Fix crash on Flutter Web

Could you show us how to use the sdk with flutter web ?

@WahdanZ it's basically straightforward with some caveats
flutter channel master
flutter upgrade --force
flutter run -d chrome

LiveQuery not working currently (I'm working on it)
And for CoreStore I'm using stub (WIP as well)
SecurityContext is unsupported, throws on constructor, just don't use it
Also be aware, that it's doesn't support dart:io and will throw even on Platform.isAndroid
You can still use it like this:
if (identical(0, 0.0) {
  // I'm Flutter Web
  // Use 'dart:html' package
} else if (Platform.isAndroid) {
  // Use 'dart:io'
}

As I already said, it's WIP, I will provide more PR's for flutter web soon, just part by part, not all at once


whack -a-mole game

I want to add a survey form please assigned it to me and label it under hacktoberfest.

the issue has been assigned to you


Where is the implementation of tags of Ionic2

I want to use Ionic 2 tags other than ionic 2 framework itself, I need to find actual implementation of Ionic 2 tags

The Ionic 2 tags are just Angular 2 components. If you want to implement such tags yourself then I would recommend you to find/read more information about creating Angular 2 components. If you want to take a look at the source of the Ionic 2 components it's here: github.com/driftyco/ionic/tree/2.0/ionic/components

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum (forum.ionicframework.com). Thanks for using Ionic!


[fix][broker] Fix unloadNamespaceBundlesGracefully can be stuck with extensible load manager  (#23349)

(cherry picked from commit e91574ac7b44348a05f1ae812c5aae3abb26fe64)
I need to export some of the changes in https://github.com/apache/pulsar/pull/23301 to resolve conflicts.
fork-test: https://github.com/heesung-sn/pulsar/pull/83

I added this change in this PR, https://github.com/apache/pulsar/pull/23496/commits/3b9e857633d06dd0300ebbc302dac50651cdc8e4
Regarding ClusterMigrationTest.testClusterMigrationWithReplicationBacklog failure, the root cause is that after the broker is disabled, the assignment operation couldn't complete, and its lookup timed out. However, this test passes in the master branch now.


html-pdf: Evaluation - ReferenceError: Can't find variable: Set

After running for months without problem we suddenly receive the following error message. Locally and on the live servers.
Server running on http://<IP_ADDRESS>:8080
Error: html-pdf: Unknown Error
html-pdf: Evaluation - ReferenceError: Can't find variable: Set
Stack:
  at https://static.cloudflareinsights.com/beacon.min.js
  at https://static.cloudflareinsights.com/beacon.min.js
  at https://static.cloudflareinsights.com/beacon.min.js
  at https://static.cloudflareinsights.com/beacon.min.js
  at https://static.cloudflareinsights.com/beacon.min.js

    at ChildProcess.respond (/pdf-printer/node_modules/html-pdf/lib/pdf.js:134:17)
    at ChildProcess.emit (events.js:223:5)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)

Is something known about this?
We had Version 2.2.0 running - tried to update to Version 3.0.1 but the same problem occurs.

I just found out that the Cloudflare Browser Insights is the Problem. Disabling this option will make the pdf generator work again. I guess they have an issue in their js file.

html-pdf: Received the exit code '1'\nhtml-pdf: Evaluation - ReferenceError: Can't find variable: $\nStack:\n  at undefined
I am facing this issue while i am trying to create a pdf using pdf.create method from html-pdf library


ERR TIMEOUT on any BACnet - Write

Which node-red-contrib-bacnet version are you using?
0.3.0
What happened?
Sending a simple request to a bacnet-write node
{"deviceIPAddress":"<IP_ADDRESS>","objectId":{"type":1,"instance":2},"values":[{"type":4,"value":98}]}
Results in the value updating in the device, however no response comes from the node, only "Error: ERR_TIMEOUT".
How can this be reproduced?
Using a bacnet-write node, send a write request to it.
What did you expect to happen?
Get some kind of response to ensure value update succeeded
Other Information
I have also tried to do a multiple write with the same result, however it does not seem to even update the value
{"deviceIPAddress":"<IP_ADDRESS>","values":[{"objectId":{"type":56,"instance":6},"values":[{"property":{"id":400},"value":[{"type":6,"value":[10,1,18,111]}],"priority":8},{"property":{"id":401},"value":[{"type":6,"value":[10,1,18,1]}],"priority":8},{"property":{"id":411},"value":[{"type":6,"value":[255,255,255,0]}],"priority":8}]},{"objectId":{"type":8,"instance":100100},"values":[{"property":{"id":77},"value":[{"type":7,"value":"Fl0W-Red5 Room","encoding":0}],"priority":8}]}]}
in this example I am only trying to change the value of property 77 (name) on type 8 (dev), however I get a timeout and the update seems to not work as no value is updated on the device.

Hi @biancode, any update on this ? :slightly_smiling_face:

This is still bugged. I can send individual writes via function nodes into BACnet write and it will update the value but will always output a timeout error.


Add a Gitter chat badge to README.md

Nbsaw/miaohu now has a Chat Room on Gitter
@Nbsaw has just created a chat room. You can visit it here: https://gitter.im/nbsaw/miaohu.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 14.77% when pulling 4f61bc5edc221e39a948ed6e28e9014595a0f07e on gitter-badger:gitter-badge into b589e83ff60ae475ce8ca0d7ae933646dfa53da3 on Nbsaw:master.


Document Postgres Surfacer Table Structure

Unless I'm missing it, there is no documentation for how the table must look like for the Postgres surfacer.

We should definitely document it. For a quick reference though, postges surfacer supports two schemas:

You dump all the labels in JSON format in a single column.
You pre-define a label to column mapping through surfacer's labelToColumn field and create schema according to that.

Here is what the schema will look like for the first option:
CREATE TABLE metrics (
    time timestamp,
    metric_name varchar(80),
    value float8,
    labels jsonb
);

Added some documentation to the config itself:
https://cloudprober.org/docs/config/surfacer/#cloudprober_surfacer_postgres_SurfacerConf


Getting access to redux dispatch in reduxForm connected component

Hey there,
I am just wondering if there is a reason that redux-form doesn't pass dispatch to the component reduxForm wraps.
For example:
var CreateCutsite = reduxForm({
  form: 'customEnzymes',
  initialValues: {
    name: 'Example Enzyme',
  },
  validate
})(MyComponent)

Why does MyComponent not have access to redux's dispatch method?
It seems like I'll need to go through a second wrapping in order to get dispatch:
var CreateCutsite = reduxForm({
  form: 'customEnzymes',
  initialValues: {
    name: 'Example Enzyme',
  },
  validate
})(connect()(MyComponent))

Wouldn't this be easier if reduxForm just passed dispatch along with its other props?
Thanks!
Thomas

Hmm upon further inspection, it looks as if the dispatch is being "eaten" by the  component.
I am passing dispatch in to my fields component like this:
<Fields names={[
    'sequence',
  'name',]} {...props} component={MyComp}

where dispatch is one of the properties on the props object
But dispatch never makes it to the props of MyComp.
I think that it must be getting stripped by the Fields component. Can you verify that this is indeed the case?
Thanks!

2+ years without a comment here, so I'm closing!

Same issue here dispatch prop is being eaten up by the <Field/> component.

@FatehAK @tnrich You can access dispatch using meta.dispatch
https://github.com/redux-form/redux-form/blob/ccebdcd0270edd699b9c381a65fefe742fdadc90/src/createFieldProps.js#L143


2-9-2021

This one passed in the comments:
Rodneyjet
<EMAIL_ADDRESS><IP_ADDRESS>
reduslim trovaprezzi
Rodneyjet

Thanks for the submission.
It looks like the phrase “trovaprezzi” is Italian for “find prices”, so I’ll definitely add that to the blocklist. The other elements are a bit too generic to do anything with.

Thanks again for these.
I have added new terms per commit 9e08c7d


add Socket::try_receive_message to avoid getting stuck for longer per…

…iod of time.
I need this to allow shutting down of the thread handling the socket. If no messages from other peers appear, the thread gets stuck in receive.

What's the policy regarding the oldest supported version of Rust? This patch uses TcpStream peek() function, which appeared in 1.18, but CI fails since it uses 1.14.
Is there some other way how to shutdown thread from Listener::start cleanly?

Unfortunately... socket.rs will be deprecated per https://github.com/rust-bitcoin/rust-bitcoin/issues/155

Thank you @dongcarl, understood.
I suppose there's no recommendation of what should be used instead of the connection functionality in the future yet - it's either not written or up to the user of library - correct?

@ks I don't think it's written up yet... Perhaps you could open up an issue about it and the maintainers will comment on the right approach (perhaps connection functionality will be a separate crate?)

@ks the last nontrivial commit to socket.rs was 99a4845719b71c9c14615450d7cac4443af0f11f "get library building on stable" in September 2015 three years ago. This was before mio, before futures, when we were still reeling from the thread::scoped is unsound shock, and it's a testament to the Rust developers' commitment to forward-compatibility that it even compiles today.
This was also over a year after I personally stopped using the socket stuff in rust-bitcoin; it never had proper test infrastructure; as far as I know it's been entirely unused ever since.
The Bitcoin protocol is pretty simple; you should be able to write your own socket management using net2 and then just send the various Bitcoin p2p network messages as raw bytes over those sockets.


Retrieve the URLs of all availale thumbnails

The clips on Tumblr pages are usually accompanied by more than one thumbnail. Their URL is stored in an array in  the source code of the page.
This patch decodes this array in its entirety and returns their URL under the thumbnails key.
It also returns the first URL under the thumbnail key.

Sorry, but the multiple thumbnails do not seem to be present anymore, at least not in our tumblr test page.


NetNewsWire opens to a read article.

This is probably related to IOS 13 (and even IOS13,4 beta5) and not to the app itself because I had the same problem with another RSS reader app, but maybe new IOS versions demanding different handling. IDK, I’m not developer, but this issue is annoying.

Hey 👋
Can you specify a bit more what exactly is going on? Is NNW opening for you when you click on a feed-url?
Also can you mention exact iOS version you are using? iOS13.4 b5 is correct?

Hello, I’m using IOS 13.4 beta 5, but the reason why I entered beta program is because I faced similar problems but with other apps including IOS email app on IOS 13.3.
Scenario:
I open NetNewsWire and start browsing feeds(I’m syncing with Feedly). I go back from article list to a feed list and close NetNewsWire. After some time, I open again NetNewsWire and it opens not in a feed list bit it opens the article I already read. This scenario happens about 20% to 30% times I open this app.
I had same problem with FieryFeeds app and as I mentioned at the beginning of my message, I had similar problem with email app and exchange account on previous versions of IOS 13.3 and all 13.4 betas before latest which seems that fixed email issue.
Thanks for fast reply.
Yeah, thanks for a such amazing app. This little gem is pure haiku: small, fast and beautiful:-)

@chalxee Do you force quit your apps by sliding them off the top of the screen?

Yes.

And phone model is iphone 7

It is a common misconception that force quitting saves on phone resources, but it isn't true.  It actually takes iOS more resources to completely quit the app than to leave it in the background.  I know that it looks like the app is still running when you switch to another application, but it isn't.  iOS suspends the app so that it isn't using up resources.  Completely stopping and starting the app is more resource intensive than just letting it suspend into the background.
I think what is happening in your case is that the app, be it Fiery Feeds or NetNewsWire, isn't having a chance to get the last place it was saved when you force quit the app.
I could be wrong and it could be related to you running the latest Beta.  If you are willing to help us out, could you try not force quitting your apps and then see if this problem still exists?  If it keeps happening, I will try to start debugging with the iOS Beta to see if I can recreate the bug.

Yes, I’ll be glad but it would need at least ad day or two to be certain. I didn’t think of that possibility to be frank, but when you mentioned it it looks possible. I will report here my findings.

We really appreciate it!

I’ll be glad. Already testing and it behaves predictably. It opens at the exact same place where it was when I suspended it.

I can confirm that suspending app works good. If I sispend app in feed list it always opens in feed list On next Start. If I suspend it in article lost it starts next time in this same article list. If I suspend it in specific article, it always opens in this same article.

Thanks @chalxee!  I'm glad to hear everything is working out for you now.


perf: TCP_CORK / better buffering pre-write

As part of the recent nginx comparison profiling:
Like most proxies Envoy uses TCP_NODELAY for all connection handling. A few improvements can be made here to reduce the number of small packet writes.

Allow for no delay to be optional during TCP proxy. During high throughput scenarios there is no reason to use no delay.
Utilize TCP_CORK and better pre-buffering in both HTTP/1.1 and HTTP/2 codecs. There are several flows where we may write several times that could be collapsed. Though modern kernels do have some level of auto corking would still be better to be explicit.

Fixed by https://github.com/lyft/envoy/pull/208 and https://github.com/lyft/envoy/pull/183. With these changes we don't need cork.


fix: Avatar changes when from offline to online

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/28518#issuecomment-1759769523
PROPOSAL: https://github.com/Expensify/App/issues/28518#issuecomment-1759769523
Tests

Launch app
Tap on profile---Preferences
Toggle on forced offline
Tab FAB and start chat
Input email with default avatar and tab on create chat
Look at the avatar in the report
Tap on profile---Preferences
Toggle off forced offline
Go back to the chat, look at the avatar and make sure it will not change.

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

Offline tests

Same as QA steps
QA Steps

Launch app
Tap on profile---Preferences
Toggle on forced offline
Tab FAB and start chat
Input email with default avatar and tab on create chat
Look at the avatar in the report
Tap on profile---Preferences
Toggle off forced offline
Go back to the chat, look at the avatar and make sure it will not change.

[x] 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: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb 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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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 grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is 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 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 code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[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.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/9712610/cc8cfd4b-d97e-433f-9f64-93844b5aed38

Android: mWeb Chrome

https://github.com/Expensify/App/assets/9712610/33e2d682-296f-424c-8407-640203f30aa1

iOS: Native

https://github.com/Expensify/App/assets/9712610/65477cb5-a078-4fc0-a38b-f30219e4adcc

iOS: mWeb Safari

https://github.com/Expensify/App/assets/9712610/a970752c-7a86-4157-8bb2-389525e8a9fb

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/9712610/7879fad1-2eaa-4c5d-8c4b-1dce3e67ad4a

MacOS: Desktop

https://github.com/Expensify/App/assets/9712610/f2ea2bc8-726f-4947-b6dd-9f271d92f511

I have read the CLA Document and I hereby sign the CLA

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] 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

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

[ ] 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

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

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

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] 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)
[ ] 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.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] 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.
[ ] 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.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@anyongjitiger Thanks for the PR! Just a couple of initial things:

