Add block status input arg

Works towards: https://github.com/onflow/flow/issues/392
Description
One possible solution for #392

For contributor use:

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

agreed with @janezpodhostnik, maybe add an enum?

Is this PR still in progress @Kay-Zee?

not sure if this is still the solution we'd like to go with for solving #392
Any objections here?


sql/*: implement rule for column resolution

Added rule that qualifies all columns resolving aliases.
Changed the transform method signatures so they may return an error.
Added support for qualified column names.

NOTE: even if this resolves the columns and annotates them with the table they belong to, getting the correct value is still TODO. That is, if you join two tables with the same column name, there is no way to get the last one. This only adds the info to the GetFields so they may be pushed down.

Updated to receive sessions


pydantic.v1.error_wrappers isn’t present in Pydantic 1.10.15

Initial Checks

[X] I have searched GitHub for a duplicate issue and I'm sure this is something new
[X] I have searched Google & StackOverflow for a solution and couldn't find anything
[X] I have read and followed the docs and still think this is a bug
[X] I am confident that the issue is with pydantic (not my code, or another library in the ecosystem like FastAPI or mypy)

Description
We are facing several test failures while using pydantic v=1.10.15 with  AttributeError: module 'pydantic.v1' has no attribute 'error_wrappers' :
Current :
============================================================================ short test summary info =============================================================================
FAILED tests/model/test_resource_validator.py::TestResourceValidatorFailure::test_connector_with_empty_properties - AttributeError: module 'pydantic.v1' has no attribute 'error_wrappers'
FAILED tests/model/test_resource_validator.py::TestResourceValidatorFailure::test_connector_with_invalid_permission - AttributeError: module 'pydantic.v1' has no attribute 'error_wrappers'
FAILED tests/model/test_resource_validator.py::TestResourceValidatorFailure::test_connector_with_invalid_permission_type - AttributeError: module 'pydantic.v1' has no attribute 'error_wrappers'
FAILED tests/model/test_resource_validator.py::TestResourceValidatorFailure::test_connector_without_source - AttributeError: module 'pydantic.v1' has no attribute 'error_wrappers'
FAILED tests/translator/test_translator.py::test_transform_invalid_document[error_graphqlapi] - AttributeError: module 'pydantic.v1' has no attribute 'error_wrappers'
=============================================================== 5 failed, 3967 passed, 5 rerun in 75.20s (0:01:15) ===============================================================
make: **
Expected:
Class 'error_wrappers' should be imported in pydantic.v1 version 1.10.15 so that this runs as expected in version 1.10.14
TOTAL                                                                  9365    267   3768    224    96%
Required test coverage of 95% reached. Total coverage: 95.83%
Line of code where this error type check is defined in our package:
https://github.com/aws/serverless-application-model/blob/develop/samtranslator/model/__init__.py#L348
Example Code
Line of code where this error type check is defined in our package:
https://github.com/aws/serverless-application-model/blob/develop/samtranslator/model/__init__.py#L348

Import defined here
https://github.com/aws/serverless-application-model/blob/44562dc000b991b99c80798e4a35ff9fbe8397d9/samtranslator/compat.py#L1-L6

    def validate_properties_and_return_model(self, cls: Type[RT]) -> RT:
        """
        Given a resource properties, return a typed object from the definitions of SAM schema model

        param:
            resource_properties: properties from input template
            cls: schema models
        """
        try:
            return cls.parse_obj(self._generate_resource_dict()["Properties"])
        except pydantic.error_wrappers.ValidationError as e:
            error_properties: str = ""
            with suppress(KeyError):
                error_properties = ".".join(str(x) for x in e.errors()[0]["loc"])
            raise InvalidResourceException(self.logical_id, f"Property '{error_properties}' is invalid.") from e

Steps to reproduce:
$ gh repo clone aws/serverless-application-model
$ cd serverless-application-model
$ python3 -m venv _e
$ . _e/bin/activate
(_e) $ pip install -e '.[dev]'
(_e) $ pip install pydantic==1.10.15
(_e) $ make test

### Python, Pydantic & OS Version

```Text
(_e) shikagg@bcd07410a1df serverless-application-model % python -c "import pydantic.utils; print(pydantic.utils.version_info())"
             pydantic version: 1.10.15
            pydantic compiled: False
                 install path: /Users/shikagg/serverless-application-model/_e/lib/python3.12/site-packages/pydantic
               python version: 3.12.3 (main, Apr  9 2024, 08:09:14) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
                     platform: macOS-14.3.1-arm64-arm-64bit
     optional deps. installed: ['typing-extensions']

Affected Components

[X] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[X] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Thanks @shikha372 for reporting this issue.
I think the problem has already been addressed in pydantc 1.10.X-fixes. this is the branch that we release the 1.10.x from it.
Please test the branch and confirm the fix. Then we will release 1.10.16 soon.

@hramezani,
Thanks for looking into this! I agree, that PR should have fixed this issue, and we'll release said fix soon!


feat: added a support for multipart form data content type in execute action request

Description

Changed the headers for the Execute Action request for passing data as multipart/form-data and receiving data back as application/json.
Changed the structure of passing the data to the Execute Action request by converting the data to FormData objects rather than key value pairs

Fixes # (issue)
#9395
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Added actions by creating API's, query and passing different params from widget to perform execute action request.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: feat/binary-data-request

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
app/client/src/sagas/ActionExecution/PluginActionSaga.ts
17.67 (0.07)
2.75 (0.02)
10.53 (0)
20.19 (0)

The changes are cherry-picked in PR 9473, so closing this PR.


Updated LIS to read ASCAT Metop-B and C from SMOPS V4

Update LIS DA obs reader to read ASCAT Metop-B and C from SMOPS V4
NOAA-NESDIS is planning to retire the legacy SMOPS product, a change that will directly impact the LIS DA system. Currently, the LIS DA system assimilates SMOPS data using the following naming convention: "NPR_SMOPS_CMAP_Dyyyymmddhh.gr2". However, the new files will adopt a different naming convention, illustrated as follows: NPR-SMOPS-CMAP-6hrly_v4r0_blend_syyyymmddhh00000_eyyyymmddxxxxxxx_cyyyymmddxxxxxxx.grib2.
Steps:
To accommodate these changes, we need to update the LIS code to enhance the capability of the DA system to:
1-  read the new file names. Additionally,
2- update the GRIB ID based on the latest version of the GRIB ID table
3- add modules to read and assimilate ASCAT-Metop-C.
Resolves #1535
Testcase
Will provided later by Ehsan Jalilvand.

Hi @LIS-navari and @ejalilva:
I have successfully compiled this code with older Intel compilers (comp/intel/2021.4.0, for the Cascade Lake nodes), and the latest Intel compilers (comp/intel/2023.2.1, for the Milan nodes).  I am awaiting a test case from @ejalilva before proceeding further.
Code review reveals several concerns:
(1) It looks like the new code released the GRIB message before checking for ASCAT-MetOp-C data, which will probably result in a runtime error.
(2) It doesn't look like the arrays are initialized before being used.  (This also appears to be the case with the older code.)  I'll test this once the usecase is available.
(3) The start date for SMOPS V4 is hardwired as 24 Apr 2024.  I assume that is sufficient for testing, but we will likely need to change that to account for actual availability of SMOPS V4 data in the Air Force archive.
(4) There are minor tweaks to be made for some of the messages ("[MSG] should read [INFO]").  (This also exists in the older code.)
Finally, this pull request points to master.  It is my understanding that this code change may be included in the upcoming LIS 7.5 patch release (with FOC changes and GALWEM radiation changes).  If so, we will have to change this pull request to point to the proper destination branch.  (We can do that after we're happy with the code, it should be straightforward.)

This pull request is contained in PR #1544.  I will mark this as DO NOT MERGE.
After we finish PR #1544, I will close this and PR #1536.


[#1608] refactor: Reuse ShuffleManagerClient in ShuffleReader and ShuffleWriter

What changes were proposed in this pull request?
(Please outline the changes and how this PR fixes the issue.)
Why are the changes needed?

The createShuffleManagerClient function is defined in both RssShuffleWriter and RssFetchFailedIterator.
The ShuffleManagerClient is created in RssShuffleManager when shuffleManagerRpcServiceEnabled is true. We can reuse the client in RssShuffleReader and RssShuffleWriter.

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UTs.

@zuston @advancedxy PTAL

Codecov Report
Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 53.28%. Comparing base (f8e4329) to head (35e9a56).
Report is 2 commits behind head on master.

Files
Patch %
Lines

...e/spark/shuffle/reader/RssFetchFailedIterator.java
0.00%
14 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #1838       +/-   ##
=============================================
+ Coverage          0   53.28%   +53.28%
- Complexity        0     2994     +2994
=============================================
  Files             0      446      +446
  Lines             0    24328    +24328
  Branches          0     2268     +2268
=============================================
+ Hits              0    12964    +12964
- Misses            0    10562    +10562
- Partials          0      802      +802

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

If we are trying to refactor ShuffleManagerClient, could you also consider to fix https://github.com/apache/incubator-uniffle/issues/1774 as well? @xumanbu rssConf in ShuffleManagerGrpcClient is hard-coded.

If we are trying to refactor ShuffleManagerClient, could you also consider to fix #1774 as well? @xumanbu
rssConf in ShuffleManagerGrpcClient is hard-coded.

ok.


[Migrator] Implemented baseline for Clickhouse Migrator

https://github.com/go-gorm/clickhouse.git

[x] Do only one thing
[x] Non breaking API changes
[ ] Tested

Description
In the scope of this PR (pull request) is aimed to get basic functionalities specified below to work on a simple gorm Model:
Tasks:

[x] Support basic table create and migrate in clickhouse
[x] Support column operations in clickhouse
[x] Support some index options and operation, but not all
[ ] Proper unit testing
[ ] Proper functional/integration testing

CreateTable baseline

Migration worked for simple model (no association) with index options, type, and expression in gorm Model tag (done)
Hardcoded GRANULARITY value if there is Index
Hardcoded engine option: using Engine=MergeTree() ORDER BY tuple()
Primary Key support is not done in this PR but upcoming PR. Primary Key has to be specified through engine parameters and depends on the choice of Engine.
Unique Index or other index class in not done since it is not supported in clickhouse

Column migrate operation

AddColumn (done)
RenameColumn (done)
AlterColumn (done)
DropColumn (done)
HasColumn (done)
DefaultExpr, Comments, and TTL not supported in this PR. But upcoming PR

Index migrate operation

RenameIndex is left out and not done in this PR
DropIndex and AddIndex with hardcoded GRANULARITY
Support index type TYPE for choices: minmax | set(min_rows) | ngrambf_v1(n, size, hash, seed) | bloom_filter(proba)

TODOs for Next PR

Parse Engine SQL Syntax when CREATE TABLE .... and check options for different engines. Some engines doesn't support Data Skipping Indices (which in any other RDBMS means having the usual INDEX ... in your table)
Allow granularity tags that is equivalent to index builder SQL INDEX ... GRANULARITY <granularity_value>
Support Primary Key after (1.) is done

https://github.com/ClickHouse/ClickHouse/issues/146


extra spaces in buffers picker

Info

nvim --version: NVIM v0.10.0-dev-653+gfbeef0d4e-Homebrew

fzf --version: 0.42.0 (brew)

[x ] The issue is reproducible with mini.sh

Description
The buffers picker contains extra spaces between buffer ids (on the left) and name (on the right): see screenshot below reproduced with mini.sh. In particular notice the space between [1] <-----> %.

I have noticed it after one of the latest commits, but I have not managed to figure out which one in particular (I do not remember this to ever be the case before, or am I wrong)? The files picker, on the other hand, looks fine (namely no extra spaces between the left and the right sides).

Hi @gennaro-tedesco, this change was on purpose as I noticed buffer numbers weren’t aligning if you had a long session and buffer numbers cross the 99 and 999 boundaries.
I’ll think of a solution to make buffer numbers dynamically aligned based on the highest buffer number.

I wonder, are buffers numbers needed to show at all in a fuzzy picker? 🤔
However I understand the purpose of the change, so do not worry too much if you cannot find any way around!

You can always hide the buffer numbers with:
:lua require("fzf-lua").buffers({fzf_opts={["--delimiter"]="' '",["--with-nth"]="-1.."}})

However I understand the purpose of the change, so do not worry too much if you cannot find any way around!

Virtually no chance of that happening, its just feeding the  right formatting alignment to string.format :)

https://github.com/ibhagwan/fzf-lua/commit/4dea46a7c61ff9f3ac3f77e2226312d50565b67d - should do what you want, @gennaro-tedesco.

Great, thank you as always!


DLL mismatch on GithHub Actions hostedtoolcache

I'm trying to debug an issue with an unusable GitHub actions build: https://github.com/builder555/PineSAM/issues/198
pyinstaller tries to bundle winrt.cp312-win_amd64.pyd etc. but issues a warning that it can't resolve api-ms-win-core-xxx DLLs.
E.g.: failed build log.
The builder nodes seem to have api-ms-win-core-libraryloader-l1-1-0.dll and api-ms-win-core-libraryloader-l1-1-1.dll not the api-ms-win-core-libraryloader-l1-2-0.dll the pyd files are built against. They are also entirely missing api-ms-win-core-path-l1-1-0.dll. I see this from listing the C:\Windows\System32\downlevel directory e.g. in this build log.
I guess this is a caching issue on GitHub Actions, the pyd files are in the site-packages directory in C:\hostedtoolcache\windows\Python etc. I tried to have pip rebuild the packages but ran into an issue building Devices.Bluetooth.GenericAttributeProfile - #2439. Skipping just that, pyinstaller still seems to grab the other packages from the hostedtoolcache path, not wherever pip installed them.
Can anyone suggest anything to make these libraries link correctly?

Mike - as I understand you might be the right person for Github Actions?


Segmentation fault from the example

I get a segmentation fault (as Travis does) when I run the example on Julia 0.5.1-pre+2 Ubuntu 16.04 and Csdp-6.1.1.
It is happening at makefill.c:74 which is basically doing
for (i=1; i<=k; i++) {
    ptr=constraints[i].blocks;
    while (ptr != NULL) {
        for (j=1; j<=ptr->numentries; j++) {
            p=ptr->iindices[j];
            q=ptr->jindices[j]; # It fails here
        }
        ptr=ptr->next;
    }
}

I think the issue is that iindices and jindices are julia arrays and you gave the pointers to Csdp pretending it is C arrays. However if iindices or jindices gets removed by the garbage collector then when Csdp tries to access it then it gets a segmentation fault.
@joehuchette @mlubin Are you giving the pointer of a Julia Vector to the C library in any of your solver wrappers or is this something that should be avoided for the above reason ?

If Csdp holds onto the Julia vector pointer instead of making a copy, then the fix is to keep a reference to the Julia vector on the Julia side (e.g., in the model object) to keep the GC happy.


Where can I find the Linux kernel version for 5.4.72-microsoft-standard-WSL2 ?

Dear Expert,
I want install the Oracle virtualbox at  on WSL2 with Ubuntu 20.04, before that, I need to manually install the Linux kernel 5.4.72-microsoft-standard-WSL2, but I can't find the release version at Github?

Here you go. FYI the release is gone, but the tag is still there.


Deleting helm chart results in deletion errors

May be related to #825 but recording here for now
  egeria git:(helm25) helm delete jazzy-aardwolf                                                                                git:(helm25|)
Error: deletion completed with 11 error(s): release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete; release "jazzy-aardwolf": object "" not found, skipping delete

Since the objects are not named, need to retry and capture more info.
Could indicate template errors on naming of objects. Or could be jobs removed via hooks. either way should be cleaned for the user
Checking resources manually, any jobs running at the time 'helm delete' is issues get left behind. In my case *job-01-data-loader - when this area is investigated the focus should be to ensure a clean deletion in all cases.

Checking resources manually, any jobs running at the time 'helm delete' is issues get left behind. In my case *job-01-data-loader - when this area is investigated the focus should be to ensure a clean deletion in all cases.

This seems to be a problem ("feature"?) in Helm itself when it comes to Jobs...  It will only remove them if they've run successfully, and within the timeout specified when running the helm install; otherwise it does indeed leave them behind, unfortunately.
(See https://github.com/helm/helm/issues/4434)


chore(updates): update the community pages per SIG discussion

This PR does the following:

removes stack overflow
removes the forum
directs people to slack first and github second

I'll also submit a PR to the community repo to update the readme there

@gcomstock Here's a PR for the changes we discussed in the SIG Meeting. I kind of want to remove the support policy page and integrate it with the main page since a lot of the information is duplicated. But that's something we can think about later.

Sorry for my tardiness here! Swamped with other work. Looks good w/my one comment.

Huh, it does not like #slack because Jekyll thinks that's an internal anchor link. Will do some cleanup.


Support Joy-Con motion

Unfortunately, there is no DSU client for MacOS that support Joy-Con.
In the README, you say "A lot of this application's code was made possible by looking at an existing DSU server implementation for Joy Con controllers".
Do you think there is a lot a work to make Joy-Con compatible with your application?
Thank you

I while ago I started working on joy4macos, but did not yet succeed in supporting motion controls properly. I started a new repo since ds4macos uses the GameController library created by apple does not support joy-con controllers. It would be cool if these projects can be merged one day.

Good to know that you started an other repo!
You should put a link in the README of this repo because a lot of people are looking for a solution and on the official DSU client dolphin page ds4macos is the only Mac app so people arrive here. Maybe some Swift developper could help if they know. I will ask on reddit.
I'm not a Swift developper but I plan to buy some Joy-Con so I will try to take a look. There are a lot of Windows open source software to get inspiration.

I found this repo: https://github.com/SymenYang/Magic-Joy-Con
Maybe it could help

Successfully connected my Joycon to DS4mac without crash.

Can it still be done?
https://github.com/magicien/JoyConSwift
nothing is work on Mac, this is the only hope
thank you.


PyWren IAM role should use correct AWS id

The IAM role installed by pywren_exec_role doesn't have the right permissions for the cloud watch logging actions. We need to get the user's aws id from boto to get this right.

For posterity, this arose when @shivaram was looking for logs in cloudwatch. We should make the command-line print-out cloudwatch logs and then add unit testing to make sure logs are showing up.

The unit test is a good idea. Also we could print out the URL to cloud watch from the CLI. For me it looks something like https://us-west-2.console.aws.amazon.com/cloudwatch/home?region=us-west-2#logStream:group=/aws/lambda/pywren1

@ericmjonas You'll also need to update the IAM role at https://github.com/ericmjonas/pywren/blob/83b7b92c93a6890284573731de2b29e222facdd1/pywren/wrenconfig.py#L89 ?


What's after step 2

I finished step 2. And now I'm stuck. Could any one explain what to do after that step. I'm on a pc btw. Id like to see a tutorial or more detailed instructions. I don't even know how to run it, all I can do is test it and i get the error for bs4 that someone else had. So if someone could give links and some help that be great! I'd paypal some money if someone gets it working for me!

I would first make sure all the files are in the same folder before you try anything. Once they are, open command prompt and  change your directory to that folder. Then you need to type pip install -r requirements.txt

That’s what I did. I downloaded the zip. Zipped in a folder. Used cmd to change directory then did pip install. It installed. But I can’t do the next steps. And idk which .py is used to run it and idk how to run it. I’m a noob at python.

On Mar 13, 2018, at 7:42 AM, enegronjr<EMAIL_ADDRESS>wrote:
I would first make sure all the files are in the same folder before you try anything. Once they are, open command prompt and change your directory to that folder. Then you need to type pip install -r requirements.txt
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

You're going to have errors so I'll walk you through them as they come

Okay thank you. Didn’t have time to work on it today. I wish we could screen share and you walk me thru it. I’d even pay alittle to get it working

On Mar 13, 2018, at 12:40 PM, enegronjr<EMAIL_ADDRESS>wrote:
You're going to have errors so I'll walk you through them as they come
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Mines not working right now, I still have an issue I'm stuck at
On Mar 14, 2018 1:37 AM, "maawbrey"<EMAIL_ADDRESS>wrote:

Okay thank you. Didn’t have time to work on it today. I wish we could
screen share and you walk me thru it. I’d even pay alittle to get it working

On Mar 13, 2018, at 12:40 PM, enegronjr<EMAIL_ADDRESS>wrote:
You're going to have errors so I'll walk you through them as they come
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sushant10/HQ_Bot/issues/33#issuecomment-372910295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjgmpcKkbGQ6y3g-UfxCsnu3xByeLdWiks5teKyHgaJpZM4SoIVH
.


Cannot install / improve installation section in README

Seems like a cool project, however none of the provided installation options work for me. What i tried:
$ python3 -m venv --system-site-packages venv
$ source venv/bin/activate
$ pip install -U pip wheel
Requirement already satisfied: pip in ./venv/lib/python3.10/site-packages (22.0.3)
Requirement already satisfied: wheel in ./venv/lib/python3.10/site-packages (0.37.1)

$ pip install bitwarden-menu
ERROR: Could not find a version that satisfies the requirement bitwarden-menu (from versions: none)
ERROR: No matching distribution found for bitwarden-menu

The Archlinux AUR link returns a 404.
And the README is not clear on how to run it after cloning the git repository.
I would like to try out the program, can you please help me?

Ah, so sorry! I haven't completed some enhancements to the project yet that I
want to before uploading to Pypi, so the instructions as written don't work!
There's also no Archlinux package yet. I started the repo as a copy of my
Keepmenu project and have been slowly
updating pieces as I find time to work on it. I'm using bitwarden-menu daily on
my laptop though and it's been super convenient and very fast to lookup and type
entries!

Download the project and change to that directory
python -m venv venv
source venv/bin/activate
pip install -U .
bwm (bwm is the name of the executable)

I'd also recommend looking into the pipx project
for installing CLI apps into virtualenvs.
Thanks for your interest! Happy to help more. Maybe I'll spend some of my
vacation time here polishing this up some more and releasing to the world :)

I've released bitwarden-menu on Pypi and the AUR, fyi. Let me know if you have
any other issues. Thanks for your interest!

Hi, sorry for the delay, and many thanks for your feedback :) Will try it out when I have some more time.


Fixed Issue 2517

This PR fixed issue #2517

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test

@preetvadaliya Note that this change should technically be on ucr since it changes the behavior of the component in the companion. That said, since we are close to a companion release and it has limited potential effect I think it's probably okay ot keep it against master. In the future you may want to double check which base branch to submit against.

Up to you. I'd like to merge it today. If you have time please do otherwise I'll do it at the end of the day.


Two identical hid devices

Hello, I want to query two hid devices with the same [idvendor] and [idproduct]
Through the following code, but it doesn't seem to work

I don't know how to operate to get two identical hid devices

usb.core.find(find_all=True, ...) returns an iterator of Device instances, so you probably want to loop over it.


ITA Translation

New language
Italian
Additional remarks
No response

Italian is already available but needs more work. See here for instructions https://music-assistant.io/help/lokalise/


Error when exporting

Error: Could not create output directory images.
Failed to export 2D123A64-9A89-4BCA-814F-361152CBFF48
Could not create output directory images.
Failed to export 2D123A64-9A89-4BCA-814F-361152CBFF48@2x
Could not create output directory images.
Failed to export 2D123A64-9A89-4BCA-814F-361152CBFF48@3x

I have the same problem. How to resolve it?


selectedOptions return need Id & Name

Hi Team,
As of now selectedOptions returns only id, but we need id and name also. Please provide any solutions for this.
Thanks you !!

I believe the original intent was that you could use the id to obtain any details about your model
e.g.
var options = [
        { id: 1, name: 'Option 1', color: 'blue' },
        { id: 2, name: 'Option 2', color: 'red' },
        { id: 3, name: 'Option 3', color: 'yellow' },
        { id: 4, name: 'Option 4', color: 'green' },
        { id: 5, name: 'Option 5', color: 'purple' },
    ];

(selectedOptions = [1,3])

options.filter(option => selectedOptions.indexOf(option.id) >= 0)

[
  {"id":1,"name":"Option 1","color":"blue"},
  {"id":3,"name":"Option 3","color":"yellow"}
]

In recent updates the id now permits objects so you could also just make your ids contain the actual id and the name, but I wouldn't recommend this.

@chris--jones is correct.  If you want the value set in "name", you should get that data from your array directly.  The "id" property is what is being set as the value model, and can be an object, as stated.  If the actual value you want is the label, then you can set that for both the id and the label.


Update with-universal-configuration example

resolves #4498 #4519

ok, let me know do I have to correct something. thx


add optional label that shows in the object description

Motivation: In my app, I log all actions. This helps with debugging. But sprinkled in now are non-descript log messages like:
💚 11:47:08: > Thunk<AppState>(body: (Function))
💚 11:47:09: > Thunk<AppState>(body: (Function))
💚 11:47:10: > Thunk<AppState>(body: (Function))

With the customizable description, users would be able to see which thunk is being dispatched, e.g.
💚 11:47:08: > Thunk<AppState>(label: "Requesting users")
💚 11:47:09: > Thunk<AppState>(label: "Requesting images for user XYZ")
💚 11:47:09: > Thunk<AppState>(label: "Requesting images for user ABC")

What do y'all think about that?

I wonder if this is something that could be done in a custom middleware?
Another reason for this is that thunks are actually fake ReSwift actions, meaning that they can't affect the state in any way. The real actions that happen inside are the ones that should be logged, IMO.
WDYT?

I wonder if this is something that could be done in a custom middleware?
Another reason for this is that thunks are actually fake ReSwift actions, meaning that they can't affect the state in any way. The real actions that happen inside are the ones that should be logged, IMO.
WDYT?

So you'd say that logging Thunk dispatching doesn't make sense, and instead one should focus on the actual actions? Hm, I can see how adding log(...) or print statements inside the thunk for debugging purposes to make sense as well. Not sure about the philosophical implications. Logging a useful line instead of "Thunk" would be nice, that's where my argumentation ends :)

So you'd say that logging Thunk dispatching doesn't make sense, and instead one should focus on the actual actions? Hm, I can see how adding log(...) or print statements inside the thunk for debugging purposes to make sense as well. Not sure about the philosophical implications. Logging a useful line instead of "Thunk" would be nice, that's where my argumentation ends :)

Yeah, that's my rationale. A thunk is just something that will eventually create real ReSwift actions.
If instead of dispatching a thunk you would implement the async operation directly in the view—for instance, when a user taps a button—, you will expect only the real actions dispatched to be logged.

Yeah, that's my rationale. A thunk is just something that will eventually create real ReSwift actions.
If instead of dispatching a thunk you would implement the async operation directly in the view—for instance, when a user taps a button—, you will expect only the real actions dispatched to be logged.


Improve the confirmation page (add directions to contact your councilperson, share on social media)

After the user submits their budget, they're directed to store-data page:
Include:

[x] Contact your councilperson!
(content at: https://docs.google.com/document/d/1ydc5Bw0Ctp8LJFEWECu9f9wMeUfM-FKlhTROik0QJiY/edit# (council person section))

Should this be a new page or displayed here?

[x] include links to share on social media (possibly use django_social_share)

Closing this for now; we can reopen or/and discuss further if we want to redesign this page.


Timeout on fast movements

Required Info

Camera Model
R200

Firmware Version
<IP_ADDRESS>

Operating System & Version
Windows 10 Pro N

Build System
VS2013

Hi,
I have noticed that the R200 will timeout waiting for frames if I make large movements with the camera. If I keep it still it will capture indefinitely.
I am capturing at 15FPS 1920x1080 colour and 30FPS 640x480 depth.
Any idea on how I can fix this? In my loop I am waiting for frames and writing out every 30th frame or so to file.

I have seen this when the USB cable moves in the socket.  If you are waving
the camera around at the end of a USB cable while doing a scan, it can stop
working.  Attach the camera to a laptop or tablet and move the whole unit.

This is a semi-known issue with some variants of the USB 3.0 connector on the peripheral. A little tape goes a long way!

I did have it attached to the back of my laptop with the magnetic sticker. Same issue.

It's a bad USB connection, probably on the peripheral side. If you can't get it to work with a little tape or hot glue for added mechanical support, I would recommend an RMA by emailing<EMAIL_ADDRESS>


Using MediaFoundationReader with MemoryStream?

Is it possible to use MediaFoundationResampler with MemoryStream?
I have a WAV file in a memory stream, not as a physical file that I would like to pitch.
I've tried using StreamMediaFoundationReader but it says that the URL is invalid.

I also encountered this problem. How do you solve it?


Is it possible to reference deploy time values from one stack as part of the build steps for another?

This is an issue I've bumped up against a couple of times now and it would be great to get a definitive answer.
Lets say I have a GraphQL stack and a React app stack. I want the React app to be able to make requests to the GQL stack. However, for the React app to build properly I need to pull in the GQL types from the GQL stack. This of course needs to happen before the React app is deployed. Is there a way to resolve the tokenized values from one stack before beginning the build/deployment of another?
I guess what I'm asking is: Does SST allow for part of an app to be completely deployed and it's exports resolved before continuing with the next step in the app?
This is a pseudo-ish code of the kind of thing I've tried before, just in case what I'm trying to do is possible, but I've been doing it completely wrong:
function GQL({stack}) {
  const gqlAPI = new GraphQLApi(stack, "Api", {
    //stuff
  });

  return gqlAPI;
}

function ReactSite({stack}) {
  const gqlApi = use(GQL);
  //also tried putting a dependsOn in here

  getGQLTypes(gqlApi.url); //This is where I have the issue

  return new ReactStaticSite(stack, "ReactSite", {
    path: "./packages/web/overlay",
  });
}

If it's not possible to do this dynamically, what's the best way to do it statically? What I'm really looking to avoid is having to go look up the gql api url manually before I can build/deploy the react app.
To be clear, this is a simplified example. I know I could reference the gql schema locally to generate the types. One example where that's not possible is with apollo federation, where you need to run a cli command before building in order to generate the federated schema. That CLI requires a yml filled with the URLs you want to federate.

Hey @liam-jones-lucout, I like the pattern you suggested. Essentially, instead of building all stacks up front, build each stack right before deploying, so it has access to stack outputs from deployed stacks.
This works for ur use case, but wouldn't work if GraphQLApi and ReactStaticSite were created in the same stack. I'd prefer a solution that handles both cases consistently.

That said, you can use SSM to work around this. Pseudo code looks like:
import * as ssm from '@aws-cdk/aws-ssm';
import aws from 'aws-sdk';

function GQL({app, stack}) {
  cons SSM_NAME = `${app.name}/${app.stage}/GQL_API_URL`;

  const gqlApi = new GraphQLApi(stack, "Api", {
    //stuff
  });

  const ssm = new ssm.StringParameter(stack, 'StringParameter', {
    name: SSM_NAME,
    stringValue: gqlApi.url,
  });

  return SSM_NAME;
}

function ReactSite({stack}) {
  const SSM_NAME = use(GQL);
  //also tried putting a dependsOn in here

  const ssm = new aws.SSM();
  const url = await ssm.getParameter({
    name: SSM_NAME
  });

  getGQLTypes(url); //This is where I have the issue

  return new ReactStaticSite(stack, "ReactSite", {
    path: "./packages/web/overlay",
  });
}

Let me know if that makes sense.

@fwang thanks for getting back to me! In your example (which I will try this evening) can you confirm that the SSM parameter will be fully resolved by the time ReactSite is called like so:
export default async function (app: App) {
   app.stack(GQL).stack(ReactSite)
}

I'm guessing from your example that SSM has a role in delaying build steps in time for the value to be resolved fully. I wasn't aware of that at all, so that if that's the case then it may be exactly what I need. Many thanks again

Yeah it will. Note that ssm.getParameter() is just a normal AWS SDK call.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


redirect from shinyapp to external resource

I am trying to simple redirect from shiny app to google via code example:
      tags$ul(
        class = "dropdown-menu",
        tags$li(
          tags$a(href = "https://google.com", "redirect to google")
        )
      )

In browser I see shiny proxy blank page, but in browser console looks like redirect is ok

Whats wrong and whats the right way to redirect from shinyapp? @fmichielssen Could you please help?

Hi @evfonarik ,
Since the shiny app is running in an iframe, your redirect needs to target the parent frame.
I'm not familiar with the tags approach, but in Javascript it would be this:
window.top.location.href = "https://google.com";
Note the window.top instead of window.

Hi @evfonarik
Thanks for your comment. Will ping developer on R language to adapt this code like on JS.

Does that tags$a line support the target attribute? If so, this might also work:
tags$a(href = "https://google.com", target = "_parent", "redirect to google")

As I believe that your question is answered, I'm closing this issue. Feel free to re-open it if you need more help.


Support for update in the bridge

Fairly extensive changes to the bridge code generation to support update, plus changes to the underlying bridge itself. The main element of getting this to work was updating API registration in the generated provider to specify provided and immutable attributes. Any field that is not required is assumed to be provided. Any field that is marked in the Terraform schema as "ForceNew" is assumed to be immutable.
There is also a small fix to the loader where services loaded from embedded plugins were correctly registered with the loader as services, while separate plugins weren't. That was affecting regeneration of Puppet types.
All five of the bridging providers and their corresponding types are regenerated.

Closes #67


无法保存商品信息, 新增商品也失败。

麻烦您把post的完整数据截图一下看看。

后台编辑商品后无法保存：

已修复

感谢！


Start and reset

I implemented a simple method StartAndReset which allows users to easily run workflow mutiple times synchronously (without using parallel option using DoAsynch method). I also added public event for workflow state change detection.
Last change is to use Parallel.Foreach instead of Task.Factory.StartNew in MultipleGoWorkflowAction for better performance when dealing with large number of items.
Added functionality should be covered with provided unit tests.

Hello !
Thanks for your contribution. I will merge your pull request and i am really happy if you use TinyWorkflow !
For the unit tests, i would prefer something based on numbers instead of strings to be more clear (maybe on my mind only...) For the name, StartAndReset, it would also be better something like StartAndRewind or StartAndResetAfterEnding, i mean, something more clear in VS, for new users :)  but nevermind.
If you want to propose something else, do not hesitate 👍


Excessive memory usage when opening large pdf file

When I try to open a large pdf file(800+ pages) in Droppy, my RAM usage suddenly increased rapidly just like what happened in #387.
Tested on archlinux with Google Chrome 80.0.3987.132.
Here is that scary book: ComputerNetworks.pdf

I've experienced this myself.

I can confirm I'm hitting the same behavior; opening a multi page PDF will consume all available host (and GPU) memory and eventually wedge my desktop. Using nvidia-smi, I can see that opening a small PDF (~2 pages) will consume ~2GB of GPU memory. This is the Chrome PID consuming all GPU memory:
/opt/google/chrome/chrome --type=gpu-process --field-trial-handle=14861158379448695967,10571079169687825361,131072 --enable-crash-reporter=bb6ddb08-####-####-####-############, --gpu-preferences=MAAAAAAAAAAgAAAQAAAAAAAAAAAAAAAAAABgAAAAAAAQAAAAAAAAAAAAAAAAAAAACAAAAAAAAAA= --shared-files

I get roughly the same behavior with Firefox, except no GPU memory consumption.
OS: CentOS 7.8
Kernel: 3.10.0-1127.19.1.el7.x86_64
Nvidia driver: 455.23.05
Browser: Chrome 85.0.4183.121 or Firefox 68.12.0esr
Droppy container: 12.2.0
This is very easy for me to reproduce, I'm happy to provide additional debugging info.


Just added DeepWallEnvironmentStyles to index.js

I just add to DeepWallEnvironmentStyles to export list. it will be good to reach EnvironmentStyles from app.

thanks

thanks


Change callback function arguments

Codecov Report

Merging #16 (51ce872) into master (6b9216f) will increase coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   98.63%   98.65%   +0.01%
==========================================
  Files          23       23
  Lines         733      743      +10
==========================================
+ Hits          723      733      +10
  Misses         10       10

Flag
Coverage Δ

unittests
98.65% <100.00%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/array.c
100.00% <100.00%> (ø)

src/lua_postgres_decode_range.h
100.00% <100.00%> (ø)

src/multirange.c
100.00% <100.00%> (ø)

src/range.c
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Irrlichtless builds

@sfan5 What do you think on the CI change? IIUC, it will build and pack all branches but only master will go into releases, all other packs will only be downloadable as build artifacts. I might’ve even tested that.

What do you think on the CI change? IIUC, it will build and pack all branches but only master will go into releases, all other packs will only be downloadable as build artifacts. I might’ve even tested that.

If that works sure. But don't you need a separate actions/upload-artifact for that to work?


Improve CSS of the section list

The glow is a little too strong in the active bubble
We should figure out how to extend a line to the "sub" item bubbles
The end of the line should terminate in the final bubble, not keep on going

Maybe instead of or in addition to the glow, we add a strong underline beneath the active item? Worth experimenting


Updates after review

Updates after Masa's review #82 and some fixes:

[x] Fix naming for the sake of clarity
[x] Hide logprobs calculation by condition to not reduce performance
[x] Add intermediate dataclass for raw logprob info
[x] Use OpenAI dataclasses for logprobs in general
[x] Fix logprob calculation from probs with randomization
[x] Other small fixes

Note: I have some doubts:

misprinting in paged_cache_model.py (see TODO)?

if do_top_p or do_top_k:
    logits = _apply_top_p_top_k(logits_random, top_ps, top_ks)

It seems that logits_random should be instead of logits
2. For parameter n > 1, only one token generated after prefill step, why are not n tokens?
cc @zxybazh, @masahi

It seems that logits_random should be instead of logits

No, here we are separately doing greedy and random sampling. Only the latter is relevant for top p/k.

Hello @masahi! Thank you for quick response!

No, here we are separately doing greedy and random sampling. Only the latter is relevant for top p/k.

The following is still not clear for me: logits from logits = _apply_top_p_top_k(logits_random, top_ps, top_ks) are not used in sample(...) function below the expression and on higher level (generate(...) function) it is also not used. Looks like we transform logits_random in _apply_top_p_top_k function and use after.
May be do it so _apply_top_p_top_k(logits_random, top_ps, top_ks) to not confuse anybody?

Hmm interesting, I never thought about sampling a token individually for different sequences after prefill, i.e. the first tokens in each of n generations are the same. I think it is fine and simpler, but curious what others think.

As I see you are right: "the first tokens in each of n generations are the same". For me it is strange why we start to randomize tokens from the second ones. And yes it is harder to implement and may be not priority. Who also can discuss and think about it?

ah yes, you are right.
logits = _apply_top_p_top_k(logits_random, top_ps, top_ks)

should be
logits_random = _apply_top_p_top_k(logits_random, top_ps, top_ks)

I will fix it ASAP

Who also can discuss and think about it?

@sunggg What do you think? Currently in parallel sampling, the first tokens in each generation are the same since we just generate one token after prefill, which is then copied into each of generation:
https://github.com/octoml/mlc-llm/blob/batch-serving/serve/mlc_serve/model/paged_cache_model.py#L491-L499

Thanks @vvchernov, I'll merge this PR as it's quite mature and let's continue the discussion thread to the original PR https://github.com/octoml/mlc-llm/pull/82.

@sunggg What do you think? Currently in parallel sampling, the first tokens in each generation are the same since we just generate one token after prefill, which is then copied into each of generation:
https://github.com/octoml/mlc-llm/blob/batch-serving/serve/mlc_serve/model/paged_cache_model.py#L491-L499

Interesting. I think both makes sense. What is OpenAI or vllm's behavior? Can we match the behavior with them as both approaches sound reasonable?

I don't know about OpenAI but vLLM samples n tokens after prefill. I created https://github.com/octoml/mlc-llm/issues/161 to fix our behavior.


Reorganize async server code

Refactor main server flow to use AsyncServer abstraction

Introduced the AsyncServer struct to encapsulate the core server functionality such as binding, client handling, and shutdown management.
Replaced atomic operations with context-based cancellation and signal handling through channels.
Updated the main function to instantiate AsyncServer, handle port binding, and manage goroutines for the server and signal handling.
Removed atomic-based state management in favor of channels and context cancellation.
Updated test server setup to use the AsyncServer structure with retry logic for port binding.

Thanks for the comprehensive reviews @soumya-codes, @AshwinKul28, @lucifercr07! It helped make the code a lot more maintainable and idiomatic!

LGTM, good to merge...


Use Kafka client API directly

If we use Kafka Java APIs directly, would be easier to move 0.9 configs

Travis builds: https://travis-ci.org/suyograo/logstash-output-kafka/builds/114991095

LGTM. can we change the name of this PR to mention the update to the new 0.9 library?


Question in terms of AmountT

Coming from cpp using mp-units https://github.com/mpusz/mp-units the quantity amount type was templated on quantity level, so each instance of quantity could adjust its representative type, in your library AmountT.
For reference, from their cpp code
template<Reference auto R, RepresentationOf<get_quantity_spec(R).character> Rep = double>
class quantity ...

As far as I understand your library the AmountT is globally set, so for different units the same AmountT is used.
Was this a design decision made to begin with or is this something that could be defined on trait Quantity level?
Or am I misunderstanding the whole thing.

This is "by design" just because I can't think of an use case where you need different numeric types for different quantities.
Do you have such a use case?

Previously in cpp, we had typed inter process communication abstraction with support for Voltage, Current, Temperature etc.
Everything was transported as signed integer with fixed precision, but in business logic it was often useful to use double for different business logic.

Why did you choose different numeric types for "transport" and business logic?

Only because I could have full control of signed integers with fixed precision to for nano meter or such, so I would control rounding and truncating fully before transmit.
This is not a hard requirement on my side so I can change the cpp code to double and use double then on everything.
This issue can be closed.

Ok. Closing this issue.

I'm not sure whether I do understand what would be required here.
To support different types for the numeric part of a quantity would mean to make AmountT a type parameter of each quantity struct. This would definitely be a breaking change.
So, I'd like to avoid that unless there is a very strong demand.


feat(ingest): snowflake using oauth

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Hi @anshbansal please pick up review on this one -- let's see if we can replicate on our Snowflake instance

Hi @anshbansal please pick up review on this one -- let's see if we can replicate on our Snowflake instance

@saxo-lalrishav Few comments

Please follow commit message format https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md#commit-message-format
Please resolve conflicts with master
Please fix the lint. You might need to re-create your local venv to get the latest updates of lint libraries

I'll try and see see if I can validate this PR on my local.

@saxo-lalrishav Is this something that you are still working on?

@anshbansal pls have a look into this and provide your feedback

https://github.com/datahub-project/datahub/pull/4647#discussion_r866049376 This comment is remaining
Please update description from "oauth configuration" to add the snowflake doc where the details are present.
Please merge latest master so that linting passes.
And please add an example configuration at least in the PR so we know what the whole config looks like for snowflake for microsoft. A better thing to do would be to add unit tests for the big validator that you have added.

Hi @saxo-lalrishav Are you still working on this PR? We can merge it after the changes requested in last comment.

Hi @saxo-lalrishav Are you still working on this PR? We can merge it after the changes requested in last comment.

Yeah, looking into this

For the lint failures suggest to re-create your local venv from scratch so that you have the latest libraries.


Can the latest v19 branch work normally on Linux?

Description

I pulled the latest code of the v19 branch today, compiled it and ran it on Linux, a gray window will appear on the screen when starting up, and the game video will appear on the computer screen instead of the headset after connecting to quest2. Last year's code from the v19 branch didn't have this problem, so will the current code work fine on Linux?
Thanks for answering.

Environment
Hardware
GPU:AMD RX 5500XT (RADV NAVI14)

try enabling direct display mode, I've had that happen sometimes and sometimes it fixes it, I don't know exactly what causes it in the first place though

Also, if neither the latest nightly, nor v19 end up tickling your fancy, I recommend trying the nightly from Jan 31.
It's the last one before nightly builds started using the OpenXR client, which introduced many changes.
Though it's not a solution to v19 not working, just putting it out there.


feat: migrate transfer params to be self managed

Description
This PR:

[x] Self manages transfer app's params.
[x] Adds params related unit and integration tests to transfer app.
[ ] Adds migration code for the params from x/params' subspace to transfer's keeper
[ ] Adds migration unit and integration tests

closes: #3502
addresses: #2010
Commit Message / Changelog Entry
feat: migrate transfer parameters to be self managed

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #3553 (f6df036) into main (cc09f72) will decrease coverage by 0.07%.
The diff coverage is 81.63%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3553      +/-   ##
==========================================
- Coverage   78.76%   78.70%   -0.07%
==========================================
  Files         182      181       -1
  Lines       12688    12719      +31
==========================================
+ Hits         9994    10010      +16
- Misses       2262     2274      +12
- Partials      432      435       +3

Impacted Files
Coverage Δ

modules/apps/transfer/keeper/relay.go
90.90% <0.00%> (ø)

modules/apps/transfer/types/keys.go
84.61% <ø> (ø)

modules/apps/transfer/keeper/msg_server.go
93.87% <70.00%> (-6.13%)
:arrow_down:

modules/apps/transfer/keeper/keeper.go
85.95% <82.35%> (-1.09%)
:arrow_down:

modules/apps/transfer/types/msgs.go
92.98% <87.50%> (-2.14%)
:arrow_down:

modules/apps/transfer/types/codec.go
89.18% <100.00%> (+1.68%)
:arrow_up:

... and 1 file with indirect coverage changes


AutoApply on customShortcuts

On mobile, when using customShortcuts, it's impossible to apply changes when autoApply is set to false.

It could be great to set dates directly and submitting it.
Here is the code of my shortcuts, maybe I missed something :

      return [
        {
          label: '7 derniers jours',
          atClick: () => {
            const date = new Date()
            return [
              new Date(date.setDate(date.getDate() - 7)),
              new Date(),
            ]
          },
        },
// ...

I have try, and work with auto-apply is false

this my code
<litepie-datepicker
  :formatter="formatter"
  :auto-apply="false"
  :shortcuts="shortcuts"
  v-model="dateValue"
>
</litepie-datepicker>

My bad, the problem occurs when it's set to true (default value in my case).

when auto-apply is true,
if you click shortcut, v-model has updated with your selection.
you can changing button cancel with new props footer, read here.
Thank you

I'll close this issue. Please open new issues if any related issues.

With this fix in 1.0.9, it's ok !
Fix bug auto-apply if shortcuts clicked.


AttributeError: 'NoneType' object has no attribute 'unsqueeze'

INFO:checkpoints:{'train': {'log_interval': 200, 'eval_interval': 1000, 'seed': 1234, 'epochs': 10000, 'learning_rate': 0.0002, 'betas': [0.8, 0.99], 'eps': 1e-09, 'batch_size': 16, 'fp16_run': True, 'lr_decay': 0.999875, 'segment_size': 8192, 'init_lr_ratio': 1, 'warmup_epochs': 0, 'c_mel': 45, 'c_kl': 1.0}, 'data': {'training_files': 'filelists/bmsslist.txt.cleaned', 'validation_files': 'filelists/bmssval.txt.cleaned', 'text_cleaners': ['chinese_dialect_cleaners'], 'max_wav_value': 32768.0, 'sampling_rate': 22050, 'filter_length': 1024, 'hop_length': 256, 'win_length': 1024, 'n_mel_channels': 80, 'mel_fmin': 0.0, 'mel_fmax': None, 'add_blank': True, 'n_speakers': 0, 'cleaned_text': True}, 'model': {'inter_channels': 192, 'hidden_channels': 192, 'filter_channels': 768, 'n_heads': 2, 'n_layers': 6, 'kernel_size': 3, 'p_dropout': 0.1, 'resblock': '1', 'resblock_kernel_sizes': [3, 7, 11], 'resblock_dilation_sizes': [[1, 3, 5], [1, 3, 5], [1, 3, 5]], 'upsample_rates': [8, 8, 2, 2], 'upsample_initial_channel': 512, 'upsample_kernel_sizes': [16, 16, 4, 4], 'n_layers_q': 3, 'use_spectral_norm': False, 'gin_channels': 256}, 'speakers': ['bmss', 'ja', 'eng', 'gd'], 'symbols': ['_', ',', '.', '!', '?', '-', '~', '…', 'A', 'E', 'I', 'N', 'O', 'Q', 'U', 'a', 'b', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'r', 's', 't', 'u', 'v', 'w', 'y', 'z', 'ʃ', 'ʧ', 'ʦ', 'ɯ', 'ɹ', 'ə', 'ɥ', '⁼', 'ʰ', '`', '→', '↓', '↑', ' '], 'model_dir': '../drive/MyDrive/vits-finetune/checkpoints'}
2023-05-25 01:26:01.854170: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
DEBUG:tensorflow:Falling back to TensorFlow client; we recommended you install the Cloud TPU client directly with pip install cloud-tpu-client.
2023-05-25 01:26:02.738299: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
DEBUG:h5py._conv:Creating converter from 7 to 5
DEBUG:h5py._conv:Creating converter from 5 to 7
DEBUG:h5py._conv:Creating converter from 7 to 5
DEBUG:h5py._conv:Creating converter from 5 to 7
DEBUG:jaxlib.mlir._mlir_libs:Initializing MLIR with module: _site_initialize_0
DEBUG:jaxlib.mlir._mlir_libs:Registering dialects from initializer <module 'jaxlib.mlir._mlir_libs._site_initialize_0' from '/usr/local/lib/python3.10/dist-packages/jaxlib/mlir/_mlir_libs/_site_initialize_0.so'>
DEBUG:jax._src.path:etils.epath found. Using etils.epath for file I/O.
INFO:numexpr.utils:NumExpr defaulting to 2 threads.
INFO:torch.distributed.distributed_c10d:Added key: store_based_barrier_key:1 to store for rank: 0
INFO:torch.distributed.distributed_c10d:Rank 0: Completed store-based barrier for key:store_based_barrier_key:1 with 1 nodes.
/usr/local/lib/python3.10/dist-packages/torch/utils/data/dataloader.py:554: UserWarning: This DataLoader will create 8 worker processes in total. Our suggested max number of worker in current system is 2, which is smaller than what this DataLoader is going to create. Please be aware that excessive worker creation might get DataLoader running slow or even freeze, lower the worker number to avoid potential slowness/freeze if necessary.
warnings.warn(_create_warning_msg(
0% 0/15 [00:00<?, ?it/s]/content/emotion-finetuning-vits/utils.py:138: WavFileWarning: Chunk (non-data) not understood, skipping it.
sampling_rate, data = read(full_path)
/content/emotion-finetuning-vits/utils.py:138: WavFileWarning: Chunk (non-data) not understood, skipping it.
sampling_rate, data = read(full_path)
/content/emotion-finetuning-vits/utils.py:138: WavFileWarning: Chunk (non-data) not understood, skipping it.
sampling_rate, data = read(full_path)
/content/emotion-finetuning-vits/utils.py:138: WavFileWarning: Chunk (non-data) not understood, skipping it.
sampling_rate, data = read(full_path)
/content/emotion-finetuning-vits/utils.py:138: WavFileWarning: Chunk (non-data) not understood, skipping it.
sampling_rate, data = read(full_path)
terminate called without an active exception
Exception ignored in: <function _MultiProcessingDataLoaderIter.del at 0x7f57365d8820>
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/dataloader.py", line 1466, in del
self._shutdown_workers()
File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/dataloader.py", line 1430, in _shutdown_workers
w.join(timeout=_utils.MP_STATUS_CHECK_INTERVAL)
File "/usr/lib/python3.10/multiprocessing/process.py", line 149, in join
res = self._popen.wait(timeout)
File "/usr/lib/python3.10/multiprocessing/popen_fork.py", line 40, in wait
if not wait([self.sentinel], timeout):
File "/usr/lib/python3.10/multiprocessing/connection.py", line 931, in wait
ready = selector.select(timeout)
File "/usr/lib/python3.10/selectors.py", line 416, in select
fd_event_list = self._selector.poll(timeout)
File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/_utils/signal_handling.py", line 66, in handler
_error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 19442) is killed by signal: Aborted.
0% 0/15 [00:23<?, ?it/s]
Traceback (most recent call last):
File "/content/emotion-finetuning-vits/train_ms.py", line 306, in
main()
File "/content/emotion-finetuning-vits/train_ms.py", line 56, in main
mp.spawn(run, nprocs=n_gpus, args=(n_gpus, hps,))
File "/usr/local/lib/python3.10/dist-packages/torch/multiprocessing/spawn.py", line 240, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/usr/local/lib/python3.10/dist-packages/torch/multiprocessing/spawn.py", line 198, in start_processes
while not context.join():
File "/usr/local/lib/python3.10/dist-packages/torch/multiprocessing/spawn.py", line 160, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 0 terminated with the following error:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/torch/multiprocessing/spawn.py", line 69, in _wrap
fn(i, *args)
File "/content/emotion-finetuning-vits/train_ms.py", line 124, in run
train_and_evaluate(rank, epoch, hps, [net_g, net_d], [optim_g, optim_d], [scheduler_g, scheduler_d], scaler, [train_loader, eval_loader], logger, [writer, writer_eval])
File "/content/emotion-finetuning-vits/train_ms.py", line 152, in train_and_evaluate
(z, z_p, m_p, logs_p, m_q, logs_q) = net_g(x, x_lengths, spec, spec_lengths, speakers)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/parallel/distributed.py", line 1040, in forward
output = self._run_ddp_forward(*inputs, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/parallel/distributed.py", line 1000, in _run_ddp_forward
return module_to_run(*inputs[0], **kwargs[0])
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/content/emotion-finetuning-vits/models.py", line 464, in forward
x, m_p, logs_p, x_mask = self.enc_p(x, x_lengths, emo)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/content/emotion-finetuning-vits/models.py", line 171, in forward
x = x + self.emo_proj(emo.unsqueeze(1))
AttributeError: 'NoneType' object has no attribute 'unsqueeze'

I have made some modifications to the "models.py" file. Please let me know if you encounter any errors after copying and pasting the code.


Improve sqlcreate to supports PostgreSQL unix domain socket

The problem:
Currently the sqlcreate generate invalid SQL statements, like:
CREATE USER  WITH ENCRYPTED PASSWORD '' CREATEDB;
CREATE DATABASE mydatabase WITH ENCODING 'UTF-8' OWNER "";
GRANT ALL PRIVILEGES ON DATABASE mydatabase TO ;

if a Django database is configured to use Unix domain socket, like:
DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.postgresql',
        'NAME': 'mydatabase',
        'USER': '',
        'PASSWORD': '',
        'HOST': '',
        'PORT': '',
    }
}

then errors can be obtained when using python manage.py sqlcreate |  sudo -u postgres psql -U postgres like:
ERROR:  syntax error at or near "WITH"
LINE 1: CREATE USER  WITH ENCRYPTED PASSWORD '' CREATEDB;
                     ^
ERROR:  zero-length delimited identifier at or near """"
LINE 1: CREATE DATABASE mydatabase WITH ENCODING 'UTF-8' OWNER "";
                                                             ^
ERROR:  syntax error at or near ";"
LINE 1: GRANT ALL PRIVILEGES ON DATABASE mydatabase TO ;

Solution
The solution I propose with this pull-request, is to change the logic so that, if USER or PASSWORD is not informed on django database configuration, then only SQL CREATE DATABASE will be generated, like:
python manage.py sqlcreate
-- Assuming that unix domain socket connection mode is being used because
-- USER or PASSWORD are blank in Django DATABASES configuration.
CREATE DATABASE mydatabase WITH ENCODING 'UTF-8';

--
Pull-request sponsored by

Coverage decreased (-71.2%) to 0.0% when pulling 95defe7cb60c6c2fd5bfbdc8fd47516fa109fb08 on luzfcb:sqlcreate-postgresql-socket-mode into 0488b46f30db0e51b375bedbc035ee4ba60c8005 on django-extensions:master.

Coverage decreased (-71.2%) to 0.0% when pulling 95defe7cb60c6c2fd5bfbdc8fd47516fa109fb08 on luzfcb:sqlcreate-postgresql-socket-mode into 0488b46f30db0e51b375bedbc035ee4ba60c8005 on django-extensions:master.

@trbs Since this changes the current default behavior (which is to generate invalid SQL), perhaps this change should be classified as breaking change.

@trbs The failing tests are not related to the changes included on this pull-request.

tests should be fixed now :-)

Yeah I agree on the breaking change thing. but since this was already broken I don't really mind ;-)

@trbs Thanks for all your hard work to maintain this library.


JavaScript preventDefault

Hello,
I am working on an interactive book for visually impaired children. I'm looking for an epub reader on android that would allow me to interact without being disturbed by the default actions of the reader. iBooks does it very well (see below).

Would it be possible to integrate this possibility in your application when using preventDefault ?
Thank you very much for the work you do.

Absolutely, and this is supposed to work already.
But we don't have much test books for this, so if you are able to share a sample ebook to test it would be super helpful.

I am preparing one for you now.
I'm working with SVGs, maybe that's why it doesn't work!?

I'm working with SVGs, maybe that's why it doesn't work!?

I'm not sure, but I can take a look tomorrow. Could you setup a test with an SVG and one without? Thanks!

Hello here is a sample epub.
I use the ibooks.js script.
It works in Thorium.
test_preventDefault.zip
Sorry for my late reply (computer problem).

No problem, thanks I will take a look on Friday.

I investigated the issue, the good news is that it's actually working and an easy fix.
The ibooks.js script is using the following to determine if the device is touch based:
iBooks.SUPPORTS_TOUCHES =
  navigator.epubReadingSystem &&
  navigator.epubReadingSystem.hasFeature("touch-events") &&
  iBooks.SYSTEM !== "Windows";

Readium mobile is not implementing the epubReadingSystem object, so it's not recognized as a touch device. I will need to bring this issue up to the dev group before making changes related to this.

It works in Thorium.

Did you test this particular test book? I tried with the latest Thorium version and it didn't work.

Did you test this particular test book? I tried with the latest Thorium version and it didn't work.
Yes and I have version 1.8 on PC

I'm sorry. It's my script that's wrong. I have to write
iBooks.SUPPORTS_TOUCHES =       navigator.epubReadingSystem &&      navigator.epubReadingSystem.hasFeature("touch-events") ||      iBooks.SYSTEM !== "Windows";
Thanks for opening my eyes. It works in your app

That's one way of handling it! But having an epubReadingSystem on mobile might still make sense. I'll keep this issue opened for now. Thanks for reporting.

I'm going to tweak my script to make it work while waiting for an update.
Thanks again for the work you do.


Dark Sky API should be replaced

Dark Sky has joined Apple and does not accept new API signups.
"Our API service for existing customers is not changing today, but we will no longer accept new signups. The API will continue to function through the end of 2021."
https://blog.darksky.net/dark-sky-has-a-new-home/

I was already afraid this topic would come up. :)
I really like the Dark Sky API, but that's probably also the reason why Apple bought them.
Does anyone have suggestions for a replacement API?

I guess most accurate will be your own or a nearby Netatmo weatherstation https://dev.netatmo.com/apidocumentation/weather

Netatmo looks promising. You need to create a free account and an app. Then we can use https://pypi.org/project/netatmo/ to query a weather station. I don't know yet how you can get such an id nearby your PV.

https://dev.netatmo.com/apidocumentation/weather#getpublicdata
The response based on square lat/lon of your area contains nearby id's.
Looks like function getmeasure required both device_id and module_id.

Yes, I was thinking in the same direction. Pick a square area, enlarge it until you find weather stations and then pick the closest. With a command line option to overrule this selection mechanism and use a fixed device id.
I don't think we need to set a module id, we get all sensor data and filter out the temperature.
BTW, there are a few more Python packages implementing the Netamo API, so we should pick the best/most mature.

I've create a feature branch with a first attempt: https://github.com/markruys/gw2pvo/tree/feature/netatmo. It supports only public Netamo weather stations, so it won't work to obtain historical temperature readings. Please let me know how it works for you.

Didnt try the netatmo branch yet but another option could be openweather. See https://openweathermap.org/darksky-openweather
Will also try out the new feature branch in the meanwhile.

I've tested the feature branch and works like expected! Leaving device-id empty uses a public weatherstation close to my location. Looks like my own device is excluded from the getpublicdata response. Using the MAC address at device-id works for using my own device.

But I've merged the Netatmo into trunk and released it as 1.3.4. You can try it like:
sudo pip3 install gw2pvo==1.3.4

I just added Netatmo to the config, it looks like the keys are correct and connection established to the API however the log is showing error and it doesn't upload to PVoutput:
INFO     Normal, 794.0 W now, 21.9 kWh today, 83.2 kWh all time, 243.7 V grid, 617.8 V PV
ERROR    'city'

My inverter reports it's own temperature I have been logging that: https://github.com/L-four/gw2pvo/commit/74d46401979ed3f0ffa0f5876ffaeb1878d7be1b

Why is a weather API needed for gw2pvo needed in the first place? PVOutput already supports this by itself.
Just create an API-key at OpenWeatherMap and fill it in at the PVOutput system settings.

Why is a weather API needed for gw2pvo needed in the first place? PVOutput already supports this by itself.
Just create an API-key at OpenWeatherMap and fill it in at the PVOutput system settings.

Because this was built a year before PVOutput started supporting OpenWeatherMap?


Adds sample settings for validator IDs for KinD example

What does this PR do?
The conjurdemos/kubernetes-conjur-demo scripts PR #140 adds support for 2
"validator" host IDs in Conjur policy that can be used to validate Conjur
authentication e.g. after a Kubernetes cluster or a Kubernetes Namespace
has been configured to support Conjur authn-k8s.
This change adds sample, customizable settings for configuring these authenticator validator
host IDs (and the Namespaces from which Conjur authentication will be validated)
for the Kubernetes-in-Docker (KinD) application deployment example.
What ticket does this PR close?
Resolves #152
Checklists
Change log

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[x] This PR does not require updating any documentation

Closing this PR since the config "knobs" that are being documented should never be needed for this demo. Anyone using this demo will be able to use the default validator IDs that are implemented in kubernetes-conjur-demo.


De namespace

Description
Remove namespace package following the formalism in https://github.com/sot/ska_sun/pull/22.
Interface impacts

None
Testing

Unit tests

[x] No unit tests

Functional tests

Installed and confirmed expected packages / functions / versions
Built docs and confirmed they look reasonable

The documentation does not build for me:
  File "/Users/javierg/SAO/git/Ska.astro/docs/conf.py", line 23, in <module>
    from ska_astro import __version__
ImportError: cannot import name '__version__' from 'ska_astro' (unknown location)

Good point.  I didn't try to build the docs but it looks like version was removed from astro.py .

After seeing that, I checked the PRs that were merged that I had not looked at, and this one also was missing __init__.py

Right.  It looks like a few of these the version was probably moved into __init__.py and then maybe not checked in to version control?  I fully expected to do need to go around and do some tiny fixes after we do more testing in the integ environment.

I added the missing __init__.py.


Use TokenId to track parser progress

Summary
This PR updates the parser progress mechanism to use a token id instead of the token kind and range. The ID is stored on the Parser and is incremented every time the next_token method is called.
The old logic would lead to panic if there were multiple levels of indentation before the program ends when the lexer would emit Dedent tokens for the same range. This makes it seem that the parser isn't progressing.
Test Plan
Tested it with the following program which gets stuck on dhruv/parser because the parser isn't able to recover from the invalid mapping key pattern:
match subject:
    case {*key}:
        pass

The reason it's only failing for invalid programs is because otherwise the parsing logic (parse_block) would consume the Dedent token.

Does this have any impact on performance?

Does this have any impact on performance?

No, I don't see any impact running the benchmarks locally.


Fix app not running after cloning

After following the installation instructions, the app could not be run right away. This PR attempts to fix that so the app can be developed on other environments.

Firstly, requirements.txt was missing a required dependency, SpeechRecognition, so I added it.
Attempting to run the program with python main.py then raises OSError: Label: File 'STHeiti Medium.ttc.ttf' not found because my computer does not have that specific font file installed, so Kivy could not load it. To fix that for any users who do not have it, I added a copy of the file to the repo, which Kivy successfully detects.
There's no sensible way to run the program with __name__ == "other", so to facilitate different run methods without clashing with Kivy's argument parser, I transformed app/ into an executable package. See the new README#usage.
I updated the README.md to reflect these changes.

I did not modify the application's core functionality in any way. I only changed the interface (how users run the app) and made the repo more complete for future clones.

LGTM! Thanks a lot for doing this lol!


Test: VS Code Figma Toolkit

Refs https://github.com/microsoft/vscode/issues/83952

[ ] any @octref

Complexity: 3
Overview
We've created a Figma toolkit for VS Code extension authors to aid in the creation of extension UI. With this toolkit, extension authors are able to re-use existing design components and create custom views. Try to design a dummy extension that covers the following the contribution points:

Custom activity bar icon
Custom viewlet with section group
Custom tree
Custom web view
Custom notification

Link: Figma Toolkit
Follow the steps in the Figma file's readme, if you are already part of the VS Code team on Figma you can skip steps 1-3 and create a file in your drafts:

I had fun. Pretty good library. Thanks 👍


AMD FSR 3.0 Causes shadowing/motion blur

Issue with excessive motion blur / sillouette shadow
This issue occurs when any movement is made and is especially apprent when the camera is moved. This only occurs when AMD FSR 3.0 is enabled. I have not tried other upscaling methods within AMD FSR 3.0 but was utilising Native AA in this screenshot. Would also note that I was in teh 100+ FPS range so low FPS was not the issue here.

I think I was able to fix it. Can you try this test build and confirm if it works on your end too?
FFXVIFix_v0_7_2_test.zip

I just tested, no more ghosting problems for me either
(3440x1440p)

Yes I can confirm that the Test build worked. My Res is 3440x1440p.
Thanks

Amazing work ! thanks

new update broke it. . .  Damn

Should be fixed again in this build.
FFXVIFix_v0.7.2_test2.zip

Fixed in v0.7.2


Parameter in sortable:sorted Event does not have the correct indexes

Parameter in sortable:sorted event callback has the following data structure
@type { Object } e - Parameter in sortable:sorted event callback
@prop { Object } e.data
@prop { Object } e.data.moves
@prop { Object } e.data.moves.oldIndex
@prop { Object } e.data.moves.newIndex

The oldIndex and newIndex are inconsistent depending on the source element's original order in the list.
Calculation of the indexes should consider the cloned source element, because before dragging, there are, say, 4 elements in the list. On dragging, a new cloned element is inserted into the list, so there will be 5 elements in the list.

Thanks for reporting, yes this has been broken since we fixed the touch issues. This will be fixed for the next release. I'll let you know once the fix is in 👍
We want to have full test coverage for the first stable release, so this won't happen again

Fix will be out for the next beta release, here the commit: https://github.com/Shopify/draggable/commit/90e0b4c098c6410513efe845688fb896d19e0102
It will now ignore the originalSource and a mirror when computing the index

Closed via: https://github.com/Shopify/draggable/releases/tag/v1.0.0-beta.3


Update README and docs to explain JWT refresh flow

Right now it is very confusing how to handle the JWT refresh flow for supabase with flutter. It is my understanding that the supabase package is supposed to generate a new access token automatically, but this is not a clear flow.
For example: are currentUser and currentSession reliably not-null anytime a user is authenticated? I have run into situations where they appear temporarily null inspite of the user being logged in. This has broken key parts of my app UI.
Is there any code needed on the client side to handle JWT refresh?
Are we supposed to listen to onAuthStateChanged to handle the JWT refresh?
Really a simple guide for Supabase and Flutter using something like Provider around persisting auth state and handling the JWT refresh would be immensely appreciated. As it stands this flow is very hard to figure out.

For example I see this code
supabase.auth.onAuthStateChanged
.listen((data) {})
.onError((error) {
  supabase.auth.refreshSession();
});```
from another issue: https://github.com/supabase/supabase-flutter/issues/452#issuecomment-1588906208

Is this the recommended way to handle refresh?

I am also not relying on the session state for any of my UI since I have found it to be unreliable. I am relying only on my user state Provider for that. Should we be able to depend on session state for UI? If it is true that we cannot this is something else which I think should be really explicitly documented.

@ellmaki
supabase_flutter will automatically refresh your access token, and there is nothing you need to do. The SDK will automatically update the token if the session has expired when making requests. You shouldn't ever have to call refreshSession() either.
Would you explain why you thought you had to implement the complicated workarounds you have shared?

I am also not relying on the session state for any of my UI since I have found it to be unreliable. I am relying only on my user state Provider for that. Should we be able to depend on session state for UI? If it is true that we cannot this is something else which I think should be really explicitly documented.
Yes, you can rely on the Supabase auth state in your UI. If supabase.auth.currentSession is not null, the user is signed in.

I think this was possibly the source of all of my problems https://github.com/supabase/supabase-flutter/pull/806

That sounds good, and sorry for the confusion.

Thanks for addressing it quickly.

@dshukertjr I want to verify that things are fixed on my end - do you have advice for simulating a JWT expiration event?

@ellmaki
Yeah, it's not really a simulation, but the easiest way to test it would be to set the JWT expiry limit to something very short. I have it set to 60 seconds here. Then your access token will expire every 60 seconds.

Thank you. Through forcing the expiration I have confirmed that my bugs came from the currentUser null issue.

Glad to hear that things are working out for you now!


[jruby] reduce generated noise + review touched code

... after https://github.com/sparklemotion/nokogiri/pull/1563 I did find some more bits worth improving/cleaning up
p.s. tried to keep the commits minimal (for easier reviewing) but feel free to squash on merge if preferred

Merging manually. Thank you so much!

thanks. managed to cause a regression - with last commit. sorry about that will redo and open another PR next week (feel free to revert if needed). once again sorry for the mess-up should have used 2 branches or waited with the PR.

ah so that commit shouldnt be here, mobile gh confused me. all should be fine, if by any chance not than I will look into it. thanks for your time.


Click commit when no git message is entered

Type: Bug
1.change file
2.git stash
3.click commit
VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T04:27:29.066Z)
OS version: Windows_NT x64 10.0.17763
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz (16 x 2904)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.86GB (5.40GB free)

Process Argv
--crash-reporter-id db904158-765e-458f-b280-0dc96990661d

Screen Reader
no

VM
0%

Extensions (26)

Extension
Author (truncated)
Version

path-intellisense
chr
2.8.1

vscode-mysql-client2
cwe
5.7.11

vscode-ssh
cwe
5.6.1

vscode-eslint
dba
2.2.6

prettier-vscode
esb
9.9.0

ftp-kr
fft
1.4.1

auto-close-tag
for
0.5.14

auto-rename-tag
for
0.1.10

githd
hui
2.3.3

jbockle-format-files
jbo
3.4.0

svg
joc
1.4.19

git-graph
mhu
1.30.0

dotenv
mik
1.0.1

vue-volar-extention-pack
Mis
1.1.5

vscode-language-pack-zh-hans
MS-
1.70.8100912

vscode-typescript-next
ms-
4.9.20221017

LiveServer
rit
5.7.9

vue-vscode-snippets
sdr
3.1.1

vscode-scss-formatter
sib
2.4.3

autoimport
ste
1.5.4

sass-indented
syl
1.8.22

vscode-conventional-commits
viv
1.24.1

volar
Vue
1.0.9

vue
Wsc
1.0.26

JavaScriptSnippets
xab
1.8.0

markdown-all-in-one
yzh
3.4.3

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyonecf:30548226
vscccc:30610679
pyindex848:30577860
nodejswelcome1cf:30587006
282f8724:30602487
gswce1:30612156
iaj6b796:30613358
dbltrim-noruby:30604474
89544117:30613380
fim-prod:30622703

@yuntian001, could you please describe further on what is happening following these steps versus what would be expected?  Also if you could provide a screenshot or recording of the problem it would help with the troubleshooting. Thanks!

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

@yuntian001, could you please describe further on what is happening following these steps versus what would be expected? Also if you could provide a screenshot or recording of the problem it would help with the troubleshooting. Thanks!

When "message" is empty, clicking "commit" will keep it in the Commit state, and the button will remain gray until "message" cannot be filled in and then click again

When you press the commit action without specifying a commit message, a full editor is being opened so that you can author the commit message using the editor. This flow is being described here. You can also disable this experience by toggling the git.useEditorAsCommitInput setting. I will go ahead and close this issue but in case you have any further questions or concerns please feel free to reopen it. Thanks!

That works for me, thank you very much.


install with vundle with many errors

Error detected while processing /usr/local/Bundles/vim-signature/plugin/signature.vim:
line   22:
E117: Unknown function: signature#utils#Set
line   23:
E117: Unknown function: signature#utils#Set
line   24:
E117: Unknown function: signature#utils#Set
line   25:
E117: Unknown function: signature#utils#Set
line   26:
E117: Unknown function: signature#utils#Set
line   27:
E117: Unknown function: signature#utils#Set
line   28:
E117: Unknown function: signature#utils#Set
line   29:
E117: Unknown function: signature#utils#Set
line   30:
E117: Unknown function: signature#utils#Set
line   31:
E117: Unknown function: signature#utils#Set
line   32:
E117: Unknown function: signature#utils#Set
line   33:
E117: Unknown function: signature#utils#Set
line   34:
E117: Unknown function: signature#utils#Set
line   35:
E117: Unknown function: signature#utils#Set
line   36:
E117: Unknown function: signature#utils#Set
line   37:
E117: Unknown function: signature#utils#Set
line   38:
E117: Unknown function: signature#utils#Set
line   39:
E117: Unknown function: signature#utils#Set
line   40:
E117: Unknown function: signature#utils#Set
line   46:
E117: Unknown function: signature#utils#Maps
Error detected while processing CursorHold Auto commands for "*":
E121: Undefined variable: g:SignaturePeriodicRefresh
Error detected while processing CursorHold Auto commands for "*":
E15: Invalid expression: g:SignaturePeriodicRefresh | call signature#sign#Refresh() | endif
Error detected while processing CursorHold Auto commands for "*":
E121: Undefined variable: g:SignaturePeriodicRefresh
Error detected while processing CursorHold Auto commands for "*":
E15: Invalid expression: g:SignaturePeriodicRefresh | call signature#sign#Refresh() | endif

system info : OS X 10.10 with vim 7.3

This seems to be a problem with Vundle.
I created a minimal vimrc:
set nocompatible
filetype off

set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()

Plugin 'gmarik/Vundle.vim'
Plugin 'kshenoy/vim-signature'

call vundle#end()
filetype plugin indent on

After running :PluginInstall, running :scriptnames shows that Vundle loads the file vim-signature/signature.vim but none of the other files.
This is why it can't find any of the functions and complains accordingly.

the list of the :scriptnames  about signature
25: /usr/local/Bundles/vim-signature/plugin/signature/mark.vim
26: /usr/local/Bundles/vim-signature/plugin/signature/marker.vim
27: /usr/local/Bundles/vim-signature/plugin/signature/sign.vim
28: /usr/local/Bundles/vim-signature/plugin/signature/utils.vim
29: /usr/local/Bundles/vim-signature/plugin/signature.vim

this only display after the install with vundle, reopen the vim it is ok

@kshenoy there is another problem about the background, look at the pic

How can i to change the gray background?

@winds18 Did you run :scriptnames immediately after installing signature or did you restart vim and then run it?
Regarding the gray background, what color scheme are you using?

@kshenoy i run it after restart vim. the background is solved, thank you for your notice.

Closing. Reopen in case of any further questions or issues.


Allow Node funcs to be accessed in outside plugins.

Does what the PR title says.
Just a simple thing that should be allowed this will help making plugins in specific ways easier.

I can't see why this would cause issues so sure :p


WIP: GeoParquet reader

I started to take a stab at this to resolve https://github.com/georust/geozero/issues/37 but ran into a few issues/questions:

I put process_geoarrow_feature_chunk in the geoarrow mod, because the parquet reader's result is automatically geoarrow (of the wkb variant). In theory we could implement GeozeroDatasource for geoarrow directly as well to convert from an arrow table, but that might be less useful.
It seemed not possible to write an impl for GeozeroGeometry in addition to GeozeroDatasource, because GeozeroGeometry's process_geom only provides a non-mutable reader, and the parquet FileReader needed a mutable reference. Or maybe I'm missing something there.
I'm not sure the best way of doing arrow -> geozero feature objects. Arrow2's "chunk" object is a vec of Array trait objects, so we need to do downcasting to get values out. Is it possible to do that downcasting outside of the loop so that we don't have to downcast on every iteration?
I assume geozero has a record-only API? I.e. I can't process all the data column by column, it needs to be row-by-row?

Rebasing should solve the failed CI test.

I did the merge, but i have no domain knowledge about this PR

@kylebarron & @pka please update on this PR's path, if it needs more changes, or should be closed or merged

Closing this because it's been (mostly) implemented in https://github.com/geoarrow/geoarrow-rs, with transitive bindings into geozero.


grammar mistake

Grammar mistake in "Vue.js [ONE] client - [ONE] client wirtten with Vue2.5", "wirtten" must be "written" :)

@SharyZ What is your question ?, if it is still relevant of course


Asset conversion get_pool_id fix (Ord does not count with is_native flag)

PR contains:

main fix for get_pool_id (Ord does not count with is_native flag) and consequnce is that e.g. on system parachain, we cannot use MultiLocation::parent() as a native asset, because MultiLocation { parents: 1, ..} > MultiLocation { parents: 0, ..}  (parents: 0 is some local asset on system parachain)
improved MultiAssetIdConverter handling (e.g. adds ability for runtime to filter supported assets and so on)
other small nits
removed NativeOrAssetId/NativeOrAssetIdConverter dedicated for tests/mocks from types.rs

Cumulus companion: https://github.com/paritytech/cumulus/pull/2860

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3185284

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3185282

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3185408

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3185409

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3185410

bot help

bot bench $ pallet dev pallet_asset_conversion

LGTM, but I would probably avoid having 3 equal versions of the NativeOrAssetId

LGTM, but I would probably avoid having 3 equal versions of the NativeOrAssetId

well, yes, I dont like it either, but I also I am not sure if it fits to types.rs,
I thought that it is dedicated for mocks/tests or as sample,
for system parachains we have custom implementation and I am not sure about other parachains if it fits for them

bot rebase

bot bench $ pallet dev pallet_asset_conversion

bot fmt

bot fmt

bot clean

Looks good. We could push the common code out into a crate under bin/node and have the three locations depend on that?

couldnt this cause some circrular dependecy?
bin/node/runtime -> common code
bin/node/runtime has deps pallet-asset-conversion, pallet-asset-conversion-tx-payment
pallet-asset-conversion-tx-payment -> need that common code from bin/node/runtime
pallet-asset-conversion ->  need that common code from bin/node/runtime
@gilescope @jsidorenko
for simplicity, I can revert that, and return NativeOrAssetId/NativeOrAssetIdConverter to types.rs with some comment, but I think that anybody who will use this pallet will provide their custom implementation, I dont know
so please both:
thumbs up - leave as it is
thumbs down - I revert that back

LGTM

bot merge


imagemin:dist Error

Hi,
I'm getting the below error after taking latest imagemin updates.
[4mRunning "concurrent:dist" (concurrent) task[24m
[4mRunning "svgmin:dist" (svgmin) task[24m
Total saved: 0 B

[32mDone, without errors.[39m

Execution Time (2015-11-14 09:31:48 UTC)
loading tasks         288ms  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 71%
loading grunt-svgmin   62ms  ▇▇▇▇▇▇▇ 15%
svgmin:dist            53ms  ▇▇▇▇▇▇ 13%
Total 404ms
    [33mWarning: [4mRunning "imagemin:dist" (imagemin) task[24m
[31mFatal error: Cannot read property 'contents' of undefined[39m

Execution Time (2015-11-14 09:31:48 UTC)
loading tasks                   373ms  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 58%
loading grunt-contrib-imagemin  180ms  ▇▇▇▇▇▇▇▇▇ 28%
imagemin:dist                    91ms  ▇▇▇▇▇ 14%
Total 645ms Use --force to continue.[39m

    [31mAborted due to warnings.[39m

Execution Time (2015-11-14 09:31:46 UTC)
loading tasks                 263ms  ▇▇▇▇ 10%
loading grunt-contrib-jshint   88ms  ▇▇ 3%
jshint:all                    580ms  ▇▇▇▇▇▇▇▇ 22%
jshint:test                   257ms  ▇▇▇▇ 10%
jsinspect:inspect             122ms  ▇▇ 5%
wiredep:app                   110ms  ▇▇ 4%
loading grunt-usemin           29ms  ▇ 1%
concurrent:dist                1.1s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 41%
Total 2.6s

i had meet this error
but i don't know how to fix this.....

I can confirm that his happens with node v5.1.0.
I've regressed to node v0.12.7, reinstalled the packages and the error is gone.

after updating to the latest(v1.0.0) ,it is gone, so closing this issue


Add docs badge to README

Hi there,
as requested here: https://github.com/inch-ci/inch_ci-web/pull/49
Thanks for giving this a shot! :+1:

Nice!


Add SRI hashes to cdn hosted content

What's changed, or what was fixed?

Adds SRI hash to CDN hosted scripts

Whoops!
There were 47 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/fundamentals/accessibility/focus/using-tabindex.md#L239 - '

:tada: This has been pushed live to https://developers.google.com/web/


Add yaml support additionally to json

Done: https://github.com/vovagorodok/ble_ota_app/commit/19cc508c326b22f0ef51f9c48efd72f9f4013e12


Capes don't seem to load their textures correctly, only show bare material.

It's as the title says, none of the capes seen to have working textures, so only their bare materials and colors show.

Yep. I'm having this issue as well. Also, it seems it will sometimes duplicate the cape, so when you make edits it just gives you a new edited cape, and you can't disable the first one.

Fixed in the latest manual publish.