FR: add tooltip

please consider adding a tooltip option to the tabs.
Using an icon only tabbed config (because the labels are not resizable and force us to swipe the tabs) it would be really nice to have a tooltip on hovering.
Just like the tooltip in the main header bar
if it could be done with configurable transition, even better
using this myself on all button-cards to give you an idea:
  styles:
    tooltip:
      - transition: opacity 0.5s. # <-- main concern here, and makes the tooltip immediate
      - color: var(--card-background-color)
      - font-size: 12px
      - background: var(--primary-color)
      - padding: 5px
      - border-width: 0px

thanks for considering!

moved to discussions...


fix filepath split bug in win

the filepath in the Windows environment looks like  C:\Users**\AppData\Local\bisheng\bisheng\Cache\bisheng
run filepath.rsplit('/', 1)[1] will not success to get the file name

great for your code


Provide dex connector for GitHub Enterprise

Background
Currently, dex connector is available for public GitHub use.
GitHub Enterprise requires additional host settings in dex-connector-def.yaml.
Related link
https://dexidp.io/docs/connectors/github/#github-enterprise
Need to be changed
def
https://github.com/kubevela/catalog/blob/e07244fe0ab185eeecf564fd759f37a369271749/addons/dex/definitions/dex-connectors-def.yaml#L60-L67
ui-schema
https://github.com/kubevela/catalog/blob/e07244fe0ab185eeecf564fd759f37a369271749/addons/dex/schemas/component-uischema-config-dex-connector.yaml#L3-L20

@TommyHwang Welcome you to contribute to this and help to test.

Update
It is currently being tested on velaux after adding hostName.
velaux returns 500 internal error as shown below.

So I'm checking api server.


feat: Setup Poetry environment

This PR adds the following:

reset-dev-env.bash to enable ease of setting up Poetry environment
.python-version
Update pyproject.toml to include project description and authors list

You'll need to resolve the conflict before I can review it.


Add encryption for CNC files

Add possibilities to encrypt or decrypt an CNC file with password.

Current status: First version of blowfish algorithm will be tested soon in a seperate project via unit tests.
The user interface integration of blowfish is written and will be tested, improved and pushed after succesful unit tests of blowfish.

This feature was published in the newest version


Parameter can be confused with a section and trigger false positives

I am using NumPy style, I did not check if the following also applies to Google style.
def function(parameters: list, second_parameter: int):
    """Process parameters.

    Parameters
    ----------
    parameters
        Listing the parameter "parameters" here triggers 5 warnings.
    second_parameter
        This is OK.
    """

The 5 warnings are:
D410 Missing blank line after section
D414 Section has no content
D405 Section name should be properly capitalized
D411 Missing blank line before section
D407 Missing dashed underline after section

Apparently pydocstyle is confused because the parameter is, apart from capitalization, spelled like a section header.

Duplicate of #481


Feature tidy fixes

:tada: This issue has been resolved in version 1.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[extractor/rcs] Improved and fixed RCS extractor

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

Fixes #5683

rewritten _get_video_src to extract m3u8 and mp3 url more reliably
implemented _create_http_formats to create http mp4 direct urls from m3u8 manifest urls
added test for audio only format
f-strings and other minor fixes
updated _TESTS with updated values

Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)

@pukkandan @bashonly this morning Core Test with Py 3.7 fails on some SSL error

We are aware of the issue. You can ignore it

ok, I collected url from the various website managed by the extractor and now everything is handled as it should.

from a rapid test if fails...
[RCSEmbeds] Extracting URL: https://video.rcs.it/video-embed/iodonna-0001585037
[RCSEmbeds] iodonna-0001585037: Downloading webpage
[RCSEmbeds] iodonna-0001585037: Downloading m3u8 information
[RCSEmbeds] iodonna-0001585037: Check filesize for https-945
WARNING: [RCSEmbeds] Unable to download webpage: HTTP Error 403: Forbidden
[RCSEmbeds] iodonna-0001585037: Check filesize for https-657
WARNING: [RCSEmbeds] Unable to download webpage: HTTP Error 403: Forbidden
[RCSEmbeds] iodonna-0001585037: Check filesize for https-358
WARNING: [RCSEmbeds] Unable to download webpage: HTTP Error 403: Forbidden
[RCSEmbeds] iodonna-0001585037: Check filesize for https-986
WARNING: [RCSEmbeds] Unable to download webpage: HTTP Error 403: Forbidden
[RCSEmbeds] iodonna-0001585037: Check filesize for https-1130
WARNING: [RCSEmbeds] Unable to download webpage: HTTP Error 403: Forbidden
[info] Available formats for iodonna-0001585037:
ID       EXT RESOLUTION FPS │   TBR PROTO │ VCODEC        VBR ACODEC    ABR
───────────────────────────────────────────────────────────────────────────
hls-358  mp4 416x234     25 │  358k m3u8  │ avc1.42c01e  358k mp4a.40.2  0k
hls-657  mp4 640x360     25 │  657k m3u8  │ avc1.42c01e  657k mp4a.40.2  0k
hls-945  mp4 960x540     25 │  946k m3u8  │ avc1.42c01e  946k mp4a.40.2  0k
hls-986  mp4 1270x720    25 │  987k m3u8  │ avc1.42c01e  987k mp4a.40.2  0k
hls-1130 mp4 1270x720    25 │ 1130k m3u8  │ avc1.42c01e 1130k mp4a.40.2  0k

[RCSEmbeds] Extracting URL: https://video.rcs.it/video-embed/iodonna-0001585037
[RCSEmbeds] iodonna-0001585037: Downloading webpage
[RCSEmbeds] iodonna-0001585037: Downloading m3u8 information
[RCSEmbeds] iodonna-0001585037: Check filesize for https-945
[RCSEmbeds] iodonna-0001585037: Check filesize for https-657
[RCSEmbeds] iodonna-0001585037: Check filesize for https-358
[RCSEmbeds] iodonna-0001585037: Check filesize for https-986
[RCSEmbeds] iodonna-0001585037: Check filesize for https-1130
[info] Available formats for iodonna-0001585037:
ID         EXT RESOLUTION FPS │  FILESIZE   TBR PROTO │ VCODEC        VBR ACODEC    ABR
───────────────────────────────────────────────────────────────────────────────────────
hls-358    mp4 416x234     25 │            358k m3u8  │ avc1.42c01e  358k mp4a.40.2  0k
https-358  mp4 416x234     25 │ ~ 3.93MiB  358k https │ avc1.42c01e  358k mp4a.40.2  0k
hls-657    mp4 640x360     25 │            657k m3u8  │ avc1.42c01e  657k mp4a.40.2  0k
https-657  mp4 640x360     25 │ ~ 7.14MiB  657k https │ avc1.42c01e  657k mp4a.40.2  0k
hls-945    mp4 960x540     25 │            946k m3u8  │ avc1.42c01e  946k mp4a.40.2  0k
https-945  mp4 960x540     25 │ ~10.24MiB  946k https │ avc1.42c01e  946k mp4a.40.2  0k
hls-986    mp4 1270x720    25 │            987k m3u8  │ avc1.42c01e  987k mp4a.40.2  0k
hls-1130   mp4 1270x720    25 │           1130k m3u8  │ avc1.42c01e 1130k mp4a.40.2  0k
https-986  mp4 1270x720    25 │ ~10.68MiB  987k https │ avc1.42c01e  987k mp4a.40.2  0k
https-1130 mp4 1270x720    25 │ ~12.23MiB 1130k https │ avc1.42c01e 1130k mp4a.40.2  0k

it does not properly creates http urls. I'll check out why.

ok, now it's good. fixed a couple of regexes that no longer matched following the Cleanup and added usage of search_json


Small sample game error!?

Current bug behaviour
I am not able to run the example at https://github.com/flame-engine/flame/tree/main/packages/flame/example
I get an error in the logs... and a red screen on the device/emulator/chrome
Expected behaviour

Steps to reproduce

Flutter doctor output

