Use https remote in github repos

In order to apply github authentication the remote needs to be https:// instead of git://. Context in https://github.com/src-d/lookout/pull/326.

I'm a little bit concerned about this change. It looks like a hack tbh.
At the same time I understand the need for it.
@smola what do you think?

Ok, it's fair enough concern. Let me give some context.
The go-vcsurl parse method builds the clone url as git://github.com/user/repo.git
https://github.com/sourcegraph/go-vcsurl/blob/v1/vcsurl.go#L96
GitHub docs say https is the recommended remote url:

Cloning with HTTPS URLs (recommended)
The https:// clone URLs are available on all repositories, public and private.

I'd say using https://github.com/user/repo.git is just as hack or as robust as what go-vcsurl.Parse is doing.

Oh. And go-vcsurl is archived repository. What do you think about forking it?

AFAIK git:// can be faster on public repositories and https:// is just required for private repositories if using auth tokens.
With respect vcsurl are we actually using its output? I'm I bit worried that we are doing this:
type RepositoryInfo = vcsurl.RepoInfo

attaching our gRPC protocol to symbol defined in an external+deprecated library

Pinging @mcuadros to join the conversation. Since you left this TODO and maybe you had some ideas how to replace vcsurl.
https://github.com/src-d/lookout-sdk/blob/e06a088af4fe40561700c057c5dacd46e23b2abe/pb/event.go#L92

Reasons to remove vcsurl:

external & deprecated
containes only few special cases for some hosts. For example it handles gitlab incorrectly.
for some providers username is empty but we rely on it in our code
supports bzr, hg which we don't
doesn't really check schema git, https, http, wrong all are valid. In our case, we can restrict only to https.

We can replace it with 1 simple function: https://play.golang.org/p/CVQ8Qa1XoQR
it's more opinionated (for example it assumes always git) but resolves all our issues. We can improve it later if needed but it don't think we will need it any time soon. Currently, we work only with github.
I propose to replace it with func linked above here in sdk and remove workaround from lookout.

LGTM :+1:
But for the final implementation I'd add a check for the Host to be a known one before doing the split.

Yes. My reasoning is that if we add support for something else, it will probably be better to have a bug because of empty owner and name than whatever happens to end in those fields.

superseded by https://github.com/src-d/lookout-sdk/pull/26


Help command and flag

Unnecessary help command given that a help flag exists.

Help command is default with the cobra package


feat: Vanessa shop, farmer dialogues

What has been done?

Adds Vanessa's shop
Adds Vanessa's dialogue
Adds missing farmer and leprechaun spawn to Catherby
Adds farmer dialogues
Adds possibility to protect allotments through paying farmers
In theory adds produce noting by leprechauns, though item-on-npc in general doesn't seem to work yet

Has your code been documented?
It's mostly dialogue, so no

Alot of good stuff here Paleocene! Looks amazing!


Matlab's inputParser for CheckOptions?

Matlab has a builtin input parser.  Should that be incorporated into CheckOptions?
https://www.mathworks.com/help/matlab/matlab_prog/input-parser-validation-functions.html

No. dsCheckOptions() performs the same function for DynaSim-relevant cases but is simpler to use.


Add force-merge option for adapters

I've got a workflow where I'm adapting a preview1 component to preview2, where that component itself has world bindings to preview2 as well. The version APIs exactly match (so there should be type equality), but I'm getting the error:
     Error: failed to merge WIT world of adapter `wasi_snapshot_preview1` into main package

Caused by:
    duplicate import found for interface `environment`

This change adds a force_merge option to the adapter to inform that imports are known to be equivalent.

Turned out I was using bad binary versions - so this is unnecessary after all.

I was about to say, every time I've hit this error it was from stale binaries


[TUTORIAL] CRUD using PersistentVector and multiple React views

on Discord @htafolla requested this:

I think a step-by-step CRUD example using PersistentVector and multiple React views would be great! Learning how to deal with state across components in React is nagging

We have CRUD tutorial now: https://docs.near.org/docs/tutorials/apps/todos-crud-app


[Suggestion] Command for Fishing Rod Strings

Using Modtweaker you can modify fishing loot for Mariculture. I am not quite fond on how it works yet but all I know is that it needs fishing rod strings.
These were listed in the code:
static {
rodTypes.put("net", RodType.NET);
rodTypes.put("old", RodType.OLD);
rodTypes.put("good", RodType.GOOD);
rodTypes.put("dire", RodType.DIRE);
rodTypes.put("super", RodType.SUPER);
rodTypes.put("flux", RodType.FLUX);
if (Loader.isModLoaded("AWWayofTime")) {
try {
rodTypes.put("blood", (RodType) ReflectionHelper.getStaticObject(Class.forName("mariculture.plugins.PluginBloodMagic"), "BLOOD"));
} catch (ClassNotFoundException e) {
e.printStackTrace();
As they always could change, I think it would be appropriate to have a command like /mt fishingRods or something similar.

Will document it myself :'D


Can't find microsoft's C/C++ Extension in marketplace

I am unable to search and install microsoft's C/C++ extension on Archlinux's code package from official repo.
Searching for "C/C++" gives other results but not the microsoft one.

Searching for @id:ms-vscode.cpptools says "No extensions found".

Searching for ms-vscode.cpptools shows only Language Packs.

VS Code version: Code - OSS 1.49.3 (2af051012b66169dde0c4dfae3f5ef48f787ff69, 2020-10-07T20:28:07.702Z)
OS version: Linux x64 5.8.14-arch1-1
Steps to Reproduce:

Open Code - OSS
Goto Extensions
Search for C++ extension.

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

Duplicate of https://github.com/microsoft/vscode/issues/107997#issuecomment-703245178

Looks like it is related to ArchLinux changing code-oss's marketplace to Open VSX.
The workaround mentioned in https://github.com/microsoft/vscode/issues/107997#issuecomment-703252446 works.


Error 404/429

I have followed the installation instructions, added my api key and tried referencing a few different models.  When using the default gpt-4 model, i receive a 404 error inside of eclipse.  When i change it to gpt-3.5-turbo, the error changes to a 429 which translates to an api limit i believe.    Is there an install step that i have missed?
I have seen the closed issue regarding 404, and i dont believe that i have the same issue. my OpenAI account looks to be part of my "personal" organization.

I think this has to do with my account's free trial being expired.  I will confirm and return here to close this issue.

Hi, I think this may be an issue with your OpenAI account. See this link:
https://help.openai.com/en/articles/6891827-error-code-404-you-must-be-a-member-of-an-organization-to-use-the-api
Cheers!
śr., 31 maj 2023, 19:51 użytkownik Mason @.***> napisał:

I think this has to do with my account's free trial being expired. I will
confirm and return here to close this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/gradusnikov/eclipse-chatgpt-plugin/issues/4#issuecomment-1570661186,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAECES4NUAEGEUPXOR3T3G3XI6ALRANCNFSM6AAAAAAYVYV65M
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I am also facing the same issue . It is giving 429 error code when using Model Name = gpt-3.5-turbo. My openapi account is part of "personal" organization .
Please let me know if you have succeed to fix this issue . Thanks in advance.
Regards.

I have recently upgraded my chatgpt account to chatgpt Pro, it is newly contracted and I have not spent any credits or limits.
I have installed the addon in Eclipse Version: 2024-09 (4.33.0)
Build id: 20240905-0614 . Added the corresponding API granting all the necessary permissions. As APIurl I have added: https://api.openai.com/v1/chat/completions. I have used a Java script to check the connection and it tells me a Connection Error: 429 - Too many requests. Waiting to retry... Any solution? Thanks in advance


nixdoc: init at 1.0.1

Motivation for this change
Adds nixdoc, a tool to generate documentation for Nix functions in the
standard library.
See #49275 for some background information.
Note that the derivation used is generated by carnix and rather large, as there is no Rust package set in nixpkgs. This is similar to the derivation used for carnix itself, and way-cooler.
cc: @grahamc @jD91mZM2 @joachifm
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@Grahamcofborg build nixdoc

Success on aarch64-linux (full log)
Attempted: nixdoc
Partial log (click to expand)
Running rustc --crate-name nixdoc src/main.rs --crate-type bin -C opt-level=3 -C codegen-units=1 --out-dir target/bin --emit=dep-info,link -L dependency=target/deps -L native=/nix/store/vxvhnmrgb54ld1lrw261cy173jyy02c1-rust_backtrace-sys-0.1.24/lib/backtrace-sys.out -l static=backtrace --extern failure=/nix/store/z1pa1ydkijikw6akzs6k4pi6r08yvk9i-rust_failure-0.1.3/lib/libfailure-e8bec5c242.rlib --extern rnix=/nix/store/v6nys1wfn2bw7qiqngcc1kkmnc9cis85-rust_rnix-0.4.1/lib/librnix-44636a2f86.rlib --extern structopt=/nix/store/lks6lg1nfb334bb90zjs143xc87v3v54-rust_structopt-0.2.12/lib/libstructopt-6cdd5a2706.rlib --extern xml=/nix/store/mh0z8a36nfkd64ngpwgz8mpld0ipqmv9-rust_xml-rs-0.8.0/lib/libxml-50e7b84675.rlib --cap-lints allow -L native=/nix/store/vxvhnmrgb54ld1lrw261cy173jyy02c1-rust_backtrace-sys-0.1.24/lib/backtrace-sys.out -l static=backtrace --color always
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1
shrinking /nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1/bin/nixdoc
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1/lib  /nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1/bin
patching script interpreter paths in /nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1
checking for references to /build in /nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1...
/nix/store/cq440dq9h8h45bd3s51s4b5p53y2j2x4-nixdoc-1.0.1

Timed out, unknown build status on x86_64-linux (full log)
Attempted: nixdoc
Partial log (click to expand)

trying https://crates.io/api/v1/crates/xml-rs/0.8.0/download
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 50974  100 50974    0     0   103k      0 --:--:-- --:--:-- --:--:--  103k
unpacking source archive /build/xml-rs-0.8.0.tar.gz
cannot build derivation '/nix/store/2m3sxhzqss6dxv9ifs8v02walw44jrv2-rust_xml-rs-0.8.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/abrzkqz0c6fvdy19368mlfl06nz5j380-nixdoc-1.0.1.drv': 29 dependencies couldn't be built
error: build of '/nix/store/abrzkqz0c6fvdy19368mlfl06nz5j380-nixdoc-1.0.1.drv' failed

I have changed it back to buildRustPackage to reduce the expression size (25 lines instead of 866 lines) https://github.com/Mic92/nixpkgs/commit/064f35037aaa007487a26637c18a45edcbee6759

Timed out, unknown build status on x86_64-darwin (full log)
Attempted: nixdoc
Partial log (click to expand)
cannot build derivation '/nix/store/7zmhhqawjvxa72phpqlvzpds1rg5s98h-rust_quote-0.6.8.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/ip2c90aawj499xzma8gh3rz6k1ixvg7z-rust_syn-0.15.15.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/5s9vql1ya88gchww64fdr3gy7yqfs8p7-rust_structopt-derive-0.2.12.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/mfhxzn1yvq0bdwssc1d4zzgfdm6f4c2g-rust_synstructure-0.10.1.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/20pc6pdypk5pknm0gr3x30a8xc1h1qfp-rust_failure_derive-0.1.3.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/k13g4fhkaq9m0x8shwc5353d1ciw46i6-rust_structopt-0.2.12.drv': 15 dependencies couldn't be built
cannot build derivation '/nix/store/myafi67zvdddbix9jva17b2xzh23pzgs-rust_failure-0.1.3.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/4s8dwd5v4wp0rvn8978kh9l7b2m7j4kp-rust_rnix-0.4.1.drv': 16 dependencies couldn't be built
cannot build derivation '/nix/store/0c5lvbv3012cl2hgwhi6xc9fcg3kccqd-nixdoc-1.0.1.drv': 28 dependencies couldn't be built
error: build of '/nix/store/0c5lvbv3012cl2hgwhi6xc9fcg3kccqd-nixdoc-1.0.1.drv' failed

@Mic92 Works for me! It looks like Hydra doesn't currently have rustc cached, so we'll have to ask for a build at some later time.

I prefer https://github.com/NixOS/nixpkgs/pull/49275

@tazjin I was confused by the UI. It looked like your other pull request contained my commit already, but It was just a reference.

https://github.com/NixOS/nixpkgs/pull/49358


Bug: Not sure how to fix this error message

Describe The Bug
"SwitchBot ERROR"secret" config is not populated, you must populate then please restart Bridge."
To Reproduce
error log
Expected behavior
unsure
Relevant log output
switchbot bridge stopping
10/18/2022, 8:04:12 AM Switchbot Bridge starting
10/18/2022, 8:04:40 AM Switchbot Bridge Loaded plugin '@switchbot/homebridge-switchbot'
10/18/2022, 8:04:40 AM Switchbot Bridge Loading 1 platforms...
10/18/2022, 8:04:40 AM Switchbot Bridge SwitchBot WARNING This plugin has been updated to use OpenAPI v1.1, config is set with openToken, "openToken" cconfig has been moved to the "token" config
10/18/2022, 8:04:40 AM Switchbot Bridge SwitchBot ERROR"secret" config is not populated, you must populate then please restart Bridge.
10/18/2022, 8:04:41 AM Switchbot Bridge Bridge is running on port 51856.
10/18/2022, 8:23:30 AM Switchbot Bridge Configuration change
10/18/2022, 8:23:30 AM switchbotbridge stopping
10/18/2022, 8:23:36 AM Switchbot Bridge starting
10/18/2022, 8:24:02 AM Switchbot Bridge Loaded plugin '@switchbot/homebridge-switchbot'
10/18/2022, 8:24:02 AM Switchbot Bridge Loading 1 platforms...
10/18/2022, 8:24:02 AM Switchbot Bridge SwitchBot WARNING This plugin has been updated to use OpenAPI v1.1, config is set with openToken, "openToken" cconfig has been moved to the "token" config
10/18/2022, 8:24:02 AM Switchbot Bridge SwitchBot ERROR"secret" config is not populated, you must populate then please restart Bridge.
10/18/2022, 8:24:03 AM Switchbot Bridge Bridge is running on port 51856.

Config for homebridge-switchbot
not available at this time

Screenshots
No response
Device & Model
unknown
Node.js Version
unknown
NPM Version
unknown
Homebridge Version
unknown
Homebridge Switchbot Plugin Version
unknown
Homebridge Config UI X Plugin Version
unknown
Operating System
raspberry

Can't tell what your config looks like since you didn't post it, but my guess is that you don't have the new Secret config set. You will have to get that from the SwitchBot app, same way you go the token, and paste in the config.

I see where to put the token, but not the Secret?  it wasn't very clear, sorry

This plugin isn't developed for HOOBs.
I would recommend moving to homebridge.

Damn, it shows as certified,   that's too much work to switch as everything works fine right now except this. .. :)  thanks anyway

@gregpa, HOOBs puts that on there, I have no control over that.
It would be better to switch now then wait.


Reuse resolution

What changes are proposed in this pull request?
Reuse resolution.
Why are the changes needed?
Improve the performance.

@HelloHorizon FYI

@Haoning-Sun Would you like to give some reason why this is necessary? Is the resolution expensive?


add a more strict linter

We have strings with both " and '. I'd like the repo to have a more uniform coding style, tbh.
Thoughts, @phillipj?

Sure, I'm also a '-fan :+1:
JSCS has gotten a lot of traction lately. In fact the codestyle rules in JSHint is being deprecated these days in favor of JSCS, so the quotmark option in JSHint will dissapear sometime soon..

check out eslint. after trying all of 'em, that's where several projects i work on have ended up.

Yeah eslint also does the trick.. Anything other than quotes you had in mind @dasilvacontin?
Although I feel most linters do the basic stuff equally well, some are more suiteable for specific things and custom styles.

Glad you guys mentioned eslint, it's the one I had in mind. :)
More things:
Writer.prototype._rawValue = function(token) {
vs
mustache.parse = function (template, tags) {
What are your thoughts about standard?

My thoughts about the differences:
I like hinting about how "public" my functions are. Therefore I often tend to add an _ for "private" and usually very specific functions, e.g.  function _somewhatPrivate(){}
I dont have a strong preference about spaces in function declarations. Looking at the source atm, there's mostly spaces function () {}, which means I have to fix my wrongly formatted Writer functions :smiley:
My two main reason for choosing the prototype-style today: better stack traces and memory profiling. Prototypes also helps raw performance, but that is usually not a big concern before running hot code which executes alot.

I dont have a strong preference about spaces in function declarations.

I don't have a preference about either style, but I like them to look the same. Will merge PR for those changes.  Therefore I often tend to add an _ for "private" and usually very specific functions, e.g. function _somewhatPrivate(){}

I usually just note it in the function's documentation and that's all.
Also, I like to name anonymous functions. Better stacktraces, as you mentioned, and more contribution-friendly code. :)

Btw, #412 has fixed alot of mixed styles.. Personally havent had any experience with google closure compiler though, ref my question about the benefits compared to jshint (or eslint for that matter)

I agree, and I would merge it if it didn't have conflicts.We could try adding eslint after I merge my v2.0 PR.


Improve documentation for shell completion

What this PR does / why we need it
This PR updates the documentation to better explain autocompletion:

update the quickstart documentation
update the main README.md
update the README.md for contributing to the CLI
update the README.md for plugin developers
update the help message for tanzu completion -h to explain how to deal with aliases or binary renaming

The PR takes the opportunity to do a bit of cleanup related to shell completion:

move a couple of functions to the completion_helper.go file
provide a constant for the "TANZU_ACTIVE_HELP" variable

Which issue(s) this PR fixes

Fixes # N/A
Describe testing done for PR
CI
Release note

Improve documentation for autocompletion and ActiveHelp.

Additional information
Special notes for your reviewer

CI failure will be resolved by #580


Exception thrown when clearing the controller of a TextField

This is a follow-up to #36065
Steps to Reproduce
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  TextEditingController _controller = TextEditingController();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: TextFormField(
          controller: _controller,
          decoration: InputDecoration(
            suffixIcon: IconButton(
              icon: Icon(Icons.clear),
              onPressed: () => _controller.text = "",
            )
          )
        )
      )
    );
  }
}

