Flutter screenshot error when run command on subfolder

Steps to Reproduce

Click run on void main().
On terminal, run mkdir screenshot
cd screenshot
Run flutter screenshot.

Expected results:
Screenshot files should save in <project_folder>/screenshot
Actual results:
Error such as follows:
Must have a connected device for screenshot type device
PS D:\Iqmal\Documents\Flutter apps\flutter_feedback_ui_by_neecoder_x\screenshots> flutter screenshot
No supported devices connected.
The following devices were found, but are not supported by this project:
sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
Chrome (web)                • chrome        • web-javascript • Google Chrome 88.0.4324.104
Edge (web)                  • edge          • web-javascript • Microsoft Edge 88.0.705.56
If you would like your app to run on android or web, consider running `flutter create .` to generate projects
for these platforms.

Logs

[√] Flutter (Channel beta, 1.25.0-8.3.pre, on Microsoft Windows [Version 10.0.19042.746], locale en-MY)
    • Flutter version 1.25.0-8.3.pre at C:\src\flutter\flutter
    • Framework revision 5d36f2e7f5 (3 weeks ago), 2021-01-14 15:57:49 -0800
    • Engine revision 7a8f8ca02c
    • Dart version 2.12.0 (build 2.12.0-133.7.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Iqmal\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\Iqmal\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.52.1)
    • VS Code at C:\Users\Iqmal\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.19.0

