Disable default LSP snippets

I can't seem to disable default LSP snippets.
I've tried using the config below to no avail
  for _,server in ipairs(lspinstaller.get_installed_servers()) do
    local opts = {
      on_attach = on_attach,
      capabilities = cmp_nvim_lsp.update_capabilities(
        vim.lsp.protocol.make_client_capabilities(),
        { snippetSupport = false }
      ),
      flags = {
        debounce_text_changes = 150
      }
    }

    local ok,config = pcall(require, 'lsp.config' .. '.' .. server.name)
    if not ok then
      config = {}
      print(server.name .. ' not found in lsp/config. Setting up defaults.')
    else
      print(' - ' .. server.name .. ' has been configuredl!')
    end

    lspconfig[server.name].setup( vim.tbl_deep_extend('force', config, opts) )
  end

But if I inspect server capability I can see that snippetSupport is definitely have been set to false
{
  capabilities = {
    callHierarchy = {
      dynamicRegistration = false
    },
    textDocument = {
      codeAction = {
        codeActionLiteralSupport = {
          codeActionKind = {
            valueSet = { "", "Empty", "QuickFix", "Refactor", "RefactorExtract", "RefactorInline", "RefactorRewrite", "Source", "SourceOrganizeImports", "quickfix", "refactor", "refactor.extract
", "refactor.inline", "refactor.rewrite", "source", "source.organizeImports" }
          }
        },
        dataSupport = true,
        dynamicRegistration = false,
        isPreferredSupport = true,
        resolveSupport = {
          properties = { "edit" }
        }
      },
      completion = {
        completionItem = {
          commitCharactersSupport = true,
          deprecatedSupport = true,
          documentationFormat = { "markdown", "plaintext" },
          insertReplaceSupport = true,
          labelDetailsSupport = true,
          preselectSupport = true,
          resolveSupport = {
            properties = { "documentation", "detail", "additionalTextEdits" }
          },
          snippetSupport = false,
          tagSupport = {
            valueSet = { 1 }
          }
        },
        completionItemKind = {
          valueSet = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25 }
        },
        contextSupport = false,
        dynamicRegistration = false
      },
      declaration = {
        linkSupport = true
      },
      definition = {
        linkSupport = true

You should check the lua-language-server's configuration.
https://github.com/sumneko/vscode-lua/blob/master/package.json


Should we started enforcing no overlap between measurement keys and parameter names?

This comes up when analyzing data afterwards. It's ambiguous whether you're talking about the measurement result "a" or the parameter "a".

I guess this could be a problem but the way you access these makes it pretty clear they are different.  I'm inclined to close this since I'm not aware of anyone making this mistake, and if they do it's mostly because we have moved too far in the direction of returning objects that don't allow you to understand what you are accessing (probably also a vote against the idea that we should eliminate TrialResult.

Closing unless we think there are places where this is causing confusion.  As was mentioned you can get confused between the rows and columns of data frames, but I don't think not allowing them to overlap is a good idea (it's a strange restriction that just adds complexity while only supporting a few people who might possibly get slightly confused).


follow deep relationships

Following relationships to attributes was allowed using tuples, but only 1 relationship deep (by limiting the tuple to length 2). For example:
like((blogs, title), *frogs*)
would work, but
like((blogs, posts, title), *amazonian frogs*)
would not.
This pull request adds the ability to go arbitrarily deep with the above syntax (so the second example would work). Tests and extra fixtures for a 2-deep relationship are also added.

That's a nice improvement, @colinhiggs. Thanks.


Any Update of this library for targetting API 26 or more?

Currently this library targetSDK is 26 and Google play console giving me warning about this. what to do?
Any help?

Why am i getting the target SDK 25 when i am using 0.8.5 and i saw your library has target 27? I am not able to upload the app to the playstore?

Currently this library targetSDK is 26 and Google play console giving me warning about this. what to do?
Any help?

Why am i getting the target SDK 25 when i am using 0.8.5 and i saw your library has target 27? I am not able to upload the app to the playstore?

@deathstroke007 please check the  tag, in that you have mentioned "android:targetSdkVerion=25". Update that to 26, that might solve your problem.

closing this as I think @SamratPyaraka  is right about the cause here


[Launcher] Check if the controller launched properly before starting the workers

We should check if the controller actually launched before starting the workers.
Imagine the case you have a controller already running on the same port you are trying to use, an old one or just somebody else controller. Then you would launch worker what would work on with the other controller which is bad.

Why not tell the controller how many worker in total it should have and
after he accept that number of worker, any later worker will get an error?
On Mon, Feb 8, 2016 at 5:20 PM, Mathieu Germain<EMAIL_ADDRESS>wrote:

We should check if the controller actually launched before starting the
workers.
Imagine the case you have a controller already running on the same port
you are trying to use, an old one or just somebody else controller. Then
you would launch worker what would work on with the other controller which
is bad.
—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/57.

Let say you are running something that takes a long time and a GPU get
freed up. You probably want to be able to add a new worker whenever you
want.
On Tue, Feb 9, 2016 at 10:32 AM, Frédéric Bastien<EMAIL_ADDRESS>wrote:

Why not tell the controller how many worker in total it should have and
after he accept that number of worker, any later worker will get an error?
On Mon, Feb 8, 2016 at 5:20 PM, Mathieu Germain<EMAIL_ADDRESS>wrote:

We should check if the controller actually launched before starting the
workers.
Imagine the case you have a controller already running on the same port
you are trying to use, an old one or just somebody else controller. Then
you would launch worker what would work on with the other controller
which
is bad.
—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/57.

—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/57#issuecomment-181915236.

I don't mean it is mandatory to use it, but I think that frequently, you
won't be able to add more GPU as you will already use all the usable GPU on
the computer. But yes, we should not make what I descrive mandatory.
On Tue, Feb 9, 2016 at 10:42 AM, Mathieu Germain<EMAIL_ADDRESS>wrote:

Let say you are running something that takes a long time and a GPU get
freed up. You probably want to be able to add a new worker whenever you
want.
On Tue, Feb 9, 2016 at 10:32 AM, Frédéric Bastien <
<EMAIL_ADDRESS>wrote:

Why not tell the controller how many worker in total it should have and
after he accept that number of worker, any later worker will get an
error?
On Mon, Feb 8, 2016 at 5:20 PM, Mathieu Germain <
<EMAIL_ADDRESS>wrote:

We should check if the controller actually launched before starting the
workers.
Imagine the case you have a controller already running on the same port
you are trying to use, an old one or just somebody else controller.
Then
you would launch worker what would work on with the other controller
which
is bad.
—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/57.

—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/57#issuecomment-181915236.

—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/57#issuecomment-181919634.


workaround for parsing block html inside arbitrary html

partially fixes #153
not 100% happy with this at the moment, as it technically fixes an edge case and introduces a new one (though a less frequent one)

LGTM


[Feature Request] Cast support while torrent streaming!

Is your feature request related to a problem?
No
Describe the solution you'd like:
Implement Chromecast casting while torrent is streaming.
Describe alternatives you've considered:
Bitlord client

Todo

[ ] Show a dialog during the stream is ready
[ ] Check for proper separation of concerns
[ ] Check for any memory leak issues
[ ] Test activity for process death


TryGetValue maight throw ArguemntNullException

2019-10-29 14:21:47.9702|ERROR|RepairMajor.GenerateMinuteBarForWeistock|9| errorfile E:\MarketRaw1\CME_CBT\2018112800.market |System.ArgumentNullException: 值不能为 null。
   在 System.Threading.Monitor.Enter(Object obj)
   在 Caching.LRUCache`2.TryGetValue(K key, V& value) 位置 c:\Users\TJ\Projects\LRUCache\src\LRUCache\LRUCache.cs:行号 114
   在 History.Lib.Common.GetProductID(String contract) 位置 E:\Code\DevelopmentDepartment\source\MessageServer\History_AddCache\HistoryLib\Common.cs:行号 372
   在 RepairMajor.GenerateMinuteBarForWeistock.<>c__DisplayClass2_0.<ExportFutureKLine>b__9(String item) 位置 E:\Code\DevelopmentDepartment\source\MessageServer\History_AddCache\RepairMajor\GenerateMinuteBarForWeistock.cs:行号 94
   在 System.Linq.Enumerable.WhereSelectArrayIterator`2.MoveNext()
   在 System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   在 RepairMajor.GenerateMinuteBarForWeistock.<>c__DisplayClass2_2.<ExportFutureKLine>b__5() 位置 E:\Code\DevelopmentDepartment\source\MessageServer\History_AddCache\RepairMajor\GenerateMinuteBarForWeistock.cs:行号 112

https://github.com/tejacques/LRUCache/blob/797ab72c425ee778827e377c5d4244c59879fdb4/src/LRUCache/LRUCache.cs#L114
I think that TryGetValue need to determine whether variable CacheNode  entry in method IS NULL.

Hi @jhy871167495
I'm having the same issue, and it is happening in random times.
Since @tejacques is not answering, did you found any workaround? Or maybe you already fixed the lib in a fork?
Thanks

Hi @jhy871167495
I'm having the same issue, and it is happening in random times.
Since @tejacques is not answering, did you found any workaround? Or maybe you already fixed the lib in a fork?
Thanks

You can try this. https://github.com/jhy871167495/LRUCache

Hi @jhy871167495
I'm having the same issue, and it is happening in random times.
Since @tejacques is not answering, did you found any workaround? Or maybe you already fixed the lib in a fork?
Thanks

You can try this. https://github.com/jhy871167495/LRUCache


fixed serializer naming strategy for tests

informations:

q
a

Fixed tickets
none

Related PRs
https://github.com/sulu-io/sulu-standard/pull/607

BC breaks
none

Documentation PR
none

@danrot tests failing code looks good

Ok, looks like the @ for defining services is only allowed in quotes in PHP7. However, there is still this problem with jackrabbit: https://travis-ci.org/sulu-io/sulu/jobs/107959731#L1253
@wachterjohannes Have you already solved that somehow?

@danrot not really ...


Wallet not created

Hi yesterday i installed the DeFi Chain App.
After 45min Synchonisizing i created a wallet.
Wrote down the 24 word for recovering.
Today I just wanted to see, if my amount of
crypto was send. But now i cannot find the wallet
and cannot restore it there is just "New wallet" or
"restore wallet" -> restoring is not working. It is coming the note:

Already have this key (either as an HD seed or as a loose private key)<<

So where can i see my wallet

Hi @Alex080988 , thanks for your feedback. To understand your user flow better, after you created wallet the screen displayed is still on Create / Restore Wallet screen? Similar to this issue? https://github.com/DeFiCh/app/issues/150
Please give us time to investigate this.

Hi @Alex080988 , may I know if the issue is resolved with the new update v2.1.4?
https://github.com/DeFiCh/app/releases/tag/v2.1.4

Same issue here with 2.1.5

Same issue here with 2.1.5

Hi @Alex080988 @nappy , update v2.1.5 is officially out https://defichain.com/downloads/.

Please give a thumbs up to this comment if your issue has been resolved, if not, please send your app logs and we'll further investigate.

Hi @Alex080988 @nappy , update v2.1.5 is officially out https://defichain.com/downloads/.

Please give a thumbs up to this comment if your issue has been resolved, if not, please send your app logs and we'll further investigate.

Seems to be resolved, at least with 2.1.5 (clean app data) I was able to restore from seed.
On a side note though, there is probably already an issue for this, the full chain is still being re-downloaded after each application start with release 2.1.5.

Seems to be resolved, at least with 2.1.5 (clean app data) I was able to restore from seed.
On a side note though, there is probably already an issue for this, the full chain is still being re-downloaded after each application start with release 2.1.5.

Hi @Alex080988 @nappy , , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue w new information (include screenshots) and your app logs. Thank you.

Hi @Alex080988 @nappy , , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue w new information (include screenshots) and your app logs. Thank you.


Question: reduce size of docker image

Current size of docker image is quite large. Can we reduce size to ~ 300MB for faster pull/push when deploy?
REPOSITORY             TAG  IMAGE ID      CREATED      SIZE
ghcr.io/weserv/images  5.x  da62b9286622  10 days ago  1.12GB

This is a known issue. I'm not sure if ~300MB is is possible, since the base CentOS 8 image is already ~239MB, see for example:
$ podman pull centos:8
$ podman images
REPOSITORY             TAG         IMAGE ID      CREATED      SIZE
quay.io/centos/centos  8           5d0da3dc9764  6 weeks ago  239 MB

Of course, any further reduction in size would be great. I've considered changing the base image to another distro, but haven't yet found a better alternative. For example, libvips in Debian 10 is still on version 8.7.4 due to their policy of never updating packages (except for security fixes and critical bugs) because they prioritize stability. Alpine is a very popular rolling release system, but unfortunately only has a default thread stack size of 128k, which can lead to stack overflows in the dependencies of libvips (see for e.g. issue https://github.com/libvips/libvips/issues/1287).
Note that PR https://github.com/weserv/images/pull/276 tried to reduce the size of the image by using a multi-stage build, but this was not merged for several reasons:

DNF (CentOS/RHEL/Fedora default package manager) is not aware of the copied dependencies.
Loadable modules were not copied along to the final image (i.e. the dependencies in /usr/lib64/vips-modules-x.y/* and /usr/lib64/ImageMagick-x.y.z/modules-Q16/*).
It was difficult to maintain.

A better approach to reduce the image size would be to remove the -devel dependencies, the build directory, and the DNF cache after configuring and building the CMake project. For example, I saw that someone did this on one of the forks:
https://github.com/weserv/images/compare/8349d64...palamccc:5.x
Let's tag this as an enhancement. Very happy to accept a PR, if you're able.

Size of image after remove remove the -devel dependencies reduced to 1.07GB (only 50MB). I think there are still a lot of files we can remove (e.g: gcc cache during build phase).
REPOSITORY     TAG     IMAGE ID      CREATED        SIZE
weserv/images  latest<PHONE_NUMBER>7a  6 seconds ago  1.07GB

The issue https://github.com/libvips/libvips/issues/1287 has been resolve by allow to set VIPS_MIN_STACK_SIZE  to change stack size. Is any help to switch to Alpine?

I able to build docker image to reduce size to 451MB. Can you double-check any issue with it?
REPOSITORY     TAG     IMAGE ID      CREATED            SIZE
weserv/images  alpine  969a998f93ce  7 seconds ago      451MB
weserv/images  latest<PHONE_NUMBER>7a  About an hour ago  1.07GB

FROM alpine:3.13.6

RUN apk add --update-cache \
  --repository https://dl-3.alpinelinux.org/alpine/edge/testing/ \
  --repository https://dl-3.alpinelinux.org/alpine/edge/main \
  build-base cmake git \
  libjpeg-turbo \
  openssl openssl-dev \
  vips vips-dev

# Copy the contents of this repository to the container
COPY . /var/www/imagesweserv

WORKDIR /var/www/imagesweserv/build

# Build CMake-based project
ARG NGINX_VERSION=1.21.3
RUN cmake .. \
  -DCMAKE_BUILD_TYPE=Release \
  -DBUILD_TOOLS=ON \
  -DNGX_VERSION=$NGINX_VERSION \
  -DCUSTOM_NGX_FLAGS="--prefix=/usr/share/nginx;\
--sbin-path=/usr/sbin/nginx;\
--modules-path=/usr/lib64/nginx/modules;\
--conf-path=/etc/nginx/nginx.conf;\
--error-log-path=/var/log/nginx/error.log;\
--http-log-path=/var/log/nginx/access.log;\
--http-client-body-temp-path=/var/lib/nginx/tmp/client_body;\
--http-proxy-temp-path=/var/lib/nginx/tmp/proxy;\
--http-fastcgi-temp-path=/var/lib/nginx/tmp/fastcgi;\
--http-uwsgi-temp-path=/var/lib/nginx/tmp/uwsgi;\
--http-scgi-temp-path=/var/lib/nginx/tmp/scgi;\
--pid-path=/run/nginx.pid;\
--lock-path=/run/lock/subsys/nginx;\
--user=nginx;\
--group=nginx" \
  && make -j"$(nproc)" \
  && cd .. \
  && rm -Rf build /var/cache/*

# Cleanup build dependencies
RUN apk del build-base cmake git openssl-dev vips-dev

WORKDIR /var/www/imagesweserv

RUN addgroup -S nginx && adduser -S nginx -G nginx

# Ensure nginx directories exist
RUN mkdir -m 700 /var/lib/nginx \
  && mkdir -m 700 /var/lib/nginx/tmp \
  # Forward request and error logs to docker log collector
  && ln -sf /dev/stdout /var/log/nginx/weserv-access.log \
  && ln -sf /dev/stderr /var/log/nginx/weserv-error.log \
  # Copy nginx configuration to the appropriate location
  && cp /var/www/imagesweserv/ngx_conf/*.conf /etc/nginx

EXPOSE 80

ENTRYPOINT ["nginx", "-g", "daemon off;"]

I'm not sure if VIPS_MIN_STACK_SIZE helps in all scenarios, for example a thread spawned by an external dependency would not honor this env variable. Another thing to consider is that Alpine packages are built with -Os by default (optimized for binary size) whereas CentOS/RHEL/Fedora uses, -O2 (optimized for performance). libvips from Remi's RPM repository also enables auto-vectorization, which would speed things up even more.
Given this, I'm a bit reluctant to switch the pre-built Docker image to Alpine Linux, but you're always free to use an other Linux distro of your choice.

Can you double-check any issue with it?

I will review PR #317 in the course of next week.

Can we compile libvips from source code for alpine?
https://github.com/felixbuenemann/vips-alpine/blob/master/Dockerfile#L5-L30

With commit https://github.com/weserv/images/commit/a9488c4112e714970c40e34a0f684688e372b36c and https://github.com/weserv/images/commit/cee18376672cf37a1e5fed3ea05c85f7cfd5f5f5 I now see:
$ podman images
REPOSITORY                TAG         IMAGE ID      CREATED        SIZE
localhost/weserv/images   custom      61c3cce52b29  3 seconds ago  588 MB
localhost/weserv/images   alpine      b04aa046a324  4 minutes ago  96.4 MB
localhost/weserv/images   latest      325490d9ea82  6 minutes ago  622 MB
quay.io/centos/centos     8           5d0da3dc9764  6 weeks ago    239 MB
docker.io/library/alpine  3.14        14119a10abf4  2 months ago   5.87 MB

The weserv/images:custom tag is based on the experimental RPMS repo from https://rpms.weserv.nl, see comment https://github.com/weserv/images/issues/298#issuecomment-955697355 on how to use that.

Can we compile libvips from source code for alpine?

I prefer to install libvips through the standard package managers, this ensures that it can be easily updated in a running container and avoids some maintenance.

FWIW, the released image on the GitHub Container Registry is even smaller :tada:
$ docker images
REPOSITORY              TAG       IMAGE ID       CREATED          SIZE
ghcr.io/weserv/images   5.x       e13dfa2278ad   15 minutes ago   595MB

The results are amazing. Thank you very much!

Hi @kleisauke, I just build alpine in local an face with a error.
giautm ➜ ~/develop $ docker run -p 8080:80 --name=imagesweserv giautm/weserv-images:alpine-3.14
nginx: [emerg] mkdir() "/var/cache/nginx/fastcgi_temp" failed (2: No such file or directory)

My bad, new docker requires a volume to mount
docker run -p 8080:80 --name=imagesweserv -v $(pwd)/nginx-cache:/var/cache/nginx -v $(pwd)/nginx-pid:/var/run giautm/weserv-images:alpine-3.14

Those host volumes mount points shouldn't be necessary, commit https://github.com/weserv/images/commit/82771fc62b27e8d32eb428edc17f71efd6273725 fixes this.
After further testing, I noticed that AVIF and GIF output doesn't work in the Alpine based Docker image, the former needs further investigation, the latter is due to the missing ImageMagick dependency. The upcoming libvips 8.12 has a gifsave operation that uses cgif, so IM should then not be necessary (for saving GIF images at least).
For now, I can recommend disabling these savers in the Alpine based Docker image by using a custom nginx configuration that includes:
weserv_savers jpg png webp tiff json; # Disable AVIF and GIF output

AVIF output in the Alpine-based Docker image should work after commit https://github.com/weserv/images/commit/a0c4337b33bd71223c0cac7870daa7b32215a6ca. GIF output should work once Alpine updates libvips to 8.12 and builds against libcgif.


fix: text bug in commit #11

Fix #11.

This is considerably more complex than it needs to be. A simpler approach is to add ?. before the delta, so it becomes:
const responseText = chunks.map(item => item["choices"][0]?.["delta"]["content"]).join("")

Please try that in your local set up and report back. If it works, no need to change anything in this PR, as I already have the change locally.

Yes, the simple approach successfully fix the issue, thank you.

@HuaDeity Until the next release is officially out, you can use this one which already has the changes.

@vitorgalvao
Thank you for kindly fixing this issue.


Recommended way to support "WITHIN GROUP"

Hi, Is there a way to accomplish percentile_cont(fraction) WITHIN GROUP (ORDER BY sort_expression)? https://www.postgresql.org/docs/9.4/functions-aggregate.html

Also interested in a similar thing: I have implemented some PostgreSQL specific AggregateFunction in our code, and I'd love to be able to pass ORDER BY sort_expression into them, ala JSONB_AGG("table_name" ORDER BY "table_name"."sort_expression")

Just to follow up; I extended AggregateFunction and implemented get_special_params_sql to add the appropriate keywords inside the function; as is done in AnalyticFunction.


PDS member files are opened in read only mode.

Development environment used

VS Code 1.84.2
Z Open Editor version:  3.3.2
Zowe Explorer: 2.12.2 configured to use z/OSMF to z/OS 3.4
Editor Platform

[x ] Visual Studio Code 1.84.2
[ ] Red Hat CodeReady Workspaces
[ ] Eclipse Che
[ ] Standalone Theia

Editor Platform Version:
Operating System (on which VS Code runs such as Windows 10 2004, otherwise name and version of platform such as OpenShift v4.3):  MAC/OS 13.6.2
Java Version (when using VS Code or Theia, execute java -version and paste the details here):  Semaru
[ ] Related to RSE API?

RSE API Plugin version:
Zowe CLI version:
Node.js version:

Logs attached (see here how to get them): yes/no

The only way we can edit in VS Code is to open a PDS member and use the editor like ISPF.  Ie.  Enter an I in the left column to add a line above the one we selected.  That works every time, but is not at all the behavior we are looking for.  We were expecting a full screen editing experience.
Detailed steps for reproducing the problem:

First step

Observed behavior

Open file.  Try to edit anywhere on the screen - edits are ignored.  Enter an I in the left column - line is added above current line and is editable.

Expected behavior

Full screen editing.

If you've not seen it that must mean it isn't happening.  Sorry to bother you.

Closing as we cannot reproduce and do not have enough information.


changefeed (ticdc): remove resolvedTs from etcd

What problem does this PR solve?

Issue Number: ref #9058, ref #9235
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/test all

/test all

/test all

/merge

/test all


API throws 500 error

API error thrown when retrieving recommendations on app engine website

It might be because your Cloud SQL v1 DB user is not created for the localhost Host name.
Can you check that you have a user similar to this


Add a Gitter chat badge to README.md

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

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

Already added the badges! :smiley:


Expose TLS setup in /api/tls

What does this PR do?
This PR add an entrypoint in the api for exposing tels configs.
Motivation
It's hard to understand troubles with https. I want to see what is done, not what I asked.
More

[ ] Added/updated tests
[ ] Added/updated documentation

I'm fighting with tls configuration. I need some explicit certificates, some star certificates, and an ACME fallback. [file] works well for adding certificates.
But now, I'm fighting with some client side crash.
curl: (35) error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error
I'm seeking some visibility.


fix: wait longer for the video

fix Co1lin/Tsinghua-Yukuotang-Autoplay#1

Thanks for your contribution!


[Draft] Move Compose file ref to the legacy node

Signed-off-by: Usha Mandya<EMAIL_ADDRESS>Move Compose file version 2 and 3 to a new legacy node

I'm still confused with the Compose file format matrix, does it make sense to keep it for the legacy part only?

Yes, that's not required. Also, the I think the page on versioning and upgrading should also move to the Legacy section. WDYT?

way better 👍
And the integration of the specification looks nice! Did you made a lot of changes on the specification markdowns?
Thanks @usha-mandya

@glours Nothing much. A minor revamp to the overview section, updated the source files in upstream compose-spec and fixed a bunch of broken URLs.


ReferenceError: window is not defined

After:
cd ./my-working-create-react-app
cra-universal init
cd ./server
yarn install
yarn start

I get:
................
[nodemon] starting `node build/bundle.js`
/home/valorloff/my-app/server/build/bundle.js:39966
var isLocalhost = Boolean(window.location.hostname === 'localhost' ||
                          ^
ReferenceError: window is not defined
    at Object.<anonymous> (/home/valorloff/my-app/server/build/bundle.js:39966:27)

Can you help?

You cannot do DOM checks on server-side, like accessing window, document.
You can only do that on client side code which didn't get executed by server (ex. componentDidMount)

What is entry point of server part? May be need some serverApp?
can you show working example of your package?

@antonybudianto Sorry to bother you but there is something I'm trying to fundamentally understand about SSR. Have you run across times where your bundle.js grabbed third party libraries with the window object in it?
As I see the code in bundle.js, I'm not even sure what is being bundled. Is there a way to simply ignore undefined objects, maybe through webpack?

@mparker11 , it's a bit tricky if your 3rd party lib doesn't support SSR/isomorphic, but you can still use them on lifecycles that didn't get called during SSR, like componentDidMount.

Thanks for the reply. I'll continue trying to figure out which libraries are affecting it. Have you had experience porting over a mature project to use SSR?

@mparker11 I hadn't unfortunately

In my case, I had mapbox lib, but to be honest I don't need it to be SSR since Googlebot does not care about parsing that kind of map


Project status

Hi,
Recently stumbled across your Python package but noticed that there hasn't been any activity in a year. Is there any plan to bring this to maturity? Any update on the current status is greatly appreciated.

Hi. I haven't found time to work on this lately, but if you can help move it forward that would be excellent, or contribute in any way in the shorter term, that would be excellent.

I don't have a great deal of time on my hands. So apart from documentation what are the places that need attention most?

Just tried to run it for a bit and it's not really usable at all, I see. I'm really curious about this right now but can't make promises about having the time for an implementation.

I don't have the capacity to move this forward now. Sooner or later it will be a necessity for my work, which will drive the implementation. That time might be approaching, but I can't promise anything.

I have started a complete re-write on an orphan branch called 'devel'. My goal was to provide a pythonic implementation and not to stick to the Java example too closely. I'll still need a few more sessions to make it really useful but it's coming along nicely. Any comments appreciated.

Great. It's definitely important to be Pythonic. My approach was to replicate the Java API pretty closely, make it work, then progressively refactor to be Pythonic, but your way will work just as well.

@Midnighter Have you pushed anything to Github?

Yes, you should see all my work on the devel branch.


Generalize data insertion capacity in orderbook

quick fix to extend the functionality of the Orderbook struct:

Added a volume field in the Level type which is used in the bids: Vec<Level>, asks: Vec<Level> within the Orderbook type. With this, now there is the optionality of either providing order-level granularity, or, just price and volume granularity, when creating a Level. And so to keep supporting the two types of filling processes:

For the Synthetic Data Generation: The volume field in the Level type, now is the sum of the generated vector of Order types.
For the Market Data Insertion: The volume field is one field within the Level type that is needed to be included.

Created the insert method for the Orderbook type.

Insertion into the Orderbook corresponding fields bids and asks now can be done by providing a collection of Level-like compatible types, including an internal validation of this.
For this PR only one case is implemented, a struct with named and typed fields that mimic 1:1 of those of the Level internal type at atelier::data::market::Level

Introduced some formatting changes.

In this PR there are changes given a few slight modifications on the rust formatting options within the added file rustfmt.toml.

Tests file modification.

For better separation of concerns, the corresponding tests will be placed at tests/orderbook_static.rs, under the fn orderbook_insert_levels()

Insertion methods for bids and asks.
Due to current dependency limitations, you will have to install protoc. I hope to lift this requirement soon.

Closing in favor of #33


[possible bug]: Maximum text size in diagram exceeded

@nhannht Thank you for this awesome plugin!
My checkboxes (tasks) contain more than 50 characters per line and the attached error showed up. This resulted in failing on rendering the gantt chart.
I know this is my data incompatibility and should keep them all shorter, but I'd appreciate it if the plugin can trim the letters like
"ThisIsAnExampleTaskTh....." etc.

Or potentially, this might be caused by the fact that my tasks (including done and not done) have over 3500 checkboxes in the vault, and exceeding the limit of the mermaid.
https://github.com/mermaid-js/mermaid-cli/issues/113

Thank you for doing research, it helps me reduce the time to fix. I just add in the new version
mermaid.initialize({
	startOnLoad: true,
	maxTextSize:99999999,
				})

Can you try it and report to me if the hotfixs worked?
I will write some scripts to generate very big size samples and test this case. But at the minimum time, I need at least tomorrow to have a new patch.

@nhannht Thank you for your quick response on this! Seems like I've faced another issue here. I forked this repository and changed the maxTextSize: 500000 as well but still the same error occurs...

I'm going to add the console error here as well. Some kind of parsing might be failing when converting the checkboxes to HTML? on rendering.

Haizz, seriously, if I don't have the same sample from your case (actually it means your private data).
It is hard for me just to guess what happened. We know there must be some task which produced the bug.
But we have thousands of tasks in your vault and impossible for us to guess which is the cause.
Now, I suggest a simple solution. Please only use the "Current file" filter.  Well, there will be a change we can narrow down which file has the task that caused the error. You don't need to report me now. Just use the plugin like it was limited in feature until you find the file that causing the error. After that, if you don't mind, please show me all the tasks in that file. (change the sensitive info if needed).

@nhannht Thank you for your quick response on this! Seems like I've faced another issue here. I forked this repository and changed the maxTextSize: 500000 as well but still the same error occurs...

And about this report, I want to make sure about the phrase "same error". Are you sure it is the same error as the above error, I don't see the sentence  "Maximum text size in diagram exceeded"

done, break up with Mermaidjs and move to gantt-task-react, it 's time to get new boyfriend


adding a way to change which reactions are associated with which species

I'd like to be able to see and change which reactions are connected to each species / aliasSpecies. maybe something like
SBMLNW.getSpeciesReactions(species, index) -> list of the reactions and indexes connected to this AliasSpecies
SBMLNW.moveReaction(reaction, index, old_species, old_species_index, new_species_index)

The feature is added in this https://github.com/sys-bio/SBMLNetwork/pull/3. Thanks for the contribution!


Splitpane Implementation

A common component used in IntelliJ is the Splitpane, the current implementation inside Compose for desktop is lacking in stability.
A simple implementation can be useful inside Jewel

Solved in #163


Update package.json

Fix #110
Title
Removed all ref to auto
Description of what's changing
package.json
What else might be impacted?
...
Which issue does this PR relate to?
...
Checklist
[ ] Tests are written and maintain or improve code coverage
[ ] I've tested this in my application using yarn link (if applicable)
[ ] You consent to and are confident this change can be released with no regression

@vvijayalakshmi21 Looks good! Can you please run "yarn" and also commit yarn.lock file ?

I see this
Please can you help>?

Sorry for committing two issues in one PR.
Please check now

I see this
Please can you help>?

@vvijayalakshmi21 For commit messages, you need to follow the format that is suggested in the error message by commitlint: Eg: chore: remove auto related dependencies.
You can read more about conventional commit message formats here: https://github.com/conventional-changelog/commitlint/tree/master/@commitlint/config-conventional

Fix for #118 also included

@vvijayalakshmi21 Changes look good! We will merge now. Thank you for contributing!

@all-contributors please add @vvijayalakshmi21 for docs and maintenance.


Update TypeScript + Jest getting started

How to use TypeScript + Jest is already documented in ts-jest: a single source of truth with up-to-date documentation always win.

I did not update README.md since it seems generated from GettingStarted.md

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4392   +/-   ##
=======================================
  Coverage   56.17%   56.17%
=======================================
  Files         191      191
  Lines        6424     6424
  Branches        6        6
=======================================
  Hits         3609     3609
  Misses       2812     2812
  Partials        3        3

Continue to review full report at Codecov.

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

Thanks!


Give posibility to run different file in existing kernel

With the new update when I run a file a Kernel opens. Okay.
I switch to a different file that may or may not be in the same directory as the first one.
When I run it a new kernel opens. This happens for every file.
I literally scanned through the options of Hydrogen. The "directory to start kernel in" is not what I am looking for.
There is no option to allow me to run all files in the existing kernel without opening a new one. I would very much like that!
(I am asking for an option to make this available by default by user's choice)

Currently I have to press so many buttons:
First Ctrl Shift P, then "Hydrogen Kernel" then Click hydrogen connect to existing Kernel, then try to guess which one is the kernel I want, since the popup window is not wide enough to read the full path AND it is not scrollable with Atom Material theme (bug).

Is there a reason to have a kernel per file?
This makes my workflow much tougher - I'm copying and pasting code into a file to run it in the existing kernel!

Same for me.
On 2 Nov 2017 18:28, "Maximilian Roos"<EMAIL_ADDRESS>wrote:

Is there a reason to have a kernel per file?
This makes my workflow much tougher - I'm copying and pasting code into a
file to run it in the existing kernel!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nteract/hydrogen/issues/1057#issuecomment-341497982,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwgYd3BHCltUyLN4-2s6I1BFMkEcnJVks5syfusgaJpZM4P_FZb
.

I think there is definitely a happy medium to be worked out on this as the 2.0 release is only a couple weeks old. I still think independent kernels are most intuitive and should be the default, but I'm hoping ideas from this thread result in making this better through config or other enhancements. I'm interested in working on that or helping anyone else who wants to contribute.

Is there a reason to have a kernel per file?

In my opinion (and I think this is a matter or preference), the reason each file should start its own kernel by default is because if you want your file to run outside of hydrogen, say in a script or an app, you'd have to pay attention to namespaces and what you import/require/using etc. It's also how jupyter notebook works unless additional setup is done to connect to existing kernels.

but it is not how Spyder works. For example I switched from spyder to atom.
I am not asking to change the default, but at least giving me the option to
recover the behaviour I need should be really easy...
On 3 Nov 2017 05:01, "Ben Russert"<EMAIL_ADDRESS>wrote:

I think there is definitely a happy medium to be worked out on this as the
2.0 release is only a couple weeks old. I still think independent kernels
are most intuitive and should be the default, but I'm hoping ideas from
this thread result in making this better through config or other
enhancements. I'm interested in working on that or helping anyone else who
wants to contribute.
Is there a reason to have a kernel per file?
In my opinion (and I think this is a matter or preference), the reason
each file should start its own kernel by default is because if you want
your file to run outside of hydrogen, say in a script or an app, you'd have
to pay attention to namespaces and what you import/require/using etc. It's
also how jupyter notebook works unless additional setup is done to connect
to existing kernels.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nteract/hydrogen/issues/1057#issuecomment-341618358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwgYfUbtukZu-FQyGylrcuzfrWuCm66ks5sypARgaJpZM4P_FZb
.

I agree it's not a complete unadulterated slam dunk and there are multiple perspectives...
I think a couple of changes we could make that would help everyone:

An option to have a global kernel
A way of setting a default kernel. I use a docker setup, and in order to get a kernel running in a new file, I need to run through 4-5 prompts (rather than Cmd-Enter if using a local kernel). Much of the frustration of the multiple kernels is around getting any kernel set up, not necessarily having the same kernel in that file

Checkout #40 for more information about this topic.
I'm open for contributions adding a config option to have a global kernel.

Indeed, it could be very useful feature bringing Hydrogen a bit closer to Spyder.  A simple checkbox in configuration that run silently Connect to existing kernel for each new file would be enough.

I messed around with the global kernel mode option last night and was able to get something rough working. I'll PR sometime this week.


Fixed error in ChildPathFilter.equals()

The equals() method checked for a wrong type of the argument which in the end leads to dependencies not being matched in JBoss 7 that lead to 100% CPU problems.

This fix was merged.


[BUG] Ingress created by route trait does not have an address

Describe the bug
Unable to successfully follow the KubeVela "quick start" instructions here.
https://github.com/oam-dev/kubevela
To Reproduce

Follow the KubeVela instructions through the Add trait step. https://github.com/oam-dev/kubevela#add-trait
Observe that the ingress has no address. kubectl get ingress -n myenv

Expected behavior
Expect to be able to get a response from https://mycomp.kubevela.io after setting up mycomp.kubevela.io to resolve to the ingress.  However since the ingress address isn't setup that can't be accomplished.
Screenshots
> k get ingress -n myenv
NAME                        HOSTS                ADDRESS   PORTS     AGE
cm-acme-http-solver-sc5m7   mycomp.kubevela.io             80        2m57s
mycomp-trait-5b576c4fc      mycomp.kubevela.io             80, 443   3m1s

Cluster information
Using an OKE cluster.
> kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.2", GitCommit:"f5743093fd1c663cb0cbc89748f730662345d44d", GitTreeState:"clean", BuildDate:"2020-09-16T21:51:38Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.9", GitCommit:"65bea96239d3fa3dc1d00f3407041719fc736fce", GitTreeState:"clean", BuildDate:"2020-07-30T12:20:29Z", GoVersion:"go1.13.9 BoringCrypto", Compiler:"gc", Platform:"linux/amd64"}

Using KubeVela vela-v0.0.7-darwin-amd64.tar.gz
Additional context
The possibly related context I can find is in the oam-core pod logs.
> k logs -n oam-system vela-core-568cf7b985-qt2gr
...
2020-10-01T21:28:09.525Z	INFO	Route	[WARN] fail to discovery port and label	{"route": "myenv/mycomp-trait-5b576c4fc", "err": "can't discovery port from workload mycomp standard.oam.dev/v1alpha1.Containerized and it's child resource, errorList: [not have spec.template in workload mycomp no port found in workload mycomp]"}

The ingress also shows some error information.
> k describe ingress -n myenv mycomp-trait-5b576c4fc
Name:             mycomp-trait-5b576c4fc
Namespace:        myenv
Address:
Default backend:  default-http-backend:80 (<error: endpoints "default-http-backend" not found>)
TLS:
  mycomp-trait-5b576c4fc-cert terminates mycomp.kubevela.io
Rules:
  Host                Path  Backends
  ----                ----  --------
  mycomp.kubevela.io
                         route-mycomp:443 (<error: endpoints "route-mycomp" not found>)
Annotations:          cert-manager.io/issuer: oam-env-myenv
                      kubernetes.io/ingress.class: nginx
Events:
  Type    Reason             Age   From          Message
  ----    ------             ----  ----          -------
  Normal  CreateCertificate  11m   cert-manager  Successfully created Certificate "mycomp-trait-5b576c4fc-cert"

Was trying to figure out what vela 0.0.7 really meant.  Grabbed the vela-core deployment.
> k get deployment -n oam-system vela-core -o yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  annotations:
    deployment.kubernetes.io/revision: "1"
    meta.helm.sh/release-name: vela-core
    meta.helm.sh/release-namespace: oam-system
  creationTimestamp: "2020-10-01T21:10:09Z"
  generation: 1
  labels:
    app.kubernetes.io/instance: vela-core
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: vela-core
    app.kubernetes.io/version: 0.1.0
    helm.sh/chart: vela-core-0.1.0
  name: vela-core
  namespace: oam-system
  resourceVersion: "3686"
  selfLink: /apis/apps/v1/namespaces/oam-system/deployments/vela-core
  uid: 8f22f944-1773-4f7b-9d09-c01298647498
spec:
  progressDeadlineSeconds: 600
  replicas: 1
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      app.kubernetes.io/instance: vela-core
      app.kubernetes.io/name: vela-core
  strategy:
    rollingUpdate:
      maxSurge: 25%
      maxUnavailable: 25%
    type: RollingUpdate
  template:
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/instance: vela-core
        app.kubernetes.io/name: vela-core
    spec:
      containers:
      - args:
        - --metrics-addr=:8080
        - --enable-leader-election
        - --use-webhook=true
        - --webhook-port=9443
        - --webhook-cert-dir=/etc/k8s-webhook-certs
        image: oamdev/vela-core:latest
        imagePullPolicy: Always
        name: vela-core
        ports:
        - containerPort: 9443
          name: webhook-server
          protocol: TCP
        resources:
          limits:
            cpu: 500m
            memory: 1Gi
          requests:
            cpu: 50m
            memory: 20Mi
        securityContext: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /etc/k8s-webhook-certs
          name: tls-cert-vol
          readOnly: true
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext: {}
      serviceAccount: vela-core
      serviceAccountName: vela-core
      terminationGracePeriodSeconds: 30
      volumes:
      - name: tls-cert-vol
        secret:
          defaultMode: 420
          secretName: webhook-server-cert
status:
  availableReplicas: 1
  conditions:
  - lastTransitionTime: "2020-10-01T21:10:09Z"
    lastUpdateTime: "2020-10-01T21:10:23Z"
    message: ReplicaSet "vela-core-568cf7b985" has successfully progressed.
    reason: NewReplicaSetAvailable
    status: "True"
    type: Progressing
  - lastTransitionTime: "2020-10-01T21:11:09Z"
    lastUpdateTime: "2020-10-01T21:11:09Z"
    message: Deployment has minimum availability.
    reason: MinimumReplicasAvailable
    status: "True"

@kminder A quick update is it seems minikube doesn't enable nginx ingress by default, so if you do minikube addons enable ingress, the public IP will show up.
I'd guess it's maybe the case for OKE as well.

I think this issue have been fixed by https://github.com/oam-dev/kubevela/pull/387, we will create a release recently.
Thanks for your report, it's very helpful！


App restarts to first screen when connecting or disconnecting bluetooth device

Hi everyone.
Whenever i connect or disconnect bluetooth device my flutter application restarts to the first screen, my flutter doctor -v goes as below
[√] Flutter (Channel stable, 3.0.3, on Microsoft Windows [Version 10.0.19044.1806], locale pt-BR)
    • Flutter version 3.0.3 at C:\Src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 676cefaaff (12 days ago), 2022-06-22 11:34:49 -0700
    • Engine revision ffe7b86a1e
    • Dart version 2.17.5
    • DevTools version 2.12.2

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\lucas\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • CHROME_EXECUTABLE = C:\Program Files\BraveSoftware\Brave-Browser\Application\brave.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.16)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.32602.291
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[√] Connected device (4 available)
    • SM A015M (mobile) • R9QNB01D7DV • android-arm    • Android 11 (API 30)
    • Windows (desktop) • windows     • windows-x64    • Microsoft Windows [Version 10.0.19044.1806]
    • Chrome (web)      • chrome      • web-javascript • unknown
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 103.0.1264.37

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Android manifest contains

<uses-permission android:name="android.permission.INTERNET"/>
    <uses-permission android:name="android.permission.BLUETOOTH" />
    <uses-permission android:name="android.permission.BLUETOOTH_ADMIN" />
    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION"/>

Hi @ldsouza93, thanks for filing the issue.
I can reproduce the issue on the latest stable and master channels with sample code using go_router and a sample code using Navigator.

Demo

go_router:

https://user-images.githubusercontent.com/104349824/177272721-60237851-a98c-45d0-80b8-2d0dfe95ac69.mp4

navigator:

https://user-images.githubusercontent.com/104349824/177273052-f670c480-8fd8-470a-9602-7a91afbce967.mp4

Sample code (Navigator)
import 'package:flutter/material.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(
        scaffoldBackgroundColor: darkBlue,
      ),
      debugShowCheckedModeBanner: false,
      home: Scaffold(
          appBar: AppBar(
          title: const Text('Main'),
        ),
        body: Center(
          child: MyWidget(),
        ),
      ),
    );
  }
}

class MyWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Column(
        mainAxisAlignment: MainAxisAlignment.center,
        crossAxisAlignment: CrossAxisAlignment.center,
        children: [
          ElevatedButton(
            onPressed: () {
              Navigator.push<void>(
                context,
                MaterialPageRoute(builder: (context) {
                  return NewPage();
                }),
              );
            },
            child: const Text('Next Page'),
          )
        ]);
  }
}

class NewPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: const Text('Back'),
        ),
        body: const Center(child: Text('Exception Propagating')));
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.4, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.0.4 at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 85684f9300 (3 days ago), 2022-06-30 13:22:47 -0700
    • Engine revision 6ba2af10bb
    • Dart version 2.17.5
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.68.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.42.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 103.0.5060.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 3.1.0-0.0.pre.1501, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.1.0-0.0.pre.1501 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 753fee42cf (89 minutes ago), 2022-07-04 22:09:02 -0400
    • Engine revision 94b47cc0eb
    • Dart version 2.18.0 (build 2.18.0-254.0.dev)
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13E113
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.68.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.42.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 103.0.5060.114

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Even though I don't need to add those above permissions, the issue is still reproducible. Can you confirm this?
Also, I don't have Apple device (iPad for example) to verify this issue. Do you only get this issue on Android?


Squiggle PT Merge & Updates

My current plans are to finish out "Squiggle PT" which became "Gruesome Claim". It's the first game made using this game base. (See here https://github.com/QueenOfSquiggles/squiggle-pt)
Once that is completed I have a few goals. I'm listing it as an issue so it's tracked alongside the code and anyone can see my plans and comment on it if they feel the need. Yes, I know this is not really a "proper" issue because it is listing a ton of different goals, but I'm lazy and it's my project so deal with it :stuck_out_tongue_closed_eyes:
Plans:

[ ] Merge updates from squiggle-pt back into this repository

This will take place in a new branch and merge into main once stable.
This will not be a formal PR so it might not be super apparent where the code changes come from.

[ ] Add a readme for any potential users so this project is understandable and usable

Describe license, what it is for those unfamiliar, as well as why it was chosen for this
Include some information about the boiler-plate structure and what kinds of features are included
How to get started on the project (Link to squiggle-pt as an example of how to use the base and what can be done.)

[ ] List out stats and requirements

requirements for development (Godot 4.0, dotnet 6, etc)

[ ] Explain exporting process and python tool scripts

Where to target exported files
How to make exports compatible with the Itch IO uploader
How to properly configure Butler for use with the scripts
How to write the configuration file such that your account and not mine is referenced (Butler requires a log in, so there's no worry about my account security, but it will never work unless the config file is properly filled out)

[ ] Consider expanding into a Wiki on the repo because this is already a lot of information that would benefit from the structure of a wiki!

This is where feedback on the game base would be meaningful, but  honestly only one other person has even starred this project as of writing so IDK if anyone else will ever use this. So maybe I'm just writing this documentation for myself.

Eventual goal (not in this issue): Work on filling out the C# documentation such that various important classes have meaningful information in-editor without demanding the user rifle through all of the code. Basically, make it so they don't have to understand everything before actually making something interesting.

Closed with PR


Implement Display trait for CompressionMethod

Thank you for the contribution.
Before I merge this, could you please explain why you would want to use the display trait instead of using the debug trait directly?

This patch is meant to provide expected behavior in user-facing release vs. programmer-facing debug contexts. Per Debug's docs, "Debug should format the output in a programmer-facing, debugging context," whereas according to Display's docs, "Display is similar to Debug, but Display is for user-facing output." For example, if someone wanted to write a CLI utility that detailed the contents of an archive, they could use {} as expected for printing name, comment, size, etc., but they'd inconsistently have to use {:?} for printing the compression method.

Ah yes, for CLI purposes this seems the best way to go. Thanks! :+1:


typeof operator used on an inferred record type produces a cryptic error

As described here, the typeof operator is probably not intended to be used on inferred types. However, in Typescript it is possible to do it, and it is in fact a very common use case.
Here is an example of using the typeof operator on an inferred record type.
const x = {
  name: "Bat",
  surname: "Man"
}

/**
 * @type {typeof x} // This raises cryptic error
 */
const y = {
  name: "Cat",
  surname: "Woman"
}

And it produces this error:
./src/js/index.js:10:12: WARNING - [JSC_TYPE_MISMATCH] initializing variable
found   : {name: string, surname: string}
required: {name: string, surname: string}
  10|   const y = {
                  ^
  11|     name: "Cat",
      ^^^^^^^^^^^^^^^^
  12|     surname: "Woman"
      ^^^^^^^^^^^^^^^^^^^^
  13|   }
      ^^^

This error message does not make a lot of sense. Two identical types do not match? A better readable error message would be along the lines of "It is forbidden to use typeof operator on inferred types / inferred record types".
Compiler Version: v20221102
Build command:
java -jar ./scripts/closureCompiler.jar \
  --entry_point=./src/js/index.js \
  --js=./src/**.js \
  --dependency_mode=PRUNE \
  --warning_level=VERBOSE \
  --js_output_file=./dist/bundle.js \
  --module_resolution=WEBPACK \
  --compilation_level=ADVANCED \
  --jscomp_error=checkDebuggerStatement \
  --jscomp_error=unusedLocalVariables \
  --jscomp_error=reportUnknownTypes;

closure-compiler does not do structural matching of types to determine their equality unless they are @record types.
// closure-compiler will construct an ad-hoc type that represents the type of this value.
// It won't get a name, and it will only appear directly in the AST attached to the node(s) referring to this value.
const x = {
  name: "Bat",
  surname: "Man"
}

// closure-compiler will generate a distinct ad-hoc type that represents the value of `y`.
// It will then check to see if that type can be considered an instance of the ad-hoc type created for `x`.
// The answer to that question is "no", because the compiler does not do structural matching of types unless
// one or both of them are declared with `@record`.
/**
 * @type {typeof x}
 */
const y = {
  name: "Cat",
  surname: "Woman"
}

If you want structural matching, you have to explicitly ask the compiler to do it by declaring an @record type.
/** @record */
const Xtype = class {
  constructor() {
    // Declare the fields in the constructor like this.
    /** @type {string} */
    this.name;
    /** @type {string} */
    this.surname;
  }
}

// Structural type matching will recognize that the value of `y` can be used where an `Xtype` is needed.
/**
 * @type {!Xtype}
 */
const y = {
  name: "Cat",
  surname: "Woman"
}

It's unfortunate that the error message is confusing, of course,
but it's also not something that comes up much in normal usage.
We might be willing to accept a PR that clarifies the error message,
but we're not likely to change the basic behavior.
One possibility would be to print something like "Anonymous type () " in the error message so the user can see that although the contents of the types match, they aren't actually the same.


Need to add an optimistic mode

Search giphy for a gif but maintain a blacklist rather than a whitelist.
Idea from @robfe

fair


[draft]: remove matplotlib dependency

well, here's a minimal example of making mpl optional.  Though, I later noticed that scikit-learn is also coming in, and that's also more than I would want to bring in in cmap, so this might not be so easy afterall
while this does allow all your tests pass without mpl (except those that explicitly need a colormap with get_cmap), i'll note the obvious reduction in support for all the many color formats that matplotlib supports if someone passes in a color that is neither a hex string or a sequence of numbers.
An alternative to this (that I guess I'm leaning towards again) is for me to just vendor the bits of glasbey that I want over in cmap (which is mostly create_palette, which doesn't need the sklearn nearest neighbors or get_cmap functionality of the extension functions).  Would you rather I did that (and credit you and the original package of course) rather than muck about too much with your package here?

... I think I'll probably go ahead and just copy the bits i need. Wanting to keep dependencies minimal is my issue, not yours :) and I'm hesitant to inject inconveniences into this project unless others are really wanting it to.     That said, if you would like me to push on it more, feel free to reopen and/or request any specifics.  thanks for your time

I'm actually quite keen to get the dependencies down for glasbey, but I'm also trying to be relatively featureful, and not have to re-invent the wheel to much to do so. Dropping a full mpl requirement would have been good. The sklearn dependency could probably have been worked around -- I'm really just using their kd-tree implementation. Still, if you just want create_palette it's probably not that hard to simply grab the bits you need from grids and internals and makes things simple for you.

Thanks for putting in the effort of the PR, and good luck with cmap.


Flatten the structure of the python package

We do not need files to be in the fsm module. We can move them to the parent module.
We will need to make the appropriate changes to outlines and benchmarks(-txt) to reflect this change.

We will need to open an issue on outlines and benchmarks to make them compatible to the new package structure.


fixes to tests for py2/3 syntax compatibility

This follows up on #5368 and adds python 2/3 compatibility fixes to some of the tests themselves.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

This needs conflict resolution.
Drop the "squashed commit messages:" from your commit log message - a list of what is changed in a commit is fine; referring to earlier, lost-to-the-world draft work in the final draft of the very thing that underwent drafting is a (natural language, not programming language) abstraction violation. :-P
The change content looks great; thank you for contributing it.

This is OK to test.

From the sanity test failure: src/python/grpcio/tests/unit/framework/interfaces/base/_control.py modified 2016 but copyright only extends to 2015.

Fixed the last copyright boilerplate

Apologies, but this needs conflict resolution again.

Rebased to resolved conflicts.

@nicolasnoble, @jtattermusch: LGTM, and what action should be taken on this? Is it agreed that the C and C# failures cannot possibly be triggered by this change?

needs resolving. Please add "ready to merge" again once tests finish.

Conflicts are resolved and tests passing.

Re-LGTM.


if what's stored in local storage to create a product is using an old structure, then remove what's saved

use an identifier to know when to update
or check entire structure that's saved to know if it's usable

could not replicate but good to proceed to done


SSG: Certain extensions go wrong.

What version of Hono are you using?
4.0.0-rc.1
What runtime/platform is your app running on?
any
What steps can reproduce the bug?
toSSG app.get('/about', (c) => c.text('About Page')).
What is the expected behavior?
Output is index.html
What do you see instead?
The output is indexhtml. This is a bug that occurs only in the determineExtension case.
Additional information
No response

I think I have an idea. I'll fix it.

Fixed by #2029


nil as NSDictionary's key

Classification (type of problem: security, crash, performance, serious bug, feature, enhancement etc)
Crash
Reproducibility (always, sometimes, rarely, N.A.)
N.A
Didn't try to reproduce crash under debugger.
Description
Fatal Exception: NSInvalidArgumentException
*** -[__NSDictionaryM removeObjectForKey:]: key cannot be nil
Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x181aad1b8 __exceptionPreprocess
1  libobjc.A.dylib                0x1804e455c objc_exception_throw
2  CoreFoundation                 0x1819aa7a4 -[__NSDictionaryM removeObjectForKey:]
3  SensorbergSDK                  0x101731828 (Missing)
4  tolo                           0x1019d54d8 (Missing)
5  SensorbergSDK                  0x1017302bc (Missing)
6  SensorbergSDK                  0x10172efd4 (Missing)
7  CoreLocation                   0x189ab1f90 (null)
8  CoreLocation                   0x189aa34b8 (null)
9  CoreFoundation                 0x181a5aa44 CFRUNLOOP_IS_CALLING_OUT_TO_A_BLOCK
10 CoreFoundation                 0x181a5a240 __CFRunLoopDoBlocks
11 CoreFoundation                 0x181a58538 __CFRunLoopRun
12 CoreFoundation                 0x1819862b8 CFRunLoopRunSpecific
13 GraphicsServices               0x18343a198 GSEventRunModal
14 UIKit                          0x1879cd7fc -[UIApplication _run]
15 UIKit                          0x1879c8534 UIApplicationMain
16 App                          0x1002800b4 main (AppDelegate.swift:24)
17 libdispatch.dylib              0x1809695b8 (Missing)
Sensorberg SDK version
2.5.3
iOS version
8, 9, 10
Device
iPhone, iPad
Additional notes

Hi rafalkitta,
I'm investigating on it.
Can you please try to run the app under debugger?

Hi,
I have it under debugger, crash didn't occur for me. I only see it in Crashlytics.

@rafalkitta Thank you for reporting this issue - we fixed it and released v2.5.4


Documentation Typo

I noticed the documentation for the RunPostBuildEvent MSBuild property contains incorrect info; I suspect a typo [link to affected file]. Specifically, I believe the OnOutputUpdated and OnBuildSuccess rows in the tables have their descriptions backwards. See below screenshot for how it appears at time of this writing.

Good catch - I'd accept a contribution to fix if you like, otherwise I'll fix it next couple of days.

@davkean I would like to contribute to VB but I have no idea how. Yes I know how to edit the code, and write tests. It is the :"contribution process" I don't understand. Is there a how to contribution, using VS2015 document for the complete novice? I have been using VB since it was released but am new to open source.

Which part of the contribution process? Git? How to submit a PR?

Both, I was able to Clone Roslyn using Desktop GitHub on Windows 10 but I don't see how I get to a specific branch, do I need to clone that too? Then I fix the issue, which I have done and tested using the Zip version. But where do I go from there.

@paul1956 If you are using the command line, git checkout -b my_branch will create a new Git branch named “my_branch”. GitHub Desktop provides a "create branch" button (near the top of the window, located just to the right of the sidebar, next to the button identifying the branch (in your case, it probably says ”master”). Click that and it will open a popup containing a blank for the name and the branch to create it from. Fill in both, click Create Branch, and ta-da: you now have a brand new branch.

OK I created a new branch "MyFix" from stabilization and I can open it in Explorer but not sure how I know it is "MyFix" I am working on. I did the fix (since it has been an issue since Master it doesn't matter to me what source I start with), then what?
.

I committed fix from Visual Studio but can't submit Pull Request.

You're trying to push to directly to the Roslyn's repo, instead of to your fork. You need to fork Roslyn, and push changes to it - from which you create PRs.
These are the steps you need to take:

Create a fork of a project (there's a Fork button on the front page of every repo) under your name
From the command-line type:
```git remote add origin https://github.com/paul1956/roslyn.git`
Then you want to push your branch to it; git push origin
Then create a new PR from the branch you pushed to the Roslyn repo.


Fix inconsistent hashing bug

I played around with a bunch of different approaches, but the best I could come up with was to use the _make_key function to produce a normalized arg list then pickle each item individually and run it through sha256 to make a final hash. This produces consistent results across executions, at least in the testing I did (including one automated test to make sure this doesn't break in the future).
This PR also includes a few other cleanups that have been recently discussed.
Fixes #95
Fixes #114

Released in v2.1.0


siconos_vview do not longer work with mesh file.

----- ~/siconos-tutorials/examples/mechanics/GeometricPrimitives -----                                                      [22:05:10][0]
[acary@ohana-2]$ siconos_vview mesh.hdf5
/Users/acary/Library/Python/3.5/lib/python/site-packages/h5py/__init__.py:34: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
Warning: Error loading configuration `/Users/acary/.config/siconos_vview.json'
/usr/local/lib/python3.5/site-packages/vtk/util/numpy_support.py:137: FutureWarning: Conversion of the second argument of issubdtype from `complex` to `np.complexfloating` is deprecated. In future, it will be treated as `np.complex128 == np.dtype(complex).type`.
  assert not numpy.issubdtype(z.dtype, complex), \
Generic Warning: In /Users/acary/Softs/VTK-8.1.2/Common/Core/vtkInstantiator.cxx, line 265
vtkInstantiator::CreateInstance was deprecated for VTK 8.1 and will be removed in a future version.

Generic Warning: In /Users/acary/Softs/VTK-8.1.2/Rendering/Core/vtkMapper.cxx, line 186
vtkMapper::ImmediateModeRenderingOff was deprecated for VTK 8.1 and will be removed in a future version.

Traceback (most recent call last):
  File "/Users/acary/.local/bin/siconos_vview", line 23, in <module>
    vview.run()
  File "/Users/acary/Library/Python/3.5/lib/python3.5/site-packages/siconos/io/vview.py", line 2363, in run
    self.initialize_vtk()
  File "/Users/acary/Library/Python/3.5/lib/python3.5/site-packages/siconos/io/vview.py", line 2322, in initialize_vtk
    self.init_instances()
  File "/Users/acary/Library/Python/3.5/lib/python3.5/site-packages/siconos/io/vview.py", line 1715, in init_instances
    self.init_instance(instance_name)
  File "/Users/acary/Library/Python/3.5/lib/python3.5/site-packages/siconos/io/vview.py", line 1711, in init_instance
    self.init_contactor(contactor_instance_name, instance,  instid)
  File "/Users/acary/Library/Python/3.5/lib/python3.5/site-packages/siconos/io/vview.py", line 1642, in init_contactor
    if not (self.global_filter or self.opts.export):
AttributeError: 'VView' object has no attribute 'global_filter'

fixed in 54f793b707aa6c6c51fdb5ea02c9c53e0cc6f9a7


add changes for v0.8.1 to CHANGELOG.md

Don't need you need all the whitespace changes but they don't hurt either..


Remove deprecated props, behaviors and components

The v1 release is an opportunity to remove deprecated features where appropriate.
Here is a list of what we should remove:

in ActionBar:

use of children instead of actions (#1412).

in BreadcrumbWithOverflow:

use of children instead of breadcrumbs (#1421).

entire component ContextHeader (#1417).
in CreateSidePanel:

use of pageContentSelector instead of selectorPageContent (#1406).

in PageHeader:

use of actionBarItems with an element or an array of elements instead of an array of shape (TODO).
use of actionBarOverflowLabel instead of actionBarOverflowAriaLabel (TODO).
use of availableSpace instead of children (TODO).
use of background instead of hasBackgroundAlways (TODO).
use of breadcrumbItems instead of breadcrumbs (TODO).
use of breadcrumbOverflowLabel instead of breadcrumbOverflowAriaLabel (TODO).
use of collapseHeaderLabel instead of collapseHeaderIconDescription (TODO).
use of collapseHeaderToggleWanted instead of hasCollapseHeaderToggle (TODO).
use of expandHeaderLabel instead of expandHeaderIconDescription (TODO).
use of pageHeaderOffset, which is no longer required (TODO).
use of preCollapseTitleRow instead of collapseTitle (TODO).
use of preventBreadcrumbScroll instead of disableBreadcrumbScroll (TODO).

in SidePanel:

use of pageContentSelector instead of selectorPageContent (#1406)
use of onActionToolbarButtonClick in actionToolbarButtons elements instead of onClick (https://github.com/carbon-design-system/ibm-cloud-cognitive/pull/1414).

in Tearsheet, TearsheetNarrow and TearsheetShell:

use of preventCloseOnClickOutside, which is no longer required (#1416).

Fixed by merge of above PRs for v1 release


Update README.md

Removed some settings resources that trows an error when you try to add them right now.
Also just changed so the text does not mention volkswagenconnect configuration block.

Ahh strange maybe there was the kodiaq_combustion_engine_heating_ventilation_status that made the others below to fail.
Then I will just remove everything in resource list and let the magic happen.
Ohh would be cool if all the entities would be linked to a device and you gather all of them under that. The car VIN should do as a unique ID...
https://developers.home-assistant.io/docs/device_registry_index

I am not so skilled in Home Assistant development, I am happy I was able to get it working in current state :) But feel free to perform changes which will do the trick, I will be happy to merge it into the code.

Ahh strange maybe there was the kodiaq_combustion_engine_heating_ventilation_status that made the others below to fail.
Then I will just remove everything in resource list and let the magic happen.
Ohh would be cool if all the entities would be linked to a device and you gather all of them under that. The car VIN should do as a unique ID...
https://developers.home-assistant.io/docs/device_registry_index

I will take a look and see if I can make it happen.
Haven't had much time for coding the last week but starting to get service call working, will take a look and see if we can make this work.


Giving fid=0 to feature gets overwritten.

In line 496 of __init__.py, it sets feat['id'] to fid if fid is truthy.  This prevents you from passing in an fid=0, which would be natural from an enumerate.  The condition should be if fid is None:, I think.
Happy to send over a PR to fix this if you want.

@jagill mercantile is more opinionated about feature ids than some other software. As it says in the docstring, we want strings. We're not failing in the integer case, but I think I would rather do that.


chore: improve issue templates

Description
Update issue templates to encourage people to write more information.
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

It seems this is the best choice.

If value was set, the <details> will appear even if the users didn't paste logs.

You rock!


Fix checkbox

Supersedes #208 — moves the input/output toggle into the repl package, which results in a more logical relationship between components and makes it easier to use <Repl>

Marking as draft because there's some remaining details to iron out around orientation


raft heartbeat in eb

After some digging, process_in_event_base has no effect for both previous and current version fbthrift, all requests received from io thread will be handled worker thread. If request do need to process in eb, need to remove process_in_event_base and add annotation.
Storage PR will be opened later.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Issue on catkin build tensorflow_ros_cpp

While building this package, tensor flow is required. Apparently, I faced some problem while running

catkin build tensorflow_ros_cpp

while gave me a error return of
________________________
Errors     << tensorflow_ros_cpp:cmake /home/youliang/segmap_ws/logs/tensorflow_ros_cpp/build.cmake.004.log
-- tensorflow_ros_cpp is probing the system
-- - The system uses C++11 ABI, the use of tensorflow from pip is possible, but problematic; consider compiling tensorflow yourself
-- - Not searching for Tensorflow installed by pip since the system uses C++11 ABI. Set FORCE_TF_PIP_SEARCH to force the search.
-- - Trying to find Tensorflow compiled by bazel
CMake Warning at /home/youliang/segmap_ws/src/tensorflow_ros_cpp/cmake/detect_tf_bazel.cmake:21 (message):
  Bazel-compiled Tensorflow library
  /home/youliang/segmap_ws/devel/../libtensorflow_cc.so not found.
Call Stack (most recent call first):
  CMakeLists.txt:89 (include)

-- - Trying to find Tensorflow from tensorflow_catkin
CMake Warning at /home/youliang/segmap_ws/src/tensorflow_ros_cpp/cmake/detect_tf_catkin.cmake:4 (message):
  tensorflow_catkin was not found
Call Stack (most recent call first):
  CMakeLists.txt:104 (include)

CMake Error at /home/youliang/segmap_ws/src/tensorflow_ros_cpp/CMakeLists.txt:115 (message):
  Tensorflow was not found

cd /home/youliang/segmap_ws/build/tensorflow_ros_cpp; catkin build --get-env tensorflow_ros_cpp | catkin env -si  /usr/bin/cmake /home/youliang/segmap_ws/src/tensorflow_ros_cpp --no-warn-unused-cli -DCATKIN_DEVEL_PREFIX=/home/youliang/segmap_ws/devel -DCMAKE_INSTALL_PREFIX=/home/youliang/segmap_ws/install -DCMAKE_BUILD_TYPE=Release; cd -
........................................................
Failed     << tensorflow_ros_cpp:cmake          [ Exited with code 1 ]
Failed    <<< tensorflow_ros_cpp                [ 0.4 seconds ]
[build] Summary: 0 of 1 packages succeeded.
[build]   Ignored:   30 packages were skipped or are blacklisted.
[build]   Warnings:  None.
[build]   Abandoned: None.
[build]   Failed:    1 packages failed.
[build] Runtime: 0.5 seconds total.

I have previously installed tensorflow from source, but I have no idea on why this cmake error happened. Hope someone can help me with this. Thanks!

The problem seems it's not finding the tensorflow installation. Can you give more details on your system/setup and how you installed tensorflow?

I tried pip install tensorflow, install tensorflow 1.8 from source, and also catkin build tensorflow_catkin package. All tensorflow installation succeed. Still the segmap compilation is unable to find the tensorflow package. Srry Iam unfamiliar with this, hope some useful information can help me out. Thanks alot

From the error message I see you are probably on Ubuntu 16 or 18. Which means you need to build tensorflow from source, install it using pip into your environment. Afterwards when you compile tensorflow_ros_cpp, see here on the cmake flags you need to set during compilation. You will need to specify this flag at least FORCE_TF_PIP_SEARCH as ON. In case it doesn't work try also correctly specifying the other flags related to python version, pip and library location.

Here's a more indepth explanation of the required flags:
https://github.com/ethz-asl/segmap/wiki/FAQ#q-issues-compiling-tensorflow_ros_cpp

Thanks, @smauq.
Had the same question of not knowing what --cmake-args to set when using custom-compiled tensorflow. Got it solved using your link.
Much appreciated,
Dennis


error: null response

hey there … today I just wanted to use trans to translate some word, but it has stopped working …
$ trans en:sk burka
[ERROR] Null response.
[ERROR] Oops! Something went wrong and I can't translate it for you :(

I am sure it use to work perfectly … all the depencences are fulfilled …

See #67 for solution.

thank You, it works with user agent :)
On 9 August 2015 at 03:54, Mort Yao<EMAIL_ADDRESS>wrote:

See #67 https://github.com/soimort/translate-shell/issues/67 for
solution.
—
Reply to this email directly or view it on GitHub
https://github.com/soimort/translate-shell/issues/70#issuecomment-129087749
.

$ trans -u chrome hello
returned:
[ERROR] Null response.
[ERROR] Oops! Something went wrong and I can't translate it for you :(


Configurable API endpoint

Allow to use other API offerings like the one from AzureOpenAi- which shall have compatible api but private model which is especially relevant if source code is not OS. Configuring another non default API url might already be sufficient

would you like to make a PR?

https://github.com/di-sukharev/opencommit/pull/80


Update composer.json

increase symfony version requirements because older versions have security warnings.

@php-http/httplug should we do these? if we don't there are security warnings on the repo because we allow vulnerable versions of symfony components. not sure if it is our job to do this - on the other hand if people realize thanks to this that they should be upgrading their vulnerable components, it has value.

thanks!


size mismatch for model.mapping

Hello again,
I tried to run the protein function annotation task after downloaded pkl file. However, I got the following error:
RuntimeError: Error(s) in loading state_dict for MultipleBinaryClassification:
size mismatch for model.mapping: copying a param with shape torch.Size([20]) from checkpoint, the shape in current model is torch.Size([33]).
[sj4@gn10 ProtST]$ python ./script/run_downstream.py --config ./config/downstream_task/PretrainESM2/annotation_tune.yaml --checkpoint /work/sj4/protst_esm2.pth --dataset GeneOntology --branch BP
The yaml is file is almost the same as one on GitHub except I changed the number of GPUS to gpus: [0]. Could you please check this issue?
Best wishes,

Hi, I tried to run the experiment with
python ./script/run_downstream.py --config ./config/downstream_task/PretrainESM2/annotation_tune.yaml --checkpoint ~/scratch/esm-model-weights/esm2_t33_650M_UR50D.pt --dataset GeneOntology --branch BP

after resetting gpus as [0]. It is successful.
Would you give more details about the checkpoint of ESM2 you have?

Hi, I tried to run the experiment with
python ./script/run_downstream.py --config ./config/downstream_task/PretrainESM2/annotation_tune.yaml --checkpoint ~/scratch/esm-model-weights/esm2_t33_650M_UR50D.pt --dataset GeneOntology --branch BP

after resetting gpus as [0]. It is successful.
Would you give more details about the checkpoint of ESM2 you have?

I have also encountered a similar issue. May I ask why you used the original model for downstream tasks in your answer? Shouldn't we use a pre-trained model like the protst_esm2.pth mentioned above?

Hi, I tried to run the experiment with
python ./script/run_downstream.py --config ./config/downstream_task/PretrainESM2/annotation_tune.yaml --checkpoint ~/scratch/esm-model-weights/esm2_t33_650M_UR50D.pt --dataset GeneOntology --branch BP

after resetting gpus as [0]. It is successful.
Would you give more details about the checkpoint of ESM2 you have?

I also meet the same issue, I try to run the zero-shot Predictors.The information is as follows:
python ./script/run_zero_shot.py --config ./config/zero_shot/PretrainESM/zero_shot.yaml --checkpoint /root/autodl-tmp/ProtST-main/protst_esm2.pth --prompt_label ./data/zero_shot_classification/subloc_name.tsv --dataset SubcellularLocalization --field "['name']"
I prepared the protst_esm2.pth model in advance in /root/autodl-tmp/ProtST-main.

Hi, sorry for the misleading comments before. I used the ESM2 checkpoint instead of the ProtST-enhanced one only to test if the released code has any inconsistency with our developing codebase. I'm downloading the ProtST-enhanced ESM2 checkpoint and would come back to you once I tested it.

Hi, I downloaded the ProtST-enhanced ESM2 checkpoint (https://protsl.s3.us-east-2.amazonaws.com/checkpoints/protst_esm2.pth, the url link is also stated in README). I have run the following command on 1 GPU, and it's successful.
python ./script/run_downstream.py --config ./config/downstream_task/PretrainESM2/annotation_tune.yaml --checkpoint /home/xinyu402/scratch/protst_output/protst_esm2.pth --dataset GeneOntology --branch BP

This suggests that the uploaded checkpoint and the released code should work fine. Potential problem I can think of is running environment. I currently use torch==1.13.1 and torchdrug==0.2.0. Could you provide more details on the environment versions ?
And if you'd like, you can provide me with the checkpoint you're using right now so that I can use it to reproduce the issue and further detect the reason behind.

Excuse me, I met the same issue when I run
python ./script/run_t2p_retrieval.py --config ./config/t2p_retrieval/go_mf.yaml --checkpoint $PRETRAIN_CHECKPOINT
I just download the esm1b checkpoint you provide, and got the similar error :
RuntimeError: Error(s) in loading state_dict for PretrainEvolutionaryScaleModeling:
size mismatch for mapping: copying a param with shape torch.Size([20]) from checkpoint, the shape in current model is torch.Size([33]).
I had tried to roll mt TorchDrug back to 0.2.0. However I got another error:
AttributeError: 'dict' object has no attribute 'startswith'
And I found that in esm/pretrained.py Line 190, the variable ''model_name'' is different between torch 0.2.0 and 0.2.1 versions.


Important Issues to stimulate development.

Following help/issue should be done to stimulate the development.

Clear Timeline setup for the project
Development setup
chat channel instructions
Weekly/monthly online meet.
Special instructions for first-time contributors to this repo.

Feel free to add others

Thank you for your comment! These are interesting, albeit maybe obvious from your PoV, points to consider.

Clear Timeline setup for the project

I would like to know more about this. If your talking about a Q1, Q2, … kind of timeline I'm afraid that there is not enough manpower behind this project to give any kind of realistic timing estimates. It might still be something we'd like to do in some general way however.
Also, there is issue #49 mentioned in the README that tries to give an up-to-date list of were work is currently needed.

Development setup

As all related work is currently happening in other repos there is not currently anything to setup to work on py-ipfs itself.

chat channel instructions

Agreed, I'm personally not the kind of persons that watches a chat channel all the time (I just cannot focus on anything else if I do that), but there are other members that maybe able to maintain such channel? (@AliabbasMerchant maybe? :grin:)

Weekly/monthly online meet.

Do you maybe have some suggestions on how to do this kind of stuff?

Special instructions for first-time contributors to this repo.

Hmm… I'm not sure what that would be other then “look at issue #49”? Any ideas?
@AliabbasMerchant: Your opinion on this would be highly appreciated as well!

Hi @Seenivasanseeni
Its great to see your interest in the project!
As @alexander255 mentioned, the problem with properly starting the project now is that the dependencies themselves are not ready.
We are waiting for the dependencies to be ready, so that we can start off with the project.
Some of the dependencies currently being worked on are:

https://github.com/ipfs/py-datastore
https://github.com/libp2p/py-libp2p/
Also, I had started with the bitswap implementation here: https://github.com/AliabbasMerchant/py-ipfs-bitswap, but its currently halted, as I am currently working on py-datastore.

Regarding setting up a channel, there is already a gitter channel: https://gitter.im/ipfs/py-ipfs, but its not at all active. (In fact, @alexander255 himself isn't a member 😀 )

Regarding setting up a channel, there is already a gitter channel: https://gitter.im/ipfs/py-ipfs, but its not at all active. (In fact, @alexander255 himself isn't a member :grinning:)

Gitter is proprietary and needs me to have yet-another-client, so I'd really not like to use that unfortunately. Since we'd have to start from zero anyways, how about using Matrix/Riot instead? :grin:

I've create a Matrix room here, if you want to test: https://matrix.to/#/#py-ipfs:ninetailed.ninja

It looks great for now. Once our community is strong, we need to move to another platform. We can finalize this chat room. @alexander255  Right ?

Once our community is strong, we need to move to another platform.

Or add bridges to those, yes. I'll start by adding the IRC bridge since that one's the easiest. :slightly_smiling_face:

We can finalize this chat room. @alexander255 Right ?

Yes, I'll add some links to the README then.


Duplicate base URL

We're running the github-release command like this: https://github.com/screwdriver-cd/toolbox/blob/master/git-release.sh#L38
With the latest release (v0.8.0), we're seeing failures due to a duplicate base URL in release command:
00:08:16 Creating release v1.0.510 for screwdriver-cd / ui
00:08:17 error: Get "https://api.github.comhttps//api.github.com/repositories/64884594/releases?per_page=100&page=2": dial tcp: lookup api.github.comhttps on <IP_ADDRESS>:53: no such host

Could it be because there's already a defaultBaseUrl for NewClient?

Hi, thank you for the report, investigating now.

Someone filed an issue with a similar complaint recently: https://github.com/meterup/github-release/issues/1
However the script indicates you're downloading from the compiled version and that should have fixed the issue linked there. I also tried using the downloaded version and wasn't able to reproduce. I'm still investigating.

Here's the full log if that helps: https://cd.screwdriver.cd/pipelines/7/builds/233174/steps/publish

Thanks, I'm attempting to reproduce using your script in a Docker container.

Here's my Dockerfile:
$ cat Dockerfile
FROM ubuntu:bionic
COPY . /go
WORKDIR /go
ARG GITHUB_TOKEN
ENV GITHUB_TOKEN=$GITHUB_TOKEN
ENV RELEASE_FILE="./git-release.sh"
ENV GIT_TAG="issue-102-testing"
ENV GIT_ORG=kevinburke
ENV GIT_REPO=github-release
RUN apt-get update && apt-get install -y wget git
RUN bash ./git-release.sh

Replace GIT_ORG and GIT_REPO with a repo that your token has write permission for. And bash script - I modified to hard code the GIT_ORG and GIT_REPO and print out all instructions as they run.
#!/bin/bash -e
# GITHUB_TOKEN = Token for updating your git repo
# RELEASE_FILE = File to release

set -euo pipefail
set -x

if [ -z "$GITHUB_TOKEN" ] || [ -z "$RELEASE_FILE" ]; then
  echo Unable to release, missing environment variables
  exit 2
fi

GITHUB_RELEASE=/tmp/linux-amd64-github-release
if [ ! -f "$GITHUB_RELEASE" ] ; then
  echo Downloading github-release
  wget -q -O - https://github.com/github-release/github-release/releases/latest \
     | egrep -o '/github-release/github-release/releases/download/v[0-9.]*/linux-amd64-github-release.bz2' \
     | wget --base=http://github.com/ -i - -O /tmp/linux-amd64-github-release.bz2
  bzip2 -dv /tmp/linux-amd64-github-release.bz2
  chmod +x $GITHUB_RELEASE
fi

GIT_VERSION=/tmp/gitversion
if [ ! -f "$GIT_VERSION" ] ; then
    echo Downloading gitversion
    wget -q -O - https://github.com/screwdriver-cd/gitversion/releases/latest \
       | egrep -o '/screwdriver-cd/gitversion/releases/download/v[0-9.]*/gitversion_linux_amd64' \
       | wget --base=http://github.com/ -i - -O /tmp/gitversion
    chmod +x $GIT_VERSION
fi

#GIT_ORG=`git remote -v | grep fetch | sed 's/ (fetch)//' | cut -d'/' -f4`
#GIT_REPO=`git remote -v | grep fetch | sed 's/ (fetch)//' | cut -d'/' -f5`
#if [ -f VERSION ] ; then
  #GIT_TAG=$(<VERSION)
#else
  #GIT_TAG=`$GIT_VERSION --prefix v show`
#fi

echo "Creating release $GIT_TAG for $GIT_ORG / $GIT_REPO"

$GITHUB_RELEASE release --user $GIT_ORG --repo $GIT_REPO --tag $GIT_TAG --name $GIT_TAG
$GITHUB_RELEASE upload --user $GIT_ORG --repo $GIT_REPO --tag $GIT_TAG --name $RELEASE_FILE --file $RELEASE_FILE

I successfully created the release and uploaded the file: https://github.com/kevinburke/github-release/releases/tag/issue-102-testing
Is it possible you have an older, or different, version of github-release on the machine that is doing the upload?

(Put Dockerfile and git-release.sh in the same directory then run
docker build --build-arg GITHUB_TOKEN=$(cat ~/cfg/personal-github-token ) .

(get your GITHUB_TOKEN from wherever you get the token from)

I just pushed v0.8.1, which should provide more defense against a (possible) way that you ran into the issue above - if you could add $(GITHUB_RELEASE) --version to your script so I can ensure you're running the version I just released that would help with troubleshooting!

Confirmed it has v0.8.1 and it published successfully! Thank you
https://cd.screwdriver.cd/pipelines/7/builds/238720/steps/publish
What changed?

Frankly the whole thing leaves me a little confused. The most likely culprit is github-release compiled an outdated version of a dependency, github.com/kevinburke/rest, which displayed exactly the error that you presented at the top of the ticket.
However, I patched the dependency a while ago, and included the patched release in both the 0.8.0 and 0.8.1 binaries. If you were downloading a 0.8.x binary, you should have gotten the newer library version without doing anything. Also, if the 0.8.0 binary included the error, I would have expected many more people to write in with complaints.
0.8.1 vendors every dependency, which was a way to triple verify that the updated library version was included in the binary.