Fixing issues with a few breaking changes

ARM API Information (Control Plane)
A couple of these were handled in the previous PR: https://github.com/Azure/azure-rest-api-specs/pull/23637
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

@mikekistler Please help review the breaking change in this PR. Thanks a lot.

@jeremyfrosti Please fix CI checks errors (Model validation), you can refer to this comments or  https://aka.ms/ci-fix on how to fix it.

@jeremyfrosti , can you work on the model validation fix?

@qiaozha Please help review JS breaking change.
@raych1 Please help review Go breaking change.
@live1206   Please help review azure-sdk-for-net-track2 error.

@chidozieononiwu Please help review the swaggerApiView error.

@live1206 Please help review azure-sdk-for-net-track2 error.

Failing due to the below error:
error   | PreCheck/DuplicateSchema | Duplicate Schema named 'DatabaseVulnerabilityAssessmentRuleBaseline' (1 differences):
02:27:31.370 cmdout 	[Invoke-GenerateAndBuildV2.ps1]     - properties.properties.$ref: "#/components/schemas/schemas:511" => "#/components/schemas/schemas:1836"; This error can be temporarily avoided by using the 'modelerfour.lenient-model-deduplication' setting.  NOTE: This setting will be removed in a future version of @autorest/modelerfour; schemas should be updated to fix this issue sooner than that.
02:27:31.371 cmdout 	[Invoke-GenerateAndBuildV2.ps1]   error   | PreCheck/DuplicateSchema | Duplicate Schema named 'DatabaseVulnerabilityAssessmentRuleBaselineProperties' (2 differences):
02:27:31.371 cmdout 	[Invoke-GenerateAndBuildV2.ps1]     - properties.baselineResults.x-ms-identifier: [] =>
02:27:31.371 cmdout 	[Invoke-GenerateAndBuildV2.ps1]     - properties.baselineResults.$ref: "#/components/schemas/schemas:512" => "#/components/schemas/schemas:1837"; This error can be temporarily avoided by using the 'modelerfour.lenient-model-deduplication' setting.  NOTE: This setting will be removed in a future version of @autorest/modelerfour; schemas should be updated to fix this issue sooner than that.
02:27:31.519 cmdout 	[Invoke-GenerateAndBuildV2.ps1]
But it's not introduced in this PR, we can ignore it for now, will need to handle it while generating .NET SDK.

@qiaozha Please help review JS breaking change.

@chidozieononiwu, @maririos  Please help review the swaggerApiView error ASAP. Thanks a lot.


[Table] 希望支持 rowSpan colSpan

ElementUI version

OS/Browers version

Vue version

Reproduction Link

Steps to reproduce
What is Expected?
希望能支持 类似的 rowSpan colSpan
What is actually happening?

+1

+1
合并行这种场景还是比较常见的.
官方忙不过来，有没有大神贡献下这个特性

+1

+1

+1

+1

+1

+1

+99

Hope support soon.

+1

+666

+999

由于目前还不支持这个功能，所以我改了它的源码，能实现自己想要的需求。有需要的，可以拿去参考一下:https://github.com/scq000/vue-element-enhanced-table

`

`
这是我自己写的一个table组件，这种可以根据内容，相同并且相邻的单元格竖排合并。

There is another related ticket that is tagged "in plan." Would the feature, how it is currently planned, provide the ability to do something like the following? Perhaps passing in an attribute (render function) to the Table component that will be called for every row, and allow the user to modify the row (change td/append or prepend another row). If this is not how it is currently planned, would another feature request for this be acceptable?

我自己写了一个基于vue2.x 的table组件，目前已支持 colSpan 和 rowSpan,感兴趣的可以看下:
http://doc.huangsw.com/vue-easytable/app.html#/table?anchor=table-cell-merge

Added in 2.0: http://element.eleme.io/2.0/#/zh-CN/component/table#he-bing-xing-huo-lie

@Leopoldthecoder Would you mind if I created another feature request to have a way to insert divider rows for grouping the data, something similar to the table in my previous comment above? The way it is implemented in 2.0 would require the divider row to be present in the bound data itself, but there may be instances where that would not be semantically correct.

@syn-zeta Having something like divider rows seems a little verbose to me, since ultimately it renders a full-span row that can be done with the current implementation.

@Leopoldthecoder Right, it can be done with the current implementation, but that would require the "grouping criteria" to be part of the data itself. So, the date would exist as the serial number property in the above image, which seems like a hack. Perhaps a generic way to group data may be useful to enough users in the future to consider this.

我再合并行列上遇到问题了 哪位可以帮我看下问题？ #10702

+1

文档中目前支持的合并单元格仅支持数组元素对象中全是基本类型的情况，多层数组嵌套类型数据的单元格拆分场景建议支持一下，目前业务中该类问题都是通过嵌套一个el-table解决的，但是边距边框处理与element原生的样式没有很好的兼容


Split the project

Hey Team,
I am going to work on code based on this project and split the request and the display into a data source and a panel. The Goal is to have a data source with the ability to have multiple URLs that you can call. This should allow you to make more dynamic UI experiences with sourced data from more than one location. Also, it will allow you to use generic data sources and make custom panels without having to make a new plugin each time.
I'm reaching out to see if you wanted to update this project and make a new one or make two totally new ones, or I can just make new ones and not bother you.
Let me know
Thanks

Hello @provenvelocity has there been any progress on your effort?


S2U-11 <IP_ADDRESS> Tests & Quizzes: Global variables in calculated questions

https://sakaiproject.atlassian.net/browse/S2U-11

Thanks!

Makes sense to me.


fix makie for RGB rasters

We're currently processing your upload.  This comment will be updated when the results are available.


Certificate not written to wallet file on import (until wallet tx)

The Issue
When a channel claim is imported to another wallet, the wallet does not write the claim certificate until there's some transaction to the wallet (incoming or outgoing). If you close the daemon and restart, the channel will no longer show in channel_list.
Steps to reproduce

Send a channel from one install to another
After importing, run channel_list_mine - confirmed there
Check default_wallet for claim - not there
Make a tx to the new wallet.
Check default_wallet for claim - now it's there.

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

This seems like a pretty large bug.


Harmonize page templates across Sections level pages only

Section Level Pages

Grid level changes and No background and larger upscale header to 50px (desktop) 32 (mobile) for section level pages
Featured Text are editable via Prismic for the following section level pages

Visit-us
Collections
Get Involved
What's on

closes #5432

Looking good – just wondering if we can get all of the non-standard font-size stuff into one place?

yeah just done that now.. ive placed this in common/styled as I hope this would be the best place for this if we need to maintain this.


[BUG] vendor package typeguard installed to entrypoints

