Vim changes cursor to pink and keeps it on exit (t_Cs, TERM=tmux-256color, urxvt)

Given t-undercurl-pink.vim:
let &term = 'tmux-256color'
hi SpellBad cterm=undercurl
set spell

" misspellet

vim -Nu t-undercurl-pink.vim t-undercurl-pink.vim will cause a pink cursor color for me (with rxvt-unicode as the terminal), and keeps it on exit.  It does not happen when using gnome-terminal.
This appears to be related to t_Cs being set incorrectly:
% TERM=tmux-256color vim --cmd 'echom &t_Cs' -c q
^[]12;%p1%s^G

Via https://github.com/vim/vim/issues/2424#issuecomment-360592096.
I've just tried https://github.com/vim/vim/pull/2126, but it does not appear to fix it.

Figured out what is going on: tmux itself has a Cs extension to set the cursor color (and Cr to reset it).
It does not show up in infocmp by default, but only with -x:
TERM=tmux-256color infocmp -A /usr/share/terminfo -x | grep Cs
Cs=\E]12;%p1%s\007

Pink appears to be the default here without an argument, but it can be set to red using TERM=tmux-256color tput Cs 1 etc.
What can be done about this?
a) Should Vim look into terminfo (via TGETSTR/tgetstrr I assume) for its internal extensions (all of https://github.com/vim/vim/blob/c75878c923034b883090aef3f08f565513d98f4d/runtime/doc/term.txt#L321-L358?) after all?
b) Should t_Cs only be used if t_Ce (to end it) is also set?

Daniel Hahler wrote:

Figured out what is going on: tmux itself has a Cs extension to set the cursor color (and Cr to reset it).
It does not show up in infocmp by default, but only with -x:
TERM=tmux-256color infocmp -A /usr/share/terminfo -x | grep Cs
Cs=\E]12;%p1%s\007

Pink appears to be the default here without an argument, but it can be set to red using TERM=tmux-256color tput Cs 1 etc.
What can be done about this?
a) Should Vim look into terminfo (via TGETSTR/tgetstrr I assume) for its internal extensions (all of https://github.com/vim/vim/blob/c75878c923034b883090aef3f08f565513d98f4d/runtime/doc/term.txt#L321-L358?) after all?

Should already happen in get_term_entries().

b) Should t_Cs only be used if t_Ce (to end it) is also set?

We usually do this, yes.
--
hundred-and-one symptoms of being an internet addict:
102. When filling out your driver's license application, you give
your IP address.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Just to be clear: I've meant to skip looking for "Cs", "Ce" etc, but it looks like it is handled normally? (i.e. looked up) - https://github.com/vim/vim/blob/c75878c923034b883090aef3f08f565513d98f4d/src/term.c#L1589
I think using b) here also would then fix it already, but I still think that querying for non-standard options could be skipped maybe.  OTOH querying them allows you to set them in your terminfo file instead of vimrc, not sure about that though.

Figured out what is going on:
It does not show up in infocmp by default, but only with -x

@blueyed Thanks for looking into this.  I didn't know about -x, that would have saved me hours of confusion last time around.

tmux itself has a Cs extension to set the cursor color (and Cr to reset it).

This is not just an obscure tmux extension, it is in the xterm terminfo as well:
$ TERM=xterm infocmp -1x | grep C[sr]
        Cr=\E]112\007,
        Cs=\E]12;%p1%s\007,

a) Should Vim look into terminfo (via TGETSTR/tgetstrr I assume) for its internal extensions (all of ?) after all?

Should already happen in get_term_entries().

The problem here is a naming conflict between the terminals' extensions and Vim's internal extensions.
Vim queries the Cs entry assuming it represents the undercurl sequence, when in fact it represents the cursor color sequence.
Maybe Vim should take into account the names widely agreed upon for well-known terminfo extensions.

Is there any workaround or fix for now? @blueyed @joshklod

Not as far as I know - I am Neovim by default, so not really affected.

FWIW for undercurls vim should use https://man.openbsd.org/tmux#Smulx and check for it in particular. t_Cs is a tmux extension as it was mentioned above https://man.openbsd.org/tmux#Cs and since it's a part of xterm, each xterm derived terminfo has t_Cs. neovim for example checks for Smulx https://github.com/neovim/neovim/blob/e11e93d1394df83a93c83de5b60639f5085d69dd/src/nvim/tui/tui.c#L1952.


Consider timezone when calculate timekey

Because the timestamp of the midnight cannot be divisible by 86400 in
localtime (JST). The timestamp of the midnight is divisible by 86400
only in UTC. Therefore we must consider offset from UTC in localtime.
For example, at 2018-07-04 01:23:23 +0900:
If timekey is 86400 and path template is /log/%Y%m%d.log.
In previous version, extract path template to /log/20180703.log.
In this version, extract path template to /log/20180704.log.
Fix #1986

Test failed. Could you check it?

Thanks!

I released v1.2.4.rc1 for testing this patch.


Split WaitUntilStartedAndForwardPorts() into two funcs

          This can be done in a follow up but this function should really be two different functions since it performs two different actions.

In fact, I would be more in favour of grouping StartAsync and WaitInstanceIsRunning together
Originally posted by @cmwaters in https://github.com/celestiaorg/celestia-app/pull/3515#discussion_r1646663199

is this still in progress @staheri14 ?


feat: extend lazy loading

resolve #1797

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@userquin
Thanks! I've updated docs from your feedback!


Broadcast perticular view in my activity

Broadcast a particular view and all other things inside that view

You have a filter to add a view to stream. This view need be inflated:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/app/src/main/java/com/pedro/rtpstreamer/openglexample/OpenGlRtmpActivity.java#L134


[Bug]: The in filter doesn't work as expected

Mapbox Implementation
Mapbox
Mapbox Version
10.7.0
Platform
Android
@rnmapbox/maps version
10.0.0-beta.10
Standalone component to reproduce
import React from 'react';
import {
  MapView,
  ShapeSource,
  LineLayer,
  Camera,
  VectorSource,
  FillLayer,
} from '@rnmapbox/maps';

const aLine = {
  type: 'LineString',
  coordinates: [
    [-74.00597, 40.71427],
    [-74.00697, 40.71527],
  ],
};

class BugReportExample extends React.Component {
  render() {
    return (
      <MapView style={{flex: 1}}>
        <Camera centerCoordinate={[-74.00597, 40.71427]} zoomLevel={14} />
        <ShapeSource id="idStreetLayer" shape={aLine}>
          <LineLayer id="idStreetLayer" />
        </ShapeSource>
        <VectorSource
          id="countries-source"
          url="mapbox://mapbox.country-boundaries-v1"
        >
          <FillLayer
            id="countries-layer"
            sourceID="countries-source"
            sourceLayerID="country_boundaries"
            style={{
              fillColor: '#6E599F',
            }}
            filter={['in', 'worldview', 'US']}
          />
        </VectorSource>
      </MapView>
    );
  }
}

Observed behavior and steps to reproduce
Observed behavior: When the worldview is an array the filter doesn't match. Even if the specified value is contained in the array.
For example, if you replace the 'US' with 'CN' then it will display the layer but if it is an array the layer won't be drawn. In nutshell, the 'in' expression works as '=='.
Expected behavior
The example above should draw a layer above China.
Notes / preliminary analysis
No response
Additional links and references
No response

@ikarasz I don't completely understand can you elaborate? Have you tried the same expression in mapbox-gl-js? Does it work there?
Shouldn't it be ['in','US',['get','worldview']]
https://docs.mapbox.com/help/glossary/worldview/

@mfazekas I think China is a good test case.
China is highlighted with ['in', 'worldview', 'CN'] but not highlighted with ['in', 'CN', ['get', 'worldview']] so I suppose the second is not the proper approach however, that would make sense to me.
And no, I haven't tested it with mapbox-gl-js. I am pretty new to mapbox so I am a bit lost with all the different libraries.
This mapbox example works with match, which doesn't work for me in rnmapbox either.
This one suggests using the approach that you mentioned but still doesn't work in my case.
Unfortunately, I have no idea how to debug this issue. If you could give me some hints then I would try it.

You were right, the problem is not with the library. The filter doesn't work as described here. It seems like you cannot combine in with ==
So instead of:
[
  "any",
  ["==", "all", ["get", "worldview"]],
  ["in", "US", ["get", "worldview"]]
]

We need:
[
  "any",
  ["in", "all", ["get", "worldview"]],
  ["in", "US", ["get", "worldview"]]
]


Handle overwrite output response

hi,  i am trying to figure out how to overwrite the output in
middleware.after = function(message, conversationResponse, callback)
i tried to modify the last line to
conversationResponse.output.text = 'This is my custom output line';
however I'm guessing its read only as this causes it to crash. in previous lines i use  conversationResponse.output.text to obtain what the bot will say so i know its valid. it just won't let me overwrite it.
we plan on using a database to get custom responses. however I'm stuck trying to figure out how to overwrite the default output.
any help is appreciated.
thanks
Dion