[√] Flutter (Channel stable, 2.5.3, on Microsoft Windows [Version 10.0.19043.1415], locale en-US)
    • Flutter version 2.5.3 at C:\flutterio\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (10 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

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

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

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

[√] IntelliJ IDEA Community Edition (version 2019.2)
    • IntelliJ at D:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2019.2.3
    • 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

[√] VS Code (version 1.61.2)
    • VS Code at C:\Users\Yo\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.110
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 93.0.961.52

More environment information

Flame version: 1.0.0
Platform affected: android, web
Platform version Tested on: android 9 emulator, chrome, and One+1 device

Log information

======== Exception caught by widgets library =======================================================
The following assertion was thrown building GameWidget<MyGame>(dirty, state: _GameWidgetState<MyGame>#d13eb):
        WARNING: Both Advanced and Basic detectors detected.
        Advanced detectors will override basic detectors and the later will not receive events
'package:flame/src/game/game_widget/game_widget.dart':
Failed assertion: line 273 pos 7: '!(hasBasicDetectors && hasAdvancedDetectors)'

The relevant error-causing widget was:
  GameWidget<MyGame> GameWidget:file:///D:/Users/Yo/AndroidStudioProjects/FlutterStuff/FlameV1.0/flutter_app/lib/main.dart:11:5
When the exception was thrown, this was the stack:
#2      _GameWidgetState.build (package:flame/src/game/game_widget/game_widget.dart:273:7)
#3      StatefulElement.build (package:flutter/src/widgets/framework.dart:4782:27)
#4      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4665:15)
#5      StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:4840:11)
#6      Element.rebuild (package:flutter/src/widgets/framework.dart:4355:5)
#7      ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:4643:5)
#8      StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:4831:11)
#9      ComponentElement.mount (package:flutter/src/widgets/framework.dart:4638:5)
#10     Element.inflateWidget (package:flutter/src/widgets/framework.dart:3673:14)
#11     Element.updateChild (package:flutter/src/widgets/framework.dart:3425:18)
#12     RenderObjectToWidgetElement._rebuild (package:flutter/src/widgets/binding.dart:1198:16)
#13     RenderObjectToWidgetElement.mount (package:flutter/src/widgets/binding.dart:1167:5)
#14     RenderObjectToWidgetAdapter.attachToRenderTree.<anonymous closure> (package:flutter/src/widgets/binding.dart:1112:18)
#15     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2573:19)
#16     RenderObjectToWidgetAdapter.attachToRenderTree (package:flutter/src/widgets/binding.dart:1111:13)
#17     WidgetsBinding.attachRootWidget (package:flutter/src/widgets/binding.dart:944:7)
#18     WidgetsBinding.scheduleAttachRootWidget.<anonymous closure> (package:flutter/src/widgets/binding.dart:924:7)
(elided 13 frames from class _AssertionError, class _RawReceivePortImpl, class _Timer, dart:async, and dart:async-patch)
====================================================================================================

The example is updated for main, you have to run the example for the version that you are using:
https://github.com/flame-engine/flame/tree/1.0.0/packages/flame/example (for 1.0.0)

I am not able to select 1.0.0 ... its as if that repo isnt available. When I use the drop down it isnt present... neither does the search show anything. May be you have extra privileges to see that repo... but I cant. I've tried from both my accounts.
If it wasnt for your link... I would have never been able to get there.
post that the game never threw an error

You are running thr example from main, select the 1.0.0 tag and use that example instead. The main branch has gesture festures that aren't available on 1.0.0 yet.

I can see the full list of tags even when not being logged in and using incognito mode, are you sure that you are looking in the tags list and not the branch list?

You can also just go here to see the tags:
https://github.com/flame-engine/flame/tags

@spydon ... thanks for getting back.
Yeah now I know what you mean... It was an error on my end.
Im able to see them clearly when I check under tags... thanks.
Sorry about the confusion.
thanks again ;-)


Project name with dash in it (e.g. man-db) loses dash on install

The culprit is here:
      PROJECT_NAME="$(echo ${ZOPEN_NAME} | cut -d'-' -f1)"

The cut is there because it wants a 'clean' project name that doesn't have the version information in it, but the cut to drop stuff after '-' is too simplistic.
After the install, instead of having a symbolic link from man-db to man-db-2.10.2 (for example), the build will generate a symbolic link from man to man-db-2.10.2. This creates all sorts of problems if you install man-db and then build a new version.
@IgorTodorovskiIBM I assigned this to you - thinking this would be a good first issue for someone?

there are a few heuristics that could be used, but looking for the last dash is probably safest. Here are a couple examples:
lrwxrwxrwx   1 FULTONM  BASIC         13 Jun 21 20:45 man-db -> man-db-2.10.2
drwxr-xr-x   8 FULTONM  BASIC       8192 Jun 21 20:45 man-db-2.10.2
lrwxrwxrwx   1 FULTONM  BASIC         12 Jun 21 20:45 groff -> groff-1.22.4
drwxr-xr-x   5 FULTONM  BASIC       8192 Jun 21 20:45 groff-1.22.4
drwxr-xr-x   4 FULTONM  BASIC       8192 Jun 21 20:45 less-632
lrwxrwxrwx   1 FULTONM  BASIC          8 Jun 21 20:45 less -> less-632
lrwxrwxrwx   1 FULTONM  BASIC         15 Jun 21 20:45 libiconv -> libiconv-master
drwxr-xr-x   6 FULTONM  BASIC       8192 Jun 21 20:45 libiconv-master
lrwxrwxrwx   1 FULTONM  BASIC         11 Jun 21 20:45 ncurses -> ncurses-6.3
drwxr-xr-x   6 FULTONM  BASIC       8192 Jun 21 20:45 ncurses-6.3

If you were to look for a number after a dash, that would break libiconv-master (it's text). I don't have all the versions available but unless a version itself had a dash in it, then this heuristic would be ok (probably best to check on perhaps a jenkins system that has everything there)

Ah yes, we have encountered this in other projects as well. The current workaround is to set ZOPEN_NAME to a name without dashes.
But I agree, we need a solution that should allow for dashes in the name so that it's consistent with the naming in other package managers like brew.

i couldn't help myself. I did a dummy install of everything:
CMake-heads.v3.24.2      expat-2.4.9              help2man                 lua-5.1.5                pcre2-10.42              unzip
autoconf                 expect                   help2man-1.49.2          lynx                     perl                     unzip-master
autoconf-2.71            expect-5.45.4            jq                       lynx-2.8.9               perl5-blead              vim
automake                 findutils                jq-1.6                   lz4                      pinentry                 vim-master
automake-1.16.5          findutils-4.9.0          less                     lz4-1.9.3                pinentry-1.2.1           wget
bash                     flex                     less-632                 m4                       pkgconfig                wget-1.21.3
bash-5.2.15              flex-2.6.4               libassuan                m4-1.4.19                python                   which
bison                    gawk                     libassuan-2.5.5          make                     re2c                     which-2.21
bison-3.8.2              gawk-5.1.1               libgcrypt                make-4.4.1               re2c-3.0                 xmlto
byacc                    getopt                   libgcrypt-1.10.2         man-db                   rsync                    xmlto-0.0.28
byacc-20230521           getopt-1.1.6             libgdbm                  man-db-2.10.2            rsync-master             xxHash-0.8.1
bzip2                    gettext                  libgdbm-master           meta                     screen                   xxhash
bzip2-1.0.8              gettext-0.21             libgpgerror              meta-main                screen-4.9.0             xz
cacert.pem               git                      libiconv                 nano                     sed                      xz-5.2.5
cmake                    git-2.41.0               libiconv-master          nano-7.0                 sed-4.8                  yq
comp_clang               gnulib                   libksba                  ncdu                     sqlite                   yq-master
comp_go                  gnulib-master            libksba-1.6.3            ncdu-1.17                sqlite-autoconf-3420000  zigi
comp_xlclang             gnupg-2.4.0              libpcre                  ncurses                  sshpass                  zigi-master
coreutils                gperf                    libpipeline              ncurses-6.3              sshpass-1.09             zip
coreutils-9.1            gperf-3.1                libpipeline-1.5.6        ninja                    sudo                     zip-master
cscope                   gpg                      libssh2                  ninja-master             sudo-1.9.13p3            zlib
cscope-15.9              grep                     libssh2-1.10.0           npth                     tar                      zlib-heads.v1.2.13
ctags                    grep-3.8                 libtool                  npth-1.6                 tar-1.34                 zoslib
ctags-5.8                groff                    libtool-2.4              openssh                  tcl                      zoslib-zopen
curl                     groff-1.22.4             libxml2                  openssh-9.3p1            tcl-core8.6.12-src       zotsample
curl-8.1.2               gzip                     libxml2-2.9.12           openssl                  texinfo                  zotsample-1.0
diffutils                gzip-1.12                libxslt                  openssl-1.1.1t           texinfo-6.8              zstd
diffutils-3.8            hello                    libxslt-1.1.37           patch                    tig                      zstd-1.5.2
expat                    hello-master             lua                      patch-2.7                tig-2.5.8

So - looks like sqlite-autoconf-3420000 will cause us grief?

if i set ZOPEN_NAME will that then use that name for 'zopen install' too?

How about reversing the string with rev and then cutting the last -.

fixed as part of move to new framework


Merge dev to 2.0

Merge dev into feat-2.0

For some reason, the changes appear twice

bros r+

bors r+


Fix #103

Rating calculations
Fix Steem Power display
enable user to select text from comment
Fix loading skeleton in user search page
Made Resource credit data more reliable

Steem Power with some delegation:

Steem Power with no delegations:


question about train_SWLDAmatlab.m

Hello.
I am looking for matlab code about stepwise Linear Discriminant Analysis. And I find bbci_public. It's really a great tool.
But I find it that train_SWLDAmatlab.m use stepwisefit function which of Statistics and Machine Learning Toolbox.
Is stepwisefit function only fit to linear model? Or there are some references support that stepwise function is also fit to LDA?
Looking for your reply.

I do not really understand your question, since LDA is a linear model.
The function train_SWLDAmatlab.m can be used for applications of
stepwise LDA as in the BCI literature.
Best regards,
Benjamin Blankertz
On 4/13/19 8:06 PM, manipulative wrote:

Hello.
I am looking for matlab code about stepwise Linear Discriminant
Analysis. And I find bbci_public. It's really a great tool.
But I find it that train_SWLDAmatlab.m use stepwisefit function which of
Statistics and Machine Learning Toolbox.
Is stepwisefit function only fit to linear model? Or there are some
references support that stepwise function is also fit to LDA?
Looking for your reply.

--
Prof. Dr. Benjamin Blankertz
Tel. (+49 30) 314-78626
Fax. (+49 30) 314-75784
Berlin Institute of Technology
Chair for Neurotechnology
Sekr. MAR 4-3, Room 4.041
Marchstr. 23
10587 Berlin
Germany

Thanks for your reply,
I think you are right. I thought that stepwisefit could only be used in general linear model. But I found some reference yesterday say that two classes LDA is very similar to GLM, so it's appropriate to use stepwisefit in LDA.
Thanks for your reply again.


How do you add your keys?

It could be I'm a bit thick, but I can't see how to add my SSH keys to the app.
I have disabled password authentication on my servers, so need key access to work.
Love how it is so minimalist! Hope I can get it to work.

Hi! Thanks :)
Basically the app itself creates a SSH key internally. Click the little "copy button" to copy the RSA public key and add that as a known key to your server.
Hope that works!

Ah - got it. I wondered how it got a public key in the first place. Totally
makes sense!
Keep up the great work!
On Mon, 30 Aug 2021 at 14:57, Tyler @.***> wrote:

Hi! Thanks :)
Basically the app itself creates a SSH key internally. Click the little
"copy button" to copy the RSA public key and add that as a known key to
your server.
Hope that works!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tytydraco/SSH/issues/2#issuecomment-907972632, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APE7DZN5RVUUIXKPG5DFTETT7LXRRANCNFSM5DASOP3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Passing width and height to apexchart

