Erroneous parsing of XML content

Hello,
I've noticed that when processing certain XML content, the parser malfunctions.
That's the snippet of the XML document that I am processing:
والجدير بالذكر أن الدورات النقابية المتعاقبة منذ عام 1950م وحتى 2011م بلغت خمسة عشر دورة تتفاوت في آجالها من عام لعامين ولأربعة أعوام ثم خمسة أعوام اعتباراً من دورة 1996_2001 م حسب نصوص القوانين وتطوراته </doc> <doc id="1432834" url="https://ar.wikipedia.org/wiki?curid=1432834" title="ريتا حايك">
However, after parsing the file (either with doc = Jsoup.parse(File f) or doc = Jsoup.parseBodyFragment(String s)), when printing the element's content (doc.html()), I notice that one doc closing tag turns to a comment closing:
والجدير بالذكر أن الدورات النقابية المتعاقبة منذ عام 1950م وحتى 2011م بلغت خمسة عشر دورة تتفاوت في آجالها من عام لعامين ولأربعة أعوام ثم خمسة أعوام اعتباراً من دورة 1996_2001 م حسب نصوص القوانين وتطوراته </doc--> <doc id="1432834" url="https://ar.wikipedia.org/wiki?curid=1432834" title="ريتا حايك">
Because of that all the remaining content (over 50MB) is loaded as one doc.
Does anything come to your mind? Am I doing something wrong or should this be considered a bug?

Hi!
Could you put the original file somewhere where I can pick it up and test? I tried to copy&paste but couldn't replicate on try.jsoup.org. I wonder if there's a BOM at the start of the file, or another tag that isn't in your snippet, that is causing the problem.

Hi!
The following file is produced by https://github.com/attardi/wikiextractor applied on the arabic wikipedia dump:
jsoup-problem.txt
$ file jsoup-problem.txt
jsoup-problem.txt: UTF-8 Unicode text, with very long lines

I guess there is no BOM, I couldn't find any other unusual tag neither.

I managed to separate as few as possible characters to reproduce. Try this one:
	String htmlFragment = "<doc id=\"1432821\">ي </1971ن </doc>";
	System.out.println(Jsoup.parseBodyFragment(htmlFragment));

But beware, original file behaves strangely when editing. For example using Notepad++ when I add space somewhere some other characters change appearance and some others disappear. I'm not familiar with encodings and characters like these, but in my pasted example after deleting some letters, remaining characters formed "/>".

Another example:

@krystiangorecki, if you look at the character array for the string you posted, it actually looks like this:
<doc id="1432821">s </1971u </doc>
(Where I have used 's' and 'u' to represent the original characters.)
XML is a LTR character based language. The Arabic letter when rendered (in browser or IDE) is putting the rendered into RTL mode, so the input doesn't render as the actual array is ordered. That's why this output looks like that, and why you're seeing the odd backspace issues.
I'm not sure if that's the same issue as the original (maybe the XML output generator has some issues with RTL), or if that's just a copy&paste issue.

That's not copy&paste issue as the same happens when parsing original example file. At some point when parsing doc#1432821 when reader bufPos=11449 state switches from Data to TagOpen then to EndTagOpen and then to BogusComment.

