Fix: Breadcrumbs redirection to patient edit consultation form

Proposed Changes

Fixes #8759
Added Consultation breadcrumb with URI for updating consultation
Updated backUrl for consultation update

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

Summary by CodeRabbit

New Features

Introduced a breadcrumb navigation entry for easier access to consultation details.
Updated the ConfirmDialog to dynamically display patient code status for consent records.

Improvements

Cleaned up URL structure by modifying the backUrl.
Enhanced upload handling logic for better interaction with existing consent records.

[ ] the issue still exist, can you recheck the PR

@shauryag2002 what is the status on the PR ?

@shauryag2002 what is the status on the PR ?

I am little busy past few days, resolving requested changes today!

LGTM


little typo

keras tutorial 5, s is missing from epochs, it should be the following:
model.fit(X_train, y_train, epochs=2, batch_size=32)

Thanks,
This is due to keras updating (keywords changed). I updated all the code.


[Snyk] Fix for 31 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

examples/v1beta1/trial-images/enas-cnn-cifar10/requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

506/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 3.7
NULL Pointer Dereference SNYK-PYTHON-NUMPY-2321964
numpy:  1.21.3 -> 1.22.2
No
Proof of Concept

399/1000  Why? Has a fix available, CVSS 3.7
Buffer Overflow SNYK-PYTHON-NUMPY-2321966
numpy:  1.21.3 -> 1.22.2
No
No Known Exploit

506/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 3.7
Denial of Service (DoS) SNYK-PYTHON-NUMPY-2321970
numpy:  1.21.3 -> 1.22.2
No
Proof of Concept

509/1000  Why? Has a fix available, CVSS 5.9
Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-SETUPTOOLS-3180412
setuptools:  39.0.1 -> 65.5.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136192
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136196
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136200
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

569/1000  Why? Has a fix available, CVSS 7.1
Out-of-bounds Write SNYK-PYTHON-TENSORFLOW-3136204
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136208
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136212
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

444/1000  Why? Has a fix available, CVSS 4.6
Buffer Overflow SNYK-PYTHON-TENSORFLOW-3136216
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
NULL Pointer Dereference SNYK-PYTHON-TENSORFLOW-3136220
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136224
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136228
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136232
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Out-of-bounds Read SNYK-PYTHON-TENSORFLOW-3136236
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Reachable Assertion SNYK-PYTHON-TENSORFLOW-3136240
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136244
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

454/1000  Why? Has a fix available, CVSS 4.8
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136248
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Improper Input Validation SNYK-PYTHON-TENSORFLOW-3136252
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Incorrect Calculation of Buffer Size SNYK-PYTHON-TENSORFLOW-3136256
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Buffer Overflow SNYK-PYTHON-TENSORFLOW-3136260
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

554/1000  Why? Has a fix available, CVSS 6.8
Out-of-bounds Read SNYK-PYTHON-TENSORFLOW-3136264
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

454/1000  Why? Has a fix available, CVSS 4.8
Incorrect Calculation of Buffer Size SNYK-PYTHON-TENSORFLOW-3136268
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Buffer Overflow SNYK-PYTHON-TENSORFLOW-3136272
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
Always-Incorrect Control Flow Implementation SNYK-PYTHON-TENSORFLOW-3136276
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

554/1000  Why? Has a fix available, CVSS 6.8
Out-of-Bounds SNYK-PYTHON-TENSORFLOW-3136280
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

579/1000  Why? Has a fix available, CVSS 7.3
Out-of-bounds Write SNYK-PYTHON-TENSORFLOW-3136509
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

589/1000  Why? Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136512
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

589/1000  Why? Has a fix available, CVSS 7.5
Heap-based Buffer Overflow SNYK-PYTHON-TENSORFLOW-3136515
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

509/1000  Why? Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-PYTHON-TENSORFLOW-3136520
tensorflow:  2.9.1 -> 2.10.1
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 NULL Pointer Dereference
🦉 Denial of Service (DoS)
🦉 Regular Expression Denial of Service (ReDoS)
🦉 More lessons are available in Snyk Learn

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 72.974%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
3116

Relevant Lines:
4270

💛  - Coveralls


Mapping for a N64 Controller

I'm making a re-map for a Bluetooth N64 controller (I'll add a pull request with the re-map after looking about this issue) and I found out something concerning about the n64 default mapping.
According to n64.c Lines 76-77, the L/R Buttons are mapped to PAD_LS/PAD_RS respectively, while the Z Trigger is mapped to both PAD_LM and PAD_RM.
But, according to gc.c Lines 65-67, the L/R buttons are mapped to PAD_LT/PAD_RT respectively, while the Z Trigger is mapped to both PAD_LS and PAD_RS.
This makes and issue when using an original N64 controller on both N64 and GameCube consoles with BlueRetro, as the Z Trigger (on the controller) will act as L+R (on the console), while the L and R buttons will do nothing.
My suggestion would be to change n64.c mapping to match to the gc.c one (only for the triggers). i.e.:
Before:
BIT(N64_Z), BIT(N64_L), 0, 0, //Line 76
BIT(N64_Z), BIT(N64_R), 0, 0, //Line 77

After:
0, BIT(N64_Z), BIT(N64_L), 0, //Line 76
0, BIT(N64_Z), BIT(N64_R), 0, //Line 77

But I wonder if that could break something with others controllers/custom-made mappings

Default mapping are fine as they are.
n64.c is done so the mapping is good by default with any controller.
using n64 controller on anything but a n64 is an edge case not worth messing everything else for.
If someone want that they have the mapping config to fix it.

also note that the adapter/wired/ source foles only affects their corresponding console. n64.c as zero impact on GameCube stuff.
Switch N64 BT controller is handled by adapter/wireless/sw.c

That's the problem, I'm not talking about the N64 BT Controller, but a self-made one acted as a Generic controller. Right now I have this on mapping_quirks.c
static void n64_bluen64(struct raw_src_mapping *map) {
    map->mask[0] = 0x771F0FFF; //Using GC mask. Maybe it should be ajusted

    map->btns_mask[PAD_RB_LEFT] = map->btns_mask[PAD_RB_RIGHT];
    map->btns_mask[PAD_RB_UP] = 0;
    map->btns_mask[PAD_RB_RIGHT] = 0;

    map->btns_mask[PAD_LT] = map->btns_mask[PAD_LS];
    map->btns_mask[PAD_RT] = map->btns_mask[PAD_RS];
    map->btns_mask[PAD_LS] = 0;
    map->btns_mask[PAD_RS] = BIT(8);

    map->btns_mask[PAD_MM] = BIT(5);
}

This will work perfectly on any GameCube console/adapter, but it will not work on a N64 console because L/R (physical buttons) does nothing, while Z Trigger act as L+R (i.e. every time I press Z, the L and R buttons will be pressed on the game).
So, what is your recommendation for that? Making the user choose/re-map everything? Because that actually is not a good solution.

Setup your quirk so that L and R goes to LS & RS, and that z goes to RM.
You really don't need to touch stuff outside the quirk.

RT LT is a special things for analog trigger with a end digital buttons. Dont use that.


Azure EventHubs classes conflicting with ServiceBus classes

When trying to use this JAR along with the Azure Service Bus SDK, I get an error saying "Exception in thread "main" java.lang.SecurityException: class "com.microsoft.azure.servicebus.ConnectionStringBuilder"'s signer information does not match signer information of other classes in the same package".
This is caused by a signature mismatch since the "ConnectionStringBuilder" class is found in both, the spark-streaming-eventhubs jar and the servicebus jar. A temporary workaround is to shade the classes, but, it is extra debugging/dev effort required on the developer's part.

The root cause is in two sides

Due to some customer scenario in HDI, we had to include EventHubs in the released jar

EventHubs copied and pasted service bus source files in their release, so those files belonging to service bus sdks are included

that means, even we refer to eventhubs client through a maven pointer, you will still see this issue because of the fact that they copy and paste the files, so, please file an issue in their repo

Yes, that is correct. The corresponding Service Bus issue is here: https://github.com/Azure/azure-service-bus-java/issues/34 & EvenHubs issue is here: https://github.com/Azure/azure-event-hubs-java/issues/95
If this issue is resolved on the EventHubs and the ServiceBus side, there needs to be an updated version of the spark-eventhubs JAR pointing to the latest version of the event-hubs sdk.

yes, there are several undergoing features here, so this problem should be fixed in the next release once we update the dependency to 0.15


Kulfy: Watch love-psychology.net

Kulfy requests the watch of the watch_keyword love-psychology\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

love-psychology\.net has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Glorfindel in Charcoal HQ


Fail of Uploading In Local Storage Mode

 Issue by zbouziane
Thursday Apr 20, 2017 at 10:37 GMT
Originally opened as https://github.com/catalogueglobal/datatools-server/issues/11

With use_s3_storage set to false:
17/04/20 12:14:10.621 [Thread-14] WARN  com.conveyal.gtfs.BaseGTFSCache - Feed tampere-20170420T121408+02-f6a79e00-b150-4725-b385-712826a46351 not found locally
java.lang.NullPointerException
	at com.conveyal.datatools.manager.models.FeedVersion.getGtfsFeed(FeedVersion.java:182)
	at com.conveyal.datatools.editor.jobs.ProcessGtfsSnapshotMerge.run(ProcessGtfsSnapshotMerge.java:130)
	at java.lang.Thread.run(Thread.java:745)

 Comment by landonreed
