pythonPackages: fix typo about input argument disabled

Description of changes
buildPythonPackage accepts disabled instead of disable to throw evaluation error on unsupported Python interpreters. Attempt to access such option as disable without the trailing d will be ignored silently, with an unexpected environment variable disable being injected into the build environment.
This patch fixes those misspelled disabled specification.
This is an apparent bug, and the fix should hardly change the output content. Maybe we could backport it to the stable release. ~(I choose to target staging, as 87 packages got rebuilt due to disable environment variable being removed.)~
Update: Target the master branch as suggested by @tjni.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

@ofborg build onionshare python3Packages.flask-openid python3Packages.furo python3Packages.graphql-server-core python3Packages.sphinx-basic-ng

Updated. Thank you for your reply and patience.

Came back here to resolve merge conflict (due to the version bump of a package), but found even more packages affected by such typo. They are now addressed by this PR.

Can you delete the disabled = !isPy3k checks? We don't support Python 2 anymore when we manage packages in nixpkgs, so we can assume it doesn't exist.

BTW, isPy3k may still be relevant for https://github.com/cachix/nixpkgs-python people.


tentative fixing master: fixing scikit-image et scikit-learn, and others >= version which is currently ok

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue

How Has This Been Tested (if it applies)

Checklist

[ ] The documentation is up-to-date with the changes I made.
[ ] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[ ] All tests passed, and additional code has been covered with new tests.

(I try to fix only scikit-image --- tests rerunning... with scikit-learn and scikit-image fixed at 1.0.1 and 0.18.3 respectively everything works)


Fix agg tests

Fix all broken tests named fix-test-* to have the same behavior both on bash and pash (must also work on the latest pash distro/debian)

We need to move aggregators in their own repo (and ideally run their tests there).
At the moment we are not using them here, and separating them would enable improving them/working on them separately than the rest of the PaSh project.

Agreed!

Aggregators have moved to https://github.com/binpash/aggregators!


How to troubleshoot MSP field value missing (UpdateCompliance Configuration)

At the Configuring devices through the Update Compliance Configuration Script page, at the bottom there is a section Verify device configuration.  I have followed the steps and my Commerical ID is showing, but the MSP field isn't there.  I've waited 72 hours and I am not seeing anything in my UpdateCompliance Summary tile.
Run the Diagnostic Data Viewer.
Go to Start, select Settings > Privacy > Diagnostics & feedback.
Under View diagnostic data, select Open Diagnostic Data Viewer.
When the Diagnostic Data Viewer opens, type SoftwareUpdateClientTelemetry in the search field. Verify the following items:
The EnrolledTenantID field under m365a should equal the CommercialID of your Log Analytics workspace for Update Compliance.
The MSP field value under protocol should be either 16 or 18.
If you need to send this data to Microsoft Support, select Export data.

My device is hybrid joined, I'm not sure if that is supported.  The website says "All devices enrolled in Update Compliance must meet all prerequisites for enabling Windows diagnostic data processor configuration, including the Azure AD join requirement. This prerequisite will be enforced for Update Compliance starting on October 15, 2022."

@jdb5 Thank you for your feedback. I have tested it in a  hybrid joined device and it shows up in the MSP field. If this continues to be a problem, we recommend you contact Microsoft support so that a support engineer can assist in fixing the issue.
Hope this helps!
Thanks
Sri

Thanks @jdb5  for taking the time to share this with the Windows documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this issue:

Post to Windows on Q&A. These are moderated by community members who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Unified Support team, such as a Customer Engineer or Customer Success Account Manager.
I'm closing this issue as there's nothing actionable for docs at this time.


ModuleNotFoundError: No module named 'optimum'

Since I updated to v0.2.0 I get this error at startup, leading to no TensorRT Tab being availabile in the WebUI.
*** Error loading script: trt.py
    Traceback (most recent call last):
      File "B:\stable-diffusion-automatic1111\modules\scripts.py", line 469, in load_scripts
        script_module = script_loading.load_module(scriptfile.path)
      File "B:\stable-diffusion-automatic1111\modules\script_loading.py", line 10, in load_module
        module_spec.loader.exec_module(module)
      File "<frozen importlib._bootstrap_external>", line 883, in exec_module
      File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
      File "B:\stable-diffusion-automatic1111\extensions\Stable-Diffusion-WebUI-TensorRT\scripts\trt.py", line 13, in <module>
        import ui_trt
      File "B:\stable-diffusion-automatic1111\extensions\Stable-Diffusion-WebUI-TensorRT\ui_trt.py", line 18, in <module>
        from exporter import export_onnx, export_trt, export_lora
      File "B:\stable-diffusion-automatic1111\extensions\Stable-Diffusion-WebUI-TensorRT\exporter.py", line 15, in <module>
        from optimum.onnx.utils import (
    ModuleNotFoundError: No module named 'optimum'

Reverting back to v0.1.0 fixed the issue.

Simplest fix would be to just go into the webUI directory, activate the venv and just pip install optimum, After that look for any other missing stuff inside the CMD. Might be that your internet skipped a beat when downloading some stuff.
I wouldn't however update to v0.2.0. There's a LOT of issues making it unstable and in some cases not usable.

please help, my stable diffusion webUI was working last time i used it , and when i launch it today, i got the same error, i ran the  pip install optmum and it makes no difference, did it install somewhere else? or update something? how tor revert? im new and i don't know coding, im just going off intuition on what those codes mean.

You would need to activate the venv in order to install.
As a general, from what I have found, the new TensorRT simply doesn't install any of the requirements.


Publish to npm

Publish ionic-filter-bar to npm so that it can be require'd by those who use browserify.

Thanks!  I will try to get to this at some point, just been pretty busy.

Seconded! Thanks!

+1


prerequisite endpoints

This page lists only the following endpoints as required:
enterpriseregistration.windows.net - For Azure AD registration.
login.microsoftonline.com - For Azure AD registration.
*.dm.microsoft.com - The use of a wildcard supports the cloud-service endpoints that are used for enrollment, check-in, and reporting, and which can change as the service scales.
What about all the other endpoints required by the "Microsoft Defender for Endpoint URL list for commercial customers" (spreadsheet)?
Please clarify any and all endpoint URL requirements.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e93f9bbd-5752-e28a-f32d-a01d81964c8f
Version Independent ID: e93f9bbd-5752-e28a-f32d-a01d81964c8f
Content: Manage Microsoft Defender for Endpoint configuration settings on devices with Microsoft Endpoint Manager
Content Source: microsoft-365/security/defender-endpoint/security-config-management.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Pls contact support.


Import and Export PDF documents - Annotation

Hi muaz-khan,
I really like this tool, looks like just the thing I need to annotate documents, I just wondered if there is an option to import .pdf files, annotate and then export them?
I can see there is the import image and export image but I presume its possible to do the same with pdf or any other document extension potentially?
As I really want to annotate something like a contact and then save the annotations and share.  Is this something you have done or a new feature I need to look into?
Many thanks
Jon

Hey @jonatap2chat  do you have this feature? I badly need this. please help

Bro please reply.

What if we share & sync PDF separately? I mean, we can open a popup in the index.html demo to embed PDF and sync pages/zoom/etc. in realtime?
Please track this thread for PDF syncing: https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421596699
PS. If its easier then I'll embed PDF inside the main-canvas as well (same like images-tool).

Yes sir that is what I'm going to do. To embed the pdf to canvas and can
draw to PDF.
Because I'm currently developing a VIRTUAL CLASSROOM sir. My inspiration is
https://www.braincert.com/try-virtual-classroom . that is what I'm going to
be.
And I using your library which is very helpful. Please help me Mr. muaz-khan
AGAIN THANK YOU SO MUCH!!! The muaz-khan/RTCMultiConnection#628 (comment)
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421596699
that
is what I'm looking for.
On Sun, Sep 16, 2018 at 12:41 AM, Muaz Khan<EMAIL_ADDRESS>wrote:

What if we share & sync PDF separately? I mean, we can open a popup in the
index.html demo to embed PDF and sync pages/zoom/etc. in realtime?
Please track this thread for PDF syncing: muaz-khan/RTCMultiConnection#628
(comment)
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421596699
PS. If its easier then I'll embed PDF inside the main-canvas as well (same
like images-tool).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/muaz-khan/Canvas-Designer/issues/43#issuecomment-421597084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnlPnFRGfCePZm7C8PL2u6zHcJFCfYd0ks5ubS3LgaJpZM4VKo_o
.

I'm going to wait for the upload feature mr muaz-khan
.
THANK YOU SO MUCH!
On Sun, Sep 16, 2018 at 12:47 AM, Benedict Aquino<EMAIL_ADDRESS>wrote:

Yes sir that is what I'm going to do. To embed the pdf to canvas and can
draw to PDF.
Because I'm currently developing a VIRTUAL CLASSROOM sir. My inspiration
is https://www.braincert.com/try-virtual-classroom . that is what I'm
going to be.
And I using your library which is very helpful. Please help me Mr.
muaz-khan
AGAIN THANK YOU SO MUCH!!! The muaz-khan/RTCMultiConnection#628 (comment)
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421596699 that
is what I'm looking for.
On Sun, Sep 16, 2018 at 12:41 AM, Muaz Khan<EMAIL_ADDRESS>wrote:

What if we share & sync PDF separately? I mean, we can open a popup in
the index.html demo to embed PDF and sync pages/zoom/etc. in realtime?
Please track this thread for PDF syncing: muaz-khan/RTCMultiConnection#628
(comment)
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421596699
PS. If its easier then I'll embed PDF inside the main-canvas as well
(same like images-tool).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/muaz-khan/Canvas-Designer/issues/43#issuecomment-421597084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnlPnFRGfCePZm7C8PL2u6zHcJFCfYd0ks5ubS3LgaJpZM4VKo_o
.

Hi MR muaz-khan Good day!!!
When I'm going to see the PDF SYCNING function? I'm so very excited for
this!!
THANK YOU SO MUCH!!!


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-2.x" 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 2368

Questions ?
Please refer to the Backport tool documentation


Unable to run_low_mem( ),  VideoSys/examples/cogvideox /sample.py in T4 Colab

Getting OOM error. Please help.

OutOfMemoryError                          Traceback (most recent call last)
 in <cell line: 2>()
1 config = CogVideoXConfig("THUDM/CogVideoX-2b", cpu_offload=True, vae_tiling=True)
----> 2 engine = VideoSysEngine(config)
3 prompt = "Sunset over the sea."
4 video = engine.generate(prompt).video[0]
13 frames
/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py in convert(t)
1148                 return t.to(device, dtype if t.is_floating_point() or t.is_complex() else None,
1149                             non_blocking, memory_format=convert_to_format)
-> 1150             return t.to(device, dtype if t.is_floating_point() or t.is_complex() else None, non_blocking)
1151
1152         return self._apply(convert)
OutOfMemoryError: CUDA out of memory. Tried to allocate 30.00 MiB. GPU 0 has a total capacity of 14.75 GiB of which 7.06 MiB is free. Process 16497 has 14.74 GiB memory in use. Of the allocated memory 14.19 GiB is allocated by PyTorch, and 456.84 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation.  See documentation for Memory Management  (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)

fixed in pr #194. it should work with t4 now.
if you still go oom. you can change the vae tiling args to use less memory here:
https://github.com/NUS-HPC-AI-Lab/VideoSys/blob/94e54cd4aadd35b817b37aed637a702a15786916/videosys/pipelines/cogvideox/pipeline_cogvideox.py#L174
an example is:
vae.enable_tiling(tile_sample_min_height=vae.tile_sample_min_height // 2)

Thanks for your support. I used this
vae.enable_tiling(tile_sample_min_height=vae.tile_sample_min_height // 2)
But I am still getting oom. But now it says ' You session crashed after using all available RAM'. So now the oom is on RAM instead of VRAM.

Ok thanks.


Re-add Option and Result.

Reverts #63, now that https://github.com/FuelLabs/sway/pull/815 is in.

@adlerjohn Are there any blockers for this, or can we wrap it up and merge soonish?

We have to wait for v0.7.0 of forc, which can be cut after the above PR is merged.

@adlerjohn can we move ahead on this with the release of Forc v0.7.0 ?

It isn't possible to instantiate a Result, so impossible to test. Still waiting on https://github.com/FuelLabs/sway/issues/814 after all!

These tests still need to be implemented: https://github.com/FuelLabs/sway-lib-std/blob/3be26db4d1bc4d133135243fea2126f211fc11bc/tests/test_projects/result/src/main.sw#L11-L25

So Emily's recent changes fixed the problems with instantiation?


Is there any way to increase transition speed?

Yes, default one is 300 ms. You can change duration like below
Navigator.push(context, PageTransition(type: PageTransitionType.rotate, duration: Duration(milliseconds: 100), child: DetailScreen()));


update docker and partial fixes on the notebooks

Description

Update Dockerfile to mount contrib along with examples
Fix the dependencies in Docker-Compose
Some stylistic changes on the notebooks

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Codecov Report
Attention: 35 lines in your changes are missing coverage. Please review.

Comparison is base (34830a7) 80.33% compared to head (6139d96) 67.14%.
Report is 129 commits behind head on main.

Files
Patch %
Lines

superduperdb/backends/dask/compute.py
47.05%
18 Missing :warning:

superduperdb/backends/base/compute.py
68.96%
9 Missing :warning:

superduperdb/backends/base/query.py
85.71%
3 Missing :warning:

superduperdb/backends/base/metadata.py
66.66%
2 Missing :warning:

superduperdb/backends/base/artifact.py
66.66%
1 Missing :warning:

superduperdb/backends/base/data_backend.py
66.66%
1 Missing :warning:

superduperdb/backends/ibis/data_backend.py
87.50%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1412       +/-   ##
===========================================
- Coverage   80.33%   67.14%   -13.20%
===========================================
  Files          95      103        +8
  Lines        6602     7272      +670
===========================================
- Hits         5304     4883      -421
- Misses       1298     2389     +1091

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


map.flyto zoom level that marker is no longer in cluster.

This is a feature request. I've only just found this plugin but I have had a look around and can't find the feature. Sorry if it already exists.
.flyto(latLng, zoomLevel) currently zooms to a predefined zoom level zoomLevel. It would be great if I could find the zoom level of marker where it is no longer in a cluster so I can enter that into the .flyto(latLng, zoomLevel) function.
Thanks for your time :)

@7ammer You can use the .zoomToShowLayer() method to do this. Remember that this is a method of the markerClusterGroup object and NOT the map object. For more info, refer to the documentation under 'Other Group Methods'. You can also read my answer to a simliar question on StackOverflow. Note that this function pans to your marker, and does not do the cool flyTo animation that you are looking for. So, if you absolutely need to get that smooth pan zoom animation in flyTo, then continue reading below.
I recently came across this exact problem myself and I was able to solve this with a dirty fix that is definitely not the official way to do it, so, if someone knows of a better way, please mention it.
The way I got the zoomLevel at which a marker is visible, is by first getting access to the parent cluster of said marker. Next, I get the bounds of this cluster using the cluster's .getBounds() method (provided by markercluster library). Once I have the bounds of this cluster, I can simply get the required zoom level use the map object's .getBoundsZoom() method (provided by the original leaflet library).
// Assuming `myMarker` is your marker object and `myMap` is your map object

const clusterBounds = myMarker.__parent.getBounds();
const zoomLevel = myMap.getBoundsZoom(clusterBounds);

myMap.flyto(latLng, zoomLevel);

As you can see, accessing the .__parent property of a marker is the only "dirty" part of this otherwise okay approach. While there is a method called .getVisibleParent(), I was not able to find a way to get the immediate parent of cluster.

@waseemyusuf even better solution:
const parentCluster = marker.__parent;
const zoomLevel = Math.min(map.getMaxZoom(), Math.max(map.getZoom(), parentCluster._zoom + 1));

map.flyTo(marker.getLatLng(), zoomLevel);


Minor typo in Section 12.4.1, page 509, version 2023-08-15

In the end of the second paragraph, "correpondence" should be "correspondence".

fixed


[Documentation]: Add documentation - Upgrade related terms are not explained in the the testcase descriptions

Describe the bug
It is not clear from the testcase descriptions

What is the difference between rolling_downgrade and rolling_version_change?
What is the difference between rolled back rollback and rolling_downgrade?

Description in rationale is unclear.
To Reproduce
Steps to reproduce the behavior:

Go to testcase descriptions
Read the test case descriptions

Expected behavior
Test case descriptions explain what are the differences.

Basically, if I understood it correctly these tests ("rolling_update", "rolling_downgrade", "rolling_version_change") have simillar implementation, but test tag decides direction (rolling_update_test_tag -> new version, rolling_downgrade_test_tag -> older version). In that case is rolling_version_change even needed? In what context? Based on https://github.com/cnti-testcatalog/testsuite/blob/main/docs/LIST_OF_TESTS.md#rollback , it looks like as rolling_version_change is just a copy of rollback_downgrade test.

The description of what these tests are doing is still not clear https://github.com/cnti-testcatalog/testsuite/blob/main/docs/LIST_OF_TESTS.md#rollback
If they are doing different things and it is suggested to test both (rather than only keeping a subset) then the documentation should be updated to make things clear.

@taylor is this related to https://github.com/cnti-testcatalog/testsuite/issues/1198 and https://github.com/cnti-testcatalog/testsuite/issues/1316?

IMO, current documentation explains the difference between tests better that it was at the moment of this issue creation (although it could be done a bit clearer).
I think, that we need to redesign rolling_version_change and all related tests, and trying to clarify the documentation before that doesn't seem like a reasonable direction of efforts. Probably, the issue could be closed.


fix(Sonos): Streamline device initialization process

The initialization routine opted for a pretty heavy-weight eventual consistency model with a bunch of repeated/duplicated checks on speakers as they onboarded. It was also doing an extra round or two of re-scanning.

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

I'm closing this PR as the onboarding process for Sonos has diverged quite a bit. There are still opportunities to optimize it, but they would look very different now.


"Slatina", "Găinești" and "Herla" villages are listed in the drop-down menu.

Description:
The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions:
The website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Slatina".

Expected results
The location is listed in the drop-down menu as "Sat Slatina SUCEAVA (Slatina)".

Step 2
Write in the search bar "Găinești".

Expected results
The location is listed in the drop-down menu as "Sat Găinești SUCEAVA (Slatina)".

Step 3
Write in the search bar "Herla".

Expected results
The location is listed in the drop-down menu as "Sat Herla SUCEAVA (Slatina)".


regarding re-train the VGG19 model

Hi,
Excuse me, I have to open a new thread regarding using your code posted in this thread.  Thanks.
Right now, I have 200 pairs of images for training. Each training image is of black and white, i.e., one channel only.  The mask image for training pair is also of size 256*256, each of its pixel corresponds to one pixel in the original image.
In the code of setting the training data set,
X = np.random.uniform(0,256, (2000, 3, 256, 256))
X = preprocess_input(X) # VGG preprocessing
Y = np.around(np.random.uniform(0,1, (2000, 1, 64, 64)))

I have three questions,

does VGG19 model accept input with one channel only?
In the code, Y was setup of having size of 64*64 due to two pooling layer. However, the mask image is of size 256*256. How can I map these 256*256 mask image to 64*64?
Here you specify to freeze the first two blocks of the VGG19 by

for i in range(7):
    model.layers[i].trainable = False

Just curious, why we have to freeze the first two blocks instead of the first block? Are there any specific reason underlying this? Or generally if we only want to re-tune the model weights, it is a normal practice to change the top layer only. Is this understanding right?
Thank you very much.

Hi,
Don't hesitate to ask if one answer is unclear :

short answer: vgg accept only RBG images so 3 channels. Long answer: you could try some black magic (like transform the one channel image into a RGB image but only with shades of grey, or changing the first layer of convolution of the vgg to conver the kernel to adapt to the formula of RGB->grey) but I strongly advise against because the vgg is used to use colors to take decisions. Grey images will confuse it.
This is a very open question. Since you can't use the vgg, I'd recommend not using poolings at all. This way, you won't have issues. This is the simplest way and should work fine.
Freezing blocks is a way of preventing overfitting. It removes parameters to learn from the model and make it less flexible.  So you should freeze more layers if you have little data, and freeze less if you have lots of data. It also depends on the closeness to you problem with the original problem. For example if you had to find if a picture is the one of a dog or a cat, you'd freeze a lot of layers. If you have to find what's the type of lightning used in a picture, you'd need to freeze very few layers since imagenet competition isn't about that at all. Training one layer is the good default when the two problems

Sorry bad manip.

When the two problems are close.
I hope I could be of some help.
If I might add one last thing, keep your network small (32 neurons per layer max) and use and abuse of data augmentation. This is because you seem to have a small dataset. Overfitting will likely be an issue.
Good luck,
Gabriel.

Gabriel, thank you so much for your detailed reply.
In accordance with your answer, it might not be a good idea for me to directly use VGG, since it relies on RGB image.  However, I can borrow the basic design of VGG, by removing the pooling layers. This way I can get output with the same size of mask image.  Since it is a new architecture, I generally have to re-train the model. Is my understanding correct?
Thanks,
surfreta

Yes. This is it. Don't forget to remove the flatten and dense layers too. It should look like this:
InputLayer -> Conv2d - > Conv2d - >... ... ... - > Conv2d. With padding="same" at each convolution layer, you should have the input size equal to the output size.
You're welcome.
Gabriel


support for number constants in specs

We have a few constants here and there in our specs that are not captured and code generated. I'm proposing to add syntax for numerical constants.
    constant: magic= 0x42

I think this is a good idea, why close?

I would rather use const magic = 0x42 in the code section though, to align with enums

it would be good to prioritize this so that we can move all the magic constants of the protocol in the spec and code-generate them for any environment. @mmoskal

Implemented as "define foo 0


serial: add tests for scheduling pod with nodeAffinity

Add new tests checking that deployment pods with nodeAffinity specified are scheduled properly on the feasible single numa node even though the resources are available on node level on two compute nodes.

to move forward, let's make sure we unlabel on error path (if it is already the case, let's explain in the comments); after that, let's review the code organization proper, which bite us already. Everything else is minor and not blocking.