Fixes #2

Thanks

@MaxResolution can you compile the dist when you have sometime

@MaxResolution can you compile the dist when you have sometime

done


feat: enable resize of weekday event slot

Checklist
Please put "X" in the below checkboxes that apply::

[x] If committing a bugfix, I have tested it in different browsers (Chrome, Firefox, Safari).
[x] If committing a new feature, I have first submitted an issue (Please note: you are free to open PRs for non-issued features, but opening an issue increases your chance of a successful PR). #221
[ ] If committing a new feature, I have also written an appropriate test suite for it.

I have tested the following:

[ ] Qalendar component in month mode.
[x] Qalendar component in week mode.
[x] Qalendar component in day mode.
[ ] All of the above modes on emulated mobile view.
[x] Dragging and dropping events.
[x] Resizing events in day/week modes.
[x] Clicking events to open event dialog.

This PR solves the following problem**.
insert your description.
How to test this PR**.
Implement events where isCustom is set to true, and watch your custom event slots resize when you grab the handles.

coverage: 97.603% (+0.03%) from 97.577%
when pulling 037c803e29064821c2bfd159d9e78fe949f3d070 on vicdotexe:resizableSlot
into 2515aede464929795cc2d6de4d6916fc47f7f886 on tomosterlund:master.

It appears to me, that this fixes and old problem I didn't know Qalendar had - namely text overflow from week events. At least sometimes when they're very short. That seems to be the reason the screenshot tests failed.
I can't record new linux screenshots based on this branch, because it's in your repo. Would you mind trying to trigger the record snapshots workflow in your fork, throw out the old linux folder and add the linux folder from the resulting screenshots in your fork?
If it doesn't work out please let me know, then I can open up a new branch based on yours and record the screenshots.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New Crowdin updates

Hey @MuckT , Please let us know do we need to perform testing from our side.
Thanks..!!


Consider adding flags to add/remove rules at command line

It likely some words will trigger false positives or a user has a special word correction.
It should be easy to do.

ignore rule is present, good enough for now.


Fileslist is not working with long gcode files

i forgot where i saw the issue but when the gcode file is longer then 100 character the command will fail

fixed with fed2a25980308e63067a57f06ca5539016cb3a13


safari is having trouble communicating with the backend

This is for mobile and desktop safari.

I see this in safari's devtools:
XMLHttpRequest cannot load https://liftoff-dev.appspot.com/api/compile. Request header field Content-Type is not allowed by Access-Control-Allow-Headers.

I think we need to allow OPTIONS as well.


Fix to BlobContainerClient CreateIfNotExists NullReferenceException

When calling BlobContainerClient.CreateIfNotExists, a NullReferenceException occurs when Response's implicit cast to BlobContainerInfo fails to unwrap a null response.
Instead of returning default, we can return a default BlobContainerInfo object and wrap a Response around it.
Resolves #9758

My two cents: I don't think the break here is worth it.  I understand the behavior here is super surprising and confusing (and likely not what we want long term), but I don't think making a breaking change to the API is the right more.  Given that we explicitly documented the behavior here, changing it feels like a bad idea.
At the same time, I understand the feedback that this API is confusing to use.  The natural code you want to write is likely going to be wrong (if you don't explicitly save the Response<T> and instead just use the implicit conversion).
If we feel like this API has usability issues, have we considered just adding a new CreateIfNotExistsWithResponse (I'm sure we could come up with a better name) that does the "right" thing (and I think we should think about what the right thing is,. it's not clear to me that returning a Response wrapped around default data is right) and then hide the existing APIs with EditorBrowsable(false) as well as provide links to the newer APIs in the older API documentation?
I get that it leaves a bit of a wart on our libraries - but I think it's a much better outcome, I struggle to think about any existing correctly written code that will not be broken by this change.  As a consumer, I would prefer warts (mitigated with documentation that points me to the right things, as well as by leveraging our tooling to make it harder to discover the bad APIs) vs the alternative of breaking changes, especially in releases were we don't bump the major version.

I completely understand and agree. It's best not to break other customers with what they're expecting from CreateIfNotExists. I will look to update the documents, that way current customers who are using this can get around this.
They might have to do a try-catch in order to get around this bug. Since the point of CreateIfNotExists over Create is to not have to do a try-catch if the container or blob already exists, maybe it's worth considering the CreateIfNotExistsWithResponse idea. Or else there's no point of calling CreateIfNotExists unless you don't want to
@seanmcc-msft @kasobol-msft What do you think about the new API idea? Or do you have any suggestions about what we should do about this bug?

I completely understand and agree. It's best not to break other customers with what they're expecting from CreateIfNotExists. I will look to update the documents, that way current customers who are using this can get around this.
They might have to do a try-catch in order to get around this bug. Since the point of CreateIfNotExists over Create is to not have to do a try-catch if the container or blob already exists, maybe it's worth considering the CreateIfNotExistsWithResponse idea. Or else there's no point of calling CreateIfNotExists unless you don't want to
@seanmcc-msft @kasobol-msft What do you think about the new API idea? Or do you have any suggestions about what we should do about this bug?

I'd make Response<T> null friendly.

We still see this null Exception a lot in a production (!) Microsoft service. Can you please merge this or at least tell me how to work around it?

https://github.com/Azure/azure-sdk-for-net/issues/9758

We still see this null Exception a lot in a production (!) Microsoft service. Can you please merge this or at least tell me how to work around it?

I don't think we will be able to merge this PR as this would introduce a breaking change. A workaround unfortunately would to do a try-catch around the CreateIfNotExists in which case it might be better off/the same as using regular Create and putting a try-catch around it. Unfortunately Response<T> is not null friendly, so on upon trying to get the value for Response<T> would throw.
Sorry for the inconvenience.

Well I tried to work around this problem a long time ago by doing this, but this does not help either:

Please don't close this bug, this is an actual bug no? Can we please have a better workaround, we can't just ignore this.

If I just try catch CreateIfNotExistsAsync, what happens when it does throw Exception and I catch it? I still do not understand what that means.
Our aim with this is to create the container if it does not exist, obviously. If this function throws, what does this tell me? Was the container not created, does it already exists, what?

@zmarty Apologies for the late response. Your response totally slipped under my radar. I tried to write a code snippet that would work around this issue, please let me know if it works for you in the thread or if not, if I can help with a better workaround.

@zmarty Apologies for the late response. Your response totally slipped under my radar. I tried to write a code snippet that would work around this issue, please let me know if it works for you in the thread or if not, if I can help with a better workaround.


php

Template
Please write your blade template that occurs issues.
Context (Environment)

write your environment

Description

Any other informations

Closed by #372


PropTypes added

Hi guys,
I've added PropTypes based on docs and existing code.
Should be useful for autocomplete in IDEs.
Please merge my pull request. Thank you.
Vitaly

@vburlai I prefer no strict types in PropTypes to allow patterns like this <Input mask={useMask && '99.99.99'} /> where useMask is false, undefined or null. Would you like to update your PR to support false props?

@sanniassin Updated with mask/maskChar to be bool/string and optional (undefined, null)


Ensure that response object from WPS includes HTTP header info

Ensure that response object from WPS includes HTTP header info:

Return: using names from HTTP headers: "location", "content-type", "content-length" etc.
Use standards defined here: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers
and maybe the auth token as well.

I have updated the nginx config to show the content-type for netcdf. I don't know about Content-Location. It does not show up in the headers.

We should double-check what ECMWF want us to include.


Add dashboard

One thing that can really add to the aesthetic and first impression on demo calls is a home dashboard with cards showing a few key metrics:

% of Policies blocked
Access map
Number of Gateways offline (or flapping Gateways)
Recent new users
Currently online clients

Inspiration: https://defguard.net/


fix prisma esm __dirname

Prisma internally uses __dirname which breaks the solid-start start command with solid-start-node
dist/server.js => import_path.default.join(__dirname, "../query-engine-darwin");

ReferenceError: __dirname is not defined in ES module scope

I'm not sure whether it should be fixed here. It's a problem with ESM vs CJS. Probably not only __dirname can make problems here. You can fix it like that:
https://github.com/pawelblaszczyk5/planotes/blob/4b24f1d74735856bc438ba72c4ac036e9d198a73/apps/web/vite.config.ts#L8

Yeah I think @pawelblaszczyk5's answer is probably the fix here. That __dirname is wrong for most files so its not stable at all.

It does the trick, thanks


is there a delete or remove function?

as titled

If you read the documentation, you'll find that there are remove functions on the caches.
https://docs.rs/quick_cache/0.6.9/quick_cache/sync/struct.Cache.html#method.remove
https://docs.rs/quick_cache/0.6.9/quick_cache/unsync/struct.Cache.html#method.remove


Can't use slab::Slab with mio::Token (compilation error)

Code:
struct ... {
  conns: Slab<Connection, Token>,
}

The error I get:
error: the trait `slab::Index` is not implemented for the type `mio::token::Token`

Versions: rustc 1.8.0 (db2939409 2016-04-11) slab: 0.2.0, mio: 0.5.1

Seems like this might be caused by the slab version used in mio, which is slab = "0.1.0"

Hit this too.

Thanks for posting this.  Can you provide a bit more code, or relevant Cargo.toml ?
I have verified that the tests to create Slabs with Tokens, and they compile and run on the branches  master and v0.5.x.   IOW,  I'm not sure how to repro this.

This fails to compile on my machine. The way I've fixed it, for now, is just to use slab = "0.1".
Cargo.toml
[package]
name = "test"
version = "0.1.0"

[dependencies]
mio = "0.5"
slab = "0.2"

main.rs
extern crate mio;
extern crate slab;
fn main() {
    let slab: slab::Slab<usize, mio::Token> = slab::Slab::new(1337);
}

Yeah, mio v0.5 requires slab v0.1. If you want to use slab v0.2, you can use mio master branch. Hope this helps.


Missing docs

Although cf-lager is a small wrapper, its purpose and intended usage is not entirely obvious.
As far as purpose is concerned, what are the benefits of using cf-lager over calling lager directly?
On usage, is it intended that AddFlags be called in each main program that uses cf-lager or is this better called during static initialisation (an init func)? Where should flags.Parse be called, since doing this in static initialisation tends to wreak havoc with test executable flags?
Also, is there a recommended technique for propagating log levels across process boundaries? E.g. is there a standard way of extracting the current log level and adding the appropriate flag value to the invocation of a process?
It would be great to capture some or all of this information in the README.

Hi Glyn,
We added some more documentation and an example of using the library. If you think we still need to go into more detail, please let us know. I think the questions about whether to include flags in a main function or in an init function are general go questions and may be outside the scope of the documentation for this small library.
For your question about propagating log levels across process boundaries, you can call GetMinLevel() on the ReconfigurableSink to figure out the current value of the log level. We included this in the example code we added to the README.
Thanks!
@jenspinney and @joachimvaldez

Good stuff. Thanks @jenspinney and @joachimvaldez!


Create Plugins: Fix plugin id mismatch

What this PR does / why we need it:
There was some mismatching in the pluginId created in the templates, this PR fix it.
Which issue(s) this PR fixes:

Fixes https://github.com/grafana/plugin-tools/issues/758
Special notes for your reviewer:
To test this, I create projects using the old version and the new version for all plugin types: app, datasource, panel and scenesapp (with all the options on). I compared the results, and they were the same for app, datasource and panel. For scenesapp the fixes made all the pluginIds to be the same and the export path new has the -app sufix instead of -scenesapp

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Closing PR in favor of https://github.com/grafana/plugin-tools/pull/837
@Ukochka merge this branch into hers.


Feature/fix undefineds

A few very minor bugfixes/cleanup items, and a fix for the otp transform that broke the map stuff

+1


[core] glfwGetTime very expensive

I've never really done much profiling before so I could be holding this wrong!
please see the analysis of two runs of an app one using GetTime each loop the other not
even the one not running GetTime show significant time spent with glfwGetTime
please see attached
profile-with.txt
profile-without.txt

still I would get rid of timeBeginPeriod(1);

How does timeBeginPeriod() get called on non-windows platforms? It looks like it has some if-defs that make it so that function is only used on win32

@ZaneL crap! those #ifdefs keep catching me out! still on windows this will cause power management issues
I restored Wait() and removed SetTargetFPS(60), and that stops the issue, cpu usage at normal time
I'm gonna have to go through all my code and nuke SetTargetFPS !!!
I think this issue needs mentioning somewhere in the docs...

timeBeginPeriod(1) is only called on Windows at InitTimer() to set the timer resolution, actually due to this function, raylib requires linking with winmm library on Windows. It's weird it is causing trouble, I'd like to know what it does internally to the system... it also seems that behaviour depends on the version of Windows used, for Windows 10 it behaves in a differernt way than previous versions... here the docs.
In any case, I agree it would be great to get rid of it if desired, I'll enable a config.h flag for that.

although 10 years old this is an interesting read download.microsoft.com/download/3/0/2/3027D574-C433-412A-A8B6-5E0A75D5B237/Timer-Resolution.docx

Ok, it seems this mechanism needs some review to improve CPU usage on Windows.
Just for reference, also adding link to Sleep() docs, actually, it's the reason to use timeBeginPeriod() and timeEndPeriod().

does it need to be set so low? it won't improve accuracy (according to the
docs) but it will increase power consumption which isn't ideal for laptop
users
On Wed, 10 Mar 2021 at 20:04, Ray @.***> wrote:

Ok, it seems this mechanism needs some review to improve CPU usage on
Windows.
Just for reference, also adding link to Sleep() docs
https://docs.microsoft.com/en-us/windows/win32/api/synchapi/nf-synchapi-sleep,
actually, it's the reason to use timeBeginPeriod() and timeEndPeriod().
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/raysan5/raylib/issues/1641#issuecomment-796009649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFO4SNCWPO35XPNPZUQXVTTC7GGHANCNFSM4Y6P7W3A
.

--
blog bedroomcoders.co.uk http://bedroomcoders.co.uk
Disclaimer:
By sending an email to ANY of my addresses you are agreeing that:

I am by definition, "the intended recipient"

All information in the email is mine to do with as I see fit and make
such financial profit, political mileage, or good joke as it lends itself
to. In particular, I may quote it where I please.

I may take the contents as representing the views of your company.

This overrides any disclaimer or statement of confidentiality that
may be included on your message.

@chriscamacho I added SUPPORT_WINMM_HIGHRES_TIMER flag to allow disabling it if needed. When disabling it, the Wait() timer resolution moves to the order of +10ms, consequently frame sync does not work as expected, getting framerates of around 50 fps when 60 fps are requested.
In any case, disabling WinMM and letting VSync do the sync works like a charm.

👍
On Fri, 12 Mar 2021 at 16:32, Ray @.***> wrote:

@chriscamacho https://github.com/chriscamacho I added
SUPPORT_WINMM_HIGHRES_TIMER flag to allow disabling it if needed. When
disabling it, the Wait() timer resolution moves to the order of +10ms,
consequently frame sync does not work as expected, getting framerates of
around 50 fps when 60 fps are requested.
In any case, disabling WinMM and letting VSync do the sync works like a
charm.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/raysan5/raylib/issues/1641#issuecomment-797604039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFO4SKNKFHJ5LUSQMCL43LTDI6YHANCNFSM4Y6P7W3A
.

--
blog bedroomcoders.co.uk http://bedroomcoders.co.uk
Free source code and tutorials!
Disclaimer:
By sending an email to ANY of my addresses you are agreeing that:

I am by definition, "the intended recipient"

All information in the email is mine to do with as I see fit and make
such financial profit, political mileage, or good joke as it lends itself
to. In particular, I may quote it where I please.

I may take the contents as representing the views of your company.

This overrides any disclaimer or statement of confidentiality that
may be included on your message.

Adding link to related raylib reddit conversation for completion.

Related: https://randomascii.wordpress.com/2020/10/04/windows-timer-resolution-the-great-rule-change/


Add many descriptions to the Window class

Added several missing descriptions for various functions and properties. I don't fully understand what every function of this node does yet, but I think I'm on the right track with these additions after some testing. Plus it's better than nothing. I intend to improve this wherever possible.
If you've any feedback, recommendations, and/or insights I'd be glad to hear it.

The descriptions of the properties should be changed to not start with "sets" or "returns", since properties can both be set and return. You can look through the docs to see how other other classes' properties' descriptions are written.
For example, instead of "Returns true if the window can't be resized.", you could write "If true, the window can't be resized." which is more consistent the rest of the documentation.

The descriptions of the properties should be changed to not start with "sets" or "returns", since properties can both be set and return. You can look through the docs to see how other classes' properties' descriptions are written.
For example, instead of "Returns true if the window can't be resized.", you could write "If true, the window can't be resized." which is more consistent the rest of the documentation.

Thank you! I'll see to fixing that!.

It seems you also changed the descriptions for the methods, those make sense to start with "Return". I was only talking about the classes' properties. Otherwise the change look good.

It seems you also changed the descriptions for the classes' methods, those made sense to start with "Returns". I only suggested changing the classes' properties' descriptions. Otherwise the change look good.

Thanks! I've updated the classes' methods to start "Returns" again!

Please squash your commits. You need to use a desktop Git client for that, you can't do it using GitHub on web.
https://docs.godotengine.org/en/stable/community/contributing/pr_workflow.html

Please squash your commits. You need to use a desktop Git client for that, you can't do it using GitHub on web.
https://docs.godotengine.org/en/stable/community/contributing/pr_workflow.html
You will have to do this after reviews as well, so it's a good time to practice now.

Ok, I think I've managed to do it! Thank you!

Please squash your commits.

That deadly advice :D

Please squash your commits.

That deadly advice :D

Yeah, I'll be honest, I'm not very familiar with Git, and I'm still learning a lot about it, so things got a little messy. Anyway, I've opened a new pull request that should be fine now with all the edits/additions.

Superseded by https://github.com/godotengine/godot/pull/49344.

Please squash your commits.

That deadly advice :D

They've done a couple of force-pushes completely fine! But oh well...


Project visibility

Projects visibility (private/public)
This PR adds the capability to define a project as public (open to all), or private (open only to owner).
This works with Okta's FGA, and is the foundation for adding inremental support for fine grained permissions.
Refer to our paper doc ("Multiplayer II: Fine Project! Grained Access?") for the detailed spec.
Implementation Contents

A new table was added to the database to save project's visibility status (currently PRIVATE, PUBLIC, WITH_LINK), and a matching definition was added in Prisma's schema.
Authorization model was written in Okta's FGA, to reflect the possible permissions.
Infrastructure for contacting Okta's FGA (fgaService), and a permissionsService that abstracts it.
When changing a project's visibility, we store it in our DB (for our usage), and also send a tuple to Okta that defines that user:* (which means "all users") have/doesn't have a viewer relation to the project, and the can_view permission is derived from it.
When calling our /project endpoint, a check is being made to see if there is a can_view permission for this project, and if not - a 404 page is being generated.
UI added - project visibility UI in the Projects page (menu and badge).
Tests that mock Okta's FGA service (since there is no official mocking for now)

Current Auth Model:

Example 404 behavior:

@ruggi I agree, we should defintely have a middleware layer for endpoints. It was left out of this initial PR since it only handles the "can_view" permission (and not editing/collaborating).

