use XDG base directory compatible cache location

fixes #12674

@Tyriar and @mjbvz  any more comments?

//CC: @billti

@vladima Is this targeting 2.2 or will it make it into 2.1?

current plan is that it will go into 2.2

Ok. At least users can work around this by installing an nightly TS 2.2 build.
We'll keep the bug open on the VSCode side until we pick up a TS build with the fix.
Thanks


ECO-1336-layout components

Motivation / Description
The style for the layout components was defined at rcb-ui level.
This made it harder to write stories reusing the style (apart from copy pasting it).
This PR refactors the existing code to create some layout components that match the ones used in the rcb-ui entrypoint component. It also moves the style in the specific components.
Changes introduced

Created the aside-block, main-block, layout and container components
Used those components in rcb-ui and cleaned up the style from there
Used those components in the state-present-offer.stories.svelte so that now I can see the result of the customisation changes (there was no style before)

          1 Error

:no_entry_sign:
Label the PR using one of the change type labels. If you are not sure which label to use, choose pr:other.

Label
Description

pr:feat
A new feature. Use along with pr:breaking to force a major release.

pr:fix
A bug fix. Use along with pr:force_minor to force a minor release.

pr:other
Other changes. Catch-all for anything that doesn't fit the above categories. Releases that only contain this label will not be released. Use along with pr:force_patch, or pr:force_minor to force a patch or minor release.

pr:RevenueCatUI
Use along any other tag to mark a PR that only contains RevenueCatUI changes

pr:next_release
Preparing a new release

pr:dependencies
Updating a dependency

pr:phc_dependencies
Updating purchases-hybrid-common dependency

  Generated by :no_entry_sign: Danger


(iron-send-motion) changes the < and > marks

This produces unexpected results when trying to use gv.

I think this is a pretty normal behavior for plugins that work over text objects. I don't do anything special on this case to set up < and >, as you can see here.

Thanks for the response!
Are you sure? because I think that this issue does not appear in
some other plugins that also work with text objects...
Also, in my .vimrc I have a function like this,
function! FillSearch(...)
    if a:0 == 0
        let l:filltext = ""
    else
        let l:is_visual=(a:1 == "v")
        let l:line=getline(l:is_visual ? "'<" : "'[")
        let [line1,col1] = getpos(l:is_visual ? "'<" : "'[")[1:2]
        let [line2,col2] = getpos(l:is_visual ? "'>" : "']")[1:2]
        let l:filltext =l:line[col1 - 1: col2 - 1]
    endif
    call feedkeys(':grep' . ' "' . l:filltext . '"'."\<Left>")
endfunction

nnoremap <silent> g/  :set opfunc=FillSearch<cr>g@

and it does not change the < and > marks either,
when called from normal mode.
Best wishes,
Urbain
On Mon, Apr 16, 2018 at 06:25:59AM +0000, Henry John Kupty wrote:

I think this is a pretty normal behavior for plugins that work over
text objects. I don't do anything special on this case to set up <
and >, as you can see
here.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/BurningEther/iron.nvim/issues/74#issuecomment-381492483

Related to this, I have noticed that <Plug>(iron-send-motion) messes with folds sometimes, for example when doing yrgg when there are folds above the current position. (yr being my binding to iron-send-motion). It seems that the plugin moves the cursor at some point, no? Is this necessary?
Also, I wanted to say that I really like this plugin - thank you for the good work! :)

because I think that this issue does not appear in some other plugins that also work with text objects...

Yeah, you got me thinking on this.. I'll do some tests, but if it's not related to opfunc than it should either be <c-u> or winsaveview()s fault.
On python side, nothing seems to cause such change on the marks.
As for the folds. I'd bet that those are related to winsaveview().
I'll test those now and will post my findings afterwards.

Also, I wanted to say that I really like this plugin - thank you for the good work! :)

Thank you very much!
Cheers,
Henry John Kupty

Hum.. None seem to be involved. My last guess - by exclusion - is that this has to do with how the remote client handles ranges and so on. If this is the case, this will be solved by migrating to lua - which I'll retake in the upcoming weeks.

Yeah, you got me thinking on this.. I'll do some tests, but if it's
not related to opfunc than it should either be <c-u> or
winsaveview()s fault.
On python side, nothing seems to cause such change on the marks.
Yeah, I had a look at the code, and I don't see where the side-effects
might be coming from either...

As for the folds. I'd bet that those are related to winsaveview().
I'll test those now and will post my findings afterwards.
Thanks! I'll let you know too if I find something.

Hi,
I'm just wondering if this is fixed after the migration to lua backend or if it still persists.
Could you help me with that?
Cheers,
Henry John Kupty

Hi,
I hope you have tested the (not so) new version by now :sweat_smile:.
Do you think we can close this one?
Best regards,
Henry

Hi Henry,
I had been using the legacy branch, but I'll switch to the master branch now. :) I'll let you know and reopen the issue if I see any of the bugs I reported before.
Best wishes,
Urbain


feat: get blockchain by domain

The problem PR #729 was that it failed to fetch the blockchain when a simple URL hit. There wasn't a way to find a blockchain just by the domain.
This PR introduces a way to do this, tested locally and also in canary. It works like a charm. All of the errors for invalid domain disappeared.

Fixing the test asap


getProfilePicFromServer does not work in Multi-Device beta

The script return timeout, any solution?

As far as I remember, that didn't work even in the non-beta.
Whatsapp changed how and when do you can access such info. If you have chat or opened contact recently it will let you, otherwise dont.
So, if you're trying to get "a lot of contact's images" you cant anymore.
Describe your situation in order to be more concise.
Btw, someone found another way to retrieve contact images somewhere in the threads, digg for it.


oj.h not found on FreeBSD