/lgtm /hold thanks for the updates! has this been verified against a cluster? if so, feel free to remove the hold

yes, the tests were tested and they passed as expected

/lgtm /hold thanks for the updates! has this been verified against a cluster? if so, feel free to remove the hold

yes, the tests were tested and they passed as expected

awesome
/hold cancel


[No QA] Conversion to Boolean

Details

Small change needed as ReportDraftComment could be '' which could lead to App Crash as we expect true|false here. https://github.com/Expensify/Expensify.cash/pull/3865#issuecomment-875412214

Changes done and tested.


Add Ructe crate

Fix for #78 - adds the Ructe crate to the templating section.

Thanks for the contribution!


Please deprecate this package OR add a notice to the ReadMe OR add maintainers

Dear @Automattic,
I ask courteously and with huge respect for both you and Kue, but please do one of the following for this project:

deprecate this package in npm (https://docs.npmjs.com/cli/deprecate)
add a note to the top of the ReadMe such as "Kue is not part of any of our current projects, and is not being actively maintained." (https://twitter.com/automattic/status/1019073146446602240)
add maintainers so that the people using it can fix problems, add updates, and perform releases (though I understand this might not actually be the best course of action)

Another nicety for (1) and (2) would be to point people to other Redis-based queue projects for Nodejs, such as Bull (https://www.npmjs.com/package/bull), which is actively maintained.
Kue is both a widely used package and one that commands a lot of respect having been around for a while. I remember it in it's infancy in 2011 having been around node back then.
Recently, having spent a fair bit of time evaluating both Kue and Bull it wasn't until I starting looking at the repo for more info that I realised that this project was inactive. I appreciate the trail Kue made in nodejs where others have followed, but it is no longer fair to just leave the project as-is.
Seeing issues such as "Is this repo maintained?" (https://github.com/Automattic/kue/issues/185) created in 2013 gives an indication that users have been wondering for a very long time now. Perhaps it's time to bring these questions to a conclusion.
Having written mongodb-queue (https://www.npmjs.com/package/mongodb-queue) and many other node packages, I understand what it is like to write it, use it, then no longer use it and I understand your pain in not wanting to do any of the above, which would effectively seal the fate of the project. I fear delaying the decision any more however is detrimental.
I have no preference and no personal buy-in to this project or others (yet) but I think it is fair of you to do any of the above options and there may be others. I'm just trying to save time for people who , like me, spent time trialling an inactive project.
I'd love to hear from you on behalf of your users and for potential users who are evaluating using Kue. Many thanks for all of your hard work over the years in both creating (indirectly) and maintaining Kue - it's a thankless task, hence my thanks above - but hopefully we can draw some of these questions to a conclusion.
Thanks again,
Your Users and Potential Users

Looks like @simison now has a PR which is essentially option 1 : https://github.com/Automattic/kue/pull/1231

add maintainers so that the people using it can fix problems, add updates, and perform releases (though I understand this might not actually be the best course of action)

This would be lovely — just requires people from the community to stand up and actively start working on Kue. Triaging open issues and PRs would be a great way to start. :-)

I will be backing everyone who is capable of being a new maintainer of Kue

Is it still open for discussion? or is the deprecation is final? Are there any active forks?

@gkorland are you suggesting you'd like to volunteer as active maintainer?  :-)

I can do my best, and I know other who would like to join too

@simison will you accept contributions?


throw bad request exception when validation fails

This  PR is to handle issue https://github.com/hwslabs/iam/issues/51
Description : - Idea is to respond back status as 4.. on validation failure of request parameters. Following statusPage will auto map it to relevant http status code.
sendStatus<BadRequestException>(HttpStatusCode.BadRequest)

I missed running test cases, which is why action is failing. Allow me sometime to fix that.

I missed running test cases, which is why action is failing. Allow me sometime to fix that.

Sure. Thanks for raising PR. I added some comments. Please check and reply back if you have any questions. We are also available in slack and we can discuss there as well.

@hari-hypto
Github Action failing with following message
Push to branch badges
remote: Permission to hwslabs/iam.git denied to github-actions[bot].
fatal: unable to access 'https://github.com/hwslabs/iam.git/': The requested URL returned error: 403
Error: Invalid exit code: 128
    at ChildProcess.<anonymous> (/home/runner/work/_actions/ad-m/github-push-action/master/start.js:29:21)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1022:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:28[7](https://github.com/hwslabs/iam/runs/5752370899?check_suite_focus=true#step:9:7):5) {
  code: 12[8](https://github.com/hwslabs/iam/runs/5752370899?check_suite_focus=true#step:9:8)
}
Error: Invalid exit code: 128

Push Commit Badge Github Action failing with following message

Seems to be a permission issue. Looking into it. We will get back.

The change looks good to me. One minor comment is to add this validation check even for the "AdminUser" password and phoneNumber when creating organization.

sure @hari-hypto.
I've added validations for admin details like (password, phone, username, email). And fixed failing test cases after adding validations

@hari-hypto There was code smell with wildcard import causing github action to fail.
Request you to trigger build action again.

@hari-hypto There was code smell with wildcard import causing github action to fail. Request you to trigger build action again.

Thanks for fixing it. Pushed the PR changes to main branch.


⚠ Bofrost.nl has degraded performance

In eec5ce7, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1010 ms

Resolved: Bofrost.nl performance has improved in a172bfd after 6 minutes.


Error 'apksigner' returned with error #1

Godot version
3.5.stable.official [991bb6ac7]
System information
Windows10, GLES3, AMD RYZEN 3 3700x, AMD R9 380
Issue description
Hi, i'm trying to export my project to android platform. I do have seen multiple tutorials and read multiple documentations files about setting it up. Below my setup:
Godot:

JDK:

Android SDK:

That's the output when i try to export the project:

Steps to reproduce
Create a basic project and try to export is for android platform
Minimal reproduction project
No response

Are you launching Godot through Steam? See https://github.com/godotengine/godot/issues/62768.

Are you launching Godot through Steam? See #62768.

Nope, no steam

Maybe I know what's wrong guys!
First of all i go to the android's build-tools folder
Your_Android_SDK_FOLDER\build-tools\30.0.3\
then i found the 'apksigner.bat' file, opened it and started to change, one at the time the exit codes, until I found what's the error that i get, see pic below

Long story short:
ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%  Please set the JAVA_HOME variable in your environment to match the location of your Java installation.
I try to solve this problem then update you. Sorry for my bad english guys

Maybe I know what's wrong guys! First of all i go to the android's build-tools folder Your_Android_SDK_FOLDER\build-tools\30.0.3\ then i found the 'apksigner.bat' file, opened it and started to change, one at the time the exit codes, until I found what's the error that i get, see pic below

Long story short: ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% Please set the JAVA_HOME variable in your environment to match the location of your Java installation.
I try to solve this problem then update you. Sorry for my bad english guys

Yeah guys, it was a bad case of JAVA_HOME path gone wrong. Close this issue and sorry for bothering you. Have a nice day! YUPPIE!!!

So what was wrong? I have the same problem and I can't solve it

I followed the same path and found that the error occurs at this location. It then goes to : note and throws an error from "shift / 1" where "set params". Java experts, tell me, what is causing the error?


Abstract out machine configuration

Currently, the framework is tightly coupled to the use of the framework. These should ideally be separated into different repositories, with the framework using a particular part of the user's services to read their configuration from there and a convenient install script that wraps things like nixos-generate-config to generate a configuration that's built with this framework.
Yeah, this is an abstraction over Nix, but it would be super useful for me when setting up new machines -- it could set up the opinionated service basics already!

This is dependent on #37 ; I don't want to introduce a configuration framework without first documenting the configuration that I've already assembled.

This is dependent on #37 ; I don't want to introduce a configuration framework without first documenting the configuration that I've already assembled.

What is this


Mongoid: .near doesn't work with scopes

Environment

Ruby 1.9.2
Rails 3.1.0.rc5
MongoDB 1.8.2
Mongoid 2.0.2

.near works as expected when called against a model:
> Item.near('London', 50)
=> #<Mongoid::Criteria
  selector: {:coordinates=>{"$nearSphere"=>[-0.1262362, 51.5001524], "$maxDistance"=>0.012630230737833089}},
  options:  {},
  class:    Item,
  embedded: false>

However, the following error occurs if I call .near against a scope:
> i = Item.all
=> #<Mongoid::Criteria
  selector: {},
  options:  {},
  class:    Item,
  embedded: false>

> i = i.near('London', 50)
ArgumentError: wrong number of arguments (2 for 1)
	from /usr/local/Cellar/ruby/1.9.2-p180/lib/ruby/gems/1.9.1/gems/mongoid-2.0.2/lib/mongoid/criterion/inclusion.rb:137:in `near'
	from (irb):7
	from /usr/local/Cellar/ruby/1.9.2-p180/lib/ruby/gems/1.9.1/gems/railties-3.1.0.rc5/lib/rails/commands/console.rb:45:in `start'
	from /usr/local/Cellar/ruby/1.9.2-p180/lib/ruby/gems/1.9.1/gems/railties-3.1.0.rc5/lib/rails/commands/console.rb:8:in `start'
	from /usr/local/Cellar/ruby/1.9.2-p180/lib/ruby/gems/1.9.1/gems/railties-3.1.0.rc5/lib/rails/commands.rb:40:in `<top (required)>'
	from script/rails:6:in `require'
	from script/rails:6:in `<main>'

It appears that the .near method from geocoder is being replaced by the standard Mongoid method.

I ran into this the other day.  If you look at the source you can see that the Geocoder .near method is only applied on the base document, not on scopes derived from that document.  To get around it barring a real fix you simply need to ensure that your Geocoder .near is always the first scope in your chain.  This was a bit ugly in my case but it worked...  In other words:
Item.awesome.near #fails
Item.near.awesome #works

Is the geocoded model embedded?
I believe that error is from calling Mongoid::Criteria#near which has a totally different signature than the geocoder near method. Additionally, the Mongoid method does not do any $maxDistance calculations.

It seems like there is a conflict between the geocoder scope :near (found here) and mongoid's method near (found here)
Is there a way to reconcile the two?

See #832.


Phrase push doesn't support "tags"

The old CLI tool allowed you to pass a "--tags" argument to add tags to translations pushed with this parameter.
Expected behavior:
phraseapp push --tags=foo,bar

Attaches one or multiple tags with your file upload. All keys added by the push will be tagged with the specified tags.

Hi @rubenvdb, this is correct. Please use the new functionality phraseapp upload create for this as described here: http://docs.phraseapp.com/api/v2/uploads/
Best, Frederik

Hi @rubenvdb, as @fredv noted this is correct. If you want to see what arguments are supported do phraseapp push --help. push / pull / init are special commands that solely rely on a .phraseapp.yml. If you would like to upload locales and use --tags as an option, use phraseapp upload create. See here for more information on push/pull:
http://docs.phraseapp.com/guides/working-with-phraseapp/command-line-client/
If that fixes the issue for you I would like to close it.
Regards,
Matthias

Hi @sacry-dyn ,
Although I understand the use of the upload command, I still can see benefit in the possibility of adding an additional tag when pushing new keys. We used the tags flag to add a tag containing the git sha1 of the commit in which the keys were added. In complex syncing situations, the push command is easier to use (because it reads the config from the file).
That being said, I would understand you closing this issue because it is an exceptional use case.

Hi @rubenvdb
we will keep the request in mind though. We want to offer as many options as possible. push and pull however can easily take 8+ command line arguments so to keep it simple we restrain from adding cmd arguments for now. If the need arises we are certainly open to consider a few command line options that enhance the interactive experience like tags or update_translations.

I’m sorry, but what would be the pull equivalent for only pulling translations for certain tags only? With the deprecated phrase gem, this would have been phrase pull de --tag=feature-messenger-v2 for example.
This was probably referred to by @sacry-dyn here, but the link referred 404s. It’s probably good to have it answered here in the issue, anyways.

Hi @j4zz, there is a new command, called "phraseapp upload create" which supports tags. Also, inside a push source, you can specify the same params upload create takes for the source file you're trying to tag - if this is supposed to happen on every push. For singular import, use upload create, for regular pushes add the params as described here: http://docs.phraseapp.com/developers/cli/configuration/
Example:
push:
-
file: "en.feature.yml"
params:
tags: "feature,other-tag"


Help - Insert data into cell based on conditions

Hi,
I am trying to insert data into the selected cell based on condition matched on another cell.
Attaching a simple excel for example.
Conditions like:
If TargetDate is not null and Task number is empty , then insert comment "Enter Task Number"
If TargetDate is lesser than Last Modified Date, Then insert Comment " Please reschedule"
if TargetDate is between 12/10/2017 and 12/12/2017 , then insert Comment " Take action"
I don't know where to start. Conditional formatting targets to text. but how to do validation and get the cell index and insert data in different column, row index.
I guess, I may be able to achieve this using com object. but checking if there is a better method here.
Test.xlsx

It's fairly easy if this is going down a column.
In one of my scripts I have
Set-ExcelColumn -Worksheet $ws -Heading 'ID'       -Width 15 -Value {($WorkSheet.Cells[$Row,5].value -replace '^<.*?;','')-replace ':.*$',''}
So here I am adding a column on the right, 15 wide - the key thing is that VALUE is a ScriptBlock.
in the Script block you can refer to $workSheet (and the properties of a work sheet) $Row (which is the row number) and  $column [number], $columnName [letter(s)], $startRow, $startColumn, $endRow, $endColumn.
So you can write a script block
$sb = {
if ("" -eq $worksheet.Cells["A$Row"].value  -and "" -eq -eq $worksheet.Cells["A$Row"].value -eq $worksheet.Cells["A$Row"].value) {"Enter task Number"}
elseif { <<whatever>>   }
<<etc>>
}

(assuming target date is column A and Task number is in column B)
Then use $sb as the value in a set-ExcelColumn
If you want comments you need to a for loop down rows and set the comment property. We'll have support for comments at some point in the future.

Thanks. I was thinking to convert into datatable and insert it and then export it back.. you saved my time. :)
But seems something I am doing wrong.

second line sets $sb based on value of A3
Value in the third line is a constant which is based on that and D3.  Try:
$excel = Open-ExcelPackage "$ENV:Userprofile\Desktop\Newfolder\test.xlsx"
Set-ExcelColumn -ExcelPackage $excel -Worksheetname "Sheet1" -Value {(
  if ($null = $excel.Sheet1.Cells["A$Row"].value ) {"Enter Task Number"}
  <<etc.>>
}

Not working. Something is wrong.
Why do you reference me A$Row ?? Are you trying to tell me that I will be looping through row by incrementing $row.
Lets make it simple. if A3 is null, then "Enter Task number" string in D3.
$excel = Open-ExcelPackage "$ENV:Userprofile\Desktop\Newfolder\test.xlsx"
Set-ExcelColumn -ExcelPackage $excel -Worksheetname "Sheet1" -Value {(if ($null = $excel.Sheet1.Cells["A3"].value ) {"Enter Task Number"})}
Close-ExcelPackage -ExcelPackage $excel -Show
The statement above doesn't make any sense to me where to insert D3.
in comments above, Yes, I need $sb set based on A3, A3 is null, and hence I want to put comments on D3 cell .. however it is choosing some random column I,J,L and fills the string to count of row. looks like a bug to me. :(

It isn't saying insert into D3.
It is saying go right until you find an unused column.
Then from the top of that column to the end of the spreadsheet insert "Enter task number" in every cell if A3 is null, and insert nothing in every cell if A3 has contents.
I told you to write A$Row, because the script block can see variables when it . Really I should have told you to use $workSheet, because that is one of the variables that the script block can see, but $Excel is also accessible in a higher scope so it will still work.
Inside Set-Excel Column it loops through the cells to set in the column and $Row is the index of the row,  $column is the Column number and $columnName is the column as letters; $workSheet is the sheet being worked on, plus $StartRow/Column and $endRow/Column which gives you the corners of the sheet.
So First write your script block, knowing that those variables are available.  ($sb for Script Block)
$MyScriptBlock = {
    if ("" -eq $worksheet.Cells["A$Row"].value  -and
        "" -eq $worksheet.Cells["B$Row"].value) {"Enter task Number"}
    elseif ( <<whatever condition>> )  { <<whatever text>>   }
}

Set-ExcelColumn -Worksheet $ws -Value $MyScriptBlock

BTW, you can create your own loop and set these values directly on the worksheet object. The you have something which steps through 2,3,4 to the end checking the value of A in the current row, and setting the value of D in the current row .

I really don't know if you understood what I am trying to do.
$excel = Open-ExcelPackage "$ENV:Userprofile\Desktop\Newfolder\test.xlsx"
$worksheet = $excel.Workbook.Worksheets.Item(1)
$MyScriptBlock = {    if ("" -eq $worksheet.Cells["A3"].value) {"Enter task Number"}  }
Set-ExcelColumn -Worksheet $worksheet -Value $MyScriptBlock
Close-ExcelPackage -ExcelPackage $excel -Show
Firstly, "" -eq $worksheet.Cells["A3"].value outputs to Falls.
$null -eq $worksheet.Cells["A3"].value outputs to True.
test.xlsx
Ok, considering I put $null and still use your snippet. I am getting the same output as I got before.
$excel = Open-ExcelPackage "$ENV:Userprofile\Desktop\Newfolder\test.xlsx"
$worksheet = $excel.Workbook.Worksheets.Item(1)
$MyScriptBlock = {    if ($null -eq $worksheet.Cells["A3"].value) {"Enter task Number"}  }
Set-ExcelColumn -Worksheet $worksheet -Value $MyScriptBlock
Close-ExcelPackage -ExcelPackage $excel -Show

And where are you mentioning cell address of the the string "Enter Task Number" should insert to. ?
Forget multiple conditions for now for my understanding, Please.

As long as you keep putting 3 every cell  will do what cell A3 says it should do.  Try putting $row where you have 3, and see what happens.

:) if i put $Row i get below error message.  I tried passing value $Row =3 with and without it, same error.
Write-Verbose : Cannot bind argument to parameter 'Message' because it is null.
At C:\Users\PavithrasagarK\Documents\WindowsPowerShell\Modules\ImportExcel\5.4.0\Set-Column.ps1:154 char:46
+                  Write-Verbose  -Message     $cellData
+                                              ~~~~~~~~~
    + CategoryInfo          : InvalidData: (:) [Write-Verbose], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Microsoft.PowerShell.Commands.WriteVerboseCommand

$excel = Open-ExcelPackage "$ENV:Userprofile\Desktop\Newfolder\test.xlsx" -Verbose
$worksheet = $excel.Workbook.Worksheets.Item(1)

$MyScriptBlock = {
    if ($null -eq $worksheet.Cells.item(3,1).value) {$worksheet.Cells.item(3,4).value = "Enter task Number"}
}

Set-ExcelColumn -Worksheet $worksheet -Value $MyScriptBlock
Close-ExcelPackage -ExcelPackage $excel -Show -Verbose

Ok If I use $worksheet.Cells.item(3,1).value instead of Cells["A$Row"] its working fine. Still I get the error in Set-excel Column as mentioned above.
Output Below:

Cannot bind argument to parameter 'Message' because it is null
Something wrong with binding  Write-Verbose  -Message  $cellData ...
Beyond my level of knowledge though :). Or may be its my syntax issues.
As usual, Glad I reached to a point where I could get the data inserted. I may be able to suppress this error in my script.

OK. Deep breath.

One of the errors is being caused by something trying to insert null into a cell. I need to look at that.
It is not writing to a random column. When Set-Column is called without a column number it writes to the first unused column. Every time you run against the same sheet it will move one column further to the right. You would be better to specify a column.
You want to get some data where column A will be null in some rows, but you don't know which ones.  Saying you want to put something in row 3 only works with your sample data.
So you either have to write a loop

for x = 2 to LastRow.
  if Cell[1, X] isnull then cell [4,x] = "Message"
Next x

or you can fill in something which says
Fill column 4 with ( "message" if cell[A,currentrow] on this row is null )

Thank you.. I got the loop part covered. Just wanted to make sure the
simplest hard coding, work first and then build loops on top of it..
On Mon, Nov 26, 2018, 08:44 jhoneill<EMAIL_ADDRESS>wrote:

OK. Deep breath.

One of the errors is being caused by something trying to insert
null into a cell. I need to look at that.
It is not writing to a random column. When Set-Column is called
without a column number it writes to the first unused column. Every time
you run against the same sheet it will move one column further to the
right. You would be better to specify a column.
You want to get some data where column A will be null in some rows,
but you don't know which ones. Saying you want to put something in row 3
only works with your sample data.
So you either have to write a loop

for x = 2 to LastRow.
if Cell[1, X] isnull then cell [4,x] = "Message"
Next x
or you can fill in something which says
Fill column 4 with ( "message" if cell[A,currentrow] on this row is null )
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/497#issuecomment-441643336,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aq2jYZCqIJSVvjWJgH9n4xI33bqMJARGks5uy_A7gaJpZM4YxRyq
.

Sorry to keep bothering you.  just FYI,  even when I specify column in Set-ExcelColumn , it still seems to be writing it to unused columns causing the excel size to grow :) However, For now, I was able to bypass the issues and workaround other options.  Below code is working for my needs.
$excel = Open-ExcelPackage "$ENV:Userprofile\Desktop\Newfolder\test33.xlsx" -Verbose
$worksheet = $excel.Workbook.Worksheets["Sheet1"]
$Rows=$worksheet.Dimension.Rows

for($row = 2; $row -lt $Rows; $row++){
if ($null -eq $worksheet.Cells.item($row,1).value) {$worksheet.Cells.item($row,4).Value = "Enter task Number"}
}

Set-Format -Range $worksheet.Cells["D2:D$Rows"]  -FontColor 'Red'
Close-ExcelPackage -ExcelPackage $excel -Show

If I run

del C:\temp\test2611.xlsx

$excel = @"
Task Number,Last Modified Date
Task1, "11/11/18"
, "10/10/18"
Task3, "10/10/18"
Task4, "10/10/18"
"@ | ConvertFrom-Csv | export-excel -path C:\temp\test2611.xlsx -WorksheetName sheet1  -PassThru

Set-ExcelColumn -Worksheet $excel.sheet1 -Column 5 -Value {if (-not $worksheet.cells["A$row"].value) {"Enter task no."} else {"-"} } -FontColor Red

Close-ExcelPackage $excel -Show

I get

If I close the file and run
$excel = Open-ExcelPackage -path C:\temp\test2611.xlsx
Set-ExcelColumn -Worksheet $excel.sheet1 -Column 5 -Value {if (-not $worksheet.cells["A$row"].value) {"Please Enter task no."} else {"."} } -FontColor Red
Close-ExcelPackage $excel -Show

It rewrites column 5

OK. I did explore and see set-excelcolumn requires else{''} to avoid Null message at this point of time.
Anyways, No worries on that, You already mentioned that you will look into it.
I do not understand how you guys managed to capture $row field without me passing on the information to it. And now I understand by what you meant it's easy looking at your first comment. :) :)
It wasn't easy for me. Lol. Once Again, Appreciate all your support.
Will get back for the next help or issue. Closing this issue for now.

When you pass a script block it can refer to variables which will exist in the context that runs in
So I can do
$MySb = {Write-host $wibble}
Wibble doesn't exist yet, and if run  & $sb there is no output
but later I can do
$wibble = "Hello world"
& $sb

And it outputs "Hello world".
So... if you know what variables will exist at the time the Script block is executed, then you can refer to them. And we tell you
>get-help Set-ExcelColumn -Parameter Value

-Value <Object>
    A value, formula or scriptblock to fill in. A script block can use $worksheet, $row, $column [number], $columnName [letter(s)], $startRow, $startColumn, $endRow, $endColumn.

Thanks for the detailed information.. Yeah, times I see examples but I tend
to not read the descriptions.
On Tue, Nov 27, 2018, 05:58 jhoneill<EMAIL_ADDRESS>wrote:

When you pass a script block it can refer to variables which will exist in
the context that runs in
So I can do
$MySb = {Write-host $wibble}
Wibble doesn't exist yet, and if run & $sb there is no output
but later I can do
$wibble = "Hello world"
& $sb
And it outputs "Hello world".
So... if you know what variables will exist at the time the Script block
is executed, then you can refer to them. And we tell you

get-help Set-ExcelColumn -Parameter Value

-Value
A value, formula or scriptblock to fill in. A script block can use $worksheet, $row, $column [number], $columnName [letter(s)], $startRow, $startColumn, $endRow, $endColumn.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/issues/497#issuecomment-442016752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aq2jYSCU2s8B5RatipfZpsbG2i7RFo47ks5uzRrWgaJpZM4YxRyq
.


⚠ Bofrost.lu has degraded performance

In ac80831, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 857 ms

Resolved: Bofrost.lu performance has improved in db0455c.


lmdb: Enable MDB_USE_PWRITEV on Linux

This is a small optimization, which turns a seek + writev into a pwritev syscall on Linux.

Oh interesting. I didn't realize that this system call was guarded by a #define.
Are there times when pwritev will actually hurt performance? It doesn't sound like it would. But I don't actually have much experience with the call. And right now I'm not sure why it would be placed behind a flag.
One of the few times I have seen the call discussed is in https://github.com/boltdb/bolt/issues/238. But that seems more like it wasn't implemented due to constrained resources than potential issues.
I will benchmark, look up anything I can find it the mailing list, and get back here.
Thanks

I don't really see a change in performance. Do you actually see any change if you run the benchmarks or in any programs you have?
benchmark                              old ns/op     new ns/op     delta
BenchmarkEnv_ReaderList-4              51130         51319         +0.37%
BenchmarkTxn_Put-4                     1526          1530          +0.26%
BenchmarkTxn_PutReserve-4              1610          1611          +0.06%
BenchmarkTxn_PutReserve_writemap-4     1344          1335          -0.67%
BenchmarkTxn_Put_writemap-4            1278          1290          +0.94%
BenchmarkTxn_Get_ro-4                  1398          1416          +1.29%
BenchmarkTxn_Get_raw_ro-4              783           798           +1.92%
BenchmarkScan_ro-4                     5890150       5901033       +0.18%
BenchmarkScan_raw_ro-4                 1521376       1527676       +0.41%
BenchmarkCursor-4                      691           687           -0.58%
BenchmarkCursor_Renew-4                179           178           -0.56%
BenchmarkTxn_Sub_commit-4              189910        188408        -0.79%
BenchmarkTxn_Sub_abort-4               209512        184603        -11.89%
BenchmarkTxn_abort-4                   14391         14075         -2.20%
BenchmarkTxn_commit-4                  14413         14634         +1.53%
BenchmarkTxn_ro-4                      144813        161165        +11.29%
BenchmarkTxn_unmanaged_abort-4         14490         14374         -0.80%
BenchmarkTxn_unmanaged_commit-4        14510         14517         +0.05%
BenchmarkTxn_unmanaged_ro-4            149717        147595        -1.42%
BenchmarkTxn_renew-4                   532           528           -0.75%
BenchmarkTxn_Put_append-4              468           465           -0.64%
BenchmarkTxn_Put_append_noflag-4       619           653           +5.49%

The benchmarks above that really swing are unrelated, I think. They should not be flushing pages. And they are generally quite noisy.
I also haven't been able to find any discussion of the flag in the official mailing list. I may have missed something. But I'm really at a loss about this flag. Perhaps I don't have the appropriate use case in the benchmarks. In general I think the benchmarks could be improved and more focused. But they do provide some utility.

I'm surprised that this doesn't help. Seems like the cutting the number of system calls in half should be an obvious win. Maybe the writes in the bench loop are so fast as it is that the difference ends up not showing up compared to whatever else is going on.

I have started writing some benchmarks which I believe will make the impact more apparent. And what I've seen so far makes me hopeful.
Looking over how pwritev gets used. It seems that you would see the flag's benefits when committing multiple writes (on multiple pages) per transaction. I don't think that this is being benchmarked effectively on master.

@mranney furthermore, to your point about it being an obvious win, I would absolutely agree with you taking the system call and LMDB code at face value. The thing that confuses me is why LMDB will not try define this for linux systems by default.
The C Makefile states

Note that the defaults [of MDB_USE_PWRITEV] should already be correct on most platforms; you should not need to change [its value].

AFAIK the library then make no attempt to define the flag for linux systems.

My unsubstantiated guess is that the flag is set in the OpenLDAP Makefile,
but I have not checked yet.
On 13 Mar 2016 11:54 am, "Bryan Matsuo"<EMAIL_ADDRESS>wrote:

@mranney https://github.com/mranney furthermore, to your point about it
being an obvious win, I would absolutely agree with you taking the system
call and LMDB code at face value. The thing that confuses me is why LMDB
will not try define this for linux systems by default.
The C Makefile states
Note that the defaults [of MDB_USE_PWRITEV] should already be correct on
most platforms; you should not need to change [its value].
AFAIK the library then make no attempt to define the flag for linux
systems.
—
Reply to this email directly or view it on GitHub
https://github.com/bmatsuo/lmdb-go/pull/58#issuecomment-195941833.

@lmb AFAICT that is not the case. It looks like there is a "grep" interface for the openldap repo.
http://www.openldap.org/devel/gitweb.cgi?p=openldap.git&a=search&h=HEAD&st=grep&s=PWRITEV
Searching for "PWRITEV" only turns up known references in the liblmdb tree.

I've put the question to the openldap-technical mailing list. A definitive answer should come from there. I will link the response here from the web archives after I have received it.

Sorry for not posting an update. But I never got a response from the openldap-technical mailing list. So I'm still a little confused there.
But it has come to my attention that the minimum system requirements for Go on Linux are kernel version 2.6.23. I was previously under the impression that the minimum required version was 2.6.32 (oops)
https://golang.org/doc/install#requirements
The pwrite(2) system call wasn't introduced until 2.6.30. So I'm somewhat concerned about people not being able to build the package if the have an old kernel.
I don't use custom build tags much. But would it be practical to put the #CGO linux CFLAGS in a separate file that is guarded by some special build tag like pwritev?
//+build pwritev

package lmdb

/*
CGO linux CFLAGS: -DMDB_USE_PWRITEV
*/
import C

That might not be completely correct syntax, but there is otherwise possible to define CFLAGS in multiple places AFAIK. I just don't really know the details of how to build non-main packages with custom build tags.

On another note I haven't made significant progress on benchmarks. I actually hate writing benchmarks for lmdb-go. While the current ones are especially bad it is impossible to completely remedy because they always be inherently artificial.

Good catch re the system requirements. I think what you propose is possible
by simply adding the build tag to the CGO comment (where it currently says
linux). Consumers of the library can then specify the build tag when
invoking go build or go install.
For benchmarks, I think that CGO is probably the main offender here. The
special scheduling alone will make it hard to get consistent results.
Especially with such a small change as this (which could be a good
indicator that its not worth worrying about this in a world of CGO
overheads).
On 23 Mar 2016 7:40 pm, "Bryan Matsuo"<EMAIL_ADDRESS>wrote:

Sorry for not posting an update. But I never got a response from the
openldap-technical mailing list. So I'm still a little confused there.
But it has come to my attention that the minimum system requirements for
Go on Linux are kernel version 2.6.23. I was previously under the
impression that the minimum required version was 2.6.32 (oops)
https://golang.org/doc/install#requirements
The pwrite(2) system call wasn't introduced until 2.6.30. So I'm somewhat
concerned about people not being able to build the package if the have an
old kernel.
I don't use custom build tags much. But would it be practical to put the #CGO
linux CFLAGS in a separate file that is guarded by some special build tag
like pwritev?
//+build pwritev
package lmdb
/*
CGO linux CFLAGS: -DMDB_USE_PWRITEV
*/
import C
That might not be completely correct syntax, but there is otherwise
possible to define CFLAGS in multiple places AFAIK. I just don't really
know the details of how to build non-main packages with custom build tags.
On another note I haven't made significant progress on benchmarks. I
actually hate writing benchmarks for lmdb-go. While the current ones are
especially bad it is impossible to completely remedy because they always be
inherently artificial.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/bmatsuo/lmdb-go/pull/58#issuecomment-200513633

Those seem like good observations about CGO. Honestly I think a real application could see benefits regardless of CGO depending on its use case for LMDB. But it's so wishy-washy that they would need actually benchmark their apps on real data to confirm. YMMV, absolutely.
One critical problem with the benchmarks is that I don't want them to take 10+ minutes to run every time. And on top of that I don't want them to take up 20GB of disk space per benchmark. So it is difficult to confirm those kinds of assertions in lmdb-go itself.
If a build tag is possible to work with then I think that is the correct way forward here. I was a little wary when I saw that go get will not allow build tag specification. But if it possible for an application to recompile it later with a build tag then that sounds ok.

I've updated the PR with a new build tag and some basic doc on how to use it.

This looks great. Thanks.
 But only go1.4 failed for known reasons. I am just going to merge this.
Thanks again.


fix: return proper error on undefined tenant

What kind of change does this PR introduce?
naive implementation to fix  #208 Probably should add a test.

add a test would be nice :smiley:


Consume flow

As part of the consume flow, please reinstate the following things missing in v4:

the OceanAssets download function and its usages https://github.com/oceanprotocol/ocean.py/blob/e914696b126e2a4304752ca4bd4c9b32380936c2/ocean_lib/ocean/ocean_assets.py#L515
unskip and adapt the test test_download_fails in test_ocean_assets.py. Hopefully by that point this PR: https://github.com/oceanprotocol/ocean.py/pull/627 will be merged
adapt and fix tests in ocean_lib/assets/asset_downloader.py which are now TODO.

CC @mariacarmina


CONTRIBUTING: Add contributions guidelines

Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

Coverage remained the same at 66.486% when pulling 0422fd8fef25ec803b040b5848abf2e418011742 on sameo/topic/contributing into 424104697a4e708013760b3ff1ad7f284beca84f on master.

LGTM

Coverage remained the same at 66.486% when pulling 87433a38737fb3d54ef93098fd0d440d0ade991b on sameo/topic/contributing into ef414d2a7f5c8a795c946102c7655d5ce292d70f on master.

Coverage remained the same at 66.486% when pulling 87433a38737fb3d54ef93098fd0d440d0ade991b on sameo/topic/contributing into ef414d2a7f5c8a795c946102c7655d5ce292d70f on master.

Coverage remained the same at 66.486% when pulling 87433a38737fb3d54ef93098fd0d440d0ade991b on sameo/topic/contributing into ef414d2a7f5c8a795c946102c7655d5ce292d70f on master.


REMOVE disqus_identifier so default is used.

This is mentioned in #150

Closing as I believe that the identifier is required, to consolidate urls one must use the url mapper mentioned in #150


CLI not properly setting the assets build property in a workspace

Command
generate
Is this a regression?

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

The previous version in which this bug was not present was
17.3.8
Description
The v18 cli does not set the assets path properly for an application created inside a workspace. No dependencies other than<EMAIL_ADDRESS>I suspect this may have something to do with the switch in syntax and folder name/location for the assets.
Expected Behavior
I would expect that the CLI would be workspace aware and properly set this value as it does with all the other config here that it generates.
Minimal Reproduction
Steps to Repro

Install the Angular CLI with npm i -g<EMAIL_ADDRESS>Generate a new workspace with ng new some-workspace --no-create-application
cd into the newly created workspace with cd some-workspace
Generate a new application within this workspace with ng g app some-app

Note that the angular.json will be updated with an entry to include a configuration for this new application but in the projects > some-app > architect > build > options > assets array the object created there has  public for the input property which does not appear to correctly reference the actual location of the public folder for this app.
Unexpected Behavior
The effect is that any assets placed in this folder will not be accessible when referenced, for example, in an <img> src attribute.
Workaround
Adjusting the application's assets configuration input property to projects/some-app/public appears to fix the issue.
Exception or Error
No response
Your Environment
Angular CLI: 18.0.1
Node: 22.2.0
Package Manager: npm 10.8.0
OS: linux x64

Angular: 18.0.0
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1800.1
@angular-devkit/build-angular   18.0.1
@angular-devkit/core            18.0.1
@angular-devkit/schematics      18.0.1
@angular/cli                    18.0.1
@schematics/angular             18.0.1
rxjs                            7.8.1
typescript                      5.4.5
zone.js                         0.14.6

Anything else relevant?
This appears to affect packages that rely on this folder to host assets such as @angular/pwa as well. Also, the relevant docs don't appear to reflect the current behavior of the project structure scaffolding (still references the old style of using an assets folder).

I have the same issue.


issue 511.2: use references instead of raw pointers for Index operations to avoid null pointer access

https://github.com/arangodb/enterprise/pull/184

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/2471/

replaced by https://github.com/arangodb/arangodb/pull/7725


Sync with upstream

Supersedes #158
Summary

Categories: protocol-units, misc.

Update to the changes in https://github.com/movementlabsxyz/aptos-core/pull/30
Changelog

The aptos-core forked crates have been updated to upstream commit 1dfd8a85c412dd08d4a61afd9e72475f54319db4 and aptos-cli 3.5.0

Testing
Updated tests should pass in CI.

This has got similar errors to main showing up in the settlement loop. But, I'll approve for for now.


Send templated Email workflow process

Given that we have a way to send templated emails (http://adobe-consulting-services.github.io/acs-aem-commons/features/email-api.html), it would be good to have a way of sending this in a workflow. We could do some intelligent things where if the workflow payload is a cq:Page, the param map gets populated with the (first-level) page properties for if it is an asset, the param map gets populated with the asset metadata.

Fixed via #350


unable to build

I think this is an issue with your Java environment and Gradle. Are you able to build a hello-world Java project using Gradle?

Closing because stale.


Import gltf generate 6x bigger material file

Might be related: #31632
Godot version:

v3.3.stable.official
OS/device including version:

Might be unrelated. Nvidia GTX1650Ti. GLES3.
Issue description:

Import gltf file exported from blender generate a 6x bigger material file in godot and similar file size increase in project export. I think it's okay that godot generate large file in project, but ideally project exports should only increase the size of original asset (gltf file).
Steps to reproduce:
Open an empty project, create a 3D scene, export project and see result binary + data size.
Import this gltf file exported from blender: https://drive.google.com/file/d/1HkD-9YrCcRWWObRZI7En2SkHihWAtIEg/view?usp=sharing
It's about 30MB. But godot generate a ~200MB .material file:

Export project again, it adds about ~200M in size:

Minimal reproduction project:

Project is too big to upload in github so I upload to google drive: https://drive.google.com/file/d/1rlYPFk1vYoal-z1XbqPESJiHm7bL46XH/view?usp=sharing. You can also just create a new project and import above glb file.

Duplicate of https://github.com/godotengine/godot/issues/34641.


Bug 1518482 Google is reporting 38,811 server errors during crawling

Description
We removed a bunch of docs from being indexed by Google which helped with bug 1518482, but since they are also in robot.txt, they can't be crawled, so will still index them, just without any content.
Original report
See https://bugzilla.mozilla.org/show_bug.cgi?id=1518482 for activity and discussion
Acceptance Criteria

[ ] zero errors in search console for docs that are indexed, but not allowed to be crawled

Google has switched to the new Search Console, and it appears the 38,811 error report was on the "old" console and no longer available.
The current error count is 337 errors on submitted page, and 369 on all known pages.

closing. superceded by other more specific issues handling this.


Enum attributes

For #383
This is my guess as to how to implement this, assuming there is some sort of string in the serialized attributes. It builds but it does not work. Some sort of reflection database error that I don't know what it means.
thread 'main' panicked at /home/quat/git/rbx-dom/rbx_reflection_database/src/lib.rs:7:76:
could not decode reflection database because: invalid type: integer `0`, expected struct Enum
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

println!("heyyy name={} value={}", enum_name, enum_value);

:sunglasses:

It builds but it does not work. Some sort of reflection database error that I don't know what it means.

This is because you're changing the (de)serialization of Enum! As a result, the reflection database can't be deserialized by the new version you're making.
Fixing this is an involved processs because we bootstrap the database. As a result I don't really recommend you try to fix it because it's potentially error prone and involves a bit of juggling with reflection databases.
PS: We have the enum attributes documented in the spec file we maintain. You didn't have to guess. 😄

What about a separate explicit enum type, like this?

This branch is now set up so that it suppresses the enum attribute errors and provides weakly typed enum functionality.  The explicit enum is moved to a different branch, never to be heard from again.

This implementation would fail a round-trip test because Enums no longer contain information on their type.

Well yes... but this is still useful as an error-suppressing patch while the full functionality isn't available

I will put aside time next week to fix this properly since it's showing up in the wild now, but I'm not willing to merge a feature that doesn't roundtrip. We won't be making a new release without the full feature anyway, so it being temporary fix is moot.
If it's desperate, I'd suggest using a custom fork of whatever tool you're using for the next little while.

I don't think it should be merged either which is why I have it as a draft, sorry for the misunderstanding. I'm not trying to get this incomplete feature merged, just a publicly available error fixer


add support for prefixed commands

Is your feature request related to a problem? Please describe.
give access to prefixed commands for certain roles and/or in certain channels or category
Describe the solution you'd like
add slash command to register and delete prefixed commands
define the cmd name with cmd parameter on the slash command
define the text content using text parameter on the slash command
usage flow:
/createtext cmd:hsr!speed text:https://pretend-this-is-a-hsr-speed-image-table.png
/deletetext cmd:hsr!speed
the bot will read message create event and if the 1st word match any prefixed command, bot will send the text content to the chat in that channel or category
role checks aren't necessary
Describe alternatives you've considered
use yagpdb
Additional context

Sounds interesting. I actually had a planning about invoking custom command long time ago


This extension will NOT install without MANY errors in Forge

Installed with URL or from repository fails.

Please post the errors

I installed it with forge. I don't get errors but I do not see the tab in the extras section.

Same as #5 then
TL;DR:

Install VS 2022 C++
Install CMake
Download a pre-built package for dlib

Thank you!


New DS Experience: Run a cell line-by-line

We have gotten mixed messages from the data science community regarding the need for a debugging tool for notebooks. For those users that have a stronger Software Engineering bent, they prefer a traditional debugging experience. For those users that have a strong Data Science bent, they prefer not to use a traditional debugging experience. It's not considered necessary. However, this latter group also does have a need for some sort of debugging, it's just usually a different context and can be solved without the use of a traditional debugger (like splitting code cells to simulate stepping and/or breakpoints).
We are going to pursue a new debugging-like experience for this class of users that is intended to make it simpler to diagnose and craft their notebooks. It is similar to debugging, but is much more lightweight and intended to be geared toward quick verification of behavior or values after lines are executed such that the need for splitting cells or using print statements is highly reduced.
Cell run-by-line basic workflow:

A user presses a button (probably not a debug button) to cause the notebook to highlight the first line of the cell. That line hasn't been run yet.
The user can inspect the values of variables with datatips
The variable explorer opens automatically.
The user can press the same button to execute the current line. The next line is highlighted and is ready to run.
There will be a keystroke that can be used in place of pressing the button. Probably F10.
If the user presses escape the execution of the cell is aborted.
This continues until every executable line of the cell is executed.
The variable explorer collapses.
Datatips no longer appear.

Additional details:

If the cell contains a function, then the lines of the function are included with each press of the run line button when that function gets called. Therefore, if the function is not actually called within the same cell, then the lines of the function will not be executed or stepped through.
If the cell contains a function call of any kind but that function is not defined in that cell, then the function call is always stepped over. There is no step-into such that another cell is brought into view.
This not debugging. There is no transition to debug mode and the debug panel never opens. There are no breakpoints and no callstacks. No step-into or step-out. It is purely: Run each line that this cell, and only this cell, executes.

Different pieces:

Send custom user defined setting to debugger - .2 days (do this first to make sure it works)- Wire up debug client - 2 days
General IP tracking - 2 days
Variables from the debugger - 2 days
DataTips returning data we show in the variable explorer? Maybe can reuse the debugger impl - 2 days
Extra button on toolbar to start all of this - .2 days
Escape hookup - .5 days
F10 hookup - .2 days

Stuff left to do:

Debug variable info not complete
DataFrame open during debugging if not same cell
Step into other cells (debugpy issue)
Hover on python files
Tests
Run by line on install debugpy
Focus/Scroll in cell after step


Having the option to not start Hotel with the OS

I did not find in the code nor in the documentation a way to prevent Hotel from starting on OS boot.
The motivation behind launching it manually to prevent having it start when I'm not coding on my laptop, to reduce battery drain.
I've tried setting permissions as 444 on the plist file in ~/Library/LaunchAgents to trick Hotel but it did not work.

Hi @czj,
If you run hotel stop, it should stop hotel  and prevent from starting with OS (it removes plist from ~/Library/LaunchAgents).
Let me know if it doesn't work.

It works well, but every time I shutdown my computer I have to think about typing hotel stop ... that's the only issue here :)

It works and if you don't want to implement the behavior you can close the issue, I understand :)

Hi!
I am looking for that feature too. Is it something that is planned ?


Cannot read properties of undefined (reading 'createCanvas')

Im trying to use the package for one of my usecases, but it throws me a TypeError: Cannot read properties of undefined (reading 'createCanvas')

@dichovsky i reproduce this error on nodejs 22.11.0
in this PR https://github.com/dichovsky/pdf-visual-compare/pull/10/files#diff-faff1af3d8ff408964a57b2e475f69a6b7c7b71c9978cccc8f471798caac2c88R12 i can't add tests for v22 for this error


Sanity check to prevent loading tiles when bounds are Infinity

Fixes #5477

@perliedman Wait, this didn't take into account the case of -Infinity

Huh. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/isFinite

@IvanSanchez doh, so much for my reviewing skills! :disappointed:


Is it possible to run the Azure Functions Dapr Extension with a Custom Handler?

I am experimenting with an odd scenario. Basically I want to subscribe to a Dapr Topic from a Go function inside Azure Functions.
For the Go function, I was able to register a Custom Handler that can handle the HTTP Requests:
func orderHandler(w http.ResponseWriter, r *http.Request) {
	log.Println("Handler invoked from an event")

	var invokeRequest InvokeRequest

	d := json.NewDecoder(r.Body)
	if err := d.Decode(&invokeRequest); err != nil {
		log.Fatal("Error while decoding the Body")
		return
	}

	log.Println(invokeRequest)
}

And inside the main function, I have the code to register the endpoints
func main() {
	customHandlerPort, exists := os.LookupEnv("FUNCTIONS_CUSTOMHANDLER_PORT")
	if !exists {
		customHandlerPort = "8080"
	}
	mux := http.NewServeMux()
	mux.HandleFunc("/orders", orderHandler)
	fmt.Println("Go server Listening on: ", customHandlerPort)
	log.Fatal(http.ListenAndServe(":"+customHandlerPort, mux))
}

I added the customHandler property inside the host.json file:
"customHandler": {
    "description": {
      "defaultExecutablePath": "handler.exe",
      "workingDirectory": "",
      "arguments": []
    }
  }

And created Function called DaprEventTrigger with the following function.json:
{
  "bindings": [
    {
      "name": "eventTrigger",
      "type": "daprTopicTrigger",
      "pubsubname": "orderpubsub",
      "topic": "orders",
      "direction": "in"
    }
  ]
}

To run this I am running the following cmd:
dapr run --app-id functionA --resources-path ./components/ --app-port 3001 -- func host start

Inside the components folder I only have one file called pubsub.yaml, which contains the Redis Component.
Right after the initialization I get this error:
time="2024-03-06T10:13:24.3482556-03:00" level=error msg="app returned http status code 500 from subscription endpoint" app_id=functionA instance= scope=dapr.runtime.processor.pubsub type=log ver=1.12.5

I suppose there's something going on with the Subscription Endpoint, which is falling with 500 HTTP Error in Dapr initialization and also when I try to call localhost:3001/dapr/subscribe
Is there a way to run a Custom Handler with Dapr Topic Subscription in Azure Functions?
I have also tried:

Creating a dapr/subscribe and /orders HTTP Trigger.

@cmelo05 did you find a solution to your issue?


fix(parser): parse empty method declaration as FunctionExpression

That a tricky one, because it's time to decide what does ESTree compliant means in the TS world (re #2463)
This code:
export declare class ByteBuffer {
  clear(): void;
     // ^^
}

Is parsed by Babel as FunctionExpression with an empty body
By TS-ESLint as TSEmptyBodyFunctionExpression
By OXC as TSDeclareFunction

I'm going the easy way to fix this to the Babel way, but I think following TS-ESLint would make sense. There is an open babel issue about that.
Edit: Ok that not so easy and require updating some logic.

Given babel raised it as issue, I'll try and align it with ts-eslint.

I didn't succeed to the task locally, but from the CI logs I see that this case:
var b = {
    foo(x = 1), // error
    foo(x = 1) { }, // error
}

was failing for a bad reason, now is parsing, but in fact should failed because with the error: A parameter initializer is only allowed in a function or constructor implementation.(2371).
What's the current philosophy of OXC parser, should I try to add this logic or is this fine?


Forward Upmerge 20230824, do not review

@de-nordic for doc build issue, please pick https://github.com/zephyrproject-rtos/zephyr/pull/62392

@de-nordic for doc build issue, please pick zephyrproject-rtos/zephyr#62392

Thanks did that to the current upmerge. This is forward sdk-zephyr branch that tests whether the current upmerge can be easily rebased to the newest Zephyr.