setuptools version
setuptools==71.0.1
Python version
python 3.8
OS
arch linux
Additional environment information
No response
Description
bug introduced by https://github.com/pypa/setuptools/compare/v71.0.1..v71.0.0#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711R181-R183
pytest would search the plugins by the entrypoint pytest11. the vendor/typeguard got installed to the entrypoint, but without a correct metadata. Got error message
Traceback (most recent call last):
  ... unrelated ..
  File "myenv/lib/python3.8/site-packages/_pytest/config/__init__.py", line 159, in main
    config = _prepareconfig(args, plugins)
  File "myenv/lib/python3.8/site-packages/_pytest/config/__init__.py", line 346, in _prepareconfig
    config = pluginmanager.hook.pytest_cmdline_parse(
  File "myenv/lib/python3.8/site-packages/pluggy/_hooks.py", line 513, in __call__
    return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
  File "myenv/lib/python3.8/site-packages/pluggy/_manager.py", line 120, in _hookexec
    return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
  File "myenv/lib/python3.8/site-packages/pluggy/_callers.py", line 139, in _multicall
    raise exception.with_traceback(exception.__traceback__)
  File "myenv/lib/python3.8/site-packages/pluggy/_callers.py", line 122, in _multicall
    teardown.throw(exception)  # type: ignore[union-attr]
  File "myenv/lib/python3.8/site-packages/_pytest/helpconfig.py", line 106, in pytest_cmdline_parse
    config = yield
  File "myenv/lib/python3.8/site-packages/pluggy/_callers.py", line 103, in _multicall
    res = hook_impl.function(*args)
  File "myenv/lib/python3.8/site-packages/_pytest/config/__init__.py", line 1152, in pytest_cmdline_parse
    self.parse(args)
  File "myenv/lib/python3.8/site-packages/_pytest/config/__init__.py", line 1501, in parse
    self._preparse(args, addopts=addopts)
  File "myenv/lib/python3.8/site-packages/_pytest/config/__init__.py", line 1388, in _preparse
    self.pluginmanager.load_setuptools_entrypoints("pytest11")
  File "myenv/lib/python3.8/site-packages/pluggy/_manager.py", line 421, in load_setuptools_entrypoints
    plugin = ep.load()
  File "myenv/lib/python3.8/site-packages/setuptools/_vendor/importlib_metadata/__init__.py", line 183, in load
    module = import_module(match.group('module'))
  File "myuser/.pyenv/versions/3.8.19/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 961, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "myenv/lib/python3.8/site-packages/setuptools/_vendor/typeguard/__init__.py", line 21, in <module>
    from ._importhook import ImportHookManager as ImportHookManager
  File "myenv/lib/python3.8/site-packages/setuptools/_vendor/typeguard/_importhook.py", line 35, in <module>
    OPTIMIZATION = "typeguard" + "".join(version("typeguard").split(".")[:3])
  File "myenv/lib/python3.8/site-packages/setuptools/_vendor/importlib_metadata/__init__.py", line 982, in version
    return distribution(distribution_name).version
  File "myenv/lib/python3.8/site-packages/setuptools/_vendor/importlib_metadata/__init__.py", line 475, in version
    return self.metadata['Version']
  File "myenv/lib/python3.8/site-packages/setuptools/_vendor/importlib_metadata/_adapters.py", line 54, in __getitem__
    raise KeyError(item)
KeyError: 'Version'

succeeded with version 71.0.0
Expected behavior
run successfully
How to Reproduce
as description
Output
as description

I think I'm seeing another aspect of the same problem where this line https://github.com/pypa/setuptools/blob/f2a6bb190202577595a45ceebdedb0016b7cf864/setuptools/_vendor/typeguard/_importhook.py#L35
fails with
...
  File "/Users/clm/dev/astropy-project/astropy-compatbility-checks/numpy-dev/.venv/lib/python3.12/site-packages/setuptools/_vendor/typeguard/__init__.py", line 21, in <module>
    from ._importhook import ImportHookManager as ImportHookManager
  File "/Users/clm/dev/astropy-project/astropy-compatbility-checks/numpy-dev/.venv/lib/python3.12/site-packages/setuptools/_vendor/typeguard/_importhook.py", line 35, in <module>
    OPTIMIZATION = "typeguard" + "".join(version("typeguard").split(".")[:3])
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'split'

But I'm only seeing this in the context of launching pytest in a complicated setup and I haven't found how to reproduce this simply yet.

Indeed, when I list the files in the wheel, I see that only the *.txt files from _vendor/*.dist-info are being included (no METADATA or RECORD or WHEEL).
I can repro the issue pretty simply:
 draft 🐚 py -m venv .venv
 draft 🐚 py -m pip install -q setuptools
 draft 🐚 py -c "import setuptools; import typeguard"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
    import setuptools; import typeguard
                       ^^^^^^^^^^^^^^^^
  File "/Users/jaraco/draft/.venv/lib/python3.13/site-packages/setuptools/_vendor/typeguard/__init__.py", line 21, in <module>
    from ._importhook import ImportHookManager as ImportHookManager
  File "/Users/jaraco/draft/.venv/lib/python3.13/site-packages/setuptools/_vendor/typeguard/_importhook.py", line 35, in <module>
    OPTIMIZATION = "typeguard" + "".join(version("typeguard").split(".")[:3])
                                         ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'split'

I'm going to yank 71.0.1, as it's worse than 71.0.0.

This issue would not have occurred if setuptools was using setuptools_scm for source code discovery.
I've confirmed that the failing test now passes when including all files from the vendored dependencies.

Confirming that the latest patch 71.0.2 fixes the issue, see:

[failed run[(https://github.com/inveniosoftware/invenio-opendefinition/actions/runs/9991927183/job/27615757684#step:4:43)
successful run

Thanks a lot for the fix :D


build: set build environment variables

Currently, GitHub action is failing to push the helm charts and some changes from #2151 need to be backported to release-v3.3 branch
Note:- build is failing at https://github.com/ceph/ceph-csi/runs/3628967761?check_suite_focus=true
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@humblec PTAL


Fixes missing example controllers dependency

As, some scripts use the uses the moveit_commander (e.g. move_to_start.py) it should be added as a ROS dependency.

@gollth Done


feat: add a11y-alt-text bot

PR Checklist

[X] Addresses an existing open issue: fixes #825
[X] That issue was marked as status: accepting prs
[X] Steps in CONTRIBUTING.md were taken

Overview
This PR adds a new bot to check for alt text in issue, PR or discussion comments.
With the .yml file, I followed the suggestion in the official repo and adapted the used version to the latest version.

Sure, I added it to a repo of mine and opened an issue with images with missing alt texts: https://github.com/DerTimonius/til-blog/issues/7

Tests are fixed now 🙂

@JoshuaKGoldberg I added the bot to the created files (good idea, did not think about that) - had to change a few types along the way. Also I added the .yml file to the filesExpectedToBeChanged Set in the migrate-test-e2e.js file - I wasn't sure if I should change the formatting of the original file or if I should add it to the set instead and went with the latter. I hope that's okay, if not I'll change it 🙂

Also I added the .yml file to the filesExpectedToBeChanged Set in the migrate-test-e2e.js file - I wasn't sure if I should change the formatting of the original file or if I should add it to the set instead and went with the latter. I hope that's okay, if not I'll change it 🙂

Mmhh yeah it's the other way around: the files in the repo should match the templates. I'd like to whittle that list down of "these are intentionally different" to zero over time. Change request: have the .yml file match the template. Thanks!

@JoshuaKGoldberg The latest change also changes the location of the if statement in the .yml file. Again, I'm a bit unsure wether the if should be in the steps as it is now or in the same scope as runs-on and steps? I already fixed this locally, but I wanted to wait for your feedback on this one


Fix pkgconfig

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Move lib/pkgconfig to omniorb-libs.
Without those files, pkg-config can't find the libraries when installing only omniorb-libs.
This should avoid having to install omniorb as build/host requirement.
I also noticed a lot of extra files included in omniorb when testing the build locally. The output was relying on conda-build to filter on new files but that's not supposed to work when specifying multiple outputs. Not sure why it worked until now.
I listed explicitly all files to include (using some regex on files - can't be used on directories).
I compared the list of files with previous packages.

@conda-forge-admin, please rerender


Generate Inline Schema

Hello,
Is it possible to configure Generator not to generate Definitions, but put all property's fields in the same tree?
For example Newtonsoft.JSchemaGenerator has option SchemaLocationHandling.Inline.
Using it - I can get schema like this:
{
  "type": "object",
  "properties": {
    "criteria": {
      "type": "object",
      "properties": {
        "basic": {
          "type": "object",
          "properties": {
            "account": {
              "type": "object",
.....

I can't find if there is a such option in NJsonSchema.
Thanks

I think there is no out of the box option for that. You’d need to customize the generator..


contrib.nccl.broadcast raises UnimplementedError

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): Docker image tensorflow/tensorflow:nightly-devel-gpu
TensorFlow version (use command below): nightly (also 1.7rc1)
Python version: 2.7
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: 7.0.5
GPU model and memory: NA
Exact command to reproduce:

Copying repro from https://github.com/tensorflow/tensorflow/issues/15425#issuecomment-361835192:
import tensorflow as tf
import tensorflow.contrib.nccl as nccl

if __name__ == '__main__':
    devices = ['/gpu:0', '/gpu:0']
    with tf.device(devices[0]):
        data0 = tf.constant([1.1, 2.2, 3.3, 4.4])
        received_tensor = nccl.broadcast(data0)
    received_tensors = []
    for device in devices[1:]:
        with tf.device(device):
            received_tensors.append(tf.identity(received_tensor))
    sess = tf.Session()
    sess.run(received_tensors)

Describe the problem
Traceback (most recent call last):
  File "<stdin>", line 11, in <module>
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 906, in run
    run_metadata_ptr)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1141, in _run
    feed_dict_tensor, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1322, in _do_run
    run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1341, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.UnimplementedError: This op should be replaced during graph optimization.
	 [[Node: NcclBroadcast = NcclBroadcast[T=DT_FLOAT, shape=[4], _device="/job:localhost/replica:0/task:0/device:GPU:0"](Const)]]

Caused by op u'NcclBroadcast', defined at:
  File "<stdin>", line 5, in <module>
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/nccl/python/ops/nccl_ops.py", line 187, in broadcast
    return gen_nccl_ops.nccl_broadcast(input=tensor, shape=tensor.shape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/nccl/ops/gen_nccl_ops.py", line 132, in nccl_broadcast
    "NcclBroadcast", input=input, shape=shape, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3305, in create_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1669, in __init__
    self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access

UnimplementedError (see above for traceback): This op should be replaced during graph optimization.
	 [[Node: NcclBroadcast = NcclBroadcast[T=DT_FLOAT, shape=[4], _device="/job:localhost/replica:0/task:0/device:GPU:0"](Const)]]

I get same issue on install from source tf-1.7.0

Get the same issue on TF-1.7 installing from source.

Nagging Assignee @chsigg: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @chsigg: It has been 32 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @chsigg: It has been 47 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @chsigg: It has been 62 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @chsigg: It has been 77 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Any updates on this? Is there any way to workaround this?

Nagging Assignee @chsigg: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hi @chsigg, is there any news here? I use tf 1.9.0 and have exactly the same problem. Thanks.

Nagging Assignee @chsigg: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Still got this problem with tf1.10

Any news here? @chsigg

Nagging Assignee @chsigg: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

The file nccl_rewrite.cc was added by ca8af1d0dbb605087a4f8ae076188f2b9a26b1ba on Oct 15 2017.  Was it working back then?

Nagging Assignee @chsigg: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @chsigg: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @chsigg: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Broaden virtualenv in project option to support custom location

[ X] I have searched the issues of this repo and believe that this is not a duplicate.

Issue
As mentioned in #278 and other issues (hmm... can't find  now). Widen the ability to support venv locations not just in the project folder but a user specified one.
This will have the benefit of increasing compatibility with more user workflows and tooling, eg IDEs that looks for venvs in the ~./virtualenvs location.

This already supported by setting the settings.virtualenvs.path setting. See https://poetry.eustace.io/docs/configuration/#settingsvirtualenvspath-string

Thanks, sorry I missed that.


Very simple tamplate not working as intended

Describe the bug
I'm trying out the below - very simple - template with 4 barcodes next to each other at the top of the page.
Contrary to expected,

the barcodes are rendered on 4 different pages (should be one single page with 4 barcodes)
the 2nd and 3rd barcode are rendered on the same spot
the 4th barcode is rendered on the place where I expect the 3rd barcode

To Reproduce
  const template: Template = {
    basePdf: BLANK_PDF,
    plugins: { code128: barcodes.code128 },
    schemas: [
      [
        {
          name: 'barcode_1',
          type: 'code128',
          position: { x: 5, y: 3 },
          width: 42.5,
          height: 15.2,
          barColor: '#000000',
          textColor: '#000000',
          includeText: true,
          content: '123456789',
        },
        {
          name: 'barcode_2',
          type: 'code128',
          position: { x: 57.5, y: 3 },
          width: 42.5,
          height: 15.2,
          barColor: '#000000',
          textColor: '#000000',
          includeText: true,
        },
        {
          name: 'barcode_3',
          type: 'code128',
          position: { x: 105.0, y: 3 },
          width: 42.5,
          height: 15.2,
          barColor: '#000000',
          textColor: '#000000',
          includeText: true,
        },
        {
          name: 'barcode_4',
          type: 'code128',
          position: { x: 157.5, y: 3 },
          width: 42.5,
          height: 15.2,
          barColor: '#000000',
          textColor: '#000000',
          includeText: true,
        },
      ],
    ],
  }
  const inputs = [
    { barcode_1: 'MLB02W3000' },
    { barcode_2: 'MLB02W3001' },
    { barcode_2: 'MLB02W3002' },
    { barcode_3: 'MLB02W3003' },
  ]

  generate({
    template,
    plugins: { code128: barcodes.code128, text },
    inputs,
  }).then((pdf) => {
    const blob = new Blob([pdf.buffer], { type: 'application/pdf' })
    window.open(URL.createObjectURL(blob))
  })

Expected behavior
barcodes next to each other at the top of the page.
Your Environment
"@pdfme/common": "^5.0.0",
    "@pdfme/generator": "^5.0.0",
    "@pdfme/ui": "^5.0.0",

Chrome 129.0.6668.60 (64-bits)
node v20.17.0
Windows 11, 64 bit

Your Error Log
No error log

Additional context
No response

A couple of issues here:

You have put your inputs on different pages. It should be a single object for these 4 items, not separate objects in the array.
You have used barcode_2 twice.

I think this is what you want:
const inputs = [
  { barcode_1: 'MLB02W3000', barcode_2: 'MLB02W3001', barcode_3: 'MLB02W3002', barcode_4: 'MLB02W3003' },
]

I'll close this one off, i hope you have what you need  😄

OK, thanks for your help !


[BEAM-12976] Enable projection pushdown in runners

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run PostCommit_Java_Dataflow

Run PostCommit_Java_Dataflow

R: @apilloud

run java precommit

portable python precommit is broken https://issues.apache.org/jira/browse/BEAM-14122

Looks like spanner test failure was just a flake. The optimizer didn't run on it, so it's unrelated.


must-gather: collect some of the odf-operator and storagesystem relat…

This commit collects log for the following:

oc get --all-namespaces storagesystem
oc logs odf-operator
oc logs odf-console

Signed-off-by: yati1998<EMAIL_ADDRESS>

@iamniting @jarrpa @umangachapagain Anything else we need to add here?

There sure is! :grin:

We need oc get storagesystem --all-namespaces -oyaml and optionally oc describe storagesystem --all-namespaces . Current command only gives us a list.

@umangachapagain Do we need to collect logs for both oc describe and get commands? or oc describe works?

We need oc get storagesystem --all-namespaces -oyaml and optionally oc describe storagesystem --all-namespaces . Current command only gives us a list.

done. please review it.

@iamniting @jarrpa @umangachapagain Anything else we need to add here?

There sure is!

@jarrpa please do let me know what else is required.

/retest

/retest-required

/ok-to-test

/cherry-pick release-4.9


Import error: from models.ternausnet2 import TernausNetV2

Hi, i was trying to use the demo.ipynb file given, but it throws an import error.
ImportError                               Traceback (most recent call last)
<ipython-input-2-b7b37541d9f9> in <module>
----> 1 from models.ternausnet2 import TernausNetV2

~\Downloads\worldview3\TernausNetV2\models\__init__.py in <module>
----> 1 from .resnext import *
      2 from .resnet import *
      3 from .wider_resnet import *
      4 from .densenet import *

~\Downloads\worldview3\TernausNetV2\models\resnext.py in <module>
      5 import torch.nn as nn
      6
----> 7 from modules import IdentityResidualBlock, ABN, GlobalAvgPool2d
      8 from ._util import try_index
      9

~\Downloads\worldview3\TernausNetV2\modules\__init__.py in <module>
----> 1 from .bn import ABN, InPlaceABN, InPlaceABNSync
      2 from .functions import ACT_RELU, ACT_LEAKY_RELU, ACT_ELU, ACT_NONE
      3 from .misc import GlobalAvgPool2d, SingleGPU
      4 from .residual import IdentityResidualBlock
      5 from .dense import DenseModule

~\Downloads\worldview3\TernausNetV2\modules\bn.py in <module>
      8     from Queue import Queue
      9
---> 10 from .functions import *
     11
     12

~\Downloads\worldview3\TernausNetV2\modules\functions.py in <module>
     16                     "inplace_abn_cuda_half.cu"
     17                 ]],
---> 18                 extra_cuda_cflags=["--expt-extended-lambda"])
     19
     20 # Activation names

C:\ProgramData\Anaconda3\lib\site-packages\torch\utils\cpp_extension.py in load(name, sources, extra_cflags, extra_cuda_cflags, extra_ldflags, extra_include_paths, build_directory, verbose, with_cuda, is_python_module)
    642         verbose,
    643         with_cuda,
--> 644         is_python_module)
    645
    646

C:\ProgramData\Anaconda3\lib\site-packages\torch\utils\cpp_extension.py in _jit_compile(name, sources, extra_cflags, extra_cuda_cflags, extra_ldflags, extra_include_paths, build_directory, verbose, with_cuda, is_python_module)
    822     if verbose:
    823         print('Loading extension module {}...'.format(name))
--> 824     return _import_module_from_library(name, build_directory, is_python_module)
    825
    826

C:\ProgramData\Anaconda3\lib\site-packages\torch\utils\cpp_extension.py in _import_module_from_library(module_name, path, is_python_module)
    965 def _import_module_from_library(module_name, path, is_python_module):
    966     # https://stackoverflow.com/questions/67631/how-to-import-a-module-given-the-full-path
--> 967     file, path, description = imp.find_module(module_name, [path])
    968     # Close the .so file after load.
    969     with file:

C:\ProgramData\Anaconda3\lib\imp.py in find_module(name, path)
    294         break  # Break out of outer loop when breaking out of inner loop.
    295     else:
--> 296         raise ImportError(_ERR_MSG.format(name), name=name)
    297
    298     encoding = None

ImportError: No module named 'inplace_abn'

conda version:  4.6.14
python: 3.7
cuda: 10.1
pytorch: 1.0
gcc: 5.4

I'm running into the same problem - did you end up finding a solution?

Have you found some solution?

Hello folks, I faced the same problem and was stuck on the same for many days. Following solution is applicable for MacOS and Linux as I tried on these both only. Firstly, error is arising due to non compatibility of entire code base of git (and the versions they are operating on) and your respective versions of Python, Pytorch and OpenCV. So, first primarily, I'll recommend you to make a virtual environment with Python 3.6.8. Warning- DO NOT bypass this step or else you wont be able to install PyTorch 0.4 and thus proceed with pip install of each dependency as it is. (Torch 0.4, etc). You'll also need to install extra libraries of matplotlib and skimage, Then, you wont be getting this error. And further, if you dont want to run it on GPU and rather on CPU then follow this link for the answer. https://github.com/ternaus/TernausNetV2/issues/16#issuecomment-691624322 . Comment, if problem still persists.


Upgrading/Updating Packages Deleted "Taskbar Search Icons" for Native Windows Apps

Brief description of your issue
I've filed an issue for this in the past #1005, but I thought it had something to do with how my OneDrive was synching the fact that the icon files were deleted.
While that may have been true—can no longer say if that's definitively true or not—but this morning I decided to run winget upgrade --all on my machine, because it had been a long while since I've booted up my personal computer (since I work from home, I tend to use my work MacBook Pro for everything).
After executing the command, recalling my concerns when I filed the original ticket last time, I double-checked to see the icons were still there, and, sure enough, they were gone!
To better explain what the actual "symptoms" are:
In the Taskbar Search Field (or if you have it hidden like I do, you hit the Window key to reveal the Start menu and then just start typing), I'll type in the name of an app that was either pre-installed as a part of the Windows 10 OS natively (i.e., Calculator, Settings, etc.) or had been installed through the Microsoft Store (i.e., Twitter, Ubuntu 20.04, Audible).
When those apps would appear in the app search results, their icons are broken. All the other icons that don't fit in the two categories I described, such as Docker Desktop, appear as they should without problem.
Now, I had checked the icons earlier when I ran my Windows Updates, because I often use the Taskbar search as sorta "Windows Spotlight or less impressive Alfred", so I know for a fact that the icons were present and worked prior to the updates.
I am open-minded to other possibilities, because anything is possible, but the timing of everything leads me to believe that the deletion of the icons has to be tied into Winget somehow.
Steps to reproduce
Environment: This was completed on a custom-built machine running Windows 10 Pro with all drivers and the OS 100% up-to-date.
Winget Version: v1.2.10941.

Open Windows PowerShell with Admin Privileges.
Enter winget upgrade --all then press Enter.
Press the Window Key to have the Start Menu appear.
With the Start Menu in focus, enter the name of an application that would either be pre-installed natively on Windows (i.e., Calculator or Settings) or any app acquired through the Microsoft Store, such as the MS version of Twitter or WSL2 Ubuntu 20.04.
When the entered apps appear in the search results, the default "broken icon" image will appear in place of the proper icons that are supposed to appear instead.

Expected behavior
The app icons for the native Windows and MS Store apps found in the Taskbar Search results should not break either after or during the Winget upgrade/update process.
Actual behavior
The app icons for the native Windows and MS Store apps found in the Taskbar Search results are broken and are instead showing the default "broken icon link" image.
Environment
Windows Package Manager v1.2.10941
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19044.1741
Package: Microsoft.DesktopAppInstaller v1.17.10941.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Let me know if y'all need additional OS, software, or hardware information.

This seems to be caused by an application upgrade as all WinGet does is download the installer and run it with silent switches which is very different from what Chocolately and Scoop does so any modifications done to your machine is done purely by the installer rather than WinGet.
Do you know what apps do you have installed on that machine? as there are some apps don't work nicely with winget upgrade --all and there are also some apps don't work nicely with the built-in application updater.

Apologies for the late response. Things have been crazy on my end. lol
I have numerous applications installed on my machine. What I could do is list all the apps I have installed on my machine via Scoop & Chocolatey along with the ones listed through Winget.
Scoop
hadolint 2.10.0

Chocolatey
autohotkey.portable <IP_ADDRESS>
chocolatey 1.1.0
chocolatey-compatibility.extension 1.0.0
chocolatey-core.extension 1.4.0
chocolatey-font-helpers.extension 0.0.4
chocolatey-windowsupdate.extension 1.0.4
discord 1.0.9004
discord.install 1.0.9004
DotNet4.5.2 <IP_ADDRESS>40902
expressvpn <IP_ADDRESS>
FiraCode 6.2
Firefox 101.0
gh 2.11.3
git 2.36.0
git.install 2.36.0
GoogleChrome 102.0.5005.63
KB2919355 1.0.20160915
KB2919442 1.0.20160915
KB2999226 1.0.20181019
KB3033929 1.0.5
KB3035131 1.0.3
kubernetes-cli 1.24.1
kubernetes-helm 3.8.2
Minikube 1.25.2
msiafterburner <IP_ADDRESS>11212
notepadplusplus 8.4.2
notepadplusplus.install 8.4.2
nvm 1.1.9
nvm.install 1.1.9
postgresql 14.3.1
postgresql14 14.3.1
vcredist140 14.32.31326
vlc <IP_ADDRESS>
vlc.install <IP_ADDRESS>
vscode 1.67.2
vscode.install 1.67.2

Winget/Windows

(I had to take a screenshot since I couldn't copy & paste the formatting cleanly.)

@KerickHowlett
Thanks for sharing!
I've found I can copy paste the text and use the three backticks on a line above and below the terminal output to improve formatting.

@KerickHowlett
Thanks for sharing!
I've found I can copy paste the text and use the three backticks on a line above and below the terminal output to improve formatting.

I had tried that originally, but the alignment of everything had still came out funky.
But do you need me to send that to you in copyable text instead of a screenshot?

No, I'm just trying to narrow down which installer is messing with icons.

Oh! Well, it happened after running the winget update, so are you asking me which app I installed through winget, explicitly? Or any of the apps winget manages in general?

If you have the logs for the winget upgrade --all it might help narrow down which upgrade had some kind of interaction. This is one of those super tricky to isolate scenarios. I'm suspecting that either an upgrade did something strange, or like you theorized, One Drive might have been doing something at the same time as well. I've seen reports of a few of the "Windows Cleaner" types of products messing with icons and associated paths, but this looks a bit different.

How do I find the logs for past executed winget commands?

@KerickHowlett
winget --info displays the log path for the Windows Package Manager.

Sorry for the very long time for me to write up a response; I've been terribly busy, which kept getting me sidetracked away from this.
Nevertheless, I ran the winget --info command to get the path of the logs like you suggested, @denelon, but by the time I looked at it, the earliest logs still stored are for 7/27/2022.
I don't know if running another update would be at all beneficial to recreate the necessary logs, but I can do it if you think it'd help.
I even promise to respond more promptly this time. 😅

Yeah, that makes sense. If I was more familiar with Powershell, I'd take a look at it myself. lol
I'd just be happy if there was a way to fix the issue whenever it does occur.
It looks like I may have to re-install Windows AGAIN! Although, I think I may install Windows 11 this time, assuming they got all the major kinks out of it by now. (I'd like to use WSLg for some apps. lol)


Feature: $ref on Observables (make _ref public)

Coming from Angular 1 and AngularFire I've been spoiled a bit with objects having direct access to the original ref and other helper functions. Before AF2 I wrote my own object but I like the Observable flow much better. Im having some issues though with relative paths and calling updates.
Consider a project and users that has the path like: /projects/<projectId>/teams/<teamId>/<userId>/favorite-colors
Ok, so first there are a lot of unknown keys that are needed to build the URL.
Let's say I want to add a favorite color to a user that's in a team. I have to get all the ID's and build the path relative to the root database.

constructor(private af: AngularFire) {
		this.teams = this.af.database.list('/projects/' + this.projectId);
	}

	addUserFieldValue(path: string,  value: string) {
		firebase().database().ref().child('projects/' + this.projectId + path).push(value);
	}

If we added the $ref you could simplify and call
addUserFieldValue(path:string, value: string) {
		this.teams.$ref.child(path).push(value);
	}

Otherwise you have to build the paths over and over when the ref is right there.
You could get it from the snapshot like this:
constructor(private af: AngularFire) {
		this.teams = this.af.database.list('/projects/' + this.projectId, { preserveSnapshot: true });
		this.teams.subscribe(snapshots => {
			this.ref = snapshots.ref;
			this.teams = [];
			snapshots.forEach(snapshot => {
				var tempVal = snapshot.val();
				tempVal.$key = snapshot.key;
				this.teams.push(tempVal);
			});
		 });
	}

	addUserFieldValue(path:string, value: string) {
		this.ref.child(path).push(value);
	}

But that seems WAY verbose to me..
The best part?
It's already there!
In the object there is _ref that is private... If we just change it to $ref and public it'll work..
I'm modding my version making _ref public, but figured it would be an easy change and then thought there might be a reason why it wasn't included so I figured I'd ask.

We have plans to support this in the future, $ref is definitely worth having :)


Increased memory usage while loading model graph  and observed a drop in performance throughput , When upgrade from TF version 2.8.0 to  either 2.8.1 or 2.9.1

Click to expand!
Issue Type
Performance
Source
source
Tensorflow Version
2.9.1 , 2.8.1
Custom Code
Yes
OS Platform and Distribution
Red Hat Enterprise Linux release 8.6 (Ootpa)
Mobile device
No response
Python version
3.8
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
we have a docker container with memory 1.5GB  where  we load our Custom RetinaNet Model and use it for inference.
We were using the Tensorflow version 2.8.0 and everything was working fine.

Due to the vulnerabilities ,  we  tried upgrading to Minor version 2.9.1. But we are getting SIGKILL - out of memory error during the load of the model graph. When checked the Docker stats, we could see that the memory usage is crossing the existing 1.5GB.  There is no other code changes other than the version upgrade.

After increasing the pod memory to 4GB we are able to load the model graph and get the inference. The same behaviour is seen when we try to upgrade version 2.8.1 as well.

We also see the  time taken for inference has increased about 40% when we upgrade the version from 2.8 to 2.9.1

Standalone code to reproduce the issue
1) Load a Retinanet Model  with Tensorflow version 2.8.0 in a docker container with memory 1.5GB
2) Load the same Retinanet model with Tensorflow version 2.8.1/2.9.1  in a docker container with the  memory 1.5GB
3) Load the same Retinanet model with Tensorflow version 2.8.1/2.9.1  in a docker container with the  memory 4GB

In this case you can see that the step1 will succeed and in step2 it will crash due to out of memory. Step3 will succeed.

Relevant log output
2022-10-12 00:33:08.206766: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:354] MLIR V1 optimization pass is not enabled
2022-10-12 00:33:08.894968: W tensorflow/core/framework/cpu_allocator_impl.cc:82] Allocation of 28753920 exceeds 10% of free system memory.
2022-10-12 00:33:09.311896: W tensorflow/core/framework/cpu_allocator_impl.cc:82] Allocation of 28753920 exceeds 10% of free system memory.
2022-10-12 00:33:12.049115: W tensorflow/core/framework/cpu_allocator_impl.cc:82] Allocation of 28753920 exceeds 10% of free system memory.
2022-10-12 00:33:12.279661: W tensorflow/core/framework/cpu_allocator_impl.cc:82] Allocation of 28753920 exceeds 10% of free system memory.
2022-10-12 00:33:13.079146: W tensorflow/core/framework/cpu_allocator_impl.cc:82] Allocation of 28753920 exceeds 10% of free system memory.
2022-10-12 00:33:16,694 INFO reaped unknown pid 8697 (exit status 0)
Process 'ForkPoolWorker-1' pid:133 exited with 'signal 9 (SIGKILL)'

Hi @rrajkumar1990 !
Can you attach the model loading code snippet too.
Thank you!

Hi @mohantym ,
We have a frozen model file "model.pb" .  we load it by :
with tf.io.gfile.GFile('model.pb', 'rb') as f:
graph_def = tf.compat.v1.GraphDef()
graph_def.ParseFromString(f.read())
        # we load the graph_def in the default graph

with tf.Graph().as_default() as graph:
tf.import_graph_def(graph_def,
input_map=None,
return_elements=None,
name="",
op_dict=None,
producer_op_list=None)
##after loading we  try to get tensor by name
graph = model_graph.graph_frozen
img = graph.get_tensor_by_name("input_img:0")
#exactly when the "get_tesnor_by_name" is called we get the memory error.
This issue happened only with TF version 2.9.1 or 2.8.1 with 1.5GB . With TF version 2.8.0 able to load the graph and get predictions

Ok @rrajkumar1990 !
Thanks for the update. Would test with a sample frozen graph model and update.
Feel free to test the loading time in 2.10/nighty version too.
Thank you!

@mohantym , Wanted to try using a stable version, but nightly will have everyday updates rite?

@rrajkumar1990 !
Yeah! Nightly version are updated on daily basis.
Did you check against 2.10 version lately?
Thank you!

@mohantym no I haven't tried yet in nightly version , as I want to run in some stable version .
where you able to replicate the issue  in 2.8.1 or 2.9.1?

Sorry @rrajkumar1990 !
Actually, I could not find a  sample .pb from in time to replicate this issue. It have been great if you provide the .pb model and code snippets in a colab gist.
My thought was as  if you are using 1.x version format( .pb)  of models, It will throw compatibility issues while replicating in 2.x version.  So, It is better to recreate and save the model using tf.saved_model.save api to get expected results.
@SuryanarayanaY !
Could you look at this issue.
Thank you!

Hi @rrajkumar1990 ,
From the error log  as we can see MLIR V1 optimization pass is not enabled  as the API  tf.config.experimental.enable_mlir_graph_optimization is under development.
Could you please spare some time to refer the attached TF Forum link regarding similar issue and please let us know if they works for your case.
Thankyou!

Hi @SuryanarayanaY ,
I don't think so the memory consumption changes are due to the "MLIR V1 optimization". I remember trying that and still  the pod memory consumption was more in 2.8.1 and 2.9.1. Could you try the steps I have mentioned and let me know

Hi @rrajkumar1990 ,
Thanks for quick response.Could you please confirm whether with 2.8v you are getting any logs like those you attached above ?
Is the behaviour tested on latest stable version 2.10 v ? Could you please test and share the logs if any ?
Also could you please share the model graph so that we can try to replicate the problem.
Thankyou!

Hi @SuryanarayanaY , can you try Retinanet weights from here https://github.com/tensorflow/models/blob/master/official/vision/MODEL_GARDEN.md and try saving it to a .pb file and load it as how I have mentioned above.

Hi @SuryanarayanaY / @mohantym - were you able to try and replicate?
Thanks

@rrajkumar1990 ,
Sorry for delay.Could you please share exact link from where to download the weights as your attached link in above comment have so many models with config and ckpt (checkpoint) files.The ckpt file has already saved_model.pb file.Are you using the same file? Please provide exact link for model also confirm which file you are using to convert into Graph?
Also I have doubt why you are using TF 1.x version when TF 2.x is available with pretrained models with much Easier and flexible API to handle Transfer learning?

@SuryanarayanaY ,
Yesterday I tired with the new 2.10.1 version. Seems like the memory consumption is even higher now.
with 2.8 => I was able to get inference with my container memory as 1.5Gb .
with 2.9.1 => 1.5GB failed wasn't enough as the memory consumption during the Inference  increased to 3.4 , so my container memory I had increased to 4GB
with 2.10.1 => The memory consumption during inference is 3.76GB
This  memory during loading of the pb file and using it to predict is jus getting higher . Please look Into this.

Also I have doubt why you are using TF 1.x version when TF 2.x is available with pretrained models with much Easier and flexible API to handle Transfer learning?

Also I don't see any direct weights available for Retinanet Model,  We have 2.x weights  for base Resnet only . Can you please point me if you  have the pretrained weights for Retinanet in 2.x

Hi @rrajkumar1990 ,
Please find the list of pretrained models here. Iam not sure which model you are interested in.Hence I have used one of resnet model to give demo on how can you save the weights of the pretrained model.Please refer to attached gist.
Please refer the attached doc also for more information on the model arguments and also some demo examples.
Thankyou!

Hi @SuryanarayanaY ,
I am looking for  Retinanet Model. But keeping that secondary, I would want to focus on  the  main issue, With the stable version 2.10.1 , the memory consumption is even higher. Can you please look into that .

@SuryanarayanaY , Yesterday I tired with the new 2.10.1 version. Seems like the memory consumption is even higher now.
with 2.8 => I was able to get inference with my container memory as 1.5Gb . with 2.9.1 => 1.5GB failed wasn't enough as the memory consumption during the Inference increased to 3.4 GB , so my container memory I had increase to 4GB with 2.10.1 => The memory consumption during inference is 3.76GB which is higher than 2.9.1 version.
This memory during loading of the pb file and using it to predict is jus getting higher . Please look Into this.

Hi @rrajkumar1990 ,
Could you please pinpoint which model weights you are using in the attached link in the above comment as there are many models available and iam a bit confused to choose right one from them.Could you please confirm so that i can try to replicate the issue.
Thankyou!

Hi @SuryanarayanaY ,
Please see the above performance numbers based on my tests. we see a performance decrease  and memory increase as we go up the version
Also please use the base  weights /pb files  from : http://download.tensorflow.org/models/resnet_v1_50_2016_08_28.tar.gz .

Hi @SuryanarayanaY , were you able to replicate the issue ?

Hi @rrajkumar1990 ,
Iam trying hard to replicate the issue but facing some problems.
I followed the instructions as per documentation here but iam facing the error as below which might be due to MacOS Host.
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
The TensorFlow library was compiled to use AVX instructions, but these aren't available on your machine.
qemu: uncaught target signal 6 (Aborted) - core dumped

I also tried to replicate on download RHEL image as mentioned below:
docker pull redhat/ubi8:8.6
docker images
docker run -it --memory=1500m bbfe57da95c7
yum install python3-pip

Then i tried to install tensorflow different versions and all giving error as below.
[root@64941ca0c428 /]# pip install tensorflow==2.10
WARNING: Running pip install with root privileges is generally not a good idea. Try `pip3 install --user` instead.
Collecting tensorflow==2.10
  Could not find a version that satisfies the requirement tensorflow==2.10 (from versions: )
No matching distribution found for tensorflow==2.10
[root@64941ca0c428 /]# pip install tensorflow==2.8.0
WARNING: Running pip install with root privileges is generally not a good idea. Try `pip3 install --user` instead.
Collecting tensorflow==2.8.0
  Could not find a version that satisfies the requirement tensorflow==2.8.0 (from versions: )
No matching distribution found for tensorflow==2.8.0

[root@64941ca0c428 /]# pip install --user tensorflow
WARNING: Running pip install with root privileges is generally not a good idea. Try `pip3 install --user` instead.
Collecting tensorflow
  Could not find a version that satisfies the requirement tensorflow (from versions: )
No matching distribution found for tensorflow

I checked that the default python version being used is 3.6.x.This might be the issue.
Am i missing something here? Iam not an expert at docker/linux commands and hence your help by providing the exact commands you are using along with the exact image ID will be very much appreciated to replicate the issue.
Thankyou!

Hi @rrajkumar1990,
Initially i thought of using TF images only.But when i see the Size of Image i dropped the idea as tf-nightly image itself having  1.49GB . 2.10.1 has 1.46GB and 2.8.1 has 1.36 GB size.As you told you are running your model in 1.5GB memory right? Are you sure we can run on 1.5 GB container? Anyway i will first test the performance on 2.5 GB container then let you know.
Thankyou!

Hi @rrajkumar1990 ,
I downloaded the .ckpt file from the attached link in your comment and saved it as model.pb file.
Then i checked the code you mentioned here and i felt something missing in the code.Hence i tried it on colab gist and i got error at this line:
graph = model_graph.graph_frozen
Error:
NameError: name 'model_graph' is not defined
From the code you provided i could find neither model_graph nor graph_frozen.Could you please cross check and provide the complete code.
Thankyou!

Hi @SuryanarayanaY  ,
After you loaded the pb file , with File :
with tf.io.gfile.GFile(frozen_graph_file, 'rb') as f:
                graph_def = tf.compat.v1.GraphDef()
                graph_def.ParseFromString(f.read())

we load the graph_def in the default graph
with tf.Graph().as_default() as graph:
                tf.import_graph_def(graph_def,
                                    input_map=None,
                                    return_elements=None,
                                    name="",
                                    op_dict=None,
                                    producer_op_list=None)

You should have ur graph stored in “graph”.
Session =  tf.compat.v1.Session(graph= graph)
With this session you can use the
img = Session.get_tensor_by_name("input_img:0")
detect_retinanet = Session.get_tensor_by_name("DetResults:0")

Then load an image with open cv, and pass to the session
 test_img = cv2.imread(input_image)[:, :, ::-1]

dets_val = Session.run(detect_retinanet, feed_dict={img: test_img})

You could see the performance throughput time  and the memory consumption has increased as we go to higher versions from 2.8.0

Hi @rrajkumar1990 ,
Iam getting problems running Docker and later I came to know that we can't use Docker on our System as per our internal policy. I believe there are other issues in this repo related to the same problem you mentioned like #58676 where Developer team already involved and definitely will do needful.
Meanwhile as you are using only 1.x Graph for prediction can you please try adding the code tf.compat.v1.disable_eager_execution() after TF import statement  which disables Eager Mode that is default from 2.X onwards followed by print(tf.executing_eagerly()) which should return False which confirms Eager mode disabled.Please test with this statements and let us know the Memory & Performance details.
Thankyou!

Hi @SuryanarayanaY ,
I'm aware that I'm using a graph for prediction and i have already added the "tf.compat.v1.disable_eager_execution() ", because eager mode is for 2.x models. The memory and performance numbers which I shared is after disabling eager execution.
Please continue your analysis and let me know

Hi @rrajkumar1990 ,
Thanks for the confirmation.Inspired from this comment can you please set the environment below and try with 2.10V and 2.11V. It should work with 2.10V but not sure of 2.11V though.You can refer the mentioned issue which also largely resembles your issue.
import os
assert os.getenv('TF_RUN_EAGER_OP_AS_FUNCTION') == 'False'

Thankyou!

Hi @rrajkumar1990 ,

Moreover the other defect which you have mentioned in the defect , they are using 2.x model "from tensorflow.keras.applications.resnet50 ". The model architecture is completely different for graph based models and models with eager execution.

The other issue mentioned also related to performance/Memory consumption w.r.t 2.X models only. I just want to clear that we are not supporting 1.X models and still tried to check in the issue. Now similar behaviour observed with 2.X models and issue already taken up by the Development team and Iam certain that they will come up with some solution definitely ASAP. I hope that might works for your case also as you said it works fine with 2.4V.
Thankyou!

Hi @rrajkumar1990 ,

Moreover the other defect which you have mentioned in the defect , they are using 2.x model "from tensorflow.keras.applications.resnet50 ". The model architecture is completely different for graph based models and models with eager execution.

The other issue mentioned also related to performance/Memory consumption w.r.t 2.X models only. I just want to clear that we are not supporting 1.X models and still tried to check in the issue. Now similar behaviour observed with 2.X models and issue already taken up by the Development team and Iam certain that they will come up with some solution definitely ASAP. I hope that might works for your case also as you said it works fine with 2.4V.
Thankyou!

Hi @SuryanarayanaY ,
My model is based on graph execution. I'm not asking support for models with 1.x. I'm already using the model with compat.v1 .  Also , I have never mentioned that there is no memory issue with 2.4V.  I was using 2.8 Version and tried upgrading to higher versions and  observing this issue.
Can you please assign this defect to the Dev team.
Thanks,

Hi @rrajkumar1990 ,
We got an update regarding High memory consumption in recent TF versions. Can you please try passing the input data in the form of tf.data.Dataset like below or any suitable method based on your input type as per the Examples in the attached API.Below is just an example but you should choose the suitable method from the tf.data.Dataset API based on your input data
x_train = tf.data.Dataset.from_tensor_slices(x).batch(32) # batch size of your choice
for _ in range(10000):
    y_True = model.predict(x_train)

Since you are using Session and tf.compat.V1 module i can't assure you performance guarantee even though if i escalate it to next level based on this documentation source. Hence please try using your input data in form of tf.data.Dataset and let us know if it helps.Based on this I can escalate the issue to next level
Thankyou!

Hi , Let me try and update here


Bumped NET Core target frameworks of Test projects to 3.0

Bumped the NET Core target frameworks of the test projects to 3.0
As according to https://dotnet.microsoft.com/download/dotnet-core they've reached end of life.

@LordMike not sure if we want to bump the version on the main library as well?

We should target netstandard3.0 in addition to the rest of the TFM's.

Do you mean netstandard2.1?
https://docs.microsoft.com/en-us/dotnet/standard/net-standard

Yes. Apparently I do.. explains why my project failed so hard last night :D


(fix) Add language information to metadata

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

Coverage increased (+0.3%) to 93.333% when pulling 5bf00850496ba3f27a4c628fc8dbec430068c1dd on jeromesimeon:js-fix-language-info into 1226c2cec5d6e473a4fa0ced0fd00d68fd469d07 on accordproject:master.


Refactor org.ga4gh.cts.api.Utils

Split it into logical groups (classes) that relate to the objects they handle.
This can only happen after the merge of the _tests branches, because it will affect the packages they contain.

moved to Maciek. Do we still need to do this?


Banner doesn't load when TOP is unchecked

Godot version
3.4
Plugin version
v2.0.0
Phone information
Android
Issue description
When I uncheck banner on TOP the banner won't load anymore.
I tried to solve the problem myself but to no avail.
Steps to reproduce
Go to Admob settings -> Banner -> uncheck on TOP
Additional context
No response

Solved on: https://github.com/Poing-Studios/godot-admob-ios/releases/tag/v2.1.0
But please try to use the latest version: https://github.com/Poing-Studios/godot-admob-ios/releases/tag/v2.1.2

hi @ZooHairDev any updates?

Closing due stale


Add Atlassian to users

Add Atlassian to Jest users. Our JIRA and Confluence are using the framework.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #6094   +/-   ##
======================================
  Coverage    64.1%   64.1%
======================================
  Files         217     217
  Lines        8335    8335
  Branches        4       3    -1
======================================
  Hits         5343    5343
  Misses       2991    2991
  Partials        1       1

Continue to review full report at Codecov.

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


Update docs for IUF and remove deprecated procedures

Summary and Scope
This PR updates some of the UAN docs to reflect the use of IUF in install/upgrade and how to use it for an image update.  Some UAN specific data was added to the reference to the IUF procedure on updating images.
The procedure on customizing images manually was removed in favor of using IUF.

Agreed with @darrenn-jackson. In the meantime I'll approve pending those changes.


colorString() not working on Windows cmd.exe

Hi @fatih
Thanks a lot for your color package.
It works great on Linux, but there's a problem under Windows.
package main

import (
       "github.com/fatih/color"
       "fmt"
)

func main() {

	color.Red("1. We have red")

	fmt.Println(color.RedString("2. We have red"))
}

The first line is printed in red, but not the second one:

I'm using the default shell (c:\Windows\system32\cmd.exe) on Windows 7.

cc @mattn

cmd.exe does not handle escape sequence in default. Please use Output.
package main

import (
       "github.com/fatih/color"
       "fmt"
)

func main() {

	color.Red("1. We have red")

	fmt.Fprintln(color.Output, color.RedString("2. We have red"))
}

Great, it works fine.
Thanks @mattn !


DX: Add type/build checks for common hard-to-notice pitfalls

During debugging of CODY-1928 I noticed we were re-registering a global command.
The fix would have been simple but encouraged by @olafurpg's urge to seize every opportunity to guard against regressions and capture reproductions in tests; I spent some time ensuring that these kind of mistakes can't creep in again.

Avoid (transitive) imports of vscode in the agent/vscode-shim.ts. This was the third time I accidentally tripped it and each time I wasted too much time remembering there is a massive disclaimer comment. Now we simply check this condition during the build...as a service to future self.

The package.json manifest now has a schema that we can expand upon. It's only basic right now, because VSCode doesn't provide a schema for us. But it at least ensures that all commands are registered with a cody prefix.

I introduced some typehacks that are extensions on top of package types which further restrict the usage to ensure not just type-safety but soundness. These types are not included by default as they would pollute the global scope. They are however run on build.

NOTE: Right now it's not really possible to show these in the editor as there seems to be no way of overriding the tsconfig for a particular subfolder. You can simply add the typehacks to the tsconfig.json and you'd get something like this (and break your build & cache in the process...tsc build --clean #ftw)

Fixes CODY-2023
Test plan

manually verified that builds still work
manually verified that introducing the above issues triggers build errors

Great idea, made a issue for it CODY-2084


GH-1330 Internal version for snapshot builds

The contents of src/template/kotlin are copied to src/generated/kotlin and @version@ gets replaced with the version.
I think src/generated/kotlin is better than build/generated/kotlin because otherwise the ide would show an error before the first compilation and after each ./gradlew clean.

Thanks!


Remove extra bang

Checklist

[x] CLA is signed

Description of change

A bang (!) slipped through in the recent UI cleanup

Merged.  Thanks


Can not resolve method 'getModel' in ChildModelIterator.java

When I download source and build it with maven, some errors displayed in IDE, the first one is:
Can not resolve method 'getModel' in row 64 in ChildModelIterator.java file.
There's no method 'getModel' in IModel interface in thymeleaf 3.0.
How do you build this source, please help!
Thanks!

My source depends on ExtensionMethod of lombok. But there's no plugin support lombok's extension method for intellij idea yet. I am working with NetBeans IDE. The build can continue if you discard the error.
Lombok Extension Method Support:
https://github.com/mplushnikov/lombok-intellij-plugin/issues/21

I revert the commits. You can pull the latest commit without any error.

@zhanhb Thanks for you help. I find lombok plugin for intelliJ and trying it now.


Configure kafka COS(S3)connector to push audit logs from eventstream to COS

Following this guide,
https://cloud.ibm.com/docs/services/EventStreams?topic=eventstreams-cos_connector
to configure kafka connector to push audit messages from EventStream to COS.

This is no longer relevant for the OpenSource project.

I'd still like to provide some guidance on how to do it at some point, probably by pointing at an audit component from https://github.com/Alvearie

I think we should open a documentation task for it.  Thoughts?

I was thinking to maybe just reopen this one and update it to be clear its documentation-only, but I really don't feel that strongly.  If we do re-open, I think we'd move it to the ice-box since this mythical Alvearie audit event consumer doesn't yet exist...

sounds good


Frontend build failing with no debug output

Describe the bug
The build step of our amplify pipeline is being marked as failed for a previously working project. All we have in the logs is what I attached below. Yarn build is able to be run on local machines with a matching version of yarn with no problems.
To Reproduce
Steps to reproduce the behavior:
Create any build in our amplify CI
Expected behavior
Builds to succeed
Screenshots
2021-01-22T21:35:50.639Z [INFO]: # Cloning repository: git@github.com:askpillar/life-ui.git
2021-01-22T21:35:50.691Z [INFO]: Agent pid 123
2021-01-22T21:35:50.695Z [INFO]: Identity added: /root/.ssh/git_rsa (/root/.ssh/git_rsa)
2021-01-22T21:35:50.849Z [INFO]: Cloning into 'life-ui'...
2021-01-22T21:35:51.067Z [INFO]: Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
2021-01-22T21:35:52.126Z [INFO]: # Switching to commit: fd8231da79a19ea27d81a4b9e93cbadb40f76b67
2021-01-22T21:35:52.135Z [INFO]: Agent pid 136
2021-01-22T21:35:52.139Z [INFO]: Identity added: /root/.ssh/git_rsa (/root/.ssh/git_rsa)
2021-01-22T21:35:52.159Z [INFO]: Note: switching to 'fd8231da79a19ea27d81a4b9e93cbadb40f76b67'.
                                 You are in 'detached HEAD' state. You can look around, make experimental
                                 changes and commit them, and you can discard any commits you make in this
                                 state without impacting any branches by switching back to a branch.
                                 If you want to create a new branch to retain commits you create, you may
                                 do so (now or later) by using -c with the switch command. Example:
                                 git switch -c <new-branch-name>
                                 Or undo this operation with:
                                 git switch -
                                 Turn off this advice by setting config variable advice.detachedHead to false
                                 HEAD is now at fd8231d Skip invite altogether if there's already a user (#498)
2021-01-22T21:35:52.165Z [INFO]: Successfully cleaned up Git credentials
2021-01-22T21:35:52.165Z [INFO]: # Checking for Git submodules at: /codebuild/output/src281026161/src/life-ui/.gitmodules
2021-01-22T21:35:52.190Z [INFO]: # Retrieving cache...
2021-01-22T21:36:32.753Z [INFO]: # Extracting cache...
2021-01-22T21:37:57.201Z [INFO]: # Extraction completed
2021-01-22T21:37:57.421Z [INFO]: # Retrieving environment cache...
2021-01-22T21:37:57.447Z [INFO]: # Retrieved environment cache
2021-01-22T21:38:08.714Z [INFO]: ## Starting Frontend Build
                                 # Starting phase: preBuild
                                 # Executing command: yarn install
2021-01-22T21:38:09.237Z [INFO]: yarn install v1.22.0
2021-01-22T21:38:09.347Z [INFO]: [1/4] Resolving packages...
2021-01-22T21:38:10.001Z [INFO]: success Already up-to-date.
2021-01-22T21:38:10.006Z [INFO]: Done in 0.77s.
2021-01-22T21:38:10.018Z [INFO]: # Completed phase: preBuild
                                 # Starting phase: build
2021-01-22T21:38:10.018Z [INFO]: # Executing command: yarn run build
2021-01-22T21:38:10.142Z [INFO]: yarn run v1.22.0
2021-01-22T21:38:10.168Z [INFO]: $ react-app-rewired build
2021-01-22T21:38:11.405Z [INFO]: Creating an optimized production build...
2021-01-22T21:39:46.472Z [INFO]: Compiled successfully.

Our amplify app ARN is arn:aws:amplify:us-east-2:489671872122:apps/d1d04rucfjnmep any help getting out builds working again would be greatly appreciated as our company is currently stuck. Thank you!

Same

Same

Bumping this again as there is really no way for our team to attack this issue with zero debugging output.
Thank you.

Bumping this again as there is really no way for our team to attack this issue with zero debugging output.
Thank you.

@bcliffor  We are looking into this now and will get back to you ASAP.

@bcliffor  We are looking into this now and will get back to you ASAP.

Closing this ticket as it was resolved directly with customer.

Closing this ticket as it was resolved directly with customer.

I'm experiencing a similar issue and wondering if there is some guidance on a fix related to this.

I'm experiencing a similar issue and wondering if there is some guidance on a fix related to this.

it seems to be an intermittent thing that occurs sometimes with their services the best way I found to get around it is to upload your zip file to a server somewhere online and then use the URL method from the AWS council website


Fix adc.management initiation bug.

@haoshen61 After manual test on OS, at L584, the original code does not behave as expected. Need your review.


Error while using setContents

Hi @JiHong88
I am facing this issue for quite some time now, however, I am not able to figure out the root cause.
When I try to set my editor's content using setContents(), I get below error:
Uncaught TypeError: Cannot read property 'scrollTop' of null
    at Object.getOffset (util.js:1021)
    at Object.setControllerPosition (core.js:755)
    at Object.call_controller (math.js:244)
    at Object.active (math.js:215)
    at Object._applyTagEffects (core.js:5305)
    at Object._onChange_historyStack (core.js:5011)
    at h (history.js:86)
    at Object.push (history.js:109)
    at Object.setContents (core.js:4411)
    at Object.setContents (core.js:7448)

The same code usually works fine, but at times gives the above error.
Once this code breaks, my entire business logic in its parent function stops.
Can you help rectify this edge case?

@dgandhi1993 When is "setContents" called?
And show me the contents of "setContents".

setContents is called with an empty string long after the editor instance initialization.
Basically, I am trying to clear the editor's content.

@dgandhi1993 If you are trying to set the initial values of the editor, you should do it in one of three ways:

call setContents inside onload
instance.onload = (core) => core.setContents("");

value option
suneditor.create("id", {value: ""})

just empty textarea

<textarea><textarea>

@JiHong88 I am using the value property. This issue appears when I am trying to change an existing editor instance value.
Also, this doesn't happen always.

The 2.34.2 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.


error in Project References when add new file

Current behavior

when "composite": true is in tsconfig.json ，add a new file and quote it in somewhere, it issue an error

Expected behavior

don't issue an error
Steps to reproduce the issue

package.json
{
  "name": "webpack-demo",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "start": "webpack serve"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "fork-ts-checker-webpack-plugin": "^7.3.0",
    "ts-loader": "^9.4.2",
    "typescript": "^4.9.5",
    "webpack": "^5.75.0",
    "webpack-cli": "^5.0.1",
    "webpack-dev-server": "^4.11.1"
  }
}

tsconfig.json
{
  "compilerOptions": {
    "composite": true,
    "target": "es2016",
    "module": "commonjs",
    "esModuleInterop": true,
    "forceConsistentCasingInFileNames": true,
    "strict": true,
  }
}

webpack.config.js
const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');

module.exports = {
    mode: 'development',
    context: __dirname,
    entry: './src/index.ts',
    resolve: {
        extensions: [".ts", ".tsx", ".js"],
    },
    module: {
        rules: [
            {
                test: /\.tsx?$/,
                loader: 'ts-loader',
            }
        ]
    },
    plugins: [new ForkTsCheckerWebpackPlugin()],
    watchOptions: {
        ignored: /node_modules/,
    },
};

start the app!, and add a new file, quote it in somewhere.
https://user-images.githubusercontent.com/8347286/219620910-70d06703-fc6f-4b58-b7e7-607aa81281a7.mp4
Issue reproduction repository

Environment

fork-ts-checker-webpack-plugin: [version from the package.json]
typescript: [version from the package.json]
eslint: [version from the package.json]
webpack: [version from the package.json]

Thanks for the report - it looks like this issue appears only with newer versions on TypeScript. I can't promise I will take a look at this soon but it's on my radar.


I'll open an issue and propose an addition to the Style Guide, coming soon!

I'll open an issue and propose an addition to the Style Guide, coming soon!
Originally posted by @jai in https://github.com/github/docs/issues/5945#issuecomment-831635058

whoops - possible duplicate of https://github.com/github/docs/issues/6009 (I opened #6009) based on the same thread
Also PR: https://github.com/github/docs/pull/6010


fix: catch validate fields so app doesn't crash TKC-863

Changes

Handle validateFields and don't crash the app on error

Fixes

https://linear.app/kubeshop/issue/TKC-863/[ui]-creating-triggerwebhook-with-empty-required-valued-would-crash

How to test it

screenshots

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

I'm not sure if the error should be ignored, but looks rather OK 👍


Adds an API for database calls

I've created a class for MSDatabase that can be used to eventually refactor a large part of our webhook code.
I've also attached unit tests!
Notably, I'm writing raw SQL queries to set up each test, which would be less clean than using MSDatabase methods, but I figured the only MSDatabase calls within each test should be those that are being directly tested - please give me your thoughts.
I also do some somewhat janky stuff when testing add_ and drop_ methods on the MSDatabase object, notably doing an assertFalse/assertTrue on the return value from the SQL selects I'm doing. This leverages the truthy/falsyness of the lists that are returned but maybe it's more obtuse than it should be? Tell me what you think. Refer to test_add_user for an example of what I mean.
This PR will help us close off on #6, though I do think that there'll have to be a webhook.py refactor first.

Nevermind, going in a different direction!


evaluation quality problem

Does the author use PSNR or SSIM metrics to evaluate the quality of images generated by LBBDM? Because the PSNR of the images generated after training using public datasets is very unsatisfactory, I would like to refer to the correct values to further modify the specific problem. I would greatly appreciate your answer！

Does the author use PSNR or SSIM metrics to evaluate the quality of images generated by LBBDM? Because the PSNR of the images generated after training using public datasets is very unsatisfactory, I would like to refer to the correct values to further modify the specific problem. I would greatly appreciate your answer！

since we used BBDM for generation, not reconstruction, we didn't adopt PSNR.


fixing {{handlebars}} that aren't showing up

Description of the change: fixes the broken handlebars in issue #3371
Reason for the change: because they're broken
Link to original source: https://sendgrid.com/docs/Classroom/Build/Add_Content/substitution_and_section_tags.html

Closes #3371

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


HDR-CX675 connection issue

I get the following error when I try and install OpenMemories-Tweak on my camcorder. Any thoughts?
Model:              HDR-CX675
Product code:<PHONE_NUMBER>
Serial number:
Firmware version:   1.00
Using drivers Windows-MSC, Windows-MTP
Looking for Sony devices
Querying MTP device
Sony Corporation HDR-CX675 is a camera in MTP mode
Switching to app install mode
Traceback (most recent call last):
File "C:\projects\sony-pmca-re\pmca-gui.py", line 74, in do
File "C:\projects\sony-pmca-re\pmca\commands\usb.py", line 239, in installCommand
File "C:\projects\sony-pmca-re\pmca\commands\usb.py", line 35, in switchToAppInstaller
File "C:\projects\sony-pmca-re\pmca\usb\sony.py", line 306, in switchToAppInstaller
File "C:\projects\sony-pmca-re\pmca\usb\sony.py", line 244, in sendCommand
File "C:\projects\sony-pmca-re\pmca\usb\sony.py", line 98, in sendSonyExtCommand
File "C:\projects\sony-pmca-re\pmca\usb_init.py", line 81, in _checkResponse
pmca.usb.InvalidCommandException: MTP error 0x2006

@efredman Feel free to re-open the issue.


0.4.21

Describe the bug
After updating till 21, I get the following error.
Eloquent Query
Stack Trace
{
"message": "Trait 'GeneaLabs\LaravelModelCaching\Traits\Buildable' not found",
"exception": "Symfony\Component\Debug\Exception\FatalErrorException",
"file": "/home/vagrant/code/vendor/genealabs/laravel-model-caching/src/CachedBuilder.php",
"line": 10,
"trace": [
{
Environment

PHP:7.2
OS: Debian stretch
Laravel: 5.8
Model Caching: 0.4.11

Checking ...

Ugh! Thanks for reporting this ... found the issue, I think. It didn't manifest in testing because of the testing environment. Fixing that now.

@tmishutin Go ahead and give 0.4.22 a try. Let me know how that works :) Thanks again!


Update onRun to reflect the latest API change

As per hooks section of the README.

:+1: the docs currently lie.

I was tricked by the guide too, due to omitting to mention this.next.

@sungwoncho @BudgieInWA @aknuds1 merged


What's a convenient way to accept an nlayers-long vector of inputs?

The current code has three such vectors: ah, hmean, and g_vec.  Currently, it reads the values from parameters.in, using the namelist mechanism.  However, apparently, allocatable arrays cannot be read by namelist input in Fortran 90 or 95, because the compiler generates the code for parsing name lists at compile time, and for some reason needs to know how many elements to expect.  This restriction is allegedly lifted in Fortran 2003, but that doesn't seem like a great reason to change standards.
So, how should MIM accept this input?  Replicating the filename-read mechanism used for the input fields and the wind time series seems rather heavy-weight (for the users).  Would it be reasonable to have a compile-time constant maximum number of layers for this purpose?  Or, more specifically, maximum number of layers that can have distinct values for those three parameters?  Are there any other options?  (I don't know Fortran I/O very well.)

I suppose one option would be to accept a list of length up to N in the parameter file, and provide an optional filename style parameter for those users who want more layers.  Then the user neither needs to mess with more input files for cases with small numbers of layers, nor needs to recompile MIM to simulate a large number of layers.
What happens when not enough values are supplied in the input for a namelist array?

That's awkward.
I agree that using the framework of putting the values in a file and reading them in is overkill. It's also a lot of hassle to go through just to load a few numbers.
Given the extremely small overhead of storing a few unused elements in these arrays I think it's reasonable to have a maximum number of layers set at compile time. Something like 10 or 20 is almost certainly enough.
I just tried running the test suite without specifying enough values for g_vec and, while the tests failed the simulation did run. So that suggests the Fortran was happy enough without getting as many values as it had spaces. This isn't something I've looked at before.

Actually, how about having the namelist read them into temporary variables that are much larger than required, then moving those values into allocated arrays? Once the model reads the namelist parameters it knows how many layers there are, can allocate vectors that are the right size to hold these values.
See the comments here: http://stackoverflow.com/questions/40935059/reading-allocatable-arrays-from-namelists

Section <IP_ADDRESS> of the standard says that having the array being read be too big is acceptable; remaining slots in it are filled with nulls.  Too small is an error.
From this I conclude that the thing to do is to define input arrays for these three things with a hardcoded maximum size, and then copy the values out of there into correctly-sized arrays for computation, if needed.  We can leave size-unlimited file-based input for these things off as a feature to be added separately.

Sounds good.


Error [ERR_MODULE_NOT_FOUND]: Cannot find module

Same issue as here: https://github.com/vercel/vercel/issues/10494
It's working locally and on fly but not on Vercel. Running<EMAIL_ADDRESS>Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/var/task/build/build-nodejs-eyJydW50aW1lIjoibm9kZWpzIn0.js' imported from /var/task/build/server-build-nodejs-eyJydW50aW1lIjoibm9kZWpzIn0.mjs
    at new NodeError (node:internal/errors:405:5)
    at finalizeResolution (node:internal/modules/esm/resolve:329:11)
    at moduleResolve (node:internal/modules/esm/resolve:992:10)
    at moduleResolveWithNodePath (node:internal/modules/esm/resolve:936:12)
    at defaultResolve (node:internal/modules/esm/resolve:1178:79)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}
Error: Runtime exited with error: exit status 1
Runtime.ExitError

Any ideas? I've migrated over to ESM, and I'd assume it's an issue with my project but it looks like it's happening in a fresh project too based on that issue.

Can you share a repository demonstrating this happening?

Invited you to the repo, it's on the remix-esm-v2 branch. Hope that's okay, I've been unable to reproduce on a fresh installation.

Unsure if it's connected but the function file sizes are dramatically smaller on the ESM build that is not working:
CJS:

ESM:

Hey @benjaminleonard, sorry, I lost track of this one. Were you still having issues?
For future reference, it would be better to create a support ticket so that it gets properly tracked.

Switching remix.config.ts to js fixes my issue. #64
Will close this.

For future reference, it would be better to create a support ticket so that it gets properly tracked.

Noted, thanks!

Thanks for the confirmation! I don't believe .ts is a supported file extension for the remix.config file (https://github.com/remix-run/remix/discussions/2934, https://github.com/remix-run/remix/pull/438).


Publish latest changes?

https://github.com/gigara/rush-github-action-build-cache-plugin/pull/2

The changes you merged in #2 has not been released on npm. No "rush", I use my own version.

@sveisvei I published the latest changes with 7e5fbb9. Thanks for your contribution.


Low level client should sent null for dictionary values.

Isolated from: #4336

Doesn't this change also affect Dictionary<TKey, TValue> and SortedDictionary<TKey, TValue> serialized by the high level client too?

Discussed, we feel this is the better default for 8.x and will call it out on the release notes.


function return type checking fails to check promise inside of a promise (angular)

eg:
function test() : ng.IPromise<number> {
   var p : ng.IPromise<number> = querySomeApi();

    return p.then(function(val : number)){
     // suppose $q was injected before
     return $q.when(val);
    })
}

This would pass the type checking but it shouldn't. promise.then() will return a promise that wraps around the returned value(val). So this function is actually returning ng.IPromise<ng.IPromise<number>>.
using ts 1.7.5

Hi @yklee1013
If we're talking about the same IPromise, then this is behaving correctly. then takes a callback that returns either a promise of some type (IPromise<TResult>) or some type (TResult). then will return a promise of the type in focus (`IPromise). This flattening is the way A+ promises are supposed to work.

Thanks for reply @DanielRosenwasser !
I wasn't specific enough when I posted the issue, but this happens when I was doing some testing for an angular app. I had to manually trigger digest cycles. The digest cycle wasn't triggered for $q.when(val) at the time the control reaches that line, so it was still an IPromise not being resolved.
The official angular doc for then():
This method returns a new promise which is resolved or rejected via the return value of the successCallback, errorCallback (unless that value is a promise, in which case it is resolved with the value which is resolved in that promise using promise chaining).
https://docs.angularjs.org/api/ng/service/$q
Since the innermost promise hasn't been resolved, then() returns a promise wrapping another promise that was in the process of getting resolved. Inspecting on console I saw the function returning an IPromise whose $$state.status = -1, which means that it has another IPromise inside. This is really an edge case that I have only encountered while doing testing. Thanks!

Hi @yklee1013
I think you may misunderstand the meaning of Promise<T> here. Promise means the value will be resolved asynchronously, that is, your test function should always return an IPromise whose status is pending, when you inspect it synchronously.  It will be settled after a while, loosely speaking. Strictly speaking, the promise will be resolved asynchronously, after your querySomeApi is resolved.
var promise = test()
console.log(promise.$$status.status) // should be pending here
setTimeout(() => console.log(promise.$$status.status), 1000) // may be resolved here, if querySomeApi is resolved before.

Promise<T> has already abstracted the notion of deferred value: your promise will not be resolved immediately after you created it, it will be resolved after the underlying work has been done, asynchronously. There is no difference between a resolved promise and a pending one at type level, in the sense that they have the same type: Promise<T>.
Accordingly, there is little point to make a type of Promise<Promise<T>>, since type information does not indicate whether a promise is resolved. Promise<Promise<T>> means a deferred value of type T, same as Promise<T>. Whether a promise is resolved depends on runtime execution.
If you still have some confusion, stackoverflow is your best friend! Also Q's wiki page may also help


Shift collaboration to file system level

Right now, Collaboration works only between two open codemirror instances. This is fine as long as we have one file, but if we have more then one file, the changes made by a user in one file will show up in the file opened by the connected users in different files.
Right now, only the text inserted/deleted in each go is transmitted to the connected peers.
We should probably add a new parameter that should be passed, the file path. Files can then be identified on the basis of this path and peers can make the change accordingly. (at the file system level or in the codemirror instance

Opened in Thimble instead.


Adapt testing section in README.md

Also enable syntax highlighting and fix indentation in one example.

Merged, cheers!


TypeScript

By making a PR you are agreeing to that contributing a hello world project allows the repo/project owner to use the hello world app in examples for their own use.

[ ] only the most simple basic hello world app
[ ] remove all tests and commands for tests and dependencies for tests
[ ] must say "Hello World" on the main component
[ ] only include the minimal components (1 ideally)
[ ] no routing, not extras whatsoever
[ ] no styling
[ ] nothing else !
[ ] must work by running npm install and npm start (or equivalent)
[ ] must be in the folder named <tech-name>
[ ] add build steps and which folder to serve in the README.md
[ ] verify that the app once built can be served from whatever folder the built assets are placed in
[ ] verify that a Pull Request doesnt already exist. If it does, feel free to contribue to it.

@johnpapa
PR #89 added typescript. Can this issue be closed?

yep. thanks


Sergio/graphic fix #59

I started with a BarChart, but it had a lot of problems when handling with Date types, so I finally did it with a LinePlot.
This is the data I've used for creating this example chart:
%{gross_amount: 10_500, issue_date: ~D[2021-11-10]}
%{gross_amount: 12_600, issue_date: ~D[2021-11-15]}
%{gross_amount: 12_000, issue_date: ~D[2021-11-15]}
%{gross_amount: 3_700, issue_date: ~D[2021-11-21]}
%{gross_amount: 7_700, issue_date: ~D[2021-11-22]}
%{gross_amount: 9_900, issue_date: ~D[2021-11-30]}
%{gross_amount: 23_200, issue_date: ~D[2021-12-01]}
%{gross_amount: 22_100, issue_date: ~D[2021-12-02]}
%{gross_amount: 1_500, issue_date: ~D[2021-12-03]}
%{gross_amount: 2_500, issue_date: ~D[2021-12-03]}
%{gross_amount: 10_000, issue_date: ~D[2021-12-03]}
%{gross_amount: 12_100, issue_date: ~D[2021-12-04]}
%{gross_amount: 3_200, issue_date: ~D[2021-12-04]}
%{gross_amount: 4_500, issue_date: ~D[2021-12-20]}

Now it has the final file structure we talked about in a previous discussion of this PR. I forced a push because I did a git rebase with main and I couldn't push to the current branch properly.


docs around container registry use

Right now, there are references to using acr for pointing to private osm images that we build and maintain. We'll want to update the docs to point to the public osm images.

PR #1358 changed the default container registry to point to public osm images.
Related feature work for issue #1416. Docs should be updated here

Looks like this has been resolved. thanks!


[SPARK-33091][SQL] Avoid using map instead of foreach to avoid potential side effect at callers of OrcUtils.readCatalystSchema

What changes were proposed in this pull request?
This is a kind of a followup of SPARK-32646. New JIRA was filed to control the fixed versions properly.
When you use map, it might be lazily evaluated and not executed. To avoid this,  we should better use foreach. See also SPARK-16694. Current codes look not causing any bug for now but it should be best to fix to avoid potential issues.
Why are the changes needed?
To avoid potential issues from map being lazy and not executed.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Ran related tests. CI in this PR should verify.

Test build #129541 has started for PR 29974 at commit 047da30.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34147/

Thanks for catching this!

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34147/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34147/
Test PASSed.

Test build #129541 has finished for PR 29974 at commit 047da30.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/129541/
Test FAILed.

Thanks! Merged to mater/3.0.

Thank you @viirya and @maropu!


Use local CNI image in pilot integration test

Part of https://github.com/istio/istio/issues/24610

@jwendell: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
de62b36a7a1208f1f5f72465c63e873bac4bddf3
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest