Running "ddev config" with project type "drupal9" creates two "homeadditions" directories

I just ran "ddev config" in an empty folder with the following setup:
Creating a new ddev project config in the current directory (/home/josh/myprojects/test)
Once completed, your configuration will be written to /home/josh/myprojects/test/.ddev/config.yaml

Project name (test):

The docroot is the directory from which your site is served.
This is a relative path from your project root at /home/josh/myprojects/test
You may leave this value blank if your site files are in the project root
Docroot Location (current directory):
Found a php codebase at /home/josh/myprojects/test.
Project Type [backdrop, drupal6, drupal7, drupal8, drupal9, laravel, magento, magento2, php, shopware6, typo3, wordpress] (php): drupal9
Ensuring write permissions for test
No settings.php file exists, creating one
Existing settings.php file includes settings.ddev.php
Configuration complete. You may now run 'ddev start'.

After running "ddev start" I realised, that there are two seemingly identical "homeadditions" folder in the .ddev directory. One named ".homeadditions" and the other "homeadditions":

Hi @joshsedl -
The one you're supposed to look at is called "homeadditions". The other one is ".homeadditions", (with a dot before it). On all operating systems besides Windows, you wouldn't see the second one by default, it's hidden.
It's part of the implementation of homeadditions feature, stuff from the global homeadditions (homeadditions in the .ddev/homeadditions directory of your home directory) is copied into the project so it can be provided and merged as well.
Thanks for using ddev, and good eye! I'll close this, but happy to continue the conversation here or elsewhere.

Ah alright, thanks for the fast reply @rfay ! Appreciate it :)


List the matches per metadata element ordering them by matching score

done as per current UI - please create new specific tasks for enhancements


Cannot copy/paste folder location

Description
I usually use copy/paste folder location in my workflow since most of my designs I send as a file via WhatsApp windows application but apparently it is not possible to copy a location from Files and paste on Windows Explorer. I've tried to paste on notepad and not even that works
ps: the github steps ask me to to go File > Settings > About to see the version - Even when I click on the Copy button to copy the Files version, it doesn't work
Steps To Reproduce

Go to the address bar
Select and copy the adress
Open windows application that has to interact with Windows Explorer to send a file
Paste the address
doesnt work

Expected behavior
Copying the address should behave as normal, a text string
Files Version
2.3.6
Windows Version
Edition	Windows 11 Pro Version	21H2 Installed on	13/6/2022 OS build	22000.795 Experience	Windows Feature Experience Pack 1000.22000.795.0
Relevant Assets/Logs
debug.log
debug_fulltrust.log

When you say you tried pasting in notepad do you mean from Files to notepad or WhatsApp to notepad

Hello! I just received a notification that case #9722 was closed since it
is completed, yet, I still can't copy and paste paths, here is a gif I made
to illustrate better![image: ApplicationFrameHost_bx5gOQKLfr.gif]
Em sáb., 13 de ago. de 2022 às 23:29, Josh-65 @.***>
escreveu:

When you say you tried pasting in notepad do you mean from Files to
notepad or WhatsApp to notepad
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/9722#issuecomment-1214271115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHEIPQ7CAYDOJXS23FPDB3VZBKZVANCNFSM56OLRCOQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
[image: photo]
Vinícius Cunha
Designer / Printer
M<PHONE_NUMBER>5
E @.***

https://imgur.com/a/gxgOueK <- gif link

The issue was closed since we didn't have enough information to debug. To be specific, it would be helpful to answer Josh's question.

I did answer the question as you can see in the attachment
Files paths can't be copied and pasted at all, I thought it was only on
WhatsApp but it's anywhere, even on windows explorer
Vinícius Cunha
On Mon, Aug 29, 2022, 11:46 Yair Aichenbaum @.***>
wrote:

The issue was closed since we didn't have enough information to debug. To
be specific, it would be helpful to answer Josh's question.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/9722#issuecomment-1230414056,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHEIPQRVBIUMLFK56JLNM3V3TENHANCNFSM56OLRCOQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks for the response, I can't access the attachment on my end 👍
If the same issue is happening from File Explorer, it's possible something is messed up with the Windows install.

Windows explorer works fine, it's Files that doesn't allow copy pasting of
file/folder paths
Vinícius Cunha
On Mon, Aug 29, 2022, 15:19 Yair Aichenbaum @.***>
wrote:

Thanks for the response, I can't access the attachment on my end 👍
If the same issue is happening from File Explorer, it's possible something
is messed up with the Windows install.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/9722#issuecomment-1230689222,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHEIPVZ3NIZPA4SXYGUD2TV3T5KLANCNFSM56OLRCOQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Where are you pasting the file path?

from files to notepad and from files to whatsapp - actually for any
windows program, just can't paste anywhere
Em sáb., 13 de ago. de 2022 às 23:29, Josh-65 @.***>
escreveu:

When you say you tried pasting in notepad do you mean from Files to
notepad or WhatsApp to notepad
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/9722#issuecomment-1214271115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHEIPQ7CAYDOJXS23FPDB3VZBKZVANCNFSM56OLRCOQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
[image: photo]
Vinícius Cunha
Designer / Printer
M<PHONE_NUMBER>5
E @.***

Is this still an issue in the preview version? https://files.community/download/preview


When adding property via config builder, add it to all sources

Simplifies explicit configuration.
E.g. we support configuring proxy with common system properties. If someone wants to quickly experiment with configuration, we want them to throw any properties into the config builder and return them if they requested by the same name.
var configuration = new ConfigurationBuilder()
    .putProperty("sys.prop.name", "value")
    .build();

// works today:
var property = ConfigurationPropertyBuilder
    .ofString("prop.name")
    .systemPropertyName("sys.prop.name")
    .build()

assertEquals("value", configuration.get(property));

// fails today, fixed by this PR
assertEquals("value", configuration.get("sys.prop.name"));

API change check
API changes are not detected in this pull request.


feat: page-number-based pagination

This PR:

Introduces page-number-based pagination to the homepage.
Refactors much of the database-related code to be more straightforward.
Changes the homepage to show only the items posted in the last week.
Supercedes #236
Moves functionality closer to #211
Closes #233
Closes #231

Note: once merged, these same changes can be made to paginate item and user pages.
Please feel free to review, @brunocorrea23 and @huai-jie.

Does this PR gonna tackle New votes per day  functionality that is mentioned in https://github.com/denoland/saaskit/issues/202?

Not so much. That was tackled in #235.

@iuioiua , thanks for the work here and for requesting the review.
Really liked the db refactor.
Couple of comments on the proposed solution.

It seems that the items older than a week are never accessible to the users, right? Is this the intended behavior?
'Loading' all items for the last week provides no guarantee of the amount of items that will be 'loaded'. Although it is better than loading the whole db at once, I feel this kind of defeats the purpose of pagination.

Wdyt?

It seems that the items older than a week are never accessible to the users, right? Is this the intended behavior?

Yes, as it stands now. Later, we can add functionality to make the window customisable.

'Loading' all items for the last week provides no guarantee of the amount of items that will be 'loaded'. Although it is better than loading the whole db at once, I feel this kind of defeats the purpose of pagination.

True, the amount of loading is sub-optimal. This will be fixed if/when Deno.Kv.count() is implemented.
Does the PR look good to you?

ok, thanks for the clarification!
LGTM


[Automatic validation] Stale branches

How to fix
Remove branches, that have not been updated in 90 days or more. Stale branches are listed in https://github.com/by-pinja/azure-subscription-cleaner/branches/stale. We also recommend automatically removing merged branches, this can be enabled from https://github.com/by-pinja/azure-subscription-cleaner/settings#merge-button-settings -page.
About validation
These issues are created, closed and reopened by repository validator when commits are pushed to repository.
If there are problems, please add an issue to repository validator
DO NOT change the name of this issue. Names are used to identify the issues created by automation.

[Automatic validation]: Issue fixed. Closing issue.


bugfix: restore config after update fail

if pouch update a container get error, restore container
config, aviod container record error config.
Fixes: #1512
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Fixes: #1512
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1513 into master will decrease coverage by 0.08%.
The diff coverage is 35.29%.

@@            Coverage Diff             @@
##           master    #1513      +/-   ##
==========================================
- Coverage   40.56%   40.48%   -0.09%
==========================================
  Files         254      254
  Lines       16535    16547      +12
==========================================
- Hits         6708     6699       -9
- Misses       8965     8980      +15
- Partials      862      868       +6

Impacted Files
Coverage Δ

daemon/mgr/container.go
48.8% <35.29%> (-0.49%)
:arrow_down:

ctrd/watch.go
75% <0%> (-3.13%)
:arrow_down:

ctrd/image.go
77.71% <0%> (-1.72%)
:arrow_down:

ctrd/container.go
49.45% <0%> (-0.73%)
:arrow_down:

daemon/mgr/network.go
68.94% <0%> (-0.49%)
:arrow_down:

I think this issue is quite severe, and we need to fix this ASAP.

since this problem is severe, i will first merge it.
LGTM


Add support LPC54114 & LPC54608 LPCXpresso boards

cc @c1728p9 @flit @jasonNXP

Hi Russ, is there any additional information required for the PR.

Hi @mmahadevan108, no additional information is needed on this PR. I'll let you know if that changes.

I have included support for LPC54608 in this branch as well.

@c1728p9 Can this be integrated into the code-base.

I think the reason global variables were getting overwritten was due to this bug:
https://github.com/mbedmicro/DAPLink/pull/283

