Split-Path removes ¨´´ from Service Pathname

From @francesco1119 on February 25, 2019 3:27
It´s as simple as that Split-Path is removing the last '"' from every Pathname you query form any service:
$service = get-wmiobject -query 'select * from win32_service where name="SQLBrowser"';
Write-output $service.pathname
Write-output $service.pathname | Split-Path

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

ID: 4fc8b0a5-8130-a6e3-7e95-92495fbcb605
Version Independent ID: 7871d186-25d2-78eb-9b50-cd04495ad981
Content: Split-Path (Microsoft.PowerShell.Management)
Content Source: reference/6/Microsoft.PowerShell.Management/Split-Path.md
Product: powershell
GitHub Login: @daxian-dbw
Microsoft Alias: dongbow

Copied from original issue: MicrosoftDocs/PowerShell-Docs#3759

This is a bug. Moving this issue to PowerShell/PowerShell/Issue

The underlying issue is how Split-Path treats paths that have embedded quotation marks (which they shouldn't):
PS> '"c:\tmp\file"' | split-path
"c:\tmp

PS> "'c:\tmp\file'" | split-path
'c:/tmp

Is the issue related only to Split-Path or to other cmdlets and providers?

Based on @mklement0's example, it looks like it is just a problem with Split-Path when the input is a string that contains quotes.

It seems we had such issue for completions (IntelliSense). It was fixed.

I think this is a fairly exotic problem and I'm not sure we need to do anything about it:
Passing a path with embedded quoting is arguably user error.
Split-Path blindly splits by the last \ or / instance, which is normally fine, especially in light of '/subdir and "/subdir being technically valid paths (the latter only on Unix-like platforms), however unlikely.
"C:\..." is not a valid path, however, because " cannot be used in paths on Windows (and if it were a legal char., it couldn't be followed by a drive spec).
So, if we want to do something about this, we could check for the following specific cases on Windows and report an error:

if a " is present anywhere in the path
if a an initial ' is followed by a drive spec / UNC share

It seems we had such issue for completions (IntelliSense). It was fixed.

In this case it was a registry value that was defined with embedded quoting, not a user typing such a value.

So use $service.pathname | Split-Path instead of Write-output $service.pathname | Split-Path.

@iSazonov:
Your two commands are equivalent. What's needed is to strip the embedded quotes.
$service = [pscustomobject] @{ pathname = '"c:\foo\bar baz"' }

$service.pathname -replace '"' | Split-Path

I think this is a fairly exotic problem and I'm not sure we need to do anything about it:
Passing a path with embedded quoting is arguably user error.

Agreed, not sure what we need to do. Checking if a path is valid (not if it exists) is not a trivial thing as one might think.

I now got it: is not Split-Path the problem but how Get-wmiobject win32_service handles the Pathname.
Look at this: if you run Get-Process  | Select-Object Name,Path each path of each process is printed without double quotes:

But if I run get-wmiobject win32_service | select name, pathname I now realize that all the services that belongs to C:\WINDOWS are listed without double quotes while all the services belonging to C:\Program Files have double quotation:

So this is a Get-wmiobject win32_service bug.
Can we select the path through Get-Service insted of using Get-wmiobject win32_service? I would like to check if the bug persists in Get-Service too or if it's just a win32_service legacy
https://stackoverflow.com/questions/31374644/get-process-location-path-using-powershell

I don't think this is a WMI problem. It is not unreasonable to expect that Split-Path should be able to handled quoted paths.

@sdwheeler:
We're not talking about quoted paths here, we're talking about paths that have extraneous embedded quotes.
It's the difference between:
$path = "c:\foo\bar"  # quoted path
$path '"c:\foo\bar"' # (quoted) path with extraneous embedded quotes

In the first case the " have syntactical function, and are stripped after parsing, in the 2nd case they're part of the data.
As stated, the best we could do in Split-Path is to rule out unambiguous errors with embedded " or '-prefixed paths that are followed by a drive spec. / UNC share - on Windows only.

As for the WMI / Get-Service's behavior:
The problem is primarily one of poor naming:
What .pathname / .BinaryPathName / ImagePath (the name of the value in a service's registry definition, which is the ultimate source of the information) actually contain is a command line, not  a single binary's file path.
Such a command line is designed for parsing by a shell / the Windows API, and of necessity must support embedded quoting.
With that in mind, you can use the following code to extract the binary (executable) file path that is embedded in the command line:
    # Sanitize the path:
    # * Some values have encosing "...", so we strip them,
    # * others have arguments, so we only take the first token.
    $binaryPath = if ($binaryPath -like '"*') {
      ($binaryPath -split '"')[1]
    } else {
      (-split $binaryPath)[0]
    }

This SO answer of mine wraps this functionality in a function, Get-ServiceFileInfo.


emmet abbreviations

Hi, i'm very new with coding, so forgive me.
All these abbreviations http://docs.emmet.io/cheat-sheet/ are not working
I'm using sublime text.
My steps
Download and install Sublime Text
Install Package control
Install Emmet
Testing writing for example
html:5
Nothing happens.
 I've read about there is a keymap, but I don't have idea how should I paste that code

Ctrl+E should work everywhere. Try to run sublime.log_commands(True) in ST console and check what command name is invoked when you hit Ctrl+E

aw now work! i've deleted everything wit appzapper, also preferences, and I've installed everything and now it works!
and about they mac os x keymap?
(thank you)

You shouldn’t paste it, Sublime loads it automatically

perfect! issue resolved!


some minor modification

fix the repeatedly alerts when the token or URL is incorrect
fix a bug that update will not be called immediately after new
configurations set

I didn't read the code very carefully, but tried to fix the issues. You may have a look at them whether they are useful

@smilingpoplar @rodoviario Yes this seem to break some configs ... gonna revert this change!


Updated loading notebook for new Steinmetz data in OSF with lots of new data

I changed the local file names to match the ones on OSF so it's less confusing.


yolov5m and yolov5l ?

Hi,
what are correct yaml files for medium and large models?
Exporting small model is ok.
I have tried with medium model, but getting error:
My yaml for medium is:
`
nc: 80  # number of classes
depth_multiple: 0.67  # model depth multiple
width_multiple: 0.75  # layer channel multiple
anchors:

[10,13, 16,30, 33,23]  # P3/8
[30,61, 62,45, 59,119]  # P4/16
[116,90, 156,198, 373,326]  # P5/32

backbone:
[[-1, 1, Focus, [64, 3]],  # 1-P1/2
[-1, 1, Conv, [128, 3, 2]],  # 2-P2/4
[-1, 3, Bottleneck, [128]],
[-1, 1, Conv, [256, 3, 2]],  # 4-P3/8
[-1, 9, BottleneckCSP, [256]],
[-1, 1, Conv, [512, 3, 2]],  # 6-P4/16
[-1, 9, BottleneckCSP, [512]],
[-1, 1, Conv, [1024, 3, 2]], # 8-P5/32
[-1, 1, SPP, [1024, [5, 9, 13]]],
[-1, 6, BottleneckCSP, [1024]],  # 10
]
head:
[[-1, 3, BottleneckCSP, [1024, False]],  # 11
[-1, 1, nn.Conv2d, [na * (nc + 5), 1, 1, 0]],  # 12 (P5/32-large)
[-2, 1, Upsample, [None, 2, 'nearest']],
[[-1, 6], 1, Concat, [1]],  # cat backbone P4
[-1, 1, Conv, [512, 1, 1]],
[-1, 3, BottleneckCSP, [512, False]],
[-1, 1, nn.Conv2d, [na * (nc + 5), 1, 1, 0]],  # 17 (P4/16-medium)
[-2, 1, Upsample, [None, 2, 'nearest']],
[[-1, 4], 1, Concat, [1]],  # cat backbone P3
[-1, 1, Conv, [256, 1, 1]],
[-1, 3, BottleneckCSP, [256, False]],
[-1, 1, nn.Conv2d, [na * (nc + 5), 1, 1, 0]],  # 22 (P3/8-small)
[[], 1, Detect, [nc, anchors]],  # Detect(P3, P4, P5)
]`
Error I am getting is:
Traceback (most recent call last):   File "export.py", line 59, in <module>     simplify_onnx(onnx_path)   File "export.py", line 44, in simplify_onnx     model_simp, check = simplify(model)   File "/home/tom/miniconda3/envs/yolact/lib/python3.7/site-packages/onnxsim/onnx_simplifier.py", line 311, in simplify     res = forward_for_node_outputs(model, const_nodes, input_shapes=input_shapes)   File "/home/tom/miniconda3/envs/yolact/lib/python3.7/site-packages/onnxsim/onnx_simplifier.py", line 160, in forward_for_node_outputs     res = forward(model, input_shapes=input_shapes)   File "/home/tom/miniconda3/envs/yolact/lib/python3.7/site-packages/onnxsim/onnx_simplifier.py", line 144, in forward     sess = rt.InferenceSession(model.SerializeToString(), sess_options=sess_options, providers=['CPUExecutionProvider'])   File "/home/tom/miniconda3/envs/yolact/lib/python3.7/site-packages/onnxruntime/capi/session.py", line 158, in __init__     self._load_model(providers)   File "/home/tom/miniconda3/envs/yolact/lib/python3.7/site-packages/onnxruntime/capi/session.py", line 177, in _load_model     self._sess.load_model(providers) onnxruntime.capi.onnxruntime_pybind11_state.Fail: [ONNXRuntimeError] : 1 : FAIL : Type Error: Shape of initializer 1099 does not match. {} != {96}

From my test, you only need to replace the yaml and pt file used in main.py and replace nn.Upsample by Upsample in corresponding yaml file. It's supposed to work in your case.

This really does not work for any other model than yolov5s

You can have a try on latest commit.

I have tried it, but unfortunately I am getting Segmentation fault (core dumped) error in the step of onnx simplifier ( pytorch 1.4.0, torchvision 0.5.0, onnx 1.6.0, onnx-simplifier 0.2.9, trt 7.0.1 )
last few lines of text dumped from the app:
  %960 : Tensor = onnx::Unsqueeze[axes=[0]](%855)
  %961 : Tensor = onnx::Unsqueeze[axes=[0]](%958)
  %962 : Tensor = onnx::Unsqueeze[axes=[0]](%959)
  %963 : Tensor = onnx::Concat[axis=0](%960, %961, %962)
  %964 : Float(1, 15360, 2) = onnx::Reshape(%957, %963) # /Development/yolov5-tensorrt/yolo.py:41:0
  %965 : Float(1, 15360, 6) = onnx::Concat[axis=-1](%951, %964) # /Development/yolov5-tensorrt/yolo.py:43:0
  %prediction : Float(1, 20160, 6) = onnx::Concat[axis=1](%739, %852, %965) # /Development/yolov5-tensorrt/yolo.py:45:0
  return (%prediction)

When I try to make the same with newer libs ( pytorch 1.5.1, torchvision 0.6.1, onnx-simplifier 0.2.10, trt 7.0.1, onnx 1.7.0 ), it goes through, but throw an error:
  %959 : Float(1, 3, 80, 80, 2) = onnx::Slice(%916, %956, %957, %955, %958) # /Development/yolov5-tensorrt/yolo.py:41:0
  %960 : Float(1, 3, 80, 80, 2) = onnx::Cast[to=1](%959) # /Development/yolov5-tensorrt/yolo.py:41:0
  %963 : Tensor = onnx::Unsqueeze[axes=[0]](%857)
  %966 : Tensor = onnx::Concat[axis=0](%963, %998, %999)
  %967 : Float(1, 19200, 2) = onnx::Reshape(%960, %966) # /Development/yolov5-tensorrt/yolo.py:41:0
  %968 : Float(1, 19200, 6) = onnx::Concat[axis=-1](%954, %967) # /Development/yolov5-tensorrt/yolo.py:43:0
  %prediction : Float(1, 25200, 6) = onnx::Concat[axis=1](%740, %854, %968) # /Development/yolov5-tensorrt/yolo.py:45:0
  return (%prediction)

model loaded
Traceback (most recent call last):
  File "main.py", line 370, in <module>
    simplify_onnx(onnx_path)
  File "main.py", line 155, in simplify_onnx
    model_simp, check = simplify(model)
  File "/usr/local/lib/python3.6/dist-packages/onnxsim/onnx_simplifier.py", line 311, in simplify
    res = forward_for_node_outputs(model, const_nodes, input_shapes=input_shapes)
  File "/usr/local/lib/python3.6/dist-packages/onnxsim/onnx_simplifier.py", line 160, in forward_for_node_outputs
    res = forward(model, input_shapes=input_shapes)
  File "/usr/local/lib/python3.6/dist-packages/onnxsim/onnx_simplifier.py", line 144, in forward
    sess = rt.InferenceSession(model.SerializeToString(), sess_options=sess_options, providers=['CPUExecutionProvider'])
  File "/usr/local/lib/python3.6/dist-packages/onnxruntime/capi/session.py", line 158, in __init__
    self._load_model(providers)
  File "/usr/local/lib/python3.6/dist-packages/onnxruntime/capi/session.py", line 177, in _load_model
    self._sess.load_model(providers)
onnxruntime.capi.onnxruntime_pybind11_state.Fail: [ONNXRuntimeError] : 1 : FAIL : Type Error: Data in initializer '992' has element type tensor(float) but usage of initializer in graph expects tensor(int64)

Do you know where could be the problem? What about providing some docker container so everything would be repeatable ( yolov5 repo have this now ) ?

It's a good idea to provider a docker container, I will add it later. But for now, I think you can either try to use onnx-sim from CLI directly rather than in main.py to see whether it works or try to run main.py in pytorch:20.01-py3 docker container from ngc.nvidia.com.

I have tried the CLI onnx-simplifier and also tried to simplify it in a separate script. The script itself is
import onnx
from onnxsim import simplify

model = onnx.load('yolov5_1.onnx')
model_simp, check = simplify(model, skip_fuse_bn=False, input_shapes={'data': [1, 3, 640, 640]})

I think that there is a problem with the fusing bn. When skip_fuse_bn=False, the code goes through. When skip_fuse_bn=True, I am getting error even the input shape is defined onnx.onnx_cpp2py_export.checker.ValidationError: Graph must be in single static assignment (SSA) form, however '1388' has been used as output names multiple times.
Nevertheles, when I simplify onnx model without the fusion_bn, your code works, but I do not see any speed improvement even your first commit made speedups like 3-4 times.
Is the whole speedup done only by the fusion_bn ?

Ok, I have tried it on V100 with skipping fuse_bn and it works quite well. Speedup is significant

hi, can u share inference trt code @batrlatom

closed according to comment https://github.com/TrojanXu/yolov5-tensorrt/issues/6#issuecomment-654086631


Add overlay and watermark transforms

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


[#7] older numpy

Closes #7

Coverage decreased (-1.01%) to 91.606% when pulling b77f30ac3299eb84c2a22c8043cce2a9cfb3d4c4 on feature/7-numpy-interface into 3a2211791a4387504603c5e7154bb6fb95f429ea on master.

Coverage increased (+0.5%) to 93.165% when pulling ed73f5d507057f4e967404b72ac7b7d3551b0d16 on feature/7-numpy-interface into 3a2211791a4387504603c5e7154bb6fb95f429ea on master.


[RAINCATCH 403+399]

Motivation
Allow auth response exclusion fields to be specified externally.
Allow session config to be specified externally.
Fixed failing tests.

Reviewed and verified in #29, #28, #24, however this PR targets the new development branch.


Mamba or Jamba models

Has any work been done with state space models.   I'd be curious how they would perform with this framework applied.

Clearly, this is a very interesting question. The use of state-space models as a base for large models has recently become a very popular topic. For example, papers like "Is Mamba Effective for Time Series Forecasting?" and "TimeMachine: A Time Series is Worth 4 Mambas for Long-term Forecasting" have made numerous attempts. In fact, we have also been studying the use of state-space models for time series forecasting for a long time and have followed the precursor to the Mamba model, the S4 model. However, we have not achieved good results in time series forecasting, which is why we decided to "let the bullets fly" for a while. We also look forward to seeing breakthroughs in time series forecasting using SSM. Finally, I would like to mention that as a general reprogramming framework, we are obviously also capable of adapting Mamba or Jamba and look forward to more like-minded friends joining us to contribute code.


⚠ Meese Enterprises has degraded performance

In 8c9a559, Meese Enterprises (https://meese.enterprises) experienced degraded performance:

HTTP code: 200
Response time: 6256 ms

Resolved: Meese Enterprises performance has improved in f87ebea after 35 minutes.


Formly warning issue

Description
formly warning issue
Sometimes the warnings are coming in misaligned (Strike through  on warnings)

Your Environment

Angular version: 8.3.10
Formly version: 5.3.9
Can someone suggest any solution here

@Abdellatif Ait boudad

Hi,
could you please please provide a reproduction using our starter example:
https://stackblitz.com/edit/ngx-formly-ui-material
Thanks!


New layout

I've added a few things to the README and split the requirements file into one for conda and another one for pip because they need different formats.
You don't need to merge this but you should update your new_layout branch by pulling from the TUW-GEO/new_layout branch. We can go through this if you are unsure about Git.

I've merged everything into the develop branch. Any new features should start from there if possible


Minor doc edit to add clarity around the --volume path format

Also add a comment to the ValidatePath func so devs/reviewers
know exactly what its looking for.
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

ping @moxiegirl @fredlf @SvenDowideit

LGTM, thanks @duglin

LGTM

dank @duglin

Oh, i18n thanking! :)
Dankjewel, @duglin

@jfrazelle cherry pick on this one please

cherry-pciked


infinity suggestions

Cool work! Might feature it on Social Media / Twitter.
I recently added multi-model deployments that default to the ENV variables. Aka you can set the default env variable
-> --port -> INFINITY_PORT or --batch-size -> INFINITY_BATCH_SIZE. Multiple args can be separated with ;, which works if the cli arg can be overloaded.
--model-id ${INFINITY_EMBEDDING_MODEL} --model-id ${INFINITY_RERANKING_MODEL}

INFINITY_MODEL_ID=mixedbread-ai/mxbai-embed-large-v1;mixedbread-ai/mxbai-rerank-xsmall-v1;

https://huggingface.co/spaces/michaelfeil/infinity/tree/305b1c2b583e9968aa153c45e8e50555af1d9575

Thanks for the suggestion, we shall check this and implement!


Hostapd STA sessions and ubus_collector

I will just create one extra bug-fix commit for all this. Is that ok?

I think actually fixing the bugs inside the respective patches is the correct way to do this. git rebase -i is ultra easy to use

Ok, just asking.

merged into pending, Thanks !

@blogic why is this not merged into pending?
CC @RickSommerville @msenecic

the CI failed on the commits so it never went from staging-pending -> pending. however looking closer, the CI timed out after 360m so i manually pushed it just now
cc @greearb @dfranusic  @cbuschfb


Closes #49: update branch naming scheme to include issue label

PR also includes some cleanup from #44: move "Creating an issue" section higher up in page in "Contributing to Documentation"

Converting to draft, so we can also knock out #45 in this PR


Real-Time Sync Switch

Current Behavior
The node is launched together with the pipeline, which, essentially, is a forever loop for querying the consensus tip and downloading/executing all the data from it. The pipeline is not able to reorg the data or to track any payloads that are received by Engine API.
Requirements
Design a switch mechanism in which, after the initial sync, the node switches from the pipeline to the BlockchainTree or some intermediary abstraction.
Related Issues

https://github.com/paradigmxyz/reth/issues/764
https://github.com/paradigmxyz/reth/issues/1104

Suggestions
Solution 1 (courtesy of @gakonst)

Run the pipeline once.
Check if the pipeline progress (the highest block number) is close to the tip
2.1. If it's close, shut down the pipeline and switch to the blockchain tree
2.2. If it's not, re-run the pipeline.

"Close" is to be defined
Solution 2

Integrate the downloaders with the BlockchainTree. The downloaders would check if the requested payload (applicable to Headers, Bodies and Execution stages) is in the tree and possibly poll for that data. Otherwise, the downloaders would fall back to download data from network.
Refactor the execution stage to poll for execution results from a process which would either execute the data or poll the BlockchainTree for the changesets.

more solutions tbd

Amending this issue:
We are going forward with implementing a SyncController which would own both the Pipeline and the BlockchainTree.
Brief Description
The SyncController will maintain a receiver channel for messages from EngineApi. The EngineApi would forward the ForkchoiceUpdated and NewPayload messages upon receiving them. Depending on the state of the BlockchainTree, the SyncController would decide whether it needs to trigger the Pipeline or simply store the new payloads and forkchoice state in the BlockchainTree. Additionally, in case the BlockchainTree encounters an error during storing/processing new payloads, this error would be returned to the SyncController, which in turn will either 1) exit if the error is fatal or 2) trigger pipeline run.


Add tag comparison

Description
When I click on "x commits to master since this release" in a Gitea release, it isn't a link. When I compare two branches but replace a branch name with the tag, it says everything is up-to-date even though 22 commits have been pushed since. I don't think this is a bug, so could you add it? Thanks! :)
(I was using the web version of Gitea (gitea.com) on a tested migration repo.)
Screenshots

