Bug in 2.6.1: incomplete SST lat-lon plot

Looks like a bug crept in between 2.6.0 and 2.6.1.

https://portal.nersc.gov/cfs/e3sm/golaz/E3SMv2/v2.LR.historical_0111/e3sm_diags/180x360_aave/model_vs_obs_1985-2014/viewer
Similar figure with 2.6.0

https://portal.nersc.gov/cfs/e3sm/golaz/E3SMv2/v2.LR.historical_0111/e3sm_diags/180x360_aave/model_vs_obs_1955-1984/viewer

It's bad and also interesting. Looks like the SST lat-lon figures are all impacted. I will take a look.

☹ How did this slip through our testing?

I tried to reproduce this figure with one line:
e3sm_diags lat_lon --no_viewer --case_id 'SST_CL_HadISST' --sets 'lat_lon' --run_type 'model_vs_obs' --variables 'SST' --seasons 'ANN' --regions 'global' --regrid_tool 'esmf' --regrid_method 'conservative' --main_title 'SST ANN global' --backend 'mpl' --output_format 'png' --canvas_size_w '1212' --canvas_size_h '1628' --figsize '8.5' '11.0' --dpi '150' --arrows --contour_levels '-1' '0' '1' '3' '6' '9' '12' '15' '18' '20' '22' '24' '26' '28' '29' --test_name '20210528.v2rc3e.piControl.ne30pg2_EC30to60E2r2.chrysalis' --test_colormap 'cet_rainbow.rgb' --ref_name 'HadISST_CL' --reference_name 'HadISST (Climatology)' --reference_colormap 'cet_rainbow.rgb' --diff_title 'Model - Observation' --diff_colormap 'diverging_bwr.rgb' --diff_levels '-5' '-4' '-3' '-2' '-1' '-0.5' '-0.2' '0.2' '0.5' '1' '2' '3' '4' '5' --multiprocessing --num_workers '5' --granulate 'variables' 'seasons' 'plevs' 'regions' --selectors 'sets' 'seasons' --reference_data_path '/global/cfs/cdirs/e3sm/e3sm_diags/obs_for_e3sm_diags/climatology/' --test_data_path '/global/cfs/cdirs/e3sm/e3sm_diags/postprocessed_e3sm_v2_data_for_e3sm_diags/20210528.v2rc3e.piControl.ne30pg2_EC30to60E2r2.chrysalis/climatology/rgr/' --results_dir '/global/cfs/cdirs/e3sm/www/zhang40/tests_e3sm_diags_issue561/'

Very interesting that this incomplete figure issue is only reproduced on cori-haswell but not on login node. It is very strange because e3sm_unified has exactly same version of matplotlib/cartopy for both e3sm_unified_1.6.0_nompi and e3sm_unified_1.6.0_cori-haswell.

Are there preprocessing steps like remapping?  Could it be that the bounds of the remapped version are wrong and that's the reason?  It certainly looks like a plotting problem but it's worth checking the NetCDF data to be sure.

Yeah, notice that the metrics numbers differ. So there is data problem involved. Checking more.

More updates, I tried generated a code snippet trying to find source of the error.
import cdms2

def regrid_to_lower_res(mv1, mv2, regrid_tool, regrid_method):
    """Regrid transient variable toward lower resolution of two variables."""

    axes1 = mv1.getAxisList()
    axes2 = mv2.getAxisList()

    # use nlat to decide data resolution, higher number means higher data
    # resolution. For the difference plot, regrid toward lower resolution
    if len(axes1[1]) <= len(axes2[1]):
        mv_grid = mv1.getGrid()
        mv1_reg = mv1
        mv2_reg = mv2.regrid(
            mv_grid, regridTool=regrid_tool, regridMethod=regrid_method
        )
        mv2_reg.units = mv2.units

    else:
        mv_grid = mv2.getGrid()
        mv2_reg = mv2
        mv1_reg = mv1.regrid(
            mv_grid, regridTool=regrid_tool, regridMethod=regrid_method
        )
        mv1_reg.units = mv1.units

    return mv1_reg, mv2_reg

file1 = '/global/cfs/cdirs/e3sm/www/zhang40/tests_e3sm_diags_issue561/lat_lon/SST_CL_HadISST/HadISST_CL-SST-ANN-global_test.nc'
file2 = '/global/cfs/cdirs/e3sm/www/zhang40/tests_e3sm_diags_issue561/lat_lon/SST_CL_HadISST/HadISST_CL-SST-ANN-global_ref.nc'

var1 = cdms2.open(file1)('SST')
var2 = cdms2.open(file2)('sst')

var1_reg, var2_reg = regrid_to_lower_res(var1, var2, "esmf", "conservative")
diff = var2_reg - var1_reg
outfile = 'new_diff.nc'
fout = cdms2.open(outfile, 'w')
fout.write(diff)
fout.close()

It turned out that the program is complaining about esmf is not:
(e3sm_unified_1.6.0_cori-haswell) chengzhu@nid00807:/global/cfs/cdirs/e3sm/www/zhang40/tests_e3sm_diags_issue561/lat_lon/SST_CL_HadISST> srun python issue_561.py
/global/common/software/e3sm/anaconda_envs/base/envs/e3sm_unified_1.6.0_cori-haswell/lib/python3.9/site-packages/cdms2/avariable.py:1289: Warning:
avariable.regrid: regridTool = 'esmf' but your version does not
seems to be built with esmf, will switch to regridTool = 'libcf'

Running on login node doesn't have this issue. And it seems like ESMF package is not included in cori-haswell.

@chengzhuzhang, ESMF was one of the packages I built with spack and removed from the conda environment on compute nodes.  This needs to be fixed somehow so cdms2 looks for ESMF in the path, ot in the conda environment.  ESMF should be in the path unless the call you are making is re-initializing the path somehow.

This was the kind of thing that we did 5 weeks of testing in the hopes of catching.  I'm really sad this slipped through.

Image checker was supposed to catch this type of image change. But that was only required to run with e3sm_diags 's integration test and complete tests. I think we should do a similar check with e3sm_unified release as well.

Okay, I think I understand.  I didn't realize that cdms2 relied on esmpy.  None of our other packages use esmpy as far as I know.
The approach where we have ESMF as its own program works great if folks are calling ESMF_RegridWeightGen from the command line.  That's what NCO and MPAS-Analysis do.  But cdms2 uses the python bindings so it needs the conda version of ESMF to go with ESMPy.
I think the solution isn't too hard.  I need to not uninstall esmf from the conda environment.  That will mean that there are 2 version of ESMF_RegridWeightGen in the path, but the Spack version should take priority.  I was just getting rid of the conda version as a precaution.
I will need to re-deploy E3SM-Unified on all the machines with Spack (which is luckily only 5) once I fix this.
Sorry for making a fuss.

Image checker was supposed to catch this type of image change. But that was only required to run with e3sm_diags 's integration test and complete tests. I think we should do a similar check with e3sm_unified release as well.

I think some kind of a check like this is important.  I don't know how successful an automated check will be unless you have a reference that uses the same packages as E3SM-Unified.

There is a permission issue:
chengzhu@nid00803:~> source /global/common/software/e3sm/anaconda_envs/load_latest_e3sm_unified_cori-haswell.sh
bash: /global/common/software/e3sm/anaconda_envs/spack/spack_for_mache_1.2.0/share/spack/setup-env.sh: Permission denied
If 'spack' is not a typo you can use command-not-found to lookup the package that contains it, like this:
    cnf spack
craype-hugepages2M(43):ERROR:102: Tcl command execution failed: set             CRAYPE_DIR      $env(CRAYPE_DIR)

Module Deprecation Warning: upgrade to 3.21.3 (new default on 2021-09-21)

I see. I thought it would work before permissions finished updating.  Obviously not. It should be done now.

Global SST figure is now completed :)
Thank you so much for a super quick turnaround.


Update the Readme.md with a valid example

Use the latest library functionality and make it a full example.