Please make sure you follow the PR template instructions carefully - you have linked to your proposal rather than the issue itself next to the $.
We require all commits to be signed, see here in the contributing guide.


🚀 [Feature Request] - add hovering effect on "get in touch" section of the page  Title

Enhancement
I want to enhance the get in touch section, when a user brings the cursor, a hover effect should come to make it more responsive.

I want to add hover effect for each box of get in touch
Kindly assign me this issue under GSSOC'23
Describe briefly how you would implement this enhancement.
Some steps to implement this enhancement-
1. Step 1
2. Step 2
3. Step 3

Additional context or screenshots
Additional context

Are you contributing for a program?
GSSoC2023
Code of Conduct

[X] I have read and agree to the Contributing Guidelines of this project.

Please assign me this issue so that can I can work on it


Listview - Xamarin Forms UWP LoadingPlaceholder fails to show image

I have a Listview referencing a custom view cell with an FFCachedImage.
I have the image in the root of my UWP project and am referencing it from PCL like this:

img.LoadingPlaceholder = "myimage.png";
img.ErrorPlaceholder = "myimage.png";

The ErrorPlaceholder  DOES however show if the image is broken.
Any idea what the difference would be between Loading/Error? Also on iOS the Loading image does immediately show.

Do you set loading placeholder before or after setting source? I'll need more info.

Hey Daniel
Below is the exact code I'm using. So it's being sent on List Item binding. Loading/Error set in constructor.
    public partial class GenericListItemTemplate : ViewCell
    {
        public GenericListItemTemplate()
        {
            InitializeComponent();

            if (Device.OS == TargetPlatform.iOS || Device.OS == TargetPlatform.Android)
            {
                 img.Transformations = new List<ITransformation>()
                {
                    new CircleTransformation(14, "#547799")
                };
            }

            img.LoadingPriority = FFImageLoading.Work.LoadingPriority.Highest;

            var imagePath = "myimage.png";

            // https://forums.xamarin.com/discussion/62120/how-do-i-acces-icons-image-resources-in-uwp-apps
            // http://stackoverflow.com/questions/38026687/how-to-use-images-in-xamarin-forms-uwp

            img.LoadingPlaceholder = imagePath;
            img.ErrorPlaceholder = imagePath;
        }

        protected override void OnBindingContextChanged()
        {
            var item = BindingContext as GenericListItem;

            if (item == null)
                return;

            img.Source = item.ImageURL;

            base.OnBindingContextChanged();
        }
    }

I don't see anything wrong here but I still can't reproduce it.

Please reopen if you have more info or working reproduction. Thanks :)


connect chaos-delegate

Did I understand correctly - with "litmusctl connect chaos-delegate" command you can connect another cluster (delegated)?
In my case it creates resources, but hangs in a "pending" state.
Delegated subscriber tries to connect to the SVC IP address from another cluster (not via litmus ENDPOINT / API)
msg="Failed to confirm cluster" data= error="Post \"http://<IP_ADDRESS>:9002/query\": dial tcp <IP_ADDRESS>:9002: i/o timeout"

@Subetov can you try this with the latest releases? It had been resolved by using the ServerEndPoint in the config


sokol_gl applying uniforms

I managed to draw some 2d texture using sokol_gl default pipeline, but now I want to test the exact same thing by creating my own pipeline. What I don't understand is how do you pass uniforms in sokol_gl?

Ah that's not possible unfortunately. The sg_pipeline_desc structs passed to sgl_make_pipeline() must only contain "fixed function state" (e.g. blend, depth-stencil and rasterizer-state). Even though it's possible to provide your own shader (as some sort of undocumented feature), this shader must accept sokol-gl's uniform layout, and there's currently no way to pass uniforms into sokol-gl.
For such "advanced features" it would probably make sense to add a "custom draw callback" to sokol-gl, which would be recorded into the internal command buffer and called during sgl_draw().

