when  a pretrained model to use,the code could not work.

according to model.py  class CellposeModel(UnetModel)  its return values is 3

but  the main.py in line 152 wants 4 return values.

I have the same problem @carsen-stringer.
I tried to find a workaround using a notebook but couldn't fix it.
Thanks a lot for your help and amazing work.

I have the same problem @carsen-stringer.
I tried to find a workaround using a notebook but couldn't fix it.
Thanks a lot for your help and amazing work.

I have the same problem @carsen-stringer.
I tried to find a workaround using a notebook but couldn't fix it.
Thanks a lot for your help and amazing work.

I have the same problem @carsen-stringer.
I tried to find a workaround using a notebook but couldn't fix it.
Thanks a lot for your help and amazing work.

Adding the --test_dir xxx\test\  option worked for me (it should be optional so it's very confusing).
When training using this command: python -m cellpose --train --dir xxx\training\ --test_dir xxx\test\ --pretrained_model nuclei --diameter 8 --chan 0 --n_epochs 100 --use_gpu.
I could use the output model later for predictions.

Adding the --test_dir xxx\test\  option worked for me (it should be optional so it's very confusing).
When training using this command: python -m cellpose --train --dir xxx\training\ --test_dir xxx\test\ --pretrained_model nuclei --diameter 8 --chan 0 --n_epochs 100 --use_gpu.
I could use the output model later for predictions.

I have been able to train using the following command:
python -m cellpose --train --dir ~\training\  --pretrained_model cyto --diameter 10 --chan 0 --chan2 0 --n_epochs 5000 --use_gpu
As you suggest @EkilGitHub I had to change  **core.py **,line 789, in self.net.train() to self.net.train().cuda() to allow GPU training.
I could predict using the following command:
python -m cellpose --dir ~\predictions --pretrained_model ~cellpose ...
However the predictions I get are way worse than the pre-trained Cellpose model ... I'd still exploring this issue.

I have been able to train using the following command:
python -m cellpose --train --dir ~\training\  --pretrained_model cyto --diameter 10 --chan 0 --chan2 0 --n_epochs 5000 --use_gpu
As you suggest @EkilGitHub I had to change  **core.py **,line 789, in self.net.train() to self.net.train().cuda() to allow GPU training.
I could predict using the following command:
python -m cellpose --dir ~\predictions --pretrained_model ~cellpose ...
However the predictions I get are way worse than the pre-trained Cellpose model ... I'd still exploring this issue.

When trying to train using the notebook using the following command:
model.train(train_data, train_labels, train_files=None, test_data=None, test_labels=None, test_files=None,       channels=None, normalize=True, pretrained_model=None, save_path=save_path, save_every=100,        learning_rate=0.2, n_epochs=500, momentum=0.9, weight_decay=1e-05, batch_size=8, rescale=True)
I get this error:
RuntimeError: running_mean should contain 50 elements not 2

When trying to train using the notebook using the following command:
model.train(train_data, train_labels, train_files=None, test_data=None, test_labels=None, test_files=None,       channels=None, normalize=True, pretrained_model=None, save_path=save_path, save_every=100,        learning_rate=0.2, n_epochs=500, momentum=0.9, weight_decay=1e-05, batch_size=8, rescale=True)
I get this error:
RuntimeError: running_mean should contain 50 elements not 2

I have been able to train using the following command:
python -m cellpose --train --dir ~\training\ --pretrained_model cyto --diameter 10 --chan 0 --chan2 0 --n_epochs 5000 --use_gpu
As you suggest @EkilGitHub I had to change **core.py **,line 789, in self.net.train() to self.net.train().cuda() to allow GPU training.
I could predict using the following command:
python -m cellpose --dir ~\predictions --pretrained_model ~cellpose ...
Weirdly sometime I also get the same error as described by @EkilGitHub ... no clue why.
However the predictions I get are way worse than the pre-trained Cellpose model ... I'd still exploring this issue.

maybe you can not choice --diameter 10. and try again.

I have been able to train using the following command:
python -m cellpose --train --dir ~\training\ --pretrained_model cyto --diameter 10 --chan 0 --chan2 0 --n_epochs 5000 --use_gpu
As you suggest @EkilGitHub I had to change **core.py **,line 789, in self.net.train() to self.net.train().cuda() to allow GPU training.
I could predict using the following command:
python -m cellpose --dir ~\predictions --pretrained_model ~cellpose ...
Weirdly sometime I also get the same error as described by @EkilGitHub ... no clue why.
However the predictions I get are way worse than the pre-trained Cellpose model ... I'd still exploring this issue.

maybe you can not choice --diameter 10. and try again.

I have been able to train using the following command:
python -m cellpose --train --dir ~\training\ --pretrained_model cyto --diameter 10 --chan 0 --chan2 0 --n_epochs 5000 --use_gpu
As you suggest @EkilGitHub I had to change **core.py **,line 789, in self.net.train() to self.net.train().cuda() to allow GPU training.
I could predict using the following command:
python -m cellpose --dir ~\predictions --pretrained_model ~cellpose ...
Weirdly sometime I also get the same error as described by @EkilGitHub ... no clue why.
However the predictions I get are way worse than the pre-trained Cellpose model ... I'd still exploring this issue.

maybe you can delete "--diameter 10." and train again.

I have been able to train using the following command:
python -m cellpose --train --dir ~\training\ --pretrained_model cyto --diameter 10 --chan 0 --chan2 0 --n_epochs 5000 --use_gpu
As you suggest @EkilGitHub I had to change **core.py **,line 789, in self.net.train() to self.net.train().cuda() to allow GPU training.
I could predict using the following command:
python -m cellpose --dir ~\predictions --pretrained_model ~cellpose ...
Weirdly sometime I also get the same error as described by @EkilGitHub ... no clue why.
However the predictions I get are way worse than the pre-trained Cellpose model ... I'd still exploring this issue.

maybe you can delete "--diameter 10." and train again.

@EkilGitHub I tried removing the diameter argument but I still have the same problem.

@EkilGitHub I tried removing the diameter argument but I still have the same problem.

Cellpose has been pretrained with a diameter of 30, so I would recommend keeping the default.
Have you tried retraining from scratch? Unfortunately I haven't had much success training from the pretrained model either, particularly if the images are far out of class from the training set. You can download the full dataset here: http://www.cellpose.org/dataset

Cellpose has been pretrained with a diameter of 30, so I would recommend keeping the default.
Have you tried retraining from scratch? Unfortunately I haven't had much success training from the pretrained model either, particularly if the images are far out of class from the training set. You can download the full dataset here: http://www.cellpose.org/dataset

When trying to train using the notebook using the following command:
model.train(train_data, train_labels, train_files=None, test_data=None, test_labels=None, test_files=None, channels=None, normalize=True, pretrained_model=None, save_path=save_path, save_every=100, learning_rate=0.2, n_epochs=500, momentum=0.9, weight_decay=1e-05, batch_size=8, rescale=True)
I get this error:
RuntimeError: running_mean should contain 50 elements not 2

Can you tell me more about this error? Sorry about these bugs, I've fixed the two other bugs you mentioned

When trying to train using the notebook using the following command:
model.train(train_data, train_labels, train_files=None, test_data=None, test_labels=None, test_files=None, channels=None, normalize=True, pretrained_model=None, save_path=save_path, save_every=100, learning_rate=0.2, n_epochs=500, momentum=0.9, weight_decay=1e-05, batch_size=8, rescale=True)
I get this error:
RuntimeError: running_mean should contain 50 elements not 2

Can you tell me more about this error? Sorry about these bugs, I've fixed the two other bugs you mentioned

Thank you @carsen-stringer. Here is the where I get the error message when using the notebook:

and the full err. message:
`RuntimeError                              Traceback (most recent call last)
 in
27 model.train(train_data, train_labels, train_files=None, test_data=None, test_labels=None, test_files=None,
28       channels=None, normalize=True, pretrained_model=None, save_path=save_path, save_every=100,
---> 29       learning_rate=0.2, n_epochs=500, momentum=0.9, weight_decay=1e-05, batch_size=8, rescale=True)
30
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\models.py in train(self, train_data, train_labels, train_files, test_data, test_labels, test_files, channels, normalize, pretrained_model, save_path, save_every, learning_rate, n_epochs, momentum, weight_decay, batch_size, rescale)
640                                      test_data, test_flows,
641                                      pretrained_model, save_path, save_every,
--> 642                                      learning_rate, n_epochs, momentum, weight_decay, batch_size, rescale)
643         self.pretrained_model = model_path
644         return model_path
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\core.py in _train_net(self, train_data, train_labels, test_data, test_labels, pretrained_model, save_path, save_every, learning_rate, n_epochs, momentum, weight_decay, batch_size, rescale, netstr)
926                 if self.unet and lbl.shape[1]>1 and rescale:
927                     lbl[:,1] /= diam_batch[:,np.newaxis,np.newaxis]**2
--> 928                 train_loss = self._train_step(imgi, lbl)
929                 lavg += train_loss
930                 nsum += len(imgi)
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\core.py in _train_step(self, x, lbl)
788             self.optimizer.zero_grad()
789             self.net.train().cuda()
--> 790             y, style = self.net(X)
791             loss = self.loss_fn(lbl,y)
792             loss.backward()
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\resnet_torch.py in forward(self, data)
189         if self.mkldnn:
190             data = data.to_mkldnn()
--> 191         T0    = self.downsample(data)
192         if self.mkldnn:
193             style = self.make_style(T0[-1].to_dense())
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\resnet_torch.py in forward(self, x)
81             else:
82                 y = x
---> 83             xd.append(self.downn)
84         return xd
85
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\resnet_torch.py in forward(self, x)
44
45     def forward(self, x):
---> 46         x = self.proj(x) + self.conv1
47         x = x + self.conv3
48         return x
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\container.py in forward(self, input)
115     def forward(self, input):
116         for module in self:
--> 117             input = module(input)
118         return input
119
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\batchnorm.py in forward(self, input)
134             self.running_mean if not self.training or self.track_running_stats else None,
135             self.running_var if not self.training or self.track_running_stats else None,
--> 136             self.weight, self.bias, bn_training, exponential_average_factor, self.eps)
137
138
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\functional.py in batch_norm(input, running_mean, running_var, weight, bias, training, momentum, eps)
2056     return torch.batch_norm(
2057         input, weight, bias, running_mean, running_var,
-> 2058         training, momentum, eps, torch.backends.cudnn.enabled
2059     )
2060
RuntimeError: running_mean should contain 50 elements not 2`
Thanks a lot for your help.
Jon

Thank you @carsen-stringer. Here is the where I get the error message when using the notebook:

and the full err. message:
`RuntimeError                              Traceback (most recent call last)
 in
27 model.train(train_data, train_labels, train_files=None, test_data=None, test_labels=None, test_files=None,
28       channels=None, normalize=True, pretrained_model=None, save_path=save_path, save_every=100,
---> 29       learning_rate=0.2, n_epochs=500, momentum=0.9, weight_decay=1e-05, batch_size=8, rescale=True)
30
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\models.py in train(self, train_data, train_labels, train_files, test_data, test_labels, test_files, channels, normalize, pretrained_model, save_path, save_every, learning_rate, n_epochs, momentum, weight_decay, batch_size, rescale)
640                                      test_data, test_flows,
641                                      pretrained_model, save_path, save_every,
--> 642                                      learning_rate, n_epochs, momentum, weight_decay, batch_size, rescale)
643         self.pretrained_model = model_path
644         return model_path
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\core.py in _train_net(self, train_data, train_labels, test_data, test_labels, pretrained_model, save_path, save_every, learning_rate, n_epochs, momentum, weight_decay, batch_size, rescale, netstr)
926                 if self.unet and lbl.shape[1]>1 and rescale:
927                     lbl[:,1] /= diam_batch[:,np.newaxis,np.newaxis]**2
--> 928                 train_loss = self._train_step(imgi, lbl)
929                 lavg += train_loss
930                 nsum += len(imgi)
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\core.py in _train_step(self, x, lbl)
788             self.optimizer.zero_grad()
789             self.net.train().cuda()
--> 790             y, style = self.net(X)
791             loss = self.loss_fn(lbl,y)
792             loss.backward()
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\resnet_torch.py in forward(self, data)
189         if self.mkldnn:
190             data = data.to_mkldnn()
--> 191         T0    = self.downsample(data)
192         if self.mkldnn:
193             style = self.make_style(T0[-1].to_dense())
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\resnet_torch.py in forward(self, x)
81             else:
82                 y = x
---> 83             xd.append(self.downn)
84         return xd
85
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\cellpose\resnet_torch.py in forward(self, x)
44
45     def forward(self, x):
---> 46         x = self.proj(x) + self.conv1
47         x = x + self.conv3
48         return x
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\container.py in forward(self, input)
115     def forward(self, input):
116         for module in self:
--> 117             input = module(input)
118         return input
119
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\module.py in _call_impl(self, *input, **kwargs)
725             result = self._slow_forward(*input, **kwargs)
726         else:
--> 727             result = self.forward(*input, **kwargs)
728         for hook in itertools.chain(
729                 _global_forward_hooks.values(),
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\modules\batchnorm.py in forward(self, input)
134             self.running_mean if not self.training or self.track_running_stats else None,
135             self.running_var if not self.training or self.track_running_stats else None,
--> 136             self.weight, self.bias, bn_training, exponential_average_factor, self.eps)
137
138
~\anaconda3\envs\cellpose\lib\site-packages\torch\nn\functional.py in batch_norm(input, running_mean, running_var, weight, bias, training, momentum, eps)
2056     return torch.batch_norm(
2057         input, weight, bias, running_mean, running_var,
-> 2058         training, momentum, eps, torch.backends.cudnn.enabled
2059     )
2060
RuntimeError: running_mean should contain 50 elements not 2`
Thanks a lot for your help.

Hey thank you so much for the great work!
I am trying to train on my custom dataset and got the same eror.
I have training images with three channels (shape: (3,820,820)) and .png labels in label format (shape: (820,820))
please have a look at the error traceback and let me know if  you know of any possible reason for this error.

Thank you so much!

I converted all training images into grayscale and now I am getting this error. I am not sure, what do 2 elements  mean in the error and from where it's getting that

Hey thank you so much for the great work!
I am trying to train on my custom dataset and got the same eror.
I have training images with three channels (shape: (3,820,820)) and .png labels in label format (shape: (820,820))
please have a look at the error traceback and let me know if you know of any possible reason for this error.

Thank you so much!

If your images have different numbers of channels you need to specify nchan=3 in the CellposeModel(nchan=3, pretrained_model=None) call, that will create the model with the right number of image channels. then train with your images.
from the command line interface, add the --all_channels flag and cellpose will determine nchan and initialize accordingly. <<this was supposed to work correctly but there was a bug I fixed yesterday. please try the latest code.


Naming conventions

Let’s try to keep our names a close to bitinformation.jl as possible.
Rename

https://github.com/observingClouds/bitinformation_pipeline/blob/master/bitinformation_pipeline/get_n_plot_bitinformation.jl to bitinformation.jl
information_content or inflevels #9
please add here

to be more aligned to bitinformation.jl, I'd propose to rename get_bitinformation to bitinformation as in the original, xr_bitround to round (how to deal with jl_bitround then?)

Could also rename package to xbitinformation

For now I think we have implemented all changes suggested so far. I close it for now.


support NW.js 0.13

Hello, check discussion in this pull request https://github.com/nwjs/nw-builder/pull/296
Dropped support for osx32 bit and osx icons needs to be resolved as well

Thanks a lot but we're going to redo how we do a lot of this. This no longer suits. See the plan at #327. Any help is more than welcome.


where to put JS  files and API's DCNs for other modules like articles?

I am new to meanjs and have a simple question. I create a new module for showing buildings on google maps on client htmls, but where to include the google map API's DCN and other javascript files? Thanks a lot!

You need to add your dependencies in config/assets/default.js for module-level dependencies.
Or you can add project wide dependencies in core/client/app/config.js as applicationModuleVendorDependencies
Hope you find it helpful...

@aniu feel free to re-open if you have further questions about this!


make target morfe-m68k doesn't work (vanilla ubuntu)

The morfe-m68k target doesn't build - on my vanilla ubuntu 20.04 install. It says it can;t find some object files, but when I look in those directories I see the files....

`rene@rene-NUC8i5BEK:~/morfe$ make morfe-m68k
cp -vf lib/musashi-c-wrapper/m68kconf.h external/Musashi/
'lib/musashi-c-wrapper/m68kconf.h' -> 'external/Musashi/m68kconf.h'
make -C external/Musashi clean
make[1]: Entering directory '/home/rene/morfe/external/Musashi'
rm -f m68kops.c m68kops.h m68kcpu.o m68kdasm.o softfloat/softfloat.o m68kops.o  m68kmake
make[1]: Leaving directory '/home/rene/morfe/external/Musashi'
make -C external/Musashi
make[1]: Entering directory '/home/rene/morfe/external/Musashi'
gcc -o  m68kmake  m68kmake.c
./m68kmake

Musashi v4.60 68000, 68008, 68010, 68EC020, 68020, 68EC030, 68030, 68EC040, 68040 emulator
	Copyright Karl Stenerud<EMAIL_ADDRESS>
Generated 1967 opcode handlers from 518 primitives
gcc -Wall -Wextra -pedantic   -c -o m68kcpu.o m68kcpu.c
gcc -Wall -Wextra -pedantic   -c -o m68kdasm.o m68kdasm.c
gcc -Wall -Wextra -pedantic   -c -o softfloat/softfloat.o softfloat/softfloat.c
gcc -Wall -Wextra -pedantic   -c -o m68kops.o m68kops.c
make[1]: Leaving directory '/home/rene/morfe/external/Musashi'
CGO_LDFLAGS_ALLOW=".+/(Musashi|musashi-c-wrapper)/.+.o"
go build --tags m68k -o morfe-m68k cmd/morfe/*.go
github.com/aniou/morfe/emulator/cpu_68xxx
gcc: error: lib/musashi-c-wrapper/shim.o: No such file or directory
gcc: error: external/Musashi/m68kcpu.o: No such file or directory
gcc: error: external/Musashi/m68kdasm.o: No such file or directory
gcc: error: external/Musashi/m68kops.o: No such file or directory
gcc: error: external/Musashi/softfloat/softfloat.o: No such file or directory
make: *** [Makefile:28: morfe-m68k] Error 2
 rene@rene-NUC8i5BEK:~/morfe$ cd external
rene@rene-NUC8i5BEK:~/morfe/external$ cd Musashi/
rene@rene-NUC8i5BEK:~/morfe/external/Musashi$ ls
example      m68kcpu.h   m68kfpu.c  m68kmake.c  m68kops.o
history.txt  m68kcpu.o   m68k.h     m68kmmu.h   Makefile
m68kconf.h   m68kdasm.c  m68k_in.c  m68kops.c   readme.txt
m68kcpu.c    m68kdasm.o  m68kmake   m68kops.h   softfloat
`

Commenting on my own submit :-)
It seems that vanilla Ubuntu with vanilla packages is not up to the task of correctly compiling the current top-of-tree.
To compile the current version directly, I needed to do the following:

Install GO from the tarball (with some manual PATH adaptations) as per the official website (the Ubuntu package manager installs an old version of golang  that does not work on all make targets)
Install libsdl2-dev with the package manager
Install make with the package manager

Even then, it compiles correctly (for the morfe and morfe-m68k targets), but the morfe-m68k conf/a2456k.ini does not run. The a2560k.ini file refers to a 'retro68.hex' file that doesn't exist. Removing that starts the morfe-m68k program, but then it crashes with a SEGSEV exception.
This could be related to the Musashi core that is intalled and compiled during the process, as that looks like a moving target itself.
I hope the author can help here ;-)