The first image shows that I just get a text cursor when I hover over the "x commits to master" text, the second shows how there is "no need to create a PR".

The first is a feature request, however the second is because you have base/head mixed up.
https://gitea.com/TheTechRobo/palc-attempt2/compare/v.0.8.4-stable...master

Suggest add compare link in release page like github.

See PR #15695.


Add some tests and fix compilation

@jakirkham @isuruf @ocefpaf I'm running out of ideas here. It seems that conda is manipulating the binaries under lib/.
If I compile in the container, without conda, it seems to work. If I compile with conda it fails the tests.
brew had the same problem until they added this: https://github.com/Homebrew/homebrew-core/blob/master/Formula/mono.rb#L15

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Ok, it turns out that this is a conda problem with the path length. Windows and therefor mono has a max path length hard-coded. This means all packages that needs to compile with mono do need to specify --prefix-length 128 during conda-build.


improve egctl to support sending multi configs at once (fix #29)

I did a fix on #29 . It sends multiple requests via egctl, based on how many YAML parts (separated by ---) it read.
I know it's quick and dirty 😌, but if possible, someone can lead me to work more on it.

Codecov Report

Merging #230 (bf3240f) into main (e1d0a82) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #230   +/-   ##
=======================================
  Coverage   80.12%   80.12%
=======================================
  Files          44       44
  Lines        4835     4835
=======================================
  Hits         3874     3874
  Misses        740      740
  Partials      221      221

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 aad887f...bf3240f. Read the comment docs.

Thanks @benja-wu for the info. Let me take a look at emctl source first.

Thanks @benja-wu for the info. Let me take a look at emctl source first.

And one more thing, the code analysis format checking is failed.
Please make sure such formatting execution successfully in your local env before contributing a PR, thx.

@benja-wu PR is updated. A new visitor for YAML parsing is added. Also @localvar 's advice is taken, to validate all the docs before sending requests to server.


examples fails with latest update for bb key needed

some examples with version 1.3.2 and it fails with
here for instance "box"
File ~/miniforge3/envs/cad/lib/python3.10/site-packages/cad_viewer_widget/widget.py:681, in CadViewer.add_shapes(self, shapes, states, tracks, normal_len, default_edge_color, default_opacity, ambient_intensity, direct_intensity, tools, glass, cad_width, tree_width, height, control, ortho, axes, axes0, grid, ticks, transparent, black_edges, collapse, position, quaternion, target, zoom, reset_camera, zoom_speed, pan_speed, rotate_speed, timeit, js_debug)
    678         print("Camera control changed, so camera was resetted")
    680 if reset_camera:
--> 681     center, radius = bsphere(shapes["bb"])
    683     if position is None:
    684         position = (normalize(np.array((1, 1, 1))) * 5.5 * radius + center).tolist()

KeyError: 'bb'

but even when changing to only showing the first part I don't see anything in the viewer??

@TermeHansen I have changed the internal format of the tesselation files slightly: Instead of having a bounding box ("bb") per assembly object, now only one overall bounding box is needed on the top-level. This is due to the change in the underlying three-cad-viewer project.
So it is important to install cad-viewer-widget 1.3.2 and pull the latest git repo with the examples.
Additionally, I forgot to migrate the examples box.json and longbox.json. This is fixed in the latest commit.


Create hrzedu.txt

District domain of Herzliya city schools. Herzliya, Israel.

@TheAmazingCodeGuru Please provide us with:

the university official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the university
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

I need to provide every school website URL? Isn't the district domain URL enough?

@TheAmazingCodeGuru We need an URL of an official website which says that students of Herzliya city schools use @hrzedu.org.il as their email domain.  It is not necessary to give us the URLs of many schools, it will be absolutely enough to give us URLs of your school website, and the website of the city district, if it confirms that all the schools share the same email domain.
Please make sure that you are familiar with the rules of this repository located at the bottom of this page, and provide necessary information in comments to your request accordingly.
Thank you!

@TheAmazingCodeGuru This request remains unanswered for a while. I will have to close it without merging unless you answer the question above. We really need the additional information about your email domain before accepting the request. Thank you for the understanding.

@TheAmazingCodeGuru I am very sorry to close your request without merging. We sometimes add school district email domains to this repository, if the schools in a district share the same email domain. However we never do it without looking at the district website, and checking which kind of schools the domain serves. There are many different school systems around the world, and we cannot be familiar with all of them without being told about them. Shall you have any update regarding the issues mentioned above, please reopen your request and comment. Thank you for your understanding.


Enable ref types by default

After Rust 1.82.0 was released enabling reftypes by default for rust wasm target https://blog.rust-lang.org/2024/09/24/webassembly-targets-change-in-default-target-features.html#enabling-reference-types-by-default I think this would be helpful for users to have this enabled by default in runtime as well, so that rust file compiled to wasm with the default flags can be run on wamr compiled with the default flags

I'm not against this change, however, I think by default WAMR tries to be as lean as possible. Not sure if @wenyongh and others have any thoughts on that.

I'm also struggling with this. On one side, ref-types are now part of the specification and should be supported by default by any WebAssembly runtime. On the other side, we need to keep the binary size as small as possible.
Here's a suggestion:

We always release binaries of iwasm and wamrc that are fully compatible with the specification and offer better performance. This way, users can easily run WebAssembly code from Rust.
(Optional) We provide separate compilation groups for full specification compatibility and for minimal binary size through something like CMakePresets.json. This allows developers to switch quickly.
By default, we enable full specification compatibility. I mean enable ref-types by default.

Let's make a vote ?

It is good to me if it brings convenience and has little impact on footprint and performance. And yes, not sure what others' opinions are, maybe a vote is better.

BTW, if to enable ref-types by default, had better also change the CMakeLists.txt under product-mini/platforms/windows and product-mini/platforms/cosmopolitan.

@wenyongh, how do we run the vote?
should I just ask all TSC members to 👍  or 👎 the initial pr comment?

https://github.com/bytecodealliance/wasm-micro-runtime/pull/3913#issue-2671535298

The recent version of the rust toolchain will emit ref types opcodes, need to enable this feature in the iwasm build.

It is a strong reason we need this one now.

Yes, @lum1n0us, that was the reason for this pr in the first place and it's in the linked article
Maybe I should have been clearer about the reason


Tips to add some personalized objects?

Hi,
thanks for the sharing of your great work.
I am currently using VIMABench to collect some dataset for the training purpose. And I would like to extend current setup for more diverse objects. Could you provide some info w.r.t the object creation and insertion?

To add new objects, you can follow the definition here and add new entry to ObjPedia. You also need to place urdf assets into this folder. After these steps you should be able to instantiate your custom objects. Let me know if you have further questions.


Render docstrings with double quotes

I've been trying running prettier on the django codebase and I noticed that they use single quotes for almost all the strings and double quotes for the docstrings, which seems to be a common things in python (I haven't seen many single quotes docstrings).