@ruggi I agree, we should defintely have a middleware layer for endpoints. It was left out of this initial PR since it only handles the "can_view" permission (and not editing/collaborating).

sure, but I don't see a reason for not going for the better option right now – also considering that with this change in its current form we already introduce technical debt (and a behavior discrepancy between that route and any other route used by the editor and the BFF). As this is an introductory PR it feels like a great place for stubbing a very simple middleware that we can improve upon going forward, rather than modifying a route that will need to be re-changed later :)


#2114 - Fix for escaping backslash

Includes the fix for escaping backslashes in the web parts content's #2114

Coverage remained the same at 100.0% when pulling 5ab51f2ef074c9708de8163943bcc81936e7981a on estruyf:issue/2114 into 9e15dcb14a9c6a3e6c0d50be7226deb76b3d3206 on pnp:main.

Coverage remained the same at 100.0% when pulling 5ab51f2ef074c9708de8163943bcc81936e7981a on estruyf:issue/2114 into 9e15dcb14a9c6a3e6c0d50be7226deb76b3d3206 on pnp:main.

Merged manually. Thank you! 👏

Merged manually. Thank you! 👏


remove default github templates

new organizational templates in place at .github.  Can remove the ISSUE_TEMPLATE and PR_TEMPLATE overrides so that the repository stays consistent with default.

For reference:

.github repo: https://github.com/edgexfoundry/.github/tree/main/.github
documentation: https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/creating-a-default-community-health-file#supported-file-types


form-element-min-height probrem

In sanitize.scss and sanitize.styl form-element-min-height variable or function, applied root-line-height with '%' unit, compile to raw scss code or min-height: ;. Is this behavior correct?
SCSS CodePen
https://codepen.io/m-iskw/pen/wMGLqo?editors=010#0
Stylus CodePen
https://codepen.io/m-iskw/pen/GoZbQo?editors=010

The form element’s min-height has been simplified to 1.5em and released as part of v3.1.1. This reduces some of the strange math needing to be introduced to each language.


Update README.md

What this PR does
Made one change each to the two python examples of using this library

need to import conversant in the first example
in 2nd example, "shakespeare_bot = PromptChatbot(" should be "shakespeare_bot = conversant.PromptChatbot("

How it was tested
tested by following the examples in code.
PR checklist

[ ] No API keys or other secrets committed to source?
[ ] New functionality is added alongside appropriate tests?
[ ] All source files have the following header?

# Copyright (c) {YEAR} Cohere Inc. and its affiliates.
#
# Licensed under the MIT License (the "License");
# you may not use this file except in compliance with the License.
#
# You may obtain a copy of the License in the LICENSE file at the top
# level of this repository.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Exclude expenses with creation_method set to debt_consolidation

We remove the exclusion of expenses from category id 18 to include
expenses from general category
To retain the ability of excluding "Settle all balances" transactions
created by Splitwise automatically we replace exclusion of category id 18 expenses with exclusion of expenses where creation_method is set
to debt_consolidation

Please rebase to current HEAD.
Commit b856775f41e3494bc7e4fc606252ebc642445f6f removes multi currency support for example. Ideally create multiple pull requests for the different topics covered in these commits.


setup .gitconfig consistently based on OS

Description
What:
Why:
When:
Where:
Tasks

[ ]
[ ]
[ ]
[ ]
[ ]

Acceptance Criteria