Hi, sorry for a long delay - I fixed that bug some times ago and now I fixed another bug, that has negative impact on compilation on current Go - but it is worth to mention, that current version of morfe has very inaccurate and outdated memory map, in comparison to real product.
Currently, my main effort is concentrated on https://github.com/aniou/morfeo - a new version of emulator, that offers a2560x with extra devices and valid memory map.


🌐 Add Chinese translation for docs/tutorial/dependencies/dependencies-with-yield.md

…ield.md
as title

📝 Docs preview for commit 979ade870b9b1272d65bb19697869a291db22e6a at: https://6499d1a2bac8cd0a1954b002--fastapi.netlify.app

Hi! This one was already handled in https://github.com/tiangolo/fastapi/pull/10870. For now, I'll close this one, but thanks for the help @jaystone776 @waynerv 🚀


Support for conda explicit lockfiles

Proposed change

Hi!
Conda currently supports exporting environments via conda list --explicit > spec-file.txt and installing them by doing conda create -n <env_name>  --file spec-file.txt.
This kind of installation bypass the solver making builds really, really fast.
Alternative options

Also, using mamba instead of conda will speed up the deployment.
Who would use this feature?

Conda environment users that wants to build their environments faster.
(Optional): Suggest a solution

Wrong repository, sorry.


