Support easy helm installation

Hi, Would it be ok if I pick this up or Maybe you have a secret helm file already :)

I've created a crude start that I'll be using internally..happy to hand the files over if you want them. Supports mysql/postgres and sqlite.

Do we have helm installation available for flagr yet ?

I have a crude chart I created..

Thats great... can you please share it?

Try this: https://github.com/travisghansen/scratch/raw/master/files/flagr-0.1.0.tgz

Does anyone have updated helm chart ?

Hello @nothing0012 @zhouzhuojie @aayush-harwani
I have created a simple Helm Chart for Flagr - it is easy to kickstart and has some docs. The repository is available here.
helm repo add flagr https://ericbsantana.github.io/flagr-helm-chart
helm install flagr flagr/flagr --devel

It has support to envs through extraEnvs value so it is easy to enable Prometheus metrics, change database configuration and also supports valueFrom to use database connection strings to it. As everything is controlled through environment variables, this simple setup should be enough for certain production environments.
The values.yaml and README.md inside the charts/flagr folder provides more information about it.
Hope this is helpful! Let me know if you have any questions or feedback.


Feature/improve set strategy

Codecov Report

Merging #51 (b90139f) into main (9500a54) will decrease coverage by 0.14%.
The diff coverage is 26.66%.

@@             Coverage Diff              @@
##               main      #51      +/-   ##
============================================
- Coverage     39.44%   39.31%   -0.14%
  Complexity      165      165
============================================
  Files            34       34
  Lines           900      903       +3
  Branches         71       75       +4
============================================
  Hits            355      355
+ Misses          512      511       -1
- Partials         33       37       +4

Files
Coverage Δ

src/main/scala/controller/Strategy.scala
75.00% <75.00%> (ø)

src/main/scala/aview/Tui.scala
8.69% <0.00%> (ø)

...ntroller/component/controllerImpl/Controller.scala
68.51% <25.00%> (-4.04%)
:arrow_down:

.../modeSelectionScreen/ModeSelectionScreenImpl.scala
0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

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


Various adjustments to improve text legibility

update styles for inline code

remove background color, make bold

update text and link colors for better contrast
add border radius to boxes (notes and code) to match nav
update code block padding to give space for copy icon and horizontal scroll
increase letter spacing on headings
update TOC spacing
make  same size as rest of the text

refs #40

@julioest I missed your review the other day. Can you also check the latest changes, then I can merge? I'll also work on keeping the PRs separated in the future
The latest change takes care of the border extending to the left:


Remove new/delete from GraphVisual

graphDataCoords has its contents allocated with new/delete.

Sorted with #269


feat: Validate that HTTP variable templates match resource pattern definitions (AIP-127)

Part of #972

Thanks for the first pass! And whoops, sorry for flooding the chat by "Starting a review"! 🙃

Great job @acamadeo thank you for your hard work on this.


catch all exceptions is not a viable solution

https://github.com/ska-telescope/lmc-base-classes/blob/41579b8b872fc5c046b1a02fe069f048f59c08b1/skabase/SKABaseDevice/SKABaseDevice.py#L64
I don't think this is an appropriate solution.

Only the relevant execption should be caught.
It would be better to have a try...except only around logging initialization
The except should provide a fallback solution, a print statement is not solving the issue. Ideally the fallback should be to initiate logging on a file instead of using rsyslog.

Added exception handling wherever applicable. PR#113 addresses this.


.Net Core 3

Is this possible in .net core 3 WPF apps? It doesn't seem to be.

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

ID: 14e9484a-4921-2ee3-c93e-787a7b104f61
Version Independent ID: 9e3db713-1558-7297-ef6d-53e5d87d4757
Content: Send a local toast notification from desktop C# apps - Windows UWP applications
Content Source: windows-apps-src/design/shell/tiles-and-notifications/send-local-toast-desktop.md
Product: uwp
Technology: design
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Hey @amartens181, it is possible with .NET Core 3.0, we're working on updating the helper library for .NET Core 3.0 along with publishing the helper library as part of the Windows Community Toolkit.
Here's the latest branch we've been working on: https://github.com/windows-toolkit/WindowsCommunityToolkit/tree/nmetulev/desktopnotifications/Microsoft.Toolkit.Uwp.Notifications/DesktopNotificationManager
You might be able to use that code and start sooner, as it'll still be some time till we publish the package and update the documentation.

closing this issue as there is no documentation change required

Mistakenly closed this issue, reopening since Andrew is already working on it.

Here's the latest branch we've been working on: https://github.com/windows-toolkit/WindowsCommunityToolkit/tree/nmetulev/desktopnotifications/Microsoft.Toolkit.Uwp.Notifications/DesktopNotificationManager

This link is dead. How do you use RegistrationServices? It's not part of .NET Core apparently.

Hey everyone, we've updated the code to work with .NET Core 3: https://github.com/WindowsNotifications/desktop-toasts/blob/master/CS/DesktopToastsApp/DesktopNotificationManagerCompat.cs
We're also in the process of adding this code to the Community Toolkit, but until then you'll have to copy/paste this file locally as the documentation says.
Closing since link in documentation has also been updated to the code that works with .NET Core 3. Let us know if you have any problems!


Add AerChemMIP, DAMIP to CMCC-CM2-SR5 activity_participation

@escoccimarro  @tomaslovato CMCC-CM2-SR5 has published data under the AerChemMIP and DAMIP activities, but this is not registered in the CVs - I will update to include this later this week - please reply if you have a problem with this
https://aims2.llnl.gov/search?project=CMIP6&activeFacets={"source_id":"CMCC-CM2-SR5","activity_id":"AerChemMIP"}
version = 20200622
https://aims2.llnl.gov/search?project=CMIP6&activeFacets={"source_id":"CMCC-CM2-SR5","activity_id":"DAMIP"}
version = 20240610

@martinaSt ping


Implement Doppler Secret Push and Delete functions

Problem Statement
Implement existing Doppler client push and delete functions.
I have manually tested both functions with KinD and an existing Doppler project.
Related Issue
None
Proposed Changes
N/A
Checklist

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

@ryan-blunden Feel free to give this PR a 👍/👎 as well.

/ok-to-test sha=9954374


Use sdnotify for systemd

This is useful if a configuration is long to load.
Systemd will continue dependency chain only when server have finish to start.
https://www.freedesktop.org/software/systemd/man/systemd.service.html#Type=

After a night of reflection... I think we can do a better design :)
More to come

Main problem is server.Start() is a blocking function. I can't evaluate when it finish to listen on socket etc.
With current patch it's working but it will send ready signal some little time before socket init...

Changes can be squashed and pushed with 1-clic

Many thanks to @Nikkau !

Let's squash the commit too if possible :angel:

@vdemeester this can be done via GH button ;)

@guilhem that's true, but I am not a huge fan of that :smile_cat: — but let's do it :stuck_out_tongue_closed_eyes:

@vdemeester this have 1 advantage: keeping history in PR.
But I'm not really sure if good or not :/
Just experiment this pattern for now ^^

@guilhem hum yeah but sometimes you want to actually have a PR with multiples commit merged as is. Anyway, this is no big deal :stuck_out_tongue_closed_eyes:


Sample feedback: basic-concepts/ts

If not pre-filled, update the title of this issue with the name of the audit this feedback pertains to.
Provide details of any issues with the documentation or improvements you'd like to see below.
There is no example code .

Hi @yeapesdt, can you clarify what issue you have with this sample? The generated code can be found in this repo under dist/basic-concepts.


Change ratelimit service to CC

Codecov Report
Patch coverage has no change and project coverage change: +3.34% :tada:

Comparison is base (0b79b47) 24.60% compared to head (429cda4) 27.95%.
Report is 15 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1548      +/-   ##
==========================================
+ Coverage   24.60%   27.95%   +3.34%
==========================================
  Files         134       77      -57
  Lines       15641     5184   -10457
  Branches     2440     2440
==========================================
- Hits         3848     1449    -2399
+ Misses      11658     3735    -7923
+ Partials      135        0     -135

Flag
Coverage Δ

adapter
?

common-controller
0.00% <ø> (ø)

config-deployer-service
23.42% <ø> (ø)

idp-domain-service
80.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 58 files with indirect coverage changes

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


Access to the Indoor dataset and specific Places images for test ( What was your test image with masks)

Hi. How we can access to the Indoor dataset for training?
How we can access  to the test set  train set for the Places and Indoor datasets?
@DQiaole

Hi~
You can download Indoor dataset from https://pan.baidu.com/s/11O1Q7gcn7dhjPDDUNrmodQ, password:hfok; and we have split the Indoor for training and validation.
As for Places2, we use the training set officially provided by Places2, which has 180w images; and we sampled 100 images randomly from each class in the Places2 test set, resulting in a total of 36,500 images as our test set.

hfok

We can not download any files from pan.baidu.com when we are not in the China. Could you please share this dataset on google drive?
I need you test set ( images and masks) for the Places and Indoor dataset for comparison. Could you please share with me?

We have upload the test set ( images and masks) for the Places and Indoor dataset.
If you cannot download the Indoor from pan.baidu.com, we recommend you download ShanghaiTech and NYUDepthV2, and construct the Indoor dataset according to our data split file.

Please upload the Indoor dataset in another Cloud.
In this file you shared test set for Indoor and places dataset. Isn't it?

Sorry, our Google Drive has no more space, we can only provide the validation set of indoor through google drive: https://drive.google.com/file/d/1LiJ1b_Cedf_9yQHPcAyDpxgkbcFrtF91/view?usp=sharing.
And this file is the test masks of our test images.

Sorry, our Google Drive has no more space, we can only provide the validation set of indoor through google drive: https://drive.google.com/file/d/1LiJ1b_Cedf_9yQHPcAyDpxgkbcFrtF91/view?usp=sharing.
And this file is the test masks of our test images.

The test mask file is for Indoor or Places dataset?

yes, indoor test images and places2 test images use the same test masks.

Besides, we have uploaded the full indoor dataset, you can find them in README.md.


[BUG] the newer version does not work

Prerequisites

[X] I have read the documentation.
[X] I have checked other issues for similar problems.