Can this be integrated into the code-base.

Seems good to me. @sg- can you merge?


Project can't fetch folders

Did you ever add any folder to the projects?

Would you check if there are any error messages in the console?

no error messages，if i use project manager open project，project folder should auto displayed in the sidebar，but it not  displayed

I know how it happens，I can not add two project in one of the open sublime text（at once），when i add one project，ctrl+shift+n （open new sublime text）add another ，it works！！! thank you!

Glad that you figured it out.


Skill Tree + Abilities

Implement Skill Tree + UI
Map Passive Stat Bonuses to Player Stats
Map Active Abilities to Player's Input

Plus ability to select player abilities

 Skill Tree + Abilitiese


Add team member's names to the project card

Ref #33

Thanks, @flickz. Close - can we make it so that it says "and others", instead of cutting off mid-name? See Fatima, below.

@RichardLitt Updated

Looks great! Thank you!


[doc] unbloat readme, fix eol-whitespace

I think the pets table is easier to grasp as a table when it has columns and requires less scrolling.

Coverage remained the same at 99.697% when pulling faaff4cd4a400b770d2bd3f549c8eba001145047 on mk-pmb:pr-doc-unbloat-readme into 6d749396e3ad1459cf902ba8fdbceb33c8da421c on SvSchmidt:master.

Merged, thank you!

Coverage remained the same at 99.697% when pulling faaff4cd4a400b770d2bd3f549c8eba001145047 on mk-pmb:pr-doc-unbloat-readme into 6d749396e3ad1459cf902ba8fdbceb33c8da421c on SvSchmidt:master.


:bug: Update command recognizing regex

Description
Updated the regex for command recognizing jack commands
Fixes #37
Type of change

Updated old regex to split the incommingMessage content to "/\s+/" , previously it was " "
(whitespace)

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Tested in discord server with all basic commands
Note : Testing on moderation commands not done

[x] Discord server

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

@Developer-R-7 Have you tested everything after changing the code? If yes, please attach some screenshots.

Yes ,I have done test with the basic commands
I haven't test for moderation commands
Screenshot

Yes ,I have done test with the basic commands
I haven't test for moderation commands

Looks good to me. Merging! ✨


Even if I delete it, WebView (or other?) remains.

Describe the bug
Let's say you have a program that creates a window to play videos.
And when I created multiple of them and then deleted some, the video in the browser that I had deleted would still be playing.
Steps To Reproduce
Prepare code of wry/examples/multi_window.rs.
Add a below snipet (YouTube embed) to here str.
<iframe class="background" id="background"
src="https://www.youtube.com/embed/JUGHD_YWOe4?iv_load_policy=3&loop=1&playlist=JUGHD_YWOe4&control=0&fs=0&mute=On&end=111&modestbranding=1&start=11&autoplay=1"
frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope;
picture-in-picture" allowfullscreen="" style="width: 1191px; height: 678px;"></iframe>

Create about five windows every almost few seconds.
After a while, when it gets too noisy, close the window a little at a time and try to get only one.
Then it remains noisy.
So the closed window is still in motion.
Expected behavior
To be successfully closed.
Screenshots
Nothing
Platform and Versions (please complete the following information):
OS: macOS 12.3
Rustc: 1.59.0
Would you assign yourself to resolve this bug?

[ ] Yes
[x] No

Additional context
Nothing

This case seems to occur on MacOS :eyes:

And I could reproduce this issue on MacOS.

Will this help? https://stackoverflow.com/questions/21161406/freeing-ios-uiwebview-resources-after-usage
-(void) dealloc
{

    [self.webview cleanForDealloc];
    self.webview = nil;
    [super dealloc];

}

I think it's this one: https://stackoverflow.com/questions/58454042/wkwebview-instances-and-memory-leak
I tried to test it, but i can't get obj-c to cooperate with me :joy:
On that note i'm going to re-open this issue because we're leaking memory right now because of this issue.

I try to fix this issue based on https://github.com/tauri-apps/wry/issues/536#issuecomment-1120459496.
I would appreciate if you review https://github.com/tauri-apps/wry/pull/587 :pray:

Also maybe we need to remove WKWebsiteDataStore by using removeDataOfTypes.


Update Firefox versions for api.HTMLElement.dragexit_event

17376

 Quality gate: NA
Details

 Quality gate: NA
Details


Update vt-guide.blade.php

Add message to V+T
typos corrected

:tada: This PR is included in version 6.27.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Implement @derivedFrom for non-list fields

This allows to define types like
type Domain @entity {
  id: ID!
  ...
}

type DomainAuction @entity {
  id: ID!
  domain: Domain @derivedFrom(field: "id")
  ...
}

and resolve them in queries by virtue of the two sides sharing the same ID.

Updated the tests to include @derivedFrom on a non-list field.

As we were discussing on Discord, if more than one entity has the matching id, this may return any of the matching entities as the value. We can consider that to be a logic error by the user, but we don't catch it in any way. So I'm not confortable merging until we either come up with a better solution or at least have a wider discussion to check if we are ok with the implications.

Ideas:

We could query matches with first: 2 instead of first: 1 and return a GraphQL error if there is more than one. This is the easiest solution, but makes the user deal with the problem rather than the subgraph author.

When writing an entity, we could check if there are any non-list @derivedFrom fields that involve the entity type. If there are any, check if there already is an entity of the same entity type that matches that @derivedFrom. If that's the case fail, similar to how we fail with ID clashes for interfaces.

@Jannis I think the first idea is good enough to allow landing this.
Having an integrity check that prevents the situation from arising is good in theory, but it's complicated. Your second idea solves half of the problem, but there is also the case that two entities already exist referencing the same id, and then a deriving entity with that id is added.

@leodasvacas Ok, first approach implemented!

@leodasvacas I guess... 😬


feat(deployment): upgrade strategy

Resolves https://github.com/cdk8s-team/cdk8s-plus/issues/365

💚 All backports created successfully

Status
Branch
Result

✅
k8s-21/main

Questions ?
Please refer to the Backport tool documentation

💚 All backports created successfully

Status
Branch
Result

✅
k8s-20/main

Questions ?
Please refer to the Backport tool documentation


loginWithRedirect behaving different between dev and prod with nextjs

Checklist

[X] The issue can be reproduced in the auth0-react sample app (or N/A).
[X] I have looked into the Readme, Examples, and FAQ and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
I'm working on a NextJS app that is completely being rendered statically by way of NextJS's Automatic Static Optimization feature. Using the next dev and next build + static export or next start has different behaviour when using LoginWithRedirect.
In dev mode it behaves as expected passing redirect_uri, organization and invitation in the authorization parameters.
When running in production mode the organization and invitation parameters are not respected in the resulting url used for redirection.
Reproduction

Use a nextjs project but only with front end functionality, nothing server side
Conditionally pass organization & invitation parameters when present.
Redirect url doesn't contain either of those parameters

Additional context
Using Next 13.3.2
Query params in production mode also have a series of strange parameters at the beginning:
https://{DOMAIN}.us.auth0.com/authorize?0=d&1=2&2=5&3=m&4=V&5=V&6=R&7=Z&8=d&9=H&10=B&11=C&12=Q&13=T&14=h&15=k&16=c&17=X&18=R&19=P&20=Y&21=k&22=x&23=Y&24=d&25=m&26=s&27=w&28=S&29=m&30=J&31=J&32=T&33=E&34=t&35=R&36=b&37=2&38=1&39=V&40=b&41=E&42=V&43=W&44=a&45=k&46=h&47=C&48=Z&49=W&50=J&51=x&52=S&53=T&54=J&55=p&56=a&57=w&58=%3D&59=%3D&client_id={CLIENT_ID}&scope=openid+profile+email+offline_access&audience={AUDIENCE}&response_type=code&response_mode=query&state={STATE}&nonce=cGQ3X35mTjNTZWpRNWNxYVZMQ0YwWFI5LnhrMUMtX0hSV3BXbER5SGZjLg%3D%3D&redirect_uri={REDIRECT_URI}&code_challenge=CfuO2VKa_FGNvb9Ol_1CK04zu2QugavZyOg3_InNUmI&code_challenge_method=S256&auth0Client={AUTH0_CLIENT}
Redirect url in dev mode looks like:
https://dev-ws8z8uiswqyyrcam.us.auth0.com/authorize?client_id={CLIENT_ID}&scope=openid+profile+email+offline_access&audience={AUDIENCE}&organization={ORGANIZATION}&redirect_uri={REDIRECT_URI}&response_type=code&response_mode=query&state=RkdiTjVHSldqUTlRbGY1anJfTDJNWDM0NWFLZ0l5NVZZfm9nWnBoTWdqZw%3D%3D&nonce=UlRrbnRpOFFZcXJHVllIOH5VNk1sTUUtTEpOM0F0RXg4Y2VRaEN2d0JjUg%3D%3D&code_challenge=03VHmwHDV56Tz7Wpe8qQtZEb_y692NueGqhmSyeJ4Ls&code_challenge_method=S256&auth0Client={AUTH0_CLIENT}
Note: I have not tested with the sample app at this point in time
auth0-react version
2.2.1
React version
18.2
Which browsers have you tested in?
Chrome, Other

Hi @ian-weir
It looks like you're passing an array as your authorizationParams in prod
new URLSearchParams({ client_id: 'foo', ...'foobar'.split('') }).toString();
// '0=f&1=o&2=o&3=b&4=a&5=r&client_id=foo'

I'd need to see some code the reproduces the issue if you want me to debug it for you

Hi @adamjmcgrath as far as I can tell, I'm not passing an array anywhere.
Here's the snippet that handles the loginWithRedirect for invitations
const authorizationParams: AppState  = {
            organization: params.get('organization'),
            redirect_uri: `${baseUrl.current}/callback`
        }

        if(params.get('invitation')) {
          authorizationParams.invitation = params.get('invitation')
        }

        await loginWithRedirect({
          authorizationParams,
          appState: {
            returnTo: '/users/user/get-current-user-id/dashboard'
          }
        })

and here's the one that handles when the user clicks our login button:
const organizationInHostname = hostname.current.split('.')[0]
    const authorizationParams: AppState = {
      redirect_uri: `${baseUrl.current}/callback`,
    }

    if(organizationInHostname !== 'login') {
      authorizationParams.organization = organizationInHostname
    }

    await loginWithRedirect({
      authorizationParams,
      appState: {
        returnTo: '/users/user/get-current-user-id/dashboard'
      }
    })

While this is the Provider
<Auth0Provider
          domain={domain}
          clientId={clientId}
          authorizationParams={{
            audience
          }}
          onRedirectCallback={onRedirectCallback}
          useRefreshTokens={true}
          cookieDomain={cookieDomain.current}
          cacheLocation="localstorage"
        >

Just an update I was able to work around this by manually appending the query params in the openUrl method

Hi @ian-weir
Could you log authorizationParams before you call loginWithRedirect in production mode and share the result?

Sure thing: {invitation: "foo", organization: "bar", redirect_uri: "http://localhost:3000/callback"}
Here's the resulting URL: https://dev-ws8z8uiswqyyrcam.us.auth0.com/authorize?0=U&1=z&2=l&3=V&4=Z&5=E&6=R&7=Y&8=V&9=W&10=M&11=t&12=V&13=0&14=F&15=l&16=Q&17=m&18=R&19=u&20=T&21=m&22=p&23=y&24=T&25=k&26=F&27=P&28=S&29=k&30=9&31=v&32=V&33=W&34=Z&35=o&36=Q&37=m&38=8&39=4&40=S&41=m&42=t&43=o&44=N&45=1&46=9&47=Y&48=X&49=0&50=Q&51=0&52=R&53=2&54=1&55=W&56=U&57=w&58=%3D&59=%3D&client_id={CLIENT_ID}&scope=openid+profile+email+offline_access&audience={AUDIENCE}&response_type=code&response_mode=query&state=UzlVZERYVWMtV0FlQmRuTmpyTkFPSk9vVWZoQm84SmtoN19YX0Q0R21WUw%3D%3D&nonce=MzI2YzF2VWdmM2tNZDFYZWthbGo4YUJUcWU5SE9VRV80MnMwV1pMNWdBeA%3D%3D&redirect_uri=http%3A%2F%2Flocalhost%3A3000%2Fcallback&code_challenge=rfTPyK0X8mO9g1nXL-KQ70spNRLq95vsaDSgZ51QZHw&code_challenge_method=S256&auth0Client={AUTH0_CLIENT}

Hi @ian-weir - thanks for sharing those
When I call
await loginWithRedirect({
  authorizationParams: {invitation: "foo", organization: "bar", redirect_uri: "http://localhost:3000/callback"}
})

I get redirected to .../authorize?redirect_uri=http%3A%2F%2Flocalhost%3A3000%2Fcallback&invitation=foo&organization=bar&response_type=code&...
So can't reproduce your issue.
In order to help you - I'll need a repo or a link that demonstrates the issue.

Sorry, spaced on this thread. Unfortunately I can't give a repo as it's for work and we're not open source. As for link I can't do that for now either as we haven't launched yet. @adamjmcgrath Out of curiosity what version of React, Next, & auth0-react were you using?


Test failure: //backends/gpu/cpp_tests:event_manager_test

The GCC + NVCC build shows a failure under //backends/gpu/...:
bazel test --config=cuda --config=gcc //backends/gpu/...
...
INFO: Elapsed time: 1348.192s, Critical Path: 952.08s
INFO: 4412 processes: 55 internal, 4357 linux-sandbox.
INFO: Build completed, 1 test FAILED, 4412 total actions
//backends/gpu/cpp_tests:memory/block_allocator_test                     PASSED in 0.4s
//backends/gpu/cpp_tests:memory/gpu_buffer_test                          PASSED in 0.2s
//backends/gpu/cpp_tests:memory/sub_allocator_test                       PASSED in 0.1s
//backends/gpu/cpp_tests:module_table_test                               PASSED in 0.9s
//backends/gpu/cpp_tests:stream/wrapper_test                             PASSED in 3.5s
//backends/gpu/mlir_tests/core_runtime:current_context_lifetime.mlir.test PASSED in 6.6s
//backends/gpu/mlir_tests/core_runtime:get_device.mlir.test              PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:logging_with_results.mlir.test    PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:op_handler.mlir.test              PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:tensor_conversion.mlir.test       PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:test_ops.mlir.test                PASSED in 6.5s
//backends/gpu/mlir_tests/core_runtime:tf_addv2.mlir.test                PASSED in 6.6s
//backends/gpu/mlir_tests/core_runtime:tf_biasadd.mlir.test              PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:tf_cast.mlir.test                 PASSED in 6.5s
//backends/gpu/mlir_tests/core_runtime:tf_const.mlir.test                PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:tf_conv2d.mlir.test               PASSED in 7.3s
//backends/gpu/mlir_tests/core_runtime:tf_fusedbatchnorm.mlir.test       PASSED in 6.6s
//backends/gpu/mlir_tests/core_runtime:tf_matmul.mlir.test               PASSED in 6.5s
//backends/gpu/mlir_tests/core_runtime:tf_maxpool.mlir.test              PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:tf_mean.mlir.test                 PASSED in 0.5s
//backends/gpu/mlir_tests/core_runtime:tf_pad.mlir.test                  PASSED in 6.5s
//backends/gpu/mlir_tests/core_runtime:tf_reduction_ops.mlir.test        PASSED in 6.6s
//backends/gpu/mlir_tests/core_runtime:tf_relu.mlir.test                 PASSED in 6.5s
//backends/gpu/mlir_tests/core_runtime:tf_softmax.mlir.test              PASSED in 1.5s
//backends/gpu/mlir_tests/core_runtime:tf_tanh.mlir.test                 PASSED in 6.7s
//backends/gpu/mlir_tests/core_runtime:tf_transpose.mlir.test            PASSED in 6.6s
//backends/gpu/mlir_tests/cuda:cublas.mlir.test                          PASSED in 1.0s
//backends/gpu/mlir_tests/cuda:dnn.mlir.test                             PASSED in 1.6s
//backends/gpu/mlir_tests/cuda:event.mlir.test                           PASSED in 0.4s
//backends/gpu/mlir_tests/cuda:launch.mlir.test                          PASSED in 0.5s
//backends/gpu/mlir_tests/cuda:memcpy.mlir.test                          PASSED in 0.4s
//backends/gpu/mlir_tests/cuda:module.mlir.test                          PASSED in 0.5s
//backends/gpu/mlir_tests/cuda:stream.mlir.test                          PASSED in 0.5s
//backends/gpu/cpp_tests:event_manager_test                              FAILED in 227.8s
  /home/lemo/.cache/bazel/_bazel_lemo/b9a83ea24871c2ac3b49c420e1a458ae/execroot/tf_runtime/bazel-out/k8-opt/testlogs/backends/gpu/cpp_tests/event_manager_test/test.log
Executed 34 out of 34 tests: 33 tests pass and 1 fails locally.

Configuration details: GCC 8.4, CUDA 11.1, Ubuntu 18.04 (x64). The GPU is a Titan RTX.

Running the failed test indicates a hard crash:
Running bazel-bin/backends/gpu/cpp_tests/event_manager_test
Run on (32 X 3400 MHz CPU s)
CPU Caches:
  L1 Data 32K (x16)
  L1 Instruction 64K (x16)
  L2 Unified 512K (x16)
  L3 Unified 8192K (x4)
***WARNING*** CPU scaling is enabled, the benchmark real time measurements may be noisy and will incur extra overhead.
I 2021-03-18 19:06:28.259879<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:28.259999<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:28.373243<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:28.373338<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:28.468926<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:28.469015<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:28.555418<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:28.555507<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:28.654265<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:28.654355<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:28.762471<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:28.762561<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:29.017772<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:29.017860<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:29.818921<PHONE_NUMBER>81376 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:29.819012<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
-----------------------------------------------------------------------------
Benchmark                                      Time           CPU Iterations
-----------------------------------------------------------------------------
BM_EventEnqueue                              161 ns        161 ns    4399306
I 2021-03-18 19:06:29.911051<PHONE_NUMBER>95968 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:29.911129<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:30.011420<PHONE_NUMBER>95968 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:30.011515<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:30.102653<PHONE_NUMBER>95968 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:30.102724<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:30.219432<PHONE_NUMBER>95968 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:30.219521<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:30.487184<PHONE_NUMBER>95968 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:30.487267<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:06:31.845045<PHONE_NUMBER>95968 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:06:31.845123<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
...
BM_MoreStreamsThanThreads/1/10            860457 ns     111927 ns       5371
I 2021-03-18 19:10:23.208142<PHONE_NUMBER>28160 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:10:23.208226<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:10:23.321154<PHONE_NUMBER>72384 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:10:23.321245<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:10:23.512955<PHONE_NUMBER>59136 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:10:23.513032<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:10:24.461173<PHONE_NUMBER>59136 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:10:24.461241<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
I 2021-03-18 19:10:28.868175<PHONE_NUMBER>59136 event_manager.cc:82] EventManager worker thread cancelled.
I 2021-03-18 19:10:28.868265<PHONE_NUMBER>70976 event_manager.cc:154] EventManager destroyed with 0 pending events.
BM_MoreStreamsThanThreads/8/10            864058 ns      90383 ns       4971
Segmentation fault (core dumped)