Implement more efficient JSON Parser?

Maybe a more efficient JSON parser may further improve performance.
This seems a comprehensive guide: https://github.com/miloyip/nativejson-benchmark

Without sourcecode, that would be near impossible
I don't think tostercx plans to do anything further on this

did a quick test - the JSON parses in ~800ms on my machine with the strlen patch, so there's not much to gain

2. I don't think tostercx plans to do anything further on this

I plan to poke around GTA internals some more when I get time, I don't plan to maintain a stable release of this project tho


Image titles as filename of the downloaded images.

Would be really nice if the title of the image were used as the images' filename instead of the HEX filename used by Imgur. Would be really useful in making sense of the downloaded files.

I have a few concerns about using image titles as filenames:
1 - Most images I've seen uploaded to imgur don't have titles. So do we just fall back to the hex for them? What happens if some images have titles and others don't, that's kinda weird in the downloaded folder.
2 - Suitably escaping the titles so that they're safe to use as filenames across operating systems (my python is a little sketchy, not sure how much os does to help with this)
3 - Scope creep. This is meant to be a quick and simple grabber, not a fully featured API client, and part of me wants to keep it that way simply to be a good citizen to imgur. If we start implementing a tonne of features that mean you can circumvent using the real API for reads, that feels a bit wrong to me. Means imgur can't rate limit / monitor usage of their services via the API key mechanism. Might sound a little ironic, me suggesting that having written this lib in the first place, but I do believe there's a certain line between "quick and dirty tool with minimal impact" and "deliberately trying to break the rules".
Interested to hear discussion on this though, if it's a feature a lot of people want, I'd consider accepting a pull request for it (with points 1 and 2 suitably addressed), but I'm not sure I fancy writing it.

I'd personally prefer if that feature is kept out of the main tool. A third-party script to retrieve page names from the image filename should be pretty easy to write. It could be linkd from the README or added as a separate file in this repo

@alexgisby Responses to your primary concerns:

Even though many images do not contain title, most of them contains an album name. So in the absence of image titles, the album name with a serial suffix can be substituted and further in the absence of an album title, the HEX value can be used.

I am new to Python programming, but can you somehow use UNICODE for naming to enable uniformity and also support multiple languages.

I understand your concerns of cluttering this simple tool. I wanted to say that this would be a nice addition. Also, Imgur allows users to download public galleries as .zip files and their filenames aren't HEX values. Instead I think they are the original filenames at the time of their upload. But am not sure.

@alexgisby  and @nodiscc As I mentioned, I am new to Python and thus would really appreciate it if you implement this feature or maybe point me to resources that could help me to create them myself. As @nodiscc  suggested, it could be added as an extension to the original script,

The titles-as-filenames problem isn't so much encoding, it's special characters. Slashes can often be problematic, as can colons. Then you get onto the filename length problem, different OS's have different filename length issues, and now you need to be truncating correctly and would anyone actually want a 255 (or more) character long filename in their download directory?
The script currently allows you to chose the name of the directory you download into (second parameter on the command line), and so the user can choose a directory name that makes sense to them, rather than this script attempting to guess at it.
We definitely don't have access to the original filenames they were uploaded as from this tool. All this script does is scrape the HTML output for  tags, and so we don't know anything apart from the image ID (the hex value). Getting the original filename would be an API call, which requires an API key and is definitely out of scope for this tool.

@alexgisby  I understand the complexities involved. Just wanted to know a method of giving meaningful names to those images stored.
Also the reason I mentioned about the .zip download was to say that Imgur themselves are okay with us downloading public galleries and wasn't asking to implement the same here.
Anyway thanks for the quick replies and more importantly for sharing your code.

I wanted this for album titles (but not images), so I've put up PR #35.


Preserve OpenCL.DebugInfo.100 through elim-local-single-store

This pass basically follows the same process as ssa-rewrite: it adds a DebugValue after each Store and removes the DebugDeclare or DebugValue Deref. It only does this if all instructions that are dependent on the Store are Loads and are replaced.

@jaebaek @s-perron Please review.


fix: import nuxt composables from #imports

This is a DX improvement when developing - we can avoid loading the entire barrel file at #app by using the new granular imports merged in https://github.com/nuxt/nuxt/pull/23951.

Hi @danielroe
Thanks for the PR.
I changed the destination branch from main to dev.
Your change will be deployed with the next version. I'll keep you posted.
Don't hesitate to create new PRs or to give us feedback on TwicPics and/or its components.
Thanks again.


Update of python3.11 image  from bullseye to bookworm broke azure-cognitiveservices-speech package

Hello,
I'm not sure if this is an issue in the package or the underlying container image, but the recent update of Python3.11 image broke the package completely and it is impossible to run the speech recognizer. There is an issue created in the Github repo of the project: https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1986
The error is not very helpful, just a list of .so files:
RuntimeError: Exception with error code:
[CALL STACK BEGIN]

/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1d2181) [0x7f7bfefd2181]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1ef2c1) [0x7f7bfefef2c1]
/lib/x86_64-linux-gnu/libc.so.6(+0x8de37) [0x7f7c08b75e37]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1f034e) [0x7f7bfeff034e]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1a6d3b) [0x7f7bfefa6d3b]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1a65e7) [0x7f7bfefa65e7]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1d9654) [0x7f7bfefd9654]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1b3d6a) [0x7f7bfefb3d6a]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1c47fa) [0x7f7bfefc47fa]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0xe9b41) [0x7f7bfeee9b41]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1d9654) [0x7f7bfefd9654]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1bc453) [0x7f7bfefbc453]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1fd4be) [0x7f7bfeffd4be]
/usr/local/lib/python3.11/site-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(synthesizer_create_speech_synthesizer_from_config+0xf3) [0x7f7bfeea1bd8]
/lib/x86_64-linux-gnu/libffi.so.8(+0x6f7a) [0x7f7c08112f7a]
/lib/x86_64-linux-gnu/libffi.so.8(+0x640e) [0x7f7c0811240e]
/lib/x86_64-linux-gnu/libffi.so.8(ffi_call+0xcd) [0x7f7c08112b0d]
[CALL STACK END]

Runtime error: Failed to initialize platform (azure-c-shared). Error: 2153

Rollback to an older version seems to help.

Running into similar issues with python:3.9-slim image. As pointed out in the azure's cognitive-services-speech-sdk issue here, azure's sdk doesn't support OpenSSL-3 which is the default in Deb 12 that the latest python images are based on.
The possible workarounds could be to