Thank you for the clarifications @floooh. I could live with that for now, but it's a chance we could see support for custom shaders in the future or do you think it would add too much complexity? I see the infrastructure is already there from sokol_gfx and the sgl_draw function already does that to some extent for draw commands:

    sgl_draw():
        - copy all recorded vertex data into the dynamic sokol-gfx buffer
          via a call to sg_update_buffer()
        - for each recorded command:
            - if it's a viewport command, call sg_apply_viewport()
            - if it's a scissor-rect command, call sg_apply_scissor_rect()
            - if it's a draw command:
                - depending on what has changed since the last draw command,
                  call sg_apply_pipeline(), sg_apply_bindings() and
                  sg_apply_uniforms()
                - finally call sg_draw()

sokol_gl.h implements a subset of the OpenGLES 1.x feature set useful for
when you just want to quickly render a bunch of colored triangles or
lines without having to mess with buffers and
shaders.

Nevermind, I think I will go back to sokol_gfx then :)

Nevermind, I think I will go back to sokol_gfx then :)

Out of interest, what did you want to use sokol_gl.h for? So far I had it mostly in mind for the occasional debug visualization or UI rendering, but didn't think it would be useful for more general 3D rendering stuff. Adding full "custom pipeline + shader" support might be possible, but it would require quite some re-architecturing of the internals and the API, so I'm a bit hesitant going in that direction :)

For a 2D game :) For prototyping I decided to still use sokol_gl.


Require matplotlib v3.5+.

scanpy requires matplotlib.colormaps, which is not in 3.4.
Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)

@conda-forge-admin, please rerender

just kidding this needs to be not on an org account


[MM-25165] Fix search bar disappearing

Summary

The search bar disappears when you search for a non-existent team or channel on system console (since the data returned is 0). This PR aims to fix this problem.
Ticket Link

https://mattermost.atlassian.net/browse/MM-25165
Screenshots

NOTE Github makes it seem there was way more code changes than there actually was. I simply removed {this.searchBar()} from the header() function and added it to render with a styled div tag. Nothing else was changed.

@srkgupta Good catch. I didn't see that. I have fixed and you can re-test. Thanks 😄

@ali-farooq0 Was this a regression on master or an old issue?

/cherry-pick 5.24

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
!!! 'upstream/5.24' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

/cherry-pick release-5.24


Read this

@mikevb1 Alright, my Destiny account was banned for cheesing and as such it's probably a good idea that this project gets discontinued.

False issue, my account was actually banned for Borderlands 2 modding.


v1.1.1: Add zenodo DOI throughout

After the V1.1.0 release we should have a DOI for the repo. This should be added to the README and the docs.

Closed by #762.


Update airbnb.swiftformat

Summary
Add the following rules:

wrapMultilineStatementBraces
wrapConditionalBodies
wrapEnumCases
wrapSwitchCases

Reasoning

Without the wrapMultilineStatementBraces rule, I feel it makes it hard to see where the conditions / function parameters end and where the action starts. Also, this rule is formulated in AirBnB's style guide, but it seems to be missing from their .swiftformat file.
See this example, where it is hard to see where the if conditions end and where the action starts:

with the current setting

with the --wrapMultilineStatementBraces rule

The other 3 rules (wrapConditionalBodies, wrapEnumCases, wrapSwitchCases) enforce what we already have in our guidelines (see the examples in the doc linked above for each one of them), so I feel it would be a good idea to include them so we don't have to think about it.

Please react with 👍/👎 if you agree or disagree with this proposal.

I don't see wrapEnumCases and wrapSwitchCases currently in our guidelines so these are brand new rules.

I thought there were PRs where that came up. I guess I was mistaken, I can remove them.

If everyone agrees to these two new rules then that is ok to keep, I just
want to make sure it's what people want to change to since it is something
new.

Don't forget to open a PR to the upstream branch at for the wrapMultilineStatementBraces rule


SSH passthrough into Docker failed: Permission denied (publickey).

Description
I tried https://docs.gitea.io/en-us/install-with-docker/#ssh-shell-with-authorizedkeyscommand and https://docs.gitea.io/en-us/install-with-docker/#docker-shell-with-authorizedkeyscommand
both end ma back
ssh -v -i "H:\Dropbox\...Josef.openssh"<EMAIL_ADDRESS>OpenSSH_for_Windows_8.1p1, LibreSSL 3.0.2
debug1: Reading configuration data C:\\Users\\josef/.ssh/config
debug1: Connecting to git.xxxx.biz [xxx.xxx.xxx.xxx] port 22.
debug1: Connection established.
debug1: identity file H:\\Dropbox\\....Josef.openssh type -1
debug1: identity file H:\\Dropbox\\...Josef.openssh-cert type -1
debug1: Local version string SSH-2.0-OpenSSH_for_Windows_8.1
debug1: Remote protocol version 2.0, remote software version OpenSSH_8.2p1 Ubuntu-4ubuntu0.4
debug1: match: OpenSSH_8.2p1 Ubuntu-4ubuntu0.4 pat OpenSSH* compat 0x04000000
debug1: Authenticating to git.xxxx.biz:22 as 'git'
debug1: SSH2_MSG_KEXINIT sent
debug1: SSH2_MSG_KEXINIT received
debug1: kex: algorithm: curve25519-sha256
debug1: kex: host key algorithm: ecdsa-sha2-nistp256
debug1: kex: server->client cipher<EMAIL_ADDRESS>MAC: <implicit> compression: none
debug1: kex: client->server cipher<EMAIL_ADDRESS>MAC: <implicit> compression: none
debug1: expecting SSH2_MSG_KEX_ECDH_REPLY
debug1: Server host key: ecdsa-sha2-nistp256 SHA256:JEDLiiKEi7sIjMyLNhHzeTBzUK8R0vpev9H8CqOlBys
debug1: Host 'git.xxxx.biz' is known and matches the ECDSA host key.
debug1: Found key in C:\\Users\\josef/.ssh/known_hosts:5
debug1: rekey out after 134217728 blocks
debug1: SSH2_MSG_NEWKEYS sent
debug1: expecting SSH2_MSG_NEWKEYS
debug1: SSH2_MSG_NEWKEYS received
debug1: rekey in after 134217728 blocks
debug1: pubkey_prepare: ssh_get_authentication_socket: No such file or directory
debug1: Will attempt key: H:\\Dropbox\\....Josef.openssh  explicit
debug1: SSH2_MSG_EXT_INFO received
debug1: kex_input_ext_info: server-sig-algs=<ssh-ed25519,sk-ssh-ed25519<EMAIL_ADDRESS>debug1: SSH2_MSG_SERVICE_ACCEPT received
debug1: Authentications that can continue: publickey
debug1: Next authentication method: publickey
debug1: Trying private key: H:\\Dropbox\\...Josef.openssh
debug1: Authentications that can continue: publickey
debug1: No more authentication methods to try.
<EMAIL_ADDRESS>Permission denied (publickey).

/usr/bin/ssh -p 2222 -o StrictHostKeyChecking=no<EMAIL_ADDRESS>/usr/local/bin/gitea keys -e git -u git -t ssh-rsa -k AAAAB3NzaC1y...
by Hand response  to: Gitea: Invalid repository path
 /usr/bin/docker exec -i -u git gitea /usr/local/bin/gitea keys -e git -u git -t ssh-rsa -k AAAAB.......
gives:
# gitea public key
command="/usr/local/bin/gitea --config=/data/gitea/conf/app.ini serv key-6",no-port-forwarding,no-X11-forwarding,no-agent-forwarding,no-pty,no-user-rc,restrict ssh-rsa AAAAB3NzaC1.....Josef.key

but I get always<EMAIL_ADDRESS>Permission denied (publickey).´ via external ssh /git clone
Gitea Version
1.16.5
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.30.x
Operating System
docker
How are you running Gitea?
via docker
Database
MySQL

duplicated with #17798


290: add VolunteerSkills model and associations; modify volunteer con…

…troller to create skills when passed

So this should handle both cases when creating a volunteer:

ID for existing skill is passed (we wouldn't need other info at this point) -> create a reference
new skill is passed -> create new skill and add reference to this volunteer.

Next Steps

frontend would need to get all skills in the DB to show. Does that route exist properly?
is there a need for edit volunteer to add a skill / remove a skill?
is this create-volunteer-with-skill robust enough?

Thanks Brian, this looks great. To answer some of your questions:

Yes, the route to get all skills exists. It hasn't been implemented in the FE yet
I don't think there is a need to edit / remove skills at this point. We may need to revisit it down the road, but right now it's kind of a one-and-done thing.
For our uses right now, create-volunteer-with-skill should be perfect
Good point! I can add a story for adding constraints / validations to the backlog.


instruction says "defaultConnection" not AZURE_SQL_CONNECTIONSTRING

In step 2.2, the instructions says to look for defaultConnection (and thumbnail shows "defaultConnection", but the name of the automatically set connection string has changed it seems, to AZURE_SQL_CONNECITONSTRING? It's confusing

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

ID: df55b676-8ff7-1470-3525-d0edfd406af6
Version Independent ID: ac264c26-7196-870b-760f-94a97ae437e8
Content: Deploy an ASP.NET Core and Azure SQL Database app to Azure App Service - Azure App Service
Content Source: articles/app-service/tutorial-dotnetcore-sqldb-app.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

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

@kmlinda I tested this for myself and yes, the name is indeed what you shared. I understand how this can be confusing.
Unfortunately, I do not have the ability to edit pictures in docs so I will need to assign this to the doc author to review further.

This is fixed. #please-close


add click functionality

completed in https://github.com/xHN35RQ/atom-notelink/commit/f53c24a7c6ab534ec83e2512961799730c71f92d


npm error 422 Unprocessable Entity

desc
npm error 422 Unprocessable Entity
exp
done
act
npm error 422 Unprocessable Entity - PUT https://registry.npmjs.org/@yors%2Fsophia-win32-x64-msvc - Error verifying sigstore provenance bundle: Failed to validate repository information: package.json: "repository.url" is "git+https://github.com/ymc-github/sophia.git", expected to match "https://github.com/YMC-GitHub/sophia" from provenance

why
npm config set provenance true
fix
npm config set provenance false
more
do you need provenance ?


Enable Type-specific Methods

This would require having type-specific signatures. I think a single Ty<T, P, R> is preferable:

T: native TS type
P: keyof any – the placeheld keys
R: boolean – whether the type can be used as the root (only objects for now)


[Question/Bug] Requesting reconciliation on changes to the included GitRepository artifact

Hey! I'd like to ask for your help in understanding the Kustomize Controller behaviour regarding requesting reconciliation on GitRepository artifact changes. Sorry for the long post, I wanted to be possibly explicit in what I want, and what I have checked so far.
I am running the Kustomize Controller v0.35.1, and the background is as follow. It reconciles some Kustomization CR, say X, in long intervals, say 24h. This Kustomization CR references the GitRepository, say A, which includes another GitRepository, say B. Now, when I push a commit to the repository the A GitRepository points to, the Source Controller rebuilds the artifact, hence its revision and checksum both change, and the reconciliation of the X Kustomization CR is triggered. This is what I want, for I do not want to wait till the next schedule. When I do the same for the repository the B GitRepository points to, the Source Controller rebuilds artifacts for both repositorys, but the revision of the A GitRepository stays the same, only its checksum changes, which is normal for nothing has actually changed in the A repository. But in this scenario, the reconciliation of the X Kustomization CR is not triggered. Is this correct?
I also tried to look for an answer by myself, and if you could tell me if I understand the behaviour correctly, I would be thankful. So, my understanding is, the Kustomization CRs get reconciled in response to artifact changes thanks to the Kustomize Controller watching the GitRepository CRs related events, as configured here. The predicate is used as a watching option to subscribe to only certain changes, for which the transformation function is then used to build a list of reconciliation requests for Kustomization CRs. The predicate in the v0.35.1 version checks for revision only, but for the sake of discussion let's assume it also checks for checksums. When the transformation function kicks in, it gets the list of Kustomization CRs, which I suspect are now indexed in the cached client by the GitRepository CRs' names and namespaces, judging on the indexBy function, but this is the part I am the most unsure of, for I am not deep into the manager / client codebase. What led me to believe this is, I've seen only certain Kustomization CRs being reconciled on artifact changes, and not all of them, so I guess the Kustomize Controller must know about the relations between them somehow. And now, after getting the list, for each item on it the last attempted revision known to the Kustomization CR is compared to the revision from the GitRepository CR, and if both are the same, the reconciliation request for this Kustomization CR is not added to the list. In the light of this part of the transformation function, indexing Kustomization CRs by GitRepository CRs' names and namespaces makes sense, for the last attempted revision in Kustomization CR status is strictly related to the referenced repository, and hence, in my case, getting the X Kustomization in this function upon updates to artifact of repository B, would always result in reconciliation, for the both revisions would always be different, but this isn't something we want, I think.
If that was the case then the consequence would be, the reconciliation of Kustomization CRs would be requested only on changes to artifact revision of directly referenced GitRepository CRs, and changes to indirectly referenced CRs, via include feature, would not trigger any updates. Also, if that was the case, checksum would feel a better indicator of detecting the change, that would also enable picking up the included repository changes. But even if predicates accounted for this, like noted earlier, then it still wouldn't work for the checksum is not part of the Kustomization CR status, so couldn't be compared in the transformation function.
But I am not sure I get the code correctly. So, first of all, I would like to ask if the code I link above is responsible for the behaviour in question, or maybe something else is at play here as well, that I missed. Second of all, do I understand it correctly, at least to some general degree? If I am wrong in my understanding, can you hint me of what should I do in order to trigger reconciliation of Kustomization CR on changes to the included repository?

Yes this is a known limitation of Git includes, maybe we could add some build metadata +checksum to the revision, like we do for charts? cc @hiddeco
To properly solve this, we could introduce a source Bundle API e.g. https://github.com/fluxcd/source-controller/issues/854#issuecomment-1226152895

Do not think we can because of indirect implications this has on the notification-controller and commit status updates :-).
In general, it would be better to just move forward with the Source bundle API, and deprecate the includes in favor of them. As this both serves more use-cases, and removes the awkwardness of one GitRepository being more of a first class citizen than another.

I get your point, having the Source Bundle API to solve this sounds also better to me. It is a shame though, because I guess it will probably take some time to implement it, right? Could we maybe for the time being add a note to the docs about this limitation of included repositories? I mean, I am not sure how obvious it is for other people, I wouldn't probably realize it myself if I didn't increase the intervals, but even after noticing something is wrong, it wasn't immediately obvious how this thing works, hence the whole question.
Anyway, thank you for answering, I think I have found a workaround to this limitation, which I need to test, so I'm fine with closing this issue, should you not need it around for potential docs update.


Update G-Cloud 10 lot descriptions

@karlchillmaid has tightened up the direct award lot descriptions as part of restructuring the lot selection page for direct award searches.
The changes in full can also be seen in a Google doc.

All changes look great :)


Errors can be declared globally and route per route

Fixes #127
Please see the golden file for OpenAPI changes :)

Per @ccoVeille suggestions why not just add:
https://golangci-lint.run/usage/linters/#usestdlibvars
to our golangci-lint config?

Per @ccoVeille suggestions why not just add:
https://golangci-lint.run/usage/linters/#usestdlibvars
to our golangci-lint config?

I'm afraid usestdlibvars won't detect this.
https://golangci-lint.run/usage/linters/#usestdlibvars
Also, there is way to configure it yet to tell him this int in this method is an http.Status
But, adding the linter is indeed a great thing.

Per @ccoVeille suggestions why not just add:
https://golangci-lint.run/usage/linters/#usestdlibvars
to our golangci-lint config?

I'm afraid usestdlibvars won't detect this.
https://golangci-lint.run/usage/linters/#usestdlibvars
Also, there is way to configure it yet to tell him this int in this method is an http.Status
But, adding the linter is indeed a great thing.

Oh I see, it only works in the context of the stdlib. It's actually already enabled for this project btw.

Exactly, it would have been reported otherwise.
One way to detect them in the context of the methods this project use is to set up forbiddigo rules.
It would work


multibyte string crashes foundry-compilers

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.2.0 (6fc7463 2024-01-05T00:21:51.712802723Z)
What command(s) is the bug in?
forge script
Operating System
Linux
Describe the bug
script had this string in the code: \u250c\u2500\u2015\u2015┄┈\n"
 forge script script/Example.s.sol --broadcast --private-key $PK --slow --rpc-url $RPC
[⠢] Compiling...
[⠆] Compiling 3 files with 0.8.20The application panicked (crashed).
Message:  byte index 47 is not a char boundary; it is inside '┄' (bytes 46..49) of `57 |                 "\u250c\u2500\u2015\u2015┄┈\n"`
Location: /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/foundry-compilers-0.1.2/src/artifacts/mod.rs:2084

This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry

Edge case - mitigated by properly escaping all unicode chars.

Interesting, cc @matthias.seitz any ideas here?

this has been mitigated by https://github.com/foundry-rs/compilers/pull/40
and now correctly displays the solc error:
Error:
Compiler run failed:
Error (8936): Invalid character in string. If you are trying to use Unicode characters, use a unicode"..." string literal.
  --> src/Counter.sol:12:28:
   |
12 |         string memory  s = "\u250c\u2500\u2015\u2015┄┈\n";


default traceback handler not applied

The default traceback handler is only added when constructing sol::state with a custom allocator function state.hpp#L75, but not without state.hpp#L68.
This means you won't get a traceback in protected functions when constructing your sol state like so:

sol::state lua;

Thanks for reporting. Fixed: pull from latest.


Add lease RBAC for leader election

/test pull-knative-net-kourier-integration-tests

@mattmoor I backed out the update of deps since it seems to cause TestKourierControlHA to degrade.

/approve

/test pull-knative-net-kourier-integration-tests

/retest

/retest

/close
I think there may be some unresolved issues here about the HA test. I'll let @jmprusi / @davidor / @mgencur fix and will try again later.


16x2 text getting pushed to the right by random char/empty spaces appearing on left of screen.

My hd44780 lcd works otherwise perfectly but while playing every few hours or so the canvas? shifts 2 blocks to the right and it keeps happening and after hours/days it will move all the way to the right and the space on the left will be a mix of random? chars and empty space.
I usually have volumio playing just 1 webradio station.
Switching to a different station or something totally different like mp3 songs does not affect the screen but when i power the raspberry down the "exiting..." text appears on the correct position in the top left and all random chars and empty spaces disappear.
After a reboot everything works fine again until slowly the random stuff pushed the test off the screen to the right again.
pic of the screen: https://drive.google.com/open?id=1fStIRT2dSe-wkvUoeuJEEzKHbrKe79nS
My system details: raspberry pi 4 with hifiberry dac+. adafruit hd44780, official rpi 4 powersupply.
volumio 2.699