Hi,
I maintain the gitlab-ce port for the FreeBSD.org system and the new gitlab-ce 15.6 version requires this gem.
I tried now to make a package for but I have problem generating the package, maybe you can help me.
All packages are generated in its own jail and only packages are available that are added before by build/run dependencies (I'm not sure if this is the problem here):
If I try to let the FreeBSD ports system generate the package for oj-introspect I see the following error message:
current directory: /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/lib/ruby/gems/3.0/gems/oj-introspect-0.7.1/ext/oj-introspect
make DESTDIR\= sitearchdir\=./.gem.20221203-45844-tsv7wz sitelibdir\=./.gem.20221203-45844-tsv7wz
compiling introspect.c
introspect.c:2:10: fatal error: 'oj.h' file not found
#include "oj.h"
         ^~~~~~
1 error generated.
*** Error code 1

Stop.
make[1]: stopped in /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/lib/ruby/gems/3.0/gems/oj-introspect-0.7.1/ext/oj-introspect

You can find the full log here:
https://pkg.fechner.net/data/131amd64-gitlab/2022-12-03_08h36m47s/logs/errors/rubygem-oj-introspect-0.7.1.log
I connected into the jail and search for the file and it is located here:
/usr/ports/devel/rubygem-oj/work/oj-3.13.23/ext/oj/oj.h
So I'm not sure if this problem is related to oj-introspect or if oj just misses to install this oj.h file in a common ruby location.
Thanks a lot for any hint.
Matthias

Hi Matthias,

All packages are generated in its own jail and only packages are available that are added before by build/run dependencies (I'm not sure if this is the problem here)

If I understand correctly you already mentioned that the oj.h exists in the system so this might not be the root cause.
You already mentioned that you can access to jail so could you please try compiling manually? If it fails again then we can print some information in extconf.rb or check the generated makefile.

If I understand correctly you already mentioned that the oj.h exists in the system so this might not be the root cause.

yes, but it is maybe not on a standard location and this is the reason why it is maybe not found.

You already mentioned that you can access to jail so could you please try compiling manually? If it fails again then we can print some information in extconf.rb or check the generated makefile.

I just looked into it, how the gem is generated and FreeBSD is calling the following command:
(cd /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/oj-introspect-0.7.1; /usr/bin/env RB_USER_INSTALL=yes LANG=en_US.UTF-8 LC_ALL=en_US.UTF-8 /usr/local/bin/gem install -l --no-update-sources --install-dir /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/lib/ruby/gems/3.0 --ignore-dependencies --bindir=/wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/bin --no-document oj-introspect-0.7.1.gem -- )
I attached you the generated Makefile.
oj-introspect_Makefile.txt
I can also modify the extconf.rb, but I'm not a ruby developer or do know ruby, I just package it ;)
(but I have good knowledge in many programming languages ;) )
I have more the impression that the oj gem does not install the oj.h header file and therefor it is clear that it cannot find it.
The header file is located in the source for oj package in /usr/ports/devel/rubygem-oj/work/oj-3.13.23/ext/oj/oj.h

Thanks for the additional information @mfechner!
I see that the makefile is using /usr/local/lib/ruby/gems/3.0/gems/oj-3.13.23/ext/oj folder but the header is located under /usr/ports/devel/rubygem-oj/work/oj-3.13.23/ext/oj so I suggest you modify your build script to compile oj-introspect with the --with-oj-include=/usr/ports/devel/rubygem-oj/work/oj-3.13.23/ext/oj flag.
I don't know how your build script looks like but the command would be something like this if it was just gem install;
gem install oj-introspect -- --with-oj-include=/usr/ports/devel/rubygem-oj/work/oj-3.13.23/ext/oj

Thanks a lot @meinac for your answer.
I think the first source of the problem is that the rubygem oj does not install the header files. We should be report this to oj package maintainer that the header files are installed and not only the oj.so.
I added now a work-around that all header files for oj are included and are available.
I was now able to continue further, but it stops now with the next error message:
cd /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/oj-introspect-0.7.1 ; /usr/bin/env RB_USER_INSTALL=yes LANG=en_US.UTF-8 LC_A
LL=en_US.UTF-8 /usr/local/bin/gem install -l --no-update-sources --install-dir /wrkdirs/usr/ports/devel/rubygem-oj
-introspect/work/stage/usr/local/lib/ruby/gems/3.0 --ignore-dependencies --bindir=/wrkdirs/usr/ports/devel/rubygem
-oj-introspect/work/stage/usr/local/bin --no-document oj-introspect-0.7.1.gem --
Building native extensions. This could take a while...
ERROR:  Error installing oj-introspect-0.7.1.gem:
        ERROR: Failed to build gem native extension.

    current directory: /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/lib/ruby/gems/3.0/gems/
oj-introspect-0.7.1/ext/oj-introspect
/usr/local/bin/ruby30 -I /usr/local/lib/ruby/site_ruby/3.0 extconf.rb
creating Makefile

current directory: /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/lib/ruby/gems/3.0/gems/oj-i
ntrospect-0.7.1/ext/oj-introspect
make DESTDIR\= sitearchdir\=./.gem.20221207-65739-ft6gkg sitelibdir\=./.gem.20221207-65739-ft6gkg clean

current directory: /wrkdirs/usr/ports/devel/rubygem-oj-introspect/work/stage/usr/local/lib/ruby/gems/3.0/gems/oj-i
ntrospect-0.7.1/ext/oj-introspect
make DESTDIR\= sitearchdir\=./.gem.20221207-65739-ft6gkg sitelibdir\=./.gem.20221207-65739-ft6gkg
compiling introspect.c
linking shared-object oj/introspect/introspect_ext.so
ld: error: unknown argument '-U'
ld: error: unknown argument '-U'
ld: error: unknown argument '-U'
ld: error: cannot open _oj_init_usual: No such file or directory
ld: error: cannot open _oj_parser_new: No such file or directory
ld: error: cannot open _oj_parser_set_option: No such file or directory
cc: error: linker command failed with exit code 1 (use -v to see invocation)
*** Error code 1

I think it maybe requires a specific compiler, I will check this.

Hi @mfechner 👋 Did you make any progress on this? Do you need anything else from my side? Otherwise, can we close this issue?

@meinac I'm sorry, but the error message while compiling this (see post https://github.com/meinac/oj-introspect/issues/4#issuecomment-1340624073) give you an idea, why it fails to build, please let me know.
Currently I do not have an idea, why this gem could not be compiled.
I ported 100reds of other modules but this is the first one I see this problem. So any help is welcome.

@mfechner This gem is following a unique approach to be able to use a header file provided by another gem so I guess that's why you are having problems. Are you able to provide a docker setup that I can pull and debug on my own?

Had the same issue on debian,  the header files were not packaged with the gem, modifying the build script to install the header files solved this issue on debian.

@mfechner This gem is following a unique approach to be able to use a header file provided by another gem so I guess that's why you are having problems. Are you able to provide a docker setup that I can pull and debug on my own?

I'm sry, but as I do not work with Linux I cannot provide with a docker setup.
But I was able to fix the problem by just removing the following lines:
--- ext/oj-introspect/extconf.rb.orig   2023-01-10 13:31:58 UTC
+++ ext/oj-introspect/extconf.rb
@@ -7,22 +7,6 @@ oj_version_file_path = Pathname.new(oj_version_file)

 OJ_HEADERS = oj_version_file_path.join('..', '..', '..', 'ext', 'oj').to_s

-cc_version = `#{RbConfig.expand("$(CC) --version".dup)}`
-if cc_version.match?(/clang/i)
-  # Ignore symbols loaded from Oj in case Ruby is compiled without
-  # "-Wl,-undefined,dynamic_lookup" (related to https://bugs.ruby-lang.org/issues/19005)
-  symfile = File.join(__dir__, 'oj.sym')
-  dynamic_symbols = File.readlines(symfile)
-  dynamic_symbols.each do |sym|
-    $DLDFLAGS << " -Wl,-U,#{sym.strip}"
-  end
-
-  # Needed for Ruby 3.2 ABI check: https://github.com/ruby/ruby/pull/5474
-  if RUBY_VERSION >= "3.2"
-    $LDFLAGS << " -Wl,-exported_symbol,_ruby_abi_version"
-  end
-end
-
 dir_config('oj', [OJ_HEADERS], [])

 create_makefile("oj/introspect/introspect_ext")


For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it's very slow. And the Utilization of GPU is low too. How to solve it? Thank you!

Originally posted by @Joke09 in https://github.com/apache/incubator-mxnet/issues/7899#issuecomment-444415247

Hi @Joke09 ,
When you put your image data into the NDArray, can you also try to copy it to GPU using as_in_context method before passing it to your model ?
http://mxnet.incubator.apache.org/test/api/python/ndarray.html#mxnet.ndarray.NDArray.as_in_context

Hi @Joke09 thanks for your question. I think you might miss this: https://mxnet.incubator.apache.org/api/python/image/image.html please take a look

Hi @Joke09 ,
When you put your image data into the NDArray, can you also try to copy it to GPU using as_in_context method before passing it to your model ?
http://mxnet.incubator.apache.org/test/api/python/ndarray.html#mxnet.ndarray.NDArray.as_in_context

Thank you! It works, But not much.
Before:
data = [mx.nd.array(im_arrays), mx.nd.array(im_infos)]
The shape of im_arrays is [8,3,896,1024]. It take 104ms.
After:
data = [mx.nd.array(im_arrays, ctx=mx.gpu(1)), mx.nd.array(im_infos)]
It take 68ms. I think it's not fast enough.

Hi @Joke09 thanks for your question. I think you might miss this: https://mxnet.incubator.apache.org/api/python/image/image.html please take a look

I think it works, when the image is in the disk. But image data is numpy array in this problem.
Sorry, I don't know how to use the image api for numpy array.

convert numpy.array to raw binary and then image.decode

convert numpy.array to raw binary and then image.decode

Thank you very much!
Can you give me some code example?
I use like this:
im = cv2.read("a.jpg")
im = cv2.resize(im, None, None, fx=im_scale, fy=im_scale)
...
image = mx.img.imdecode(im)

But it's fail.
File "test.py", line 12, in
image = mx.img.imdecode(im)
File "/data/usr/incubator-mxnet/python/mxnet/image/image.py", line 142, in imdecode
return _internal._cvimdecode(buf, *args, **kwargs)
File "", line 36, in _cvimdecode
File "/data/usr/incubator-mxnet/python/mxnet/_ctypes/ndarray.py", line 98, in _imperative_invoke
stype=out_stypes[0])
File "/data/usr/incubator-mxnet/python/mxnet/ndarray/sparse.py", line 1185, in _ndarray_cls
raise Exception("unknown storage type: %s"%stype)
Exception: unknown storage type: -1

I have this code:
data = [mx.nd.array(im_arrays), mx.nd.array(im_infos)]
data_shapes = [('data', im_arrays.shape), ('im_info', im_infos.shape)]
data_batch = mx.io.DataBatch(data=data, label=None, provide_data=data_shapes, provide_label=None)
self.module.forward(data_batch)

im_arrays is a numpy array and read from cv2. im_arrays.shape is [8,3,896,1024]
Can someone help me speed up this code with image api. I'm not familiar with Mxnet.

Hi @Joke09 instead of trying to use cv2 to read, you should use this:
https://mxnet.incubator.apache.org/api/python/image/image.html#mxnet.image.imread
Which convert an image into NDArray directly

Hi @Joke09 instead of trying to use cv2 to read, you should use this:
https://mxnet.incubator.apache.org/api/python/image/image.html#mxnet.image.imread
Which convert an image into NDArray directly

Hi @lanking520 , the client use cv2 to read and send the image data to the server. I can't control it.
I don't really want to write the image data to disk.
Another question:
the shape of data in mx.io.DataBatch must be [8,3,896,1024]. The im_infos.shape is (8, 3). They are both used for proposal in image detection. But image.imread and image.imdecode just get only one NDArray and the shape is [3,896,1024]. How to get NDArray with the shape is [8,3,896,1024].
I have try this:
im_arrays = im_arrays.swapaxes(2, 3).swapaxes(1, 2)
#data = [mx.nd.array(im_arrays), mx.nd.array(im_infos)]
img_encodes = []
for im_array in im_arrays:
  img_encodes.append(cv2.imencode('.bmp', im_array)[1])
batch_images = [mx.img.imdecode(img_encode) for img_encode in img_encodes]
# note: I don't know how to convert this list of NDArray to an NDArray of batch image data.
data_batch = mx.io.DataBatch(data=data, label=None, provide_data=data_shapes, provide_label=None)
self.module.forward(data_batch)

I  also try this:
im = cv2.imread("0.jpg")
start = time.time()
img_encode = cv2.imencode('.bmp', im)[1]
image = mx.img.imdecode(img_encode)
print(time.time() - start)
start = time.time()
image = mx.nd.array(im)
print(time.time() - start)

the fisrt print time is almost half of the second.  Maybe the mx.nd.array is too slow.

The first dim is the batch size which seemed you should set 8 images as a batch to feed them in. You can use nd.expand_dim to expand one more dimension on the image and use concat to concatenate images together.

@Joke09 - Are you still facing the performance issue?


UI design for calculator app

In this PR I have done the following

Create a new directory called components.

Inside components, i have created a new Calculator.js file.

In Calculator.js  I  created a React component that matches the  requirement

Implemented the design with (JSX and CSS)

Develop just the UI of the calculator; the math logic to make it work will be added in the next project.

Used functional components

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @misikirdelelew,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Some links (Install, Usage, etc) in the Table of Contents are broken and have no corresponding sections on README. Kindly add the sections and ensure they are properly linked in the ToC.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Experiment with multi-resolution pipelines