pin the docker image to an older python docker using its digest info (which I successfully confirmed); or
downgrade the OpenSSL to 1.x (which I haven't confirmed but seems to be an approache discussed on the azure github issue).

Same comment as https://github.com/docker-library/python/issues/835#issuecomment-1593459023, although I am unsure if azure-cognitiveservices-speech in your environment is hitting any libseccomp issues or just the OpenSSL version change.

From #837 (comment):

I'd suggest updating docker and libseccomp on the host. Newer base OS's use newer system calls and an older libseccomp can block them since they are unknown to it. You can verify that it is libseccomp by running the bookworm image with --security-opt seccomp=unconfined.
This is similar to the update to Ubuntu focal: docker-library/mongo#606 (comment)

If you are having issues with the new bookworm based images, then switching to the *-bullseye images is one possible workaround.
Similar to #836 and #837

Closing for now. Solution seems to be to use the bullseye images.


MAPREDUCE-7210. Replace mapreduce.job.counters.limit with mapreduce.job.counters.max in mapred-default.xml

Because the mapreduce.job.counters.limit is deprecated, the new property name is mapreduce.job.counter.max, so we need update the mapred-default.xml.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
33
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
1048
trunk passed

+1
compile
36
trunk passed

+1
mvnsite
41
trunk passed

+1
shadedclient
1740
branch has no errors when building and testing our client artifacts.

+1
javadoc
15
trunk passed

_ Patch Compile Tests _

+1
mvninstall
30
the patch passed

+1
compile
28
the patch passed

+1
javac
28
the patch passed

+1
mvnsite
32
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
1
The patch has no ill-formed XML file.

+1
shadedclient
687
patch has no errors when building and testing our client artifacts.

+1
javadoc
16
the patch passed

_ Other Tests _

+1
unit
325
hadoop-mapreduce-client-core in the patch passed.

+1
asflicense
28
The patch does not generate ASF License warnings.

3063

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-878/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/878

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml

uname
Linux 60531233ee99 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 1ae062c

Default Java
1.8.0_212

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-878/2/testReport/

Max. process+thread count
1631 (vs. ulimit of 5500)

modules
C: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core U: hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-878/2/console

versions
git=2.7.4 maven=3.3.9

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

+1

Thanks @aajisaka  for reviewed and committed work.


fix: ACNA-1786 - serve-static hook error

Closes #609
How Has This Been Tested?

npm test
manual test (link local app plugin, set serve-static hook, then aio app run)

Types of changes

[X] 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:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

Not sure what's going on with codecov. Investigating...


bson has no exported member Int32

Trying to build throws the error
node_modules/@types/connect-mongo/node_modules/@types/mongodb/index.d.ts(24,45): error TS2305: Module '"C:/.../node_modules/@types/bson/index"' has no exported member 'Int32'.

Installing @types/bson 1.0.11 resolves the error
Windows 10 pro
Node 10.15.0
TypeScript 2.7

This should be fixed in updated deps as the @types/mongodb dependency supports this already:
https://github.com/DefinitelyTyped/DefinitelyTyped/commit/43de02ea1d87534427c0b1ae95e13a26d11e4e42#diff-c4368217c3da6217f635d632d0bd4c87
If the problem persists:
npm install --no-cache


[ConfigTransformer] Merge multiple section under one name together

Sometimes, such snippet is created from commented out code:
    $containerConfigurator->extension('framework', [
        'secret' => '%env(APP_SECRET)%',
    ]);

    $containerConfigurator->extension('framework', [
        'http_method_override' => false,
    ]);

:x:

The correct result should look like this:

    $containerConfigurator->extension('framework', [
        'secret' => '%env(APP_SECRET)%',
        'http_method_override' => false,
    ]);

:heavy_check_mark:
This can be achieved with node traverser run after the node conversion.

Resolved i


feat!: set default inputFormatter and keyboardType to DigitField

BREAKING CHANGES
Closes #21

@rultor merge it.

@rultor merge it.

@rafamizes OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge it.

@rafamizes Done! FYI, the full log is here (took me 2min)


tensorflow/contrib/lite/toco/tflite/export.cc:192] Unsupported operator: Sub

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Linux Ubuntu 17.04
TensorFlow installed from (source or binary):pip
TensorFlow version (use command below):1.4.0
Python version: 2.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:8.0/6.0
GPU model and memory:GTX 1060 with 6GB memory
Exact command to reproduce:bazel run --config=opt tensorflow/contrib/lite/toco:toco -- --input_file=/home/wh/gitmodel/tensorflow/wh/frozen_1.pb --input_format=TENSORFLOW_GRAPHDEF --output_format=TFLITE --output_file=/home/wh/gitmodel/tensorflow/wh/frozen_lite.lite --inference_type=FLOAT --input_type=FLOAT --input_arrays=image_1 --output_arrays=InferenceTower/output6 --input_shapes=1,320,480,3

Describe the problem
I am trying to convert a graph from .pb to .lite format using toco, but I get this error:
2017-12-04 20:14:38.202653: F tensorflow/contrib/lite/toco/tflite/export.cc:192] Unsupported operator: Sub
I think Sub is the basic op, Lite shoud support it. Is it right?

/CC @aselle, can you comment?

I use summarize_graph tool to my model. it output :
Found 3681310 (3.68M) const parameters, 0 (0) variable parameters, and 0 control_edges
Op types used: 99 Const, 47 Identity, 19 Conv2D, 18 BiasAdd, 13 Relu, 10 Conv2DBackpropInput, 10 MirrorPad, 10 Mul, 10 Shape, 10 StridedSlice, 4 MaxPool, 1 ConcatV2, 1 Placeholder, 1 Sigmoid, 1 Sub

Lite unfortunately currently doesn't support Sub. We are planning to add support to it soon.
If you could share the python source of your model, we might be able to point out an alternative approach. You could possibly simulate a subtraction by doing a 1x1 conv with same padding with the filter -1 and then do an addition... i.e.
foo - bar
is equivalent to
foo + bar* -1


Redirects do not work when using Django's i18n_patterns() wrapper

Issue Summary
When using Wagtail wrapped by Django's i18n_patterns, redirects only work for "exactly that URL" instead of also for the localised versions. E.g. /from-page => /to-page only works for http://site/from-page but not http://site/en-us/from-page.
Steps to Reproduce

Set up a standard wagtail install
follow the wagtail instructions on turning on internationalisation, with at least on LANGUAGES entry, I'd recommend en-us.
create a redirect for /test to the homepage.
navigate to localhost:8000/test and observe it redirecting to localhost:8000
navigate to localhost:8000/es-us/test
this is a 404 instead of redirecting properly

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

The fix for this is updating the wagtail.contrib.redirects middleware to, in _get_redirect, remove the locale prefix from the path to resolve, if request.LANGUAGE_CODE exists.
Technical details

Python version: 3.7.7
Django version: 3.1
Wagtail version: 2.11.2
Browser version: irrelevant

Sorry if I haven't fully understood the issue, but the existing behaviour looks like the expected behaviour to me. Redirects are supposed to bypass all of the page routing logic because they are mainly used to support legacy URLs or allow editors to define a shortened URL for a page. I'm not sure how also making them work with all the language prefixes by default helps either of these use cases.

When one uses wagtail-modeltranslation (which is recommended as one of the two localization strategies) there are no "different pages". Every localised page is the same page, and redirects should kick in for everything with a local code. Right now, redirects simply don't work, at all, for setups that use that approach.
If there is additional code that can be checked to determine whether a wagtail instance is using "full, distinct, page trees" vs. "field-based localisation", then the PR can be improved to take that into account.

setting a redirect without the locale code will redirect only when users directly navigate to that specific unprefixed URL

This is exactly what the redirects module is supposed to do, if you need to redirect from a locale prefix then you can add this on to the redirect.
Since this issue is specific to wagtail-modeltranslation, I think the best approach for solving this issue is probably going to be copying middleware.py into wagtail-modeltranslation and customising the logic there.

Can you go into what you mean with "you can add this on to the redirect"? How would one do that, and are there wagtaill docs already written so that folks can actually read up on how to do that? If not, those definitely need to be written and linked to on https://docs.wagtail.io/en/stable/advanced_topics/i18n.html?highlight=internationalisation

Sorry if I wasn't clear enough. I meant add it to the redirect path.
So instead of creating a redirect for /test, you create a redirect with the path /en-us/test and this should give the behaviour you are looking for.

If I do that, the redirects do not work, so that is not the solution.