These type of errors generally relate to timing issues with the display.  You can try adjusting the pulse timing setting to something higher to see if that has any impact.  If it doesn't, I would suggest trying a different HD44780 display.

If i understand correctly that would be "self.delayMicroseconds(1) # 1 microsecond pause - enable pulse must be > 450ns" on line 330 in hd44780.py ? (i am using 4bit mode)
And how does it work with docker ? if i increase the delay, save and reboot it works in docker ?
I'm sorry for being such a noob lol, this is the first time i'm messing with linux.

Yes.  That's correct.  Normally that value would be adjustable in the pydPiper.cfg file but I saw that in the current version of the HD44780.py driver that it is not reading from the config file to determine what the pulse time should be.  You should try a longer value like 10 instead of the current value of 1.
Note:  Make sure that your ExecStart line includes the -v clause needed to reference the local copy of pydPiper, not the one that is inside the docker image itself.  It should look like this...
ExecStart = /usr/bin/docker run --network=host --privileged -v /var/log:/var/log:rw -v /home/volumio/pydPiper:/app:rw dhrone/pydpiper:v0.31-alpha python /app/pydPiper.py

So far it seems to be working, thanks for the help :)


[QUERY] Using RequestContext with Azure.Data.Tables

Library name and version
Azure.Data.Tables v12.5.0
Query/Question
This sits somewhere between a feature request and a query, I think.
My use case is that I'd like to disable the automatic-throw-on-error-status behavior of the TableClient. The Azure.Core documentation implies that I should be able to do exactly this by attaching a RequestContext object to a request, but I can't seem to find anything on TableClient or TableServiceClient that would accept one.
Am I missing something, or is this just a hole in the implementation?
If RequestContext is indeed not something that can be used with Azure.Data.Tables, is there another way to disable the auto-throw behavior? I don't really like using exceptions to handle basic HTTP failure states.
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @pingzing
This feature is forthcoming. Currently it only applies to purely code generated clients but will be coming to all clients in the next couple of months.
duplicate of #25626


Extend FTorch to cover 5-dimensional tensors.

Closes #172
We have had a request for 5D tensor support.
This is from the ConvLSTM: https://paperswithcode.com/method/convlstm which is a type of NN used to analyse gridded data.
Given our initial application (climate sciences) it would make sense to support this as standard.
I have extended the ranks argument in the fypp code to now support 5-D (rank 5) tensors.
One point to discuss is whether we want to support slightly higher in anticipation of other applications (Rank 6 or perhaps 7) bearing in mind that doing so adds quite a lot of code/functions in the f90 file.
We should also perhaps add documentation noting the limit in Rank, and perhaps describing how to extend it for specific cases.

As discussed over in #166 The tests for autograd were producing an error after this change, despite not touching them.
This could be fixed locally on my machine by checking for (.not. associated(out_data)) rather than the shape of unassociated/unallocated out_data.
However, the code is still failing the CI test here.
My guess would be that this is perhaps due to a regex mismatch that is being dealt with over in #166
So question is whether to:

merge as is here
turn autograd test off for now until fixed in #166 as not currently an active feature for users

I have just removed the commit that worked around the pointer issues that will be addressed by #175
This should now be merged after:

#175
#166

Rebased after #175 merged to main.
@TomMelt's changes to regex matching in autograd test as part of #175 mean that #166 is no longer a blocker so this is ready for review.


CI: Continue BSP building workflow if build of tier >1 BSP fails

Summary
Potential fix for https://github.com/atsamd-rs/atsamd/issues/744 - this change to CI has the "Build BSPs" workflow continue regardless of tier >1 BSP builds failing.
One downside to this approach is that the overall workflow run may show success despite there actually being BSP build failures.  This PR is a test case to see how obvious this situation is in practice - a tier 2 BSP is expected to fail.

The required statuses above come from the project's branch protection settings, so this change is really just about whether the CI jobs are run or not.

I wouldn't say that a Tier 2 is necessarily expected to fail. As discussed on Matrix, Tier 2 BSPs are locked to a specific crates.io HAL version, so they normally should succeed when nothing has been changed (unless a dependency breaks semver, like is the case for the itsybitsy_m0.
I don't know how much complexity this would add, but just a suggestion to get around this:
Would it be feasible to only run CI on BSPs if:

It is a Tier 1 BSP
If it's a Tier 2 BSP, run CI only if the BSP had some files changed in the PR

I wouldn't say that a Tier 2 is necessarily expected to fail

Sorry, that was poor wording on my part, meant that in this specific PR the itsybitsy m0 is expected to fail.

Would it be feasible to only run CI on BSPs if...

As external issues arise, with this change we'll know about them but they won't interfere with work.  To me, this seems a bit better than letting issues silently accumulate, or relying on users to report them.
That said, I think that would be possible, and likely worth doing if we start running in to issues with time/cost of building all the BSPs.


The "show all" button does not work

In the caniuse website, the "show all" button does not work, click it take no effect, please fix it.
example page: https://caniuse.com/#search=display

You can toggle between "Apply filters" and "Show all", where "Show all" is usually already enabled as default. What is the behavior you're expecting to see?

Sorry, I made a mistake.


Enhance minimal distance algorithm in gardener-scheduler

How to categorize this PR?

/area usability
/area cost
/area performance
/kind enhancement
What this PR does / why we need it:
This PR enhances the Gardener Scheduler to perform scheduling based on actual/configured distances.
Co-authored-by: Rafael Franzke<EMAIL_ADDRESS>Which issue(s) this PR fixes:
Fixes #7383
Special notes for your reviewer:
Release note:

Gardener Scheduler's Minimal Distance strategy can take scheduling decisions based on region distances configured by operators. This especially improves the allocation for shoots of providers regions for which the standard Levenshtein distance is inappropriate. Please see `docs/concepts/scheduler.md` for more information.

/assign

/assign


Restore custom printer possibility for Circe usage

Circe equivalent issue of https://github.com/hseeberger/akka-http-json/issues/135

Fixed by #137.


$.html() always tries to set HTML.

I just ran into an issue, where calling $(...).html() would set the innerHTML to undefined.
My version of Cash:
<EMAIL_ADDRESS>~/W/BIRD3 $ cat bower_components/cash/bower.json | grep version
    "version": "1.0.0",

if (content === "undefined") { is improper. Add the typeof keyword like so:
if (typeof content == "undefined") {.

Thanks!

Still not fixed?

Still not fixed?

@novns Did you check?

@novns Did you check?

Yes, I caught this bug myself and found this issue.
As you can see the improper comparison is not fixed as it was suggested earlier:
https://github.com/kenwheeler/cash/blob/master/dist/cash.js#L933

I'm not sure what you mean by that. It was reported that:

if (content === "undefined") is improper

And we are instead doing:

if (html === undefined)

Which is equivalent to (assuming the html variable is actually defined):

if (typeof content == "undefined").

In fact I can't reproduce the issue you're talking about. What version of cash are you using?

In fact I can't reproduce the issue you're talking about. What version of cash are you using?

I use the latest cash.min.js v1.0.0 available here

And the comparison is still content === "undefined" there.

That's so old switching to v4.1.5 will require significant efforts.
@kenwheeler I really think this page should redirect to this repo.

Thanks, I will try the new version.


Bm/create header

@Doralyp @aceburgess @AlexTaber , Install / Format header, Jquery to sign out on link, format sign up page

Won't merge, but thanks!


Move the documentation related to ./x.py from https://forge.rust-lang.org

As per discussion https://github.com/rust-lang-nursery/rustc-guide/issues/84#issuecomment-382978810 create a new documentation for ./x.py.
Copy the contents from link https://forge.rust-lang.org/x-py.html into the new file.

It would be good to systematically move over relevant stuff from there. A PR on this repo should probably be accompanied by a PR on https://github.com/rust-lang-nursery/rust-forge

@rajcspsg Did you want to work on this?

@mark-i-m - my pleasure :)
I've to copy the x-py.md correct?

I think that would be a great start. Some of the things on the forge page are already in the rustc-guide (https://rust-lang-nursery.github.io/rustc-guide/how-to-build-and-run.html), so we should try to avoid duplicating content.  I think we can add the content from the forge page to the existing chapter in the guide. (TBH, I haven't read the forge page, but let me know if you get stuck :) )

Nice! I'll probably come around to this next week. Thanks :)

@mark-i-m we can close this ticket correct?

Yep! Thanks :)


Composer adjustments

Is the pull request based off the lastest version?
Yes
What features have you added?
Small adjustments on composer.json
What bugs did you fix?
Installing this library on an environment without IMAP PHP extension.
Is your code valid PSR-2?
Not applicable.
Has anything in your pull request already been fixed?
No.
Anything else?
No.
Optional things below
[] Added your name to the credits
[] Add a update name below:
Rubens Takiguti Ribeiro

Thank you! Looks good to me and no conflicts so I will go ahead and merge!


Uncaught TypeError: Cannot read property 'location' of undefined

ncaught TypeError: Cannot read property 'location' of undefined
at new Router (main.969d8.js:8215)
at main.969d8.js:19965
at measureLifeCyclePerf (main.969d8.js:19745)
at ReactCompositeComponentWrapper._constructComponentWithoutOwner (main.969d8.js:19964)
at ReactCompositeComponentWrapper._constructComponent (main.969d8.js:19950)
at ReactCompositeComponentWrapper.mountComponent (main.969d8.js:19858)
at Object.mountComponent (main.969d8.js:3171)
at ReactCompositeComponentWrapper.performInitialMount (main.969d8.js:20041)
at ReactCompositeComponentWrapper.mountComponent (main.969d8.js:19928)
at Object.mountComponent (main.969d8.js:3171)
"react-router": "^4.1.0",

+1

+1

+1

+1

+1

+1

+1

+1

Im getting this error

+1

I've found some solution, that works for me. I think, router module is damaged, so I've removed whole node modules folder, and install again with other versions:
"react": "^15.5.4",
"react-router": "^3.0.2",
I hope it will be helpful for you to.

Worked to downgrade! Thanks. At least I can continue.

Downgrade worked for me too. thanks

+1

@GolkowyBenQu Thanks this fixed it!


feat: adding shape support for gather ONNX operation

Pull Request Template
Checklist

[ ] Confirmed that run-checks all script has been executed.
[ ] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
#2115
Changes
Summarize the problem being addressed and your solution.
Testing
Describe how these changes have been tested.

I've resolved the user's issue. There's a section of code that checks if the rank of the indices tensor is greater than 1, and if so, it triggers a panic. This check seems unnecessary since the underlying gather operation in Burn supports multi-dimensional tensors (though I might be mistaken). The user's model uses a multi-dimensional indices tensor, causing a crash before completing the parsing of the Gather node. If I remove this check, the operation graph progresses further. Should I remove the indices_dim check?

I've resolved the user's issue. There's a section of code that checks if the rank of the indices tensor is greater than 1, and if so, it triggers a panic. This check seems unnecessary since the underlying gather operation in Burn supports multi-dimensional tensors (though I might be mistaken). The user's model uses a multi-dimensional indices tensor, causing a crash before completing the parsing of the Gather node. If I remove this check, the operation graph progresses further. Should I remove the indices_dim check?

If you check the code, the ONNX Gather op actually translates to tensor.select(dim, indices). There is another ONNX op called GatherElements which corresponds to burn's gather operation on a tensor.
So the rank 1 tensor limitation is because select only accepts 1d indices.
Linking the original gather PR #1843  for reference.

@mepatrick73 this is ready for review right? The changes look good to me w.r.t supporting shape input.

Looks good to me so far

When Gathering from a Shape, shouldn't the output also be an ArgType::Shape instead of converting it to a Tensor?

The rank of the output tensor could be bigger than 1, therefore Shape is not appropriate.

@mepatrick73 this is ready for review right? The changes look good to me w.r.t supporting shape input.

Yes it is ready for review :)


