Standard Image Features

Signed-off-by: Mathias Fechner<EMAIL_ADDRESS>

@garloff can you comment?

thanks for the feedback @garloff.
@matfechner can you look at the feedback next feedback? Thanks!

Discussion in Team IaaS on 2022-07-13
The question is: wether we're not in the position (or wether it is not our job) to provide a biased list for SCS environments?
List:
Ubuntu
Debian
Fedora
Rocky/Alma (centos stream)
openSUSE Leap
grml (rescue purposes)
where applicable LTS and regular. nothing with rolling releases.
@matfechner can you update the PR with this please?
Discussion on wether cluster api image should be listed here. The definition of the cluster image should come from the container layer, however every cluster api image needs to be part of a SCS environment.

Am pulling this into the spring, since I want to have this finalized before R3. Let's finish this up in tomorrows edition of the SIG Standardization.

Discussion on wether cluster api image should be listed here. The definition of the cluster image should come from the container layer, however every cluster api image needs to be part of a SCS environment.

Our reference implementation for KaaS with Cluster-API does know where to get the node images from and registers them on the fly as needed. The images come from https://minio.services.osism.tech/openstack-k8s-capi-images/ and work for OpenStack CAPI environments (certainly for SCS IaaS). That said, it is better if the provider already has those images registered, saving 8mins or so image registration time and being more efficient for the disk and image storage as well.
I would expect SCS certified KaaS platforms to have node images for all popular k8s versions available.

A view on standard images:

Some providers prefer "optimized" images that have some cloud-specific pre-configuration; in the worst case, provider-specific images may even be required b/c special drivers are needed. (Full disclosure: OTC originally required special images, and continued build and providing omtimized images later, when the requirement was lifted.)
Some customers appreciate the specific images; many don't, as they want something that is well-known and works across clouds with identical properties.

Considerations:

Require unchanged upstream images to be available under specific names. (Downside: If these don't work due to specifics in the infrastructure, the provider can not offer them or get granted an exception.)
The vendor_data mechanism can be used to inject configuration information, software, ... into freshly started VMs by default. (E.g. using local mirrors for package updates to provide a very valid example.)
Additional, "optimized" images could be added under different names -- if we standardize the names, these would not have a cloud-specific name ... The source and build process would need to be made transparent if we somehow standardize this, though. Both post-processed upstream images and built-from-the-ground-up images could be done, though we might want to differentiate between those by namespace?

As discussed (among others with @frosty-geek and @matfechner) this PR will be closed. The document has been updated by @garloff and this PR is now obsolete.


Implemented more complete testing suite

Resolves #76
Run the testing suite with: py.test --cov AxonDeepSeg/ --cov-report term-missing
Major changes:

Added a more complete testing suite using pytest (82% code coverage)

Tests assertions vary in their specificity of functionality vs. only looking at if the code ran/output exists or is formatted as expected, depending on time limitations for certain functions or if it was even feasible to quantify specific assertions of the function outputs (e.g. core data augmentation transformation like rotations were written to be random, so I couldn't verify if the images were rotated by a specific angles).
A lot more specific test cases for different functionality cases could likely be written, but I think this is a good first step beyond the old integrity test.

Added files required for testing (~4mb), located in test/__test_files__

Otherwise, we could delete them here, include them in osf.io, and pull them prior to TRAVIS tests.

Coverage analysis added (coveralls.io), added badge in readme.

Minor changes:

Minor bug fixes discovered while writing tests

e.g. config_tools.py, segmentation_scoring.py

Minor documentation clarification, or consistent filenaming

e.g. data_augmentation.py

Minor code refactoring (e.g. removing function argument that went unused in all possible cases)

e.g. segment.py

Some added exception handling

e.g. patch_extraction.py, compute_morphometrics.py

Notes:

This PR should probably be squashed & merged instead of a simple merge.
Tests are organized in the folder test/, one test_FILENAME.py file for each core ADS file, and matching the same directory structure as AxonDeepSeg/.
Tests can be evaluated on local computer using py.test --cov AxonDeepSeg/ --cov-report term-missing
Tests were separated in two test categories using flags unit and integration. Append -m FLAG where flag is a valid one to the py.test call to run just those tests.
Command-line interface calls (e.g. "main") were not tested – these should be written/tested later when improved exception handling is worked on (issue #29).
Files mapping_results.py, trainingforhelios.py, and the majority of visualize.py were not tested, as the code looks either stale, incomplete, or severely outdated and non-functional in its current state. We should maybe looking into deleting or fixing these if they are still useful. (e.g. in mapping_results.py, one if statement looks for if DS_STORE files are in the folder before running the core functionality which seems completely unrelated to that file, so is likely an incomplete function that was in development), see below

https://github.com/neuropoly/axondeepseg/blob/4fb50073a22d6358451567a4004236796d84058c/AxonDeepSeg/mapping_results.py#L11-L31

One more thing, currently the badge points to this branch. After the merge and the first coverage report for master is pushed to coveralls.io, we'll have to update the badge branch to point to master.

Great – thanks @jcohenadad !
I'll wait on a review by @MelanieLu and/or @oumayb before merging, as I think they had an interest on doing so (I met with them yesterday about it).

@mathieuboudreau I ran the testing but it seems like only the init.py files are 100% covered, all the other files are at 0%. Is it normal?

@oumayb Hmm no, it shouldn't come out that way. Would you mind saying which branch you're on, (git branch, it will be the one with the * next to it), which commit number you're at (git log --pretty=format:'%h' -n 1), and the directory that you tried that command/test from (pwd)?

@mathieuboudreau Ah I see I was on the master branch, commit #a7e70d3 and axondeepseg directory. Should I switch to unit_tests?

@oumayb yup! Just switch to it using git checkout unit_tests, and do a git pull to make sure it's up to date. And you're in the correct directory to run that test!

@mathieuboudreau thanks, I was able to run the test correctly. I still have one error message though (ImportError: No module named prettytable) I'm working on.

@oumayb Ahh I forgot to mention that I added new requirements, so that ADS needs to be reinstalled (which is why you're getting the missing module message). I also just merged the Sentry branch to master, and updated this branch to be in sync with master, so more requirements are needed to be installed.
I'd recommend trying the following steps:

git pull to update the branch again.
pip install -e . to (re)install the ADS requirement.
axondeepseg_test to run ADS once, which will ask your for your consent to track bugs (say yes if you'd like).
py.test --cov AxonDeepSeg/ --cov-report term-missing

Sorry that this has been a bit convoluted! Let me know if this works. If so please approve again, as your latest approved review has been auto-dismissed because I just pushed a bug fix.

@mathieuboudreau no worries. I just followed your steps and it's working perfectly! I also got to see the Sentry message for the first time :)
I'm approving again.


test(config): consider all files for test coverage

PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

Codecov Report

Merging #512 (f74ae79) into main (c29cee4) will decrease coverage by 30.21%.
The diff coverage is n/a.

:exclamation: Current head f74ae79 differs from pull request most recent head c69f051. Consider uploading reports for the commit c69f051 to get more accurate results

@@             Coverage Diff             @@
##             main     #512       +/-   ##
===========================================
- Coverage   81.55%   51.33%   -30.22%
===========================================
  Files          35       65       +30
  Lines        1903     3023     +1120
  Branches      307      528      +221
===========================================
  Hits         1552     1552
- Misses        351     1446     +1095
- Partials        0       25       +25

Impacted Files
Coverage Δ

src/routes/devices/disconnect.ts
0.00% <0.00%> (ø)

src/routes/amt/auditLogValidator.ts
0.00% <0.00%> (ø)

src/index.ts
0.00% <0.00%> (ø)

src/routes/amt/amtFeatureValidator.ts
0.00% <0.00%> (ø)

src/routes/health/index.ts
0.00% <0.00%> (ø)

src/routes/devices/index.ts
0.00% <0.00%> (ø)

src/routes/devices/create.ts
0.00% <0.00%> (ø)

src/routes/amt/index.ts
0.00% <0.00%> (ø)

src/routes/auth/login.ts
0.00% <0.00%> (ø)

src/test/helper/ciraclient.js
0.00% <0.00%> (ø)

... and 20 more

Continue to review full report at Codecov.

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


Experiment: auto-import from yet-unimported referenced projects

Still to-do:

[x] Deduplicate files from multiple programs

[ ] How does this change based on useSourceOfProjectReferenceRedirect? Are there cases where the main program will directly contain .d.ts outputs but the referenced project will show us equivalent .ts inputs?

[ ] Audit which program should be used to create the module specifier resolution host—it probably should be the main program (look at paths)
[ ] Test cache invalidation (I think it should be working without changes based on conversation with @amcasey. If it doesn’t, this PR is likely not worth it.)
[ ] Consider limiting how many total files/projects we put into the ExportInfoMap
[ ] Test paths module specifier resolution
[ ] Test a lot more stuff

Fixes #39778

@typescript-bot pack this

@typescript-bot pack this

So it turns out that in TS Server, referenced project changes do not trigger any kind of notification / update / reload of the referencing project, so cache invalidation here doesn’t automatically work. Probably the best thing for this feature will be not to add any referenced-project exports into the ExportMapCache. Seeing as there’s been zero feedback since I’ve started working on this, I’m going to table it for now.

I come from https://github.com/microsoft/TypeScript/issues/39778 and I tried this patch but I was not lucky.
None suggestions from referenced projects unless I do the first import.


Do not log memcap warnings

The issue is fixed in Suricata 8.0.x, but 7.0.x won't get a backport, so lets just start ignoring these two specifically.
Closes #15

excellent! thank you


Do not have identifiers with quotes in the name (AA0100) -> Wrong for PageExtensions

Describe the bug
When I modify a field with quotes inside the name I receiv a warnig for this field name.
This Warning doesn't make sense, since I can't change its name.
To Reproduce
Steps and to reproduce the behavior:

Create a new Page Extension for the "Sales Order Subform"
Add  a modify to the layout for the field "TotalSalesLine.""Line Amount"""

layout
{
    modify("TotalSalesLine.""Line Amount""")
    {
        Visible = false;
    }
}

Expected behavior
Don't show this warning since I cannot change this name.
Screenshots
If applicable, add screenshots to help explain your problem.
5. Versions:

AL Language: 5.0.288712

Business Central:

Hi @pri-kise, thank you for reporting the issue. I found that the issue is not reproducible on the master (You can try to get a repro on the latest daily build). Apparently it has been already fixed there and will be delivered in the future release. So I am closing the issue now. Thanks

@AndreyKorepanov is there an ETA when this will be delivered to the VS Code Store?

Hi! The extension will be available in the VS Code Store starting with the 1st of October. Can you confirm that the issue is solved by using the developer preview?


Open Test Browser TypeError: 'NoneType' object is not callable when using useralias param

Describe the bug
Hi,
Trying to open a test browser with a different user of a persistent org gives me this error TypeError: 'NoneType' object is not callable. If I use a scratch org the keyword works perfectly and it allows me to open 2 browsers with different accounts. To authenticate i use  'sfdx auth:web:login' for both the persistents accounts, while for the scratch org ones  i generated a password and then created a new profile with sfdx following this youtube video.
Here the robot file i execute
`*** Settings ***
Resource        cumulusci/robotframework/Salesforce.robot
Library         cumulusci.robotframework.PageObjects
Test Teardown    Delete Records and Close Browser
*** Test Cases ***
Utente admin
Open Test Browser       useralias=Chatter
Utente UTest
Open Test Browser            useralias=UTest`
Here the log with the same error in each case

Reproduction steps
1.connect to a persistent org with sfdx auth:web:login with the first account
2.connect to a persistent org with sfdx auth:web:login with the second account
3.execute the file robot with cci task run robot  --suites pathToFile.robot
4.get the TypeError: 'NoneType' object is not callable
Your CumulusCI and Python versions
CumulusCI version: 3.51.0 (C:\Users${user}.local\bin\cci)
Python version: 3.9.8 (C:\Users${users}.local\pipx\venvs\cumulusci\Scripts\python.exe)
Operating System
Microsoft Windows 10.0.19042
Windows environment
Command Prompt
CumulusCI installation method
pipx
Error Gist
No response
Additional information
No response

The useralias for the Open Test Browser keyword refers to an org in the CumulusCI keychain, not the sfdx keychain. In other words the alias needs to show up in cci org list in order for it to work. For a persistent org, that means you would need to connect the org using cci org connect rather than using sfdx. Unfortunately we don't currently provide a way to import a persistent org from sfdx into the cci keychain.

@ZephyrusMB I'm saying that it will not work unless the persistent orgs have been connected that way. But perhaps there is another problem as well. Could you run cci error gist after the NoneType error occurs and provide the link to the gist so that we can see more information about where the error is happening?

Here it is, I edited some paths or email for privacy reasons.
CumulusCI version: 3.51.1
Python version: 3.9.8
Environment Info: Windows / AMD64

Last Command Run
================================
cci task run robot --suites .\robot\demo\tests\PersistentOrg\PersistentOrgAdmin\openBrowser.robot --robot_debug true
[01/26/22 17:46:31] Org info updated, writing to keychain
                    Beginning task: Robot
                    As user<EMAIL_ADDRESS>
                    In org: IDORG

==============================================================================
openBrowser
==============================================================================
Utente Chatter                                                        F.

Utente Chatter                                                        | FAIL |
TypeError: 'NoneType' object is not callable
------------------------------------------------------------------------------
Utente Standard                                                       F.

Utente Standard                                                       | FAIL |
TypeError: 'NoneType' object is not callable
------------------------------------------------------------------------------
openBrowser                                                           | FAIL |
2 tests, 0 passed, 2 failed
==============================================================================
Output:  C:\Users\Desktop\RobotFramework\Cumulusci\PyconDemo\robot\demo\results\output.xml
Log:     C:\Users\Desktop\RobotFramework\Cumulusci\PyconDemo\robot\demo\results\log.html
Report:  C:\Users\Desktop\RobotFramework\Cumulusci\PyconDemo\robot\demo\results\report.html
Error: 2 tests failed.
Run this command for more information about debugging errors: cci error --help
Traceback (most recent call last):
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\cumulusci\cli\cci.py", line 79, in main
    cli(args[1:], standalone_mode=False, obj=runtime)
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 782, in main
    rv = self.invoke(ctx)
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\click\core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\cumulusci\cli\task.py", line 181, in run_task
    task()
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\cumulusci\core\tasks.py", line 160, in __call__
    self.result = self._run_task()
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\cumulusci\tasks\robotframework\robotframework.py", line 282, in _run_task
    raise RobotTestFailure(
cumulusci.core.exceptions.RobotTestFailure: 2 tests failed.

@ZephyrusMB Thanks. Unfortunately since the error happened in a Robot Framework test, the stack trace does not give enough information (it's showing where Robot Framework was called, not where the test failed while Robot Framework was running).
If you look at the Robot Framework log (the log.html file mentioned in the output), does it show any more information about where the error happened?

I believe you should be able to repeat this error following the steps:

log in to a persistent org with the first user with cci org connect
log in to the same persistent org with the second user with cci org connect
check the successful connection with cci org list
run the script in the first post with cci task run robot --suites path/to/file.robot

Repeating the test with Setting log level to DEBUG i receive this more info. Thank you
Traceback (most recent call last):
  File "C:\Users\.local\pipx\venvs\cumulusci\lib\site-packages\cumulusci\robotframework\CumulusCI.py", line 120, in login_url
    access_token = org.get_access_token(**userfields)

@ZephyrusMB Thanks. That traceback makes me realize that unfortunately this is currently implemented in a way that will only work for scratch orgs. Thanks for reporting it.

@davidmreed Can you comment on the likelihood of this being added to our roadmap?

Tracking as W-10502445


[v4] batch effects

I've noticed degraded performance when working with a large number of components on the screen during a color scheme change, especially noticeable on dev builds.
It seems to occur when each component gets re-rendered after effect.rerun() call.
Batching setState statements that happen after rerun resolves that issue.

Before (Release)
After (Release)

Not sure if it is a good long-term solution.

unstable_batchedUpdates is deprecated as in React 18 all setState calls within the same window are batched. NativeWind runs these updates in sync, so they should be all within the same window.
I'm hoping you closed this issue as you found the source was a different problem? If not I'm happy to look into this further.

@marklawlor I have decided to file an issue, because this change introduced undefined behavior.


only CBOR-serialize the passphrase when checking its hash

Issue Number

#1483
Overview

The actual encryption passphrase that is used is the hash provided to the API
by clients, it can be either null or a 32-byte string. Then, this passphrase
is serialized to CBOR before being hashed, but, that is the non-serialized
passphrase that is used for encrypted the root key!
Comments

so we need to revert this commit https://github.com/input-output-hk/cardano-wallet/pull/1511/commits/702cc229faaa33ba1117be690632d9db7400195b right?

bors r+

bors r+


Underlying error: The operation couldn’t be completed. The server responded with an error:

Bug report
Describe the bug
A clear and concise description of what the bug is.
Steps to reproduce
Steps to reproduce the behavior:

Check everything is fine with the flutter doctor -v command

$ flutter doctor -v
[✓] Flutter (Channel stable, 3.0.4, on macOS 12.4 21F79 darwin-x64, locale en-DE)
    • Flutter version 3.0.4 at /usr/local/Caskroom/flutter/1.22.6/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 85684f9300 (5 days ago), 2022-06-30 13:22:47 -0700
    • Engine revision 6ba2af10bb
    • Dart version 2.17.5
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc2)
    • Android SDK at /Users/gabriel/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0-rc2
    • ANDROID_HOME = /Users/gabriel/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

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

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

[✓] Android Studio (version 2021.2)
    • 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 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2021.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 212.5632

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

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554                            • android-x64    • Android 12 (API
      31) (emulator)
    • iPhone de GABRIEL (mobile)   • 078f58cccda2cf44b3a505bc1c89250983c45612 • ios            • iOS 15.5 19F77
    • macOS (desktop)              • macos                                    • darwin-x64     • macOS 12.4 21F79
      darwin-x64
    • Chrome (web)                 • chrome                                   • web-javascript • Google Chrome
      103.0.5060.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Run the app on the iPhone device connected over USB.3.

$ flutter run -v -d 078f58cccda2cf44b3a505bc1c89250983c45612
.
.
.
[ +139 ms] "status: ready to play"
[ +573 ms] Launching a Dart Developer Service (DDS) instance at http://<IP_ADDRESS>:0, connecting to VM service at
http://<IP_ADDRESS>:64454/.
[ +295 ms] flutter: Instance of 'InterstitialAd' loaded
[ +681 ms] flutter: Instance of 'RewardedInterstitialAd' loaded.
[  +68 ms] <Google> <Google:HTML> A valid SKAdNetworkIdentifier for Google (cstr6suwn9.skadnetwork) is missing
from Info.plist. See [Enable SKAdNetwork to track conversions]
(https://googlemobileadssdk.page.link/enable-skadnetwork)
[  +48 ms] DDS is listening at http://<IP_ADDRESS>:64470/VWC3h3vChGk=/.
[   +6 ms] 9.2.0 - [FirebaseFirestore][I-FST000001] AppCheck failed: 'The operation couldn’t be completed. Too
many attempts. Underlying error: The operation couldn’t be completed. The server responded with an error:
[        ]  - URL:
https://firebaseappcheck.googleapis.com/v1/projects/carpediem-d3916/apps/1:1051055649031:ios:ab86024b52bce8c3f2d5a
2:exchangeDeviceCheckToken
[        ]  - HTTP status code: 400
[        ]  - Response body: {
[        ]   "error": {
[        ]     "code": 400,
[        ]     "message": "App not registered: 1:1051055649031:ios:ab86024b52bce8c3f2d5a2.",
[        ]     "status": "FAILED_PRECONDITION"
[        ]   }
[        ] }
[        ] '
[        ] 9.2.0 - [FirebaseFirestore][I-FST000001] AppCheck failed: 'The operation couldn’t be completed. Too
many attempts. Underlying error: The operation couldn’t be completed. The server responded with an error:
[        ]  - URL:
https://firebaseappcheck.googleapis.com/v1/projects/carpediem-d3916/apps/1:1051055649031:ios:ab86024b52bce8c3f2d5a
2:exchangeDeviceCheckToken
[        ]  - HTTP status code: 400
[        ]  - Response body: {
[        ]   "error": {
[        ]     "code": 400,
[        ]     "message": "App not registered: 1:1051055649031:ios:ab86024b52bce8c3f2d5a2.",
[        ]     "status": "FAILED_PRECONDITION"
[        ]   }
[        ] }
[        ] '
[  +67 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:64454/
[  +39 ms] DevFS: Creating new filesystem on the device (null)
[  +66 ms] DevFS: Created new filesystem on the device
(file:///private/var/mobile/Containers/Data/Application/0509AD9B-EDB0-48B2-B220-F30C7F6858D2/tmp/carpediemgXmBjg/c
arpediem/)
[   +2 ms] Updating assets
[ +282 ms] Manifest contained wildcard assets. Inserting missing file into build graph to force rerun. for more
information see #56466.
[   +6 ms] Syncing files to device iPhone de GABRIEL...

See error or incorrect behavior

Expected behavior
App runs on iPhone without a black screen.
Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.17.5
Flutter SDK 3.0.4
carpediem 1.0.1+81

dependencies:
- animate_do 2.1.0 [flutter]
- assets_audio_player 3.0.4+4 [flutter rxdart uuid http path_provider assets_audio_player_web]
- auto_size_text 3.0.0 [flutter]
- awesome_notifications 0.6.21 [flutter platform intl]
- cached_network_image 3.2.1 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- carousel_slider 4.1.1 [flutter]
- clipboard 0.1.3 [flutter]
- cloud_firestore 3.2.1 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- cupertino_icons 1.0.5
- day_night_time_picker 1.1.2 [flutter]
- devicelocale 0.5.3 [flutter flutter_web_plugins]
- email_validator 2.0.1
- firebase_app_check 0.0.6+16 [firebase_app_check_platform_interface firebase_app_check_web firebase_core firebase_core_platform_interface flutter]
- firebase_auth 3.4.1 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.19.1 [firebase_core_platform_interface firebase_core_web flutter meta]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]
- flutter_launcher_icons 0.9.2 [args image path yaml]
- flutter_local_notifications 9.6.1 [clock flutter flutter_local_notifications_linux flutter_local_notifications_platform_interface timezone]
- flutter_native_timezone 2.0.0 [flutter_web_plugins flutter js]
- flutter_onboard 2.0.0 [flutter freezed_annotation flutter_hooks hooks_riverpod]
- flutter_spinkit 5.1.0 [flutter]
- flutter_svg 1.1.1+1 [flutter meta path_drawing vector_math xml]
- font_awesome_flutter 10.1.0 [flutter]
- form_field_validator 1.1.0 [flutter intl]
- google_fonts 3.0.1 [flutter http path_provider crypto]
- google_mobile_ads 1.3.0 [meta flutter]
- google_sign_in 5.3.3 [flutter google_sign_in_android google_sign_in_ios google_sign_in_platform_interface google_sign_in_web]
- image_gallery_saver 1.7.1 [flutter]
- intl 0.17.0 [clock path]
- lottie 1.3.0 [archive flutter path vector_math]
- modal_bottom_sheet 2.1.0 [flutter]
- package_info_plus 1.4.2 [flutter package_info_plus_platform_interface package_info_plus_linux package_info_plus_macos package_info_plus_windows package_info_plus_web]
- path_provider 2.0.11 [flutter path_provider_android path_provider_ios path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- permission_handler 10.0.0 [flutter meta permission_handler_android permission_handler_apple permission_handler_windows permission_handler_platform_interface]
- provider 6.0.3 [collection flutter nested]
- rxdart 0.27.4
- screenshot 1.2.3 [flutter]
- share_plus 4.0.10 [meta mime flutter share_plus_platform_interface share_plus_linux share_plus_macos share_plus_windows share_plus_web]
- shared_preferences 2.0.15 [flutter shared_preferences_android shared_preferences_ios shared_preferences_linux shared_preferences_macos shared_preferences_platform_interface shared_preferences_web shared_preferences_windows]
- shimmer_animation 2.1.0+1 [flutter]
- timezone 0.8.0 [path]
- url_launcher 6.1.4 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]
- yaml 3.1.1 [collection source_span string_scanner]

dev dependencies:
- flutter_lints 2.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph]

transitive dependencies:
- archive 3.3.0 [crypto path]
- args 2.3.1
- assets_audio_player_web 3.0.4+3 [flutter flutter_web_plugins]
- async 2.8.2 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.1 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.2.0
- charcode 1.3.1
- clock 1.1.0
- cloud_firestore_platform_interface 5.5.10 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.6.19 [cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- collection 1.16.0
- crypto 3.0.2 [typed_data]
- dbus 0.7.4 [args ffi meta xml]
- fake_async 1.3.0 [clock collection]
- ffi 1.2.1
- file 6.1.2 [meta path]
- firebase_app_check_platform_interface 0.0.4+10 [firebase_core flutter meta plugin_platform_interface]
- firebase_app_check_web 0.0.5+16 [firebase_app_check_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_auth_platform_interface 6.3.1 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 3.3.19 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.4.3 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 1.6.6 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- flutter_blurhash 0.7.0 [flutter]
- flutter_cache_manager 3.3.0 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_hooks 0.18.5 [flutter]
- flutter_local_notifications_linux 0.5.0+1 [flutter flutter_local_notifications_platform_interface dbus path xdg_directories]
- flutter_local_notifications_platform_interface 5.0.0 [flutter plugin_platform_interface]
- flutter_riverpod 1.0.4 [collection flutter meta riverpod state_notifier]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- freezed_annotation 0.15.0 [collection json_annotation meta]
- google_sign_in_android 5.2.8 [flutter google_sign_in_platform_interface]
- google_sign_in_ios 5.3.1 [flutter google_sign_in_platform_interface]
- google_sign_in_platform_interface 2.1.3 [flutter quiver]
- google_sign_in_web 0.10.1+3 [flutter flutter_web_plugins google_sign_in_platform_interface js]
- hooks_riverpod 1.0.4 [collection flutter flutter_hooks flutter_riverpod riverpod state_notifier]
- http 0.13.4 [async http_parser meta path]
- http_parser 4.0.1 [collection source_span string_scanner typed_data]
- image 3.2.0 [archive meta xml]
- js 0.6.4
- json_annotation 4.5.0 [meta]
- lints 2.0.0
- matcher 0.12.11 [stack_trace]
- material_color_utilities 0.1.4
- meta 1.7.0
- mime 1.0.2
- nested 1.0.0 [flutter]
- octo_image 1.0.2 [flutter flutter_blurhash]
- package_info_plus_linux 1.0.5 [package_info_plus_platform_interface flutter path]
- package_info_plus_macos 1.3.0 [flutter]
- package_info_plus_platform_interface 1.0.2 [flutter meta plugin_platform_interface]
- package_info_plus_web 1.0.5 [flutter flutter_web_plugins http meta package_info_plus_platform_interface]
- package_info_plus_windows 1.0.5 [package_info_plus_platform_interface ffi flutter win32]
- path 1.8.1
- path_drawing 1.0.0 [vector_math meta path_parsing flutter]
- path_parsing 1.0.0 [vector_math meta]
- path_provider_android 2.0.15 [flutter path_provider_platform_interface]
- path_provider_ios 2.0.10 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.7 [ffi flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.6 [flutter path_provider_platform_interface]
- path_provider_platform_interface 2.0.4 [flutter platform plugin_platform_interface]
- path_provider_windows 2.0.7 [ffi flutter path path_provider_platform_interface win32]
- pedantic 1.11.1
- permission_handler_android 10.0.0 [flutter permission_handler_platform_interface]
- permission_handler_apple 9.0.4 [flutter permission_handler_platform_interface]
- permission_handler_platform_interface 3.7.0 [flutter meta plugin_platform_interface]
- permission_handler_windows 0.1.0 [flutter permission_handler_platform_interface]
- petitparser 5.0.0 [meta]
- platform 3.1.0
- plugin_platform_interface 2.1.2 [meta]
- process 4.2.4 [file path platform]
- quiver 3.1.0 [matcher]
- riverpod 1.0.3 [collection meta state_notifier]
- share_plus_linux 3.0.0 [share_plus_platform_interface file flutter meta url_launcher]
- share_plus_macos 3.0.1 [share_plus_platform_interface flutter]
- share_plus_platform_interface 3.0.3 [flutter meta mime plugin_platform_interface]
- share_plus_web 3.0.1 [share_plus_platform_interface url_launcher flutter flutter_web_plugins meta]
- share_plus_windows 3.0.1 [share_plus_platform_interface flutter meta url_launcher]
- shared_preferences_android 2.0.12 [flutter shared_preferences_platform_interface]
- shared_preferences_ios 2.1.1 [flutter shared_preferences_platform_interface]
- shared_preferences_linux 2.1.1 [file flutter path path_provider_linux path_provider_platform_interface shared_preferences_platform_interface]
- shared_preferences_macos 2.0.4 [flutter shared_preferences_platform_interface]
- shared_preferences_platform_interface 2.0.0 [flutter]
- shared_preferences_web 2.0.4 [flutter flutter_web_plugins shared_preferences_platform_interface]
- shared_preferences_windows 2.1.1 [file flutter path path_provider_platform_interface path_provider_windows shared_preferences_platform_interface]
- sky_engine 0.0.99
- source_span 1.8.2 [collection path term_glyph]
- sqflite 2.0.2+1 [flutter sqflite_common path]
- sqflite_common 2.2.1+1 [synchronized path meta]
- stack_trace 1.10.0 [path]
- state_notifier 0.7.2+1 [meta]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.0 [charcode source_span]
- synchronized 3.0.0+2
- term_glyph 1.2.0
- test_api 0.4.9 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.1 [collection]
- url_launcher_android 6.0.17 [flutter url_launcher_platform_interface]
- url_launcher_ios 6.0.17 [flutter url_launcher_platform_interface]
- url_launcher_linux 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_macos 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_platform_interface 2.1.0 [flutter plugin_platform_interface]
- url_launcher_web 2.0.12 [flutter flutter_web_plugins url_launcher_platform_interface]
- url_launcher_windows 3.0.1 [flutter url_launcher_platform_interface]
- uuid 3.0.6 [crypto]
- vector_math 2.1.2
- win32 2.6.1 [ffi]
- xdg_directories 0.2.0+1 [meta path process]
- xml 6.1.0 [collection meta petitparser]

Seems like that you have enabled AppCheck but wrongly configured on Firebase site. Do you have right p8 file and team id?

Seems like that you have enabled AppCheck but wrongly configured on Firebase site. Do you have right p8 file and team id?

I do not have AppCheck enable (not that I am aware of). On Android, the application is running pretty well.

Seems like that you have enabled AppCheck but wrongly configured on Firebase site. Do you have right p8 file and team id?

I do not have AppCheck enable (not that I am aware of). On Android, the application is running pretty well.

Can you check your Firebase Console and App Check there if there is some enforcement?

@gmalmeida
Looking at the error log you shared:

  +68 ms] <Google> <Google:HTML> A valid SKAdNetworkIdentifier for Google (cstr6suwn9.skadnetwork) is missing
from Info.plist. See [Enable SKAdNetwork to track conversions]

This seems to be related to ads mob library you might be using and need to check if you are using correct entry in info.plist file for it.

AppCheck failed: 'The operation couldn’t be completed. Too
many attempts. Underlying error: The operation couldn’t be completed. The server responded with an error:

https://firebaseappcheck.googleapis.com/v1/projects/carpediem-d3916/apps/1:1051055649031:ios:ab86024b52bce8c3f2d5a
2:exchangeDeviceCheckToken
[        ]  - HTTP status code: 400
[        ]  - Response body: {
[        ]   "error": {
[        ]     "code": 400,
[        ]     "message": "App not registered: 1:1051055649031:ios:ab86024b52bce8c3f2d5a2.",
[        ]     "status": "FAILED_PRECONDITION"
[        ]   }
[        ] }

For this, check https://stackoverflow.com/a/70966447 and see if it helps.

Seems like that you have enabled AppCheck but wrongly configured on Firebase site. Do you have right p8 file and team id?

I do not have AppCheck enable (not that I am aware of). On Android, the application is running pretty well.

Can you check your Firebase Console and App Check there if there is some enforcement?

Thanks!
I have enabled it and was able to get rid of the error.
The issue now is:
On Android) application runs out of the box (both on simulator and on device)
On IOS) application shows a black screen (no ERROR reported / both on simulator and on device).

@gmalmeida Looking at the error log you shared:

  +68 ms] <Google> <Google:HTML> A valid SKAdNetworkIdentifier for Google (cstr6suwn9.skadnetwork) is missing
from Info.plist. See [Enable SKAdNetwork to track conversions]

This seems to be related to ads mob library you might be using and need to check if you are using correct entry in info.plist file for it.

AppCheck failed: 'The operation couldn’t be completed. Too
many attempts. Underlying error: The operation couldn’t be completed. The server responded with an error:

https://firebaseappcheck.googleapis.com/v1/projects/carpediem-d3916/apps/1:1051055649031:ios:ab86024b52bce8c3f2d5a
2:exchangeDeviceCheckToken
[        ]  - HTTP status code: 400
[        ]  - Response body: {
[        ]   "error": {
[        ]     "code": 400,
[        ]     "message": "App not registered: 1:1051055649031:ios:ab86024b52bce8c3f2d5a2.",
[        ]     "status": "FAILED_PRECONDITION"
[        ]   }
[        ] }

For this, check https://stackoverflow.com/a/70966447 and see if it helps.

Yes, I had other .plist files for debugging and release, which were not updated.
Now the error got resolved, and the black screen issue remains :)
Thanks!


How to run with low vram/ram?

I want to run this model locally so I can caption a lot of images, my problem however is that I get the following error message every time:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 20.00 MiB. GPU 0 has a total capacty of 8.00 GiB of which 0 bytes is free. Of the allocated memory 14.51 GiB is allocated by PyTorch, and 117.23 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
I've tried to do set 'PYTORCH_CUDA_ALLOC_CONF=max_split_size_mb:10' and some other lower values and nothing worked.
I have 8gb VRAM and 16gb RAM

You can try setting "device_map" to "auto" and possibly using HuggingFace accelerate to offload the model to RAM.  Quantization could also help fit the entire model in VRAM, but is currently isn't working (see issue #3 ).  Hopefully I'll have that fixed soon.

Alpha Two has errors in bitsandbytes, but a quantized repo in GPTQ format was released today. I was wondering if this could be used to load with less VRAM. Also, I think GPTQ is compatible with CPU. It is an easy to use format, although the disadvantage is that it does not allow on-the-fly quantization.
https://huggingface.co/OPEA/llama-joycaption-alpha-two-hf-llava-int4-sym-inc


Handled crash in case of failure in store initialisation

Description:  Handled crash caused due to failure in store initialisation, now FIC handling the crash gracefully.
Changes Proposed in PR: Handled crash caused due to failure in store initialisation
Fixes: #Github issue id

/azp run


Embedded build broke

The debian embedded build broke, looks like a bcc function signature changed
 /home/runner/work/bpftrace/bpftrace/src/attached_probe.cpp: In member function 'void bpftrace::AttachedProbe::attach_uprobe(bool)':
/home/runner/work/bpftrace/bpftrace/src/attached_probe.cpp:706:51: error: too few arguments to function 'int bpf_attach_uprobe(int, bpf_probe_attach_type, const char*, const char*, uint64_t, pid_t, uint32_t)'
                                         probe_.pid);
                                                   ^
In file included from /home/runner/work/bpftrace/bpftrace/src/attached_probe.h:10,
                 from /home/runner/work/bpftrace/bpftrace/src/attached_probe.cpp:14:
/usr/local/include/bcc/libbpf.h:87:5: note: declared here
 int bpf_attach_uprobe(int progfd, enum bpf_probe_attach_type attach_type,
     ^~~~~~~~~~~~~~~~~
/home/runner/work/bpftrace/bpftrace/src/attached_probe.cpp: In member function 'void bpftrace::AttachedProbe::attach_usdt(int)':
/home/runner/work/bpftrace/bpftrace/src/attached_probe.cpp:816:77: error: too few arguments to function 'int bpf_attach_uprobe(int, bpf_probe_attach_type, const char*, const char*, uint64_t, pid_t, uint32_t)'
       eventname().c_str(), probe_.path.c_str(), offset_, pid == 0 ? -1 : pid);
                                                                             ^
In file included from /home/runner/work/bpftrace/bpftrace/src/attached_probe.h:10,
                 from /home/runner/work/bpftrace/bpftrace/src/attached_probe.cpp:14:
/usr/local/include/bcc/libbpf.h:87:5: note: declared here
 int bpf_attach_uprobe(int progfd, enum bpf_probe_attach_type attach_type,
     ^~~~~~~~~~~~~~~~~

Originally posted by @fbs in https://github.com/iovisor/bpftrace/pull/1583#issuecomment-719586301

Maybe this change? https://github.com/iovisor/bcc/commit/50f2009c7521e7d245372839cdb6b9cd6924202b

Yeah, I changed the bcc API.
https://github.com/iovisor/bpftrace/pull/1567 adds code for signature detection and the proper ifdefs

Would adding https://github.com/iovisor/bpftrace/pull/1567/commits/25446cd47c21be561de6aa2bdb26fa2bcba8aae4 solve it? We can merge that first and have the rest of 1567 pending review

Would adding 25446cd solve it? We can merge that first and have the rest of 1567 pending review

Yeah, it would. I'll put that out into a separate PR

#1589 splits it out


Feature/parse local

Here's the draft version, please check if that's okay with you

How about something like fromLocalFormat or fromCallerFormat or similar? To point out better that the given isocode specifies the formatting, not the destination of the number.
Am 28. Juli 2022 00:08:30 MESZ schrieb cedvdb @.>:
@. commented on this pull request.

@@ -42,6 +42,46 @@ class PhoneParser {
return PhoneNumber(isoCode: isoCode, nsn: national);
}

/// parses a [phoneNumber] given an [isoCode] when it is unknown
whether the phone number is a local or an international one,
/// we just know it is specified in the country's locale
///
/// @.*** phoneNumber}
///
/// throws a PhoneNumberException if the isoCode is invalid
@internal
static PhoneNumber fromLocale(IsoCode isoCode, String phoneNumber) {

I don't think it should be merged but I could be wrong. currently it is
like this:
fromIsoCode(IsoCode.FR, ..) will try to parse a phone number with the
hint that it is supposed to be a phone number located in France.
Just a better terminology would fit better imo, the fromLocale name I
think is confusing too since the first parameter in other functions is
the location of the phone number. Here it's the location of the caller.
--
Reply to this email directly or view it on GitHub:
https://github.com/cedvdb/phone_numbers_parser/pull/27#discussion_r931620801
You are receiving this because you authored the thread.
Message ID:
@.***>

--
Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.

What about modifying fromRaw instead ?
fromRaw(phoneNumber,  { IsoCode callerCountry, IsoCode destinationCountry })

What about modifying fromRaw instead ?
fromRaw(phoneNumber,  { IsoCode? callerCountry, IsoCode? destinationCountry })

This gives some flexibility, and should adhere to the following rules

the international prefix removed needs to be the one from callerCountry if present
the countryCode (33 for france), needs to be the one from the destination country if present, or it's computed
the nationalPrefix, this is the national prefix from the destination country, which we figured above.

What do you think ?

Sounds reasonable, that gives the user the maximum flexibility when using fromRaw.
I adapted my PR accordingly, please check.

Thanks for this 👍, taking some time to not rush this was worth it I think.

Thanks for this 👍, taking some time to not rush this was worth it I think.

Yes definitely, the result is much better than my initial proposal in my opinion.

Could you please bump pubspec.yaml and publish the latest version to pub.dev?

yes will do today

@mrclauss I'm waiting for a PR in the metadata library to pass before publishing this one.


Update Composer dependencies (2022-06-01-00-07)

Visual regression test passed!
View the visual regression test report


/bin/sh not able to parse bash functions

Command failed: /bin/sh -c [ -f ~/.bash_profile ] && source ~/.bash_profile;[ -f ~/.bashrc ] && source ~/.bashrc;rbeautify "/var/folders/78/3jyqdny925n348fpr6wh23bc0000gn/T/input115211-94410-1yp2t0n" /Users/miketheman/.bash_it/lib/helpers.bash: line 76: `bash-it': not a valid identifier

This error occurs since I have some functions being loaded in my bash_profile that to not work with the default /bin/sh shell on OSX.
This is due to the default behavior of child_process and exec. See more here: https://nodejs.org/api/child_process.html#child_process_child_process_exec_command_options_callback
I tried tinkering with the code to allow accepting bash as the new shell, since you end up sourcing the functions declared in bash_profle anyhow, but I couldn't get it working.

Kind of the same problem here since I'm using zshell and have not configured the correct environment in Bash config.

This will be consequently fixed by #164
For now, if you are able to comment out https://github.com/Glavin001/atom-beautify/blob/master/lib/langs/cli-beautify.coffee#L49-L50 in your local installations.
Has anyone tried passing bash as the shell to https://github.com/Glavin001/atom-beautify/blob/master/lib/langs/cli-beautify.coffee#L57 ? I am unable to reproduce and also in the middle of exams, so it'll be a few weeks before I have time to look into this further.

I've been able to make this kind of thing work on master code:
...
                if typeof cmd is "string"

                  config =
                    env: process.env
                    shell: 'bash'

                  isWin = /^win/.test(process.platform)
...

This might work for zshell as well.
I know you're working on something better longer term, so I didn't know if it was worthwhile sending a PR for this change, or leaving it for now, since there's a workaround.
I guess it would most make sense to try and detect the user's $SHELL variable and react to that, however that brings into problems of detecting where the Atom instance was started from - it you launch via Spotlight, Alfred or Start Menu, you won't have access to the envvars during launch time.
See here for how another package 'solved' this problem.

This should not longer be applicable now that #282 is published. Published to v0.25.0 and above.


Improve Logging

The API currently has a limited level of logging.
This should be improved to allow for easier debugging.
This should be configurable and leverage Pino

I'll take this.

Tracking in branch https://github.com/project-vagabond/covid-green-backend-api/tree/cg-logging


How to reproduce your best result?

Hi, I was trying to reproduce your best result (mAP=88.7%) by simply runing
python tools/train.py --config_file='configs/softmax_ranked.yml' DATASETS.NAMES "('market1501')"

but I got a much worse result at mAP=86.5%, with 2% margin to claimed result. Moreover, it's even lower than using triplet loss.  What I can do to get your claimed result?

model
mAP
loss

r50-ibn-a(my result)
87.1%
softmax + triplet

r50-ibn-a(my result)
86.5%
softmax + ranked

r50-ibn-a(your result)
88.7%
softmax + ranked

Thank you!

Did you change training paramters ?
There are some exp result, you can refer (Resnet50_ibn_a).
Try more epoch ?
`

w
r
T
DukeMTMC-ReID
Market1501

mAP
rank1
mAP
rank1

0.2
0.7
1.0
80.2
90.1

0.4
0.7
1.0
80.8
90.9

0.6
0.7
1.0
80.6
90.2

w
r
T
DukeMTMC-ReID
Market1501

mAP
rank1
mAP
rank1

0.4
0.6
1.0
80.3
90.6

0.4
0.7
1.0
80.8
90.9

0.4
0.8
1.0
80.8
90.4

w
r
T
DukeMTMC-ReID
Market1501

mAP
rank1
mAP
rank1

0.4
0.7
0.5
80.4
90.6

0.4
0.7
1.0
80.8
90.9

0.4
0.7
5.0
80.3
90.6

`

I used your RankedListLoss for car196 dateset but i can't reproduce the results. the loss is increasing with every epoch. any thoughts?

Hello, what is the meaning of these hyperparameters？
Looking forward to your reply！thank you


RSS feed is backwards

The RSS feed has the right date, but shows the last items first, which is somewhat unexpected, to say the least. :)

The current implementation is correct. Check out the GitHub status feed.

wait, then there's a problem with my install. in both the demo and the github status feed, latest iterms are shown first. in our RSS feeds, most recent items are last.
sorry if my description wasn't clear...

That's the demo, it looks fine to me?

that's what i am saying: the demo looks fine, the order is correct. but i notice the <pubDate> field are all the same. if you add a new element, it will not show up on top of the feed, but at the bottom! here's the current RSS output:
<?xml version="1.0" encoding="UTF-8"?>
<rss xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:wfw="http://wellformedweb.org/CommentAPI/" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:atom="http://www.w3.org/2005/Atom" xmlns:sy="http://purl.org/rss/1.0/modules/syndication/" xmlns:slash="http://purl.org/rss/1.0/modules/slash/" version="2.0"><channel><title>Cachet Demo</title><description>Status Feed</description><link>https://demo.cachethq.io</link><item><title>Awesome</title><description>We totally nailed the fix.</description><status>Fixed</status><pubDate>Thu, 26 Mar 2015 11:30:44 +0000</pubDate></item><item><title>Monitoring the fix</title><description>We're checking that our fix will first work.</description><status>Watching</status><pubDate>Thu, 26 Mar 2015 11:30:44 +0000</pubDate></item><item><title>Update</title><description>We've found the problem, so we're looking at it.</description><status>Identified</status><pubDate>Thu, 26 Mar 2015 11:30:44 +0000</pubDate></item><item><title>test</title><description>test</description><status>Investigating</status><pubDate>Thu, 26 Mar 2015 12:00:30 +0000</pubDate></item></channel></rss>

my test is at the end.

In the demo they will be correct because everything is seeded at the same time.

right, which is why we don't see the bug there, but I am now pretty sure it's a bug. :)

This is fixed now.

83e979cc251673a79ef4ad7262ddb85e2cae0b6f

great, thanks!


Reducing recursive functions under blocked matches: zeta_full

This adds a new normalization step called zeta_full that ... from pervasives.fsti
(** Unroll recursive calls

    Note: Since F*'s termination check is semantic rather than
    syntactically structural, recursive calls in inconsistent contexts,
    or recursive evaluation of open terms can diverge.

    When asking for the [zeta] step, F* implements a heuristic to
    disable [zeta] when reducing terms beneath a blocked match. This
    helps prevent some trivial looping behavior. However, it also
    means that with [zeta] alone, your term may not reduce as much as
    you might want. See [zeta_full] for that.
  *)
val zeta : norm_step

(** Unroll recursive calls

    Unlike [zeta], [zeta_full] has no looping prevention
    heuristics. F* will try to unroll recursive functions as much as
    it can, potentially looping. Use with care.
 *)
val zeta_full : norm_step

Note, there are still a couple of orthogonal open issues related to this:

There's a normalizer bug that changes the configuration options prematurely, See nik_norm_cfg_bug

Even with just zeta (not zeta_full), the normalizer turns off delta steps when reducing under blocked matches. That should also be fixed. See progress in #1910 Once that's fixed, the name of this flag will be accurate: i.e., it only toggles the behavior of zeta under matches.

Looks pretty good. I would add a note in pervasives that ZetaFull (effectively) implies Zeta (I was thinking previously ZetaFull would be "in addition" to Zeta).


Cloudevent SDK for Angular

I want to use standard cloudevent format for sending events from frontend applications to the backend API.
I came to know about the following package which conform to the standard cloudevent format.
https://github.com/cloudevents/sdk-javascript
But I want to use it in angular application. Are there any cloudevent package exists for angular 12.

@shaheer-k there aren't any packages that i know of that wrap this sdk in an angular component(or whatever it is called in Angular land).
But this is just a regular javascript library that should be able to be integrated into any JS framework

@lholmquist
thank you for the response.
I installed cloudevent sdk in Angular 12.0.0 application.
npm install cloudevents
Installation completed successfully.
I tried the following code in my .ts file
import { CloudEvent, CloudEventV1, CloudEventV1Attributes } from "cloudevents";

const ce: CloudEventV1<string> = {
  specversion: "1.0",
  source: "/some/source",
  type: "example",
  id: "1234"
};
const event = new CloudEvent(ce);

The line const event = new CloudEvent(ce); is causing the following error
./node_modules/cloudevents/dist/transport/http/index.js:11:31-46 - Error: Module not found: Error: Can't resolve 'http' in '\node_modules\cloudevents\dist\transport\http'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "http": require.resolve("stream-http") }'
        - install 'stream-http'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "http": false }

./node_modules/cloudevents/dist/transport/http/index.js:12:32-48 - Error: Module not found: Error: Can't resolve 'https' in '\node_modules\cloudevents\dist\transport\http'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "https": require.resolve("https-browserify") }'
        - install 'https-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "https": false }

Could you please help me in solving the above error.

@shaheer-k I am going to close this, assuming you've had some success. If you have not been able to get this to work, please re-open this issue or create a new one. Thanks.


[BUG] TRL SFT - AutoGPTQ Quantization Issues

Hello AutoGPTQ friends,
I've been dealing with some difficult to debug quantization issues with AutoGPTQ Quantization using the TRL SFT script.
The vanilla base model quantizes perfectly fine with AutoGPTQ scripts.
However, when I run SFT using the TRL SFT script. I get a lingalg.cholesky error.
You will notice that on the AutoGPTQ Github, there are several related issues related to this error. Unfortunately, with the TRL trained model. Upping damp_percent and group_size or desc_act had no impact on quantization. With or without triton.
The following is the error I experience while running AutoGPTQ quantization. Hopefully you guys could give some feedback:
Traceback (most recent call last):
  File "/root/my-model/workflow/process/gptq.py", line 55, in <module>
    converter(model_path, output_path)
  File "/root/my-model/workflow/process/gptq.py", line 25, in converter
    model.quantize(examples)
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/auto_gptq/modeling/_base.py", line 368, in quantize
    scale, zero, g_idx = gptq[name].fasterquant(
  File "/usr/local/lib/python3.10/dist-packages/auto_gptq/quantization/gptq.py", line 110, in fasterquant
    H = torch.linalg.cholesky(H)
torch._C._LinAlgError: linalg.cholesky: The factorization could not be completed because the input is not positive-definite (the leading minor of order 1 is not positive-definite).

Okay, figured it out.
You need to update the SFT example script a bit.
Update to feed Tokenizer into SFT Training
tokenizer = AutoTokenizer.from_pretrained(script_args.model_name, use_fast=True)

# Step 5: Define the Trainer
trainer = SFTTrainer(
    model=model,
    args=training_args,
    max_seq_length=script_args.seq_length,
    train_dataset=dataset,
    dataset_text_field=script_args.dataset_text_field,
    peft_config=peft_config,
    tokenizer=tokenizer
)

Lower Gradient Accumulation Steps
Set it to 4 when running SFT.
Hope this helps future people who have issues.


Fix docs AsciinemaPlayer on page load

Right now, when you first visit the docs, the AsciinemaPlayer doesn't load. However, if you click somewhere or navigate to another page, it suddenly appears. Also, if it's already loaded and you refresh the page, it disappears once more.
Initial load:

After a click:

Any idea why it responds to a click event? It works locally with me but online it has this strange behavious. I also face a double load sometimes... 🤔

Any idea why it responds to a click event?

I haven't done extensive research on this yet, so I can't say why it behaves this way.

I will check this issue. We made some changes to the AsciinemaPlayer component, replacing useIsBrowser with BrowserOnly, and that may be causing this issue.

Thank you @Saurabh7019 🤩

I will check this issue. We made some changes to the AsciinemaPlayer component, replacing useIsBrowser with BrowserOnly, and that may be causing this issue.

Thank you for the time you want to put in here! Should I assign the issue to you?

Yes, please!


Mergify: configuration update

This change has been made by @jackkoenig from https://mergify.io config editor.

I tested this with the Mergify config editor to see if it would do the right thing for PRs like https://github.com/freechipsproject/www.chisel-lang.org/pull/121 and it does.
I got the author regex from https://docs.mergify.io/examples/#dependabot and the rest from FIRRTL's mergify config.


Signatures are ignored for workflow actions after nested workflow

Here is the MWE:
[to_be_nested]
parameter: input_file = str
parameter: output_file = str
input: input_file
output: output_file
bash: expand = True
  sleep 5
  echo done > {_output}

[default]
input: '1.txt'
output: f'{_input:n}.a', f'{_input:n}.b'

sos_run('to_be_nested', input_file = _input, output_file = _output[0])

bash: expand = True
  echo "I am being executed"
  sleep 5
  echo done > {_output[1]}

The problem is that when this workflow is executed twice in a row, supposedly all computation should be skipped the 2nd round, but it is not the case:
first run,
$ touch 1.txt
$ sos run test.sos
INFO: Running default:
INFO: Running to_be_nested:
INFO: to_be_nested is completed.
INFO: to_be_nested output:   1.a
I am being executed
INFO: default is completed.
INFO: default output:   1.a 1.b
INFO: Workflow default (ID=66acb2efba52ab38) is executed successfully with 2 completed steps.

second run, part of the workflow is still executed:
$ sos run test.sos
INFO: Running default:
INFO: Running to_be_nested:
INFO: to_be_nested (index=0) is ignored due to saved signature
INFO: to_be_nested output:   1.a
I am being executed
INFO: default is completed.
INFO: default output:   1.a 1.b
INFO: Workflow default (ID=66acb2efba52ab38) is executed successfully with 1 completed step and 1 ignored step.

This is also by design, because sos does not keep signatures for steps with subworkflows. This will give SoS a chance to visit the subworkflow and check if anything over there has been changed and needs rerun. An obvious example is that if the code for a subworkflow has been changed but the master step has not, the substep would not be executed.

Okay, then I guess the recommended alternative is to use two separate steps for it?

Yes, perhaps depends: sos_step would also work, but separate step is more straightforward for complex cases.

I was able to reimplement my pipeline and get it work. Ideally when truly nothing is changed the computation should be skipped regardless of implementation (nested workflow or not) but current design choice is not too limiting anyways. It's okay with me if we label it wontfix and close the ticket.

Yes, ideally sos should be able to track the content of the nested workflow, but in really this is very difficult as even name of workflows can be dynamic.. and nested workflows can call nested workflows.
We could solve the problem partially by recording the content of the entire file, but then we have sos_run(source='another_file').
I suppose I will keep the ticket open as reference for future discussions.


Card hangs if accessed before RPi is ready on power-up

This appears to be stuck on the Apple II side as a reboot of the II resolves it. The start up sequence in the menu firmware will need to be altered to handle the case where the RPi starts up with its read line low which confuses the II code into thinking the RPi is ready when it's not.

Fixed by commit 460440ea4299d5a2145dea092fd77cbab4f628d6


Generate list of contributors at build time

Currently Contributors.js hits the GitHub API to retrieve the list of contributors to this repository and then makes n (number of contributors) calls again to the GitHub API to retrieve additional information about contributors.
Consider building the list of contributors at build time since this list only ever changes when new commits are pushed to the main branch.
It will make the Contributors page load super quick, https://protege.dev/contributors and avoid errors when the GitHub API has been it too much.
It would also remove the need for #94 since there would no longer be a loading phase for the Contributors page.

resolved with https://github.com/drewclem/protege/pull/115


Add carte-blanche

What about adding Carte Blanche to this project?

There are many alternatives in this space, so I'm not sure I see the benefit of Slingshot integrating a particular one. And since Carte Blanche is about having an isolated environment, I don't see the benefit of integrating it with Slingshot. If one found it useful, they'd likely logically keep them separate.


This webpage is not available

This webpage is not available
ERR_CONNECTION_CLOSED
ReloadHide details
The webpage at https://wakatime.com/ might be temporarily down or it may have moved permanently to a new web address.

Can you attempt this again? I am not having an issue connecting to it now

Was doing database maintenance at that time, closing this issue.


fix: Queue delete not touched tags command if adapter is not connected

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DBAL Profiler doesn't work with PHPEngine Templating

I have setup my Silex application such that it uses the PHPEngine instead of Twig and I have just installed this library but it is failing.
Below is a message I get:
Fatal error: Uncaught exception 'InvalidArgumentException' with message 'Identifier "twig" is not defined.'
InvalidArgumentException: Identifier "twig" is not defined.
In my bootstrap file I have included the following service providers as shown below:
if(ENV != 'LIVE'){
$app->register(new Silex\Provider\WebProfilerServiceProvider());
$app->register(new Sorien\Provider\DoctrineProfilerServiceProvider());
}
Is the a tweak I can perform without touching this library to make it work with php?

it's just extension/module to https://github.com/silexphp/Silex-WebProfiler and i don't think you can run silex webprofiller without twig


Hard to follow because some outdated information

Hi,
unfortunately, this article is very hard and cumbersome to follow, for a couple of reasons:

Structuring isn't very good sometimes. In some numbered paragraphs, more than one step is explained, but in most, each step gets their own paragraph and number.
"then select Add dynamic content" is not really clear at first, because I was lookig for a button at first. I had a quite while until i realized there's a text link appearing. A picture to explain what that is meant would have been great.
". Copy and paste the GET request that has been parameterized using the @concat string function below into the dynamic content box." and the actual box that should be copied follows in the second box. Because the first box also has parameters, it takes time to realize that it is really meant the second box. The text structure with indication to the actual box that needs be corrected should be directly under the text that points this out.
"Select MSI as the Authentication type." The term "MSI" gets mentioned several times in this text, yet this radio button (now a dropdown-list) has changed. Also, "MSI" goes by a slightly different name and on top of that, not as abbreviation.
Most of the screenshots are outdated.
Why is eg. https://management.azure.com/ as inline-text and not in a code box from which you can copy? If something is meant to be copied, then be consistent and use code-boxes for all such things or then inline text.

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

ID: 239e657e-15d8-05a3-466d-4159b178307e
Version Independent ID: 866f6f93-0302-324b-668e-b41f67581d6d
Content: How to pause and resume dedicated SQL pools with Synapse Pipelines - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql/how-to-pause-resume-pipelines.md
Service: synapse-analytics
Sub-service: sql
GitHub Login: @kromerm
Microsoft Alias: makromer

@kaysauter
Thanks for your feedback! We will investigate and update as appropriate.

@kaysauter
I've delegated this to @kromerm, a content author, to review and share their valuable insights.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Genesis build

Why this should be merged
How this works
How this was tested

In general looks good, I think we can do it with less changes in avax code. Beside this to clearify: why was genesis.Camino replaced ?

The structure was to multipurposal: it was containing actual actual part of genesis, but also was used for keeping some small config part from genesis. The genesis itself is only needed on initial state sync, but not after, so there is no reason to still keep in state struct with all genesis stuff like deposit txs and offers - they'r also exist in their own parts of state after sync.


Convert to unplugin

This may be outside the scope of what you want to do with this package (which is fine, just let me know and I can fork/publish).
This PR:

Converts rollup-plugin-pure to unplugin instead of exclusively rollup.
Updates dependencies

Worth noting that the original rollup plugin used the rollup-exclusive syntax to position the plugin post — since this is not supported (as far as I can tell) with unplugin this could be considered a slight regression.

Summary by CodeRabbit

New Features

Enhanced build configuration with support for popular build tools.
Introduced a new component definition method for improved development experience.

Refactor

Reorganized code structure for better maintainability and performance.
Updated transformation logic to enhance plugin functionality.

Going to go ahead and just run with a fork of this — I think the two are sufficiently different.
https://github.com/justin-schroeder/unplugin-pure

Oh no! I'm very happy to merge and reuse; this just dropped between the cracks

No worries! It probably doesnt belong here anyway since this is specifically a rollup plugin. ❤


Refactor Python files and update dictionary

I discovered your library and thought I could contribute since I plan to use it. I've done the following:

Conventionalised Python files.
Updated scrabble/dictionary.json according to the Collins Official Scrabble Dictionary (2019).
Added dependency for setuptools since it's used in setup.py.

Upon re-reading my PR, I've realised that adding setuptools as a dependency is pointless.

I think that's all. Something else I will bring up as an option to you, though, is to use the qrs library's wordlist_full list instead of having to maintain your own copy.
$ pip install qrs
$ py

>>> import qrs
>>> print(qrs.wordlist_full)


[.dev] private profile + night mode = can't read text

QA label is usually to be used for pull-requests.


files-cli: 2.13.141 -> 2.13.148

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/files-com/files-cli/releases.
meta.description for files-cli is: Files.com Command Line App for Windows, Linux, and macOS
meta.homepage for files-cli is: https://developers.files.com
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.13.148 with grep in /nix/store/8fsmh9ndkvma1pzkjkrdmiilfp810baz-files-cli-2.13.148
found 2.13.148 in filename of file in /nix/store/8fsmh9ndkvma1pzkjkrdmiilfp810baz-files-cli-2.13.148

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

files-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8fsmh9ndkvma1pzkjkrdmiilfp810baz-files-cli-2.13.148 \
  --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 files-cli https://github.com/r-ryantm/nixpkgs/archive/725e4570b93145b100bcc75d4b207799c57d4b6a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/725e4570b93145b100bcc75d4b207799c57d4b6a#files-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8fsmh9ndkvma1pzkjkrdmiilfp810baz-files-cli-2.13.148
ls -la /nix/store/8fsmh9ndkvma1pzkjkrdmiilfp810baz-files-cli-2.13.148/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

1 package built:

files-cli

Maintainer pings
cc @kashw2 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Chrome 32, SizeWatcher can't set scroll position of scroll watchers.

Hi,
Great library but unfortunately,
I get this error and it says not all fonts were loaded.
SizeWatcher can't set scroll position of scroll watchers.
Im trying to use this with "weathericons". Here's the link to the CDN for the font css file:
https://cdnjs.cloudflare.com/ajax/libs/weather-icons/2.0.5/css/weather-icons.css
Best Regards,
Bhargav

Hi @rrphotosoft,
In which OS and browser version is it happening,
Also, can you paste a code snippet of how you are trying to load the fonts? Are you setting timeout for example?

Here, the unicode character of the "day-cloudy-high" icon is "F07D". Adding it to the FontLoader.referenceText should solve the issue:
<!DOCTYPE html>
<html>
<head>
	<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/weather-icons/2.0.5/css/weather-icons.css">
	<script type="text/javascript" src="FontLoader.js"></script>
</head>
<body>
	<script type="text/javascript">
		FontLoader.referenceText += "\uf07d";
		var fontLoader = new FontLoader(["weathericons"], {
			"fontLoaded": function(font) {
				console.log("font loaded: " + font.family);
			},
			"complete": function(error) {
				if (error !== null) {
					// Reached the timeout but not all fonts were loaded
					console.log(error.message);
					console.log(error.notLoadedFonts);
				} else {
					// All fonts were loaded
					console.log("all fonts were loaded");
				}
			}
		}, 3000);
		fontLoader.loadFonts();
	</script>
</body>
</html>

@smnh Hi! First, thanks for this library :)
I got this SizeWatcher can't set scroll position of scroll watchers. message in console on Safari and Firefox while at the same time I don't get it on Chrome (I'm on Mac).
I'm using this library to load several fonts in a text editor case, so I can not figure the characters out because it can have a lot of fonts used. So I can not really try your previous answer/fix.
But because it works on Chrome I wonder if it is actually the same issue, and if it could fix Firefox and Safari errors anyway ....
Do you have any idea what could make your lib works on Chrome but not other browers? :/

Hi @heymath,
Does this error appear for specific fonts you try to load, or for any font?
You can run the tests located in Tests/compat-tests.html in Firefox and Safari to see if the FontLoader works for the general case. If these tests are passing successfully (except the 3rd test which should pass only in IE10 and lower), then probably there is a problem with a specific font. Try to use the elimination method to find the problematic font.

Hi @smnh, indeed I was in a rush so I just moved to WebFontLoader instead of FontLoader, however thank you for your help.
FYI: I just ran tests in Chrome, Firefox and Safari, and they passed excepted the 3rd as described.


Styling broken for ProductiveCard in v2.39.0

Package
Carbon for IBM Products
Description
They styling for the ProductiveCard component looks like it has broken between v2.38.0 and v2.39.0 (possibly because of this PR? - https://github.com/carbon-design-system/ibm-products/pull/5054/files)
Screen shot of the ProductiveCard using v2.38.0 (working as expected)

Screen shot of the ProductiveCard using v2.39.0 (styling is broken)

I believe that you can also see the problem when looking at the storybook for ibm-products
https://ibm-products.carbondesignsystem.com/?path=/story/ibm-products-components-cards-productivecard--with-caption&globals=viewport:basic

Component(s) impacted
ProductiveCard
Browser
No response
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
v2.39.0
Severity
Severity 2 = Aspects of design is broken, and impedes users in a significant way, but there is a way to complete their tasks. Affects major functionality, has a workaround.
Product/offering
IBM App Connect
CodeSandbox or Stackblitz example
https://ibm-products.carbondesignsystem.com/?path=/story/ibm-products-components-cards-productivecard--with-caption&globals=viewport:basic
Steps to reproduce the issue (if applicable)
Navigate to the ibm-products storybook (https://ibm-products.carbondesignsystem.com/?path=/story/ibm-products-components-cards-productivecard--with-caption&globals=viewport:basic) and view the ProductiveCard component
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

thank you for pointing this out! i've identified the bug and will push a fix ASAP


bug: searchbar.setfocus() needs a short delay

Type: bug
Ionic Version: 2.x
Platform: all
Hello Ionic team,
when a component tries to set the focus on searchbar the setFocus() method needs a short delay to work.
Here a plunker to demonstrate the problem
https://plnkr.co/edit/0mwSiLWMHhtn9vy7U8vl?p=preview
Change the time in home.page.ts to 100 ms and the focus is set correctly.
setTimeout(()=>sb.setFocus(), 100);
greetings

With RC2 seems to work fine now.


We could have the scraper as a separate process.

it was not fixed, no

I believe it already is running as a separate process.
Alucard, got any more info on the error ? I remember seeing logs of it, but lost them.

it hits the database hard with bulk update transactions from multiple goroutines
the scraper already runs as a separate process and has from the start

Woops in my head the issue was about rss feed

Now that we have like all the scrapped data, would it not that be better to have it run only when you look a specific torrent. Of course, you check before that the seed/leech data is not fresh enough in the database.
So only torrents that have been looked recently will have fresh data. Then you can run a daily scrapping of the last 500 torrents that do not have up to date scrapped data.
500 being the number of torrents that we are sure doesn't crash the server for example.
You can also stop doing transactions and just creating all the queries and make a single big transaction with the query built. Or doing the same but every 20 torrents for example.
Anyway, that's just ideas to improve it, dunno how the scrapper actually work.

That fucks up sorting by seeders I think

Fixed?

I think =)


Use a crawl to fetch all the invalid herf in doc

I wrote a crawl to fetch all the invalid herf in website:  https://kubernetes.io/docs/home/
This is my crawl.go: http://paste.openstack.org/show/614826/
Run this command:
go run findlinks.go https://kubernetes.io/docs/home/
Then we can find some invalid herf.
I fixes part of them:
https://github.com/kubernetes/kubernetes.github.io/pull/4330
But I found that there are so many invalid herf.
http://paste.openstack.org/show/614827/     (part of them)
I opened this issue to trace the invalid herf.

@zhangxiaoyu-zidif  @jianglingxia may have interest to this issue.

@chenopis

Thanks @CaoShuFeng , We have a similar tool, right? @chenopis .
This would be a good supplement.

@CaoShuFeng @zhangxiaoyu-zidif We do have a tool, but it might be good to also have a published script that anyone else can run too.


Honour regex in dependsOnMethods

Closes: #141
Fixes #141  .
Did you remember to?

[X] Add test case(s)
[X] Update CHANGES.txt
[X] Auto applied styling via ./gradlew autostyleApply

We encourage pull requests that:

Add new features to TestNG (or)
Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.
Note: For more information on contribution guidelines  please make sure you refer our Contributing section for detailed set of steps.

@juherr - I have added the tests. Please take a look and let me know if there's anything additional that you are looking for.

@juherr - I have addressed all the review comments. There's one bug that I found which seems to exist when you use dependsOnMethods and use a fully qualified method name. Since that defect exists in regular dependsOnMethods usage (without regex), when I fix that defect I will do it for regex as well.
Here's the issue link https://github.com/cbeust/testng/issues/2840

What do you thing to add the following case:
public class TestClassSample {

  @Test(dependsOnMethods = "test_C[0-9]{7}")
  public void randomTest() {}

  public static class Inner {
    @Test
    public void test_C6390323() {}
  }
}

randomTest is supposed to fail here because test_C6390323 is out of scope.

@juherr - I have added that test as well. Please check.


Release build must reload cache manually to work

Bug Report
Please answer these questions before submitting your issue. Thanks!
What did you do?
Load TiDB Dashboard webpage.
What did you expect to see?
It works.
What did you see instead?
It will fail to work due to cache is not forced to invalidate in the latest HTML:
    <link rel="stylesheet" href="/dashboard/dashboardApp.css" />

    You need to enable JavaScript to run this app.

What version of TiDB Dashboard are you using (./tidb-dashboard --version)?
master

Yep, in clinc service ui, I add an extra timestamp to the link, is it ok for dashboard, or maybe it better to calculate its content hash?
    <link rel="stylesheet" href="index.css?t=TIME_PLACE_HOLDER" />

function buildHtml(inputFilename, outputFilename) {
  let result = fs.readFileSync(inputFilename).toString()

  const placeholders = ['TIME_PLACE_HOLDER']
  placeholders.forEach((key) => {
    result = result.replace(new RegExp(key, 'g'), new Date().valueOf())
  })

  fs.writeFileSync(outputFilename, result)
}

@baurine It's your turn to decide the engineering part now :) I'm fine as long as it does resolve the issue.

The content hash is more popular. But considering we're only build the dist on release and not on project deployment, timestamp is not a problem.


Assertion triggered when system is going to PM_STATE_SOFT_OFF

Describe the bug
When I try to switch to SHUTDOWN mode using applicatiom PM policy then there is triggered system assertion from sched.c.
The call stack reveals that there is scheduled semaphore waiting from the idle thread and it is not allowed.
Platform: STM32WB55
Call stack:
z_priq_rb_add -> __ASSERT_NO_MSG(!z_is_idle_thread_object(thread))
add_to_waitq_locked
pend
z_pend_curr
z_impl_k_sem_take
k_sem_take
bt_hci_cmd_send_sync
send_stack_reset
shutdown_ble_stack
pm_power_state_set
pm_state_set
pm_system_suspend
idle
To Reproduce
Steps to reproduce the behavior:

Request system to switch to PM_STATE_SOFT_OFF on STM32WB55

Expected behavior
No assertion should be triggered when system try to switch to PM_STATE_SOFT_OFF.
Impact
showstopper
Logs and console output
Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK 0.13.1
Commit SHA e8df8a579033be238beb774e4909d637da649f3a

Additional context

yep, that is a real problem. The bluetooth stack takes a sempahore and we have an assert checking if it is the idle threa doing it. We don't want the idle thread being blocked  ... also this operation can not be async since the soc will be powered down.
This particular situation seems ok to ignore the assert because is is happening when powering off the system ... if you build without asserts you everything should work fine ( I don't have this target to exercise it though).
The problem is real and I'll take some time investigating a proper solution for it.

@erwango Would it be possible to use asynchronous call (with polling) instead of bt_hci_cmd_send_sync?

@r2r0 I think that this would work indeed. Not need to add a supplemenary polling as it is already present with the following check in function shutdown_ble_stack.
while (LL_PWR_IsActiveFlag_C2DS() == 0) {

So the new code would looks like:
static void send_stack_reset(void)
{
	int err = 0;

	err = bt_hci_cmd_send(ACI_HAL_STACK_RESET, NULL);

	if (err) {
		LOG_ERR("M0 BLE stack reset issue");
	}
}

Thanks @erwango - PR submitted.


Navbar Search Form Messed Up After Chrome 50 Update

It seems the navbar search form is not functioning properly after Chrome was updated. As used on this page: http://materializecss.com/navbar.html, You can see you have to click in the upper half of it to get into the search bar.

Experiencing this as well after the Chrome 50 update.
My navbar has centered "logo" text and when you click somewhere on the navbar (logo text included) it becomes a search bar. Well, that's what used to happen at least.
Now if I click on the logo text it actually acts as a link, and the click area to trigger searching is messed up as shown if you visit the materializecss navbar search example (that alexknipfer linked).

I can confirm this.  The clickable area is now 20px tall.  Works fine in Safari.

This has already been fixed apparently. You can apply the reference Stack Overflow hotfix too- https://github.com/Dogfalo/materialize/issues/3082

Please close this issue :)


Contributions

Hi,
I like this project and I'd like to contribute.
Do you take PR? Are there specific features in the pipeline for implementation?
Thanks,
Em

Hey! I haven't done much on this project for a while, but i'd definitely accept a pull request if you have a feature or bugfix you wanted to contribute. I don't have a feature pipeline or anything, but if you wanted to pick up something specific, the implementation of using arrow keys to navigate seems kinda unreliable across different operationing systems, so if you can find a better way to handle that function then that would be immense!


New Data Source from ADS - RDP Session

New RDP session data source.

I believe this will be captured between user account:auth, logon session, and network traffic


Containers in a "Created" status are being displayed as "Running" on the doughnut graph

Here's the ps list:
$ docker ps -a
CONTAINER ID        IMAGE                               COMMAND                  CREATED             STATUS                      PORTS                    NAMES
ea3edfd44f7d        selenium/standalone-firefox-debug   "-name firefox-standa"   12 minutes ago      Created                                              distracted_brattain
cbd7173b84cc        uifd/ui-for-docker                  "/ui-for-docker"         46 hours ago        Up 46 hours                 <IP_ADDRESS>:9000->9000/tcp   pedantic_cori
c4872aa026c8        selenium/node-firefox-debug         "/opt/bin/entry_point"   46 hours ago        Up 46 hours                 <IP_ADDRESS>:5556->5900/tcp   root_selenium-firefox_1
5bbacdf31116        selenium/node-chrome-debug          "/opt/bin/entry_point"   46 hours ago        Up 46 hours                 <IP_ADDRESS>:5555->5900/tcp   root_selenium-chrome_1
1c32ace8d60d        selenium/hub:latest                 "/opt/bin/entry_point"   46 hours ago        Up 46 hours                 <IP_ADDRESS>:4444->4444/tcp   root_selenium-hub_1
d286495b8ad7        selenium/node-firefox-debug         "/opt/bin/entry_point"   47 hours ago        Exited (137) 47 hours ago                            selenium-node-firefox-debug2
20012e4fc62f        selenium/node-chrome-debug          "/opt/bin/entry_point"   47 hours ago        Exited (137) 47 hours ago                            selenium-node-chrome-debug2
8989785b01ca        selenium/node-chrome-debug          "/opt/bin/entry_point"   47 hours ago        Exited (137) 47 hours ago                            selenium-node-chrome-debug
d8a113b463aa        selenium/node-firefox-debug         "/opt/bin/entry_point"   47 hours ago        Exited (137) 47 hours ago                            selenium-node-firefox-debug
61d8ba297c34        selenium/hub                        "/opt/bin/entry_point"   2 days ago          Exited (143) 47 hours ago                            selenium-hub
df5b5970e3b7        selenium/standalone-firefox-debug   "/opt/bin/entry_point"   2 days ago          Exited (137) 2 days ago                              standalone-firefox-debug
21ad680d3152        selenium/standalone-firefox-debug   "/opt/bin/entry_point"   2 days ago          Exited (137) 2 days ago                              kickass_davinci

What the doughnut shows, is 5 in running, and 7 stopped. But there are actually only 4 running.

I have a potential fix for this. I just need to submit a PR

Here's the PR: https://github.com/kevana/ui-for-docker/pull/258 -- admittedly, this was a fairly naive change. I don't have a full understanding of what's going on in the rest of the codebase. If this doesn't make sense, feel free to dump it.

Merged, will be in next release.


Only one image from multiple is used (browser side via ArrayBuffer)

Hello,
Problem when using pdfkit client-side (browser only).
When adding multiple different images on one page only one of them (first one) is used at every iteration of doc.image().
Every image is first downloaded via XHR as ArrayBuffer
eg. You want to add images from such urls:
img1 = https://pdfkit.org/docs/images/test.jpeg
img2 = https://pbs.twimg.com/profile_images/519367942866104320/PB96rDH_.png
img3 = https://img.freepik.com/free-icon/github-character-silhouette_318-40485.jpg?size=338&ext=jpg
img1, img2 and img3 are downloaded via XHR with 'arraybuffer' responseType
We add them to doc:
doc.image(img1, 455, 80, {fit: [80, 80]});
doc.image(img2, 455, 200, {fit: [80, 80]});
doc.image(img3, 350, 200, {fit: [80, 80]});
In output PDF only one image is used (first one - in above example 'img1') at every coordinations.
JSFiddle demonstration

I'm Having the same problem. Cann't figure it out. Please Help somebody

I guess this is not possible with pdfkit


HTCondor CI is failing

I've put some time in trying to debug HTCondor CI (see https://github.com/dask/dask-jobqueue/pull/562#issuecomment-1204389580).
If it did work on my laptop, I've not been able to make it works in github Actions, don't know why yet.
Some details:

I've updated Docker images to run the correct Python version (same as other CI environments),
I've updated docker-compose setup with a configuration on HTCondor negociator Interval, the time between to scheduling cycle, as the default of 60s made it possible tests fail because of that, or made tests really long.
I've added some debugging docker commands to try to understand what's happening.

From the last run here, we can see from the Cleanup sections and condor_history command output that the HTCondor queuing system is working. The problem seems to come from the worker jobs which complete really fast,  Dask workers never connect to Scheduler.
I guess we would need to see the jobs stdout/stderr to debug further the problem.
cc @riedel @mivade @jolange.

Thanks  @guillaumeeb ! Yes, this looks like the worker jobs are failing directly.
The command is starting with a semicolon now:
 Arguments = "-c '; /opt/anaconda/bin/python -m distributed.cli.dask_worker tcp://<IP_ADDRESS>:33755 --nthreads 1 --memory-limit 100.00MiB --name HTCondorCluster-1 --nanny --death-timeout 60'"

I certainly broke this with my changes, I guess because env_extra does not stay None due to this line.
I'm not sure if this would cause this to fail, but I'll fix it first before looking further into this.
Otherwise, seeing stderr would help a lot, probably -- I remember that I had to do some manual downgrades of click for some python versions. But I tried a number of setups, so I don't remember really...

Closed by #570.


Upgrade to webpack-dev-middleware@7

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(docs): add spec for capnproto as well-known encoding

Public-Facing Changes
add spec for capnproto as well-known encoding
Description
The spec is overly verbose to be descriptive but we will move the details into the tooling.
Follow up:

use cgo to allow the CLI to perform the schema serialization. (there is currently no canonical serialization of capnproto schemas. See e.g. https://groups.google.com/g/capnproto/c/Ivl3u4JJoEQ)

I've validated that it correctly load the openpilot log.capnp:Event schema in the browser

@jon-chuang based on your messages in the Foxglove Community Slack, it seems like you've already got deserialization working for capnproto in Studio? Would you be willing to make a PR to foxglove/studio to add deserialization support as well as add campnproto to foxglove/schemas?

Would you be willing to make a PR to foxglove/studio to add deserialization support as well as add campnproto to foxglove/schemas?

Yes. But let’s get agreement on the spec here first.
For foxglove/schemas, I believe this is lower priority (L2) since it doesn’t block useability (user-script/plugin workarounds are just fine)

This could be solved by including both display name and ID in the Schema.name field, perhaps separated by a space character.

Does the display name include any path information? Or is that not considered part of the display name?

Does the display name include any path information? Or is that not considered part of the display name?

This is optional/configurable. Generally, it does.

@jon-chuang is this pr still on your radar? I'm still aligned that its a good idea if you're able to address @james-rms's feedback.


Scheduler & Executor counter

Keep global scheduler and executor counters used for generation of UPID (i.e. scheduler(1), scheduler(2), ... , scheduler(n))

there's an example of this in PR #53 as part of the SASL authenticator


fix SimpleNLG -> SimplerNLG in code examples

@mattgillooly Thanks, good catch!

My pleasure!  Took me a minute to figure out why the examples weren't working as-is, so hopefully this will save the next developer a couple minutes of head-scratching.  Thanks for this gem, and the punditbot source, which I'm also finding to be instructive.

@mattgillooly: "next developer" -- you're very optimistic! :) Good luck and holler if you've got any questions...


Add s3 dev uploads bucket

This is to add a single bucket, digitalmarketplace-dev-uploads to the dev aws account. This bucket will be used for all operations in the dev environment instead of separate buckets for separate upload types - see this PR for more details.
The policy assigned to the bucket should allow any user in the main account all bucket object operations. I don't think it will allow any permissions for bucket operations (such as deleting the bucket or it's policies).
I've tried to keep the module fairly generic, but how much scope there is in future for creating more s3 buckets with this particular policy I'm not sure. It may be worth just moving this bucket resource and policy to the dm-aws-dev directory instead of importing the module. Thoughts are most welcome.

The overall module name "s3-bucket" is too general, you should rename it to "dev-s3-bucket" or similar.

@bandesz I've made a few changes based on the above, if you could let me know what you think? It plans fine on the aws-dm-dev side of things, but I don't have permissions on the aws-dm side.