Am closing this issue, as the redirects app was never designed to be compatible with i18n_patterns. I concur with @kaedroho that this would be best resolved as part of wagtail-modeltranslation - as the requirements seem quite specific to that approach (which I believe haven't been recommended as a localisation solution by Wagtail for some time).

Am closing this issue, as the redirects app was never designed to be compatible with i18n_patterns. I concur with @kaedroho that this would be best resolved as part of wagtail-modeltranslation - as the requirements seem quite specific to that approach (which I believe haven't been recommended as a localisation solution by Wagtail for some time).

fair enough, but note that the docs still tell folks to use either wagtailtrans or wagtail-modeltranslations over on https://docs.wagtail.io/en/stable/advanced_topics/i18n.html?highlight=localization#alternative-internationalisation-plugins if they need an alternative, meaning it's basically one of the two official recommendations.

fair enough, but note that the docs still tell folks to use either wagtailtrans or wagtail-modeltranslations over on https://docs.wagtail.io/en/stable/advanced_topics/i18n.html?highlight=localization#alternative-internationalisation-plugins if they need an alternative, meaning it's basically one of the two official recommendations.

Same problem :(

@raminjrmz this issue has been closed for some time - see above comments.
I recommend you start a discussion about your problem and see if others can provide a work around. Or if you feel there is a bug that is not covered by this response above - maybe a new bug is needed.


Force IPv4 DNS lookups until IPv6 is supported

Rancher Version: 1.0
Docker Version: 1.10.3
OS: Debian jessie
Steps to Reproduce: Add host with full IPv6 stack and AAAA records
Results: Same as #1946
Expected: Everythings runs smoothly just like with 0.59
Could you please force DNS lookups to IPv4 until IPv6 is supported? I wasn't able to get my host to run any stacks until i explicitly changed the Host Registration URL to a non-HTTPS IPv4 URL.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Added SwiftWebVC to UI section

This is a Swift implementation of SVWebViewController (already in the list).

Also added a period/full stop to the description of the framework a couple of lines above.


Link the private documents to hocuspocuse database via uuid, also add username and cursor color to the db

For private documents we have to serve the document under the /userNamespace namespace.

I need to revise this task again. Now I have this flow for public documents, but for private documents, I need to worry about integration between the hocusepocuse server and subapase auth service.


Image file for reference from Herd-UI wiki

I admit to accepting my own pull request but it's just a harmless additive change to gh-pages


feat: application change history (HL-1061)

Description :sparkles:
This adds the change key to the application payload. change key is a list of change dictionaries. Change dictionaries are in format {"field": <field>, "old": <old_value>, "new": <new_value>}.
For the handler, the changes key includes changes that are made when the application status is additional_information_needed.
For the applicant, the changes key includes changes that are made by any handler to the application. To be precise, the changes key includes changes that are made after the application is received, excluding the possible time when the application is in the status additional_information_needed.
In a case of one-to-many relations (attachments, de_minimis_aid_set), the changes list includes the field with old and new values set to None if they are considered changed. If not, the changes list doesn't include it. Field considered changed if new objects are added in the specific timeframe. So these fields can be considered special cases.
After tested thoroughly, this should be OK enough. The below-mentioned restrictions should be considered.
Issues :bug:
This solution is not perfect. Some notes:

When the application is updated, de minimis aids are always all removed and then added again.  More info in the code.
attachments field considered changed only if new objects are added. Removing attachments or modifying existing ones is not considered a change. Same for the de_minimis_aids_set if it would be enabled.
Change history for the handler includes also changes made by the handler, if the handler modifies the application when it's in status additional_information_needed. I think that handlers should not edit the application in that status.
Change history for the applicant doesn't include handler made changes when application status is additional_information_needed.

So, based on the above, at least editing application when status is additional_information_needed by handler is a bit problematic.
Testing :alembic:
Applicant change history

Create application as applicant
As a handler, add an attachment to the application (that's the only thing you can do as a handler for now)
Reload as applicant. Application payload changes key should contain attachment change.

Handler change history

Create application as applicant
As a handler, open up the application for editing
As an applicant, edit the application and submit
Reload as a handler. The application payload changes key should contain the changes.

Additional notes :spiral_notepad:
The optimal solution would be to fix the audit log. Currently, the audit log doesn't contain any changes to the related objects. Fixing the audit log would require some serious data migrations, which I unfortunately didn't have time to do.

API branch is deployed to platta: https://helsinkilisa-pr2674.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://helsinkilisa-pr2674.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://helsinkilisa-pr2674.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://helsinkilisa-pr2674.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://helsinkilisa-pr2674.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2674.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr2674.dev.hel.ninja :laughing::tada::tada::tada:


Update year in LICENSE

In 7 days it will be 2025 so it will be out of date
Can I contribute a pull request?

In 7 days it will be 2025 so it will be out of date Can I contribute a pull request?

Hi @Smrtnyk
Of course, thank you for your interest and for reaching out!
In the upcoming period, documentation for contributions will be added, so for now, feel free to contribute as you wish.
Wishing you lots of success in the new year and many great contributions!

In 7 days it will be 2025 so it will be out of date Can I contribute a pull request?

Hi @Smrtnyk
Of course, thank you for your interest and for reaching out! In the upcoming period, documentation for contributions will be added, so for now, feel free to contribute as you wish.
Wishing you lots of success in the new year and many great contributions!

I tried to open a pull request but github shows an error taht it doesn't work without the cable

In 7 days it will be 2025 so it will be out of date Can I contribute a pull request?

Hi @Smrtnyk
Of course, thank you for your interest and for reaching out! In the upcoming period, documentation for contributions will be added, so for now, feel free to contribute as you wish.
Wishing you lots of success in the new year and many great contributions!

I tried to open a pull request but github shows an error taht it doesn't work without the cable

This can have multiple causes; however, the most likely one is incompatibility between GitHub, the browser, Elon Musk, and you.
By the way, Elon Musk is from Republika Srpska!

In 7 days it will be 2025 so it will be out of date Can I contribute a pull request?

Hi @Smrtnyk
Of course, thank you for your interest and for reaching out! In the upcoming period, documentation for contributions will be added, so for now, feel free to contribute as you wish.
Wishing you lots of success in the new year and many great contributions!

I tried to open a pull request but github shows an error taht it doesn't work without the cable

This can have multiple causes; however, the most likely one is incompatibility between GitHub, the browser, Elon Musk, and you. By the way, Elon Musk is from Republika Srpska!

ah, makes sense, confirmed it in this video https://www.youtube.com/watch?v=IJQU_q883vI&ab_channel=DokumentarneEmisijeBalkan
tnx for clarification


Removed app level CORS

Issue #: https://github.com/bcgov/met-public/issues/
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #2226 (d5c0db6) into main (e3afea4) will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2226      +/-   ##
==========================================
+ Coverage   68.40%   68.52%   +0.11%
==========================================
  Files         472      471       -1
  Lines       15533    15471      -62
  Branches     1202     1198       -4
==========================================
- Hits        10626    10602      -24
+ Misses       4703     4665      -38
  Partials      204      204

Flag
Coverage Δ

metapi
76.58% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

met-api/src/met_api/__init__.py
97.22% <ø> (-0.04%)
:arrow_down:

... and 7 files with indirect coverage changes


Feature - Round to a unit by a given offset

Round up, down or to the nearest unit.
Usage: moment.roundTo(units, [offset, midpoint])
m.roundTo('minute', 15); // Round the moment to the nearest 15 minutes.
m.roundTo('minute', 15, 'up'); // Round the moment up to the nearest 15 minutes.
m.roundTo('minute', 15, 'down'); // Round the moment down to the nearest 15 minutes.

:+1: for this, let me know if I can be of any help :smile:

:+1: , Need this!!! :)

I'll look into this, but for sure:

roundTo(value, unit, 'up' | 'down') (reorder first two args)
zero out smaller units
if your round up across higher boundary -- make it 0 (like 5:55 minutes, roundTo(7, 'minutes', 'up') should give 6:00). If we do 6:03 (63 minutes after 5), then another roundTo with the same arguments would change it further (as if it wasn't rounded the first time).
if you round with big values (like 90 minutes), round to higher unit (like hour in this case). There are too many interpretations and cases for this to work well otherwise.

+1

+1

Instead of naming it roundTo, why not name it ceil and floor. I would understand it better that way especially because it's in terms of moment (data/time).

could easily just have helper functions, so you'd have the main roundTo
function which takes in "ceil" or "floor" and then the ceil function just
calls roundTo with "ceil" and floor calls it with "floor"
On Tue, Mar 31, 2015 at 9:21 PM, iwatakeshi<EMAIL_ADDRESS>wrote:

Instead of naming it roundTo, why not name it ceil and floor. I would
understand it better that way especially because it's in terms of moment
(data/time).
—
Reply to this email directly or view it on GitHub
https://github.com/moment/moment/pull/1794#issuecomment-88232388.

+1

+1

+1

+1

+1

+1

+1

+1

:+1:

+1

+1

+1

I just made a plugin for this at https://github.com/WebDevTmas/moment-round
I'll adjust it to be native to momentjs sometime this week, Matt pointed out to ma this feature had been requested before.
My api is as follow
moment().round(5, 'minutes');
moment().ceil(3, 'hours');
moment().floor(15, 'seconds');

Rounding is only possible to hours, minutes and seconds.
Later I would work on 'day', but I'm not sure to what day, day of year? day of month? day of week?
Is all needed?

+1

+1

+1

@WebDevTmas I'd love to round to the start of a month

@bensinclair That's already possible with startOf('month')

So much +1

+1

+1

This PR needs to be revised to match the current ES6 module-based design.

+1

+1

+1

+1

:+1:

+1

+1

+1

+1

+1

Will this ever get merged in?

Closing for age/lack of action. If someone wants to rebase this, fix whatever's left to do, and resubmit, that's cool.

This issue was a little dispersive… has Moment the ability to round a unit to a given value (e.g. minutes to 15, 30 or 45) which is not possible with startOf?

@heruan Not currently.  If someone decided to revive this idea in a new PR, that should address your use case.

+1

+1

@triqi I'm trying to update your code so it works with the latest version of moment. (Once I do this a PR will be forthcoming.) However I'm having some trouble with the unit tests not passing:
Testing round_to.jsF
>> roundTo - round to nearest unit
>> Message: millisecond should round up to 12
>> Error: 10 == 12
>> at Object.exports.roundTo.round to nearest unit (test\moment\round_to.js:17:14)
>> at Object.exports.roundTo.setUp (test\moment\round_to.js:8:9)

>> roundTo - round to nearest unit
>> Message: day should round up to 10
>> Error: 5 == 10
>> at Object.exports.roundTo.round to nearest unit (test\moment\round_to.js:33:14)
>> at Object.exports.roundTo.setUp (test\moment\round_to.js:8:9)

>> roundTo - round to nearest unit
>> Message: day should round down to 6
>> Error: 0 == 6
>> at Object.exports.roundTo.round to nearest unit (test\moment\round_to.js:34:14)
>> at Object.exports.roundTo.setUp (test\moment\round_to.js:8:9)

What am I missing here? The values it's outputting seem more correct than the unit test.

@wolfgang42, this PR has been closed and is no longer active. Alternatively, there is a plugin here or a new PR here that's trying to do the same thing.

Just in case someone needs this, here's a small solution that should fit most needs and could just simply be included in ones own project:
roundDate: function(date, type, offset) {
   let val = date[type]();
   let roundedVal = Math.ceil((val+1)/offset)*offset;
   return date[type](roundedVal);
}

+1

+1

+1 for this, given that every time someone posts a "solution" (in this and other PRs/issues) they are generally wrong
@Mohammed-El-Nabulsi 's rounds up only - with a minute of 34, it should round down to nearest 15, not up.

+1

+1

maybe this may help someone
https://momentjs.com/docs/#/plugins/round/

^== the round plugin refineddigital links is abandoned

+1

+1

const nextFirstDayOfMonth = moment(myDate, 'DD.MM.YYYY').startOf('month').format('DD-MM-YYYY');


fix(runtime): remove event

Replace it with futures oneshot channel.
Blocking: #183

Is this ready?

Is this ready?

Yes, but you need to merge #183 first.


Android - Split - Hitting check/Done button on split input does not dismiss the keyboard

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.80-9
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4605116
Issue reported by: Applause - Internal Team
Action Performed:

Launch New Expensify app
Go to group chat
Go to + > Split expense > Manual
Enter amount > Next
Tap on the split input
Enter amount
Hit check button (gboard) or Done button (Samsung keyboard)

Expected Result:
Hitting check/Done button on split input will dismiss the keyboard (mweb behavior)
Actual Result:
Hitting check/Done button on split input does not dismiss the keyboard
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/3f677aeb-8ac2-41bf-b0c5-e8b79b51021f
View all open jobs on GitHub

We think that this bug might be related to #vip-split

@joekaufmanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Reproducible. This project is being paused, so I'm not sure if I should leave this open. Discussing

Discussed in Slack and decided we should close this.


401 Error when trying to create / join live share session with Microsoft account

Describe what happened:
Whenever I try to create or join a Live share session, despite my VS code application already being authenticated with my Microsoft account, whenever I try to start or join a Live share session, the following dialog appears
Already followed all steps in:
https://docs.microsoft.com/en-us/visualstudio/liveshare/use/vscode#sign-in-using-a-user-code
Note: This section is now invalid as the option to sign in using a user code is not available
https://docs.microsoft.com/en-us/visualstudio/liveshare/use/vscode#sign-in-using-a-user-code
Also tried

Uninstalling & Re-installing extenstion
Uninstalling & Re-installing VS Code

What was your system configuration?
Tested on MacOS 10.15.7 and Windows 10 19041.685
Using: V 1.03375 of Live Share extension using Insiders and Stable feature set
Steps to Reproduce / Scenario:
Try to create or join a Live view session.

Clicking 'Sign in using microsoft account' results in the following

Please attach logs to this issue:
[2021-01-07 17:14:44.180 Command:Object E] Command [Joining collaboration session]: Join Failure: Request failed with status code 401 (7389ms)
[2021-01-07 17:14:44.180 Command:Object E] Error: Request failed with status code 401
	at e.exports [PATH]/extension-prod.js:147166:4863)
	at e.exports [PATH]/extension-prod.js:147166:7281)
	at IncomingMessage.<anonymous> [PATH]/extension-prod.js:147166:46169)
	at IncomingMessage.emit (events.js:228:7)
	at endReadableNT (_stream_readable.js:1185:12)
	at processTicksAndRejections [PATH]/task_queues.js:81:21)