fix(db push): make sure to stop migrate engine when it errors

Related
https://github.com/prisma/prisma/issues/7795
https://github.com/prisma/prisma/pull/7990
Thanks @millsp for finding the test that uncovered this 👍🏼

If you can't reproduce this locally, let me know.

Ok I'll merge main later and see what this is about

That's the danger of the timeouts I think - they cancel the job, and do not fail it by default.

Yes... but better that it times out after 10 minutes than 6 hours by default 😄


'RNPaypal.h' file not found

My AppDelegate.m file is here:
Get this error when I try to build:
AppDelegate.m:4:9: 'RNPaypal.h' file not found

// Copyright 2015-present 650 Industries. All rights reserved.

#import "AppDelegate.h"
#import "RNPaypal.h"

@implementation AppDelegate

// Put your app delegate methods here. Remember to also call methods from EXStandaloneAppDelegate superclass
// in order to keep Expo working. See example below.

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
  [[RNPaypal sharedInstance] configure];
  return [super application:application didFinishLaunchingWithOptions:launchOptions];
}

- (BOOL)application:(UIApplication *)application openURL:(NSURL *)url
  sourceApplication:(NSString *)sourceApplication annotation:(id)annotation
{
  return [[RNPaypal sharedInstance] application:application openURL:url sourceApplication:sourceApplication annotation:annotation];
}
@end

Mi archivo AppDelegate.m está aquí:
Recibe este error cuando intento construir:
AppDelegate.m:4:9: 'RNPaypal.h' file not found

// Copyright 2015-present 650 Industries. All rights reserved.

#import "AppDelegate.h"
#import "RNPaypal.h"

@implementation AppDelegate

// Put your app delegate methods here. Remember to also call methods from EXStandaloneAppDelegate superclass
// in order to keep Expo working. See example below.

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
  [[RNPaypal sharedInstance] configure];
  return [super application:application didFinishLaunchingWithOptions:launchOptions];
}

- (BOOL)application:(UIApplication *)application openURL:(NSURL *)url
  sourceApplication:(NSString *)sourceApplication annotation:(id)annotation
{
  return [[RNPaypal sharedInstance] application:application openURL:url sourceApplication:sourceApplication annotation:annotation];
}
@end

My AppDelegate.m file is here:
Get this error when I try to build:
AppDelegate.m:4:9: 'RNPaypal.h' file not found

// Copyright 2015-present 650 Industries. All rights reserved.

#import "AppDelegate.h"
#import "RNPaypal.h"

@implementation AppDelegate

// Put your app delegate methods here. Remember to also call methods from EXStandaloneAppDelegate superclass
// in order to keep Expo working. See example below.

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
  [[RNPaypal sharedInstance] configure];
  return [super application:application didFinishLaunchingWithOptions:launchOptions];
}

- (BOOL)application:(UIApplication *)application openURL:(NSURL *)url
  sourceApplication:(NSString *)sourceApplication annotation:(id)annotation
{
  return [[RNPaypal sharedInstance] application:application openURL:url sourceApplication:sourceApplication annotation:annotation];
}
@end

I have the same problem, did you manage to solve it?

Never solved, I think this package is broken.

Do you know any alternative?

Nope, but I do believe Paypal has a web checkout flow that you can tap into. Haven't gotten around to working on it.

Would you implement it with a WebView or how would you do it?

Yeah, open up the webview flow and do it from there.

Looking around again, I found this package: https://github.com/taessina/react-native-paypal-wrapper
If you implement it can you let me know if it works?

I am using react-native-paypal-lib on android but it has errors in IOS
https://github.com/zeddz92/react-native-paypal

Hi all! Please try add
$(SRCROOT)/../node_modules/react-native-paypal/ios

to
target > Build Settings > search header > find Header Search Paths.
Or see picture

Hi guys, sorry for the late reply.
I think we might have some issue in the project.pbxproj config file, as I recall having to do something similar as the latest comment suggested to have react-native-paypal build correctly in our project.
I think that's the issue, and I was wondering if the workaround suggested by @ptdede is working for anyone else?

Thank you @ptdede, it worked for me.

Closed by #13


[popup] Display current layout information

Is your feature request related to a problem? Please describe.
It would be great to have the current layout information on a record page (name + Link )
Describe the solution you'd like

Idea by Camille Guillory

Request is partially develop in legacy version:
Image from the article:
https://www.salesforceben.com/salesforce-inspector/


HDDS-4152. Archive container logs for kubernetes check

What changes were proposed in this pull request?
In the last few days kubernetes check has been failing very frequently.  This change let's it save logs from all containers to make it easier to find any problems.  (docker-compose-based acceptance check also saves container logs.)
https://issues.apache.org/jira/browse/HDDS-4152
How was this patch tested?
$ unzip -t kubernetes.zip
Archive:  kubernetes.zip
    testing: getting-started.xml      OK
    testing: getting-started/         OK
    testing: log.html                 OK
    testing: minikube.xml             OK
    testing: minikube/                OK
    testing: ozone-dev.xml            OK
    testing: ozone-dev/               OK
    testing: ozone.xml                OK
    testing: ozone/                   OK
    testing: report.html              OK
    testing: summary.html             OK
    testing: getting-started/pod-datanode-0.log   OK
    testing: getting-started/pod-datanode-1.log   OK
    testing: getting-started/pod-datanode-2.log   OK
    testing: getting-started/pod-om-0.log   OK
    testing: getting-started/pod-s3g-0.log   OK
    testing: getting-started/pod-scm-0.log   OK
    testing: minikube/pod-datanode-0.log   OK
    testing: minikube/pod-datanode-1.log   OK
    testing: minikube/pod-datanode-2.log   OK
    testing: minikube/pod-om-0.log    OK
    testing: minikube/pod-s3g-0.log   OK
    testing: minikube/pod-scm-0.log   OK
    testing: ozone-dev/pod-datanode-0.log   OK
    testing: ozone-dev/pod-datanode-1.log   OK
    testing: ozone-dev/pod-datanode-2.log   OK
    testing: ozone-dev/pod-jaeger-0.log   OK
    testing: ozone-dev/pod-om-0.log   OK
    testing: ozone-dev/pod-prometheus-584b7948d9-8fcb6.log   OK
    testing: ozone-dev/pod-s3g-0.log   OK
    testing: ozone-dev/pod-scm-0.log   OK
    testing: ozone/pod-datanode-0.log   OK
    testing: ozone/pod-datanode-1.log   OK
    testing: ozone/pod-datanode-2.log   OK
    testing: ozone/pod-om-0.log       OK
    testing: ozone/pod-s3g-0.log      OK
    testing: ozone/pod-scm-0.log      OK