Update the Readme.md with a valid example.
[Fix #45]


Python: add dates of the end of the active support period

Fixes #3377

And thanks again @presidento !

I added some paragraph breaks and changed wording slightly.


8.1

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 49.90%. Comparing base (2ca1689) to head (289fc62).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

models/star.go
0.00%
4 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
- Coverage   50.00%   49.90%   -0.10%
==========================================
  Files          26       26
  Lines        2086     2090       +4
==========================================
  Hits         1043     1043
- Misses        999     1003       +4
  Partials       44       44

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


Catalog route refactor

@RackHD/corecommitters @brianparry @geoff-reid @srinia6 @abildw

+1

sorry issue with jenkins slaves... test this please

:+1:

test this please


LSTM: Training - missing file /langdata/radical-stroke.txt

Other case É of é is not in unicharset
Setting unichar properties
Setting properties for script Common
Setting properties for script Latin
Failed to load radical-stroke info from: ../langdata/radical-stroke.txt
Warning: given outputs 105 not equal to unicharset of 113.

@theraysmith I am trying to run the commands given in training tutorial.

the above messages are from basetrain.log.
Does the langdata repo need to be updated for 4.0 alpha?

.

Fixed in https://github.com/tesseract-ocr/langdata/commit/3299c600323a511486fdab58c8e31258c308a7bc.
I'm retesting now. It seems the tutorial works without it, so I imagine the accuracy numbers in the tutorial will come out different.

Thanks, Ray.
On 07-Dec-2016 10:29 PM, "theraysmith"<EMAIL_ADDRESS>wrote:

Fixed in tesseract-ocr/langdata@3299c60
https://github.com/tesseract-ocr/langdata/commit/3299c600323a511486fdab58c8e31258c308a7bc
.
I'm retesting now. It seems the tutorial works without it, so I imagine
the accuracy numbers in the tutorial will come out different.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/542#issuecomment-265505151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_o1Gwtcn4WKaDYuXl83XfkwNT8-mUks5rFuYMgaJpZM4LGOuy
.

I've just updated the numbers in the training tutorial.
It seems to work apart from one thing that needs looking at - it doesn't run the eval  from the trainer.
It doesn't harm the tutorial, but will be required before people start serious training.

Ray,
Please add what are the minimum requirements for doing LSTM training in terms of hardware, software, etc.
I realized after running the process that I needed to build Scrollview.jar. I am not sure whether it is REQUIRED or only optional for those who would like to see visual debugging output. It is not built as part of the regular make install of tesseract and training tools.

It will stop at 5000 iterations, (in about half an hour)

I think that is probably dependent on the hardware used. I did not get any progress for more than one and a half hour - not sure whether it was because I did not have scrollview.jar at that point. I ran it later with 500 iterations.
I think it maybe helpful to have just a single iteration as the first step in tutorial to make sure that the process is working.
Also, the case that I think most people would like to use for LSTM training would be to use Finetuning to add a font to the existing trainingdata. It would be helpful to have a separate page on wiki for it. It would also be great to know how to add training data based on scanned images for typefaces that are not available as fonts. Please add a sample of the linedata box file.
I will try to test 'finetuning' the Hindi traineddata for Sanskrit and post here.

@theraysmith Is unicharambigs used while recognition with LSTM engine?

p;lease help me out :
Config file is optional, continuing...
Failed to read data from: data/langdata/Apex/Apex.config
Failed to read data from: data/langdata/radical-stroke.txt
Error reading radical code table data/langdata/radical-stroke.txt
make: *** [Makefile:293: data/Apex/Apex.traineddata] Error 1

tesseract "data/Apex-ground-truth/eng_64.tif" data/Apex-ground-truth/eng_64 --psm 13 lstm.train

tesseract data/Apex-ground-truth/eng_64.tif data/Apex-ground-truth/eng_64 --psm 13 lstm.train
set -x;
tesseract "data/Apex-ground-truth/eng_44.tif" data/Apex-ground-truth/eng_44 --psm 13 lstm.train
tesseract data/Apex-ground-truth/eng_44.tif data/Apex-ground-truth/eng_44 --psm 13 lstm.train
python3 shuffle.py 0 "data/Apex/all-lstmf"
head -n 90 data/Apex/all-lstmf
tail -n 10 data/Apex/all-lstmf
combine_lang_model
--input_unicharset data/Apex/unicharset
--script_dir data/langdata
--numbers data/Apex/Apex.numbers
--puncs data/Apex/Apex.punc
--words data/Apex/Apex.wordlist
--output_dir data

--lang Apex
Failed to read data from: data/Apex/Apex.wordlist
Failed to read data from: data/Apex/Apex.punc
Failed to read data from: data/Apex/Apex.numbers
Loaded unicharset of size 113 from file data/Apex/unicharset
Setting unichar properties
Other case É of é is not in unicharset
Other case FI of fi is not in unicharset
Setting script properties
Failed to load script unicharset from:data/langdata/Latin.unicharset
Warning: properties incomplete for index 3 = C
Warning: properties incomplete for index 4 = H
Warning: properties incomplete for index 5 = E
Warning: properties incomplete for index 6 = S
Warning: properties incomplete for index 7 = -
Warning: properties incomplete for index 8 = R
Warning: properties incomplete for index 9 = I
Warning: properties incomplete for index 10 = K

tesseract "data/Apex-ground-truth/eng_64.tif" data/Apex-ground-truth/eng_64 --psm 13 lstm.train

tesseract data/Apex-ground-truth/eng_64.tif data/Apex-ground-truth/eng_64 --psm 13 lstm.train
set -x;
tesseract "data/Apex-ground-truth/eng_44.tif" data/Apex-ground-truth/eng_44 --psm 13 lstm.train

tesseract data/Apex-ground-truth/eng_44.tif data/Apex-ground-truth/eng_44 --psm 13 lstm.train
python3 shuffle.py 0 "data/Apex/all-lstmf"

head -n 90 data/Apex/all-lstmf

tail -n 10 data/Apex/all-lstmf
combine_lang_model
--input_unicharset data/Apex/unicharset
--script_dir data/langdata
--numbers data/Apex/Apex.numbers
--puncs data/Apex/Apex.punc
--words data/Apex/Apex.wordlist
--output_dir data
--lang Apex
Failed to read data from: data/Apex/Apex.wordlist
Failed to read data from: data/Apex/Apex.punc
Failed to read data from: data/Apex/Apex.numbers
Loaded unicharset of size 113 from file data/Apex/unicharset
Setting unichar properties
Other case É of é is not in unicharset
Other case FI of fi is not in unicharset
Setting script properties
Failed to load script unicharset from:data/langdata/Latin.unicharset
Warning: properties incomplete for index 3 = C
Warning: properties incomplete for index 4 = H
Warning: properties incomplete for index 5 = E
Warning: properties incomplete for index 6 = S
Warning: properties incomplete for index 7 = -
Warning: properties incomplete for index 8 = R
Warning: properties incomplete for index 9 = I
Warning: properties incomplete for index 10 = K

did you find the solution

p;lease help me out :
Config file is optional, continuing... Failed to read data from: data/langdata/Apex/Apex.config Failed to read data from: data/langdata/radical-stroke.txt Error reading radical code table data/langdata/radical-stroke.txt make: *** [Makefile:293: data/Apex/Apex.traineddata] Error 1

Before running make training, please run make tesseract-langdata. It will download the file radical-stroke.txt as well as many SCRIPT.unicharset files into your data/langdata directory.

@nkrot
hello dear , thx for ur suggestion , I did follow it and currently I do have langdata downloaded into tesstrain but once I run this cmd :
make training MODEL_NAME=Apex TESSDATA=tessdata_best MAX_ITERATIONS=10
I got this error

I'd greatly appreciate ur help alot

@manuthvann216
I cannot give you an easy answer to your question. I am still learning how to train tesseract and I feel like Hercules in Augean stables.
what is in your directory data/Apex-ground-truth/?
*.lstmf files are generated from *.box and *.tiff ( or *.png) files by these lines in Makefile: https://github.com/tesseract-ocr/tesstrain/blob/main/Makefile#L250-L263
%.lstmf: %.tif %.box
	tesseract "$<" $* --psm $(PSM) lstm.train

for each pair of box and image files there will be one *.lstmf file. PSM is set to 13 (as can be seen above in Makefile). The file lstm.train is probably this file: https://github.com/tesseract-ocr/tessconfigs/blob/main/configs/lstm.train


[Bug] webview panel

Hello,
Not sure if this is a bug, or part of the app, but it feels a bit strange at first to see a "Safari-esque" panel open when pressing command option + i / I know the app is build on CodeMirror and uses native Webview, but maybe this trigger can be muted, to blend in? I am probably nitpicking :)
Thanks, love the app.

It doesn’t matter at all, I’m really nitpicking. I’m just putting myself in the shoes of someone triggering it by accident and being really confused and or surprised by this, and wondered if it was by design or not. Now I know! Thanks!


python2Packages.rednose: fix build

Motivation for this change
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

I don't why, but python2.pkgs.rednose build just fine on staging.

Actually, why is this even needed? python2.pkgs.rednose also builds on master.

@dotlambda I forgot to mention, it's failing on darwin

In that case, please only disable the tests on Darwin and amend the commit message.

Fixed, thanks for your patience

@marsam I assume you've tested the functionality of the package on darwin (as it can't be confirmed anymore with disabled tests), right? In that case I'm :+1: for merging :)

yes! btw the tests pass, just get stuck in the tearDown methods, specifically in the reload(sys)

@marsam thanks!


[BUG]: Changing the title of Website

The title of the Website must be changed to Foodie from Food Ordering Website.
Here is a screenshot of how it looks-

Hey PA @khushi-joshi-05 ,
I can solve this issue under GSSOC'24, if you assign me this.
Thank you!

I can work on it , can you assign it to me please
under GSSOC'24

@sunny0625
please assign me the issue


Option to disable registering custom SPARQL aggregation functions

Option (ontop.registerCustomSPARQLAggregateFunctions) is included in OntopKGQuerySettings as it is accessed in RDF4JQueryTranslatorImpl.
When option is set to false (default true), custom aggregation functions (STDEV, VARIANCE...) will no longer be registered. This will make SPARQL queries using any of these functions fail during parsing.

Thanks!


Robust attestation

This PR contains a more fault tolerant way to do attestation. The remote attestation code runs in a loop in a separate thread in order to constantly check for unattested machines. This is done through the use of an accumulator variable -- every time a new enclave is started, the accumulator variable's state is changed to indicate that attestation needs to be re-run.
One caveat is that this attestation design relies on Spark's fault tolerance design. If a task attempts to run on an executor with an unattested enclave, the code will throw an exception. Therefore, the spark.task.maxFailures must be set to a number greater than 1 so that Spark will rerun the task. Currently this configuration variable is set to 10 for the tests.

Found another alternative solution that has a correct wait:
  def wait(sc: SparkContext): Unit = {
    if (!sc.isLocal) {
      numExecutors = sc.getConf.getInt("spark.executor.instances", -1)
      val rdd = sc.parallelize(Seq.fill(numExecutors) { () }, numExecutors)
      while (
        (rdd
          .mapPartitions { (_) =>
            val id = SparkEnv.get.executorId
            Iterator(id)
          }
          .collect
          .toSet)
          .size < numExecutors
      ) {}
    }
    logInfo(s"All executors have started, numExecutors is ${numExecutors}")
  }


check-chrony plugin with same api as check-ntp

@Xiaohua-Shen could u please review this PR.

LGTM


Akshita Gupta

@eddiejaoude please have a look

Please read the documentation in the README.md https://github.com/EddieJaoudeCommunity/awesome-github-profiles#to-add-your-github-profile

@eddiejaoude i am very sorry, i will follow the rules and give issue.
Thankyou for guiding.

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one. Nice Profile btw 🚀

Thanks!


fix up ,eureka client renew request is error#569

Ⅰ. Describe what this PR did
if user / to splicing instanceId,the renew request will to ***/name/ip:port, but should to ***/(name/ip:port) ,change the instanceId to name:ip:port
Ⅱ. Does this pull request fix one issue?

fixes #569
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #571 into develop will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             develop     #571   +/-   ##
==========================================
  Coverage      31.65%   31.65%
  Complexity       765      765
==========================================
  Files            200      200
  Lines           7916     7916
  Branches         937      937
==========================================
  Hits            2506     2506
  Misses          5153     5153
  Partials         257      257

Impacted Files
Coverage Δ
Complexity Δ

.../discovery/registry/EurekaRegistryServiceImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


fix unsafe acces on arm

see https://github.com/docker/docker/pull/28285 for context.
fixes https://github.com/docker/docker/issues/28284

ping @tonistiigi

Tested swarm working properly in arm with this vendored into docker. LGTM

LGTM


Can't download wasmd

