Wrong SQL generated in where clause when is specified the table with alias

Hi guys
I'm trying to use a complex query with gorm, the base of my query is:
SELECT c.* FROM channel c
JOIN location l ON c.id=l.channel_id
GROUP BY c.id,l.lat,l.lng
HAVING ( ( ACOS( SIN(RADIANS(:lat)) * SIN(RADIANS(l.lat )) + COS(RADIANS(:lat)) * COS(RADIANS(l.lat))    * COS(RADIANS(l.lng) - RADIANS(:lng)) ) * 6371000) < :radius )
ORDER BY  ( ACOS( SIN(RADIANS(:lat)) * SIN(RADIANS(l.lat )) + COS(RADIANS(:lat)) * COS(RADIANS(l.lat))    * COS(RADIANS(l.lng) - RADIANS(:lng)) ) * 6371000)
LIMIT 50

In the above query you can see that you send the parameters :lat, :lng :radius, So rescribing it with gorm  I did :
    channels := []models.Channel{}
    distance := " ( ACOS( SIN(RADIANS(%[1]s)) " + "* SIN(RADIANS(l.lat )) + COS(RADIANS(%[1]s)) " + "* COS(RADIANS(l.lat))    " + "* COS(RADIANS(l.lng) - RADIANS(%[2]s)) ) " + "* 6371000)"
    distance = fmt.Sprintf(distance, lat,lng)
    having :=  fmt.Sprintf("%s < ? ", distance)
    if models.DB.Table("channel c").Select("c.*").
        Preload("Location").
        Joins("JOIN location l ON c.id=l.channel_id").
        Where(params).
        Group("c.id,l.lat,l.lng").
        Having(having, radius).
        Order(distance).
        Limit(limit).Find(&channels).Error != nil{
        return nil, models.DB.Error
    }

In the above code I added the WHERE clausule but the query generated by gorm is:

SELECT c.* FROM channel c JOIN location l ON c.id=l.channel_id
WHERE (channel c."type" = 'office') GROUP BY c.id,l.lat,l.lng
...

In the WHERE clause gorm add an extra information about the table   WHERE (channel c."type" = 'office'),  Is a bug or is my mistake? if is my mistake, how can I fix it?

Please follow https://github.com/jinzhu/gorm/blob/master/.github/ISSUE_TEMPLATE.md to reopen another issue if it is still a problem for you.
Thank you.


npm install fails

Getting 403 Forbidden - GET https://artifactory.twitter.biz/api/npm/js-virtual/tsutils/-/tsutils-2.29.0.
Anyway to fix this?

Have you found the solution? @pmpavan


Char literal of '\n' is 97 instead of 10.

This program should print 10, but it prints 97 instead. For every other character i tried the correct ASCII value was returned.
module body Example.Foo is
    function main(): ExitCode is
        printLn('\n');
        return ExitSuccess();
    end;
end module body.

I noticed this when using Standard.IO.Terminal (readLine) because it would only read up to the first 'a' ('a' is 97 in ASCII).
I wondered whether this could be intentional and maybe character literals in austral aren't supposed to be ASCII, but since '\r' for example correctly equals 13 i don't think that's the case.

This looks like a typo to me, in the function parse_ascii_char(string) in Util.ml: https://github.com/austral/austral/blob/master/lib/Util.ml#L116
I have prepared a trivial PR that fixes it: #580


The pretrained model "crowdhuman_final.pth" not available

The link of the pretrained weights file crowdhuman_final.pth is not found by the server (Error 404). Can you provide a new link to the pretrained model file?

Nevermind, I found the alternative links in the README.md


chore: again some format

Summary by CodeRabbit

Refactor

Improved log message formatting across several modules for better readability and consistency.
Reordered import statements in setup.py for improved code organization.

Style

Added empty lines between import blocks for enhanced code clarity.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 24.20%. Comparing base (cc5dbfb) to head (457e781).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #137   +/-   ##
=======================================
  Coverage   24.20%   24.20%
=======================================
  Files          79       79
  Lines        6888     6888
=======================================
  Hits         1667     1667
  Misses       5221     5221

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


Clear text function

Currently the text input field in the search bar can only be cleared by the user, via either the cancel or clear button. There are however other situations where one might want to clear the input, for example after the user has completed their search.
This pull request address this by adding a clearText function, that allows for the text input to be cleared programmatically

Hey @PhilipMortiboy new maintainer here. We recently released v3.1.0 so now your changes are out of sync with latest. Can you rebase your branch against the next branch?

Implemented in https://github.com/umhan35/react-native-search-bar/pull/133/commits/4749a909f5c272a7fdaaac10d46cf506be47bb66, will be available in next release.


Change LHN header to Expensify

Details

Continuing the work https://github.com/Expensify/App/pull/18449.
Fixed Issues

$ https://github.com/Expensify/App/issues/18382
Tests

LHN(Chat list)

Sign in to the new dot
Confirm the top of the LHN says Expensify and not Chats and has the proper environment badge

Signin Page

ensure you're logged out
load new dot
ensure the Expensify wordmark shows in green with the proper environment badge

You can test the various badges locally by editing:
https://github.com/Expensify/App/blob/a29631b947530833f4722adf56fa9e267850176b/src/components/ExpensifyWordmark.js#L41
and use the constants from above: CONST.ENVIRONMENT.DEV, CONST.ENVIRONMENT.STAGING, CONST.ENVIRONMENT.PRODUCTION

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Just want to confirm that the wordmark is 108px wide, like so?

If that's the case then we can create different wordmark files that have bigger badges. This is why I originally pushed not to reuse the same logos from the home screen as they were created for different sizes cc @bondydaa we should have just replaced "Chats" with the wordmark and then control the badge styles with CSS

Yup, that's my thinking. Use just a wordmark svg (no badge there) and then keep the previous badges that were styled via CSS. Then, we can make some small adjustments to those badges to make them a bit smaller but still appear legible.

@parasharrajat Will you be able to take over here? I will assign myself for a review since Bondy cannot merge this

Use the links below to test this build in android and iOS. Happy testing!

:test_tube::test_tube: Use the links below to test this build in android and iOS. Happy testing! :test_tube::test_tube:

android :robot:
iOS :apple:

https://ad-hoc-expensify-cash.s3.amazonaws.com/android/18802/index.html
https://ad-hoc-expensify-cash.s3.amazonaws.com/ios/18802/index.html

desktop :computer:
web :spider_web:

https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/18802/NewExpensify.dmg
https://18802.pr-testing.expensify.com

This is how the adHoc build looks like

@parasharrajat its not super important to address this but it would be great if we could fix it already in this PR

Definitely dont have to block on that though

Thank you very much 🙇

Could you try with a font-size of 7px, and using a bold font weight? Then maybe we can get the height of the badge down to 12px too?

adhoc should use a purple color, not red. And maybe we can move the badge over to the left by 1 or 2 pixels. Otherwise it's looking good!

Great, looks good to me!

I think it's fine. None of this is customer facing either.

Asked in #contributor-plus if anyone is available to jump in and test this now.

going to check on the status of the app deploy to see if we should add cp staging label prior to merge.

🚀 Cherry-picked to staging by https://github.com/bondydaa in version: 1.3.14-14 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.14-14 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Deleted comma

Support for the edit:
Grammarly: Comma before "and"
Thank you for the beautiful article and your contentious work to improve it!
Leo

The video in the section "Briefly on the subject of Microsoft" is not being displayed any more, because of the channel being banned on the basis of many reports about copyright infringements.

I exchanged the old link against for an alternative identical one.
This should be tested, before it is merged.


Eta077 main

Based on #159, we update the Dockerfile to support local development for non-unix platforms.

I'm able to successfully build a container from the updated Dockerfile, but when running the ./bin/test -t all command, I get the following error:
error: failed to run custom build command for `cast v0.2.7`

Caused by:
  process didn't exit successfully: `/project/target/debug/build/cast-dd26f4947cc4de68/build-script-build` (exit status: 1)
  --- stderr
  /project/target/debug/build/cast-dd26f4947cc4de68/build-script-build: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.29' not found (required by /project/target/debug/build/cast-dd26f4947cc4de68/build-script-build)
warning: build failed, waiting for other jobs to finish...
error: build failed
test failed with exit code 101

@eta077 can you try running with something like:
docker run --rm -it -v $(pwd):/project -e CARGO_TARGET_DIR=/tmp <image> ./bin/test -t all

and let me know if that works? I'm wondering if it is trying to use existing files in your target dir

@mindriot101 looks like that was the issue, the test command works after running cargo clean - thanks so much for your help!

That's great, it sounds like some files from your target folder were being used to link against new compiled outputs. I think either a cargo clean first, or setting the CARGO_TARGET_DIR as I showed is enough to ensure a clean build. I'll merge this into the main branch. Thank you for helping test the crate!


[Feature] Download Resource Packs and Inessential Mods After Initial Download

In addition to my earlier request, I propose a feature where the launcher downloads essential mods and allows the user to launch the pack, but continues to download files such as resource packs and other large inessential files. Such files would be marked in solder as something like "post-initial-download."
If my other feature, Launcher/Solder Multi-File Download and Reassembly, is implemented, I also request compatibility with this feature, for even more streamlined modpack deployments.
Related feature requests:
https://github.com/TechnicPack/LauncherV3/issues/36
https://github.com/TechnicPack/LauncherV3/issues/35
https://github.com/TechnicPack/TechnicSolder/issues/293

Why would we implement this? You'd end up with broken modpacks, this is not how Minecraft loads things.


API Permissions

In following step 3 above, I found the "My APIs" tab is blank unless I first expose the the Azure API for FHIR I previously deployed.  Perhaps it ought to indicate such.
Similarly, step 4 above seems to depending on the presence of defined scope(s) such that the permissions can be associated to one or more scopes.  It could be helpful to reflect this dependency.

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

ID: 7d6f2e64-ad33-1f0d-5d8a-3462a68fcfad
Version Independent ID: 1999ca19-8836-9bc9-d964-d8c0706a7f93
Content: Register a confidential client application in Azure Active Directory - Azure API for FHIR
Content Source: articles/healthcare-apis/register-confidential-azure-ad-client-app.md
Service: healthcare-apis
GitHub Login: @hansenms
Microsoft Alias: mihansen