From a quick glance, it seems that the entries in tfrt::HostContext::all_host_contexts_ are never freed, and next_host_context_index eventually overflows

Thanks for reporting this issue. It should have been fixed in commit d1d7f44f.

I can confirm that the test case now passes.
//backends/gpu/cpp_tests:event_manager_test                              PASSED in 239.1s


DT-1983: 🔒 Add security policy

Link to relevant moj slack threads:
https://mojdt.slack.com/archives/C52LPHPK8/p1614275799048400
https://mojdt.slack.com/archives/C52LPHPK8/p1620131693171600
It sounds like all MoJ public projects should at the very least have a security policy defined.

Hi @jakemulley, thanks for the review.
I had a read of the health file and then added the file to that repo as suggested - https://github.com/ministryofjustice/.github/blob/main/SECURITY.md.
Had a bit of a panic when I realised that my branch (pgp-DT-1983-security-policy) automatically became the default branch because no-one had committed to that repo before and no branch protection policy is defined.  Have now renamed it to main.  Worth defining a branch protection policy so that future commits can go through a PR process - I can do that if you'd like me to.
Looks like the security policy file is now working a treat on all the MoJ repos - hadn't realised that it was so straightforward, thanks.  Please have a check to ensure you're happy by looking at any of the MoJ repos e.g. https://github.com/ministryofjustice/template-repository/security/policy.
Totally agree on the security policy being defined in a blog post, I'd noticed that at the time and thought it seemed crazy.  I'll pick up with security tomorrow and see if they fancy moving it.  It is even referenced from their security guidance github repo at https://ministryofjustice.github.io/security-guidance/vulnerability-disclosure-policy/#vulnerability-disclosure-policy.


[BUG] PUT Alias creates index and not an alias

What is the bug?
When creating aliases it returns successful but actually creates an index instead.
How can one reproduce the bug?
resp, err := b.openSearch.Indices.Alias.Put(context.Background(), opensearchapi.AliasPutReq{
		Alias:   alias,
	})
	b.logger.Info("Created alias", slog.Any("response", resp))
	return err

What is the expected behavior?
An alias is created
What is your host/environment?
osx
Do you have any screenshots?
Do you have any additional context?

Want to try and write a (failing) test + fix for this @vrecan?


Phylogenetic tree: job completed, tree not rendered

One of my tree jobs completed successfully.
In the HTML tree it produced, some of the nodes have genome names, some show only genome ids.
When I click on the tree viewer, nothing is displayed.
<EMAIL_ADDRESS>Trees/.Escherichia and Shigella Tree RAxML/Escherichia and Shigella Tree RAxML.html

I believe this has been fixed or no longer an issue.


[Release] Missing v0.10.dev0 is generating wrong dev version numbers

After the release of v0.9.0, we'll need to find the exact point in the repository that should be tagged with the beginning of v0.10.dev0, so that the tooling in this repository can start producing the correct version numbers in the packages.
At the moment, we're still generating v0.9.devX versions, which are wrong.
The hash https://github.com/apache/tvm/commit/97b3076c3532f73a9d9eeba26a3f329f8e0f803d seems to be the latest one that  v0.9.0 and main have in common, so I wonder if https://github.com/apache/tvm/commit/0e23122846aa3b7a5350102d8c06fa21695d34be should be the one that marks the beginning of 0.10 development cycle.
Creating a tag is simple, but the main motivation I created this issue is to see whether our release guide should be amended to reflect the tagging of the beginning of next cycle to be done as part of the release process. What do others think?
cc @u99127 @areusch @u99127 @driazati @manupa-arm

I agree we should document this in our release process

Yeah we shouldn't wait until a release next time to bump the version numbers, we should do them as soon as we do the release candidate branch cut, that way there's no gaps. At that point the only common commit will be the one before the branch cut since any further changes will be individually cherry-picked into the rc.

I think in this case it is not about version number, but to create a tag v0.10.dev0 in the branching off pt when we cut off the release branch

OK, so in the interest of having the package being generated this Wednesday with the right name on PyPI, I'm planning to tag https://github.com/apache/tvm/commit/0e23122846aa3b7a5350102d8c06fa21695d34be with v0.10.dev0, while we discuss how the release guide should be amended.


Handle exception pickling

See the comments on PR #47 at the end -- @ooq is still seeing unpickleable exceptions.

@ooq We were referencing an exception that you had seen that couldn't be serialized and yet still had a problem -- do you have a testcase perhaps ? I can't find the original problem.

@ericmjonas Sorry that I could not recall the exact exception I had.
Just curious, do we have a rough list of 1) what exceptions cannot be pickled? 2) what exceptions can be pickled but cannot be unpickled?
And, is Exception the only object that causes pickle/unpickle issues? Just want to make sure we don't have to stress out about exc_traceback. If exc_traceback is not the worry, why are we doing str(exc_traceback) here?

Sometimes exc_traceback contains pieces of the exception that cannot be pickled and I wanted to be 100% sure things propagated correctly. I don't have a list, but we have example exceptions that I test in test_pickle.py where we create objects (and exceptions) that fail on either pickle or unpickle.


1.21 Update

I know I'm very early, but your mod is one of the ones we would like to use on our 1.21 server so an update would be nice. No pressure though! :D

Nah, you're an hour late :P
I will port it, but for now, a dependency override should work.
https://fabricmc.net/wiki/tutorial:dependency_overrides

@cqm3ron Uploaded to Modrinth :)


Bump version

Codecov Report

Merging #8 (086971d) into main (231f411) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          136       136
=========================================
  Hits           136       136

Impacted Files
Coverage Δ

abcattrs/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Remove chainerx.md

Fixes #6657

Jenkins, test this please.

Thanks, LGTM.

Jenkins CI test (for commit 92b46cf66d372a32982602fddce270486ae232da, target branch master) succeeded!


Terragrunt init fails while checking AWS config profile for S3 bucket

Hi ,
We already have terragrunt module for dev,qa, prod to install an application on AWS Ec2. I cloned the repository and I want to initialize the terragrunt to make sure its connecting with AWS or not. But Im getting below error while running 'terragrunt init' command on gitbash and powershell.
Any suggestion and help is most welcome. Thanks !
Set up have -
aws-cli/2.1.1 Python/3.7.7
terragrunt version v0.25.4
Terraform v0.13.4
Error -----------------------------------------------------
[terragrunt] 2020/12/03 21:54:49 Initializing remote state for the s3 backend
[terragrunt] 2020/12/03 21:54:49 Error finding AWS credentials (did you set the AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY environment variables?): NoCredentialProviders: no valid providers in chain. Deprecated.
For verbose messaging see aws.Config.CredentialsChainVerboseErrors

https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-files.html

Looks like an AWS auth error. Check out A Comprehensive Guide to Authenticating to AWS on the Command Line for instructions.

Closing this as the original question has been answered. If there are any follow ups, please open a new ticket with updated context. Thanks!


Only create the sarif files in a cloud build

note: TSA upload seems to have intermittent issues, not because of this change


Enable tests passed on POSIX

Enable all tests PASSED on POSIX platform but not passed on other platforms
@OHerasym, @dev-gh please review

Reviewed


[feat]: 🐋 Add start9 user to docker group

Prerequisites

[X] I have searched for existing issues that already suggest this feature, without success.

Describe the Feature Request
Elevate start9 user rights to run docker command without sudo.
Describe the Use Case
Transitioning from old habits working as root can be a painful experience.
Describe Preferred Solution
Lets make it bearable by simply adding start9 user to docker group.
usermod -aG docker start9 will do
Describe Alternatives
Listen to users asking about docker not working because:
Got permission denied while trying to connect to the Docker daemon socket

Anything else?
If accepted just tell me at what stage you want this to happen?

Ah, was about to add this but @k0gen beat me to it!


Potential early-adopter ✋, How to use it?

Hi @fmassa!
I like your next NestedTensor abstraction.

Should I take it by cp-ing this into my codebase?

Would you recommend to use this package? Can I pip install it?

My use case

I'm currently passing a dictionary into a couple of modules (loss, layers). However, each module take different tensors.

I did an awful hack in my dataloader to support this.

It seems that your abstraction serves my use case. That's what I understood from your DETR project. Am I wrong?
Cheers :beers:

Closing the issue as I misunderstood the purpose. Yet, I will keep an :eye: on this package.


Persist active language?