Describe the problem
We all know that the image resampling is one of the most challenging aspect for our models. Most of the time, they are trained with large datasets that all have a single resolution, or at the very least where pixel-size is very homogeneous. This creates a strong relationship between what the model learns and the "perceived" size of the axons within a patch. And it is bad.
What happens is that these dedicated models do a good job for the data from which were selected the training samples. For new images, they either generalize poorly, or we have to apply tedious pre-processing step and mess around with the resampling (what the zoom factor effectively does) to make them perform well.
The resampling step is "fixed": we resample every image to a common resolution. However, depending on the body part, the species and a whole bunch of other factors, the axon size can vary and this common resolution has no guarantee of being easier to segment for a given model.
Proposed solution
The resampling step should leverage a deep-learning method. The models need to learn themselves how to resample these images so they can segment appropriately. This is the first step towards a modality-agnostic model, because SEM and BF images have 10X the TEM resolution.

Possible solutions include:

train a model with this "fixed" resampling, but with a dataset that contains heterogeneous resolutions.
build a pipeline that will segment an image with multiple zoom factors using a given model, then input these multi-resolution segmentations along with the original image into a conditional GAN to clean up the mask
build a pipeline that will take a dataset with a wide range of pixel sizes (like in 1.) and use deep-learning methods to learn the resampling pre-processing operation to apply before the image is actually fed to the Unet model.

I think 3 is the most robust solution but we could try 1 first pretty easily to see what happens.

Note regarding this project: there is a similar problem with a "multi-contrast pipeline", which is that we need to encode the metadata as input data so that the model knows what resolution is currently being processed. Currently, this pixel size is used to obtain the scaling ratio required to resample the input image into dimensions that match the model's resolution.
This leads me to think that we need something like FiLM Layers to encode this metadata. Fortunately, this is already implemented in ivadomed.

@mathieuboudreau yes, this is absolutely relevant. If I was less busy this would be my priority - hopefully I can start tackling this early 2025.
Related to this issue: HyperNetworks https://arxiv.org/pdf/2407.03681


Remove unused variables

Description
Remove unused variables
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[ ] Extended the documentation

@RafaelJCamara Hello and thank you for your contribution!
As you have the time, I'd appreciate it if you could take at look at the review @KrylixZA provided and consider signing-off and committing them to your PR so I can accept and merge it.

@WhitWaldo @KrylixZA Ready to be reviewed again.
I can't replicate the merge conflict on my side.

@RafaelJCamara I'll take a stab at the merge this afternoon.

@WhitWaldo No need to worry about it, I was able to fix the conflicts. I was having a tempo. problem.
The merge was solved in my latest commit.
Thanks!

@RafaelJCamara I've been merging a pile of PRs into master the last couple of days and it looks like the CI build and tests are failing.
Would you mind jamming out another commit that builds and tests as you've got the availability?

@WhitWaldo Fixed the failing test. Build is now going strong on my side.

@holopin-bot @RafaelJCamara Thank you Rafael!


Cleanup and PgUpdate support

overall we should not duplicate the same class with a params and a param-less version. Param-less should be params command with empty params list.

Done  https://github.com/vietj/vertx-pg-client/issues/14 https://github.com/vietj/vertx-pg-client/issues/15 and https://github.com/vietj/vertx-pg-client/issues/18

The documentation is not covering the update case, it should be updated (after query and before batch).


What determines XDebug's path?

I notices my installation couldn't find xdebug, and seeing at #86 it's happened before, but I seem to have an older version, is this based on what's on my computer?

@emeaguiar the path is automatically chosen by pecl when xdebug is installed. Afaik, the timestamp identifies which PHP build is being used. We can't change this on our images, but I've created a workaround so we don't need to keep updating that reference to work with it.

Closing. Fixed in https://github.com/10up/wp-local-docker-images/pull/9 and #111


tartiflette.io: Tutorial and API documentation:

This pull-request contains the documentation which will be displayed on tartiflette.io.
The integrate is described in the open-source project tartiflette.io

LGTM to me again :).


Extension doesn't respond on VS Code launch in macOS

Issue Type: Bug
Extension Name: html-css-class-completion
Extension Version: 1.19.0
OS Version: Darwin x64 18.6.0
VSCode version: 1.34.0

Zignd.html-css-class-completion-unresponsive.cpuprofile.txt
{
	"messages": [],
	"activationTimes": {
		"startup": true,
		"codeLoadingTime": 7846,
		"activateCallTime": 3,
		"activateResolvedTime": 22,
		"activationEvent": "*"
	},
	"runtimeErrors": []
}

Thanks for attaching the file. I'll be closing the issue and mentioning it on #155. Just so we can centralize the discussion there.


Canvas not defined or null reference IE issue

I have nearly 20k data to be drawn, so I am looping through each data to frame the icon feature(below is the code). In chrome it seem to be working fine but in IE, I get unable to get property of canvas in line iconStyle=new ol.style.Style Attached the screenshot as well.  Can I get help on this?
        var locations = data;
        var iconStyle, markerSet = [];

        for (var i = 0; i< locations.length; i++) {

            var iconFeature = new ol.Feature({
                geometry: new ol.geom.Point(ol.proj.transform([Number(locations[i].longitude), Number(locations[i].latitude)], 'EPSG:4326', 'EPSG:3857')),
                name: locations[i].city,
                id: guid()
            });

                **iconStyle = new ol.style.Style({**
                    image: new ol.style.Circle({
                        radius: 12,
                        fill: new ol.style.Fill({
                            color: [0, 82, 136, 1]
                        })
                    }),
                    text: new ol.style.Text({
                        font: '15px Roboto-Regular',
                        text: (i + 1) + '',
                        fill: new ol.style.Fill({
                            color: '#000'
                        })
                    })
                });

             iconFeature.set('type', options.locationsType);
             iconFeature.set('cords', [locations[i].latitude, locations[i].longitude]);
              iconFeature.set('sites', locations[i].sites);

            iconFeature.set('category', locations[i].category);
            iconFeature.set('params', locations[i]);
            iconFeature.setStyle(iconStyle);
            markerSet.push(iconFeature);
        }

You are probably using IE in some kind of compatibility mode. Make sure you have <!DOCTYPE html> as first line in your HTML, and also add <meta http-equiv="X-UA-Compatible" content="IE=Edge"> to your HTML document's <head>.

Its there on my header already. Please note if i use some 3000 data, website is rendering properly but when the data counts increases more than 5k then this kind of error comes in. Also some time when the feature loops runs I get  OUT OF MEMORY exception.


v.4.2.3 if no

Describe the bug
if no  in the file svelte.
System Info
System:
    OS: Linux 5.13 Linux Mint 20.2 (Uma)
    CPU: (4) x64 Intel(R) Core(TM) i5-2400 CPU @ 3.10GHz
    Memory: 13.60 GB / 23.37 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 16.11.0 - ~/.nvm/versions/node/v16.11.0/bin/node
    npm: 8.2.0 - ~/.nvm/versions/node/v16.11.0/bin/npm
  IDEs:
    Nano: 4.8 - /usr/bin/nano
    VSCode: 1.62.3 - /usr/bin/code
  Browsers:
    Chrome: 96.0.4664.93
    Chromium: 96.0.4664.45
    Firefox: 94.0
  npmPackages:
    svelte: ^3.44.2 => 3.44.2
    svelte-windicss-preprocess: 4.2.3 => 4.2.3
    windicss: ^3.2.1 => 3.2.1

Used Package Manager
npm

Thanks.. Known.
There are no major new features activated by default, so you can force 4.1 for now.

Ok... I will... Thank you.

Hi, had the issue with dark mode with windi + svelte, after updating svelte-windicss-preprocess to 4.2.3 my issue with dark mode overwriting seems solved.

Hi, had the issue with dark mode with windi + svelte, after updating svelte-windicss-preprocess to 4.2.3 my issue with dark mode overwriting seems solved.

Glad you have your expected behavior. However 4.2 will get some more updates to handle regression. If you have a specific issue please open an additional issue.

Just, one detail:
If style is like that, inline classes don't work:
<style></style>

If style is like that, inline classes work fine:
<style>
</style>


Added 'operator' to the API

Enabled searching via the api to include the 'operator' key for performing searches using the elasticsearch 'and' operator.

Thanks @michaelfillier! This looks good
I'm going to add this the the v2 API which will be introduced in 1.4 (release date around Janurary).

Great!

Merged into #1974 Thanks!


docs: explicitly describe switching btw wdi5- and WebdriverIO-api

and make @hmanchev a reviewer

on me


Problem running on mcr.microsoft.com/dotnet/aspnet:8.0 docker image

Versions: .net 8.0, Pkcs11Interop 5.1.2, docker 27.1.1, Utimaco Security Server <IP_ADDRESS>
I prepared a repo: Pkcs11Test, which is a default .net 4.8 MVC application, with slight modifications:

CustomDllImportResolver in Program.cs
custom HsmAdapter.cs class
on both GET methods in HomeController a key pair is generated
changes in Dockerfile from line 5 to line 16, where I set the environment variables and copy *.so and *.cfg files

The problem: when using the default Utimaco libcs_pkcs11_R2.so library, the application starts, home page is displayed, key pair is generated, but then the application crashes. I'm not able to find any logs, not in Visual Studio, docker, pkcs11 log in docker container... so I suspect something like segmentation fault?
Another observation - if I use the pkcss11 library from https://github.com/Pkcs11Interop/pkcs11-logger (use line 15 in HsmAdapter.cs instead of line 14), then everything works 🤨
Any ideas?