https://github.com/adoroszlai/hadoop-ozone/runs/1031423764

Thanks @elek for reviewing and committing it.


Data-Sweet-formula is not throwing "this.viz is not defined" error with kibana 6.6.2

Hi,
I am working with kibana 6.6.1. I have to install data-sweet-formula plugin manually to work on this version of kibana. Unfortunately, after successful installation kibana is throwing "this.viz is not defined" error.
Can you please look into this issue.
Regards,
Sujata

Hi,
For now, the plugin is not compatible with kibana version >= 6.5.0

Hi,
please give it a try to the last release (support for kibana v6.5.x to 6.7.x).
Regards


Xcode 9 warnings fix

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation or CocoaDocs
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
11.0

CocoaLumberjack Version
3.3.0

Integration Method
cocoapods

Xcode Version
Xcode 9.0.1

Repro rate
all the time (100%)

Issue Description and Steps
There is one warning in Xcode 9
Pods/CocoaLumberjack/Classes/DDOSLogger.m:19:8: 'DDOSLogger' is partial: introduced in iOS 10.0
Pods/CocoaLumberjack/Classes/DDOSLogger.m:19:20: Annotate 'sharedInstance' with an availability attribute to silence
It will be nice to fix it.

@ivulyov90
Thanks for your opening issue.
This issue should be fixed by https://github.com/CocoaLumberjack/CocoaLumberjack/pull/895 that includes some other fixes. See this commit to be more precise.
As you see, https://github.com/CocoaLumberjack/CocoaLumberjack/pull/895 has been already merged to master branch, but not released yet. I think it probably will come together with CocoaLumberjack 3.4.0.

Hey, any update on when 3.4.0 will be released? This warning has been nagging me for some time 😛


Merge master into gh-pages

Done, the live demo at http://apinf.org/open-api-designer/ is now updated


Increase GstDiscoverer timeout to make it more robust

When I try to open an RTSP stream it fails around 80 % of the times, because GstDiscoverer doesn't succeed getting stream info within the timeout. It even fails sometimes when opening a video file that is stored on the SD card in the Jetson Nano.
I increased the timeout from 1 to 5 seconds, which has resulted in 100 % success rate in my experiments, without increasing the discovery time when it works.

Thanks @magnus-eigenvision, I was unaware of this setting.  Cheers and thanks for the good work you have been doing!


Merge node configurations for bootstrap and client

Overview / Motivation
Bootstrap and "normal" client nodes have the same parameters in tests. currently, they have separate args sections which might cause tests failing due to misconfigurations.
The Task
TODO: Clearly describe the issue requirements here...
Implementation Notes
TODO: Add links to relevant resources, specs, related issues, etc...
Contribution Guidelines
Important: Issue assignment to developers will be by the order of their application and proficiency level according to the tasks complexity. We will not assign tasks to developers who have'nt introduced themselves on our Gitter dev channel

Introduce yourself on go-spacemesh dev chat channel - ask our team any question you may have about this task
Fork branch develop to your own repo and work in your repo
You must document all methods, enums and types with godoc comments
You must write go unit tests for all types and methods when submitting a component, and integration tests if you submit a feature
When ready for code review, submit a PR from your repo back to branch develop
Attach relevant issue to PR

this is regards our k8s on gcp


Pass through Jupyter settings

Pass Jupyter settings through via kwargs. Need to check whether "default_url" exists within kwargs, if it does make it clobber the scriptedforms default url.

Don't do this. Just do it on a case by case basis as needed.


Fix blade directive

This syntax seems to be broken on Laravel 9.47 and up:
<div>
    <span @class([
        'lt-text-xs lt-inline-block lt-py-1 lt-px-2 lt-rounded',
        'lt-text-green-600 lt-bg-green-50' => !@empty($success),
-        'lt-text-yellow-600 lt-bg-yellow-50' => !@empty(
-            $warning
-        ),
        'lt-text-red-600 lt-bg-red-50' => !@empty($danger),
    ])>
        {{ $value }}
    </span>
</div>

In my project, the error is thrown when opening the products page in the hub.
Looks like this issue in Laravel was introduced here: https://github.com/laravel/framework/pull/45490
Setting the laravel/framework version to 9.46.0 fixes the issue in my project.

Looks like this was fixed in main with https://github.com/lunarphp/lunar/pull/801

Closing as this should be fixed in 0.2


bug(mat-list-option): There's no visual selected state on mobile browsers

Reproduction
https://stackblitz.com/angular/aexyvybjrlm?file=src%2Fapp%2Flist-single-selection-example.html (taken from your "List with single selection" example: https://material.angular.io/components/list/examples)
Steps to reproduce:

Open the StackBlitz link above on a mobile browser (Safari on iOS, or Chrome on Android)
Click the items

Expected Behavior
The selected item should have a visual selected state, like on desktop.
Desktop screenshot (macOS, Chrome):

Actual Behavior
There's no visual selected state.
Mobile screenshot (iPhone, iOS, Safari):

Environment

Angular: 10.0.0
CDK/Material: 10.0.1
Browser(s): Safari
Operating System (e.g. Windows, macOS, Ubuntu): iOS

Please fix.
PS: Any workaround we can use while this gets fixed and released?

For anyone wondering, I'm using the following as a temporary workaround:
@media (hover: none) {
  .mat-list-single-selected-option {
    &:not(.mat-list-item-disabled):hover {
      background: rgba(0, 0, 0, 0.12);
    }
  }
}


db not defined

where the hell did you even define DB?

The latest commit (eb7b8bb) fixes your issue. I simply forgot to require quick.db, apologies.


update the "Configuring Apache Airflow on Kubernetes for use with Elyra" docs

Currently, we describe how to install airflow on Kubernetes in the docs, but they describe using the old 7.X.X version of the Airflow Helm Chart (User Community), which only supports Airflow 1.10.X.
We should update the docs to reference the new 8.X.X versions of the chart, and thus for Airflow 2.X.X.
PS: Here is the CHANGELOG for the Airflow Helm Chart (User Community).

See related issue involving support for airflow 2.0 : https://github.com/elyra-ai/elyra/issues/2124


What do you mean by same mel-spectogram representation?

Will this work if:

I train the tacotron2 model here using the settings in the repo.
Use the pretrained weights from the waveglow repo?

How can I verify the settings for the mel-spectogram are the same?

The defaults here are the same as the defaults in waveglow repo.
You're looking at these params: https://github.com/NVIDIA/tacotron2/blob/master/hparams.py#L35

ok awesome. So just the sampling rate has to be the same? Or do you mean the whole audio section?
Btw I generated the following sample (i only trained  it for 20 epochs) i assume this has to go for something closer to 500 epochs?
https://www.dropbox.com/sh/jlrfrucpd1riisj/AAB5WbLNTVn3bm5N86JgR4Lva?dl=0

        sampling_rate=22050,
        filter_length=1024,
        hop_length=256,
        win_length=1024,
        n_mel_channels=80,
        mel_fmin=0.0,
        mel_fmax=8000.0,

For tacotron 2 with the current setup and mel representation a validation loss of ~3 and below sounds like speech for sure.


chore(connlib): pass to client new fields

Fixes #4885

@conectado Was this tested with all the clients to make sure adding the fields doesn't cause serialization errors? Or was that not implemented yet?

@conectado Was this tested with all the clients to make sure adding the fields doesn't cause serialization errors on the other side of the FFI? Or was that not implemented yet?

This PR only added fields to the messages received from the portal. No changes to FFI.


new functions

hi zhw2590582, I have added the following functionalities to your project :
1- some styling changes.
2- you can now change the end and start of the subtitle from the subtitles box as well as the time meter at the bottom.
3- you have a right menu option to copy subtitle text directly to the clipboard.
4- you can select multiple subtitles and do functions on them.
5- if you select one or more subtitles the vedio will loop from the start of the first select sub until the finish of the last sub, and will never pass until you choose that, so this will give you more power to work on a single sub.
6- dynamically choose the shifting offset.
7-  more uploading option like from a url or Google drive.
if you interested in adding those functions please tell me so I can make a pull request with the changes.
cheers
Ahmad

Thank you very much for your suggestions. At present, the code of the two components of the editor is messy Metronome.js and Block.js. I need time to optimize them, but the other components are easy to modify.
1-I want to know which elements need to be modified
2-I do n’t think it ’s common to adjust the start time and end time of the subtitles on the right, because usually people do n’t know the exact time, so I forbid to modify the time directly, but drag and drop the slider directly through the bottom will be more convenient and intuitive.
3-Right-click copy function sounds good
4-Multi-select subtitles are also good, but the code changes will be great, I do n’t know if it ’s worth it
5-This suggestion is awesome and can save a lot of editing time
6-I don't quite understand this
7-The editor itself supports getting subtitles and videos from url, but it is not perfect, so I temporarily disabled it, or I can enable it now


wdi5 ts init adds wrong ui5 type devDependency

