undefined symbol error for _dataset_ops.so on rasp pi

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

$ uname -a
Linux pi_c 4.4.13-v7+ #894 SMP Mon Jun 13 13:13:27 BST 2016 armv7l GNU/Linux
$ cat /etc/issue
Raspbian GNU/Linux 8 \n \l

TensorFlow installed from (source or binary):

installed from nightly wheel http://ci.tensorflow.org/view/Nightly/job/nightly-pi-python3/

TensorFlow version (use command below):

$ python3 -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
v1.6.0-rc1-1295-g851c289 1.6.0

Python version:

$ python3 --version
Python 3.4.2

CUDA/cuDNN version:  N/A   running cpu on pi

Have I written custom code NA
Bazel version NA
GPU model and memory NA

Exact command to reproduce:

$ python3 -c "import tensorflow.contrib.slim as slim"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/__init__.py", line 33, in <module>
    from tensorflow.contrib import data
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/__init__.py", line 63, in <module>
    from tensorflow.contrib.data.python.ops.error_ops import ignore_errors
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/error_ops.py", line 20, in <module>
    from tensorflow.contrib.data.python.ops import contrib_op_loader  # pylint: disable=unused-import
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/contrib_op_loader.py", line 24, in <module>
    resource_loader.get_path_to_datafile("../../_dataset_ops.so"))
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/util/loader.py", line 56, in load_op_library
    ret = load_library.load_op_library(path)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/load_library.py", line 58, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename, status)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl15AllocateAlignedEj

Describe the problem
Import of slim fails with .so error (as described above). doubt it's actually slim, it was just the first import to reference _dataset_ops.so

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
Bazel version
GPU model and memory

It looks like you are missing the protobuf library. Can you try to locate it?

the following snippet of code runs in the same environment. i think this shows that, generally, proto buffers are ok ?  (this was the simplest tf related proto buff bit of code i could think of)
$ cat test.py
#!/usr/bin/env python3
import tensorflow as tf
c = tf.ConfigProto()
c.intra_op_parallelism_threads = 5
print(c)
 $ ./test.py
intra_op_parallelism_threads: 5

there might be something specific with contrib though; as mentioned here
https://petewarden.com/2017/08/20/cross-compiling-tensorflow-for-the-raspberry-pi/#comment-108790

The error you got is this:
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl15AllocateAlignedEj

which says that it can't load the protobuf runtime.
I just saw what you are trying to do. You can't import directly. You have to use import tensorflow as tf, then  use tf.contrib.slim.

that's true. i was just looking for a minimal reproduction case
$ cat test.py
#!/usr/bin/env python3
import tensorflow as tf
import tensorflow.contrib.slim as slim

$ ./test.py
Traceback (most recent call last):
  File "./test.py", line 3, in <module>
    import tensorflow.contrib.slim as slim
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/__init__.py", line 33, in <module>
    from tensorflow.contrib import data
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/__init__.py", line 63, in <module>
    from tensorflow.contrib.data.python.ops.error_ops import ignore_errors
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/error_ops.py", line 20, in <module>
    from tensorflow.contrib.data.python.ops import contrib_op_loader  # pylint: disable=unused-import
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/contrib_op_loader.py", line 24, in <module>
    resource_loader.get_path_to_datafile("../../_dataset_ops.so"))
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/util/loader.py", line 56, in load_op_library
    ret = load_library.load_op_library(path)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/load_library.py", line 58, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename, status)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl15AllocateAlignedEj

the code i first saw this with was a much bigger piece https://github.com/matpalm/bnn/blob/master/predict.py
(developed on my desktop, but fails on the pi)
$ ./predict.py
$ ./predict.py
Traceback (most recent call last):
  File "./predict.py", line 9, in <module>
    import model
  File "/home/pi/dev/bnn/model.py", line 5, in <module>
    import tensorflow.contrib.slim as slim
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/__init__.py", line 33, in <module>
    from tensorflow.contrib import data
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/__init__.py", line 63, in <module>
    from tensorflow.contrib.data.python.ops.error_ops import ignore_errors
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/error_ops.py", line 20, in <module>
    from tensorflow.contrib.data.python.ops import contrib_op_loader  # pylint: disable=unused-import
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/contrib_op_loader.py", line 24, in <module>
    resource_loader.get_path_to_datafile("../../_dataset_ops.so"))
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/util/loader.py", line 56, in load_op_library
    ret = load_library.load_op_library(path)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/load_library.py", line 58, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename, status)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl15AllocateAlignedEj

and, just for my sanity, regarding potential lazy loading / eval or something...
$ cat test.py
#!/usr/bin/env python3
import tensorflow as tf
c = tf.ConfigProto()
c.intra_op_parallelism_threads = 5
print(c)
import tensorflow.contrib.slim as slim

$ ./test.py
intra_op_parallelism_threads: 5

Traceback (most recent call last):
..... AS BEFORE
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl15AllocateAlignedEj

OK. I'm not 100% sure what it is, but you're really not supposed to import directly from tensorflow package. Just don't do that and you should be fine. Also, we are deprecating high level APIs in favor of tf.keras.

Fair enough. This might be the nail in the coffin for me & slim... I'll use this as an excuse to upgrade.

(oh, i did forget to mention, thanks for the attempted workaround but referencing slim through tf import still didn't work)
$ cat test.py
#!/usr/bin/env python3
import tensorflow as tf
print(dir(tf.contrib.slim.conv2d))

$ ./test.py
Traceback (most recent call last):
  File "./test.py", line 3, in <module>
    print(dir(tf.contrib.slim.conv2d))
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/util/lazy_loader.py", line 53, in __getattr__
    module = self._load()
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/util/lazy_loader.py", line 42, in _load
    module = importlib.import_module(self.__name__)
  File "/usr/lib/python3.4/importlib/__init__.py", line 109, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 2254, in _gcd_import
  File "<frozen importlib._bootstrap>", line 2237, in _find_and_load
  File "<frozen importlib._bootstrap>", line 2226, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 1200, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 1129, in _exec
  File "<frozen importlib._bootstrap>", line 1471, in exec_module
  File "<frozen importlib._bootstrap>", line 321, in _call_with_frames_removed
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/__init__.py", line 33, in <module>
    from tensorflow.contrib import data
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/__init__.py", line 63, in <module>
    from tensorflow.contrib.data.python.ops.error_ops import ignore_errors
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/error_ops.py", line 20, in <module>
    from tensorflow.contrib.data.python.ops import contrib_op_loader  # pylint: disable=unused-import
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/contrib_op_loader.py", line 24, in <module>
    resource_loader.get_path_to_datafile("../../_dataset_ops.so"))
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/contrib/util/loader.py", line 56, in load_op_library
    ret = load_library.load_op_library(path)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/load_library.py", line 58, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename, status)
  File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.4/dist-packages/tensorflow/contrib/data/python/ops/../../_dataset_ops.so: undefined symbol: _ZN6google8protobuf8internal9ArenaImpl15AllocateAlignedEj

Oh, right I forgot about the lazy loading of contrib. It is basically loaded on demand. There is something broken in your install. It's not to do with slim, but with tf.contrib.data. If you try to use it directly, it will likely break.

agreed it's something broken with that nightly built wheel (from http://ci.tensorflow.org/view/Nightly/job/nightly-pi-python3/) when i built from source (awhile back) it was a pain to get working, but didn't have this problem.
thought it might also be something weird on that pi (since i've built tensorflow on it multiple times) but i tried to install this wheel on a completely fresh install of jessie (on another pi) and got the same problem

and agree it's not slim (that was just the first thing hitting this in my original code (see initial report)) . so i guess moving to whatever other framework for layers isn't going to help. (since they will all reference data i guess)
i'm going to try to use a frozen graph (in the hope the frozen graph doesn't use data) and if that doesn't work i'll give up on the wheel for now & go back to building from source.


Update governance version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12353440428

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12353440428
Status: success


Preview doesn't show whole picture

I've build a fragment containing a CameraView instance and a toolbar, just like the default Android Camera app. I am trying to achive a smooth transition between a preview and a taken photo, but on some devices preview showing only a part of a picture, while resulting image is looking normal. I've tried to tweak the adjustViewBounds parameter, but it changes nothing. It seems like photo is top-cropped inside a CameraView preview, so user can't see a whole picture before taking a shot.
P.S.
I've figured out that changing of a width and height according to an optimal size value here may help, but a preview image is not centered according to its frame.

I've got the same issue. The preview from the CameraView is cropped or zoomed, but the picture taken is full size

same here

I'm having the same issue.

seeing same issue. can't use the lib...

Anybody found any solution for this? I too facing similar issue while catching with front cam.


Upgrade grape and jsonapi-resources gems

jsonapi-resources
There were a few internal changes which are mostly related to a serializer. But, the most important change is a _routed flag which specifies whether there is a route for a resource. It affects links added to responses, resources without routes doesn't get links.
Grape
Now classes providing modifications to Grape API must inherit Grape::API::Instance instead of Grape::API. In our case blocks specified in Grape::JSONAPI::API wouldn't have had access to methods of Grape::JSONAPI::API. Basically, internal classes of Grape wouldn't have seen them. More info about a motivation behind the instance change is there https://github.com/ruby-grape/grape/blob/master/UPGRADING.md#upgrading-to--120