Tuesday Nov 13, 2018 at 17:43 GMT

This code path has been significantly altered and no longer produces this NPE.


Fix various warnings from rustc

This includes improper_ctypes in lua52-sys, but implemented as #![allow(improper_ctypes)] on the module root. #[allow(improper_ctypes)] on the lua_State struct does not work. If this is merged, #72 should be closed.
It also removes the #[allow(raw_pointer_derive)] from hlua::LuaContext, since that is no longer needed and produces a warning on nightly.
The pr also provides fixes for rust-lang/rust#19925.

Thanks!


correlation id in response header

Is there any way to return correlation Id not just in log, but also in response?  Having that value in response (header) will allow developers / testers to provide id of the request/response that should be tracked for bugs etc.

Does https://github.com/zalando/tracer solve your problem? Using it a trace can span more than just a single request-response-pair.

I've just tried, and this is what I understand:

tracer adds header in response X-Trace-ID
logbook log this header in headers section
still logged request doesn't have X-Trace-ID (from logbook) in headers section

so - that makes it a bit difficult to join:

need to know X-Trace-ID
from X-Trace-ID we need to search for logged response,
get correlation attribute from response
search for request with the same correlation

the best option would be to have correlation attribute from logbook in each response header (then we won't need tracer and additional X-Trace-ID)

@lsntomaszw Sorry for the late response:

tracer adds header in response X-Trace-ID

It also adds it to the request, if it's not already present. Therefore the following statement is incorrect:

still logged request doesn't have X-Trace-ID (from logbook) in headers section

In order to make this setup work you need to register the TracerFilter before the LogbookFilter.

the best option would be to have correlation attribute from logbook in each response header (then we won't need tracer and additional X-Trace-ID)

That would be too invasive for logbook. The purpose of the library is to log traffic, not change it. Tracer is the best match for your use case. All you need to know is the trace id. Either the client got it back as a response or they already provided it themselves with the initial request. Either way, you'll have it.
The following doc explains why we have correlation id in the first place: https://github.com/zalando/logbook/blob/master/docs/scalyr.md


Capacitor calculation not working

Netease has changed the way the capacitor gets calculated. The file effects.json does not contain the key discharge_attribute_id anymore, that used to reference what type of energy a module used. Previously, it contained either 310 for capacitor, or 650 for fuel (or not set if the module needs neither).
Instead the file now contains two booleans consume_cap and consume_fuel for every effect that decides what kind of energy an item uses. The attribute ids are hardcoded into the game (dogma_common/dogma_const_keys.py -> DogmaAttribute class) instead.
We can either change the capacitor simulator of sweet to reflect the new system (there might be items that consume both capacitor and fuel in the future), or we just insert the two values into the effects table when creating the database export. The first solution would break all existing iOS users, as they can't obtain the update through the app store at the moment.

The issue is already fixed in the database (not sure if already uploaded), but because of #17 iOS is broken anyways because some attribute ids got changed and drones/fighters do no longer work. So once I get a bit more time, I'll change the capacitor simulator to the new system. But we can keep the fix inside the db for now, the column isn't used for anything else.


graph-grid.py is the plotting script

mass_windows.txt is necessary for mapping DIDs to masses

Note that we will have to update the code to have the same style as optimize.py in terms of the similar packages. Maybe we can merge it into optimize.py if we're clever enough.


TypeScript definition doesn't play well with strict:true tsconfig settings

Hi @sublimator, would you care to provide more information?
If you have a fix in mind, PR's welcome!

Ahh I see #134 is the PR


lazy loading node with homebrew installed nvm issues.

It doesn't work properly, doesn't load node from nvm & doesn't load globally installed modules too. Some issues I found:
if [[ -s "$HOME/.nvm/nvm.sh" ]]; then # 1
  lazy_load_nvm "$HOME/.nvm"

# Load package manager installed NVM into the shell session.
elif (( $+commands[brew] )) && [[ -d "$(brew --prefix nvm 2>/dev/null)" ]]; then
  lazy_load_nvm "$(brew --prefix nvm)" #2

# Return if requirements are not found.
elif (( ! $+commands[node] )); then
  return 1
fi

NODE_GLOBALS=(`find $NVM_DIR/versions/node -maxdepth 3 \( -type l -o -type f \) -wholename '*/bin/*' | xargs -n1 basename | sort | uniq`) #3

[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" #4

nvm.sh is not inside ~/.nvm it's inside $(brew --prefix nvm)
When lazy_load_nvm gets passed $(brew --prefix nvm) as $1
$NODE_GLOBALS doesn't work because the versions are inside ~/.nvm/versions not inside $(brew --prefix nvm)/versions
Which loads the nvm but without $NODE_GLOBALS

Maybe this was an old behaviour that's why this script was created like this, I dunno.
For example this works for me
lazy_load_nvm() {
  NVM_DIR=$1
+ VERSIONS_DIR=$2
  # Skip adding binaries if there is no node version installed yet
  if [ -d $2/versions/node ]; then
+   NODE_GLOBALS=(`find $VERSIONS_DIR/versions/node -maxdepth 3 \( -type l -o -type f \) -wholename '*/bin/*' | xargs -n1 basename | sort | uniq`)
  fi
  NODE_GLOBALS+=("nvm")

  load_nvm () {
    # Unset placeholder functions
    for cmd in "${NODE_GLOBALS[@]}"; do unset -f ${cmd} &>/dev/null; done

    # Load NVM
+   [ -s "$(brew --prefix nvm)/nvm.sh" ] && source "$(brew --prefix nvm)/nvm.sh"
-  # [ -s "$NVM_DIR/nvm.sh" ] && source "$NVM_DIR/nvm.sh" # No clue why this doesn't work although they are identical

    # (Optional) Set the version of node to use from ~/.nvmrc if available
    nvm use 2> /dev/null 1>&2 || true

    # Do not reload nvm again
    export NVM_LOADED=1
  }

  for cmd in "${NODE_GLOBALS[@]}"; do
    # Skip defining the function if the binary is already in the PATH
    if ! which ${cmd} &>/dev/null; then
      eval "${cmd}() { unset -f ${cmd} &>/dev/null; [ -z \${NVM_LOADED+x} ] && load_nvm; ${cmd} \$@; }"
    fi
  done
}

...

+  lazy_load_nvm "$(brew --prefix nvm)" "$HOME/.nvm"

I'm by no means a shell expert though :)

maybe prezto should incorporate https://github.com/lukechilds/zsh-nvm, which has a lazy-load mode that seems to work at least as well as prezto's built-in version

Thanks for this issue.
Would either you like to work up a pull request?
@lukechilds, would you be interested in replacing what we've got with your module?
I'm having trouble with nvm lazy-loading on Mac OS and would be happy to review and test.

@paulmelnikow I think you can use zsh-nvm with prezto normally if it's not a module not 100% sure but it should work check the manual install https://github.com/lukechilds/zsh-nvm#manually
I don't use prezto anymore, I switched to zim with zplug & installed zsh-nvm too & that fixed all my issues.

Ah cool, maybe someone can give that a shot. Would be great to have it integrated as an external module, especially given our nvm support is buggy.
I've been following Zim, which started off as an endeavor to maintain a Prezto fork, before Matt, the author decided to rewrite it instead. It looks really promising. Technically in great shape, and also takes a smart approach of a small core. I'm curious if Matt would be interested in transitioning it to a community-managed project. There are some big advantages in what you can do when you empower and engage a community.
I've also been playing around with the plugin managers. There are a lot of them! There's zplug, Antigen, and two projects (1, 2) called zpm. Why did you decide on zplug?

zim now has it's own organization BTW.
Regarding zplug, honestly, it just looked simple, fast & familiar. Since it's similar to vim-plug—  which I use to manage my vim plugins — they even mention that it was inspired by it

Re zim, that's good to know! I hadn't seen the new organization yet. I'll have to catch up on those discussions.
Interesting, a bunch of the zsh plugin managers seem to be inspired by vim plugin managers. Seems like the vim community is ahead of the game.

I just recently tried to switch off of prezto to a zsh plugin manager, but they all had slower zsh startup times for my setup than prezto. I'll have to try zim.

@paulmelnikow Sorry, I'm pretty busy with other projects at the moment.
I'd probably recommend using a plugin manager and just using zsh-nvm. It gets updated fairly frequently.
If you really want to embed it in prezto it shouldn't be a big job. You should just be able to just copy/paste the lazy load function.

@sodiumjoe Yeah the tradeoff for the extra modularity is increased complexity in the loaders and therefore slower startup. However most modern plugin managers have some sort of caching functionality to make the difference pretty small.

@lukechilds yeah that makes sense, though I each with all available caching and prezto was still faster. For my own personal use, I don't mess with my zsh setup that often, while I open new shells all the time, so the tradeoff doesn't really make sense. This makes me hopeful about zim, though!


Xcode11 exhibits warnings if you bind in viewDidLoad

Since Xcode11 we're seeing the warnings that were addressed in #75 return.   The example project exhibits this warning also if you tap the Customization using table view delegate in the example app.  Obviously this happens a lot more in a large application with lots of screens.
It's not clear to me if this is a UIKit bug or something that needs addressing in RxDatasources.  It feels like the former as I can't imagine how the toolkit can complain that a view isn't in the hierarchy in viewDidLoad but I wanted to log something to track the issue as this beta season develops.

Can confirm. This is a warning that was introduced in Xcode 11 when views are laid out before the viewController is added to the view hierarchy.
The warning in the console is this:
Warning once only: UITableView was told to layout its visible cells and other contents without being in the view hierarchy (the table view or one of its superviews has not been added to a window). This may cause bugs by forcing views inside the table view to load and perform layout without accurate information (e.g. table view bounds, trait collection, layout margins, safe area insets, etc), and will also cause unnecessary performance overhead due to extra layout passes. Make a symbolic breakpoint at UITableViewAlertForLayoutOutsideViewHierarchy to catch this in the debugger and see what caused this to occur, so you can avoid this action altogether if possible, or defer it until the table view has been added to a window. Table view: <UITableView: 0x7fcef4069e00; frame = (0 0; 414 896); clipsToBounds = YES; autoresize = RM+BM; gestureRecognizers = <NSArray: 0x600002515c20>; layer = <CALayer: 0x600002a342c0>; contentOffset: {0, 0}; contentSize: {414, 0}; adjustedContentInset: {0, 0, 0, 0}; dataSource: <RxCocoa.RxTableViewDataSourceProxy: 0x600000e475a0>>

Adding the symbolic breakpoint breaks on DelegateProxyType line 329.
The workaround is to bind your dataSource after the view was added to the view hierarchy. In a UIViewController, that moment is when viewDidAppear is called. However, this is quite inconvenient since this method may be called more than once in the lifetime of a view (when navigating to and from the view for instance). So that means you need to manage the Disposable manually or place guards around the subscription. Either way, not ideal.
Since this lives in RxCocoa, should we also open an issue there?

However, this is quite inconvenient since this method may be called more than once in the lifetime of a view ( this isn't even the worst aspect.  You can visually see the UI change if your labels have placeholder values from a storyboard, etc.  It's just way too late in the lifecycle to do this.

Bump. I got the same issue. Any update ? 😄

In case you don't want to manage the Disposable manually, as you would lose, for example, the position in your tableView if you are binding to it. You could use a way to execute this only once. For now, and with some time pressure, I have come to use:
private lazy var rxReady: Bool = setupRx()

override func viewDidAppear(_ animated: Bool) {
        super.viewDidAppear(animated)

        if rxReady {
            //Needed for circunvent issue:
            //https://github.com/ReactiveX/RxSwift/issues/2081
            //https://github.com/RxSwiftCommunity/RxDataSources/issues/331

            //Once it's solved, it's save to move the call to setupRX back into viewDidLoad()
        }
    }

Not proud of it ...but it's a way to circumvent the issue with minimal changes for now until it's fixed in the library.
The other is to ignore the warning... but ...that doesn't seem right.

Hey guys how is this going?

This pull request fixes this issue: https://github.com/ReactiveX/RxSwift/pull/2076.
It is already in develop branch, so should appear on next RxSwift release.

Any idea when the next RxSwift release is coming out? :)

in viewDidLoad or loadView func where you bind dataSource,delay a little while then do the bind task. the waring goes away,but I dont think this a good way.
Observable.timer(RxTimeInterval.milliseconds(10), scheduler: MainScheduler.instance).subscribe(onNext:{ e in
self.itemsSub.bind(to: self.addressTabelView.rx.items(dataSource: dataSource)).disposed(by: self.disposeBag)
self.itemsSub.onNext(self.dataList)
}).disposed(by: disposeBag)
`

same issue for me. I am using RxFeedback:
i run Driver.system() in viewDidLoad(), then bind tableView using:
func bind() -> Feedback {
        return bind(self) { me, state in
            let subscriptions: [Disposable] = [
                state.data().drive(me.tableView.rx.items(dataSource: me.dataSource))
            ]
            let events: [Observable<Event>] = []
            return Bindings(subscriptions: subscriptions, events: events)
        }
    }

i tried here to replace my data state.data() driver with Driver<[DataSection]>.empty() not to send any data on tableView.rx.items, but still get a warning:

[TableView] Warning once only: UITableView was told to layout its visible cells and other contents without being in the view hierarchy ...... to a window. Table view: <UITableView: 0x7b7800006c00; frame = (0 0; 414 896); clipsToBounds = YES; autoresize = W+H; gestureRecognizers = <NSArray: 0x7b0c000b6f70>; layer = <CALayer: 0x7b0800078f60>; contentOffset: {0, 0}; contentSize: {414, 0}; adjustedContentInset: {0, 0, 0, 0}; dataSource: <RxCocoa.RxTableViewDataSourceProxy: 0x7b180000bee0>>

Has anyone found the decent fix since? None of the fixes above satisfied my needs.

I have been encountering this issue and believe I may have a fix.  By setting a symbolic breakpoint for UITableViewAlertForLayoutOutsideViewHierarchy I determined the following call to layoutIfNeeded in DelegateProxyType was being called before the tableView is added to window:
extension ObservableType {             func subscribeProxyDataSource<DelegateProxy: DelegateProxyType>(ofObject object: DelegateProxy.ParentObject, dataSource: DelegateProxy.Delegate, retainDataSource: Bool, binding: @escaping (DelegateProxy, Event<Element>) -> Void)                 -> Disposable                 where DelegateProxy.ParentObject: UIView                 , DelegateProxy.Delegate: AnyObject {                 let proxy = DelegateProxy.proxy(for: object)                 let unregisterDelegate = DelegateProxy.installForwardDelegate(dataSource, retainDelegate: retainDataSource, onProxyForObject: object)                 // this is needed to flush any delayed old state (https://github.com/RxSwiftCommunity/RxDataSources/pull/75)                 /// Potential fix for https://github.com/RxSwiftCommunity/RxDataSources/issues/331                 if let _ = object as? UITableView, object.window != nil {                     object.layoutIfNeeded()                 }


Prevent deadlock when unregistering notification handler

A call to unregister notifications while a current notification was being sent could cause a deadlock. This stops holding the callbackMapLock when calling HcsUnregister*Callback to prevent the deadlock.
@swernli @jhowardmsft PTAL
Signed-off-by: Darren Stahl<EMAIL_ADDRESS>

Yup, LGTM

Don't merge yet... Darren and I chatted a bit and there are a couple updates he's making.

OK

Discussed changes have been made, but I've got a few more things to figure out before we merge.

LGTM


Enhancement: Add Media Devices

Would be nice to have the ability to have Televisions or media devices in the bridge

You are right, I forgot about those. BUT no vendor (apple, amazon, google, Samsung) support media players via matter. So I’ll map them to simple on-off-switches.

So I’ll map them to simple on-off-switches.

I personally would rather have a configurable option for this. I would prefer to have my devices show up as what they actually are (TVs, speakers, whatever), and wait for vendors to add functionality. I know obviously most people wouldn't want this though, which is why I'm voting for a config toggle.

yes i understand that, but starting with the on-off switch will be easier for me. I'd need to dive deep into how media players work with matter - i already know that this is a lot harder than other device types.
one addition which speaks (at least) for the configuration option: if you are adding an unsupported device to the bridge, at least alexa won't connect the whole bridge. so one will not able to control any of the other devices.


Externalize and consolidate character alias logic

Re-created pull request with only my commits.
Will hope to make more changes like this in the future.

I was actually planning on doing this exact thing myself, I'll look at it when I have some time but it seems good at a glance.

It needs a bit more work, it no longer responds with an error embed if it doesn't find anything with the character name. Also names with space in them still don't work, but it looks like they would since there's some in the character nickname file.

Yeah, it just occurred to me that spaces won't even work, no matter how you spin it. Everything should be good now, I tested it to make sure it works.

I used your idea but I changed the implementation to one I liked better and which also works with the existing error messages.


Support sending raw messages to statsd daemon

Sometimes the API abstraction doesn't help much. With this functionality, the user can directly send a message across the wire. This can be useful when a statsd server supports different metric types (ie. Datadog).

This project hasn't been updated in years.  Biased but I would suggest checking out https://github.com/brightcove/hot-shots


Overlay problem

When you close modal and open again overlay become darker.. i got 0.3 but when you close and open  modal several times it become to black color. I think it is 0.3 + 0.3....
$('#click').click( function(){ $('modal').easyModal({top: 100, overlay : 0.2, autoOpen: true, zIndex: function () {return 100;}}); return false; });

I had the same issue. Each time you call easyModal() it creates a new element with class 'lean-overlay' which creates layers with greater opacity each time.
To resolve, make sure you only call easyModal() once and use .trigger('openModal') each time you need to open/reopen the modal.


[Icon] - Icon does not display.

import React, { Component } from 'react';
import { Icon } from 'semantic-ui-react';
...

...
but it doesn't display at all. How can we solve this problem?

This is a SUI React issue, please create the issue over on the React repo https://github.com/Semantic-Org/Semantic-UI-React


chore(webui): upgrade angular cli version

What does this PR do?

update webui thanks to angular-cli update feature
mainly everything was done by the ng update @angular/cli cmd

Motivation

make app up to date with current Angular version

Additional Notes

angular cli update doc

@ldez I need to update node.js to ^10.15. in this PR to make it work


Rewrite hostmap to allow for subdomains

Right now subdomains fail because the host map is too strict.

Done. it will look for a.b.c.d first then b.c.d, then c.d and so on.


[18.09 backport] Remove a network during task SHUTDOWN instead of REMOVE

built on top of https://github.com/docker/engine/pull/204; I'll rebase if that one is merged
backport of https://github.com/moby/moby/pull/39174 and https://github.com/moby/moby/pull/39242 for 18.09
Make sure the LB sandbox is removed when a service is updated
with a --network-rm option
- What I did
Ensured that the LoadBalancer Sandbox is deleted when a service is updated with --network-rm option
- How I did it
Removed the removeNetworks logic from task REMOVE to task SHUTDOWN
- How to verify it
docker network create -d overlay netTest1
docker network create -d overlay netTest2
docker service create  --name test --network netTest1 --network netTest2 --replicas 2 nginx
docker service update --network-rm netTest2 test
docker network inspect netTest2  # Make sure there are no endpoints including LB

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

ping @mavenugo @arkodg PTAL

Hm... looks like I'd have to cherry-pick some changes in SwarmKit;
01:12:50 Congratulations! Seccomp profile generation is done correctly.
01:14:13 integration/service/update_test.go:221:17:warning: RunningTasksCount not declared by package swarm (gosimple)
01:14:14 Build step 'Execute shell' marked build as failure

marked this "WIP", pending https://github.com/moby/moby/pull/39242 / https://github.com/moby/moby/issues/39225

Added https://github.com/moby/moby/pull/39242 to the cherry-picks, removing "WIP" again

updated, because https://github.com/moby/moby/pull/39242 was slightly modified

rebased because Jenkins results were purged, and https://github.com/docker/engine/pull/204 was rebased

rebased again because #204 was rebased


Add links to upstream documentation

rpm-ostree = https://rootpages.lukeshort.cloud/linux_distributions/fedora.html#build
Anaconda and Kickstart = https://rootpages.lukeshort.cloud/virtualization/virtual_machines.html#anaconda

These are not official documents and soon they may not be very relevant anyways. Closing.


Test failing Locally

Forked the repo and ran npm install and npm run bootstrap. When i ran npm run test i got the following errors ```

<EMAIL_ADDRESS>test /Users/johnmurray/school/388/wasmos
jest

FAIL  packages/as-node/spec/as-node.spec.ts
● Test suite failed to run
TypeScript diagnostics (customize using `[jest-config].globals.ts-jest.diagnostics` option):
packages/as-node/spec/as-node.spec.ts:3:10 - error TS2305: Module '"../../../../../../../../Users/johnmurray/school/388/wasmos/packages/as-node/node_modules/@wasmos/kernel/node_modules/@wasmos/fs/assembly"' has no exported member 'fs'.

3 import { fs } from "@wasmos/fs";
           ~~

FAIL  packages/assemblyscript/spec/compiler.spec.ts (5.104s)
● Console
console.log packages/assemblyscript/src/compiler.ts:43
  [ '/Users/johnmurray/school/388/wasmos/packages/assemblyscript/node_modules/@wasmos/fs/assembly',
    '/Users/johnmurray/school/388/wasmos/packages/assemblyscript/node_modules/assemblyscript/lib/host/assembly' ]
console.log packages/assemblyscript/src/compiler.ts:43
  [ '/Users/johnmurray/school/388/wasmos/packages/assemblyscript/node_modules/@wasmos/fs/assembly',
    '/Users/johnmurray/school/388/wasmos/packages/assemblyscript/node_modules/assemblyscript/lib/host/assembly' ]

● echo › should compile
ENOENT: no such file or directory, lstat '/Users/johnmurray/school/388/wasmos/packages/assemblyscript/dist'

Ah this is the same issue that I had in class when I showed the file.  This is because in@wasmos/fs's package.json the types point to assembly/ so instead you need to add /src after it so that it reads the typescript instead of the assembyscript.

Hmm it seems that indeed as-node.spec.ts does use the wrong import, but as I hoped it wouldn't be, it seems that the symlinks are the issue...


[#61] [iOS] [UI] As a user, I can see text area answer

Closes #61

What happened 👀
Added text editor for answer view
Insight 📝

I also added placeholder for the text editor as it's not supported.

Proof Of Work 📹
https://user-images.githubusercontent.com/19943832/211706082-57028eba-1891-488c-a1c7-f6cf6298672a.mov

All good, except for one weird thing in the intro. Please investigate it 🙏 .

@nkhanh44 For the default case, we are showing the text of the display type. Please ignore this one, I will update on the integrate task!


New feature vgw

New feature vgw for vpn gateway (also known as virtual gateway).

Please include availability_zone argument. In my environment, terraform plan shows diff.
~ aws_vpn_gateway.vpn1
    availability_zone: "ap-northeast-1a" => ""

~ aws_vpn_gateway.vpn2
    availability_zone: "ap-northeast-1a" => ""

Thanks, I added availability_zone attribute.

LGTM 👍 Thanks :exc


Integration test for rates

I think we need some kind of integration test which checks the rates of the essential modules.
This would especially be handy for the bringup of new hardware such as the Snapdragon or RPi.
Potentially part of the solution is already available using the perf_counters.

uorb top?

I guess so :smiley:.


beautiful soup deprecated

Once you install beautiful soup4 the script will run, but the HA observer returns no data.

Are you seeing no data when you run it at the command line?  How is it configured in CheckMK?

OMD[monitor]:~/local/bin$ ./check_haobserver.py homeassistant <<<check_mk>>> Version: 1.0 AgentOS: HomeAssistant <<<local>>> 0 Supervisor connected=1 Supervisor reports Connected 0 Supported connected=1 Supported reports Supported 0 Healthy connected=1 Healthy reports Healthy

I think I have mitigated a potential failure scenario with the checkin for issue #3.  Let me know if you are still getting no output and we can debug further.


fix: Try removing compile-files for SO

Something is causing a Stack Overflow on Windows; maaaybe it could be this line? https://github.com/PRQL/prql/pull/2700/files#diff-2e1623f80d59e3d48345bf175c77cecd15f88c8c7edb81a03ead35194b839720R16

This does seem to be the source. Given we don't use the example much, I've disabled compile-files while we resolve the count * issue.
I'm happy to have a look at the query more if that's helpful (but I know @aljazerzen is already planning to make changes with the count *)

ref https://github.com/PRQL/prql/pull/2700#issuecomment-1575206792


call lack of progress if object leaves the field

fixes #77

Signed-off-by: Adrian Matejov<EMAIL_ADDRESS>

@mrshu Yeah. Created an issue for that.


Css Refactor

Added Autoprefixer Gem and cleaned out all vendor prefixes from css.

Awesome work, I made a few changes in master this morning so I resolved conflicts and merged this branch into master :+1:


feat(tutorial): complete step 5

URL: https://carbon-tutorial-abitvenkat.mybluemix.net/#/

Per instructions in the tutorial step 5, please include your mybluemix.net URL for your deployed app in your pull request description.

I have edited the description to include the URL. Sorry forgot about adding it.


Simple terminology question

Hi, thank you for sharing such a great work.
In the codebase, especially when preparing the inputs for training, you use terms like "t1_points" and "t2_points". Does "t" here indicate trajectory? In that case, what is the role of "t1" and "t2"?
Plus, I would really appreciate if you could brief/simple logic of the training pipeline. For example, what exactly is the input/outputs.
Thanks a lot.

hi @HyeonHo99,
t1 and t2 basically denote the sets of source (t1) and target (t2) points that correspond to each other.
Regarding the training -- the pipeline gets a single video as input (and optionally foreground masks, if they're not provided, we compute them using DINO saliency masks), and outputs a DINO-Tracker instance for that video, which can be later queried for tracking points. The technical details of using the pipeline are described in the readme.

Thank you for the answer @tnarek , I would like to ask you one more question.
So I can understand that
given sampler = DinoTrackerSampler(...)
sampler()["t1_points"] and sampler()["t2_points_normalized"] are the corresponding points. But I somewhat expected they are coordinates representing points but the values are floating numbers between (-1,1).

Given a sampler instance (sampler = DinoTrackerSampler(...)), how can I extract the coordinate pairs that correspond to each other?
Also, what does "frames_set_t" from sampler()["t1_points"] mean?

Thanks a lot.

During training, the coordinates are being normalized to [-1, 1], notice the range_normalizer that is used in DinoTrackerSampler. You can use the range_normalizer.unnoramlize function to convert them back to the original resolution
For efficiency, in each training iteration, we consider only a limited number of frames form the video (e.g. 8), out of which we extract all correspondences. frames_set_t indicates the time indices of those frames.


The link for the website, beside the description is broken

I guess its because is https instead of http, maybe you are planning to install the ssl but it made me difficult to find the correct url to the website xD btw: it works! http://🐀.ws/🌺

I didn't find the https link, can you attach a screen or something?

Ok link of the repo, facepalm. Fix it, thanks!

You are welcome! :smile:


Question regarding pseudo mask generation

Hi, In the code where I can find the pseudo mask generation process? I want to save the pseudo masks. How can I do this?

You can obtain them from here:
https://github.com/facebookresearch/deepmask

@IssamLaradji  thanks for your quick response. Then how the PRM is helping in generating a pseudo mask? It would be very helpful for me if you could clarify this to me.

PRM is there to choose between the proposals.

@IssamLaradji  Thanks for the clarification.

@IssamLaradji One more question, are you saving the output from the PRM somewhere?


[feature] on loaded callback

Hello,
could we please get a callback that fires when the image has loaded, no matter where it is loaded from (memory cache, disk cache, network)
also a callback for when the image has failed to load :)

It's done in v0.3.4 :)

Amazing turnaround my man, thank you.


headers with the same name get clobbered

some headers, such as Authentication-Results, can occur multiple times in a message. the current code clobbers previous values.
it seems like the following places need to be updated to support headers having lists of values found in a message. all headers could have values that are lists, or only the ones that have more than one value.

https://github.com/SpamScope/mail-parser/blob/develop/mailparser/mailparser.py#L534
https://github.com/SpamScope/mail-parser/blob/develop/mailparser/utils.py#L448
https://github.com/SpamScope/mail-parser/blob/develop/mailparser/mailparser.py#L298

the current code uses email.message.get() but needs to use email.message.get_all(). https://docs.python.org/3/library/email.message.html#email.message.EmailMessage.get_all

:+1: Came here to add this. This goes for Received headers as well.

In the mean time, you can solve this for Received headers by:
headers = message.headers
headers['Received'] = [h.replace('\n', '') for h in message.received_raw]

For the other headers, I don't think you'll be able to currently, due to the lack of _raw for others.

@mortea15 are you seeing this happen for received headers? The receiveds property looks like it uses message.get_all().

I need to have a raw mail to test it.
If you still have this problem, open another issue.

you just need to duplicate a header with a different value...
Return-Path<EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: (qmail 11769 invoked from network); 22 Aug 2016 14:23:01 -0000
Received: from smtprelay0207.b.hostedemail.com (HELO smtprelay.b.hostedemail.com) (<IP_ADDRESS>)
  by smtp.server.net with SMTP; 22 Aug 2016 14:23:01 -0000
Received: from filter.hostedemail.com (<IP_ADDRESS>.rfc1918.com [<IP_ADDRESS>])
	by smtprelay06.b.hostedemail.com (Postfix) with ESMTP id 2CC378D014
	for<EMAIL_ADDRESS>Mon, 22 Aug 2016 14:22:58 +0000 (UTC)
Received: from DM6PR06MB4475.namprd06.prod.outlook.com (2603:10b6:207:3d::31)
 by BL0PR06MB4465.namprd06.prod.outlook.com with HTTPS id 12345 via
 BL0PR02CA0054.NAMPRD02.PROD.OUTLOOK.COM; Mon, 1 Oct 2018 09:49:22 +0000
Received: from DM3NAM03FT035.eop-NAM03.prod.protection.outlook.com
 (2a01:111:f400:7e49::205) by CY4PR0601CA0051.outlook.office365.com
 (2603:10b6:910:89::28) with Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1185.23 via Frontend
 Transport; Mon, 1 Oct 2018 09:49:21 +0000
X-Session-Marker: 6A64617A657940616C6578616E646572736D6974682E636F6D
X-Spam-Summary: 69,4.5,0,,d41d8cd98f00b204,suvorov.s@nalg.ru,:,RULES_HIT:46:150:152:379:553:871:967:989:1000:1254:1260:1263:1313:1381:1516:1517:1520:1575:1594:1605:1676:1699:1730:1747:1764:1777:1792:1823:2044:2197:2199:2393:2525:2560:2563:2682:2685:2827:2859:2911:2933:2937:2939:2942:2945:2947:2951:2954:3022:3867:3872:3890:3934:3936:3938:3941:3944:3947:3950:3953:3956:3959:4425:5007:6001:6261:6506:6678:6747:6748:7281:7398:7688:8599:8824:8957:9009:9025:9388:10004:10848:11604:11638:11639:11783:11914:12043:12185:12445:12517:12519:12740:13026:14149:14381:14658:14659:14687:21080:21221:30054:30055:30065:30066,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:5,LUA_SUMMARY:none
X-HE-Tag: print38_7083d7fd63e24
X-Filterd-Recvd-Size: 64695
X-Test-Key: value1
X-Test-Key: value2
Received: from computer_3436 (unknown [<IP_ADDRESS>])
	(Authenticated sender<EMAIL_ADDRESS>	by omf06.b.hostedemail.com (Postfix) with ESMTPA
	for<EMAIL_ADDRESS>Mon, 22 Aug 2016 14:22:52 +0000 (UTC)
From: =?UTF-8?B?0YHQu9GD0LbQsdCwINCk0J3QoSDQlNCw0L3QuNC40Lsg0KHRg9Cy0L7RgNC+0LI=?=<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: =?UTF-8?B?0L/QuNGB0YzQvNC+INGD0LLQtdC00L7QvC3QtQ==?=

This was an issue of headers function but mail is correct.
def get_header(message, name):
    """
    Gets an email.message.Message and a header name and returns
    the mail header decoded with the correct charset.

    Args:
        message (email.message.Message): email message object
        name (string): header to get

    Returns:
        str if there is an header
        list if there are more than one
    """

    headers = message.get_all(name)
    log.debug("Getting header {!r}: {!r}".format(name, headers))
    if headers:
        headers = [decode_header_part(i) for i in headers]
        if len(headers) == 1:
            # in this case return a string
            return headers[0].strip()
        # in this case return a list
        return headers
    return six.text_type()

Get get_header can get string and list, but I didn't use it in headers. It was a bug. Thanks a lot for your issue.


Inference using ctc decoder

🚀 Feature Request
Inference with wav2vec using ctc decoder
Motivation
I'm trying to build a pipeline using wav2vec embeedings on an arabic data set. I seem to have issues with wa2letter installation. So can anyone provide us with a simple inference using CTC decoder for example  or beam search

look in ctc.py for an example of how to do argmax decoding which doesnt require a decoder or language model

This is a snippet of code of what I'm trying to do (I'm a beginner so no judgment):
def greedy_decoder(data): 	# index for largest probability each row 	return [np.argmax(s) for s in data] from fairseq.models.wav2vec import Wav2VecModel cp = torch.load('/home/haythem/Downloads/wav2vec_small_960h.pt') model = Wav2VecCtc.build_model(cp['args'],target_dict) model.load_state_dict(cp['model'],strict=False) with torch.no_grad():     net_output = model(**sample["net_input"])     lprobs = model.get_normalized_probs(             net_output, log_probs=True         ).contiguous()   lprobs=utils.softmax(lprobs.float(),dim=-1) data = np.array(lprobs) result = greedy_decoder(data) import pandas as pd txt=pd.read_csv('/home/haythem/Desktop/libri/dict.ltr.txt',sep=' ',names=['letter','numbers']) a2=list(txt.iloc[:,0]) results1=[a2[result[i]] for i in range(len(result))] results1

heres the relevant code that you could plug in after your normalized_probs() call:
lprobs_t = lprobs.transpose(0, 1).float().contiguous().cpu()
toks = lp.argmax(dim=-1).unique_consecutive()
pred_units_arr = toks[toks != self.blank_idx].tolist()
pred_units = self.task.target_dictionary.string(pred_units_arr)

I'm facing the same problem widh wav2letter. So, I'm trying to decode a sample using a simple decoder ctc. Here my code:
import torch
import fairseq
import librosa
model, cfg, task = fairseq.checkpoint_utils.load_model_ensemble_and_task(['/home/ubuntu/Self-Supervised/code/librispeech/finetuning/checkpoint_best.pt'], arg_overrides={"data": "dict.ltr"})

model = model[0]
model.eval()
print('model loaded')

with torch.no_grad():
	net_output = model(**sample["net_input"])
	lprobs = model.get_normalized_probs(
	net_output, log_probs=True
	).contiguous()

	lprobs=utils.log_softmax(lprobs.float(),dim=-1)
	data = np.array(lprobs)
	result = greedy_decoder(data)
	print(result)

error :
NameError: name 'sample' is not defined
I did not inderstand where I should put the wav data and how.
I will appreciate any help @alexeib and @haythemdhieb  .
Thanks.

theres a problem in your python code. you need to iterate over the dataset to obtain "sample" which you use for your model

wav format is for sound. while you can almost certainly use the same kind of model/training for handwritten ocr (and i think i saw some papers that do something similar), you'd have to implement a different data loader + encoder

I am using Fairseq for Handwriting text recognition, and I am using an encoder + CTC architecture for my model. More specifically, I am using an architecture like Fairseq speech recognition "vggtransformerencoderonly" + CTC. Because I am using Google Colab to run my model, I was wondering if there is a way to infer translations for such a model without installing and using flashlight? @alexeib

@haythemdhieb
TorchAudio recently ported flashlight CTC decoder. It currently provides lexicon constrained beam search decoding with/without KenLM.
You can try it with nightly build.
The API documentation is found
https://pytorch.org/audio/main/prototype.ctc_decoder.html#lexicon-decoder
https://pytorch.org/audio/main/tutorials/asr_inference_with_ctc_decoder_tutorial.html
cc @carolineechen

torchaudio is a good choice. additionally the "viterbi" (aka greedy) decoder doesnt rely on flashlight and can be used if you dont need to fuse with an LM


fix(styles): Used popper library to handle tooltip

Fixes #532
Notes
Changes are only done in twig

Implemented Popper.js to handle the tooltip.
Modified styling and js as required to support popper.js
Priority of tooltip will be Top,Bottom,Right,Left as @inesdgomes suggested.
Considered mobile devices when screen size is below medium (Same screen size measuring in other components), the max width remains at 180px.

Cc :- @justintemps
Fixed bugs:-

[x] the position of the icon is wrong
[x] the arrow is pointing in the wrong place
[x] the box needs to be variable according to the amount of content there is
[x] weird behaviour on hover
[x] font should be consistent in tooltip

Screenshots
Long tooltip (More than 50 chars)

Normal tooltip (Less than 50 chars)

Hey great work on this @Shashika6. Before merging I need to make sure that it can be deployed on the website without them having to change bundling configuration. It's not clear to me whether or not we should consider adding external libraries from node_modules a breaking change in the Twig package, because it makes some assumptions about the way they're bringing in the JavaScript.

Sure @justintemps let me know if any changes are required to be done from my side :)

Hey @Shashika6 can you make sure this also addresses all of the design issues raised in https://github.com/international-labour-organization/designsystem/issues/705#issuecomment-1843569041?

@justintemps Verified that the design issues raised in https://github.com/international-labour-organization/designsystem/issues/705#issuecomment-1843569041? are addressed.


Throw Error on No Good UninstallStrings for Passthrough and InstallShield

Remove-BloatwareWin32Passthrough and Remove-BloatwareWin32Installshield should throw errors if there are no good uninstallstrings to use.

Slated for release 1.1.3

Release 1.1.3 #10


Upload field component refresh page without saving results in trash files in storage

Describe the bug
Upload field component instantly creates Attachments model and uploads file, without waiting for save function to run. As result, if you upload and refresh the page, we have trash Attachments without having any Model relations.
To Reproduce
Steps to reproduce the behaviour:

Upload a file, but don't run the save function.
Refresh the page.
upload component is empty but Attachment Model and uploaded files remain.

Expected behaviour
Expected to save and preview from browser storage instead of uploading and fetching from URL. The actual upload should be delayed till the save function is called.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [Windows 11]
Browser [*]
Version [*]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Server (please complete the following information):

Platfrom Version: [10.^]
Laravel Version: [8.0^]
PHP Version:  [8.0]
Database:  [MySql]
Database Version:  [8.2^]

Additional context
Storage is costly on AWS. So this bug can drive up server costs.

This is a compromise between manual file saving and convenience.

Storage is costly on AWS. So this bug can drive up server costs.

Each time you download the file to the server, we calculate its hash and do not allow downloading two identical files in one storage. This check is carried out using a database and calls AWS or another repository without apparent need.
This leads to a decrease in storage costs.
But if you mean referring to reading, I can't remember any place where we read the contents of the file, but you can enable recommendations in the Laravel documentation:

If my words did not convince you or you want to control more files, you can use the classic file field:
Input::make('Upload')->type('file')

This will send the file along with the sending form, and you can get control:
use Illuminate\Http\Request;

public function save(Request $request)
{
   $request->file('photo');
}

I'm open to any suggestions.

The upload component is very much functional with preview, size, format check. A custom implementation will need all these to be built as well. The main issue here is the Attachmentable model is made without an Attachmentable model relation. I don't think attachable are meant to exist on their own.
Workaround:
https://github.com/orchidsoftware/platform/blob/master/resources/views/fields/upload.blade.php#L19
Add a lazyUpload property on Uploads that accepts a function. It will create/upload the Attachment and Attachmentable Model when the function for saving data is run.


docs: Aries framework pkg godocs

Part of #739
Signed-off-by: Rolson Quadras<EMAIL_ADDRESS>

@sandrask I agree with both the points. Will update the example and push the changes.


'React/RCTViewManager.h' file not found

Hello Guys,
I encountered this error using and ejected project from expo:
"react": "16.8.3",
"react-native": "https://github.com/expo/react-native/archive/sdk-32.0.0.tar.gz",
"expo": "^32.0.0",
"expokit": "^32.1.1",

And I also setup the project using this:

Thanks for any help

the same

same with RN 0.60.5...

Hy guys, any solutions?

same with RN 0.62.3

Hey any solution?. I get this problem with RN 60.


Don't remove static chart PNGs if the kernel is not connected and refresh > 0

Currently the chart code will remove a static chart PNG before it draws a chart dynamically, and then re-add a PNG for the updated version. These static PNGs are so that a notebook viewed in GitHub displays something for a %chart cell,
We are adding support for charts that are refreshed periodically. In such a case, the static PNG may have more up-to-date content than the initial call to render() has, and so opening such a notebook in Datalab or Jupyter could replace a more recent static PNG with a less recent dynamic chart.
This issue is just to note this possibility, and to consider checking in the case where we have a periodically refreshing chart whether the session is connected to a kernel, and if not, to not remove the PNG (and in fact to not attempt a refresh).

Moved to https://github.com/googledatalab/datalab/issues/14


Fold the storage service behaviour into delta and introduce roles to select the behaviour to be run

Nexus Storage will be decommissioned in 1.11


No default route

in the router config, if I don't define a default, all the moduleIDs become undefined.
so in the browser I click any Route menu item, I go to /undefined.
I know this is not a bug, but rather a use case.

I suggest, normally even if the user don't define a router entry point, the router should keep compiling the router moduleIDs (#hrefs).

i read the api doc, i have to use 'href' attribute in the routes to overcome this use case.

@EisenbergEffect We can close this issue it looks like.


user documentation does not explain "just" target modifier

I still don't know what "just" does.  The doc mentions it but does not explain it.  At first I thought that it shrank a target to exclude the selection-type separator (space for token, \n for line, etc), but I am beginning to think that it does more than that.
The doc mentions "take every instance just air" does something very different than what I described above, so apparently I don't understand the "just" modifier in full generality.

I think this issue and #1522 are perhaps more important than they first appear.
Awhile ago, I asked how to do something in cursorless, and the answer involved "before, "start of", and "just".  That was the moment where I most felt that I did not understand the underlying structure of cursorless.  And now I notice that none of those modifiers are really documented.  They are important modifiers that are useful to doing very common editing operations.

Yeah we should really have more on "just". I believe there's a Cursorless kata about it that's prob the best place to learn about it for now

What is the best way to show a little bit more on the cheatsheet?  Edit https://github.com/cursorless-dev/cursorless/blob/a8ea10d83c55811f363c32e75d7941f01dece25a/cursorless-talon/src/cheatsheet/sections/modifiers.py#L48 ?
I think there is a big difference in impact between an explanation in the official doc and cheatsheet vs buried in a 3rd party readme.  I still haven't looked at more than a handful of katas.  I am willing to work on just doc but will probably need correction.

See https://github.com/cursorless-dev/cursorless/pull/1538#discussion_r1233304500


useMergeState

I wanted to make a PR but I don't know how Gatsby works so I figured I can leave the code here and let you decide if you'd like to publish it or not.
The idea is to mimc the functionality of the old setState as closely as possible:

Provide an object to be merged into state OR
Provide a function which gets passed the old state and returns a new object to be merged into state.

To me this is especially helpful in 3 scenarios:

You have a lot of state and don't want to have a ton of useState declarations at the top of your component.
You want to stick with what you're used to from setState.
You want to avoid the overhead of using useReducer.

useMergeState
import { useState } from 'react'

// Object-checking helper fxn since this is only meant to work with objects.
const objCheck = thing => {
  if (({}).toString.call(thing) !== '[object Object]') {
    throw '`useMergeState` only accepts objects.'
  }
}

const useMergeState = (initialState = {}) => {
  objCheck(initialState)
  const [state, setState] = useState(initialState)

  /*
    Just like the old `setState` with React classes, you can pass a fxn to `mergeState`.
    Why would you want to do this? Because certain scenarios, such as using the `useEffect` hook
    create closures around values that may become stale or outdated by the time they get used.
    Providing a fxn ensures that fxn has the latest available state object to work with.
  */
  const mergeState = objOrFxn => {
    // Passing a fxn to `mergeState`.
    if (objOrFxn instanceof Function) {
      setState(prevState => {
        const newState = objOrFxn(prevState)
        objCheck(newState)
        return { ...prevState, ...newState }
      })

    // Passing an object to `mergeState.
    } else {
      objCheck(objOrFxn)
      setState(prevState => ({ ...prevState, ...objOrFxn }))
    }
  }

  return [state, mergeState]
}

Example usage
import React from 'react'

const MyComponent = () => {
  const [state, mergeState] = useMergeState({
    date: new Date(),
    num: 1
  })

  // Using an object.
  const button1Click = () => mergeState({ date: new Date() })

  // Using a function.
  const button2Click = () => mergeState(oldState => ({ num: oldState.num + 1 }))

  return (
    <>
      <button onClick={button1Click}>{state.date.toString()}</button>
      <button onClick={button2Click}>{state.num}</button>
    </>
  )
}

const reducer = (prev, updater) => {
  if (typeof updater === 'function') {
    return { ...prev, updater(prev) }
  } else {
    return { ...prev, updater }
  }
}
const [state, setState] = React.useReducer(reducer, initialState)

Well ain't that sexy

This refactor for the win:
import { useReducer } from 'react'

const reducer = (prevState, updater) => (
  typeof updater === 'function'
    ? { ...prevState, ...updater(prevState) }
    : { ...prevState, ...updater }
)

const useMergeState = (initialState = {}) => useReducer(reducer, initialState)

Sorry for getting to this so late. This is super useful and especially like how it's implemented with useReducer. Might be fun to talk a bit about how React actually builds useState with useReducer. I'm going to do a post on this soon :)


Download videos (Offline sync/cache)

Is there a way to download and save the videos at a specific resolution so that I can use the videos offline?

Not right now, but will keep it in mind! Curious, can you tell me more about why you need this feature?

I am currently developing a VR app for an event so they would like to "cache" the videos just in case the wifi connection drops

@caseypugh any news on this feature? Im now working on a video app which would really benefit from a download function :-)

Sorry didn't mean to close it

checking back...

👍

Hi @yosun @MohHeader thanks for the feedback! There is still no timeline for this feature, but given the interest I will try to prioritize it in the future 🚀 thank you for keeping this thread going

So this download feature works now, is anybody still working on this project? There has been no activity for a long time

Any news on the download feature?

For downloading I would throw the videos into an AWS s3 bucket and see if that works for you.

This feature will be really helpful.


docker-compose: 1.28.2 -> 1.28.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for docker-compose is: "Multi-container orchestration for Docker"
meta.homepage for docker-compose is: "https://docs.docker.com/compose/"
meta.changelog for docker-compose is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.28.3 with grep in /nix/store/n2gv7k8p05zp44chpqlld161al4f93bx-docker-compose-1.28.3

directory tree listing: https://gist.github.com/1154edb61dcf2551575462ddabad1001

du listing: https://gist.github.com/c7dc317809fa979dcbc04f9a0822170a

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
arion
docker-compose

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n2gv7k8p05zp44chpqlld161al4f93bx-docker-compose-1.28.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A docker-compose https://github.com/r-ryantm/nixpkgs/archive/0a7c4f6a32514ddf502f020d1863adbf2cb83821.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n2gv7k8p05zp44chpqlld161al4f93bx-docker-compose-1.28.3
ls -la /nix/store/n2gv7k8p05zp44chpqlld161al4f93bx-docker-compose-1.28.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

arion
docker-compose (docker_compose)

Maintainer pings
cc @Frostman for testing.

Result of nixpkgs-review pr 113526 at 0a7c4f6a run on aarch64-linux 1

2 packages built:

arion
docker-compose


Bug in OutlineInputBorder's left border when focused

Hello, I was playing with Theme customizations, I tried changing the borders, but it seems that the left borders have problems in rendering.
Even if I change only borders in the right side, the left side seems to be buggy.
The problem happens on both iOS and Android.
Steps to Reproduce

This is the styling I use
focusedBorder: OutlineInputBorder(

    borderRadius: BorderRadius.only(

      bottomLeft: Radius.circular(15.0),

      topRight: Radius.circular(15.0),

    ),

    borderSide: BorderSide(color: Colors.green),

  )

Screenshots

With RTL

Same issue with changing the other sides:

Same thing with only changing the topRight border only

The output of flutter analyze
Analyzing mdc_100_series...
No issues found! (ran in 1.6s)

The output of flutter doctor -v
[✓] Flutter (Channel beta, v0.9.4, on Mac OS X 10.14 18A391, locale en-IQ)
    • Flutter version 0.9.4 at /Applications/flutter
    • Framework revision f37c235c32 (4 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/mohammed/Library/Developer/Xamarin/android-sdk-macosx
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /Users/mohammed/Library/Developer/Xamarin/android-sdk-macosx
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.4)
    • IntelliJ at /Users/mohammed/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 28.0.4
    • Dart plugin version 182.4129.13

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

[!] Connected devices
    • Device QC7LRWRWHUYHDMVG is not authorized.
      You might need to check your device for an authorization dialog.

! Doctor found issues in 2 categories.

My best regards,

Is this still an issue?
Can you please post a minimal runnable reproduction
as a single file so that we can just copy your code into lib/main.dart of a new project and run to reproduce.

Dear @zoechi , thanks for the response
This is the code that reproduce the issue.
import 'package:flutter/material.dart';

void main() => runApp(LoginPage());

class LoginPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    // TODO: implement build
    return MaterialApp(
      home: Scaffold(
        body: Container(
          padding: EdgeInsets.all(50.0),
          child: TextField(
            decoration: InputDecoration(
              labelText: 'Username',
              focusedBorder: OutlineInputBorder(
                borderRadius: BorderRadius.only(
                  bottomLeft: Radius.circular(15.0),
                  topRight: Radius.circular(15.0),
                ),
                borderSide: BorderSide(color: Colors.blue),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

Regards,

This doesn't produce any borders for me

[✓] Flutter (Channel master, v1.2.2-pre.21, on Mac OS X 10.14.3 18D42, locale en-AT)
[✓] Connected device (2 available)
    • Pixel XL                  • HT69V0203649  • android-arm64 • Android 9 (API 28)
    • Android SDK built for x86 • emulator-5554 • android-x86   • Android 9 (API 28) (emulator)

on these 2 devices.

Hi @mohammed-io
Looks like this bug is fixed, not able to reproduce on latest dev channel

flutter doctor -v
[✓] Flutter (Channel dev, v1.18.0-dev.5.0, on Mac OS X 10.15.4 19E287, locale
    en-GB)
    • Flutter version 1.18.0-dev.5.0 at /Users/taha/Code/flutter_dev
    • Framework revision 7f56b53de4 (3 days ago), 2020-04-12 12:00:01 -0400
    • Engine revision beb8a7ec48
    • Dart version 2.8.0 (build 2.8.0-dev.20.0 89b0f67261)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4, Build version 11E146
    • CocoaPods version 1.9.1
[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
[✓] VS Code (version 1.44.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.9.1
[✓] Connected device (4 available)
    • Taha’s iPhone • 00008020-001059882212002E • ios            • iOS 13.4.1
    • macOS         • macOS                     • darwin-x64     • Mac OS X
      10.15.4 19E287
    • Chrome        • chrome                    • web-javascript • Google Chrome
      81.0.4044.92
    • Web Server    • web-server                • web-javascript • Flutter Tools
• No issues found!

if you're still experiencing this problem, please create a new issue and provide the information
required in the template.
Thank you


Encryption id fix #issue34

following this issue: https://github.com/OmarElGabry/miniPHP/issues/34#issuecomment-723137154
that's a solution I have implement long time ago and its stable and works fine at one of my production systems
@OmarElGabry

@OmarElGabry


Build Fix for GCC 12.2.0 , Fedora

While building on Fedora with GCC 12.2, the CMake throws the following error in src/base/sequence_checker.cc & src/base/synchronization/auto_signaller.cc
/libbase/src/base/sequence_checker.cc: In constructor ‘base::SequenceChecker::SequenceChecker(base::SequenceChecker&&)’:
/libbase/src/base/sequence_checker.cc:14:23: error: ‘exchange’ is not a member of ‘std’
   14 |   sequence_id_ = std::exchange(other.sequence_id_, std::nullopt);
      |                       ^~~~~~~~
/libbase/src/base/sequence_checker.cc: In member function ‘base::SequenceChecker& base::SequenceChecker::operator=(base::SequenceChecker&&)’:
/libbase/src/base/sequence_checker.cc:23:23: error: ‘exchange’ is not a member of ‘std’
   23 |   sequence_id_ = std::exchange(other.sequence_id_, std::nullopt);

/libbase/src/base/synchronization/auto_signaller.cc: In constructor ‘base::AutoSignaller::AutoSignaller(base::AutoSignaller&&)’:
/libbase/src/base/synchronization/auto_signaller.cc:16:19: error: ‘exchange’ is not a member of ‘std’
   16 |     : event_(std::exchange(other.event_, nullptr)) {}
      |                   ^~~~~~~~
/libbase/src/base/synchronization/auto_signaller.cc: In member function ‘base::AutoSignaller& base::AutoSignaller::operator=(base::AutoSignaller&&)’:
/libbase/src/base/synchronization/auto_signaller.cc:23:19: error: ‘exchange’ is not a member of ‘std’
   23 |     event_ = std::exchange(other.event_, nullptr);

This has been fixed by adding <utility> header in src/base/sequence_checker.cc and src/base/synchronization/auto_signaller.cc.

The macOS builds are failing because of a linker issue with - Xcode 14.0.1 as discussed here gcc12. I partially resolved it via Xcode 14.1(beta) here patch.
Unless the Xcode issue doesn't issue a patch for linker errors and the gcc9 issue( with Xcode 14.1) we can ignore g++9 on macos-12


Make mypy pass from the command line

Similar issue to https://github.com/dbt-labs/dbt-core/pull/5947 ~(which this includes, and so requires merging #5947 first)~ — currently running mypy from the command line fails, because the configs are in .pre-commit-config.yaml, which isn't available to editors etc.

Description

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

closes #6029


Fall back to MockPin over Native Pin for non-Pi users

If I have gpiozero on a Pi the existing pin factory priority makes sense:

RPi.GPIO
RPIO
pigpio
Native Pin

(i.e. if they have no pins library installed, native pin kicks in and their hardware still works)
However, if I'm on a PC and am running gpiozero, falling back to Native Pin is pointless. It would be better to simply look for pigpio, and fall back to MockPin (or probably MockPWMPin to be safe).
Would it be possible to have different priorities according to whether or not you're on a Pi? If so, do you agree, @waveform80 @lurch?

I'm guessing this didn't get implemented? When I try this on my Ubuntu PC:
>>> from gpiozero import LED
>>> led = LED(4)

I get an error that gpiozero.exc.BadPinFactory: Unable to load any default pin factory!


Uncaught TypeError: Cannot read property 'addEventListener' of undefined in Electron

Hi, I'm trying to get the Demo.html working with an Electron App. However, I get the error:
Uncaught TypeError: Cannot read property 'addEventListener' of undefined

On line 2239 of squire-raw.js:
win.addEventListener( 'focus', this, false );

Any ideas what would be causing this? The html/js is identical to the Demo. Any help appreciated.

I can get it running without errors by using:
var editor = new Squire( document, { ...

instead of:
var editor = new Squire( div, { ...

But that means I lose the toolbar.

Have you tried using Squire from the master branch? I don't think this line exists anymore, so it's probably worth trying the latest version first.

I've since moved on to implementing my own editor. Thanks for responding but I'll close this off.


fl_chart and InteractiveViewer doesn't allow pan gestures.

Describe the bug
When using InteractiveViewer to create a pan-able, zoom-able chart in mobile, the default gesture recognizers of Line Chart absorb all gestures and so it stops the InteractiveViewer from panning the view.
I assume this is related to the refactoring of the touch system, and what should be used is another breaking change where the touchCallback returns a boolean where it says if it handled the gesture and such, to allow the gesture event to bubble up the chain.
EDIT: Checked the code and apparently it's best to simply stop setting handlers for all possible touch behaviors and instead pass handlers manually. https://github.com/imaNNeoFighT/fl_chart/blob/47d83844c10def4167f6bf16e8ab68693b8c82ae/lib/src/chart/base/base_chart/render_base_chart.dart#L50-L74
To Reproduce
Soon ™
Screenshots
This seems similar to the change in issue #772.
Versions

Flutter: stable 2.5.2
FlChart: 0.4.0

Any updates on this? Been using fl_chart and now have a new requirement of having a zoom feature within the charts.

Zoom feature is discussed in another issue #71.
Interactive Viewer is not our preffered way to do this.


Format Only Modified Lines?

Hi,
Thanks for this plugin, solves some major issues we were having with import organization issues between prettier and VSCode and lots of pain as a result.
That said, I was wondering if you might consider adding an option for only formatting modified lines?  I have previously set editor.formatOnSaveMode to modifications which causes it to use the Format Modified Lines command instead per: https://github.com/microsoft/vscode/issues/44075#issuecomment-678227183 - not sure what the best way to integrate it here would be, but it would be nice to have the option to not have to format the whole document every time now in order to use this plugin / get the fixes it provides elsewhere.
Thanks again!

@rohit-gohri when do this go live? :D
in dire need

@rohit-gohri when do this go live? :D
in dire need

There were some bugs with the implementation, will try to at least release a beta for testing purposes by the weekend. So I can collect some feedback.

There were some bugs with the implementation, will try to at least release a beta for testing purposes by the weekend. So I can collect some feedback.

If anyone has experience with extension development, I would appreciate their review a on the PR (#6)

If anyone has experience with extension development, I would appreciate their review a on the PR (#6)

Reviewed! I'm not the most experienced with extension development but it all made sense / looked good to me.

Reviewed! I'm not the most experienced with extension development but it all made sense / looked good to me.

Sorry for the delays, have finally released the feature. I haven't personally used formatModified before so not sure if the behavior is 100% correct. Let me know if you face any problems

Sorry for the delays, have finally released the feature. I haven't personally used formatModified before so not sure if the behavior is 100% correct. Let me know if you face any problems

a quick test without eslint seems to be working

a quick test without eslint seems to be working

Closing this now. If anyone faces any new problems, feel free to open a new issue.
And, thanks for the PR review @cojo 👍

@rohit-gohri not sure if this i should create a new issue but

when i use formatModified. but it does its job fixes eslint errors, but quite annoying

@CoLdAs1cE It'll be great if you could open a new issue with a small code sample that triggers this and your eslint & prettier configs.

@CoLdAs1cE the error could be because prettier extension itself doesn't support formatModified maybe? https://github.com/prettier/prettier-vscode/issues/1677

@CoLdAs1cE the error could be because prettier extension itself doesn't support formatModified maybe? prettier/prettier-vscode#1677

I did ask, they say it saves on modified

UPDATE:
Wow i had the old version and could not get this to work at the time so i was switching back and forth between prettier and this, so i had on format on save which gives this error 😞

So, working now I guess?

Yeah, sorry about unclear answer :P


Added missing asset for magic monarch & added eth wire fleece

I changed my PR based on your review input and created another commit for this PR (https://github.com/aeon0/botty/pull/70/commits/70602353de36e06c2fa9895ba8f9d15a3b3e39eb). Is this the correct way or do you need anything else from me?

I changed my PR based on your review input and created another commit for this PR (7060235). Is this the correct way or do you need anything else from me?

Nope, that's how I wanted it : ) thx


skip_covered, skip_empty for LCOV file?

Is it possible to use skip_covered, skip_empty options when generating LCOV file? I only see it works on general coverage report command.

Similar to #871, I would prefer that generated data files (xml, json, lcov) have all the data, and let the consumer/renderer of the data handle filtering like this.


In-App Purchases

Any plans to add In-App Purchases to Parse Server?

No plans to add receipt validation to parse-server from our end, but, totally open to it being added as a community feature.

@gfosco was there validation before on Parse? That would need to be added into the iOS SDK in that case for convenience don't you think?

Follow on in #134.


I couldn't find set-up-alert doc

I couldn't find set-up-alert doc. the old link is : https://docs.microsoft.com/en-us/azure/billing/billing-set-up-alerts

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

ID: 694a543a-e2f6-1046-b618-01acc45c4dfe
Version Independent ID: 392351bb-9bba-0e62-c6ef-229c508e311d
Content: Prevent unexpected costs, manage billing in Azure
Content Source: articles/billing/billing-getting-started.md
Service: billing
GitHub Login: @tonguyen10
Microsoft Alias: cwatson

It seems like this feature is not working so the document is deprecated and removed.
Probably, Microsoft Billing team can give you more details. I was just looking for an issue to create a PR so when I debugged the issue I got the info.
checkout this link - https://github.com/MicrosoftDocs/azure-docs/tree/master/articles/billing

Billing alert is mentioned in the new AZ-100T01 course. Has it been deprecated or renamed?

reassigning to @cwatson-cat  to take appropriate action. unable to resolve @tonguyen10 in GH assignee list.

Hi @tbkobberoe - The billing alert feature as it was implemented wasn't working well for customers and  is deprecated - so we removed the article. @manish-shukla01 FYI

But you put it in the brand new class?

@travellingtrainer Ive been told that there is a 3 month period from a course has been created and signed off until it is published as there are some finishing that needs to be done. Thats probably why we see this issue

Hello All,
We recently released the Azure Budgets feature in Ibiza to replace the cost thresholds that you have previously been using (supporting both Enterprise and Pay-As-You-Go customers). While the current cost thresholds will continue to work, please begin using our Budgets feature moving forward. To learn more, visit our documentation page found here: https://docs.microsoft.com/en-us/azure/cost-management/tutorial-acm-create-budgets?toc=/azure/billing/TOC.json

#please-close


Can't find protobuf-c package since #999

Describe the bug
Since the merge of #999 compiling failed. If you check my Dockerfile's 48 & 49 line, both libprotobuf-c-dev and protobuf-c-compiler is already installed but still it gives me below error while compiling:
#24 60.29 configure: error: The protobuf-c package was not found with pkg-config. Please install protobuf-c!

You could --enable-systemd, that could accidentally work, or instead specify with a path, --with-protobuf-c=/usr. But my colleague has found the correct fix for this.


from relative to fixed position

it's throwing errors when I try to get from relative to fixed position(Cannot read property 'length' of null) should I use upRemoveClass for this or something?

yeah, probably, looking at the demo...
Btw I really like the directive, haven't yet tested the performance, but other than that, really nice, better than skrollr which was my previous favorite.. if only it had a better documentation, wouldn't even know about spark-id if I didn't check the source..

Thanks fxck. If you'd like to improve the documentation I'm happy to accept PRs.


KERI-9: C Programming memory management

6.21 Demonstrate skill in controlling memory

With attention given to implementation defined behavior, compare and contrast standard memory allocation functions (e.g ., malloc () calloc(), realloc ( ), and free ())
Demonstrate appropriate error checking when managing memory allocations
Describe programming techniques that reduce the occurrence of memory leaks (e.g ., behaviors that reinforce a clear ownership model)
Demonstrate effective use of Valgrind with --leak-check=full to identify memory leaks
Given code samples, identify and remove memory leaks

Done and merged