Run app
Enter text into the TextFormField
Click the "x" icon.

Expected results: The TextFromField should be empty
Actual results: The following exception is thrown:
The following assertion was thrown while handling a gesture:
invalid text selection: TextSelection(baseOffset: 10, extentOffset: 10, affinity: TextAffinity.upstream, isDirectional: false)

When the exception was thrown, this was the stack:
#0      TextEditingController.selection= (package:flutter/src/widgets/editable_text.dart:193:7)
#1      EditableTextState._handleSelectionChanged (package:flutter/src/widgets/editable_text.dart:1485:23)
#2      RenderEditable._handleSelectionChange (package:flutter/src/rendering/editable.dart:417:7)
#3      RenderEditable.selectPositionAt (package:flutter/src/rendering/editable.dart:1568:5)
#4      RenderEditable.selectPosition (package:flutter/src/rendering/editable.dart:1539:5)
...
Handler: "onTapUp"
Recognizer: _TransparentTapGestureRecognizer#48765
  debugOwner: _TextSelectionGestureDetectorState#bd4cc
  state: ready
  won arena
  finalPosition: Offset(368.3, 378.9)
  finalLocalPosition: Offset(368.3, 23.2)
  button: 1
  sent tap down

That being said, the TextFormField is empty after all.
Flutter analyze
No issues found!

flutter doctor -v
[√] Flutter (Channel stable, v1.12.13+hotfix.9, on Microsoft Windows [Version 10.0.18363.752], locale de-DE)
    • Flutter version 1.12.13+hotfix.9 at C:\Development\devTools\flutter-sdk
    • Framework revision f139b11009 (3 days ago), 2020-03-30 13:57:30 -0700
    • Engine revision af51afceb8
    • Dart version 2.7.2

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at C:\Users\Julian\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] VS Code (version 1.43.2)
    • VS Code at C:\Users\Julian\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.5.0

[√] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)

• No issues found!

Hi @julianbraasch
It is fixed here https://github.com/flutter/flutter/issues/35848,
consider to use another channel - beta or dev, the fix is already in there, but not in stable yet
Closing, if you disagree please write in the comments and I will reopen it
Thank you


OPTION configure not showing in mod's menu

It did't show the configure button in the mod menu. Can someone help?

Same thing for me, no configuration button for this mod :

Minecraft 1.19.3
Fabric API 0.73.0+1.19.3
Mod Menu 5.0.2
Twitch Chat Bridge 0.14.1b

twitch_chat.error.log

Are you both using Minecraft 1.19.3?

Could you check if this version works for you?
twitchchat-0.14.2b.jar.zip

Great! I'll release this version as a fix.
I hope it works well in your livestream!


fix redis dependencies

The latest version if node redis has big breaking changes.
We need to support it or switch to an alternative such as ioredis.
see also https://github.com/day1co/fastcache/issues/5

https://github.com/day1co/fastbus/pull/9 에서 해결되었습니다


[trel] Update errorCode when sending packet over disabled trel radio

When trel is disabled intentionally, sending broadcast packets can cause log entries of
INFO: [I] Mac-----------: Frame tx failed, error:Abort, len:73, seqnum:213, type:Data, src:1234567887654321, dst:0xffff, sec:no, ackreq:no, radio:trel
The error code Abort is used both in this case and other cases e.g. when a trel peer cannot be found. Changing the error code to InvalidState to differentiate with other Abort cases.

In general, I am not sure changing the error code from "TxDone" callback (for TREL to update emitted logs) is a good idea. It may have other unintended side-effects.
We require an explicit set of error codes that we allow and expect  in "TxDone" callback. This is documented in the otPlatRadioTxDone():
https://github.com/openthread/openthread/blob/fc307401a6aee332b36a94816033c8884ed283c5/include/openthread/platform/radio.h#L880-L896
The background/reason for this is that we want to limit the error so that behavior/action taken on each is clear. We have issues in past where some radio platforms returned other errors.
I know we check this in SubMac and assert if error is not one allowed ones:
https://github.com/openthread/openthread/blob/fc307401a6aee332b36a94816033c8884ed283c5/src/core/mac/sub_mac.cpp#L595-L626

Submitted https://github.com/openthread/openthread/pull/9522 as a different solution which skips logging trel frame tx error when trel interface is disabled.

Superseded by #9522


ORC-628. Add tool to count number of rows in ORC files under a directory

I found that orc-tools uber jar has been broken due to ORC-599 and make a PR to fix it. Could you review that, please?

https://github.com/apache/orc/pull/507

Sorry about that! I wrote and tested the patch on branch 1.5, so it worked. Thanks for fixing master.

Thank you, @omalley .


unsupervised adaptation of model

Hi, great work!
I am very curious about the unsupervised adaptation.
Unfortunately, the supplementary material you mentioned is not available in the paper(v3) in arxiv.
It would very nice if you could share the supplementary part. Looking forward to the implementation as well.

Details about unsupervised training can be found here:
https://openaccess.thecvf.com/content/ICCV2021/supplemental/Luo_Score-Based_Point_Cloud_ICCV_2021_supplemental.pdf
The code is here:
https://github.com/luost26/score-denoise/blob/e5fc1fe0c5cfd1713271044b3df3274d561035ad/models/denoise.py#L81


Support for faceted NRQL

Added support to the QueryData object for faceted nrql results.

Thanks for this. I should be able to look at it tomorrow.


Display verification status as a badge atop the user profile icon

We should provide a visual indicator to the user whether or not their current session has been verified, which is important because you cannot view E2EE messages without your device session being fully verified.
Design suggestion
The best place to display the user's verification status is as a small badge icon overlaid atop the user's profile picture. Currently the user's profile picture is just a placeholder (the gray "U" circle on the top left of the window), as the user profile/settings page hasn't yet been implemented.

We should place the badge here on the top-right corner of the user profile picture), like so:

Note that we also need to handle the case for the mobile UI, when the tab bar is at the bottom:

Then, when the user hovers over the badge/profile picture, we should display a tooltip that states the verification status, which is either "Verification unknown", "Unverified", or "Verified".
Implementation
Robrix already implements a way to receive a stream of updates about the user's verification state, it just doesn't do anything yet. This is basically a placeholder for this issue. See the code here: https://github.com/project-robius/robrix/blob/185459f8cfb6443e38c57149c6d7afd796d770e8/src/verification.rs#L20-L30
And then for the three possible VerificationStates, use icons like this:

Unknown: a gray shield with either a question mark icon (or no icon) within it

icon suggestion

Unverified: a red shield with an exclamation mark icon within it

icon suggestion

Verified: a green shield with a checkmark icon within it

icon suggestion

Once the state transitions to Verified, we can break out of the loop and allow the async task to exit normally, since there is no reason to continue waiting on updates once we know we're verified.

I am looking at this.

Closed by #244


Values not available for System Generated Fields

I'm experiencing an issue pulling values for fields that are generated systematically in Jira. We have an Analyst (user) custom field that is set to the authoring user on create of a Jira ticket. This value doesn't show up in the issue history, but does contain a value in the field on the issue form. If a user changes the value to another user, the field value will change and will then show up in the history log and therefore is available in the Jira JSON export.
I have confirmed that this field does not display in the JSON results when running the following Jira API calls:
a.	https://[company].atlassian.net/rest/api/latest/issue/[issuekey]/
b.	https://[company].atlassian.net/rest/api/latest/issue/[issuekey]/changelog
Is there any way to update the config file to either always include this value or statically set that field to the author on the first revision? I'm also open to other suggestions.
Thanks!