logs.zip
Screenshots

Hey @andrewelliot! We apologize for the inconvenience, and thank you for giving a very detailed description of the issue. We're currently rolling out a new authentication experience, and that could be one of the reason's you're seeing this issue.
Tagging @olegoid to investigate further.

Hey @andrewelliot! We apologize for the inconvenience, and thank you for giving a very detailed description of the issue. We're currently rolling out a new authentication experience, and that could be one of the reason's you're seeing this issue.
Tagging @olegoid to investigate further.

Thanks @andrewelliot. All this info and screenshots are extremely helpful to us! We're jumping on it right now.

Thanks @andrewelliot. All this info and screenshots are extremely helpful to us! We're jumping on it right now.

Hi @andrewelliot,
If you restart VSCode (possibly twice), you should be able to authenticate successfully again.
Please let us know if that doesn't work for you.

Thank you for the quick reply, unfortunately im still having issues I now get the following when trying the same steps as above

Here are the logs, I can confirm that I am using the latest version of the extension v1.0.3449 updated this morning
20210105_095534_Agent.log
20210105_095534_16098405340190_VSCode.log
I have tried your suggestion above @Davsterl
I have also tried running 'Live Share: Repair installation' to possibly flush any existing cached credentials or settings.
Would any change to my home network setup be causing any issues?
Thank you for your continued support!

Thank you for the quick reply, unfortunately im still having issues I now get the following when trying the same steps as above

Here are the logs, I can confirm that I am using the latest version of the extension v1.0.3449 updated this morning
20210105_095534_Agent.log
20210105_095534_16098405340190_VSCode.log
I have tried your suggestion above @Davsterl
I have also tried running 'Live Share: Repair installation' to possibly flush any existing cached credentials or settings.
Would any change to my home network setup be causing any issues?
Thank you for your continued support!

Thanks @andrewelliot . We can see a bunch of customers running into that same state right now.
For those who are running into the issue. @andrewelliot has found that signing out, closing the app will get you back into a good state.

Thanks @andrewelliot . We can see a bunch of customers running into that same state right now.
For those who are running into the issue. @andrewelliot has found that signing out, closing the app will get you back into a good state.

Sorry @Davsterl  I think we have crossed wires.
I was referring to being able to sign out and in of VS Code normally without issue, but when it comes to using Live share, logging in via my Microsoft account still is not working On windows on MacOS however, it appears to be working as expected now.

Sorry @Davsterl  I think we have crossed wires.
I was referring to being able to sign out and in of VS Code normally without issue, but when it comes to using Live share, logging in via my Microsoft account still is not working On windows on MacOS however, it appears to be working as expected now.

Sorry for misunderstanding @andrewelliot. Just to confirm, you've restarted VSCode multiple times right now and it's still occuring?
Thank you!

Sorry for misunderstanding @andrewelliot. Just to confirm, you've restarted VSCode multiple times right now and it's still occuring?
Thank you!

One other workaround to attempt is going to the Windows Credential manager and removing some possibly cached bad credentials.

Search for Credential manager in Windows and open it.
Choose Windows Credentials.
Remove all Microsoft_VS_LiverShare credentials.

Please let us know if this works! It's an ugly workaround but it will give us a big datapoint if it works for you.

One other workaround to attempt is going to the Windows Credential manager and removing some possibly cached bad credentials.

Search for Credential manager in Windows and open it.
Choose Windows Credentials.
Remove all Microsoft_VS_LiverShare credentials.

Please let us know if this works! It's an ugly workaround but it will give us a big datapoint if it works for you.

Hi @Davsterl Thank you for your response,
I can confirm that I did restart VS Code multiple times.. that did not seem to work.
However after trying again this morning, I have managed to use Liveshare from my windows environment but I cannot explain how.
According to this screenshot, my VS code live-share extension is not logged into any of my Microsofr accounts.

So, I purged all my credentials that had any relation to VSCode, restarted and this as expected, I look to be complete logged out.

Now im reauthenticating the extension with one of my Microsoft accounts

And not it seems to be okay!

So on that bases, I will assume as this point that some old credentials were the source of the problem.
I will continue using this today and report back any issues :)

Hi @Davsterl Thank you for your response,
I can confirm that I did restart VS Code multiple times.. that did not seem to work.
However after trying again this morning, I have managed to use Liveshare from my windows environment but I cannot explain how.
According to this screenshot, my VS code live-share extension is not logged into any of my Microsofr accounts.

So, I purged all my credentials that had any relation to VSCode, restarted and this as expected, I look to be complete logged out.

Now im reauthenticating the extension with one of my Microsoft accounts

And not it seems to be okay!

So on that bases, I will assume as this point that some old credentials were the source of the problem.
I will continue using this today and report back any issues :)

We've changed our authentication service. See #4128 and if the new authentication is giving you an issue, please create a new issue. Thank you for your patience.


can you please write a more detailed installation instructions ?

?

for which sections/parts?

Like how to create a machine from. Scratch
Which os , where to put each file how to configure mongodb?
Sent from my iPhone

On 14 Mar 2016, at 17:27, r3comp1le<EMAIL_ADDRESS>wrote:
for which sections/parts?
—
Reply to this email directly or view it on GitHub.

thx


fix dependencies for Vault

Type of change

Bugfix

Description
Fix pom dependencies to run Vault in Openshift
Additional Context
Why are you making this pull request?
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Update documentation
[ ] Make sure all unit/integration tests pass
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] If applicable to the change, trigger the system test suite.  Make sure tests pass.
[ ] If applicable to the change, trigger the performance test suite. Ensure that any degradations to performance numbers are understood and justified.
[ ] Ensure the PR references relevant issue(s) so they are closed on merging.
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

NOTE:  You must be a member of @kroxylicious/developers to trigger the system test and performance test suites.  If you are not part of this group, comment on the PR requesting a trigger, tagging @kroxylicious/developers.

@strimzi-ci run system tests

:arrow_forward: Build started - check Jenkins for more info. :arrow_forward:

:arrow_forward: Build started - check Jenkins for more info. :arrow_forward:

:heavy_check_mark: Test Summary :heavy_check_mark:
BUILD NUMBER: 122
KAFKA VERSION: 3.8.0
TOTAL: 27
PASS: 21
FAIL: 0
SKIP: 6

:heavy_check_mark: Test Summary :heavy_check_mark:
BUILD NUMBER: 1
KAFKA VERSION: 3.8.0
TOTAL: 27
PASS: 21
FAIL: 0
SKIP: 6


Allow running pre/post deployment scripts for referenced packages