APP/0Fatal TypeError: message.watsonData.output.text.join is not a functionFeb 13, 2017 8:49:13 AM
APP/0 at /home/vcap/app/node_modules/botkit/lib/CoreBot.js:866:32Feb 13, 2017 8:49:13 AM
APP/0 at Object.Botkit.botkit.trigger (/home/vcap/app/node_modules/botkit/lib/CoreBot.js:779:49)Feb 13, 2017 8:49:13 AM
APP/0 at Object. (/home/vcap/app/node_modules/botkit/lib/CoreBot.js:753:28)Feb 13, 2017 8:49:13 AM
APP/0 at /home/vcap/app/node_modules/botkit/node_modules/ware/node_modules/wrap-fn/index.js:121:18Feb 13, 2017 8:49:13 AM
APP/0 at next (/home/vcap/app/node_modules/botkit/node_modules/ware/lib/index.js:82:27)Feb 13, 2017 8:49:13 AM
APP/0 at /home/vcap/app/node_modules/botkit/lib/CoreBot.js:862:21Feb 13, 2017 8:49:13 AM
APP/0 at /home/vcap/app/node_modules/botkit-middleware-watson/lib/middleware/index.js:154:9Feb 13, 2017 8:49:13 AM
APP/0 at Promise._settlePromise (/home/vcap/app/node_modules/botkit-middleware-watson/node_modules/bluebird/js/release/promise.js:580:21)Feb 13, 2017 8:49:13 AM
APP/0 at Promise._settlePromises (/home/vcap/app/node_modules/botkit-middleware-watson/node_modules/bluebird/js/release/promise.js:691:18)Feb 13, 2017 8:49:13 AM
APP/0 at Promise._settlePromise0 (/home/vcap/app/node_modules/botkit-middleware-watson/node_modules/bluebird/js/release/promise.js:612:10)Feb 13, 2017 8:49:13 AM
APP/0 at Promise._settlePromiseFromHandler (/home/vcap/app/node_modules/botkit-middleware-watson/node_modules/bluebird/js/release/promise.js:510:31)Feb 13, 2017 8:49:13 AM
APP/0 at Object. (/home/vcap/app/bot-facebook.js:120:55)Feb 13, 2017 8:49:13 AM
APP/0 at Promise._fulfill (/home/vcap/app/node_modules/botkit-middleware-watson/node_modules/bluebird/js/release/promise.js:636:18)Feb 13, 2017 8:49:13 AM
APP/0 at tryCatcher (/home/vcap/app/node_modules/botkit-middleware-watson/node_modules/bluebird/js/release/util.js:16:23)Feb 13, 2017 8:49:13 AM
APP/0 at Object.bot.findConversation (/home/vcap/app/node_modules/botkit/lib/Facebook.js:103:13)Feb 13, 2017 8:49:13 AM
APP/0npm ERR! argv "/home/vcap/app/vendor/node/bin/node" "/home/vcap/app/vendor/node/bin/npm" "start"Feb 13, 2017 8:49:13 AM
APP/0npm ERR! Linux 4.4.0-45-genericFeb 13, 2017 8:49:13 AM
APP/0npm ERR<EMAIL_ADDRESS>start: node server.jsFeb 13, 2017 8:49:13 AM
APP/0npm ERR! node v4.7.2Feb 13, 2017 8:49:13 AM
APP/0npm ERR! code ELIFECYCLEFeb 13, 2017 8:49:13 AM
APP/0npm ERR! Exit status 2Feb 13, 2017 8:49:13 AM
APP/0npm ERR! Feb 13, 2017 8:49:13 AM
APP/0npm ERR! This is most likely a problem with the botkit-middleware-watson-multi-bot-example package,Feb 13, 2017 8:49:13 AM
APP/0npm ERR! npm v2.15.11Feb 13, 2017 8:49:13 AM
APP/0npm ERR! node server.jsFeb 13, 2017 8:49:13 AM
APP/0npm ERR! Tell the author that this fails on your system:Feb 13, 2017 8:49:13 AM
APP/0npm ERR! npm owner ls botkit-middleware-watson-multi-bot-exampleFeb 13, 2017 8:49:13 AM
APP/0npm ERR! not with npm itself.Feb 13, 2017 8:49:13 AM
APP/0npm ERR! npm bugs botkit-middleware-watson-multi-bot-exampleFeb 13, 2017 8:49:13 AM
APP/0npm ERR! There is likely additional logging output above.Feb 13, 2017 8:49:13 AM
APP/0npm ERR! Or if that isn't available, you can get their info via:Feb 13, 2017 8:49:13 AM
APP/0npm ERR! Feb 13, 2017 8:49:13 AM
APP/0npm ERR! Please include the following file with any support request:Feb 13, 2017 8:49:13 AM
APP/0npm ERR! /home/vcap/app/npm-debug.log

Hi,
The output must adhere to the format of Conversation Service's output, which is a list of Strings.
Also, unless you change the middleware.interpret's callback function, you don't have to change the output yourself. You just have to return the Conversation response object in the after function.
Let me know if you have more questions,
Steven

Hello People, sorry but i use this way to request some help to have votes for my idea here in my country to use Watson and the power of bluemix to help the tourism in my country. So, could you bring me your vote.
http://comunidad.socialab.com/ideas/ver/31815
Thanks a lot.


[tests-only] Make sharing and providerState helper functions modular/flexible

This PR refactors sharingHelper and providerStateHelper functions to be more flexible and modular

each function for each share (user, group, public)
includes possible share options

closes https://github.com/owncloud/owncloud-sdk/issues/786

lgtm :+1:


closes #2 by enabling html5Mode

Is there any cons of enabling it?
Does that means less browser support? (I mean, angularjs already requires an updated browser)

Not a problem, for old browsers there is a fallback mechanism. Not that we need that since we're working with angular material that only supports evergreen browsers.


Sorbet: init

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

This PR adds the sorbet/ directory and typed sigil to all files in our codebase. Some tests are expected to fail ATM, I am pushing changes to fix them.

Also: is there any other way to specify typed/not than at the top of every file (e.g. a configuration file that lists all files)? Is/can there be a default e.g. default to false/true/ignore?

Sorbet supports YAML configuration for this, which seems like a good option: https://sorbet.org/docs/cli#overriding-strictness-levels

Sorbet supports YAML configuration for this, which seems like a good option: sorbet.org/docs/cli#overriding-strictness-levels

This would also not interrupt with test/support/fixtures, which is a good thing. I will start working on this then!

Nice work, thanks @vidusheeamoli!


libnifalcon: init at 1.1

Motivation for this change
A library for development using the Novint Falcon haptic device.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] 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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Closing due to unresponsive author. Feel free to reopen the discussion.

Closing due to unresponsive author. Feel free to reopen the discussion.


Use correct class name on installation template

This is related to https://github.com/hschne/rails-mini-profiler/issues/4
Maybe a before :suite to run the installer before the tests is a good idea to catch installer errors.

Thank you! :raised_hands:
Testing the install generator is definitely on my list of things to do :+1:


Remove extraneous text in header

Noticed there was some extraneous text in the header while looking at the docs.

Definitely aliens.


Mixed chart layout issue

I am getting an issue where using a mixed chart (bar with second dataset of line). The first bar is getting draw over the y axis when using a 'time' x axis. Using just bar, or just line for both works, but in mixed the issue is present.

Expected Behavior

First column should be inline with first tick.

Current Behavior

Bar is overlapping the y ticks.

Steps to Reproduce (for bugs)

'bar' type of chart
Two datasets, one set to 'bar', the second to 'line'
xAxes scale set to 'time'

Environment

Chart.js version: 2.4.0
Browser name and version: Chrome 56

Orijinal repodaki issueya link koyar misiniz?

https://github.com/chartjs/Chart.js/issues/3913

ok


Prefill done, but Linux Steam Client gets no hit [Nativ Linux Steam Games]

Could it be that prefill only loads the Windows versions of Steam games?
Tested Game: Counter Strike 2
Mostly that's no Problem because you have the Proton layer and the Linux Steam Client loads the Windows game.
But for native Linux games you would need the prefill option "load Linux version"
Additional Details

Platform you are running SteamPrefill on:  Ich777's Unraid App
Region : [Germany]

You'll want to use the --os flag to specify both Linux and windows. See prefill docs for the full list of options

Thank you very much and sorry for wasting your time. I hadn't seen that section yet!

No worries at all!

Once more: is it possible to download windows AND linux Versions with this paramter, or only Linux OR Windows?
"--os linux, windows" doesn't work?

You can do both. Try it without the comma

https://github.com/ich777/docker-lancache-prefill/issues/9
I crossposted the problem, because I don't know what is the correct repo to post. Sorry

The prifill App says all the Files are up to date, but if I open Steam, there are no hits.
There seams to be a bug in the linux update path. Or I do still something wrong.
If I now uninstall CS2 from Steam, and reinstall it, I get the hits. Until there's a new update. Then I have to download the update again via Steam, because prefill says all up to date!

I've been able to reproduce this using CS2 on Ubuntu. I'll try to figure out what's going on here

I've confirmed that what you're seeing isn't the Linux version of the game, but its actually the shader predownloading that Steam now does for Linux.
<IP_ADDRESS> / - - - [18/Jan/2024:03:49:15 +0000] "GET /depot/730/manifest/1428942837211374601/5/9103427991922747470 HTTP/1.1" 200 84666 "-" "Valve/Steam HTTP Client 1.0" "MISS" "cache15-iad1.steamcontent.com" "-"
[steam] <IP_ADDRESS> / - - - [18/Jan/2024:03:49:15 +0000] "GET /depot/730/chunk/2866c3a8c143e48bd09f6a595dea83ddd69d6933 HTTP/1.1" 200 30640 "-" "Valve/Steam HTTP Client 1.0" "HIT" "cache15-iad1.steamcontent.com" "-"
[steam] <IP_ADDRESS> / - - - [18/Jan/2024:03:49:15 +0000] "GET /depot/730/chunk/9c1ccd54c3a5b296d8d514cef99f0ffb386f8c5b HTTP/1.1" 200 35872 "-" "Valve/Steam HTTP Client 1.0" "HIT" "cache15-iad1.steamcontent.com" "-"
[steam] <IP_ADDRESS> / - - - [18/Jan/2024:03:49:15 +0000] "GET /depot/730/chunk/b7827f00810fe9bc3131bc65858bd8b3e45adb39 HTTP/1.1" 200 30080 "-" "Valve/Steam HTTP Client 1.0" "HIT" "cache15-iad1.steamcontent.com" "-"

Access log that shows app 730 downloading a manifest for what should be an app.
access.log

I've updated the issue title to better reflect whats going on here : SteamPrefill isn't downloading any shaders for the SteamDeck.
I'm going to close this issue out as I couldn't figure out exactly how Steam determines which shaders to download, as well as that this project is aimed at LAN parties where most machines are going to be running Windows.
I am however open to merging in the changes to support this if anyone is willing to put in the effort to figure out how to implement shader downloading.


Update ug dg rahul

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master      #79   +/-   ##
=========================================
  Coverage     68.07%   68.07%
  Complexity      662      662
=========================================
  Files           112      112
  Lines          2111     2111
  Branches        230      230
=========================================
  Hits           1437     1437
  Misses          588      588
  Partials         86       86

Impacted Files
Coverage Δ
Complexity Δ

.../seedu/address/logic/commands/FindItemCommand.java
100.00% <100.00%> (ø)
8.00 <0.00> (ø)

Continue to review full report at Codecov.

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


Auto-enable Dynamic Imports for CJS files in Next.js projects

Describe the bug
Next.js has started using dynamic imports in CJS files, which is causing certain Next.js projects to fail.
Link to the blitz that caused the error
https://stackblitz.com/github/vercel/next.js/tree/canary/examples/with-i18n-next-intl
Steps to reproduce
For example, here is one of their official examples that is now breaking: https://github.com/vercel/next.js/tree/canary/examples/with-i18n-next-intl
And here's the working version w/ ENABLE_CJS_IMPORTS flag turned on: https://stackblitz.com/edit/github-8ezthy?file=.stackblitzrc
Expected behavior
IMO we should auto enable CJS imports if we detect a certain project is using Next.js (or even better, somehow generically check files for dynamic import statements before evaluation).
Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 96.0.4664.55
Major version = 96
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.55 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 59243069,
  "usedJSHeapSize": 55549529,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 381d8e23c5bf5b1d29278d4e63be21122d702b6e

Additional context
No response

I believe we can close this now? As CJS is enabled by default.


Add validation check to block duplicate title_abbrev in the same language:version.

This applies during late validation, so merging tributaries can cause it, even if it doesn't happen in each tributary.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Done. Test in https://github.com/awsdocs/aws-doc-sdk-examples/pull/6659


Sunburst Label overlapping last node

Is there any way to control the Label to not appear on the inner arc, but to use the outer arc for where it needs to begin drawing?  I am running into the current scenario.
Sandbox Link