[ ]
[ ]
[ ]
[ ]
[ ] Resolve linter errors
[ ] Remove to-do, dead code, & unnecessary comments
[ ] Existing documentation is updated (readme, .md's)
[ ] New documentation needed to support the change is created
[ ] Validation test run if applicable

Constraints

N/A

References:

References #

here is a starting point for discussion

autocrlf = true should only be used on Windows

Each dev runs:
git config --global core.whitespace blank-at-eol,blank-at-eof,space-before-tab
git config --global core.autocrlf true

git config --global pull.rebase false
git config --global init.defaultbranch main

git config --global user.name your_user_name
git config --global user.email<EMAIL_ADDRESS>
Each repo contains in .git/config (I think)
[core]
  whitespace = blank-at-eol,blank-at-eof,space-before-tab
[pull]
        rebase = false
[init]
        defaultBranch = main

We should probably also add a .gitattributes where .sh line endings are always lf as they won't run otherwise. Just in case someone tries to check in a CRLF in a .sh file.

here is a starting point for discussion

autocrlf = true should only be used on Windows

Each dev runs:
git config --global core.whitespace blank-at-eol,blank-at-eof,space-before-tab
git config --global core.autocrlf true

git config --global pull.rebase false
git config --global init.defaultbranch main

git config --global user.name your_user_name
git config --global user.email<EMAIL_ADDRESS>
Each repo contains in .git/config (I think)
[core]
  whitespace = blank-at-eol,blank-at-eof,space-before-tab
[pull]
        rebase = false
[init]
        defaultBranch = main

We should probably also add a .gitattributes where .sh line endings are always lf as they won't run otherwise. Just in case someone tries to check in a CRLF in a .sh file.


support alternate gun load outs

support alternate/optional gun load outs, for example see Bristol F. 2B

armement is now an array of configurations


Add QA view to admin

toggle "show datasheet next to forms" for proofreading

customise admin change_view in templates/admin/observations/change_form.html
to include 50% width / 100 % height area with iframes for all attachments

interim solution: open (attached or native) data sheet in new tab on other screen, enter data and view data sheet side by side


allow nested JSONPs

The extension currently doesn't recognizes jsonp in the form .(
I've added the period character to the allowed characters in the function part.

Thanks! I will release a new version with this pull request applied, thanks again.

New version released


how to identify payment is success or not when payment is done

Version of react-native-iap
4.4.4
Version of react-native
0.61.5
Platforms you faced the error (IOS or Android or both?)
IOS
Tested environment (Emulator? Real Device?)
Real Device
Steps to reproduce the behavior

I use "finishTransactionIOS" method for the transaction. With particular function I am not able to get purchased item from apple so is there any mistake by me here to call function. Please find attached sources as well to check.
When open app at that using "requestPurchase" this method. With this function also I am not able to get product purchased by user. It is working fine with sandbox environment by not working with live.

note : for android it works fine.

click purchase

check purchased (componentDidMount )

Also asked in #1100

Well, the question seems pretty ambiguous. The payment is successful when it comes to purchaseUpdatedListener. You are calling finishTransactionIOS just to acknowledge the purchase so that it won't refund.


Share the bottom-up and top-down parameters

In your paper, it's said "the variational inference is designed in a bidirectional way based on bottom-up path and top-down path, while letting the inference network (left) and generative network (right) share their parameters as shown in Figure 1."
But in your code, I didn't find a place to share parameters. Can you tell me what I missed?
Thank you in advance.

It means that NN modules belonging to the top-down path are used as a part of both the encoder (variational inference) and the decoder (generative network) of the BVAE.


add some mllm features and try to incorporate Chinese-LLaVA-Med project

What does this PR do?

add processor save to merge Lora with MLLM
support full parameter fine-tuning of MLLM for zero2. (zero3 has some problems) We should set only mm_projector and lm requires_grad=True to be the same as LLaVA and should not autocast_projector_dtype. A complete case can be found at Chinese-LLaVA-Med.
incorporate a project Chinese-LLaVA-Med using the latest version of llama-factory to fine-tune a Chinese medical multimodal llm

[x] test has passed as shown in https://huggingface.co/BUAADreamer/Chinese-LLaVA-1.5-Med

Before submitting

[x] Did you read the contributor guideline?

bug fix log (only for my own

in config of llava-hf/llava-1.5-7b-hf there is no hidden_size, patch hidden_size can satisfy ds3
only qlora need autocast dtype


Geopackage job export

Geopackage temp storage
!story G15.5
!component kettle

Closed out and will be replaced with issues 144-147

Have initial communications working.  Still need to add the full message. Need to update the client UI then we can post the full job export message to the queue for processing

I will mark this as completed.  We have a geopackage export fro the tile layer.  We probably need to support reprojecting but the flow for geopackage is now working and proven


generalize script with cli options, improved header decoding/joining, message-id filepath fix

as promised in https://github.com/davisdude/mbox2html/issues/2, here are my changes - the commits were redone to be sorted by most- to least useful for you to cherry-pick, starting with the 'add cli options' and the changed 'header decoding'. Replacing slashes helped with github message-ids that are formed according to their issue threads http paths.
The bottom commits get more subjective: an option only to extract text/plain message parts and lastly to control date formatting for the index list differently than in the message view.
You kept to a style of spacing the parantheses. It wasn't deliberate to not have minded this for newly written lines, I can redo the branch if you're strict on this, it only got apparent to me when finalizing the branch push. If you prefer individual PRs for each change instead of cherry-picking, I'd happily do so for those you want to merge.
I did test against randomly assorted mailing lists, personal mails and newsletters. The header decoding really seemed improved. There are still some message-ids that can go missing that I haven't tracked down the reason for yet, but this should be as before.

Sorry this took me a while to looking into. And no worries on the formatting - plan to run it through a formatter after merging anyways.
I tested out your changes and ran into an issue on my archive, so I made a small tweak to handle unlisted email formats. It's possible that this wasn't handled properly before, either. Hopefully that's okay. If not, feel free to revert. Thanks for the contribution! :slightly_smiling_face:

your script is so useful because it has no external dependencies. For people yet unfamiliar with python/venv/pip and needing it for one-offs that must be welcome.
When you've merged, I wonder if you'd have preferred each change as separate pull request. I'm never sure when it's better to do this vs branches. Thanks for taking the time, these external contributions must always come around untimely.

In this case, I think either one would work, though I generally lean towards preferring separate PRs if changes are different enough to warrant it.


issue #124 Email is checked in lower case.

This is because email addresses are case insensitive by standard.

I've run out of time today, but I will look at this for you first thing tomorrow morning.

After this change, I am not able to save my profile. Error message: There is already a member with the same ...

Seems this was an oversight. I have pushed a fix (@rasstislav), so please let me know if you're still having problems.

@nglasl What is the fix as we need it as well.

https://github.com/silverstripe-australia/silverstripe-memberprofiles/commit/ce98c97037dd6ab1ff6e3fb4ab7386c2377febc5

@nglasl it works, thanks.

FYI: Just learnt that I can merge a specific commit with
git cherry-pick <commit hash>

In this case:
git cherry-pick ce98c97037


Scroll the download pane into view when a version is selected

Compatibility:
CSS scroll-margin: https://caniuse.com/?search=scroll-margin
JS DOMElement.scrollIntoView(..): https://caniuse.com/?search=scrollintoview
One thing I noticed is that tailwind scroll-mt-20 wasn't really doing much, not sure if it's just me (it was doing something, just not enough, not sure if I was using it wrong 🤷♂)
I was checking out the mobile download page and noticed that when you press one of the versions there's no indicator that it has actually opened, so I made it scroll into view.

I believe scroll-mt-20 only works with link scrolling (scrolling to an ID with a link)

Not the way I would have done it, although with my luck the way I would have done it probably wouldn't work. This works so LGTM


search scopes must have an :against or :associated_against option even if it's blank

This works:
pg_search_scope :full_search,
                associated_against: {},
                using: {
                  tsearch: {
                    prefix: true,
                    tsvector_column: 'search_contents'
                  }
                }

But this doesn't:
pg_search_scope :full_search,
                using: {
                  tsearch: {
                    prefix: true,
                    tsvector_column: 'search_contents'
                  }
                }

Shouldn't we be able to leave this option out if it's just going to be blank anyway?

I'll add (I work with @aviv) that the key is that we're using the tsvector_column option.

Yeah it makes sense that you wouldn't need it.
One weird thing about the way pg_search works is that you specify the columns at the top-level, but really it's each individual search feature (tsearch, trigram, dmetaphone) that actually consumes the columns array. It'd be great if we only raise if one of the features can't figure out its columns, and the top-level array was more of a fallback default if you don't specify it in the feature-level hash.


Implement Eq and Hash for Symbol.

Hello,
This PR adds implement Eq and Hash for Symbol.
If it is the same RString, it is the same Value, so Symbol can be implemented.

Merged.  Thanks for this!


Remove jest warning regardng update to SimpleImage

Is your feature request related to a problem? Please describe.
Since we changed to the svg transformer the native images in our tests aren't mocked anymore.

Stacktrace
  console.error
    Warning: An update to SimpleImage inside a test was not wrapped in act(...).
When testing, code that causes React state updates should be wrapped into act(...):

act(() => {
  /* fire events that update state */
});
/* assert on the output */

This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
    at SimpleImage (/Users/digitalfabrik/projects/integreat-app/native/src/components/SimpleImage.tsx:28:3)
    at StyledNativeComponent (/Users/digitalfabrik/projects/integreat-app/native/node_modules/styled-components/native/dist/styled-components.native.cjs.js:5949:29)
    at Styled(SimpleImage)
    at View
    at Component (/Users/digitalfabrik/projects/integreat-app/native/node_modules/react-native/jest/mockComponent.js:28:18)
    at StyledNativeComponent (/Users/digitalfabrik/projects/integreat-app/native/node_modules/styled-components/native/dist/styled-components.native.cjs.js:5949:29)
    at Styled(View)
    at PoiDetails (/Users/digitalfabrik/projects/integreat-app/native/src/components/PoiDetails.tsx:54:23)
    at useContext (/Users/digitalfabrik/projects/integreat-app/native/node_modules/styled-components/src/models/ThemeProvider.js:48:22)
    at wrapWithTheme (/Users/digitalfabrik/projects/integreat-app/native/src/testing/wrapWithTheme.tsx:6:26)

Describe the solution you'd like
Mock SimpleImage where the error occurs or wrap in act where needed.
Describe alternatives you've considered
Mock svgs in tests. But I'm unsure whether this actually fixes the issue.
Additional context
N/A

I think we only use simpleimage for non-svgs? Not sure though. Possibly related to #1882 and #2378.

Maybe. I'll take a look at this, as soon I've got some time on my hands. Don't like it if tests are 🔴red


test build not for review

Pull Request Test Coverage Report for Build 562

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

Totals

Change from base Build 561:
0.02%

Covered Lines:
8792

Relevant Lines:
11675

💛  - Coveralls


Change the percentage log file name

As @mitchellkrogza  reported here it may be better to rename the percentage log file name to percentage.txt

I fixed it for now on my Travis build by just adding sudo cp $TRAVIS_BUILD_DIR/.dev-tools/_funceble/output/logs/percentage/_* $TRAVIS_BUILD_DIR/.dev-tools/_funceble/percentage.txt


Examples needed

We need at least a good simple and advanced example.
Simple will include just a basic server with static store.
Advanced example might include something like data retrieving from the server via a sqlite db or json file.

I've added a first simple example. Suggestions are welcome ;)!


Exception Details

"Otherwise … this method fails and throws an exception."
Need details about what exceptions are thrown when.

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

ID: ee5f9867-2800-7596-24d1-3e46069c352f
Version Independent ID: af51891d-f411-fea2-6b73-8d40e72abbf5
Content: StorageFile.MoveAsync Method (Windows.Storage) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.storage.storagefile.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: knicholasa
Microsoft Alias: nichola

What he said.

Thanks for submitting your feedback and we apologize for the delayed response. I see that updates have not been made since the creation of this Issue, so I went ahead and created an internal work item to investigate.
I created the exception by getting a list of folders grouped by type:
var queryOptions = new QueryOptions(CommonFolderQuery.GroupByType);
var query = KnownFolders.PicturesLibrary.CreateFolderQueryWithOptions(queryOptions);
IReadOnlyList<StorageFolder> folders = await query.GetFoldersAsync();

When I try to do StorageFile.MoveAsync into one of these folders, which isn't a physical folder on disc, I get a System.Exception with this message:
Value does not fall within the expected range.
It's not a very helpful message, but I can update the docs for this page to let developers know what to expect from the exception.
I will update and close this Issue after the work item is complete to update the docs.

The internal work item to update this page with exception details has been completed. The changes should be live soon. Thanks for your suggestion!


feat: make el.closest() support IE8+

see
https://developer.mozilla.org/en-US/docs/Web/API/Element/matches#Polyfill

Thanks, but we only support IE10+


Right to left

Any way for us to switch the placement of where the server list starts to the right side instead of the left.

It's finally added!
Either redownload the theme or add this variable: --HSL-server-direction: column-reverse;


Usage with nextjs

Question
How to use this library with nextjs api routes and getserversideprops?
Other details
I was trying to use the search function but get the following error while using with nextjs

Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

Could you show your code? I'm using it with next.js too and it is working fine

I don't have the original code but I used it like this.
stackblitz demo


Error: "Update your device to use Messages for web"

I updated to v0.9.1 and ended up deauthorizing my computer, and when I signed back in, I got a message reading:
Update your device to use Messages for web
To keep using Messages for web, you'll need to update your Phone app
I tried rolling back to v0.8.0, which I've found to be very stable, but the issue persisted. It is true that I am not using the most up-to-date version of Android Messages on my phone (I have an older version in order to retain use of the widget), HOWEVER when I go to Messages for Web in my browser, it still connects fine. I'm not sure what it is about the desktop app that is preventing it from connecting to my phone.

I updated to v0.9.1 and ended up deauthorizing my computer, and when I signed back in, I got a message reading:
Update your device to use Messages for web To keep using Messages for web, you'll need to update your Phone app
I tried rolling back to v0.8.0, which I've found to be very stable, but the issue persisted. It is true that I am not using the most up-to-date version of Android Messages on my phone (I have an older version in order to retain use of the widget), HOWEVER when I go to Messages for Web in my browser, it still connects fine. I'm not sure what it is about the desktop app that is preventing it from connecting to my phone.

Use V1.0.0 and when you scan the qr code make sure to flip the remember this computer toggle.

@elijahtlang @rvcjew This sounds like Google may be enforcing a minimum version of the Android app for new sign-ins.
If you try to sign in to Messages in an incognito window (or a different browser that you're not already signed in with), do you get the same message telling you to update the app on your phone?

@chrisknepper thanks for the quick response. I tried updating to v1.0.0 with no change in results. I did as you suggested, and new sign-ins from a browser still work fine. It's only in this desktop app that I'm encountering the issue.

@chrisknepper thanks for the quick response. I tried updating to v1.0.0 with no change in results. I did as you suggested, and new sign-ins from a browser still work fine. It's only in this desktop app that I'm encountering the issue.


docs: clarify ExternalServer DNS resolution behavior

From reading the source code of workerd and KJ, I can see that ExternalServer does DNS resolution only once ever. This was certainly surprising to me, and I suspect it would be a surprise to many, as ExternalServer is the seemingly most natural way to get a binding to a particular networked service, but it is in fact only suitable to look up domain names that are guaranteed to not change over the lifetime of the workerd service on which it is configured, like container sidecars.
So, be more explicit about the behavior in the documentation.

I think this is a bug. If we document it, we should say that it's a bug that may be fixed later. But maybe we should just fix it?

You added the TODO two years ago in https://github.com/cloudflare/workerd/commit/dc27ddcb80f29a2da23ad370f138a2b5b21ae585. So it doesn’t seem to be a bug as much as consciously acknowledged missing functionality. And I don’t think that commit regressed DNS resolution in any way, I think ExternalServer has always been this way.
Either way, I have got what it takes to clarify the documentation but not, I think, to add this functionality. So if you’d prefer the latter to the exclusion of the former, go ahead and close this PR.


Long code blocks cut off until focus in

Bug Report
When you put a component inside of a <Playground /> that extends past 11~ lines it cuts the bottom of the text off, but still presents a scrollbar. However, scrolling scrolls up the non-visible code instead of scrolling the code into view.
To Reproduce

Include a <Playground /> element in an mdx file. The component(s) need to be longer than 11 lines (in my case) for it to be cut off.
View the page in the browser and the code should be cut off.
Click inside of the Playground and the code should then appear.

Expected behavior
Code is visible at all times without requiring the container to gain focus.
Environment

OS: Windows 10
Node/npm version: Node 10.11.0/npm 6.4.1

Additional context/Screenshots
Below is the bug in action. I see the same result in both Chrome and Firefox.

Hey @Jared-Dev this still happening in the newest versions?

Pretty awesome @Jared-Dev 🙏

@pedronauck I stashed the changes I was working on for our internal components and updated to the latest version of docz. I am still having the same issue as the image above. I did for safe measures delete all the site data and do a hard refresh to make sure I didn't have a caching issue going on.

I am having the same problem and on the same line 11.
I have not done anything other than what was shown in docz documentation and I am using the latest version of docz.
online demo: https://httpiago.github.io/ios-theme-toolkit/#/components/Button

Launched on v0.13.0 ✅


Delete unused codes

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

It seems to be duplicated. 🙇
https://github.com/pipe-cd/pipecd/pull/3147#issuecomment-1023819321

@knanao Thank you. I reverted it.

Nice catch!
/approve


Can this library write/upload media file like image, video?

Hello!
I want to implement that user's local file or fetched file upload to application's local storage like android's scoped storage.
But I don't know well about upload to local storage.
I understand this library's documentation explanation like this library can only upload .txt file.
I want to know my understanding is right.
Can I upload media file to local storage like image, video file?
Thank you!

On Android, the following should copy an image to the system image storage.
// Produce an image file somehow, storing here:
const source = Dirs.CacheDir + '/img.jpg';

await FileSystem.cpExternal(source, 'img.jpg', 'images');

If you want more control over the destination, you could do something like:
import * as ScopedStorage from 'react-native-scoped-storage';

const targetDir = await ScopedStorage.openDocumentTree();
const target = AndroidScoped.appendPath(targetDir, 'img.jpg');
await FileSystem.cp(source, target);

Thank you for your comment!
Is this process also working right on IOS?

Thank you very much!!


add deployOnOCP field to the charts to support deploy on aks

Description
add deployOnOCP field to the charts to support deploy on aks.
on aks case only need cluster-manager, foundation, hypershift components.
Related Issue
https://issues.redhat.com/browse/ACM-11040
Changes Made
Provide a clear and concise overview of the changes made in this pull request.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/assign @rokej
/assign @dislbenn

/retest

/lgtm


make swagger know about auth

Should make /ping pingable in swagger dev with no oauth, and give you an oauth button for the others

I guess that's because you're running at localhost... not sure what we do for Rawls here. I think there's some Google-config shenanigans involved.

Yeah ... the real solution there is to make the Local environment work.  Here's my initial stab at it: https://github.com/broadinstitute/firecloud-develop/pull/618
In the meantime yes this is an improvement.

Oh awesome @jmthibault79 . Thanks!


TN-2274 CLI to merge duplicate users as described in TN-2240

Robust, careful process won't allow merge unless several criteria are met.  Asks admin user to verify each step of the process.

Hello @pbugni! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file manage.py:

Line 482:1: W293 blank line contains whitespace
Line 499:80: E501 line too long (97 > 79 characters)
Line 500:80: E501 line too long (84 > 79 characters)
Line 522:80: E501 line too long (82 > 79 characters)
Line 523:80: E501 line too long (80 > 79 characters)
Line 524:80: E501 line too long (83 > 79 characters)

In the file portal/models/user.py:

Line 1476:20: E131 continuation line unaligned for hanging indent


Orientation of controller vs hmd after sync

After sync position and orientation of hmd and controllers is synced. Orientation is not always same so it can be that if u move in one direction controllers movement is skewed to hmd. You have to try few times with rotation of the controller to get both close.  Can we just set controllers orientation same as where hmd is pointing to avoid the problem?

The problem is that we have two completely independent tracking systems. In order to align them we need a correspondence between a rotation and translation in steamvr tracking space  and a rotation and translation in the PSMove tracking space:

When you press start and select we are correlating the position/rotation of the HMD with the position/rotation of the psmove controller.
Now all that said, if we KNOW that the PSMove tracking space has the same rotation as the SteamVR tracking space then we could do what you propose. This might be the case with Rift cat tracking setups. Not sure. I could see adding a toggle in the steamvr.settings that sets the alignment mode (rotation/translation or translation only).

could we set same direction of the controller as hmd and leave its rotation?
cause we play with rotation now and perhaps it is not necessary.

I mean if orientation consists of direction and spin then we know that they both have good spin and we need only calibrate direction element of orientation.
Right?

@HipsterSloth
so basically each time when I run riftcat they are alighed in direction and spin with hmd as you can see (i am looking down directly). all I need is to change position.
I know more or less where this happens however there quite a few computation which is not easy to decode for me. I appreciate you can advise how to make this change.
thanks

I think we could just add a "ignore_alignment_rotation" flag that we check in CPSMoveControllerLatest::FinishRealignHMDTrackingSpace.
If that flag is true then:
-we set hmd_pose_meters.Orientention to PSMoveQuaternion::identity() here:
https://github.com/cboulay/PSMoveService/blob/master/src/openvr_plugin/driver_psmoveservice.cpp#L2336
And we set controller_pose_meters to PSMoveQuaternion::identity() here:
https://github.com/cboulay/PSMoveService/blob/master/src/openvr_plugin/driver_psmoveservice.cpp#L2385
This would assume that both the PSMove controller and the HMD were pointing down -Z in the SteamVR tracking space.

do u mean to PSMovePose::identity()
https://github.com/cboulay/PSMoveService/blob/master/src/openvr_plugin/driver_psmoveservice.cpp#L2385
as result it is still synced for position and orientation in totally different place.
i would be happy to have orientation ignored for the start. i can try with these code to search for solution hoever if you have further suggestion it would be grt.

No I meant PSMoveQuaternion::identity(). Basically I wanted to remove the rotation from the HMD Pose and the controller pose. That might not be the right thing to do. That alignment code is super confusing even to me. I'll try to take a look tonight and see if I can add that flag. I can't really look at that code right now because I'm at work.

I added a the following new flag in the steamvr.vrsettings:

"psmove_settings" : {
"use_orientation_in_alignment" : false
},

This will assume that the controller is aligned straight upright facing -Z rather than using the existing controller orientation (which could be slightly off). Let me know if this works for you.
https://github.com/cboulay/PSMoveService/releases/tag/v0.9-alpha6.0.10

@HipsterSloth
thanks for adding this. but I must say that I was very stupid this time. basically the directions were similat because direction of psmove area is inalignment with direction of game area in my case :) . but it does not mean they are lighted in some way... sorry for this.
so basically I must carefully try original method and go to different part of game arena to see if direction is in synch on the proper level. after some practice it goes very well. the problem is that even small rotation of controler leads to big difference in direction. i think matematically its
sin(error angle) * distance
i have proposal for alternative synch mechanism to avoid this error but there is no rush to do it.
basically you need capture two points. when first point is taken you can align location of controller and hmd. then you go like one or two meters in other direction and capture point. this gives you precisely the angle. the only assumption here is that you have ability to keep controller in same location on hmd in two points which is my case. so basicall you are sure that in two points hmd and controller are relatively same to each other.
i wonder if I have some time and implement this then would you be able to help me finish 3d calculations? i think it can be bring to actual calculations.
thanks.


Flutter doctor fails when Flutter SDK install path contains spaces

Steps to Reproduce

Install the SDK in a path containing spaces. e.g., /Users/someone/Google Drive/Development/dart/flutter-sdk.
Run flutter doctor

$ flutter doctor
...
/Users/someone/Google Drive/Development/dart/flutter-sdk/bin/flutter: line 50: $PROG_NAME: ambiguous redirect

flutter doctor is now working. What's still left to do:

Add quotes around the xcode_backend.sh path in the two build phases we call it in the project templates + all examples.
A ton of quoting in xcode_backend.sh itself.
flutter run on both Android and iOS
probably more :)

xcode_backend.sh is cleaned up and flutter build ios now works.
iOS Xcode project templates updated.

There's also a bug in the engine (specifically in the Tonic loader) wherein file:// URIs are not URI-decoded during conversion to filesystem paths. I'll put together a patch and send upstream.

Note: dart-lang/sdk#30059 is a blocker for this.

@bkonyi You did this right?

Verified we handle spaces in SDK path, project path, and Android SDK path for:
[x] macOS: Android apps
[ ] macOS: iOS apps
[ ] Linux: Android apps
[ ] Windows: Android apps

Also verified accents and emoji in paths on macOS.

Follow-up work in progress:

[ ] Update the bots to include a space in the SDK path
[ ] Remove the doctor warning about spaces

One more patch for flutter_tester landing in https://fuchsia-review.googlesource.com/c/topaz/+/121856

This is now resolved. Travis/Appveyor are updated to run all tests in a flutter SDK directory named "flutter sdk".

As an added bonus, we now also support any path that's a valid UTF-8 string.

This patch had a minor regressive impact on the aot_snapshot size metrics.

Please disregard my most recent comment.


Does not start on MacOS arm64

$ ft -h
Killed: 9
$ uname -a
Darwin MBP.local 23.5.0 Darwin Kernel Version 23.5.0: Wed May  1 20:12:58 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T6000 arm64

Happy to help test different binaries if there's a way to make it work

Thanks for the info Ahmed!
I'm not very familiar with Macs. What is the model of the laptop? What version of OSX are you running?

It's an Apple M1 pro macbook running MacOS 14.5 (23F79) (latest)
Update: I could get it to run. This was needed
$ xattr -d com.apple.quarantine ~/Downloads/ft  # Remove attribute
$ codesign --force --deep --sign - ~/Downloads/ft  # Sign the binary (this will only work on my machine)

I think you can use GitHub actions runners for MacOS to verify the binary can at least run there.
It still doesn't seem to work very well. For example, it shows two mismatching version numbers 2.1 and <IP_ADDRESS> and when invoked with -h to just show the help message, it should do that and exit. Instead it does not exit, it just waits there, and I have to kill it via Ctrl+C
$ ~/Downloads/ft -h
8/9/2024 12:40:29 PM: File Tunnel 2.1.0
ft <IP_ADDRESS>
Copyright ©  2023

ERROR(S):
  Option 'h' is unknown.

  --tcp-listen        Listen for TCP connections. Example --tcp-listen <IP_ADDRESS>:11000

  --tcp-connect       Connect to a TCP server. Example --tcp-connect <IP_ADDRESS>:22

  --read-duration     The duration (in milliseconds) to read data from a TCP connection. Larger values increase throughput (by reducing the number of small writes to file), whereas smaller values improve responsiveness.

  --udp-listen        A local address on which to listen for UDP data. Example --udp-listen <IP_ADDRESS>:11000

  --udp-send-to       Forwards data to a UDP endpoint. Example --udp-send-to <IP_ADDRESS>:12000

  --udp-send-from     A local address which UDP data will be sent from. Example --udp-send-from <IP_ADDRESS>:11000

  -w, --write         Where to write data to. Example: --write "\\nas\share\1.dat"

  -r, --read          Where to read data from. Example: --read "\\nas\share\2.dat"

  -p, --purge-size    The size (in bytes) at which the file should be emptied and started anew. Setting this to 0 disables purging, and the file will grow indefinitely.

  --tunnel-timeout    The duration (in milliseconds) to wait for responses from the counterpart. If this timeout is reached, the tunnel is considered offline and TCP connections will be closed at this point.

  -v, --version       Print the version and exit.

  --help              Display this help screen.

  --version           Display version information.

^C

Awesome info, thanks Ahmed. I didn't know about GitHub actions runners - I'll check it out :)

