Memory Error (Runtime Error)

Runtime error:
Traceback (most recent call last):
File "/home/ubuntu/.local/lib/python2.7/site-packages/luigi/worker.py", line 203, in run
new_deps = self._run_get_new_deps()
File "/home/ubuntu/.local/lib/python2.7/site-packages/luigi/worker.py", line 140, in _run_get_new_deps
task_gen = self.task.run()
File "/home/ubuntu/.local/lib/python2.7/site-packages/luigi/contrib/external_program.py", line 107, in run
stderr = self._clean_output_file(tmp_stderr)
File "/home/ubuntu/.local/lib/python2.7/site-packages/luigi/contrib/external_program.py", line 86, in _clean_output_file
return ''.join(map(lambda s: s.decode('utf-8'), file_object.readlines()))
MemoryError
Hi! I don't know if this is a luigi issue, but I checked my code without luigi and it works, In my task I am using scrapy to save a lot of info and save it in S3

I would say this is issue with program which you are executing through ExternalProgramTask.
Line which results in MemoryError takes all program output from stdout or stderr and creates one big string from it. If you print a lot of data in stdout or have a lot of error messages in stderr it could be bigger then available memory.

I found out that  this is not the real problem, I am gonna open other issue, thanks @veyu

thank @veyu So do you think that it is not possible with luigi? or Can I change that configuration? Or maybe a bigger server?

It's is difficult to answer your questions without a better understanding of the problem.

You need to run an external program
Said program generates lots of stdout and/or stderr

Do you require the stdout or stderr of the program? If not, try calling it directly through python's subprocess module. You could write the exit code to a file and check that in a custom complete method.


Proof of concept object nodepath caching

Just in case you haven't already implemented something which fulfills a similar purpose. So far I've only tested basic rename cases propagating to properties, and in the Wildeburn project this makes moves and renames extremely fast. It does have a few weaknesses such as not quite duplicated code between recursively building the cache and recursively doing the rename, increasing memory overhead if a project very heavily uses NodePaths, and pretty simplistic checking of paths against the cache.
I think ideally rather than this incremental solution there would be a direct link tracked by each node for each property which is directly referencing it via NodePath while in the editor, but I didn't want to mess with the existing logic too much, just filter out recursive calls which were not going to do anything.

I think the only thing that really needs to be updated to make this fast is the SceneTreeDock::_fill_path_renames() call via SceneTreeDock::_node_prerenamed(). I think the current behavior is actually a bug, as it creates a list of path_renames which is just every single node. This logic makes no sense at all. There's no need to fill this structure at all if you're then going to just go through each node in the tree.

@a-johnston yeah, I'll have a look to see if I can get that information from signals, and then create a separate cache of nodepath/node mappings from that. Maybe all of that can indeed just be removed from scenetreedock.

Would you still like to see this merged?


Yocto build is broken

Following the instructions provided here: https://wiki.seeedstudio.com/reTerminal-Yocto/, the build fails here:
| make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
|   CC [M]  /home/tim/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.o
| /home/tim/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.c: In function 'ili9881d_dsi_probe':
| /home/tim/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.c:501:12: error: 'struct drm_panel' has no member named 'prepare_upstream_first'
|   501 |  ctx->panel.prepare_upstream_first = true;
|       |            ^
| make[3]: *** [/home/tim/reterminal-yocto/build/tmp/work-shared/seeed-reterminal/kernel-source/scripts/Makefile.build:280: /home/tim/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.o] Error 1
Looking at the github actions, this failure is 2 months old.  Is there a plan to fix this?

I have also come across this today
Also documented here https://github.com/Seeed-Studio/seeed-linux-dtoverlays/issues/76
ERROR: seeed-linux-dtoverlays-1.0-r0 do_compile: oe_runmake failed
ERROR: seeed-linux-dtoverlays-1.0-r0 do_compile: Execution of '/home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/temp/run.do_compile.2983622' failed with exit code 1
ERROR: Logfile of failure stored in: /home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/temp/log.do_compile.2983622
Log data follows:
| DEBUG: Executing shell function do_compile
| NOTE: make -j 16 ARCH=arm64 KBUILD=/home/marcel/reterminal-yocto/build/tmp/work-shared/seeed-reterminal/kernel-source O=/home/marcel/reterminal-yocto/build/tmp/work-shared/seeed-reterminal/kernel-build-artifacts CROSS_COMPILE=aarch64-poky-linux- all_rpi
| make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
|   CC [M]  /home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.o
| /home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.c: In function 'ili9881d_dsi_probe':
| /home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.c:501:12: error: 'struct drm_panel' has no member named 'prepare_upstream_first'
|   501 |  ctx->panel.prepare_upstream_first = true;
|       |            ^
| make[3]: *** [/home/marcel/reterminal-yocto/build/tmp/work-shared/seeed-reterminal/kernel-source/scripts/Makefile.build:280: /home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d/ili9881d.o] Error 1
| make[2]: *** [/home/marcel/reterminal-yocto/build/tmp/work-shared/seeed-reterminal/kernel-source/Makefile:1825: /home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/git/modules/ili9881d] Error 2
| make[1]: *** [Makefile:185: __sub-make] Error 2
| make: *** [Makefile:157: all_rpi] Error 2
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
| ERROR: Execution of '/home/marcel/reterminal-yocto/build/tmp/work/seeed_reterminal-poky-linux/seeed-linux-dtoverlays/1.0-r0/temp/run.do_compile.2983622' failed with exit code 1
ERROR: Task (/home/marcel/reterminal-yocto/layers/meta-seeed-cm4/recipes-kernel/seeed-linux-dtoverlays/seeed-linux-dtoverlays.bb:do_compile) failed with exit code '1'

Fix proposed here: https://github.com/Seeed-Studio/seeed-linux-dtoverlays/issues/76#issuecomment-1792372585

@swrpug
As you have a reTerminal DM in hand. It's better for you to use the yocto kirkstone to build the image.
The compile step on seeed's wiki need to update now.
Here is the detail steps for you to compile the kirkstone image:
mkdir -p kirkstone/layers && cd kirkstone/layers

git clone -b kirkstone https://github.com/openembedded/meta-openembedded.git
git clone -b kirkstone https://github.com/meta-qt5/meta-qt5.git
git clone -b kirkstone git://git.yoctoproject.org/meta-raspberrypi
git clone -b main https://github.com/Seeed-Studio/meta-seeed-cm4.git
git clone -b kirkstone git://git.yoctoproject.org/poky

cd ..
source layers/poky/oe-init-build-env

bitbake-layers add-layer ../layers/meta-raspberrypi
bitbake-layers add-layer ../layers/meta-seeed-cm4
bitbake-layers add-layer ../layers/meta-qt5
bitbake-layers add-layer ../layers/meta-openembedded/meta-oe
bitbake-layers add-layer ../layers/meta-openembedded/meta-python

MACHINE="seeed-reterminal-plus" bitbake rpi-test-image

If you don't want to compil it by yourself. We just release a image of it. You can download and test it in here
We recommand you to use the balenaEtcher as the flash tool for yocto image.

@hecko
First of all, thank you for your work on fix the compile error of this repo.
As the dunfell will EOL at Apr 2024.We recommand you to use the kirkstone instead.
However, the LCD black issue is still there with kirkstone.I am trying to fix it.
Until now,the reason of lcd blank is that the drm driver of kernel mounted later than lcd driver.
You can remount the lcd driver after system is up with the following commands:
modprobe -r mipi_dsi
modprobe mipi_dsi

@hecko
The LCD blank issue is fixed now. You can just download the firmware in here to test it. I test it for more than 10 times and all of them work fine.
  One notable issue encountered is the LCD display brightness slider.  Moving the slider does not change the screen brightness, except that moving it all the way to the left, turns the screen black.  Reloading the LCD drivers via ssh has no effect.  Only solution so far is to reset the unit.
I did not have success with the image I built my self, still investigating.

@swrpug
About the brightness slider.Have you tried to control file system node of lcd brightness with following command?
echo 127 > /sys/class/backlight/1-0045/brightness

@bigbearishappy I just tried your script on comment above (https://github.com/Seeed-Studio/meta-seeed-cm4/issues/12#issuecomment-1811858602) and it's throwing me an error.
| make[2]: *** No rule to make target 'arch/arm64/boot/dts/overlays/reTerminal.dtbo'.  Stop.
| make[1]: *** [/home/rd/yocto-reterminal/kirkstone/build/tmp/work-shared/seeed-reterminal/kernel-source/scripts/Makefile.build:500: arch/arm64/boot/dts/overlays] Error 2
| make: *** [/home/rd/yocto-reterminal/kirkstone/build/tmp/work-shared/seeed-reterminal/kernel-source/Makefile:1474: overlays/reTerminal.dtbo] Error 2
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
ERROR: Task (/home/rd/yocto-reterminal/kirkstone/layers/meta-raspberrypi/recipes-kernel/linux/linux-raspberrypi_6.1.bb:do_compile) failed with exit code '1'

I've also posted on the forum, with no replies so far:
https://forum.seeedstudio.com/t/reterminal-reterminal-dm-yocto-kirkstone-build-failing/275536
Can you give any hint?

@bigbearishappy I just tried your script on comment above (#12 (comment)) and it's throwing me an error.
| make[2]: *** No rule to make target 'arch/arm64/boot/dts/overlays/reTerminal.dtbo'.  Stop.
| make[1]: *** [/home/rd/yocto-reterminal/kirkstone/build/tmp/work-shared/seeed-reterminal/kernel-source/scripts/Makefile.build:500: arch/arm64/boot/dts/overlays] Error 2
| make: *** [/home/rd/yocto-reterminal/kirkstone/build/tmp/work-shared/seeed-reterminal/kernel-source/Makefile:1474: overlays/reTerminal.dtbo] Error 2
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.
ERROR: Task (/home/rd/yocto-reterminal/kirkstone/layers/meta-raspberrypi/recipes-kernel/linux/linux-raspberrypi_6.1.bb:do_compile) failed with exit code '1'

I've also posted on the forum, with no replies so far: https://forum.seeedstudio.com/t/reterminal-reterminal-dm-yocto-kirkstone-build-failing/275536
Can you give any hint?

OK, I will have a try and feed back to you.

@bigbearishappy any news? I have been trying for more than a week to get this working... I already forgot all the different things I've tried so far.

@bigbearishappy any news? I have been trying for more than a week to get this working... I already forgot all the different things I've tried so far.

Sorry for my late reply. I have just fix this compile error in here. You can pull the latest code and try it again.
The reason of this error is that when I work with kirkstone before, there is no bb for kernel 6.1, so I add it by myself. But later the meta-raspberrypi add the same bb file for kernel 6.1, and my bb file is just ignored.

Closing the issue. You can reopen the issue if you have any further and relative problem~

@bigbearishappy build has just finished! I was finally able to compile it successfully.
Thank you so much for the fix!


Fix unit tests for after latest changes

@bmorrise Merge it to master also, please

@pamval, @pedrofvteixeira, run WM over this PR please

running wm

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Djunit.jvmargs=-XX:PermSize=256m -f engine/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
engine/test-src/org/pentaho/di/trans/steps/fieldsplitter/FieldSplitterTest.java
engine/test-src/org/pentaho/di/trans/steps/metainject/MetaInjectTest.java
Checkstyle violations:
See: https://github.com/pentaho/pentaho-coding-standards
File engine/test-src/org/pentaho/di/trans/steps/fieldsplitter/FieldSplitterTest.java

Message: Using a static member import should be avoided - org.mockito.Matchers.any. at Line: 25 Column: None
Message: Using a static member import should be avoided - org.mockito.Matchers.anyString. at Line: 26 Column: None
Message: Using a static member import should be avoided - org.mockito.Mockito.*. at Line: 27 Column: None

File engine/test-src/org/pentaho/di/trans/steps/metainject/MetaInjectTest.java

Message: Using a static member import should be avoided - org.junit.Assert.assertEquals. at Line: 25 Column: None
Message: Unused import - org.junit.Assert.assertFalse. at Line: 26 Column: 15
Message: Using a static member import should be avoided - org.junit.Assert.assertFalse. at Line: 26 Column: None
Message: Using a static member import should be avoided - org.junit.Assert.assertTrue. at Line: 27 Column: None
Message: Using a static member import should be avoided - org.mockito.Matchers.any. at Line: 28 Column: None
Message: Using a static member import should be avoided - org.mockito.Matchers.refEq. at Line: 29 Column: None
Message: Using a static member import should be avoided - org.mockito.Mockito.atLeastOnce. at Line: 30 Column: None
Message: Using a static member import should be avoided - org.mockito.Mockito.doReturn. at Line: 31 Column: None
Message: Using a static member import should be avoided - org.mockito.Mockito.mock. at Line: 32 Column: None
Message: Using a static member import should be avoided - org.mockito.Mockito.spy. at Line: 33 Column: None
Message: Using a static member import should be avoided - org.mockito.Mockito.verify. at Line: 34 Column: None

Unit test coverage change:
org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta:

instructionPercentCoverageChange : +0.17%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta:

instructionPercentCoverageChange :-0.28%
branchPercentCoverageChange :-3.33%
complexityPercentCoverageChange :-1.85%

org.pentaho.di.trans.steps.randomvalue.RandomValueMeta:

instructionPercentCoverageChange : +0.15%
branchPercentCoverageChange : +2.08%
complexityPercentCoverageChange : +2.13%

org.pentaho.di.trans.steps.fieldsplitter.FieldSplitter:

instructionPercentCoverageChange : +7.34%
branchPercentCoverageChange : +15.22%
linePercentCoverageChange : +8.82%
complexityPercentCoverageChange : +10.71%

org.pentaho.di.trans.step.StepMeta:

instructionPercentCoverageChange : +0.19%
linePercentCoverageChange : +0.27%

org.pentaho.di.job.entries.sql.JobEntrySQL:

instructionPercentCoverageChange : +1.64%
linePercentCoverageChange : +1.97%


Why ffcv is faster than native pytorch data loading?

Good work !
I am just curious why ffcv is faster than native pytorch data loading? Once users understand the mechanism under the hood, they are more likely to adopt it : )

Hello!
Thank you very much. There are many reasons but on the top of my head:

Single file vs many smaller files (less stress on the file system)
Using the knowledge about which samples will be needed in the future to start reading the data ahead of time and absorb the latency of slow storage
The use of threads instead of processes remove the need of any inter-process communication (shared memory etc...) and reduces memory consumption
Compilation of the CPU augmentations transforms in machine code at the start of training (JIT).
Copying the next batches while the compute is happening instead of simply asynchronously (but still on the critical path)
Ability to use to move augmentations to CPU or GPU depending on where is your bottleneck.
The use of pre-resized datasets
In the case of systems with a lot of RAM, leveraging operating system level caching to implicitly store the entire dataset in RAM
Being able to use alternative or a mix of different encoding for the data. JPEG can be slow to decode, if you have enough storage/RAM but not enough compute using uncompressed images might be faster, and FFCV enables that simply by switching a flag.


Best Practices for Schema Design???

The page "Best Practices for Schema Design" feels mislabeled.  I read it expecting to get tips on designing Data Explorer Schemas.  Instead I got a bunch of tips on manipulating and querying schemas, not designing them.
(If you have any tips on designing Data Explorer Schemas, I'd love to read them.  In the meantime, please consider retitling the page.)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 04e54dce-472a-ac8a-e8e0-d31c25602f71
Version Independent ID: 91cb215e-e203-5ff1-dae7-08eda1250375
Content: Best practices for schema design - Azure Data Explorer
Content Source: data-explorer/kusto/management/management-best-practices.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@DavidBerg-MSFT Thanks for the suggestion. Assigning to the author to have a look.

@DavidBerg-MSFT thanks for your feedback. We will change the page title as requested.
I created a WI re: your request for "tips on designing Data Explorer Schemas"
I will now close this issue.

Thanks.  Glad you found the suggestion helpful.
From: orspod<EMAIL_ADDRESS>Sent: Tuesday, December 1, 2020 9:36 AM
To: MicrosoftDocs/dataexplorer-docs<EMAIL_ADDRESS>Cc: David Berg<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dataexplorer-docs] Best Practices for Schema Design??? (#470)
@DavidBerg-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDavidBerg-MSFT&data=04|01|David.Berg%40Microsoft.com|982c5aabf5734f10e84508d8961f9ded|72f988bf86f141af91ab2d7cd011db47|1|1|637424409819498434|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=3dxnl8URmKCohCSp86n59xUk7O%2BVDeDsFtL2nM5F7sw%3D&reserved=0 thanks for your feedback. We will change the page title as requested.
I created a WIhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmseng.visualstudio.com%2FTechnicalContent%2F_queries%2Fquery%2F977d69dc-a57d-4219-8f3c-fba78447ccf4%2F&data=04|01|David.Berg%40Microsoft.com|982c5aabf5734f10e84508d8961f9ded|72f988bf86f141af91ab2d7cd011db47|1|1|637424409819498434|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=2xtLuylrDx8p02li0OcrEjuvCkqwHAB5LLfSEz2IE%2FQ%3D&reserved=0 re: your request for "tips on designing Data Explorer Schemas"
I will now close this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdataexplorer-docs%2Fissues%2F470%23issuecomment-736706905&data=04|01|David.Berg%40Microsoft.com|982c5aabf5734f10e84508d8961f9ded|72f988bf86f141af91ab2d7cd011db47|1|1|637424409819508422|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=oVpUQTOIQQF%2Fj7jVFSg1E2OcVhLWo65f3r8vAYacNPE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACGECRUK3ZPJ4PALEF7NPFDSSUSRJANCNFSM4UFSWTZA&data=04|01|David.Berg%40Microsoft.com|982c5aabf5734f10e84508d8961f9ded|72f988bf86f141af91ab2d7cd011db47|1|1|637424409819508422|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7b%2Bbp59%2FcNqFyHU7dYx9iRCpMnko9%2BIe8IurH6JqXec%3D&reserved=0.


Add consts

Current coverage is 100% (diff: 100%)

No coverage report found for master at 4c7a625.

Powered by Codecov. Last update 4c7a625...484bf72


App Shell crops when scrolling horizontally

this might not be a big issue if you have a smaller screen than a laptop size


A step-by-step tutorial with deployment to Azure App service

I created this video as step-by-step with animated workflow, it can help many people
This video will show step-by-step ASP.NET Core development with Deployment to Azure
https://www.youtube.com/watch?v=1QVRe7UHaro

What tutorial should this be surfaced from?
Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select This page .


Provide better logging utility in Functions SDK/libraries

A problem that function developers run into: Emitting a debug log message to stdout. Because stdout is used for machine-to-machine protocol [1], it leads to cryptic failures from kpt fn. In Typescript, we actually override console.log to write to stderr, but we need a more uniform solution across different languages.
Consider:

Introduce logger utilities in the libraries as the recommended approach to debug logging
Improve the error message when the developer accidentally writes to stdout when do they don't use the logging utility provided.

[1] https://github.com/kubernetes-sigs/kustomize/blob/master/cmd/config/docs/api-conventions/functions-spec.md

@mengqiy feature to keep in mind for when you get to the SDK part of the function work.


Audio Samples Not Working

I am looking to work with audio and many of the iOS samples are working with xcode 6.
avTouch - XIB has an error won't compile
converter demo - never finishes rendering (I didn't bother debugging)
APTapQueue - I can't get any sound to play (although it looks like it's processing data)
maybe it's just me, but these are kind of old and might need updating.

I checked these samples with current stable and they work fine.
avTouch – compiles and run
AQTapDemo – was fixed a month ago
AudioConverterFileConverter – was fixed a few months ago
Please check it again. If you have any troubles just reopen this issue


Friendrequest doc

add jsdoc for friend request

The FriendRequest design has some problems which is not resolved yet.
See: #70
However, this is a good workaround for the document, so let's merge it first.
Thanks!


Merged code to eliminate redundancy for AsyncWiredSwitch8 and AsyncOp…

Breaking change

Proposed change
Combined the two elif conditions for AsyncWiredSwitch8 and AsyncOpenCollector8Module into a single condition for cleaner code and remove duplicate code.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Deprecation (breaking change to happen in the future)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist
[ ] I have followed the perfect PR recommendations
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
[ ] Untested files have been added to .coveragerc.

To help with the load of incoming pull requests:

[ ] I have reviewed two other open pull requests in this repository.

Waled Mohammad Group 10
Rule 2: The reason for why this issue/code smell is relevant is because code redundancy can reduce code readability and increase the cognitive load.
Rule 3: I have merged the two 'elif' conditions for AsyncWiredSwitch8andAsyncOpenCollector8Module` into a single condition and this solution solve the problem because the code now handles both cases with a single implementation for cleaner and more efficient code.

looks good


Rewrite Author section of README

The link to contributors needs to be presented better.

I'll take that one from hacktoberfest.

Updated the author section: PR #129

COmpleted with PR #129


De-duplicate the feedback form footer

https://github.com/SuffolkLITLab/docassemble-AssemblyLine/blob/c9607e1b5cfedda7e5877ef33bdabbffe64ad5bb/docassemble/AssemblyLine/data/questions/al_visual.yml#L49-L53
Both if get_config('debug'): and its else: seem to use the same code. If that's correct, we can get rid of the if.

They do. In MAVC, those went to separate places, which was a limit of Google Forms. I think there's no reason to use two separate forms anymore.

I think this is a useful place to suggest that we move the feedback form into this repository, and update the links accordingly. I'm updating the issue title to match the broader scope.

We actually decided to add a footer to every page with a "feedback" link. #137


Find and replace doesn't open files anymore

The new behaviour is very confusing. Is there a way to change it through configuration or is it a bug?

VSCode Version: Version 1.23.0 (1.23.0)
OS Version: MacOS 10.12.6 Sierra

Steps to Reproduce:

Use find-and-replace from sidebar
Click the replace all button

Expected: Opens all editors with unsaved changes to each file
Actual: Opens nothing and just changes files on disk

Does this issue occur when all extensions are disabled?: Yes
[edit: additional info]
I see it's an improvement on the release from 1.22 save-automatically
I'd like to have an option where this behaviour can be modified into the 'old' way.

Related #46983

@Alber70g

I'd like to have an option where this behaviour can be modified into the 'old' way.

Do you mean you do not want to auto save or you want the files to be opened after replace operation is done?
If you do not want auto save, may I know why you don't want that?


How long time do you guys nomally run with your own data?

I'm running it with the command "./findstems 16 0.2 2 cloud.pcd cloud.pcd"
size of cloud.pcd is 369.1 kB, about 10000 cloud point.
the process is as followed:
Reading slice: complete
Cluster extraction: cloud.intermediate.slice.clusters.pcd | 1
Region-based segmentation: cloud.intermediate.slice.cluster.regions.pcd | 5
RANSAC cylinder fits:
and it keeps running for about 5 hours and is still running now.
am I doing something wrong? or it's slow because I'm running it using CPU r75800X.
I'll be really grateful if someone could answer me.
Jayla

Certain steps can take a considerable amount of time.
However the RANSAC shouldn't take extremely long from memory, It seems you have only found 5 regions in your slice.
Have you got less than 5 trees in your data?
This number should normally be greater than the number of trees your have in your data.
What may be happening is that: In your your slice, there has grown some very large regions and these very large regions may take a very long time to finish.
You should inspect cloud.intermediate.slice.cluster.regions.pcd to make sure each color (region) contains only one section of a stem. If stems are connected by ground/branches/vines you will need to adjust the height that the slice is taken at or change the smoothness value (first argument).
it should look similar to this

Certain steps can take a considerable amount of time.
However the RANSAC shouldn't take extremely long from memory, It seems you have only found 5 regions in your slice.
Have you got less than 5 trees in your data?
This number should normally be greater than the number of trees your have in your data.
What may be happening is that: In your your slice, there has grown some very large regions and these very large regions may take a very long time to finish.
You should inspect cloud.intermediate.slice.cluster.regions.pcd to make sure each color (region) contains only one section of a stem. If stems are connected by ground/branches/vines you will need to adjust the height that the slice is taken at or change the smoothness value (first argument).
it should look similar to this

Thanks for your answer!
My original .pcd data only contains around 10 trees, which are cut from a large park scene. I think the reason I run it slowly is that cloud.intermediate.slice.cluster.regions.pcd file shows relatively discrete points, not as same as yours are clear stems, which means my original data is not good enough to do segmentation.
I'm wondering if it's convenient for you to share your .pcd file with me?  Really appreciate your help.
Regards

Thanks for your answer!
My original .pcd data only contains around 10 trees, which are cut from a large park scene. I think the reason I run it slowly is that cloud.intermediate.slice.cluster.regions.pcd file shows relatively discrete points, not as same as yours are clear stems, which means my original data is not good enough to do segmentation.
I'm wondering if it's convenient for you to share your .pcd file with me?  Really appreciate your help.
Regards

More discrete points are normal,  the image above was just to illustrate the separated-ness of the stems.
You could try adding some extra prints (std::cout << "done:"<< i <<std::endl;) to the stem finding loop, This wont make it any faster but adds some mental reassurance that things are moving.
You could also lower the number of max iterations:
treeseg.cpp:429		seg.setMaxIterations(1000000);
to 10,000.
This number controls how many attempts the RANSAC algorithim attempts before deciding it has found the best possible cylinder fit.
This will slightly reduce the chance that a good match will be found for each stem,(ie some trees may be missed) but will greatly reduce the maximum possible time spent per tree.
note. In this context, cylinder fitting refers to "finding stems" not volumetric models such as the ones produced by TreeQSM
Regards Tony

More discrete points are normal,  the image above was just to illustrate the separated-ness of the stems.
You could try adding some extra prints (std::cout << "done:"<< i <<std::endl;) to the stem finding loop, This wont make it any faster but adds some mental reassurance that things are moving.
You could also lower the number of max iterations:
treeseg.cpp:429		seg.setMaxIterations(1000000);
to 10,000.
This number controls how many attempts the RANSAC algorithim attempts before deciding it has found the best possible cylinder fit.
This will slightly reduce the chance that a good match will be found for each stem,(ie some trees may be missed) but will greatly reduce the maximum possible time spent per tree.
note. In this context, cylinder fitting refers to "finding stems" not volumetric models such as the ones produced by TreeQSM
Regards Tony

Hi Tony,
Thanks for your advice! I've added print code and set MaxIterations lower to 10000, but it still stuck on the same place for hours:
zsq@qingshan-zsq:~/Downloads/treeseg-0.2.0/build2$ ./findstems 15 0.2 2 kloud.slice.pcd kloud.slice.pcd
Reading slice: complete
Cluster extraction: kloud.intermediate.slice.clusters.pcd | 2
Region-based segmentation: kloud.intermediate.slice.clusters.regions.pcd | 5
RANSAC cylinder fits:
the prior step is:
zsq@qingshan-zsq:~/Downloads/treeseg-0.2.0/build2$ ./getdemslice 2 0.5 5  kloud.tile.downsample.kloud.pcd
-27.4909 -19.0517 3.5466
-27.4909 -17.0517 0.444468
-27.4909 -15.0517 -0.294579
-27.4909 -13.0517 1.78738
-27.4909 -11.0517 3.40282e+38
-27.4909 -9.05169 3.40282e+38
........(omitted)
I did as you said slightly change the slice shorter by changing second and third parameters to make each color only contains one stem as much as possible, what does first parameter mean? does it affect much on segmentation?
Another problem I'm confusing, Andy used ../FGC01_coords.dat as the fourth parameter in the command:
findstems 5 0.2 2 ../FGC01_coords.dat ../FGC01.slice.pcd
but I'm using kloud.slice.pcd, which only contains three parameters in a line, the source code in findstems.cpp need four parameters a line:
        std::ifstream coordfile;
	coordfile.open(argv[4]);
	float coords[4];
	int n = 0;
	if(coordfile.is_open())
	{
		while(!coordfile.eof())
		{
			coordfile >> coords[n];
			n++;
		}
	}
	coordfile.close();
	float xmin = coords[0];
	float xmax = coords[1];
	float ymin = coords[2];
	float ymax = coords[3];

I've seen the answer you wrote in question 16, but still not sure about how to use .pcd file in this command.
Could you please give me some guides, many thanks!
Regards,
Jayla

Hi Tony,
Thanks for your advice! I've added print code and set MaxIterations lower to 10000, but it still stuck on the same place for hours:
zsq@qingshan-zsq:~/Downloads/treeseg-0.2.0/build2$ ./findstems 15 0.2 2 kloud.slice.pcd kloud.slice.pcd
Reading slice: complete
Cluster extraction: kloud.intermediate.slice.clusters.pcd | 2
Region-based segmentation: kloud.intermediate.slice.clusters.regions.pcd | 5
RANSAC cylinder fits:
the prior step is:
zsq@qingshan-zsq:~/Downloads/treeseg-0.2.0/build2$ ./getdemslice 2 0.5 5  kloud.tile.downsample.kloud.pcd
-27.4909 -19.0517 3.5466
-27.4909 -17.0517 0.444468
-27.4909 -15.0517 -0.294579
-27.4909 -13.0517 1.78738
-27.4909 -11.0517 3.40282e+38
-27.4909 -9.05169 3.40282e+38
........(omitted)
I did as you said slightly change the slice shorter by changing second and third parameters to make each color only contains one stem as much as possible, what does first parameter mean? does it affect much on segmentation?
Another problem I'm confusing, Andy used ../FGC01_coords.dat as the fourth parameter in the command:
findstems 5 0.2 2 ../FGC01_coords.dat ../FGC01.slice.pcd
but I'm using kloud.slice.pcd, which only contains three parameters in a line, the source code in findstems.cpp need four parameters a line:
        std::ifstream coordfile;
	coordfile.open(argv[4]);
	float coords[4];
	int n = 0;
	if(coordfile.is_open())
	{
		while(!coordfile.eof())
		{
			coordfile >> coords[n];
			n++;
		}
	}
	coordfile.close();
	float xmin = coords[0];
	float xmax = coords[1];
	float ymin = coords[2];
	float ymax = coords[3];

I've seen the answer you wrote in question 16, but still not sure about how to use .pcd file in this command.
Could you please give me some guides, many thanks!
Regards,
Jayla

Hi Jayla,
Apologies for the delay - I've just replied to #33 so I'm assuming this is now fixed. Feel free to reopen if not.
Cheers,
Andy


Entity / JPA Diagram is inconsistent with boolean fields

Generating Entity Code from JPA Diagramm with Boolean Fields - you get access Methods with is prefix. But if you want to create JPA Diagramm from Entity Code this Fields with is Prefix Methods are ignored in the Diagramm. Primitive "boolean" Fields are always ignored - only Boolean Wrapper Objects with get Methods are correct copied in JPA Diagramm.

Thanks for reporting the issue.


Error built with HDF5 1.8.18

HDF5 is built from source.  When configure matio, it reported can't found HDF5:
./configure --enable-mat73=yes --with-hdf5=/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18 --with-default-file-ver=7.3
...
checking for HDF5 software...  no
...
MATIO Configuration Summary
==============================================================
           C Compiler: gcc
               CFLAGS:  -g -O2
     Shared Libraries: yes
     Static Libraries: yes
  default MAT version: MAT_FT_MAT73

Features --------------------------------------------
  MAT v7.3 file support: no
Extended sparse support: yes

Packages --------------------------------------------
                 zlib: -lz
                 hdf5:
               MATLAB:

config.log:
...
configure:15844: checking for HDF5 software
configure:15881: gcc -o conftest -I/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18/include  -g -O2   conftest.c -L/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18/lib -lhdf5 -lz -lm  >&5
/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18/lib/libhdf5.a(H5PL.c.o): In function `H5PL_term_interface':
H5PL.c:(.text+0x32c): undefined reference to `dlclose'
/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18/lib/libhdf5.a(H5PL.c.o): In function `H5PL_load':
H5PL.c:(.text+0x5e8): undefined reference to `dlsym'
H5PL.c:(.text+0x7ac): undefined reference to `dlopen'
H5PL.c:(.text+0x7bc): undefined reference to `dlsym'
H5PL.c:(.text+0x934): undefined reference to `dlclose'
H5PL.c:(.text+0x99c): undefined reference to `dlerror'
H5PL.c:(.text+0xca4): undefined reference to `dlclose'
/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18/lib/libhdf5.a(H5Z.c.o): In function `H5Z_init_interface':
H5Z.c:(.text+0x958): undefined reference to `SZ_encoder_enabled'
/home/pi/matio/CMake-hdf5-1.8.18/HDF5-1.8.18-Linux/HDF_Group/HDF5/1.8.18/lib/libhdf5.a(H5Zszip.c.o): In function `H5Z_filter_szip':
H5Zszip.c:(.text+0x104): undefined reference to `SZ_BufftoBuffDecompress'
H5Zszip.c:(.text+0x19c): undefined reference to `SZ_BufftoBuffCompress'
collect2: error: ld returned 1 exit status
...

Fix is
change HDF5_LIBS  in matio_hdf5.m4
From "-L${HDF5_DIR}/lib -lhdf5"
To "-L${HDF5_DIR}/lib -lhdf5 -ldl -lszip"

Hmm, I guess it needs a slightly more clever fix, because its really not clear if HDF5 was built
with szip support or not? One solution is of course pkg-config that was meant exactly for this
kind of problem. An alternative might be to try without szip and add -ldl -lszip as a fallback if
linking fails without it?
Just my two cents...

I regularly build HDF5 from source and test it on Travis CI and never met the issue. Can you please provide yout HDF5 config (call).

@tbeu are you building against the (optional) szip library?

No, I don't since it is not needed (by matio to have the szip support for hdf5).

This is the reason. I think many people build hdf5 with szip because its one of the "official"
filters. Maybe even the official Windows dll is linked against szip?
In any case, I would not add -ldl -lszipas a default, but it might be good to have a way to
add it for users who need it. @bxwllzz did you try adding it to LD_LIBS or LD_FLAGS
when running configure?

OK, I will check it if I configure hdf5 with --with-szlib.

I build HDF5 with cmake as https://support.hdfgroup.org/HDF5/release/cmakebuild518.html which is "RECOMMENDED by HDF5". Following the Build Instructions in that page, I run build-unix.sh to built HDF5.
I just noticed it says
These files will build the static HDF5 C and C++ libraries with SZIP and ZLIB compression included.
But I stiil don't know how to use cmake to build HDF5 without SZIP?
@emmenlau Adding -ldl -lszip to LD_LIBS or LD_FLAGS when running configure didn't solve this problem.

@bxwllzz I did not test this myself but here are cmake instructions from HDF5:
https://support.hdfgroup.org/HDF5/release/chgcmkbuild.html
According to my understanding -DHDF5_ENABLE_SZIP_SUPPORT=OFF should disable
SZIP. If you want to be double sure, add also -DHDF5_ENABLE_SZIP_ENCODING=OFF.
I did not test it, but it seems promising. Does it help?

@emmenlau build-unix.sh run ctest -S HDF5config.cmake,BUILD_GENERATOR=Unix -C Release -V -O hdf5.log
In HDF5config.cmake, I found the options:
...
### disable ext libs building
#set(ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_Z_LIB_SUPPORT:BOOL=OFF")
#set(ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_SZIP_SUPPORT:BOOL=OFF")
#set(ADD_BUILD_OPTIONS "${ADD_BUILD_OPTIONS} -DHDF5_ENABLE_SZIP_ENCODING:BOOL=OFF")
...

After uncommit these lines, I can make HDF5 without SZIP. Thanks for your advice.

It is all working for me - using plain configure and not cmake.
curl https://support.hdfgroup.org/ftp/lib-external/szip/2.1.1/src/szip-2.1.1.tar.gz -O -J
tar -zxf szip-2.1.1.tar.gz
cd szip-2.1.1
./configure --quiet CFLAGS="-w"
make -C src install
cd ..
git clone --depth 1 --branch hdf5_1_8_18 https://bitbucket.hdfgroup.org/scm/hdffv/hdf5.git hdf5_1_8_18
cd hdf5_1_8_18
./configure --quiet --enable-shared --disable-production --enable-debug=all --with-pic --disable-deprecated-symbols --disable-hl --disable-strict-format-checks --disable-clear-file-buffers --disable-instrument --disable-parallel --disable-trace --with-default-api-version=v18 --with-szlib=~/szip-2.1.1/szip CFLAGS="-w"
make install -C src

You can run cmake -DHDF5_ENABLE_SZIP_SUPPORT:BOOL=OFF -DHDF5_ENABLE_SZIP_ENCODING:BOOL=OFF ./ to disable szlib in hdf5.


Tabnine was unable to install javscript semantic completion

Hello, I installed tabnine on VSCODE right now using the instructions provided.
When I type TabNine::version I obtain a popup with the message I put on the file attach.

Ok I managed to install it...

@velteyn how did you get it to work?

I`ve posted the same issue on another repo cause cannot install it

Hi.
I followed this guide https://tabnine.com/semantic
Follow the link on the table about javascript. After I installed the server the error evaporated.


Why use a localhost callback URL?

Albeit I'm a OneDrive fan I'm using the empty space in Google Drive for my test server, and after I click accept... it's attempting to use localhost. Thing is, this server is running on a Linux box, so I'm using my personal computer to visit the link... and since I have a firewall on there too I can't just change localhost to its IP...

Sorry you're having issues! To be honest, I have no idea. I simply use the Google Drive API, and it generates the credential for me. All of the web browser oauth2 stuff is entirely done in Google's API, and I have no control over it.
To make generating the initial credential easier, I have created a companion program called DriveCredentials, You can run it on any computer with Java, and provided that it has a browser you can generate a credential file from there. This credential file can then be copied to any server you wish to use Google Drive with. You can also use DriveCredentials to generate OneDrive credential files, but OneDrive doesn't even have a Java API so generating a credential is mostly based on manual input.
If you're a bit weary of using another program to generate credentials, I totally understand. Keep in mind that DriveCredentials and DriveBackup are completely open-sourced so you can ensure that nothing suspicious is being done with your information.
Again, I'm sorry about these issues, but there's really no way for me to do anything about them as it's all done upstream. Let me know if you've managed to get things working so that I can close this thread.

Hmm, I know BackupLikeMe allows you to paste in an auth key. That being said, I forgot about the alternative method so I'll go ahead and use that, thanks.


Make CosmosDB query execute the selected part of query

Problem
It is often required to work/switch between related or different variations of the same query. Currently I have to either:

spread related queries to different tabs. But this gets cluttered quickly and is difficulty to see which tabs are "of the same group".
constantly comment queries in and out in the same tab. But this is cumbersome, especially  for multi-line queries.

Also currently, it is not intuitive that should user select a part of query (i.e. to exclude some trailing where conditions) and press "execute" then it is not the selection being executed (like in almost all other similar client tools) but the entire query.
Proposed solution
Change the behavior of "Execute query" to:

if some text is selected in query editor window then execute just the selection
otherwise, execute entire query.

This seems like a rather trivial thing to implement and make the user experience more intuitive.
Alternatives
Alternative could be to implement some hierarchy to tabs, but it is significantly costlier to implement and would clutter the UI.

The Cosmos team recently deprecated our integration with Azure Storage Explorer. Here is the full deprecation notice from the 1.15.1 Storage Explorer release notes:

The Azure Cosmos DB integration with Storage Explorer has been deprecated. Any existing functionality will not be removed for a minimum of one year from this release. Users should use the Azure Portal, the Azure Portal desktop app, or the standalone Cosmos Explorer instead – all of which contain many new features that aren’t currently supported in Storage Explorer. Additionally, the Azure Cosmos DB team has also open sourced the data explorer project and will be investing in their own desktop tooling at a later date. Please keep an eye on their repository for more information.

Currently, we are not prioritizing new features or bug fixes for Cosmos in Azure Storage Explorer. If you the issue you are reporting also applies to the Azure Portal or https://cosmos.azure.com, please file an issue in our open source Cosmos Explorer repo


vscode-extensions.WakaTime.vscode-wakatime: Fix path

Motivation for this change
WakaTime vscode extension now tries to find wakatime-cli executable which we don't provide, so the extension will download it and then fail.
Log from vscode developer tools:
[Extension Host] [WakaTime][DEBUG] Initializing WakaTime v4.0.9
[Extension Host] [WakaTime][DEBUG] Using standalone wakatime-cli.
[Extension Host] [WakaTime][DEBUG] Downloading wakatime-cli standalone...
 ERR EROFS: read-only file system, open '/home/berberman/.vscode/extensions/WakaTime.vscode-wakatime/wakatime-cli.zip': Error: EROFS: read-only file system, open '/home/berberman/.vscode/extensions/WakaTime.vscode-wakatime/wakatime-cli.zip'

Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 122795 at 0f398e4e run on x86_64-linux 1

1 package built successfully:

vscode-extensions.WakaTime.vscode-wakatime

3 suggestions:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/misc/vscode-extensions/vscode-utils.nix:21:12:
   |
21 |     inherit configurePhase buildPhase dontPatchELF dontStrip;
   |            ^

warning: missing-phase-hooks
configurePhase should probably contain runHook preConfigure and runHook postConfigure.
Near pkgs/misc/vscode-extensions/vscode-utils.nix:21:12:
   |
21 |     inherit configurePhase buildPhase dontPatchELF dontStrip;
   |            ^

warning: build-tools-in-build-inputs
unzip is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/misc/vscode-extensions/vscode-utils.nix:25:5:
   |
25 |     buildInputs = [ unzip ] ++ buildInputs;
   |     ^


pretrained weights for gt-encoder

First of all, nice work! Do you mind sharing the pretrained weights for the GT encoder. I did not see the download link for that. Thanks.

I also did not find it. Is this available? Thanks.

I did not find it either. As a matter of fact, I am not convinced that the gt-encoder supervision would help at all.


refactor: fix parameter order in create_pull according to PyGithub 2.1.X

Due to bump of pygithub from 1.59.1 to 2.1.1, the call of create_pull should be adapted. See breaking change in release notes for pygithub for more reference.
new function signature for create_pull:
def create_pull(self, base, head, *, title=NotSet, body=NotSet, maintainer_can_modify=NotSet, draft=NotSet, issue=NotSet)

Has this caused any errors? Since we use named parameters the order shouldn't matter.

It is related to https://github.com/cloudfoundry/haproxy-boshrelease/pull/530 which has been just merged.

Close the PR as change is not needed.


C_CPP : Level order Traversal in Binary Tree

Description
In this i give full explain about the approach of the level order traversal in binary tree in document form and with good code snippet and also with understandable comments .
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

Closing as the PR got merged !


Create one.is-a-fullstack.dev.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description
Introducing imagination and nature connection activities
Website Link / Preview

Record corrected


Angular behavior

This is not a bug, the issue is that I am beginning with meteor and this module sounds really great.
The issue is that I don't grasp yet the full picture of Meteor and this module.
Now, my question is how will it fast-render content from the server in a angular application (urigo:angular) ?
Of the top of my head I know that we still have to wait for the angular app to bootstrap itself before showing content, so even if the content arrives quicker we still have the bootstrap and content processing time, right?

I hope now this works well with angular. You can use our route APIs for that: https://github.com/meteorhacks/fast-render#using-fast-renders-route-apis
I'm closing this issue since, we don't specifically add angular related features into FR.


make generated typescript bindings adhere to strict mode

At the moment the generated .ts files does not pass the strict typescript mode.
A possible (interims) solution could be to add // @ts-nocheck to the generated files.
So you don't have to do it every time again, if the file is regenerated.
It would be counterintuitive to disable strict mode project wide. Many projects which want type safe ipc calls also want strict type checking (imho).
thanks for the great work.

Yeah this makes perfect sense to me, do you know what errors ts throws when given a code generated file?

most of them are: Parameter 'xyz' implicitly has an 'any' type.
in serializeResult the function serializeU8 is called, which is not declared. (this is probably only related to my generated file)
if you drop a tsconfig.json file in the folder above the test folder with the TS test files you should get all errors presented. (at least in VS Code)
{
    "compilerOptions": {
        "strict": true,
    }
}

Aye yeah, okay that makes sense. The deserialization and serialization code doesn't have any types so that's why. I don't expect these functions to remain code generated this way (will be moved into the ipc router hopefully) so let's just go with the ts-ignore option


unknown log format "compression" in /etc/nginx/nginx.conf:34

Hi Andrew,
I experience an error as in the subject with nginx.

Ah, fixed, cheers :+1:

apologies was a corrupted dockerfile

more updates coming soon.


Lost all changes made in my notebook

Environment data

VS Code version: 1.57.1
Jupyter Extension version (available under the Extensions sidebar): v2021.6.999662501
Python Extension version (available under the Extensions sidebar): v2021.6.944021595
OS (Windows | Mac | Linux distro) and version: Mac
Python and/or Anaconda version: 3.9
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Jupyter server running: Local

Expected behaviour
Changes are saved in my jupyter notebook.
Actual behaviour
Changes are not saved.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

I saved the changes I made to the notebook
I closed the file and opened the source control panel to stage my changes. But the panel said that there were no changes. I could only see a staged changed of the same file that I did 2 days ago but no the changes I made to the file since then.
I proceed to open the file and all my changes were gone

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

Hi
Is there a way I can recover the changes I made? I need to submit this project tomorrow :( :(
Thanks in advance.

Apologies for the delay in getting back to you .
We have shipped a new UI for the Notebook. Please could you take a screenshot of the notebook and upload it.

Issues labeled with "Info needed" are closed if we do not receive more information from the issue author in 2 weeks. If you are the issue author and you still are running into this problem, please reactivate with additional information.


[Event Change Request] Codeunit 12 "Gen. Jnl.-Post Line" - OnFinishPostingOnBeforeResetFirstEntryNo

Describe the request
Hello, can you please add "GenJournalLine" and "IsTransactionConsistent" to the event OnFinishPostingOnBeforeResetFirstEntryNo in codeunit 12 "Gen. Jnl.-Post Line"?
    procedure FinishPosting(GenJournalLine: Record "Gen. Journal Line") IsTransactionConsistent: Boolean
    var
.
.
.
        GlobalGLEntry.Consistent(IsTransactionConsistent);

        if CostAccountingSetup.Get() then
            if CostAccountingSetup."Auto Transfer from G/L" then
                TransferGlEntriesToCA.GetGLEntries();

        OnFinishPostingOnBeforeResetFirstEntryNo(GlobalGLEntry, NextEntryNo, FirstEntryNo, GenJournalLine, IsTransactionConsistent);
        FirstEntryNo := 0;

    [IntegrationEvent(true, false)]
    local procedure OnFinishPostingOnBeforeResetFirstEntryNo(var GLEntry: Record "G/L Entry"; NextEntryNo: Integer; FirstEntryNo: Integer; GenJournalLine: Record "Gen. Journal Line"; IsTransactionConsistent: Boolean)
    begin
    end;

Additional context
We would like to add further information to the posted entries and write additional data but only if some conditions in the GenJournalLine are fulfilled and the transaction is complete.
Internal work item: AB#550927

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 26401.


5.5 cherry pick

Do not merge until PM approval given.

Closing this PR as this will be in for 23.10/ROCm 5.6

Merge approved by Valentin.


Update README.md

Add a reinforcement learning example

Thanks for adding the example @windmaple.

Thanks for creating the plugin in the first place!


⚠ Bofrost.be has degraded performance

In 4121921, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1512 ms

Resolved: Bofrost.be performance has improved in ceba22e.


Update docs to reflect 1.1.0

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/chart])

@austinbv
Could you sign DCO and bump this chart to latest version? Thank you.


Rebuild failed Error: unable to find build ->...

HKube micro-service
To which micro-service the bug related to, Dashboard ,Monitor-Server etc.
Describe the bug
Rebuild failed Error: unable to find build ->...
To Reproduce
build algorithim
wait a day for ETCD to be deleted
try to rebuild the algorithm
"Error: unable to find build -> get-twitt-a-jxbqls at _watchBuild (/hkube/algorithm-builder/lib/builds/docker-builder.js:173:15) at processTicksAndRejections (internal/process/task_queues.js:93:5) at async Object.runBuild (/hkube/algorithm-builder/lib/builds/docker-builder.js:475:17) at async Bootstrap.init (/hkube/algorithm-builder/bootstrap.js:23:30)"
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

tested on
systemVersion: "v2.2.45",
fullSystemVersion: "v2.2.45-1645610255014",


Fixed bug in server.rules.math::calcNewBotSpeed

Previously, when the currentSpeed = 0, it was possible to accelerate backwards at a limit bound by DECELERATION, rather than ACCELERATION.
PREVIOUS OUTPUT:
    ...
    calcNewBotSpeed(0, -0.9) -> -0.9
    calcNewBotSpeed(0, -1)   -> -1
    calcNewBotSpeed(0, -1.1) -> -1.1
    ...
    calcNewBotSpeed(0, -1.9) -> -1.9
    calcNewBotSpeed(0, -2)   -> -2
    calcNewBotSpeed(0, -2.1) -> -2
    ...

The condition when currentSpeed = 0 is now evaluated separately from the condition when currentSpeed > 0.
CURRENT OUTPUT:
    ...
    calcNewBotSpeed(0, -0.9) -> -0.9
    calcNewBotSpeed(0, -1)   -> -1
    calcNewBotSpeed(0, -1.1) -> -1
    ...
    calcNewBotSpeed(0, -1.9) -> -1
    calcNewBotSpeed(0, -2)   -> -1
    calcNewBotSpeed(0, -2.1) -> -1
    ...

@mnbrandl Thank you for the contribution and help fixing an issue. 😊👍
Before merging this into the main branch, I will first create some tests that identifies the bug, but also suit for refactoring code later etc. So please be patient with me till I have some test ready.

Glad to help!
I suppose I should add, I was getting an error that Gradle was unable to locate the kotlin-stdlib dependency, as no repositories had been added. I'm assuming it is probably an issue to my specific build environment (I'm having other issues trying to build), but maybe not. Whatever the issue, it was fixed by adding a repositories block to the dependencyResolutionManagement block in settings.gradle.kts. Not being very Gradle saavy, I don't know if this is something that needs to be looked into.
As for the tests, I have written some code that involves using the method, but did so in my own project for developing bots. The code wasn't written as a test, but it happened to function as a test find and fix the bug.
The identifying feature of the bug was that the output was not symmetrical for inputs that were negated. I.e. the magnitude of the output for inputs (0, 8)-> (1) was not the same magnitude of the output for inputs (0, -8)->(-2).
I am willing to come up with some tests, but I'm not sure how to proceed given your comment about future refactoring.

@mnbrandl

I am willing to come up with some tests, but I'm not sure how to proceed given your comment about being suited for future refactoring.

I am adding some tests very soon (work in progress) that proves that there is a bug, meaning that some of the tests will fail.
After that, I will merge your fix to main, and the test should all get green. 🙂

@mnbrandl I have now added some basic tests that proves that there are several issues:
https://github.com/robocode-dev/tank-royale/commit/1258a6ad339d533d7cde09173e20b9d905a7f7dd
The issue you found with the speed being -2 instead of -1.
But also some issue when the speed is going from a positive to negative value, and vice versa, and comparing with the behaviour of the original Robocode. So I will have a look at this as well. 😊

@flemming-n-larsen
I did not think to compare behavior with the original Robocode, that would have made too much sense! I do believe you are correct, the decel time needs to be calculated (from which we can easily obtain the accel time as well). The new speed is simply acceleration * time. However, to correctly calculate the new position, the time accel and decel time is also required in the method MutableBot::moveToNewPosition.
Also, in instances crossing zero from a given current speed where the target speed could result in time spent decelerating, accelerating, and traveling at constant speed (eg. -0.5, 0.5) would also be required in moveToNewPosition.
I see the original is doing things differently, relying on the distance remaining rather than a target speed to calculate movement. Target speed is now calculated in the BaseBotInternals class, and seems to be doing much of the work we are looking to implement in the server module. I'll have to look more into myself as well, let me know your thoughts once you have taken a look!

@mnbrandl I merge you pull request now as promised. Then I will figure out how to incorporate the crossing of zero without the distance so we can fix the remaining part of the test as it would be best to let tank royale be as compatible with the original game as possible to support the bridge. Especially because they are so similar.

@mnbrandl I fixed the remaining bug when speed 0 is crossed, i.e. when the current speed and target speed have different signs. 😊


Synchronization of the UPN changes between two federated domains

Hello!
Is anyone aware of the recent change in AAD Connect behavior on synchronization of the UPN changes between two federated domains? Recently my colleagues have noted Azure AD Connect synchronizes changes made in AD from one federated UPN Suffix to another federated UPN Suffix. However, I have not managed to find any announcement about this change. Could someone comment on it? Has it been done deliberately? Will AAD Connect continue to synchronize changes of UPNs between federated domains further? Or it has been done accidentally and will be reverted back?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8a4a5a4d-f097-a0f8-f81e-75a5837f202b
Version Independent ID: 58c43f72-9f93-b30f-2645-8b259aa3b516
Content: Azure AD Connect - AD FS management and customization
Content Source: articles/active-directory/hybrid/how-to-connect-fed-management.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@akulnev Thank you for your query. We will investigate and update further.

@akulnev We checked this internally with the program managers. The team was already working on this as it was a long requested item on the backlog of PG as per this uservoice feedback item.  This is a service side backend change which was introduced just a few weeks ago . A release notes update is on the way for this. This change is going to stay as it simplifies a lot of things for the customers who needed to run custom scripts to change a user's UPN suffix from one federated domain to another within the same tenant . They earlier needed to run scripts to change federated UPN suffix1 -->> managed UPN (@.onmicrosoft.com -->> federated UPN suffix2.
Hence if you have any script setup within your workflow , you can remove the same as the AAD connect will be able to sync such users without any issue as there is no service side limitation anymore related to this. The change is seamless and automatic. We will now close this issue.  Should you have any further queries regarding this , feel free to tag me to your reply and I will continue to answer them. Alternatively , you can always open a new issue and we would be happy to help.
Thank you.


bug: App doesn't report internal errors during labware calibration

Current behavior
As @mcous commented in #4810, during calibration...

The app collects the error from the API and does not display it.

Instead, it swallows the error silently. This makes things slightly more confusing for users. When something fails, it will look as if clicking the button in the app just didn't do anything. See #4810 for a concrete example.
Expected behavior
If something goes wrong during calibration, that should somehow be made clear to the user.

Check back in on this after 4.0.0 release

According to @mcous, this is a problem with our RPC layer.
In v4.0.0, that RPC layer is no longer used for deck calibration or pipette offset calibration, so this should no longer be a problem for those.
It's still a problem for labware calibration. But labware calibration is going to be overhauled soon, including replacing its RPC layer.

This ticket is now only relevant to labware calibration (as the other calibration stuff is handled by http versus rpc). This ticket is on the way too, as the labware cal project comes to completion.
Any lack of calibration errors using the new infrastructure should get their own tickets.

Fixed in v5


Please add feature to sync xAxis zoom for multiple charts in ng-repeat

Thanks for your work on this, pablojim!
I have a number of charts within a table using ng-repeat, all of which need to synchronize the xAxis on zoom and min/max. I've created a plunker here http://plnkr.co/edit/ZjJHKIpp9tgq6vSU657e?p=preview . Please let me know if this is possible or how to add functionality on my end for this.
Thanks again!

Your fiddle is broken now..
This is the solution I use to sync the xAxis zoom between 2 charts.
First, in the config on each chart:
$scope.chart1 = {
	options: {
		chart: {
			events: {
				selection: function(event) {
					self.onZoomHandler(event, 'chart2');
				}
			}
		}
	}
}

then the custom handler:
self.onZoomHandler = function(event, chartToUpdate) {
    var chart = $scope[chartToUpdate].getHighcharts();
    if (event.xAxis) {
        chart.xAxis[0].setExtremes(event.xAxis[0].min, event.xAxis[0].max);
        if (!chart.resetZoomButton) {
            chart.showResetZoom();
        }
    } else {
        chart.xAxis[0].setExtremes(null, null);
        if (chart.resetZoomButton) {
            chart.resetZoomButton = chart.resetZoomButton.destroy();
        }
    }
};

@jeanpool407 Can you please share the code for synchronize multi chart in plunker?


Tagging your NFT as NSWF does nothing

What happened?
For example this NFT. I can see in properties that it is NSFW but still, it's possible to see it in explorer, newest list and etc.
Suggestions:

add in settings a button that will allow you to hide all NSFW content (make sure that settings stay the same even after you log out - right now, everything you set in settings will be changed back to default when you log out and log in again)
blurr picture + small icon on the picture with NSWF text (so people know that it is not blurry NFT uploaded but SWF)
when you click on this blurred NFT you get a notification that this NFT is NSWF do you want to open it?
add a filter in collection/NFT explorer - filter out NSF works.

I would not:

hide picture from explorer + newest list, latest sale - this can be discriminating for artists (blurring + other suggestions should be enough)

Please reproduce in steps
Check this collection.
Expected Behavior
Do something to prevent us from becoming a porn site.
What browsers are you seeing the problem on?
No response
At which address did you encounter bug?
No response
Are you logged in?
None
Which wallet you are using?
No response
At which chain did you encounter bug?
No response
Screenshots

Relevant log output
No response
Payment link for reward
https://kodadot.xyz/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

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

Link #285 & #5455


[doc update] Structured list

Update the usage doc for structured list  component with a new template so that it is ready for V11.
Related: Update all component usage docs with the new template

This is great @thyhmdo ! We actually have some v10 feature updates for structured list that have never been documented which would actually be great to add in here.
We have both a condensed list and a flush alignment list now. Check out in knobs in the story.
https://react.carbondesignsystem.com/?path=/story/components-structuredlist--selection

@aagonzales oh and recently found this one related to the issue that Lauren started https://github.com/carbon-design-system/carbon-website/issues/2310


"Error - Not Found" with 1.0.0

Error: Not Found
    at /home/user/public_html/adminMongo/app.js:181:15
    at Layer.handle [as handle_request] (/home/user/public_html/adminMongo/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/home/user/public_html/adminMongo/node_modules/express/lib/router/index.js:312:13)
    at /home/user/public_html/adminMongo/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/home/user/public_html/adminMongo/node_modules/express/lib/router/index.js:330:12)
    at next (/home/user/public_html/adminMongo/node_modules/express/lib/router/index.js:271:10)
    at /home/user/public_html/adminMongo/node_modules/express/lib/router/index.js:618:15
    at next (/home/user/public_html/adminMongo/node_modules/express/lib/router/index.js:256:14)
    at next (/home/user/public_html/adminMongo/node_modules/express/lib/router/route.js:121:14)
    at /home/user/public_html/adminMongo/routes/index.js:8:5