Poking around the source code I swapped the getRadius0 call to getRadius to use the row's radius.  Would this be ideal to have as a prop for the user to be able to change?  I could open a PR for this feature if so.


Update to target release date of Mixed Content Download Block

Updating due to Chrome updates to their release schedule from targeting Chrome 82 to Chrome 84

#sign-off

#sign-off and merged.


use system line feed for inserted lines

grunt-injector always inserts lines with unix style line feed '\n'. This breaks our build on windows computers, we want to concat all

We experienced problems when writing the file, enforcing a write solved the problem


[MLS-272] Fix Special Token Encode Difference

Currently in MLC Serve, we may encounter slightly different input tokens compare to expected due to the handling of special tokens. Take the recent codellama 70b model as an example, in the official document, a chat example is given as follows:
chat = [
    {"role": "system", "content": "System prompt    "},
    {"role": "user", "content": "First user query"},
    {"role": "assistant", "content": "Model response to first query"},
    {"role": "user", "content": "Second user query"},
]

After tokenizer's template application tokenizer.apply_chat_template(chat, tokenize=False) it becomes
'<s>Source: system\n\n System prompt <step> Source: user\n\n First user query <step> Source: assistant\n\n Model response to first query <step> Source: user\n\n Second user query <step> Source: assistant\nDestination: user\n\n '

And the reference of generated token is:
reference_tokens = [1, 7562, 29901, 1788, 13, 13, 2184, 9508, 32015, 7562, 29901, 1404, 13, 13, 3824, 1404, 2346, 32015, 7562, 29901, 20255, 13, 13, 8125, 2933, 304, 937, 2346, 32015, 7562, 29901, 1404, 13, 13, 6440, 1404, 2346, 32015, 7562, 29901, 20255, 13, 14994, 3381, 29901, 1404, 13, 13, 29871]

However, in our implementation, even if the generated prompt is the same
'<s>Source: system\n\n System prompt <step> Source: user\n\n First user query <step> Source: assistant\n\n Model response to first query <step> Source: user\n\n Second user query <step> Source: assistant\nDestination: user\n\n '

The generated token has an extra token 1 in the beginning
[1, 1, 7562, 29901, 1788, 13, 13, 2184, 9508, 32015, 7562, 29901, 1404, 13, 13, 3824, 1404, 2346, 32015, 7562, 29901, 20255, 13, 13, 8125, 2933, 304, 937, 2346, 32015, 7562, 29901, 1404, 13, 13, 6440, 1404, 2346, 32015, 7562, 29901, 20255, 13, 14994, 3381, 29901, 1404, 13, 13, 29871]

The difference comes from the different usage of the add_special_tokens option when doing tokenizer encoding. The transformers lib's apply_chat_template function uses add_special_tokens=False as default when doing tokenization, code can be found here. This PR uses the same option default and can get the same generated token as official reference code.
Also it remains a question whether we should adopt apply_chat_template function as a potential way to do chat template application and tokenization using tokenizer directly.
CC @sunggg

Comments addressed, please take another look @sunggg


Chrome Web Store - Not Currently Available

This extension should be submitted to the Chrome Web Store.

@eavab @bctt @reharri7 I think it's time that the Justin Bahn Design Extension is brought to the world. I've paid the $5 extension developer fee and will be submitting the extension to the public Chrome Web Store.


cant find private key

hi
Public key of F0001000000000001 is
04F50CF235C38DAF3106727063CC3EE9EFC631A08F99F77C6A5B8AAA03D9007DBB439209AE5FB59096FCC5874ED940906D2580928CF74028C1C3E1F5AF496D1414
i added in 1.txt
then run this code
python bsgs_hybrid_dll_secp256k1.py -pfile 1.txt -b bPfile.bin -bl bloomfile.bin -n<PHONE_NUMBER>00000 -keyspace F0001000000000001:F0001000500000001
result : bsgs found private key F0001000000000001
but when i add the Public key of F0001000000000002 to 1.txt .
It cant find it
why?

@iceland2k14
04CB2DA061F7C5A8178651A0BBA315B7AF83CBA3EB4EC7553F81A1F4C58C7BB3613C6758765787DC03C7D079169A780A9421C902DB098B1EE7BB527954BC50C318
cant find this .

@iceland2k14
04CB2DA061F7C5A8178651A0BBA315B7AF83CBA3EB4EC7553F81A1F4C58C7BB3613C6758765787DC03C7D079169A780A9421C902DB098B1EE7BB527954BC50C318
cant find this .

@@iceland2k14

Please see example in v6_dll_bsgs


Some fixes related to ISPN-658, these should only be about logging and should not change functionality (I hope)

ISPN-1469 looks like it should break some stuff, but it seems the affected commands are invoked with RpcManager.broadcastRpcCommand() and all exceptions are swallowed.

Galder, I don't think I can really improve the logging in StateTransferLockImpl but I found another NPE in the logs and I added it to this pull request. It was during commit so it was also ignored, just polluting the logs.

Ok Dan, thanks. It's integrated now and I'll see if it improves those random issues in the Hot Rod testsuites.


tests added

Codecov Report

Merging #39 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   99.64%   99.64%
=======================================
  Files           4        4
  Lines         280      280
=======================================
  Hits          279      279
  Misses          1        1

Continue to review full report at Codecov.

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

@yzainee Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-39


Add missing symbol functions in dbghelp.rs

Thanks for this project!
I'm trying to add windows support to rbspy using winapi-rs - and I found that I needed a couple of symbol handling functions that weren't already exported. This PR adds the missing symbol handling function definitions from dbghelp32
Let me know if anything needs changed here,


fix bug: get profile api cannot be accessed #1882

To solve this problem #1882,  I modified the route of getprofile API as /profiles

@zyxxoo Do you think this issue may be caused by upgrading jersey?

https://github.com/apache/incubator-hugegraph/pull/1893


file's mime type defaulting to documents

Hi

I tried to upload a csv file using gdrive but it keeps defaulting to document instead of spreadsheet
I tried the normal command [gdrive upload] and still the same
I even use the options --mime (force mime type) still no difference

Can you help me to upload my csv file from linux to automatically convert it to "application/vnd.google-apps.spreadsheet" or google sheet.
Thank You

Seems to be related: https://github.com/prasmussen/gdrive/issues/118
Try uploading the csv manually in your browser & check the mime type. In my case, using a signature file, the mime type was correctly set to application/octet-stream, which means that gdrive seems to be the problem.

Hi thank you. I had tried to resolve it by comparing the scenario between my mac and my linux. if i upload  a file on my linux, the file would only be available on google docs but if i upload it using my mac i can open this file on google apps available.
but the options "--mime" doesn't work for spreadsheet it only forces the file to be its original mime
anyway I made an alternatives to this issue. thank you so much


Fix #640 Update whois server list

Discussing before I complete it

Do we want to maintain both whois functions in Custom and Email Adapters or just replace it with for example: https://github.com/io-developer/php-whois/blob/master/src/Iodev/Whois/Configs/module.tld.servers.json

If we want to maintain it we should update the full list and add the missing ones?

This is about 15% done and then I got fed  up with it.  I am considering updating the whois functions to a more up to date whois library

Replaced by #655


Revert "[cleanup]: refactor code to show the use of fakeClock explicitly in pkg/queue(#3380)"

Sorry for the mess maintainers!!

cc: @tenzen-y


Fix/accesibility pagination

Codecov Report

Merging #68 into master will decrease coverage by 7.49%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   94.25%   86.76%   -7.50%
==========================================
  Files          12       13       +1
  Lines         331      393      +62
  Branches       71       86      +15
==========================================
+ Hits          312      341      +29
- Misses         19       52      +33

Impacted Files
Coverage Δ

src/Pagination/components/BreakView.ts
100.00% <100.00%> (ø)

src/Pagination/components/PageButton.ts
100.00% <100.00%> (ø)

src/Pagination/components/PageNumberButton.ts
100.00% <100.00%> (ø)

src/Pagination/components/PageNumberView.ts
100.00% <100.00%> (ø)

src/Pagination/components/Pagination.ts
100.00% <100.00%> (ø)

src/DropDownSort/components/DropDownSort.ts
87.50% <0.00%> (ø)

src/Pagination/utils/ContainerUtils.ts
29.78% <0.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #68 into master will decrease coverage by 7.62%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   94.25%   86.63%   -7.63%
==========================================
  Files          12       13       +1
  Lines         331      389      +58
  Branches       71       84      +13
==========================================
+ Hits          312      337      +25
- Misses         19       52      +33

Impacted Files
Coverage Δ

src/Pagination/components/PageSizeSelect.ts
78.37% <ø> (ø)

src/Pagination/components/BreakView.ts
100.00% <100.00%> (ø)

src/Pagination/components/PageButton.ts
100.00% <100.00%> (ø)

src/Pagination/components/PageNumberButton.ts
100.00% <100.00%> (ø)

src/Pagination/components/PageNumberView.ts
100.00% <100.00%> (ø)

src/Pagination/components/Pagination.ts
100.00% <100.00%> (ø)

src/DropDownSort/components/DropDownSort.ts
87.50% <0.00%> (ø)

src/Pagination/utils/ContainerUtils.ts
29.78% <0.00%> (ø)

Continue to review full report at Codecov.

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


Token indices sequence length is longer than the specified maximum sequence length for this model

When I try to use this code with he sdxl model I get this:
Token indices sequence length is longer than the specified maximum sequence length for this model (176 > 77). Running this sequence through the model will result in indexing errors
The code I use is:
def prompt_worker_sdxl(prompt,negative_prompt,pipe_sdxl):
   compel = Compel(tokenizer=[pipe_sdxl.tokenizer, pipe_sdxl.tokenizer_2],
                       text_encoder=[pipe_sdxl.text_encoder, pipe_sdxl.text_encoder_2],
                       returned_embeddings_type=ReturnedEmbeddingsType.PENULTIMATE_HIDDEN_STATES_NON_NORMALIZED,
                       requires_pooled=[False, True],
                   truncate_long_prompts=False)

   with torch.no_grad():
       conditioning, pooled = compel([prompt,negative_prompt])
   return conditioning, pooled

the pipelines I've tested are:
pipe_sdxl_cntrlnt = StableDiffusionXLControlNetPipeline.from_pretrained(
    "stabilityai/stable-diffusion-xl-base-1.0",
    controlnet=controlnet_sdxl,
    vae=vae_sdxl,
    torch_dtype=torch.float16,
)
pipe_sdxl = StableDiffusionXLPipeline.from_pretrained(
    "stabilityai/stable-diffusion-xl-base-1.0",
    vae=vae_sdxl,
    torch_dtype=torch.float16,

)