[√] Connected device (3 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 88.0.4324.104
    • Edge (web)                  • edge          • web-javascript • Microsoft Edge 88.0.705.56

• No issues found!

Hi @iqfareez
Thanks for filling the issue, it's not a sub-directory or any directory outside of the project that's causing the issue BUT flutter screenshot is confused with multiple devices  but when you specify device -id it works fine
Ex:
tahatesser@Tahas-MacBook-Pro screenshot % flutterm screenshot -d rmx
Screenshot written to flutter_01.png (53kB).
tahatesser@Tahas-MacBook-Pro screenshot % `

tahatesser@Tahas-MacBook-Pro screenshot % flutterm screenshot
No supported devices connected.
The following devices were found, but are not supported by this project:
RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 10 (API 29)
macOS (desktop)  • macos            • darwin-x64     • macOS 11.1 20C69 darwin-x64
Chrome (web)     • chrome           • web-javascript • Google Chrome 88.0.4324.96
If you would like your app to run on android or macos or web, consider running `flutter create .` to generate projects for these
platforms.

Must have a connected device for screenshot type device
tahatesser@Tahas-MacBook-Pro screenshot %

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.6, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.22.6 at /Users/tahatesser/Code/flutter_stable
    • Framework revision 9b2d32b605 (12 days ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.4)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[!] Android Studio (version 4.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_242-release-1644-b3-6915495)

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

[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.

[✓] Flutter (Channel master, 1.26.0-18.0.pre.156, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.26.0-18.0.pre.156 at /Users/tahatesser/Code/flutter_master
    • Framework revision 45508985b1 (8 hours ago), 2021-02-02 22:46:04 -0500
    • Engine revision 2c144c3eeb
    • Dart version 2.12.0 (build 2.12.0-282.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

Based on the behavior, this makes a duplicate of https://github.com/flutter/flutter/issues/50039
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you

Thank you. It works!


Get error: module 'bokeh' has no attribute '__version'

Describe the bug
A clear and concise description of what the bug is and how to reproduce it. If applicable, add screenshots.
Environment
OS: Windows 10
Python: 3.11.7
IDE: PyCharm
Package versions
ipython==8.20.0
ipython-genutils==0.2.0
pandasgui==0.2.14
plotly==5.9.0
PyQt5==5.15.10
PyQt5-sip==12.13.0
PyQtWebEngine==5.15.10
bokeh==3.3.0
In source file utility.py:446, following code:
in get_figure_type
if int(bokeh.__version.split(".")[0]) < 3:
^^^^^^^^^^^^^^^
AttributeError: module 'bokeh' has no attribute '__version'. Did you mean: 'version'?
Manual fix: after changing '__version' according to the instruction above provided by PyCharm, everything works OK.

Just noting that there has never been a bokeh.__version attribute without trailing underscores. Instead, there has always been a bokeh.__version__ attribute, as is common for python packages.


Add x-request-id in debug logs for tvm client

TVM already has x-request-id header added to response.
TVM Client needs to add this to debug log for easy troubleshooting
Fixes - https://github.com/adobe/aio-tvm/issues/44
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

TVM already has x-request-id header added to response

We should also make sure that the TVM should logs the request id, so that we can map an exception to a log on the server

TVM already has x-request-id header added to response

We should also make sure that the TVM logs the request id, so that we can map an exception to a log on the server

Added PR for TVM - https://github.com/adobe/aio-tvm/pull/47


5Mbps data bitrate not working

Not sure if this is an issue with the Cantact Pro firmware or the gs_usb_fd driver, but I can't generate a 5mbit CANFD data bitrate. I haven't tested this extensively, hopefully I can do some more testing soon.
Test case:

CANtact Pro with CAN0 connected to CAN1
Both interfaces brought up with the same bitrates:

1Mbps standard bitrate
5Mbps data bitrate

Using gs_usb_fd driver at your latest commit https://github.com/linklayer/gs_usb_fd/commit/76af2fde69f57d6872863f010541a83df56871d3

Result:
Repeated TX of packet that does not appear to have anything in the data portion of the packet, after some attempts it appears to give up on TXing packets that are not ACKed.
Scope Shot

Have you successfully tested other/slower data bitrates on the cantact yet?
Assuming you probed CAN_L it seems that the "receiving" CAN node is destroying the transmit by sending an error frame.
More details are harsh to see due to bad screen representation.
FD without BRS works?

Hey @normaldotcom,
I've changed the clocking of the CAN cores in the CANtact, they are now clocked with 40 MHz not 24 MHz. If you want to test it. This procedure works for me, hopefully you will not brick your device:

unplug
press and hold the button next to USB connector
plug USB

Now the light should be quite dimm. The device should show up as an USB mass storage device:

mount USB mass storage
make a copy of the firmware.bin in a secure place (that's the original FW)
erase the firmware.bin
extract the attached firmware.zip somewhere else (e.g. /tmp)
copy the new firmware.bin to the mounted USB mass storage
unmount
unplug + plug form USB

$ ip -details -statistic link show can0
9: can0: <NOARP,UP,LOWER_UP,ECHO> mtu 16 qdisc pfifo_fast state UP mode DEFAULT group default qlen 10
    link/can  promiscuity 0 minmtu 0 maxmtu 0
    can state ERROR-ACTIVE restart-ms 1000
          bitrate 500000 sample-point 0.875
          tq 125 prop-seg 6 phase-seg1 7 phase-seg2 2 sjw 1
          gs_usb: tseg1 1..16 tseg2 1..8 sjw 1..4 brp 1..1024 brp-inc 1
          gs_usb: dtseg1 1..16 dtseg2 1..8 dsjw 1..4 dbrp 1..1024 dbrp-inc 1
          clock 40000000

The new firmware has a clock of 40000000 the original one uses 24000000.
          re-started bus-errors arbit-lost error-warn error-pass bus-off
          0          0          0          0          0          0         numtxqueues 1 numrxqueues 1 gso_max_size 65536 gso_max_segs 65535
    RX: bytes  packets  errors  dropped missed  mcast
    0          0        0       0       0       0
    TX: bytes  packets  errors  dropped carrier collsns
    0          0        0       0       0       0

firmware.zip


Altcoin issue

Hello Iceland , i test the program using doge coin , any peticular tricks so i can use it ? i had error
Loading addresses from 'doge.txt'
terminate called after throwing an instance of 'std::out_of_range'
what():  basic_string::substr: __pos (which is<PHONE_NUMBER>3709551615) > this->size() (which is 0)
Aborted (core dumped)
Please Help ,
Regards

I think this error maybe, you are giving Dogecoin Address in the input file. You must give Hash160 file not the address.
HASH160 is the RipeMD160 hash of the SHA256 from the Pubkey.
This can be back calculated from the Dogecoin address (and for all of those Altcoin Address) by Base58decode and removing Checksum and network bytes.

thank you for your kind reply iceland , i convert the doge address  using btcposbal converter , are they same ?
best regards

No idea about this converter. But if using python with library bit, you can convert entire doge_list of address into h160_list...
h160_list = [bit.base58.b58decode_check(address)[1:].hex() for address in doge_list]

thank you for your kind reply iceland , now its working like charm
regards


Support later versions of STIX 2

We need to figure out how to support later versions of STIX 2 in the validator. It should probably default to the latest published version of the spec, with an option to set the version to validate against. People may want to validate against multiple versions of the spec if they are supporting multiple versions at once. Each version of the validator could have a maximum supported STIX version, or we could do something with subpackages like https://github.com/oasis-open/cti-python-stix2/issues/79. I'm leaning towards the latter.

I think you can have multiple submodules for the same repo (at different commits) for multiple copies of the JSON schemas. In terms of supporting multiple versions, the validator should:

if validating a bundle, use the version specified in the bundle.
validate against the most recent version by default (this means that we'll need a new major release for each STIX version, since behavior could be backwards-incompatible unless the specs are truly backwards compatible).
Allow users to specify an explicit version via a command line argument or function parameter.

At first I was confused by "multiple submodules for the same repo" - now I understand you mean git submodules, not python submodules.

I updated my comment above to be a bit more clear 😀

@clenk: Do you think it's important to figure this out before we release a 1.0, in case it causes significant internal refactoring of the public API?

Yes, to be on the safe side I think so. I've been meaning to work on this for a while but haven't found the time.

We'll need a major version bump anyway when we switch from validating STIX 2.0 by default to validating STIX 2.1 by default, so there's no reason this needs to be done prior to 1.0.


Product price should be casted as ShopCurrency

Product price should be casted as ShopCurrency.
It seemed symbol changes only come through in the PriceRange function and not the standard sellingPrice function.
Changing the casting to ShopCurrency seemed to fix this.

Sorry, I've just noticed your Object::useClass... ignore this!


Edit and view buttons at top of chapter 2 lead to 404 errors

back-link: en/ch02/index.html#get-started
The two buttons "edit" and "view" at the top of the chapter page lead to 404 errors. The URLs they link to are:
https://github.com/qmlbook/qmlbook//edit/master/index.rst
https://github.com/qmlbook/qmlbook//blob/master/index.rst

Another 404 happens when I try to add an issue to a subsection:

Link goes to https://github.com/qmlbook/qmlbook//new?labels=ch02&body=back-link%3A+en%2Fch02%2Findex.html%23hello-world

Thank you for this report. I'll log this as a bug and try to address it when overhauling the travis setup.


Move translations in JSON format to a different key

Is your feature related to a specific framework or general for this extension
No
Is your feature request related to a problem? Please describe.
Yes, I want to be able to move translations around in JSON format without having to go through each language file and do the move manually. Example JSON:
{
  "a": {
    "1": "Value!"
  },
  "b": {
    "2": "Value!"
  }
}

I would like to be able to say for path a --> 1 to move it to b --> 1.
Describe the solution you'd like
Maybe via right-click --> move command?
Additional context
Babeledit has the functionality I described, but you need a paid license to do so.

Do you mean merge the two together?

@antfu Let me clarify with the resulting JSON:
{
  "b": {
    "1": "Value!",
    "2": "Value!"
  }
}

Which, in this case means merging them together, but effectively I've moved a1 to b1. I could also have moved a1 to c1 for example and then it wouldn't be merging. Does that make sense?

It's called rename key in this extension. You can right click the field in sidebar.

Alright, how can I look past that... Thanks!

I noticed it is quite slow initially so I excluded all of my html files from scanning for keys and that definitely speeds things up a lot. Managing the translations quickly is more important for me than being able to see translated values in the html. Also being able to see the translated values in the html requires a deeper understanding of how transloco scopes work.


[HotbarPlus Suggestion] Custom configurable swap delay instead of a boolean

I just want to say that I'm sorry for sending this here, I know this isn't the right repo.
I didn't know where to go to suggest this feature since there's no obvious way to contact you, and there's also no repo for HotbarPlus.
Basically, right now you can either have a ton of delay or no delay at all.
I think it would be great if you could make it so that we can set the delay ourselves.
Oh and it would be nice if you could have an option for number keys/hot keys to not be affected by the delays
Again, sorry for sending this report here.

Nah, it's all good! When I have some free time, maybe this weekend, I'll look into this!

Moved report to https://github.com/cmooref17/Lethal-Company-HotbarPlus/issues/1


Speed up boost vector iterators

Fixes #7639
Boost Iterators are amazingly slow for iteration, but accessing by index is very fast.  This converts the iter protocol to use the getitem protocol and results in a > 300x speedup for iteration.

I've added the test note book to:
Code/GraphMol/ChemReactions/testData/GetPosition.ipynb
Here are the tests
from rdkit import Chem
m = Chem.MolFromSmiles("CCCCCCCCCCCC")
ps = Chem.FindAllPathsOfLengthN(m, 3)
%timeit [list(x) for x in ps]

Before: 5.04 ms ± 24.9 μs per loop (mean ± std. dev. of 7 runs, 100 loops each)
after: 24.1 μs ± 53.9 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

from rdkit import Chem
from rdkit.Chem import AllChem
from rdkit.Chem import rdChemReactions

for N in range(3, 40):
    carbons = [f"[C:{i}]" for i in range(1,N+1)]
    rxn = f"{'.'.join(carbons)}>>{''.join(carbons)}"
    rxn = AllChem.ReactionFromSmarts(rxn)
    mol = Chem.MolFromSmiles("C")
    bbs = []
    for i in range(N):
        bbs.append( [mol] * 5 )

    lib = rdChemReactions.EnumerateLibrary(rxn, bbs)
    print(f"{N=}", len(lib.GetPosition()), end=" ")
    %timeit [x for x in lib.GetPosition()]

It looks like the bulk of the cost is returning the position as the timings are not dependent on the return size.  On OSX at least, it's about twice as fast.
Before:
N=3 3 652 μs ± 465 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=4 4 653 μs ± 2.56 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=5 5 652 μs ± 1.28 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=6 6 652 μs ± 1.28 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=7 7 653 μs ± 1.79 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=8 8 654 μs ± 1.55 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=9 9 655 μs ± 2.49 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=10 10 654 μs ± 1.49 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)

After:
N=3 3 383 μs ± 1.39 μs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=4 4 383 μs ± 879 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=5 5 383 μs ± 874 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=6 6 383 μs ± 575 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=7 7 382 μs ± 668 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=8 8 383 μs ± 697 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=9 9 383 μs ± 563 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
N=10 10 383 μs ± 840 ns per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
...

Running on linux (using g++) I see a factor of ~4 speedup on the first case (the one with FindAllPathsOfLengthN) and no change for the second example.

It looks like the vector wrapped in rdChemReactions isn't in rdBase, adding this adds the speedup
rdChemReactions.VectSizeT.__iter__ = __vect__iter__

Before
N=3 3 24.9 μs ± 283 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)
N=4 4 24.9 μs ± 161 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)
N=5 5 25.2 μs ± 105 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

After
N=3 3 3.64 μs ± 29.4 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)
N=4 4 4.17 μs ± 47.3 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)
N=5 5 4.78 μs ± 81.8 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

I'll see what I can do here, but fixing the underlying vector indexing suite would be a generic solution I think.

I wasn't using the same build for testing which was an oversite.  Maybe
clang got better at optimizing something since the last release.
On Wed, Aug 21, 2024 at 9:35 AM Greg Landrum @.***>
wrote:

It looks like the vector wrapped in rdChemReactions isn't in rdBase,
adding this adds the speedup
If this is true, and it makes sense that it would be, how did you see the
factor of two speed improvement that you showed above?
I'll see what I can do here, but fixing the underlying vector indexing
suite would be a generic solution I think.
Agreed, but that is likely to be "deeply non-trivial"
—
Reply to this email directly, view it on GitHub
https://github.com/rdkit/rdkit/pull/7719#issuecomment-2302077182, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLOFIAH2Y65ZY6GOWM7EDLZSSJQ5AVCNFSM6AAAAABMQSTC7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBSGA3TOMJYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

The newer tests used exactly the same build.
On Wed, Aug 21, 2024 at 9:49 AM Brian Kelley @.***> wrote:

I wasn't using the same build for testing which was an oversite.  Maybe
clang got better at optimizing something since the last release.
On Wed, Aug 21, 2024 at 9:35 AM Greg Landrum @.***>
wrote:

It looks like the vector wrapped in rdChemReactions isn't in rdBase,
adding this adds the speedup
If this is true, and it makes sense that it would be, how did you see the
factor of two speed improvement that you showed above?
I'll see what I can do here, but fixing the underlying vector indexing
suite would be a generic solution I think.
Agreed, but that is likely to be "deeply non-trivial"
—
Reply to this email directly, view it on GitHub
https://github.com/rdkit/rdkit/pull/7719#issuecomment-2302077182, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLOFIAH2Y65ZY6GOWM7EDLZSSJQ5AVCNFSM6AAAAABMQSTC7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBSGA3TOMJYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

We use RegisterVector iterators in a lot of places that would need to be monkey patched.  Like GetPositions.  I'll see how hard this is to consolidate, I think most std::vector should be wrapped at the lowest level, i.e. RDBoost.  And we could wrap std::vector<RWMOL_SPTR/ROMOL_SPTR> in rdkit.Chem which should cover most of the major issues.  I'll get this done this weekend.

@bp-kelley maybe that can be done in a separate PR? If you remove the notebook from this one, I can go ahead and merge it and we'll have the partial solution out there in the upcoming patch release.

@greglandrum after this is merged I have a quick follow up where I register numeric vectors in RDBase.cpp, this should fix the original issue.

@greglandrum I think removing the notebook was your change request, let me know if there is anything else


determine why our routing/network dataset can only be added through arc/catalog

network = C:\\temp\\RegionalTransitDatabase\\data\\TomTom_2015_12_NW.gdb\\TomTom_2015_12_NW.gdb\\Routing\\Routing_ND"

arcpy.na.MakeRouteLayer(network, "WorkRoute", "TravelTime")

throws the following error:
Runtime error  Traceback (most recent call last):   File "<string>", line 1, in <module>   File "c:\program files (x86)\arcgis\desktop10.4\arcpy\arcpy\na.py", line 1280, in MakeRouteLayer     raise e ExecuteError: ERROR 000732: Input Analysis Network: Dataset C:\temp\RegionalTransitDatabase\data\TomTom_2015_12_NW.gdb\TomTom_2015_12_NW.gdb\Routing\Routing_ND does not exist or is not supported

or, if executed on the data through the toolbox dialog:
ExecuteError: ERROR 030033: Parameter does not contain a network dataset data element.

Adding the network dataset by dragging it from Catalog to the TOC makes it available for use. Arc puts up a dialog titled Adding Network Dataset, which indicates that there is some special subset of the arcpy Layer adding tool that should allow us to add the network dataset to the mxd. But in my effort to use that tool I hit the same as the first error above, and I can't seem to find a tool in the na.py arcpy tool that would allow us to validate this network dataset or do any kind of introspection on it.
So for now, we'll just have to manually drag things from catalog into the TOC.

looks like this was because the .gdb was doubled up in a folder /x.gdb/x.gdb


Postman collection issue

Description:
If the user has a resource " /* "  [1] then when user downloads the postman collection, the request with " /* " has a " /* " in the URL [2]. Which is invalid and will result in 404 error.
[1]
[2]

Steps to reproduce:

Create an API with Resource "/*"
deploy and publish
Go to devPortal
Go to Try out
Download Postman Collection
Import the collection to postman
Try to invoke the api with token generated

Affected Product Version:
API-M 4.0.0-beta
Environment details (with versions):

OS: Windows

Suggested Labels:
API-M 4.0.0

Having /* at the end implies that user can add any resource afterwards. They have to modify it anyway when using postman.


Infinite token

I would like to know if there is a way to have an infinite token, so I dont have to refresh the expiration time.
Thanks.

I think if you leave out the exp claim token will never expire.

Just like this?
`
$payload = [
"iat" => $now->getTimeStamp(),
"jti" => $jti,
"sub" => $server["PHP_AUTH_USER"]
];
`

@thiagok Even if you technically could, I would argue that you should not. Think about it like this: if you provide a user with an infinitely lasting token, how would you remove access for that token in the future if you needed to? As far as I know, the only way would be to change your application secret, something that would impact all users by invalidating every single token.
Furthermore, I think there is no good reason to hand out infinitely lasting tokens. Having an expiry date (even if it is a year) guarantees nobody will be able to abuse your services infinitely.


opensleigh.net is down

Hey bro, it seems like opensleigh.net is down. Could you possibly up it?

hi @NotIlya4, I dropped the website a while ago. I am working on a new version of the library, it's in alpha on Nuget right now. I will probably push the docs here on GitHub Pages in the near future.
Just out of curiosity, where did you find the link opensleigh.net?

here https://github.com/mizrael/OpenSleigh/blob/develop/src/OpenSleigh/README.md


Updated Clarifai functions with new clarifai-python SDK

Updated functions

added inference params to model predict

modified embed size

modified pat args

Description
The aim of this change is to utilize clarifai python SDK functions in the class.

Updated bulk predict with SDK funciton.
updated embeddings and llm class to support CLARIFAI PAT as an arg.
Updated notebooks with new examples.

Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Tested new functions in the notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review as well as peer-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@jerryjliu  could you please review this ?

This looks fine to me -- sorry for the delay!


chore: modify latest version msg

Context
This PR updates the latest version message to enhance user engagement and clarity.
The change from it is the latest version to You are running the latest version adds a personal touch, making the message feel more user-centric.
This small adjustment aims to create a more positive and user-friendly experience, reinforcing the idea that the user is in control and up-to-date.
Description
Safe to merge PR: just modified the message for the latest version.
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @R3DRUN3, I like that personal touch, thanks!
Please, in order to pass our tests, can you run our pre-commit? or just execute the black library poetry run black . ?

poetry run black .

@sergargar I've run the code formatter, the PR should pass the tests now! 😊

It seems that you have an old version of black, can you do a poetry install? Also, please discard the changes in tests/providers/aws/services/iam/iam_inline_policy_no_administrative_privileges/iam_inline_policy_no_administrative_privileges_test.py file.

iam_inline_policy_no_administrative_privileges_test.py

@sergargar  Done!

iam_inline_policy_no_administrative_privileges_test.py

@sergargar  Done!

I still see changes in that file 😕

Closing this, I'll submit a new PR in the future


[Bug] Folder permission is causing delay to play videos

Describe the bug
Video takes some time to play after permission to a folder is granted. (maybe caused by searching for a subtitle?)
To Reproduce

Long press on the folder icon in player
Proceed with the dialog by pressing ok
give permission to a folder containing multiple video files, maybe 100 or more (folder with a few or no items will not cause this)
Open a video and see how long it takes to play (mine is 5-8 secs)

Expected behavior
No delay on opening a video.
Device

Model: Motorola One Fusion
OS: Android 11
Version: RPLS31.Q2-63-10-2-9

Yes, the subtitle search is indeed that slow. It can take a few seconds (if there are just dozens or hundereds of files) even on high end devices. This would really be good to improve. I could search for subtitles in the background and try to attach it but should I restore playback to that position or start it again from the beginning in case the user might miss a line?

what if it tries to find only one file with the same name as the video? would it be faster? I think vlc works that way.


Importing delete "Don't Import" matrix field content

On updating Commerce_Product (match by product slug & variant's SKU => update variant's QTY)
Other Matrix fields (on another tab) on the product get removed even set to "Don't Import"
Check the url below to see an example feed i used.
http://gmyle.s3.amazonaws.com/feed/craftcms/general-product.xml
====
Import Strategy: Update existing elements
A+ content (the matrix field) is set to "Don't Import"

Confirming that I have this problem too

@stianmandarin @idolab Have you both tried the latest master to confirm if it is still happening?

@engram-design Hey, thanks for getting back. I'm already on 2.0.7 (that is the latest right?)

@stianmandarin The master has got a few fixes I'm about to roll into a 2.0.8 release

@engram-design Aha, I see. Tried the master now, but the problem persists

@engram-design Tried the master too, but the problem persists +1

Hi, any news on this? I've got the same problem too ...

Sorry to be a pain, but are there any news on this? We're about to launch a site, so it's a bit critical

Hi all, this seems to be an issue with the newly added 'Enabled' block option. Selectize.js apparently doesn't let you select blank values (for Don't Import) so they only way around this at the moment is to select the Enabled dropdown field and hit backspace - this will save with an empty value.
Pushing a fix for this regardless shortly.

This is fixed in the latest master, but as I mentioned, for all current feeds, you'll need to actively remove the value from the Enabled default field by pressing backspace. This is because its already saved as importing as enabled.
The way field mapping works at the moment, is that if any value is mapped to a field (even defaults) it'll enact the parsing. This will change in the Craft 3 version of Feed Me.


list top artists/playlist, add mime types, fallback to default mime type

Aggregates the top artists per playlist and exposes them comma-separated within the comment field of each playlist. Also optimizes direct/single playlist access.
Map mime types for m4a, mp4 and opus file extensions.
When no mime type registered for file extension, fallback to serving the file using mime type application/octet-stream (and log a warning) instead of failing the request. (At least GStreamer/Mopidy seems to do its own format detection anyway since it still plays such streams.)

@BinaryBrain it would be great if you could review and merge this PR soon. Not having the mime type fix/enhancement prevents me from serving my music library using upstream Beetstream and I guess others have the same problem, e.g. see #11.

Yes, I've been quite busy recently and the current version is broken if you don't have the smartplaylist plugin.

ah, right, it fails here when neither the smartplaylist plugin was enabled nor the playlist_dir option specified. I'll add a commit to this PR to fix it...

Done: Now I've pushed another commit to not fail but log a warning when the smartplaylist plugin is not enabled and the playlist_dir configuration option was not specified.


Update Account Settings page

Why:

The current account settings page shows the user profile info and billing information.  The billing system is not currently implemented and can be hidden for the time being.  As for the user profile information, it was decided that it would probably be better to send the user to the 'Actors' show page for the currently logged in user.

@bmanifold please add basic tests and LGTM.

Apologies for forgetting tests. With this page being so static tests slipped my mind.  I'll get some added.


small typo I18n -> i18n

Please verify the following:

[ ] yarn ci:test jest tests pass with new tests, if relevant
[ ] README.md has been updated with your changes, if relevant

Describe your PR
This is a small typo I noticed.

:tada: This PR is included in version 6.9.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


don't run Google's analytics.js on local

This is for the Hugo docs.
This helps resolve #28351

@coliff : can you rebase against the upstream twbs:v4-dev-xmr-hugo-2 branch?
BTW does this fix #28351 completely?

Nvm, I don't think it resolves #28351.

Oh, and please remove the resolve from your commit message while at it.

hey @XhmikosR - sorry not got back to this yet. Do I need to do anything now?

@coliff: yeah, all of the above :)

Not sure how to proceed with this honestly. I personally have no issue with this, nor with the analytics running.
But I still need to above things fixed in order to merge it.
/CC @MartijnCuppens @Johann-S @mdo for thoughts.

Rebase here got messed up.

Yeah but do you want this change? I will take care if it myself, I'm just
not sure 100% I want this. Both ways are Ok to me
On Tue, Mar 12, 2019, 22:22 Mark Otto<EMAIL_ADDRESS>wrote:

Rebase here got messed up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/28384#issuecomment-472164916, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVVtRwMpD4w5SCElBI2v7VWTms8LnR-ks5vWAyOgaJpZM4bYKFL
.

Yeah but do you want this change? I will take care if it myself, I'm just not sure 100% I want this. Both ways are Ok to me

I can't see what the diff is easily, but based on the title here, I'm good not running Google Analytics on localhost while doing development.

The diff is just the last patch, the rest will go away with a proper rebase. @coliff I will make the changes, but it'd be nice if you could take care of the comments yourself.


clusterpool: allow setting max size and max concurrent actions

xref: https://issues.redhat.com/browse/CO-1278
Today the cluster pool keeps creating clusters to match .spec.size as ready clusters
are claimed by users. This is usually not possible as the secret used by the cluster pool usually
maps to an account in cloud that has limits/quotas and therefore there is need to allow admin
to set a limit on the maximum number of clusters created by a pool (already claimed + unclaimed).
The new field .spec.maxSize allows the admin to set this limit on maximum number of clusters
that are created for a pool. This includes the already claimed clusters, clusters ready to be claimed and
clusters that being provisioned, and does not include clusters are being deleted.
NOTE: when unset, this implies Inf capacity.
Similar to quotas in accounts, most cloud providers have API quotas that limit number of cloud API interactions
over a period of time. An close approximation of this can be made in terms of number of clusters being brought
up/down as these are periods of high cloud API interactions. So if the admin can define a limit on the
number of UP/DOWN activity that can happen for a pool, it allows the admin to prevent rate limiting.
The new field .spec.maxConcurrent specifies the maximum number of cluster create/delete can be in progress at
a given instant.
NOTE: when unset, this implies Inf concurrent limit.

/assign @dgoodwin
/cc @abutcher

/lgtm


Reasonable error message and error state on failed notifications

As user
I want error state and reasonable error message in case notification approval fails
so that could track the correct status of the notification
Hints / Details

Acceptance Criteria

[ ] ERROR Status for Quality topics in error
[ ] Detail view contains reasonable error messages in message history
[ ] Reasonable error messages with cause why sending fails
[ ] Quality topics in status error could be retried with command "Approve". Retry is limited to 10 times. After that command, "Approve" is disabled with a reasonable disabling message. "Approve of quality topic retry is limited to 10 times"

Out of Scope

...

We need to align on a mapping of errors, this could be implemented already with this issue - https://github.com/eclipse-tractusx/traceability-foss/issues/787


Claims are missing in PreAuthorisationMiddleware in v16.0.0

Expected Behavior
In a PreAuthorisationMiddleware, the claims should exist on the ClaimsPrinciple on the HttpContext.  This is needed for custom Authorization logic.
Actual Behavior
There are no Claims.
Steps to Reproduce the Problem
Using Ocelot 15.0.7, set up the pipeline with a custom PreAuthorisationMiddleware:
app.UseAuthentication();
var pipelineConfiguration = new OcelotPipelineConfiguration
{
    PreAuthorisationMiddleware = async (context, next) =>
    {
          var claimCount = context.User.Claims.Count();
          await next.Invoke();
    },
};

Value of claimCount is 13 (at least for my case).
Upgrade to Ocelot 16.0.0 and run again.  Value of claimCount is 0.
Specifications

Version: 16.0.0, 16.0.1
Platform: .NET Core SDK (3.1.202)

I believe this is the same as: https://github.com/ThreeMammals/Ocelot/issues/1252 it started in 15.0.7 for me.

@amweiss, it does seem related, but this is working for me in 15.0.7, so maybe not the same issue.

Hello,
The breaking change in 16.0.0 was changing names ReRoutes to Routes (it should be changed in ocelot.json), but it shouldn't cause such problems I think.
I've tried to reproduce it basing on the code you showed but the claims exist in my case. Maybe you have (or can prepare) a full simple example of application with this problem?
Do you

@jlukawska, thank you for taking the time to look into this.  When I came back to this and attempted to reproduce the error, it was working this time.  I'm not sure what was going on before, but I'm all set now.  Sorry to waste your time.


Anki backend userstory10

all backend of US 10 ACCEPT freakin 409 test for POST mapping --> in progress

Looks good. Resolve the conflicts and its ready for merge :)


#17654 #17064 SQL Terminal

Refactoring
Preference to add query text
Preference to add server output

Looks like Terminal opens more than once

Output data is shown after the sql query result even if it was executed before query

Focus is set on the statistics tab instead of resultset (probably, another issue is needed)

That's how the server output is fetched. We discussed and don't want to change it for now.
I guess it's this issue #17763

verified


OutputTooBigUTxO / MaxTxSizeUTxO Warning

Create a user warning prior to OutputTooBigUTxO / MaxTxSizeUTxO when minting (long asset name may cause issue)? so the receiving wallet does not lock up if the size is more than 16kb (or increase the max size).
Or when attempting to send have a warning script in Yoroi or Daedalus be called before attempting the invalid transaction letting the sender know what the problem is and a possible way to resolve it.
Similar to the "you can't mint ada" code
The warning is preferable so the issue does not occur in the first place?

Asset names are restricted to at most 32 bytes.
I don't understand how a receiving wallet could be effected by an invalid transaction, it would never receive anything.

https://forum.cardano.org/t/possible-attack-method-what-am-i-missing/75412
See the link above as this individual had a similar issue.
I think it is more about "after" receiving the problem asset the wallet can't send the native assets. In effect, it locks the wallet.

Also on Page 22 here
https://hydra.iohk.io/build/5949624/download/1/shelley-ma.pdf

Here is another example of an issue where tokens were minted but now cannot be sent/
https://forum.cardano.org/t/i-minted-some-tokens-but-cant-send-outputtoobigutxo-maxtxsizeutxo/75336

Thank you for the context @nnamdert , that helps a lot. This sounds like a problem with the wallet, not the ledger or the protocol. The OutputTooBigUTxO check was put in place to prevent the creation of an output that was nearly the size of the max transaction size. The wallet may be attempting to combine outputs that are near the limit.


Learning Pathways Configuration Issues [BUG]

PROVISIONING NOTE: If you are having issues with the provisioning service while trying to provision M365 learning pathways please do not post that issue to this repo as we cannot help you here. Please post all questions and issues for the provisioning service to https://github.com/SharePoint/sp-dev-docs/issues.

Describe the bug
I have Learning Pathways Webparts on 2 of my Office 365 Hub SharePoint Online sites. The icons next to the options are no longer there. The options are working and clickable but the icons aren't there. I have tried different browsers and machines. When I click on the settings gear in that Webpart it takes me to the Learning Pathways admin page where I get an error message at the top stating that the Microsoft 365 Learning Pathways has a configuration issue.
To Reproduce
Steps to reproduce the behavior:

Go to O365 Hub and click on Edit
Click on the settings icon in the Learning Pathways Web part
Page tries to load
See error - Microsoft 365 Learning Pathways has a configuration issue.

Expected behavior
I am supposed to see options for editing Learning Pathways. Icons are supposed to be visible in the options in the Web Parts on the SharePoint page
Screenshots
If applicable, add screenshots to help explain your problem.
Learning Pathways version number:
<IP_ADDRESS>

Additional context
Add any other context about the problem here.

Closing issue due to inactivity. If you're still having an issue or a similar issue please start a new issue.


⌨ Document Rails backtrace cleaner integration

Used here: https://github.com/appsignal/appsignal-ruby/blob/d9227616d624ec13fedea3b4c0711cfc667a2211/lib/appsignal/transaction.rb#L475
We can expand the Rails integration page (https://docs.appsignal.com/ruby/integrations/rails.html) to mention how to use the backtrace cleaner with AppSignal.

test (will remove this comment later)