yes the problem is in "routes/index.js"
https://github.com/mrvautin/adminMongo/compare/master...lefdilia:patch-5

Yep, fixed. Thanx 😉

Seems the link above is dead. Was fixed in PR: https://github.com/mrvautin/adminMongo/pull/96.


METROL-103: update WebKitPlugin

This is the same PR as created against the Master branch ((https://github.com/rdkcentral/rdkservices/pull/257) but now against the Sprint branch. It also includes the changes resulting from the latest discussion on the DiskCacheDir and WebKit responsiveness check code.

A new one against the sprint/2010 branch was created


views/chat: set title attribute on topic

The title attribute is only set when receiving a topic change event right now, not on page load.

Good catch, thanks!


VM has reported a failure when processing extension 'install_azure_devops_agent'

Describe the bug
From Step 3. Customize and deploy the Azure DevOps Agents (runners)
Error of type
module.vm_extensions["level0"].azurerm_virtual_machine_extension.devops_selfhosted_agent["devops_selfhosted_agent"]: Still creating... [1m0s elapsed]
module.vm_extensions["level3"].azurerm_virtual_machine_extension.devops_selfhosted_agent["devops_selfhosted_agent"]: Still creating... [1m0s elapsed]
module.vm_extensions["level1"].azurerm_virtual_machine_extension.devops_selfhosted_agent["devops_selfhosted_agent"]: Still creating... [1m0s elapsed]
module.vm_extensions["level2"].azurerm_virtual_machine_extension.devops_selfhosted_agent["devops_selfhosted_agent"]: Still creating... [1m0s elapsed]
module.vm_extensions["level4"].azurerm_virtual_machine_extension.devops_selfhosted_agent["devops_selfhosted_agent"]: Still creating... [1m0s elapsed]
Terraform apply return code: 0
Terraform returned errors:
╷
│ Error: Code="VMExtensionProvisioningError" Message="VM has reported a failure when processing extension 'install_azure_devops_agent'. Error message: "Enable failed: failed to execute command: command terminated with exit status=1\n[stdout]\ndll\n./bin/Microsoft.TeamFoundation.Test.WebApi.dll\n./bin/System.Diagnostics.Tools.dll\n./bin/System.Web.HttpUtility.dll\n./bin/Microsoft.Azure.Storage.DataMovement.dll\n./bin/System.Security.Cryptography.Encoding.dll\n./bin/es-ES/\n./bin/es-ES/strings.json\n./bin/Microsoft.TeamFoundation.Core.WebApi.dll\n./bin/System.Private.DataContractSerialization.dll\n./bin/System.Net.WebProxy.dll\n./bin/System.Security.Cryptography.Cng.dll\n./bin/System.Private.Uri.dll\n./bin/Agent.Worker.dll\n./bin/Microsoft.VisualBasic.dll\n./bin/System.Xml.XPath.XmlDocument.dll\n./bin/Minimatch.dll\n./bin/Microsoft.TeamFoundation.Policy.WebApi.dll\n./bin/Microsoft.TeamFoundation.TestClient.PublishTestResults.dll\n./bin/System.Net.WebSockets.dll\n./bin/System.Globalization.dll\n./bin/Agent.Listener.runtimeconfig.json\n./bin/CommandLine.dll\n./bin/System.Threading.AccessControl.dll\n./bin/System.IO.Compression.Brotli.dll\n./bin/update.sh.template\n./bin/System.CodeDom.dll\n./bin/CodeSignSummary-a05ae9f4-33f9-45d4-9a39-6a2e91593084.md\n./bin/System.IO.Abstractions.dll\n./bin/System.IO.MemoryMappedFiles.dll\nextracted\n--------OS Information--------\nNAME="Ubuntu"\nVERSION="20.04.2 LTS (Focal Fossa)"\nID=ubuntu\nID_LIKE=debian\nPRETTY_NAME="Ubuntu 20.04.2 LTS"\nVERSION_ID="20.04"\nHOME_URL="https://www.ubuntu.com/\"\nSUPPORT_URL=\"https://help.ubuntu.com/\"\nBUG_REPORT_URL=\"https://bugs.launchpad.net/ubuntu/\"\nPRIVACY_POLICY_URL=\"https://www.ubuntu.com/legal/terms-and-policies/privacy-policy\"\nVERSION_CODENAME=focal\nUBUNTU_CODENAME=focal\n------------------------------\nThe current OS is Debian based\n--------Debian Version--------\nbullseye/sid\n------------------------------\n/usr/bin/apt\nHit:1 http://azure.archive.ubuntu.com/ubuntu focal InRelease\nHit:2 http://azure.archive.ubuntu.com/ubuntu focal-updates InRelease\nHit:3 http://azure.archive.ubuntu.com/ubuntu focal-backports InRelease\nHit:4 https://packages.microsoft.com/repos/azure-cli focal InRelease\nGet:5 http://security.ubuntu.com/ubuntu focal-security InRelease [114 kB]\nFetched 114 kB in 0s (229 kB/s)\nReading package lists...\nBuilding dependency tree...\nReading state information...\n14 packages can be upgraded. Run 'apt list --upgradable' to see them.\nReading package lists...\nBuilding dependency tree...\nReading state information...\nliblttng-ust0 is already the newest version (2.11.0-1).\nlibkrb5-3 is already the newest version (1.17-6ubuntu4.1).\nzlib1g is already the newest version (1:1.2.11.dfsg-2ubuntu1.2).\n0 upgraded, 0 newly installed, 0 to remove and 14 not upgraded.\nReading package lists...\nBuilding dependency tree...\nReading state information...\nlibssl1.1 is already the newest version (1.1.1f-1ubuntu2.4).\n0 upgraded, 0 newly installed, 0 to remove and 14 not upgraded.\nReading package lists...\nBuilding dependency tree...\nReading state information...\nReading package lists...\nBuilding dependency tree...\nReading state information...\nlibicu66 is already the newest version (66.1-2ubuntu2).\n0 upgraded, 0 newly installed, 0 to remove and 14 not upgraded.\n-----------------------------\n Finish Install Dependencies\n-----------------------------\ndependencies installed\n\n  ___                      ______ _            _ \n / _ \                     | ___ ()          | ()\n/ /\ \_____   _ _ __ ___  | |/ / _ __   | | _ __   ___  \n|  _  |  / | | | '/ _ \ |  /| | ' \ / _ \ | | ' \ / _ \/ __|\n| | | |/ /| || | | |  __/ | |   | | |) |  / | | | | |  /\ \\n\| |/|\,||  \| \|   || ./ \|||| ||\||/\n                                   | |\n        agent v2.187.2             ||          (commit 0cfc45c)\n\n\n>> End User License Agreements:\n\nBuilding sources from a TFVC repository requires accepting the Team Explorer Everywhere End User License Agreement. This step is not required for building sources from Git repositories.\n\nA copy of the Team Explorer Everywhere license agreement can be found at:\n  /home/adminuser/agent/agent-1/externals/tee/license.html\n\n\n>> Connect:\n\nError reported in diagnostic logs. Please examine the log for more details.\n    - /home/adminuser/agent/agent-1/_diag/Agent_20210620-194325-utc.log\n\n\n[stderr]\nWARNING: Error loading config file: .dockercfg: $HOME is not defined\nsudo: ./svc.sh: command not found\nsudo: ./svc.sh: command not found\n\nWARNING: apt does not have a stable CLI interface. Use with caution in scripts.\n\n\nWARNING: apt does not have a stable CLI interface. Use with caution in scripts.\n\n\nWARNING: apt does not have a stable CLI interface. Use with caution in scripts.\n\n\nWARNING: apt does not have a stable CLI interface. Use with caution in scripts.\n\nE: Unable to locate package libicu67\n\nWARNING: apt does not have a stable CLI interface. Use with caution in scripts.\n\nVS30063: You are not authorized to access https://dev.azure.com.\n\x1b[41mError on or near line 101; exiting with status 1\x1b[0m\n"\r\n\r\nMore information on troubleshooting is available at https://aka.ms/VMExtensionCSELinuxTroubleshoot "
│
│   with module.vm_extensions["level0"].azurerm_virtual_machine_extension.devops_selfhosted_agent["devops_selfhosted_agent"],
│   on extensions/devops_selfhosted_agent.tf line 2, in resource "azurerm_virtual_machine_extension" "devops_selfhosted_agent":
│    2: resource "azurerm_virtual_machine_extension" "devops_selfhosted_agent" {
│
╵
Screenshots

OS:Win 10 / Docker 3.4
Rover Version aztfmod/rover:0.15.1-2104.2711

For me it was just a matter of restarting my system (docker for windows error), remove the extension for the linux agent and change the entry to map your organisation
url           = "https://dev.azure.com/azure-terraform/"
in this file
/configuration/sandpit/level1/gitops/azure_devops_agents_vm/landingzone.tfvars


CreateTransactionRequest not works for 'void' type of transaction (CIM api).

CreateTransactionRequest require 'amount' parameter for all type of transactions. As described here http://www.authorize.net/support/CIM_XML_guide.pdf  'amount' not needed for 'void' transaction. But CreateTransactionRequest create '<amount>...</amount>' tags for 'void' also. Because of this, authorize.net return the error message:
"The element 'profileTransVoid' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd' has invalid child element 'amount' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd'. List of possible elements expected: 'customerProfileId, customerPaymentProfileId, customerShippingAddressId, transId' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd" .
And transaction doesn't pass.

Could you please try this fix in the branch 'ticket12' and see if it works for you?

Yes, it works great. Only need to correct the typo in line 697 if amount and self.transaction_type != "Void": from amount to self.amount and it will work fine.
Thank you!


Where is the hacked ASP.NET Core DLLs?

it seems the project use dlls in the following directories, so where to get the hacked dlls?
I wish to compile the project in macOS.

Update C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App\8.0.0\*.dll,
use your dotnet root path to replace the C:\Program Files\dotnet\


Place scrollbarSlider.background below syntax blocks/characters in minimap

Hi. Thanks for the latest release! Loving the latest features.
I'm a themer and I've noticed that to make the minimap background look good, I need to use a transparent colour for scrollbarSlider.background; otherwise the built-in amount of transparency used is still quite dark:

I was wondering if it'd be a good idea if the syntax characters/blocks could be on top of the highlight, meaning I can work easier with solid colours and not need think about how much transparency Code is going to add for me (because it wouldn't be needed any more).

It is very convenient to have the slider rendered on top of the minimap pixels, as they can be independently optimized. From frame to frame, the minimap will reuse its pixels from the previous frame, and painting the slider as part of the minimap pixels would make such an optimization a lot more difficult.

Huh? Isn’t it just a z-index tweak?

To change the dom elements order, we would need to:

tweak z-index
rethink how mouse handling works (a listener is now registered on the slider dom node)
make the minimap canvas have a transparent background.

I've only had a peek at the dev tools and the DOM structure but my thoughts would be:

Move slider behind canvas (the map) using z-index
Allow mouse events to pass through the canvas by using pointer-events: none
Canvas is transparent by default so whatever is colouring this in can be turned-off.

Thanks @alexandrudima I've had a poke around in there and I can't see an easy way to simply remove the background without removing items that may be important to other parts of the UI. However, I've never looked as Code's code before and so I'm going to have to leave this for the professionals.


Can not leverage NewDialer function due to cql-proxy version conflict

In the critical gocql-astra change, we introduced https://github.com/datastax/gocql-astra/pull/6/files:
func NewDialer(b *astra.Bundle, timeout time.Duration) (gocql.HostDialer, error) {
	return &dialer{
		bundle: b,
		timeout: timeout,
	}, nil
}

And this is the new function that we will leverage. To use it, I need to create an astra.Bundle instance, and that turns out to be the problem.
In cql-proxy v0.1.3, astra.Bundle is defined as
type Bundle struct {
	tlsConfig *tls.Config
	host   string
	port   int
}

As the fields are not exported, there is no way I can create an instance. It has a bunch of other constructors like LoadBundleZip but I don’t see how I can easily use them without having a zip file. Note that the entire purpose of our change is to avoid needing a secure bundle zip file.
In cql-proxy v0.1.4, the fields in astra.Bundle are exported, meaning that I can directly instantiate them. But as said above, using v0.1.4 will cause build errors due to two versions.
Thus, the new functionality cannot be used without updating to 0.1.4 cql-proxy in gocql-astra.

@StevenLacerda I've merged the PR @joao-r-reis created above to update the cql-proxy version used by gocql-astra.  Please let us know if you hit any other problems!


basic puffin profiler

This is a basic implementation of the puffin profiler.
It's pretty nice, but it's not a profiler that can generate a complete flamegraph (unless you disseminate the puffin::profile_function!() macro everywhere) but useful nonetheless.
I preferred to implement it with http protocol to avoid having to integrate egui (so we will be free to choose the GUI we would like).
To connect to the profiler during the game use this:
https://github.com/EmbarkStudios/puffin/tree/main/puffin_viewer
To enable/disable the profiler the line to be modified is this: puffin::set_scopes_on(true).
There is also this macro that identifies where the frame ends (I hope I put it in the correct place): puffin::GlobalProfiler::lock().new_frame();

awesome! taking a look at this today

Just yesterday the new version of tracy_full came out, case that I had missed as usable but potentially more interesting than Puffin because it has support for WGPU:
https://github.com/SparkyPotato/tracy_full
I think for now we can use Puffin anyway, since the changes are very easy to refactor and don't mess the code.
If we see that the puffin is not enough to profile the wgpu part maybe we will evaluate whether to use it.

yes let's definitely try puffin and see how we like it. gunna check out tracy, wgpu support sounds very cool. I've been using cargo flamegraph so far and it's good for profiling cpu but doesn't tell you much on the gpu side

Looks awesome! gunna merge it in. I like how I can press 'M' while the game is running and instantly see the render function time shrink, it's super cool. It would've helped me track down some of these perf issues more quickly, like 'get all bone data' taking 6ms, wayyyy too slow.


Create Rajesh Gunasekaran PR to sign CLA

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

What this PR does / why we need it:
This PR is being created so I can sign the CLA
Does this PR introduce a user-facing change?:
None


fix for https://github.com/Geta/geta-notfoundhandler/issues/64

fix for "Pages' conflict
https://github.com/Geta/geta-notfoundhandler/issues/64

The only thing is that we cannot release master yet. It has some WIP changes.

And thanks for the fix!


Valid way to exclude specific consrutctions(for ex. loops)

Hello. We are thinking about excluding some specific constructions from parsing - for example loops.  I saw there are limitation abilities in "Safe parser" documentation section https://github.com/lunet-io/scriban/blob/master/doc/runtime.md#safe-runtime which can help to limit them, but is there valid way to forbid some of them at all? Thanks.

Unfortunately, there is nothing builtin. You can always iterate on the AST and remove the loops (but there is no a visitor pattern currently provided, so you have to manually go through the statements). If you feel to have this at parse time, feel free to send a PR, modifying the ParserOptions and the Parser to discard loop after parsing them (and log an error).


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-23-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1045

Questions ?
Please refer to the Backport tool documentation


[BUG]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce

Provide a link to a StackBlitz sandbox which reproduces the issue
OR: provide a link to a repository that reproduces the issue

If you cannot do one of the above, list steps to reproduce the behaviour below:

Go to '...'
Click on '....'
Scroll down to '....'
See error '....'

Expected behaviour
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here. eg.

Hydrogen version
Node version
Device details

Closed in favor of #2135


Revert "changed Pswrd togg button position"

Reverts MonalikaPatnaik/UMatter#523

This PR merged with lot of bug there are some components and css properties which imported twice and project not running locally right.

Hey..yes there were issues with the code..now I think it's working fine!


Inference acceleration doesn't work

I'm trying to use DeepSpeed to accelerate a ViT model implemented by this project. I following this tutorial to optimize the model:
from vit_pytorch import ViT
import torch
import deepspeed
import time

def optimize_by_deepspeed(model):
    ds_engine = deepspeed.init_inference(
        model,
        tensor_parallel={"tp_size": 1},
        dtype=torch.float,
        checkpoint=None,
        replace_with_kernel_inject=True)
    model = ds_engine.module
    return model

device = "cuda:0"

model = ViT(
    image_size=32,
    patch_size=8,
    channels=64,
    num_classes=512,
    dim=512,
    depth=6,
    heads=16,
    mlp_dim=1024,
    dropout=0.1,
    emb_dropout=0.1
).to(device)

input_data = torch.randn(1, 64, 32, 32, dtype=torch.float32, device=device)

model = optimize_by_deepspeed(model)

start_time = time.time()
for i in range(1000):
    res = model(input_data)
print("time consumption: {}".format(time.time() - start_time))

and launch the program with following command:
deepspeed --num_gpus 1 example.py

However, I found that the time consumption is roughly the same whether or not model = optimize_by_deepspeed(model) is executed. Is there any problem in my code?

Hi @broken-dream, init_inference injects optimized kernels but doesn't work for all models. I think it was basically designed for transformer-based models on HF.

I will try it with the huggingface ViT model. Thanks for your help!


fix edge case when unlocking the fortress

if you have just reached level 25 but not build anything the fortress is technically unlocked but you still don't have a rank
this commit fixes that by making the rank an optional and correcting the logic for when you have the fortress unlocked

Thanks! Is the raw rank from the server in that case just -1, or 0?


Feature request: disable overlapping selections

It would be awesome to disable overlapping selections :+1:

What do you mean by "overlapping selections"? Please give an example.

@mir3z I think he means to not be able to select parts of already selected text.


App home is showing not just BAU task

https://tools.hmcts.net/jira/browse/DTSPO-181

Fixed in https://github.com/hmcts/cnp-flux-config/pull/8509


Add missing things in typings

the pr show it has changes in Client, Shard and Server but actually there is nothing change, it happen because i just throw all files to the fork repo

the pr show it has changes in Client, Shard and Server but actually there is nothing change, it happen because i just throw all files to the fork repo

I can't merge the first 3 files, since it will break the commit history
I will manually add the typings in my local branch and then push it

I can't merge the first 3 files, since it will break the commit history
I will manually add the typings in my local branch and then push it

ok

ok


Revert "[DebugInfo] Set the autogenerated bit to request recycled locations"

Reverts apple/swift#15929
This breaks PR testing because it was only smoke tested.
/swift/test/DebugInfo/returnlocation.swift:188:22: error: CHECK_INIT-NEXT: is not on the line after the previous match
 // CHECK_INIT-NEXT: call void @swift_bridgeObjectRelease{{.*}}, !dbg [[retnLoc:![0-9]+]]
                     ^
<stdin>:752:2: note: 'next' match was here
 call void @swift_bridgeObjectRelease(%swift.bridge* %2) #4, !dbg !275
 ^
<stdin>:749:149: note: previous match ended here
 call swiftcc void @"$Ss5print_9separator10terminatoryypd_S2StF"(%swift.bridge* %2, i32 %16, i32 %17, i32 %18, i32 %20, i32 %21, i32 %22), !dbg !274
                                                                                ^
<stdin>:750:1: note: non-matching line after previous match is here
 call void @"$Ss13_StringObjectV8_VariantOWOe"(i32 %20), !dbg !275
^

https://ci.swift.org/job/oss-swift_tools-RA_stdlib-RD_test-simulator/869/consoleFull#174822253ba62d58e-7248-467b-91e0-c7508d5cf947

@swift-ci test.

Build failed
Swift Test OS X Platform
Git Sha - 53a0676cc2e2f19f4ccf9317ae086aa6ad7c9b67

Reverting this is blocked by PR #15930.

@swift-ci smoke test OS X Platform.

@swift-ci test.

Sorry about this. There should be a simpler way to write this test.


browserify loads every shim instead of just one

According to this:

http://stackoverflow.com/a/28965677/607033
Don't require polyfills in your modules, that's an anti-pattern. Your modules should assume that the runtime is patched (when needed), and that should be part of the contract. A good example of this is ReactJS, where they explicitly define the minimum requirement for the runtime so that the library can work: http://facebook.github.io/react/docs/working-with-the-browser.html#browser-support-and-polyfills

it is an anti pattern to use code like this in our modules
var platform = require("../platform");

var StackFrameExtractor;

if (platform.isNode() || platform.isChrome())
    StackFrameExtractor = require("./v8");
else if (platform.isPhantom())
    StackFrameExtractor = require("./phantom");
else
    StackFrameExtractor = require("./mock");

module.exports = StackFrameExtractor;

actually this is called by ErrorAdapter, which is called by UserError, which needs an adapter.
What appears to be a bug, that all 3 implementations are loaded by browserify, but only one is used. So what I can do is defininig classes, or I can define those modules as functions and run them by code. E.g.
var getStackFrameExtractor = require("./xxx");
StackFrameExtractor = getStackFrameExtractor();

so if the module overrides global variables, etc... by default then it won't have a problem. probably yadda is affected, but I have to send an issue about this to browserify whether this is normal or not.

The simplest test:
index.js
var x;
if (Math.random()<0.5)
    x = require("./a");
else
    x = require("./b");
console.log(x());

a.js
console.log("a is loaded");
module.exports = function (){
    console.log("a is running");
    return "a";
}

b.js
console.log("b is loaded");
module.exports = function (){
    console.log("b is running");
    return "b";
}

In node I expect:
a is loaded
a is running

In browserify I expect:
a is loaded
b is loaded
a is running

it should not do b is loaded when a is selected.

I am not sure whether this is a browserify bug or this is a feature. I removed the bug flag, since it does not cause any error in this system, just a little overhead, because the js engine has to load the frame extractor of every environment, but it is negligible.

Closing this in favor of #94 , since I moved the error api to a separate project...


Device changes hostname every restart

Description
Some devices change their hostname every time they are restarted while others don't.
This means their entry from device list is no longer valid and a new entry appears in the pending list. All tags associated need to be re entered and connection credentials need to be updated.
Edition
Cloud
Version
v0.12.2-rc.3

Hi @tiago-cerveira, by default, ShellHub utilizes the MAC address of the primary interface as its identifier. Therefore, if the identifier changes, the name changes as well.
Which device are you currently installing the ShellHub agent on?

I am getting device change for jetson xavier devices
Otavio Salvador @.***> escreveu no dia quinta,
25/05/2023 à(s) 12:17:

Hi @tiago-cerveira https://github.com/tiago-cerveira, by default,
ShellHub utilizes the MAC address of the primary interface as its
identifier. Therefore, if the identifier changes, the name changes as well.
Which device are you currently installing the ShellHub agent on?
—
Reply to this email directly, view it on GitHub
https://github.com/shellhub-io/shellhub/issues/2833#issuecomment-1562726694,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMWBT6UTP4NTPO7AP3Z72LXH45ULANCNFSM6AAAAAAYOOQ3GE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you please check the output of ifconfig -a before and after a reboot? I think the MAC address might change.

@tiago-cerveira I had similar scenario, and it was not possible to remove dummy0 since it was pre-compiled with the kernel. the workaround I used is with the SHELLHUB_PREFERRED_HOSTNAME parameter, I did something like SHELLHUB_PREFERRED_HOSTNAME=$(hostname) in an automation on the relevant machines.

To ensure a long-term solution, it is essential to update the SHELLHUB_PREFERRED_IDENTITY, which serves as the device identifier. You can provide a CPU serial number, network MAC address, or any other equivalent identifier to complete the process.
In one of our customers we use:
#!/bin/sh

# This script set the CPU serial number as ShellHub ID
# If it is a RaspberryPi based machine, it reads from /sys/firmware or /sys/devices/ path
if [ -e /sys/firmware/devicetree/base/serial-number ]; then
    export SHELLHUB_PREFERRED_IDENTITY=$(tr -d '\0' < /sys/firmware/devicetree/base/serial-number)
else
    export SHELLHUB_PREFERRED_IDENTITY=$(cat /sys/devices/soc0/serial_number)
fi

Closing as it is a device characteristic and we offered a way to fix it.


Support Swift 3

There are already forks that did it, like https://github.com/acrtec/Pantomime.

I installed this framework via pod but got 34 errors ! It seem it is not compatible with swift 3 !

Just found that Swift 3 compatibility was actually implemented. It's on master branch, but for some reason wasn't released (no tag and no Pod update). So, if you need it for an app just add the pod by git with branch.

Thank you , but how should I git it with branch ?

Something like that:
pod 'Pantomime', :git => 'https://github.com/thomaschristensen/Pantomime.git', :branch => 'master'
See https://guides.cocoapods.org/using/the-podfile.html for more options.

Thank you  👍  it works fine , but there is no way to extract video file from the .m3u8 , I need to parse the URL and then download the video file, this is my code :
let builder = ManifestBuilder()
      ```

let masterManifestURL = URL(string: "https://xxxxxxxxxxx.com/videos193dbe05552d449f81057009b11defe1475564711-640-360-900-h264.mp4/master.m3u8")
let videoFile = builder.parseMasterPlaylistFromURL(masterManifestURL!)


chore: updated business profiles

Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Updated business profiles
View profile details and update webhook details inside business profile (that currently is under payment-settings)

Motivation and Context
moving all profile related details to business profile instead of payment-setting

How did you test it?
working as described in description

Where to test it?

[x] INTEG
[x] SANDBOX
[ ] PROD

Checklist

[x] I ran npm run re:build
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible

Please add edit profile name functionality . Currently its always going to the profile details Refer to existing business-profile page in sbx

okay

will pick this up again when the product requirement regarding these changes are clear


Highlight current month

Changes the header bar's background color to a lighter blue when
viewing the current month to better differentiate between months.

I'm going to go ahead and hold off merging this one in until we can figure out how it'll interplay with the localisation / internationalisation stuff @egens is working on.

I bet this does what you need:
ynabToolKit.shared.parseSelectedMonth();

OK, I'll take a look. Thanks.

I've updated the code to use the parseSelectedMonth() function. Let me know if this will work?
Also, how did you go about finding that particular function? Something to do with the Ember Inspector, I'd imagine?

It's a Toolkit library function from src/common/res/shared.
I'm not sure who originally contributed it. :-)

Got it, thanks. I'll have to look at that file in case there's other useful stuff in there.

Alrighty, in it goes. The L10n feature may still break this, but if it does we'd need to update the library function anyway, and we'd address it there so any features relying on that will just work as they always have.

Forgot to mention, thank you!