(Closing out as we've had no other related reports which I would expect if there is an endemic issue - expect this was root-caused by a document flipping around the text order.)


 Have so many things which can be implemented

Describe the project
A mental health project that helps the suffering from it and also creating awareness about it in the society.
Issue Type:

[x] Bug Report
[✅] Feature Request
[✅] Documentation
[✅] New Project

Screenshots
Front page demo

If applicable, add screenshots/references to help explain your problem.
https://www.who.int/india/health-topics/mental-health
https://timesofindia.indiatimes.com/blogs/voices/why-mental-health-needs-a-big-boost-in-budget-2023/

Please assign thi issue to me


UfS sync causing duplication of data

Alluxio Version:
alluxio-2.3.0/ alluxio-2.4.0-Snapshot
Describe the bug
I am facing an issue with alluxio ufs sync.  I have some tables whose data get overwrite every hour in S3 but Alluxio not purging old files so my data is getting duplicated while I am query through Alluxio catalog in Presto.
As per doc:
Metadata sync keeps a fingerprint of each UFS file so that Alluxio can update the file if it changes. The fingerprint includes information such as file size and last modified time. If a file is modified in the UFS, Alluxio will detect this from the fingerprint, free the existing data for that file, and reload the metadata for the updated file. If a file is added or deleted in the UFS, Alluxio will update the metadata in its namespace as well.
Data in Alluxio showing files of 2 hrs.
./bin/alluxio fs ls /catalog/dataplatform/tables/task/hive/
-rwx------  systemmanager  systemmanager                0       PERSISTED 09-08-2020 19:08:15:000 100% /catalog/dataplatform/tables/task/hive/_SUCCESS
-rwx------  systemmanager  systemmanager        129016033       PERSISTED 09-09-2020 08:07:55:000 100% /catalog/dataplatform/tables/task/hive/part-00000-e4b562fe-3d03-41ef-8b43-727e26c8aa1a-c000.snappy.parquet
-rwx------  systemmanager  systemmanager        128775563       PERSISTED 09-09-2020 09:08:26:000   0% /catalog/dataplatform/tables/task/hive/part-00000-e6ec7eac-f87a-498d-8429-e209a922b26e-c000.snappy.parquet
-rwx------  systemmanager  systemmanager        117141346       PERSISTED 09-09-2020 08:07:54:000 100% /catalog/dataplatform/tables/task/hive/part-00001-e4b562fe-3d03-41ef-8b43-727e26c8aa1a-c000.snappy.parquet
-rwx------  systemmanager  systemmanager        115965603       PERSISTED 09-09-2020 09:08:26:000   0% /catalog/dataplatform/tables/task/hive/part-00001-e6ec7eac-f87a-498d-8429-e209a922b26e-c000.snappy.parquet
-rwx------  systemmanager  systemmanager         91051189       PERSISTED 09-09-2020 08:07:53:000 100% /catalog/dataplatform/tables/task/hive/part-00002-e4b562fe-3d03-41ef-8b43-727e26c8aa1a-c000.snappy.parquet
-rwx------  systemmanager  systemmanager         86995170       PERSISTED 09-09-2020 09:08:23:000   0% /catalog/dataplatform/tables/task/hive/part-00002-e6ec7eac-f87a-498d-8429-e209a922b26e-c000.snappy.parquet
-rwx------  systemmanager  systemmanager         44151789       PERSISTED 09-09-2020 08:07:49:000 100% /catalog/dataplatform/tables/task/hive/part-00003-e4b562fe-3d03-41ef-8b43-727e26c8aa1a-c000.snappy.parquet
-rwx------  systemmanager  systemmanager         49253364       PERSISTED 09-09-2020 09:08:21:000   0% /catalog/dataplatform/tables/task/hive/part-00003-e6ec7eac-f87a-498d-8429-e209a922b26e-c000.snappy.parquet
Data in S3:

To Reproduce

add a file to the test s3 UFS path
s3://.../test-a
force the sync (expected to show just test-a)
./bin/alluxio fs -Dalluxio.user.file.metadata.sync.interval=0 ls /dir/
remove the existing file and add a different file
remove s3://.../test-a
add s3://.../test-b
Use a longer sync interval (still expected to show just test-a)
./bin/alluxio fs -Dalluxio.user.file.metadata.sync.interval=5m ls /dir/
wait over 5 minutes
Use a longer sync interval to see what it returns (expected to show just test-b but it will show both test-a and test-b)
./bin/alluxio fs -Dalluxio.user.file.metadata.sync.interval=5m ls /dir/

Expected behavior
When the sync is happening it is able to detect new files that got added in S3 but it not able to detect deleted files. So it detects delete also and those files should be deleted from Alluxio.
Urgency
It's a very critical issue as we can not use alluxio with this issue as all the data is incorrect because of duplication.
Additional context
Add any other context about the problem here.

@jainanuj07  thanks for the report! We will investigate this issue

@yuzhu  mind take a quick look at this and see if there is anything obvious?

@jainanuj07 I was not able to reproduce this error. Could you share your alluxio configuration (be sure to remove any sensitive information)
When I tried it, deleted files in the s3 was also deleted from the alluxio space after the sync.
[ec2-user@FsmetaV2S3AlluxioS3aCACHE-THROUGHdavid-masters-1 alluxio]$ bin/alluxio fs ls -R -Dalluxio.user.file.metadata.sync.interval=0 /testdir
          19640       PERSISTED 09-22-2020 13:30:48:000   0% /testdir/LICENSE3
[ec2-user@FsmetaV2S3AlluxioS3aCACHE-THROUGHdavid-masters-1 alluxio]$ aws s3  cp ./LICENSE  s3://autobots-fsmetav2s3alluxios3acache-throughdavid-lai3l6/autobots-test/testdir/LICENSE4
upload: ./LICENSE to s3://autobots-fsmetav2s3alluxios3acache-throughdavid-lai3l6/autobots-test/testdir/LICENSE4
[ec2-user@FsmetaV2S3AlluxioS3aCACHE-THROUGHdavid-masters-1 alluxio]$ aws s3  rm   s3://autobots-fsmetav2s3alluxios3acache-throughdavid-lai3l6/autobots-test/testdir/LICENSE3
delete: s3://autobots-fsmetav2s3alluxios3acache-throughdavid-lai3l6/autobots-test/testdir/LICENSE3
[ec2-user@FsmetaV2S3AlluxioS3aCACHE-THROUGHdavid-masters-1 alluxio]$ bin/alluxio fs ls -R -Dalluxio.user.file.metadata.sync.interval=1m /testdir
          19640       PERSISTED 09-22-2020 13:38:39:000   0% /testdir/LICENSE4

@gpang @yuzhu  any update on this issue? are we targeting fixing it in 2.4?


Event subscription cannot be renewed (events_asyncio)

Environment
SoCo version: 0.22.1
SoCo events module: events_asyncio
Python version: 3.8, 3.7
Error
When trying to renew an event subscription (either with auto_renew=True or manually calling await renew()), the following error is thrown (coroutine 'Subscription._request.<locals>._async_make_request' was never awaited) and the subscription is not renewed:
.../python3.8/asyncio/base_events.py:1860: RuntimeWarning: coroutine 'Subscription._request.<locals>._async_make_request' was never awaited
  handle = None  # Needed to break cycles when an exception occurs.
Object allocated at (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/soco/events_asyncio.py", lineno 491
    return _async_make_request()

Code to reproduce this issue
Use the example from the docs page and modify the main function like this:
async def main():
    # pick a device at random and use it to get
    # the group coordinator
    loop = asyncio.get_event_loop()
    device = await loop.run_in_executor(None, _get_device)
    sub = await device.renderingControl.subscribe()
    sub2 = await device.avTransport.subscribe()
    sub.callback = print_event
    sub2.callback = print_event

    async def before_shutdown():
        await sub.unsubscribe()
        await sub2.unsubscribe()
        await events_asyncio.event_listener.async_stop()

+    await asyncio.sleep(10)
+    print('Renewing subscription..')
+    await sub.renew()

    await asyncio.sleep(100)
    await before_shutdown()

Hi @bdraco : Do you feel like taking a look at this one?

I wasn't able to replicate with pysonos, but there have been some downstream fixes so I'll try directly soon.
I'll open PRs to get the fixes into SoCo later today and then retest directly.

Fix for the session race is here https://github.com/SoCo/SoCo/pull/823. black version seems to have changed and its failing on existing files
Working on replication now

@cyrilwanner Can you give https://github.com/SoCo/SoCo/pull/824 a shot?

Fix for the session race is here #823. black version seems to have changed and its failing on existing files.

As noted in the PR, #825 has the changes to keep Black happy.

@cyrilwanner Can you give #824 a shot?

I can confirm that #824 fixes this bug, both for the manual renewal, but also for auto_renew=True which had the same error.
Thank you so much for resolving this issue so fast!

I can confirm that #824 fixes this bug, both for the manual renewal, but also for auto_renew=True which had the same error.

Great -- thanks for the quick confirmation. Once I have a final PR, I'll cut a v0.22.2 release containing the fix.

Rebased, good to go now 👍

Fixed in v0.22.2.


Add methods to InMemoryQueueStorage to allow requests without serialization

Added the methods AddRequestPointer and GetRequestPointer to allow for working with InMemoryQueStorage without having to use Serialization.
I had to make SerializableRequest public to allow it to be used from the Queue package

adding a a bit to this and will create a new pr when finished.


Update to spec v5.1

Description
Tasks related to spec upgrade v5.1:

[x] On Genesis Updates (Part of Spec PR #711): #545
[x] Use signed_root For Block ID Purposes in Blocks/State - (Spec PR #816): #518
[x] Employ consistent style for function naming: #500
[x] Minor formatting/typos/etc #494
[ ] Update Eth2Genesis Event - (Deposit_Contract PR #22): #493
[ ] Update Eth2 Deposit Event - (Deposit_Contract PR #21): #492
[x] Minor refactor to process_slashings to reduce repeated computation (Spec PR #714): #490
[ ] Make 'Any active validator' statements stricter (Spec PR #703): #489
[x] Do not check withdrawal credentials for existing validators (Spec PR #750): #487
[x] Cache justified/finalized roots in state for use in state transition (Spec PR #756): #483
[x] Only attempt to reward proposer if attestation exists for validator index (Spec PR #744): #481
[x] Handle get_base_reward when no previous balance (Spec PR #718): #480
[x] Handle when no attestations available for crosslinking (Spec PR #717): #479
[x] Fix committee assignment bugs (Spec PR #699): #478
[x] Fix latest_randao_mixes initialization value (Spec PR #713): #477
[x] Add HistoricalBatch tuple, remove merkle_root in favor of hash_tree_root functions. - (Spec PR #739): #476
[x] Fix the "ready to exit" condition in update_validator_registry (Spec PR #705): #475
[x] Fix up previous epoch logic around genesis (Spec PR #672): #474
[x] Epoch Transition at Start of Epoch - (Spec PR #732, #741): #473
[x] Favor Higher Hash Tree Root for Ties in LMD GHOST - (Spec PR #737): #472
[ ] Clarify basic/non-basic SSZ types - (Spec PR #724): #471
[ ] Add tuple lengths to eliminate length mixin for some BeaconState, Deposit fields - (Spec PR #723): #470
[ ] SSZ Spec Changes - (Spec PR #696): #469
[ ] Remove signature/proof_of_possession from message_hash calculation. - (Spec PR #725): #468
[ ] Bugfixes
[ ] Merge to master

Did a run-through of all of the checked tasks and implemented those that were missing. The tasks that still need to be looked at are mostly the ones related to SSZ and Eth2Genesis/Deposit events.

Did one more pass-through; all that's missing is 3 SSZ-related tasks that @schroedingerscode will look over, 2 deposit contract tasks I will work on, bug fixes which @cemozerr, @schroedingerscode, and I are working on, and then we have merge to master!

resolved by #687


Error on Mask RCNN when change BN to GN

I have changed the
model = dict (                        backbone=dict(                                                 norm_cfg=dict(type='GN', requires_grad=True)))
By training, it raises error:
`Traceback (most recent call last):
File "/home/chao/mmcv/mmcv/utils/registry.py", line 179, in build_from_cfg
return obj_cls(**args)
File "/home/chao/swa_mmdetection/mmdet/models/backbones/resnet.py", line 417, in init
self._make_stem_layer(in_channels, stem_channels)
File "/home/chao/swa_mmdetection/mmdet/models/backbones/resnet.py", line 567, in make_stem_layer
self.norm_cfg, stem_channels, postfix=1)
File "/home/chao/mmcv/mmcv/cnn/bricks/norm.py", line 112, in build_norm_layer
assert 'num_groups' in cfg
AssertionError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/chao/mmcv/mmcv/utils/registry.py", line 179, in build_from_cfg
return obj_cls(**args)
File "/home/chao/swa_mmdetection/mmdet/models/detectors/mask_rcnn.py", line 24, in init
pretrained=pretrained)
File "/home/chao/swa_mmdetection/mmdet/models/detectors/two_stage.py", line 26, in init
self.backbone = build_backbone(backbone)
File "/home/chao/swa_mmdetection/mmdet/models/builder.py", line 39, in build_backbone
return build(cfg, BACKBONES)
File "/home/chao/swa_mmdetection/mmdet/models/builder.py", line 34, in build
return build_from_cfg(cfg, registry, default_args)
File "/home/chao/mmcv/mmcv/utils/registry.py", line 182, in build_from_cfg
raise type(e)(f'{obj_cls.name}: {e}')
AssertionError: ResNet:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./tools/train.py", line 187, in
main()
File "./tools/train.py", line 161, in main
test_cfg=cfg.get('test_cfg'))
File "/home/chao/swa_mmdetection/mmdet/models/builder.py", line 77, in build_detector
return build(cfg, DETECTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/home/chao/swa_mmdetection/mmdet/models/builder.py", line 34, in build
return build_from_cfg(cfg, registry, default_args)
File "/home/chao/mmcv/mmcv/utils/registry.py", line 182, in build_from_cfg
raise type(e)(f'{obj_cls.name}: {e}')
AssertionError: MaskRCNN: ResNet: `
Have i did something wrong by the changing?
Thanks for the help.

I have change the pre train model to
`base = '../mask_rcnn/mask_rcnn_r50_fpn_1x_coco.py'
conv_cfg = dict(type='ConvWS')
norm_cfg = dict(type='GN', num_groups=32, requires_grad=True)
model = dict(
pretrained='open-mmlab://jhu/resnet50_gn_ws',
backbone=dict(conv_cfg=conv_cfg, norm_cfg=norm_cfg),
neck=dict(conv_cfg=conv_cfg, norm_cfg=norm_cfg),
roi_head=dict(
bbox_head=dict(
type='Shared4Conv1FCBBoxHead',
conv_out_channels=256,
conv_cfg=conv_cfg,
norm_cfg=norm_cfg),
mask_head=dict(conv_cfg=conv_cfg, norm_cfg=norm_cfg)))
learning policy
lr_config = dict(step=[16, 22])
runner = dict(type='EpochBasedRunner', max_epochs=24)`
And set the norm_cfg as the code above, it runs perfect. Still would like to ask what means 'ConvWS' and is it necessary to change the num_groups in norm_cfg. My training data is 512*512.
Thanks

num_groups necessary
ConvWS is another conv layer


Backwards compatibility

Thank you all for the great package! It is really lovely to be able to include pmlb in a project and only fetch datasets as need (while avoiding having to personally host and provide access to others directly). Recently I ran into the situation that older versions of pmlb now return 404 errors, and when upgrading pmlb some datasets no longer exist or have been renamed. In general, not a huge deal for ad-hoc projects, but pmlb provides this great functionality for some longer term artifacts.
With that in mind, is there any interest in receiving a patch for (some amount) of backwards compatibility? It would be great if datasets that were removed because they are duplicates re-directed to their canonical names, and similarly that failed requests made a best-efforts attempt to provide an alternative request that succeeds.
I'd be happy to take a look at this, but wanted to gauge interest first (in the meantime I've found myself having to include old datasets directly in repos).

hi @josepablocam, we'd be happy to receive such a patch and would really appreciate it! I think it's a bit tricky to handle, so I'm not sure of how to do it, but let us know what you're thinking. Maybe @weixuanfu or @trang1618 or @JDRomano2 can weigh in as well.

@lacava great, let me take a look and plan something out and I'll share here and see what everyone thinks. Thanks again for the library, really appreciate all the work you and the other maintainers/developers have put into this!

Thank you for the suggestion and for offering to help @josepablocam! I'm wondering if a mapping from old dataset names to current dataset names would help?
I do want to emphasize that using the current benchmark collection would be most recommended to avoid errors in past data.

The easiest way to do this is - as @trang1618 said - create a file that lives someplace in the source tree, mapping obsolete database names to their current names, and have fetch_data() check against the contents of this file every time you run it.
Obviously, the tough part will be retroactively identifying all of these changes up until the current version. Has there been any convention (formal or informal) for mentioning when a database name changes, like in commit messages?
As part of this, there is probably a more graceful way PMLB can fail when it can't find the database other than returning a 404. Even just error text with a link to currently valid databases and a link to a Github Issue template for reporting it when legacy database access has 'broken', so we could add it to this hypothetical mapping file.

create a file that lives someplace in the source tree, mapping obsolete database names to their current names, and have fetch_data() check against the contents of this file every time you run it.

that seems like a good solution for handling revisions to dataset names within the current version. i realize now I was thinking of "forwards" compatibility when mentioning trickiness, i.e. coming up with some kind of link redirect strategy or using symbolic links to work with older released versions of fetch_data() (e.g. in https://pypi.org/project/pmlb/0.3/).

Obviously, the tough part will be retroactively identifying all of these changes up until the current version. Has there been any convention (formal or informal) for mentioning when a database name changes, like in commit messages?

I don't think so.

As part of this, there is probably a more graceful way PMLB can fail when it can't find the database other than returning a 404. Even just error text with a link to currently valid databases and a link to a Github Issue template for reporting it when legacy database access has 'broken', so we could add it to this hypothetical mapping file.

I like this idea as well.


Another critical vulnerability with immer

react-dev-utils is using immer 8.0.1 I am receiving critical vulnerability with immer 8.0.1 that is fixed in 9.0.6

This is also affecting our current deployments

Same here, we are also impacted by this

Thanks for the report!
PR already pending in https://github.com/facebook/create-react-app/pull/11364, (duplicated in https://github.com/facebook/create-react-app/pull/11444, https://github.com/facebook/create-react-app/pull/11454),

related to https://github.com/facebook/create-react-app/issues/11174
duplicate of https://github.com/facebook/create-react-app/issues/11443

Hey @petetnt - is there a way to force update this dependency as it's causing dependabot alerts for us.


Redesign fields

The current prop fields are functional, but could be redesigned.
Considerations

Be wary of breaking UI changes
Consider a plugin for alternate UI designs

Related

#701
#64

Adding a request for fields to also be collapsable in the right side panel.


Bootcamp with Markdown

From website to MarkDown

Image must be changed, i will share a zip with the image
BootcampImages.zip

Thank you for this. Moved to PR https://github.com/near/docs/pull/998


error on PackerSync

a little confusing message on PackerSync (i'm on rolling and did a LvimUpdate before that)
Error executing vim.schedule lua callback: ...ite/pack/packer/start/packer.nvim/lua/packer/display.lua:280: Expected 4 arguments
stack traceback:
	[C]: in function 'get_extmark_by_id'
	...ite/pack/packer/start/packer.nvim/lua/packer/display.lua:280: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

eh that seems weird 🤔
can you print output of these commands:
git -C ~/.local/share/lunarvim/site/pack/packer/start/packer.nvim status
git -C ~/.local/share/lunarvim/site/pack/packer/start/packer.nvim rev-parse HEAD

  ~/Source/nix-config  mychanges !1                                                                                 3m 0s 12:29:2
❯ git -C ~/.local/share/lunarvim/site/pack/packer/start/packer.nvim status

HEAD detached at 6afb674
nothing to commit, working tree clean

  ~/Source/nix-config  mychanges !1                                                                                12:29:2
❯ git -C ~/.local/share/lunarvim/site/pack/packer/start/packer.nvim rev-parse HEAD

6afb67460283f0e990d35d229fd38fdc04063e0a

  ~/Source/nix-config  mychanges !1                                                                                12:29:3
❯ lvim -vv
NVIM v0.9.0-dev
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by _nixbld1

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "
/nix/store/iwsgdnc48rhkcs2m3qvn9k32599jxynz-neovim-unwrapped-master/share/nvim
"

Run :checkhealth for more info

might be related to https://github.com/wbthomason/packer.nvim/issues/768

can you run these, and see if it gets fixed or not?
cd ~/.local/share/lunarvim/site/pack/packer/start/packer.nvim
git reset --hard
git checkout master
git pull

yea i saw this, had a little trouble to find out which addon it could be..
i reseted packer to master.. same error and it went back to the same packer version (6afb674)

yea i saw this issue, had a little trouble to find out which addon it could be..
i reseted packer to master.. same error and it went back to the same packer version (6afb674)

maybe checkout my latest commit, and see if that helps
it probably comes from noice

its not happen on main.. only when i add my changes to your theme "magic" :D
i will figure it out..

disabling themes is still not working.
i had to copy the setups from the theme to plugins.lua


Diff split view not working on pull requests

The diff split view is not working on pull requests. Clicking on the Split view button does not throw an error, it just shows no difference to the unified view. It does work on single commits though.
I have made an example on try.gogs.io:
https://try.gogs.io/meb/splitViewTest/compare/master...feature/test?style=split

Gogs version (or commit ref): <IP_ADDRESS>6
Git version: 2.1.4
Operating system: Linux x86_64
Database:

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at http://try.gogs.io:

[x] Yes
[ ] No
[ ] Not relevant

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.


lazy creation/deletion of variables and constraints

Official API for C++ and some community-driven libs (like https://github.com/supermihi/gurobimh) support to add/remove variables without calling APIs.
It seems to be better than current implementation.

It is unfortunate that variables are no longer Copy having to write &x + &y = 0 instead of x + y = 0. I wonder if there is a way around this, maybe if rust-gurobi gave unique indexes to variables and internally mapped them to the real indexes?

@leodasvacas
As you pointed out, writing constraints without borrowing is better than &x + &y. It was removed during the trial and error of the API.
I'll add derivation of Copy trait again until next release.


Support memoryview encoding/decoding as a no-op, calling tobytes() on…

…ly when decode_responses=True
Pull Request check-list
Please make sure to review and check all of these items:

[x] Does $ tox pass with this change (including linting)?
[x] Does travis tests pass with this change (enable it first in your forked repo and wait for the travis build to finish)?
[x] Is the new or changed code fully tested?
[x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description of change
Supports passing memoryviews to redis-py as first outlined in #1265 and discussed further in #1266

Codecov Report

Merging #1285 into master will increase coverage by <.01%.
The diff coverage is 96.96%.

@@            Coverage Diff             @@
##           master    #1285      +/-   ##
==========================================
+ Coverage   92.45%   92.45%   +<.01%
==========================================
  Files          19       19
  Lines        6415     6442      +27
==========================================
+ Hits         5931     5956      +25
- Misses        484      486       +2

Impacted Files
Coverage Δ

tests/test_encoding.py
97.29% <100%> (+1.29%)
:arrow_up:

redis/connection.py
80.65% <87.5%> (-0.06%)
:arrow_down:

redis/_compat.py
86.4% <0%> (-0.8%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks for the review!  I think most of the changes are straightforward.  I like the idea of subclassing memoryview, but I found out that's not possible.  Quoting from the docs: "It is not currently allowed to create subclasses of memoryview."  So I think we can't use a custom class to test that we're never calling tobytes(), since the code relies on isinstance(x, memoryview).  What we really care about is whether the argument is still a memoryview by the time it gets sent over the socket by sendall here, so testing the return value of pack_command() should be sufficient (it gets called here).

This seems ready for another review.  To summarize the changes, everything was straightforward except:

I couldn't subclass memoryview to make a warning-emitting view.  But I think the tests to pack_command and pack_commands are sufficient to test that tobytes() doesn't get called (i.e. that we're saving a memory copy)
It's undecided whether we should allow Encoder.decode to work with memoryviews (see my comment for details).

I also noticed that the argument length-checking and joining code in pack_commands is redundant with that of pack_command.

Thanks @Cody-G. This has been merged.


Use valid version in examples

The version that was previously in the examples won't work as it doesn't exist.

The README on the main branch describes the unreleased version.
If you want to use the (current) latest version, please refer to https://github.com/zulip/github-actions-zulip/tree/v0.1.2


Sync scripts and makefile with equivalent WLO files

For issue #457

Included with https://github.com/application-stacks/runtime-component-operator/pull/485. Thanks


Sample/React. Cannot run the process on a background thread

I ran the Sample/React project and tried to keep the donut from lagging under stress. Enabling CreateWorker didn't work for me :/ After which I saw your comment: "After some testing I've figured worker interaction layer adds too much complexity and makes debugging harder. It'll make more sense and will probably be more performant to use threading on C# side (https://github.com/Elringus/DotNetJS/issues/79)."
So I tried using Task on C# side.
        public async Task StartStress()
        {
            cts?.Cancel();
            cts = new CancellationTokenSource();
            await Task.Run(() => Stress(cts.Token));
        }

        private async Task Stress(CancellationToken token)
        {
            while (!token.IsCancellationRequested)
            {
                var time = DateTime.Now;
                await Task.Run(() => ComputePrime(frontend.GetStressPower()), token);
                frontend.NotifyStressComplete((DateTime.Now - time).Milliseconds);
                await Task.Delay(1, token);
            }
        }

I'm not strong in multithreading, but I thought this should work. Understandably, it didn't work
How can I run the process on a background thread? is it possible now? I will be very grateful for the answer

It should be possible once multithreading support is implemented on .NET's wasm runtime (#79) and after we move to the stock runtime (#20).

thanks a lot 🤗

Running C# backend on worker thread is actually a desirable scenario for my own project, so I'll most likely add an option to serve interop calls via an async queue, so that they can be processed off the main thread by default. If you'd like to feature-proof for that usage scenario, make interop APIs async so that later you can just enable the option w/o breaking anything.


New Chart extending BAR chart doesn't work

Expected Behavior
In the version 2.8.0, creating new chart extending a bar chart, it works.
See the code:
Chart.defaults.derivedBar = Chart.defaults.bar;
var custom = Chart.controllers.bar.extend({
   draw: function(ease) {
          Chart.controllers.bar.prototype.draw.call(this, ease);
   }
});
Chart.controllers.derivedBar = custom;

var ctx = document.getElementById("myChart");
var myChart = new Chart(ctx, {
    type: 'derivedBar',
    data: {
        labels: ["Red", "Blue", "Yellow", "Green", "Purple", "Orange"],
        datasets: [{
            label: '# of Votes',
            data: [12, 19, 3, 5, 2, 3]
        }]
    }
});

Current Behavior
In the new vesion 2.9.1, new chart is not rendered and completely empty (without any message into log)

The problem is ONLY for BAR chart (at least on my tests).
Environment

Chart.js version: 2.9.1
Browser name and version: FF 70.0

@stockiNail I'm guessing you also need the dataset defaults, which were introduced in 2.9. There's some info in the docs and you can see it being done in the current bar controller source. The reason for this is to better support mixed charts

@benmccann Thank you! It works. A hint, if I may. In the doc, about Controllers, I don't find (but I'm getting old and maybe there is....) something like that:
Chart.defaults.global.datasets.derivedBar = Chart.defaults.global.datasets.bar;

@benmccann honestly in another issue I asked for that (without feedback) what you  suggested.
See issue #6482
Thanks a lot!

Sorry but still does'nt work on 2.9.4. I got the example code from this topic including your suggestion:
Chart.defaults.derivedBar = Chart.defaults.bar;
Chart.defaults.global.datasets.derivedBar = Chart.defaults.global.datasets.bar;
Chart.controllers.derivedBar = Chart.controllers.bar.extend({
	draw: function(ease) {
		Chart.controllers.bar.prototype.draw.call(this, ease);
	}
});
new Chart($('[data-chart]').get('0'), {
	type: 'derivedBar',
	data: {
		labels: ["Red", "Blue", "Yellow", "Green", "Purple", "Orange"],
		datasets: [{
			label: '# of Votes',
			data: [12, 19, 3, 5, 2, 3]
		}]
	}
});

It works with type: 'bar' but doesn't work with type: 'derivedBar',.

Sorry but still does'nt work on 2.9.4. I got the example code from this topic including your suggestion:
Chart.defaults.derivedBar = Chart.defaults.bar;
Chart.defaults.global.datasets.derivedBar = Chart.defaults.global.datasets.bar;
Chart.controllers.derivedBar = Chart.controllers.bar.extend({
	draw: function(ease) {
		Chart.controllers.bar.prototype.draw.call(this, ease);
	}
});
new Chart($('[data-chart]').get('0'), {
	type: 'derivedBar',
	data: {
		labels: ["Red", "Blue", "Yellow", "Green", "Purple", "Orange"],
		datasets: [{
			label: '# of Votes',
			data: [12, 19, 3, 5, 2, 3]
		}]
	}
});

It works with type: 'bar' but doesn't work with type: 'derivedBar',.

@vincentwinkel I'm assuming you made an error and it actually works,
since you deleted the comment?
vincentwinkel kirjoitti 19.01.2021 13:20:

Sorry but still doesn't work on 2.9.4. I got the example code from
this topic including your suggestion:
Chart.defaults.derivedBar = Chart.defaults.bar;
Chart.defaults.global.datasets.derivedBar =
Chart.defaults.global.datasets.bar;
Chart.controllers.derivedBar = Chart.controllers.bar.extend({
draw: function(ease) {
Chart.controllers.bar.prototype.draw.call(this, ease);
}
});
new Chart($('[data-chart]').get('0'), {
type: 'derivedBar',
data: {
labels: ["Red", "Blue", "Yellow", "Green", "Purple",
"Orange"],
datasets: [{
label: '# of Votes',
data: [12, 19, 3, 5, 2, 3]
}]
}
});
It works with type: 'bar' but doesn't work with type: 'derivedBar',.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Links:
[1]
https://github.com/chartjs/Chart.js/issues/6636#issuecomment-762777862
[2]
https://github.com/notifications/unsubscribe-auth/AGVNCUM7JPKFWBQMZUPDFLTS2VTG3ANCNFSM4JGE3MHA

@vincentwinkel I'm assuming you made an error and it actually works,
since you deleted the comment?
vincentwinkel kirjoitti 19.01.2021 13:20:

Sorry but still doesn't work on 2.9.4. I got the example code from
this topic including your suggestion:
Chart.defaults.derivedBar = Chart.defaults.bar;
Chart.defaults.global.datasets.derivedBar =
Chart.defaults.global.datasets.bar;
Chart.controllers.derivedBar = Chart.controllers.bar.extend({
draw: function(ease) {
Chart.controllers.bar.prototype.draw.call(this, ease);
}
});
new Chart($('[data-chart]').get('0'), {
type: 'derivedBar',
data: {
labels: ["Red", "Blue", "Yellow", "Green", "Purple",
"Orange"],
datasets: [{
label: '# of Votes',
data: [12, 19, 3, 5, 2, 3]
}]
}
});
It works with type: 'bar' but doesn't work with type: 'derivedBar',.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Links:
[1]
https://github.com/chartjs/Chart.js/issues/6636#issuecomment-762777862
[2]
https://github.com/notifications/unsubscribe-auth/AGVNCUM7JPKFWBQMZUPDFLTS2VTG3ANCNFSM4JGE3MHA

@kurkle yes it's my bad, the error occurred after including this plugin.
I didn't investigate yet but I assume it's linked to this repo so I deleted not to pollute the topic.

@kurkle yes it's my bad, the error occurred after including this plugin.
I didn't investigate yet but I assume it's linked to this repo so I deleted not to pollute the topic.


Content Store is not getting a response value from sendMessage

I noticed that for some actions that get dispatched via mapDispatchToProps, I'm getting the following error:
TypeError: Cannot read property 'payload' of undefined

This error pops up here. I'm opening this in case anyone could possibly think what the issue may be, but I'm investigating.
Note: I'm using redux-thunk and frequently dispatch functions instead of action values with a type, but that shouldn't be an issue. I just thought I'd leave that note here in case it may be useful to debug.

An update. Looks like both problem actions are actually aliases. The response is:
{ error: null }

Seems like I've figured it out. This happens when I don't return a dispatch within a redux-thunk function. For instance, if I have a thunk as follows:
export const someActionCreator = () => {
  return (dispatch, getState) => {
    if (<some_condition>) {
      return;
    }
    return dispatch({ type: SOME_ACTION });
  };
};

That premature return raises the error in this issue. If I swap it out with return dispatch({ type: '' });, it seems to work. This is strange though, and it seems like it's completely kosher to not dispatch (conditional dispatches) from within a thunk (unless I'm mistaken about that - I am still pretty new to redux).

So the issue seems to relate to the default promiseResponder here. If you don't call dispatch, then it looks like the result is returned as undefined (makes sense). I'm adding a simple work around now. Watch for a PR in the next couple minutes.

Awesome thank you @tshaddix! Appreciate the quick response!

This will be included in 0.0.8. Building release now.

💃

Done! Thanks for reporting, @vhmth !

https://www.opentest.co/share/a54ffc8059c911e6a1ffdd04f7a207b4

Sounds great! I'll be in SF in a couple weeks.
Also, sweet extension. Wonderfully clean experience!


problem

when i run "history = model.fit_generator(train, steps_per_epoch=1000,  use_multiprocessing=True, epochs=18, verbose=2, validation_data=(Xval, Yval, w))"
i got the AttributeError: Can't pickle local object 'GeneratorEnqueuer.start..data_generator_task'
do you know how to fix it . win10
and i am confused about “slice_at_t = Lambda(lambda x: tf.slice(x, [0,i,0], [-1,1,CuDNNGRU-1]))”  what's the meaning of this

Could you try to disable multiprocessing?

@LenzDu thank you  , you are right.
and i am confused about “slice_at_t = Lambda(lambda x: tf.slice(x, [0,i,0], [-1,1,CuDNNGRU-1]))” what's the meaning of this


v0.2.2

Documentation
General

[ ] Data model
[ ] How to make bindings
[ ] Write serialize/deserialize by hand
[ ] How to test serialize and deserialize procedures
[ ] Errors
[ ] All examples in documentation are tested via runnableExample. pragmas too.
[ ] Move to own domain
[x] Pragmas
[x] Quick example in README
[x] Quick example in documentation

Deserialize

[ ] Visitor
[ ] SeqAccess
[ ] MapAccess
[ ] Deserializer
[ ] Helper templates documentation and examples
[ ] How deserialization works for case objects with untagged pragma and multiple conditions in of.

Serialize

[ ] SerializeArray
[ ] SerializeMap
[ ] SerializeSeq
[ ] SerializeStruct
[ ] Serializer
[ ] Helper templates documentation and examples

Contribution

[ ] Issue and PR templates
[ ] How to debug make macros
[ ] How does code generation works (about intermediate representation and functions)

Not documentation

[x] Drop support for aliases (because aliases supported by default). make... macros must raise an error, User must make serializable base type, l
[x] Add support for distinct types
[x] Add full support for inheritance. Now the parent's fields are not taken into account
[x] Better errors instead of Error: Expected one of {nnkSym, nnkObjectTy, nnkRefTy} for make... macros
[ ] More hints for pragmas. Like this and this. Example: Use renamed instead of renameSerialize and renameDeserialize
[ ] Add alias pragma. Like this
[ ] Add deserWith pragma. Like this
[x] Add skipPrivate pragma
[x] Add support for field {.defaultValue.} instead of field {.defaultValue(value).}
[x] Reimplement renameAll as pragma for support distinct types and inheritance

Maintaining code that respects styleCheck is hard, especially with third-party dependencies. Therefore, support for styleCheck has been removed from the priorities for this release.
Support for aliases should be completely removed, including for the Foo = ref Bar cases, for consistency. The library should provide a generalized function to de/serialize ref types based on generated serialization functions for base types.
Support for distinct is removed from the priorities for this release.


add IMU error

before I add the IMU， the result is quite good.
After adding IMU, I found the laser is  sloping, about 15 degree. When I rotate my equipment, the laser also move, but not in the way i want .  I have added IMU for hector_slam, everything works fine. But this time, it does not work! BELOW is my Problem:
I checked the IMU data in another rviz window, it seems OK, but I don't known what 's the problem

`

`

here is my TF about the imu_link
node name="tf_base_link2imu_link" type="static_transform_publisher" pkg="tf" args="0 0 0 0 0 0 /base_link /imu_link  40"

Please provide a fork of cartographer_ros with your configuration files added and an example bag that shows the problem.

Sounds like this is a problem with an incorrect transform between the IMU and the LIDAR. Closing due to inactivity.


CI also builds tests

Pull Request Functional Test Report for #2867 / ea8ef10d396f7f49e9fa65feae0577d2bb040ea3
No tests identified for validation.
Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 54.484%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1847

Relevant Lines:
3390

💛  - Coveralls

Infeasible:
# github.com/openconfig/featureprofiles/feature/experimental/bgp/otg_tests/bgp_2byte_4byte_asn.test
/opt/hostedtoolcache/go/1.21.8/x64/pkg/tool/linux_amd64/link: mapping output file failed: no space left on device
# github.com/openconfig/featureprofiles/feature/experimental/bgp/otg_tests/base_bgp_session_parameters.test
/opt/hostedtoolcache/go/1.21.8/x64/pkg/tool/linux_amd64/link: mapping output file failed: no space left on device
# github.com/openconfig/featureprofiles/feature/experimental/bgp/otg_tests/bgp_always_compare_med.test
/opt/hostedtoolcache/go/1.21.8/x64/pkg/tool/linux_amd64/link: mapping output file failed: no space left on device
# github.com/openconfig/featureprofiles/feature/experimental/bgp/otg_tests/bgp_remove_private_as.test
/opt/hostedtoolcache/go/1.21.8/x64/pkg/tool/linux_amd64/link: mapping output file failed: no space left on device
# github.com/openconfig/featureprofiles/feature/experimental/bgp/otg_tests/bgp_afi_safi_defaults.test
/opt/hostedtoolcache/go/1.21.8/x64/pkg/tool/linux_amd64/link: mapping output file failed: no space left on device


Sometimes the cached page becomes blank page.

I use mvcdonutcaching on one page of my site. It runs well except that sometimes the page becomes blank page, so that I must restart IIS to recover. the problem happens every a few days.
I wonder if this problem is related to mvcdonutcaching.
Thanks a lot!

Actually we have similar issue - sometimes is page just empty.
Should we rewrite controller actions to async, or why @leen3o mention it?

I had it too. Running on azure. Problem is, the status code is still OK, so my monitors dont alert me...

In my case I found that removing this line fixed my issue,
"ControllerContext.RequestContext.RouteData.Values.Remove(value.Key);"
there should be some problems removing values from ControllerContext.RequestContext.RouteData that cause that the page become a blank page without throwing any error.


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.167

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.167


Add percent-* (discount price tag) icons

Came across percent-circle in the wild the other day… Although others seem more appropriate representations of discount/price  tag…

The percentage sign inside tag isn't centered.

I'm not sure about this one, I get sort of the use case, it's the same as percent-badge. But I'm why we should bloat the library with all kinds of percent icons. I think we should a bit more selective.

all kinds of percent icons. I think we should a bit more selective.

@ericfennis Yeah maybe we don’t need them all, but most symbols in the library have -square and -circle variants… Although I think percent-diamond is the most relevant one. So since percent-tag doesn’t really sit very well, as @jguddas pointed out, I have dropped it…


Update ruby versions to latest in Travis and update Thor to latest version in Gemfile

I would accept an update to the Ruby versions but updating the thor dependency would put it out of sync with the vendored thor.

@ddollar ok, I'll revert Thor changes, one moment

Please update this PR to be only an update to the Ruby versions and I would be happy to merge. Thanks!

@ddollar done! Sorry for a big delay)

@ddollar do you have time to review it again?)

@kkohrt @micwoj92 almost a year ago I made my changes but @ddollar gone somewhere =(

I think, I can close it now, because nobody cares


Encrypt/Decrypt all notes

Can you add a new feature where you can encrypt/decrypt all notes(markdown) in a single execution instead of manually encrypting/decrypting each notes? Thanks in advance. I like this plugin. Hopefully this works in mobile too.

@reversed-heartbeat
Thanks for your feedback. This is a great idea. Would it be okay to encrypt it during the next write or should all notes encrypted during activation?
Out of curiosity: do you use OpenPGP.js or the gpg CLI wrapper?

That's a good one too! I think that would be really useful.
I use the latter since I'm a bit familiar with it. And thanks you for your dedication :) Looking forward~

Implemented in release https://github.com/tejado/obsidian-gpgCrypt/releases/tag/0.2.0
I would be delighted if you could give me some feedback on this.

Hey bro, I'm really sorry for the late reply. I've been really swamped with work lately, and I completely forgot about this until I saw your notification, but I just wanted to say that the improvement updates you made to your plugin are absolutely amazing! I've been using your plugin for a while now, and I'm always impressed with how you continue to improve it. <3
You've made the plugin even better than it was before and definitely a valuable asset to the community.  Thanks for all your hard work!
Keep up the good work bro! :)


Reorder arguments to index, adjust, and adjust'

For other arrays, containers, strings, etc., including Sequence that users will often be converting from, index takes the container before the Int, and adjust and adjust' take the (a -> a) function as the first argument. In addition to being consistent with these cases, I think one can argue that the standard order is more often useful for currying.
I realize this is a breaking change, but I think it'd be much better to make it now, before there are too many users, rather than wait. Users that are surprised by the change should at least get a compiler error, plus you'll obviously bump the library's version number.
Thanks very much for this library!! Sorry to nit-pick.

Whether you do this change or not, you could add a WARNING pragma to these functions, at least temporarily. Also, if you're not sure, you could include a request for feedback or an opinion/vote in the warning.
Finally, this issue applies to index and update in Data.AMT in extended-containers also.

(index v) can be passed to a linear or binary search algorithm, or anything else that expects a (possibly memoized) function, such as . (composition) or maybe or really anything else in functional programming. (index i) [for an Int i] would only be useful to pass to a function that works on potentially many vectors by always looking at just their ith element, which is rarer than a memoized function of Ints. For adjust and adjust', I guess the (a -> a) function could be something that you want done to a bunch of vector elements, like incrementing/decrementing them by 1 or some other fixed amount. Again, it's rarer that you want the ability to adjust a fixed-index set of elements by various (a -> a) functions. (I do think this currying argument is weaker for adjust/adjust' than for index, and the consistency argument with other container types is the main one.)
FWIW, the ListLike package uses the standard order for index also.


I'd love to see the ability to use two cameras

At the office, I have an external USB camera and on the go the laptop has one built in. I don't know how I'd get this working on both.

Just edit app config, specify an external device. But you'll probably need to add more face samples...
I'd propose a small patch to support camera fallback:
--- compare.py.original	2018-11-18 12:29:05.533869737 +0700
+++ compare.py	2018-11-18 12:29:40.258341917 +0700
@@ -58,7 +58,14 @@
 timings.append(time.time())

 # Start video capture on the IR camera
-video_capture = cv2.VideoCapture(config.get("video", "device_path"))
+devices = (d.strip() for d in config.get("video", "device_path").split(','))
+for dev in devices:
+    if not os.path.exists(dev):
+        continue
+    video_capture = cv2.VideoCapture(dev)
+    break
+else:
+    sys.exit(1) # not sure about exit code

 # Force MJPEG decoding if true
 if config.get("video", "force_mjpeg") == "true":

This patch will allow to specify multiple device paths and pick up first available camera.
Or did you mean both cams simultaneously (like windoze does on my laptop)?

I appreciate the effort, but i think this might be out of scope. I just don't see many uses for this feature and more code to maintain.
However, you could solve this with a dynamic symlink. Make a dummy symlink and configure Howdy to use it as device path. A very simple python script could then point that symlink to the right camera for the situation.


Feature Request: Optionally store Audio and Transcription under 'yyyy/mm/dd/hh'

Hello,
Currently the WAV files are stored in S3 at the root of the bucket.
client.put_object(Body=wav_bytes.getvalue(), Bucket=f'{os.getenv("s3_recordings_bucket")}', Key=f'{self.contact_id}.wav', Tagging=self.encoded_tags, ContentType='audio/x-wav')

We have an 11 year retention period in place for all files related to Amazon Connect, so these buckets become unmanageable very quickly.
By environment variable, it would be good to store these files similar to how Kinesis Firehose stores in S3, under yyyy/mm/dd/hh/{self.contact_id}.wav
I can make the change to the code after we pull it down, but that makes upgrades challenging.  I'm not a Python coder or I'd offer to do a PR.  My tweak will be hacky at best.  :)
The same environment variable should also ensure the transcriptions are stored in the same location.
Thanks!

can add this in next release. Makes sense to do so.


[YouTube] [ChannelTab] extract subscriber count and channel views

Checklist

[X] I'm reporting a site feature request
[X] I've verified that I'm running yt-dlp version 2021.12.27. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
https://www.youtube.com/channel/UCR1IuLEqb6UEA_zQ81kwXfg
Description
I have implemented a scraper with BeautifulSoup to extract some additional metadata from YouTube Channel for a project of mine. I was wondering if there would be interest to integrate that into yt-dlp? Both of these fields are extractable from the page without an API call.

Channel Subscribers: This information is available in the ytInitialData script in header -> c4TabbedHeaderRenderer -> subscriberCountText

The number is unfortunately truncated and as a string, e.g. "2.03M subscribers"
As far as I have observed the unit can be M for millions, K for thousands and none for below 1000.
That is language specific, but as far as I know yt-dlp defaults to english already?

Channel Views: This information is in ytInitialData at itemSectionRenderer -> contents -> viewCountText.

This is as a string and will need some logic to extract the numbers.

Additionally, for extracting banners there is already an issue open: #2237.
This would be a great addition to have upstream directly in yt-dlp.
Verbose log
Does not apply...

For subscriber count we maybe could add a field such asfollowers or playlist_followers? This can probably be extracted in other places the subscriber count shows up too. Language is not an issue, we enforce English now for YouTube regardless due to reasons like this.
For the channel view count, I'm not sure where you were looking. I think you might've gotten mixed up with individual video view counts? The channel view count is only provided in the response for the about tab.

This can probably be extracted in other places the subscriber count shows up too.

I did some testing, the subscriberCountText field is also available on the Video page (https://www.youtube.com/watch?v=<video-id>) but at a different location in the json: videoSecondaryInfoRenderer -> owner -> videoOwnerRenderer
But it is for example not available in the playlist page (https://www.youtube.com/playlist?list=<playlist-id>).

For the channel view count, I'm not sure where you were looking. I think you might've gotten mixed up with individual video view counts? The channel view count is only provided in the response for the about tab.

Yes, you are right about this one, I've meant to specify, that field is only available at the about page (https://www.youtube.com/c/<channel-name>/about). None the less, still useful, but I guess less easy to access if you need to request a specific page /about to get that...


Switch to binary and toml formats for saves/config

Description

Use binary format for save files to improve saving/loading
Use toml file for configuration and pretty print it to make it very much more readable.

Alternate Designs
N/A
Benefits

Faster loading/saving
Config files are easier to read and write

Possible Drawbacks
Saves files not readable, which might actually be good for anti-cheating. If you actually care :)
Applicable Issues
N/A

Codecov Report

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

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   95.48%   95.48%
=======================================
  Files           3        3
  Lines         133      133
=======================================
  Hits          127      127
  Misses          6        6

Impacted Files
Coverage Δ

src/resources.rs
93.75% <ø> (ø)
:arrow_up:

src/main.rs
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


ImGui: Cherry-picked optimizations for OpenGL backend from master (possible fix for #3455)

Currently ImGUI is overall outdated, but OpenGL backend is especially outdated and based on one of the early releases. Because latest version is significantly different and incompatible with rest of Osiris imgui, i've decided to cherry-pick performance-impact commits from master branch to our version. As a bonus i've cherry-picked commits that enabling usage of SSE-powered ImRsqrt and ImRecip functions for imgui_draw.cpp calculations on both Windows and Linux.

As a bonus i've cherry-picked commits that enabling usage of SSE-powered ImRsqrt and ImRecip functions for imgui_draw.cpp calculations on both Windows and Linux.

Is OpenGL even used on Windows?

As a bonus i've cherry-picked commits that enabling usage of SSE-powered ImRsqrt and ImRecip functions for imgui_draw.cpp calculations on both Windows and Linux.

Is OpenGL even used on Windows?

This referring to changes inside imgui_draw.cpp/imgui_internal.h, it's about general calculations, not drawing.


first search is always 5 seconds longer then following

Im not sure if it is fixable or not but processingTimeMs of the first search after initialisation of client is always 5.2+ seconds, while all the following no matter if I'm trying same index or any other are around 100ms

We now pre-warm some models on startup via a startup script https://github.com/marqo-ai/marqo/blob/mainline/src/marqo/tensor_search/on_start_script.py#L77-L84. Setting this via configuration is something for the future.


Sped up centering operation in transform_data

Using (Boehm, 2014) data, participants 1-5:
Before:
CPU times: user 52.8 s, sys: 516 ms, total: 53.3 s
Wall time: 53.3 s
After:
CPU times: user 6.32 s, sys: 91.4 ms, total: 6.42 s
Wall time: 6.43 s
Should also scale better to larger datasets.

Does now print this:
RuntimeWarning: Mean of empty slice
  mean_last_dim = np.nanmean(data.values, axis=-1)

When running, let me know if this is an issue then we can catch the warning so it does not print:
https://stackoverflow.com/questions/29688168/mean-nanmean-and-warning-mean-of-empty-slice


proxy_protocol: Introduce custom TLV support for upstream PP2 headers

Commit Message:
This commit introduces support for injecting custom TLVs into the Proxy Protocol v2 (PP2) header for upstream transport sockets. This enables xDS control planes to build upstream PP2 headers with greater flexibility. Previously, upstream PP2 headers only passed through TLVs from downstream connections when using the Proxy Protocol listener, limiting customization.
With this change, users can define custom TLVs by specifying host metadata in a well-known namespace, providing dynamic, granular control over PP2 header content. For example:
clusters:
      - name: httpbin
        load_assignment:
          ...
          endpoints:
          - lbEndpoints:
            - metadata:
                filter_metadata:
                  envoy.transport_socket_match:
                    outbound-proxy: true
                typed_filter_metadata:
                  envoy.transport_sockets.proxy_protocol:
                    "@type": type.googleapis.com/envoy.extensions.transport_sockets.proxy_protocol.v3.CustomTlvMetadata
                    entries:
                      - type: 0x96
                        value: Zm9v # foo
                      - type: 0x97
                        value: YmFy # bar
               ...

By decoupling upstream PP2 customization from downstream listener config, this unlocks more flexible use cases for Proxy Protocol in upstream paths.  Earlier approaches considered extending upstream_proxy_protocol to support TLV configuration but were rejected due to added control plane complexity. Similarly, reusing the envoy.network.proxy_protocol_options namespace was evaluated but required significant refactoring and risk.
Additional Description: N/A
Risk Level: Low
Testing: Unit & integration
Docs Changes: TODO. Include in the protobuf docs
Release Notes: Included in the changelog as a new feature
Platform Specific Features: N/A
[Optional Runtime guard:]
[Optional Fixes #Issue] https://github.com/envoyproxy/envoy/issues/18520
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

For more discussion on some of the implementation choices, see https://github.com/timflannagan/envoy/pull/1 for an initial round of review. Also, I haven't written C++ in a while so I'm happy to make any style-related changes.

Looks like integration tests are failing. I'll fix that tomorrow. At a glance, the assertions are expecting a certain ordering for the custom TLVs that are being defined, and there's deterministic output issues.

/api lgtm


Hatch 628 - Abstract Delete pattern functionality

HATCH-628

Wrote cypress test for delete pattern navigation
Pulled table functionality from delete.js and created new DeleteTable.js component in components/ directory
Added option for medium-severity deletion and made new DeleteConfirm.js component in components/ with confirmation modal

Sorry for all the commits 🤦🏼♀ now I know how to do the eslint check before pushing code!

Sorry for all the commits 🤦🏼♀ now I know how to do the eslint check before pushing code!


drone`s issue

Description

drone has no welder, omnitool and else
Reproduction

spawn drone
play drone
Screenshots

Additional context

It doesn't really matter, since drones as a ghost role are disabled until they can reworked


IPD-51646: Skip the bin NPM feature

Original fix by @cjwilliams. Up for initial testing.

Attention: this fix will likely require us to gain access to the qemu nodes to make sure they get the latest version of this repo. As it stands this access is limited to people outside of the company, so we'll have to figure out a way to do this.


validator: indicator for loading keys

Loading more than 100 keys or loading a validator on a low-resource machine lets the user believe the client is "stuck" as it does not indicate any status.
Describe the solution you'd like
10% .. Loaded 10/100 keys (0xbb3d65dd002050ac12d963226f41dd88)
20% .. Loaded 20/100 keys (0xb0f4f2afa59948b642fd831d8dae30c6)
30% .. Loaded 30/100 keys (0x6362f93a37edaa6337ca0eb670872bbc)
40% .. Loaded 40/100 keys (0x9dc4325370645bc9ff1440519db86db3)
50% .. Loaded 50/100 keys (0xe257ca039b831fb548cdc37bd6b35439)

Describe alternatives you've considered
Getting a drink

Getting a drink

Getting a drink :rofl:

Keeping this issue open and it only requests to display progress. For actually speeding this up, see https://github.com/ChainSafe/lodestar/issues/4179


fix(carousel): adjust types for embla-carousel-react@8

Fixes #2322 , #2281
Thanks to @akynau & @nianiam

Thanks @Gomah, @akynau , and @nianiam. Just updated carousel.tsx locally and this worked perfectly for me 👍

Thanks @Gomah, @akynau , and @nianiam. Just updated carousel.tsx locally and this worked perfectly for me 👍

May have spoke too soon! When attempting to integrate it into a Zustand store (or even in just a component), I get Unsafe assignment of an any value for any variable that referred to the CarouselApi type.
For example, when just doing
const [api, setApi] = React.useState<CarouselApi>()
api would throw the eslint error Unsafe assignment of an any value.
What fixed it was adding embla-carousel as a dev dependency. For pnpm users, they suggest doing so here

Fixes #2322 , #2281
Thanks to @akynau & @nianiam

Hi so I tried both of the above fixes but it breaks the type for plugins in useEmblaCarousel hook call.
const [carouselRef, api] = useEmblaCarousel(       {         ...opts,         axis: orientation === "horizontal" ? "x" : "y",       },       plugins //Type error with plugins)
Error -
 Argument of type '(CreatePluginType<LoosePluginType, {}>[] | undefined)[] | undefined' is not assignable to parameter of type 'CreatePluginType<LoosePluginType, {}>[] | undefined'.      Type '(CreatePluginType<LoosePluginType, {}>[] | undefined)[]' is not assignable to type 'CreatePluginType<LoosePluginType, {}>[]'.        Type 'CreatePluginType<LoosePluginType, {}>[] | undefined' is not assignable to type 'CreatePluginType<LoosePluginType, {}>'.          Type 'undefined' is not assignable to type 'CreatePluginType<LoosePluginType, {}>'.            Type 'undefined' is not assignable to type 'LoosePluginType'.


Overwriting log file

Hello!
I am attempting to use this package but it seems like it's just overwriting the log file in S3 each time a logging event happens. My setup is below, is there something I need to do to make it append to the file rather than overwrite it?
 loggerConfig.WriteTo.AmazonS3(
     "log.txt",
     "bucket",
     Amazon.RegionEndpoint.USEast2,
     "key",
     "secret",
     Serilog.Events.LogEventLevel.Verbose,
     null,
     new Serilog.Formatting.Json.JsonFormatter());

Closing this since this seems to be a limitation of S3

To be honest, I have never seen this behaviour, but thanks for the information.


v4: tsconfig and alias not working as expected all the time

Not getting expected results on all parts when using tsconfig alias/paths
Atm its messing up if ./ is added in fusebox alias config, and its not able to make tsconfig paths correctly when using paths inside the packages
For testing code in issue use this:
https://github.com/vegarringdal/bug-5

npm install
node fuse to run

TSconfig
"baseUrl": "./",
    "rootDir": "./",
    "paths": {
      "@tsconfig/test1": ["packages/test1"],
      "@tsconfig/test2": ["./packages/test2"],
      "tsconfig-test3": ["packages/test3"],
      "tsconfig-test4": ["./packages/test4"]
      }

fusebox config alias
 alias: {
        "fusebox-alias-test5": "packages/test5",
        "fusebox-alias-test6": "./packages/test6" //  "./" <- part breaks it
      }

folder structure of project

/**
 *  "@tsconfig/test1": ["packages/test1"],
 */
import "@tsconfig/test1"; // <- working 😁
import<EMAIL_ADDRESS>// <- failing ❌, I would have expected this to work..
import<EMAIL_ADDRESS>// <- failing  ❌, I would have expected this to work..

/**
 *  "@tsconfig/test2": ["./packages/test2"],
 */
import "@tsconfig/test2"; // <- working 😁
import<EMAIL_ADDRESS>// <- failing ❌, I would have expected this to work..
import<EMAIL_ADDRESS>// <- failing ❌, I would have expected this to work..

/**
 *  "tsconfig-test3": ["packages/test3"],
 */
import "tsconfig-test3"; // <- working 😁
import "tsconfig-test3/test3.js"; // <- failing ❌, I would have expected this to work..
import "tsconfig-test3/test3.css"; // <- failing ❌, I would have expected this to work..

/**
 *  "tsconfig-test4": ["./packages/test4"]
 */
import "tsconfig-test4"; // <- working 😁
import "tsconfig-test4/test4.js"; // <- failing ❌, I would have expected this to work..
import "tsconfig-test4/test4.css"; // <- failing ❌, I would have expected this to work..

/**
 *  "fusebox-alias-test5": "packages/test5"
 */
import "fusebox-alias-test5"; // <- working 😁
import "fusebox-alias-test5/test5.js"; // <- working 😁
import "fusebox-alias-test5/test5.css"; // <- working  😁

/**
 *  "fusebox-alias-test6": "./packages/test6", the "./" part messes it up..
 */
import "fusebox-alias-test6"; // <- failing ❌, I would have expected this to work..
import "fusebox-alias-test6/test6.js"; // <- failing ❌, I would have expected this to work..
import "fusebox-alias-test6/test6.css"; // <- failing ❌, I would have expected this to work..

import<EMAIL_ADDRESS>// <- working 😁

// last is prb correct
// uncomment- cant see other working/failing with this one active
// import<EMAIL_ADDRESS><- failing ❌ but 😁 I guess
// since it cant know to go under dist/esm or do you want that as alternative if it does not fine any files?

// this one is to much noise
// import "@lit-html-free/grid" // <- working 😁 but not using ts:main of package.json, is it correct ❔

@vegarringdal It will not work since you tsconfig paths is incorrect.
 paths: {
          '@app/*': ['foo'],
},

The reason why this works  import "@tsconfig/test1"  and this import "@tsconfig/test1/bar" doesn't is obvious.
  "@tsconfig/test1/*": ["packages/test1/*"],

Refer to this test
It's 100% compliant with typescript specs.

great 👍


Added partition_id in RelationalOperator::feedInputBlock.

As a part of adding partition support, this PR added an optional partition_id in RelationalOperator::feedInputBlock.
Assigned to @hbdeshmukh. Thanks!

Looks good to me. Merging.


Timber 2.0 - Menu item "current_item_ancestor = false" for Archive item menu when i'm on single (CPT)

Expected behavior

Menu item archive has "current_item_ancestor" to true when i on single.
Actual behavior

I created a CPT and in my menu I added the archive of this CPT (and not a custom link) but when i'm on single the "current_item_ancestor" is equal to false.

And what is weird is that it is the article archive that has "current_page_parent" to true
Steps to reproduce behavior

Create CPT
Add in menu archive Post and Archive of CPT
Go to single on CPT

What version of WordPress, PHP and Timber are you using?

"timber/timber": "2.0.0-alpha.1"
"roots/wordpress": "5.7.2",
Php : 7.4.3
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)

Composer

If we go down that road

I wouldn't go down that road 😄 Did you have a look at this function? This is a mess. We'd better wait for WP to fix this (in some years 😅) and stick to it in the meantime.


hf_olmo: support flash attn 2

https://github.com/allenai/OLMo/issues/460, tested with a simple snippet as below:
import transformers, torch

model = transformers.AutoModelForCausalLM.from_pretrained("allenai/OLMo-7B-Instruct", use_flash_attention_2="flash_attention_2", trust_remote_code=True).cuda()
tokenizer = transformers.AutoTokenizer.from_pretrained("allenai/OLMo-7B-Instruct", trust_remote_code=True)

print(tokenizer.decode(model.generate(torch.tensor(tokenizer.encode("Hello World! My name is")).unsqueeze(0).cuda())[0]))
# Hello World! My name is Emily and I am a second year student at the University of California,

Someone familiar with transformers internals should review this (maybe @AkshitaB). I'm not sure what transformers does with this, but I'd be very cautious if they're monkey-patching our attention mechanism since flash-attn expends a different input shape (the head and sequence dimensions are flipped compared to what we normally do).


add screenshot of the app and a few sentences how to use it and what is the source of iracing seasons (pulled directly from iRacing or from PDF?)

Great idea. Will consider adding this in the future


Collection of tips to get Yode-Nvim working with other plugins

The seditors are special buffers of type "acwrite", which may cause problems with other plugins. If you have any tips how to change settings to make a plugin work with the seditors, post them here as a comment.

How I handle closing the yode-nvim buffers with my session saving: https://github.com/hoschi/yode-nvim/issues/7#issuecomment-1038300549


rotation_matrix working?

I tried the following:
julia> v = [1.0, 1.0, 0.0]
julia> z = [0.0, 0.0, 1.0]
julia> M = rotation_matrix(z, 45.0)
julia> M * v
3-element Vector{Float64}:
 -0.3255815357163887
  1.3762255133518482
  0.0

Since v has an angle of 45 deg with either x or y, should this rotation about z yield a vector along x (or y depending on the direction that it is rotating?)

Yeah, this is what happens when you don't convert from degrees to radians.......

Ok, makes sense. So two things

Documentation
Maybe some way to specify degree/rad?

In Julia, a function f normally has another version fd e.g. cosd for degrees. Maybe you could have a rotarion_matrixd? Or just add a keyword argument.

Not really a bug why the tag? :laughing:


Fix for GHC 8.2.

This fixes #20 and makes it compile with GHC 8.2.
Trivial API revert.

Thanks a lot!

Could you please release it on Hackage?

Released. Sorry for the delay and thanks again!


chore: remove dependency on syn_derive

Removes dependency on syn_derive crate.
The added dependency is saving 21 lines of code at the cost of added Cargo.lock entries for a large list of projects depending on borsh-rs. The motivation came from the fact that proc-macro-error crate is unmaintained and the rust ecosystem is slowly transitioning into proc-macro-error2 (already 800k downloads) but this change did not land on syn_derive. This causes crates depending on borsh-rs transtively depending on syn_derive and thus proc-macro-error.  Also it is generally thought as a good practice to remove external dependencies especially if they are not gaining us much to begin with.
Related to https://rustsec.org/advisories/RUSTSEC-2024-0370.html

I just pushed an update to switch syn_derive to proc-macro-error2, but yeah I have to agree that it seems pretty silly to use syn_derive for such a small part.

@dzmitry-lahoda it is your call, my original intention opening this was basically:

syn_derive is nearly not used in the repo, just removes 21 lines of code while adding a dependency tree which contained an outdated dependency (proc-macro-error).
I was under the impression that @Kyuuhachi was not active and deprecated the repo. But I left hem a bump comment (https://github.com/Kyuuhachi/syn_derive/pull/5#issuecomment-2456266351) and they updated the version.

So I think you can either go to 0.2.0 or remove the dependency completely. If I were you I would be leaning towards removing the dependency if there are no other places you can impact with the added syn_derive in your code base.

@kayagokalp this removes 2 dependencies from dependency tree,
proc-macro-error(2) and proc-macro-error-attr(2)

collapse
135d134
<  "syn_derive",
587,607d585
< name = "proc-macro-error-attr2"
< version = "2.0.0"
< source = "registry+https://github.com/rust-lang/crates.io-index"
< checksum = "96de42df36bb9bba5542fe9f1a054b8cc87e172759a1868aa05c1f3acc89dfc5"
< dependencies = [
<  "proc-macro2",
<  "quote",
< ]
<
< [[package]]
< name = "proc-macro-error2"
< version = "2.0.1"
< source = "registry+https://github.com/rust-lang/crates.io-index"
< checksum = "11ec05c52be0a07b08061f7dd003e7d7092e0472bc731b4af7bb1ef876109802"
< dependencies = [
<  "proc-macro-error-attr2",
<  "proc-macro2",
<  "quote",
< ]
<
< [[package]]
900,911d877
< ]
<
< [[package]]
< name = "syn_derive"
< version = "0.2.0"
< source = "registry+https://github.com/rust-lang/crates.io-index"
< checksum = "cdb066a04799e45f5d582e8fc6ec8e6d6896040d00898eb4e6a835196815b219"
< dependencies = [
<  "proc-macro-error2",
<  "proc-macro2",
<  "quote",
<  "syn 2.0.87",

syn_derive would've been worth exactly the 21 lines it replaces, but i
didn't see any improvement in error location output in the variant when syn_derive is bumped,
despite proc-macro-error(2) and proc-macro-error-attr(2) being used.
At least for this simple use case.

@race-of-sloths score 5

@dj8yfo Thank you for calling!
@kayagokalp Thank you for the contribution! Join Race of Sloths by simply mentioning me in your comment/PRs description and start collecting Sloth Points through contributions to open source projects.
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths and receive a reward
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Invite contributor @race-of-sloths invite to invite the contributor to participate in a race or include it, if it's already a runner.
Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called

@Kyuuhachi oh, well, probably proc-macro-error2 is for reporting errors when syn_derive macros are being derived, and not for the ones, which happen during syn::parse::Parse parsing. That got me confused for a moment

syn_derive would've been worth exactly the 21 lines it replaces, but i

@dj8yfo I would still argue that is not the case. You have ~580 crates depending on borsh whereases there are 7 crates depending on syn_derive. By removing this dependency you are most likely removing the syn_derive entry from >573 crates' Cargo.lock which is very nice outcome. This numbers are considering only the number of crates published, the real numbers are probably more than this.
I guess this is the effect of being in a hot-path so it makes even more sense to be careful in these type of repos.
On top of this added level of indirection while managing your dependencies is something else and in my opinion should also be taken into account while accessing whether to depend on something or implement it yourself.
Thanks for the quick fix, and release!

@kayagokalp only 2 mentioned ones (proc-macro-error2... )aren't depended on otherwise. Other 5 are included as dependencies anyway.


[HUDI-5583] multiple writers dead lock while cleaning

Change Logs
deadlock
Impact
None
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

the source code fix may not be required. #7739 we found an issue w/ locking.

OK, I test master branch with this UT, it succeed

@nsivabalan  can I just add TestMultipleHoodieJavaWriteClient? WDYT

yes, sure. adding additional tests is def good.

rebase w/ latest master as well

@fengjian428 : any updates here

@fengjian428 : any updates here

conflict resolved.  now this pr only contain one UT for multiple java client

@hudi-bot run azure

Azure CI is green.


Need a logo

Any ideas for a utron logo?

Bsed on Ultron? ;-)
http://www.google.com.br/imgres?imgurl=http://vignette2.wikia.nocookie.net/marvelmovies/images/b/bc/Ultron_Empire_textless.jpg/revision/latest%3Fcb%3D20150126200212&imgrefurl=http://marvel-movies.wikia.com/wiki/Ultron&h=793&w=640&tbnid=HVhF_evEqg8YbM:&tbnh=186&tbnw=150&usg=__TohwugVvX_8vHYb_U3HqmSBNPd4=&docid=lRxxQkip-g9X-M&itg=1&client=ubuntu

@OsvaldoTCF cool. Is the image copyrighted? We need something cool like that.

Well, this is the image of the cinema, so I imagine that has copyright. Perhaps a good idea is to use it as a base.

Having Ultron as a reference could butthurt someone, and you really doesn't want to be sued by Disney. How about using a robot gopher to be the concept for the logo?

@euller88 you are right.  A robot gopher will be cooler.

@euller88 do you use gitter? We can discuss more about utron there https://gitter.im/gernest/utron

Ok guys :) maybe something like that ?

gr8
in zip you have ai,svg and png version. Enjoy :)
ultron_logo.zip
;)

ok, i back to my webapp powered by ultron :) thx for gr8 frame ;>

@paramah minor detail, what is the purpose of the red line on the gopher fore head. I think it doesnt mix well with gopher

If the redline is gone, the gopher looks even more bad ass than utron. :)

@paramah Thank you.

@gernest hmmm, right. Give me some time i will make some little changes with colors and shapes.

Okay, I will just add this for now to see how it looks like. Later you can just update the image and submit a PR for easy contributing.

see #63


Error in Imputation using impute.knn in matix method

Hi Jeff,
I noticed that an error occurred in impute() when using "impute.knn" with matix method, see below:

rdata <- hela_proteins
library(impute)
rdata %>% impute(.function = impute.knn, method = 'matrix')
Error in .function(as.matrix(this_matrix), cores = cores) :  (data, k = 10, rowmax = 0.5, colmax = 0.8, maxp = 1500, rng.seed = 362436069) { r…
unused argument (cores = cores)
✖ Imputing by matrix using the function base::quote function (data, k = 10, rowmax = 0.5, colmax = 0.8, maxp = 1500, rng.seed = 362436069) { r…

Could you please resolve it? Thank you.

Thank you for pointing this out, my documentation was insufficient here.
The R package in bioconductor::impute, allows for the popular imputation method KNN. The generalized impute function for the method matrix assumes the underlying function is multithreaded, as is the impute.randomforest method. Therefore, to make any function operable, you need to make a wrapper function to allow for the cores variable to be accepted. In addition, the impute package returns an incompatible data object, that you must convert to a matrix - fortunately, the impute package’s data object contains the matrix in $data. See the article on imputing with a matrix.


Importing tensorflow fails on travis-xenial because of glibc version

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):travis-ubuntu-16.04
TensorFlow installed from (source or binary):pip (to get whatever is the latest version there)
GPU model and memory:travis default build target

You can collect some of this information using our environment capture script
Describe the problem
Since the last few days, our utility which imports tensorflow has been failing automated builds on travis-ci. We are using a ubuntu-xenial target (16.04). This issue was NOT present a few days ago, but suddenly appeared causing travis builds to fail. I am assuming it is an issue with dependencies or version checks.
Source code / logs
Here is the travis build history:
https://travis-ci.org/autonomio/core-module
Here is the .travis.yml:
https://github.com/autonomio/core-module/blob/master/.travis.yml
Here is the error we are getting:
Using TensorFlow backend. Traceback (most recent call last):   File "./test_script.py", line 1, in <module>     from autonomio.commands import data, train, test   File "/home/travis/build/m-anish/autonomio/autonomio/__init__.py", line 3, in <module>     from train_new import kuubio   File "/home/travis/build/m-anish/autonomio/autonomio/train_new.py", line 7, in <module>     from transform_data import transform_data   File "/home/travis/build/m-anish/autonomio/autonomio/transform_data.py", line 6, in <module>     from y_transform import y_transform   File "/home/travis/build/m-anish/autonomio/autonomio/y_transform.py", line 2, in <module>     from keras.utils import np_utils   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/keras/__init__.py", line 3, in <module>     from . import activations   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/keras/activations.py", line 4, in <module>     from . import backend as K   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/keras/backend/__init__.py", line 73, in <module>     from .tensorflow_backend import *   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 1, in <module>     import tensorflow as tf   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/__init__.py", line 24, in <module>     from tensorflow.python import *   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/python/__init__.py", line 51, in <module>     from tensorflow.python import pywrap_tensorflow   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 52, in <module>     raise ImportError(msg) ImportError: Traceback (most recent call last):   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 41, in <module>     from tensorflow.python.pywrap_tensorflow_internal import *   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>     _pywrap_tensorflow_internal = swig_import_helper()   File "/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper     _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description) ImportError: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.17' not found (required by /home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so)
`
Here is the full log:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/231031263/log.txt?X-Amz-Expires=30&X-Amz-Date=20170511T053131Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170511/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host&X-Amz-Signature=de398e3a80f8114605f1a334832b1dc024321d3c46dc8453384b53620c38ec37

I'm closing this since apparently, the correct version of ubuntu was not being selected by travis.
Sorry for the noise.


[SPARK-45092][SQL][UI] Avoid analyzing twice for failed queries

What changes were proposed in this pull request?

As a discussion starting from https://github.com/apache/spark/pull/42481#discussion_r1316776270, for failed queries, we need to avoid calling SparkPlanInfo fromSparkPlan, which triggers another round of analyzing.
This patch uses Either[Throwable, () => T] to pass the throwable conditionally and bypass plan explain functions on error.
Why are the changes needed?

improvements of https://github.com/apache/spark/pull/42481
Does this PR introduce any user-facing change?

no
How was this patch tested?

existing tests
Was this patch authored or co-authored using generative AI tooling?

no

cc @cloud-fan

thanks, merged to master


Support code generation for body-string.json

If operation is a swift keyword (e.g. String, Enum), add 'Operation' as suffix
if name is a swift keyword, add 'Type' as suffix
refactor package dependency in XCworkspace to not use xcodeproj. Use swift Package.swift instead.
remove all the xcodeproj files in the repo

Wiki is updated.


fixed href tag creation for URLs in cell text

fixes #2519

added href tag creation for URLs in cell text.
added tests.

@thadguidry hmm, since when do you merge PRs without a code review?

@wetneb Hi Antonin, I tested his changes, reviewed them, requested changes, he made the changes, I verified, and merged.  Sorry, which parts am I allowed to do or not? Oh, did our process change where we need to have 2 reviewers now?  Please explain and happy to adjust.

I think we have a long-standing agreement that you do not merge PRs with code changes without the approval of a developer. If you want to stick to the roles defined in the gouvernance document, then I think you should not be a committer anymore.

Hi @wetneb @thadguidry , can this PR be reverted as it changed around 3k lines in code, or should I open a followup pr to fix them? It has undone many formatting changes in the cypress-test-suite, which will create more conflicts with the ESlint.

@kushthedude yes, done.
@singhakshita I am sorry that we had to undo this merge - your PR was merged erroneously by @thadguidry without review by a developer. If you could open a new pull request with your changes it would be great. If you want help with that let me know.

Hi @wetneb @thadguidry , can this PR be reverted as it changed around 3k lines in code, or should I open a followup pr to fix them? It has undone many formatting changes in the cypress-test-suite, which will create more conflicts with the ESlint.

@kushthedude i have added changes to two files ,one is cell.ui.js and other is the test file proper-displayjs ,if i run prettier
it apply changes to all the tests files and increases the number of changes , and without runing prettier if i send the PR its fails my ui.integration test ,can you please suggest me  what should be done.

I agree with @singhakshita as I saw the same thing local with prettier and thought it was conforming against those 3k lines
I think we need to understand why that is and document the right steps for verifying the correct formatting locally.
@wetneb sure I will remove my watching on repo.

This changes were due to seperation of concerns in eslint and prettier PR,
as can be seen prettier flow is already merged but eslint is still
remaining to go into master.
On Sun, 7 Feb, 2021, 18:30 Thad Guidry<EMAIL_ADDRESS>wrote:

I agree with @singhakshita https://github.com/singhakshita as I saw the
same thing local with prettier and thought it was conforming against those
3k lines
I think we need to understand why that is and document the right steps for
verifying the correct formatting locally.
@wetneb https://github.com/wetneb sure I will remove my watching on
repo.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/3588#issuecomment-774670486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLXAVSXBXPVPV2XMFRDS52FGHANCNFSM4XFDUVRA
.


关于加入SPP提取多尺寸特征但loss不下降的问题咨询

@brotherNanNan Sorry for the slow response. I don't have experience with SPP so won't be able to provide any insights on this. Please consider sharing your findings with us here if you figured out a solution. Thanks!

I have given up this attemp, the loss just doesn't decline..


UDPPacketPeer .get_var() returning decode_variant error

Operating system or device:
Windows 64 bit
Godot 2.0.2 stable official
Issue description (what happened, and what was expected):
When trying to receive string data through UDP using udp.get_var() the console shows the following error:
ERROR: decode_variant: Condition ' type>=Variant::VARIANT_MAX ' is true. returned: ERR_INVALID_DATA
At: core/io/marshalls.cpp:47
I noticed this problem on both the udp_chat demo and my own personal implementation.
Steps to reproduce:
run udp_chat demo and receive text packet from a remote host.
Link to minimal example project (optional but very welcome):
Not necessary, use udp_chat demo.

This commit I think is what's triggering the error. https://github.com/godotengine/godot/commit/35959f9c5a314effc9a442931baccec2408b8ece
Not sure though.
One thing I should mention is the data I'm receiving is 3 numbers separated by spaces. Could the first character being an integer followed by a space and then another float be what's confusing the decoder?

Just tested with characters at begining of packet, still receiving error.  No change.

Looks like the problem was with my sender...I never tested the UDP chat from one to the other.  Sender is labview which does not support utf8. Will close after more testing if necessary.

Yup, it was the sender not sending utf8.  Not a bug, closing.


Rebasing release-0.8.0-RC2 branch to master

This pull request rebases release-0.8.0-RC2 branch to master.

Closing this pull request, will apply release branch changes by cherry picking the commits.


Repository field is missing from package.json

There's no repository field in this repo's package.json so no link appears on https://www.npmjs.com/package/@textile/powergate-client - adding one would help discoverability.

Good call. Done.


Why force the app to listen on <IP_ADDRESS> in docker mode?

Pretty much what the title says. I'd like it to only listen on a specific ip/host (e.g. <IP_ADDRESS>); why does the app bind to <IP_ADDRESS> when it's launched in docker?

Mostly laziness, since I don't use Docker and no one had complained about it yet. I'll look into making it configurable.

The restriction should be gone in 2.8.0-beta.2. It's mostly untested though, so let me know if it's not working as expected.


Add login protection

#22

Hmm, thanks for the observation @schmijos. I followed the approach from redmine tracky. I will remove the custom logic and replace it the protected controller with the application controller if it works.

After further investigation, it turns out that the ProtectedController is not used anywhere and the sync controller is already extending the Redmine ApplicationController.


Has anyone been able to match Dreamscope results?

I think the result of dreamscope is really beautiful.Anyone can get closed to it?

If anyone could answer this that would be awesome. Their transitions are amazing. It looks like they're weighting between more than one style.

@dovanchan @WMaur This issue already exists, here: https://github.com/jcjohnson/neural-style/issues/257


WSFCFileServerWaitCondition failure

I am attempting to create an Always On Sql cluster in an existing VPC. However, I receive the following error when the template runs WSFCFileServerWaitCondition:
WaitCondition received failed message: 'Command failure in Disable-WSManCredSSP C:\cfn\scripts\Disable-CredSSP.ps1 on line 5 Exception: System.InvalidOperationException:
This command cannot be executed because the setting cannot be disabled.
 at System.Management.Automation.MshCommandRun...' for uniqueId: i-0dbd7e32db3e83ba2

Has anyone found a good workaround for this issue or might know how I can get things working?
Thanks!

Hi Doug,
This might be a workaround: winrm set winrm/config/client/auth '@{CredSSP="true"}'
Let us know if that helped.

Right now I'm using all the samples straight from the AWS quickstart site (https://s3.amazonaws.com/quickstart-reference/microsoft/sql/latest/templates/sql.template). I guess I'll need to download and modify them before trying again.
Do you know where in the sample I should run the winrm set... command?

You can run the Quick Start in your personal bucket. You could create a try/catch block in the Disable-WSManCredSSP script and try running that command if the initial attempt fails. I will try to implement it myself as well when I have a few cycles.

Hi Doug,
Before trying that command, please check the permissions of the Admin user running the command and also if a GPO policy is preventing you from running the command. Thanks.

Thanks for the suggestions. It turns out we do have a group policy in place that was conflicting with the AWS sample. I downloaded the sample and modified it a bit to match our environment. Things seem to be working better now.


feat: Add warning for large payloads

Add warning for large payloads, which is typically caused by excessive custom attribute sizes.

Overview
This PR adds a check to the harvest cycle to check the body size.  If the body is large (>750kb), it will log a warning to the console noting that customers should check their custom attributes.

Related Issue(s)
NR-137463

Testing
A new jest test representing this case has been added to harvest.test.js

Should follow up with plan about surfacing "warnings" to NR1


A quick note

Last night close to 500 folks showed up to an event put on by the Greenville news to help bridge the gap between the Greenville of Top Ten Lists (the cool downtown Greenville) and the rest of Greenville. A theme that was so loud and clear in the meeting that it made it to the write-up in the Greenville News http://www.greenvilleonline.com/story/news/local/2015/06/09/crowded-unseen-greenville-forum-begins-larger-community-conversation/28777223/ was transportation.
"In particular, the theme of transportation arose throughout the night."
This morning, I was at Upstate Transit Coalition meeting and the need for a better and more advanced technology solution for transit came up again. They want to help the Trolley Tracker succeed.
This project is something that's cool and going to be a nice to have for the trolley, but it could also be something that helps solve a bigger issue in our community. I know we all know this already, but I figured it was worth sharing again as we Voltron this bad boy up next week.

Awesome! Thank you for that Adam! We have had a lot of people RSVP to the
Hack Night including Ryan Poolos of the CocoaHeads group. Looking to be a
great night to get it over the hump!
On Wed, Jun 10, 2015 at 10:19 AM, Adam Gautsch<EMAIL_ADDRESS>wrote:

Last night close to 500 folks showed up to an event put on by the
Greenville news to help bridge the gap between the Greenville of Top Ten
Lists (the cool downtown Greenville) and the rest of Greenville. A theme
that was so loud and clear in the meeting that it made it to the write-up
in the Greenville News
http://www.greenvilleonline.com/story/news/local/2015/06/09/crowded-unseen-greenville-forum-begins-larger-community-conversation/28777223/
was transportation.
"In particular, the theme of transportation arose throughout the night."
This morning, I was at Upstate Transit Coalition meeting and the need for
a better and more advanced technology solution for transit came up again.
They want to help the Trolley Tracker succeed.
This project is something that's cool and going to be a nice to have for
the trolley, but it could also be something that helps solve a bigger issue
in our community. I know we all know this already, but I figured it was
worth sharing again as we Voltron this bad boy up next week.
—
Reply to this email directly or view it on GitHub
https://github.com/codeforgreenville/trolley-tracker/issues/23.

Publicly accessible PivotalTracker Project for the different components https://www.pivotaltracker.com/n/projects/1367484