If I uncomment the _pkcs11?.Dispose() then it works.
And even weirder - if I put a breakpoint on the _pkcs11?.Dispose() and wait a couple of seconds, then it works. But if I continue as soon as breakpoint is hit, then the application crashes again.
PS - are there any best practices regarding pkcs11 and session objects lifecycle on web pages? I guess each request generates new pkcs11 object, opens session and disposes everything once it is done.

PS - are there any best practices regarding pkcs11 and session objects lifecycle on web pages? I guess each request generates new pkcs11 object, opens session and disposes everything once it is done.

Found your answer here: https://github.com/Pkcs11Interop/Pkcs11Interop/issues/10#issuecomment-38201532
Recap:

create a Singleton which gets instantiated during app startup, which

loads the pkcs11 library
opens a master session
logins
keeps master session open

when a new thread needs to perform a crypto operation it retrieves a new session from the singleton class, and closes it at the end (but it shouldn't log out)


Note that Trigger delivers all events if no filter defined

Changes

:broom: Add rule that Trigger to deliver all events if no filter defined
See the default filter value in https://github.com/knative/specs/blob/main/specs/eventing/spec.md#spec
Impl: https://github.com/knative/eventing/blob/d8265bb5b08f1bebe61fdecd166f43962b01a0e2/pkg/apis/eventing/v1/trigger_defaults.go#L37

/kind enhancement

Fixes #

Release Note

Docs

/approve


Use JDBC execute rather than executeQuery for updates in HiveWarehouseSession

merged to branch-2.3-3.0


Update to more recent version of tpm2-tss

The installer pull in tss v2.0, which is old and not supported.
https://github.com/keylime/keylime/blob/master/installer.sh#L17
Please update this to a more recent version.

I believe this has been fixed by @THS-on


Missing openj9 build for mac

Dear devs,
the macos openj9 build is available for macos as a download.
At least for JDK11.
I've seen closed source j9-jdk8 builds bundled with IBM Notes.
The matrix in the Readme.md file is missing macosx-openj9 alltogether.
Could you please look into, why:

… why openj9 builds for jdk8 (and possibly jdk9/10) are missing?
… why macosx-openj9 is missing in the build matrix, although it was build?

Thanks a lot!

I'm working on JDK8 (spent most of yesterday on it as it happens!) but there are some issues with the build process as they are introducing extra prereqs. There are no plans for any OpenJ9 builds with JDK9/JDK10
The current builds can be downloaded from https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-mac-x64-openj9/ (They're not on the lightlies page yet) but as mentioned you might hit dependency issues (at a minimum it'll need libstdc++ which you can probably get with brew install gcc49. If you try that install, let me know how it goes as I haven't checked that in a clean environment yet :-)

My Notes install on Mac doesn't appear to utilise any j9-jdk8 build ...
  501  1297     1   0 10:11am ??         3:02.45 /Applications/IBM Notes.app/Contents/MacOS/rcp/eclipse/plugins/com.ibm.rcp.base_<IP_ADDRESS>50831-1518/macosx/x86/eclipse --launcher.suppressErrors -nosplash -nl en_US -dir ltr -name IBM Notes -personality com.ibm.rcp.platform.personality -product com.ibm.rcp.personality.framework.RCPProduct:com.ibm.notes.branding.notes -data<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications -configuration<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications/.config -plugincustomization /Applications/IBM Notes.app/Contents/MacOS/rcp/plugin_customization.ini -vm /Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home/bin/java -startup /Applications/IBM Notes.app/Contents/MacOS/rcp/eclipse/plugins/com.ibm.rcp.base_<IP_ADDRESS>50831-1518/launcher.jar -vmargs -Xdock:icon=/Applications/IBM Notes.app/Contents/Resources/appicon.icns -Djava.ext.dirs=/Applications/IBM Notes.app/Contents/MacOS/ndext:/Applications/IBM Notes.app/Contents/MacOS/jvm/lib/ext:/Library/Java/Extensions:/System/Library/Java/Extensions:/Library/Internet Plug-Ins/JavaAppletPlugin.plugin//Contents/Home/lib/ext -Dorg.eclipse.swt.internal.carbon.smallFonts -Xverify:none -XX:MaxPermSize=128M -Xmx256m -Xms48m -vm /Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home/bin/java -Djava.security.policy=/Applications/IBM Notes.app/Contents/MacOS/java.policy -Dorg.eclipse.swt.fixCitrix=false<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications/logs/java_error%p.log -XX:+HeapDumpOnOutOfMemoryError<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications/logs -Xverify:none -Xmx512m -Dosgi.framework.extensions=com.ibm.rcp.core.logger.frameworkhook -Drcp.home=/Applications/IBM Notes.app/Contents/MacOS<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications -Dosgi.splashPath=platform:/base/../../shared/eclipse/plugins/com.ibm.notes.branding,platform:/base/../../shared/eclipse/plugins/com.ibm.notes.branding.nl1,platform:/base/../../shared/eclipse/plugins/com.ibm.notes.branding.nl2,platform:/base/../../shared/eclipse/plugins/com.ibm.notes.branding.nl3 -Dcom.ibm.rcp.install.id=1706182214 -Drcp.install.config=multiuser -Declipse.registry.nulltoken=true -Dcom.ibm.pvc.webcontainer.useLocalAddress=true -Dautopd.logfile.generations=3 -Dorg.apache.xerces.xni.parser.XMLParserConfiguration=org.apache.xerces.parsers.XIncludeAwareParserConfiguration -Dcom.ibm.pvc.webcontainer.http.address=localhost -Dosgi.nl.user=true<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications/autopd -Dorg.eclipse.swt.browser.XULRunnerPath=/Applications/IBM Notes.app/Contents/MacOS/rcp/eclipse/plugins/com.ibm.rcp.xulrunner.runtime.macosx.x86_64_<IP_ADDRESS>50831-1518/xulrunner -Djava.util.logging.config.class=com.ibm.rcp.core.internal.logger.boot.LoggerConfig -Dcom.ibm.pvc.webcontainer.port=0,59449 -Dcom.ibm.pvc.webcontainer.vhost.configfile=/Applications/IBM Notes.app/Contents/MacOS/shared/eclipse/plugins/com.ibm.collaboration.realtime.webapi_<IP_ADDRESS>50831-0324/virtualhost.properties<EMAIL_ADDRESS>Support/IBM Notes Data/Expeditor/Applications/logs/derby.log -Djava.security.properties=file:/Applications/IBM Notes.app/Contents/MacOS/rcp/eclipse/plugins/com.ibm.rcp.base_<IP_ADDRESS>50831-1518/rcp.security.properties -Dcom.lotus.sametime.skipRandomGeneratorSeed=true -Djava.protocol.handler.pkgs=com.ibm.net.ssl.www.protocol -Dosgi.hook.configurators.exclude=org.eclipse.core.runtime.internal.adaptor.EclipseLogHook -Drcp.osgi.install.area=/Applications/IBM Notes.app/Contents/MacOS/rcp/eclipse -Xbootclasspath/a:/Applications/IBM Notes.app/Contents/MacOS/shared/eclipse/plugins/com.ibm.collaboration.realtime.stjavatk_<IP_ADDRESS>50831-0324/sslite140-v3.16.jar:/Applications/IBM Notes.app/Contents/MacOS/rcp/eclipse/plugins/com.ibm.rcp.base_<IP_ADDRESS>50831-1518/rcpbootcp.jar -Dfile.encoding=UTF-8 -XstartOnFirstThread

bens-mbp:bin<EMAIL_ADDRESS>"/Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home/bin/java" -version
java version "1.8.0_151"
Java(TM) SE Runtime Environment (build 1.8.0_151-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.151-b12, mixed mode)

@bmhm Ref your confused reaction on Ben's post - he was asking if you had the java -version output from the JVM you've seen in Notes, since we're not aware that there has been an IBM J9 VM on macos.

@sxa555 I was told it was Notes, but now I was told it might have been another IBM product which included a OpenJ9 JRE (no JDK). Sorry for the false information!

@bmhm No worries. To my knowledge there has never been a J9 on macos until this one :-)

So this can be closed?

How do you want to track Java 8 J9 issues?
@sxa555 I just got this when doing a 'maven clean install' within IntelliJ idea on macOS with the J9 build. The regular adoptopenjdk openjdk hotspot builds (18x) are fine.
/Users/jonesn/java/jdk8u192-b12/Contents/Home/bin/java -Dmaven.multiModuleProjectDirectory=/Users/jonesn/IdeaProjects/egeria -Dmaven.home=/Users/jonesn/IdeaProjects/maven/apache-maven-3.5.4 -Dclassworlds.conf=/Users/jonesn/IdeaProjects/maven/apache-maven-3.5.4/bin/m2.conf "-javaagent:/Users/jonesn/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/183.3795.13/IntelliJ IDEA 2018.3 EAP.app/Contents/lib/idea_rt.jar=54304:/Users/jonesn/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/183.3795.13/IntelliJ IDEA 2018.3 EAP.app/Contents/bin" -Dfile.encoding=UTF-8 -classpath /Users/jonesn/IdeaProjects/maven/apache-maven-3.5.4/boot/plexus-classworlds-2.5.2.jar org.codehaus.classworlds.Launcher -Didea.version=2018.3 clean install
Assertion failed: (!"JVM_SocketShutdown() stubbed!"), function JVM_SocketShutdown, file j7vmi.c, line 2375.
JVMDUMP039I Processing dump event "abort", detail "" at 2018/10/24 09:57:11 - please wait.
JVMDUMP032I JVM requested System dump using '/Users/jonesn/java/jdk8u192-b12/Contents/Home/bin/core.20181024.095711.12684.0001.dmp' in response to an event
[INFO] Scanning for projects...

That looks like a VM crash to me and should be reported at the Eclipse OpenJ9 upstream project.  Reporting issues here initially is fine (in case it's an Adopt Binary issue).

Any thoughts @sxa555 - I know we've had some issues with libraries etc so I'm a little doubtful if it is just an upstream issue. Will try on windows shortly as a comparison

@planetf1 I'd definitely raise it at https://github.com/eclipse/openj9 ... Bear in mind that JDK8/J9/macos has so far undergone minimal testing, so this information will be useful to the OpenJ9 team whether it's a dependency issue or an underlying problem. Please link back to this issue once you raise it :-)

Reporting issues here initially is fine (in case it's an Adopt Binary issue).

Indeed - plus it gives us awareness of it in case someone else reports it to us!

I went one step further in the build - this time to run some unit tests only to hit another core dump
[INFO] Surefire report directory: /Users/jonesn/IdeaProjects/egeria/open-metadata-implementation/frameworks/open-connector-framework/target/surefire-reports
JVMDUMP010I System dump written to /Users/jonesn/java/jdk8u192-b12/Contents/Home/bin/core.20181025.082618.45081.0001.dmp
JVMDUMP032I JVM requested Java dump using '/Users/jonesn/java/jdk8u192-b12/Contents/Home/bin/javacore.20181025.082618.45081.0002.txt' in response to an event
This DOES NOT occur on windows (the previous one kinda-did) so it could be a mac or build regression?
javacore.20181025.082618.45081.0002.txt
I've not opened a J9 issue for this one -- maybe worth seeing how the first is received.
How are tests running with the build? I'm thinking it's worth waiting until the build is deemed 'good'? THough if the feedback is helpful I can note other issues as I work on my project

We run a full pipeline of tests on the builds, might be interesting to add your ones to that?  Is it an OSS project?

looks like this was closed upstream at the j9 project. Closing


CQ-4290733 : [Visual Requirements-AEM Dynamic Media-Image Set Editor]…

…: Luminosity ratio fails for multiple controls, does not meets minimum luminosity ratio 4.5:1.

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

@shingari007 You need to sign the Adobe CLA.

@shingari007 You need to sign the Adobe CLA.

done


Transaction support OR custom callbacks on the query stack

We've been doing our own implementation of transaction support wrapped around node-mysql. It's quite a tough thing to get right, and easy to use. It is natural behavior in node-mysql that queries are queued. The problem is that queuing a COMMIT onto a list of queries, means that if there is an error with a query inside the transaction, the successful part of the transition will still be committed. The decision to issue a ROLLBACK or a COMMIT can only be made after all queries that needed executing have finished.
For this reason, we would love to either have some awesome native transaction support that deals with these issues, OR have the following implemented:
nodeMysql.queueCallback(function myCallback() {});
Which would append the callback to the end of the query stack.
If we had this, we could queue a callback that when called can decide whether to COMMIT or ROLLBACK based on any previously registered error state.

Check out this... https://github.com/bminer/node-mysql-queues

I have added transaction support to node-mysql.  Check out this project:
https://github.com/bminer/node-mysql-queues


Bootstrapping fails with clang 15.0.7 due to warning "array-parameter" in BoringSSL

I'm trying to bootstrap-build justbuild on ArchLinux using CLang 15.0.7. Unfortunately, the compilation fails due to a warning in BoringSSL, which is built with -Werror (even when configuring "COMPILER_FAMILY": "unknown"):
Running ['/usr/bin/clang', '-O2', '-DNDEBUG', '-std=gnu17', '-Wa,--noexecstack', '-D_XOPEN_SOURCE=700', '-Wall', '-Werror', '-Wformat=2', '-Wsign-compare', '-Wmissing-field-initializers', '-Wwrite-strings', '-Wshadow', '-fno-common', '-I', 'work', '-isystem', 'include', '-c', 'work/src/crypto/fipsmodule/bcm.c', '-o', 'work/src/crypto/fipsmodule/bcm.o'] with env {'PATH': '/bin:/sbin:/usr/bin:/usr/sbin'} for action 'dee9161ccc9d5793778e365f064834ab836dcc47'
In file included from work/src/crypto/fipsmodule/bcm.c:38:
work/src/crypto/fipsmodule/bn/asm/x86_64-gcc.c:427:51: error: argument 'a' of type 'const uint64_t[8]' (aka 'const unsigned long[8]') with mismatched bound [-Werror,-Warray-parameter]
void bn_sqr_comba8(BN_ULONG r[16], const BN_ULONG a[8]) {
                                                  ^
work/src/crypto/fipsmodule/bn/asm/../internal.h:292:51: note: previously declared as 'const uint64_t[4]' (aka 'const unsigned long[4]') here
void bn_sqr_comba8(BN_ULONG r[16], const BN_ULONG a[4]);
                                                  ^

I'm using the following command line:
env JUST_BUILD_CONF='{"COMPILER_FAMILY": "unknown", "CC": "/usr/bin/clang", "CXX": "/usr/bin/clang++", "AR": "/usr/bin/ar"}' python3 ./bin/bootstrap.py .

Output of clang --version:
clang version 15.0.7
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

As a workaround, I have removed the -Werror from the posix_copts array in etc/defaults/CC/TARGETS.boringssl.

Thanks @mhthies, your workaround is actually a good solution. We will drop -Werror for externals, as it is not our job to lint external projects at all.
Would it be fine for you if we are putting you as a co-author for the upcoming change?

Would it be fine for you if we are putting you as a co-author for the upcoming change?

Of course, if you consider this to be justified. :)

Sure, we really appreciate your efforts. :)
The change is in with commit https://github.com/just-buildsystem/justbuild/commit/31cb23a32d0665777a82799f3f476266882516cb. Closing this issue.


Potential ways forward regarding a smoother experience in Jupyter

At the moment, using Vizro in a Jupyter notebook may feel annoying when re-evaluating cells, because

adding a new model with the same ID causes the an error.
in addition, there is a further issue, that currently every instantiation of a model without ID  adds it newly to the model_manager (MM), thus creating a situation where models are in the MM that are not used within the dashboard

Solving this may entail multiple levels of investigation/considerations. See also https://github.com/mckinsey/vizro/pull/56 for some notes and code around this issue.
Short term solution
In the short term, https://github.com/mckinsey/vizro/pull/59 ensures that users get a more comprehensive error message when working in Jupyter.
Note that this does not catch all cases, as a cell could also not contain a model with manual ID, thus never triggering an error that catches the additional models added to the MM.
Medium term solution

remove the restrictions to add duplicate ID's but at the same time improve all reference to the MM, especially _items_with_type
_items_with_type in particular should not be a wild card search to any type, but if at all search for a Dashboard, all subsequent scans should be hierarchical and on a per need basis
likely further investigation about consequences needed
note also: removing the duplicate ID restriction can also cause confusion when users do NOT want to update a model, but by mistake copied one

Long term solution
Idea 1

go back to hierarchical registering approach (ie not register upon instantiation)

Reasoning from @antonymilne as to why this is NOT a no-brainer:

I think this is one of the disadvantages of the recursive approach to registering models in the model manager. Let’s say you go for the recursive approach like this:

call dashboard.register that recursively registers all models
pre_build uses models existing in model manager
pre_build can itself create models
if we want those models to also be registered in the model manager then we need to somehow register them e.g. by manually calling selector.register or by re-running dashboard.register after the pre-build

This step 4 with the extra manual registration stage is a bit ugly, annoying and easy to forget, whereas if it’s done automatically upon model instantiation it’s not required at all.
tbh the current solution with pre_build creating models is also not very satisfactory, as described in the comment:

  # Note that a pre_build method can itself add a model (e.g. an Action) to the model manager, and so we need to
  # iterate through set(model_manager) rather than model_manager itself or we loop through something that
  # changes size.
  # Any models that are created during the pre-build process *will not* themselves have pre_build run on them.
  # In future may add a second pre_build loop after the first one.

Overall if we do want to change how components are registered in the model manager then we’d need to think about it in combination with pre_build. Hopefully there’s some better overall solution, but I don’t think we can change one without thinking about the other at the same time. I’d love if we could come up with an improvement here, but I don’t think there’s going to be an easy solution.

Idea 2

introduce some form of context_manager that associates every model with a dashboard

Idea 3

introduce a flag when overriding is allowed

I'll need to spend some time properly thinking this all through, since it's a tricky topic and not something we should rush. But just to note another possible idea while I remember: maybe we need another method on VizroBaseModel which enables you to instantiate and register a model in one go. This would avoid you needing to remember to manually call register after you've created a model in pre-build, but I'm not sure exactly what that register method would do.

Another possible solution might be to clear the managers on shutdown of the flask process. I did this in an old solution where we used to copy assets to a temporary directory. It's not super reliable (e.g. from memory there's some complexities with multiple threads), but maybe in the context of Jupyter where you're unlikely to be doing that it would work well.