Update this works very well on macos. I've established tunnels and pushed data through it, works beautifully. It would still be great to figure out this codesign'ing thing so that users don't have to do it.
The slightly weird behaviours I had noticed (two version numbers, the cli not exiting after -h) also happen on Windows, so it seems to be expected behaviour. Thanks for such a great tool!

That's great! And thanks so much :)

It turns out that this is a known issue with C#: https://github.com/dotnet/sdk/issues/34917

Interesting! Thanks @DUOLabs333


Fix FIPS coding in PUDL

Is your feature request related to a problem? Please describe.
In #3531 we've learned that the addfips package we rely on to map state/county data into a FIPS code is irregularly maintained. This is causing problems for downstream users, and we should consider deprecating this package and moving to a different solution.
Describe the solution you'd like
There are a few different use cases for FIPS encoding that we could hypothetically imagine for PUDL.
Right now, add_fips_ids in pudl.src.helpers.py is the only place we import the addfips package. This method is expects a county and state name, and a FIPS vintage:
def add_fips_ids(
    df: pd.DataFrame,
    state_col: str = "state",
    county_col: str = "county",
    vintage: int = 2015,
) -> pd.DataFrame:
    """Add State and County FIPS IDs to a dataframe.

    To just add State FIPS IDs, make county_col = None.
    """
    # force the columns to be the nullable string types so we have a consistent
    # null value to filter out before feeding to addfips
    df = df.astype({state_col: pd.StringDtype()})
    if county_col:
        df = df.astype({county_col: pd.StringDtype()})
    af = addfips.AddFIPS(vintage=vintage)
    # Lookup the state and county FIPS IDs and add them to the dataframe:
    df["state_id_fips"] = df.apply(
        lambda x: (
            af.get_state_fips(state=x[state_col]) if pd.notnull(x[state_col]) else pd.NA
        ),
        axis=1,
    )

    # force the code columns to be nullable strings - the leading zeros are
    # important
    df = df.astype({"state_id_fips": pd.StringDtype()})

    logger.info(
        f"Assigned state FIPS codes for "
        f"{len(df[df.state_id_fips.notnull()])/len(df):.2%} of records."
    )
    if county_col:
        df["county_id_fips"] = df.apply(
            lambda x: (
                af.get_county_fips(state=x[state_col], county=x[county_col])
                if pd.notnull(x[county_col]) and pd.notnull(x[state_col])
                else pd.NA
            ),
            axis=1,
        )
        # force the code columns to be nullable strings - the leading zeros are
        # important
        df = df.astype({"county_id_fips": pd.StringDtype()})
        logger.info(
            f"Assigned county FIPS codes for "
            f"{len(df[df.county_id_fips.notnull()])/len(df):.2%} of records."
        )
    return df