Describe the bug
The easy ui5 generator includes a sub-generator that handles the wdi5 integration into the generated app. This mostly consists of npm init wdi5 -- --ts. Init wdi5 seems to hard coded introduce a devDependency to the latest version of @openui5/types. But in the context of the easy ui5 generator the user can select between sapui5 and openui5 and also which specific version to be used.
See also my initial issue for the ui5 ts app generator: https://github.com/ui5-community/generator-ui5-ts-app/issues/33
I think this is caused here: https://github.com/ui5-community/create-wdi5/blob/main/src%2Findex.ts#L20 so maybe this is the wrong repo for this issue 🙈
To Reproduce
yo easy-ui5 -> select generator ts-app and follow the instructions. select sapui5 (e.g. 1.120.11) and have a look at the package.json afterwards.
Expected behavior
Depending on the selection in the easy ui5 generator either @openui5/types or @sapui5/types should be included as devDependency in the version entered in the respective generator step.

Hi @heimwege
Yes, this is the wrong repo - it should be moved to the create-wdi5 repository. But the issue is valid. In case of the UI5 generators added the types already, they must not be added additionally. This should be checked in the generator steps.

Thanks for the confirmation @petermuessig I created a new issue in the wdi5-create repo. Let's close this one.


Code comments

Bad code comments
	if (x) { //If x is true then continue on
        do $zu(45,$zu(27,$ZCVT(x,"UTF8"),a,t+1))
        write "Sucsess" //tell the user we did it
        return x+1 //End the function and give them the increment
 	}

So what could we say about it? Beyond the standard "comment your code reasonably"

At least, that


light.lifx_effect_pulse and light.lifx_effect_colorloop not working with Light Groups

Home Assistant release with the issue: 0.69.1

Last working Home Assistant release (if known): Not sure... it wasn't working with 0.68 either.
Operating environment (Hass.io/Docker/Windows/etc.): Hassbian

Component/platform: light.lifx_effect_pulse and light.lifx_effect_colorloop

Description of problem:
Executing light.lifx_effect_pulse with the following service data fails:
{
  "entity_id": "light.hall_lights", "color_name": "pink", "mode":"breathe", "brightness":255, "period":0.5, "cycles":2, "power_on":true
}

However this succeeds (using individual entities from the light group above):
{
  "entity_id": "light.hall_1, light.hall_2, light.hall_3, light.hall_4, light.hall", "color_name": "pink", "mode":"breathe", "brightness":255, "period":0.5, "cycles":2, "power_on":true
}

light.lifx_effect_colorloop failed here:
{
  "entity_id": "light.bathroom_lights", "brightness": 180, "period":5, "spread":216
}

But worked here:
{
  "entity_id": "light.bathroom_door, light.bathroom_mirror, light.bathroom_shower", "brightness": 180, "period":5, "spread":216
}

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
light:

  - platform: lifx
    broadcast: <IP_ADDRESS>

  - platform: group
    name: Hall Lights
    entities:
      - light.hall
      - light.hall_1
      - light.hall_2
      - light.hall_3
      - light.hall_4

  - platform: group
    name: Bathroom Lights
    entities:
      - light.bathroom_door
      - light.bathroom_mirror
      - light.bathroom_shower

Traceback (if applicable):

Additional information:

I think this works as expected, the lifx_ prefix of the service call name means that it only works for the LIFX platform.
Maybe @OttoWinter can comment?

@amelchio Exactly 👍 platform-specific service calls will not be supported by the light group platform.

Ohhh okay. Now I understand. But funny though seeing as if you put it in a normal group it would work.
I think the documentation on the group/light group page should more explicit about the functionality you gain/lose.
That is, just because you can put lights in a Light Group, you shouldn’t necessarily do it.

On 10 Jun 2018, at 14:44, Otto Winter<EMAIL_ADDRESS>wrote:
@amelchio Exactly 👍 platform-specific service calls will not be supported by the light group platform.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I will close this issue since the light group works as intended.


Reference implementation

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Do you have a reference implementation for java web risk?  Do you have examples of a simple system which has implemented the local database?  Do I set up rpc server to manage the local database?  What do you recommend for a basic service helper, and channel provider?
I would like to see a java project which implements looking up a url, database support (inserts, updates, deletes) and also submitting a suspicious url to Google using java web risk, version 1.
I do not have experience with google tools, so a reference implementation would be a big help.
Thank you.

Do you have a timeframe for a reference project?  If not, is there any advice on how to write a system to use java web-risk?  I'd be happy to do the coding and submit it to you, but I do need some help.

@lwilsojn thanks for reaching out. I'm not up to speed on the webrisk API but there are likely some good pointers on in the product documentation. There is also a sample application in Go here.


[Hub Generated] Review request for Microsoft.Insights to add version stable/2017-04-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

@InbalZim Hi.  Please create new branch with this recommendation for faster review.

While adding new version the recommendation is to copy the existing version into new directory structure for first commit and then push new changes for easy review. Please create new branch with this recommendation for faster review.

Created a new PR as requested https://github.com/Azure/azure-rest-api-specs/pull/11579


[feature] Allow heater and cooler be the same entity  in heat_cool_mode

Hey!
I have a very specific request. I'm using a heatpump with floor heating which automatically switches between heating and cooling mode depending on outside temperature. The cooling mode is limited to round about 5K below outside temp and is secured by some dew point sensor.
So if it's cold, the room-thermostat has to be opened to let warm water inside the floor heating.
If it's warm, the same valve has to be opened to let cold water inside the floor "heating" and cool the room.
Reason for using this module: It shall open the very same valve/switch if it's colder as selected temperatur (+ some tolerance) or if it's hotter (+ some tolerance). Maybe with manual switch of the mode. Obviously I have to make sure, that my heatpump has switched it's mode from heating to cooling.
- platform: dual_smart_thermostat
  name: Wohnzimmer
  heater: switch.shelly1minig3_redacted-id_switch_0
  cooler: switch.shelly1minig3_redacted-id_switch_0
  target_sensor: sensor.wohnzimmer_hut_temperature
  heat_cool_mode: true # <-required
  initial_hvac_mode: "heat_cool"
  unique_id: wz_heatcool

With this configuration, the entity allows only heating.

This question has been asked many times. It's a tricky edge case as the thermostat component currently cannot determine if it's summer or winter.
My suggestion is to:

create an input boolean for winter mode that you will manually switch, or if you have a sensor for that, use that
create a template switch for heating using the above input_boolean/sensor + switch.shelly1minig3_redacted-id_switch_0
create a template switch for cooling using the above input_boolean/sensor + switch.shelly1minig3_redacted-id_switch_0

The switch you create in step 1 should reverse the logic of the switches you create in steps 2-3.


Create a New Repo - Splice

Please include:

GitHub repo to add: splice
Associated lab proposal: (include the original lab proposal) https://github.com/hyperledger-labs/hyperledger-labs.github.io/pull/261

Here you go!
Invites sent


Add explanation of Python Path Manager

This dialogue is not explained in the docs
http://pythonhosted.org/spyder/search.html?q=pythonpath
Especially the "synchronise" function would need more explanation.
BTW, it would be great to display the pythonpath paths that may have been defined from outside spyder within this Python Path Manager

@jnsebgosselin could you give us a hand with this one?

Yes, sure, I'll assign it to me.


.semver:minor Dockerfiles without node, kubernetes, helm and terrafor…

removes node, kubernetes, helm, terraform and mssql-cli from the Dockerfile
updates version of the user agent to 2.188.3
fixes configureAgent.sh to support AGENT_POOL environment variable with spaces (eg Vienna Linux)

Sorry, mistake; I did not want to make a pull request to this original Lambda3:master branch.


Posting Function

Implement a function that, gives the user a way to rate a gallery with stars and words
Design Template


Add a note for v1.15.0-alpha.3

Since v1.15.0-alpha.3 , the --allow-privileged is removed from kubelet options.
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.15.md#action-required-1
ACTION REQUIRED: The deprecated Kubelet flag --allow-privileged has been removed. Remove any use of --allow-privileged from your kubelet scripts or manifests.

thanks, welcome to join the ding talk group.


ColumnType for where clause

Hi, I am trying to set column type, which is used only for where clause. not for select.
Initially, I wrote it like this. but I encountered a type error on where clause.
export interface SomeTable {
  id: Generated<number>
  date: Date
  // Other properties...
}

const result = await db.selectFrom('SomeTable')
                         .selectAll()
                         .where('date', '=', '2024-01-01') // type error
                         .limit(1)
                         .executeTakeFirst();

So I changed date: Date to date: ColumnType<Date | string>.
It works on where clause, but the result type is Date | string.
Consequently, result.date.getTime() doesn't working this time.
In my timezone "Asia/Tokyo", new Date('2024-01-01') will be converted to 2024-01-01 09:00:00.
Therefore, I would like to search in string, but get the result in Date.
Do you have any ideas for this?
thanks.

It's best to just make the db return strings. JavaScript dates suck.
https://kysely.dev/docs/recipes/data-types


Make imports of numpyro, jax, ... optional

This issue has been solved for the moment. However, it may be better to import the modules when the inferer are set up, because here the most informative error messages can be provided. In addition, it will speed up pymob import. The downside is that the modules are not accessible via import.
I could also fix this with a silent try except block or something like this in the init PLUS an import attempt in the set_inferer method


Add migration integration test

Towards giantswarm/roadmap#30
Adds new migration integration test. Ensures that chartconfig CRs can be migrated to chart CRs without affecting the related helm release.
This is prep for removing the legacy chartconfig controller. Its only purpose is deleting chartconfig CRs as part of the migration.

I don't think test is controversial. Going with this.

Sorry, I just got busy with other things.

@tomahawk28 No worries. ❤
I got an error trying to add my comment due to the outage so I gave up.