with this schedulers:
pipe_sdxl_cntrlnt.scheduler = DPMSolverMultistepScheduler(use_karras_sigmas=True).from_config(pipe_sdxl_cntrlnt.scheduler.config)
pipe_sdxl_cntrlnt.scheduler.algorithm_type = 'sde-dpmsolver++'
pipe_sdxl.scheduler = DPMSolverMultistepScheduler(use_karras_sigmas=True).from_config(pipe_sdxl.scheduler.config)
pipe_sdxl.scheduler.algorithm_type = 'sde-dpmsolver++'

compel is 2.0.2

I have the same issue, did you find a solution ?

No, I decided to not use it and to shorten prompt lenght

ah, not much an option for me.

this isn't an error, the result should be fine..


Allow license to be listed in the dynamic section

Meson already has project(license:). Would it be possible to autodetect it dynamically? For instace, I have 'Apache-2.0 OR MIT' as the value in project(). I would like this to populate the license = { text = "XXX" } metadata like this project already does with version.

Thanks for the suggestion! However, I think the semantics are a little bit different, XXX in your example would be the actual license text, so "Apache-2.0 OR MIT" wouldn't be the correct value. There's no way to specify that currently, but hopefully there will be with PEP 639, which adds a License-Expression core metadata field and assigns the value of project.license in pyproject.toml to the expression. So, this is another issue where we are blocked until that PEP, or an alternative, lands.

It's unfortunate this wasn't thought of earlier when the pyproject.toml was designed. Seems so fundamental, but hopefully the PEP is accepted.
For now, I'll just leave it as is since I see no better way to express dual licensed software.

My recommendation for that case would be to have a single license file with both licenses, and specifying it in pyproject.tml. But I agree, it's unfortunate, hopefully the PEP will land though and fix this.

Unfortunately that wouldn't be REUSE compliant from what I understand.
https://reuse.software/

I didn't know about REUSE, thank you for pointing it out. I've only skimmed it a bit, but I can't tell how it wouldn't be compliant? You can still have the SPDX headers, you'd just have an extra license file for the Python tooling to read. In fact, using the REUSE spec you should even be able to automatically generate this file! Am I missing something here? :rofl:

Hmm touch LICENSE in the root doesn't seem to cause the tool to fail. Oh well. Just throw REUSE on the pile of reasons that PEP should be advanced.

Agreed. I have pinged the discourse thread. Let's see if we can get it to move forward.
(cc @CAM-Gerlach)

Thanks for your efforts.

Sorry, finishing what is hopefully the last major update got stuck beyond a pile of other things that came up over the past few months, but I've now got most of the next rewrite complete and it's near the top of my priority list to finish, hopefully in the next few days :crossed_fingers: I can also mention REUSE compliance and this issue in the rationale and on the thread.

@CAM-Gerlach no problem whatsoever. At least someone thought about it in the first place!

Yeah, but to be fair that person with the original idea wasn't me—credit for that is due to @pombredanne

Could one of y'all link to the discourse thread where I presume discussion of the referenced PEP is? I am not familiar with the Python PEP process.

It's here :blush:
https://discuss.python.org/t/pep-639-round-2-improving-license-clarity-with-better-package-metadata

It's here :blush:
https://discuss.python.org/t/pep-639-round-2-improving-license-clarity-with-better-package-metadata

Meson 1.1 now has license_files, which it would be nice to automatically distribute as a set of license files (especially if you have a dependency subproject that may be bundled in a wheel, but not if built in, e.g.,  a Linux distro).

Thanks @QuLogic. That does look potentially useful, especially for the subproject case you mention, since that cannot be done from pyproject.toml. I think we're still waiting for PEP 639 to land first though? Not sure what we'd do with license_files right now.

PEP639 is now provisional, i.e., waiting for two build backends to implement it. meson-python could be one of those two.


Update Posts “test-post”

Automatically generated by Netlify CMS

👷 Deploy Preview for shimmering-marigold-4f5b4a processing.

Name
Link

🔨 Latest commit
81c647210180fc8cb4d0d0006f8efff3eb70524b

🔍 Latest deploy log
https://app.netlify.com/sites/shimmering-marigold-4f5b4a/deploys/64ff8e57a0a1c400080af916


Various findings after test

This issue is imported from pivotal
Various small findings.

(Arnout Terpstra - Jan 13, 2020)


[bug] Lifecycle error occurred when registering plugins after migrating from v1 to v2

Describe the bug
When migrating from v1 to v2 using the CLI, after registering the 'shell' plugin, the compiler reports a lifecycle error when running the program with tauri dev.
Creating a project directly with v2 does not produce this compilation error.
error[E0107]: struct takes 0 lifetime arguments but 1 lifetime argument was supplied
   --> /Users/thepoy/.cargo/git/checkouts/plugins-workspace-fd0259fcb18f51ad/8645a02/plugins/shell/src/commands.rs:106:20
    |
106 |     command_scope: CommandScope<'_, crate::scope::ScopeAllowedCommand>,
    |                    ^^^^^^^^^^^^ -- help: remove this lifetime argument
    |                    |
    |                    expected 0 lifetime arguments
    |
note: struct defined here, with 0 lifetime parameters
   --> /Users/thepoy/.cargo/registry/src/mirrors.ustc.edu.cn-4affec411d11e50f/tauri-2.0.0-beta.3/src/ipc/authority.rs:364:12
    |
364 | pub struct CommandScope<T: ScopeObject>(ScopeValue<T>);
    |            ^^^^^^^^^^^^

error[E0107]: struct takes 0 lifetime arguments but 1 lifetime argument was supplied
   --> /Users/thepoy/.cargo/git/checkouts/plugins-workspace-fd0259fcb18f51ad/8645a02/plugins/shell/src/commands.rs:107:19
    |
107 |     global_scope: GlobalScope<'_, crate::scope::ScopeAllowedCommand>,
    |                   ^^^^^^^^^^^ -- help: remove this lifetime argument
    |                   |
    |                   expected 0 lifetime arguments
    |
note: struct defined here, with 0 lifetime parameters
   --> /Users/thepoy/.cargo/registry/src/mirrors.ustc.edu.cn-4affec411d11e50f/tauri-2.0.0-beta.3/src/ipc/authority.rs:404:12
    |
404 | pub struct GlobalScope<T: ScopeObject>(ScopeValue<T>);
    |            ^^^^^^^^^^^

For more information about this error, try `rustc --explain E0107`.

Reproduction
#[tokio::main]
async fn main() -> AlleyResult<()> {
    // ...

    tauri::Builder::default()
        .plugin(tauri_plugin_shell::init())
        .invoke_handler(tauri::generate_handler![
            // ...
        ])
        .run(tauri::generate_context!())?;

    Ok(())
}

[build-dependencies]
tauri-build = { version = "2.0.0-beta", features = [] }

[dependencies]
tauri = { version = "2.0.0-beta", features = [] }
tauri-plugin-shell = { git = "https://github.com/tauri-apps/plugins-workspace", branch = "v2" }
tokio = { version = "1", features = ["macros"] }
# ...

Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.3.1 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.76.0 (07dca489a 2024-02-04)
    ✔ cargo: 1.76.0 (c84b36747 2024-01-18)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 20.11.1
    - pnpm: 8.15.1
    - npm: 10.2.4
    - bun: 1.0.22

[-] Packages
    - tauri [RUST]: 2.0.0-beta.3
    - tauri-build [RUST]: 2.0.0-beta.2
    - wry [RUST]: 0.36.0
    - tao [RUST]: 0.25.0
    - @tauri-apps/api [NPM]: 1.5.3
    - @tauri-apps/cli [NPM]: 2.0.0-beta.2

[-] App
    - build-type: bundle
    - CSP: unset
    - devUrl: http://localhost:1420/
    - framework: SolidJS
    - bundler: Vite

Stack trace
No response
Additional context
No response

The shell plugin is currently not updated to beta.3. It needs to be removed temporarily from your dependency.
https://github.com/tauri-apps/plugins-workspace/pull/971

Just a heads up that this issue even breaks developing a fresh application using tauri-create-app yarn create tauri-app --beta.

Just a heads up that this issue even breaks developing a fresh application using tauri-create-app yarn create tauri-app --beta.

Yes probably due to the PR above not being merged. I guess the create-tauri-app scripts take all latest versions, that are not compatible between each other right now

Just a heads up that this issue even breaks developing a fresh application using tauri-create-app yarn create tauri-app --beta.

Yes, When I follow Create a Project to make a new tauri-2.0.0 project, encountered the same problem.


use default python first

You say in the README that "The default python will be used when get-pipenv is called. The default python is used for all other pipenv calls." Currently, python3 is tried first, and then python (the default).

Should we update the documentation to match python3 (And all other code places) or the code to try python first?

I would update the code. Python2 is not deprecated yet. And systems are moving the default python to be python3 slowly, so this will fix itself.

I don't think we need to favor python 3. Python 2 is not deprecated yet. And because distros are slowly changing the default python to python 3, then this situation will resolve itself---i.e., python3 will be favored by default.

This is not about deprecating python2, if it was, python2 would have been removed. I think we should be favoring python3 always, we have passed the tipping point, and should not look back to python2 excepts for compatibility reasons.
Consider what this suggested order means:

First try python. In other-words it is less predictable which python gets called. Some OS still link python to python2 when python3 is installed too. This means we are trying the least predictable behavior first.
Second try python3 Now we know we are trying pyhon3 for sure, which we already placed ahead of the third option
Third, try python2. If there is no python3 at this point, just use python2.

@decrispell

I personally think defaulting to whatever the  "python" command resolves to is the most intuitive / least surprising.  On the other hand, pipenv itself seems to default to python3, even when "python" on the host gives you 2.7, so there is good precedent for the current behavior.  I don't think it's too big a deal either way, as long as there's an easy way for the user to specify which version they actually want.

pipenv does not default to python3. It defaults to default python.

Odd. I get this on my mac:
~/test $ python --version
Python 2.7.14

~/test $ pipenv install
Creating a virtualenv for this project...

(test-CrH5_LA2) ~/test $ pipenv shell
Launching subshell in virtual environment…
bash-3.2$  . /Users/sgrichar/.local/share/virtualenvs/test-CrH5_LA2/bin/activate

(test-CrH5_LA2) bash-3.2$ python --version
Python 2.7.14

What does pip --version say?

Also, what does pipenv --version say?
@scott-vsi Do you get same on a not a mac?

On my mac:
$ pipenv --version
pipenv, version 2018.7.1

$ pip --version
pip 18.0 from /usr/local/lib/python2.7/site-packages/pip (python 2.7)

head -n1 "$(which pipenv)" ?

and echo PIPENV_PYTHON?

This script is specific to favor python3, and in 26 more days it won't matter
https://pythonclock.org/
So lets leave it how it is.


ksp audit cassandra stig v 72671

Based on the reference provided approving the policy
cc: @syeddhadi @raviknox @salman-accuknox


add tests for scrape_metrics_for_device

fix this:
https://github.com/PhilipMay/smart-prom-next/blob/fdc4a96da917d663a7e2dcbd8c9a9ab4e9b9bf66/tests/test_scrape_metrics_for_device.py#L12-L14