Approaches to resolving the problem
County and state names -> FIPS Code
This is our current FIPS encoding use case. We could:

create a fork of addfips, and implement the fixes we want to see that solve the problem identified in #3531. We'd relatively easily benefit from any upstream changes, and if the project returns to being more actively maintained it wouldn't be particularly complex to switch back.
write our own method to convert state/county codes to FIPS codes drawing on the addfips methodology. As is noted in #338, the fuzzy matching implemented in addfips is definitely of relevance to us, so we shouldn't think we can just pull down a CSV and merge county and state names without losing records.
explore a maximalist version, which involves encoding FIPS from addresses or lat/lon coordinates rather than county/state. However, this function is currently used for EIA 860, 861 and 923 data, and at least 861 and 923 data don't have complete addresses or lat/lon coordinates, so this likely won't meet our needs. The pygris package seems like a great API-free option for implementing this functionality down the line, though!

Address -> Lat lon -> FIPS Code or lat/lon -> FIPS code
As mentioned above, this wouldn't address the use case for 2/3 EIA datasets we're currently using the addfips package for. See  pygris package, which seems interesting if we want to explore this further, but this is out of scope for the actual problem we're trying to resolve. See also #3531 for some discussion on other ways to implement this.
Proposed solutions

Fork addfips and implement our desired changes, then do our own release so we can import this as a package.
Pros: We don't have to write much code, and we can keep up to date with any changes made in addfips.
Cons: Extremely annoying and we're likely to forget about this. We're still relying on a largely unmaintained package.

Recreate the logic of addfips in pudl.helpers.py using Census data directly (e.g., this 2023 data or by calling the Census's geometry API, see [here](https://www.census.gov/data/developers/data-sets/geo-info.html) for more info).
Pros: We are our own maintainers.
Cons: We have to recreate the fuzzy-matching logic in the package we're currently using to prevent performance loss.

Find some other new Python package to meet our needs.

us is maintained but doesn't yet support counties despite an open PR on this topic from 2017: https://github.com/unitedstates/python-us/. The census package depends on it, so there's no county support there either.
I have yet to find another comparable alternative

The only fuzzy matching addfips does is to replace 12 diacritics (like "é") and standardize 3 abbreviations like "Saint"/"St.". The code is like 10 lines. Your input data has to have perfect quality modulo those two replacements or the matches will fail.
The cardinal sin of addfips is to not maintain the source census data or handle the slight changes over time, which is the only thing that needs maintenance! But I'll stop there because I think I've written more lines complaining about addfips than there are lines of code in the entire package 😂


Bug: Can not get caption after Image add caption

Lexical version:
Steps To Reproduce

Pull repository lexical
yarn dev
Go to \lexical\packages\lexical-playground\src\Editor.tsx
After useEffect add code as below:

  function handleChange(editor: LexicalEditor) {
    editor.update(() => {
      const rawHTML = $generateHtmlFromNodes(editor, null);
      console.log(rawHTML);
    });
  }

Add Plusgin as below after

<OnChangePlugin onChange={(_, editor) => handleChange(editor)} />

The current behavior

After insert image we click to add caption
After add caption and get the html. Caption doesn't include html

The expected behavior

In image must have caption text include

You have to teach Lexical how to serialize and deserialize to/from HTML:
https://lexical.dev/docs/concepts/serialization#html

Sorry, I don't know how to export exportDOM for caption after clone https://github.com/facebook/lexical and run yarn dev


The comparison query

Hello,
In the paper, all the domain adaptation and the cycler2r methods use RAWi and RGBb to train.
Then, the detectors are trained using the simRAWb generated by the cycler2r or other da methods.
And the testing dataset is the RAWi again.
My question is that whether you try other RAW data which doesn't appeared in the cycleR2R training process so that the performance still surpass the rgb base line.
So there is a doupt that the reason it surpasses rgb baseline is the style of simRAWb is the same as the RAWi, which means the detector has seen the style of testing data in advance.

Hi,

My question is that whether you try other RAW data which doesn't appeared in the cycleR2R training process so that the performance still surpass the rgb base line.

If I've understood your question correctly, you're inquiring about the potential of training the CycleR2R model on RAW data from a different camera (RAW_{other cam}) and RGB_b, followed by training a detector on the generated simRAW_{other cam}, and ultimately testing it on simRAW_i. The answer is no, we have not conducted such tests. Our primary objective is to utilize large-scale labeled RGB data and unlabeled RAW data from the target camera (RAW_{target cam}) to train a detector specifically for the target camera's RAW images.

So there is a doupt that the reason it surpasses rgb baseline is the style of simRAWb is the same as the RAWi, which means the detector has seen the style of testing data in advance.

To partially address your point, yes, one reason our approach might surpass baseline models is that simRAW_i closely resembles RAW_i more than the RGB baselines do. This observation supports the hypothesis that the similarity in style between simRAW_i and RAW_i contributes to our model's performance advantage. Additionally, as detailed in our supplementary materials, testing on different Image Signal Processor (ISP) outputs of RGB can result in significantly varied accuracy rates, due to the stylistic disparities introduced by different ISPs.