@kirtnoel what do you mean by "first expose the Azure API for FHIR I previously deployed."? What you would probably see in there is the confidential client app that you would create in order to obtain a token. The Azure API for FHIR has a first part app with URI https://azurehealthcareapis.com that will be present in your Azure AD tenant once you have deployed an instance of the Azure API for FHIR. You should not be adding any scopes from your confidential client for anything.
Azure AD is a pretty big topic on its own. There are a lot more details here: https://docs.microsoft.com/en-us/azure/active-directory/develop/index
Hope this helps,
Michael

Concerning you question:  what do you mean by "first expose the Azure API for FHIR I previously deployed."?
In the diagram (below), the "My APIs" tab was blank when navigating my Azure Portal.  As such, being new to the Azure portal, I didn't realize that I could have equally chosen the "Azure Healthcare APIs" under the tab "APIs my organization uses".  I errantly concluded I needed something under the My APIs tab and succeeded to get a couple "scopes" under that tab through the "Expose an API" configurations.
As per your guidance, this was indeed unnecessary.  I've eliminated the scopes and the API working just fine now.  This is really terrific stuff.  Thanks!

#please-close


[loc][Query][mlnet-tools] Localizability of strings

Hi,
please help localization team to answer question from translator.
In case translator is not sure about context/meaning/localizability of string we raise a question to Core team.
Thanks in advance for the reply

Id
File Name Live Page Extended
String Resource ID
Full Source String
Extended Description
Answer Needed By

49,758
Microsoft.ML.ModelBuilder\Resources\Strings\xlf\EvaluateTabResources.xlf
0; "SampleDataHeader"
Sample data
Is sample a noun or a verb?

49,987
mlnet\xlf\Strings.xlf
0;   "CacheHelperText"
Specify   [On|Off|Auto] for cache to be turned on, off, or auto-determined (default).
Is   [On|Off|Auto] translatable? Please, confirm. Left as is by now.

50,069
Microsoft.ML.ModelBuilder\Resources\Strings\xlf\CodeTabResources.xlf
0;   "TrainInAzureCardText"
and get the most out of the   cloud.
What will precede this string?   Is this a continuation of the header, "Train in Azure"? If so, is   it legitimate to make this a independent sentence? A screen would be helpful,   thank you.

50,072
Microsoft.ML.ModelBuilder\Resources\Strings\xlf\CodeTabResources.xlf
0;   "ImproveModelCardText1"
Check   out
It   assumes that this segment "Check out" (ResID   "ImproveModelCardText1") is followed by "to learn more about   how to use the model in your app." (ResID   "ImproveModelCardText2").                    But in Japanese, the translation of "Check out" should be located   in the last of the sentence.          So should it be "blank"?

50,076
Microsoft.ML.ModelBuilder\Resources\Strings\xlf\CodeTabResources.xlf
0;   "ImproveModelCardText2"
to learn more about how to use   the model in your app.
It seems that this segment   "to learn more about how to use the model in your app." (ResID   "ImproveModelCardText2") follows "Check out" (ResID   "ImproveModelCardText1").                    In Japanese, verb should be located in the last of sentence.                    So can we add the tranlslation of "Check out" into this segment?

49,758 "Sample" is a noun in "Sample data"
For 50,069 50,072 and 50,076 we are making a change and I will checkin the strings in their complete form. Can we not translate these for right now? I will checkin the new strings on Thursday or Friday.
@LittleLittleCloud to comment on 49,987 Specify [On|Off|Auto] for cache to be turned on, off, or auto-determined (default).  Is [On|Off|Auto] translatable? Or are those always in English?

@PDostalek Please let me know if I should answer these in the grid or another form. Thanks!

@beccamc  Thanks for your replies. Strings are continuously being translated, please let me know with changes. Thx

@PDostalek  50,069 50,072 and 50,076 are checked in.
@LittleLittleCloud for 49,987

For 49987, it's not, please keep the origin string, thanks

@beccamc Could you please provide us the whole concatenation of your strings?

50,069 "Train your model in Azure and get the most out of the cloud"
50,072 "Check out our docs to learn how to to improve your model's performance"
50,076 Removed

THank you


More functionality

New options:

enter / leave - options for animation type
closeButton - bulma or bootstrap close buttons
mode - HTML or TEXT, HTML mode enables HTML interpolation in notification body
permanent - this options makes notification permanent
New methods:
removeAll - use it to remove all notifications

Also some CSS fixes (margin between notifications) and README.MD improvement

Hi Adam, please review my PR.


In-game lua editor/clientside lua viewer crash

Lua editor Metastruct and Clientside Lua Viewer use crash the game

If you need a test sample: https://github.com/glua/clientside-lua-viewer/tree/master/lua/menu_plugins/luaviewer

Alright, this should be useful in figuring out where all the damn crashes are coming from.

Any update on this at all?

Finally bothered to enable debugging symbols and I've figured out where one of the crashes is coming from. Hopefully I can get a more stable release out soon.

This particular crash seems to be caused by an issue with the incomplete JSON parsing. Should be an easy fix. Note that I've only tested with the metastruct editor, not the client lua viewer as I didn't want to install it.


Power-down loss after burn-in

After flashing, I perform ap <myssid> <mypassword> test, but when it is re-powered, the WIFI hotspot is gone? This is also the case when I do the sta test, besides, he can't connect to my smart phone hotspot, can you help me?

Can you provide a log of your not being able to connect?

Hello, @zhy102
Closing this issue, feel free to reopen with more updates. Thanks.


[Bug] -c doesn't exchange pokemons