Hi @masghah, I'll take a look at this. Can you share the definition of the Analyst field in Jira for us to setup a scenario to test?

Sure!
Field Name: Analyst
DataType: User Picker (com.atlassian.jira.plugin.system.customfieldtypes:userpicker)
Orderable: True
Navigable: True
Searchable: True
Hopefully that's what you're looking for. Let me know if need other information.

@masghah sorry for the slow response. Did you manage to resolve it?

@masghah closing this one for now, feel free to open it up again.


AUTDGetFPGAInfo always returns 0

This is not a bug. The lower 1 bit of the FPGA info is connected to the thermal sensor, and is set to 0 when the fan is driven by setting the force fan flag to On.


[main] Update community files (#329)

Update community files
Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


CFD in AssetClass

Hi, Could you please add CFD to the AssetClass Enum? Thanks.

Your wish is my command, oh exalted arbitrageur.

Wow, that’s fast! Thanks a lot!

On 8 Nov 2019, at 12:41 , Christopher Singley<EMAIL_ADDRESS>wrote:
Closed #6 https://github.com/csingley/ibflex/issues/6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/csingley/ibflex/issues/6?email_source=notifications&email_token=ANW4VVOY63NH2DLTRFRBBNTQSVF7DA5CNFSM4JKV653KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUXNXDGI#event-2782622105, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANW4VVJYEGXDYCCPLPLJK73QSVF7DANCNFSM4JKV653A.


Allow E2E key verification using QR codes

As listed in #627 the UI can show your own key and the key of the other side. However, it would be more convenient to verify these by simply scanning a QR code, like https://github.com/siacs/Conversations does for OMEMO on jabber.

A FAR MORE widespread option is
Aztec Code
it is an ISO standard, and far better


Add hal::pixel_display_332 interface

An interface for display that support 8-bit resolution pixel frame buffer data.

Superseded by #37 . That interface would manage to handle this interface as well.


C++ compiling & CMake integration & Linking order of static dependencies

Hi,
I had some issues using your library in a small project of mine. I solved those issues and I thought I'd share here.
Setup
Ubuntu 19.10
gcc 9.2.1
cmake 3.17.0
meson 0.53.2
ninja 1.9.0
Issue compiling the standalone project (standard)
$ cd mini-async-log-c
$ meson build --buildtype=release
$ ninja -C build

I get the following error:
In file included from ../subprojects/base_library/include/bl/base/thread.h:12,
                 from ../subprojects/base_library/test/src/bl/base/time_test.c:6:
../subprojects/base_library/include/bl/base/impl/thread_c11.h: At top level:
../subprojects/base_library/include/bl/base/impl/thread_c11.h:52:48: error: unknown type name ‘bl_tss_cleanup_func’
   52 | static inline bl_err bl_tss_init (bl_tss* key, bl_tss_cleanup_func cleanup)
      |                                                ^~~~~~~~~~~~~~~~~~~

Cause
It seems that there is a discrepancy in the implementation files when using C11 threads instead of POSIX threads in the base-library dependency:
bl/base/impl/thread_C11.h
bl/base/impl/tss_posix.h
The implementation choice is defined in bl/base/thread.h which depends on bl/base/platform.h and since I use a recent version of gcc (I guess ?) the choice is made to enable BL_HAS_C11_THREAD.
NB: Interestingly this issue didn't arise when I tried to compile logger-bench and base-library as a standalone but I didn't dig much farther.
NBB: malcpp did not compile with logger-bench though :(
Solution
I first chose the quick and dirty way of switching the preprocessor directives in bl/base/thread.h to give priority to the use of POSIX threads. I ended up with this error:
c++ -Isubprojects/base_library/b3ec909@@bl-nonblock-mpmc-bpm-relacy@exe -Isubprojects/base_library -I../subprojects/base_library -I../subprojects/base_library/include -I../subprojects/base_library/src -I../subprojects/base_library/test/src -I../subprojects/base_library/gitmodules/relacy -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wnon-virtual-dtor -std=c++11 -O3 -pthread -MD -MQ 'subprojects/base_library/b3ec909<EMAIL_ADDRESS>-MF 'subprojects/base_library/b3ec909<EMAIL_ADDRESS>-o 'subprojects/base_library/b3ec909<EMAIL_ADDRESS>-c ../subprojects/base_library/test/src/bl/mpmc_bpm_relacy/mpmc_bpm_relacy.cpp
../subprojects/base_library/test/src/bl/mpmc_bpm_relacy/mpmc_bpm_relacy.cpp:304:10: fatal error: relacy/relacy.hpp: No such file or directory
  304 | #include <relacy/relacy.hpp>
      |          ^~~~~~~~~~~~~~~~~~~

With this error, I gave up the idea of using a C threading implementation since the logger would eventually be used in a C++ library.
Issue compiling the standalone project (C++ implementation)
I use the -xc++ flag to force gcc to compile c++ code regardless of the compiled file:

$ cd mini-async-log-c
$ CFLAGS=-xc++ meson build --buildtype=release
$ ninja -C build

I end up with a bunch of compiling errors related to the use of cmocka as unit testing framework (not reproduced here). Next try without building the tests:

$ cd mini-async-log-c
$ CFLAGS=-xc++ meson build --buildtype=release -Dbare=true
$ ninja -C build

I get this kind of error:
../subprojects/base_library/src/bl/base/dynamic_string_replace.c:178:1: error: jump to label ‘destroy_bl_dynarray’
  178 | destroy_bl_dynarray:
      | ^~~~~~~~~~~~~~~~~~~
../subprojects/base_library/src/bl/base/dynamic_string_replace.c:162:10: note:   from here
  162 |     goto destroy_bl_dynarray;
      |          ^~~~~~~~~~~~~~~~~~~
../subprojects/base_library/src/bl/base/dynamic_string_replace.c:164:9: note:   crosses initialization of ‘char* rptr_prev’
  164 |   char* rptr_prev   = s->da.str + bl_dstr_len (s);
      |         ^~~~~~~~~

Cause
This  was caused by the use of goto labels that jumped over the declaration with initialization of temporary variables which seems to be allowed in C but not in C++.
In the malc project:

src/malc/bounded_buffer.c
src/malc/malc.c

In the base-library project:

src/bl/base/dynamic_string_replace.c
src/bl/getopt/getopt.c
src/bl/getopt/getopt.c#2

Solution
I went the q-a-d way again and (stupidly) rewrote the branching logic in those code segments using flags to emulate the goto mechanism. I later stumbled on one of your commit or in-code commentary specifying that the goto mechanism was intended to improve code readability and it did... quite a lot.
I gave another try at solving this and simply created local scopes for the temporary variables jumped over by gotos. It did solve the goto issue.
Issue compiling the standalone project (C++ implementation #2)
So, again:
$ cd mini-async-log-c
$ CFLAGS=-xc++ meson build --buildtype=release -Dbare=true
$ ninja -C build

And this time I'm met with this kind of error:
../subprojects/base_library/src/bl/serial/serial_posix.c: In function ‘bl_err bl_serial_init(bl_serial**, bl_uword, const bl_alloc_tbl*)’:
../subprojects/base_library/include/bl/base/allocator.h:54:29: error: invalid conversion from ‘void*’ to ‘bl_serial*’ [-fpermissive]
   54 |   (bl_alloc_tbl_ptr)->alloc ((bytes), (bl_alloc_tbl_ptr))
      |   ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                             |
      |                             void*

I didn't search this one through a lot and directly went:

$ cd mini-async-log-c
$ CFLAGS="-xc++ -fpermissive" meson build --prefix=/ --buildtype=release -Dbare=true
$ ninja -C build

And thus, I was able to compile the logger. Now to the CMake integration.
CMake integration & Linking order
This part is more for those interested by using this logger inside a CMake project.
So, having everything compiled, I installed it with the complete compile + install command being:

$ cd mini-async-log-c
$ CFLAGS="-xc++ -fpermissive" meson build --prefix=/ --buildtype=release -Dbare=true
$ ninja -C build
$ DESTDIR=$PWD/install ninja -C build install

And, based on the README, I wrote my Findmini-async-log-c.cmake and managed to gather everything as STATIC IMPORTED libraries. I created an INTERFACE target to regroup everything under the same umbrella like this:

add_library(mini-async-log-c INTERFACE IMPORTED)
target_link_libraries(mini-async-log-c
    INTERFACE
        malc
        malcpp
        bl-base
        bl-nonblock
        bl-time-extras
        Threads::Threads
)

I used your example hello-malcpp.cpp inside my project to test the integration. In doing so, I did stumble on some linking error like:

/usr/bin/ld: /home//mini-async-log-c/install/lib/x86_64-linux-gnu/libmalcpp.a(src_malcpp_destinations.cpp.o): in function `malcpp::array_dst::size() const':
destinations.cpp:(.text+0x99): undefined reference to `malc_array_dst_size'

Which appeared to be related to my wrong ordering of static libraries when creating my umbrella target.
Furthermore I got some errors looking like:

/usr/bin/ld: /home//mini-async-log-c/install/lib/x86_64-linux-gnu/libmalc.a(src_malc_entry_parser.c.o): in function `push_obj_data(void*, char const*, char const*, malc_obj_log_data const*)':
entry_parser.c:(.text+0x598): undefined reference to `bl_itostr_dyn_arr'

Those errors stemed from the dependency on bl-tostr library from base-library, I added it to the mini-async-log-c target and got this target which, linked with the example, could compile:

add_library(mini-async-log-c INTERFACE IMPORTED)
target_link_libraries(mini-async-log-c
    INTERFACE
        malcpp
        malc
        bl-tostr
        bl-nonblock
        bl-time-extras
        bl-base
        Threads::Threads
)

And that's it !
Sorry for the big chunk of text, I hope you'll find some interesting feedback in it. And thanks for releasing this project, now having everything setup, I can't wait to give it an in-depth try.
NB: I attached the Findmini-async-log-c.cmake for those interested. You can use it as such:

list(INSERT CMAKE_MODULE_PATH 0 "path/to/find/modules")
set(mini-async-log-c_ROOT "abs/path/to/mini-async-log-c" CACHE PATH "Location of the mini-async-log-c library")
find_package(mini-async-log-c MODULE)

Thanks for reporting! I'll see when I have some time and I'll try to fix these issues.

Maybe I should use NIX to pin the build tools...

I read your message:

I have to look at compiling with gcc 9.2, that shoudn't happen.

-The missing relacy.hpp header could have been caused because you didn't initialize the submodules recursively. The header could fail to compile, but not be missing:
git submodule update --init --recursive
-Some other issues are expected WRT to compile C11 code as C++ are expected. There are subtle differences between the languages, as you found on C++ pointers don't explicitly cast to void*.
-Notice that for compiling this from CMake you can use ExternalProjectsAdd as done from logger-bench. The URI can be remote or local I think. Or you can just install the project and use pkgconfig:
https://stackoverflow.com/questions/29191855/what-is-the-proper-way-to-use-pkg-config-from-cmake

I installed both gcc and g++9 on Ubuntu 18.04 and I wasn't able to reproduce.
CC=gcc-9 CXX=g++-9 meson build
Do you face problems if you clone a clean repo from Ubuntu 19, and run:
git submodule update --init --recursive
meson build
ninja -C build

Github's CI is enabled in this project and is passing on "ubuntu-latest", which I guess is 19.04.

Hi, thanks for the reply.
I tried the following commands:

$ git clone <malc-url> malc
$ cd malc
$ git branch
* master
$ git submodule update --init --recursive
$ gcc-9 --version
gcc-9 (Ubuntu 9.2.1-9ubuntu2) 9.2.1 20191008
$ g++-9 --version
g++-9 (Ubuntu 9.2.1-9ubuntu2) 9.2.1 20191008
$ CC=gcc-9 CXX=g++-9 meson build
$ ninja -C build

I still get the following error:
In file included from ../subprojects/base_library/include/bl/base/thread.h:12,
                 from ../subprojects/base_library/test/src/bl/base/time_test.c:6:
../subprojects/base_library/include/bl/base/impl/thread_c11.h: At top level:
../subprojects/base_library/include/bl/base/impl/thread_c11.h:52:48: error: unknown type name ‘bl_tss_cleanup_func’
   52 | static inline bl_err bl_tss_init (bl_tss* key, bl_tss_cleanup_func cleanup)
      |                                                ^~~~~~~~~~~~~~~~~~~

There is this issue with the typedefs used with C11 threads in the base-library dependency:
bl/base/impl/thread_C11.h
If I Crl+F this header for bl_tss_cleanup_func I find only one result of it being used as a parameter and AFAIK it is not defined in included headers. What do you think about this ?

I see, you are right on this. This is a clear bug.
I might not be able to reproduce because this depends on libc too, not gcc/g++ only.

If you download the attached patch and
cd subprojects/base_library
tar -xf <path>/patch.tar.gz
git am 0001-Fix-broken-C11-thread-local-storage-implementation.patch
cd -

Do things get better
patch.tar.gz

Hi, thanks for you quick reply and sorry for mine being late. So I applied your patch and still had some errors.
First build error:
In file included from ../subprojects/base_library/include/bl/base/thread.h:12,
                 from ../subprojects/base_library/test/src/bl/base/time_test.c:6:
../subprojects/base_library/include/bl/base/impl/thread_c11.h: In function ‘bl_mutex_destroy’:
../subprojects/base_library/include/bl/base/impl/thread_c11.h:87:35: error: invalid use of void expression
   87 |   return bl_thread_error_convert (mtx_destroy (m));
      |                                   ^~~~~~~~~~~~~~~

Because mtx_destroy returns void.
Second build error:
In file included from ../src/malc/memory.h:11,
                 from ../src/malc/malc.c:24:
../src/malc/tls_buffer.h: At top level:
../src/malc/tls_buffer.h:51:34: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘tls_buffer_out_of_scope_destroy’
   51 | extern void bl_tss_dtor_callconv tls_buffer_out_of_scope_destroy (void* opaque);
      |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The macro bl_tss_dtor_callconv was removed from thread_c11.h in the patch.
So I modified the patch (attached), tested it on a clean repo and everything compile perfectly. Thank you very much for your help.
patch.tar.gz

Thanks for fixing this out. d6e32de71ee826c0d6fd8868e57283d0dba89082 solves your issue, so you don't have to patch.
git pull && git submodule update --init --recursive

Great, I think this can be closed. I'll open another issue dedicated to some failed tests from test-all-variants.sh once I investigate it a bit further.


Census: The name piping on personal details screen is not according to the spec

Expected behaviour
Must display the the Name of the person under the Title: The following questions are for: (Please Find Attached)
Actual behaviour
Name is not displayed under the Title: The following questions are for: (Please Find Attached)
Steps to reproduce the behaviour
Launch Census Schema
Navigate to the Personal Details screen (Any Path)
Technical information
Browser
All
Operating System
All
Screenshot

@yoganandkunche Can you confirm if the group title should contain First and Last names (i.e. John Doe) and the descriptive test should include middle names if they exist (i.e. John James Doe). It's not clear from the spec, but that's what it looks like in you're issue screen shot. What are the requirements?

@samiwel Checked with Jenny and she thinks that it should match the spec file.. I can always check with @Louise Morries when she is in tomorrow and get back to you on this.

Okay, once we have final confirmation from Louise I'll deliver the fix. I think it should be easy to resolve based on my testing so far.

@samiwel Checked with louiseM and she confirmed that it must be exactly as it is on the spec file.
"Title should contain First and Last names (i.e. John Doe) and the descriptive test should include middle names if they exist (i.e. John James Doe)

I believe this is fixed by PR #613 . There is some front-end work required to tidy up the solution. But if you want to test there, feel free.

Defect passed retest


More than 149 devices on Switches Alone

One this one setup for our new office, we have well over 149 legs (switches and or dimmers) and the issue is the warning that
plugin will continue to crash... it imports up to 149 of the switches alone (no dimmers, shades, or scenes on the child bridge, or installed at all in this instance) and then truncates the rest with the warning.
Is there any further way to segregate the switches alone into 2 parts (child bridges) as to overcome this issue/limitation?
Log
5/2/2023, 6:49:09 PMCrestron Home HOOBs starting
5/2/2023, 6:49:20 PMCrestron Home HOOBsLoaded plugin 'homebridge-crestron-home'
5/2/2023, 6:49:20 PMCrestron Home HOOBsLoading 1 platforms...
5/2/2023, 6:49:20 PMCrestron Home HOOBsCrestron Home HOOBsLoading accessory from cache: 1.1 - Main Washroom Main Washroom Blind - Blackout
5/2/2023, 6:49:20 PMCrestron Home HOOBsCrestron Home HOOBsLoading accessory from cache: 1.1 - Main Washroom Main Washroom Blind - Shade
5/2/2023, 6:49:20 PMCrestron Home HOOBsBridge is running on port ********.
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsSuccsessfully authinticated, working with version:  2.000.0003
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsWARNINGReturning more than 149 devices, Bridge may crash -  257
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 2.2 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 2.1 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 2.2 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 2.2 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 2.1 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 2.1 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.4 - ********
5/2/2023, 6:49:21 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.2 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.0 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.0 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.0 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.0 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.5 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.4 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 1.4 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 3.3 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 3.1 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 3.1 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsAdding new accessory: 3.2 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsRestoring existing accessory from cache: 1.1 - ********
5/2/2023, 6:49:22 PMCrestron Home HOOBsCrestron Home HOOBsRestoring existing accessory from cache: 1.1 - ********
5/2/2023, 6:49:50 PMCrestron Home HOOBsCrestron Home HOOBsUpdating Devices state
5/2/2023, 6:49:50 PMCrestron Home HOOBsCrestron Home HOOBsWARNINGReturning more than 149 devices, Bridge may crash -  257
5/2/2023, 6:50:20 PMCrestron Home HOOBsCrestron Home HOOBsUpdating Devices state
5/2/2023, 6:50:20 PMCrestron Home HOOBsCrestron Home HOOBsWARNINGReturning more than 149 devices, Bridge may crash -  257
Plugin Config:
{
"platform": "CrestronHomePlatform",
"name": "Crestron Home HOOBs",
"loginInterval": 4,
"crestronHost": "192.168.1.",
"token": "*****",
"enabledTypes": [
"Switch"
]
}

Plugin Version: 1.1.5
Homebridge Version: 1.6.0
Node.js Version: 16.20.0
Operating System: Hoobs 4.3.0

In the current implementation there is no way to split lights if you have more than 149 (this is Apple HomeKit limitation).
How would you split it? By room?


mathematical operations

I can not do the multiplication gives an error in the code I wrote

Which error do you see?

Ben herhangi bir hata kodu sorun derleme yapımında tanınmaz alamadım
PRoGrAmER
2016-08-23 20:20 GMT+03:00 ficristo<EMAIL_ADDRESS>
Which error do you see?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adobe/brackets/issues/12708#issuecomment-241807370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT_Jm-UkJxmMP9Q6DofVvFL6VtmGe1Peks5qiyvygaJpZM4Jq_EY
.

I do not get any error code is not recognized in making trouble compiling
PRoGrAmER
2016-08-24 17:56 GMT+03:00 Evliya BALIK<EMAIL_ADDRESS>
Ben herhangi bir hata kodu sorun derleme yapımında tanınmaz alamadım
PRoGrAmER
2016-08-23 20:20 GMT+03:00 ficristo<EMAIL_ADDRESS>
Which error do you see?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adobe/brackets/issues/12708#issuecomment-241807370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT_Jm-UkJxmMP9Q6DofVvFL6VtmGe1Peks5qiyvygaJpZM4Jq_EY
.

Could you give us more info like a snippet of code, which language you are using, maybe a screenshot, what are you compiling, etc? I really don't know how to help otherwise.
Maybe try also Debug \ Reload without extensions to see if it is an extension that is causing problems.

Şimdi tamamen matematiksel bir çarpma (*) operasyonu olarak tek başına
sorunu çözüldü bilmek teşekkür ederim
PRoGrAmER
2016-08-24 20:52 GMT+03:00 ficristo<EMAIL_ADDRESS>
Could you give us more info like a snippet of code, which language you are
using, maybe a screenshot, what are you compiling, etc? I really don't know
how to help otherwise.
Maybe try also Debug \ Reload without extensions to see if it is an
extension that is causing problems.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adobe/brackets/issues/12708#issuecomment-242152102,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT_Jm-FeXHi3gOzHrify1l7toJwUTsaaks5qjITFgaJpZM4Jq_EY
.

now completely solved the problem by itself as a mathematical
multiplication (*) operation , thank you to know
PRoGrAmER
2016-08-24 20:52 GMT+03:00 ficristo<EMAIL_ADDRESS>
Could you give us more info like a snippet of code, which language you are
using, maybe a screenshot, what are you compiling, etc? I really don't know
how to help otherwise.
Maybe try also Debug \ Reload without extensions to see if it is an
extension that is causing problems.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adobe/brackets/issues/12708#issuecomment-242152102,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT_Jm-FeXHi3gOzHrify1l7toJwUTsaaks5qjITFgaJpZM4Jq_EY
.


Team ID required in ios Appfile

Next I also needed to provide my Team ID in the ios Appfile. Perhaps this is only required because i have multiple teams? But again, it could be added to the manual.
Originally posted by @yvanvds in https://github.com/mmcc007/fledge/issues/2#issuecomment-471615971

Do you have an error message or anything that indicates this (so it can be reproduced and documented correctly)?

Ok. I filled in a fake Team ID and get this result.
The first error reads:
[13:25:59]: Could not set team ID to '118607454', only found the following available teams
- 46266300 (Team1 Name)
- 2015341 (Team2 Name)

Fairly sure this is what I got the previous time. After all, when fledge is installed in my project, this is in the Appfile:
itc_team_id("118607454") # App Store Connect Team ID
team_id("ET2VMHJPVM") # Developer Portal Team ID

I guess these are fake, or from your account? Then follows the Fastlane summary, with the last failed step: upload_to_testflight. At the last line, the previous error is repeated.

See comment in #13

Please close if you think this issue is resolved

I don't think I can close this issue since you've created it.


fix: 修复维值带有 '-' 时刷选复制无法复制表头 close #2684

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #2684

📝 Description

🖼 Screenshot

Before
After

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

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

Project coverage is 79.20%. Comparing base (1adc25d) to head (475e4c3).
Report is 155 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2691      +/-   ##
==========================================
+ Coverage   76.48%   79.20%   +2.71%
==========================================
  Files         240      160      -80
  Lines       12312     9517    -2795
  Branches     2630     2053     -577
==========================================
- Hits         9417     7538    -1879
+ Misses       1377      665     -712
+ Partials     1518     1314     -204

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

:tada: This PR is included in version @antv/s2-v1.55.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


WIP: Use helm repo for integration tests instead of tarballs

Signed-off-by: Faseela K<EMAIL_ADDRESS>Make the helm integration tests use istio helm repo.
This is needed as part of helm graduation from Alpha to Beta (https://github.com/istio/istio/issues/42565)

@kfaseela: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I like the simplicity but am a bit concerned it goes the opposite direction of #35835. WDYT?
Since its GCS it may not be so bad since we already heavily depend on it

Ohh!! I missed this discussion. I thought we wanted to get rid of tarballs based install everywhere as we discussed in the helm promotion task, so thought this was also needed!

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

lint_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

lint_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test lint

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

lint_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test lint

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

analyze-tests_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test analyze-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

lint_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test lint

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

analyze-tests_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test analyze-tests

integ-helm_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-helm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

lint_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test lint

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

analyze-tests_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test analyze-tests

integ-helm_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-helm

integ-pilot-istiodremote_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

lint_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test lint

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

analyze-tests_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test analyze-tests

integ-helm_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-helm

integ-pilot-istiodremote_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-istiodremote

integ-pilot-multicluster_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-basic-arm64

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

analyze-tests_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test analyze-tests

integ-helm_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-helm

integ-pilot-istiodremote_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-istiodremote

integ-pilot-multicluster_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-multicluster

lint_istio
010d614bf2c15571fd2dc77ea80900c3f1f72795
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

If we don't test the helm repo then we need to figure out a way to make updating the tarballs and helm integration test. Right now it's a bit difficult to update them. It'd be nice if it was part of the release process, but it's too cumbersome to do for a release manager without understanding how they are packaged and tested.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ds_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ds

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-ipv6_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-ipv6

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

analyze-tests_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test analyze-tests

integ-pilot-istiodremote_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-istiodremote

integ-pilot-multicluster_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-multicluster

lint_istio
010d614bf2c15571fd2dc77ea80900c3f1f72795
link
true
/test lint

integ-helm_istio
010d614bf2c15571fd2dc77ea80900c3f1f72795
link
true
/test integ-helm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-distroless

integ-cni_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-cni

integ-pilot-istiodremote_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-istiodremote

integ-pilot-multicluster_istio
ee6e256bfe5d84f32f930570522c90b80fa690ca
link
true
/test integ-pilot-multicluster

lint_istio
010d614bf2c15571fd2dc77ea80900c3f1f72795
link
true
/test lint

integ-helm_istio
010d614bf2c15571fd2dc77ea80900c3f1f72795
link
true
/test integ-helm

analyze-tests_istio
010d614bf2c15571fd2dc77ea80900c3f1f72795
link
true
/test analyze-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
498c8598fcd658f946f1a5f2d78f15d9540a15e0
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kfaseela: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
498c8598fcd658f946f1a5f2d78f15d9540a15e0
link
true
/test lint

gencheck_istio
498c8598fcd658f946f1a5f2d78f15d9540a15e0
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

If we don't test the helm repo then we need to figure out a way to make updating the tarballs and helm integration test. Right now it's a bit difficult to update them. It'd be nice if it was part of the release process, but it's too cumbersome to do for a release manager without understanding how they are packaged and tested.

@howardjohn what do you think?

I like the simplicity but am a bit concerned it goes the opposite direction of https://github.com/istio/istio/issues/35835. WDYT?
Since its GCS it may not be so bad since we already heavily depend on it

With the configurable chart repo, a local web server that hosts the charts can likely be added later right?

yep lgtm (at ahigh level, will review the code in a bit)

@kfaseela as of now, what is the current way of running the helm e2e tests in /tests/integration/helm against helm chart tarballs that are built and packaged locally?
For instance, if we had the tarballs for base-1.16.1.tgz  cni-1.16.1.tgz  gateway-1.16.1.tgz  istiod-1.16.1.tgz living in some /dir, how do we point the e2es to install / test against these charts?

@kfaseela as of now, what is the current way of running the helm e2e tests in /tests/integration/helm against helm chart tarballs that are built and packaged locally?
For instance, if we had the tarballs for base-1.16.1.tgz cni-1.16.1.tgz gateway-1.16.1.tgz istiod-1.16.1.tgz living in some /dir, how do we point the e2es to install / test against these charts?

currently the tarballs are hardcoded as far as i can understand, and there is no way to override. If you replace the tarballs with your own, under the testdata folder, and run the tests it will be picked up

@kfaseela thank you!!

/cherrypick release-1.17

@kfaseela: new pull request created: #42972

In response to this:

/cherrypick release-1.17

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Feature: TypeScript Support

+1 for Typescript support -- would love that, I haven't been able to get the adapter to work properly in my Typescript project

Duplicate of #8

Should be closed as duplicate (same state as not planned)


sql: EXPLAIN statement

Document the EXPLAIN statement.
Some nuggets from Gitter:
kena @knz Jun 30 16:16
guys
who invented explain(debug)?
who can explain what it is supposed to do and what its output means?
RaduBerinde @RaduBerinde Jun 30 16:19
it is supposed to show the exact keys and values that are processed
i can answer any specific questions
the columns are rowid, key, value, and the last one indicates if the key corresponds to the end of a row, and if so what happens to that row (filtered, buffered, or a result)
kena @knz Jun 30 16:22
what do you mean with "processed"?
what does filtered mean? what does buffered mean?
RaduBerinde @RaduBerinde Jun 30 16:26
the SQL layer does KV ops and receveis KVs, right? then our code processes those KVs to form rows and whatnot
the debug output has an entry for each KV that our code looks at
filtered means that a row is thrown out because of a filter like WHERE
buffered means it is stored in memory in some node (e.g. sortNode). it willl show up again if that node outputs it
kena @knz Jun 30 16:28
aha
ok this becomes clearer now
but here you're talking about k/v reads
what about writes?
RaduBerinde @RaduBerinde Jun 30 16:32
reads. i have never used it on anything other than a SELECT, I am not sure if it even works
kena @knz Jun 30 16:32
yeah
ok

Another nugget from Gitter:
(18:34) < knz> @jseldess regarding explain. I'd recommend explaining (hah!) explain(plan) and explain(types), which are both useful to end users. Then simply
mention that explain(trace) and explain(debug) also exist but are mostly for our internal use. Then invite the reader to come and ask on
gitter if they think they need to know.

Explain now works correctly for VALUES containing subqueries. https://github.com/cockroachdb/cockroach/pull/8970. If relevant, you can just wrap this into your open work on EXPLAIN, @sploiselle.

Additional improvements to EXPLAIN in https://github.com/cockroachdb/cockroach/pull/12506.

Hold onto EXPLAIN for a little bit, there's a rather larger change upcoming.

K. Thanks, @knz.

Latest improvements have landed: https://github.com/cockroachdb/cockroach/pull/12553

More improvements: https://github.com/cockroachdb/cockroach/pull/12637


RedisRateLimter increment and expire is not atomic operation

RedisRateLimter increment and expire is not atomic operation
In the middle of two steps，if Redis is down，The RedisRateLimter will create forever key
when request exceed the limit number，zuul filter will return 429 response
I think the increment and expire operation should use Lua script

Read the new Code, master has been solved the problem


How to use it

I dont know how to use this danser bot, please help.

Really? Not only on the wrong repo, and issue like this? Please learn to use git and cli tools. Spam.

r u rlly? at least tell me what to do.

and why spam?


Correction for raw affiliation IPSiM, Univ Montpellier, CNRS, INRAE, Institut Agro, 34060, Montpellier, France

Correction needed for raw affiliation IPSiM, Univ Montpellier, CNRS, INRAE, Institut Agro, 34060, Montpellier, France
raw_affiliation_name: IPSiM, Univ Montpellier, CNRS, INRAE, Institut Agro, 34060, Montpellier, France
new_rors: 03rnk6m14;02feahw73;003vg9w96;051escj72;05r332y60
previous_rors: 03rnk6m14;02feahw73;003vg9w96;051escj72
works_examples: W4379622671
contact: 17eda04bd5c22d45eaf3c1c6c2da5302:f67e98ad960f5b729793dd59403b22bfa7c6 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


unable to download subtitles from youtube.com

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.03.24
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

% youtube-dl --list-subs 'https://www.youtube.com/embed/3UawxEP33FE?feature=oembed' --verbose --print-traffic
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--list-subs', u'https://www.youtube.com/embed/3UawxEP33FE?feature=oembed', u'--verbose', u'--print-traffic']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.03.24
[debug] Python version 2.7.16+ (CPython) - Linux-5.4.0-4-amd64-x86_64-with-debian-bullseye-sid
[debug] exe versions: avconv 4.2.2-1, avprobe 4.2.2-1, ffmpeg 4.2.2-1, ffprobe 4.2.2-1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 3UawxEP33FE: Downloading webpage
send: u'GET /watch?has_verified=1&bpctr=9999999999&hl=en&v=3UawxEP33FE&disable_polymer=true&gl=US HTTP/1.1\r\nAccept-Language: en-us,en;q=0.5\r\nAccept-Encoding: gzip, deflate\r\nConnection: close\r\nAccept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8\r\nUser-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.123 Safari/537.36\r\nAccept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7\r\nHost: www.youtube.com\r\nCookie: PREF=f1=50000000&hl=en\r\n\r\n'
reply: 'HTTP/1.1 200 OK\r\n'
header: Cache-Control: no-cache
header: Strict-Transport-Security: max-age=31536000
header: Content-Type: text/html; charset=utf-8
header: X-Frame-Options: SAMEORIGIN
header: Content-Encoding: gzip
header: X-Content-Type-Options: nosniff
header: P3P: CP="This is not a P3P policy! See http://support.google.com/accounts/answer/151657?hl=en for more info."
header: Expires: Tue, 27 Apr 1971 19:44:06 GMT
header: Date: Fri, 10 Apr 2020 10:34:46 GMT
header: Server: YouTube Frontend Proxy
header: X-XSS-Protection: 0
header: Set-Cookie: GPS=1; path=/; domain=.youtube.com; expires=Fri, 10-Apr-2020 11:04:46 GMT
header: Set-Cookie: s_gl=1d69aac621b2f9c0a25dade722d6e24bcwIAAABVUw==; path=/; domain=.youtube.com
header: Set-Cookie: YSC=QfBVh5GTBCY; path=/; domain=.youtube.com; secure; httponly; samesite=None
header: Set-Cookie: VISITOR_INFO1_LIVE=QNVxRY_hVN8; path=/; domain=.youtube.com; secure; expires=Wed, 07-Oct-2020 10:34:46 GMT; httponly; samesite=None
header: Alt-Svc: quic=":443"; ma=2592000; v="46,43",h3-Q050=":443"; ma=2592000,h3-Q049=":443"; ma=2592000,h3-Q048=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,h3-T050=":443"; ma=2592000
header: Connection: close
header: Transfer-Encoding: chunked
send: u'GET /timedtext?disable_polymer=true&type=list&hl=en&v=3UawxEP33FE HTTP/1.1\r\nAccept-Language: en-us,en;q=0.5\r\nAccept-Encoding: gzip, deflate\r\nConnection: close\r\nAccept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8\r\nUser-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.123 Safari/537.36\r\nAccept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7\r\nHost: video.google.com\r\n\r\n'
reply: 'HTTP/1.1 404 Not Found\r\n'
header: Date: Fri, 10 Apr 2020 10:34:47 GMT
header: Pragma: no-cache
header: Expires: Fri, 01 Jan 1990 00:00:00 GMT
header: Cache-Control: no-cache, must-revalidate
header: X-Content-Type-Options: nosniff
header: Content-Type: text/html; charset=UTF-8
header: Server: video-timedtext
header: Content-Length: 1633
header: X-XSS-Protection: 0
header: X-Frame-Options: SAMEORIGIN
header: Alt-Svc: quic=":443"; ma=2592000; v="46,43",h3-Q050=":443"; ma=2592000,h3-Q049=":443"; ma=2592000,h3-Q048=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,h3-T050=":443"; ma=2592000
header: Connection: close
WARNING: unable to download video subtitles: Unable to download XML: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update.  I had no trouble getting the subtitles (even dowload them) until yesterday :(
This seems to be the culprit:
send: u'GET /timedtext?disable_polymer=true&type=list&hl=en&v=3UawxEP33FE HTTP/1.1\r\nAccept-Language: en-us,en;q=0.5\r\nAccept-Encoding: gzip, deflate\r\nConnection: close\r\nAccept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8\r\nUser-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.123 Safari/537.36\r\nAccept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7\r\nHost: video.google.com\r\n\r\n'
reply: 'HTTP/1.1 404 Not Found\r\n'```

I think I am having the same problem.  A video that I can watch with subtitles will not download them, although similar videos have no issue. ( https://youtu.be/bdlgsSMc09I  made by the same user has the same issue ).

% youtube-dl -f 'best[ext!=webm]' --prefer-ffmpeg --write-sub -v 'https://youtu.be/5NsITeJibEs'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-f', u'best[ext!=webm]', u'--prefer-ffmpeg', u'--write-sub', u'-v', u'https://youtu.be/5NsITeJibEs']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.03.24
[debug] Python version 2.7.17 (CPython) - Linux-4.15.0-96-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6, phantomjs ., rtmpdump 2.4
[debug] Proxy map: {}
[youtube] 5NsITeJibEs: Downloading webpage
WARNING: unable to download video subtitles: Unable to download XML: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[youtube] 5NsITeJibEs: Downloading MPD manifest
[debug] Invoking downloader on u'https://r4---sn-ab5l6nzy.googlevideo.com/videoplayback?expire=1586750116&ei=RI6TXpCzMfmB2LYPi5uiWA&ip=<IP_ADDRESS>&id=o-AIT_4CBLbNQRL2huYi_SIBTG7OS0klJufAhv6xtpZowY&itag=18&source=youtube&requiressl=yes&mh=uw&mm=31%2C29&mn=sn-ab5l6nzy%2Csn-ab5sznl7&ms=au%2Crdu&mv=m&mvi=3&pl=17&initcwndbps=7533750&vprv=1&mime=video%2Fmp4&gir=yes&clen=118701708&ratebypass=yes&dur=2610.944&lmt=1562696524415699&mt=1586728418&fvip=4&c=WEB&txp=2211222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cratebypass%2Cdur%2Clmt&sig=AJpPlLswRQIhANTrhSaQFUY-g3_BEybgY8hWu3EHRQzv8gADlkV4yMchAiB2b3-SpuV1I595Rycoc8iaPOQ7pUzwm2AVNgYyICdRSg%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=ALrAebAwRAIgTBwkOBcc4B_4P2TR8ggLCbNppZMarBjGaiQtDWXlTjMCIFVteCBINagQr9vcnaiaXiUHt8WOR6ooFQ0uO_tbqPJh'
[download] 20160404 MAT126 L18-5NsITeJibEs.mp4 has already been downloaded
[download] 100% of 113.20MiB

% youtube-dl --list-subs 'https://youtu.be/5NsITeJibEs'
[youtube] 5NsITeJibEs: Downloading webpage
WARNING: unable to download video subtitles: Unable to download XML: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update.  Seems related to the video privacy setting: if I use Public intead of Unlisted, youtube-dl correctly fetches subtitles.

if I use Public intead of Unlisted
what do you mean by that?

I update my comment. I'm the owner of the video, so I changed the video privacy setting

That's what I thought, unfortunately those vids are not mine :(

I can confirm the issue is only for Unlisted videos, like this one for example.
Using youtube-dl v 2020.03.24 and calling:
youtube-dl --list-subs --all-sub "https://www.youtube.com/watch?v=lvIDEzO60w4"

Expected
To get one sub available pt
Actual
[youtube] lvIDEzO60w4: Downloading webpage
WARNING: unable to download video subtitles: Unable to download XML: HTTP Error 404: Not Found (caused by <HTTPError 404: 'Not Found'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[youtube] lvIDEzO60w4: Looking for automatic captions
[youtube] lvIDEzO60w4: Downloading MPD manifest
Available automatic captions for lvIDEzO60w4:
Language formats
af       vtt, ttml, srv3, srv2, srv1
sq       vtt, ttml, srv3, srv2, srv1
...~100 lines truncated...
yo       vtt, ttml, srv3, srv2, srv1
zu       vtt, ttml, srv3, srv2, srv1
lvIDEzO60w4 has no subtitles

As a side question --- is there a way to know which of the automatic captions actually exist instead of seeing the full list?  This specific example video only shows the pt-BR automatic captions in the youtube player, but --list-subs returns a full list of 109 language codes. I'm guessing all those other languages are technically available because of auto-translate feature, but it would be useful to know which ones are already there vs. the ones that could be there.

@jaimeMF Would you have any pointers for possible workaround for this? Basically video.google.com/timedtext?hl=en&type=list&v={youtube_id}&disable_polymer=true doesn't work anymore for Unlisted videos as of two weeks ago.
If saw in the git commit history you worked on this part of the code so I'm wondering if there was another option for getting the subtitles. Can we imitate how the browser client does it? Any pointers would be appreciated, and if it seems feasible I can try to fix.

Okay, this issue seems to have been only transient (for about two weeks).
I just tried running:
youtube-dl --list-subs --all-sub "https://www.youtube.com/watch?v=lvIDEzO60w4"

and got no errors and the correct output (matching what I see online):
[youtube] lvIDEzO60w4: Downloading webpage
[youtube] lvIDEzO60w4: Looking for automatic captions
[youtube] lvIDEzO60w4: Downloading MPD manifest
Available automatic captions for lvIDEzO60w4:
Language formats
af       vtt, ttml, srv3, srv2, srv1
....
....
zu       vtt, ttml, srv3, srv2, srv1
Available subtitles for lvIDEzO60w4:
Language formats
pt       vtt, ttml, srv3, srv2, srv1

@starenka @ttocs47 I checked on the videos you posted too and I can get subs for them now. So motion to close this issue .... and hope it never comes back :)


fix: added missing moved block

Description
One of the internal resources is missing a moved block.
https://github.ibm.com/GoldenEye/issues/issues/5968

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

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


helpppp

giúp tôi cho chúng nó trở lại như cũ, làm ơn

Mine is locked too :(( My discord ID is Jacky Setiawan#1269... Please, i wanna ask several things. Thank you


Provide nicer parse errors and suggest "full-syntax"

The included compile_fail test has this output:
error: expected `,`
  --> src/test.rs:13:24
   |
10 |     VarA = get_an_isize!(), // error without "syn/full"
   |                        ^

error: this might need the "full-syntax" feature of `num-derive`
 --> src/test.rs:11:10
  |
8 | #[derive(num_derive::FromPrimitive)]
  |          ^^^^^^^^^^^^^^^^^^^^^^^^^
  |
  = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)

Closes #16.

bors r+


Failing HDD

My HDD is failing right now... It's dieing little by little. I'm trying to recover data and backup on GitHub the last untested changes (which will probably not work as intended/crash)...
I won't be able to touch UMod until I have the time to look at this HDD...
[EDIT] : Done recovering UMod from HDD...

Ok So, now I disconnected the Sata HDD from the computer now running with 2 HDDs (one for system, one for studies).
I noticed the PC seam to be faster, and no longer crashes windows explorer so I should be able to download a copy of UMod on the study HDD...
I'll let you informed of the next...

YEAH UMod is back !


Syntax error test Internet Explorer 11

I got following syntax error running ie 11.
 Line:59160			return (key, value, index) => {
SCRIPT1002: Syntax error Internet explorer 11.437

PWA STOREFRONT ! >|

I can confirm this issue is still present.