fixed above with comment

see https://github.com/prometheus/client_python/discussions/828

done


Typo in Jira Feature Collector Java Tests

The folder "Hygieia/jira-feature-collector/src/test/java/com" has a capitalone directory as well as an incorrectly named captialone directory.

Closed. Download latest code.


added return type String to serialize()

added return type String to serialize() in data_encryption_key.dart

Checks failure fixed in another PR

Checks failure fixed in another PR


Markdown link underline appears above its line

Type: Bug
I noticed the underline appeared 3 lines above the link's line IIRC. This fixed itself once I scrolled.
 Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

/gifPlease

i wish i thought about recording it before scrolling

Closing as without a screenshot of the issue (or steps to reproduce it) I can't tell what might be happening here


[BEAM-2798] rename simpleQuery to query and query to queryMulti

R: @takidau @xumingming

retest this please

Changes Unknown when pulling c580b52198d475dd56272bdf4d751991c0c51a15 on XuMingmin:BEAM-2798 into ** on apache:DSL_SQL**.

close #3752. Thanks @takidau


RYA-73 added timestamp to mongoDB DAO

Changed the mongo storage strategy to now
include timestamp in the [de]serialization
of RyaStatements.
The timestamp is added at insertion timein the DAO
Added test for [de]serialize
Added test for timestamp in the DAO
Added failsafe to mongo rya for integration test framework
Jira

Hi @isper3at ,
While running apache-yetus pre-commit checks with this PR manually, I am getting following warnings,
"The patch has 6  line(s) that end in whitespace."
"The patch 48 line(s) with tabs."
Seems like this PR contains those formatting issues.

updated, should be fixed.

looks good.  will merge tomorrow.  good job adding tests.

oh kind of not related to this pr, but should we consider adding context to the mongo composite indices?

merged.


chore: test-python-jenkinsx to 0.0.5

chore: Promote test-python-jenkinsx to version 0.0.5

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


livenessProbe and readinessProbe cause errors in log

Kubernetes livenessProbe and readinessProbe cause that ubiquity container logs errors every readinessProbe period (default 5 secs). Probes are defined as default as:
readinessProbe:
  tcpSocket:
    port: 9999

Probes make these lines in ubiquity pod log:
2019/04/18 10:48:18 http: TLS handshake error from <IP_ADDRESS>:48322: EOF
2019/04/18 10:48:23 http: TLS handshake error from <IP_ADDRESS>:48334: EOF
2019/04/18 10:48:28 http: TLS handshake error from <IP_ADDRESS>:48368: EOF
2019/04/18 10:48:33 http: TLS handshake error from <IP_ADDRESS>:48380: EOF
2019/04/18 10:48:35 http: TLS handshake error from <IP_ADDRESS>:48392: EOF
2019/04/18 10:48:38 http: TLS handshake error from <IP_ADDRESS>:48414: EOF
...

Similarly probes cause errors in ubiquity-db pod log every 2 second:
LOG:  incomplete startup packet
LOG:  incomplete startup packet
LOG:  incomplete startup packet
LOG:  incomplete startup packet
LOG:  incomplete startup packet

@horakmar Any updates on this? I'm also experiencing the same issue.


Add SKIP_IAP_VERIFICATION magic value.

Setting this for IAP_BACKEND_SERVICE_ID will skip verification of the audience, allowing us to avoid the circular dependency from #324, although it will still validate that the JWT audience contains the correct project.
This option should be used by clients who have already verified that the caller can only recieve traffic already validated by IAP and that the project does not have any other backend services with less secure IAP settings.

I'm not the best at Java, so I'd appreciate it if you were willing to make these changes. Thank you so much for your help with this!

Created #450, could you give this one a try?

looks great - thank you!

#450 is merged, so I'll close this one.


Update Xcode Code Signing

Issue Description
I have a white-label app project on iOS that needs to use different provisioning files and certificates for specific environments. I've successfully integrated match with a git repo and different branches per each project, it creates the new certificates and provisioning if needed, otherwise fastlane download and install them.
The issue I have is that when fastlane is compiling with gym seems that the project settings hasn't been updated after match action has run without errors. The gym action throws the following error:
Code Signing Error: No profile for team 'XXX' matching 'YYY' found:  Xcode couldn't find any provisioning profiles matching 'XXX/YYY'

Where XXX is the team id, YYY the name of the provisioning file. The issue is that those values are not the new ones downloaded and installed by match action, they are the old ones that are set by default by our Xcode project.
Is there anything I should do in order to update the Xcode settings? If run fastlane using the default environment settings, the match action downloads and install cert and provisioning files as well, and the gym action runs without any issues.

The code signing guide above should yield the answers you're looking for


Add '-Fsystem' framework search option to indicate path for frameworks that should be treated as 'system'

This has the effect of propagating the search path to the clang importer as '-iframework'.
It doesn't affect whether a swift module is treated as system or not, this can be done as follow-up enhancement.

@swift-ci smoke test

@jrose-apple could you review ?

@swift-ci smoke test

I have a commit to update lldb, I'll apply after merging this.

Filed JIRA for the DYLD_FRAMEWORK_PATH issue: https://bugs.swift.org/browse/SR-3967


CKA speedups

added support for torch>=2.0.0 and torchvision=0.15.* (fixes issue #165)
implemented CKA in NumPy and PyTorch (torch is 2x faster than NumPy)
added CUDA support for CKA in PyTorch (100x faster than CKA in NumPy)
added new function called get_cka where user can set backend (torch or numpy)

Just wondering while you're "in the zone" with refactoring CKA, maybe you can add the debiased version of CKA? It's an alteration of HSIC and only a minor change, already given in the PyTorch implementation of Simon's notebook demo:
https://github.com/google-research/google-research/blob/master/representation_similarity/Demo.ipynb
I have some recent work looking at some issues with the regular formulation of CKA and that the debiased/unbiased version might actually be what people need to be looking at in some cases:
https://openreview.net/forum?id=E1NRrGtIHG
Super, super busy at the moment otherwise I would add it. I still might, if you can't, but the changes to be made are in the notebook above and should only require an extra parameter to the CKA call.

Just wondering while you're "in the zone" with refactoring CKA, maybe you can add the debiased version of CKA? It's an alteration of HSIC and only a minor change, already given in the PyTorch implementation of Simon's notebook demo:
https://github.com/google-research/google-research/blob/master/representation_similarity/Demo.ipynb
I have some recent work looking at some issues with the regular formulation of CKA and that the debiased/unbiased version might actually be what people need to be looking at in some cases:
https://openreview.net/forum?id=E1NRrGtIHG
Super, super busy at the moment otherwise I would add it. I still might, if you can't, but the changes to be made are in the notebook above and should only require an extra parameter to the CKA call.

Done.

Great! Thanks for adding that. Code looks nice!

Great! Thanks for adding that. Code looks nice!

You're welcome. Glad to hear you like the code! Could you approve?


get_events_for_block_i_ds and get_events_for_height_range methods fail

Not sure, if it's just me doing something wrong, but I did not manage to make get_events_for_block_i_ds and get_events_for_height_range work.
Example:
async def run_events():
    return await client.get_events_for_height_range(type = "A.c1e4f4f4c4257510.Market.MomentPurchased", start_height = 15141027, end_height = 15241127)

Error:
File "/Users/me/.local/lib/python3.9/site-packages/betterproto/__init__.py", line 640, in _type_hint
    type_hints = get_type_hints(cls, vars(module))

TypeError: vars() argument must have __dict__ attribute

Double checked the official Flow Documentation, it expects a string for the type. I adapted the syntax for the type argument from this article.

Thank you for the report. Let me check this today and get back to you.

I was not able to replicate your problem. the same code worked fine for me. It is likely related to the python setup. I still wish to investigate this further.
Any more Info you can give me would help a lot.
Did you install flow-py-sdk using poetry? which version are you using? how are you running this code? anything really...

So looking at this again. I just want to make sure you doing something equivalent to
from flow_py_sdk import flow_client

# ...

async def run_events():
    async with flow_client( host="some_node_url", port=9000 ) as client:
        return await client.get_events_for_height_range(type = "A.c1e4f4f4c4257510.Market.MomentPurchased", start_height = 15141027, end_height = 15241127)

thanks a lot! I installed it with pip install flow-py-sdk.
Python version: 3.9.5
Ah ok, no I'm doing it a bit differently, let me try again.
from flow_py_sdk import Config
from examples.common import Config #set to mainnet

ctx = Config()
client = flow_client(host=ctx.access_node_host, port=ctx.access_node_port)

Same with poetry add flow-py-sdk.  Now it works :) But I don't know if it was because of the version downgrade or the reinstallation of the environment. Nevertheless, thanks a lot!

I'll give replicating this problem another go on a fresh pc. Which OS are you using @chrfla if you don't mind me asking.

Great! I'm using MacOs 11.2.3.


Ios scan find only 1 device

Hi, around i have multiple device with the same bluetooth name.But when i start the scan with IOS it finds only 1 device.
i try to put a delay of 1 sec after i start the scan, before starting listening the results.
Any solution?

Hello, this plugin is obsolete and maintenance is stopped.
Use the continuous and active package : flutter_blue_plus


Add Lenovo

Backstory
Tried the support forums because I had a problem. They weren't helpful. So I deleted the account.
The Lenovo account can actually be split into two accounts (which is why there are two "data"s): the main one (known as Lenovo ID) and the Lenovo Forums account (which depends on Lenovo ID, creating a forum account automatically creates the Lenovo ID account) (I couldn't actually find an official name for this, the closest is "Community account"). According to this, if you delete Lenovo ID first, you won't be able to delete the forum account.
Also, I don't know if the labels count as "deleting": for Lenovo ID it's "Deactivate Account" (trying to login again says that the account is locked), for the forum it's "Close Account".
By the way, some information might not be 100% accurate: Lenovo's website is a mess, there is barely any documentation on this, there are multiple URLs for managing the profile with different interfaces (I have no idea what passport.lenovo.com (which many sources point to) is supposed to do, since there are basically no options there), it even changes interface and features depending on the language (for example, Italian Lenovo doesn't support login via Auth).
Data

Lenovo ID

name: Lenovo ID
url: https://account.lenovo.com/us/en/account/myprofile.html
difficulty: medium
notes:

Delete the Lenovo Forums account first if you have it, otherwise you won't be able to delete it anymore.
Password is required: if you don't have one (because you signed in with Auth), you need to reset the password with "Forgot Password".
You must manually sign out after deactivation.

domains: www.lenovo.com, account.lenovo.com, passport.lenovo.com

Lenovo Forums

name: Lenovo Forums
url: https://forums.lenovo.com/user/myprofilepage/personal/closeaccount
difficulty: easy
domains: forums.lenovo.com

Since the system is able to recognize a deactivated account, I'll put Lenovo ID as impossible.
Otherwise thanks for submitting a very detailed entry :)


runtime error: invalid memory address or nil pointer dereference