Another idea: a global dashboard object and navigating through that hierarchy (as in the medium term solution above) might actually be a good long term solution. This would probably avoid difficulties with needing to recursively register.

Another point against hierarchical registration or at least in favour of keeping _items_with_type: when we implement components nested in tabs or containers, it gets harder to collect e.g. all components on the page.

Closing this just so we don't have the comments spread over too many different places. #120 is a good fix for Jupyter for now, and the real issue here is much deeper than just Jupyter - will continue the conversation in #109 and #111 anyway.


Make interaction recording asynchronous in bus

I'm not sure if we should make it asynchronous on the bus side. It feels weird to have an API that's not blocking and awaits a successful insert of whatever you've asked to insert. Did you consider calling RecordInteractions in a goroutine on the worker side?

By the way I checked and this does fix uploads for me. Very rudimentary testing:
 2023/01/25 18:48:52 upload succeeded, 142.4 MiB in 46.928058375s
 2023/01/25 18:49:39 upload succeeded, 142.4 MiB in 46.928058375s

Averaging only 60mbps though, peaked at 700. I've seen it averaging 500 not too long ago so we just have to hook it up to Jaeger and see what's up. I'll try and make time for that tomorrow. Definitely good to merge a fix like this though.


Confirmation popup windows for all the processing steps?