PEP 257 requires double quotes for docstrings. It also requires a blank line after the docstring for classes, but not for functions.

Hey guys! I've read in the README that the plugin is deprecated, so I'd like to know if you're still interested in solving this issue. Case yes, I'd like to claim it :smile:

@rcsm are you using this plugin? if so I'd be happy to review an merge fixes :)

Yep! I've started using prettier very recently and used this plugin for a little python app of mine. Right now I'm looking for an OSS to contribute, and that's why I'd like to solve this issue :)

@rcsm cool! I use black for formatting python, but could be nice to get prettier python stable anyway


[System Tests] Unnecessary duplicated describe-key call when deleting a KEK in AWS KMS

Is your feature request related to a problem? Please describe.
At AfterEach phase we are deleting the KEK created for Record Encryption system tests. In case we use AWS KMS, we are calling to describe-key twice and this can be optimized.
2024-07-11 13:59:04 INFO  io.kroxylicious.systemtests.executor.Exec:275 - Command: kubectl --namespace localstack exec localstack-8b685bb46-k2zgk -- awslocal kms describe-key --key-id alias/KEK_my-topic-7ee736
2024-07-11 13:59:04 INFO  io.kroxylicious.systemtests.executor.Exec:279 - RETURN code: 0
2024-07-11 13:59:05 INFO  io.kroxylicious.systemtests.executor.Exec:275 - Command: kubectl --namespace localstack exec localstack-8b685bb46-k2zgk -- awslocal kms describe-key --key-id alias/KEK_my-topic-7ee736
2024-07-11 13:59:05 INFO  io.kroxylicious.systemtests.executor.Exec:279 - RETURN code: 0
2024-07-11 13:59:07 INFO  io.kroxylicious.systemtests.executor.Exec:275 - Command: kubectl --namespace localstack exec localstack-8b685bb46-k2zgk -- awslocal kms schedule-key-deletion --key-id a5a8dd18-5bcb-425f-9645-1647a2465a19 --pending-window-in-days 7
2024-07-11 13:59:07 INFO  io.kroxylicious.systemtests.executor.Exec:279 - RETURN code: 0
2024-07-11 13:59:10 INFO  io.kroxylicious.systemtests.executor.Exec:275 - Command: kubectl --namespace localstack exec localstack-8b685bb46-k2zgk -- awslocal kms delete-alias --alias-name alias/KEK_my-topic-7ee736
2024-07-11 13:59:10 INFO  io.kroxylicious.systemtests.executor.Exec:279 - RETURN code: 0