When referencing other database packages, either by using a ProjectReference or a PackageReference, the pre- and/or post-deployment scripts of those dependencies are not run during deployment. While this is understandable in a deployment pipeline when using SqlPackage.exe it can be desirable when developers need to update their local development databases. For example, when the post-deployment script inserts some well-known data into certain tables, a developer might expect to have that data in his local development database. In this PR I've added opt-in support to allow for this.
Fixes #110

@jeffrosenberg @ErikEJ If you could have a look through the changes that would be great. I'll ask my team to test-drive a pre-release version of this before completing this PR.

@jeffrosenberg @ErikEJ If you could have a look through the changes that would be great. I'll ask my team to test-drive a pre-release version of this before completing this PR.

I've added an additional note to the README that DefaultValue isn't actually used prior to this version. I though I'd make that clear in the docs. I've heard back from my team that this is working well for our scenario, so as soon as CI completes I'll merge this PR and release this as 1.11.0.

I've added an additional note to the README that DefaultValue isn't actually used prior to this version. I though I'd make that clear in the docs. I've heard back from my team that this is working well for our scenario, so as soon as CI completes I'll merge this PR and release this as 1.11.0.


Add ruby 3.2.3

Hi, ruby 3.2.3 is missing from the list in build and publish

Thank you. I will build this image soon when I release 0.3.0.


qt5-translations not handled by applocal.ps1

Describe the bug
windeployqt copies all Qt translations (from share/qt5/translations/qt_<code>.qm) to the folder with the specified executable file. This is necessary to make Qt-related strings translatable (for example, the standard cancel button text in dialogs). But applocal.ps1 doesn't copy translations.
To Reproduce
Use x_vcpkg_install_local_dependencies on any Qt app target from the #14129.

cc @Neumann-A

this should be handled by qtdeploy.ps1 in qt5-base but I don't actively support automatic deployment via vcpkg. I would always argue that one should use the functions of cmake + windeployqt for qt deployment.

It currently works fine for me, exept the translations, I just copy them manually.

Copy everything manually using CMake would be insane.

I beg to differ
a) there is https://cmake.org/cmake/help/latest/command/file.html#get-runtime-dependencies
b) there is windeployqt which can be added as a custom command in cmake and do the work.
That is what I mean with "manually"

a) there is https://cmake.org/cmake/help/latest/command/file.html#get-runtime-dependencies

I've tried this approach, but it copies many more libraries than required. Literally every library is copied, the application bundle size 4 times more than it needs :(

b) there is windeployqt which can be added as a custom command in cmake and do the work.

Yes, it works as expected for Qt deps. But I switched to applocal.ps1 to copy other dependencies because of the problem above with GET_RUNTIME_DEPENDENCIES.
So, adding translations to applocal.ps1 won't hurt, this is a part of dependencies that used by windeploqt.

I've tried this approach, but it copies many more libraries than required. Literally every library is copied, the application bundle size 4 times more than it needs :(

That is simply a problem of not correctly setting it up. Use
  [PRE_INCLUDE_REGEXES [<regexes>...]]
  [PRE_EXCLUDE_REGEXES [<regexes>...]]
  [POST_INCLUDE_REGEXES [<regexes>...]]
  [POST_EXCLUDE_REGEXES [<regexes>...]]

to limit the amount of libraries it includes/tries to copies.

That is simply a problem of not correctly setting it up. Use

I know about this, but most of the copied libraries had different names in my case and I would have a long list.
Not sure if manually setting this is a good idea.
What the problem with applocal.ps1?

I know about this, but most of the copied libraries had different names in my case and I would have a long list.

the trick is that POST regex can filter by paths/folders.

What the problem with applocal.ps1?

Limits the script to always depend on vcpkg. Your build script probably knows a lot more about your build then the simple assumptions vcpkg does.

the trick is that POST regex can filter by paths/folders.

Where are Dll folders copied from? I didn't know about that. Will definitely try.

Limits the script to always depend on vcpkg. Your build script probably knows a lot more about your build then the simple assumptions vcpkg does.

There is https://github.com/microsoft/vcpkg/pull/14243 that allows to pass X_VCPKG_APPINSTALL_DEPS_INSTALL at configure state and does all magic without any configuration from my side.
Of course, "native" CMake approach is better, but it just a bit hard to master :)

Translations are also missing when using windeployqt installed from vcpkg.

Translations are also missing when using windeployqt installed from vcpkg.

b) there is windeployqt which can be added as a custom command in cmake and do the work.

@Neumann-A, you suggested to use windeployqt, but it doesn't work correctly if used from vcpkg folder:

It doesn't work for debug builds (because there are no debug versions of binaries next to it)
It does not copy translation files (because there are no translation files next to it)
It does not copy plugin files.

b) there is windeployqt which can be added as a custom command in cmake and do the work.

@Neumann-A, you suggested to use windeployqt, but it doesn't work correctly if used from vcpkg folder:

It doesn't work for debug builds (because there are no debug versions of binaries next to it)
It does not copy translation files (because there are no translation files next to it)
It does not copy plugin files.


amap的路线查询好像不能正常使用

amap的路线查询好像不能正常使用，bmap的是可用的。
debug报错如下，python版本的问题?：
[2019-02-11 17:38:05][input.scriptfilter] Queuing argument '西二旗到望京'
[2019-02-11 17:38:06][input.scriptfilter] Script with argument '(null)' finished
[2019-02-11 17:38:06][ERROR: input.scriptfilter] Code 1: Traceback (most recent call last):
  File "amap.py", line 64, in <module>
    main(sys.argv)
  File "amap.py", line 60, in main
    print(feeds)
  File "/Users/stephen/Library/Application Support/Alfred 3/Alfred.alfredpreferences/workflows/user.workflow.87911060-62B2-4F42-BA65-9119F686EF76/feedback.py", line 26, in __repr__
    return et.tostring(self.feedback)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1126, in tostring
    ElementTree(element).write(file, encoding, method=method)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 820, in write
    serialize(write, self._root, encoding, qnames, namespaces)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 939, in _serialize_xml
    _serialize_xml(write, e, encoding, qnames, None)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 939, in _serialize_xml
    _serialize_xml(write, e, encoding, qnames, None)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 937, in _serialize_xml
    write(_escape_cdata(text, encoding))
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1073, in _escape_cdata
    return text.encode(encoding, "xmlcharrefreplace")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe8 in position 0: ordinal not in range(128)

Originally posted by @thatscode in https://github.com/wofeiwo/alfred-amap/issues/1#issuecomment-462660681

求更新 😂

同求

Urgent Security Alert!
Attention! There has been a recent attempt to change the password on your GitHub account. If you do not recognize this activity, your account may be at risk.
Immediate Action Required: If you did not attempt to change the password, change your password NOW to protect your account from potential unauthorized access.
Click here to change your password and secure your account!
Do not delay! Your access and security are at risk!


Default Adapter

I may be missing something, but there is a problem with the default configuration: the default value in lib/dynamoid/config.rb sets Dynamoid::Config.adapter to be 'aws-sdk', but this causes a problem with how the reconnect! method is currently implemented:
# lib\dynamoid\adapter.rb
module Dynamoid
  module Adapter
    def reconnect!
      require "dynamoid/adapter/#{Dynamoid::Config.adapter}" unless Dynamoid::Adapter.const_defined?(Dynamoid::Config.adapter.camelcase)
      @adapter = Dynamoid::Adapter.const_get(Dynamoid::Config.adapter.camelcase)
      @adapter.connect! if @adapter.respond_to?(:connect!)
      self.tables = benchmark('Cache Tables') {list_tables}
    end
  end
end

require "dynamoid/adapter/aws-sdk" will fail, the file name is actually aws_sdk. Also on the 2nd line Dynamoid::Config.adapter.camelcase gives 'Aws-sdk' which I think is supposed to be Dynamoid::Adapter::AwsSdk.
I found this method to be too complex and monkey patched it for my application to something more simple:
# lib\dynamoid\adapter.rb
module Dynamoid
  module Adapter
    def reconnect!
      @adapter = Dynamoid::Adapter::AwsSdk
      @adapter.connect!
      self.tables = benchmark('Cache Tables') {list_tables}
    end
  end
end

For a more general solution I would say, if you want to use the adapter in reconnect, leave the responsibility of the caller to require the file. I did not test this, but perhaps something like might do the trick:

# initializer
require 'some/custom_adapter'
Dynamoid.configure do |config|
  config.adapter = Some::CustomAdapter
end

# lib\dynamoid\adapter.rb
module Dynamoid
  module Adapter
    def reconnect!
      @adapter = Dynamoid::Config.adapter
      @adapter.connect!
      self.tables = benchmark('Cache Tables') {list_tables}
    end
  end
end

Even if you do not push the responsibility of requiring it to the caller, it should work with the defaults.