I followed the instructions to download wasmd from this tutorial (https://docs.cosmwasm.com/docs/getting-started/installation) but when i run 'wasmd version' i get the following error: wasmd: command not found.
Im running this in wsl with Ubuntu and followed all the previous steps from the tutorial mentioned before. I also tried making the same in dev instead of checking out to v0.27.0 but i get the same results.

Hey @MateoGargano, check this issue to understand how to fix it https://github.com/CosmWasm/wasmd/issues/821


[ROCm] Fix for ROCM CSB breakage - 200930

The following commit introduces a new subtest that is failing on the ROCm platform
https://github.com/tensorflow/tensorflow/commit/326a4b419c07141eeb82a8f1087dfe55fbec9187#diff-d56271856aeccc4bf59c34b80b40bafa
The failure we see is
======================================================================
ERROR: testFoo(False) (__main__.FunctionInlineControlTest)
testFoo(False) (__main__.FunctionInlineControlTest)
decorated(False)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/efb88f6336d9c4a18216fb94287b8d97/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/function_test_gpu.runfiles/absl_py/absl/testing/parameterized.py", line 267, in bound_param_test
    test_method(self, testcase_params)
  File "/root/.cache/bazel/_bazel_root/efb88f6336d9c4a18216fb94287b8d97/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/function_test_gpu.runfiles/org_tensorflow/tensorflow/python/framework/test_util.py", line 1799, in decorated
    return func(self, *args, **kwargs)
  File "/root/.cache/bazel/_bazel_root/efb88f6336d9c4a18216fb94287b8d97/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/function_test_gpu.runfiles/org_tensorflow/tensorflow/python/framework/function_test.py", line 1654, in testFoo
    self.assertEqual(noinline, Cell.definition.attr["_noinline"].b)
  File "/root/.cache/bazel/_bazel_root/efb88f6336d9c4a18216fb94287b8d97/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/function_test_gpu.runfiles/org_tensorflow/tensorflow/python/framework/function.py", line 327, in definition
    context.add_function(self._c_func.func)
  File "/root/.cache/bazel/_bazel_root/efb88f6336d9c4a18216fb94287b8d97/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/function_test_gpu.runfiles/org_tensorflow/tensorflow/python/eager/context.py", line 2361, in add_function
    context().add_function(fdef)
  File "/root/.cache/bazel/_bazel_root/efb88f6336d9c4a18216fb94287b8d97/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/function_test_gpu.runfiles/org_tensorflow/tensorflow/python/eager/context.py", line 1126, in add_function
    pywrap_tfe.TFE_ContextAddFunction(self._handle, fn)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Attempting to add a duplicate function with name: Cell_CACqFJ4l9ho where the previous and current definitions differ. Previous definiton: signature {
  name: "Cell_CACqFJ4l9ho"
...
...
attr {
  key: "_noinline"
  value {
    b: true
  }
}
...
...
 and current definition: signature {
  name: "Cell_CACqFJ4l9ho"
...
...
attr {
  key: "_noinline"
  value {
    b: false
  }
}
...
...

The difference in the value of "_noinline" is to be expected.
I do not understand enough about the testcase / underlying functionality to do root cause analysis here.
This error does not seem to be ROCM specific, but it is causing ROCm CSB to fail, and hence need to skip it for ROCm.

/cc @cheshire @chsigg @nvining-work

@cheshire @chsigg gentle ping


WebWindow in kiosk mode

Please consider kiosk style applications via a simple WebWindowOptions.Mode enumerable flag, which makes the window fixed full screen (no close, minimize, maximize, restore) title or menu + top-most; and can be ran as a single-app kiosk in windows 10 https://docs.microsoft.com/en-us/windows/configuration/kiosk-single-app or similar on other platforms.

+1
If this is ever to be taken seriously, we need to be able to disable the right click menu; the touch gestures, etc.

@616E64726173 I believe these should be raised as a different issue; IMHO disabling touch gestures and mouse click would be done on a case-by-case basis; some kiosk application may require these i.e.. mapping pan/zoom/area selection etc.  I'm of the understanding that touch gestures map to mouse operations?
AFAIW right click menu can be done already intercepting the mouse click event.

We able to disable most of these events in Javascript, yes, but it's going to differ per renderer.
Ideally we would have some kind of configuration object, per 'window', that determines the functionality scope on the invocation level in a manner that is reliably cross-platform.
I agree, maybe this should be a separate issue but it is related to the Kiosk Mode since what determines 'kiosk' is subjective - disabling panning, disabling exiting, minimization, etc should all configurable rather than just "KioskMode = true". Ideally we would store these flags inside of enums with a "KioskMode" where all the flags are set to true.
Naturally there are much more pressing issues such as having multiple windows open at once and the ability to programmatically close them from C# which right now is my biggest blocker from using it since you can't close a window from within javascript. (e.g a login screen that prompts the user to login, and closes upon success.)
I am just glad that this project is receiving any attention at all because it makes 'electron' style applications much more viable in NETCore without the need to bundle 2 tons of cement to ship a calculator.

Thanks for the suggestion! I'd be happy to consider accepting a PR that adds this functionality.


Fix cases where request is undefined for user passed 'state' checks

Fix cases where request is undefined for user passed 'state' checks before passing it ServerRequestParameters

#691

lgtm

I'm having the same issue today with following error
Error in logging in:  ClientAuthError: endpoints_resolution_error: Error: could not resolve endpoints. Please check network and try again. Detail: ClientConfigurationError: untrusted_authority: The provided authority is not a trusted authority. Please include this authority in the knownAuthorities config parameter.
at t [as constructor] (https://xxxx/static/js/2.8c752529.chunk.js:2:136326)
at new t (https://xxxxx/static/js/2.8c752529.chunk.js:2:103927)
at Function.t.createEndpointDiscoveryIncompleteError (https://xxxxx/static/js/2.8c752529.chunk.js:2:104619)
at Function. (https://xxxxx/static/js/2.8c752529.chunk.js:2:1872752)
at https://xxxxx/static/js/2.8c752529.chunk.js:2:62868
at Object.throw (https://xxxx/static/js/2.8c752529.chunk.js:2:62973)
at s (https://etdcorestrdevne.z16.web.core.windows.net/static/js/2.8c752529.chunk.js:2:61762)
It was working until yesterday but today when we deploy the build to server, starting getting above error. from localhost its still working fine.


Connect to the Anilist account with the API

Hi!
I would like to know if it is possible to connect your anilist
account to this api, in order to view your souls in progress / concluded.
Thanks so much

Hey, I don't think this is possible right now. OAuth is not supported at the moment, only graphQL


Add support for aufs to graphstorage.

This adds support for aufs as a graphstorage implementation. This
enables Kurma to work on Ubuntu 14.04 and 12.04.
The container stager has also been updated to default to automatically
detecting which driver to use, if one isn't explicitly defined.
A few updates have been made to the Ubuntu packer definitions for Ubuntu
14.04 and 12.04. On both, the cgroup-lite package was installed to
ensure all cgroups mounts are available on boot. Additionally, the 12.04
defintion will install the 3.13 kernel from Ubuntu trusty. By default,
12.04 used the 3.2 kernel, which is older than kernels typically
recommended for container workloads for full and stable functionality.
Fixes #33.
@alextoombs @mbhinder

Mostly added nits, LGTM


toString() in original object and toString in proxy are different

In Proxy object the toString() method returns something different than original object does.

Hi,
Thank you for this PR! Valid point 👍
I don't see any changes to production code. Is this intended or you have not pushed everything? Currently I see only a failing test added to RuleProxyTest.
For the fix, I suggest doing the same as the compareTo method: we should see if there is already a toString method. If there is one, we should return it, otherwise use the rule name.
Kr
Mahmoud

I merged the failing test and fixed it. I also added you to the contributors list.
Thank you for your time.

Thanks!!!
I only provided the test, because this weeks I was really busy to fix it.
Sometimes I only have time to hunt the bugs and prove it with a unit test,
and when I have more time I fix them.
Thanks for adding me in contributors list I hope find and hunt more issues
and bugs.
regards
Daniel Rivero Capellán
PS, Sorry for my english
2018-04-08 13:06 GMT+02:00 Mahmoud Ben Hassine<EMAIL_ADDRESS>
I merged the failing test and fixed it. I also added you to the
contributors list.
Thank you for your time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/j-easy/easy-rules/pull/144#issuecomment-379541531,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT9oyi3_SIKGozLp7WScRLE4Y_-X_Uynks5tme8_gaJpZM4TCi3c
.

--
Es duro caer, pero es peor no haber intentado nunca subir.


Plugin should also support split diff mode

The plugin currently only supports unified diff mode. Also only unmodified and added lines are being annotated.
We should also support:

[ ] annotate deleted lines
[ ] support split diff mode

done by #9


Publish code on GitHub

The goal is to publish OntoME source code on github.com/geovistory/ontome.

Example:
https://github.com/geovistory/design-system/blob/main/LICENSE


chore: log version when running ucc-gen

This can be useful for debug purposes.

:tada: This PR is included in version 5.9.0 :tada:
The release is available on:

v5.9.0
GitHub release

Your semantic-release bot :package::rocket:


Update missing e2e changes

This PR is for backporting some changes for the e2e tests from midstream. No longer using go get to fetch the applifecycle-backend-e2e repo.

/lgtm


#243: Rename pgtde_is_encrypted() to pg_tde_is_encrypted()

See: https://github.com/Percona-Lab/pg_tde/issues/243

Happy to help!

It needs a couple of more changes in tests' expected files. See the regression.diff here: https://github.com/Percona-Lab/pg_tde/actions/runs/10025861865/artifacts/1725344792

It needs a couple of more changes in tests' expected files. See the regression.diff here: https://github.com/Percona-Lab/pg_tde/actions/runs/10025861865/artifacts/1725344792

@dAdAbird can you please tell me what changes need to be made? It's not really obvious to me, from the diff, what the problem is. Thanks.

@jeffreydwalter The dashed lines need one more dash because now the function name is one character longer. It's more visible if you add a space manually to the diff - without that, it's difficult to see because the diff also uses the - as a marker:
 SELECT pg_tde_is_encrypted('test_norm');
  pg_tde_is_encrypted
-  --------------------
+  ---------------------

You can also execute the tests locally and use the test runner to generate the expected files, that prevents issues like this.

@jeffreydwalter The dashed lines need one more dash because now the function name is one character longer. It's more visible if you add a space manually to the diff - without that, it's difficult to see because the diff also uses the - as a marker:
 SELECT pg_tde_is_encrypted('test_norm');
  pg_tde_is_encrypted
-  --------------------
+  ---------------------

You can also execute the tests locally and use the test runner to generate the expected files, that prevents issues like this.

Ah, I opened the diff in a text editor, so it wasn't obvious because the - blended into the line. 😆

Anyway, I think I get why you guys do testing that way, but surely there's got to be a better way... Thanks for your help!

Yeah, those alignment things might be quite annoying. But renaming or similar changes concerning existing tests shouldn't happen much...
Anyway, thanks for your contribution


Distinguish between "no match" and "match with no groups"

I could be wrong, but it seems to me that there is no distinction between "no match":
https://github.com/cucumber/cucumber/blob/662b3662422c5301f685999704d9dd38e6341757/cucumber-expressions/go/regular_expression_test.go#L45-L47
And when there is a match with no groups:
https://github.com/cucumber/cucumber/blob/662b3662422c5301f685999704d9dd38e6341757/cucumber-expressions/go/regular_expression_test.go#L61-L63
The other implementations return null for the former and [] for the latter. How will Cucumber know which case it is if we always get []?

For the first test we should check if nil, and the second could check not nil built is empty.
require.Nil(t, Match(t, "hello", "world"))

match := Match(t, `Across the line\(s\)`, "Across the line(s)")
require.NotNil(t, match)
require.Empty(t, match)

I'll try adding this today

I think Empty passes with nil and empty arrays


It's too dangerous to use too  much hardcode to judge the direction of scrollView

As you can see,in theSwipViewController's changeButtonColor method,there are too much hard code to judge the direction of the scrollView,there are many if else things,and all of these if else do not cover all scope,you just consider the scope from -374 to 374,so how it work on iPhone 6 plus,iPhone 6 plus's screen width is 414,it is out of the scope you considered.
I think this issue maybe has a better solution,like this https://stackoverflow.com/questions/2543670/finding-the-direction-of-scrolling-in-a-uiscrollview
How about you?

The thing is just the direction won't do it since I need to change the color exactly halfway through the animation.
But it's true that the values are hardcoded, kind of didn't struck me to support there other sizes (silly me - I really should have thought of it).  Will look into how to change that, thanks for the suggestion.

Yeah,this solution is a bit verbose,because you should write code for different size screen,I think the stackoverflow solution is better because you can write code once,but for all size of screen it can work well.
I'm working on a simuliar project,it also has a simuliar animation,but the project only has few customize feature
https://github.com/cielpy/CPYPagerViewController

New 0.1.1 release uses more abstract values to work with all screen sizes. I might do some more refactoring in the future but this version should do for now 👍

Thanks
On Tuesday, 29 March 2016, Marek Fořt<EMAIL_ADDRESS>wrote:

New 0.1.1 release uses more abstract values to work with all screen sizes.
I might do some more refactoring in the future but this version should do
for now 👍
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/fortmarek/SwipeViewController/issues/5#issuecomment-202487785


sources (draft)

hinoki should take all its factories from a source.
sources are where factories come from.
sources are simply functions which take a string and return either a factory function or null.
sources can be made from objects (to look up names in an object):
hinoki.sourceFromObject(object);

multiple sources can be merged into a single source:
hinoki.mergeSources([source, source]);

sources compose !
sources can be decorated:

to rewrite names for namespacing and aliasing.
to wrap factories (for tracing).
sources can be used to implement factory-arrays and factory-objects #3:
as a modified sourceFromObject.
by collecting all the dependency names on factories which are objects or arrays on decoration time (for performance).
by returning a fake factory that depends on all those names and correctly injects the object or array when called.
hinoki.sourceFromDir

this is now present in master and released > 1.0.0-rc.1


Rely on path env resolve

No longer scans the following

%UserProfile%\scoop\apps\beyondcompare\current\
%UserProfile%\scoop\apps\diffinity\current\
%ChocolateyToolsLocation%\neovim\*\
%UserProfile%\scoop\apps\rider\current\IDE\bin\
%UserProfile%\scoop\apps\vscode\current\

Instead add the directory for the desired app to the %PATH% environment variable

@oskar FYI

It took me some time to figure this out since it reverted #237 but that was a good call since it's much more maintainable. 👍🏻

@0xced sorry it caused u some friction


add small improvement to contract, test & event listener

This PR includes the following features:
Smart Contract level:

Highlight the difference between the fallback and receive methods. Remove compilation warning regarding missing receive function
Added extra events to track on the UI
Fix unit test on myTest.js

Frontend:

EventListener.js: added support to track multiple events
UI updates to track multiple contract events

Going to close this due to age. There is a reason the MyContract is basic, this allows for the developer to learn solidity while he builds his/her project.


Caret position is wrong after select a candidate from auto-complete for Powershell New-object cmdlet

Here are the steps to repro.

Launch VSCode
Change the mode to Powershell
Type  $obj = New-Object ""
Move the caret between double quotations.
Type system.col
Auto-complete shows Collections (For System.Collections)
Choose it

Expected:
"Collections" will be inserted and the caret will be moved just before the second double quotation.
Actual:
"Collections" is inserted but the caret remains the same position.
Note:
VS Version 0.10.10

@daviwil is this a problem in the PowerShell extension or in VS Code? If the former I can move this issue over.

That depends on what VS Code uses to determine where the caret goes after the completion.  I'm able to reproduce this issue also.  Nothing has changed in our completion provider in the last couple PowerShell extension updates.
Is it possible that the semantics of CompletionItem.textEdit have changed in this release?

@daviwil I just looked into this and the edits I get in the editor:
The file contents are (the | denotes the cursor):
$obj = New-Object "system.col|"

And the generated edit from accepting the suggestion is replace "system.col" with "System.Collections"
This makes it that the cursor is inside the replaced range, in which case it stays put.
If the generated edit from accepting the suggestion would be replace system.col with System.Collections <-- notice that double quotes are left untouched then the cursor is at the end of the replaced range, in which case it goes to the end of the new inserted text.
Here is the current behaviour:

And here would be the behaviour if the suggestions would not contain the leading and trailing double quotes:

Thanks for looking into this, Alex!  Even if we removed the quotation marks aroung the completion text, the problem still remains that the cursor is not being placed after the quoted string (which is the behavior in the PowerShell ISE:

Since we're replacing a whole substring, is there any way we can have the caret be placed just after the completed substring?  Could we possibly add some way to configure the cursor position after the completion?

Awesome, thanks a lot for making that change Alex!  It seems like this behavior should come automatically with no extra work on my side, correct?

Yes, here is the behaviour on master:

It is still not what the OP wanted, but that can only be changed by providing different suggestions.


[BSO] Goldemar doesn't drop boxes anymore

🐛 Bug Report
For quite a bit now, there have been no boxes in the Goldemar mass channel. Not sure if they were accidentally removed or not, since theres been no mention on a nerf.

pls fix. 4k+ dry at goldemar is bad enough but at least the boxes made doing the masses /slightly/ worthwhile.
A week of nonstop goldemar should yield more than a bunch of hair.

pepeRiot pepeRiot pepeRiot pepeRiot pepeRiot pepeRiot pepeRiot pepeRiot bring back boxes

Fixed in next update


WIP: stats: bins=auto in docstrings

Try improving histograms in the docstrings of distributions.
E.g. in master, https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.lognorm.html there's a single bin; let's see if it's going to get better with bin autoselection.

TBH I'm baffled by the doc build failure. It works ok locally and the change looks harmless, but somehow some distribution (not sure which) hangs CircleCI.

Let's see if merging main helps, since CIrcleCI doesn't do that.

Well that is wild.

Do you plan to dig into this @ev-br?

Thanks for looking into it Matt. Sadly, I won't be able to look into this CircleCI weirdness, in the next few weeks at least.

There was an error in the doc builds that didn't report as an error here on GitHub. It looks different than before, so rerunning in the off-chance that it was a fluke.

@tacaswell Tom, this seemingly harmless change causes our doc build to fail. Do you have any insight into how this might happen?

I wrote some code to try to track this down. I'm pretty sure levy and levy_l are the culprits. I'll see if I can change their parameters to make them nicer.

Sorry, I missed the ping.  This appears to be sorted?

Well I have a workaround, but we had trouble here because it looks like hist with bins='auto' is causing hanging/crashing despite there being only 1000 data points. (Memory hogging, maybe?) Is that to be expected, or should I file a bug report?

Even if there are only 1k points, how many bins are being generated by the auto algorithm?  My guess is that there are a very large number of empty bins and we are handling that badly.
Can you open an issue with the example?  Either it is something we will be able to fix or serve as a marker for why we can not switch to "auto" as the default binning.

Refguide failure looks unrelated.
(scipy.sparse.linalg.svds)

This is an improvement, and CI is finally passing, so let's merge it. Thanks @ev-br!

Refguide failure looks unrelated. (scipy.sparse.linalg.svds)
===================
scipy.sparse.linalg
===================

Non-deprecated objects in __all__: 33
Objects in refguide: 33

scipy.sparse.linalg.svds
------------------------

File "build/testenv/lib/python3.9/site-packages/scipy/sparse/linalg/_eigen/_svds.py", line 239, in svds
Failed example:
    (np.allclose(s3, s) and
     np.allclose(np.abs(u3), np.abs(u.toarray())) and
     np.allclose(np.abs(vT3), np.abs(vT.toarray())))
Expected:
    True
Got:
    False

That hit me too out of the blue while I am working on https://github.com/scipy/scipy/pull/16712 I do not know if it is a coincidence, but I am making changes there to fix this. @WarrenWeckesser


fix(api-gateway): graphql relay connection findAll do not work

Codecov Report

Merging #126 (d93c71b) into develop (f2fcf20) will decrease coverage by 1.63%.
The diff coverage is 58.82%.

@@             Coverage Diff             @@
##           develop     #126      +/-   ##
===========================================
- Coverage    95.16%   93.53%   -1.64%
===========================================
  Files           69       70       +1
  Lines         1096     1144      +48
  Branches       129      145      +16
===========================================
+ Hits          1043     1070      +27
- Misses          53       71      +18
- Partials         0        3       +3

Impacted Files
Coverage Δ

packages/api-gateway/src/graphql-common-utils.ts
55.31% <55.31%> (ø)

packages/api-gateway/src/graphql-relay.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


llvmPackages: Split tablegen into its own derivation

Alternative implementation to https://github.com/NixOS/nixpkgs/pull/359967 (rejected).
Two commits:

llvmPackages.bolt: Use ninja, drop libclang dependency

Minor improvements to bolt build while we're here; could go as its own PR.

llvmPackages: Split tablegen into its own derivation

In addition to the stated change, this conditionalises clang-pseudo-gen which was dropped in LLVM 20.

Goal is to avoid unnecessary rebuilds of LLVM for some cross scenarios.
Background:
LLVM has some tools that run at build time. In native builds, these are
built as a part of the usual build, but in cross builds they need to
come from buildPackages.
In many scenarios this is a small problem because LLVM from
buildPackages is already available as a build; but if cross building a
version of LLVM which is not available (e.g. a new git commit of LLVM)
this results in two builds of LLVM and clang, one native and one for the
cross.
Full builds of LLVM are expensive; and unnecessary in this scenario. We
don't need a native LLVM, only a native copy of the tools which run at
build time. This is only tablegen and related tooling, which are cheap
to build.
Implementation-wise, we introduce a derivation llvmPackages.tblgen,
which specifies the tablegen targets which need to be built and has a
custom installPhase to copy them to the output.
A previous attempt in https://github.com/NixOS/nixpkgs/pull/359967
dropped the use of LLVM_TABLEGEN_EXE and friends on the grounds that
llvm can already cross build these things, but that is false since it's
necessary in that case to wire in a cross compiler. This PR avoids that
problem by allowing access to buildPackages.tblgen.
Signed-off-by: Peter Waller<EMAIL_ADDRESS>
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I like this idea a lot! Surprise it just works without cmake modifications from upstream.

Made it unconditional. For posterity, so we can look back with fond memories:

More making-unconditional-needed

Please can you point to it? The existing open thread did have its one hoisted out of the conditional. https://github.com/NixOS/nixpkgs/compare/c22a2b32967f893aae5f4d20e2ce018f5ad7d7f3..59331c3a174455ff68cdaa959dd98a5fd13a6930


Provide a sensible default for the MemberLogGroup parameter

In the interest of simplifying the deployment of the solution, it would be good to provide a sensible default value for the LogGroupName parameter in the MemberLogGroup class. This would allow deployment to override the value if they choose to, or to accept the default value.

We will make a note of this, thanks for the suggestion!


Ubuntu 18.04: Required and recommended packages are installed: 'php_ver' is undefined

OS: Ubuntu 18.04
Playbook:
---
- name: test
  hosts: all
  remote_user: root
  vars:
    nextcloud_trusted_domain:
      - "my.host.name"
    nextcloud_webroot: "/var/www/nextcloud"
    nextcloud_data_dir: "/srv/nextcloud"
  roles:
      # https://galaxy.ansible.com/aalaesar/install_nextcloud
     - aalaesar.install_nextcloud

Error:

TASK [aalaesar.install_nextcloud : [APACHE] -  Required and recommended packages are installed.] ***
fatal: [fresh1804srv]: FAILED! => {"msg": "'php_ver' is undefined"}
to retry, use: --limit @/etc/ansible/site.retry

Workaround:
  vars:
    php_ver: '7.2'

But than it hangs at:
TASK [aalaesar.install_nextcloud : [APACHE] -  Required and recommended packages are installed.] ***
changed: [fresh1804srv] => (item=apache2)
changed: [fresh1804srv] => (item=smbclient)
changed: [fresh1804srv] => (item=libapache2-mod-php7.2)
changed: [fresh1804srv] => (item=php7.2-ldap)
changed: [fresh1804srv] => (item=php7.2-imap)
changed: [fresh1804srv] => (item=php7.2-gd)
ok: [fresh1804srv] => (item=php7.2-json)
changed: [fresh1804srv] => (item=php7.2-curl)
changed: [fresh1804srv] => (item=php7.2-intl)
failed: [fresh1804srv] (item=php7.2-mcrypt) => {"changed": false, "item": "php7.2-mcrypt", "msg": "No package matching 'php7.2-mcrypt' is available"}

There is no mcrypt in php7.2 anymore

Seems that galaxy is not up to date. When downloading the role directly from github it works fine.

hi @klemens-u . i'll update galaxy soon.
Thank you for your feedback.


Add ADR for RTL support

The title will need an update as seen in fb91cf8 once #105 is merged (incl. an update to storybook 6).


Add a logo

Congratulations on releasing 1.0.0! I'm very much looking forward to finding excuses to use this package.
I'd like to propose a logo. I tried very hard to work with just the shapes available in the Elm tangram, but I just couldn't make it work 😄. Here's what I came up with after some reconciliation:

If the layered nodes is creating too much of a 3D effect, here's a flatter alternative I could work into this PR:

Amazing! I'll merge and add it to the readme. Thank you!


pull request for e2e test at 11/12/2016 06:26:11

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix(types): fix missing type exports

Fixes #1080
This change marks all @algolia/* packages as bundledPackages for algoliasearch, which means that they are treated as part of this package and not external dependencies.
As a result API Extractor will throw an error if all types from @algolia/* packages that are accessible publicly through algoliasearch are not properly re-exported.
This PR is separated into 2 commits:
The 1st one updates API Extractor configuration to mark ae-forgotten-export messages as errors and updates the build script to mark all @algolia/* packages as bundledPackages (for algoliasearch only). This is done dynamically so we do not have to update the configuration if we add/remove some packages.
The 2nd one updates the actual TS files to fix the missing exports.

Hi there, what's the status? Is there something I can help with, maybe?
I use the latest version of the client and I can't access the type for the Settings and TS is bothering me with the queryType specifically.


There is any limitations on team members?

Hi is there any restrictions about the team members, such as quantity, grade?

There is absolutely no restriction.


iOS 14 - Play button disappears resulting in no way to start video playback.

Bug
Once the video preview has loaded, the play button appears for a second to then disappear on iOS 14 beta simulators and devices, resulting in a video preview that cannot be played.
Platform
Which player are you experiencing the problem on:

iOS (all iOS 14 betas)

Environment info

React native info output:
System:
    OS: macOS 10.15.6
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 2.98 GB / 32.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 12.18.2 - ~/.nvm/versions/node/v12.18.2/bin/node
    Yarn: Not Found
    npm: 6.14.5 - ~/.nvm/versions/node/v12.18.2/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.9.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 13.7, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK: Not Found
  IDEs:
    Android Studio: 4.0 AI-193.69<IP_ADDRESS>26763
    Xcode: 11.7/11E801a - /usr/bin/xcodebuild
  Languages:
    Java: 14.0.2 - /usr/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: ^16.11.0 => 16.11.0
    react-native: ^0.62.2 => 0.62.2
  npmGlobalPackages:
    *react-native*: Not Found

Library version: 5.1.0-alpha8 though this happened on non-alpha versions of the library
Steps To Reproduce

Run application on iOS 14 simulator or device.
Does not have to be built using XCode 12 (building with XCode 11.x and installing on an iOS 14 device via TestFlight or App Store yields same result)
...

Expected behaviour

Play button stays and can be interacted with.

Hi there, by chance, Does anyone have an update on this?

To get around this, I had to add a touchable overlay with a play icon which onPress flipped the paused prop on the Video component to true and hid the play button. Once the video playback started the normal controls were displayed and functioned properly.

Update: Looks like it works just fine in 14.2 Beta 1.

@mdcone any chance you've got a snippet of that play button component overlay? I can't get the icon to display as an overlay to the video, not sure where I'm going wrong

To get around this, I had to add a touchable overlay with a play icon which onPress flipped the paused prop on the Video component to true and hid the play button. Once the video playback started the normal controls were displayed and functioned properly.

Can you write example please with play method?

Have anybody idea how it can be fixed?

For anyone that's searching for an implemenation that worked for me, I set the controls to false and toggled the pause/play state using a boolean flag that was toggling using a hook:
  <TouchableOpacity onPress={() => setVideoIsPaused(!videoIsPaused)}>
      <Video
        source={{
          uri: source
        }}
        style={{
          height: 400,
          position: 'relative',
          bottom: 25,
          width: Dimensions.get('window').width,
          zIndex: 0,
          backgroundColor: 'black',
        }}
        resizeMode="cover"
        controls={false}
        paused={videoIsPaused}
        repeat={true}
      />
      <Icon
        type="font-awesome-5"
        name="play"
        size={65}
        style={{
          zIndex: 9999,
          display: videoIsPaused ? 'flex' : 'none',
        }}
        containerStyle={{
          position: 'absolute',
          height: 400,
          bottom: 25,
          left: 0,
          right: 0,
          flexDirection: 'row',
          alignItems: 'center',
          justifyContent: 'center',
          backgroundColor: videoIsPaused ? 'rgba(0,0,0, .5)' : 'transparent',
        }}
        color={'rgba(255, 255, 255, 0.5)'}
      />
    </TouchableOpacity>

Also experiencing this issue.

I had other completely unrelated video issues on iOS 14 that were also resolved in 14.2 Beta as pointed out by @mdcone

@Asalem1, sorry for the late response. Your solution looks great!

Also have the same video issues on iOS 14 and also noticed that they were resolved in 14.2 Beta


special-checks

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: special-checks
Document link: special-checks.md
My Issue/Suggestion
In section 5 Checking Whether an Object Is Synced
The guide says "This function is available only in expressions for conditional visibility or editability, as only they are evaluated on client side."
However, I found that the function does work in a nanoflow called in native, which was very useful. If I hadn't tried it anyway, I wouldn't have known. So it would be worth correcting the guide.

Hi @Chris-Karona,
Thanks for pointing this out!
I'll check this and keep you posted here.
Best,
Yiyun


Fix signals, UserProfiles are now created

Fixed the signals emitted and added a few files. Don't forget to add your name to pyproject.toml.
EDIT: I also added a new app called accounts for User handling. Please review the changes carefully, its quite major and then merge the second commit.

The idea of having a separate app for user handling is quite amazing!  Cheers :tada:


Select firmware drop down sorts incorrectly.

Looks like major version may be ignored.
Also check to make sure we're using numeric sorting not alphabetic sorting.

Fixed in #1c4a62b8088b5813bf3e66d78866616391eafc92. See Digilent/dip-angular2#3646c9bcf55ec231b0c2a9e169171267604672de


pinogra_core::protocols::http get back the underlying stream

What is the problem your feature solves, or the need it fulfills?
Let me start with this: I am very new to pingora and to Rust in general, so forgive me if I am asking obvious questions and making silly mistakes.
Given an existing HTTP session (server/client, h1/h2), I am looking for a universal way to extract the underlying Stream so that I can perform other operations with it. The way I see it, we would consume self and return the underlying stream.
The closest thing that I can see today is the reuse() method which works only for h1 and only if Keepalive is enabled https://github.com/cloudflare/pingora/blob/main/pingora-core/src/protocols/http/v1/server.rs#L806
However, for h2 and non-keepalive h1 there is no way to get ownership back of the underlying stream and do other things with it (in the same app).
Sample use case is implementing a MITM forward proxy server. We start by accepting an HTTP/1.1 CONNECT request over plain TCP, return a 200 and then start a TLS handshake over the same TCP socket. Currently the
http::server::Session::new_http1(stream) takes ownership of the passed stream. In order to continue with a subsequent protocols::ssl::server::handshake() we need ownership of the stream back. Currently the only way to do this is to call the .finish() method, which calls reuse() of the prior http server, but that returns the Stream only if it's h1 and keepalive. What if we want to support H2 for example to the client ?
Describe the solution you'd like
Would it be reasonable, and would you accept a PR, that implements a new method that looks something like this:
pub fn into_stream(self) -> Stream {
    self.underlying_stream
}

Alternatively, perhaps we could have a similar method like protocols::http::server::Session::new_http1() that takes a reference to the the underlying stream rather than ownership. Would that work?
Describe alternatives you've considered
A workaround that I put together is to duplicate/clone the original stream by going all the way down to the raw socket fd, try_clone()ing it and back up to the Stream type but it feels unnecessary and inefficient to go through this so that we have descriptors/streams down to the same kernel socket, only to solve the ownership problem.

I feel like this is a common-ish rust pattern, where you have into_inner(), that then returns the original thing the object was built from. We'd probably want to flush, but I think a reasonable way to make the API is something like:
At least for the HTTP v1 client:

unsafe pub async fn into_inner(self) -> Result<Stream> {
   self.underlying_stream
            .flush()
            .await
            .or_err(WriteError, "flushing body")?;
Ok(self.underlying_stream)
}

In this, since the client / server isn't generecized, you'll have type erasure of the stream (so, if you have a special stream, you'll either have to cast it back and check, or deal with downcasting).
Unsafe, because I think the initial impl of the API should be "experimental".
What's folks think?


Use immutable library for List/Set/Map

Closes #102

Think before a stable release is made, a release of 0.1.0 needs to be made for immutable so that its API is committed to and there are no "surprise" changes associated with an alpha lib.


Route poll creator to result page with unique poll id

Fixes #135
Describe your changes
After creating poll, the user is redirected to the result page: path={'/result/' + pollId}
pollId is unique for the poll, and that can be used when answering the poll as well.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added thorough tests

Codecov Report

Merging #138 (a50baaf) into dev (8935d31) will decrease coverage by 0.02%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##              dev     #138      +/-   ##
==========================================
- Coverage   53.11%   53.09%   -0.03%
==========================================
  Files          65       65
  Lines        2773     2776       +3
  Branches      360      360
==========================================
+ Hits         1473     1474       +1
- Misses       1275     1277       +2
  Partials       25       25

Flag
Coverage Δ

backend
61.93% <ø> (ø)

dev
53.09% <40.00%> (?)

frontend
14.20% <40.00%> (+0.11%)
:arrow_up:

staging
?

unit
53.09% <40.00%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

frontend/src/components/App.tsx
57.14% <ø> (ø)

frontend/src/components/PollCreation.tsx
28.42% <33.33%> (+0.46%)
:arrow_up:

frontend/src/services/pollService.ts
7.82% <50.00%> (-0.07%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Container using a vhost over tcp / access Redis in a container from another container

Hi,
it doesn't seem possible to use tcp connection over a vhost.
Whenever I add the vhost to the configuration the mode changes to http.
I'd like to use Redis in a container and speak to it from another container, for this I would like to add a vhost to be able to find the specific Redis container (there could be multiple Redis containers on the same host). Is there another way to do this?
thanks,
Richard

I'm not sure but I think that if you use virtualhost it's just for http service, if you need to see your service from another container you have to use a DNS service.
Sorry for my english.

VHost are strictly an HTTP specific feature.


Questions about cell type annotation

Hi, Thanks for your great work. I want to use scGPT to do cell type annotation task on my own dataset. Is there any demo code about how to do this task? Thanks

Yes, we are working on wrapping it up and will release notebooks for cell type annotation and other tasks soon in the next week.

Thanks

Hi, wondering any follow-ups on this issue? Thank you for the great work. It would be great to see the release of downstream tasks!

Hi @KatarinaYuan , the cell type annotation tutorial has already been released. Please find it in the tutorial folder and here.


3.4 globalThis support makes incorrect assumptions about let/const bindings

TypeScript Version:  3.4.0-dev.20190316
Search Terms:
3.4 globalThis let const property
Code
// Compile with `tsc -t es2015`
const foo: number = 42;
const bar: null = globalThis.foo; // Type 'number' is not assignable to type 'null';

For reference, the following demonstrates how these bindings behave in browser and node, respectively:
<script>
const foo = 42;
alert(window.foo); // --> "undefined"
</script>

const foo = 42;
console.log(global.foo); // --> "undefined"

Expected behavior:
TypeScript models types consistently with what's actually happening in VMs.  Specifically, variables bound by let or const are not translated to properties on globalThis.
Actual behavior:
TypeScript converts let/const bindings onto properties of globalThis, but these properties do not exist in ES2015+ output (when let and const are retained in the output).
This is a problem because it enables coding patterns that depend on transpilation to ES5 or lower to work correctly.  When the output level is switched to ES2015, these will break at runtime with no compile-time warning.
Playground Link:
N/A: playground does not support 3.4 yet
Related Issues:
None found

Verified, and checked with MDN. Not sure how that is going to translate to our var-declarations in our .d.ts files.

The lib.d.ts file that changes the most is es5.d.ts, with Function, String, et al declare const instead of declare var. Bigint from esnext also changes, and that's it.
After some discussion, I vote for shipping as-is and then adding the restriction for 3.5, along with changes to d.ts.
I don't think being incorrectly permissive is going to cause problems given how permissive global used to be, and the risk of changing Function et al from const to var seems scary to me.


Getting started info

Hi! I was working on a tool to suggest concertina fingerings, and to provide tunes as input an ABC parser would be great, and your package looks like the best option in python. I'm just starting, but I think I may want to suggest a new feature at some point... but before we get there I wanted to document the steps a new developer should take, and check with you about the PR process that you'd like to follow.
Feel free to edit, or bounce back to me for changes, or close if it's not the direction you want.
To get myself oriented to the codebase, would it be ok if I tried to address one of the current issues? ...

#40
#38
#34

Would you like to assign me to one of those? Thanks.

Codecov Report

Merging #41 (54a7419) into main (fbb365e) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #41   +/-   ##
=======================================
  Coverage   89.15%   89.15%
=======================================
  Files           9        9
  Lines        1300     1300
=======================================
  Hits         1159     1159
  Misses        141      141

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thank you for the feedback! I'm going to mark this as draft, fix all the typos, then move the whole section to a new document, as you suggest.

Sounds cool, what type(s) of concertina?

Anglo, so there are different notes on the push and the pull. There are some other projects out there that already do this in different ways, but seems like a tool with a good command line interface would still be useful:

Anglo Concertina Fingering Generator: Web page which takes ABC notation, and returns ABC notation, and renders it using abcjs
concertina-pbqp: C++; Models it as an NP-hard problem, but uses a solver library for an approximate solution.
juliamullen/concertina: Python classes. Melody needs to be written into source code.

I play C/G Anglo a bit, interested to see what you come up with. That PBQP approach is interesting.


chore(react-tinacms-editor): don't insert previewSrc into content body

previewSrc is generated by the nodeView and should not mutate content

@kendallstrautman I refactored this to pass the full imageProps object down to prosemirrorMenu. could use some extra scrutiny.


incorrect mapping between points and shapes in point_features()

Retrieving mapped shapes to points mistakenly uses shapes DataFrame index instead of instance_key column value. Addressed in d80290a9b13bf0690a8f52cf7b9c81c1c649ccb8. Needs test cases.

fixed in #133


docs: fix missing children in theming example

📝 Description
Adds {children}to theming example @ https://chakra-ui.com/docs/theming/component-style#return-value
⛳ Current behavior (updates)
Theming example destructs children in line 222 but doesn't return it
🚀 New behavior
removes destructed children from theming example props
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information

Good catch, @aaarichter! Although, I'd prefer if we remove the children destructuring completely, so it becomes a part of the ...rest spread. Since we're not doing anything specific to the children, destructuring it and manually rendering it seems redundant. So let's remove it. What do you think?

@nikolovlazar feedback addressed.
I wasn't sure which way to write it. Destructing children in this example is redundant but from a understanding point of view a bit clearer (to me)

@allcontributors please add @aaarichter for bugs.


Refactor/data matcher

Coverage increased (+1.02%) to 46.219% when pulling c41cf0cfb824afe4b982dad95d82050dfef3346f on u9520107:refactor/data-matcher into e6c60d77f0209654a66342b520f190ff31e374a3 on ringcentral:master.

Coverage increased (+1.02%) to 46.219% when pulling 5ae799b646be879e6e42b965de45c3aeacc65402 on u9520107:refactor/data-matcher into e6c60d77f0209654a66342b520f190ff31e374a3 on ringcentral:master.

Coverage increased (+1.008%) to 46.207% when pulling 3ebf950145eb79543e2ce2dd59b993b927c0be3f on u9520107:refactor/data-matcher into e6c60d77f0209654a66342b520f190ff31e374a3 on ringcentral:master.

rerun

Coverage increased (+0.5%) to 46.207% when pulling 2aa61d5cca6f9ee1587c2335f3473d0bd5d73dcb on u9520107:refactor/data-matcher into c2c649de88d6b609c3ad26765e63b2fea8b2db0c on ringcentral:master.

Coverage increased (+0.5%) to 47.172% when pulling abfb6064a5ded4d96ce597330f3e698340ff21b6 on u9520107:refactor/data-matcher into bc874cd3b1730a0c6682c45dc321c3f1734f8828 on ringcentral:master.


Publishing config file: Nothing to publish (Laravel 5)

Hi,
when I try to publish the config file with php artisan vendor:publish --provider="Krucas\Notification" --tag="config" I always get Nothing to publish for tag [config]..
Any idea?
Thanks!

Hey, just merged pull request concerning this topic, you should read readme.md again.


Cheap character attention for normalizer and spelling correction

From this example /example/normalizer, you can see some error from the algorithm,
another = 'i mmg xska mknn kampng'
normalizer.normalize(another,debug=True)

'saya memang tak saka makanan kampung'

The problem with saka, you can check the debug,
[('saka', 86), ('spa', 67), ('sika', 86), ('sia', 67), ('seka', 86), ('sua', 67), ('ski', 67), ('suka', 86), ('soka', 86)]

saka, sika, suka have same levenshtein distance from ska, which is 86.
To check contextual meaning for before and after words are very expensive in term of computation and rules-based.
Any idea?

There actually exist other string matching algorithms other than Levenshtein distance.
From Jellyfish library:
https://jellyfish.readthedocs.io/en/latest/comparison.html
Other algorithms might be better or well-performed than the currently used algorithm. I had in the past have a little success with Jaro Winkler algorithm in detecting and removing repeating tweets spam.

Never tested them before, thanks!
I will submit the result here later.

Tested using jellybean, I still prefer to use fuzzywuzzy, which more faster and results totally no different.


Str::wrap() method does not take into account existing start and end of string

Laravel Version: 9.39.0
PHP Version: 8.1
Database Driver & Version: irrelevant

Description:
In https://github.com/laravel/framework/commit/8ea50bc1b7bab7507ca5b9d3761c45cee450b9f0 a new static method Illuminate\Support\Str::wrap($str, $before, $after) was added. This method does not take into account if $str already starts / ends with the given $before and $after sequences.
I mark this as a bug because I believe a number of people already had a Str::wrap() mixin in place that includes these checks and is in line with how the Arr::wrap() method works. The new Str::wrap() method takes precedence over the mixin and changes behavior in existing applications.
Steps To Reproduce:
$this->assertEquals('/some/string/', Str::wrap('some/string', '/')); // ok
$this->assertEquals('/some/string/', Str::wrap('/some/string/', '/')); // should succeed but fails

We unfortunately cannot account for use cases where this method was already introduced.


Introduce ubicloud/cache package

Introduce ubicloud/cache package
The actions/toolkit repository provides a set of packages to make creating actions easier. We've forked this repository and refactored the actions/cache package into the ubicloud/cache package. This updated version can interact with Ubicloud's cache service. We aimed to retain as much of the original code as possible, but differences in the architecture of Azure Blob Storage and S3 compatible storage necessitated some changes.
This commit only modifies branding literals and the package name. Subsequent commits will implement changes to the code.
To stay in sync with GitHub's release cycle for the cache package, I've decided to keep our versioning as similar as possible. The only difference is that I've multiplied the patch version by 100. So, if the original version is 3.2.4, the corresponding version in the
ubicloud/cache package will be 3.2.401. This approach simplifies version tracking and allows us to release multiple build versions for the same version of the original package.
I've also renamed ACTIONS_CACHE_URL to UBICLOUD_CACHE_URL and ACTIONS_RUNTIME_TOKEN to UBICLOUD_RUNTIME_TOKEN. We pass these variables to our runners during environment setup.
Make saveCache Ubicloud compatible
saveCache consists of three main components: reserveCache, uploadFile, and commitCache. We use presigned URLs to enable the cache client to interact with blob storage without passing the blob storage credentials. The design of Azure blob storage and S3 compatible API cache differs in terms of concurrent file uploading. As Ubicloud uses an S3 compatible API, we need to make saveCache compatible with Ubicloud.
To enable concurrent file uploading, we first send a CreateMultipartUpload request to the blob storage. This request returns an UploadId, which we then use to upload the file in parts. Each uploaded part generates an ETag, which is essential for completing the upload. After uploading all parts, we send a CompleteMultipartUpload request, which includes all the ETags of the parts and the UploadId. This finalizes the upload, making the file available in the blob storage.
When the cache client sends a reserveCache request to the Ubicloud cache service, it triggers a CreateMultipartUpload request to the blob storage. This returns the UploadId and a presigned URL for each part to the cache client. The cache client then uses these presigned URLs to upload the file parts to blob storage and collects the ETags from each response. After all parts are uploaded, the cache client sends a commitCache request to the Ubicloud cache service, including the ETags and UploadId. The cache service then sends a CompleteMultipartUpload request to the blob storage to finalize the upload.
Make downloadCacheHttpClientConcurrent Ubicloud compatible
We also use presigned URLs to restore caches from blob storage. The design of Azure blob storage and the S3 compatible API are more similar in terms of file downloading than uploading. Since Ubicloud uses an S3 compatible API, we need to ensure restoreCache is compatible with Ubicloud.
When the restoreCache method of the cache client called, it first sends a getCacheEntry request to the Ubicloud cache service. The response includes archiveLocation, a presigned URL used to download the cache. If this location is a URL from Azure Blob Storage, the download occurs concurrently. Otherwise, it downloads as a single thread.
The current implementation of downloadCacheHttpClientConcurrent is not compatible with the S3 compatible API. The main difference lies in how we obtain the file size. In Azure Blob Storage, we can get the file size by sending a HEAD request to the URL. However, with the S3 compatible API, we need to send a GET request to the URL with a Range: bytes=0-1 header to get the file size. This PR ensures downloadCacheHttpClientConcurrent is compatible with the S3 compatible API.

One thing to watch out for is dangling multipart uploads that failed, they need to get cleaned up somehow (bucket policy?)

There is indeed a bucket policy for that.
        // Example: abort all incomplete multipart uploads after a week
        {
          ID: "Abort Incomplete Multipart Uploads",
          AbortIncompleteMultipartUpload: {
            DaysAfterInitiation: 7,
          },
        },

One thing to watch out for is dangling multipart uploads that failed, they need to get cleaned up somehow (bucket policy?)

There is indeed a bucket policy for that.
        // Example: abort all incomplete multipart uploads after a week
        {
          ID: "Abort Incomplete Multipart Uploads",
          AbortIncompleteMultipartUpload: {
            DaysAfterInitiation: 7,
          },
        },

For R2, the default seems to be 7 days: https://developers.cloudflare.com/r2/objects/multipart-objects/


Fixed RUSTLS on Windows Compilation

Migrated my project to windows and had issues compiling the underlying pingora lib's modifed the compilation of the impl UniqueID for TlsStream and it allowewd me to compile. I am still familiarizing myself with the code base so I am unsure if this is the best way to do this but it worked for me.

I don't think this change is necessary now that #471 / 1756948df77d257bddf7ab798cc3fddf348a91c8 is merged


[Snyk] Upgrade react-leaflet from 2.6.3 to 2.8.0

Snyk has created this PR to upgrade react-leaflet from 2.6.3 to 2.8.0.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 4 months ago, on 2020-10-22.

Release notes

Package name: react-leaflet

2.8.0 - 2020-10-22
Added React v17 to peer dependencies (PR #773 by anajavi).
[website] Added react-esri-leaflet plugin (PR #768 by slutske22).

2.7.0 - 2020-05-06
Added viewBox and preserveAspectRatio props to SVGOverlay component (PR #704 by spectras).
[website] Bump Leaflet docs links to v1.6.0 (PR #686 by moondef).
[website] Set react-leaflet-markercluster v2 support (PR #693 by yuzhva).
[website] Added react-leaflet-kml plugin (PR #707 by aviklai).

2.6.3 - 2020-03-13Fixed regression in v2.6.2 handling maxZoom and minZoom options in Map component.

    from react-leaflet GitHub release notes

Commit messages

Package name: react-leaflet

3c216dc v2.8.0
b478fcd Merge pull request #773 from anajavi/react-17
db43ad9 Allow react 17 in peerDependencies
2edcfa8 Merge pull request #768 from slutske22/patch-1
ba28c2d Update plugins.md
f56a1ab Merge pull request #744 from PaulLeCam/dependabot/npm_and_yarn/website/prismjs-1.21.0
03d6b03 Bump prismjs from 1.17.1 to 1.21.0 in /website
fd68fd5 Merge pull request #723 from chbrandt/patch-1
a0c7659 Add <hr/> lines to mark section limits
ee56ffd v2.7.0
f84e6bc Merge pull request #707 from aviklai/master
978a730 Add viewBox and preserveAspectRatio props to SVGOverlay element (#704)
cc4ccc8 Added react-leaflet-kml plugin
51283ae Merge pull request #693 from yuzhva/docs/plugins
cdb05b1 docs: react-leaflet-markercluster v2 support
c13eead Update dependencies
4eba8f5 Merge pull request #686 from moondef/bump-reference-version
25dccbb bump-reference-version

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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


Add copyright notice for Intel contributed files.

@DrTodd13: The unicode copyright character in parfor.py is making Python 2.7 unhappy due to the lack of a declared encoding and general Python 2.x inconsistencies with handling of unicode.
Can you change the header comment to use the ASCII encoding friendly (c) form of the copyright symbol?

Codecov Report

Merging #2421 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2421      +/-   ##
==========================================
- Coverage   86.19%   86.19%   -0.01%
==========================================
  Files         315      315
  Lines       59761    59762       +1
  Branches     6481     6481
==========================================
- Hits        51513    51510       -3
- Misses       7205     7207       +2
- Partials     1043     1045       +2

Fixed.


Updating Shared Libs is HARD

In order that all developers can easily make updates to shared libraries and be sure of adoption to other projects we should:

Have a mechanism to auto-update & test downstream clients with new library versions
Have an established review mechanism for shared libraries
Have clear documentation on how to build/release and user shared libraries.

See https://github.com/wellcomecollection/docs/tree/master/rfcs/024-library_management for a potential solution.

Closing as dupe of https://github.com/wellcomecollection/platform/issues/4539


Data Integrity Layer ( Uncommitted )

Description
The Idea is to integrate a Frontend for the Data Integrity Layer. So that the Quality User can be assured, that the data has not been tempered with
Impact
This will impact:

Trace-X

Acceptance Criteria

[ ] User is aware if integrity of the data chain is given or not

Additional information

[ ] Integrity is implemented currently only for asBuilt downward direction (SingleLevelBomAsBuilt)

@mkanal is this still valuable ? otherwise please close it. greetings from the NES Committee ;)


Fix classification of application as builtin on Windows

This fixes a bug on Windows, where under certain circumstances an application is misclassified as a builtin, due to case-sensitive path comparison on a case-insensitive filesystem. See #52 for more details.
Closes #52
Proposed fixes

Compute the common path prefix of the module path and the application directory, using os.path.commonpath. Check if this path prefix is the same directory as the application directory, using os.path.samefile.

Handle the case where the module path is on a different drive than the application directory. The os.path.commonpath function raises ValueError if the paths are on different drives.

Use os.path.samefile to check if the real and absolute paths of the module are the same directory.

Tests
The test case uses an application directory named SRC and a sys.path entry of src. This is slightly different from the real world case described in the issue, but suffices to reproduce the issue. The test case is marked xfail on Linux, because this platform typically uses case-sensitive file systems.
Surprisingly (at least to me), the test case also needs to be marked xfail on macOS, even though this platform is traditionally case-insensitive. The reason for this is that os.path.commonpath is case-sensitive on macOS (arguably a bug in CPython?).
There is no test case yet for the branch where module path and application directory are on different drives, because I have not come up with a good way to test this.

Thanks for the review. I'll probably get to this tomorrow morning, CEST.

Replied to your suggestions inline, with links to the commits that address them. The commit messages provide some more detail.
I also made two additional changes:

https://github.com/asottile/aspy.refactor_imports/pull/53/commits/2fe7cb3bebe938a84eec7590d945ea9b487892bb
https://github.com/asottile/aspy.refactor_imports/pull/53/commits/4bad6f0528345dd47921bd2ed11a8190936600fb

These change the PR to compare paths with string equality modulo os.path.normcase, instead of using os.path.samefile. This avoid breaking existing tests, and has some other advantages. See the commit message of https://github.com/asottile/aspy.refactor_imports/pull/53/commits/2fe7cb3bebe938a84eec7590d945ea9b487892bb for details.
Cheers :)


As a group admin I'd like to be able to remove users from my group

Precondition: the user is logged in, the user is a group admin
People: group admin, users to be deleted
Subjects: group

From the menu "user" user chooses "go to groups" and chooses the group with which he/she would like to work by double click or pressing the corresponding button.
User is redirected to a group members tab with the list of the group members.
User can remove one, several or all the users from the chosen group:

to delete one user admin presses a delete button in the line of this user in the table
to delete several users admin ticks the users to be removed and presses delete button on the top of the table
to delete all the users those belong to the group, admin ticks a checkbox on the top of the table and presses delete on the top of the table.

The system responses with a popup that contains the following:

question if the admin really wants to remove the user(s)
name of the user / list of the names of the users
button to confirm and button to abort.

The admin confirms the deletion by pressing the corresponding button.
The system checks if everything is correct and returns:

a message that the user(s) was(were) removed if everything is fine; admin will also immediately see that the user(s) disappeared from the group members list
a popup with the error description in case if user removal cannot be performed. This case is unlikely and can happen only because of some database or server error.

The deleted users will see a popup that informs them about their removal from the group when they log in for the first time after their removal. The group will disappear from their group list as well.
The programs shared in the group are not shared with the removed users anymore. If people want to keep the programs they have to save it to they own programs in advance -- that can be explained in a pop-up.  The programs shared by the removed users are not shared with the group anymore.

This can also be used as a test case.
This issue is related to #384

@vlebedynska, I close this issue for you


Line length shrinks if Matrix element is used when TWISS is called

A line with a MATRIX element, eg:
wiggler: matrix,l:= 4,rm11:= 1.000449786249,rm12:= 4.000639844715,rm21:= 0.00022577719708,rm22:= 1.00045326313,rm33:= 0.9786569514876,rm34:= 3.970145599493,rm43:=-0.01063922161568,rm44:= 0.97864807445;
from MADX can be exported to Xsuite.
However when I load the line and then do TWISS the line appears to shrink:
line = xt.Line.from_dict(loaded_dct)
tab = line.get_table()
print(tab['s'][-1])
OUTPUT: 2303.952493011991
line.build_tracker()
tw = line.twiss(method='4d')
print(tw0.s[-1])
OUTPUT: 2223.9524930120483
I can provide the lattice, if needed.

Hello. Yes it would be good to have the full lattice

I've sent it.


TypedArrays are js.Iterable

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3745/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4946/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3746/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4947/
Test FAILed.

Updated.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3751/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4953/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3750/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4952/
Test PASSed.


Is it possible to abort if one of the formats is unavailable?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question
When I request a download from Youtube in formats "137,140" (for video and audio, without merging, because it is lengthy and useless), the following happens:

Trying to download 137... not available, whatever, moving on
Trying to download 140... success!

Your video is successfully downloaded!
(...but there is no video, only audio.)
Is it possible to do anything about this situation?

Maybe add an option to abort if one of the requested formats is not available?
Maybe add an option to disable merging when downloading in "137+140" form?
Maybe download the video in both formats at the same time, merging them on the fly, and then saving only the output file?

Using "137,140" keeps its from merging-- whereas "137+140" merges.

I know, but I wanted to avoid doing the merge afterwards while still requiring both inputs. This was a possible workaround idea, since "137,140" does not require both inputs.
The problem is, currently I need twice as much space (for the inputs and for the output), and a lot more time (first to download, then to merge). I just want to watch a video in my favorite player instead of YouTube's proprietary slow player!..

* Maybe download the video in both formats at the same time, merging them
on the fly, and then saving only the output file?

You could run 2 instances of youtube-dl and then use something like "ffmpeg -i someVid.mkv -i someAudio.mp3 MergedVid.mkv" to merge them.

How would this help with "on-the-fly" merging?.. Maybe if I find a way to redirect inputs with pipes, but I don't know how to redirect BOTH pipes with simple bash scripting... I think this would be easier to implement within youtube-dl . The audio stream is smaller and it's downloaded first, so it shouldn't take too much resources to keep it in a buffer and merge with the video stream as it's downloading.
Alternatively, how about this: download the audio stream first, saving it as a file like it's currently done, and then download the video stream, merging it on the fly?..
I guess the best solution would be to simply require both inputs in case of "137,140". Maybe add another notation (or option) that does not require, but requests them as it's currently done.


build: upgrade to webpack5

Types

[x] 🏗 Build System

Background or solution
Changelog

改的有点多了，重新提


Remove last comma in bundles

It seems like webpack is adding a superfluous comma at the end of the bundle. Although it's not valid JavaScript modern browsers are handling this error decently. But older browsers like IE7 fail.
/******/},     <----
/******/
/******/})
// END OF BUNDLE

Has the problem been solved ，How about this one

We don't support IE7, sorry


Do we need MobileUsingWifi.js?

I can't find a reference to it being used anywhere

Not needed anymore. I was thinking it gets removed as part of https://github.com/thaliproject/Thali_CordovaPlugin/issues/621 where we clean the legacy code that isn't needed anymore.


API protection breaks use of non-standard methods

Example:  codecombat.com/play/level/mod-squad-supplies-party
Basically, I added the pickMushroom(), buySmallPotion(), buyMediumPotion(), buyLargePotion() methods, but if the user tries to call them, they get the error:
TypeError: Cannot read property 'pickMushroom' of undefined

It works with http://codecombat.com/play/level/mod-squad-supplies-party?skip_protect_api=true

We've been just putting these methods in the programmableProperties list and assigning them from the Referee or a custom Component, and it works fine.


remove machine-approver probes

https://github.com/openshift/cluster-machine-approver/pull/211 is changing the metrics endpoint to listen only on localhost so these probes will not work

/lgtm

/lgtm

/jira refresh

/jira refresh

/override ci/prow/e2e-kubevirt-aws-ovn


Improved: markup for home page header

Related Issues

Closes #
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

Closing as done through https://github.com/hotwax/launchpad/commit/a94c62a94f5fff2a05fcdd884b708011c33cddfb

Closing as done through https://github.com/hotwax/launchpad/commit/a94c62a94f5fff2a05fcdd884b708011c33cddfb


Could you give more information about your student model on pubfig65?

I want to use your pre-trained strudent model in my experiments. I cannot find your training dataset and the test data file cannot be read by h5py normally. I have tried the dataset you metioned in readme, but  student model input need 224224 images, while pubfig83 is 100100, so I don't know how your model deal with transforming. So is it convenient for you to publish your pubfig65 dataset and give more details about your student model(e.g. label) or tell me how you transform images from 100100 to 224224?

What we did in our experiments was to upsample all images to 224*224 (cause we need to fit it into VGG-Face). This should be fairly simple to do when you load images. If you use Keras.preprocessing.image, here is the code snippet you can use to specify the resolution you want.
from keras.preprocessing import image
target_size = (224, 224)
img = image.load_img(img_path, target_size=target_size)
x = image.img_to_array(img)


[Android] i get a error when i fetch a image

I use fetch to download a image,
fetch(imageURL).then((response)=>{ ... }).catch((err)=>{...})
it work good in ios, but Android will get a error Exception creating JSON string: null
and if i open the js debug, no error. why?

Can you provide a code sample ? A bit more than your last message.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/android-i-get-a-error-when-i-fetch-a-image
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.


Rework OMUSE packaging to match AMUSE.

See https://github.com/amusecode/amuse/pull/736. This still needs testing, but apparently we don't have CI setup for omuse yet?

Argh, ruined it by not having a separate branch right away.


add CLI option to set log level

fixed forever https://github.com/subgraph/roflcoptor/commit/22276c0d36df0206af02aefb73fbfbe8eac99b17


Separated the tquery logic from the Table component.

It is now possible to easily create a non-tquery table by providing just the TanStack Table object and a couple of options to the <Table> component.
Cleaned up the i18n of the table - column names and summary are based on the i18n files after providing the table name. Also simplified the language interface structures which was not very good.

Coś zstanityzowało opis PRa i jest
the
component.

Tak jest jak się wstawi <taga> bez backticków.


parseColorCodes doesn't handle &-characters well

CommonUtils.parseColorCodes replaces all &-characters with §-characters to support the vanilla color code system (text bolding and other formatting features that you can't do with hex colors)
Should the function only check for &-characters that are used for bolding, italics & underlining? Current system just replaces the character no matter the context.

I believe only replacing & characters used for bolding, italics & underlining would work best.


Questionnaire a

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

A2: INCOMPLETE
Please specify the sampling strategy and justify it (SRS/STRATA/CLUSTER, etc...)
I would also avoid open ended questions on a questionnaire as the follow up analysis on such questions can be very difficult.

Repulling request after committing updates in response to suggestions provided.

A2: INCOMPLETE
Please justify how a SRS will be representative of this population. How did you come to an n=20,000?
I would focus on the scientific question of interest you are investigating, to then best determine the sampling strategy that will best answer your question. Hope that can help :)