Describe the solution you'd like
Call only once to describe-key.

Not needed.


More informative infos for target installation

When installing cross-target packages, indicate the target triple
in the download and installation notifications.

Error is spurious I think. Related to windows uninstallation.


EBS mirgration

xref: kubernetes/kubernetes#70590
Tested that CSI Controller service is called for attach while in-tree driver is still used for CreateVolume.
There is a problem I found with following message:
I1031 23:24:06.158943       1 connection.go:239] GRPC error: rpc error: code = Internal desc = Could not get volume with ID "aws://us-east-2a/vol-0d3b8a174ebb7e16f": InvalidParameterValue: Value (aws://us-east-2a/vol-0d3b8a174ebb7e16f) for parameter volumes is invalid. Expected: 'vol-...'.
        status code: 400, request id: bd4680c1-3e16-40a4-88b2-573aec494c03

This is caused by EBS in-tree driver has a special volume naming convention while CSI driver expected a regular naming that starts with vol-.
I will debug more for the cause.

@davidz627  @ddebroy

Here is where the volumeName is generated: https://github.com/kubernetes/kubernetes/blob/master/pkg/cloudprovider/providers/aws/aws.go#L2214

Here is the PV:
>> kk describe pv
Name:              pvc-0c5325eb-dd62-11e8-8dc5-0203bf87cc1c
Labels:            failure-domain.beta.kubernetes.io/region=us-east-2
                   failure-domain.beta.kubernetes.io/zone=us-east-2a
Annotations:       kubernetes.io/createdby: aws-ebs-dynamic-provisioner
                   pv.kubernetes.io/bound-by-controller: yes
                   pv.kubernetes.io/provisioned-by: kubernetes.io/aws-ebs
Finalizers:        [kubernetes.io/pv-protection external-attacher/com-amazon-aws-csi-ebs]
StorageClass:      gp2
Status:            Bound
Claim:             default/claim1
Reclaim Policy:    Delete
Access Modes:      RWO
Capacity:          4Gi
Node Affinity:
  Required Terms:
    Term 0:        failure-domain.beta.kubernetes.io/zone in [us-east-2a]
                   failure-domain.beta.kubernetes.io/region in [us-east-2]
Message:
Source:
    Type:       AWSElasticBlockStore (a Persistent Disk resource in AWS)
    VolumeID:   aws://us-east-2a/vol-0d3b8a174ebb7e16f
    FSType:     ext4
    Partition:  0
    ReadOnly:   false
Events:         <none>

I think this is because in-tree provisioner is still used. The problem should be gone once we start using CSI provisioner

@leakingtapan were you able to test with pre-provisioned volumes successfully?

Chatted offline on slack. I'm able to test pre-provisioned volumes with this change. So far I verified that attach succeeds on AWS after feature gate is turned on with CSIMigration and CSIMigrationAWS enabled. I confirmed  @davidz627 's concerns that MountVolume might fail and it is due to volumeToMount.DevicePath is populated with csi volumeattachment name instead of real path in GenerateMountVolumeFunc.

The CSI devicePath issue is related to: kubernetes/kubernetes#70852


PETSc formula should provide the PETSc python bindings : petsc4py

Provide a detailed description of the proposed feature
petsc4py development is now hosted alongside the petsc library. It is part of the PETSc API now.
the formula should install the bindings
What is the motivation for the feature?
petsc4py provides the python bindings for both sequential and parallel computation and it is quite complete with respect to what PETSc does.
How will the feature be relevant to at least 90% of Homebrew users?
it would be relevant to most PETSc formula users as it provides scripting capabilities for PETSc
What alternatives to the feature have been considered?
I don't see any alternatives

Please submit a PR for discussion, thanks. For future reference: please submit issues to homebrew-core and group very similar issues into one.

Just to be explicit: no guarantee we'll accept the PR but it's easier to discuss and we're unlikely to open it for you.


rebase Cloud 2.0 branch on main with latest SDK

The Cloud 2.0 branch has diverged from the main branch. This commit rebases
the Cloud 2.0 branch on main. The individual Cloud 2.0 commits no longer
build on their own, because they've been rebased on the main version of
the SDK. This commit pulls in the Cloud 2.0 version of the SDK to address
that issue.
Release note: None
Commit checklist

[X] Changelog
[X] Doc gen (make generate)
[X] Integration test(s)
[X] Acceptance test(s)
[X] Example(s)

I think I'll have to force push this into the cloud-20 branch, since it will change the history. I want the changes in the branch to always be on top of main.


Joining slack channel via heroku is broken

Heroku app doesn't load and gives back an application error when trying to sign up for slack.
###Questions
Is communications still done via slack if so please add me to the slack channel
my email is<EMAIL_ADDRESS>

Lets remove this link for the time being?


Typo in Kubernetes Deployment documentation

"I have noticed a typo in the heading of this https://docs.open-metadata.org/deployment/kubernetes#1-create-a-dockerfile-based-on-openmetadataingestion.
The heading should be
"Create a Dockerfile based on openmetadata/ingestion"
instead of
"Create aDockerfilebased onopenmetadata/ingestion".

The documentation has a space, but there is a problem with the rendering. @harshach I need help with fixing this in docs rendering. Can you please assign this to someone else.

hey @aniketkatkar97 can you take a look


Add ability to customize prometheus configuration, e.g. add VictoriaMetrics backend support

What is missing?
Add ability to customize prometheus configuration, e.g. I wanted to add
VictoriaMetrics setup to prometheus configuration file but I can't find how it can be easily done. The only change is required is a change to prometheus config file (not only its scrape section).
Why do we need it?
Users should be able to provide their own prometheus configuration file, possibly via secret functionality. As such I want to customize my prometheus service to use  VictoriaMetrics fast, cost-effective and scalable time-series database.
Environment

Prometheus Operator version:

quay.io/coreos/prometheus-operator:v0.34.0

Anything else we need to know?:

You can already configure remote write and external labels, do you need anything else?

How? I can't find proper documentation for that. Any pointers/examples will be useful.

The Prometheus spec has the remoteWrite field. Check out the API docs.

great, that's what I was looking for. I just tested deployment and now I see the remote_writes in /etc/prometheus/config_out/prometheus.env.yaml Closing the ticket.


Can the save-data client hint request header be supported in your script?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Browsers are starting to support the save-data client hint request, where a browser is specifically asking a web experience to prioritize low data usage when serving content.  Currently, there is no way to define specific images to users with this preference turned on using this lazy loading library.
See https://developers.google.com/web/fundamentals/performance/optimizing-content-efficiency/save-data/ for more information on the save-data client hint.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Would it be possible, or make sense, to include this in your library so as a user I can specify images that should lazily load to those w/ save-data turned on?
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
We could handle this on our own by passing different, lower resolution, srcs into the lazy loader before initializing it.
Additional context
Add any other context or screenshots about the feature request here.

Hey there. You can already use responsive images with the srcset attribute, so that browsers will choose the best format to satisfy the users request to save data. This should be a browsers implementation and work also on regular images and native lazy images, so I wouldn't implement it here.


Add pre-load CPU feature guard check

Follow-up from #48024: building the library and Python code loading it.

Thanks for the review. The MacOS failures look unrelated?

Any input @penpornk?

@hakos Thank you for checking! Yes, the MacOS CPU Python build is unrelated. The PR has already been pulled inside and is just stuck with internal test failures. I'll try to get to it soon.

@penpornk Thanks, should I try rebasing to see if it can get rid of the MacOS CPU Python build failure?

@hakos You don't need to. They are unrelated. :)

@hakos The PR is merged! Thank you very much for your patience! :)
What changed: I ended up moving the .so to //tensorflow/core/platform to fix the segfaults.

Thanks @penpornk!


Report DataItemTableView allows invalid data

1. Describe the bug
AL environment and compiler does not give an error when adding an invalid DataItemTableView in a report.
In CAL you are not allowed to enter 0D as for a date field.

Microsoft Dynamics NAV Development Environment
'0D' is not a valid date.
OK
In AL it is posible, and no error is given when entering this values, packaging or pushing to the system.
The resulting error is only thrown when you run the report that the date is not valid.
2. To Reproduce
Add a Date field (My Date Field) as new field to the sales header table.
Create the following report and publish it to the system.
report 50103 "Sales Headers"
{
    UsageCategory = Administration;
    ApplicationArea = All;

    dataset
    {
        dataitem(SalesHeader; "Sales Header")
        {
            DataItemTableView = where("My Date Field" = const(0D));
            column(No; "No.")
            {

            }
        }
    }
}```

**3. Expected behavior**
Like in CAL where a compiler error raised that 0D is not allowed I expect AL gives some similar errors,

**4. Actual behavior**
You can publish the report to the system and only getting an error during runtime.

**5. Versions:**
BC 19.2

- AL Language: v8.3.570094
- Visual Studio Code:1.63.2
- Business Central:19.2

I think in those expressions, to filter for value 0D, you must do const(''). Not sure though. Agree it should be an error at build time if it won't work at run time. Sadly, many things aren't...

Thanks for raising this, we should definitely block this at runtime. Unfortunately since this is not a (recent) regression but something that has been around since C/AL, it's unlikely that we'll prioritize this soon. I'll keep this issue open in case we do manage to get to it in the next few months, but if it's something that you'd like prioritized faster it would be best to go through customer support as they can create a support request that will give us time to work on this issue specifically.

Update - this issue has been raised by our support team and is now in our internal queue. :)

Thanks for raising this, we should definitely block this at runtime.

or build time, even? ;-)

Thanks for raising this, we should definitely block this at runtime.

or build time, even? ;-)

Indeed, I meant at build-time, not runtime. ): ...fingers working on muscle memory too much.

Hello,
The Runtime error you get, is:
The format of the Date '''' does not match your device's date settings.
The al compiler has no idea of the device settings where an app will run. C/AL was an onprem compiler.
This is not a compiler issue. Maybe a report runtime to recognize null dates, but then this is a new feature, since 0D was not valid even in C/AL.
What do you think?

Due to prioritization, we are currently not planning to change the behavior of this. '0D' is syntactically a valid Date, so the compiler accepts it. Having the compiler accept a filter criteria does not guarantee that it is a valid runtime filter - particularly because triggers can modify the filters / table views.
In the future we may change the runtime handling of this - but for now, you cannot rely on the compiler to guarantee that it works at runtime, if it is not a type mismatch


ExpressionAnimation should support resting points that can invoke keyframe animations

Take a pull-to-refresh scenario for example, if I pull down the list by say, 120px, I'd like the expression animation to kick off a keyframe animation and ignore the expression calculation.
Does this make sense?

Hey @JustinXinLiu,
Thanks for the feedback! Just to clarify, you're looking for a way to conditionally trigger the start of an animation yes? Outside of when the output of an animation or property meets a condition, are there any other ways you find useful to conditionally trigger an animation?

Yes, that's correct.
Currently when I want to stop the expression animation and trigger a keyframe reset animation when a condition is met, I will need to monitor this by subscribing to an input event, stop the expression animation and start the keyframe one.
Since it's not possible to get the animated value from an expression animation, this approach only works when an input event is given.
It's almost like the Condition and RestingValue in InteractionTrackerInertiaRestingValue. So I guess it would be cool if the resting animation could be either a keyframe or an expression animation.

I will give you a scenario. :) Take your Pointer Rotate demo for an example, when the cursor enters the picture, how to smoothly animate the rotation of the picture instead of changing the initial rotation instantly; when the cursor leaves, how to again smoothly animate it to the original position? This will require A LOT more code than what's currently there.

Ok - that makes sense. Thanks for sending this info over, we'll use this information to help evaluate this feature against the rest of our work. If other scenarios come to mind, send them our way :)
Thanks again @JustinXinLiu!

@JustinXinLiu thanks for the feedback. We've created this UserVoice feedback item and will be tracking it there.


Joseph Leckie Academy Pull Request

Pull request for josephleckieacademy.co.uk

@mry-stack lib/domains/uk/co/josephleckieacademy: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@mry-stack First stage of verification passed, official domain for Joseph Leckie Academy is josephleckieacademy.co.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

[like]  M.Ryder reacted to your message:

From: Philip Torchinsky @.>
Sent: Wednesday, July 17, 2024 7:48:04 AM
To: JetBrains/swot @.>
Cc: M.Ryder @.>; Mention @.>
Subject: [EXTERNAL] -Re: [JetBrains/swot] Joseph Leckie Academy Pull Request (PR #21805)
CAUTION: This email originated from outside of Joseph Leckie Academy. Do not click links or open attachments unless you recognise the sender and know that the content is safe, if you have any doubts do not click links or open attachments. You should never be asked to enter your username and password into an external link.
@mry-stackhttps://github.com/mry-stack First stage of verification passed, official domain for Joseph Leckie Academy is josephleckieacademy.co.ukhttps://josephleckieacademy.co.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.
—
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/21805#issuecomment-2232653677, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJYVBZVJKKHSFMFBU5TLNNTZMYOTJAVCNFSM6AAAAABLAA65AWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSGY2TGNRXG4.
You are receiving this because you were mentioned.Message ID: @.***>
Confidentiality Notice & Disclaimer: This email maybe Private and/or Confidential. If you have received this email in error please do not distribute it in anyway, without the express permission of the author. The information within this message should be regarded as confidential and is intended for only the named recipients unless explicitly stated otherwise. If you believe you have received this message in error, please delete and contact @.*** so that we can perform an internal investigation. Any views expressed within this message are personal to the sender, and not necessarily those of Joseph Leckie Academy. Emails sent and received by members of Joseph Leckie Academy may be intercepted and read by the Academy. This interception will only occur to ensure compliance with any policies, procedures or statutory obligations, as well as part of necessary maintenance and support of the communication systems. Please be aware that any email communications may also be open to Subject Access Requests under Data Protection, Freedom of Information Requests or Environmental Information Legislation and may therefore need to be disclosed to third parties. Email Security: Email communications are not 100% secure and can be intercepted and read by others under certain circumstances. We strongly advise that you do not divulge information which if disclosed to third parties would like cause you or others distress, without a layer of additional security or in a more secure manner, such as By using an encrypted service or ensuring content is password protected. Joseph Leckie Academy has taken reasonable measures to ensure that any attachment(s) to this email has been checked for viruses, however we would advise that you scan this e-mail and any attachments for viruses. We do not, to the extent permitted by law, accept liability (whether in contract, negligence or otherwise) for any damage sustained.

@mry-stack Pull request merged. Thank you.


[Theme] SpotifyXP installer pane/picture refresh

I have created a SpotifyXP installer pane, because I was bored and I thought this may help improve the theme of the SpotifyXP installer.
Not saying that your current SpotifyXP installer pane is bad, I just wanted to create a (potentially) better installer pane that you may use without my credit.
Final(?) Pane:

Various revisions for your use:  {.Zip file}
Images.zip
If you think this is better, let me know and use the images.
If not, no worries! I was just bored, and I like graphic designing.
No credit is needed at ALL, but if you want to credit, I would appreciate it.

Video of me testing the images:
https://github.com/SpotifyXP/SpotifyXP/assets/145695642/06d44442-1917-402c-9b85-d7542a547c07
Sorry for the audio delay. Didn't notice the mic mess-up when I edited it in Audacity and bonded the audio.

Thank you 😁. This will be in the next commit

Glad I could help with designs!

What is this test.jar?

Just a test to ensure the picture worked in the setup utility. Nothing else.
The test build was based on the latest public release of SpotifyXP.
I had extracted the Jar file, replaced the setup.png with the one I made, then used JarBuild to rebuild the Jar for testing.
Note that I deleted the test jars after I recorded the video. I can't upload the test jar due to it's size.

Can you make the background of the final version transparent? I think that would look nice

Let me see what I could do...
Here:


No section: 'client'

So, I recently ran into a weird issue, and I've got no idea what caused it. But anytime I try to run any ee commands It just says " No section: 'client' ". Any help would be much appreciated!
Also, any site on the system, which as far as I can tell should work fine return error 404.

[ ] lsb_release -a

No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 8.4 (jessie)
Release:        8.4
Codename:       jessie

[ ] ee -v

No section: 'client'

[ ] ee info

No section: 'client'

[ ] wp --allow-root --info

-bash: wp: command not found

It seems your /etc/mysql/conf.d/my.cnf file is missing.
Also wp (wp-cli) is not present.
Is this fresh EasyEngine setup?

It's not fresh, it was set up and worked fine around the 4th of may. I've no clue what caused this to occur.
The /etc/mysql/conf.d/my.cnf file does exist, but had no client tag. I was using ee to run a php site, so minimal installs.

@xreeckz  Can you please verify /etc/mysql/conf.d/my.cnf file content as below :
[client]
user = root
password = PASSWORD

PASSWORD would be any random generated string.


Crash When Opening RFTools Machines with Moar Signs Toolbox

Hello,
I'm getting a consistent crash when right clicking on the RFTools Machines with a Moar Signs toolbox being held:
http://pastebin.com/SFssfezp
http://pastebin.com/MhR9hyA5
McJtyLib: 2.3.4
RFTools: 5.82
RFTools Dimensions: 4.53
MC: 1.10.2
Forge: 2221
Thank you!

Seems Moar Signs toolbox doesn't implement a needed method. Did you report to them? Not sure it is their fault though but I would like to hear their feedback


\u0000 cannot be converted to text

If some jsonb columns contains  \u0000  characters then the pertitence fails on postgres.
Is there a common way to replace those invalid json chracters before persist ?

There's always a way. Looking forward to your Pull Request that demonstrates the issue and provides the fix for it

Closed as no Pull Request was provided.


logout

When user logout, I can only get "accessToken:" at server side:
app.on('logout', data => {
    console.log('logout', data);
});

How can I retrieve user's account, such as email?

Hi.
I use this part of code at client side and it should be what you want:
import decode from 'jwt-decode';

const data = decode(token)
data.userId

Like that (not tested):
import decode from 'jwt-decode';
app.on('logout', ({accessToken}) => {
  const {userId} = decode(accessToken)
  const user = app.service("users").get(userId)
});

The above worked for me. Thanks @Buom01 :-D


Messaging after entering intent (LUIS)

Im using Bot Framework and LUIS, and I need to find a way to ask the user messaging to enter some parameters if the parameters werent detected on the utterance that caused the intent.
So, after an intent gets detected and called,  I need to check if a certain parameter is in the LuisResult, like this:
[LuisIntent("DoSomething")]
public async Task DoSomething(IDialogContext context, LuisResult result)
{
     string aParameter;
     if (TryGetParameter(result, out aParameter))
      {
         // do something with the parameter
      }
      else
      {
         // ask the user to enter the needed parameter... and so on if there are more parameters needed
        // here a new message flow starts, asking the user for the parameters... How can I do this without 'leaving' the intent?
       }
       context.Wait(MessageReceived);
 }

I need something simpler than FormFlow, as FormFlow use hardcoded enums to options, and in my case parameters options are something dynamic.
Thanks in advance.

Instead of calling
context.Wait(MessageReceived)
unconditionally, add a new method
private async Task AfterUserInput(IDialogContext context, IAwaitable result)
and call
context.Wait(AfterUserInput);

Ok cool thanks! Now I have a couple of more questions:

How do I get the text the user entered. Inside the AfterUserInput function I'm doing:

 var message = await result;
But examining that object I didn't found a property corresponding to my text.

And supossing that I can find the text, how can I, inside that function (AfterUserInput), call LUIS again so It can analyse the utterance again?

Code right now is like this:
[LuisIntent("DoSomething")]
public async Task DoSomething(IDialogContext context, LuisResult result)
{
     string aParameter;
     if (TryGetParameter(result, out aParameter))
      {
         // do something with the parameter
         context.Wait(MessageReceived);
      }
      else
      {
          context.Wait(AfterUserInput);
      }
 }

// And my AfterUserInput function goes like this right now:

  private async Task AfterUserInput(IDialogContext context, IAwaitable<IMessageActivity> result)
  {
      var message = await result;
      // assuming that I can get the text entered...
      // how can I call LUIS again (without doing the explicit call to the service), to make him analyse the text? or should I do this by hand
  }

Thanks in advance!

Look at message.Text to get the user's text.
Inside of AfterUserInput, call MessageReceived(context, result) to use the normal LuisDialog dispatching.

Thanks, I was just trying that but it seems that now whenever I type something to the bot, it always enters on 'AfterUserInput' function and then it enters on the 'DoSomething' intent function. Is it related to the dialog stack or something? Code now look like the one before but with the AfterUserInput function  as you said:
 private async Task AfterUserInput(IDialogContext context, IAwaitable<IMessageActivity> result)
        {
            var message = await result;
            string text = message.Text;
            await context.PostAsync(text);
            await MessageReceived(context, result);
        }

Any thoughts about that?

After receiving a user's message, it should enter whichever method was passed to context.Wait last.  If you point to the same method, it will re-enter the same method.

Thanks!


Total of weights cannot exceed 1

Hi,
In your new version there is a breaking change. The total weight can max 1
      if (weightsTotal > 1) {         throw new Error('Total of weights cannot exceed 1')       }
it is ok but how can i handle this limitation.
My intention is:
from        {name: 'zip', weight: 0.5},       {name: 'city', weight: 0.4},       {name: 'street', weight: 0.3},       {name: 'land', weight: 0.02}       {name: 'housenummer', weight: 0.1}, to        {name: 'zip', weight: 0.05},       {name: 'city', weight: 0.04},       {name: 'street', weight: 0.03},       {name: 'land', weight: 0.02}       {name: 'housenummer', weight: 0.01},

Same problem here. After updating to 3.6.0 fuse throws the mentioned error from above.

By design, the total weights should not exceed 1, as it throws off the new scoring calculation. Could you not change the weights to accommodate this restriction?
Alternatively, I could normalize the weights so that they are between (0, 1)

I think it would be fine if you could normalize the weights.

I would agree, it would be great if the library could handle it. Seeing as the error was not thrown before, this is sort of a breaking change, that maybe one that should not be introduced in a minor version bump?


Spring latest

Upgrading Spring Boot to 3.0.6

@jdavidtorres Thank you for signing the Contributor License Agreement!

Would be superseded by #1253


Respect .gitignore

When running codeowners on a typical git repository, many files are printed which are actually not part of the repository because of .gitignore. Is that correct behavior? Would you consider that to be an improvement if this tool respects (and parses?) .gitignore if present?

I'd like to work on this.
I will adapt the parser with a new Switch, like I did for sections, but will wait with adapting the cmd until the section support is merged, or further adapted to your expectations.

Another potential solution to respect gitignore: run this command to get the list of files in the repo git ls-tree -r --name-only HEAD
If the codeowners tool could take the list of files as an input, the user could also do this themselves and modify the list however they want before evaluating the code ownership.

This has been on my backlog for a few months and finally took the time to try out a few solutions yesterday. I put up some patches here: https://github.com/hmarr/codeowners/pull/32. Looks like the library already uses rev-parse for finding the repo root, so using ls-files as a best-effort solution should be fine. Please take a look.

Thanks for the contribution @ti-mo. Anyway you @rwese could get this reviewed?


bugfix: use real version to check if custom attachment is broken

#3558
Checklist

[x] the [contributor license agreement][] is signed
[x] commit message follows [commit guidelines][]
[ ] tests are included
[ ] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
If a Custom Attachment links to the latest version of an Item,  it uses 0 to represent the latest version. However, when we check whether the attachment is broken, we need to use the real latest version number rather than 0.
#3558

All good, except...
Really should have a test around this kind of thing. Any chance when this functionality was added there was more than one customAttachment and we could configure one to point to 'latest'/version 0?

I have updated the testing data. Without the fix, new search page shows 500 error. And with the fix it works fine. :smile:


Add build file

How do you compile the Scala code? Seems like the build file is missing...

As described at the end of the readme, in a spark-shell, I load the scala code with :
:load [pathtoScalaCode]/spark-fm-parallelSGD/fm/fm_parallel_sgd.scala


LANG-1118: Fix StringUtils.repeat(char, int)

Close PR #68. Recreated due to rebase on master.
Now doing what is said in JavaDoc comment :
when passing a negative repeat value to the StringUtils.repeat(char, int) function, it returns an empty String instead of throwing a NegativeArraySizeException.

Coverage decreased (-0.0%) to 93.83% when pulling 9cc91332ff252f6e3f41719526f930dcb5908c31 on rikles:fix-LANG-1118 into 63d8a025e6f38ddfc966c4104ac783bcb59031e0 on apache:master.

Hello @rikles I'm still trying to find out how to correctly merge PRs from the command line. Looks like it didn't work with this PR. Your changes have been merged. Can you close this PR please? :) Thank you!

Merged into master. Close PR.