Backend
Local
Interface Used
CLI
CLI Command
No response
UI Screenshots & Parameters
No response
Error Logs
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
cudf-cu12 24.10.1 requires pandas<2.2.3dev0,>=2.0, but you have pandas 2.2.3 which is incompatible.
gcsfs 2024.10.0 requires fsspec==2024.10.0, but you have fsspec 2024.9.0 which is incompatible.
google-colab 1.0.0 requires pandas==2.2.2, but you have pandas 2.2.3 which is incompatible.
tensorflow 2.17.1 requires tensorboard<2.18,>=2.17, but you have tensorboard 2.16.2 which is incompatible.
Successfully installed Mako-1.3.6 PyWavelets-1.7.0 albucore-0.0.20 albumentations-1.4.21 alembic-1.14.0 authlib-1.3.2 autotrain-advanced-0.8.31 bitsandbytes-0.44.1 brotli-1.1.0 colorlog-6.9.0 datasets-3.1.0 diffusers-0.27.2 dill-0.3.8 evaluate-0.4.3 fastapi-0.115.4 fsspec-2024.9.0 hf-transfer-0.1.8 inflate64-1.0.0 invisible-watermark-0.2.0 ipadic-1.0.0 jiwer-3.0.5 lightning-utilities-0.11.8 loguru-0.7.2 multiprocess-0.70.16 multivolumefile-0.2.3 nvidia-ml-py-12.535.161 nvitop-1.3.2 optuna-4.0.0 packaging-24.1 pandas-2.2.3 py7zr-0.22.0 pybcj-1.0.2 pycryptodomex-3.21.0 pyngrok-7.2.1 pyppmd-1.1.0 python-multipart-0.0.17 pyzstd-0.16.2 rapidfuzz-3.10.1 rouge-score-0.1.2 sacremoses-0.1.1 seqeval-1.2.2 simsimd-6.0.5 starlette-0.41.2 tensorboard-2.16.2 texttable-1.7.0 tiktoken-0.6.0 torchmetrics-1.5.1 tqdm-4.67.0 trl-0.12.0 uvicorn-0.32.0 werkzeug-3.1.2 xxhash-3.5.0
Additional Information
No response

doesnt look like an error but more like a warning. are you unable to train models?

yes

abhishekkrthakur
i get Traceback (most recent call last):
File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.10/dist-packages/autotrain/trainers/dreambooth/main.py", line 5, in
from diffusers.utils import convert_all_state_dict_to_peft, convert_state_dict_to_kohya
File "/usr/local/lib/python3.10/dist-packages/diffusers/init.py", line 5, in
from .utils import (
File "/usr/local/lib/python3.10/dist-packages/diffusers/utils/init.py", line 38, in
from .dynamic_modules_utils import get_class_from_dynamic_module
File "/usr/local/lib/python3.10/dist-packages/diffusers/utils/dynamic_modules_utils.py", line 28, in
from huggingface_hub import cached_download, hf_hub_download, model_info
ImportError: cannot import name 'cached_download' from 'huggingface_hub' (/usr/local/lib/python3.10/dist-packages/huggingface_hub/init.py)

autotrain doesnt support dreambooth anymore. to train loras please use: https://huggingface.co/spaces/autotrain-projects/train-flux-lora-ease


Update static (client side)

Landed in https://github.com/metarhia/Example/commit/f20b21bcc7dcd6f837160b00e96beaca3fc0be9d, https://github.com/metarhia/Example/commit/5f590376fc7305143589229c4e30e7b53d75150a, https://github.com/metarhia/Example/commit/ee6714087bc9e94ffc0fa9a3d57391bc45c8983a, https://github.com/metarhia/Example/pull/246/commits/115f0c0c9e0ff0cc3b2c17eb36b9128223c49e7a, and https://github.com/metarhia/Example/commit/5ee455c1d55b5b5723105ecb79c646664d358438


tenant: Correctly log number of tries when deleting

The change to a f-string format for the error raised when the deletion of an agent failed was not done correctly, resulting in the number of tries before giving up to not be printed.
A message similar to the following would be logged instead:
2024-12-02 12:54:11.424 - keylime.tenant - ERROR - ('Agent d432fbb3-d2f1-4a97-9ef7-75bd81c00000 (<IP_ADDRESS>:9002)',) was not deleted from Verifier default (<IP_ADDRESS>:8881) after %d tries

I'll merge this as it is very straightforward


Create .gitignore

You should probably add this, but the .gitignore file will ignore the __pycache__ directory, which will be helpful.
It will also ignore other future unneeded files, which is helpful as well :)

Thanks!

No problem! Glad I could help! :D

Wait do I have to delete the pycache files first so they get ignored from the next time?

No, if you do delete them, once you run it on Replit again, it will re-appear anyways. The .gitignore file ignores it for you, so that all the data will be ignored. It won’t delete the file, but it will ignore all the data, which will be helpful.

Oh okay thanks


Ability to select a block of streams in the UI

In most panes containing streams in the Stroom UI you can either select ALL streams or need to click streams 1 by 1.
It would be useful to to be able to select a block of streams by clicking a stream selector box, holding the Shift key, then clicking a subsequent stream selector box, to effectively select the entire block of streams in between these two stream positions.

+1

Is this for reprocess/delete type scenarios?

@stroomdev66 Yes, that's where I'd see it being most used

If we were to do this then I think we would ideally lose the tick icons for each row and just use the multi row selection. Batch operations on the filtered data that are currently performed clicking the header tick box would be performed by a batch operation menu hopefully making batch functionality a bit more obvious.

+1
I use this also for selecting a bunch of streams to export to our dev environment for testing. I hear rumours that there is a new approach to doing this though.


@haise0

@haise0 Hello, bro. Could you refork the project? There is something private for me in the old codes.

Sure!


[Tracking Issue] Unified Device, Target and Memory Scope Planning

This issue is to track progress for unified-device-target-and-memory-scope-planning
See https://github.com/apache/tvm-rfcs/pull/38.

The RFC is moribund at the moment while we clear off some more pressing issues. I'd like to pick this up again in 2022Q1 with more detail in the RFC after discussions with @Mousius who's tackling the companion Target vs 'codegen compiler name' disconnect.
I see this as enabling more uniform support for mixing and matching between library implementations, external codegen, and the built-in TIR codegen.


Add IMDb API

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been [squashed][squash-link] into a single commit

I don't want to receive email

Hi,
you can check the URLs available in the Wiki Pages,
Example: https://imdbot.vercel.app/?q=avengers end game
An example repository which is using the api is: https://github.com/SpEcHiDe/Media-Search-bot/blob/master/bot/sessalc/search_imd_b.py#L35

Hi,
you can check the URLs available in the Wiki Pages,
Example: https://imdbot.vercel.app/?q=avengers end game
An example repository which is using the api is: https://github.com/SpEcHiDe/Media-Search-bot/blob/master/bot/sessalc/search_imd_b.py#L35

Very good. Thanks for showing me


Feature: add cpu/mem info in top node side

Signed-off-by: xiechengsheng<EMAIL_ADDRESS>Fix part of #3 , codes about top job would come soon.

@GarnettWang codes have been updated, PTAL, thanks~

@xiechengsheng , thanks for your quick and great contribution!
I suggest we should get the same result with kubectl top node.
kubectl top node
NAME                     CPU(cores)   CPU%      MEMORY(bytes)   MEMORY%
i-bp10ql620010exb478op   166m         4%        2237Mi          28%
i-bp14vuwrktdfxioibi8i   192m         4%        2124Mi          27%
i-bp1afgfln76507diuvl9   234m         5%        2222Mi          28%
i-bp1fzgva3kl983om6iwy   61m          3%        2772Mi          40%
i-bp1fzgva3kl983om6iwz   41m          2%        2358Mi          34%

Can you help investigate and implement it? Thanks!

@cheyang OK, It's my pleasure to take this. BTW, do you mean that we should get and output the real GPU/CPU/mem usage in cluster node, instead of pod request resource value/node total resource value?

For CPU/Memory, I think it should be the same implementation as https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/cmd/top_node.go. But for GPU,  we don't need change. Thanks.

I'm sorry to ask that which metrics server would be used to monitor the whole cluster's resource usage info? heapster or metrics-server? Because different metrics server will make me to write different client code to get cluster info from metrics server's RESTFul API. Thanks a lot.

According to the implementation in top_node.go, we should use heapster. If heapster is not available in the cluster, we will not display the cpu/memory  usage.

And should we really print the real-time resource usage in top node command? This resource usage changes all the time.

According to the definition of top, I think it's for real-time resource usage.

Hi @cheyang , sorry for my late updated codes. Please re-check this pr.
BTW, I run kubectl top pod --all-namespaces on k8s master to get whole cluster's namespace pods cpu resource usage, and I simply add the each pod's cpu usage to get cluster's cpu sum usage. Then I run kubectl top node to get whole cluster's node cpu resource usage, and I add each node's cpu uasge, I find that sum(node_cpu_usage) is always bigger than sum(pod_cpu_usage). These two ways to get cpu/mem usage would get different result, then which one do you think is better to be adopted in arena top node and arena top node -d command?

/assign @xiaozhouX

Hi there, when I add "k8s.io/metrics" in Gopkg.toml, and use dep to generate dependent package in vendor directoory, but the bulid process make would always fail with undefined error in my server, I have no idea to fix this, need yours help...
Here is the make log output:

And I think maybe a version error exist here, so I change the "k8s.io/metrics" from branch release-1.9 to release-1.10 to release-1.11, but it seems useless...

Any help would be appreciated!

I signed it!

/assign @wsxiaozhang

/assign @xiaozhouX

@xiechengsheng  Sorry for late response. I exec arena top node in my host for a 1.11.x version k8s cluster. It turns out a error the server could not find the requested resource (get services http:heapster:), it seems that there is compatibility problem, Can you check for that?  Thanks very much for your contribution.


pycharm marketplace

Why isn't latest in the marketplace of pycharm? its there version 0.4.2

There were incorrect version constraints set up. The next version should show up shortly!


flake: allow-all/pod-to-service/curl-2 timeout during curl exec, no flows retrieved