See discusion here https://github.com/biocore/qiita/pull/732#discussion_r22235787

If we can't have a review page or something similar, I think it's better to have the confirmation, especially with things that are compute-heavy.

We agreed that only steps that delete files should have these confirmations.

Can this be closed? I think this was addressed in https://github.com/biocore/qiita/pull/863, right? If not please describe what's missing.

This was in reference to the popups that happen on virtually every action.
Like, are you sure you want to process xyz, etc
On Feb 19, 2015 2:41 AM, "Antonio Gonzalez"<EMAIL_ADDRESS>wrote:

Can this be closed? I think this was addressed in #863
https://github.com/biocore/qiita/pull/863, right? If not please
describe what's missing.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/754#issuecomment-75022495.

Yup, I think they have been erased at least on pre and processing. Not sure
if there are other places that this is still happening that shouldn't.

Okay, didn't realize. Safe to close, can always reopen later if necessary
On Thu, Feb 19, 2015 at 8:35 AM, Antonio Gonzalez<EMAIL_ADDRESS>wrote:

Yup, I think they have been erased at least on pre and processing. Not sure
if there are other places that this is still happening that shouldn't.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/754#issuecomment-75072457.

I'm gonna wait for @josenavas to close as he open it.

I think it can be closed. We can re-open if there is any missing bit.

I believe this still happens when you add a new raw data to a study, unless
i was working with an old version of the code base
On Feb 19, 2015 9:07 AM, "josenavas"<EMAIL_ADDRESS>wrote:

Closed #754 https://github.com/biocore/qiita/issues/754.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/754#event-238473742.

 Reopened, any other place?