Thanks for the updates, @dnesteryuk! :clap:


I can not install or I may not be. Want to help someone teach or install by teamviewer? Thank you very much.

Expected Behavior

Actual Behavior

Your FULL config.json (remove your username, password, gmapkey and any other private info)

Output when issue occurred

Steps to Reproduce

Other Information
OS:

Branch:

Git Commit:

Python Version:

Any other relevant files/configs (eg: path files)

Short Description

Possible solution

How it would help others

https://github.com/PokemonGoF/PokemonGo-Bot/blob/master/docs/installation.md
or
https://github.com/PokemonGoF/PokemonGo-Bot/blob/master/docs/manual_installation.md
also u can join our discord server ---> https://discord.gg/n3g5puF


chore: removing plugins-node-all

Removing plugins-node-all as it was moved to contrib repo and it is now
@opentelemetry/plugins-node-core-and-contrib
Package @opentelemetry/plugins-node-all should be deprecated

Thanks for doing this. I'll remove from my list.


Fix a number of problems reported by PHPCS

@jdrever I had a look at your PR #33 and tested it. The view worked, but clicking on one of the links results in an error in species_records.php. Assuming that is what is expected at the moment because species records don't work yet?
I merged the pull request anyway.
Opening another lightweight one here tidying up some problems PHPCS picked up.
In other files (such as NbnRecords.php), there are quite a lot of problems it is detecting. Didn't want to touch those just yet in case I break something.

Yes I know I saw that too, but I strongly presume that is because we are having to use the Worcestershire dataset and the location functionality doesn’t work when you are looking at data with Worcestershire locations.  Given that all I have done is changed the existing link to the page, and that the second page worked previously when using Shropshire data,  I doubt it is a problem with the code change.
From: Charles Roper @.>
Sent: 10 March 2021 18:48
To: jdrever/ShropBotSoc @.>
Cc: James Drever @.>; Mention @.>
Subject: Re: [jdrever/ShropBotSoc] Fix a number of problems reported by PHPCS (#34)
This is the error I am seeing when clicking on a species name: https://www.youtube.com/watch?v=zGyxHy5fl7Y
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/jdrever/ShropBotSoc/pull/34#issuecomment-795899889, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPKYJONFIUO3VY6UP6IC3LTC65GVANCNFSM4Y46SV4A.

Can I suggest we disable the use of tabs rule for now? - it just creates a lot of noise in the Problems tab

Or (just found the Convert Indentation to Tabs command in VC) we either convert files as we work on them, or do a mass change/single commit)

re. the with locationid, could we add in a wildcard here perhaps?
https://github.com/jdrever/ShropBotSoc/blob/11f7656ac2a03271fa70d74db2f0230fd18da17b/src/app/Controllers/Records.php#L15
Not looked into closely, but guessing Joe's code is making assumptions about what fields are present in the returned data (e.g. a location or site field), which isn't being gracefully handled.
Would need to compare raw Json returned by query on Shrop vs Worc to know for sure.

re. the with locationid

I have fixed this problem in PR #43.


Save spatial scales without float imprecisions