Hit in GKE on #390 https://github.com/cilium/cilium-cli/actions/runs/1001017290
2021-07-05T12:33:47.221179076Z   [.] Action [allow-all/pod-to-service/curl-2: cilium-test/client2-666976c95b-mc9mq (<IP_ADDRESS>) -> cilium-test/echo-other-node (echo-other-node:8080)]
2021-07-05T12:33:47.221185112Z   📄 Following flows...
2021-07-05T12:33:47.221856284Z   ❌ command "curl -w %***local_ip***:%***local_port*** -> %***remote_ip***:%***remote_port*** = %***response_code*** --silent --fail --show-error --connect-timeout 5 --output /dev/null http://echo-other-node:8080" failed: Timeout occurred
2021-07-05T12:33:47.221975647Z   📄 Validating flows for peer cilium-test/client2-666976c95b-mc9mq
2021-07-05T12:33:57.222499529Z   ❌ Aborting flow matching: context deadline exceeded
2021-07-05T12:33:57.222546717Z   ❌ Flow validation failed for peer cilium-test/client2-666976c95b-mc9mq: 0 failures (first: -1, last: -1, matched: 0)

Attaching sysdump:
cilium-sysdump-out.zip.zip
List of PRs blocked:

#390 Manually unblocked after 2 weeks as an exception because it seems we have not reproduced this flake ever since.

Hit by multicluster test in #529, though on no-policies/pod-to-service/curl-0, not allow-all/pod-to-service/curl-2: https://github.com/cilium/cilium-cli/runs/3547587561?check_suite_focus=true


How to use input parameters with SQL kernel?

I came to this page looking for help on creating a parameterised Notebook to run SQL queries. Not python.
Please provide some examples how to create a SQL Notebook that uses input parameters.

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

ID: 57656112-bcf3-de19-f90d-92680191e4f6
Version Independent ID: e456625f-9be8-20e1-5933-deb31d7b80a2
Content: Parameterize notebooks in Azure Data Studio with the Run with Parameters action - Azure Data Studio
Content Source: docs/azure-data-studio/notebooks/run-with-parameters.md
Service: azure-data-studio
GitHub Login: @VasuBhog
Microsoft Alias: vabhog

Hi @frab3 - We do not currently support parameters with the SQL kernel. It is something that we have been looking at but has not made the cut. Here is a link to the issue where we are tracking this feature request: https://github.com/microsoft/azuredatastudio/issues/18709. Please feel free to add any additional thoughts there.
Thanks,
Dave
#please-close


Add warnings/checks for common chartbuilder errors

warning: axis ticks divided unevenly
warning: no prefix/suffix given
numbers too long (should be divided)

Done 4e8d5d45b29d5ab5561bd1ef52842bb83f49e067


Feature/add more logging for handle message

Changes Unknown when pulling 0fc0d85682b6e41d5fc58a19a0afad3b1e155d9a on feature/add-more-logging-for-handle-message into ** on master**.


I cannot get this version to work with 1.9.1

Version 4.0 does not load under any circumstance.  I have the latest IRBT MOD UTILS installed which loads fine once you fix the mod.json error, an extra bracket in it.  I do not have many mods that I use.
Here are my logs. I have turned off every other DLL mod and your mod will not load. I have also tried removing the IRBT requirement and it will not load.  I think there is an error! Can you please check.
harmony_summary.log
ModTek.log
ModTek_runtime_log.txt
output_log.txt

Depends on line is incorrect, needs to have irbt, cc, and cac then remove optional.


[BEAM-14535] Added support for pandas in sklearn inference runner

Scikit learn developers build pipelines using pandas very often and we should support it if we want to support pandas pipelines.
Here's an example from their site.

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

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

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

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

R: @yeandy

R: @TheNeuralBit

Run PythonDocker PreCommit

I'll go ahead and merge this for the base support, but as discussed offline, let's follow-up with a change to break out the numpy array and dataframe implementations into separate ModelLoaders, and mark the dataframe one experimental


Improve appender looping performance in Category

Replace Vector with a better implementation for lopping on appenders

Hi @ceki
As I understand, you fixed this issue in the 1.2.20 with this commit :
https://github.com/qos-ch/reload4j/commit/63ff036d11b14767365197a79606cb6df6102c3a
I think this issue can be closed ?

Yes, thank you.


Chart assumes default cluster DNS name for GELF container

I've been trying to set up Seq in our Kubernetes cluster with GELF input coming from Fluentbit based on this documentation. I have deployed Seq using the Helm chart as described in the documentation. While setting up Fluentbit I've seen that our cluster isn't using the default DNS name of *.cluster.local but is using a different name instead, so I had to configure Fluentbit slightly different. This could be because our cluster has been rolled out using Rancher.
After fixing that though, I still didn't see any logs. So I had a look at the logs of the seq-gelf container inside my Seq pod and I saw the following error message:
Failed to send an event batch
System.Net.Http.HttpRequestException: Name or service not known
 ---> System.Net.Sockets.SocketException (0xFFFDFFFF): Name or service not known
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean allowHttp2, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.GetHttpConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at SeqCli.Ingestion.LogShipper.SendBatchAsync(SeqConnection connection, String apiKey, IReadOnlyCollection`1 batch, Boolean logSendFailures) in /home/appveyor/projects/seqcli/src/SeqCli/Ingestion/LogShipper.cs:line 155
   at SeqCli.Ingestion.LogShipper.ShipEvents(SeqConnection connection, String apiKey, ILogEventReader reader, InvalidDataHandling invalidDataHandling, SendFailureHandling sendFailureHandling, Func`2 filter) in /home/appveyor/projects/seqcli/src/SeqCli/Ingestion/LogShipper.cs:line 55

After inspecting the pod configuration I noticed that the seq-gelf container had an environment variable SEQ_ADDRESS set to an address that ended with *.cluster.local as well, which caused the above error message.
Unfortunately there isn't a way to override this from the Helm chart. I had to manually patch the deployment to change the value of that environment variable. Interestingly I don't think it needs to be a FQDN at all, since the seq-gelf container is running within the same pod, so it can just use localhost and it will work. I've manually made that change on my cluster and now I'm seeing logs coming into Seq.

Thanks for the suggestion @jmezach - we'll try to get this into a future release 👍


Added "Spacebucks" barsign

Added the animated barsign for "Spacebucks" because what do our corporate overlords love more than franchises? You can't escape them, even in space!
Media

[x]  I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
🆑
Added: The "Spacebucks" barsign.

Can you remove "4 the emprah" while at it, feels like it'll be sussy copyright.

Does Spacebucks come from any ss13 server or is it a custom sprite? If it's not in either of the copyright commits then please add another line of copyright. Otherwise this is fine.

It's a totally custom sprite I made. I can totally add in a line of copyright, to be honest I just wasn't sure how to attribute copyright to myself in this context.

One more change, the CL shouldn't have Added: To the game
It should mention what it adds in the description:

add: The "Spacebucks" barsign was added.

Or something similar to that.

One more change, the CL shouldn't have Added: To the game
It should mention what it adds in the description:

add: The "Spacebucks" barsign was added.

Or something similar to that. This is mostly due to how the CL bot works.

Just updated it!


Initial implementation of a JetBrains YouTrack hook for GitBlit.

In response to #1036

:+1: Thanks!


Change viewport (new setViewPort method)

New to git, so I'm sure there's an easier way to do this, but here's the code (from drawing.py, 51):
 def setRenderSize(self, w=None, h=None):

      self.renderWidth = w

      self.renderHeight = h

      if origin == 'center':

        self.viewBox = (-width/2, -height/2, width, height)

      return

Haven't tested the code, and it already works well on most borwsers/etc, but I'm trying to develop on Kivy, and they barely support SVG files.
Feedback is welcome
EDIT: looks like you may need to make the 'origin' global, or create a setViewPort func

If I understand correctly, you want to change the drawing's view port size.
setRenderSize changes the final image scale, not the internal coordinate system.  There is currently no way to edit the view port after a drawing has been created.  You could make a Pull Request to add a method setViewPort contining these lines to do this.
For now, you could just edit the viewBox of a drawing directly:
d = draw.Drawing(...)
...
width = ...
height = ...
d.viewBox = (-width/2, -height/2, width, height)

but this may result in badly scaled SVGs.

Closing as stale.


[EDGE-896] Update pipeline dashboard screencaps

Updating screencaps for the pipeline dashboard walkthrough page as they are very out of date.
Also removed all-builds.png as that is no longer used in the docs :)
Linear ticket
Link to doc page with updated screenshots:
https://2580--bk-docs-preview.netlify.app/docs/pipelines/dashboard-walkthrough

I missed this while I was out last week but stoked to see these get updated! Thanks! 💚


Add support for GoogleAPI OAuth service

Add support for the GoogleAPI Oauth service.

This is amazing, and I'm very grateful that you put this together. Just a few minor things I brought up.

Thanks for the review @mattmassicotte.
I will submit another PR that addresses your comments early next week.
As for the retrieveLogin() public accessor, I'm glad to discuss and perhaps my use-case is too specific to what we are doing.

As for the retrieveLogin() public accessor, I'm glad to discuss and perhaps my use-case is too specific to what we are doing

You know, I doubt it. As is, there's no way to have more than on login per authenticator, which totally makes sense. I just want to keep this PR focused so we can address this other problem well.

Yes I agree.
We need a better construct to support multi-account authorization (multiple Login objects).
I started working on this internally in our app but internal OAuthenticator support  would be much better.
Let's discuss the best way to lay out that capability jointhly.
Feel free to open the previous Issue to track this.

Fantastic!

New release package soon ?


Latest contracts & Goerli deployment

Caveat
The execute request command does not currently work. Actively fixing.

Can we remove the prettier PR for my own sanity?  We can do this later if it is necessary.

I want to prioritize getting this merged so we can merge the update-name branch to main to avoid any further confusion on which branch to use.

Can we remove the prettier PR for my own sanity? We can do this later in an isolated PR if it is necessary.

Sure if it makes it easier for you. You can also review by commit rather than all files changed.
We need a prettier config in this repo. I see it's added as a dependency, but since there is no config it is pulling from my global prettier config, which seems to be pretty different from your global config.

Thanks!  We should probably set some standard at some point, but that is certainly out of scope for the near future.

I am still seeing a lot of changes in this PR replacing single-quotes w/ double quotes and other ancillary differences...

I am still seeing a lot of changes in this PR replacing single-quotes w/ double quotes and other ancillary differences...

..this is the prettier thing that I just mentioned.
I can try to manually resolve, but it would be a lot easier if you just paste your prettier config so I can use the same.

Can we eliminate everything except contract changes? The only changes in this PR outside of /contracts should the change from FunctionsRegistry to FunctionsBillingRegistry

There is also a bug fix or two in here and the new deployments.
Let's see how things look after the formatting consistency change and then if you really need it separate I guess I can do that.

Can you rebase this on top of PR#10 please?


Issues with debugging stack (tcb->uxStack moves?)

Hi, I'm having memory issues and was trying to make a tool to debug stack usage, for that I need to check how much free stack remains, by default IDF provides only the stack high water mark, but not current use or stack size. But these can be extracted from FreeRTOS structs... or so I thought.
I get  stack current position (top) and end (end) from uxTaskGetSnapshotAll and stack begining (uxStack) from pxTaskGetStackStart. The second one required #define INCLUDE_pxTaskGetStackStart			1 in FreeRTOSConfig.h.
Things look OK, but only for some time. For some reason unknown to me, the begining of the stack jumps around at random times. Note that it's probably just a debugging aid and maybe needed on task destruction, because in normal task operation only the current position is important.
Here is a log of some measurements of the main task, note that the size is OK (default is 4096), but sometimes it changes by 107492, because uxStack jumped by that value. Even more, then it jumps back and again to that value.


allow for VERSION and REVISION to be passed in during docker builds

solve for #2953
Allows VERSION and REVISION to be set via build args.
E.G docker build --build-arg VERSION=somerelease --build-arg REVISION=internalRev .

weird... i don't understand how I'm reducing code coverage in registry/handlers/app.go given I am not editing that file at all...

@manishtomar @caervs is it possible to get eyes on this?


sles11: syslog is not redirected to serial port

For image_tests, to wait for some image to boot, I add a startup script to print the message "BOOTED" and I use Daisy WaitForInstancesSignal, but sles11 doesn't seem to send syslog outputs to the serial port and the test fails.
I tested on the latest sles11:
sles-11-sp4-v20180104                                 suse-cloud           sles-11

You can't assume startup script stdout will go to the serial console, it's
better to write to /dev/ttyS0 directly unless your specifically testing
syslog output to the serial console.
I don't think we should rely on syslog writing to the serial console at all
as it makes a lot of assumptions of the host OS (syslog redirection is
setup or that the OS even uses syslog) but that's a separate discussion
entirely.
On Fri, Mar 23, 2018, 8:53 PM Helen Koike<EMAIL_ADDRESS>wrote:

For image_tests, to wait for some image to boot, I add a startup script to
print the message "BOOTED" and I use Daisy WaitForInstancesSignal, but
sles11 doesn't seem to send syslog outputs to the serial port and the test
fails.
I tested on the latest sles11:
sles-11-sp4-v20180104 suse-cloud sles-11
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/compute-image-tools/issues/373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQWyS6aZGaqrdmeBBhJfAcOPeChFdeaQks5thcMkgaJpZM4S5luc
.

Right, I see, I'll update the scripts to write to ttyS0 directly then, so I can rely on WaitForInstancesSignal.
Thanks.


Bug Report - Application error : (0Xc0000142) Help-Wanted

Environment
Windows 10 Home
Windows build number: [run "ver" at a command prompt]
Windows Terminal version (if applicable):
Microsoft Windows [Version 10.0.18362.145]
(c) 2019 Microsoft Corporation. All rights reserved.

Any other software?

Steps to reproduce

Expected behavior
Expected to see the terminal open into the Linux bash. Trying to open the cmd.exe produces the same error.

Actual behavior
Terminal window opens with a message cmd.exe - Application Error - "The application was unable to start correctly (0xc0000142). Click OK to close the application"

Help-Wanted

Are you launching the built-in cmd.exe/powershell.exe/wsl.exe/conhost.exe, or are you trying to launch OpenConsole.exe/Windows Terminal built from source in this repo?
What architecture is your machine, and if you're building from source, what platform are toy targeting?