Expected Behavior
The command--cp 200 exchanging pokemons under 200 for candy.
Actual Behavior
The pokemon is not exchanging for candy. You can see it in the cmd it but its still in the inventory - also the message  Your inventory is full!  is wrong.
[#] Something rustles nearby!
[#] A Wild Weedle appeared! [CP94]
[x] Using Pokeball...
[x] Captured Weedle! [CP94] - exchanging for candy
[###] Your inventory is full! Please manually delete some items.
[#] A Wild Eevee appeared! [CP103]
[x] Using Pokeball...
[x] Captured Eevee! [CP103] - exchanging for candy
[###] Your inventory is full! Please manually delete some items.
### Steps to Reproduce

Other Information
OS:
Git Commit: dev 19

duplicate again.. #237

#127 #142 #15
i just wish people would check if something is reported already..

This is fixed in master already. Thanks for the report.


Module names

Module names should not contain path delimiter: "cldr/event", "cldr-event". This makes a lot of trouble with require.

Please, check this file for an example on how you should configure your AMD path settings.
Please, make additional comments in case it's still a problem for you. Thanks


[Issue 2339] Extension - Sort the token by balance

Describe the task

[x] Show the balance on the right

[x] Sort the token by the balance on the Select token when performing actions:

[x] Send fund
[x] Staking

🚀 Deployed on https://pr-2420--sw-web-runner.netlify.app


Multirepo SCM support

With August's release, VS Code will start supporting multiple SCM providers simultaneously. This means that the user will be able to use both a Perforce repository's features alongside a git repository's features.
This new world is entirely compatible with the existing API, yet there is one small consequence. Calling scm.createSourceControl will create a new repository view in the SCM viewlet. This extension always calls it on activate. We suggest to change the code in order not to call scm.createSourceControl until a Perforce repository is actually detected in the user's workspace.
Please let us know if you need any additional information to adopt this change.

fixed with #88.  still needs work to support multiple roots / p4 providers

Now part of version 2.2.0, please update!

Awesome!


Create per-target .config file

This issue tracks fixing the naming and kconfig issues associated with a single project name as well as the varying configuration which is the result of having multiple targets.
The issue requires changes to:
https://github.com/unikraft/kraftkit/blob/b8eb19f/schema/loader.go#L388
Which should not use the projectName but the target name.
Additionally, an associated ".config" file should be created, in the format: .$TARGET_NAME.kconfig which is then fed to Unikraft via UK_DEF_CONFIG=path/to/$TARGET_NAME.kconfig.

Fixed a while ago, each target has it's own config now


List out names and domains for apps already used with chosen username during auth

On this screen for clarity since it's not always easy to recognize the existing apps by their icons:

https://www.figma.com/file/aXnNi8MOmN3CDrVwaDXXuqX3/Authenticator?node-id=5721%3A11526
We'll need to truncate at some point("..., and 3 more"). I'd suggest after 3

I'm closing out this issue since the need will be addressed in Connect 2.0 as part of #671.


Suggesting addition for sign up feature

Missing Feature:
No sign up feature on the plone website
Feature Suggestion i would like to add:
I would like to add sign up feature to the plone website via firebase authentication so that user can sign up using google account.
Here is the attached image:

Only Admin login is Possible:

@displacedaussie can you please assign this issue to me if this is a genuine requirement for the website?

Volto can use external auth providers via this addon: https://github.com/collective/volto-authomatic


[CSL-196] [Stats] Don't use global mutable variable

@shersh
Now we are using global mutabl variabl to collect statistics. This practice is known to be bad in Haskell world. And it probably can lead to some issues. This should be replaced with more smart solution. Like putting this variable into ReaderT context.
https://github.com/input-output-hk/pos-haskell-prototype/blob/744c2a737b30605515a15b9088bc8affa4bcd60e/src/Pos/Statistics/MonadStats.hs#L145
https://github.com/input-output-hk/pos-haskell-prototype/blob/744c2a737b30605515a15b9088bc8affa4bcd60e/src/Pos/Statistics/MonadStats.hs#L164

@georgeee
Actually, it's strange this variable isn't part of Stats monad (via ReaderT).


fix: support old Node.js version

https://github.com/eggjs/egg-core/pull/269

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add prettier & prettier config to the repo (and/or tsdx)

Currently it seems that prettier is not included as a devDependency and there is no prettier config checked in. As a result there's no consistency in the file formatting - if I make a change to a file I'm getting the whole thing lighting up in the diff because the default prettier config is different to how the previous developer formatted the file.
What I would suggest as a way to solve this issue and simplify the whole build/lint/test process at the same time is that we could use tsdx to manage those tasks, because it handles both tsc and babel and ships with prettier included.

@madhavajay I could make a PR for this if you like, as I did the same setup recently on the tcn-node project.
One thing that would have to be agreed on is what prettier config to use. The defaults are printWidth 80, semicolons enabled, and double quotes over single quotes. I'm not really into those settings but there is always something to be said for sticking to defaults just to avoid arguments.

This is obsolete now that we've switched to Go.


Two commits: spellchecking and translation of the MPL license

Thank you for help @korasaccharin  :+1:


checkbox value not updating

import ipysheet

kwargs = dict(row_start=0, row_end=0, squeeze_column=True, squeeze_row=True)

grid = ipysheet.Sheet(rows=1, columns=2, column_headers = False, row_headers=False, column_width=[20, 100])
grid.cells = (ipysheet.Cell(value=True, column_start=0, column_end=0, type='checkbox', **kwargs),
              ipysheet.Cell(value='True', column_start=1, column_end=1, **kwargs))

grid.cells[0].value = False
grid.cells[1].value = 'False'

grid

This shows a checked box and a cell containing "False"
Somewhat related to https://github.com/QuantStack/ipysheet/issues/3

I am not able to reproduce your issue. I'm using master. Maybe it was fixed since the last version. Did you try using master?

My bad, thank you


*: add admin permission checking

/cc @mitake

@xiang90 how about making other operations, e.g. membership change, admin ones? They shouldn't be done by ordinal users.

@mitake For membership, yes.

@mitake The membership commands not do not go through v3 apply loop. I will make it happen in another pr.

@xiang90 sure, supporting the membership commands can be done later

@mitake I fixed the typo. PTAL.

@xiang90 thanks, lgtm after CI pass


Support unmatched requests and clear request log

Add support for clearing the request log
Add support for getting the number of requests that didn't match any stub

Is this a solution when you can just clear requests count without deleting all stubs?


Pydantic ValidationError (UserResponse and LoginRequest)

Tests affected: test_user_response_valid, test_login_request_valid
Error type: ValidationError
Potential cause: This error occurs when the data provided to a Pydantic model does not pass the validation checks.

Similar to the last issue. The code itself was not properly sending the right data when executing the pytests therefore I was getting that error. This change was in the test_user_schemas.py and including the uuid.


⚠ Samsung has degraded performance

In aff6d7a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2391 ms

Resolved: Samsung performance has improved in 48db6b6.


slaughter brace charge counter in inventory resetting to "30" on every proc.

title

also in worn inventory

duplicate #12299


demo login

you guys have a demo up on http://demo.ghost.io but no credentials how does one login to the admin I tried to install it on my local machine took 2 hours to setup the whole thing

Hi @alirz23, demo.ghost.io is a demo of what a Ghost blog looks like. If you want to try it out you can go to ghost.org, sign up and try our Ghost for 14 days - there are no card details required.

Just blunt marketing feedback here --- I just installed Ghost locally. Problem is I use node 9.8 which is unsupported. And that's fine, I crossed my fingers and hoped it would work well enough to check out admin page. It didn't work, and I assume it will if I abide by the recommended versions, but let's face it I'm trying to research 5 different products and while I could get into NVM or a different solution, how many minutes have I now spent trying to get a glimpse at your admin panel? Too many.
+1 to this

@CurtisBelt
https://docs.ghost.org/docs/install-local
It describes everything you need. It's a very simple three step installation to get a local blog working.

check supported node versions
install the CLI
install a local blog

If there is something unclear in the docs - please either tell us in slack or suggest a edit.

@kirrg001
I appreciate your response however I did not intend to say your docs were unclear. I was just backing up what pabru said https://github.com/TryGhost/Ghost/issues/6026#issuecomment-356239994.
Any bumps in installation (even if it's my own fault) can result in someone skipping Ghost as an option. That's why many other CMS' have the admin dashboard readily available for testing.

(This may not necessarily be a github-relevant issue, as it's more about the platform's marketing materials than about the CMS code. If there is a more relevant location for this post, please feel free to direct me there.)
Chiming in to add that it would be extremely helpful to be able to visit a public URL running the Ghost admin, or at least its text editor. Of course allowing anyone to save posts/content on a public instance would be disastrous, so perhaps persistence would be disabled there.
The ghost local installation process was a significant barrier to me in selecting ghost as a platform. Not necessarily a technical barrier - it's just an npm command and a ghost-cli command - but even after completing the process, there is no indication in the install local docs or CLI output of how to log into the newly created local instance. I checked out the source to discover that a new site's admin is open at /ghost.
tl;dr Platform looks great! This is a roadblock.

We do now have a better place for this sort of discussion -> https://forum.ghost.org.
However, the lack of admin URL output is tracked here: https://github.com/TryGhost/Ghost-CLI/issues/665 I will chase getting it sorted out.

Just use Stapi!

Just use Stapi!


Develop tool to convert a virtual disk image to a docker image

Problem
Develop tool to convert a virtual disk image to a docker image
Solution
Use a tool like qemu-nbd (or equivalent) to mount a virtual disk, extract
the filesystem, and convert that into a single large docker image.  Bonus
points for allowing the user to specify a base image to start with
that's "close" to the original OS image and then ensure the single delta
image contains only the changes and not the entire disk content.

#dibs


Zabbix plugin and graph support

Zabbix plugin should support obtaining graphs via Zabbix API. It should have the following synopsis:
graph  <hostgroup/host> [@period]
keyword is a regular expression or pattern, which matches the graph for specific host/hostgroup.
period represents time period, e.g 1h will show data for the last hour.
Some interesting URLs:
https://www.zabbix.com/documentation/2.2/manual/api/reference/graph/get
https://github.com/sorah/hubot-zabbix/blob/master/scripts/hubot-zabbix.coffee

We have separated zabbix into its own plugin. This bug was re-created there https://github.com/erigones/ludolph-zabbix/issues/1 no need for it to be in core, there fore I am closing it here.


Create compatibility with BO4E > v202401.0.0

The current implemented BO4E version is legacy. The newest version has implemented e.g. reworked BO objects and changed some technical fields as _typ. It will be a big load of work to adapt ibims to these changes. It may also be necessary to adapt code in powercloud2lynqtech in a next step if fields or models got renamed or if the modelling changed.
Related PR: https://github.com/Hochfrequenz/intermediate-bo4e-migration-models/pull/134
Necessary steps (and knowledge) are:

[ ] Understand how the BO4E-config files work
[ ] Compare configuration with actual (new) implementation of BO4E
[ ] Maybe rename fields
[ ] Maybe remove additional models if they are now implemented in BO4E standard
[ ] Maybe remove additional fields if they are now implemented in BO4E standard
[ ] E.g. Move from BoTyp to Typ
[ ] Maybe more complicated stuff if the modelling changed
[ ] Eventually, adapt code in powercloud2lynqtech (out of scope for this issue)

I See ;)


Update benchmarks/nel for spacy<3.5.0

Description

Add spacy_version requirement
Remove unused virtualenv requirement
Sync README

Types of change

Bug fix of sorts.
Checklist

[x] I confirm that I have the right to submit this contribution under the project's MIT license.
[x] I ran the tests, and all new and existing tests passed.
[x] I ran the update scripts in the .github folder, and all the configs and docs are up-to-date.

This only works in the current CI (hopefully) because the CI is also installing spacy<3.5.0.


Feature request: Link to CNCF's merchandise page and have a banner for "news alerts"

@jhdalek55 requested the following:

I think a merch link on the site would be fun.
It would also be good if press links like this one--ones that include video and are relatively high profile— could be accessible from the splash page as well. People probably won't go looking for such clips, but if they can access them easily, I bet many would watch. Can we somehow bring a clip like this forward when we have something without tieing us to always having some kind of clip in that place? I think it would be a great way to generate interest in the site and maybe bring the reader who just stumbles upon us, as opposed to those looking for us, into the site.

A link to the merchandise page from a top-level heading on the splash page, and a temporary banner, like the "news alert" banner commonly used by news websites, is one way we can do it.

The link to the CNCF merchandise page: https://store.cncf.io/collections/tuf

I think these options are fine. What would we need to make this happen?

Do we want to link to the merch from the ABOUT page? Probably makes the
most sense.
Lois
On Thu, Jun 28, 2018 at 5:58 PM, Vladimir Diaz<EMAIL_ADDRESS>wrote:

The link to the CNCF merchandise page: https://store.cncf.io/
collections/tuf
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theupdateframework/theupdateframework.github.io/issues/39#issuecomment-401186047,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AN5qXwZmjNf88Ed4-cXaDATC9uG5jmCzks5uBVGFgaJpZM4U8Itd
.


Update s3, sdk-core, sqs to 2.20.118

About this PR
📦 Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.17.295 to 2.20.118
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #647.


fw_pos_control_l1: for tailsitters we need to rotate the body acceler…

…ation
vector as it has it's reference in the mc frame
Signed-off-by: Roman<EMAIL_ADDRESS>Otherwise altitude hold in fixed-wing doesn't work properly

Applied to master


[docs] request for documentation: lazymint

Hi guys thanks a lot for outsourcing this massive protocol
as a user, I want to sell items using lazy mint. let's say I have a contract
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
contract ERC721ACBR is AccessControl, ERC721 {
  bytes32 public constant MINTER_ROLE = keccak256("MINTER_ROLE");

  constructor(
    string memory name,
    string memory symbol,
  ) ERC721(name, symbol) {
    _setupRole(DEFAULT_ADMIN_ROLE, _msgSender());
    _setupRole(MINTER_ROLE, _msgSender());
 }

  function mint(address to, uint256 tokenId) public virtual onlyRole(MINTER_ROLE) {
    _mint(to, tokenId);
  }
}

now I want to create an order signed by wallet with MINTER_ROLE
that contains an invocation of mint(address, uint256) instead of transfer(address,address,uint256)
to sell the not minted items for a fixed price using seaport

@Slokh please, just a general idea in a few words

Any update on this?

@callum-elvidge I'll give you a hint, your zone should call mint function

@callum-elvidge I'll give you a hint, your zone should call mint function

Can you give a bit more info on this and also which testcases should I focus on.


update README for handleFindAll

and added test case demonstrating using handleFindAll and handleDelete.  This would close #128.

thanks a ton @eliotpiering .. I really like good documentation .. so that was super great.  Nice test too.


Initialize Transaction Metadata Data Type is String

Going through the Paystack metadata documentation here, https://paystack.com/docs/payments/metadata/
Metadata is of type object. Why is this different in this package? When I checked metadata is of type string.
Is there something I am missing? Kindly help.

Nothing is missing @seunafara . Please refer to the API docs section for Transactions
There are no noticeable discrepancies between the Official Paystack API Reference and this package
See below:
;

Oh, I'll check this out and test. Thanks

I just tried it tho. Does the error show in this image?
https://paystack.com/docs/payments/metadata/

@seunafara like I said you are meant to stringify it. Like so:
paystack.initializeTransaction({
   amount: business.claim_price,
   email: req.user.email,
   metadata: JSON.stringify({
      cart_id: 398,
      custom_fields: [
         {
              display_name: "Invoice ID",
              variable_name: "Invoice ID",
              value: 209
         },
         {
              display_name: "Cart Items",
              variable_name: "cart_items",
              value: "3 bananas, 12 mangoes"
         }
      ]
   }),
   callback_url: `${theAppUrl}/pay/process-payement`
});

I will close this PR out now because there is really no issue here.
Thanks for using paystack-node 😄 . Be sure that the package will continue to be maintained and upgraded 👍


chore(project): drop mockio

Codecov Report

Merging #45 into master will increase coverage by 13.27%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #45       +/-   ##
===========================================
+ Coverage   73.68%   86.95%   +13.27%
===========================================
  Files           5        4        -1
  Lines          76       46       -30
===========================================
- Hits           56       40       -16
+ Misses         15        4       -11
+ Partials        5        2        -3

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 bf4754f...148818b. Read the comment docs.


Makefile: Anything actually UNIX specific?

Hello there!
I am experimenting with WASM via Emscripten and Wasm3 - but I want to go a little further and take the WASI route to learn more about the possibilities that get with WASM.
So I took a look at the makefile. A lot of CMake-ing and basic shell ops.
Is there anything that I may not be spotting that is specifically UNIX only? I use "UNIX" as an opaque term here, since it was mentioned that environments like git-bash do work with this - and, well, for the lack of a better description ;)
If this is not the case, it'd be quite easily possible to automate the build on plain Windows by utilizing Chocolatey - there are packages for Ninja, CMake and Make. Even VS2019 can be installed that way. i.e.: choco install cmake ninja make visualstudio2019community visualstudio2019community-buildtools
Then you'd only have to change cp and mv to Windows compatible variants and could just make in the root.
Before I go and try to port the makefile myself, I just want to be sure that I am not overlooking anything before running straight into a wall, you see. :)
Kind regards,
Ingwie

Welcome! The wasi-libc Makefile uses some tools like sed, find, and diff, though those parts are effectively just doing consistency checking, which could be disabled without affecting the main product of the build.

Also, if just want to experiment with SDK you can download the existing mingw SDK from the releases page: https://github.com/WebAssembly/wasi-sdk/releases.

The original question is answered; thanks!


[Diagnostics] Fix bad diagnostic for failed assignment from Any to a more specific type

When generating constraints for subscript convert InOutType into LValueType,
because base of the subscript should never be marked as inout, but rather as
@lvalue to denote mutability.

Resolves rdar://problem/25601561.

/cc @rudkx

@swift-ci Please smoke test.
I'll take a closer look at this change this evening.

@swift-ci Please smoke test

@xedin FYI the bug number in the commit message still has two digits swapped.
So I'm torn on this one. This really isn't fixing the core issue here, but is papering over it.
The core issue is that during diagnostics we apply a solution that adds an InOutExpr, and that particular pass of typeCheckChildIndependently() doesn't turn up anything, so we continue on and build a new constraint system with this mutated AST.
We could take your patch, amended with a comment explaining that it is a pragmatic hack for the time being, or could defer this until we have time to rework (what I expect to be a lot of stuff) to design things like this out of the system.
@DougGregor What do you think?
If we do take the patch I think the more principled thing to do is check to see if the InOutExpr's subexpr is an LValue and only make the result an LValue if it is.

FWIW, I believe I've seen other bugs that are probably hitting the exact same issue so we might be able to kill off more than one bug with this change.

@rudkx Pushed a fix for a commit message (sorry about that, apparently i can't even copy-paste correctly...)!
Let me elaborate a bit more about why I did what I did here: When you look at the example in the test - here was the initial state with force_value_expr at the top:
(force_value_expr type='$T8' location=<REPL Input>:4:36 range=[<REPL Input>:4:23 - line:4:36]
  (call_expr type='$T7' location=<REPL Input>:4:29 range=[<REPL Input>:4:23 - line:4:35]  arg_labels=
    (unresolved_dot_expr type='$T6' location=<REPL Input>:4:29 range=[<REPL Input>:4:23 - line:4:29] field 'value' function_ref=single
      (force_value_expr type='$T5' location=<REPL Input>:4:27 range=[<REPL Input>:4:23 - line:4:27]
        (subscript_expr type='$T4' location=<REPL Input>:4:24 range=[<REPL Input>:4:23 - line:4:26]  arg_labels=_:
          (unresolved_dot_expr type='@lvalue [S_r25601561?]' location=<REPL Input>:4:23 range=[<REPL Input>:4:23 - line:4:23] field 'a' function_ref=unapplied
            (declref_expr implicit type='C_r25601561' location=<REPL Input>:4:23 range=[<REPL Input>:4:23 - line:4:23] decl=REPL.(file).C_r25601561.func decl.self@<REPL Input>:3:10 function_ref=unapplied specialized=no))
          (paren_expr type='(Int)' location=<REPL Input>:4:25 range=[<REPL Input>:4:24 - line:4:26]
            (declref_expr type='Int' location=<REPL Input>:4:25 range=[<REPL Input>:4:25 - line:4:25] decl=REPL.(file).C_r25601561.func decl.i@<REPL Input>:3:15 function_ref=unapplied specialized=yes)))))
    (tuple_expr type='()' location=<REPL Input>:4:34 range=[<REPL Input>:4:34 - line:4:35])))

When solving that the only constraint which is going to fail is contextual type conversion - $T8 conv String, which forces diagnostics as follows - first it goes into diagnoseExprFailure and tries to diagnose ForceValueExpr via FailureDiagnosis::visitForceValueExpr, what that actually does is extracts a sub-expression which is CallExpr and tries to solve it instead to check if there are any structural errors - none in that case.
(call_expr type='Any?' location=<REPL Input>:4:29 range=[<REPL Input>:4:23 - line:4:35]  arg_labels=
  (dot_syntax_call_expr type='() -> Any?' location=<REPL Input>:4:29 range=[<REPL Input>:4:23 - line:4:29]
    (declref_expr type='(S_r25601561) -> () -> Any?' location=<REPL Input>:4:29 range=[<REPL Input>:4:29 - line:4:29] decl=REPL.(file).S_r25601561.value()@<REPL Input>:2:10 function_ref=double specialized=no)
    (load_expr implicit type='S_r25601561' location=<REPL Input>:4:27 range=[<REPL Input>:4:23 - line:4:27]
      (force_value_expr type='@lvalue S_r25601561' accessKind=read location=<REPL Input>:4:27 range=[<REPL Input>:4:23 - line:4:27]
        (subscript_expr type='@lvalue S_r25601561?' accessKind=read location=<REPL Input>:4:24 range=[<REPL Input>:4:23 - line:4:26]  decl=Swift.(file).Array.subscript [with S_r25601561?]  arg_labels=_:
          (inout_expr implicit type='inout Array<S_r25601561?>' location=<REPL Input>:4:23 range=[<REPL Input>:4:23 - line:4:23]
            (member_ref_expr type='@lvalue [S_r25601561?]' accessKind=readwrite location=<REPL Input>:4:23 range=[<REPL Input>:4:23 - line:4:23] decl=REPL.(file).C_r25601561.a@<REPL Input>:2:9
              (declref_expr implicit type='C_r25601561' location=<REPL Input>:4:23 range=[<REPL Input>:4:23 - line:4:23] decl=REPL.(file).C_r25601561.func decl.self@<REPL Input>:3:10 function_ref=unapplied specialized=no)))
          (paren_expr type='(Int)' location=<REPL Input>:4:25 range=[<REPL Input>:4:24 - line:4:26]
            (declref_expr type='Int' location=<REPL Input>:4:25 range=[<REPL Input>:4:25 - line:4:25] decl=REPL.(file).C_r25601561.func decl.i@<REPL Input>:3:15 function_ref=unapplied specialized=no))))))
  (tuple_expr type='()' location=<REPL Input>:4:34 range=[<REPL Input>:4:34 - line:4:35]))

So following that diagnoseExprFailure happy returns with correctly type checked sub-expression attached to the ForceValueExpr and now it's time to do diagnoseContextualConversionError (which is correct), what that does is - it strips away all of the already assigned types from the expressions but not their newly found declarations, which results into the problem at hand. So since subscript in the normal case doesn't deal with re-written AST which contains InOutExpr, and I think it's valid to have InOutExpr there since that particular subscript is mutable and CallExpr had successfully type-checked, I decided to simply fix subscript generation to handle inout'ness.

@xedin Yes, that's what I saw under the debugger and it's good to detail exactly what's going on.
My concern with this particular fix is it doesn't look like under normal circumstances we would ever have an InOutExpr as a child of a SubscriptExpr on entry to the type checker. We only get these as a result of a successful type check, and you wouldn't normally expect to feed a successfully type checked expression back into the type checker.
The current mechanism used in the diagnostics (of type checking pieces and mutating the AST) isn't ideal. I believe we see a significant number of crashers directly as a result of this kind of mutation happening, for reasons similar to this one. I think the right long term design would be to either not mutate the AST during diagnostics emission, or to roll back those changes before attempting to build a new constraint system. It's not clear to me how large a change that would end up being.
My specific concern about taking changes like these is that it's easy for other people who are coming up-to-speed on our code to see something like this and assume that we should expect InOutExprs in subscripts, and that the right way to deal with them is to force them to LValues.
Being pragmatic, though, it's a fine fix to avoid this particular issue - I think we just need ample warning to go with the code explaining why it's there (e.g., "// FIXME: We don't normally expect InOutExpr subexpressions of subscripts on entry to constraint generation but can get them as a result of diagnostics passes mutating the AST. We should ideally fix diagnostics to not do that.").
One other suggestion on your particular change which I think would make it a bit cleaner is to explicitly check for InOutExpr and walk through it rather than looking at and modifying the types. That should work, and with a comment explaining why we're doing this I think we could probably take the change with the hope that one day we can fix the diagnostics passes.
Thanks for working on this - as I mentioned I think this probably fixes more than one reported issue.

@rudkx We are totally on the same page, I just wanted to write it down so we have it in logged.
What you are describing is exactly what I tried to do, but I couldn't find a way to get that original sub-expression back after type checking of the ForceValueExpr rewrote whole AST :( I think that would require bigger change to fix properly, I'll make a change for checking for InOutExpr explicitly and put FIXME for that, and you are right, that would make things for clean for new people (like myself).
I can also attempt to properly solve this situation as a separate issue, WDYT?

@xedin I think the proper solution is probably best to defer for the moment. At some point it's definitely something worth tackling, though!

Got it!

@rudkx ^ All done, please let me know what you think about FIXME message.

@rudkx ^ all requested changes are addressed.

@swift-ci Please smoke test and merge

Looks like swift-ci doesn't want to merge automatically for some reason...


Closes #509

Wrap ${extensionPath} in single quotes to alleviate issues with spaces existing in extensionPath var.

Thank you!

Thank you!


Add Remove option to GestureRecognizerCollection

Is your feature request related to a problem? Please describe.
While working on Fabulous.Avalonia gesture recognizers, I found that GestureRecognizerCollection uses IReadOnlyCollection<GestureRecognizer> and only exposes  Add.
Describe the solution you'd like

Update GestureRecognizerCollection to inherit from.AvaloniaList< GestureRecognizer>
Or keep IReadOnlyCollection<GestureRecognizer> and expose a Remove method that will modify the private List<GestureRecognizer>? _recognizers

Describe alternatives you've considered

Create a type based on GestureRecognizerCollection and use reflection
Additional context
Fabulous.Avalonia
GestureRecognizers PR

@emmauss adding an option to remove a gesture sounds like a valid thing to do to me. What do you think, could we accept a PR introducing this API?
If yes, @edgarfgp a PR would be welcome.

@timunie I would be happy to contribute so I can complete the support of Avalonia in F# Fabulous

@timunie I was wondering if there are any issues with the gestures currently. Looking at https://github.com/AvaloniaUI/Avalonia/blob/master/samples/ControlCatalog/Pages/GesturePage.cs I could not trigger any gesture. Thanks in advance :)

@timunie I was wondering if there are any issues with the gestures currently. Looking at https://github.com/AvaloniaUI/Avalonia/blob/master/samples/ControlCatalog/Pages/GesturePage.cs I could not trigger any gesture. Thanks in advance :)

Some gestures require touch input.


ORMAdapter problem with EntityManager overriding

Hello,
Can you properly type this code in the ORMAdapter by adding the EntityManagerInterface type to the $manager attribute and to the afterConfiguration method in order to replace the test "$manager instanceof EntityManager"?
Indeed, if we try to override the manager with a decorated manager, the test $manager instanceof EntityManager fails and throws an exception.
But it works correctly by adding an instanceof EntityManagerInterface and it's more SOLID to test an Interface.
class ORMAdapter extends AbstractAdapter
{
    private ManagerRegistry $registry;
    protected EntityManagerInterface $manager;
    protected ClassMetadata $metadata;

....
}

/**
     * @param ORMOptions $options
     */
    protected function afterConfiguration(array $options): void
    {
        // Enable automated mode or just get the general default entity manager
        $manager = $this->registry->getManagerForClass($options['entity']);
        if (!$manager instanceof EntityManagerInterface) {
            throw new InvalidConfigurationException(sprintf('Doctrine has no valid entity manager for entity "%s", is it correctly imported and referenced?', $options['entity']));
        }
....
}

@shades684 seems to make sense right?


Fix build failures for Android on Windows

The issue is caused by https://github.com/godotengine/godot/pull/64306 which makes use of a 3.7 feature while the current recommended python version is 3.5 for 3.x and 3.6 for master.

Does this work? To reduce code duplication and make the version specific parameter more obvious:
diff --git a/methods.py b/methods.py
index 02d2d32184..c29ff3036f 100644
--- a/methods.py
+++ b/methods.py
@@ -412,27 +412,17 @@ def use_windows_spawn_fix(self, platform=None):

         startupinfo = subprocess.STARTUPINFO()
         startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
+        popen_args = {
+            "stdin": subprocess.PIPE,
+            "stdout": subprocess.PIPE,
+            "stderr": subprocess.PIPE,
+            "startupinfo": startupinfo,
+            "shell": False,
+            "env": env,
+        }
         if sys.version_info >= (3, 7, 0):
-            proc = subprocess.Popen(
-                cmdline,
-                stdin=subprocess.PIPE,
-                stdout=subprocess.PIPE,
-                stderr=subprocess.PIPE,
-                startupinfo=startupinfo,
-                shell=False,
-                env=env,
-                text=True,
-            )
-        else:
-            proc = subprocess.Popen(
-                cmdline,
-                stdin=subprocess.PIPE,
-                stdout=subprocess.PIPE,
-                stderr=subprocess.PIPE,
-                startupinfo=startupinfo,
-                shell=False,
-                env=env,
-            )
+            popen_args[text] = True
+        proc = subprocess.Popen(cmdline, **popen_args)
         _, err = proc.communicate()
         rv = proc.wait()
         if rv:

Thanks!

Cherry-picked for 3.6.

Cherry-picked for 3.5.1.


Added some very basic example to simplify the start for users

As I did struggle to get started using the readline lib I thought creating a basic example on how the completions can be used would be useful for others, too.

Hello,
Users wishing to add a tree of commands to the shell should use the dedicated console library, and if we strip down this example from command code, it remains pretty much nothing, so closing this.


Pixelfed posts with 5 or more photos cropped in weird dimention when logged in via Pixelfed account

Phanpy displays the full amount of photos from the post when logged into via Pixelfed but depending on the screen width crops them to a strange dimension.

post in question: https://pixelfed.social/p/Lacalle/684124192137313050
the view from the search screen page (attached below) looks quite fine to me with adding 4+ on the forth photo and then loading the lightbox on click

tested on dev.phanpy.social (version)
Firefox 124.0.2 (64-bit)
macOS 13.6.4

I just noticed this view is also there when logged in from only a mastodon account but when looking at the remote Pixelfed timeline as well

There's no design done yet for posts with >5 media attachments. Most styling work is optimized for Mastodon's limit of 4 media attachments. So, what we're seeing here is best-effort minimum-viable design 🙂
The "4+" thing was added to anticipate >4 media attachments, limit them and prevent them from taking too much space in the timeline, but show all on the post page.
As for the odd cropping, I'll need to figure out what's the best way to display them and think about various cases like:

Different dimensions. Edge case: very big or small images.
Different aspect ratios.  Edge case: very tall/wide images.
Even/odd/max number of media attachments: Edge case: hundreds of images? (if any instance even allows it?)
Mixed list of video + audio + photo + unknowns (Also depends on the instance allowing it)


Unable to handle an exception

I am getting this error and I unable to handle the exception, even if I put the operation under try and except.
Error:
Error Code: 404 URL:https://sandboxapic.cisco.com/api/v1/interface/network-device/17184480-2617-42c3-b267-4fade5f794a9 Data:{} Headers:{'X-Auth-Token': 'ST-4896-fVSZ1kDWAu0aKwnJtfcb-cas', 'Cookie': 'JSESSIONID=4DFF3F3BAC4B9E8C1F4F1B026DE4821F; Path=/apic-em-inventory-manager-service/; HttpOnly; Secure', 'Content-Type': 'application/json', 'Accept': 'application/json', 'X-CSRF-Token': 'soon-enabled'} Message:{"response":{"errorCode":"Not found","message":"Request data not found","detail":"No resource found with deviceId: 17184480-2617-42c3-b267-4fade5f794a9"},"version":"1.0"}
Traceback (most recent call last):
  File "C:\Python35-32\lib\site-packages\uniq\apis\client_manager.py", line 187, in call_api
    response.raise_for_status()
  File "C:\Python35-32\lib\site-packages\requests\models.py", line 862, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://sandboxapic.cisco.com/api/v1/interface/network-device/17184480-2617-42c3-b267-4fade5f794a9

Code:
from uniq.apis.nb.client_manager import NbClientManager

apicEmServer = NbClientManager(server="sandboxapic.cisco.com",
                                 username="devnetuser",
                                 password="Cisco123!",
                                 connect=True)

allNetworkDevices = []
allInterfaces = {}
try:
    allNetworkDevices = apicEmServer.networkdevice.getAllNetworkDevice()
    allNetworkDevices = apicEmServer.serialize(allNetworkDevices)
except:
    print("Error")
if allNetworkDevices is not None:
    for networkDevice in range(len(allNetworkDevices['response'])):

        try:
            deviceInterfaces = apicEmServer.interface.getInterfaceByDeviceId(deviceId = allNetworkDevices['response'][networkDevice]['id'])
            deviceInterfaces = apicEmServer.serialize(deviceInterfaces)
            print(allNetworkDevices['response'][networkDevice]['id'])

            if deviceInterfaces is not None:
                for line in range(len(deviceInterfaces['response'])):
                    if deviceInterfaces['response'][line]['description'] != "" and deviceInterfaces['response'][line]['description'] is not None:
                        allInterfaces[deviceInterfaces['response'][line]['description']] = allNetworkDevices['response'][networkDevice]['hostname'], allNetworkDevices['response'][networkDevice]['managementIpAddress'], deviceInterfaces['response'][line]['portName']
        except:
            print("Unable to get interface information from " + allNetworkDevices['response'][networkDevice]['id'])
print(allInterfaces)`
```

I just ran your code verbatim, and it works perfectly fine, this is the output.
2dc30cac-072e-4d67-9720-cc302d02695a
Unable to get interface information from 17184480-2617-42c3-b267-4fade5f794a9
3ab14801-5c88-477d-bbb5-0aca5e5ba840
80036c7d-c82e-412c-ac01-b05f9badfcba
Unable to get interface information from 80036c7d-c82e-412c-ac01-b05f9badfcba
bc14ad0c-197b-476c-b9f9-10758879b4e4
Unable to get interface information from bc14ad0c-197b-476c-b9f9-10758879b4e4
24ac6aa8-7759-44d5-90a3-00c83e96583d
1a167f1a-feac-4ff1-8f70-9e20af22d99b
ed8dbe19-5dd0-48e4-a40f-c8787f1d93a5
8d32f709-cbcf-4c9b-99eb-ee6aabde8e74
554477e0-46ac-47bb-990e-4e20f0450e6f
d23ba4fb-cfe8-465b-be23-a9b0d89ce898
93ad4feb-5d17-4999-b46d-387f69113fc3
5a3bdb62-5def-40a1-be98-944ba2a7d863
Unable to get interface information from 5a3bdb62-5def-40a1-be98-944ba2a7d863
{' Connected to CAMPUS-CORE1': ('CAMPUS-Dist1', '<IP_ADDRESS>', 'GigabitEthernet5/7'), ' Connected to CAMPUS-Access1': ('CAMPUS-Dist1', '<IP_ADDRESS>', 'GigabitEthernet5/5'), ' Connected to Dist1': ('CAMPUS-Access1', '<IP_ADDRESS>', 'GigabitEthernet1/0/1'), ' Connected to Uplink Router': ('AHEC-2960C1', '<IP_ADDRESS>', 'FastEthernet0/1'), ' Connected to WLC5508': ('CAMPUS-Dist1', '<IP_ADDRESS>', 'GigabitEthernet5/38'), ' Connected to CAMPUS-DIST2': ('CAMPUS-Dist1', '<IP_ADDRESS>', 'GigabitEthernet5/48'), 'Unit: 0 Slot: 0 Port: 4 Gigabit - Level 0x6070001': ('Campus-WLC-5508', '<IP_ADDRESS>', 'GigabitEthernet0/0/4'), ' Connected to PnP Demo Switch': ('Branch2-Router.yourdomain.com', '<IP_ADDRESS>', 'GigabitEthernet0/0'), ' To CAMPUS-Core2': ('CAMPUS-Core1', '<IP_ADDRESS>', 'GigabitEthernet3/47'), ' #WAN#100M#SPP1#': ('CAMPUS-Router2', '<IP_ADDRESS>', 'GigabitEthernet0/0/3'), ' Connected to Wireless AP': ('CAMPUS-Access1', '<IP_ADDRESS>', 'GigabitEthernet1/0/26'), ' To CAMPUS-Core1': ('CAMPUS-Core2', '<IP_ADDRESS>', 'GigabitEthernet3/47')}

Actually you are handling the exception, that is the reason the code proceeds to display the interface information for other network devices.
This is the response I see:
2dc30cac-072e-4d67-9720-cc302d02695a
Error Code: 404 URL:https://sandboxapic.cisco.com/api/v1/interface/network-device/17184480-2617-42c3-b267-4fade5f794a9 Data:{} Headers:{'Accept': 'application/json', 'X-Auth-Token': 'ST-5937-EdyBXOysZOt0vdmwFtwa-cas', 'Content-Type': 'application/json', 'Cookie': 'JSESSIONID=00A28CF92507CFD916686E31865F026B; Path=/apic-em-inventory-manager-service/; HttpOnly; Secure', 'X-CSRF-Token': 'soon-enabled'} Message:{"response":{"errorCode":"Not found","message":"Request data not found","detail":"No resource found with deviceId: 17184480-2617-42c3-b267-4fade5f794a9"},"version":"1.0"}
Traceback (most recent call last):
  File "/Users/amepatil/.virtualenvs/uniq_github_package_installed/lib/python3.5/site-packages/uniq/apis/client_manager.py", line 187, in call_api
    response.raise_for_status()
  File "/Users/amepatil/.virtualenvs/uniq_github_package_installed/lib/python3.5/site-packages/requests/models.py", line 862, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://sandboxapic.cisco.com/api/v1/interface/network-device/17184480-2617-42c3-b267-4fade5f794a9
Unable to get interface information from 17184480-2617-42c3-b267-4fade5f794a9
3ab14801-5c88-477d-bbb5-0aca5e5ba840

The following line is printed after the error message
Unable to get interface information from 17184480-2617-42c3-b267-4fade5f794a9

Thank you very much for the fast response, This means I will see the error no matter what ?

To give more context, that exception is getting logged as part of
https://github.com/CiscoDevNet/uniq/blob/master/apis/client_manager.py#L189
Also, for why the other print messages that are not preceded by the same exception, that could be due to the script failing for some other reason (may be a KeyError or IndexError) in these lines:
deviceInterfaces = apicEmServer.serialize(deviceInterfaces)
            print(allNetworkDevices['response'][networkDevice]['id'])

            if deviceInterfaces is not None:
                for line in range(len(deviceInterfaces['response'])):
                    if deviceInterfaces['response'][line]['description'] != "" and deviceInterfaces['response'][line]['description'] is not None:
                        allInterfaces[deviceInterfaces['response'][line]['description']] = allNetworkDevices['response'][networkDevice]['hostname'], allNetworkDevices['response'][networkDevice]['managementIpAddress'], deviceInterfaces['response'][line]['portName']

The recommended practice is to have a smaller block (ideally a single line) between an all encapsulating try and except.
Something like this:
        try:
            deviceInterfaces = apicEmServer.interface.getInterfaceByDeviceId(deviceId = allNetworkDevices['response'][networkDevice]['id'])
        except:
            print("Different EXCEPTION \n")

        try:
            deviceInterfaces = apicEmServer.serialize(deviceInterfaces)
            print(allNetworkDevices['response'][networkDevice]['id'])

or make the except catch a specific exception like so:
from requests.exceptions import HTTPError
.
.
 except HTTPError as e:
            print("Unable to get interface information from "

Another recommendation is to always print/log the Exception itself
except HTTPError as e:
            print(e)

except Exception as e:
            print(e)

Thank you! I will work on it.

All information that this library logs is done through python's logging module.
We have defined a logger named uniq
So in case you do not want to see that log message, you can do some thing like so:
import logging
logger = logging.getLogger("uniq")
logger.disabled = True

Closing as not an issue.


Add knative-eventing-tutorial example

Work in progress, as I am debugging an obscure error that I encountered only today (not yesterday or the day before...)
In the meanwhile, you may review the code and the respective wiki page. I abstained from copy-pasting the code to the wiki as code changes often, but I'll add a pointer to here instead.
Signed-off-by: Bora M. Alper<EMAIL_ADDRESS>

Now that we have bumped Knative to v0.23, this too is ready to merge.

@ustiugov Is there anything else you have requested that I am missing?

apart from the base image, lgtm. great job

please add info to the changelog in the Added section

@ustiugov Done.


457 : prompt for unsubscrigin private stream & deactive private strea…

fix bug #457 - Private stream cannot be deleted after no one remains

Automated message from Dropbox CLA bot
@devstorm21, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Automated message from Dropbox CLA bot
@devstorm21, thanks for signing the CLA!

Hi @devstorm21 - congrats on the tests passing!
The next time you need to update a pull request, you don't need to close the old one and open a new one. You can update your pull request and we recommend you do that (instead of closing the old PR and opening a new one) because then it's easier to see the history of what you've done, to to check that you've responded to previous reviewers' comments, and to preserve links we've mentioned (in comments) with other issues and PRs.
Thanks for writing a descriptive commit message. Here's a guide to help you write commit messages that are more consistent with our style, which makes it easier for us to skim them in logs and quickly understand what's happening. Here are some tips on tidying your commit history. And when you update your code and your commit message, you can update your pull request and the changes will appear here. :)
Thanks again! Looking forward to seeing the next revision. I'm not a frontend or Django expert so other people will take the lead on reviewing those parts of your code.

hi @devstorm21 - how's it going?

Hello @devstorm21, a Zulip maintainer reviewed this pull request over a week ago, but you haven't updated your pull request since. Please take a look at the requested changes and update your pull request accordingly.
Thank you for your valuable contributions to Zulip!


OAuth unable to login user when signing in through facebook with a rails api only applicaiton

Hello and thanks for creating such a good package!
I have a setup with rails api only app (built in rails 5) as my back end with devise_token_auth, omniauth-facebook and front end angular2 application with angular2-token integrated.
After thoroughly going through the documentation I have implemented the user token authentication which works just fine, however I since yesterday am attempting to log user in via facebook.
For that I went through the documentation of this package and on a deeper research I found that with rails api only appliction the OAuth flow is not working which was a bit concerning but diving much deeper I found this sample app which was really helpful
https://github.com/piotrkaczmarek/ng2AuthRouterStarter
I used this app as well as the documentation as a guide to implement the facebook auth.
What happens is when I attempt to login through facebook, I am taken to facebook in the same window (i configured it to be the same window) and upon providing my email and password I see user being created in the backend and then I am taken back to my angular app at the following URL:
 http://localhost:4200/oauth_callback?auth_token=************&blank=true&client_id=*******************&config=&expiry=*******************&uid=*******************#_=_
As per the documentation I have made a route and a component (OauthCallbackComponent) to intercept this URL request on the client end inside the OauthCallbackComponent I can see the this._tokenService.currentAuthData with all the data that was passed in the params, however the user remains logged out and no relevant information is stored in the local storage. Now I am not sure what will be the next step because this is all the flow I have and I think I am almost at the finish line, I want to know if after all this process I have to somehow manually set these auth credentials I receive in the URL in browser's localstorage and then log user in myself or is there something I may have missed.
Thanks in advance.
Inside the sign in component.ts this is the method triggered from UI
facebookAuth(){
    this.authService.initiateFacebookAuth()
  }

This is the corresponding call in the auth.service.ts
initiateFacebookAuth(){
    this._tokenService.signInOAuth('facebook');
  }

This is the route I made to intercept the oauth_callback
{
  path: 'oauth_callback',
  component: OauthCallbackComponent
}

In the ngOnInIt of the OauthCallbackComponent
this._tokenService.processOAuthCallback();
console.log(this._tokenService.currentAuthData);

I have read that processOAuthCallback() fetches the auth data returned but even if I remove this method I can see the currentAuthData in my browser's console.
Is there anything I am missing ?
I can see the user being created via omniauth-facebook inside the database and on every subsequent request I see their login count increasing  but I am actually not signed in the front end.
Is there anything I need to do or have I missed anything, I would really appreciate any guidance.
Thanks a lot!
UPDATE
For the moment what I have done is, I have manually stored the auth data I received in the oauth_callback into the local storage. I think there might be a better way to do this. If there is please let me know.

Hi @91sticker can you please show how did you handle to store the data manually ? I may be getting the same issue here.

Hello @marlonmantilla m sorry for responding so late, was out of station.
Here is how I handled the issue:
So basically after going through the documentation and through the source of sample project, I created a component named "oauth-callback.component" and inside my router the path 'oauth_callback' points to that particular component. Upon redirection from facebook (after logging in) this component intercepts the response and inside the onInit lifecycle hook of this method I was able to get the user's data in response and then store it inside the local storage.


Tolerations set are not seen on the workload in Edit workload page

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
Deploy a cluster.
Create a workload with tolerations
Key: test
Equals
Value: value1
Effect: NoExecute

Click on Edit workload after it comes up Active.
Expected Result:
Tolerations set earlier must be available on the UI.
Actual Result:
Tolerations set are not available.
View in API on the workload shows the tolerations
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): rke
Kubernetes version (use kubectl version):

1.15.3

Docker version (use docker version):

18.9.8

Verified on master-head - commit id: d7b425da7
Steps:

Deploy a cluster.
Create a workload with tolerations

Key: test
Equals
Value: value1
Effect: NoSchedule

Click on Edit workload after it comes up Active.
Tolerations set earlier must be available on the UI.


(GER) ASOP terminal + Minor elevator + part of hull scraping

ASOP terminals translated
Minor improvements on the elevator ---- Replaced the word "Stockwerk" to "Etage"
accidentally translated a small part for hull scraping 😄

@all-contributors Please add @Auhrus for translation

@all-contributors Please add @ROBdk97 for projectManagement


user12986714: Watch [^<]*+<a href

user12986714 requests the watch of the watch_keyword <strong>[^<]*+<a href. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

<strong>[^<]*+<a href has been seen in 7154 true positives, 290 false positives, and 40 NAAs.

Approved by tripleee in Charcoal HQ


How to parse an array of string

Does anyone know how to parse an array of string. Like this:
"zoneInfo": {
    "tariffInfoLines": [
        "In this city you pay per minute."
    ]
}

We have a zoneInfo object that contains a tariffInfoLines array. This tariffInfoLines array contains strings. To store in Realm, i define RealmString object and store it in RLMArray
@interface RealmString : RLMObject
@property NSString *stringValue;
@end
@implementation RealmString
@end

RLM_ARRAY_TYPE(RealmString)

@interface TariffInfo : RLMObject
@property RLMArray<RealmString> *storedTariffInfoLines;
@end

But i dont know how to parse it from JSON using Realm-JSON. The application will crash if you try to. So i really need some help to solve this problem!

That's because you haven't defined the mapping for stringValue:
@interface RealmString : RLMObject
@property NSString *stringValue;
@end
@implementation RealmString
+ (NSDictionary *)JSONInboundMappingDictionary {
  return @{
         @"self": @"stringValue",
  };
}
@end

Thanks @matthewcheok but it seems not work. It still crash at line 190 of RLMObject+JSON.m
for (id item in(NSArray*) value) {
	[array addObject:[elementClass mc_createObjectFromJSONDictionary:item]];
}

Because item is type of NSString not NSDictionary

Your issue looks very similar to matthewcheok/Realm-JSON#4 Mind sharing your code sample?

Oh, it works now after clean database and project. Thanks so much @matthewcheok


frees-rpc new Generation

frees-rpc will adopt the new generation of Scala Macros in the Freestyle ecosystem. This new version of the scala macros will be supported natively by the compiler in 2.13, under the scalac option -Ymacro-annotations, thanks to https://github.com/scala/scala/pull/6606.
However, the migration of the current scalameta macros (no longer under development and won't support 2.13.x) to the new Scala Macros can be huge, mainly taking into account both https://github.com/frees-io/freestyle and https://github.com/frees-io/freestyle-rpc.
As you probably know, frees-rpc is currently depending on frees-core for defining the tagless final algebras, so we are suggesting two steps work to achieve this migration.

First, in order to parallelize the work, we need to decouple temporarily frees-rpc from frees-core. The reason would be just to minimize the potential blockers from the migration in frees-core side, which presumably would be bigger, so we could migrate Freestyle RPC faster and independently from the core. As you can imagine, we want frees-core back to frees-rpc as soon as both are inline in terms of binary compatibility (it could be considered the third stage).

As part of the second step, we'll do the migration itself, where the RPC @service macro annotation will be migrated to use the new generation of Scala Macros.

The Freestyle Maintainers team hope this makes sense and want to say that any kind of feedback is very welcomed :)

Related to https://github.com/frees-io/freestyle/issues/539.

Closed by #339


Ratelimiting

So, like I told @daveajones , I found out there's a rate limiting system in place... the hard way :}
So thank so much for the warning and plz update docs <3
And also:
As a developer, when I create an service on the API, I would like to be informed on the current status of my requests and limits, so I can avoid hitting that 429 or predict when I am able to send new requests.
Normally this is done through headers, plus some explanation in the documentation on how to interpret these headers. A countdown header and knowing cooldown/reset time would suffice I think.
Protip: custom headers usually start with x- and I have a feeling that might help pushing them through Cloudflare.

What is the rate limit set to and for what period?

@daveajones said, "You want to keep your requests at less than one per second" (#16 ). Is this per account, per key, or per client?

We don't publish rate limits because we have no SLA's or guarantees.  We are a free project that runs on donations and I don't want everyone pushing their usage right up to the edge of the published rate limits.  If everyone does that we can't support the load.  We adjust things continuously as needed to support the load we have at the time.

No. A rate limit is a 429. Make sure you are using a proper user agent header and not the ones from the sample code.

No. A rate limit is a 429. Make sure you are using a proper user agent header and not the ones from the sample code.

I think I'm doing everything right, I had it working and then all of a sudden it stopped.
This is my request code, I don't know if anything jumps out to you when you look at this:
const headerTime = Math.floor(Date.now() / 1000).toString()
const key = import.meta.env.VITE_PODCASTINDEX_KEY
const secret = import.meta.env.VITE_PODCASTINDEX_SECRET
const hashInput = key + secret + headerTime
        const hash = CryptoJS.SHA1(hashInput).toString(CryptoJS.enc.Hex)

        const episodesResponse = await axios.get(
            `https://api.podcastindex.org/api/1.0/episodes/byfeedid?id=${id}`,
            {
                headers: {
                    'User-Agent': 'AppleMusicDashboard/1.0',
                    'X-Auth-Key': key,
                    'X-Auth-Date': headerTime,
                    Authorization: hash,
                },
                params: {
                    fulltext: true,
                    max: 10,
                },
            }
        )
        console.log('episodes', episodesResponse)

all the variables print out fine when I log them to console, but I'm getting the hash auth error:
The hash in the Authorization header doesn't match up.  Please check your hash compute code: https://podcastindex-org.github.io/docs-api/#overview--authentication-details

Resolved by making a new key secret pair. Maybe I didn't copy/paste them correctly last time..


Select and Cancel buttons are not Working.

https://user-images.githubusercontent.com/23289583/167604258-9d53d2bb-b561-470f-98c4-ae5a0278a196.mov

https://stackoverflow.com/questions/72195696/submit-and-cancel-button-not-working-in-react-google-picker

Anyone ever find a solution? I am not using this library but I have the exact same issue in my react application....
I can select a file by double clicking, but that is the only way to select.
I have a "select folder" option in my app and there is no way to select a folder since double click does not work there.

I have been plagued with this issue for a long time (somewhat intermittently) but now it is very consistent.
I finally figured out the cause of my issue, may or may not be the same as others - but I had a modal open to allow the user to choose where to save a file and if they opened the google picker to choose a folder it would not work - but it seemed to work fine in other places.
Not sure exactly why, but my modal seemed to have a conflict with the google drive picker modal. Select/cancel/X/filter menus all did not work. I had to keep track of when the google picker was open and omit some of the CSS classes for my modal and it appeared to resolve the issue.
In summary - I think there was some css conflict from items on my page that only applied to a few components on the picker.


Add option to disable trailing commas on multi-line collections

Extends the configuration options to allow disabling the addtion of trailing commas on multi-line collections.

I gleaned the name multilineCollectionTrailingCommas from an older PR, but since this does affect single line dictionaries, I wonder if collectionTrailingCommas would be more appropriate.

I gleaned the name multilineCollectionTrailingCommas from an older PR, but since this does affect single line dictionaries, I wonder if collectionTrailingCommas would be more appropriate.

Yeah, "line" is the wrong word here, because the logic isn't based on whether the collection wraps across multiple lines, it's about whether it has multiple elements—single element arrays/dictionaries never have a trailing comma inserted, even if they wrap (because of Swift parser subtleties around how collection types are parsed in expression contexts). I think there are other places in swift-format where we use "line" when we really mean "statement" or "element", but fixing those up require ensuring compatibility with older configs.
How do you feel about multiElementCollectionTrailingCommas?

I gleaned the name multilineCollectionTrailingCommas from an older PR, but since this does affect single line dictionaries, I wonder if collectionTrailingCommas would be more appropriate.

Yeah, "line" is the wrong word here, because the logic isn't based on whether the collection wraps across multiple lines, it's about whether it has multiple elements—single element arrays/dictionaries never have a trailing comma inserted, even if they wrap (because of Swift parser subtleties around how collection types are parsed in expression contexts). I think there are other places in swift-format where we use "line" when we really mean "statement" or "element", but fixing those up require ensuring compatibility with older configs.
How do you feel about multiElementCollectionTrailingCommas?

It's a mouthful, but it certainly self-documents what the setting is for. I'll refactor it.

Refactored the setting name to multiElementCollectionTrailingCommas and use literal rather than initializer in the docs.

Added a DocC header to auto-gen proper documentation based on the styling of indentSwitchCaseLabels


Add french translation

Hi! Here's a French translation 🥖. Should I do anything else?

Hey, thank your for the translation! I'll upload it in the next few days!

thanks :)

I've uploaded a new release for it! Sorry for the little delay! :)


Line break in tooltipster

Hi!
Is it possible to add a linebreak in the titel so tooltipster show a new sentence?
I searched and tried everything but can't find it.
regards
Jardin

This is not working for me, it shows then


Fix disappearing subscription tab when switching accounts

It fixes #186
Test plan

Add user with enterprise account
Add another user with dotcom account
Switch between enterprise and dotcom accounts
Subscription tab should appear for dotcom account and disappear for enterprise account

https://github.com/sourcegraph/jetbrains/assets/9321940/309974c6-fc1a-45a6-912f-ce3cacfab9d7

TODO: There is an issue with accounts added before this fix

Tested locally. I can confirm that it works 🚀


[BUG] Projected vs. Flatten: could they be unified?

Describe the bug
Current project_to_simple is bad in both performance and memory usage, as it will copy the projected v/e's structure, even when the graph has only one vertex label and one edge label.
That is really bad, as the cost of project will greater than the cost of running an application.
Now we have flatten fragment, could the ProjectedFragment be replaced with the flatten fragment? Or, could we make it doing less copy? e.g., even a special case for graphs that contains only one vertex label and one edge label.
Additional context
Related to #1300.

The add_column may not work for Flatten currently.

The add_column may not work for Flatten currently.

It can be done as long as we have the meta, seems not a blocking issue:)


Stabilise server for production release

DoD:

All automated tests pass.
Manual field tests of all features: try with realistic usage scenarios.
Open a GitHub issue for each discovered bug.
Fix bugs that are of critical importance for release.

This should have been closed already.


Updated CMS, fixed img rendering bug, updated podcast/ episode pages

Updated the CMS layout per discussion with Rafna. Fixed the img rendering bug. 👍  Updated podcast and episode pages.

Looking great.


Big fonts for old eyes?

Thank you for the new library.  All your hard work is appreciated.
Is it possible to get fonts around 72pt to work with this library?
I created a 72pt Roboto Black font using http://oleddisplay.squix.ch/#/home but it ended up only displaying the left hand side of the numerals.
44pt works great.  50 has some issues at the margins(time in the pic below) but is passable but 72 is mostly incomplete (the temp in the pic below.
I suspect it has something to do with this compiler warning which I don't get with regular size fonts.
src\roboto_black_72.h:458:1: warning: large integer implicitly truncated to unsigned type [-Woverflow] src\roboto_black_72.h:458:1: warning: narrowing conversion of '405' from 'int' to 'const char' inside { } [-Wnarrowing]

roboto black big fonts.zip

I'm running into the same issue that fonts in this side not shown correct. Getting the same error message and the same output on the screen. Would be great if there is a way to get this fixed :)

The problem is the generated code is creating numbers that are too large for the type.
If you look at the output of the generating site you'll see this:
 const char Dialog_plain_78[] PROGMEM = {
0x4E, // Width: 78
0x5C, // Height: 92
0x20, // First Char: 32
0xE0, // Numbers of Chars: 224

// Jump Table:
0xFF, 0xFF, 0x00, 0x19,  // 32:65535
0x00, 0x00, 0xEE, 0x1F,  // 33:0
0x00, 0xEE, 0x13D, 0x24,  // 34:238
    ... more

So you see it is creating an array of char.  A char will hold values 0-0xFF (0-255).  But the values in the list are bigger than that.  e.g. 0x13D is bigger than 0xFF.
To fix this you need to do two things:

Change the type:
See if it works.

In the generated output change  const char Dialog_plain_78[] PROGMEM = { to:
  const int Dialog_plain_78[] PROGMEM = {

That will ensure those values are correct.  Of course the next step is seeing if what consumes the fonts will work with int rather than char.  You'll either get lucky, or not.  If it fails to compile you'll soon spot the problem and I suspect it should be as simple as updating the types appropriately.  (i.e. Carrying on changing char to int as you see error-messages about truncation, or mis-matched types.)

So I'm a bit new to c/c++ but followed skx's explanation above (thanks btw) and have updated my font type to int.
Unfortunately the Minigrafx library itself does seem to be struggling to consume the int font data, with the compiler erroring out with the following in a few places in my code.
no matching function for call to 'MiniGrafx::setFont(const int [64671])'
Attempting to update the Minigrafx library is probably beyond my current skill level without guidance at the moment - has anyone managed to successfully update Minigrafx to show large fonts?

Same issue, please fix it, who can.
Thank you in advance


Call ninja automatically if bf is called without any arguments

So we don't need to type bf && ninja but just bf

In generated makefile we are using buildfox.py (without args) as a configure rule - it will result in ninja call.
We need to do something with it - maybe use some argument for it.

Might be something like --do-not-run-ninja :/

--just-generate? :)

That's better :+1:

It's quite buggy on Windows atm:
C:\work\try_material>bfd
b'[1/7] cc src/css-micro-parser/test.c'
b'[2/7] cc src/aheasing/easing.c'
b'[3/7] cc src/material_gl/material_gl.c'
b'[4/7] cc src/tigr/tigr.c'
b'[5/7] cc src/glew/glew.c'
b'[6/7] cc src/main.c'
b'[7/7] link build/app.exe'
b'FAILED: build/app.exe'
b'cl /nologo @build/app.exe.rsp /MT /link  /LIBPATH:src/libcurl/static-release-x64  /out:build/app.exe'
b'LINK : warning LNK4031: no subsystem specified; CONSOLE assumed'
b'main.obj : error LNK2019: unresolved external symbol _curl_easy_strerror referenced in function _main'
b'main.obj : error LNK2019: unresolved external symbol _curl_easy_init referenced in function _main'
b'main.obj : error LNK2019: unresolved external symbol _curl_easy_setopt referenced in function _main'
b'main.obj : error LNK2019: unresolved external symbol _curl_easy_perform referenced in function _main'
b'main.obj : error LNK2019: unresolved external symbol _curl_easy_cleanup referenced in function _main'
b"src/libcurl/static-release-x64\\libcurl_a.lib : warning LNK4272: library machine type 'x64' conflicts with target machine type 'X86'"
b'build/app.exe : fatal error LNK1120: 5 unresolved externals'
b'ninja: build stopped: subcommand failed.'
Traceback (most recent call last):
  File "C:\work\buildfox\buildfox.py", line 497, in <module>
    main()
  File "C:\work\buildfox\buildfox.py", line 491, in main
    sys.exit(run_command("ninja"))
  File "C:\work\buildfox\buildfox.py", line 320, in run_command
    output = process.stdout.readline()
KeyboardInterrupt

The worst - it gets stuck in the end.

What is bug about here and what do you mean by 'stuck'?

b'ninja: build stopped: subcommand failed.' is just wrong, stdout bytes are not converted to string, it should print ninja: build stopped: subcommand failed. instead.
bf just get struck waiting for something after it prints b'ninja: build stopped: subcommand failed.', basically at this point ninja is already finished, but bf is waiting for something.

Guess it's fixed.


Swift Language Version” (SWIFT_VERSION) is required to be configured correctly for targets

I'm following each and every instruction that is mentioned for installing the cordova iosrtc plugin still I'm facing the below error after the cordova build ios.
“Swift Language Version” (SWIFT_VERSION) is required to be configured correctly for targets which use Swift. Use the [Edit > Convert > To Current Swift Syntax…] menu to choose a Swift version or use the Build Settings editor to configure the build setting directly.
I have already seen the questions that were raised in the forum regarding the same, but none of those solutions worked for me.
Below is some information about my system:
iosrtc plugin:  cordova-plugin-iosrtc 4.0.2 "iosrtc"
ionic version: 2.2.2
cordova version: 6.5.0
Xcode version: 8.3
Please help me out, thanks in advance.

@coolvasanth can you link to the forum post.

No I have not received any solution yet, which is badly affecting my product delivery ...

Have you tried setting it in the Xcode project by hand? Go on the build settings option and set "Swift Language Version" to 3.

Ya tried that but getting 103 errors. Image is attached for better clarity.

You seem to be missing the bridging header.

Hi I have added even bridging header but no help.

Does it find the bridging header though? Is it the same error?
If you follow the instructions here https://github.com/BasqueVoIPMafia/cordova-plugin-iosrtc/blob/master/docs/Building.md to the letter is should work. Otherwise I honestly don't know what the problem could be :-S

Hi @saghul ya it was the Bridging header mistake.I'm using this plugin in my ionic project. If anyone else using ionic project and facing the same issue then below is the solution.

First goto project/tsconfig.json and set target as es5 ("target": "es5").

Then open the project in Xcode. You will find it in project/platform/ios/project name.xcodeproj or project name.xcworkspace

In Xcode navigate to projectname/other resources/Bridging-Header.h

Add below line there:
// cordova-plugin-iosrtc
#import "cordova-plugin-iosrtc-Bridging-Header.h"

Goto project- Build settings - build settings option and set "Swift Language Version" to 3. (as @saghul  suggested above.)

Now clean the project (command + shift + k is the shortcut)

Now build the project (command + B is the shortcut).

Whola everything works like a charm now


Support subject identifier types

See http://openid.net/specs/openid-connect-core-1_0.html#SubjectIDTypes

Implemented by https://github.com/tekul/broch/commit/f6569e9c1707dad7ac905dee3f21eab32f4afad8


Does the Azure Cognitive Services Speech SDK allow for Pronunciation assessment?

I only see it in the REST API...which works but hoping to have more options. Am I missing it somewhere in the SDK? Any help would be appreciated.

Thanks for contacting us. You're unfortunately not missing anything--the SDK and the service results it uses don't yet have the pronunciation information that the REST API's endpoint provides. It's a great feature request for us to prioritize. I've added it to the team's work backlog with a backtrace to this issue and we'll update here once there's progress or new information.

Closing the issue since a backlog work item has been created. Once the work is done, we will update this to let you know what release the feature will be in.

@swashtek The pronunciation assessment feature was released with SDK 1.14.0. Check our how-to for quick starting!


Offline issues

I am having some problems with that the logging gets stuck after I go offline for some time (and then back on ofc).
Is there any known work-around for this? Otherwise I can look into it.

Hey @hirro, what sort of behaviour do you see? Are you seeing logs being dropped? What I think is happening for you you is that when you go offline we start filling up the in memory queue, when this queue is full we actually still add to it but remove the last item in the queue. That could explain any beahviour which would see your logs dropped or out of order.

Hi there! I have sent a PR with some comments. I tested it successfully on my device.