Hello Ryan, we have the react-localize-redux working in our project but currently we encounter an issue. If the user switch the language from English to French, he/she can view the French version of the website all the way util when they refresh the website.
Every time they refresh the website, the router will initialize the language again with the defaultLanguage (in our case it's English).
We saw that when action @@localize/SET_ACTIVE_LANGUAGE was called, the payload {languageCode:"fr"} was sent,  but it's not in the redux store.
Is there a way to persist the chosen language? We have redux-persist set up already, if the languageCode is saved anywhere, we can pick it up again and use it as the defaultLanguage when next time we initialize the localication.

@rockia I have added a FAQ question to answer this on the official docs that should answer your question.
How do I persist active language?

Never thought of using the native method. Thanks @ryandrewjohnson ! Localisation fully up and running, will deploy to production in our next release. Thanks so much for this awesome library and support for the last two weeks! Have a great upcoming Christmas, eh?


Question on LLMRequests

I'm trying to implement an agent that has a Yelp tool to find the best restaurants in an area. I have been using  LLMRequests, but it seem as though the beautiful soup output of the requests response is very noisy (see below). For something like this, should one implement a different chain (e.g. a 'Yelp' chain that integrates with Yelp API) instead?
 'Between >>> and <<< are the raw search result text from yelp\n    Extract the names of top restaurants or say "not found" if there are no results.\n    >>> \n\n\nSearch Businesses In San Francisco, CA - Yelp\nYelpFor BusinessesWrite a ReviewLog InSign UpRestaurantsHome ServicesAuto ServicesMoreMoreFilters$$$$$$$$$$SuggestedOpen Now\xa0--:--WaitlistFeaturesOffering a DealOffers DeliveryOffers TakeoutGood for KidsSee allNeighborhoodsAlamo SquareAnza VistaAshbury HeightsBalboa TerraceSee allDistanceBird\'s-eye ViewDriving (5 mi.)Biking (2 mi.)Walking (1 mi.)Within 4 blocksBrowsing San Francisco, CA businessesSort:RecommendedAllPriceOpen NowWaitlist1.\xa0Bi-Rite Creamery10004Ice Cream & Frozen Yogurt$$MissionThis is a placeholder“Nice little shop near Mission Dolores Park! Like a 1 min walk away\n\nThey had a buncha unique flavors to choose from, I tried banana, cream brûlée, honey…”\xa0more2.\xa0Brenda’s French Soul Food11895Breakfast & BrunchSouthernCajun/Creole$$TenderloinThis is a placeholder“Today I go there with my coworker and then have orders foods \nAfter having the food \nWe settle bills and the server charges more than usual price \nAfter that I…”\xa0more3.\xa0Gary Danko5800American (New)FrenchWine Bars$$$$Russian HillThis is a placeholder“Amazing service and food! An error was made with our reservation so we showed up and weren\'t on their books! After showing proof of the res, they offered for…”\xa0more4.\xa0Tartine Bakery8666BakeriesCafesDesserts$$MissionThis is a placeholder“So cute here! The treats were so delicious. Honestly, the brownies were so very delish! We had a a good variety of treats to enjoy and they were all really…”\xa0more5.\xa0Mitchells Ice Cream4623Ice Cream & Frozen YogurtCustom Cakes$Bernal HeightsThis is a placeholder6.\xa0Hog Island Oyster6782SeafoodSeafood MarketsLive/Raw Food$$EmbarcaderoThis is a placeholder“Great place. Fresh oysters ! I love the area and the view of the bay! Great food and ambiance!”\xa0more7.\xa0House of Prime Rib8300American (Traditional)SteakhousesWine Bars$$$Nob HillThis is a placeholder“I\'m not going to say much but this is basically the best Prime Rib restaurant in America. It was so good that I brought my mom here from out of town. Here is…”\xa0more8.\xa0Fog Harbor Fish House8700SeafoodWine BarsCocktail Bars$$Fisherman\'s WharfThis is a placeholderOutdoor seatingFull barLive wait time: 12 - 22 mins“First time here and I wasn\'t disappointed, the foods are yummy. The staffs are very good specially to our server Shawn.”\xa0moreFind a Table9.\xa0B Patisserie3196BakeriesPatisserie/Cake ShopMacarons$$Lower Pacific HeightsThis is a placeholder“One of my absolute favorite pastry shops in SF. Their Kouign amann are the best that I\'ve ever tasted. They have classic flavors (plain and chocolate) and also…”\xa0more10.\xa0Burma Superstar7367Burmese$$Inner RichmondThis is a placeholder“Great food. great service. \nOutdoor seating has heater when it\'s cold out. \nHighly recommend the place.”\xa0moreStart Order1234567891 of 24Can\'t find the business?Adding a business to Yelp is always free.Add businessGot search feedback? Help us improve.More NearbyActive LifeArts & EntertainmentAutomotiveBeauty & SpasEducationEvent Planning & ServicesFoodHealth & MedicalHome ServicesHotels & TravelLocal FlavorLocal ServicesNightlifePetsProfessional ServicesPublic Services & GovernmentRestaurantsShoppingMore categoriesAboutAbout YelpCareersPressInvestor RelationsTrust & SafetyContent GuidelinesAccessibility StatementTerms of ServicePrivacy PolicyAd ChoicesYour Privacy ChoicesDiscoverYelp Project Cost GuidesCollectionsTalkEventsYelp BlogSupportYelp MobileDevelopersRSSYelp for BusinessYelp for BusinessBusiness Owner LoginClaim your Business PageAdvertise on YelpYelp for Restaurant OwnersTable ManagementBusiness Success StoriesBusiness SupportYelp Blog for BusinessLanguagesEnglishCountriesUnited StatesAboutBlogSupportTermsPrivacy PolicyYour Privacy ChoicesCopyright © 2004–2023 Yelp Inc. Yelp, ,   and related marks are registered trademarks of Yelp.Some Data By Acxiom\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n <<<\n    Extracted:

yes! if there a good yelp api, it will probably be a lot easier to integrate with that

@pruksmhc did you manage to get a working Yelp chain integrated? Would appreciate any tips if you've got any.

Following this discussion!


How to set center item?

Is there any option/method to set center by manually(programatically)?
Please help?
Thanks in avdance.

@anik123
Call RecyclerView.scrollToPosition method

@mig35 i am getting
java.lang.IllegalArgumentException: position can't be great then adapter items count.
after using your solution.

@anik123
Are you using last library version ('com.azoft.carousellayoutmanager:carousel:1.1.7')?

My code is :
    layoutManager.setPostLayoutListener(new CarouselZoomPostLayoutListener());
    recyclerView.setLayoutManager(layoutManager);
    recyclerView.setHasFixedSize(true);
    recyclerView.setAdapter(adapter);
    recyclerView.addOnScrollListener(new CenterScrollListener());
    recyclerView.scrollToPosition(contacts.size()-1);

@anik123
have you looked what version do you use?

@mig35 that solve the issue. Thanks


Fix display shield testcase

Twister's extra arguments feature only works correctly if the fully
qualified platform name is given, which was not the case. Make all
platform names for the shield testcase fully qualified to resolve this.
Also, remove the LPC54114 from the test, as it was not supported (no arduino_gpio definition is present)
Fixes #80876

@hakehuang FYI, this should fix the issue on the lpcxpresso54114 board

@kartben friendly ping :)

Adding myself on the assignee to help move this along. Changes are specific to NXP platforms.