Currently, the .zattrs files contain spatial scales as in:
"datasets": [
                {
                    "coordinateTransformations": [
                        {
                            "scale": [
                                1.0,
                                0.16249999999999998,
                                0.16249999999999998
                            ],
                            "type": "scale"
                        }
                    ],
                    "path": "0"
                },

The correct x & y scales would be 0.1625 though. Our processing can handle this minimal float rounding, but it would be better to save the actual correct numbers in the metadata. This is a (minimal) distortion of the metadata and could have weird side-effects (e.g. what if it's rounded slightly differently for 2 OME-Zarr files that are opened in the same viewer? will all downstream processing always be stable to pixel sizes that are slightly off?)

One way to force fractal to write "0.1625" is that we keep track of pixel sizes as strings (first in the metadata parsing, and then downstream), and then include the strings in the .zattrs file by hand.
TBH I would not recommend this approach, which introduces a lot of friction in the corresponding functions for little benefit (see below). I'm open to other ideas on how to ensure that the written number if 0.1625.
More in general, I'm not sure this goal is really important:
The difference between 0.1625 and 0.16249999999999998 is of 2e-17 micrometers, and it should come from floating-point finite precision. All downstream processing will still use floats, and then it may still produce this kind of built-in errors (they simply go unnoticed because we do not write every single variable to a text file). If a 2e-17 pixel-size error breaks a task (as we noticed when building ROI indices with ceil instead of round), chances are that other (less-predictable) rounding errors would also break it. Then I think we should fix the task, rather than providing a more polished input.

I agree that this should not affect any of our downstream processing and if it does, we should fix that. Not sure we can guarantee the same for e.g. visualization tools, but I'd mostly hope they wouldn't be affected.
In the end, while the error is tiny, we still should do our best not to introduce any changes to the metadata. If we're limited in the precision of reading out the metadata for processing, so be it (=> let's make sure our processing is stable to that level). But would be great if the metadata is actually saved as it was originally parsed.
I agree though, this isn't a high priority issue, I assigned it to backlog for the time being.


jobs: turn on notifications by default

I think more people want this than not, and those that don't can turn it
off in the options.

@rcj (and @Echoring) do you have opinions on this? Mostly I see people asking for sounds for jobs all the time and so I think being slightly flashier/noisier by default isn't that bad since people can turn this off.

In my thought this is fine. Even someone has asked for tts output in jobs module.

I don't mind.
Regarding TTS, I have intentions to add that to Jobs. I'm currently thinking about how to best expose this in the configuration UI as the user probably wants to customize the TTS output per skill. Same might go for the threshold. So I'm wondering if some kind of UI where you can configure per skill how it should behave (show a timerbox, with sound, with TTS, threshold, etc) would be beneficial.
Maybe initially we could have individual sections in the configuration UI per job to group job specific configuration.
Ideally the Jobs overlay would be way more configurable. What I would like to do eventually is to be able to place the different UI elements individually at different locations (drag/drop them to where you want them to be), so you could add n containers which could contain m timerbox or other objects which could be placed anywhere on the screen.

Regarding TTS, I have intentions to add that to Jobs. I'm currently thinking about how to best expose this in the configuration UI as the user probably wants to customize the TTS output per skill. Same might go for the threshold. So I'm wondering if some kind of UI where you can configure per skill how it should behave (show a timerbox, with sound, with TTS, threshold, etc) would be beneficial.
Maybe initially we could have individual sections in the configuration UI per job to group job specific configuration.
Ideally the Jobs overlay would be way more configurable. What I would like to do eventually is to be able to place the different UI elements individually at different locations (drag/drop them to where you want them to be), so you could add n containers which could contain m timerbox or other objects which could be placed anywhere on the screen.

Sounds great! In fact I'm also considering about allowing user custom setting for elements in jobs module, but your destination seems far better.


far future: take in dataframes or other sparse data structures directly

right now taking in python dictionaries is awesome for it's flexibility and ease of development, but is killing us on memory, even if it is a super sparse data structure.
one workaround we could do for this is described in https://github.com/ClimbsRocks/auto_ml/issues/40, though that feels fairly hacky. taking in a DataFrame seems much more obvious.

@ClimbsRocks
currently i am adding the dataframe support.
Just wanted to know you want to keep dictionary objects along with data frames?

Hey @vishnumani2009 ! Sorry again for the delayed response time.
I think it's totally fine to switch our internals to rely on only DataFrames.
I'd still like to keep our API flexible to accept either. So what I'm picturing is that as soon as the user passes in a dataset, we immediately test whether it's a list of dictionaries, or a DataFrame. If it's a list of dictionaries, the first thing we do is convert it to a DataFrame. Then the rest of the internal workflow that's based on DataFrames should be good to go.
Thanks again for working on this! Let me know if there's anything I can do to help. I just fixed that other bug you reported, but let me know if you run into anything else.
Warmly,
preston

@ClimbsRocks
I am currently end of completing data frame support.
This took my time as had to make changes through out  the code base.
Currently I have followed  approach specified by you above.
I am stuck at one point and need your help with following.

I would like to know what does fit and transform method doing in CustomSparseScaler
I would like to what is returned from get_all_attribute_names() function
I would like to know what does fit and transform method doing in  AddPredictedFeature
I would like to know what does fit and transform method doing in  AddSubpredictorPredictions

best,
Mani

Awesome! I'm becoming more and more excited by how easy (and memory efficient) DataFrames are.

CustomSparseScaler is inspired by scikit-learn's RobustScaler. The goal is to find the 5th percentile value and the 95th percentile value for that column. We set these as 0 and 1, and then scale all values to fit between them. I wrote my own version to be compatible with sparse data, since scikit-learn's doesn't take sparse data by default.
This function just figures out the list of columns (attribute names in current implementation) in our dataset. It ignores everything in column_descriptions (which is either categorical data which can't be scaled, output column, a column marked ignore, or the y values to train a subpredictor on).
AddPredictedFeature can be deprecated for now. In it's current form it is used to run clustering on our input rows, and add the prediction for which cluster this row falls into as a feature. It's not super useful, and the code isn't great.
One of the most powerful things auto_ml does is automatically train and ensemble subpredictors for you on sub-portions of your problem. Once we've trained up all our subpredictors, we want to run each row through the subpredictors to get predictions from them. AddSubpredictorPredictions is where we pass each row into each subpredictor to get it's prediction.

Thanks so much for working on this PR. I'm excited to see the results! It should make working with large datasets much easier, and allow the user to do things like save their data in more efficient file formats like HDF5. Big Data here we come!

@ClimbsRocks
Thank you the details.
Will send you the PR soon.

@ClimbsRocks
Thanks for all the help.
I completed dataframe support.
Currently i am testing it with multiple datasets.
I do have some questions with respect to certain part of the codes.
I will consolidate all of them and send you soon
Later in weekend i will send PR.

heck yeah! awesome progress on this. feel free to ask any questions you have about any specific parts of the code.
i think i'd actually prefer to submit a PR now, if you can. the new PR tools are really nice for having specific conversations about specific lines of code, and keeping it all nicely organized.
either way, i'm super excited to support dataframes! i just crashed a few computers trying to load lists of dictionaries into memory that were too large :)

@ClimbsRocks
PR sent succesfully waiting for you review.
Following are some of my additional questions.
Since this being an auto ML tool, we will be bench marking this sometime in future.
Lets say we have all of the fundamental functionality ready by June 2017 , why not take this to auto ml challenge  and do a benchmark? what do you think?
See here for complete details on challenge.
Also i was planning if we could write a paper on auto_ml ,just to get a word out among user community.
Wanted to know your opinion on this?

Thanks for the help here @vishnumani2009 ! For what it's worth, we're training on datasets at work that would easily overwhelm 128GB of RAM if our dataset were a list of dictionaries. Happily, with DataFrames, this is entirely doable.


Enable format on type endpoint

So that my code is pretty without having to explicity run codeformat.

v0.4.13 has format on ; and } but not <enter>. This is broken on the server


ORCID assigned to wrong person

Bug description
ORCID assigned to wrong person. Can we move it, or should that person try to connect once again?
Steps to Reproduce

Log in to the front-office and download http://biblio.ugent.be/orcid/person.csv
See person https://telefoonboek.prd.rad.ugent.be/nl/people/802002307424 has the ORCID https://orcid.org/0009-0008-7064-5599
This ORCID belongs to https://telefoonboek.prd.rad.ugent.be/nl/people/801000581956

We will ask the person connected wrongly to the ORCID to remove the connection, and to ask the right person to reconnect. The one that is mistakenly connected is an ATP member, so perhaps this was a mistake from an administrative viewpoint. We can still investigate the problem you mentioned @nicolasfranck later.


PUB-1326 - Content changes for manual upload / remove pages

JIRA link
https://tools.hmcts.net/jira/browse/PUB-1326
Change description
Updates to content for Manual Upload, Confirmation and Remove List page.

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Looks like some unit tests have failed on this

Oops! Now sorted (I left some extra h3's in that wern't needed in the manual upload view)


Wait for result before setting chosen task name

When the task enqueue request fails, it doesn't make sense to include a chosen task name in the error response.
This likely has no effect beyond the cleanliness of the API proxy response.

http://ocd.appscale.net:8080/job/Daily Build/6156/


Add confirm dialog on disable Piped key

What would you like to be added:
Currently, disable piped key (Setting -> Piped -> Piped three dots -> Disable) has no confirm dialog before disable. Of course, it's not hard delete action but we have no way to revert this action on web view currently, so with this kind of dangerous action, confirm dialog should be added 🙏
Why is this needed:

I think that is not for disabling the Piped itself, instead of the Piped's key. And you can re-enable it so I think it is not that dangerous.

but we have no way to revert this action on web view currently
I think we are having a way to re-enable it with the "Enable" button.

Ops, you're right 🙏 I forgot about the Filter function, can access disabled keys and "enable" it from there 👍
So the problem should be, shall we make that "filter" appear by default such that someone like me could aware of that function 🤔

/close


[Request] Add submission guidelines

Having submission guidelines would help people wanting to contribute (like me).
It could recap:

What important files do, to have an idea of which files should be changed to add a functionality / fix a bug
What commands are used to compile files (if needed)
Any additional tips from the main contributor (you)

This project is very interesting, and I'd like to help with my free time. Having guidelines would help me a lot to contribute!

Good idea, this would be helpful. For all contributors, including myself. I'll work on contribution guidelines alongside the handful of bugfixes when I get a solid block of time later this week(end).

Added CONTRIBUTING.md in b3f9dde which outlines the repository structure and other useful information for contributors.
Let me know if you think of anything else that could be useful there.


Composer Installation

Not sure what this even requires as I've never looked into making my own composer package but would be cool. Also you should probably use a namespace....

I've used Composer exactly one time so I'm not an expert on that... yet.
However, there are brilliant people who add pull requests:
https://github.com/jenstornell/tiny-html-minifier/pull/14
When I have the skills or get a pull request on installation instructions, I will add it.

I would like to contribute to the project, as soon as I saw it I liked it, I was looking for a simple way to compress the html code and I found this. I have knowledge of composer, I agree with @Modicrumb, you have to use namespaces. I could do a pull-request with this change.

@EdixonAlberto Good you liked it! I have hidden your comment as it's off topic on Composer installation. Instead I've created a new issue out of it to keep them apart. Continue the namespace discussion there.
https://github.com/jenstornell/tiny-html-minifier/issues/30

jaja, ok is that the namespace has a lot to do with composer, that's why I'm missing the subject. We will continue here #30  then.

Looks like you didn't add your package here https://packagist.org/


proc_channel should be able to indicate STD/SEM as 'tubes'

So far can only indicate them as dotted lines (I think).

Daniel takes care of this one.

closed?!

Thanks!


[RFC][Relay] Dynamic Dimensions

Supporting Dynamic Dimensions
I recently opened an RFC proposing a new dynamic runtime (see #2810).
A missing piece of the puzzle for supporting fully dynamic models is typing, and code generation  for tensors with statically unknown shapes.
There are three critical steps to supporting dynamic tensors which I believe are distinct tasks:

Extending the type system
Computing allocation sizes
Code generation for dynamic shape

First we should consider a simple example which exhibits the hardest version of this problem, how do we type such a function:
operator arange(%start: int32, %stop: int32, %step: int32) -> Tensor((Any,), int32)

fn (%i: int32) {
  let seq = arange(0, %i, 1);
  sum(seq);
}

This example is challenging to type check due to arange's output shape
depending on the value of %i. Currently we are able to statically determine the
output shape if it is a function of the input's shape, but not if the output shape is a function of the input itself.
*Note: we could use full dependent typing here, but this introduces new challenges. *
In the cases where the output shape is symbolic function of the input, or fully dynamic (such as arange) we must generate code which computes the precise size of the output buffer for the return value at  execution time.
For example even when if we know f: (Tensor<n>, Tensor<m>) -> Tensor<n + 1, m> we must
generate a function which can allocate space based on the runtime values of n, m.
These cases are rare because most programs are closed program we know all shapes statically,  but once we introduce a dynamic shape, it is possible we must defer arbitrary shape
computation until runtime.
Extending the type system
The first proposed change is finish support for symbolic shapes.
For example f can be specialized to concrete shapes, and users
can pass around functions and operations which work over symbolic
shapes.
The second change is support for an Any shape.
Any represents a tensor dimension which is not statically known.
One possible design is to use a sub-shaping rule. This rule would be
similar to the traditional subtyping rule used in popular programming
languages.
For example in C++:
class T { ... };
class U : public T { ... };

We know U <: T (that is U is a subtype of T).
In this version of Relay this would mean (1, 2, 3) <: (1, Any, 3).
Unfortunately this typing rule is not what we want. For example if
we have a function which expects a tensor of shape (1, 2, 3) we
can no longer call it with a tensor of type (1, Any, 3) we must
introduce explicit casting to properly check the runtime type.
Instead we propose borrowing ideas from gradual typing,
and introducing the notion of "gradual shaping". Any occurence
of a dynamic dimension is potentially unknown at compile time,
and when we demand a concrete dimension we must dynamically
check. This introduces the ability for runtime shape mismatches
but ensures that dynamically shaped and statically shaped code
can interact gracefully.
This is important, for example in many cases Any would pollute
the types of functions that are static, but their argument has
an unknown shape.
For example, we can optimize the below code under the assumption
that it will always have shape (10, 10), and then perform
one runtime check of shape when calling f.
fn @f(%x : Tensor[(10, 10)]) { ... }

let x: Tensor(Any, Any) = ...;
@f(x)

Computing allocation sizes
The second component is computing the allocation size required.
We must due this because TVM requires its output buffers to be preallocated.
When the output is dependent on input, or a symbolic relationship which has not
been specialized we need to generate a function at runtime which computes
the output buffer size.
Array<IndexExpr> ShapeFunc(const Array<Input>& inputs) {
  ...
}

We can then invoke this function to allocate the appropriate output storage,
during execution.
Code generation for dynamic shape
The final challenge is performing code generation, this RFC only proposes solving the
representation problem and generating naive TVM code in these cases which use fully
symbolic shapes which are not known until runtime.
I believe there are a variety of different code-generation strategies which
each may suit different situations, and we implement a mechanism for specifying
the code generation policy in the face of dynamic shapes seperately from
shape functions, and typing of dynamic shapes.
Some of the possible code generation strategies:

Generate placeholder code when symbolic relationship holds.
Generate placeholder code and make use of shape functions.
Implement bucketing style approach to code generation.
Your ideas here!

This is a great progress towards deploying real-world NLP models. Thanks Jared for proposing this!
As far as I could tell, however, in the long term, we probably need more support for more general dependent typing for more data types. I am afraid that we will have to consider composite types in the very end.

Another immature idea is to unify composite types with operator's attributes (but maybe far long-term)

I think for now it is okay to just support code gen with fully symbolic shape. Later if we want specific optimization strategies for different programs, we can revisit and adding more code gen strategies.

This looks like a great design, and should handle all the applications I'm currently thinking of.  @jroesch  let us know if we can help.

@ajtulloch @icemelon9 and I have been quietly hacking on a prototype the last few months but have been busy with other things (such as VM 😄 ). We are going to start pushing now, I opened a draft PR which will contain type checking changes, and we will follow-up with code generation next.
One thing would to help validate, criticize the design from the standpoint of a user who is deploying models day to day, and comment on the code generation strategies.
One change that we have decided on is adding the ability to type hint when generating IR.
When generating IR from frameworks we don't necessarily have all type information up front.
That is you may know a static shape for a value, but the shape is not necessarily a stable static property.
For example imagine a loop variable which starts as a (1, 1) matrix, but will grow by one in the first dimension on every loop. Eventually the matrix will have the shape (n, 1) where n is the number of loop iterations but we can't know that statically.
Our current solution is to generate IR with unknown types, plus a type hint, allowing inference to eventually solve the most general type.
For example if we were to generate IR for a loop we can do:
def my_loop(%x: ?T, %y: ?U) {
     ...
     can_unify(%x, shape=(1, 1))
     my_loop(concat([%x, %x], axis=0))
     ...
}

These could later be used to inform code generation, but ensure that we get the correct typing.

The any dimension feature has landed into the mainline


Fix try-catch for Windows

Need both ValueError to catch signal.signal not available and AttributeError to catch signal.SIGALRM missing.
Fixes #227 introduced by #223

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 89.689%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

django_q/timeout.py
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

django_q/core_signing.py
1
80.85%

django_q/timeout.py
1
80.77%

Totals

Change from base Build<PHONE_NUMBER>6:
-0.03%

Covered Lines:
3375

Relevant Lines:
3763

💛  - Coveralls

Fixed the linter complaint. :)


Update Localizable.strings

Checklist

[ ] This PR does not change text that hasn't been signed off with the RKI. Have a look at the pinned issue 440
[ ] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[ ] If this PR comes from a fork, please Allow edits from maintainers
[ ] Set a speaking title. Format: {task_name} (closes #{issue_number}). For example: Use logger (closes # 41)
[ ] Link your Pull Request to an issue - Pull Requests that are not linked to an issue with you as an assignee will be closed.
[ ] Create Work In Progress [WIP] pull requests only if you need clarification or an explicit review before you can continue your work item.
[ ] Make sure that your PR is not introducing unnecessary reformatting (e.g., introduced by on-save hooks in your IDE)

Description

Approved.
Best Regards
Andreas
Von: janetback<EMAIL_ADDRESS>Antworten an: corona-warn-app/cwa-app-ios<EMAIL_ADDRESS>Datum: Donnerstag, 13. August 2020 um 18:07
An: corona-warn-app/cwa-app-ios<EMAIL_ADDRESS>Cc: "Vogel, Andreas"<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Betreff: Re: [corona-warn-app/cwa-app-ios] Update Localizable.strings (#1052)
@janetbackhttps://github.com/janetback requested review from @corona-warn-app/cwa-app-ios-members on: #1052https://github.com/corona-warn-app/cwa-app-ios/pull/1052 Update Localizable.strings as a code owner.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/corona-warn-app/cwa-app-ios/pull/1052#event-3652780131, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB4HXRI2QVKSH5R6OPMQUFTSAQFLTANCNFSM4P6TDWWA.


Bug in Snelson?

https://github.com/GAMES-UChile/mogptk/blob/c4bfff1375a348c212618334e3ccb413bc873e0d/mogptk/model.py#L119-L120
I think it should be self.variance.

Thanks! Fixed now


Fix node 10 travis problem

What happens with node 10? Would be good to capture that in the commit as well :octocat:

It spits out the error: TypeError: Cannot read property '1' of null

I did re-run it and it failed again, different timeout this time.  Trying again


Feature Suggestion

Auto Split Images More than 5000px
Sometimes, i encounter images over 20K long tall page, if possible split the images to 5000px each and rename it into imagesname_A imagesname_B or similar
Auto Renumber chapter filenames and chapters
use 001 020 (3 digit system instead) so 001.png 002.png, with the split images, the name will be 001_A.png 001_B.png etc,
same as chapter, i use advanced renamer but, quite troublesome, so chapter-001 chapter-002 because android and things like natural number sort order will sort 1 then 10 and 100 if no first 0-digit to next

Split Images:
Auto Renumber: In the case of images, they are implicitly formatted with two digits (for example, 01, 05, 23, etc.). Generally, this format should prevent sorting issues. Did you saw comics/manga that had more than 99 images in chapter ?.

Sorry for late answer, An Outsider Way's in on Toonily are ranged from 80 to 160 images per chapters so i think 3 digit number format will do.
for the image you can look "That’s Not How You Do It" chapter-45, there's 14K pixel image tall and 23K image tall there, or "Circles", Ch 13, all of them are like 20K pixel tall, this will problematic when converted to webp wich only can support 16.3K pixel width/height

@Shikitsuna from what I noticed in the manga indicated by you "An Outsider Way" the images have 3 digit number format. However, I have added the change to the part with chapters - thank you.
I must mention that the chapters with names other than "chapter-N" are downloaded regardless of the indicated range (like "epilogue")
During tests with such large chapters, a high processing time was observed (right side of the image). In this sense, a concurrent mode of downloading the images from the chapter has been adopted (left side of the image).

Splitting Images
What about splitting images, I decided to include them in next release in the golang version only. Adding functionality to the shell version involves external dependencies such as ImageMagic, which goes against the project's goal of using no dependencies.
Release v0.20

Thanks, I guess the split image parts can't be done much without dependencies, but overall this is very good improvement. tested and it's working nicely (for now).


feat(testing): make retry plugin accept async methods

This PR make retry plugin work correctly with async methods.
If Chai extended with async methods like following:
matchCode(code: string): Promise<Assertion>;
it would give false positive:
it('FALSE POSITIVE', async () => {
    await expect(() => `const a =    7;`)
        .retry()
        .to.matchCode(`const b = 7;`);
});

Changes was included in #756


Database Gives An Error When Trying To Filter

Describe the bug
When trying to filter both databases in Obsidian vault it comes back with an error. Error seems to be focused on column on the right of the File column on both databases.
To Reproduce
See attached log and video screen capture.
Expected behavior
Filtering a database should function as normal.
Desktop (please complete the following information):

OS: masOS Monterey - Version 12.5.1
Version: Obsidian Version 0.15.9 (Installer 0.15.8); DB Folder Version 2.3.3

obsidian.md-1662240264927.log
https://user-images.githubusercontent.com/106125987/188288101-05e44e52-0904-428b-935d-e405ef97c440.mov

fixed on https://github.com/RafaelGB/obsidian-db-folder/issues/322 with 2.3.4


Change default delete action

In gdu the default button in the delete dialog is "Yes" but in ncdu it is "No". This isn't a big deal but muscle memory can be hard to adapt! I would like to add an option to change the default in gdu to match the behaviour of ncdu.
Is this feature something the gdu project would accept to merge, and if so what do you think would be the best way to do it? Flag? Env? Other?
Thanks!
PS: I did implement this feature for myself, but it's not elegant (added --default-delete-action=yes|no but it required changing CreateUI signature to pass the flag which of course ends up changing way too much stuff... There's probably a better way😃).

I prefer the yes default myself but if it can become a configurable item it would make sense.

@ducalex: Yes, it has to bubble through quite a lot of code. Adding it makes sense to me as well.

@ducalex: Yes, it has to bubble through quite a lot of code. Adding it makes sense to me as well.

Thank you, I'll spend some time studying the code and prepare a draft with hopefully minimal and non disruptive changes!
On a related note, would it make sense to pass a UIConfig object (or similar appropriate construct) to CreateUI instead of a list of args? Because there are other feature requests to add flags/settings (#12/#123, #103) and I suspect it would involve updating ~70 CreateUI calls (which admittedly is mostly tests) for every addition?

Yes, passing the options as config object makes sense to me. We can make separate PR for it. Would you like to do it?

I'm fairly new at Go so I don't feel confident enough to architect this change myself at the moment.
But maybe someone will be able to give me some guidance after seeing the extent of code that had to be changed by my initial PR.

I wish I could specify a custom function/script for deletion... so I could script to tarball then delete. ;D


Use term "Micro Frontend" consistently in README, fix spelling

Resolves https://github.com/microsoft/redux-micro-frontend/issues/37

@harunsmrkovic - thanks for your interest and contribution. Please create an issue and attach it with the PR. We will approve it.

@patrickCode I did it, but can't figure out how to sign the CLA - there is a CI job stuck


Python crashes when trying to write an output file

I'm running the example code from the README:
import OpenEXR, Imath

width = 10
height = 10
size = width * height

h = OpenEXR.Header(width,height)
h['channels'] = {'R' : Imath.Channel(Imath.PixelType.FLOAT),
                 'G' : Imath.Channel(Imath.PixelType.FLOAT),
                 'B' : Imath.Channel(Imath.PixelType.FLOAT),
                 'A' : Imath.Channel(Imath.PixelType.FLOAT)}
o = OpenEXR.OutputFile("hello.exr", h)
r = array('f', [n for n in range(size*0,size*1)]).tobytes()
g = array('f', [n for n in range(size*1,size*2)]).tobytes()
b = array('f', [n for n in range(size*2,size*3)]).tobytes()
a = array('f', [n for n in range(size*3,size*4)]).tobytes()
channels = {'R' : r, 'G' : g, 'B' : b, 'A' : a}
o.writePixels(channels)
o.close()

When I get to the OpenEXR.OutputFile, I get a "Python has stopped working" dialog box, or a Jupyter dialog box "The kernel has stopped unexpectedly."
I'm running on 64-bit Miniconda on Windows 11, in a clean environment where I just installed python and jupyter, then pip installed OpenEXR.  I tried Python 3.12, then made a new environment with Python 3.7 (since that's the oldest version supported by the pyproject.toml).  I also tried in conda environments on a different computer running Windows Server 2016.  None of this works.
There's also a conda OpenEXR on conda-forge.  I tried with and without that; I'm not sure what it is, but installing it is not sufficient to import OpenEXR without also pip installing OpenEXR.
Please let me know if there's any more info, or tests I can run, that would be useful in debugging.

Oops, sorry about that, that example code is wrong. I'll submit a fix momentarily, but the proper line should read  Imath.Channel(Imath.PixelType(Imath.PixelType.FLOAT) ). And it's also missing a from array import array.

Thank you!  That works for me.
I made a PR (#1714) and am trying to figure out who from my institution I should reach out to about the CLA.  I'm hoping to get back to you about that later today.


The .ini config files should be compiled to PHP files on first access

We should see if this increases performance, in theory it should as the resulting .php files will be stored in the opcode cache.

I got this working locally and tested it via the ab command, and to be honest I could not measure the difference, even with hundreds of simultaneously threads pounding my local laptop app the difference between .ini versus .php config files was non-existent.
I think it is helped by the fact that our config manager class is a singleton, so for every request/response cycle we only ever parse the .ini config file exactly once so the overhead it already tiny.
Not worth the effort so I will close this as a won't fix.


Crash with GTK4 > 4.10

The crash started when GTK4 changed the unstable API of SignalListItemFactory from Gtk::ListView to GObject::Object.
This triggered a bug in gi-crystal.
To reproduce: Try to open any documentation.

Fixed by https://github.com/hugopl/gi-crystal/pull/133


Resort to fallback method if we can't get the active window (Fixes #38)

I'm not sure I'm so comfortable putting a goto into the code

Why's that?

Upon further thought, I suppose it's fine. I think I just have a negative response to goto statements ingrained in me, perhaps unfairly.

They can certainly be used to write extremely confusing and hard-to-follow code, but we're just doing a very short hop here to avoid a bit of code duplication.


Add / handler to admin server with a minimal dashboard

This is the weburi the user arrives at from dcos.

What functionality do you see this dashboard (or API) exposing?


[$1000] Web - Auto navigation back to user contacts when magic code is incorrect full filled and network is offline then online

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

Action Performed:

Login any account
Go to Settings -> Profile -> Contact method
Click "New contact method" button -> Fill in any email that is not Expensify account yet
Click on the secondary login that just added to verify it
Fill in incorrect magic codes
Go offline then online

Expected Result:
It should stay in the validate secondary login after go online
Actual Result:
It navigates back to the user contacts page automatically although magic codes are incorrect
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.43-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/c5561bd6-d18b-4993-84ed-e12db969665d
https://github.com/Expensify/App/assets/93399543/04d69770-fd67-4471-8f60-917b0f75a1dc
Expensify/Expensify Issue URL:
Issue reported by: @hoangzinh
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689868296938909
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f2d16d4070455f05
Upwork Job ID:<PHONE_NUMBER>552568832
Last Price Increase: 2023-07-28

I can reproduce and can't find a dupe - I think this is unexpected behavior, but I think i've seen similar issues

@aimane-chnaif two proposals above!

pending proposal reviews

I don't want to adjust the bounty on this until the proposals above have been reviewed

pending proposal review

@aimane-chnaif bump on the review above!

pending review

@aimane-chnaif can review the updated proposals above early next week or provide an eta on when you'll be able to review?

@aimane-chnaif will you review the proposals this week?

@aimane-chnaif any thoughts?

Payouts due:
Contributor: $1250 @hoangzinh (reporting bonus + contributor)
Contributor+: $1000 @aimane-chnaif
Eligible for 50% #urgency bonus? Y
Upwork job is here.

@adelekennedy can we close this out?

I just checked the Upwork job and it looks like everyone is paid out


record the final mint transaction

It is not easy to check an onchain transaction mint successfully, because of the mint cap term. Saving the final mint transaction id, we can ensure all transactions before that tranaction id satisfy the cap condition.

Users can see if a mint succeeded by looking up the output on /output/TXID:VOUT.

@casey I think the output is only temporary available? When an output is spent, it gets deleted from the database.
https://github.com/ordinals/ord/blob/744a3d6bec4422b2b1d599f0fa98bf69df04e4ad/src/index/updater/rune_updater.rs#L419-L421


Empty form submits on homepage

I can submit the form to validate on the homepage without providing a URL. I don't think that should be allowed.

This was addressed in v0.9.2 of yURL. Thanks for flagging this!


Sessions all down suddenly

I got this error after 20 min after activating a session, and then it goes down:
`'NoneType' object has no attribute 'replace'
Traceback (most recent call last):
File "C2Server.py", line 325, in do_POST
cookieVal = (s.cookieHeader).replace("SessionID=", "")
AttributeError: 'NoneType' object has no attribute 'replace'
'NoneType' object has no attribute 'replace'
Traceback (most recent call last):
File "C2Server.py", line 325, in do_POST
cookieVal = (s.cookieHeader).replace("SessionID=", "")
AttributeError: 'NoneType' object has no attribute 'replace'`

Hi,
Sorry to hear about this, can you provide a little more information on this. Does this happen every time after 20 minutes or did you run a certain command to trigger this error. If so can you let us know what those commands were.
Thanks

Also, was the implant fully working in the 20 minutes you had it up? E.g. could you run commands and see the output?

Actually it was may launch break, i left the sessions active (idle without any task) and when I came back I found the error.
I've got new sessions but I wanted to share the issue to perfect this nice tool.
( I trying to fix the issue with an initialisation of the variable s(.cookieHeader) and a try catch. but I think that maybe it can be fixed with a more reliable way.
Thanks

Thanks for your kind words :)
This is a strange one, @benpturner there should always be a cookie right for a post request?
@kumara19bara can you confirm you're using the most recent version and you're using the posh and posh-server commands to run the tool or equivalent?

yes I just installed the latest version because of the issue. and got the same error again. I think that some of request may be stopped by the AV.

Are you still encountering this issue?

it is working perfectly after the last update, thank you.
Keep the hard work in this awesome project.
cheerzz


Add notification type updated

@aml-development/ozp-backend-team
Requires
ozp-react-commons branch orgstewardNotifications aml-development/ozp-react-commons#105
ozp-backend branch orgstewardNotification (no s at the end) aml-development/ozp-backend#375
Use Case:
An admin has the ability to send out a notification to all Content Stewards, notifying them to review their org's Listings and make any necessary changes.
Content Steward updates Listing as needed via Create New Listing Form (the usual means of updating listings)
Content Steward and listing owner(s) are notified when listing has been updated and approved
Acceptance Criteria:
Only admins can send notifications to content stewards
All content stewards receive the notification to update their apps
Only content stewards receive the "update app" notification type
Content stewards can click on a link in the in-site notification to go to the same Listing Management Recent Activity tab. (I chose this tab because all org stewards will have it and they can easily go to published listings from overview list on the left. They can also see which ones have been updated recently)
The system notification type still functions as before there was no "Type" dropdown
To test:
As bigbrother,
Go to Center Settings (from the dropdown menu)
Go to the Notifications Tab
Select the Update Request type
See that the Notification text is populated with a default message (you can change this message if you like)
Set expires on date
Press send
Login as david, julia, obrien and wsmith to make sure that they have received the message and that it has the link as detailed in acceptance criteria
bigbrother should not receive the notification
hodor should not receive the notification
jones should not receive the notification
Referencing original pr: https://github.com/aml-development/ozp-center/pull/893
This is simply just Lauras work with master merged in.

https://github.com/aml-development/ozp-center/pull/893 has the approval.  I'm going to merge this updated branch.


Performance recommendation: Singleton extensions

Parent issue: https://github.com/silverstripe/silverstripe-admin/issues/199
At the moment, every time we construct a class that has extensions, we construct a number of extension instances for each of these instances (effectively o(n^2)). This is extremely expensive, given that most extensions provide behaviour only to the owned object and do not themselves contain instance-specific properties (although some do!).
As we currently rely on setOwner() before calling methods on extensions, it should be feasible to safely share instances across multiple parents (although the risk does increase if setOwners aren't updated down the call chain).
Extensible::getInstances() will be modified with the following change:
-$instance = Injector::inst()->create($extension);
+$instance = Injector::inst()->get($extension);

Extensions which must NOT be shared can still be declared as type: prototype via injector, which will cause get() to return a safely isolated new instance.
Referenc: https://blackfire.io/profiles/dea4e266-e469-4976-abf9-c7844503b04b/graph?settings[dimension]=wt&settings[display]=focused&settings[tabPane]=nodes&selected=SilverStripe\View\ViewableData%3A%3AgetExtensionInstances&callname=SilverStripe\View\ViewableData%3A%3AgetExtensionInstances
Note that getExtensionInstances() is called about 1100 on a normal /admin/pages url. Of these 1100 requests there are about 100 Injector::create() calls.

Effort set to medium for implementation: easy, testing: hard

To clarify, you'll need to ensure that setOwner() is called everywhere before the extension class is used, this seems like a similar thing that is being done with the Controller class and setRequest().
and the type: prototype that you mention will be a config on the Extension class right?

Seems borderline as a late stage change, e.g. modules that have already upgraded will need to go back and check their extensions. But we allow opting out of this behaviour, so at least you don't need to rewrite custom code. It's a bit counter-intuitive to show devs $this->owner references in example code, but then have the caveat that other $this references won't work. We should just pass in $owner as a first arg to all extension methods to make these stateless, but that's a 5.x change. My assumption is there's non-trivial performance benefits which drive this change?

and the type: prototype that you mention will be a config on the Extension class right?

Actually it's a key you add to the injector service reference, not on Extension directly.

It's a bit counter-intuitive to show devs $this->owner references in example code, but then have the caveat that other $this references won't work as expected

They will still work, they'll just be singletons. For example, SiteTreeLinkTracking::$parser class will now be shared across all instances, but since it itself is a stateless class, there's no negative effect.
An example where an extension is currently stateful is Versioned; We would need to change this to prototype to ensure it behaved as before to avoid symptoms.

Seems borderline as a late stage change.

I think that the amount of speed-up to complex pageviews this provides will help us determine whether it's worth the frustration.  A slow SS4 will harm the community more than an late-beta Api change. ;-)
So if this is a significant speed-up I'd say go for it. My assumption is that it will affect a pageview with lots of pages on a site with lots of extension-supplying modules installed.

Seems borderline as a late stage change, e.g. modules that have already upgraded will need to go back and check their extensions.

So far we haven't released anything higher than an alpha release for the blog module, so I imagine that we'll be revisiting and regression testing closer to the time anyway. This is from a commercially supported module point of view - of course this may affect other community modules, e.g. Symbiote which are tagged as stable already.
I'd be mildly concerned about changing to a singleton pattern since there will be and unknown number of extensions that expected to be able to persist things to their own unique state, or perform certain actions based on the state of the current owner. My main concern with that is that they won't know to do that until things start breaking (not obviously breaking though, just behaviours not working).
An example of this would be BlogCommentExtension from the blog module:
public function getExtraClass()
{
    $blogPost = $this->owner->getParent(); // <--- danger!

    if ($blogPost instanceof BlogPost) {
        if ($blogPost->isAuthor($this->owner->Author())) { // <-- more danger!
            return 'author-comment';
        }
    }

    return '';
}

BlogMemberExtension is another example where it's relying on a specific DataObject to determine how to augment its functionality: https://github.com/silverstripe/silverstripe-blog/blob/master/src/Model/BlogMemberExtension.php#L48-L97
I guess the point is that it's possibly quite common. Do you think it might be safer to make "act as singleton" an opt-in thing rather than opt-out?

$blogPost = $this->owner->getParent(); // <--- danger!

This would work because all method calls would have setOwner() called on it prior, even though it's a singleton. Note that setOwner() and clearOwner() actually works a stack, so it handles even nested calls.

Ah, right. In that case it's really only going to affect a small number extensions that may be storing data on its own instance, assuming it won't be used anywhere else. This is less of a problem IMO because it's probably less likely.

The danger is when a method does $this->myCache = []; and then rely on the cache needing to be specific to each owner.
I can imagine a few extensions doing that, but it's a small proportion of the total number.

Here's an example of that from something I'm working on at the moment: https://github.com/symbiote/silverstripe-advancedworkflow/blob/b284612bc9a0a52051ae7e8ace662974c89557af/code/extensions/WorkflowApplicable.php#L31-L52

ah... hahah, that was what I was thinking too!
yeah, so for that case in particular, you'll need to a PublishJobsRunning map of owners and no longer have just a simple boolean to define it...
Or turn off singleton for that extension :P

Yeah - in that case I think turning off singleton behaviour would be best - but it's a good example of an extension that would need to know to do this, because it's not going to start throwing exceptions, it'll just start behaving in strange ways

Yes that's the risk; Do we turn on singleton by default (opt out), or off by default (opt in)? Requiring opt-in would slow adoption, but be safer.

What kind of improvement are we talking about? I realise it's probably something that will be more obvious at scale, but if its substantial then it might be worth it

For example, all of the $Var, $Title etc calls get turned into DBField::create_field() calls for casting in a template. Each of those will potentially have extensions applied to them, which multiplies the number of object creation by the number of fields. With static extensions only one of each instance type is constructed, regardless of how ever many times the extended object is constructed.

If the performance improvements are significant, then we should have it on by default.
Can we use magic getters/setters to apply these local calls to the owner object, or is that too risky?

It could increase the risk of infinite loops.


nixos/adguardhome: init

Motivation for this change
Adds a system service for AdGuard Home.
The service definition is largely based on the one generated by AdGuard Home itself when passed adguardhome --service install (this won't work on NixOS, as it will try to write to a read-only filesystem), with an additional choice of config, work, and pid file locations that should follow the FHS.
May provide some relief for followers of #61617 while they wait for #108055 & the following PRs (but won't actually work on a pi without overriding the adguardhome package because of #113900)
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

I think it might make sense to introduce an openFirewall option to this module, although I am not exactly sure what it should cover. The port the webUI is served on is specified by port, but not the port the DNS server is served on (which is specified in the config file / on initial setup). However, ports 53 will be used in the vast majority of cases.

Marking as draft while I look into DynamicUser

Pushed a commit that uses DynamicUser, remove options with no clear use case, and adds an openFirewall option for the webui.
Probably should all be squashed on merge.


XDG Support for ivi-shell: ivi-id-agent

This is a reference implementation of an ivi-id-agent plugin. It
creates surface-ids for desktop-surfaces depending on the configuration
provided in "weston.ini". For more please refer to the reference
implementation in this commit.
The only public interface available is the "id_agent_module_init"
function. It is responsible for initialization of structs and reading
the configuration.
In the reference "weston.ini" two types of configurations can be found:
[desktop-app] is used to configure a particular application. Therefore
the desired surface-id must be provided. Moreover "app-title" is the
title that is provided by the desktop application (xdg-protocol).
"app-id" behaves accordingly. Although both parameters can be set, it
is not mandatory. Finally at least one has to be set.
[desktop-app-default] enables the id-agent to generate generic
surface-ids for unconfigured applications, e.g. for development. This
tag is optional. To generate the id an interval starting from
"default-surface-id" to "default-surface-id-max" is used. The id is
incremented until the interval is exceeded.
In the function "get_id" a surface-id is assigned by means of the
configuration. It can be adjusted, if another behavior is desired.
In this plugin the parameters, that are described above, are evaluated.
To use these patches please also apply the dedicated patches for
weston: https://github.com/mtey/weston/tree/xdg_support_ivi_id_agent
Since libweston-desktop is used to introduce desktop-surface support,
xdg-protocol is supported.
More Info:
https://wiki.automotivelinux.org/_media/agl-distro/agl_amm_xdg_support_a04.pdf

Closed this PR due to Weston v6 release. Please refer to the following PR:
https://github.com/GENIVI/wayland-ivi-extension/pull/105


Update Dockerfile

The CLI is not copied to the container. This commit fixes it.

@bert-laverman pls take a look.

You have -debug versions of the base image, which has shell (bash). But at this point I'm not sure if this is the right source repository for https://hub.docker.com/r/axoniq/axonserver-enterprise/tags

You have -debug versions of the base image, which has shell (bash). But at this point I'm not sure if this is the right source repository for https://hub.docker.com/r/axoniq/axonserver-enterprise/tags

Actually, the "-debug" images of distroless don't have a real shell, but more something that resembles the minimal one from busybox. The official Axon Server images use the Eclipse Temurin images, which contain bash, but those images are managed somewhere else. The "Running Axon Server" repository is there to provide examples.
Still, a separate Axon Server CLI image might be useful to add. I'll have a look if we can add that.


pear: command not found

my pear command isn't found. i am on mac os x 10.10 on a fairly new macbook and i use homebrew + phpbrew btw. i have 5.6.6 + defaults installed via phpbrew. what should i do? $PATH issue?

Hi, whats the output of echo $PATH?
BTW, I totally recommend you to abandon PEAR as it has too many issues. Suggestion: http://www.phptherightway.com/#handling-pear-dependencies-with-composer

It's all fixed.


About initializing learnable parame φi and ηi

hello! in Algorithm 1 -Lines3,I want to kown how to update φi and ηi by φi-1 and ηi-1,could you enjoy a pecific methods？
And others, In i-th phase,plastic blocks will learn new  parameter by novel data,so this new paramter how to merge with stable blocks in i+1-th phase?
Thank you!!!

Hi @zjy359
Thanks for your interest in our work!
For your Q1: At the beginning of each phase, we directly use the old model to initialize the current-phase model. i.e., letting φi=φi-1 and ηi=ηi-1.
For your Q2: we don't merge the stable and plastic branches. We train them separately.

Thank you for your reply!!!
So in the i-th phase,the plastic branches learned new knowledge will be keep in plastic branches rather than transfer to stable branches in the i+1-th phase(But plastic branches learned new knowledge in i-th phase will become old knowledge in i+1-th phase,and this konwledge still in plastic branches rather transfer to stable branches,this is my question).In all class-incremental-learning process,the plastic branches learn all new-class knowledge ,and the stable branches only keep base-class knowledge。
Is my understanding correct?
------------------ 原始邮件 ------------------
发件人:                                                                                "yaoyao-liu/class-incremental-learning"                                                                                @.>;
发送时间: 2022年8月18日(星期四) 晚上11:14
@.>;
<EMAIL_ADDRESS>主题: Re: [yaoyao-liu/class-incremental-learning] About initializing learnable parame φi and ηi (Issue #45)
Hi @zjy359
Thanks for your interest in our work!
For your Q1: At the beginning of each phase, we directly use the old model to initialize the current-phase model. i.e., letting φi=φi-1 and ηi=ηi-1.
For your Q2: we don't merge the stable and plastic branches. We train them separately.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@zjy359
In each phase, we update both plastic and stable branches with new data and old exemplars. The stable branch tends to keep more knowledge because it uses a different architecture.
If you have any further questions, please do not hesitate to contact me.
Best,
Yaoyao

Thank you very much!!!
------------------ 原始邮件 ------------------
发件人: "Yaoyao @.>;
发送时间: 2022年8月19日(星期五) 凌晨1:55
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [yaoyao-liu/class-incremental-learning] About initializing learnable parame φi and ηi (Issue #45)
@zjy359
在每个阶段，我们用新的数据和旧的范例更新塑料和稳定的分支。稳定分支倾向于保留更多的知识，因为它使用不同的体系结构。
如果您有任何进一步的问题，请不要犹豫与我联系。
最好的，
瑶瑶
—
直接回复此邮件，在GitHub上查看，或取消订阅.
@.***与>.


HBA / HBD selection fragile, should use elements

            # we only want to show HBD/HBA
            if contacted_lig_at.type == "N" and contacted_res_at.type == "O" or \
                            contacted_lig_at.type == "O" and contacted_res_at.type == "N":
                contacts.append([contacted_lig_at, contacted_res_at])

In util is a bit fragile as it depends on atom types, instead should query atom elements, as types can be anything.

can you point me to the atom element method in MDA?

should just be the contacted_lig.element https://userguide.mdanalysis.org/stable/topology_system.html

fixed in https://github.com/asapdiscovery/choppa/pull/16/commits/d7eada4d6729bb86750b7a71fc327c1b539105b2


Cleanup and extensions

Now runPostCrab should handle injecting in the DB an 'extended' sample if you pass more than 1 config file as argument:

each crab task should preexist in the DB first, and inherit from the same process (aka: the first part of the dataset name should be the same)
the name will be of the type process_extended_i_plus_j_tags where i and j are the original sample IDs
the sumw and nevents are summed, an 'or' of the processed_lumi jsons is done

The PR also include the cleanup from the fact datasets are now bookkept in the Datasets repo

Ah sorry, forgot to add: this was tested and injection seems successful:
>search_SAMADhi.py sample -n *v2.0.4+7415_HHAnalysis_2016-02-14.v0 | grep ggZH_HToBB_ZToNuNu
1206	ggZH_HToBB_ZToNuNu_M125_13TeV_powheg_pythia8_v2.0.4+7415_HHAnalysis_2016-02-14.v0
1255	ggZH_HToBB_ZToNuNu_M125_13TeV_powheg_pythia8_ext1_v2.0.4+7415_HHAnalysis_2016-02-14.v0
1313	ggZH_HToBB_ZToNuNu_M125_13TeV_powheg_pythia8_extended_1206_plus_1255_v2.0.4+7415_HHAnalysis_2016-02-14.v0

PR updated with @blinkseb comments : the tool has been moved to its own script for clarity, current naming is mergeDBsamples, but better ideas are welcome !

Great thanks! I'm wondering also if it would not be easier / prettier to pass sample ids to mergeDBsamples instead of crab tasks? Indeed, it has no links anymore with crab, since it's only merging two database entries into one. What do you think?

yup, can be done, the only slightly troublesome point would be to find back the tags from the urls stored in code_version, but definitely not a show stopper...

If it's finish, would it be possible to squash the 7 last commits into one?

I'm still adding the option to pass SAMADhi IDs instead of crab configs, but then yes, I can squash

Investigating a possible issue in extra sum of weights invalid json

Now rebased (except for the two 'cleaning' commits') !

Can we remove the 'do not merge' ? Or is there still wanted features ?

Ah sorry I forgot it's was me who added it ...


DHCPv6 server should handle request without iana option

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #9843
Special notes for your reviewer:
Release note:

DHCPv6 server handle request without iana option

/cc @phoracek @EdDev @ormergi

DHCPv6 server should handle request without iana option

Could you please reason for it?

DHCPv6 server should handle request without iana option

Could you please reason for it?

I found my VM could send InformationRequest, which not contains IA_NA Option, the request will cause virt-launcher crash.
More details please look at #9843

@EdDev I found my VM could send dhcpv6 Information-request, which not contains IA_NA Option, the request will cause virt-launcher crash.
More details please look at #9843

Thank you.
Could you please update the commit message with this info so we will have it logged for future us?
E.g. something like:

When sending DHCPv6 InformationRequest (not contains IA_NA option), the request will cause
virt-launcher DHCPv6 server crash.

assuring

@EdDev I have updated the commit message. The bugfix works well on my Ubuntu GuestOS :-)

/assign @AlonaKaplan

/test pull-kubevirt-e2e-k8s-1.27-sig-compute

/test pull-kubevirt-unit-test

/retest

Thanks!
/approve

/retest


the latest macosx release reports version 0.0.0 with terraform v0.11.2

The latest macosx release reports version 0.0.0 with terraform v0.11.2 and is not enabled giving then the "apigee not found error"
2018/01/31 22:19:20 [WARNING] found legacy provider "terraform-provider-apigee-v0.0.8_x4"
2018/01/31 22:19:20 [DEBUG] found valid plugin: "apigee-v0.0.8_x4", "0.0.0", "~/terraform/terraform.d/plugins/darwin_amd64/terraform-provider-apigee-v0.0.8_x4"

terraform init --plugin-dir=~/terraform/terraform.d/plugins/darwin_amd64

Initializing provider plugins...

Missing required providers.

The following provider constraints are not met by the currently-installed
provider plugins:

* apigee (0.0.8)

Terraform can automatically download and install plugins to meet the given
constraints, but this step was skipped due to the use of -get-plugins=false
and/or -plugin-dir on the command line.

If automatic installation is not possible or desirable in your environment,
you may manually install plugins by downloading a suitable distribution package
and placing the plugin's executable file in one of the directories given in
by -plugin-dir on the command line, or in the following directory if custom
plugin directories are not set:
    terraform.d/plugins/darwin_amd64
``

How is this issue affecting you?  Is the issue only that terraform is seeing an incorrect version number?

ha investigating a bit more I found the issue is in the name, terraform picks the last - as the name:

NOT WORKING: terraform-provider-apigee-v0.0.8_x4
vs
WORKING terraform-provider-apigee_v0.0.8_x4

maybe rename the release ?

I notice you have a:
_x4

which is mixed with the dashes on the end of your filename..  That isn't how it is named in the release.
What happens if you just use:
terraform-provider-apigee-v0.0.8
I think that the above is the correct way.

@zambien I thought so too, but that is the standard suffix that the terraform adds when doing terraform init (not sure what it means).  I suspect something has changed in the logic that parses the name of the plugin and so the apigee plugin name is picked up as apigee-v0.0.8 with no version

Ok.  I found this issue on the terraform GH repo:
https://github.com/hashicorp/terraform/issues/16874
Do you have a .terraformrc file that is not matching?  That is what the issue reporter was seeing.

@Mistobaan I updated the release binary file names.
Sorry for all the back and forth.... this is an odd issue because I don't have this problem on my Mac or when using Virtualbox running Ubuntu Linux.  I have been trying to recreate the issue but have not been successful.

Yeah is an annoying detail but that is the spec and seems that now is getting enforced.

The naming scheme for provider plugins is terraform-provider-NAME_vX.Y.Z, and Terraform uses the name to understand the name and version of a particular provider binary. Third-party plugins will often be distributed with an appropriate filename already set in the distribution archive so that it can be extracted directly into the plugin directory described above.

from: https://www.terraform.io/docs/configuration/providers.html
I also have another plugin that does not have that exact naming and it reports version=0.0.0
see above reference.

Good to know.  Thanks!

The latest release has the proper naming conventions.  I had meant to close this but forgot.  Please let me know if you experience other issues and thanks for the contribution!


Add example questionnaire with autocomplete control

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #520
Description
Created a sample Questionnaire with autoComplete item control and added to the Gallery App list of Questionnaires
Type
Feature
Checklist

[X] I have read and acknowledged the Code of conduct
[X] I have read How to Contribute
[X] I have read the Developer's guide
[X] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[X] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[ ] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

Please note: This has not been tested locally.

When I try to run the gallery app locally it is crashing at the moment.
There has been no change to underlying code.
Just addition of Questionnaire and an item to the Gallery App


MySQL column and table names containing backticks not escaped properly

Apparently MySQL table and column names are allowed to contain backticks.  Not that I would ever do that on purpose, but anyway...
http://dev.mysql.com/doc/refman/5.0/en/identifiers.html

Permitted characters in quoted identifiers include the full Unicode Basic Multilingual Plane (BMP), except U+0000:

ASCII: U+0001 .. U+007F
Extended: U+0080 .. U+FFFF

...and...

The following statement creates a table named a`b that contains a column named c"d:
mysql> CREATE TABLE `a``b` (`c"d` INT);

So the proper way to escape them is with a double-backtick, just like how PGSQL escapes " with "".
I've updated the tests to check that backticks in identifiers are being properly escaped.  It's in PR #752.
Related to issue #737.

This seems to work correctly already:
> knex('tab`le').select('d`oh').where('i`d', 'm´e').toString()
'select `d``oh` from `tab``le` where `i``d` = \'m´e\''


fix(web): version and deployment patch editing

Codecov Report
Base: 17.97% // Head: 17.97% // No change to project coverage :thumbsup:

Coverage data is based on head (e3802f5) compared to base (b16279b).
Patch has no changes to coverable lines.

: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
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #459   +/-   ##
========================================
  Coverage    17.97%   17.97%
========================================
  Files           57       57
  Lines         5965     5965
========================================
  Hits          1072     1072
  Misses        4818     4818
  Partials        75       75

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


github-actions: Ensure cancel previous run job never fails.

Testing plan:

Updated the workflow API url to be incorrect and made sure this job didn't fail.
Tested by force-pushing and making sure that the previous runs cancel successfully.

This is great, merged, thanks @priyank-p!


Cannot use a dependency in a yarn workspace

Command
build, serve
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Hi.
Thank you for the awesome work around angular. I don't know whether this is actually a bug or I did not understand something about any of yarn, typescript, esm, commonjs, angular, webpack...
When using a simple angular project as a yarn workspace, I cannot use the exports of a sibling project.
Minimal Reproduction
Here is the description of how to reproduce the issue, I attached the whole project as well
myproject.zip

create a yarn workspace root package.json, pointing to the two workspaces we will create now

{
  "name": "myproject",
  "private": true,
  "workspaces": [
    "angular",
    "lib"
  ]
}

create a simple angular project
ng new angular (then, keep the default value for everything that is asked)
add the "private": true entry in its package.json to indicate it is a workspace
create a very simple library called lib
with the following package.json

{
    "name": "@mylib/lib",
    "version": "0.1.0",
    "private": true
}

and the following index.ts
export const somevariable = "something";

indicate in the package.json of the angular workspace the dependency toward the sibling library, adding "@mylib/lib": "^0.1.0", in its dependency section.
in the root component on the angular workspace, make use of the variable
import { somevariable } from '@mylib/lib';
and then, set the title to somevariable

...
export class AppComponent {
  title = somevariable;
}
...

run ng serve

You will see that somevariable does not appear.

Strangely, changing import { somevariable } from '@mylib/lib'; into import { somevariable } from '../../../lib'; works around this issue. You need to rm -rf .angular/ && ng serve to actually see the impact of the change.

Running ng build in development mode, I could find out that the webpack export was actually empty where somevariable should have been.
the bundled code looks like this
class AppComponent {
  constructor() {
    this.title = _mylib_lib__WEBPACK_IMPORTED_MODULE_0__.somevariable;
  }
}

With _mylib_lib__WEBPACK_IMPORTED_MODULE_0__ defined like this /* harmony import */ var _mylib_lib__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! @mylib/lib */ 6479);
Then, the content associated to 6479 is the following
/***/ 6479:
/*!***********************!*\
  !*** ../lib/index.ts ***!
  \***********************/
/***/ (() => {

/***/ })

I assume there has been some too eager tree-shaking somewhere.
Thank you again for angular. I hope this bug report is helpful. In case it is just me doing something wrong, I would be very welcome if you provided some guidance about how to make this work.
My best,

I realized that I did not give the instruction to use the myproject.zip

unzip myproject.zip
cd myproject
yarn
cd angular
ng serve

Also, I realized that angular (or node, or typescript, or webpack, or... I'm not sure what tools is doing this) behaves differently when I move lib into node_modules.
import { somevariable } from '../../../lib'; works well
but moving lib to node_modules and import { somevariable } from '../../../node_modules/lib'; fails.
Therefore, to the best of my understanding, the fact that import { somevariable } from '@mylib/lib'; gets the content from ../node_modules/@mylib/lib causes the issue.
For other dependencies work well in general and are also got from node_modules, I assume there is something I forgot to do to avoid @mylib/lib from being dropped. I hope someone in here could provide me with a hint of what I did wrong.

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.

Thank you for your answer. I will do this. At least this confirms that I need some guidance :-)


link with 3dcitydb, 3d-we-map, and plugins using git-submodule

This PR dramatically simplifies the workflow for managing the resources e.g. 3dcitydb, 3dcitydb-web-map, plugins (spreadsheet Generator und ADE Manager) by means of the Git Submodule.
It is now possible to specify the branch names of the corresponding resources In the properties.gradle file. Wenn running e.g. the buildInstaller task, each submodule will be automatically fetched from the corresponding branch in Github, built, and packaged into the importer-exporter installer.

Thanks @yaozhihang, long wanted feature :-)


Update README.md, correct iOS build pre-action

There's a comment as part of the review of pull request #281 by @danielwinkler that the build pre-action for iOS should copy the .env.staging (or whatever) file to ${PROJECT_DIR}/../.env, which turns out to be correct.  This change updates the README to reflect that (and also addresses his other comment about the second reference to /tmp/envfile).
@maharjanaman also mentioned on that pull request a problem with the script, and I believe that was from not having a "Provide build settings from" selection, so I added a comment about that, as well.

With Xcode 11.5, I didn't need to select a value for "Provide build settings from".
This worked,
cp ${PROJECT_DIR}/../.env.production .env
"${SRCROOT}/../node_modules/react-native-config/ios/ReactNativeConfig/BuildXCConfig.rb" "${SRCROOT}/.." "${SRCROOT}/tmp.xcconfig"

Somehow, building the app took env.production but I haven't run the app on actual device.

@maharjanaman I found that it was building with whatever happened to be in .env at the time, not copying the correct .env.production (or whichever) file into place.  If you remove the .env file that happens to be there now, does it rebuild correctly for you?  When I do it, all my Config values are undefined, because no .env file gets created, unless I make the changes I noted.

@jjd314 I did had to select a value for "Provide build settings from". And also I have to use cp ${PROJECT_DIR}/../.env.staging ${PROJECT_DIR}/../.env.


Scan PDF file having PDF417 Barcode

Hi,
Is this possible?

This would work fine against a PDF417 barcode, it has enogh resolution, but it wont do the PDF related work for you.


bug: lazy's menu conflict with neo-tree

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.5
Operating system/version
Arch Linux6.6.22-lts  plasma6  wayland
Describe the bug
The bug is very easy to reproduce.
1.use "l" or"l" open lazy's menu
2.use"e" open neo-tree
then you can find you lose control of lazy's menu, even if close the neo-tree or reopen lazy can't get focus on lazy, can't control or close it.  you can quit nvim but can't just close the lazy. it' s little but annoying
Steps To Reproduce
1.use "l" or"l" open lazy's menu
2.use"e" open neo-tree
3.done
Expected Behavior
solve the conflict between lazy and neotree
Repro
I just install lazyvim didn't change anything, it's completely new

my fault never mind