linux, xsel: Primary paste in the tmux window doesn't work immediately

With set -g mouse on, when I highlight text tmux-yank copies it to the primary buffer using xsel. When I immediately try to paste this text in any pane (using shift+middle-click) in the same tmux instance it fails to paste. xsel -p prints the previously highlighted text - so the copy part is working, just paste isn't.
If I middle-click into any other window, the paste works. And from then on, the paste also works in tmux. This suggests to me that there is something blocking the shift-middle-click paste in tmux, until the terminal window loses focus. Copy/primary-paste continues to work for a short time after this, then it happens again (after ~2 mins).
This happens in fullscreen or windowed mode and seems to also apply to the clipboard buffer (when text is copied by pressing "y" instead of mouse-up).
I'm aware I can just bypass tmux by holding down Alt, but this doesn't work with panes as it selects a whole line across the display. I'd like to copy multi-line per pane.
I'm not sure how to debug the middle-click paste event in tmux. What debug options can I turn on to print more information about this? Thanks
tmux 2.9a, Fedora 32, gnome-terminal 3.34.2

I use mobaxterm on win10 locally ,ssh to  remote ubuntu 20.04, run in tmux.

Look forward to improvement.
Thx!


Update request: VSCODE 1.84.2→ 1.85.2

Package name:vscode
Latest released version:

Current version on the unstable channel:1.85.1
Current version on the stable/release channel:1.84.2

[ +] Checked the nixpkgs pull requests

Notify maintainers

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

@samuela @K900 Thank you very much!

done in https://github.com/NixOS/nixpkgs/pull/282178


 It was working on v 0.56.
My code:
    import * as Progress from 'react-native-progress';

    // in render()
    <Progress.CircleSnail size={80} color='#ffffff' indeterminate/>

update:
I checked on both Windows and Linux. I also checked with Android v7.0 & v8.0, nothing changed and the problem still exists. I took a look at the ReactNativeART and found nothing because both versions(0.56 & 0.57) are almost identical.

i got the same issue when upgrade RN 0.57.
in <Progress.Circle />. in ios work fine, but every android devices, just a black square.

Same issue here, very frustrating!

Just ran into this too.

I see two behaviours:

black background
blue color of the text shown - should be the pie color
transparent (or maybe white) color of the pie (so it's inivisible)

<Progress.Circle ...>
Using react 0.57.2 - this has worked nicely with 0.56

ReactNative: 0.57
React-Native-Progress: 3.5.0
Upon launching of the app I use <Progress.Circle >.  Since upgrading it just shows a black box.  Within my app I have a refresh button and upon tapping the button the <Progress.Circle > shows up but does not animate.
Hope this helps at all.
It did work with RNP 3.4 and RN 0.55

Same issue and it's weird, it's only happen the first time the component is <Progress.Circle/> is loaded !

Facing the same issue. Only happens when the component is loaded. After that it works fine. RN 0.57

Same issue. Do you have any workaround for this issue?

Yes,same issue. i use version RN 0.57.3 Circle And CircleSnail color Black.

Same issue

I found out the workaround. In my case, It works well as intended on Android devices.
I hope this helps.
constructor(props: Props) {
  super(props);
  this.state = {
    isVisible: false,
  };
}

componentDidMount() {
  requestAnimationFrame(() => {
    this.setState({
      isVisible: true,
    });
  });
}

render() {
  if (!this.state.isVisible) {
    return null;
  }

  return (
    <View style={styles.container}>
      <Progress.Circle
        size={52}
        progress={0.5}
        unfilledColor="#fff"
        color="#ff457f"
        thickness={2}
        borderWidth={0}
      >
      </Progress.Circle>
    </View>
  );
}

@ifsnow Is your workaround to set it to be invisible initially and then load it in?

@Slooowpoke Yes. I don't fully know why this solves the issue. but, It work well for me.

I found out the workaround. In my case, It works well as intended on Android devices.
I hope this helps.
constructor(props: Props) {
  super(props);
  this.state = {
    isVisible: false,
  };
}

componentDidMount() {
  requestAnimationFrame(() => {
    this.setState({
      isVisible: true,
    });
  });
}

render() {
  if (!this.state.isVisible) {
    return null;
  }

  return (
    <View style={styles.container}>
      <Progress.Circle
        size={52}
        progress={0.5}
        unfilledColor="#fff"
        color="#ff457f"
        thickness={2}
        borderWidth={0}
      >
      </Progress.Circle>
    </View>
  );
}

returning null didn't work, but I used an ActivityIndicator and it worked. Thank you!

I found out the workaround. In my case, It works well as intended on Android devices.
I hope this helps.
constructor(props: Props) {
  super(props);
  this.state = {
    isVisible: false,
  };
}

componentDidMount() {
  requestAnimationFrame(() => {
    this.setState({
      isVisible: true,
    });
  });
}

render() {
  if (!this.state.isVisible) {
    return null;
  }

  return (
    <View style={styles.container}>
      <Progress.Circle
        size={52}
        progress={0.5}
        unfilledColor="#fff"
        color="#ff457f"
        thickness={2}
        borderWidth={0}
      >
      </Progress.Circle>
    </View>
  );
}

I applied your solution, but the issue sometime appear for me, any idea?

same issue with RN 0.59.5.
<Progress.Circle /> display with the black square, but ios fine.

I'm using this with the latest Expo v32.0.0 which is using React Native 0.57 but don't seem to face this issue.

Any fix for this :(?

I can't fix it,  it is so weird, and all the solutions above didn't work. So i give up.
In my condition, i just wrote it with two views instead —— use borderWidth, borderRadius, AnimatedView.
It worked.

@x330650779 please share your solution

The issue came back :( any solution ?


Change class of FCM category slug to remove extra spacing

There's too much spacing between the FCM category slug and the FCM heading. In accordance with the updated markup recommendations in cfpb/capital-framework#320, this PR fixes that.
Changes

FCM category markup from p.h4 to div.category-slug
Some minor CHANGELOG cleanup

Testing

Pull branch
Reload a local page with an FCM

Review

@jimmynotjim
@anselmbradford
@KimberlyMunoz
@sebworks

Screenshots
See cfpb/capital-framework#320
Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

You'll want to update the CF dependency and re-shrinkwrap to get the margin changes you made in CF#320

This update doesn't actually require the update to CF, but if that gets merged and released before this, I can do that.

Ah ok, in that case 👍 from me

:shipit:


SVG Image does not support HorizontalOptions? (Android)

Hello,
When I try to set any HorizontalOptions on an SvgImage in a Xamarin.Forms project the images does not appear and then the application crashes.
I am setting HorizontalOptions to center the image within its view.
Exception
05-11 16:15:15.167 E/mono    (13129):  05-11 16:15:15.167 E/mono    (13129): Unhandled Exception: 05-11 16:15:15.167 E/mono    (13129): Java.Lang.IllegalArgumentException: width and height must be > 0 05-11 16:15:15.167 E/mono    (13129):   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Users/builder/data/lanes/2923/39a97ef2/source/mono/external/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143  05-11 16:15:15.167 E/mono    (13129):   at Java.Interop.JniEnvironment+StaticMethods.CallStaticObjectMethod (JniObjectReference type, Java.Interop.JniMethodInfo method, Java.Interop.JniArgumentValue* args) [0x00082] in /Users/builder/data/lanes/2923/7334f6a9/source/Java.Interop/src/Java.Interop/Java.Interop/JniEnvironment.g.cs:12649  05-11 16:15:15.167 E/mono    (13129):   at Java.Interop.JniPeerMembers+JniStaticMethods.InvokeObjectMethod (System.String encodedMember, Java.Interop.JniArgumentValue* parameters) [0x0001b] in /Users/builder/data/lanes/2923/7334f6a9/source/Java.Interop/src/Java.Interop/Java.Interop/JniPeerMembers.JniStaticMethods.cs:97  05-11 16:15:15.167 E/mono    (13129):   at Android.Graphics.Bitmap.CreateBitmap (Int32 width, Int32 height, Android.Graphics.Config config) [0x00054] in /Users/builder/data/lanes/2923/7334f6a9/source/monodroid/src/Mono.Android/platforms/android-23/src/generated/Android.Graphics.Bitmap.cs:670  05-11 16:15:15.167 E/mono    (13129):   at NGraphics.AndroidPlatform.CreateImageCanvas (Size size, Double scale, Boolean transparency) [0x00020] in /Users/fak/Dropbox/Projects/NGraphics/Platforms/NGraphics.Android/AndroidPlatform.cs:23

Nevermind, only happens if you don't set HeightRequest and WidthRequest.

Good catch. I would probably argue we should figure out a better way to convey that requirement, even if it is in the docs, and assuming it is even ultimately necessary. I would love for it to be able to render from the default space it is allocated during sizing passes.

Yeah ideally it would be optional.
Until then, perhaps an exception should be thrown.


Excellent and concise documentation

This is the best documentation I've ever seen, I don't know about you guys.

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

ID: 84b97c27-6026-d52a-6955-eb7696e3a204
Version Independent ID: ee8cecea-7ff2-bd7b-bea3-4c50da8860a3
Content: Migrator Class (Microsoft.EntityFrameworkCore.Migrations.Internal)
Content Source: dotnet/xml/Microsoft.EntityFrameworkCore.Migrations.Internal/Migrator.xml
Product: ef-core-api
GitHub Login: @dotnet-bot
Microsoft Alias: divega

lol, we're in the process of removing the all the .Internal types from the API docs. See the interface for (slightly) more substantial docs.

Should read, "Pay no attention to the man behind the curtain!"

To be fair, I was warned many times not to use these types, my code is full of green squiggles

I've opened the PR to push the changes live:
https://github.com/aspnet/EntityFramework.ApiDocs/pull/61
BTW, do you want to keep receiving the issues for API docs here or would you prefer to go to https://github.com/aspnet/EntityFramework.ApiDocs/?

@ajcvickers @bricelam - Should we get issues for API docs filed on product repo directly? Since that is where we need to make change anyway.

Cool. Thanks for confirming @smitpatel.
I'm closing this issue since the page no longer exists and I'll ask the engineering team to reindex the data for the API browser search: https://docs.microsoft.com/en-us/dotnet/api/?view=efcore-2.1.


[5.3] just sick of typing this :)

Looks fine, though, I'm concerned about performance if people call this, then call get again.

I don't see how this provides value enough to be part the framework code. Should we really encourage people to use the helpers for this trivial stuff? Couldn't it just be added to a local helper file in @alnutile's project?

I agree @sisve. Though, tbh, that helper method is called json_decode, and it's in the PHP core, IMO. 😆

Filesystem is a Macroable class so I would suggest adding a macro for it in your project :)

Sounds good, thanks!
On Tue, Dec 20, 2016 at 9:02 AM Taylor Otwell<EMAIL_ADDRESS>wrote:

Closed #16866 https://github.com/laravel/framework/pull/16866.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/pull/16866#event-900147590, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWTSSuAjLRia_7BxcFVpgqorZ5fCn0-ks5rJ9_KgaJpZM4LQ6tI
.

--
Alfred Nutile
DevelopersHangout Podcast
http://www.developershangout.io/
ReadOrListen Podcast
http://www.readorlisten.com/
About
https://alfrednutile.info/


fixing hash function

Fixing hash function

What's the problem with hash function?

This adds a __hash__ dunder method so that doing hash(Block) returns the Block proof.


[Do Not Merge]Remove copyright Notice

See https://github.com/nunit/governance/issues/12

Replaced by https://github.com/nunit/nunit-csharp-samples/pull/12


Address review comments from PR 24986

Review comments there were not addressed before it was merged.  See https://github.com/project-chip/connectedhomeip/pull/24986#discussion_r1103481942
Specific actions:

Undo the changes to the certification tests that were not accompanied by test plan changes.
Add the test that was asked for in the review comments.

@jrhees-cae @yufengwangca

Of course ideally the test plans would in fact be modified so they catch this issue and then the test would be one of the certification tests.... @cjandhyala

@jrhees-cae Could you update the test plans, if the expected outcome in a TC is modified , we must execute the TC in a TE or SVE.

@cjandhyala I looked at the test plan document related to this test case (DRLK-10) and there aren't any needed changes. The test plan does not specifically call out what UserIndex or CredentialIndex to use for the test (the only thing I changed was the credentialIndex used during the steps).

Yes that is fine. It does not need to be verified.
On Tue, Feb 14, 2023, 4:52 PM cjandhyala @.***> wrote:

@jrhees-cae https://github.com/jrhees-cae Ok, I see verification steps
are updated to verify certain expected values, since the expected outcome
is changed in some of the steps in the script, will need to run this script
in a TE/SVE.
I see the script is updated to verify the below values.

CredentialIndex: 1 is changed to CredentialIndex: 2
verify "UserIndex and Credentials are set to null"

Also I see that expected outcome in one of the step is 'DataIndex is set
to 4' but in the actual verification that value is not verified. Is that
fine ?
—
Reply to this email directly, view it on GitHub
https://github.com/project-chip/connectedhomeip/issues/24992#issuecomment-1430542243,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOU6QVX5UFMKXKNMBT6GJHTWXQLCTANCNFSM6AAAAAAUYNNMVE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Fixed by https://github.com/project-chip/connectedhomeip/pull/25030


get python unit test coverage to 100% (a pipe dream?!)

As soon as I get some time back, I'd like to contribute to this!

Hey @jcrowgey. How you getting on with the testing? I've been away for a while but still interested in how you're doing with this.

Hi, I've been super-busy finishing up a PhD that should have been completed a while ago.  I'm mainly just trying to stay on top-of bug-reports at the moment---haven't worked on testing improvements at all.  If you have some time to improve the testing, I'd love to see it.
Cheers!

IMO 100% coverage is a non-goal, but incremental coverage goals are good, especially when they can be encoded into the tooling (so it can nudge people "hey, our goal is n% coverage for this file, consider adding test coverage along with your change").


Feature/get test cases map

I added a function that returns the test cases map in https://github.com/GoogleCloudPlatform/dfcx-scrapi/blob/main/src/dfcx_scrapi/core/test_cases.py
the name of the function is "get_test_cases_map"
The description is like below:
    """Exports Agent Test Cases and UUIDs into a user friendly dict.

    Args:
      agent_id: The agent to create the test case for. Format:
          `projects/<Project ID>/locations/<Location ID>/agents/<Agent ID>`
      reverse: (Optional) Boolean flag to swap key:value -> value:key
    Returns:
      Dict containing Test Case UUIDs as keys and display names as values.
    """

Made some typo changes on the last commit

@kmaphoenix Ready to merge!


Delayed initial orderbook sync until first message is received (#62).

The orderbook sync issue happens when the initial loadOrderbook finishes before the websocket client has subscribed to any products, which will cause some initial updates to be missed. This can also cause several unnecessary resyncs. Waiting until after the first message for that product is received resolved the issue (#62, #80) in my testing. A setTimeout for several seconds on the initial loadOrderbook call works most of the time as well, but it is not guaranteed.

@rsparlin Good idea, would you mind rebasing this?
Also, just curious: What's your opinion on #82?

Done. However, I did notice an unrelated issue that made orderbook syncing fail unless I corrected it. Line 79 of orderbook_sync.js needed this.processMessage to have this bound to it. I can separate this into a different PR if you like, but I have included the change for now.
As for #82, I would welcome the change; I can't think of a situation where I've been concerned with specific orders other than my own.

Decided to fix the nit in #83. Thanks a lot @rsparlin for this, should make the orderbook sync much more reliable.


Add feels_like to sensor list

climacell provides a data point called feels_like which is the apparent temperature that takes into account wind and other factors. Would be great to see this added (and added to the new lovelace card, which I will put in a request on that repository)

Hi, you can manage the field feels_like now. I have released a new version with the new feature now.
Thanks,
Renato


Exception when using build-info-extractor-gradle latest version with

I'm using the  build-info-extractor-gradle plugin(version 4.25.5) and gradle.plugin.com.palantir:gradle-flexversion version 0.9.2 in our projects. And since yesterday all my builds started to fail due to the following error
No signature of method: org.eclipse.jgit.internal.storage.file.FileRepository.getRef() is applicable for argument types: (String) values: [HEAD]
  Possible solutions: getAt(java.lang.String), getFS(), grep(), exactRef(java.lang.String), findRef(java.lang.String), getAllRefs()

It looks like when it's paired with gradle.plugin.com.palantir:gradle-flexversion:0.9.2 this issue starts happening. May be newest plugin using a newer groovy version?
Versions

Gradle Artifactory plugin version: 4.25.5
Operating system: macOS Monterey
Gradle version: 4.10.2

Thank you @aj-swipejobs, for taking the time to report this issue.
We have released Gradle Artifactory plugin 4.26.1, which includes the fix for this issue.

Hi @Or-Geva, thanks for fixing this one :). I can confirm it's working on my side at-least :).


Replace Gallery View with DPAD Aware RecyclerView

GalleryView is way beyond it's usefulness.  In the move to make a more matrial design and leanback application, need to replace it with the dpad aware exentsion of RecyclerView.
https://github.com/vganin/dpad-aware-recycler-view

https://github.com/sungerk/GalleryView/blob/master/galleryview/src/main/java/org/net/sunger/widget/GalleryView.java
This provides existing gallery type functionality.

Great plugin! I was having problems making kodi and plex work together using external players but your application made it possible. I would like to thank you. On another note, may I ask when the next update will be? I've seen you post on the plex forums stating that you'll be updating the app.

@Kuro07 thanks.. It all depends on my availability to work on it.   I work on it when I have time, which comes in short spurts.   Refactoring to Recycler Views was taking longer than I had anticipated.

I see. Is there anything that I can help with perhaps?

@Kuro07 I'm always willing to take pull requests if you have some android development skills.   You can checkout the branch https://github.com/NineWorlds/serenity-android/tree/dpadaware_recycler
Which contains the current work in progress.  I think I have both the main screen, movies, and tv shows migrated over.   Music and Search still need to be migrated.   Also I know that I have multiple tests that are failing.
With that said I should be getting some more time over the next couple of weeks to get back to this but any pull requests would be appreciated.
The dpadrecycler-view branch should work with the latest Intellij Community edition.  I haven't migrated yet to gradle (it is on the list).

This portion is mostly done.   I did end up removing the hardly ever used Music library support, mainly.  Also it looks like Google TV support will need to be dropped.  It had a nice run, but too many limitations on library support to make it worth while continuing to support the platform.
With this change, I may need to temporarily drop Tablet support, and just concentrate on D-Pad aware devices.
Should have this merged in later this weekend.

This has been merged into develop.


cargo fmt gives gobbledegook errors about 'nightly' that cannot be fixed

If I try using some cargo fmt options such as wrap_comments or format_code_in_doc_comments it just gives me errors that do not mean anything, and which cannot be fixed.
Typical messages are "unstable features are only available in nightly channel" and "the 'cargo' binary, normally provided by the 'cargo' component, is not applicable to the 'nightly-x86_64-unknown-linux-gnu' toolchain" and "the 'rustfmt' binary, normally provided by the 'rustfmt' component, is not applicable to the 'nightly-x86_64-unknown-linux-gnu' toolchain".
Please fix this.

Thanks for writing all that.
I was very frustrated as the error messages and the documentation did not give me any way to fix the problem. I would think that other users would got the same issue in the same situation. I think that the best would be to add a sentence or so to the documentation to explain the situation.
As it was, given your explanation, I uninstalled nightly and then reinstalled it, and then cargo +nightly fmt ran without issue.


[Feature]: expect library vite support

🚀 Feature Proposal
Currently expect library is unusable with vite:
| 11:34:12 PM [vite] warning:
| ./.vite_generated/deps/expect.js
| 13064|            try {
| 13065|              const moduleUrl = (0, _url().pathToFileURL)(filePath);
| 13066|              const importedModule = await import(moduleUrl.href);
|    |                                                  ^
| 13067|              if (!applyInteropRequireDefault) {
| 13068|                return importedModule;
| The above dynamic import cannot be analyzed by Vite.
| See https://github.com/rollup/plugins/tree/master/packages/dynamic-import-vars#limitations for supported dynamic import formats. If this is intended to be left as-is, you can use the /* @vite-ignore */ comment inside the import() call to suppress this warning.
|   Plugin: vite:import-analysis
|   File: ./.vite_generated/deps/expect.js?v=bb9d55bc

Motivation
Both vite and expect are popular libraries.  We use expect on our server side codebase, but can't on client side due to this issue.
Example
No response
Pitch
Because it's a fundamental limitation of expect codebase.

I tried to create a minimum reproduction example here:
https://github.com/kkimdev/expect-vite-reproduce/tree/main
steps:

clone the above repository
npx vite
open http://localhost:5173 in a brower

Result:

The above example is a fresh project using npm create vite --template react-ts and all I've done is importing expect in src/App.tsx. For some reason, it stuck at a different place, but it looks like expect doesn't work with vite out of the box anyways...

Interesting it tries to do some fs stuff...

That would be really great! expect is our assertion library of choice, and would love to use in more places!

I added a vite plugin vite-plugin-node-polyfills, the previous error disappeared. But it had a new one produced by the jest-util/src/isInteractive.ts#L10 as shown in the picture.
Because it's the process.stdout didn't polyffill in the plugin. And I did not find out a way to solve this, maybe some one who is good at vite or other bundler tool can give more help informations.

I guess this issue is not really specific to Vite since some of the errors are due to node library usages.

I think ideally expect should work out of the box on browsers without third party transformations.

Yes, I agree. But currently Jest still needs some additional work to implement. It may include adding another bundler tool to satisfy it can be used in a browser environment. this needs some investigation. And this maybe added in Jest v30?  @SimenB

I'd love to see https://github.com/jestjs/jest/pull/12348 picked up

I'd love to see #12348 picked up. Can start with the more reusable modules (like expect)

Cool~ Is there any plans for this PR?

PR welcome 🙂

With the node polyfills linked above, I only need to fix the process.stdout thing mentioned above. That's due to https://github.com/defunctzombie/node-process/issues/41. Easy enough to fix 🙂

This is great, thanks!
btw, how about having a node module shim layer that uses node module if available, or falling back to browser polyfill.  In this way, it's no longer dependent on build time transformation.
e.g.,
import * as af from 'node_shim/fs'  // Imports `fs` on node, polyfill version on browser/

It looks like vite-plugin-node-polyfills is using https://github.com/niksy/node-stdlib-browser underneath .

https://github.com/jestjs/jest/releases/tag/v30.0.0-alpha.1


Database

Hi,
I wanted to test the project but it seems that the database is missed in the package.
Can you provide the DB ?
Thanks a lot for your job
Here the error :
Error Number: 1146
Table 'pryper.common_tab' doesn't exist
SELECT * FROM common_tab WHERE position = 'head_content'
Filename: .../system/database/DB_driver.php
Line Number: 691

Initial Controller makes Database by php script.
You can set up config/database.php and start app with http://localhost/trivia-qa/initial or change default controller to "initial" controller in config/routs.php and start app with http://localhost/trivia-qa .
after getting successfully generation of tables change default controller to "page".
trivia-qa is your application folder in local


CYG-47830 | Added build support on jdk11

[CYG-47830 | Added build support on jdk11]
Officially, Samza code does not build on jdk11 as per : Discussion
Since we have to migrate to jdk11 by vrni-6.3.0, we have to make Samza work on jdk11. As of now, vrni does not use all the modules from the samza codebase. The modules used are mentioned below :

Through this PR, we have tried building all the modules required by vrni.

Build is Successful on jdk11

For now, we have skipped static checkstyle and javadoc jar creation issues on jdk11 in order to build. Command used to build the codebase : ./gradlew clean build -x checkstyleMain -x checkstyleTest

Unit Tests are passing on jdk11

All the UTs are passing. Some minor changes were done so as to make them work on jdk11. Ignored ones are already marked ignored in the codebase.

Integration test suite provided by Samza is working on jdk11

All the integration tests are now passing after doing some minor changes such as library upgrades etc.
Command Used : ./bin/integration-tests.sh /tmp/yarn-tests yarn-integration-tests
Yarn-Integration-Tests

Stand-Alone Integration Tests
Command Used : ./bin/integration-tests.sh /tmp/samza-tests standalone-integration-tests

Notes:

Samza code was having an issue with scala 2.11 on jdk11. Scala version 2.12 is used now for building the code on jdk11.
Rat and Checkstyle plugin issues have been ignored during build right now as the major task in hand right now is to integrate samza 1.5.1 into vrni.
No major code changes has been done in the core codebase as of now
Everything is working on jdk8 as well after doing the changes
yarnVersion has been changed to 3.0.0-cdh6.3.2 which is compatible with jdk11 and vrni also uses this version

Couldn't this be part of private vnera/third-party repo?


[FEATURE] Add green background for better chromakeying of gifs

Right now, if you want to window capture/browser source a gif you have to apply a chromakey to a dark grey background which could be a problem with some gifs. Maybe give the option to put the gif on a solid bright green background to make chromakeying cleaner.

Will keep this in mind for the Beta release. Perhaps a settings option to change your background color, that way you’re not limited to just green?


Missing Coding Challenges 73 and 118

[ ] Coding challenge 73 - Acrostic (https://www.youtube.com/watch?v=jwoK5WKVXGw)
[ ] Coding challenge 118 - Mastodon Fractal Tree Bot (https://www.youtube.com/watch?v=luxczHFn1rU)

in case someone wants to port these, here is a helpful tool to get a starter json file, based on data from the old website

@shiffman @jasontheillustrator Are there thumbnails for these challenges that we could use?

Looks like we don't have any new thumbnails. The current thumbnails are older ones that @shiffman might have access to.

Apologies for the slowness in replying/reviewing I'm traveling until August 6. @jasontheillustrator can make new thumbnails for the YT videos but for the index.jpg files that go along with the challenges we usually just take the visual element frmo the thumbnail or custom design our own (w/o a photo of me or text). @kfahn22 has made most of them in fact!
I'm happy for anyone to just create something for these two challenges and I'll work with @jasontheillustrator to upload the YT thumbs themselves!

I made some thumbnails for you.

Thanks @jasontheillustrator!
For challenge 118, I'm wondering if a thumbnail only showing the fractal tree would be misleading. The videos are mostly about using the Mastodon API in Node.js to make a bot, and executing a Processing sketch from the command line. The tree fractal is not coded on screen, I believe it's one of the official Processing examples lightly altered to demonstrate the bot behavior.
For challenge 73, @kfahn22 updated the source code to make it functional again, and provided a matching thumbnail in #1671 so maybe we should go with that one?

Thanks for the feedback. I think it's probably best if @shiffman weigh in on this.

What if we try a thumbnail like this? The only thing is the tree angle in the render might not actually be 52 degrees, an easter egg for the detail-oriented viewer?

Closing this because all Coding Challenges are not on website!


MissingMessageHandlerException when passing derived class to Send

I understand why this doesn't work, but would it be possible to implement support for this?
This is my use-case:
In domain project (with Mediator.Abstractions)
public interface IMyReq : IRequest<string>
{
    string MyParam { get; }
}

public sealed class MyReqHandler : IRequestHandler<IMyReq, string>
{
    // implementation...
    public ValueTask<string> Handle(IMyReq req, CancellationToken ct)
    {
        // implementation...
    }
}

In asp.net application project (with Mediator.SourceGenerator)
public sealed record MyReq : IMyReq
{
    [FromQuery]
    public required string MyParam { get; init; }
}

public static partial class Endpoints
{
    public static void MapEndpoints(this IEndpointRouteBuilder endpoints)
    {
        endpoints.MapGet("/my_endpoint", (IMediator mediator, [AsParameters] MyReq req) => mediator.Send((IMyReq)req));
        // ...
    }
}

because handler member should by class
https://github.com/martinothamar/Mediator/blob/5b500ad9bd61606d96b0e8ab045a5a6eff1499f5/src/Mediator.SourceGenerator.Implementation/Analysis/CompilationAnalyzer.cs#L442C64-L442C64

Mmm.. I don't think that's the problem. First because records are classes (unless it's a struct record). Second, because I'm already using records as commands and it works just fine.
So, the issue must be anything else, maybe related to the use of an interface?

I believe feiyun0112 is correct. It would be nice to support interfaces or abstract classes


Reduce compile time

The commits are mostly independent, so if you think any of the commits is bad for maintainability, I can easily drop it. The commits I'm most confident about are:

https://github.com/Smithay/wayland-rs/commit/49779784766a15d937b098289124d4aaffb754b5 "Remove futures dependencies from wayland-client"
https://github.com/Smithay/wayland-rs/commit/84aad31150043622a5ffd42c9c9a71ec96e2d98f "Reduce code bloat of smallvec![] macro"

Rebased


I2c API

This API has not been designed yet. What methods should it provide?

I2C is interesting. Since we very likely want to retain the async property of the interface, there needs to be a way to model master and slave operation and hence the ability to either initiate and keep a new "session" with a slave having a specific address (in the simple case, there're also variants where group addressing is possible) where the operation can be read, write or combined. Or in the slave case the device only has a specific address and needs to respond to the read or write operations initiated by the master.


Duplicate get_wan_ip and possible missuse.

I noticed you are using a get_wan_ip function from pyp2p https://github.com/Storj/storjnode/blob/develop/examples/network/node.py#L2 how does this work and why cant you use the build in stun functions of the protocol https://github.com/Storj/storjnode/blob/develop/storjnode/network/server.py#L241?
Exposed via:

storjnode.network.node.sync_get_wan_ip
storjnode.network.node.async_get_wan_ip
storjnode.network.node.sync_has_public_ip
storjnode.network.node.async_has_public_ip

This also seems to be used wrong here https://github.com/Storj/storjnode/blob/develop/storjnode/network/node.py#L186 and possible other places. You cannot assume the wan facing port is the same as the port on the local interface, the NAT may map it to another one.
I suggest replacing the get_wan_ip function from pyp2p with the one from the DHT protocol as is uses other nodes as stun servers and doen't relay on external services.
I'm not quiet happy with the interface so I will add a more efficient get_transport_info call that returns { "wan": (ip: port), "local": (ip: port) } and keep the above methods but implement them using this.
There would also be no need to pass the wan_id in the constructor https://github.com/Storj/storjnode/blob/develop/storjnode/network/node.py#L178 as the node is perfectly capable of getting it itself.

Removed in https://github.com/Storj/storjnode/commit/b543dc1d8f8342788c699d32a41cc5c0b8a48e39


How to deal with data parallel and distributed data parallel?

On my eyes, your code is just running with single gpu while I need to test this code with multi-gpu for other implementations. I just want to check that you have ran your code using data parallel and distributed data parallel.

Good question, you cannot warp the model with data parallel functions
To support multi-GPU reconstruction, you should initialize distributed environments and allreduce the gradients manually.
I will add the dist mode in a few days.

@yhhhli
hi，multi-GPU reconstruction does not reduce the time consumption. Is there any way to speed up reconstruction.


[BUG] After the workers group, specify a group. The task cannot view the log, but can only download and view it

Describe the bug
After grouping, the task cannot view the log, but if the default grouping is used, the log will be displayed
To Reproduce
Steps to reproduce the behavior, for example:

Create a workers group
Create a test project
run the project
after look the log is '暂无更多'

Expected behavior
i think the log is show the background
Screenshots

Which version of Dolphin Scheduler:
-1.2.1
Additional context
Add any other context about the problem here.
**Requirement or improvement

i want look the log

is logger server started up ?

could other workflow see the log or not?

Do not reply for a long time, temporarily close the issue


[RV_DM]jtag_dtm_idle_hint

This test verify that debugger does not need to stay any longer than advertized idle time indicated in dtmcs register.
@jdonjdon please review it. Thanks.

@BasitMehmood
Did you do rebase?

@BasitMehmood Did you do rebase?  you have a conflict in these files.

yes i have done it.

You have this error too.
2023-10-12T06:39:15.6919005Z Fixing file: "hw/ip/rv_dm/dv/env/seq_lib/rv_dm_vseq_list.sv"
2023-10-12T06:39:15.6963503Z ##[error]Whitespace check failed. Please run util/fix_trailing_whitespace.py on the above files.

@jdonjdon i will do again PR to resolve conflicts.

i am closing this PR and make another one.


Please extend this and show how POST, PUT, and PATCH methods can be implemented for XML and JSON media types

See this issue for more details.
It is not obvious how one should:

Fetch the entity with a given ID from the reposiory
Update its properties from the JSON/XML payload (PATCH)
Replace the entity (PUT)
Save it back to the repository

I concur that that would be good content. However, we try to make the guides be completable in 15 minutes. Adding those bits would take us over the 15-minute limit. You could write a blog post and share it with the Spring community.


Position of slider is updated on Compare cost fragment.

How reproduce:

Start app
Select node of app with compare button
Touch compare cost button -> Position of slider is updated after screen is opened

I'm not quite sure what you mean here, can you share some screenshots or a screen recording?

https://github.com/ev-map/EVMap/assets/87148630/a40eb9d1-fbf6-4cc5-b140-843633d7f29f


speed and memory plots

This PR updates the speed_and_memory.ipynb notebook.
The notebook now plots

generation speed vs. compression rate
peak memory vs compression rate
cache size vs compression rate
for both default and quantized cache.

I did not include prefilling speed, as this is mostly independent of the cache size (and the repo isn't designed to optimize this part).
Apart from that, the spelling of "Hugging Face" was changed in various files.

Could you add a .gitattributes file with:
*.ipynb linguist-documentation

and check the GitHub page stop displaying "74% Jupyter" in the languages section ?

(I don't manage to add inline comment in the notebook on the GitHub UI so I'll add them in this main thread)
Curiosity question: is get_size_of_cache necessary ? We know the formula for it: $\text{size} = 2 * n_{layers}  * n_{heads} * d * \text{precision}$ with $\text{precision}=2$ for float16 and $\text{precision}=0.5$ for int4. The quantized cache adds some parameters with _scale and _shift, does it make a big difference ?

About the plots:

it's great to display the 3 metrics (peak memory, cache size and generation time)
I would remove the dashed line because it's not very clear what it is. For instance if it's max memory, why isn't it always reached ?
the generation time for quantized cache is twice slower than with float16, is it expected ? in hf blog post it's not supposed to be slower. See also this issue
could you use the same y-axis for both bfloat16 and int4

It would be great to add a single plot that illustrates why using kvpress. I propose to recycle the data you created in the notebook with what I believed will interest users the most:

x-axis: context length
y-axis: peak memory usage
4 curves:

bfloat16 cache
int4 cache
bfloat16 cache + 50% compression
int4 cache + 50% compression

Also, I would add an horizontal dashed line at X and clip all curves below this X value to clearly show that with compression / quantization you can fit more context length in your GPU

Thanks for the feedback!

attn_implementation: IMO, it's Ok to assume flash attn is available when running the benchmarks.
As you pointed out, get_size_of_cache is not directly necessary. I added the function, as it is more explicit IMO.
I would remove the dashed line: Makes sense, thanks for spotting
4. the generation time for quantized cache is twice slower than with float16, is it expected? Interestingly, the prefilling time is similar for both methods. Seems that there's a different between prefilling and generation time.
could you use the same y-axis for both bfloat16 and int4 Yeah, I can change
maybe remove the 400 tokens curve ? Makes sense, I replot with 8_000 tokens start.

Thanks for the updates and cool plots. Could you add the last plot in the main README ? (and maybe create an assets dir for the image and kvpress.jlg)


Implement C'thun quests from spec

We've designed some C'thun quests that just need to be implemented: https://docs.google.com/document/d/142wKOD2FKQwCwIBhoLqT9n6A2v987oJRqhyCT_nxaUI/edit
@SebastianTanoni will do a first pass, implementing objects and quest frameworks.
@YakaryBovine will finish it off with quest flavour and any code requirements.

Just noticed this is a duplicate. Let's use #2683 instead


Several community members seeing nodes "offline" and not syncing to best tip. jrwashburn and akbasemre1 sent their logs in for this

Looking closer at our log rotation code, it seems to be correct. It's unclear why there would be a large time gap in the rotated logs vs the current log. I double checked some of the Core.Unix functions we call and they seem to do the correct thing on the filesystem. Will continue to try and track down that bug some, but until then, we may not be able to properly debug this until we observe it in an environment where we replicate the logs and store them to inspect. If we are able to recreate this bug in one of our QA net environments (on our nodes), we already have that setup, so we could debug there.

:man_facepalming: turns out the script I wrote the find the most recent rotated log was wrong. I was able to find a log file where the validation timeout began. I see a lot of successful validations, followed by some odd connection issues, which is then followed by a wall of validation time outs. The odd thing is, at this point, some validations still succeed before the timeout. However, the messaging in the logs seems to indicate something is very wrong with these validations. They are either super old (logs could be reporting wrong time format as well though), or there is something wrong with the timing logic.
[2020-11-28 16:31:31.500000]Libp2p_helper.Go.codanet.Helper: validation for item 1744 took<PHONE_NUMBER>59 seconds
[2020-11-28 16:31:31.500000]Libp2p_helper.Go.codanet.Helper: validation for item 2041 took<PHONE_NUMBER>49 seconds
[2020-11-28 16:31:31.500000]Libp2p_helper.Go.codanet.Helper: validation for item 2165 took<PHONE_NUMBER>77 seconds
[2020-11-28 16:31:31.500000]Libp2p_helper.Go.codanet.Helper: validation for item 1871 took<PHONE_NUMBER>31 seconds
[2020-11-28 16:31:31.500000]Libp2p_helper.Go.codanet.Helper: validation for item 1789 took<PHONE_NUMBER>58 seconds

Confirmed that the logs for libp2p_helper validation timing is misleading. It's reported in nanoseconds instead of seconds. I will fix that separately, but this means the logs indicate that it is taking upwards of 12 and a half minutes to return a validation callback. Seems as though the node is stuck behind a wall of validation spam, and is never able to catchup to still valid validations. This could be because, when a validation times out from libp2p, we still process it in ocaml. If it takes longer to drain the entire queue of transactions to validate than it does for a timeout, and the validation callbacks do not timeout in ocaml, then that means that a node in this state will likely be unable to ever begin validating messages again. I will do some thinking about the best way to fix this.

Here is my official suggestion on how to fix this:
Configure validation timeouts from the ocaml side. Send the validation timeout duration as a configuration parameter to libp2p (this ensures they will both be in sync). Create a better type for libp2p callbacks which, in addition to a function to call for sending the callback message to go, also includes a "created_at" timestamp sent by the go process. When we pull validation work off of queues (most notably, in the transition processor and in the network pools), check to see if the validation callback still valid using the configured timeout duration and the timestamp the go process sent us. If the validation callback is outdated, skip it.
This seems like the easiest solution with the lowest surface area for future configuration issues.

Make sure that snark coordinator is up-to-date and recheck


pointer-events: none remain on a closed dialog, making it impossible to interact with when reopened

To reproduce:
npm init
npm install @vaadin/vaadin-crud

create index.html:
<script type=module<EMAIL_ADDRESS><vaadin-crud></vaadin-crud>
<script>
document.querySelector("vaadin-crud").items = [
{"foo": "bar"},
{"foo": "baz"}
];
</script>

Run with polyserve and

Click on "edit" for the first row
Click Delete...
Click Delete
Click edit on the remaining row
Try to do anything in the popup

Downgrading from 3.2.7 to 3.2.5 seems to fix the issue


feat: allow SmartInput and Field to receive autoComplete prop

Feature request
Current Behavior
Right now, there is no way to give a Field or a SmartInput an autoComplete prop, e.g. autoComplete="username".
Let's say you have a custom Input configured and a Zod schema like this:
import { forwardRef } from 'react';
import type { FormProps } from 'remix-forms';
import { Form as RemixForm } from 'remix-forms';
import type { SomeZodObject } from 'zod';
import { z } from 'zod';

const Input = forwardRef<HTMLInputElement, JSX.IntrinsicElements['input']>(
  ({ type = 'text', ...props }, reference) => (
    <input
      className="shadow-sm focus:ring-indigo-500 focus:border-indigo-500 block w-full sm:text-sm border-gray-300 rounded-md"
      ref={reference}
      type={type}
      {...props}
    />
  ),
);

const Form = <Schema extends SomeZodObject>(props: FormProps<Schema>) => (
  <RemixForm<Schema>
    className="space-y-8 divide-y divide-gray-200"
    inputComponent={Input}
    {...props}
  />
);

const schema = z.object({
  firstName: z.string().min(1),
  email: z.string().min(1).email(),
})

And you want to have an input with an autoComplete property, you have to render it like this:
<Field className="sm:col-span-3" name="firstName">
  {({ Label, Errors }) => (
    <>
      First name

      <Input
        {...register('firstName')}
        autoComplete="given-name"
      />

      <Errors />
    </>
  )}
</Field>

<Field className="sm:col-span-3" name="emailAddress">
  {({ Label, Errors }) => (
    <>
      Email address

      <Input
        {...register('emailAddress')}
        autoComplete="email"
      />

      <Errors />
    </>
  )}
</Field>

This renders using SmartInput as well as configuring inputComponent on the Form useless in these cases.
Desired Behavior
Ideally, we could give the Field or the SmartInput the autoComplete property, so we could render:
<Field className="sm:col-span-3" name="firstName">
  {({ Label, SmartInput, Errors }) => (
    <>
      First name

      <SmartInput autoComplete="given-name" />

      <Errors />
    </>
  )}
</Field>

<Field autoComplete="email" className="sm:col-span-3" name="emailAddress" />

(Note that in this case, we could also render firstName straight up as a Field, but I wanted to show the "Custom Field, standard components" version, too.)
We also have to consider this for the Form's renderField prop, e.g.:
renderField={({ Field, autoComplete, ...props }) => {
  const { name } = props;

  return (
    <Field key={name as string} {...props}>
      {({ Label, SmartInput, Errors }) => (
        <>

            <SmartInput autoComplete={autoComplete} />

          <Errors />
        </>
      )}
    </Field>
  );
}}

Alternatives
Alternatively, maybe there is a way to infer the autoComplete prop from the schema for certain all the keys that are possible autoComplete values.

@janhesters, thank you also for offering to create a PR for this one. I will share what I'd do if I were to implement it at a high level, but please bring as many follow-on questions as you need:
First, I'd create three examples on our test-examples folder. The first one would include a few Fields receiving different autoComplete props each, but without Field receiving children.
The second would include similar Field's receiving the autoComplete prop, but they would all receive children and render inputs like Input, SmartInput, etc. This will make sure that Field passes its own autoComplete prop along to inputs when the inputs don't receive it directly.
The third would do the same with SmartInputs receiving the autoComplete prop directly.
Then I'd write failing E2E tests, allowing me to TDD my way to implementation.
Only then I'd start implementation. For the Field, I'd add the autoComplete prop here, destructure it here, and then pass it to all appropriate inputs inside this long if/else mapping, which is used for when Field receives children.
For the SmartInput, I'd first add the prop here and then make sure it receives the one passed to Field here, which is used for when Field does not receive children.
Then I'd add destructure it here and pass it to the appropriate inputs inside this chain of ternaries.
I probably missed something, and I hope I didn't take away the fun by giving you the (probably) precise places to change. But I know this code is complex, and I thought this would be an incentive for a good first PR :D
Let me know if you have any doubts!

Wow thanks a ton! I'm going to finish a PR over the coming days. Just worked on it for an hour, but its much to learn in this codebase haha.

I hope I didn't take away the fun by giving you the (probably) precise places to change.

No, you honestly enable me be able to write the PR in the first place! Thanks again, will post here when the PR is ready again, or if I have more questions 🙏


stylesheet issue - 404 Not Found

For some download, when viewing a downloaded OEBPS/*.xhtml page with a browser, the page rendered did not seem to be styled correctly (quite different from what is viewed on-line which seems to offer better readability).
Looking at OEBPS/Styles/ and found that some downloaded stylesheet file (OEBPS/Styles/Style*.css) has the following content:
<title>404 Not Found</title>

404 Not Found

nginx/1.19.7

Can it be an error with the downloading process or something else?
Is there anything can be done to fix it?
Thanks!

This issue can be easily reproduced by downloading a book, e.g.<PHONE_NUMBER>553.
# ./safaribooks.py<PHONE_NUMBER>553

 ██████╗     ██████╗ ██╗  ██╗   ██╗██████╗
██╔═══██╗    ██╔══██╗██║  ╚██╗ ██╔╝╚════██╗
██║   ██║    ██████╔╝██║   ╚████╔╝   ▄███╔╝
██║   ██║    ██╔══██╗██║    ╚██╔╝    ▀▀══╝
╚██████╔╝    ██║  ██║███████╗██║     ██╗
 ╚═════╝     ╚═╝  ╚═╝╚══════╝╚═╝     ╚═╝

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[-] Successfully authenticated.
[!] Working on book<PHONE_NUMBER>553 [ 1 of 1 ] ...
[*] Retrieving book info...
[-] Title: C++ Crash Course
[-] Authors: Josh Lospinoso
[-] Identifier:<PHONE_NUMBER>553
[-] ISBN:<PHONE_NUMBER>885
[-] Publishers: No Starch Press
[-] Rights:
[-] Description: Upgrade your Code with C++C++ is one of the most widely used languages for real-world software. In the hands of a knowledgeable programmer, C++ can produce small, efficient, and readable code that any programmer would be proud of.Designed for intermediate to advanced programmers, C++ Crash Course cuts through the weeds to get you straight to the core of C++17, the most modern revision of the ISO standard. Part 1 covers the core of the C++ language, where you’ll learn about everything from types ...
[-] Release Date: 2019-09-24
[-] URL: https://learning.oreilly.com/library/view/c-crash-course/9781098122553/
[*] Output directory:
    /home/root/safaribooks/Books/C__ Crash Course<PHONE_NUMBER>553)
[*] Retrieving book chapters...
[-] Downloading book contents... (38 chapters)
    [####################################################################] 100%
[-] Downloading book CSSs... (2 files)
    [####################################################################] 100%
[-] Downloading book images... (67 files, 45 uniq)
    [####################################################################] 100%
[-] Creating EPUB file...
[*] Done: /home/root/safaribooks/Books/C__ Crash Course<PHONE_NUMBER>553)/9781098122553.epub

    If you like it, please * this project on GitHub to make it known:
        https://github.com/lorenzodifuccia/safaribooks
    e don't forget to renew your Safari Books Online subscription:
        https://learning.oreilly.com

[!] Bye!!
[*] Done: ['9781098122553']

    If you like it, please * this project on GitHub to make it known:
        https://github.com/lorenzodifuccia/safaribooks
    e don't forget to renew your Safari Books Online subscription:
        https://learning.oreilly.com

[!] Bye!!

Download on Oct 15, 2021: the C__ Crash Course<PHONE_NUMBER>553)/OEBPS/Styles output directory contains 2 .css files but the second one (style01.css) was invalid and its content was showing 404 Not Found:
# ls -l
total 20
-rw-r--r--. 1 root root 14342 Oct 15 01:26 Style00.css
-rw-r--r--. 1 root root   555 Oct 15 01:26 Style01.css
[root@24234f2a3a16 Styles]# cat Style01.css

<title>404 Not Found</title>

404 Not Found

nginx/1.19.7

Comparing to a previous download on Nov 4, 2020 for the same book: it was the same book, however the C__ Crash Course<PHONE_NUMBER>553)/OEBPS/Styles output directory contains 2 .css files which were both valid. Style01.css contains valid content. Also, a diff on the Style00.css shows it is same with recent download.
$ ls -l
total 232
-rw-r--r--. 1 root root  14342 Nov  4  2020 Style00.css
-rw-r--r--. 1 root root 220357 Nov  4  2020 Style01.css

The issue appears to be true for other book downloads.
Is any other information needed?
Thanks.


Clicking clear button in edit view reverts text for submit button

If you edit a lesson and click the CLEAR button to remove the form input text, it also reverts the submit button text from 'UPDATE LESSON' to 'ADD LESSON'. It should remain as 'UPDATE LESSON'.

hi @jrobind I would like to handle it, well I have fixed the problem. just I have to make a pull request.

Awesome, go for it! 👍

Closed by #104


Creating multiple ui implementation (branch for xml, anthor for Jetpack compose)

I recommend utilizing the separate layers and creating two ui versions: The XML existing one and a new compose one. The main activity shows a button to go for each.
I think this will showcase the real power of separation of concerns.

see #34

We can also demonstrate the Navigation library, it will be a nice addition and should ease the flow.

Well, I don't mean separate branches, I mean like in the same app. Imigain a main activity with btn for each ui way of implemenation. I don't mind separate branches, just with separate branches we'll not need the Navigation library, so we'll use the chance to demonstrate it. : )
I'm just thinking out loud, feel free to share what you think. :)


FAI-12269 Conform to new Bitbucket spec

Adapt to some spec renames here: https://github.com/faros-ai/airbyte-connectors/pull/1638
No need to be backwards compatible since we always pull the latest source image for Bitbucket.

@ypc-faros do we need to update this? https://github.com/faros-ai/faros-community-edition/blob/main/init/resources/airbyte/workspace/airbyte_config/SOURCE_CONNECTION.yaml#L22
or maybe this?
https://github.com/faros-ai/faros-community-edition/blob/main/init/resources/airbyte/workspace/airbyte_config/STANDARD_SYNC.yaml#L472

Thank you @thomas-gerber 🙇 !
https://github.com/faros-ai/faros-community-edition/pull/358


Fixes the hang on BGP password

This is a fix for #226, the logic around parsing annotations would never end if a BGP password wasn't supplied. The password is now optional.
This also fixes a debug message.
root@k8s:~# k logs -n kube-system   kube-vip-ds-8m9qk
time="2021-07-05T12:37:45Z" level=info msg="server started"
time="2021-07-05T12:37:45Z" level=debug msg="Using the internal Kubernetes token"
time="2021-07-05T12:37:45Z" level=info msg="Kube-Vip is waiting for annotation prefix [metal.equinix.com] to be present on this node"
time="2021-07-05T12:37:45Z" level=debug msg="<IP_ADDRESS> / 65000 / <IP_ADDRESS> / 65530 \n"
time="2021-07-05T12:37:45Z" level=info msg="Exiting Annotations watcher"
time="2021-07-05T12:37:45Z" level=info msg="Starting Kube-vip Manager with the BGP engine"

Can this be included in a tagged release?


NASA Test Plan draft.

Hi Team!
Here is the first draft of the NASA Test Plan.
I am adding the same file in two formats to simplify the reading and editing. Please review, critique, and contribute.

Ok

Good


Mistake promise/prefer-await-to-callback

How do I avoid an error with promise/prefer-await-to-callback for this following code which use events:
dbConn.on('error', (err) => logger.warn('MongoDB event `error`:', err));

For now, I would disable that rule for the line using:
// eslint-disable-next-line promise/prefer-await-to-callbacks
dbConn.on('error', (err) => logger.warn('MongoDB event `error`:', err));

This seems like a bug though. Perhaps that rule should ignore callbacks in an on() function (which indicates using events). That's not something we currently check for but should probably be considered. I'm not 100% sure how to accomplish that off the top of my head, but suggestions or PRs are welcome!

I think I found a fix for this - pushing up to #117.

Maybe this plugin can help you: eslint-plugin-unicorn, check the rule prefer-add-event-listener for example.

Also the rule page is missing examples:
https://github.com/xjamundx/eslint-plugin-promise/blob/master/docs/rules/prefer-await-to-callbacks.md


[WIP] Add Cloud Volume Type model

https://bugzilla.redhat.com/show_bug.cgi?id=1592900
Adds a basic model for Cloud Volume Types, which are needed for mapping new Cinder volumes to specific backends. This is especially important for Openstack V2V infrastructure mapping. Depends on the schema migration in https://github.com/ManageIQ/manageiq-schema/pull/223

@roliveri This is Cinder-related, your feedback would be appreciated.

I'm not sure I understand the need for a basic model for volume types. Can't volume types be represented as subclasses of a common base class, and queried accordingly?
It seems the data stored in this new table will not vary within type, so they can probably just be defined as class-specific values of the type subclasses.

Volume types are user defined, so I believe they would need to be collected from the Cinder API like any other inventory: https://developer.openstack.org/api-ref/block-storage/v3/index.html#volume-types-types

@mansam
Ah, I see. It's a bit confusing because type is overloaded. There are types of volumes like: Cinder, EBS, etc. These are user-defined sub-types within type. Do you think it's worthwhile renaming these classes (and tables) to make this distinction more clear?

@roliveri Do you think calling this class/table "CloudVolumeBackingType" would avoid the possible confusion?

@mansam I'm not sure. To me, backing type indicates the type of storage on which a virtual volume is based.
Are we really talking about sub-types or sub categories of a given storage type?
I'm not sure it's worth spending too much time on. Maybe a comment in the model class definition will be enough to clear up any confusion.

Cinder Volume Types are mostly a way to set the backend on which to create a volume, so using BackingType in the name probably isn't too confusing. But I agree with you that a comment is probably more than sufficient.

:+1:

https://github.com/ManageIQ/manageiq-schema/pull/223 has been merged, so this can be merged.


Navbar responsive

Description
Created new navbar using bootstrap which is responsive too.
Fixes #102
Type of Change:

New Feature

How Has This Been Tested?
On my local system and github pages on forked repo.
Checklist:

[x] My PR follows the style guidelines of this project
[x] I have performed a self-review of my own code or materials
[x] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

@jigar-sable merged. Thank you for your contribution 🥇

@manankohlii thanks for merging my pr..


Sonos Bug

Thanks for the great repo!
Have a small issue with trackDescription data coming from Sonos:
postToInfluxDB(): Something went wrong! Response from InfluxDB: Headers: [content-length:489, request-id:c464608c-7473-11e7-b65d-000000000000, http/1.1 400 bad request:null, content-type:application/json, date:Sat, 29 Jul 2017 15:37:02 GMT, x-influxdb-version:1.3.1], Body: {"error":"unable to parse 'trackDescription,deviceId=9c93ae4e-671d-4ffa-998d-895cb3d16e45,deviceName=Office,groupId=null,groupName=Home,hubId=655bcc7a-dfd6-4458-a163-21c2b670325e,hubName=Home\\ Hub,locationId=d0288ea8-7456-442d-b4c8-41e83f5a69a5,locationName=Home,unit=null value=Okay\\ Then': invalid boolean\nunable to parse 'by\\ Sokro': missing fields\nunable to parse 'from\\ LA\\ FAMILIA\\ PT.\\ 1': missing fields\nunable to parse 'Grouped\\ with\\ Living\\ Room': missing fields"}

I've seen this issue with my setup. In my case names of rooms and sensors aren't only in ASCII but in Unicode (UTF-8). I suspect platform not working correctly with UTF-8 encoded POST payload. Please see  PR #20 if that helps.


Alpha sorted function support page

This PR adds to the documentation a page inspired by the web page at http://utensil-site.github.io/available-in-katex/. The proposed page is sorted alphabetically. It contains functions both supported by KaTeX and un-supported by KaTeX. It shows you which is which. It thus can serve as a replacement for the original page and for the current wiki page that shows things not supported by KaTeX.
I started with the other page and I also scraped the function list at https://docs.mathjax.org/en/latest/tex.html, but having then done so, I created a static page. It will not automatically update with new functions.
docusaurus does not recognize the title that I've given to the page. I don't know why and would appreciate a suggestion on how to fix that.

Fixes #103, fixes #276, and fixes #1480.

‘’’
(
)
]
|
>
\allowbreak
\arcctg
\blue
\blueA
\blueB
\blueC
\blueD
\blueE
\bull
\C
\cancer
\ce
\cf
\clubs
\diamonds
\empty
\exist
\goldA
\goldB
\goldC
\goldD
\goldE
\gray
\grayA
\grayB
\grayC
\grayD
\grayE
\grayF
\grayG
\grayH
\grayI
\green
\greenA
\greenB
\greenC
\greenD
\greenE
\harr
\Harr
\hdashline
\hdashline0
\hearts
\hfil
\kaBlue
\kaGreen
\L
\l
\lang
\maroonA
\maroonB
\maroonC
\maroonD
\maroonE
\mathsterling
\mintA
\mintB
\mintC
\N
\nobreak
\orange
\overparen
\part
\pink
\purple
\purpleA
\purpleB
\purpleC
\purpleD
\purpleE
\R
\rang
\red
\redA
\redB
\redC
\redD
\redE
\relax
\scr
\sixptsize
\smiley
\spades
\sub
\tealA
\tealB
\tealC
\tealD
\tealE
\textdegree
\underparen
\varvdots
\xleftequilibrium
\xrightequilibrium
\xrightleftarrows

@ylemkimon Thank for the list of functions. There is good stuff in there.
I believe that the color functions were intended to be un-documented and used internally by Khan Academy. Similarly, the extensible arrows are there to support mhchem. They don't exist in any LaTeX package, so we don't intend to put them in the documentation either.
I'll go to work on the others.

@ronkok Sorry, I had no time to append description. The list is from various sources including KaTeX source. So it may include typos, non-standard commands, and KaTeX internal commands, like KaTeX colors. And it is simply compared using Excel, so comparison may be inaccurate.

Following seems to be missing:

math environment in text mode: \(, \), \[, \]
\| (\Vert)
\hfil, \vfill, \vfil
\bull, \exist, \lang, \rang, \N, \R: mediawiki (non-standard)
text small caps: \textsc

@ylemkimon There was a reason some functions were omitted. I could not find independent confirmation that things like \bull and \hfil actually exist somewhere. I've now checked the texvc package, which  underlies mediawiki, and I've been able to confirm that many of those functions do indeed exist.
But I still can't find \hfil and \vfil anywhere. Do you know where they come from?

@ylemkimon I've picked up review comments, with the exception of \hfil and \vfil. I'd be happy to include those once we're sure that they exist somewhere.

\hfil and \hfill (and vertical variants) are all TeX built-ins.  I believe they're equivalent to \hskip1fil and \hskip1fill (why is there no \hfilll for \hskip1filll?) but given that they're not macros, I might be missing some subtlety.  We can check their meaning in The TeXbook.

Speaking of which, it might be nice to link to relevant Github issues in that third column for unsupported functions.  For \hfil etc., it's #990.

There is also a \! (negative thin space). https://github.com/Khan/KaTeX/pull/1536#discussion_r206973492 and https://github.com/Khan/KaTeX/pull/1536#discussion_r206973778 seem to be not addressed.

I believe that I have now picked up @ylemkimon's comments and I have also added references to several GitHub issues. I've probably missed some issues, but's it's a good first effort.
I did not add \text{\[\frac a b \]} because that doesn't work when I test it on the demo page.

@ylemkimon Good stuff. All done.

@ylemkimon Hold off on the merge for a minute. I've got one more to add.

Now it's good to go.

Thank you! This fixes four issues.

@ronkok this is super cool!

@ronkok Great work!
Where can I see this list? It seems there's still only https://khan.github.io/KaTeX/docs/supported.html after this PR is merged. And the gh-pages branch seems to be behind the master branch?

@utensil It is available on https://deploy-preview-1536--katex.netlify.com/docs/support_table.html until #1544.

@utensil 208 out of how many?

@kevinbarabash 208 out of 1055, which is the same as the total count of this Alpha sorted function support page.


"copy" does not actually copy data, only creates a link to original data

I published imagery for the World Wildlife Fund using my own AGOL account, then used ago-assistant to copy from my account to their account.  We published the imagery from their account and it worked fine, but when I deleted what I believed to be the original copy from my account, the imagery was lost from their account.
Having an option to create a link would also be useful, but the "copy" command needs to actually make a copy, not a link!

What was the item type for the content that you copied? There is an option to "Full Copy" a hosted service. That process creates a full replica of the original service complete with data in the destination account. The Simple Copy option creates a reference (link) to the original.

Evan
This has been a couple of weeks, and I have to be honest that I don’t recall what I did.
I looked at the assistant UI again today, and I am sure I hit “copy” but I don’t understand why some items are not selectable – I am guessing now that what I must have done was copy an imageservice* (or maybe a web map), but did not realize I did not select the source imagery for that service.  If correct, in this case it makes sense that a service points to the original data and did not copy the source data.
However I can’t see the “Full Copy” option you’re referring to – this is all I see:
<EMAIL_ADDRESS>Should there be a follow-on confirmation menu after a copy, “execute Simple Copy or Full Copy?”…?   I don’t see many features so don’t understand how to invoke a Full Copy.  I’m on Chrome – should I be using a different browser?
Cody
From: Evan Caldwell<EMAIL_ADDRESS>Sent: Wednesday, May 18, 2016 2:27 PM
To: Esri/ago-assistant<EMAIL_ADDRESS>Cc: Cody Benkelman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Esri/ago-assistant] "copy" does not actually copy data, only creates a link to original data (#134)
What was the item type for the content that you copied? There is an option to "Full Copy" a hosted service. That process creates a full replica of the original service complete with data in the destination account. The Simple Copy option creates a reference (link) to the original.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ago-2Dassistant_issues_134-23issuecomment-2D220163323&d=CwMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=SD5yC5DpYWeG5jFHVhUfKs533_rBp8fz2P_t7Cxtqmw&s=wvL4zpvzs_ec5DpiBjJ-7bGuesmmqwj2tOpPdZ9YIRo&e=

The full copy option only appears for Hosted Feature Services. If you were copying an image service, then all it was doing is cloning the ArcGIS Online item, which itself was just a bookmark to the image service which lived on an instance of ArcGIS for Server somewhere. Here is an example. Copying that item will register the service in your own organization, but the image service itself still lives at http://imagery.arcgisonline.com/ArcGIS/rest/services/LandsatGLS/FalseColor/ImageServer.
Is there anything that could be done differently in the UI that would have helped you better understand what the copy option was doing?

Evan
Thanks for the info.  I’ve tested more and it appears that in any case where the item is a data file, it is white and can’t be selected for copy – so I should have known what I was attempting (move hosted data) was not possible – but this does make me ask why that can’t be done (or perhaps “when?”) because I think users will want it (as I did).  Since we charge people for hosting data, I’m sure there will be cases where users want to transfer data between accounts.
Other than that, I’d suggest
A)     Some kind of help or guide for why some items are white and can’t be selected,
B)     And perhaps a confirmation message for any copy stating what will be copied (along the lines of what you wrote below).
If I think of other suggestions I’ll write again
Thx
Cody B.
From: Evan Caldwell<EMAIL_ADDRESS>Sent: Wednesday, May 18, 2016 4:33 PM
To: Esri/ago-assistant<EMAIL_ADDRESS>Cc: Cody Benkelman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Esri/ago-assistant] "copy" does not actually copy data, only creates a link to original data (#134)
The full copy option only appears for Hosted Feature Services. If you were copying an image service, then all it was doing is cloning the ArcGIS Online item, which itself was just a bookmark to the image service which lived on an instance of ArcGIS for Server somewhere. Here is an examplehttp://www.arcgis.com/home/item.html?id=161bcbbee9a94c0a992c5bd5428b13dd. Copying that item will register the service in your own organization, but the image service itself still lives at http://imagery.arcgisonline.com/ArcGIS/rest/services/LandsatGLS/FalseColor/ImageServerhttps://urldefense.proofpoint.com/v2/url?u=http-3A__imagery.arcgisonline.com_ArcGIS_rest_services_LandsatGLS_FalseColor_ImageServer&d=CwMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=9E_LRxKdxKirFHUfedESWkk_Ncc-ldhpaEl1r1SArYc&s=wzt5VXWgEeMYgSEGZDyijQasJjfPatTBuCFNH1vopnc&e=.
Is there anything that could be done differently in the UI that would have helped you better understand what the copy option was doing?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ago-2Dassistant_issues_134-23issuecomment-2D220188730&d=CwMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=9E_LRxKdxKirFHUfedESWkk_Ncc-ldhpaEl1r1SArYc&s=yMPjGXeMpcLOHgcez_K-gyKs3Ans4ngp-IORHgQqxhg&e=

There is a lot of work to be done in intuitively explaining why some actions can't be worked with. Specifically, files aren't supported yet because they require a proxy to transfer the files between accounts and one of the goals is to keep this app 100% client side so that it can be easily deployed on networks.
I'm trying to plan out some moderate UI changes and I'll try to incorporate this as best I can. One thought is to have a "mode" selection for the type of copy: Copy a reference to the original vs Replicate this in my account.
This is great feedback. Keep it coming! 👍

Thanks Evan
I hope it’s helpful
Thanks for your feedback
Cody B
From: Evan Caldwell<EMAIL_ADDRESS>Sent: Thursday, May 19, 2016 2:04 PM
To: Esri/ago-assistant<EMAIL_ADDRESS>Cc: Cody Benkelman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Esri/ago-assistant] "copy" does not actually copy data, only creates a link to original data (#134)
There is a lot of work to be done in intuitively explaining why some actions can't be worked with. Specifically, files aren't supported yet because they require a proxy to transfer the files between accounts and one of the goals is to keep this app 100% client side so that it can be easily deployed on networks.
I'm trying to plan out some moderate UI changes and I'll try to incorporate this as best I can. One thought is to have a "mode" selection for the type of copy: Copy a reference to the original vs Replicate this in my account.
This is great feedback. Keep it coming! 👍
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ago-2Dassistant_issues_134-23issuecomment-2D220446513&d=CwMFaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=kvJsLKwtbJlle3QsDdW2VMIzrKOdrNkdxl67i5q7LW4&s=SkF9LHRFilFLqs5qt9lJmWBNYIw_wjv2eUTBG_PgM1c&e=

@CodyBenkelman are you ok with closing this issue?

Yes, this can be closed
Thanks
Cody B
From: Sam Libby<EMAIL_ADDRESS>Sent: Monday, June 19, 2017 12:04 PM
To: Esri/ago-assistant<EMAIL_ADDRESS>Cc: Cody Benkelman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Esri/ago-assistant] "copy" does not actually copy data, only creates a link to original data (#134)
@CodyBenkelmanhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_codybenkelman&d=DwMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=UZsKUYSHl_WH4yYOUOizGv8kHL4Oo9LOAAiQ4zg9b_o&s=y3z3C5beODoYgp8TMZACQYe7NiNHfquujaYs6SHUAqY&e= are you ok with closing this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ago-2Dassistant_issues_134-23issuecomment-2D309535984&d=DwMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=UZsKUYSHl_WH4yYOUOizGv8kHL4Oo9LOAAiQ4zg9b_o&s=en26k0ugYfgfX1189UdoNRPI5zVnAEIFWQPN1yIcM0g&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ADEeIxsSYCaPMOB86d41GTjrpxVSIxpAks5sFsY5gaJpZM4IhgZ1&d=DwMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=Yap0V0N_PRBXBZRDH4QoBmUI-748lHmGKn3LYkVffkc&m=UZsKUYSHl_WH4yYOUOizGv8kHL4Oo9LOAAiQ4zg9b_o&s=cVn9lYyYFiVrZVVa0_OmPNtOFMODNcYDmG6yLLvO-gg&e=.


WIP: Create Swift Package

Currently not building

I've managed to do it for swift here https://github.com/jkrukowski/SQLiteVec
The problem is that the version of SQLite on macOS does not support loading extensions that's why my package bundles the sqlite together with sqlite-vec.

@jkrukowski
I've started to try to make changes to GRDB which would allow for loading the SQLite-Vec extension. Basically what you have with your package, but with all the benefits of what GRDB has to offer.
https://github.com/groue/GRDB.swift/discussions/1591#discussioncomment-11425229
Any help or thoughts would be greatly appreciated


Implementing cupyx.scipy.signal.oaconvolve

This implements cupyx.scipy.signal.oaconvolve which uses the recently merged cupyx.scipy.signal.fftconvolve to do most of the work by breaking the larger data into smaller blocks to make the FFTs faster. Nearly all of the other logic comes from scipy with some simplifications.
A custom version of scipy's special.lambertw function is provided that only works for the situation needed to determine the block size (called _optimal_oa_block_size). This function is only called once per call to oaconvolve and takes a 1-2 microseconds in my tests, if necessary this can be moved to Cython which is 10x faster.

Additionally, if #4450 is addressed, cupyx.scipy.special.lambertw could be used instead, but it will unlikely be faster than this custom version which has parts optimized for the hardcoded values of z and k.

pfnCI, test this please.

Jenkins CI test (for commit 827b4eef2c6b0d89818f75ec426c1e18ce2210c8, target branch master) succeeded!

Would you add oaconvolve to docs/source/reference/signal.rst so we can generate its reference page?

LGTM except for the points!

Should now be ready to go! Thanks for the input!

Jenkins, test this please

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

LGTM, thanks!

LGTM, thanks!


Setting up a ServiceEntry for HTTPS/TLS with SNI fails with Istio on GKE

Describe the bug
Using the example config
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https
spec:
  hosts:
  - api.dropboxapi.com
  - www.googleapis.com
  - api.facebook.com
  location: MESH_EXTERNAL
  ports:
  - number: 443
    name: https
    protocol: TLS
  resolution: DNS

from https://istio.io/docs/reference/config/networking/v1alpha3/service-entry/, running curl https://www.googleapis.com fails with curl: (35) error:1408F10B:SSL routines:ssl3_get_record:wrong version number on Ubuntu 18.04 and with curl: (35) error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol on Ubuntu 14.04. The same command works when adding
  - number: 80
    name: http
    protocol: HTTP

to the ServiceEntry and using http:// instead of https://. Using protocol: HTTPS does not make a difference.
I understand that Istio won't be able to inspect the HTTP traffic; however, the documentation indicates that the above setup should allow detecting the remote host via SNI and then proxying the encrypted connection through the egress.
Expected behavior
curl to the https:// url should work as well.
Steps to reproduce the bug

Launch an Ubuntu 18.04 image on Kubernetes:

apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: xxx
  namespace: xxx
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: xxx
    spec:
      containers:
      - name: xxx
        image: ubuntu:18.04
        imagePullPolicy: Always
        command: ["/bin/sleep"]
        args: ["36000"]e

Version
version.BuildInfo{Version:"1.1.1", GitRevision:"2b1331886076df103179e3da5dc9077fed59c989", User:"root", Host:"7077232d-4c6c-11e9-813c-0a580a2c0506", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Clean", GitTag:"1.1.0-17-g2b13318"}
Error: error execing into istio-citadel-76f674f64-wskvg citadel container: command terminated with exit code 255

No idea why that command fails — according to the GKE dashboard, I am running Istio 1.0.6-gke.1. The error also occurred with Istio 1.0.3, though.
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.7", GitCommit:"65ecaf0671341311ce6aea0edab46ee69f65d59e", GitTreeState:"clean", BuildDate:"2019-01-24T19:32:00Z", GoVersion:"go1.10.7", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.6-gke.7", GitCommit:"aeaa96020ec0614a8773799058c3b8d58c19b9ff", GitTreeState:"clean", BuildDate:"2019-03-13T11:22:57Z", GoVersion:"go1.10.8b4", Compiler:"gc", Platform:"linux/amd64"}

Installation
Installed via Istio on GKE, with permissive mTLS. (I did have strict TLS enabled before, but have since disabled that again.)
Environment
Vanilla GKE with Container-Optimized OS, see above for the Kubernetes version.
Cluster state
(Partial dump attached.)
istio-dump.zip

/cc @rshriram @costinm

I've tested your config, and only replace add one more host with www.github.com, and I can access by curl https://www.github.com -ivk. But I can not access  api.dropboxapi.com. I know this is expected, because in my vm, I can not access api.dropboxapi.com
/ # curl https://api.dropboxapi.com/  -ivk
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to api.dropboxapi.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
  CApath: none
* TLSv1.2 (OUT), TLS handshake, Client hello (1):
* LibreSSL SSL_connect: SSL_ERROR_SYSCALL in connection to api.dropboxapi.com:443
* stopped the pause stream!
* Closing connection 0
curl: (35) LibreSSL SSL_connect: SSL_ERROR_SYSCALL in connection to api.dropboxapi.com:443

@hzxuzhonghu thank you for looking into this!
I've tried with a completely new cluster now, with the following config:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: xxx
  namespace: xxx
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: xxx
    spec:
      containers:
      - name: xxx
        image: ubuntu:18.04
        command: ["/bin/sleep"]
        args: ["36000"]
        imagePullPolicy: IfNotPresent
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https
spec:
  hosts:
  - fcm.googleapis.com
  location: MESH_EXTERNAL
  ports:
  - number: 80
    name: http
    protocol: HTTP
  - number: 443
    name: https
    protocol: HTTPS
  resolution: DNS

---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https2
spec:
  hosts:
  - heise.de
  location: MESH_EXTERNAL
  ports:
  - number: 80
    name: http
    protocol: HTTP
  - number: 443
    name: https
    protocol: HTTPS
  resolution: DNS

Now, curl https://fcm.googleapis.com works. However, curl -vv https://heise.de indicates that this connection is also routed to fcm.googleapis.com. Is it possible that I am simply affected by https://github.com/istio/istio/issues/9744? If so, what are my options to work around the issue, given that Istio on GKE does not yet support Istio 1.1? I can hardly believe that it should be impossible to connect to two different services via HTTPS.
(Listener config will follow in a minute.)

Unfortunately, I'm unable to extract the listener config:
$ istioctl pc listener xxx-89985d997-m2f8b.xxx --port=443 --address="<IP_ADDRESS>" -o json
Error: error unmarshalling config dump response from Envoy: json: cannot unmarshal object into Go value of type []json.RawMessage

Okay, that was due to an istioctl version mismatch. The full config is attached; looks like only the service entry for fcm.googleapis.com is applied.
istio-config.json.zip

Yeah, the two serviceEntry has duplicated port, so the first one take effect.

But even when I add both hosts to the first ServiceEntry, heise.de never gets added to the config. How am I supposed to connect to more than one HTTPS service outside my cluster?

On 29. Mar 2019, at 09:46, Zhonghu Xu<EMAIL_ADDRESS>wrote:
Yeah, the two serviceEntry has duplicated port, so the first one take effect.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

make protocol TLS in this case,  for other tcp protocol, one of the solution i can think of is to provide addresses in serviceentry.

@hzxuzhonghu I have created the following ServiceEntries now:
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https
spec:
  hosts:
  - fcm.googleapis.com
  - bing.com
  location: MESH_EXTERNAL
  ports:
  - number: 80
    name: http
    protocol: HTTP
  - number: 443
    name: https
    protocol: HTTPS
  resolution: DNS
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https2
spec:
  hosts:
  - heise.de
  location: MESH_EXTERNAL
  ports:
  - number: 80
    name: http
    protocol: HTTP
  - number: 443
    name: https
    protocol: HTTPS
  resolution: DNS
---
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-arstechnica
spec:
  hosts:
  - arstechnica.com
  location: MESH_EXTERNAL
  ports:
  - number: 80
    name: http
    protocol: HTTP
  - number: 443
    name: https
    protocol: TLS
  resolution: DNS

All four domains work via HTTP. So clearly I can have more than one ServiceEntry on the same port, provided the hosts are different.
However, all four domains point to fcm.googleapis.com when using HTTPS or TLS (see the entry for arstechnica.com). Also, the config only mentions fcm.googleapis.com, even though all four hosts work via HTTP.
The docs specifically state that at least fcm.googleapis.com and bing.com should work in this case. Also, looking up the IP addresses of each service I use (and which may change in the future) can't be the solution?

Note: if should be
  ports:
  - number: 443
    name: https
    protocol: TLS

https works as tcp

@hzxuzhonghu I have tried using both TLS and HTTPS in that field. Neither has worked.

Update:
I have now been able to connect to remote services with the following configuration:
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https
spec:
  hosts:
  - api.appfigures.com
  location: MESH_EXTERNAL
  resolution: NONE
  ports:
  - number: 443
    name: https
    protocol: TLS
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: tls-routing-https
spec:
  hosts:
  - api.appfigures.com
  tls:
  - match:
    - sniHosts:
      - api.appfigures.com
    route:
    - destination:
        host: api.appfigures.com

Initially, this did not work because I had exposed a Service based on GCP LoadBalancer on port 443 whose port name was grpc. Changing that port name to https changed the Envoy proxies to listen on that port with the TCP protocol instead of the HTTP protocol (as evidenced by istioctl proxy-config listeners istio-test-7f478f476c-qzrlf -n istio-test).
However, the docs still make it sound like a single config of type
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: external-svc-https
spec:
  hosts:
  - api.dropboxapi.com
  - www.googleapis.com
  - api.facebook.com
  location: MESH_EXTERNAL
  ports:
  - number: 443
    name: https
    protocol: TLS
  resolution: DNS

should be sufficient to access external services. That only works for the first service declared in that list; all others would also be routed to api.dropboxapi.com in this case even if the host www.googleapis.com is specified.
Am I missing something?

@MrMage We just ran into the same issue. From what I can see, the issue is that GKE only has Istio 1.0.6 and you need 1.1+ for the single ServiceEntry (no VirtualService) configuration.

This should be fixed in 1.1, please try upgrading if possible as we are no longer supporting 1.0. If you still run into this issue feel free to open a new one/reopen

I am facing exactly this issue again in Istio 1.8.2.

I am facing exactly this issue again in Istio 1.8.2.

@iris-garcia please open another issue, this one is almost 1.5 years old and unlikely the same root cause.

@iris-garcia please open another issue, this one is almost 1.5 years old and unlikely the same root cause.


When the return value defined by the @ Success tag is equal to a null value, a null pointer exception occurs

Describe the bug
When the return value defined by the @ Success tag is equal to a null value, a null pointer exception occurs
To Reproduce
Steps to reproduce the behavior:

Response {data=nil} in the structure

Run the command swag init in cmd

Invalid memory address or nil pointer dereference discovered through debugging

Expected behavior
Can swaggo parse comments in the returned structure that are equal to nil, such as utils. Response {data=nil}
Expected swaggo doc definition
// deleteFile 删除文件
//
//	@Summary		删除文件
//	@Description	删除文件
//	@Tags			member
//	@Accept			json
//	@Produce		json
//	@Param			fileIds	query		string	true	"文件ID集合字符串"
//	@Success		200		{object}	utils.Response{data=nil}
//	@Failure		400		{object}	utils.ErrorResponse
//	@Router			/app/deleteFile [POST]
func deleteFile(c *gin.Context) {
}

Expected JSON return value
{
   "status": 0,
   "message": "success",
   "data": null
}

Your swag version
v1.16.1
Your go version
go1.20.5
Desktop (please complete the following information):

OS: [windows 10 19045.3324]

May I ask why need null?

The project coexists with the Springboot project and requires a unified JSON return value

What do you think of the solution in #1667 ?

@sdghchj
What do you think of the solution in #1667 ?


SQL Syntax Highlighting not working when escaping strings

VSCode Version: Version 1.21.0-insider (1.21.0-insider)
OS Version: macOS 10.13.3

Steps to Reproduce:
$ code-insiders --disable-extensions <my-project>

Does this issue occur when all extensions are disabled?: Yes
Locked issue: #39822

duplicate of https://github.com/Microsoft/vscode-mssql/issues/955


Separate Linux builds/packages and some refactoring.

Accidentally closed other PR and couldn't reopen :door:.
Fixes #894

So the proper syntax will be
npm run package:linux:32

for 32-bit and
npm run package:linux:64

for 64-bit?

Sorry, I thoroughly brutalized this PR with conflicts.  I took the base logic and pushed it to master after some refactoring


Fix some small docs issues

These aren't rendering correctly: https://pumasai.github.io/SimpleChains.jl/dev/

Thanks!


Interpret optional extras in pyproject.toml as pixi features

Problem description
#999 will bring in initial pyproject.toml support.
I propose to interpret groups of extras dependencies in [project.optional-dependencies] as pixi features specifying some pypi dependencies.
Each of these could then be mapped to a pixi environment, depending on one or several features (a group of extra dependencies can contain a reference to another group of optional dependencies of the package itself).
I think it would make sense to create a tool.pixi.environments table during init.
A couple of questions:

Should that table be amended automatically after init, to propagate changes in [project.optional-dependencies]? I would assume not as one may not want to generate an environment per group of extra. Another option would be to keep it in sync, but for a list of excluded extras
Should we define a single solve group across all environments? I would assume so, as I think it maps better with how the groups of extra dependencies are being used in python.

So for instance:
[project]
name = 'my-package'
dependencies = [
    # ...
]

[project.optional-dependencies]
# development dependency groups
extra1 = [
    # ...
]

# feature dependency groups
extra2 = [
    # ...
]
extra3 = [
    "my-package[extra2]", # could also be .[extra2]
    # ...
]

Would inject upon pixi init the following table
[tool.pixi.environments]
extra1 = {features = ["extra1"], solve-group = "default"}
extra2 = {features = ["extra2"], solve-group = "default"}
extra3 = {features = ["extra3","extra2"], solve-group = "default"}

I'll be happy to implement this once #999 is merged in.

@ruben-arts the PR for this is in. I am not 100% sure, but I believe that being able to lock all the extra groups consistently like this is pretty unique to pixi :-)


set debounceTime on empty observable

Bug Report
Current Behavior
The debounceTime operator is not working on empty obervable. I tested on other operators such as delay and it is perfectly fine.
when I repeatedly subscribe with debounceTime(x)
the complete function just executes reapeatly and immedietly.There is no "debounce" effect.
Reproduction

REPL or Repo link:
https://stackblitz.com/edit/angular-8gdsau

  debouceEmpty = empty().pipe(debounceTime(800));
  delayEmpty = empty().pipe(delay(1500));

  testDelay() {
    this.delayEmpty.subscribe({
      complete: () => console.log('delay Complete!')
    });
  }

  testDebounceTime() {
    this.debouceEmpty.subscribe({
      complete: () => console.log('debounceTime Complete!')
    });
  }

Expected behavior
It should wait 't' amount time before executing the complete function.
Environment

Runtime: window 10,  chrome  69, angular 6.2
RxJS version: 6.3.3

Related with https://github.com/ReactiveX/rxjs/issues/4249 - this is more about behavior of delay. There are some inconsistent semantics around completion notification delay. In case of debouncetime, it delays value, not completion.

Related with #4249 - this is more about behavior of delay. There are some inconsistent semantics around completion notification delay. In case of debouncetime, it delays value, not completion.

Any idea ? How can I use denounce on void without return value?

what usecases you'd like to delay completion via debounce on empty stream?

Can't say for sure without seeing full flows, but that doesn't sound like jobs of debounce operator of rx. If you'd like debounce event emission itself you'd need to convert event handler into observable, otherwise if you'd like function to be debounded that's not scope of rx. maybe _.debounce or something else is what you'd looking for instead.


Feature/Question: consume a batch of messages

I'm not sure if this is already possible, but I'm wondering if there is a way to consume a batch of messages off a topic (I'm working with Kafka).
I want to consume either 5 minutes of messages or 50,000 messages (whichever comes first) and then process that collection of messages as a batch.
Right now, each handler is executed once for each message so I don't see a straightforward way to pull multiple messages in one function.
Is there a way to do this currently? If not, do you think this is something that could be possible?

@chrisgoddard sorry, but in the current implementation there is no way to consume batches
But you can find this functionality in the V2, just wait 2 weeks for release

It would be great to add support for Redis-Streams too.

@chrisgoddard please, check FastStream for this feature
Tomorrow will be a great release
All your Propan code should works with FastStream correctly: just migrate @handle to @subscriber and change some imports paths


make rviz related packages relocatable

Atm various .cmake files installed by rviz packages and its dependencies contain absolute paths to the install space which makes them non-relocatable. See:

https://github.com/ros2/ros2/issues/915#issuecomment-623552072
https://github.com/ros2/ros2/issues/904

Reviewing the latest packaging jobs I think resolving this for all the rviz dependencies has also resolve this for rviz itself.


Subtitle Under A Button

It would be useful if we could add a subtitle under each button (optional).
Thanks

Hi, thanks for the suggestion! I'm not planning on adding this feature to AHKActionSheet, though, as it's quite complicated as it is.
Since this project is open source, I encourage you to fork it and make your own version of it :smiley:!


Don't throw exceptions because of uncaught rejection

This is a feature request - or better: a request to remove a feature.
I'm using zone.js 0.6.12 installed via npm.
By throwing an exception because queue.length == 0 && state == REJECTED every test fails where someone is not writing promise.catch(() => {}) or promise.then(() => {}, () => {}). It makes the code more ugly and is not helpful.
At least you should be able to disable this feature (inform about uncaught rejections).
Let's assume this code:
export class Simple {
  private i:number = 0;
  public function setIncrement(i:number):void { this.i = i; }
  public function make():Promise<void> {
    return new Promise((resolve, reject) => {
      if (this.is < 10) { resolve(); }
      else { reject('increment to high'); }
    };
  }
}

One from this tests fails but the code works fine:
import {fakeAsync, flushMicrotasks} from '@angular/core/testing';
import {Simple} from '../simple';

describe('simple', () => {
  var simple;
  beforeEach(() => {
    simple = new Simple();
  });
  it('resolves', fakeAsync(() => {
    simple.setIncrement(3);

    var promise = simple.make();

    expect(promise['__zone_symbol__state']).toBe(true);
  }));
  it('rejects', fakeAsync(() => {
    simple.setIncrement(11);

    var promise = simple.make();

    expect(promise['__zone_symbol__state']).toBe(false);
  }));
});

To reproduce:
> Promise
< function ZoneAwarePromise()

> Promise.reject().then(() => {}, () => {})
< Object { __zone_symbol__state: null, __zone_symbol__value: Array[0] }

> Promise.reject().then(() => {})
< Object { __zone_symbol__state: null, __zone_symbol__value: Array[0] }
Object { __zone_symbol__error: Error, fileName: Getter, lineNumber: Getter, columnNumber: Getter, message: Getter, name: Getter, stack: Getter, originalStack: Getter, zoneAwareStack: Getter, toString: [524]/</</createMethodProperty/props[key].value(), 5 more… }

This feature provides positive value when tracking unhandled promises. However, there are cases where you want to store a promise regardless of its status and process the outcome later. If this promise is then rejected, a massive stacktrace is printed. This gets very prominent in tests when all kinds of behaviour is tested.
As a work around in our tests we are doing this right now:
spyOn(console, 'error').and.callFake(() => { console.log('Suppressed call to console.error'); });

Now zone.js has provide unhandledpromiserejectionhandler

Browser
use window.unhandledrejection

nodejs
use unhandledRejection

Can this meet your requirement?

Thanks for the pointer. I am digging through the source code but fail to see any quick explanation on how to use this.

Here is the usage
https://github.com/angular/zone.js/pull/627

If I understand correctly that would be the following?
Zone[Zone.__symbol__('ignoreConsoleErrorUncaughtError')] = true;

That's an internal API likely to change in the future though, right?

I like @cburgmer  answer the most. But anyway:
Promise.reject('This is not failing at all');

should IMHO not cause an error in the console. The default value for this setting should be true.

@tflori , now  you can set
Zone[Zone.__symbol__('ignoreConsoleErrorUncaughtError')] = true;

to ignore the console output.

yes, I understand that.
for me it is a bug. without zonejs it does not throw an error. with zonejs it throws. but as i understand zones it is just an execution zone and not a "you have to do it correct" zone.
and therefore the symbol should be true by default.
if you don't think so, you can close this issue. it's solved with this symbol.

@tflori , by default it is true because it is helpful for debug, and BTW the symbol only control to output the error information to console or not. Without the symbol, the error will just be output to console.
I tried the code in nodejs without zone.js, it will also output warning, and in the future it will not be warning but an error.

onst promise = new Promise((resolve, reject) => {
  setTimeout(() => {
   reject('fail');
}, 0);
});

setTimeout(() => {
   console.log('promise', promise);
}, 100);

(node:30630) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): fail
(node:30630) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled w
ill terminate the Node.js process with a non-zero exit code.

wow, ok. where to discuss about es6 :smile:?
it feels wrong for me. because it means I don't have the option to ignore the error. on a bash for example when I don't care about non zero exit codes I write ; instead of &&.
but that's a es6 discussion then. and thanks for the info