Fix default session timeout values to 30 minutes everywhere (close #542)

This is a tiny PR that just updates the default foreground and background timeouts in Tracker to 30 minutes to match the SessionConfiguration defaults and avoid confusion. Issue #542

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Protractor field editor

This field editor is specifically good for servos.

Since this includes a new field editor. You'll need to publish a new build of pxtblockly to deployment-config/doccdn


graal: generate part of the config

@imkiva

Does this mean the generator won't work if we start a fresh build (like in CI)? Because we will not have .class files.

bors d=ice1000

bors r+


TST: Removed regex warning for read_csv

Follow-up to #13481 and partially fixes #13932.  Regex mistakenly matches to the empty string, which will cause Python 3.x to issue a warning.  In addition, not sure why this test was placed in python_parsers_only.py...

Current coverage is 85.30% (diff: 100%)

Merging #13943 into master will decrease coverage by <.01%

@@             master     #13943   diff @@
==========================================
  Files           139        139
  Lines         50157      50157
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          42785      42784     -1
- Misses         7372       7373     +1
  Partials          0          0

Powered by Codecov. Last update b7abef4...b93325e

ty!


[$1000] iOS - Split bill - Keyboard displayed abruptly when split bill animation slides in

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/21310
Action Performed:

Open iOS app
Open group DM
Set focus to message composer (so keyboard is displayed)
Tap on "+" near composer
Tap on "Split bill"

Expected Result:
Split bill screen animation slides in smoothly with no unnecessary elements displayed
Actual Result:
Keyboard is still displayed for a couple of seconds when Split bill screen animation slides in
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/42ac89e6-2ece-428c-830a-6de75399c0e3
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fd7b79c5ae42e842
Upwork Job ID:<PHONE_NUMBER>936536576
Last Price Increase: 2023-06-26

Hi!
I'm Artem from Callstack - expert contributor group
Maybe I can help with this one?

@CortneyOfstad Can you please assign @waterim here

@s77rt Do you want me to remove your assignment in favor of @waterim, or have you both assigned?

@CortneyOfstad Both. I will be the C+ here for review

I am trying to run iOS application. For that i want to read iOS document.
But It is redirecting me to page asking my Enter your @expensify.com email to get started
I don't have this email. plz help setting up this.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01ecb1d62beec30401

@waterim Any updates here?

@s77rt Found a root of the issue, will add proposal today.

@s77rt Will send it tomorrow, found one issue with random black blink after the navigation

@s77rt I was reviewing Popover animation, everything was correct there. After I decided to try to restart IOS simulator and this black blink just disappeared.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Keyboard displayed abruptly after navigation to split bill screen
What is the root cause of that problem?
The main issue here is a behaviour of Native IOS keyboard. If the keyboard dismissal occurs after the navigation has started, it may result in a momentary blink as the new screen is displayed.
File we are working with: ReportActionCompose.js
What changes do you think we should make in order to solve the problem?
The best way to fix this issue is to hide Keyboard with Keyboard.dismiss() before the actual Popover with (Split Bill and Add Attachment) was opened. This will add a natural feeling of the application - Keyboard will disappear with a native animation before Popover is opened.
What alternative solutions did you explore?
N/A

@waterim Thanks for the proposal. I don't think your RCA is correct/complete. Your provided little info on why the bug is occurring in the first place. For instance why the keyboard is even getting pulled up? Is it because we are re-focusing the input after closing the modal or something else? I think the point that I'm trying to make here is that instead of pulling the keyboard up and then dismiss it, we should not even pull it in the first place.

@s77rt okay, I will add more information to the Proposal.
here is an answer to your question:
original behavior - we are not closing the keyboard itself when we are opening Popover, and after the navigation to the split bill screen we are dismissing the keyboard, thats why it still blinks for a second, because we are closing it only on the split bill screen.
In my proposal Im adding dismiss before even popover opens, with this solution we are getting smooth keyboard close before opening popover(like telegram for example has) and with this solution when we are navigating to a new screen we don’t have any keyboard pulled up before.

@waterim After we click on the plus grey button, the keyboard is already dismissed. Why after we click on "Split bill" it appears again?

@s77rt After additional investigation:
"Why after we click on "Split bill" it appears again?"

"grey button" opens a modal from react-native-modal which handles automatically the keyboard. It means it dismisses the keyboard on open and shows the keyboard after it close(if it was opened before we opened the modal itself)
After the click on the button "split bill" we are making a navigation and closing the modal and it means that our keyboard will show up. But on the page Split bill we have a logic to dismiss a keyboard and thats why keyboard will be opened just for a second.
With my solution we are closing the keyboard manually before modal is showing thats why it will not dismiss the keyboard automatically and will not open it again after modal is closed.(because it was closed before we opened the modal)

Heading OoO for the week (back July 10) so re-assigning this in the meantime 👍

@waterim Thanks for the follow up.

"grey button" opens a modal from react-native-modal which handles automatically the keyboard. It means it dismisses the keyboard on open and shows the keyboard after it close(if it was opened before we opened the modal itself)

Can you please back this up with link to code? I know this behaviour exists in RNW for accessibility reasons as part of the FocusTrap implementation. But I don't see any relevant code in react-native-modal. Perhaps this is something coming from RN's <Modal />?
Also, how come this issue is only reproducible in iOS and not in Android?

Feedback: Everytime you make a statement it's really recommended that you link to code supporting that statement e.g. in "But on the page Split bill we have a logic to dismiss a keyboard", it would have been great and much helpful if you linked to that logic. This makes the review process much faster.

Sorry @waterim I just realised that this issue may be a dupe of https://github.com/Expensify/App/issues/13922.
@lschurr I think we can close this one.

@s77rt actually with my solution add attachment works perfectly, without any keyboard blinks

@s77rt thank you for your feedback, will add more details, more relevant code to future proposals

@waterim What I meant to say is that we may already be working on fixing this issue. I see we have an open PR https://github.com/Expensify/App/pull/15337 for that.

@s77rt okay, sure)
Just one suggestion to add a native animation for a keyboard dismiss after clicking on the “+” button, it will feel more natural and native

@waterim Thanks, this feels like an improvement. I think we should let the assignees handle the bug fix first then we can discuss such animations in Slack.

Just to confirm - should this one be closed for now @s77rt?

@lschurr Yes


Integration Test - Release - 2020/08/25 14:04:43

Auto create for integration test. IssueCommandFlow

@PRMergerTest2 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Error: Invalid delegation amount error when broadcasting transaction

gm, we're running into the following error whenever we try to broadcast a transaction:
Error: Broadcasting transaction failed with code 18 (codespace: sdk). Log: invalid delegation amount: invalid request
example signed transactions that experience this error when broadcasting:
#1
#2
The last multisig transaction that we were successfully able to execute (also a delegation tx) was on April 18th, and then both signers (ledgers) started experiencing this error.

@abefernan could you have a look into this? Any recent change that might MsgDelegate amounts to break?

The fix was just deployed. @willyogo could you retry now?

thanks so much for the quick resolution!
will report back soon on whether this fixes for us

we're back in business @webmaster128, thank you again for fixing this so quickly!


Segfault on Mix_FreeMusic

We are currently segfaulting on shutdown. This seems to have been introduced during WidgetManager development, and these are almost always caused by a missing or extraneous init, create, or free in an SDL library. Fix this.
It's actually caused during a Mix_FreeMusic call on shutdown. Can't tell why it's breaking.

Bisect shows the bug was introduced in 37a609577d48c42f485e1b4433632c0b095bf0bc. There's nothing wrong with the code at first glance.

@seisatsu: I can't reproduce this on my system. Do you think it might be a Linux thing?

Fixed in c386c3c6b655e65b24196f19fe48d36643f730bd.


Newest version doesn't support navigateToLoginRequestUrl anymore

Unfortunately the newest version doesn't have a flag where you can prevent the lib from redirecting to an url after a successful token refresh. Therefor an angular SPA will load two times. Please reintroduce this flag as a part of the options object in the UserAgentApplication constructor.
Best regards,
Tobias Brenner

@tobiasbrenner The latest version has this property which allows you to do your custom navigation.


[docs] Add the introduction of "admission control" module

A new module of "admission control" is added into the HwameiStor, it's request to keep the document up to date.

Add the brief intro of "admission control" into https://hwameistor.io/docs/intro
Add a new item of the Components section to describe the "admission control"
Add the info about "admission control" into the "Quick Deployment"

Suggest pod name be the explicit admission control

Descriptions about admission has already added in https://github.com/hwameistor/hwameistor/pull/144


Support touch events

Old iPads are limited to iOS 12.4.6 whose Safari doesn't support pointer events. We should add a fall-back to touch events in such case; maybe there's already a compatibility layer we can just load when pointer events are detected as not supported. (They're also not that different, so might also be possible to add the fallback directly.)

@tomhull0 confirmed that this is working now. Yay!


Update setup docs to mention cloud-auth command

Summary
Blocked by

https://github.com/mattermost/mattermost-plugin-jira/pull/928

Ticket Link

Codecov Report
Patch and project coverage have no change.

Comparison is base (52bf3f9) 34.18% compared to head (2acf160) 34.18%.

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

Additional details and impacted files
@@               Coverage Diff                @@
##           jira-docs-3.0.1     #946   +/-   ##
================================================
  Coverage            34.18%   34.18%
================================================
  Files                   52       52
  Lines                 5991     5991
================================================
  Hits                  2048     2048
  Misses                3738     3738
  Partials               205      205

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

@mickmister I have done some minor changes according to our QA's comments. To merge this and include in our latest release we need to fix the CI here and one possible solution for this is to sync the branch with its base branch. But i am not able to understand these doc branches like [jira-docs-3.0.1](https://github.com/mattermost/mattermost-plugin-jira/tree/jira-docs-3.0.1) and similar branches. Can you please let me know how this PR is created and possibly fix the CI.

@Kshitij-Katiyar Since the recent PR https://github.com/mattermost/mattermost-plugin-jira/pull/948, we have deprecated the Gitbooks docs, and are instead favoring the README to hold all of the information. So we just need to make sure the README is updated on master.
cc @cwarnermm Do you think we should remove the docs folder to avoid confusion on where documentation lives? Or does it play a role in some capacity?

Closing this since we are now putting docs in the README instead of Gitbooks

@mickmister - I'm in the process of removing the docs folders in the GitHub repos for community-supported plugins. That docs folder no longer plays a role since it was created as part of GitBook synchronization functionality, and we're no longer hosting the source of truth via GitBook. (Note: We are replacing that GitBook content with redirects to help visitors find the README and plugin docs.)


operators not supported by tflite: CAST, ExpandDims, FLOOR, Fill, Pow, RandomUniform, SPLIT, Stack, TensorFlowGreater, TensorFlowMaximum, TensorFlowMinimum, TensorFlowShape, TensorFlowSum, TensorFlowTile.

This is a feature request issue.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):

Yes

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

CentOS 7

TensorFlow installed from (source or binary):

From pip install tensorflow-gpu==1.6

TensorFlow version (use command below):

tensorflow-gpu-1.6

Python version:

Python 2.7.5

Bazel version (if compiling from source): None

GCC/Compiler version (if compiling from source): None

CUDA/cuDNN version:

CUDA: 9.0

GPU model and memory:

NVIDIA GTX 1080Ti, 11178MiB

Exact command to reproduce:

toco --input_file=/tmp/mymodels/model_frozen.pb --output_file=/tmp/mymodels/converted_model.tflite --input_format=TENSORFLOW_GRAPHDEF --output_format=TFLITE --input_shape=1,320,320,3 --input_array=input_images --output_array=output_num_array --inference_type=FLOAT --input_data_type=FLOAT