Hi,
What architecture is your machine X64 , and if you're building from source Yes from the GitHub repo, what platform are toy targeting wsl.exe?
I can open wsl.exe , cmd.exe, and even OpenConsole.exe , which results in a single terminal window which is not the new terminal application in appearance.
See the screenshot below. The second screenshot shows the error I am getting with the dev build terminal.
<EMAIL_ADDRESS><EMAIL_ADDRESS>Kind Regards,
Stephen
From: Mike Griese<EMAIL_ADDRESS>Sent: Tuesday, May 28, 2019 3:32 PM
To: microsoft/terminal<EMAIL_ADDRESS>Cc: Stephen Morgans | Code Craft<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] Bug Report - Application error : (0Xc0000142) Help-Wanted (#1018)
Are you launching the built-in cmd.exe/powershell.exe/wsl.exe/conhost.exe, or are you trying to launch OpenConsole.exe/Windows Terminal built from source in this repo?
What architecture is your machine, and if you're building from source, what platform are toy targeting?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/1018?email_source=notifications&email_token=AA22CZEMDT5M26LSZHWGPQLPXUX47A5CNFSM4HPVNLQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWMEBTI#issuecomment-496517325, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA22CZGQAVUJ6R54BW5Y2GTPXUX47ANCNFSM4HPVNLQA.

Hey for the record, your screenshots didn't work through email replying to github. You might have to respond on github for them to work.
When I ask about what platform you're targeting, I mean what's the Solution Platform set to in VS?

Here it's set to x64.

This is the event viewer log created when I try to open the application.
The application was working 100%, but after a Windows update it broke.
Log Name:      Application
Source:        Application Error
Date:          2019/06/09 12:31:17 PM
Event ID:      1000
Task Category: (100)
Level:         Error
Keywords:      Classic
User:          N/A
Computer:      Dell2016
Description:
Faulting application name: WindowsTerminal.exe, version: <IP_ADDRESS>, time stamp: 0x5ce9ca15
Faulting module name: Microsoft.UI.Xaml.Markup.dll, version: <IP_ADDRESS>, time stamp: 0x5ce9c93c
Exception code: 0xc0000005
Fault offset: 0x000000000002decf
Faulting process id: 0x7748
Faulting application start time: 0x01d51eae6f1e2d27
Faulting application path: C:\Terminal\src\cascadia\CascadiaPackage\bin\x64\Debug\AppX\WindowsTerminal.exe
Faulting module path: C:\Terminal\src\cascadia\CascadiaPackage\bin\x64\Debug\AppX\Microsoft.UI.Xaml.Markup.dll
Report Id: 64717541-ed38-4710-a1c5-774f2b55f243
Faulting package full name: WindowsTerminalDev_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App
Event Xml:

1000
2
100
0x80000000000000

27114
Application
Dell2016

WindowsTerminal.exe
<IP_ADDRESS>
5ce9ca15
Microsoft.UI.Xaml.Markup.dll
<IP_ADDRESS>
5ce9c93c
c0000005
000000000002decf
7748
01d51eae6f1e2d27
C:\Terminal\src\cascadia\CascadiaPackage\bin\x64\Debug\AppX\WindowsTerminal.exe
C:\Terminal\src\cascadia\CascadiaPackage\bin\x64\Debug\AppX\Microsoft.UI.Xaml.Markup.dll
64717541-ed38-4710-a1c5-774f2b55f243
WindowsTerminalDev_<IP_ADDRESS>_x64__8wekyb3d8bbwe
App

Hey for the record, your screenshots didn't work through email replying to github. You might have to respond on github for them to work.
When I ask about what platform you're targeting, I mean what's the Solution Platform set to in VS?

Here it's set to x64.

I did compile it in x64 architecture not ARM64.


B 18259

[## Agility ticket](https://www13.v1host.com/USTRANSCOM38/story.mvc/Summary?oidToken=Story%3A873093&RoomContext=TeamRoom%3A808731&concept=TeamRoom)
Summary
NFPR - Partial Delivery Out of Destination SIT Automated Testing
Verification Steps for the Author
These are to be checked by the author.

[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test
Run playwright tests for prime simulator flows
Prime simulator user :
is able submit payment request on SIT without destination SIT Out Date

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 3d180d9e26b6cd26a7789bdf3d48c2062e98cb32

Test coverage on server failed.
Last run: Tue 19 Dec 2023 04:00:05 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


Second argument a la Object.create?

In particular, this would be useful to do Object.fromEntries(entries, null). It could reuse the same spec machinery as Object.create.

This would have the opposite signature from Object.create, which would be somewhat annoying.

That’s true. Do you think there’s a reasonable way, that doesn’t have to use proto to provide the [[Prototype]] without a second call?

I don't think so, though I could easily be missing something obvious. I don't think a second call (or __proto__) is all that bad, though. Still might be worth adding something here; it just doesn't seem like it's absolutely required.

Definitely not required; but it feels like it'd be a win for optimizeability and code clarity.

The current signature is appealing to me for a two reasons:

single arg functions play nicely with Array.prototype.map out of the box
it’s harder to draw a line re: what functionality is included if functionality goes beyond "turn these entries into a generic object"

The proto prop was mentioned, but it seems like Object.setPrototypeOf would keep this clean and without much trouble, no?
const instance = Object.setPrototypeOf(Object.fromEntries(entries), prototype);

Certainly that's a solution; but that creates an object that extends Object.prototype, and then alters its prototype chain later - Firefox had a scary console warning about the performance impact of this for many years.

If there were a second arg, one thing I’d suggest is that it seems it would cover more use cases if it were a target object to assign to, rather than a prototype to use: Object.fromEntries(entries, Object.create(null))

That might indeed be a better solution; but in that case, "from" might not be the proper word to use in the name - all the "from" methods generate a new object.
Regardless of our choice here, I'd hope that the default and most ergonomic usage creates a new object instead of mutating an existing one.

@bathos' idea has merit:
It doesn't introduce a new way to create objects with non-Object.prototype __proto__s.
The second, optional argument is essentially the target. If target is not specified or evaluates to undefined, then Object.fromEntries would create its own target inheriting from Object.prototype.
The implementation is essentially:
Object.fromEntries = function (entries, target = {}) {
  for (const [key, value] of entries) {
    target[key] = value;
  }
  return target;
}

This does not solve the issue raised by attempting to use Object.fromEntries the callback for Array.prototype.map, as the second argument passed to Object.fromEntries would be a number.
One (gross) possibility would be to allow the user to call Object.fromEntries.call(Object.create(myProto), entries) and introspect on the this value within Object.fromEntries — this has a failing in that Object would be the this in the typical case.
I'm leaning more toward if you need a custom prototype for your object, perhaps a different function than vanilla Object.fromEntries is more appropriate.

I don't think we should be motivated by "passing it as a callback to map and friends"; that should be done in an arrow function to ensure explicit passing of args rather than implicit ones.


Issue_45 and Issue 46

Added new Room "Small Book Corner" to the ZDD Text Adventure (Issue 45)
Added new Item "Laptop" to the ZDD Text Adventure (Issue 46)

Hello @SophZoe ,
Your code is clearly structured and easy to understand. You have named your variables in a naturally understandable way, your comments within the code are clear to understand. I also like the way you have described the journey through the space you have created and the interactions.


Typescript - "cannot find name OffscreenRenderingContext" error when upgrading to 3.x.x version

I am developing an Angular 12+ platform. I am currently using PDF.js v2.16.105 but whenever I try to upgrade to 3.x.x (eg. 3.2.146) application doesn't build anymore due an error coming from pdfs dependency itself.
I obtain the following error coming from the file text_layer.d.ts: cannot find name OffscreenRenderingContext
Configuration:
PDF.js version: 2.16.105 -> 3.x.x
Angular version: 15.0.4
Typescript version: 4.8.4
It seems to me an issue due the fact since 3.x.x the types are now included within the library but this one cannot be found.

Try to upgrade your typescript version.
I faced the same issue, and I've upgraded my typescript version to 4.9.4 and the error was gone

Unfortunately Angular 15 doesn't support TS 4.9.4 yet, a bunch of warning and errors might popup. It makes more sense to me to fix the issue even for those ones not using the latest release of TypeScript.

If this works simply by updating to the latest TypeScript version, according to https://github.com/mozilla/pdf.js/issues/15915#issuecomment-1380201330, then that's really the preferred/correct solution here.
Please keep in mind that PDF.js is a JavaScript project, which only provides type-definitions. Even keeping that working takes a fair amount of effort/time for the (few) core PDF.js contributors, who do not know TypeScript, and thus trying to support older TypeScript versions is most unfortunately not really feasible.
(Also, note that Angular isn't something that we use here and thus can't provide support for.)

@giuseppebrb
I encountered this same thing, still struggling a lot to get this working in my Angular13 repo.
I also couldn't find anything on https://github.com/microsoft/TypeScript relating to OffscreenRenderingContext
I added this to my main project's root src/typings/index.d.ts which should be then captured by something in your tsconfig.app.json (see 'include': [ 'src/**/*.d.ts' ]
// some comment about why this exists relating to pdfjs-dist install
interface OffscreenRenderingContext {}

It's still not fixing my import issues, but it's a simple way to override the typescript/buld error.


README feedback

I am a complete n00b to Helm charts.
I’d like to see the following in the README:

What is a Helm chart?
What is the use case for using Helm charts with Nautobot?
What problem does this solution solve?
Once this solution is installed, what then? How is it used?
Screen shots of this solution in action
The README says This repo is intented to house helm charts for the Nautobot project. (Yes, today that is only the Nautobot chart, but we are ready for more).  Could there be multiple Helm charts for the Nautobot project?  Why would there need to be more than one?  What is the use case for more?

Having this info in the README will help a user who may be unfamiliar with Helm charts to determine if this solution can help them. Otherwise, this solution will only appeal to people who already understand the use case and already understand how a Helm chart adds value.

@tim-fiola did we get this one resolved?

Yeah.  looks good!


Ensure that we install Thrust and Cub correctly.

Since thrust ships a thrust-config.cmake in the source tree, a re-configure of cmake can disable the built-in thrust install
rules.
In addition thrust 1.17 has a bug in how it install cub so instead rapids-cmake needs to manage installing but packages.

@gpucibot merge


Add forgot/reset password function in frontend for registerd accounts

Adds forgot password/reset password functionality to the frontend.
Current process flow of resetting the password:

User clicks reset password
User enters username and email-id for password reset link
Tokenized reset link is sent to registered email-id
User can reset the password on clicking the link if token has not expired and is still valid

Hi @JarbasAl any updates on reviewing this PR ?

deprecated by #44


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #347   +/-   ##
=======================================
  Coverage   85.04%   85.04%
=======================================
  Files           1        1
  Lines         214      214
=======================================
  Hits          182      182
  Misses         32       32

Continue to review full report at Codecov.

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

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New config options for continous scroll and Open ID Connect Auth are missing in JSON schema

Currently the config module treats the UI config as a generic map. As a consequence the ui option only can be overwritten as a whole.
The config module needs to be reviewed and refactored, enabling partial overwrite of options and with proper validation of all sub options.


(IGNORE) Clarify use of test cert/key and general doc edits

Doc-only PR to try to clarify use of test cert/key when using c2patool, along with numerous clean up edits.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 48.68%. Comparing base (4ed1799) to head (f787026).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #193   +/-   ##
=======================================
  Coverage   48.68%   48.68%
=======================================
  Files           4        4
  Lines         684      684
=======================================
  Hits          333      333
  Misses        351      351

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

@mauricefisher64 I removed the Example column from the table in the "Configuring trust support" section because it made the other columns very narrow and hard to read. The example values are very straightforward anyway... I also changed the wording a bit and added another example to help clarify.


Fix lint/tsc issues when the strict tsc flag is true

Recently the Stacktrace team have found issues in their spas/libs that could have been caught earlier by the tsc if strict was turned on, particularly strictNullChecks. If we try to turn on strict in our spas/libs we get errors that originate in skyux-builder because the builder is built in to the spas/libs. This PR aims to do the least amount of work possible to allow consumers to turn on strict if they choose.
It fixes the linting issues without changing any functionality or public / private api of skyux builder, mostly by:

Changing the type with the issues into a union of undefined
Using the Non-null assertion operator

Codecov Report

Merging #464 into master will increase coverage by <.01%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master     #464      +/-   ##
==========================================
+ Coverage   99.42%   99.42%   +<.01%
==========================================
  Files          71       71
  Lines        1914     1915       +1
  Branches      305      305
==========================================
+ Hits         1903     1904       +1
  Misses         11       11

Flag
Coverage Δ

#builder
100% <ø> (ø)
:arrow_up:

#runtime
96% <85.71%> (+0.01%)
:arrow_up:

#srcapp
100% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

runtime/bootstrapper.ts
100% <ø> (ø)
:arrow_up:

runtime/i18n/host-locale-provider.ts
100% <ø> (ø)
:arrow_up:

runtime/pact.service.ts
41.17% <0%> (ø)
:arrow_up:

src/app/app.component.ts
100% <100%> (ø)
:arrow_up:

runtime/i18n/resources.service.ts
100% <100%> (ø)
:arrow_up:

...time/directives/sky-app-link-external.directive.ts
100% <100%> (ø)
:arrow_up:

runtime/params.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@Blackbaud-BobbyEarl I took a look and I think I'm fine with these changes, but I'd like to get your take on it as well.

Hey @mattfurness , can you clarify how I might best see these errors prior to your changes?  I ran skyux new, updated the tsconfig.json file to include strict and strictNullChecks in the compilerOptions section.  I do get a couple errors, but none that would seem addressed by your PR.

The errors I found and fixed here were from turning on strict temporarily in this repo and running the tests. I have created a temporary PR in the internal BB sky-lib-p2p-events repo that shows around 11 errors from builder in the failedskyux test step.  My hunch is that you need to import from @blackbaud/skyux-builder/runtime explicitly to start seeing the errors.

Hi y'all. I work with @mattfurness. We noticed that you have updated a bunch of the files in @blackbaud/skyux-builder/runtime to import from other precompiled npm packages. Fantastic work! This has resolved most of the compile errors that result from setting strict: true.
We have updated our PR to fix the one remaining error. You can reproduce this yourself by running skyux new, setting compilerOptions": { "strict": true } in tsconfig.json, and then updating HomeComponent to import SkyuxConfig as follows:
import { Component } from '@angular/core';
import { SkyAppConfig } from '@blackbaud/skyux-builder/runtime';

@Component({
  selector: 'my-home',
  templateUrl: './home.component.html'
})
export class HomeComponent {
  constructor(config: SkyAppConfig) {
    console.log(config);
  }
}

We'd really appreciate you accepting this PR, as strict: true will significantly improve our ability to handle null and undefined properly.

@mattfurness and @bentefay.  First off, my sincerest apologies for being unresponsive on this issue.
Thanks for hanging with us.  This LGTM, but will look for additional validation from @Blackbaud-PaulCrowder and @Blackbaud-SteveBrush.


target .net framework vs .net core in cs-script <IP_ADDRESS>

Indeed, the current release of CS-Script targets .NET Core family runtime (.NET 6).
However, I still needed to write, debug, and test code in .net Framework 4.0.
How can I configure cs-script to target .net 4.0?
My system info is provided below.
cs-script.npp <IP_ADDRESS>
cs-script <IP_ADDRESS>
Window 10 Pro 10.0.19044 N/A Build 1904
C:\Program Files\dotnet>dotnet.exe --list-runtimes
Microsoft.AspNetCore.App 3.1.15 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.18 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.15 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.15 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.6 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
C:\Program Files\dotnet>dotnet.exe --list-sdks
5.0.300 [C:\Program Files\dotnet\sdk]
C:\Program Files\dotnet>reg query "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\NET Framework Setup\NDP"
HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\NET Framework Setup\NDP\CDF
HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\NET Framework Setup\NDP\v4
HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\NET Framework Setup\NDP\v4.0

It is a difficult one. Since .NET Framework is no longer in the .NET roadmap CS-SCript also has switched to .NET Core family.
You can definitely install CS-Release that targets .NET Framework (from here).
When it comes to Notepad++ you would be able to simply install an older version of the plugin but unfortunately, Notepad++ team has implemented recently some hosting API changes that invalidate some many older plugins including CS-Script.
Thus you will need to use Visual Studio for debugging. Or...you can use VSCode with CS_Script extension that supports debugging in the same IDE.

I use notepad++ (npp) portable. I'm considering downloading an older version of the application just to use cs-script.npp targeting .net 4.0.
I went back to npp 8.0.0 and  CSScriptNpp.<IP_ADDRESS>.x64 but intellisense wasn't working.
Would you happen to recall the combination of npp and cs-script.npp that was the most stable (i.e., all features available including intelligence) when targeting .net 4.0?

This combination seems to work:


wallet instance lifecycle

Define the Wallet Instance lifecycle by a formal state machine definition.

Solves https://github.com/italia/eudi-wallet-it-docs/issues/214

@balanza watch also this: https://github.com/italia/eudi-wallet-it-docs/actions/runs/8521706303/job/23340414290?pr=242#step:4:61
if possible, please fix in your next round of commits.


[Feature] When the content of the code block is too long, the class .cm-gap hides part of the code block content.

Initial checklist

[X] I agree to follow the code of conduct
[X] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Problem
When I use the code block component codeBlockComponent and the code content is too long, the class .cm-gap hides part of the code block content.
This seems to be a configurable option for the CodeMirror editor, but how can I configure the codeBlockComponent to display the full content of the code block, so that no matter how much content there is, the cm-gap does not hide any content?

Solution
The codeBlockConfig exposes configurable options.
Alternatives
null

I've already exposed the codeBlockConfig.extensions from code mirror, could you please describe more precisely about how you want to configure codemirror?

https://discuss.codemirror.net/t/improve-scroll-performance-tradeoff/8825
From the above link, it seems that due to performance reasons, the new CodeMirror editor does not provide configuration options. The documentation I looked at some time ago for the older version seemed to offer configurable options, which is a bit awkward now. 😅

Since I want to obtain complete HTML DOM elements after editing multiple lengthy code blocks and display them on a webpage, if .cm-gap hides some actual DOM elements, it will not be possible to get the complete HTML.


feat(footer): arrange uneven margin and change hover color

Fixes #4563 #4953
Describe the changes you have made in this PR -
This PR addresses the issue of uneven margins and lack of hover effects color on footer links for better UI. The following changes have been made:

Adjusted the margins of footer elements to ensure a consistent and balanced layout.
Added a hover effect on footer links, changing the link color on hover for better user experience and accessibility.

Screenshots of the changes (If any) -
The proposed Figma design can be accessed here.
After implementing this pull request, the footer will resemble the design showcased below:

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Summary by CodeRabbit

New Features

Redesigned footer layout for a more balanced and cohesive appearance.
Enhanced spacing between social icons for improved visual clarity.
Updated hover state color for footer links, providing clearer interaction feedback.

Bug Fixes

Improved text alignment for the copyright section to enhance overall aesthetics.

Style

Adjusted CSS styling for better responsiveness and user experience, including class modifications and increased column widths.

Hii @tachyons, We Should merge this PR and the design was approved in issue #4563
as the footer partial  is migrated to the view component that's why I made this PR to resolve this uneven margin issue


kernel indicator not appearing

Description:
The kernel status indicator is no longer appearing in Atom when I run python kernels (the button on the bottom bar of the Atom window that shows the kernel is running and gives options to shutdown or restart the kernel when clicked).
Steps to Reproduce:

Open a python script
Execute a selection
Observe that code executes (output appears), but kernel indicator does not appear. Keyboard commands to shutdown kernel still work.

Two days ago this worked fine. I think I updated Hydrogen since then, but I had a lot of things going on (OS crashes, python updates) and I'm not 100% sure that the problem appeared exactly when the Hydrogen updates took effect. I have tried uninstalling and reinstalling Atom and Hydrogen, and clearing window state.
Versions:
OS: Windows 10
Python: 2.7.12 and 3.6.6
Hydrogen: 2.51
Atom: 1.29.0
Electron: 2.0.5
Chrome: 61.0.3163.100
Node: 8.9.3
Logs:
This is what my debug log looks like. Enable Debug Messages is turned on for Hydrogen.
C:\Users\Brad\.atom\packages\chary-tree-view\lib\chary-tree-view.js:11 Uncaught (in promise) TypeError: Cannot read property 'entryClicked' of undefined
    at atom.packages.activatePackage.then.treeViewPkg (C:\Users\Brad\.atom\packages\chary-tree-view\lib\chary-tree-view.js:11:45)
    at <anonymous>
<embedded>:105004 Unable to locate git workspace root for C:\Users\Brad\Google Drive\Research\Python3. Expected if C:\Users\Brad\Google Drive\Research\Python3 is not in a git repository. GitError: git rev-parse --show-toplevel in C:\Users\Brad\Google Drive\Research\Python3 exited with code 128
stdout:
stderr: fatal: not a git repository (or any of the parent directories): .git

    at new GitError (C:\Users\Brad\AppData\Local\atom\app-1.29.0\resources\app\node_modules\github\lib\git-shell-out-strategy.js:121:24)
    at Promise (C:\Users\Brad\AppData\Local\atom\app-1.29.0\resources\app\node_modules\github\lib\git-shell-out-strategy.js:385:29)
    at <anonymous>
revParse @ <embedded>:105004
4<embedded>:22769 'Hydrogen.kernelspec' could not set the default. Attempted default: undefined; Schema: {}
setDefaults @ <embedded>:22769

I don't see hydrogen related items in the output there, except for the unrelated kernelspec config thing I've been meaning to fix
Just to be sure, can you make sure your hydrogen settings have not disabled the status bar? This is an available setting.

Thanks Ben, was it. I must have somehow clicked that setting without noticing. I feel a little foolish now.

Glad you got it working!


Transitions in data card

The data card initially contains a default value of 0. Make some animations in the card such that it will only appear when there is the data and all the other times it will disappear.

This issue is completed and merged successfully.


🚀  Adding Create Post Form

Added a post creation form to let users have the ability to publish posts.
Used django CreateView class.

Added a CreateView for a new post.
Added a redirection function on the post model to be used after creating new post.
Set the post creation page url.
Added a post creation html template.
Added relevant tests.

I tried to not include the post creation html template in this PR in order to not include UI but I had to set a html page for testing purposes.
Future tasks:

Adding the ability to create a job-offer post (#96)
Adding a link to the create new post form (#95)
Style the form using crispy-forms (#94)

Close #92

Added a post creation form to let users have the ability to publish posts. Used django CreateView class.

Added a CreateView for a new post.
Added a redirection function on the post model to be used after creating new post.
Set the post creation page url.
Added a post creation html template.
Added relevant tests.

I tried to not include the post creation html template in this PR in order to not include UI but I had to set a html page for testing purposes.
Future tasks:

Please add an Issue here

Adding the ability to create a job-offer post
Adding a link to the create new post form
Style the form using crispy-forms

Close #92

Added a post creation form to let users have the ability to publish posts. Used django CreateView class.

Added a CreateView for a new post.
Added a redirection function on the post model to be used after creating new post.
Set the post creation page url.
Added a post creation html template.
Added relevant tests.

I tried to not include the post creation html template in this PR in order to not include UI but I had to set a html page for testing purposes.
Future tasks:

Please add an Issue here

Adding the ability to create a job-offer post
Adding a link to the create new post form
Style the form using crispy-forms

Close #92

Created the issues and edited this PR description to include them.

@paOmer Nice work, I just do not see any client tests in the PR. When we test View/Form it is not guaranteed the the hole path is not broken from http request to http response

Updated the tests accordingly.


Edit Pinboard Form

We need the following option(s) at the "edit pinboard" - form:

on creating a new column, a button to put the column to the top of the list
additional buttons to the existing columns to move them directly to the top/bottom of the list

Is it planed to implement these functionality soon?

With the last version, you can change the position of the columns by drag and drop.


typeahead with headers

There's any option available to create Headers on typeahead like in Multiple Sections with Headers example here.
Cheers

@kefengdeng-ibboost, please do not use the issues forum for support related-requests.  Rather, please follow the instructions here.  The issues forum is reserved for bugs.  Thanks.

You can get something like this working now by using a custom template and sorting the results. See this question on stackoverflow.

That's really good to know. Thanks a lot.

Closing this as support - looks like it's doable with custom templates.


Websocket docker

TODO:

[x] Deploy docker images somewhere

The proxy is at: wss://websocket-tcp-proxy-fywbx.ondigitalocean.app, hopefully soon is at wss://proxy.mutinywallet.com
You can put that in to the proxy box. If you want to expose a local port so you can connect to, you can run bore: bore local {your_local_port} --to bore.pub and note the port you get.
Then that connection string would be something like {pubkey}@<IP_ADDRESS>:{bore_port} - just replace the port with the one you got from bore


[hotfix] fix javadoc link of ClusterClient#triggerSavepoint

What is the purpose of the change
fix javadoc link of ClusterClient#triggerSavepoint

merging.


Enable copy and pasting of nodes

Use as reference https://github.com/paceholder/nodeeditor/pull/252

Someone could not notice, but as copy is done using json MIME Data, a node copied from the scene can be pasted as raw json in any text editor or wherever string MIME data are supportod

Someone could not notice, but as copy is done using json MIME Data, a node copied from the scene can be pasted as raw json in any text editor or wherever string MIME data are supportod


cargo no std output on windows

Cargo doesn't return anything to cmd or powershell. I've installed git bash on windows and there it works perfectly.
Cargo version: cargo 0.21.0-nightly (50b1c24d1 2017-06-17)
OS: Windows 10

Thanks for the report! I think this was a bug that was fixed recently, mind updating Cargo and seeing if it works?

Hi. It works but I think it displays linux bash style output on windows as well.
?[m?[m?[31m?[1merror:?[m could not find Cargo.toml in C:\Users\singhgag or any parent directory
where as on bash it displays error in red color

What shell are you invoking Cargo in? What's the value of your TERM environment variable?

Oh Mistake. I was using hyper that gives this error. Thankyou :) I'll close the issue.


Good session

I want to attend -

Hi xu,sorry for not noticing your comment .I shared another session 'how to play music in html5',you can view it on https://kkkkkxiaofei.github.io/jekyll/update/2015/06/11/lyric.html


Update README and versions for 23.06 branch (#708)

Update README and versions for 23.06 branch

Update README.md for 23.06

LGTM
out of curiousity, why does this PR not mirror this one: https://github.com/triton-inference-server/model_analyzer/pull/708/files

LGTM
out of curiousity, why does this PR not mirror this one: https://github.com/triton-inference-server/model_analyzer/pull/708/files

True, it very similar but has some differences to the READMED.md file, and that's the only reason.
However it also may contain some release related commits.

LGTM
out of curiousity, why does this PR not mirror this one: https://github.com/triton-inference-server/model_analyzer/pull/708/files

True, it very similar but has some differences to the READMED.md file, and that's the only reason. However it also may contain some release related commits.

Thanks for the explanation.


Requirement list incomplete

Your requirement list in the README shows only a couple of PHP requirements, but what about all the Bower/Gulp/NPM stuff?
Obviously, for just running it, PHP is enough, but I will probably have to make my own design and add some functionality, so I would like to know what exactly I will need to achieve this.

The list is comprehensive - it's a list of requirements.
You may choose any development tools of your preference, but we don't compile core assets and so on. You just need composer for dependencies.

Then what's with the following files:
bower.json
gulpfile.js
package.json
?

These are part of Laravel

https://github.com/laravel/laravel/tree/5.1
bower.json does not exist (same for 5.0)
gulpfile.js contents are quite different: https://github.com/laravel/laravel/blob/master/gulpfile.js
package.json contents are also different

See development branches to find the preferred Laravel version. We try to keep "inherent" files as close to the Laravel master as possible, for developers' sake, but you should expect minor differences. Feel free to modify them as you see fit, they have no direct relation to Microweber code.

So they don't actually matter for this project?

Depends on what you mean exactly, but you can delete them and everything will still work. I hope that answers the question.

Yep, that's what I meant.
Is it a weird design decision on Laravel's part to include those files or is it that this project actually deleted the dependencies on those files/isn't relying on them?

We're planning some changes in that regard, since right now things aren't exactly where we want them, with regard to your comment, but for the record they serve a "bootstrapping" purpose, since many developers usually create some of these files anyway.

hi @jurchiks
those files are not really used, its safe to be deleted.
they left from earlier laravel version https://github.com/laravel/laravel/tree/5.0

So you do not use any JS package managers or CSS precompilers or anything like that?

nothing else besides composer for the PHP packages

Ok, then I'm closing this issue.


Create wiki and example for Cesium Client js

create a wiki for people to get started with cesium js with GEE tile support.
add example
add screenshot

Initial wiki posted here
https://github.com/google/earthenterprise/wiki/Cesium-3D-Client-Library

here is screenshot. I cannot update the wiki with the images beacause only way to do it is through the url. We should decide a common place to upload the wiki images.

@asmitaw Sorry, I just noticed this issue. We have the start of a procedure for adding images to the Wiki. We're working on writing it up now. Basically you will create a fork of my copy of the Earth Enterprise Wiki repo. Then add your images into the /images directory and create a pull request from the above. You can use this pull request as an example.
Also, if you want to preview it, you can push it to your personal wiki (of any repo I think).
This process is the best I can come up with for doing PRs on the wiki.

Looks like the only thing missing is the screenshot. I think enough has been done to close the ticket.


windows_feature_powershell resource fails on PowerShell 3.0

Cookbook version
4.3.2
Chef-client version
13.9.4
Platform Details
Windows 2008 R2, PowerShell 3.0
Scenario:
Install Windows Feature w/ windows_feature_powershell install method
Steps to Reproduce:
windows_feature 'RDS-RD-Server' do
  action :install
  install_method :windows_feature_powershell
end

Expected Result:
Feature is installed
Actual Result:
Error executing action install on resource 'windows_feature[RDS-RD-Server]'
 RuntimeError
 windows_feature_powershell[RDS-RD-Server] (c:/chef/cache/cookbooks/windows/resources/feature.rb line 47) had an error: RuntimeError: The windows_feature_powershell resource requires PowerShell 3.0 or later. Please install PowerShell 3.0+ before running this resource.

Notes:
I believe there is a typo in feature_powershell.rb. Should read ['languages']['powershell']['version'].to_i >= 3
def raise_on_old_powershell
    # be super defensive about the powershell lang plugin not being there
    return if node['languages'] && node['languages']['powershell'] && node['languages']['powershell']['version'].to_i > 3
    raise 'The windows_feature_powershell resource requires PowerShell 3.0 or later. Please install PowerShell 3.0+ before running this resource.' if powershell_version < 3
  end

Fixed in 4.3.3. Thanks for the report


[AVM Module Issue]: virtual machine module diskname

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
Module Name
avm/res/compute/virtual-machine
(Optional) Module Version
0.2.3
Description
I am trying to change the defaultname off the osdisk within the virtual machine template, the syntax looks right but it seems to ignore my settings. I included the example from my code
osDisk: {
  caching: 'ReadWrite'
  createOption: 'FromImage'
  deleteOption: diskDeleteOption
  diskSizeGB: '128'
  managedDisk: {
    storageAccountType: diskStorageAccountType
  }
  name: 'OSDISK'
}

(Optional) Correlation Id
No response

@jkamping Thanks for bringing it up. I will look into it

@jkamping Please use the newest version 4.0 of the VM module. There it is fixed.


Mentioning someone in tags doesn't notify them

Should mentions work in tags? I told someone "tag me in your observation @carrieseltzer" and he did exactly that, but I didn't get a notification from iNaturalist.
Link to observation: http://www.inaturalist.org/observations/3784985
If this isn't an easy fix I'll just be more explicit with my instructions to new users. Usually they use the app so it isn't an issue since there isn't a tags field there.

IMO this is an extreme edge case that has more to do with people not understanding what the very "tag" in this context. I'd say close this unless there are strong objections.

Agreed. I'll be more explicit in my instructions to new users in the future. Thanks!


Create Event System

Probably worth creating a subscription event system so we can better listen for events. Will be especially useful for handling async work.

Moving up to the UI milestone as it is likely this will be needed for under the hood API changes

Available events will be searchable via :h netman-events
They will be added as needed. Starting out with the following events
    - netman_provider_load
    - netman_provider_unload

The event system will "live" inside the API (since the API is where most everything comes and goes, its a great time to fire events). The event system should probably also consider exposing the event system via :h aucommands, to allow for providers that aren't directly using lua (I don't believe this is currently an option, but its better to not code ourself into a corner).
I envision this working as such
-- api.register_event_callback
-- @param event string
--     The event you wish to listen for. Must be a valid `:h netman-events` event.
-- @param callback function
--     The function you want me to call back.
--     When the event if fired, your function will be called, and provided a `data` param. The contents of the param will be
--         - event string
--             - The event that was fired
--         - source string
--             - The thing that triggered this event. May be nil, and will likely contain the name of the URI on things like
--                remote saves/reads/stat/exec, etc
-- @return integer
--     An integer ID that will be unique to your registered callback. Must be used to unregister. @see api.unregister_event_callback
-- @throws
--     INVALID_EVENT_ERROR
--        An error that is thrown if the requested event is not valid per `:h netman-events`
--     INVALID_EVENT_CALLBACK_ERROR
--        An error that is thrown if there is no callback provided
-- @example
--     -- Dummy callback, use the data provided to it however you want
--     local callback = function(data) print(vim.inspect(data)) end
--     require("netman.api").register_event_callback("netman_provider_load", callback)
api.register_event_callback(event, callback)

-- api.unregister_event_callback
-- Unregisters the callback associated with the id
-- @param id integer
--     The id that was provided on the registration of the callback. @see netman.api.register_event_callback
-- @example
--    local id = require("netman.api").register_event_callback('netman_provider_load', function(data) end)
--    require("netman.api").unregister_event_callback(id)
api.unregister_event_callback(id)

-- api.emit_event
-- Emits the event, and will also call any functions that might care about it
-- @param event string
--     A valid netman event
-- @param source string | Optional
--    Default: nil
--    If provided, this will be the URI source of the event.
-- @example
--     require("netman.api").emit_event("netman_provider_load")
api.emit_event(event, source)

Completed with #129 , though it is not yet in use.


Can't access EditorFileSystem

Tested versions
Godot 4.2
System information
Godot v4.2.2.stable - Ubuntu 22.04.4 LTS 22.04 - X11 - GLES3 (Compatibility) - NVIDIA GeForce GTX 1060 3GB (nvidia; 535.171.04) - AMD Ryzen 9 3900X 12-Core Processor (24 Threads)
Issue description
an example from the docs do not work for me:
var editor_settings = EditorInterface.get_editor_settings()
and the same for  EditorFileSystem
It returns error:

Steps to reproduce
Just copy paste the code I provided and try to run
Minimal reproduction project (MRP)
Bug report.zip

Try running with @tool, this is editor only so won't be available in a running game

Try running with @tool, this is editor only so won't be available in a running game

Ah... Thank you. Didn't see it mentioned in the docs


Pass through files if no format is configured

Currently patternplate ignores any file that has no matching format defined in configuration/patternplate-server/patterns.js. Setting up a patternplate project thus requires a user to configure formats and transforms every single time.
I think we should discuss if defining a set of pass-through formats/extensions that mirror the standardized web formats: html, js and css.
Example
# Given no configuration at /configuration/patternplate-server/patterns.js
patterns
|- button
|  |- pattern.json
|  |- demo.css # passed through
|  |- index.css
|  |- index.html # passed through
|  |- index.js # passed through

These pass-through formats would be delivered with (almost) no transformation. This concept works reasonably well for patterns without dependencies or source files with the need to import anything.
Importing other pattern sources
Given resolving and linking currently is part of the concern transforms need to solve there is no obvious way to introduce patternplate-conforming import resolving to pass-through pattern sources.
An idea on how to express imports in the pass-through formats. Please note: This disregards implications to current patternplate architecture.
<!-- index.html -->
<link rel="import" href="pattern://dependency">
<div>
   <dependency/>
</div>

/* index.css */
@import 'pattern://dependency';

/* index.js */
const dependency = import 'pattern://dependency';

Open questions

Should we handle imports in passed through source files in any patternplate-specific way?
If yes: When and how to resolve imports in passed through source files?
If yes: How to handle errors in pattern resolving?
If no: Whats our modularization story for pass-through formats?

cc @KnisterPeter @Dangoo

What do you mean with #used?
I've add pattern.json to the files tree.

@KnisterPeter: I changed # used to # passed through and added a comment about pattern.formats configuration being not there. Hope this makes things clearer.

Prototypical changes to patternplate-server:
https://github.com/sinnerschrader/patternplate-server
Demo project:
https://github.com/sinnerschrader/patternplate-getting-started/tree/feat/%23115-demo-passthrough-for-webformats
Deployed demo:
https://sinnerschrader.github.io/patternplate-getting-started/demos/passthrough-webformats/

@KnisterPeter @LKuechler could you have a look at the prototype and give some feedback? More ideas for implementing pattern imports in the passthrough formats are welcome.

Something does not work with the button-row pattern

I don't like the html-import. The other formats could only be used if the pattern dependency resolver learns about uris/urls before.
There is no real good way to allow passthrough with modularization, because html/css/js does not support it on their own.
We should discuss this very seriously. Otherwise this will cost a lot for later versions.

@KnisterPeter Concerning the html imports i think about emulating / wrapping the behavior you'd expect from browsers natively. This is still pretty rough in my head, but do you get the general idea? Perhaps something like https://github.com/felipeccastro/html-component shows the correct way to go about this.

Obsolete as we will remove the transform chain


[Event Request] page 6510 "Item Tracking Lines" - OnBeforeAddToGlobalRecordSet

We would like to add a new event OnBeforeAddToGlobalRecordSet in the following procedure:
local procedure AddToGlobalRecordSet(var TempTrackingSpecification: Record "Tracking Specification" temporary)
var
    ExpDate: Date;
    EntriesExist: Boolean;
begin
    TempTrackingSpecification.SetCurrentKey("Lot No.", "Serial No.");
    if TempTrackingSpecification.Find('-') then
        repeat
            TempTrackingSpecification.SetTrackingFilterFromSpec(TempTrackingSpecification);
            TempTrackingSpecification.CalcSums("Quantity (Base)", "Qty. to Handle (Base)",
              "Qty. to Invoice (Base)", "Quantity Handled (Base)", "Quantity Invoiced (Base)");
            if TempTrackingSpecification."Quantity (Base)" <> 0 then begin
                Rec := TempTrackingSpecification;
                Rec."Quantity (Base)" *= CurrentSignFactor;
                Rec."Qty. to Handle (Base)" *= CurrentSignFactor;
                Rec."Qty. to Invoice (Base)" *= CurrentSignFactor;
                Rec."Quantity Handled (Base)" *= CurrentSignFactor;
                Rec."Quantity Invoiced (Base)" *= CurrentSignFactor;
                Rec."Qty. to Handle" := Rec.CalcQty(Rec."Qty. to Handle (Base)");
                Rec."Qty. to Invoice" := Rec.CalcQty(Rec."Qty. to Invoice (Base)");
                Rec."Entry No." := NextEntryNo;

                // skip expiration date check for performance
                // item tracking code is cached at the beginning of the caller method
                if not ItemTrackingCode."Use Expiration Dates" then
                    Rec."Buffer Status2" := Rec."Buffer Status2"::"ExpDate blocked"
                else begin
                    ExpDate := ItemTrackingMgt.ExistingExpirationDate(Rec, false, EntriesExist);
                    if ExpDate <> 0D then begin
                        Rec."Expiration Date" := ExpDate;
                        Rec."Buffer Status2" := Rec."Buffer Status2"::"ExpDate blocked";
                    end;
                end;

                //OnBeforeAddToGlobalRecordSet(Rec, EntriesExist, CurrentSignFactor); A109F0006.Event.o
                OnBeforeAddToGlobalRecordSet(Rec, TempTrackingSpecification, EntriesExist, CurrentSignFactor); //A109F0006.Event.n
                Rec.Insert();

                if Rec."Buffer Status" = 0 then begin
                    xTempItemTrackingLine := Rec;
                    xTempItemTrackingLine.Insert();
                end;
            end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeAddToGlobalRecordSet(var TrackingSpecification: Record "Tracking Specification"; var TempTrackingSpecification: Record "Tracking Specification" temporary; EntriesExist: Boolean; CurrentSignFactor: Integer)
begin
    //A109F0006.Event Parameter TempTrackingSpecification
end;

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.


Throw SSHClient.CommandFailed when the status code is non-zero

Thank you for your commit, but I am encountering an issue when using thesshClient.executeCommand method.
When a command fails (i.e., returns a non-zero exit code), I cannot capture the stdin or stderr output.
Am I using the wrong method?
do {
    let buf = try await client.executeCommand(command)
    let result = String(buffer: buf)
    return result
} catch let error as SSHClient.CommandFailed {
    // Missing error reason
    let message = "Command failed: `\(command)`, ExitCode: \(error.exitCode)"
    throw SessionError.executionFailed(message: message)
}

If you're interested in an output, rather than the convenience wrappers around it - look into streamContinuation

Also noteworthy: #71 adds support for an stdin stream as well. But awaiting feedback

Thanks!


Make launcher icon removeable

Once the keyboard installation is finished, it's not really necessary to have a
persistent launcher icon.

Look into adding a button to give users the option of removing it. Cf.
http://stackoverflow.com/questions/5659118/allow-a-user-to-remove-app-icon-from-
launcher

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 May 2011 at 5:08

necropost


QString to const char* conversion

I wanted to convert a QString to a const char * but  for some Qstring I had a weird behaviour.
const char* c = path.toStdString().c_str(); does not work as expected
But
std::string pathString = path.toStdString();
const char* c = pathString.c_str(); do the job as it should
for example:
QString pathToFolder = /home/pauline/simulation/essai/Creso
if the convertion is done with the first method the 8 first character are "empty"
if the conversion is done with the second method everything is fine.
see:  https://stackoverflow.com/questions/17343714/convertion-from-qstring-to-char-pointer-generates-empty-string

This is because c_str() points to data contained in the std::string. I didn't realise this before (but it's actually obvious now), so all instances of .tStdString().c_str() in our code should be modified if the returned pointer is kept.
Well noticed. This is a big memory bug.

I checked all the code and found no problematic cases.

Ok good.


Overvoltage limit overflow fix

the bat config calculation did calculate the absolute max voltage
too high for NMC/NMC_HV, it was doing an unnecessary multiplication.
This triggered an issue in the voltage limit set code, which did not
handle the overflow of the 16bit limit value properly.
Both are fixed now. We limit the limit value to what the ADC is able
to deliver, so that we detect overvoltage for sure, even if the setting
is out of ADC range.

You are right about 0xFFF0.
Your simplification was my previous solution, but I abandoned it in favor of the current one. The problem is that for very large float input values the result may exceed the uint32_t range and we would not notice this here. For that reason I switched to using float compare which handles this without problems (we get "inf" in this case).

Alright, I understand. Although it's not very probable, as the user must be very far off the original value that is normally read by the ADC. We could even increase the range and do the shift by 4 bits as the latest step in the return statement :)

Yes, I don't think such a wrong value would happen by user interaction, more by a programming mistake or something like this. Of course, I also agree this is far fetched. But safe than sorry.

By the way, one question: Why did you choose const variable here. Is it only a matter of showing that they will not be changed anymore, or does it give any speed improvements?
And could you please also add the unit-test? Otherwise I need to add it in a separate commit later on.

No speed improvements, the compiler is of course smart enough to figure out that these values are never changed. However, I started to place const if I want to express, this is a variable which should be not be changed over its lifetime. With 'const' the compiler prevents accidental breaking this rule. Of course again not necessary here.
I will add the unit test here now.

I will add the unit test here now.

Great. And do you also update the comparison in adc_update_value, as mentioned above? Thanks!

No speed improvements, the compiler is of course smart enough to figure out that these values are never changed. However, I started to place const if I want to express, this is a variable which should be not be changed over its lifetime. With 'const' the compiler prevents accidental breaking this rule. Of course again not necessary here.

Ok, thanks for the explanation. So that's a bit like in Rust, where you have to explicitly define that a variable can be changed later on, otherwise they are always constant.

Ok, done with this, can be merged.

Please have a look at the last commit in this branch https://github.com/hogthrob/MPPT-Charger_Software/tree/overvoltage_fix and see if you think this is a good idea.
I used some C++ concepts to avoid code repetition. It will not change the performance significantly (or increase/decrease code size).


For setting up in the local environment in Windows

Fix: #442
Issue with setting up the code locally for contributor in Windows
Context
While working on the package/shared directory, running the pnpm run build command was functioning correctly in WSL (Windows Subsystem for Linux) but failing in native Windows environments. This issue occurred because the rm -rf ./dist command used in the prebuild script is a Unix/Linux command and is not supported natively in Windows.
Error Details
When running pnpm run build in the package/shared directory on Windows, the following error was encountered:

This prevented the build process from completing successfully.

Resolution
To address this, the prebuild script in package.json was updated to use a cross-platform solution. This ensures compatibility with both Linux and Windows environments.
Changes Made

Replaced the rm -rf ./dist command in the prebuild script with the rimraf package, a cross-platform tool for deleting files and directories.
Updated the package.json file in the package/shared directory.

Outcome

The updated prebuild script ensures compatibility across both Linux and Windows platforms.
The issue caused by the unsupported rm -rf ./dist command is resolved.
Files now run locally on Windows without errors, and the workflow remains unaffected for Linux users.

Screenshot of the Final Output

This could help to fix the error for upcoming contributors independent of their OS

I added worked on this on best of my knowledge but if you have any better approach for this I will be glad to learn and work on that.
PS: I confirmed that this file is automatically created and may be this is happening in Windows. Adding few resources to check
https://github.com/vitejs/vite/issues/13267
https://stackoverflow.com/questions/76510164/can-i-safely-delete-vite-config-ts-timestamp-files