I get the following output when I want to pull the translations within Android Studio.

phraseapp pull
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x5aa7]
goroutine 1 [running]:
main.(*Target).DownloadAndWriteToFile(0xc2080cf200, 0xc20803a340, 0xc2081a1340, 0x0, 0x0)
/Users/gfrey/Development/clients/src/github.com/phrase/phraseapp-client/pull.go:151 +0x13a7
main.(*Target).Pull(0xc2080cf200, 0xc20803a340, 0x0, 0x0)
/Users/gfrey/Development/clients/src/github.com/phrase/phraseapp-client/pull.go:133 +0x3b4
main.(*PullCommand).Run(0xc208099400, 0x0, 0x0)
/Users/gfrey/Development/clients/src/github.com/phrase/phraseapp-client/pull.go:38 +0x199
github.com/dynport/dgtk/cli.(*Router).Run(0xc20802ae60, 0x0, 0x0, 0x0, 0x0, 0x0)
/Users/gfrey/Development/clients/src/github.com/dynport/dgtk/cli/router.go:53 +0x253
github.com/dynport/dgtk/cli.(*Router).RunWithArgs(0xc20802ae60, 0x0, 0x0)
/Users/gfrey/Development/clients/src/github.com/dynport/dgtk/cli/router.go:59 +0x89
main.Run()
/Users/gfrey/Development/clients/src/github.com/phrase/phraseapp-client/main.go:15 +0x3a
main.main()
/Users/gfrey/Development/clients/src/github.com/phrase/phraseapp-client/main.go:10 +0x1b
goroutine 5 [syscall]:
os/signal.loop()
/Users/gfrey/go/src/os/signal/signal_unix.go:21 +0x1f
created by os/signal.init·1
/Users/gfrey/go/src/os/signal/signal_unix.go:27 +0x35
goroutine 10 [runnable]:
net/http.(*persistConn).readLoop(0xc20808e9a0)
/Users/gfrey/go/src/net/http/transport.go:928 +0x9ce
created by net/http.(*Transport).dialConn
/Users/gfrey/go/src/net/http/transport.go:660 +0xc9f
goroutine 17 [syscall, locked to thread]:
runtime.goexit()
/Users/gfrey/go/src/runtime/asm_amd64.s:2232 +0x1
goroutine 11 [select]:
net/http.(*persistConn).writeLoop(0xc20808e9a0)
/Users/gfrey/go/src/net/http/transport.go:945 +0x41d
created by net/http.(*Transport).dialConn
/Users/gfrey/go/src/net/http/transport.go:661 +0xcbc

I installed the command line tool with brew and also updated the path in the preferences of the plugins within android studio.
The same also happens when I try running the tool: phraseapp pull

Could you please update to version 1.1.9 of the CLI client (brew update && brew upgrade should do the trick). This issue was fixed in that revision.

@gfrey thanks that did the trick. Just when I was editing my post and tried to use the CLI tool I wanted to check for updates.
Thank you very much!


[release-1.2] Prober targets service instead of pods directly

Manual cherry pick of https://github.com/knative-sandbox/eventing-kafka-broker/pull/2159
Receiver's prober targets services instead of pods directly to allow components to be part of an Istio mesh