Describe the problem
I want to convert a trained model, which works on my GPU workstation, to a tflite model to work on Android phone. But I found that there are lots of operation not supported (see also the log below): CAST, ExpandDims, FLOOR, Fill, Pow, RandomUniform, SPLIT, Stack, TensorFlowGreater, TensorFlowMaximum, TensorFlowMinimum, TensorFlowShape, TensorFlowSum, TensorFlowTile. Do I need to provide customized operator for that (that is a lot of job...)? Or do newer version of toco support that? By the way, toco seems to not supported batch normalization either, see this issue.
Source code / logs
After running the command above,
2018-06-19 10:17:59.783022: I tensorflow/contrib/lite/toco/import_tensorflow.cc:1171] Converting unsupported operation: RandomUniform
2018-06-19 10:17:59.920127: I tensorflow/contrib/lite/toco/import_tensorflow.cc:1171] Converting unsupported operation: Pow
2018-06-19 10:17:59.964930: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 306 operators, 577 arrays (0 quantized)
2018-06-19 10:17:59.971329: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 306 operators, 577 arrays (0 quantized)
2018-06-19 10:18:02.355631: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 216 operators, 476 arrays (0 quantized)
2018-06-19 10:18:02.360368: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before dequantization graph transformations: 216 operators, 476 arrays (0 quantized)
2018-06-19 10:18:02.364625: I tensorflow/contrib/lite/toco/allocate_transient_arrays.cc:311] Total transient array allocated size: 78643200 bytes, theoretical optimal value: 78643200 bytes.
2018-06-19 10:18:02.365858: F tensorflow/contrib/lite/toco/tflite/export.cc:304] Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If you have a custom implementation for them you can disable this error with --allow_custom_ops. Here is a list of operators for which you will need custom implementations: CAST, ExpandDims, FLOOR, Fill, Pow, RandomUniform, SPLIT, Stack, TensorFlowGreater, TensorFlowMaximum, TensorFlowMinimum, TensorFlowShape, TensorFlowSum, TensorFlowTile.
Aborted (core dumped)

I met this error, and resolved by synchronizing tensorflow code.

@wenhui7909 You mean that lastest tensorflow code has solved that ?

Yes,
Before, I also met this error , about can't find slice and some other op.
But now, those are ok.
Did u rebuild toco?
2018-06-22 10:43 GMT+08:00 Yubin<EMAIL_ADDRESS>
@wenhui7909 https://github.com/wenhui7909 You mean that lastest
tensorflow code has solved that ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/20110#issuecomment-399303217,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcPuIS2T97TcYdZOhn5uVnAw_GU27TADks5t_FnHgaJpZM4UswkF
.

@wenhui7909 I have met the same question,could you tell more about how to fix it?

Update source code, and build toco as below:
bazel build tensorflow/contrib/lite/toco:toco

@wenhui7909 I download the tensorflow r1.7 from github again,and bazel build again,but I didn't fix it.

Try latest version or tensorflow 1.8,
and you also can check the op cc file is exist or not.
My error report that slice op is not supported, so check slice_op.cc.

My error report that randomuniform,stack,tensorflowshape,floor
Tensorflow 1.8 don't support command line.I have try the 1.9 version.But the problem is still exist.
My model code don't appear to have these ops ,if I check the op not exist,it seems complex to fix it.

Oh, I don't know why I can see the op files and you can't.
such as, I can see floor.cc, and stack_ops.cc...
2018-07-03 17:36 GMT+08:00 KB<EMAIL_ADDRESS>
My error report that randomuniform,stack,tensorflowshape,floor
Tensorflow 1.8 don't support command line.I have try the 1.9 version.But
the problem is still exist.
My model code don't appear to have these ops ,if I check the op not
exist,it seems complex to fix it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/20110#issuecomment-402077840,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcPuIYtBOmUuTRgSAzfPsUVd21jPcq7wks5uCzslgaJpZM4UswkF
.

Would it be possible to provide your model ? It would help to understand if you could share what you are using those ops for.

Nagging Assignee @andrehentz: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Slice should now be supported (as well as shape and floor). Would you mind running the converter again using the latest build (1.10, or head)? randomuniform is not yet supported, and we don't yet have an ETA for that op. If you share the model we might be able to advise some intermediate workarounds.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Include diga error code for code validation requests in response

Example of updated response format:

The errorCode and errorText  are added in lieu of message field
error is removed as is redundant as the http status code is present both in the header and the body. (FBM to keep it though , as it AFAICS is default behavior in spring::boot)

{
    "status": 400,
    "errorCode": "CODE_NOT_FOUND",
    "errorText": "Freischaltcode / Rezeptcode nicht gefunden: 77AA AAAA AAAA ADGE",
    "path": "/validate/77AAAAAAAAAAADGE",
    "timestamp": "08-27-2021 06:41:07"
}

Example of current response format:
{
    "timestamp": "2021-08-22T10:35:48.992+00:00",
    "status": 400,
    "error": "Bad Request",
    "message": "DigaCodeValidationResponseError(errorCode=CODE_NOT_FOUND, errorText=Freischaltcode / Rezeptcode nicht gefunden: 77AA AAAA AAAA ADGE)",
    "path": "/validate/77AAAAAAAAAAADGE"
}

Addresses #10

This PR would make dealing with errors way easier. What still needs to be done to get it ready to be merged?

To be honest i didn't had much time to look into this lately, but my current concern with the proposed solution is that the new field "errorCode" is holding information that is not always available (only in the case of an invalid insurance response).
Maybe you or @bokchan can explain again a bit more in detail what is currently missing and what you would like to improve (only including the diga-client errors directly to the api response)?

As @icemac points out the proposed solution is mainly to make it easier to parse the error response.
Currently we extract the errorCode and  errorText  as regex capture groups from the message field in the response.
Having errorCode as a top-level field will allow the client to handle response errors in a more structured and simpler way.
@gtuk for which cases is information from the not available for the new errorCode field?
Would it be possible to define custom error codes to cover these cases?
Also, my bad also for not following up on the initial discussion, https://github.com/gtuk/diga-api-service/pull/13#issuecomment-907455835

If i understand it correctly you would like to have easy access to the errorCode, coming from the underlying diga-api-client

Yes. That is the wish from our side. Seems to be the same @icemac is requesting.
Your suggestion to keep the current structure of the api response and have an additional field to include the underlying error code from the diga-api-client sounds good to me.

We also have to parse the message using a regex. If it could be done easier on the server side it would be nice. On the other hand: The regex parser is already there.

If one of you would be able to provide a PR for the proposed i would be happy to review it as soon as possible. Unfortunately i can't work on it myself until mid of July, because of some time constrains. Let me know would you prefer @bokchan @icemac

@gtuk sorry for not getting to you earlier. For our case it is also a nice to have. As @icemac notes we already have the regex parsing implemented.
So I am fine with closing this PR and deferring implementation of the feature until a more general approach is agreed upon.

Closing this for now


Added Licensing Information

Add the Apache 2 license information to the code and the readme.

Selected the Apache 2.0 licensing agreement and added it to the application.


[move-lang] add support for friend visibility (parser to typing)

parse public(friend) as a function visibility modifier
parse and compile (up to typing) of the friend list declarations
unit tests for the parser, expansion, naming, and typing passes

Motivation
This is the first piece of move source compiler changes for friend visibility. This PR covers the changes in parser, expansion, naming, and typing. Expects more PRs to follow that cover dependency checking and translation to IR.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
New test cases for the source compiler are added

Rebased past #7786

/land


Remove empty MyHarmony.app on logitech-myharmony uninstall

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

I didn't include the version because this is not a version-specific issue correction. Let me know if that is incorrect.

Thank you.


Have you tried applying the politary loss function to the traditional classification problem instead of the zero shot object classification problem? I would like to know whether the effect will be better or not.

Have you tried applying the politary loss function to the traditional classification problem instead of the zero shot object classification problem? I would like to know whether the effect will be better or not.

Yes, we have tried the PL on the traditional detection problem (not classification as our base case is detection). We get comparable performance with the Focal Loss, which shows that the PL is specifically useful for ZSD case where proper alignment with the word-vectors and separation between background and unseen classes are important considerations.


Preliminary support for mixed cell type flow

This adds preliminary support and a simple test for flow with mixed cell types.  Items missing due to assumptions in petsc include:

using with probes/dm_locate
coupling with particles
interpolating to triangle cell (this is unrelated to the mixed cells).

This PR will require an update to PETSc (knepley/fix-plex-hybrid-ops) and test update when the triangle interpolation is fixed.


headers should be a dictionary, it should be a list

Summary.
Requests lib has implemented headers as a dictionary, yet the web supports a repeating header with the same key.  Subversion via WebDAV is tech that uses that.  For example, "svn log" sends up:
DAV: http://subversion.tigris.org/xmlns/dav/svn/depth
DAV: http://subversion.tigris.org/xmlns/dav/svn/mergeinfo
DAV: http://subversion.tigris.org/xmlns/dav/svn/log-revprops

Expected Result
All three 'A' headers got sent to the server
Actual Result
Only one did
Reproduction Steps
foo = requests.request(verb, url, headers={"A": "one", "A": "two", "A": "three"})

System Information
All versions of requests

Ha!  You can fool it with trailing spaces on the key:
 headers={
     "DAV" : "http://subversion.tigris.org/xmlns/dav/svn/depth",
     "DAV " : "http://subversion.tigris.org/xmlns/dav/svn/mergeinfo",
     "DAV  " : "http://subversion.tigris.org/xmlns/dav/svn/log-revprops"
 }

Did you try doing headers=[('A', 'one'), ('A', 'two'), ('A', 'three')]? I believe we support this use case when sent that way.