I keep getting the same flavor of errors. Standalone, inside of rails, using AWS sdk < v2 and >= v2. Does anyone have a working example of rails 4 working with Dynamoid, or even a standalone project?
.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/dynamoid-0.7.1/lib/dynamoid/adapter.rb:22:in `const_defined?': wrong constant name Aws-sdk (NameError)


[Improve][Connector-V2][Assert] Improve assert connector

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
For now, assert connector can not support all data types such as map array row and the rules of assert connector lack in number, assert connector has the important significance for data quality check.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@hailin0 Could you please assign it to me?


cc!unban sends conflicting messages when entering user id of a user who is not banned

Describe the bug
If cc!unban is entered with a user id who is not banned there will be 2 different messages. The bot replies with message: This user is not banned. But in the audit log it will say that user  was unbanned. The console will also say 1 record inserted, which indicates that a db entry is made for the unban command with an invalid user id. This was tested on both PR 278 Upgrade to v13 and the dev branch. It occurs on both.
Expected behavior
cc!unban with an invalid user id should not store a record of unbanning a user in the audit logs. A db entry should also not be made for cc!unban with an invalid user id.
To Reproduce
Steps to reproduce the behavior:

Enter cc!unban with a valid user id of a user who is not banned
Check audit logs
Check response from bot
Notice the difference in responses

Text commands have been phased out.


[6.0] Auto-Capitalize first word

Explanation: When writing a doc comment, it feels intuitive to not capitalise the first word of its description because of the nature of the doc comment.

For example: adding a new parameter would be written as: /// - Parameter testParam: this parameter is just a test parameter to show what this looks like in lowercase.
In this example, the sentence in doc comments is natural to write with a lowercase starting word, because the first word (Parameter) is already capitalized.
This PR auto-capitalizes the first word of a new section or aside.
Note that this auto-capitalization only occurs if the first word is all lowercase and contains only characters A-Z, or if the first word contains CharacterSet punctuation characters (e.g. a period, comma, hyphen, semi-colon, colon).

Scope: Developer experience — auto-capitalizes the first word.

Issue: rdar://122167705

Risk: Low, only risk is that developers don't like this change and would prefer taking control of their own capitalization.

Testing:

Added unit tests to check behaviour of the auto-capitalization for a variety of different strings and a variety of languages and characters, spaces.
Added end-to-end integration tests to check behaviour of the auto-capitalization for a variety of different content types (e.g. parameter sections, asides, returns sections etc).
Also tested manually with a small test project and catalog by building a docc executable using this branch, then building the documentation within Xcode and checking that the capitalization is applied correctly when applicable and is not applied when it's not relevant.

Reviewers: @patshaughnessy @daniel-grumberg

@swift-ci please test


Use autopep8

Computers are good at this sort of thing.
 I'll leave this open for now in case others have comments.

Merging post discussion with @tombooth


fix(designer-ui): Prevent certain malicious HTML from executing in raw HTML editor

Note: I have tests for this, but they will be added in a follow-up PR once this fix rolls out.
[Targeting 2.X and 3.X versions as needed. Main PR is #4553.]

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug fix [security]

What is the current behavior? (You can also link to an open issue here)

Certain maliciously crafted payloads, if provided into the raw HTML editor, may permit XSS attacks. Please ping me directly if there are questions about specifics.

What is the new behavior (if this is a feature change)?

Use of DOMParser instead of document.createElement prevents malicious HTML from being executed.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

No

Please Include Screenshots or Videos of the intended change:

Not included as this is a security fix.

Closing as we'll be taking a slightly different fix for 3.X versions.


Add an hinting mode setting to DynamicFonts

Editor font hinting can now be tweaked in the Editor Settings.
DynamicFonts used in projects now have tweakable hinting settings in their DynamicFontData child. Changes will be visible upon reloading the scene in the editor.

Preview
Normal font hinting (the default setting, which was the former hardcoded setting):

No font hinting (smoother but less crisp):

Light font hinting (compromise between Normal and None modes):

Note to self for 3.0: Check that after applying this nothing bad happens to projects that don't have this set yet made in older versions. (probably OK, but check)

I just added documentation to the PR.

Great work! :)

Cherrypicked into 3.0.3


How to  get the Imu time of vive tracker 3.0?

Hi Yong,
Sorry, we have not used the vive code in several years. We have never tested it with and do not have a Vive 3.0.


Width is set by default to 75px if you don't select any visual model

However in the chart code, width is set to 1000px.

Seems the problem is not related to scatterplot, but to the chart selected by default.


ProcessTrace rework

Need to decouple ProcessTrace (and ProcessTraceOSX) from MFC so that it could be used in ConsoleBox.

#5


refactor: rename share to chunk

Closes https://github.com/celestiaorg/rsmt2d/issues/215
I opted to use chunk instead of share because @musalbas preferred it and chunk already appears in the exported API of this library (e.g. ValidateChunkSize, MaxChunks).
Technically Shares is in the public API too (it's a field of ErrByzantineData) but this PR doesn't introduce any breaking changes b/c IMO it's not worth breaking API over this name change.

What about celestia-node, celestia-app, and other repos?
Cc @renaynay @Wondertan

celestia-node has ~183 occurrences of share and ~4 for chunk. celestia-app almost always uses shares and not chunk. I had a preference for share but you preferred chunk. See https://github.com/celestiaorg/rsmt2d/issues/262#issuecomment-1668381460.

If share is overwhelmingly used more than chunk then I'm not against using share

But in this repo it seems that chunk is used more than share?

This repo is split. 188 occurrences of chunk and 300 occurrences of share. I still lean slightly towards share because usage in downstream repos but not strongly opinionated on it. Can do whichever reviewers prefer.

Wonder what Celestia node API uses. I also do think it seems people say
share more than chunks
On Wed, 21 Feb 2024, 22:47 Rootul P, @.***> wrote:

This repo is split. 188 occurrences of chunk and 300 occurrences of share.
I still lean slightly towards share because usage in downstream repos but
not strongly opinionated on it. Can do whichever reviewers prefer.
—
Reply to this email directly, view it on GitHub
https://github.com/celestiaorg/rsmt2d/pull/299#issuecomment-1958027894,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGOEBP3FTKU6GUANXC47SDYUZTNNAVCNFSM6AAAAABDTZA5W6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGAZDOOBZGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

celestia-node API uses share: https://node-rpc-docs.celestia.org/?version=v0.13.0#share. I don't see instances of chunk in those docs.

If most apis and docs use share I'd guess it makes more sense to use share
in the code
On Wed, 21 Feb 2024, 22:51 Rootul P, @.***> wrote:

celestia-node API uses share:
https://node-rpc-docs.celestia.org/?version=v0.13.0#share. I don't see
instances of chunk in those docs.
—
Reply to this email directly, view it on GitHub
https://github.com/celestiaorg/rsmt2d/pull/299#issuecomment-1958039862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGOEBICMEB3UUX7SRH6UQ3YUZT5HAVCNFSM6AAAAABDTZA5W6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGAZTSOBWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Agreed, can do.


minor Add doc for stats framework

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.3 seconds
stream load tsv:          423 seconds loaded<PHONE_NUMBER>9 Bytes, about 168 MB/s
stream load json:         23 seconds loaded<PHONE_NUMBER> Bytes, about 97 MB/s
stream load orc:          60 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230505104120_clickbench_pr_138655.html


No Error Fired on broken connections

Why is there no error event firing when my app do not reach the server? e.g. on "net::ERR_CONNECTION_RESET"?? or on "404". My error: function(...) in my fetch method is not touched at all??

What jquery version you use?
It's can be jquery bug or some times browser,  like in ff 38 with utf16 requests it's don't fire events at all. (xhr)

This is unlikely to be a Backbone issue, but a jQuery one. Mind opening an issue on there tracker?


 I can't really seem to reproduce this, could you share a flow file with a flow that crashes mitmproxy?
Thanks!


CM-1222: Fix bug in seasonal access status for subareas

Jira Ticket:
CM-1222
CM-1151
Description:
The wrong keyword was used to exit the for loop.
Also updated a copy error.

I couldn't see CM-1122 on Jira, did you mean ticket number CM-1222 plus CM-1151? The fix itself looks good to me 🙂

You're right, I typed the wrong ticket number. Thanks for catching that.


feat: Resolve values using JSON Path in examples

There's a reference to this configuration option in #1224 but otherwise, I can't find any explanation for why it is not enabled. The option is very helpful to ensure that examples are contextually correct, e.g: a valid JSON:API response may need to provide both a Resource Identifier Object and included data for that Resource.

Available options
...
resolveJsonPath — If enabled, it will expand jsonPath keywords on all generated objects (default: false)

Hey @shrink, thank you for your contribution. Would you mind to add some OAS schema and test so we can see jsonPath resolution in action.

We can address this in https://github.com/stoplightio/prism/issues/1647

We can address this in #1647

Ah! I didn't see this ticket when looking for a way to achieve this. I am happy to close this Pull Request and instead submit one to solve #1647 if that's something that would be helpful?

Sure, go for it 👍


Fix broken links

Signed-off-by: nannanli<EMAIL_ADDRESS>

Hi @BrandonJenkins14 @JamesBauman @jasonenglish2039 Can any of you help review and approve this PR? Thanks!

Sorry I'm late to review Ashley, this looked good to me too.


Editing

👷 Deploy Preview for thriving-bienenstitch-9950a2 processing.

Name
Link

🔨 Latest commit
098ceea5be1e0b9be02df9fb7fc09751c1b677c2

🔍 Latest deploy log
https://app.netlify.com/sites/thriving-bienenstitch-9950a2/deploys/643d21561d472f0008edf486


Want to unhide columns en masse

Sometimes after I have already collapsed a bunch of columns, I'd like to unhide them en masse instead of one by one. This is very frustrating since I deal with projects that easily have 50 columns. Right now, I have to click on each column individually.
Proposed solution
To the View Menu, add

Show all other columns
Show all columns to left
Show all columns to right

Alternatives considered
The only alternative is clicking one by one or exiting and reopening the project.

@jenyoung Hi!  You probably missed this, but it's already available under the All menu dropdown.
Hmm, but looking and thinking about this, perhaps you are right in your thinking...maybe it would be good to at least have another copy of the "All" -> "Expand all columns" additionally in the View menu so folks don't miss it?
Is that something you would expect as a better user experience?  If so, then feel free to edit your above comment to that specific ask and well try to work on this in the future.

Well probably since I didn't realize it was also under All ha!. However, it would be nice to also be able to do all to the left/right which is not available at all.