/test unit-tests_eventing-kafka-broker_release-1.2
(https://repo.maven.apache.org/maven2): transfer failed for https://repo.maven.apache.org/maven2/org/apache/maven/plugins/maven-shade-plugin/3.2.1/maven-shade-plugin-3.2.1.pom: Connection reset -> [Help 1]

/test channel-reconciler-tests-ssl_eventing-kafka-broker_release-1.2

/test channel-reconciler-tests-ssl_eventing-kafka-broker_release-1.2
Doesn't look good :(
helpers.go:109: Failed to patch the ChannelImpl {error 26 0  Internal error occurred: failed calling webhook "validation.webhook.kafka.eventing.knative.dev": failed to call webhook: Post "[https://kafka-webhook-eventing.knative-eventing.svc:443/resource-validation?timeout=2s](https://kafka-webhook-eventing.knative-eventing.svc/resource-validation?timeout=2s)": EOF}

/test channel-reconciler-tests-ssl_eventing-kafka-broker_release-1.2
ERROR: timeout waiting for pods to come up
strimzi-cluster-operator-76ff9bcc-2hzdj   0/1   Pending   0     5m59s


Optimize and re-encode SVG Sass variables

I noticed that some of the SVGs still had Sketch meta data and a few frivolous attributes that could be removed. In the process of removing these I discovered that there is a newer method of encoding that can both improve readability and reduce the amount of required characters.
Optimizing SVGs in data URIs
Encode SVG SCSS

May even be smart to just implement this function to be able to story unencoded SVGs as Sass variables (just need to be make sure to always use the function when using one.

Hey Gray! Thanks for this suggestion. Though we are actively moving towards inlining our SVGs, we will take a look and see what we can use from this.

👍 inlining provides significantly more flexibility (and probably boosts accessibility, too)


 To align with Vulndb's recommendations and ensure a secure connection, we have implemented the retrieval of an access token using our client credentials and secret.
Considering that HTTPS handles URL signing automatically, there is no need for explicit URL signing in this implementation. The security aspects are effectively managed by the HTTPS protocol.
Additionally, we have retained the inclusion of a User Agent in our GET request to provide comprehensive information about our VulnDB Data Mirror application.
I am not sure if I should remove the oauth signpost references in pom.xml https://github.com/stevespringett/vulndb-data-mirror/blob/d65c619002e4bc96127ca86c51edfd61b53a4798/pom.xml#L129

Thanks for the PR @LaVibeX. At the moment, I have no way to test this, as I'm waiting for Flashpoint to provide OWASP access to the service again. But the PR looks good to me.


Interesting issue with single argument children alongside multi argument children

Consider the following command tree:
dispatcher.register(literal("goal")
    .then(argument("y", integer())
        .executes(c -> {
            setGoal(new GoalYLevel(
                c.getArgument("y", Integer.class)
            ));
            return 0;
        })
    )
    .then(argument("x", integer())
        .then(argument("z", integer())
            .executes(c -> {
                setGoal(new GoalXZ(
                    c.getArgument("x", Integer.class),
                    c.getArgument("z", Integer.class)
                ));
                return 0;
            })
        )
    )
    .then(argument("x", integer())
        .then(argument("y", integer())
            .then(argument("z", integer())
                .executes(c -> {
                    setGoal(new GoalBlock(
                        c.getArgument("x", Integer.class),
                        c.getArgument("y", Integer.class),
                        c.getArgument("z", Integer.class)
                    ));
                    return 0;
                })
            )
        )
    )
);

This should have 3 possible usages
goal <y>
goal <x> <z>
goal <x> <y> <z>

However, when I try to execute the command with a single integer argument (via goal 1) an exception is thrown!
com.mojang.brigadier.exceptions.CommandSyntaxException: Unknown command at position 6: goal 1<--[HERE]
  at com.mojang.brigadier.exceptions.SimpleCommandExceptionType.createWithContext(SimpleCommandExceptionType.java:21)
  at com.mojang.brigadier.CommandDispatcher.execute(CommandDispatcher.java:283)
  at com.mojang.brigadier.CommandDispatcher.execute(CommandDispatcher.java:178)
  at com.mojang.brigadier.CommandDispatcher.execute(CommandDispatcher.java:143)
  ...

Both the 2-arg and 3-arg usages work as intended:

Interesting.
the CommandDispatcher when parsing looks at all child nodes and tries to parse the first node and add it to potential commands. When the node could not parse, it also collects the exceptions in a list. However, in this case we have 3 integer arguments as child of the literal. Of course, all 3 children will parse without any exceptions.
Later on, the parsing method will try to find the best child. When doing so, it prefers children that parsed the complete command string and then, children that did collect exceptions.
In this case, all 3 children will be evaluated to be suitable.
This means that your example is, as far the command parser is conserned, just as ambiguous as the following would be:
dispatcher.register(literal("goal")
		.then(argument("x", integer())
				.executes(c -> {
					System.out.println(c.getArgument("x", Integer.class));
					return 0;
				})
		)
		.then(argument("y", integer())
				.executes(c -> {
					System.out.println(c.getArgument("y", Integer.class));
					return 0;
				})
		)
		.then(argument("z", integer())
				.executes(c -> {
					System.out.println(c.getArgument("z", Integer.class));
					return 0;
				})
		)
);

Maybe the parse-method should already collect an exception, when the command is not executable and the reader doesn't have anything to read anymore, rather than leaving that for the execute-method.

Accidental duplicate due to GitHub downtime. See #39


Signals

cc @gkoz

https://developer.gnome.org/gtk3/stable/GtkCellRendererText.html ,
https://developer.gnome.org/gtk3/stable/GtkCheckMenuItem.html ,
https://developer.gnome.org/gtk3/stable/GtkContainer.html ,
have children, so <T>

GtkContainer signals can be generated https://gist.github.com/EPashkin/977773a192bf633bb8413a96a1930870, but currently don't contains IsWidget<T> (IMHO needed)

..(forgot add main idea), so if IsA<Widget> not needed and no need in this signals currently, IMHO not need add signals as it simply.

IsA<Widget> couldn't work: the handler can't assume anything about the types of the child widgets, we can't predict anything statically.
More generally, I suspect that the container signals are not really supposed to be used by applications. Note the absence of documentation. I haven't tested this but I have the impression that Box doesn't emit any of Container signals (e.g. packing a widget might not emit the add signal).

@gkoz: Is it really an issue? At least we'll have them if anyone needs them.
@EPashkin: I updated the PR, do you see anything else? (I made CellRendererText, CheckMenuItem and Container "serializable".)

Thanks, sees it all ok.

Is it really an issue? At least we'll have them if anyone needs them.

If someone needs them but they don't match reasonable expectations it is an issue. Can you test with a few containers to see if add and remove are fired as expected?
Regarding someone needing any obscure signal, I think it might be possible to make a dynamically-typed interface for connecting any signals.

@gkoz: It's working (tested on VerticalLayout in process-viewer here). :)

So, can we merge it?

Thanks for testing these signals @GuillaumeGomez


Font size in Popup on iOS

When content is displayed in a popup, if there are images, the font size will be larger. This happens only on iOS and does not happen when there are very few text. Also, if I change the height and width once, it goes away.
Is it possible to fix it?

Are you talking about umap.plot.interactive? Sorry, but it isn't quite clear which component you are talking about.

Sorry, I posted in the wrong thread.


Need Help

Describe the bug
trying to decrypt key
Expected behavior
Hey bro ,
Need small help from your side actually i am trying to make a script that can extract m3u8 from aniwatchtv
i found “source”: content but it is encrypted and it is salted  i need key for that can you tell me how can i find key from site and logic pls :)
Actual behavior
.
Additional context
No response

@jaephefd12 this might be the resource for the keys: https://raw.githubusercontent.com/cinemaxhq/keys/e1/key, but it's not quite working

Can you tell me like in which file there is key on site aniwatchtv like which i can scrape and how can i decrypt it i m trying find way but couldnt find anything
https://raw.githubusercontent.com/cinemaxhq/keys/e1/key
this myb isnt working i tried ur repo but it is saying {"status":500,"message":"Cannot decrypt sources. Perhaps the key is invalid."}
And could you tell like how did u find that cinemaxhq/key/e1/key
Give me some hints so i would try to find it will take not much time to ge… pls

Can you tell me like in which file there is key on site aniwatchtv like which i can scrape and how can i decrypt it i m trying find way but couldnt find anything
https://raw.githubusercontent.com/cinemaxhq/keys/e1/key
this myb isnt working i tried ur repo but it is saying {"status":500,"message":"Cannot decrypt sources. Perhaps the key is invalid."}
And could you tell like how did u find that cinemaxhq/key/e1/key
Give me some hints so i would try to find it will take not much time to ge… pls

So I don't take any credits for the extractor of this API as its directly taken from consumet

I just checked consumet's discord, and many are trying to get the keys working. Try - https://keys4.fun/
@ghoshRitesh12 u should chk the cosumet and anify discord, maybe you'll find a clue about how to fix this scrape issues.

I just checked consumet's discord, and many are trying to get the keys working. Try - https://keys4.fun/
@ghoshRitesh12 u should chk the cosumet and anify discord, maybe you'll find a clue about how to fix this scrape issues.

I am already in their discord and I'll try the one you mentioned.

@animemandir, @jaephefd12 I have added another extractor for getting anime episodes. The response is a bit all over the place, I will arrange it sometime.

it works decent. Server switching is not possible ig. idk why other servers wont work. it gives response only with the default server. Which gives only sub as default, this makes switching to dub impossible.

it works decent. Server switching is not possible ig. idk why other servers wont work. it gives response only with the default server. Which gives only sub as default, this makes switching to dub impossible.

Aniwatch's vidstreaming and megacloud share the same servers ig and the rest dont work as well. You can choose dub in the api endpoint as category. So, it should solve the problem if I understood you correctly. Let me know if that's all.


Initial version of install instructions

Closes #13
Add trustchain-mobile app installation & development instructions Mac.

In step 2, should there be a git clone command for didkit? Seems to be missing and I'm wondering if there's a particular branch I should be on.

In step 2, should there be a git clone command for didkit? Seems to be missing and I'm wondering if there's a particular branch I should be on.

Yep, this was a typo - we were accidentally cloning the ssi repo twice instead of didkit + ssi. I just pushed a correction.

I'm getting this error in Step 8 (running the command from /Users/thobson/spruceid/didkit):
thobson@521-D7M932T9F2 didkit % make -C lib ../target/test/flutter.stamp
cd flutter && LD_LIBRARY_PATH=/Users/thobson/spruceid/didkit/lib/flutter \
                flutter --suppress-analytics test
Running "flutter pub get" in flutter...
Resolving dependencies...
  async 2.10.0 (2.11.0 available)
  characters 1.2.1 (1.3.0 available)
  collection 1.17.0 (1.18.0 available)
  ffi 2.0.2 (2.1.0 available)
  js 0.6.5 (0.6.7 available)
  matcher 0.12.14 (0.12.16 available)
  material_color_utilities 0.2.0 (0.8.0 available)
  meta 1.8.0 (1.9.1 available)
  pedantic 1.11.1 (discontinued replaced by lints)
  source_span 1.9.1 (1.10.0 available)
  stack_trace 1.11.0 (1.11.1 available)
  stream_channel 2.1.1 (2.1.2 available)
  test_api 0.4.17 (0.6.1 available)
Got dependencies!
Resolving dependencies in ./example...
Resolving dependencies in ./example failed. For details run `flutter pub get --directory ./example`
pub get failed
command: "/Users/thobson/flutter/bin/cache/dart-sdk/bin/dart __deprecated_pub --color --directory . get
--example"
pub env: {
  "FLUTTER_ROOT": "/Users/thobson/flutter",
  "PUB_ENVIRONMENT": "flutter_cli:verify:test",
  "PUB_CACHE": "/Users/thobson/.pub-cache",
}
exit code: 1
make: *** [../target/test/flutter.stamp] Error 1


[Test] Integration test fails

Integration tests are executed against Chrome and Firefox browsers. If we initiate the tests and after the test passes successfully in the first browser (Chrome) then the test in the second one fails, with the error "Git repository already exist".
Related to : #998

This could be related to a recent change in Eclipse Dirigible (part of the 6.1.17 release): https://github.com/eclipse/dirigible/commit/5eed856dd795a62afb9c619fb22fe233510d62b2
With this commit the location where git projects are cloned was changed as follows:

.../target/.git → .../target/dirigible/repository/.git


fix: validators link

Closes #8498
Summary

Please summarize your changes, describing what they are and why they were made.

...
Changelog
- Please write a list of changes

Testing
Platforms

Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[ ] Linux
[ ] Windows

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...
Checklist

Please tick the following boxes that are relevant to your changes.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

The issue comes from https://github.com/iotaledger/explorer/issues/1460


Temperature offset setting?

Would it be possible to add temperature offset settings to the indoor and outdoor motion sensors? The values are way lower, but still relative, to the real ones.
How Xiaomi Aqara app solves it can be found here (I think) https://github.com/TedTolboom/com.xiaomi-mi-zigbee/blob/master/app.json .
And as you (and @johanbendz ha!) know I don't know nothing about coding so I will not copy paste! Hehe.

Nice feature request! Added it to our internal todo list.

Closed with #81.


Provide facilities to query for failed schedules and rerun.

Just committed code to allow manual run of a scheduled plan.

commit 55a14aa534f91189e5cb2bf0f3fc06085d483fcb provides a report of failed plans with the command gzr plan failures. It will only report if the most recent attempt to execute the plan failed.


How to handle autoapprove afdpe if the region did not support?

How to handle autoapprove afdpe if the region did not support? E.g. Australia southeast

Hi @thomas-tran, Why is auto approval not available in Australia southeast? I see that ACI is available in Australia Southeast, check here.
However even if you need to use a region where ACI is not supported, you can just approve the PE manually with this script from cloudshell, or you can go to the Azure Portal > Private Link Center > Pending connections

@thotheod Sorry I mean Australia East, I got error on bad request as the Azure front door did not support private link in the location when set the autoapproval  to true

Hi @thomas-tran,
Thanks for sharing the error with me. Unfortunately Azure Front Door private link is not available in Australia South East, but I see in documentation that it is available in "Australia East". Would that region work for you?

@thotheod  The client request the solution to be deployed in 2 regions Australia east and Australia south east. Azure Front Door routes traffic based on availability of the primary region. When the primary region is unavailable it will route traffic to the secondary region. The app service will be blocked from the internet and only allow access from the front door .
It would not be a problem to approve the private endpoint manually. However, the Azure Front door would not allow to mix one region with private link and another without private link through the origin group
module afd '../../shared/bicep/network/front-door.bicep' = {
  name: take ('AzureFrontDoor-${resourceNames.frontDoor}-deployment', 64)
  params: {
    afdName: resourceNames.frontDoor
    diagnosticWorkspaceId: logAnalyticsWs.outputs.logAnalyticsWsId
    endpointName: resourceNames.frontDoorEndPoint
    originGroupName: resourceNames.frontDoorEndPoint
    origins: [
      {
          name: webAppRegion1.outputs.webAppName  //australia east
          hostname: webAppRegion1.outputs.webAppHostName
          enabledState: true
          privateLinkOrigin: {
            privateEndpointResourceId: webAppRegion1.outputs.webAppResourceId
            privateLinkResourceType: 'sites'
            privateEndpointLocation: webAppRegion1.outputs.webAppLocation
          }
      },
     {
          name: webAppRegion2.outputs.webAppName // australia south east
          hostname: webAppRegion2.outputs.webAppHostName
          enabledState: true
          privateLinkOrigin: null
      },
    ]
    skuName:'Premium_AzureFrontDoor'
    wafPolicyName: resourceNames.frontDoorWaf
  }
}

@thomas-tran Since Azure Front Door private link is not available in Australia South East you need to do some modifications in your deployment.
You need to allow public access to the web apps. But once you do that you need to control who can access the web apps. You can manage access restrictions rules in the portal. So you can have an unmatched rule to default to deny, and then an allow rule for the service tag AzureFrontDoor.Backend. More details you also find here
This approach is not as secure as the one we demonstrate in the LZA, but it is good enough. Since the AFD Private link is not available in both regions you wish to deploy, this is the only viable alternative. The upside of that approach is that you can reduce cost, since you do not need premium AFD anymore.


Adds KMS support to mock_cloudformation

This adds support for mocking out KMS key manipulation using Cloudformation. Please let me know if there's any changes you'd like to see!

Great, thank you!


Issues/11 upgrade dsf 0.6.0

resolves #10
resolves #11

EncryptData and StoreData are using the deprecated constants CODESYSTEM_HIGHMED_ORGANIZATION_TYPE and CODESYSTEM_HIGHMED_ORGANIZATION_TYPE_VALUE_COS, if possible upgrade to the new constants.


Add test for usort

Depends on #59

 Test common library hints USORT

Nice one!
The test looks great.
When I run the hints_tests.py test, this program returns the error ValueError: Custom Hint Error: NameError: name 'positions_dict' is not defined
Just comment the test and we will be good to go

A fix was merged yesterday to fix this error. The test should be passing now, have you updated the compiled cairo-rs-py module before running the test?

Nice one!
The test looks great.
When I run the hints_tests.py test, this program returns the error ValueError: Custom Hint Error: NameError: name 'positions_dict' is not defined
Just comment the test and we will be good to go

A fix was merged yesterday to fix this error. The test should be passing now, have you updated the compiled cairo-rs-py module before running the test?

Oh nice, I forgot to run maturin develop!


[BACKPORT v1.4.x] feat(volume): remount volume if it becomes read only

ref: https://github.com/longhorn/longhorn/issues/7843
When detecting the filesystem become read only
Try to remount the global mount point to read write again

ref: https://github.com/longhorn/longhorn/issues/7845

We backport this change to instance-manager v1.4.x
https://github.com/longhorn/longhorn-instance-manager/pull/382#issuecomment-1911186398
but we didn't backport it in manager
https://github.com/longhorn/longhorn-manager/pull/2533
cc @derekbit @innobead

@ChanYiLin the PR check failed.

Won't fix in 1.4, as it's EOL. If users encounter this issue, suggest them to upgrade to 1.5.x.


How does one connect a private app?

I wanted to test this package for my laravel app and i am very confused on how to define my private app. Which fields do i need? Because the fields in a private app are different from a public app. For example i don't need to get my session etc. because i only want to write products to my store from my Laravel app.
Thanks in advance.

Solved it my self

For anyone in the future. If you want to interact with the REST php api you do the following:
This is in laravel
use Shopify\Context;
use Shopify\Clients\Rest;

$shopify = Context::initialize(
            env('SHOPIFY_API_KEY'),
            env('SHOPIFY_API_SECRET'),
            ["read_products", "write_products"],
            env('SHOPIFY_APP_HOST_NAME'),
            new FileSessionStorage('/tmp/php_sessions'),
            '2021-04',
            true,
            true,

        );

        $domain = env('SHOPIFY_APP_HOST_NAME'); //domain.myshopify.com
        $secret = env('SHOPIFY_API_SECRET'); //Your shopify password of the private app
        $client = new Rest($domain, $secret);
        $body = [
            "product" => [
                "title" => "Hiking backpack"
            ]
        ];

        $response = $client->post('products', $body);


RFC: Run from nearest package.json

Problem
In monorepos, some modules are only available in the sub-package under test, thus aren't available if cwd the monorepo root.
For example:
<root>
  packages/
     foo/
       package.json
       __tests__/
         foo.spec.ts
     bar/
       package.json
       __tests__/
         bar.spec.ts

If cwd is <root>, and foo.spec.ts calls jest.mock on a package that is only present packages/foo/node_modules, jest crashes!
Discussion

it is common for monorepo users to open their project from the <root>
it is common for monorepo users to try and exec a test from a child package from the VSCode root workspace ctx
the aforementioned CWD mismatch can lead to this ext crashing (no fault of the extension)

Support for a feature like cwdNearestPackageJson: true could easily:

discern the spec filename under test
find the nearest package.json filename
set the cwd to the dirname of the prior found package.json filename

With such a feature, monorepo users could run a test from any sub-package, and reliably run in the same pwd and node_modules context as if they had just cd <path-to-package> && jest .!

I'll reopen this one for discussion 👍
I think this case is already supported, but I could be wrong.
If you try it with  jestrunner.changeDirectoryToWorkspaceRoot, it will run cd <path> before running the jest command.
You can see here it is falling back to currentPackagePath, which upsearches for package.json:
https://github.com/firsttris/vscode-jest-runner/blob/43cc89b213698c2971b8b3437345ad6d04b1eb6d/src/jestRunnerConfig.ts#L51-L77
Based on the code, perhaps the current docs could be improved:

jestrunner.changeDirectoryToWorkspaceRoot
Changes directory to workspace root before executing the test

I think this setting should really be:

Changes directory to the resolved project path. The fallback order is jestrunner.projectPath, falling back to the nearest package.json, falling back to the workspace root.

Does this cover your use case? Thanks


Update index.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

It's useful, but please expand the article.

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: We do not accept PRs which only add links when there are already valid reference links for the article.
Thank you and happy coding.


I want to see when the reporter can't find the test result xml

I want to see in the test report when the reporter can't find the test result xml.
Now everything is pass.

@pdoggyster i.e. you mean that Maven plugin should return errors when no XML files are present.

@vania-pooh
when it try to generate a report but the source dir is empty than the report is created but it is shown everything okay but the report is empty.
example: I use the TC report generator or command line tool.

I propose the following solution:

Add failIfNoResults parameter to AllureReportGenerator and throw a ReportGenerationException with cause if no results and failIfNoResults is true.
Add failIfNoResults parameter to AllureReportBuilder, and all our report build tools.

@eroshenkoam what are you think?

Or better always fail the report generation if no results. No one want to see the empty report

Is this change already available?

@pdoggyster since 1.4.15. I going to release it next week

Note: exception will be thrown only when you generate report 1.4.15 or higher. Other reports will be empty

Great, thanks!


Adds color support for tmux

Even with 256 color support enabled in tmux, glog previously would only render plain text.  This pull request adds glog color support for 256 color tmux terminals.

I just completed a CLA for my individual 'darkwind-mt' account.  Thanks!

updated my alternative email address


(#104) Improve README.md template for packaged modules

This commit adds to README.md.erb:

information that modules are generated automatically
link to module templates, so it's easier for people to find and
contribute to it

Updated.


[cluster-test] Introduce ct - shortcut to run cluster test docker

ct runs cluster test inside docker container
locally built image is used by default, but can be overwritten with --image

Running from Circle CI will look something like that:
ct --prune --container cluster-test-ci --image <RUNNER_IMAGE> --
   --deploy <upstream_xxx> --run_ci_suite

@bors-libra r+

:pushpin: Commit ee1047c has been approved by ankushagarwal

:hourglass: Testing commit ee1047c9c41102cc694556fa43ea08d4384257a6 with merge 905e0dbc3bcab506cafbe1b80938af315e5dd300...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: ankushagarwal
Pushing 905e0dbc3bcab506cafbe1b80938af315e5dd300 to master...


IPv6 documentation

In this document https://support.packet.com/kb/articles/elastic-ips
Public IPv6 - We assign a (publicly routable) /56 on the project level, which is divisible into 256 /64s (“LAN subnets” in IPv6 parlance), each routable to a server. IPv6 addressing is also provided at no charge.
How is this to be done?

Yes, we've met several problems when setup the elastic IPs for IPv6.
I've a /56 pools in my projects and I want to reserve a IPv6 pools for my OpenStack clusters to use.
So I come to Elastic IPs. In one machine, whose Network mode is Mixed/Hybrid, I click the "Network" - ”Elastic IPs“ - "assign new Elastic" - "public IPv6" and then I get an Elastic IP pools:
IP / RANGE | TYPE | QUANTITY |
-- | -- | -- | --
2604:1380:4111:3e02::/64 | Public IPv6 | >2 billion IP(s)
But in the document, I don't find how to setup the network gateway for this type of IP, and how to make IPv6 IP worked. Even configure the IP to an network device, I could not ping to them via another IPv6 IP.

@zalkar-z Hey guys, do we have plan to produce the doc?

@kevin-zhaoshuai yep, working on it! sorry for the delay

@zalkar-z Let me clarify my requests.
I know that after installation, for example IPv4, it will have a /28 IP and gateway.
So that I have a network gateway and IP range, I can use this for vm instance to use.
For IPv6, now my issue is:
I have a IPv6 IP block, for example 2604:1380:4111:3e02::/64. And I want my VMs provisioned by OpenStack(running on Packet node) to use the IP from the IPv6 Ip block 2604:1380:4111:3e02::/64, But I really don't know how to use it. I don't know which can be served as the network gateway, and I configure 1 IP to a node, but it never worked.


[1.x] Allow output format supported by PHP-CS-Fixer

Hi
This is PR to enable --format option of PHP-CS-Fixer and --report option to save report to the file. This PR covers cs2pr support? and Ability to specify format of the output issues.
From README.md file:
Format output
You can output results in different format supported by PHP-CS-Fixer.
This is especially useful in CI/CD pipelines (for example, Annotations via the Checks API):
pint --test --format=checkstyle

Then you can send output to the tool that will read it and create annotations for you. For example, for GitHub Actions
you can implement following job:
- name: Show Pint results in PR
run: pint --test --format=checkstyle | cs2pr

Save report to the file
If you run pint with --format option, it will suppress standard pint's output. If you want to see
both standard and formatted output, you can save formatted output to report file:
pint --test --format=checkstyle --report=checkstyle.xml

This is especially useful in CI/CD pipelines (for example, Annotations via the Checks API)
when you want to have both job logs and job annotations. You can have one job to generate report file and second job
to read from report file. For example, for GitHub Actions you can implement following jobs:
- name: Check PHP code style
continue-on-error: true
run: pint --test --format=checkstyle --report=./pint-report.xml

- name: Show Pint results in PR
run: cs2pr ./pint-report.xml

Will let @nunomaduro review this and mark as ready for my review when he is done

Going to re-work this pull request on the feat/formatters-and-reporters branch. Thank you!


[models] Proposing to add GetModelBoundingBox()

I noticed there is no functionality present for getting the BoundingBox of a model, but only a single mesh.
This lack of functionality is also exhibited in the example models_loading.c . Since there is no functionality to get the bounds of a Model, the example simply gets the bounds of the first mesh, which may or may not be the full bounds of the model if additional meshes are present in the model.
This lack of functionality can for the most part be easily overcome as an individual can write their own GetModelBounds function, however this is functionality that almost anyone dealing with a model will need / end up writing, and as such it would seem to indicate such functionality should be part of the RayLib itself.
Additionally, solving this issue for those working through bindings in languages that used manages memory is a tad more complicated, though doable, without additional underlying functionality. Simply adding this function removes that problem all together.
BoundingBox GetModelBoundingBox(Model model)
{

    int meshCount = model.meshCount;
    BoundingBox bounds = { 0 };

    if (meshCount < 1)
    {

        return bounds;
    }else{

        Vector3 tPos;
        bounds = GetMeshBoundingBox(model.meshes[0]);
        for (int i = 1; i < meshCount;i++)
        {

            BoundingBox tempBounds = GetMeshBoundingBox(model.meshes[i]);

            tPos.x = bounds.min.x < tempBounds.min.x ? bounds.min.x : tempBounds.min.x;
            tPos.y = bounds.min.y < tempBounds.min.y ? bounds.min.y : tempBounds.min.y;
            tPos.z = bounds.min.z < tempBounds.min.z ? bounds.min.z : tempBounds.min.z;
            bounds.min = tPos;

            tPos.x = bounds.max.x > tempBounds.max.x ? bounds.max.x : tempBounds.max.x;
            tPos.y = bounds.max.y > tempBounds.max.y ? bounds.max.y : tempBounds.max.y;
            tPos.z = bounds.max.z > tempBounds.max.z ? bounds.max.z : tempBounds.max.z;
            bounds.max = tPos;
        }
    }

    return bounds;
}

Ok, feel free to send a PR to add it.


"Merge All Windows"

MacOS Window menu item "Merge All Windows" would be awsm.

+1

This would be super convenient, for instance when you restart your computer and that previously all grouped windows reopen as independent windows ;) this is quite annoying.

Depends on underlying support in Electron, which is being added here:
https://github.com/electron/electron/pull/10271
and a forthcoming PR for the JS API.

I did not end up sending another PR for the API because it was pointed out by @bpasero that

Electron 1.8 is out 👍

Very frustrating trying to open multiple projects in one window.

Current Atom v1.33.0 is using Electron v2.0.11. I still don't see Merge All Windows option..

BUMP!
I'm wondering if there is any traction on this. It would be awesome since I use Project Manager and it will open a new window any time I open a project.
Here's what I'm running at the moment:
Atom: 1.34.0 x64
Electron: 2.0.16
Chrome: 61.0.3163.100
Node: v8.9.3

+1

Thanks for the feedback.
Upon review with the team, this isn't something that we want to add. It would require significant architectural changes or workarounds and we don't feel that it would be of enough benefit to warrant the work that it would require. Because of this, we won't be accepting pull requests for this feature either.
Thanks again for the suggestions and interest in Atom.

VS Code it is.


fix: update setup.py and pyproject for PyPI distributions

Updates:

add more information to setup.py for better distributions
add CONTRIBUTING.md

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