headers must be a dictwhich is verified here:
https://github.com/requests/requests/blob/54ad3d6dcc173576d3f9784985e88d31a4997677/requests/utils.py#L858-L861
A list is not supported.
@paul-hammant What API would you propose to verify multiple-value headers? headers={"A": "one", "A": "two", "A": "three"} makes no sense because this is {'A': 'three'}. What about a list:
headers = {
    "A": ["1st value of multi-value header", "2nd value of multi-value header],
    "B": "single value header"
}

@lutzhorn, I don't believe we'll allow the user to pass a list since it would go against our stance of not guessing on what the user wants in their header.
While we don't currently support a list of tuples, we use that in other places such as the data param and it seems like a possible option.
I'm not convinced we need to make a change here though. RFC 7230 states pretty clearly that:
3.2.2.  Field Order
   [...]
   A sender MUST NOT generate multiple header fields with the same field
   name in a message unless either the entire field value for that
   header field is defined as a comma-separated list [i.e., #(values)]
   or the header field is a well-known exception (as noted below).

So @paul-hammant you should be able to accomplish what you're looking for with something like:
dav_vals = ["one", "two", "three"]
headers = { 'DAV': ','.join(dav_vals)}
foo = requests.request(verb, url, headers=headers)

I'm proposing we close this, because this is solvable with one line and probably doesn't warrant the weight of adding tuple support.

Yes, the RFC is strict on that. Using join is a good idea for how to follow the RFC and accomplish what @paul-hammant  has in mind.

@paul-hammant I think is related to #4290

Apache2/Subversion are not as strict as the RFC. No matter - there are multiple ways through this. Thanks for not insta-closing :)


Documentation and help incorrect for 'generate' command in Flyway v11.0.1.

Which version and edition of Flyway are you using?
11.0.1
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
CLI
Which database are you using? (Type & version)
SQL Server
Which operating system are you using?
Windows
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Tried to use the generate command with a named diff artifact. I used the examples provided and none worked In the end I discovewred that the version of Flyway I was using said that each command contained the generate. prefix. In fact it rejected this prefix and instead used the basic key
e.g. -generate.description  didnt work but description did
-generate.artifactFilename didn't work but -artifactFilename did
-generate.location didn't work but -location did
The help txt is...
Configuration parameters: (Format: -key=value)
generate.target             The target to generate the script for, must be either diff.source or diff.target fr
om flyway diff. Uses initial target if unspecified
generate.changes            A comma seperated list of change ids. If unspecified, all changes will be used.
May specify - to read changes from stdin
generate.types              Comma seperated list of types of script to generate.
Valid values: versioned|undo|baseline Default: versioned
generate.version            The version part of the migration name.
If unspecified this will be calculated to either the next migration version if a ve
rsioned migration is specified, or the current version otherwise
generate.description        The description part of the migration name
generate.baselineFilename   The filename (or full path) to use for the generated baseline migration. Mutually e
xclusive with description & version
generate.versionedFilename  The filename (or full path) to use for the generated versioned migration. Mutually
exclusive with description & version
generate.undoFilename       The filename (or full path) to use for the generated undo migration. Mutually exclu
sive with description & version
generate.location           The location to generate the migration to
generate.artifactFilename   The location of the diff artifact to apply to the target.
Defaults: diff.artifactFilename, %temp%/flyway.artifact.diff
generate.force              If the migration script already exists, overwrite it. Default: false
generate.timestamp          Add a timestamp to the calculated version if one is not already present.
Valid values: always|auto|never Default: auto
What did you expect to see?
tried to run the examples
What did you see instead?
lots of errors
Flyway : ERROR: Invalid flag: -generate
At line:1 char:1

Flyway generate -generate.artifactFilename="PubsArtefact.diff"

  + CategoryInfo          : NotSpecified: (ERROR: Invalid flag: -generate:String) [], RemoteException
  + FullyQualifiedErrorId : NativeCommandError

Hi. The help is actually strictly correct, though we might consider updating it anyway. I suspect that the error you are getting is because the . in the names needs escaping in PowerShell (but not in bash or windows command prompt).
The working behaviour you are observing is due to namespace short-circuiting, see here.

Yes, it looks like PowerShell is getting thrown by the dotted key in the parameter.
if I use the syntax ...
Flyway generate '-generate.description="GeneratedBuildScript"'  '-generate.artifactFilename="PubsArtefact.diff"'
... then it works. Also if I 'splat' the commands ...
$Gen=@(
'-generate.description=GeneratedBuildScript',
'-generate.artifactFilename=PubsArtefact.diff'
)
Flyway generate $Gen
... then that works. In all my work so far, I'd been splatting arrays of parameters, which don't suffer the same problem. Because the generate command  had so few parameters it seemed like over-engineering to use arrays of parameters
What threw me originally was that it worked when I left out the 'generate.' prefix, and also that the examples in the documentation didn't mention the general requirement for escaping the dot in PowerShell.  Why not just leave out the requirement  for the prefix? It seems to serve no useful purpose. complicates life for the users, and I suspect it is being ignored at the moment anyway

Yes, it looks like PowerShell is getting thrown by the dotted key in the parameter. if I use the syntax ... Flyway generate '-generate.description="GeneratedBuildScript"' '-generate.artifactFilename="PubsArtefact.diff"' ... then it works. Also if I 'splat' the commands ...
$Gen=@( '-generate.description=GeneratedBuildScript', '-generate.artifactFilename=PubsArtefact.diff' ) Flyway generate $Gen ... then that works. In all my work so far, I'd been splatting arrays of parameters, which don't suffer the same problem. Because the generate command had so few parameters it seemed like over-engineering to use arrays of parameters
What threw me originally was that it worked when I left out the 'generate.' prefix, and also that the examples in the documentation didn't mention the general requirement for escaping the dot in PowerShell. Why not just leave out the requirement for the prefix? It seems to serve no useful purpose. complicates life for the users, and I suspect it is being ignored at the moment anyway

As mentioned, the prefix isn't required. It was initially, before we fully released the new verbs, but it is completely optional when invoking the verbs in isolation like this.
The reason the prefixes need to exist at all is:

verb chaining; you can invoke multiple verbs in one line and they could have arguments with the same name but different prefix (see examples in the namespace short-circuiting documentation I linked earlier)
flyway has name associations between command line parameters and config file settings and the config file settings need to be namespaced, both to prevent collisions of names and to give some order/readability to the config file

Very helpful. Yes, once chaining is better known by Flyway users, it makes sense to be able to spell out the correct namespace for ambiguous parameters. For the time being, it makes sense to indicate where a published example won't work in PowerShell. I'll makes sure that all parameters to flyway are cast as strings.


Update to rspec 2.99

We're  upgrading to rspec 3 and would like to still use suit.

@salticus this is best done on a branch


Compiling with CUDA_COMPUTE_CAP=75 does not lead to Runtime compute cap 75

System Info
Hi & Thanks for the amazing work !
I'm building the docker image with:
docker build --no-cache . -f Dockerfile-cuda --build-arg CUDA_COMPUTE_CAP=75
Starting with:
docker run --gpus all --env CUDA_VISIBLE_DEVICES=3 -p 8090:80 -v $volume:/data --pull always ghcr.io/huggingface/text-embeddings-inference:0.2.2 --model-id BAAI/bge-base-en-v1.5
Returns:
Error: Could not create backend

Caused by:
    Could not start backend: Runtime compute cap 75 is not compatible with compile time compute cap 80

My lowest compute capabilities in the system are RTX 2080s with capability 7.5. Starting on a 3090 or 4090 works though.
Best
Alex
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
docker build --no-cache . -f Dockerfile-cuda --build-arg CUDA_COMPUTE_CAP=75
Expected behavior
text-embeddings-inference starts and serves model

Solved: I missed specifying the architecture in the docker image string -> ghcr.io/huggingface/text-embeddings-inference:turing-0.2.2


train vqgan

I want to use my own Dataset to train. So do I need to retrain vqgan? if so I see that vqgan training seems to be missing the discriminator. how do I train vqgan?

Hello,
It depends if your dataset is "far" from ImageNet images. If so, you may want to retrain the VQGAN, but that is beyond the scope of this repository.  Please refer to the official repository of the [VQGAN] (https://github.com/CompVis/taming-transformers) for more details on how to train the model.
If your dataset is "close" to ImageNet images, there is no need to retrain the VQGAN, you can use the provided one directly.
Best,
Best,
Victor

Hello,
It depends if your dataset is "far" from ImageNet images. If so, you may want to retrain the VQGAN, but that is beyond the scope of this repository. Please refer to the official repository of the [VQGAN] (https://github.com/CompVis/taming-transformers) for more details on how to train the model.
If your dataset is "close" to ImageNet images, there is no need to retrain the VQGAN, you can use the provided one directly.
Best,
Best,
Victor

Thank you for your reply. I have one more question. If I want to use it for image inpainting tasks, do I only need to retrain the second part? So how do the input images and masks become code

Hello,
It depends if your dataset is "far" from ImageNet images. If so, you may want to retrain the VQGAN, but that is beyond the scope of this repository. Please refer to the official repository of the [VQGAN] (https://github.com/CompVis/taming-transformers) for more details on how to train the model.
If your dataset is "close" to ImageNet images, there is no need to retrain the VQGAN, you can use the provided one directly.
Best,
Best,
Victor

Thank you for your reply. I have one more question. If I want to use it for image inpainting tasks, do I only need to retrain the second part? So how do the input images and masks become code

I think you do not need to retrain the model but just do some modification in the inference step is enough


⚠ Fortnite has degraded performance

In 81131c0, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1498 ms

Resolved: Fortnite performance has improved in 7aab546.


Fix links in template README

Packages impacted by this PR
Describe the problem that is addressed by this PR

Fix broken links in the template example README caused by App Config sample regeneration

API change check
API changes are not detected in this pull request.