That's the only place I noticed while using the site, but I just grepped all HTML files in the codebase for the word "sure" and came up with these other instances:
./qiita_pet/templates/edit_study.html:218:  if (confirm("Are you sure you want to go to the study description page? None of the changes will be stored")){
./qiita_pet/templates/search_studies.html:105:    {# make sure processed data exists for the data_type before displaying#}
./qiita_pet/templates/study_description.html:62:  if (confirm("Are you sure you want to (re)process the sample template?")) {
./qiita_pet/templates/study_description.html:85:    if (confirm("Are you sure you want to create a new raw data?")) {
./qiita_pet/templates/study_description.html:118:  if (confirm("Are you sure you want to add a new prep template?")) {
./qiita_pet/templates/study_description.html:156:  if (confirm('Are you sure you want to delete raw data: ' + raw_data_filetype + ' (ID: ' + raw_data_id + ')?')) {
./qiita_pet/templates/study_description.html:174:  if (confirm('Are you sure you want to delete prep template: ' + data_type + ' (ID: ' + prep_template_id + ')?')) {
./qiita_pet/templates/study_description.html:194:  if (confirm("Are you sure you want to make this study public?")) {
./qiita_pet/templates/study_description.html:208:  if (confirm("Are you sure you want to approve this study?")) {
./qiita_pet/templates/study_description.html:222:  if (confirm("Are you sure you want to ask for approval? IMPORTANT: This will not allow you to add any new raw data or modifications to this study")) {
./qiita_pet/templates/study_description.html:236:  if (confirm("Are you sure you want to revert to sandbox status?")) {
./qiita_pet/templates/study_description.html:250:  if (confirm("Are you sure you want to update the investigation type?")) {
./qiita_pet/templates/study_description.html:368:  if (confirm("Are you sure you want to unlink all files? They will be removed from the system")){
./qiita_pet/templates/study_description.html:386:  if (confirm("Are you sure you want to submit a preprocessing job?")){
./qiita_pet/templates/study_description.html:393:  if (confirm("Are you sure you want to submit a processing job?")){
./qiita_pet/templates/upload.html:56:      <button type="button" class="btn btn-danger" onclick="if(confirm('Are you sure you want to delete the selected files? You can not undo this action')) { $('#delete-files-form').submit() }">

grep for confirm( to get all of them, but those all look like sane places to do confirmation checks, save a couple. The job submissions at the bottom could be removed, the create raw data as said abive, and possibly the add new prep template.


training stage parameters are getting saved

long term - the stage parameters should not be changed when the individual sessions parameters are changed
short term - I can set up an auto-job to just copy the training stage parameters from the repo periodically, which will wipe any changes from the individual session parameters

@hanhou @XX-Yin I can confirm the TrainingStagePar.json files ARE getting modified

I updated the computers to automatically update the Training stage parameters json file every 30minutes from the version on the repo. This is a short term fix, we need a long term fix

Long term fix would be using AutoTrain instead of using locally saved parameters.


for big endian types use fallback

NEON intrinsics are not supported on big endian targets. Change to use fall back types for big endian aarch64.
tested on aarch64_be-unknown-linux-gnu_ilp32 and aarch64_be-unknown-linux-gnu

here is the issue https://github.com/rust-lang/stdarch/issues/1484

Please fix this by fixing the intrinsics upstream. Fixing this in every single crate is not less effort than fixing it in stdarch: https://github.com/rust-lang/stdarch/issues/1484#issuecomment-1942934003


parse error near `&&'

Issue
Got error message when login to console
/home/vagrant/.oh-my-zsh/custom/themes/spaceship.zsh-theme:329: parse error near `&&'

Screenshot

Environment
Operating system: Linux Ubuntu 14.04 VM
Terminal emulator: Xshell 5
ZSH version: 5.05
Relevant .zshrc
ZSH_THEME="spaceship"

idem

Same issue on Bash for Windows (Ubuntu 14.04.5)
ZSH 5.0.5

@fascox zsh 5.0.2 isn't supported (see the badges at the top of the README).

@ilyador I don't see that issue on zsh 5.0.5, but technically 5.0.5 shouldn't be supported either (see issue #190). You're probably encountering a different issue.

I upgraded Ubuntu from 14 to 16 (on WSL) and thus upgraded ZSH to 5.1 so now the issue is gone

I've only used spaceship a couple times on 5.0.5. It's definitely possible this is an edge case I didn't hit, which may indicate another reason this is incompatible with 5.0.5.

Same issue on RHEL 6.7 (Santiago)
ZSH 5.4.2 (x86_64-unknown-linux-gnu)

Can anyone share the relevant log ?

Thanks for that, I found the issue:
+/home/thompsons/.oh-my-zsh/oh-my-zsh.sh:9> fpath=( /home/thompsons/.oh-my-zsh/functions /home/thompsons/.oh-my-zsh/completions /home/thompsons/.oh-my-zsh/plugins/git /home/thompsons/.oh-my-zsh/functions /home/thompsons/.oh-my-zsh/completions /usr/share/zsh/site-functions /usr/share/zsh/4.3.11/functions )

My zsh installed in a different location, and is 5.4.2, but the zsh that's called by oh-my-zsh is the default that came with my distro, which is 4.3.11 (which obviously isn't supported).

Occurring to me as well in Trusty, zsh 5.0.5

Same issue here Yosemite 10.10.1, zsh 5.0.5 or zsh 5.4.2 occurs the issue. Another mac on sierra 10.12.3 same zsh configs works fine. Any solutions?

@cheesepaulo what does echo $ZSH_VERSION tell you?

On 5.0.5, I see problems with these lines:
https://github.com/denysdovhan/spaceship-zsh-theme/blob/68068b61e5e8852dcdfff1f19a796758f86d2cde/spaceship.zsh#L337
and
https://github.com/denysdovhan/spaceship-zsh-theme/blob/68068b61e5e8852dcdfff1f19a796758f86d2cde/spaceship.zsh#L803
Both of these lines can be resolved by replacing [[ $foo ]] with [[ -n $foo ]]. I'm pretty sure this is a dupe of zsh-users/zsh-syntax-highlighting#225 (it uses the solution I've described). #121 had the same root cause too.
It sounds like this only appears for old versions of zsh (the borderline is somewhere between 5.0.5 and 5.1)

We can add the fix I mentioned, but spaceship still won't work properly on 5.0.5 (until #190 is resolved)

Thanks @nfischer I solve this upgrading my zsh.
If you have Homebrew installed, you can do this.
check the zsh info
brew info zsh
install zsh
brew install --without-etcdir zsh
add shell path
sudo vim /etc/shells
add the following line into the very end of the file(/etc/shells)
/usr/local/bin/zsh
change default shell
chsh -s /usr/local/bin/zsh
Mac has a native zsh 5.0.5 and brew install 5.4.2 BUT not always change the 5.4.2 as a default zsh. You need to override this behavior to works fine!

I'm having the same issue (though line 337) with up to date zsh on Mac OS X 10.10.5
~/.oh-my-zsh/custom/themes/spaceship.zsh-theme:337: parse error near `&&'

$ zsh --version
zsh 5.4.2 (x86_64-apple-darwin14.5.0)

It seems there is no official backport for ZSH newer version for Ubuntu 14.04.


Feature/embed discourse

Adds discourse comments to the same spot on the tool/workflow page as discuss comments were

Note that dockstore.model.ts has been updated.

Codecov Report

Merging #113 into develop will increase coverage by 0.88%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #113      +/-   ##
===========================================
+ Coverage    65.15%   66.03%   +0.88%
===========================================
  Files           93       93
  Lines         2514     2591      +77
  Branches       319      331      +12
===========================================
+ Hits          1638     1711      +73
- Misses         705      713       +8
+ Partials       171      167       -4

Impacted Files
Coverage Δ

src/app/shared/dockstore.model.ts
100% <100%> (ø)
:arrow_up:

src/app/shared/versions.ts
69.04% <0%> (+1.04%)
:arrow_up:

src/app/workflow/versions/versions.component.ts
91.83% <0%> (+2.64%)
:arrow_up:

...pp/workflow/version-modal/version-modal.service.ts
80.26% <0%> (+4.4%)
:arrow_up:

src/app/shared/container.service.ts
96.61% <0%> (+6.36%)
:arrow_up:

...rganization/refresh-tool-organization.component.ts
61.53% <0%> (+9.15%)
:arrow_up:

...ization/refresh-workflow-organization.component.ts
68% <0%> (+10.1%)
:arrow_up:

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 d6f196f...212de4d. Read the comment docs.


Distinct collaborators receiving the same task

Describe the bug
When two collaborators log into the same project they receive the same task
Backend indicates an error:
[2021-04-12 22:11:03,640] [tasks.models::set_lock::103] [ERROR] Current number of locks for task 2 is 1, but overlap=1: that's a bug because this task should not be taken in a label stream (task should be locked)

In the user screen that logged last, sometimes, the "Submit" button disappear.
To Reproduce
Steps to reproduce the behavior:

Log 2 users with different accounts
Click on "..." button to label the project

Expected behavior
Users receiving different tasks each.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Ubuntu 20.04 focal
Label Studio Version 1.0.1

@rodrigocaus

Click on "..." button to label the project

Do you mean blue Label button on Data Manager ?

I have the same problem

@makseq
The blue Label button also applies. But what I meant was this Label button:

I think they do the same.

@bolotyuh @rodrigocaus
The problem is not so trivial as it might seem. When LS generates the next task for the annotator, it must lock this task for a while until the annotator does his job. And this duration is unpredictable: it can be 1 second, 1 minute, 1 hour, etc.
We tried to use locks and the label stream looks like what you expect: annotators are given different tasks all the time. Although if you try to press Label again, you won't see unlabeled tasks, because they have been locked by some annotators for several ours in edge cases. And it's confusing.
So, you can have two fast workarounds:

Rewrite this line
https://github.com/heartexlabs/label-studio/blob/4f3df84fd38872c6a2405a2d8e330076bebf22db/label_studio/projects/api.py#L434 to

next_task = not_solved_tasks.order_by('?').first()

Create an individual tabs for each annotator and split tasks via filter id ranges.

@makseq Thanks you really help me.

@makseq How is used the parameter sampling I thought it resolve this type of problem(randomize tasks)?

@bolotyuh It seems sampling is skipped always because of Data manager tabs usage.

I did it  but get a new problem when disappear top toolbar

Yep, once I saw this problem, but can't reproduce it.

Yep, once I saw this problem, but can't reproduce it.

When you refresh the page on the same task top toolbar disappearing as I understood it because we already have annotation

@bolotyuh Trying to reproduce it, but still can't..

@makseq thank you for your answer. Using something like:
# ordered by data manager
if external_prepared_tasks_used:
    next_task = next((t for t in not_solved_tasks.all() if not t.has_lock()), None)
    if not next_task:
        raise NotFound('No more tasks found')
    return self._make_response(next_task, request)

I can "simulate" a filter on not_solved_tasks (something like not_solved_tasks.filter(has_lock=False).first()). Since the task lock seems to expire, eventually unsolved tasks can be retrieved again later. Could this work?

Yes, it might be working. Although this block should also be taken into account too:
https://github.com/heartexlabs/label-studio/blob/4f3df84fd38872c6a2405a2d8e330076bebf22db/label_studio/projects/api.py#L439

I see, my approach has this problem: Same User locks different tasks unnecessarily, so the project can falsely run out of tasks quickly (Users can lock as many tasks as they want as soon as they click on Label button)
I can suggest a similar solution:
# ordered by data manager
if external_prepared_tasks_used:
   try:
       next_task = next((
           t for t in not_solved_tasks.all()
           if t.locks.filter(user=user).count() > 0
       ))
       return self._make_response(next_task, request, use_task_lock=False)
   except StopIteration:
       next_task = next((t for t in not_solved_tasks.all() if not t.has_lock()), None)
       # could be next_task = not_solved_tasks.first()
       if not next_task:
           raise NotFound('No more tasks found')
       return self._make_response(next_task, request)

It is not pretty and it might have a better solution using filters on not_solved_tasks, but it seems to solve the problem you pointed.

#805


Do not doubly allocate

Hey, this is an attempt to get Thermochimica working with MOOSE. I ran into a multiple allocation issue with the reinitDataTcFromMoose function that is meant to be called repeatedly.

The approach that I used was to call ResetReinit (as well as two other reset functions) at the end of each Thermochimica call. You can look in any of the classes built on ThermochimicaNodalData to see an example.
I realize this is a bit clunky, but I think it is important to do things this way for 2 reasons.

Each different application will want to be different post-processing on the Thermochimica-generated data, so the reset should not come in the base class execute().
Resetting and reallocating is safer because, in general, there is nothing to prevent the array sizes from changing between calls. For example, a new element might be introduced to the simulation between time steps.

Ok, sounds good. I ran in to a repeated allocation error with the ThermochimicaNodalData class. I'll check again and make sure there is a call to ResetReinit!


feat(ui): display line numbers in object-editor. Fixes #12807.

Related issue:
Fixes #12807
Motivation
Adding support for displaying the line number would enhance the user experience, as it would aid in debugging
Modifications
Remove lineNumbers key in object-editor.tsx.
Verification
after changes:

before changes:

It looks good. I also wanted to display the line number on the left to help debug.


Add in a check if keys for all wallets are not accessible on startup

Probably have to iterate this a few times. Ideally something like:

Launch (not resume) app
Check keys are accessible
Inform user and "probably" go to the create new wallet screen

This is a bit tricky because we want to make sure the keys are really not accessible anymore (due to migration to a new device).
Related to: #1849

@James-Sangalli
Correct me if i mistake, for this task i need to to check wallets private key/ seed phrase accessibility while application is launching, right?
i need to check accessibility with prefix Keys.ethereumSeedUserPresenceRequiredPrefix or Keys.ethereumSeedUserPresenceNotRequiredPrefix too? and how to test this issue?

how to test this issue?

@vladyslav-iosdev the best way (and most important) test for this is:

Launch app, create wallet(s)
Run iCloud backup
Wipe phone or with a new phone
Restore phone from iCloud
Run app

Expected

Show the app like it's a new install (or tell user they keys are not restored)

Observed

The app looks like it still has the wallets and keys, but it doesn't.

Unless you have a spare phone, I'll try to test this :)

I might have been wrong about how to solve this.
So the problem we want to solve is described in the test above ^
The list of wallets is stored in UserDefaults with the keys Keys.ethereumAddressesWithPrivateKeys and Keys.ethereumAddressesWithSeed. So the problem is this is synced to iCloud. What if we wrote this 2 values to a file in the documents directory and don't back up the file to iCloud? (sorry I might have sent you down the wrong path! Vlad)
@vladyslav-iosdev what do you think?

@vladyslav-iosdev what do you think?

so while backup to icloud user defaults also move there. ok, got it, i will check it and  try to resolve it.

Unless you have a spare phone, I'll try to test this :)

i got one, will try to reproduce it

@hboon do we want to replace UserDefaults with storing data in file?

@hboon do we want to replace UserDefaults with storing data in file?

@vladyslav-iosdev I’m thinking just for these 2-3 values (list of wallets). Let’s think about this a bit. This piece of data is critical because if it’s broken, the user loses access to their wallets.

@hboon am i understand right, wallets can be accessible only on one device,  and can't be becked up to another one?

Yes. We don’t want them to be backed up

So the problem is caused by:

Keys are stored and not backed up (good)
The list of wallets is in UserDefaults and appears to be backed up and restored

I thought the solution was to check that (1) is not available (in a new device), but maybe the better and much easier solution is (2)?

So the problem is caused by:

Keys are stored and not backed up (good)
The list of wallets is in UserDefaults and appears to be backed up and restored

I thought the solution was to check that (1) is not available (in a new device), but maybe the better and much easier solution is just to make (2) not happen?

ok, now its much clear

And maybe the better way to store the list of wallets (2), currently in UserDefaults, should just be to store it in the keychain and flagged as not to backup. Seems safer than as a file?

And maybe the better way to store the list of wallets (2), currently in UserDefaults, should just be to store it in the keychain and flagged as not to backup. Seems safer than as a file?

seems like it will work, so it this case seems like this one will not need, right?
if !UIApplication.shared.isProtectedDataAvailable {
            throw EtherKeystoreError.protectionDisabled
        }

Why not? That mainly checks for whether the Realm database is protected

Why not? That mainly checks for whether the Realm database is protected

oh, ok, i got it


Update README.md

explicitly mention Linux
explain that connect must be issued before the "OK (Change)" button appears (connection is what triggers it)
format commands with monospace font
also, explain that the wifi connection must be done in ad-hoc mode

Thanks!


Allow for coalescing of operations

It should be possible to squash a set of operations such that related operations are merged. This functionality will be useful to condense any set of operations to a minimal set, which in turn will minimize network usage.
I'd like to start this work with a generic function that takes an array of operations and returns a "squashed" array of operations.

Closed via 95eaffaf0f961f5c20bdb853968d6ae1bf8ca441


Add weekly updates for 2015-08-21

https://github.com/nodejs/evangelism/pull/140/files

원문링크입니다.
https://medium.com/node-js-javascript/io-js-and-node-js-news-of-august-21st-6c8c49ff5cda

억..깜박하고있었네요 =ㄱ=);;

^^


Custom field valure are returned ONLY if contain value

I have made several test to recover the fields of one Sharepoint Document library with graph api:
https://graph.microsoft.com/v1.0/sites/{SiteId}/lists/{ListId}/items/{ItemId}?$expand=fields
I think it is important highlight that the answer contains the "custom" fields only if they are set.

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

ID: 182645cd-d48d-da9b-7a25-0f41126bb8eb
Version Independent ID: e58c571e-b339-88e1-e8b8-ada8457f09af
Content: Get listItem - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/listitem-get.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: MSGraphDocsVteam

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


libcloud storage support

Ciao Hetzner,
We use your cloud and storage boxes. We use Apache libcloud to abstract from a specific implementation and found that your storage is not in the library (you can find Amazon, Google, etc).
https://github.com/apache/libcloud
We wrote a driver for libcloud that connects to your storage box via WebDAV. You could add it to Apache libcloud so people could find it and also find you as a cloud provider. I attached the source code below. If you decide to use it I can explain how to add to libcloud, etc and also provide testing code. License is open source.
webdavdrivers.py.zip
Cheers

Hello @gionatamettifogo,
thank you for your message and your work on this! The scope of this library is only the Hetzner Cloud API (docs.hetzner.cloud), so we can not support our Storages Boxes, which are parts of our Hetzner Robot (robot.hetzner.com). However, we will have a look on libcloud and will decide if we contribute to this. Thank you!


Breaks Plex's databases (v<IP_ADDRESS>40)

I used this script to copy the state of one user to another on Plex v<IP_ADDRESS>40 on my Synology NAS, and it corrupted the databases. I had to restore from a DB backup.

Corrupted in what way? Did Plex not launch anymore after the transfer?


Vertical build consistency

In GitLab by @skahn on Oct 8, 2020, 11:46
Description
Not sure the vertical build is entirely consistent.
I would expect for double null the TF vertical bore to be 2.0D0 * hmax, but it does not seems to be the the way it is treated.
code extraction
    ! Height to inside edge of TF coil
    ! Rem SK : definition only valid for double null!
    hmax = rminor*kappa + vgap + divfix + shldlth + ddwi + vgap2 + thshield + tftsgap

    ! TF coil vertical bore [m] (Not sure it is entirely consistent !)
    ! Rem SK : not consistend for single null!
    dh_tf_inner_bore = 2.0D0*(rminor*kappa + vgaptop + fwith + blnktth + vvblgap + &
        shldtth + ddwi+ vgap2 + thshield + tftsgap)

Proposal
Set dh_tf_inner_bore = 2.0D0 * hmax if the double null option is considered.
Ref : @jmorris-uk, @mkovari and @stuartmuldrew
Best regards
Seb

@mkovari I see the changes you have made have clarified this code, so I will close the issue.
I think a more systematic scheme for the build variables sounds like a great idea.


servicerouter.py - enhances the error output for bad requests

When trying to run servicerouter.py as a callback on any marathon
instance that does not have callbacks enabled, it raised a nonsense
error message.
This MR enhances the provided error reason with the marathon specific message, if present.
Example old error:
[12:32] marfri@dhcp-workcat ~/work/marathon # ./bin/servicerouter.py -m "http://marathon1:8080" -l http://lb1:4242
Traceback (most recent call last):
  File "./bin/servicerouter.py", line 853, in <module>
    run_server(marathon, args.listening, args.haproxy_config, args.group)
  File "./bin/servicerouter.py", line 792, in run_server
    groups)
  File "./bin/servicerouter.py", line 727, in __init__
    marathon.add_subscriber(addr)
  File "./bin/servicerouter.py", line 468, in add_subscriber
    params={'callbackUrl': callbackUrl})
  File "./bin/servicerouter.py", line 446, in api_req
    return self.api_req_raw(method, path, **kwargs).json()
  File "./bin/servicerouter.py", line 442, in api_req_raw
    response.raise_for_status()
  File "/usr/local/lib/python2.7/site-packages/requests/models.py", line 825, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request

New error message:
[12:32] marfri@dhcp-workcat ~/work/marathon # ./bin/servicerouter.py -m "http://marathon1:8080" -l http://lb1:4242
Traceback (most recent call last):
  File "./bin/servicerouter.py", line 853, in <module>
    run_server(marathon, args.listening, args.haproxy_config, args.group)
  File "./bin/servicerouter.py", line 792, in run_server
    groups)
  File "./bin/servicerouter.py", line 727, in __init__
    marathon.add_subscriber(addr)
  File "./bin/servicerouter.py", line 468, in add_subscriber
    params={'callbackUrl': callbackUrl})
  File "./bin/servicerouter.py", line 446, in api_req
    return self.api_req_raw(method, path, **kwargs).json()
  File "./bin/servicerouter.py", line 442, in api_req_raw
    response.raise_for_status()
  File "/usr/local/lib/python2.7/site-packages/requests/models.py", line 825, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request (http event callback system is not running on this Marathon instance. Please re-start this instance with "--event_subscriber http_callback".)

Also I am unsure if this really is a 400 error but rather a 500 as its the server side that isnt properly configured but anyhow the request is "bad" in this case so I guess 400 is still ok.

Although I personally do not agree with PEP-8 for its line length limit, I "fixed" the issue and amended it into my original commit. Thanks for looking into it!

Yea, PEP-8 bugs me as well but it is well-known and I don't have to build any infrastructure to enforce it. The value of having everything match is greater than my style nits, esp. since this is a small helper utility script in a language different than the majority of the project.
All merged (With a little patch on top because not only does PEP-8 want you to not pass 79 chars, it is also particular about how you wrap the line to preserve "visual indent")


Update production

Mostly tested GenomicsAssayTemplate and ProteomicsAssayTemplate and checked config.json.

Hey @anngvu ; due to changes in schematic, https://github.com/nf-osi/nf-metadata-dictionary/pull/139 needs to be merged and released before we can update production. Would you be able to review that too? Thanks!


also patch estraverse-fb - fixes #73

Was trying to get jscs working and got a similar error.
Not sure if this is right but eslint is also using estraverse at https://github.com/eslint/eslint/blob/master/lib/eslint.js#L11. Patching the keys for that reference of estraverse seems to fix the Unknown node type ExportDefaultSpecifier error.

And checking some of the other issues - it seems to fix #71, #81 for me.

Released as of 3.1.1, thanks!

awesome.


Fix docs action

This is at least part of the problem of #11 (I guess a DOCUMENTER_KEY might still be needed for deployment?).
Edit: Added DOCUMENTER_KEY settings.

Thanks @devmotion, the GitHub key should be enough but didn’t realise the main branch at called ‘main’ by default now

Yes, the token should be sufficient here - but the dedicated deploy key has more permissions and hence is needed e.g. if a Github action should trigger another Github action (e.g. if using TagBot, see https://documenter.juliadocs.org/stable/man/hosting/#GitHub-Actions or the TagBot README). So in general I think it's useful to create (and then also use) a deploy key.


Marathon 1.5 release script

JIRA issues: MARATHON-8153

I was evil and rebased your commits onto releases/1.5 and force pushed 😈.

@meln1k status on this?

@meln1k status?

Won't do for now.


Where V-L coupling function?

Hi author, your work is excellent and I am very interested in your work. However, I have a question. As mentioned in your article, the first 8 layers of text prompts can provide visual prompts to the visual encoder through a V-L coupling function, but I have not found the corresponding implementation code for this place. Can you provide the corresponding code location for me? I am very grateful for that.

Hi @Sandy-program,
Thank you for showing interest in MaPLe!
Regarding your query, the corresponding code for the deep prompting at text side can be found at these lines.
I hope it is clear now.
Thank you and kind regards!

Thank you very much for your help and look forward to your better work


Update adviser text for lane arrow tool

The advisor (tutorial) text doesn't mention the new Ctrl and Alt modifiers for the recently merged #538 .
Although information is shown in tooltip on the menu button, would it be worth updating the adviser text popup too?

Also, it's not particularly clear from just the tooltip that the new modifier keys only make sense on roads with more than one lane in a given direction (eg. asymmetric roads or medium/large roads).

we have adviser text popup too?

Yes, each tool has it's own adviser text popup.
Automatic display: There's a TM:PE mod option (General tab?) to determine whether they automatically appear when first selecting a tool from TM"PE menu bar - I assume you turned that off at some point like I have.
Manual display: You can also select a tool then click the small (?) advisor button near main game build bar to manually show the adviser text for selected tool.
Before making any changes, see my recent comment in #548

We just need to update the crowdin. Please assign to me if you want me to do this.

You should already have access to alter those strings for the tutorial/adviser text.

@kvakvs is this fixed?
If so can you please close?

Have a look when you start the game, i believe it is fixed.
I do not have plans to run the game today.

If it's not currently done add a note in #1344 as I'm planning to do a bunch of crowdin updates this week.

I don't even know how to view tutorials anymore. did they disable this feature in Airport DLC?

@kianzarrin the game setting also needs enabling as does the setting in TMPE options. See https://github.com/CitiesSkylinesMods/TMPE/issues/1240 for details

yeah I enabled game settings but I can only see tutorials once.

how do I force show them again? there used to be a ! button.

I deleted the config file. lane arrow tool does not seem to have tutorial at all.

I think need to deal with #1240 first, so put this #547 on hold for now?


Update PULL_REQUEST_TEMPLATE.md

This PR fixes #73.  I thought I fixed a link in the PR template but I didn't.
I have run the link checker manually before marking this ready to merge.
All Submissions:

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Does your PR link to an issue?
[X] Have you described the changes this PR will make?

The updated link works for me!


OC <IP_ADDRESS>

Hello, Is there any version for the Hosted payment that will work with OC <IP_ADDRESS>?

Hey @esk1mox
Versions v1.0.0 to v1.0.4
Compatible with OpenCart versions up to <IP_ADDRESS>.