Drag and drop

Add a feature to drag and drop files into Lapce. That would be useful because it is frustrating to open files from numerous different locations when they're already open in a file manager or the desktop. The feature has already been implemented in several basic text editing applications.

Duplicate of https://github.com/lapce/lapce/issues/1181


Healthvault lib zip file curl is broken, if building your own Docker image

If you are building your own docker image (option 3), line 35 of the Dockerfile uses curl to fetch the Healthvault Java Lib.  However, the URL is no longer valid, and returns an HTML file sending you to:
http://healthvaultjavalib.codeplex.com/downloads/get/878881
If you try the curl to that address, it still fails.  A similar issue was discussed here, but I was not able to get that to work.
As a work around, I just downloaded the .zip file to the /docker/source directory, and then had the Dockerfile copy it to the container while building:
COPY ./healthvault-sdk-1.6.0.zip /root/healthvault-sdk-1.6.0.zip

Thanks for the heads up @jaredsieling. It looks like MS released a new internal build, so I've updated the HealthVault library link and it should work again.
On a related note, since we wrote those Docker instructions, we've become more proficient with Docker and Docker Compose (previously called Fig) has been gaining ground. We're looking into simplifying deployments using Compose, so I'm curious if/why people are using the single source Dockerfile to know if it's worth keeping around. Could you let us know why you prefer that method over the binary build? And whether a straightforward Compose build would be easier?

Thanks. We've made some updates to the shim code for our purposes and added another API, so I am packaging it with those updates.  A straightforward Compose build would be great, and I actually was looking into that myself. It would be great to connect about that, or if you have an early alpha version of that you could make that available.

Compose builds are available as of 0.3.0, in two flavours. The default docker-compose.yml configuration downloads images, whereas the docker-compose-build.yml builds the images. When using the latter configuration, we recommend using the convenience script called run-dockerized.sh that compiles the code before delegating to Docker Compose.
PS. If you'd like to contribute the API you added, let us know.


Replace Travis CI badge with GitHub Actions badge

Looks like TravisCI is not used anymore, so I've replaced its badge with GHA one.

Coverage: 98.551%. Remained the same when pulling 358cbcb4aee225969a2b6611148e4e63e7cf1a72 on andyone:master into 83c0eeb63b77acbf9d087e8de04b4f4ecf343fcf on funbox:master.


[Test][Spark] Make sure the value in spark not be reused.

Purpose of this pull request
We should make sure Spark InternalRow value not be reused after invoke SeaTunnelSinkWriter::write method. This is a bug long time ago (seem like already be fixed in dev branch). Please refer #5091 . I removed clone row in jdbc sink in https://github.com/apache/seatunnel/pull/5620/files#diff-7038c93a00208c28985861ea0b2ea931d05c2b1eb3b4acd8291b92cf59ce0707L80. So I think we should add test case to avoid regression about reused InternalRow in Spark.

Does this PR introduce any user-facing change?
no

How was this patch tested?
add new test.

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

cc @ic4y

Please update https://github.com/apache/seatunnel/tree/dev/seatunnel-dist/release-docs.

Done

By the way, why we need to upgrade the dependency version in this PR.

Our spark/flink dependency scala version not same. It should be 2.12. Also update common-lang3 and jackson version so that the test case can be execute. (Too old version cause Spark app can not start)


How to install the GCM module of DoWHY from Git ?

Hello!
I trying install DoWhy package in PyCharm by pip install git+https://github.com/py-why/dowhy
but i have this error:
Collecting git+https://github.com/py-why/dowhy
Cloning https://github.com/py-why/dowhy to c:\users\prgU\appdata\local\temp\pip-req-build-jwov6bkp
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'done'
Preparing wheel metadata: started
Preparing wheel metadata: finished with status 'error'
WARNING: Ignoring invalid distribution -andas (c:\users\prgU\appdata\local\programs\python\python36\lib\site-packages)
WARNING: Ignoring invalid distribution -andas (c:\users\prgU\appdata\local\programs\python\python36\lib\site-packages)
Running command git clone -q https://github.com/py-why/dowhy 'C:\Users\prgU\AppData\Local\Temp\pip-req-build-jwov6bkp'
ERROR: Command errored out with exit status 1:
command: 'C:\Users\prgU\AppData\Local\Programs\Python\Python36\python.exe' 'C:\Users\prgU\AppData\Local\Programs\Python\Python36\lib\site-packages\pip_vendor\pep517\in_process_in_process.py' prepare_metadata_for_build_wheel 'C:\Users\prgU\AppData\Local\Temp\tmp6lf94zli'
cwd: C:\Users\prgU\AppData\Local\Temp\pip-req-build-jwov6bkp
Complete output (14 lines):
Traceback (most recent call last):
File "C:\Users\prgU\AppData\Local\Programs\Python\Python36\lib\site-packages\pip_vendor\pep517\in_process_in_process.py", line 280, in
main()
File "C:\Users\prgU\AppData\Local\Programs\Python\Python36\lib\site-packages\pip_vendor\pep517\in_process_in_process.py", line 263, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
File "C:\Users\prgU\AppData\Local\Programs\Python\Python36\lib\site-packages\pip_vendor\pep517\in_process_in_process.py", line 133, in prepare_metadata_for_build_wheel
return hook(metadata_directory, config_settings)
File "C:\Users\prgU\AppData\Local\Temp\pip-build-env-2kfu5mgf\overlay\Lib\site-packages\poetry\core\masonry\api.py", line 43, in prepare_metadata_for_build_wheel
poetry = Factory().create_poetry(Path(".").resolve(), with_dev=False)
File "C:\Users\prgU\AppData\Local\Temp\pip-build-env-2kfu5mgf\overlay\Lib\site-packages\poetry\core\factory.py", line 43, in create_poetry
raise RuntimeError("The Poetry configuration is invalid:\n" + message)
RuntimeError: The Poetry configuration is invalid:
- Additional properties are not allowed ('group' was unexpected)

Thanks @c7dn7 for reporting this.  Could you try using a newer version of poetry (>1.2.0)
@darthtrevino could you take a look at this error.  Do we need to change our required version of poetry-core to >1.20 to support grouped dependencies?

Yep, having a look

This install vector worked fine in a fresh codespace using Python 3.10. In looking at this a bit closer, it looks like @c7dn7 is using Python3.6, which isn't supported by the library. I believe that we need at least 3.8

Okay, so just need to use Python 3.8+ ?  But for use gcm module i need add DoDhy from git or from PyPi ?

@c7dn7 the gcm module is available in DoWhy 0.9.1 via pip install.  I believe it was first released as an experimental API in v0.8
https://github.com/py-why/dowhy/releases

@c7dn7 the gcm module is available in DoWhy 0.9.1 via pip install. I believe it was first released as an experimental API in v0.8
https://github.com/py-why/dowhy/releases

i try install 0.9.1 but error:
ERROR: Could not find a version that satisfies the requirement dowhy==0.9.1 (from versions: 0.1.1, 0.2, 0.4, 0.5, 0.5.1, 0.6, 0.7, 0.7.1, 0.8)
ERROR: No matching distribution found for dowhy==0.9.1
this error because i use python 3.6 ?

this error because i use python 3.6 ?

Yes.


fix mean_ap

When using scale_ranges in ***Metric, average_precision() returns ap as a list, which causes the round() exception in  ***Metric.

TypeError: type list doesn't define round method

I've also encountered this issue and applied the patch.
With the applied patch, the exception is no longer raised.
Instead a single mAP value is calculated and returned.
But the mAP should be taken with a grain of salt, since it will not equal to the mAP without scale_ranges.
This pull request only averages between the scale ranges without factoring in the number of ground truths in each range.
E.g. when having two ranges

one for very small objects, of which there are not a lot, and which are very hard to detect and
one for regular sized objects, of which there exist plenty

then the mAP of the small objects will be overly represented in the resulting mAP.
E.g. the result would look somewhat like this:
---------------iou_thr: 0.5---------------
Scale range (0, 1024)
+------------+-----+------+--------+-------+
| class      | gts | dets | recall | ap    |
+------------+-----+------+--------+-------+
| Class 1    | 2   | 98   | 0.000  | 0.000 |
| Class 2    | 0   | 99   | 0.000  | 0.000 |
+------------+-----+------+--------+-------+
| mAP        |     |      |        | 0.000 |
+------------+-----+------+--------+-------+
Scale range (1024, 9999999)
+------------+-----+------+--------+-------+
| class      | gts | dets | recall | ap    |
+------------+-----+------+--------+-------+
| Class 1    | 98  | 98   | 1.000  | 1.000 |
| Class 2    | 98  | 99   | 1.000  | 1.000 |
+------------+-----+------+--------+-------+
| mAP        |     |      |        | 1.000 |
+------------+-----+------+--------+-------+
OrderedDict([('AP50', 0.5), ('mAP', 0.5)])

The mAP is 0.5, but actually nearly all objects except the two in the smaller scale range where detected.
I currently fixed it, by always specifying a range  (0, 9999) as the first item and returning the mAP of the first result instead of the average with:
    if isinstance(mean_ap, list):
        mean_ap = mean_ap[0]

I guess it should first be discussed, what the expected result should be.

I think that when using scale_ranges, we will focus on the mAP of each scale_range, rather than the mAP of the whole range.


[NNPA] Add one more constraint to the pattern DimUnstickRemovalPattern

The pattern DimUnstickRemovalPattern is for ONNXDim to get dimension directly from a ztensor without unstick. However, we have another rule to reconstruct stick -> zdnn op -> unstick back to an ONNX op if NNPA is not beneficial. That situation results in a generated code where both the onnx op and zdnn op exists while the zdnn op would be redundant.
This produces the overhead caused by the redundant zdnn op.
This patch adds one more constraint to the pattern so that the pattern is applied only when the consuming ops are zdnn ops so that unstick is unnecessary, and getting dim directly from the ztensor is really beneficial.

Jenkins Linux s390x Build #14574 [push] [NNPA] Add one more cons... started at 10:17

Jenkins Linux amd64 Build #14544 [push] [NNPA] Add one more cons... started at 09:17

Jenkins Linux ppc64le Build #13569 [push] [NNPA] Add one more cons... started at 10:26

Jenkins Linux amd64 Build #14544 [push] [NNPA] Add one more cons... passed after 1 hr 7 min

Jenkins Linux s390x Build #14574 [push] [NNPA] Add one more cons... passed after 1 hr 26 min

Jenkins Linux ppc64le Build #13569 [push] [NNPA] Add one more cons... passed after 1 hr 54 min


Improvements to self-signed-ca.sh

On my testing environment (debian 11, openssl 1.1.1k, minikube with k8s 1.20,
go 1.15, kfserving 0.5) I get some TLS errors when testing kfserving,
namely
"x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching"
when trying to deploy a simple InferenceService resource.
After some digging I found this problem:
openssl req -noout -text -in server.csr | grep "Subject Alternative Name" -A 1
            X509v3 Subject Alternative Name:
                DNS:kfserving-webhook-server-service, DNS:kfserving-webhook-server-service.kfserving-system, DNS:kfserving-webhook-server-service.kfserving-system.svc, DNS:kfserving-webhook-server-service.kfserving-system.svc.cluster, DNS:kfserving-webhook-server-service.kfserving-system.svc.cluster.local

openssl x509 -in server.crt -text -noout | grep "Subject Alternative Name" -A 1

Meanwhile if I add this patch:
openssl x509 -in server.crt -text -noout | grep "Subject Alternative Name" -A 1
            X509v3 Subject Alternative Name:
                DNS:kfserving-webhook-server-service, DNS:kfserving-webhook-server-service.kfserving-system, DNS:kfserving-webhook-server-service.kfserving-system.svc, DNS:kfserving-webhook-server-service.kfserving-system.svc.cluster, DNS:kfserving-webhook-server-service.kfserving-system.svc.cluster.local

As far as I got go 1.15 introduces a stricter policy for TLS certs, and
openssl by default doesn't copy SANs from a .csr to .crt if not
explicitly told.
Also updated the self-signed-ca.sh script to patch the caBundle of the conversion webhook to avoid runtime issues when deploying v1alpha2 services on a recent version of kfserving.
gh: #1657

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

@googlebot I signed it!

@googlebot I fixed it.

@haijohn I think that the CLA is blocked because I added you as co-author. I am not sure if it still complains because I formatted the Co-authored-by incorrectly or because you don't have a google-cla. Let me know what's best for you :)

@haijohn I think that the CLA is blocked because I added you as co-author. I am not sure if it still complains because I formatted the Co-authored-by incorrectly or because you don't have a google-cla. Let me know what's best for you :)

Hi @elukey I already signed the CLA. the correct format is Co-authored-by: name<EMAIL_ADDRESS>according to this

@haijohn yep you are right, I found your email from a previous commit and amended it. Now the google bot wants you to type @googlebot I consent. if possible :)

@googlebot I consent

@pvaneck @yuzliu Hi! What do you think about the patch?

Any thoughts about what do to do unblock the tide check?

thanks @elukey for the fix!
/lgtm
/approve


DID Resolution Result in case of upstream issues

Seeking commentary on what the resolve() function in our DID method/resolver implementations should do when there is either a network failure or upstream gateway failure.  Examples include:

for did:web, when attempting a GET to the server specified by the DID URI the network call might return a failure or the remote server could respond with status codes 404, 500, 503, etc.
for did:dht, there are multiple steps in the resolution process where network calls or upstream gateways might return an error status

[!NOTE]
This only pertains to DID methods in which resolution relies on an external service.  In other words, we need to consider for did:dht and did:web but not did:jwk or did:key.

We haven't yet added test vectors for DID DHT but there is one test for DID Web checks for didResolutionMetadata containing an error value of notFound given an invalid domain.
Do we want to mirror this pattern for other DID test vectors? Should implementations raise an exception? (Kotlin/JS) or return an error result (Go, Rust). Is it up to the implementation and all of the Web5 SDKs will do it differently?
Additional Reference Information

List of DID Resolution Errors in draft community group report
§ 8.3 notFound

If during DID Resolution or DID URL dereferencing a DID or DID URL doesn't exist, the value of the DID Resolution or DID URL dereferencing Metadata error property MUST be notFound as defined in sections DID Resolution Metadata and DID URL Dereferencing Metadata in [DID-CORE].

For did:dht, the proposed impelementation does:

notFound, when the given did isn't found by the gateway.
internalError for all other types of errors, e.g. a timeout.

@andresuribe87 Given that, what are your thoughts on returning internalError in the event the HTTP GET request during resolution of a did:web DID either outright fails (network transport layer issue) or returns a status code other than 200?

FWIW, this reference implementation always returns notFound when there are any issues.


[terra-date-time-picker] onBlur called before/during DST resolution, selections lost

Bug Report
Description
This relates to https://github.com/cerner/terra-framework/issues/521 and https://github.com/cerner/terra-framework/issues/522. In short, with 521 requested for the ambiguous date resolution to occur before onChange, as onChange is what was causing our component to call our service to update the date. With 522 that was going to change to the newly consolidated onBlur, meaning we would need the ambiguous date resolution to occur before onBlur as well.
521 went through and worked well for our onChange implementation. Once 522 was completed and we updated our implementation (making it much cleaner and easier for us, thank you again), DST issues reappeared.
Steps to Reproduce

Have some external service called when onBlur is invoked from a date-time component.
Select an ambiguous hour (Nov 3, 1:15) and navigate away from the component.
Modal appears simultaneously with update going through, the value from the modal is essentially lost.

Additional Context / Screenshots
Below you can see that the time zone is not being honored.

These screenshots were generated by updating to the ambiguous hour, selecting and deselecting the component, which generates the modal (our wrapper will not call our service if the underlying time did not change), and picking one and then the other of the offsets. Despite changing the offset, the underlying time did not change.
Expected Behavior
If the ambiguous hour is entered, onBlur is not called until after the modal is dismissed. The modal does not put focus back into the other parts of the component. To a consumer/user, interacting with the modal is still part of modifying the time, and onBlur (at least as requested in 522) is a signal that the user is done modifying the time.
Possible Solution
Check if we're in the ambiguous hour prior to calling the onBlur prop, call the onBlur prop whenever the modal is dismissed.
@ Mentions
@benbcai

@benbcai @jmsv6d
I put together an initial draft here: https://github.com/cerner/terra-framework/compare/on-blur-DTP Please note that I had to hack around checkAmbiguousTime a bit to get TimeClarification dialog open in India (it shall be removed later).
It'll be great if you could take a quick look and let me know if there are a certain cases that I should probably test before I go ahead and open a PR if this draft looks good.

@nramamurth if it's not too much work to get a snapshot out there I'd be happy to play around with it some

@prmeznar
Great! Thank you and I appreciate it. I have cleaned it up a bit and here is the branch that you could point to and play around - https://github.com/cerner/terra-framework/tree/on-blur-DTP. I have also reached out to you on Teams.

@prmeznar
I have a snapshot built as well and you could use the snapshot if you do not want to point to the branch.
"terra-date-time-picker": "https://github.cerner.com/terra/terra-snapshots/blob/master/terra-framework/on-blur-DTP/terra-date-time-picker.tgz?raw=true",

Snapshot behaves better in that the blur does not occur until the dismissal of that window.
However, in some of our crazier workflows there's still some unexpected behavior. For the workflow:

Update to the ambiguous hour, modal pops
Specify the hour, close modal, our service is called and page refreshed with updated value

On our refresh, we will no longer display the timezone face up, which our strategists have OKed (Since we provide times in UTC, ideally the component could figure out (or at least intelligently guess) at before/after daylight saving given system time zones, and display that guess without needing a blur, but it's not a priority)

Click into the field, blur without changing any values, modal pops
Specify the same hour (so CDT -> CDT), modal closes, I assume blur is called but it does not update due to our consuming logic (the time has not changed)
Edit the time (say, from 1:30 -> 1:40). Blur does not seem to get called but I would expect it to.

You can see a video of the last three steps of that process here
I'm guessing this is because the logic is "if you're in the ambiguous hour only invoke blur when the modal closes" but I think better reasoning is "only allow blur when the ambiguity is resolved and the date time component loses focus" -- and then anytime that little timezone specifier thing shows it would qualify as 'resolved' (along with closing the modal). You can see the current approach suggested by me above but I hadn't thought it fully through - sorry.
As a side, I would expect to be able to update the time and immediately click that time zone specifier without blur being invoked (i.e., I'd say that time zone qualifier is part of the date time component). That may be 'free' - I'm not sure how y'all managed the magic of not calling blur when hopping from one 'subcomponent' to another, but I'm super glad you did.

@prmeznar
Thanks for the feedback and I appreciate it. I think that's a legit scenario and I have updated the snapshot to account for it. If it's not too much work, then could you test it out and lmk how it behaves?

@nramamurth Looks good! Thanks for the quick turnaround.


[Question] How to get filesystem access if running in browser (yarn run web)

Hi,
can anybode give me a few hints how to access a (persistant) filesystem if running vscode in the brwoser via
yarn install
yarn compile
yarn run web

On a clean run yarn run web --host <IP_ADDRESS> --port 8089 its complaining about a non exisiting filesystemprovier for MemFs.
If I add a (the?) memfs Extension "jrieken.vscode-memfs-0.0.3.vsix" via yarn run web --host <IP_ADDRESS> --port 8089 --extention /path/to/the/vsix it works in-memory.
What is the correct approach to access a persistent filesystem (at the moment, regardless its on the host or client, but preferable on the host)?
Thanks a lot

Thanks but we are currently giving L1 support for running VSCode for Web. The yarn web support is for testing purposes only.


Anisotropic Filtering

I've been unable to find any good resources on anisotropic filtering (best being https://registry.khronos.org/OpenGL/extensions/EXT/EXT_texture_filter_anisotropic.txt which doesn't exactly contain much info, and https://www.sciencedirect.com/science/article/abs/pii/S0097849399001594 which is locked behind a paywall) and I'm not a professional graphics programmer.
Leaving it on help wanted for now in case someone who knows more about texture filtering comes across the repo.

This is no longer valid with the library's shift in scope


How to custom key for route

I want to use back by key, And the key is random.
what can i do ?

i have a same problem

@skevy

did you guys figure this out?

Hi! In an effort to get the hundreds of issues on this repo under control I'm closing issues tagged as questions. Please don't take this personally - it's simply something we need to do to get the issues to a manageable point. If you still have a question I encourage you to re-read the docs, ask on StackOverflow, or ask on the #react-navigation Reactiflux channel. Thank you!

Still don't know how to set a key

cc @ericvicenti - we need some better docs around keys ;)

any progress here? still don't know how to specify key.
I need to know the key to access a shared view across app and avoid multiple instances of that shared view.

any update on this ?

please create a new issue and provide info about what exactly you're trying to accomplish @smitthakkar1 :)


Bracket Colorization Not Working

Issue Type: Bug
First, have bracket pair colorization working. Then turn off the setting in the settings page. Then turn it back on again.  It doesn't work. I've tried restarting vscode, editing the settings file by hand, and I've restarted my computer.
This is what I've tried adding in by hand.
"editor.bracketPairColorization.enabled": true,
"workbench.colorCustomizations": {

        "editorBracketHighlight.foreground1": "#00ff4c",
        "editorBracketHighlight.foreground2": "#00ffe5",
        "editorBracketHighlight.foreground3": "#009dff",
        "editorBracketHighlight.foreground4": "#ff65de",
        "editorBracketHighlight.unexpectedBracket.foreground": "#ff0000"

}

VS Code version: Code 1.67.1 (da15b6fd3ef856477bf6f4fb29ba1b7af717770d, 2022-05-06T12:37:03.389Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 7 4800HS with Radeon Graphics          (16 x 2895)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.42GB (10.29GB free)

Process Argv
--crash-reporter-id da4ce562-e8f1-4c5c-bb8c-6df6cacd61b9

Screen Reader
no

VM
0%

Extensions (26)

Extension
Author (truncated)
Version

project-manager
ale
12.5.0

doxdocgen
csc
1.4.0

c-cpp-compile-run
dan
1.0.15

vscode-html-css
ecm
1.12.2

vscode-great-icons
emm
2.1.86

beautify
Hoo
1.5.0

better-cpp-syntax
jef
1.15.13

clojure-snippets
leo
0.0.6

csharp
ms-
1.24.4

python
ms-
2022.6.0

vscode-pylance
ms-
2022.4.3

jupyter
ms-
2022.4.1011282140

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.6

remote-containers
ms-
0.234.0

remote-ssh
ms-
0.80.0

remote-wsl
ms-
0.66.2

atom-keybindings
ms-
3.0.9

cmake-tools
ms-
1.10.5

cpptools
ms-
1.9.8

cpptools-extension-pack
ms-
1.2.0

vsliveshare
ms-
1.0.5561

vsliveshare-audio
ms-
0.1.91

LiveServer
rit
5.7.5

guides
spy
0.9.3

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebug:30475216
vscaat:30438848
vsclangdf:30486550

I made the bug report after hours of searching and not being able to figure it out. I finally figured out the issue. The issue is that this was also in my settings.json.
"editor.language.colorizedBracketPairs": [
],

So if you have the same issue, that might be something to look at.


Added webknox.com

Thanks
Fake: https://api.webknox.com/q/failed-to-import-spring-security-bean-configuration-file-from-url-location
Original https://stackoverflow.com/questions/6122358/failed-to-import-spring-security-bean-configuration-file-from-url-location


[Snyk] Security upgrade setuptools from 65.5.0 to 65.5.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

resotolib/requirements-test.txt

⚠ Warning
virtualenv 20.16.7 has requirement importlib-metadata>=4.8.3; python_version < "3.8", but you have importlib-metadata 4.2.0.
pytest-cov 4.0.0 requires coverage, which is not installed.

Vulnerabilities that will be fixed
By pinning:

Severity
Issue
Upgrade
Breaking Change
Exploit Maturity

Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-SETUPTOOLS-3113904
setuptools:  65.5.0 -> 65.5.1
No
No Known Exploit

Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

Superseded by #1282


Contact maintainer?

How do I contact the maintainer? pip is serving a really, really, old copy of gps.

If you mean the maintainers and adminstrators of warehouse, this is as good a place as any. 😄
If the project you mention is https://pypi.org/project/gps/, I can't reproduce a problem with pip installing an old version. The current version is 3.9, and that's what I get.
It could be a caching issue on PyPI's CDN, I suppose, in which case some details about your network environment (those requested by https://github.com/pypa/warehouse/blob/master/.github/ISSUE_TEMPLATE/access-issues.md) could help locate the problem.

@jamadden "The current version is 3.9, and that's what I get."
No, the current gpsd version is 3.18.1.  3.19 will ship in July.  3.9 is six years old and not compatible with any gpsd of the last 5 years.  This package is useless.
Feel free to install gpsd from your fave distro and see if that pip package works with it.
Note the upstream dev is noted as Eric Raymond.  Feel free to contact him for confirmation.

I see that @frostoov is the PyPI user who uploaded version 3.9. The release history says he uploaded it on July 3rd, 2018.
I suggest @frostoov upload 3.18.1 (released 2018-10-19); if he doesn't respond to this GitHub mention, I see his email address is listed on his GitHub profile, so @garyemiller could contact him that way.
If he doesn't respond, then please comment here and say so; in that case the process of claiming an unmaintained/abandoned project needs to proceed and we'll tag and follow up on this issue accordingly.
Hope this helps.

How do I contact @frostoov ?
I see no email or contact info for himm on his PyPi user page.
I see no @frostoov on github.
I'm lost here.  Seems to me his PyPi user page should have contract info?

PyPI does not disclose email addresses of users.
https://github.com/frostoov is working for me and shows an email address, is that not working for you?

"PyPI does not disclose email addresses of users."
Nor did I ever ask for that.  I just want a contact method.  An html form would do.
"https://github.com/frostoov is working for me:
Thank you, I did not find that.  It should not be that hard to contact a package maintainer.
Email sent to him.  Fingers crossed.

When I mentioned @frostoov in my comments, in the GitHub web interface that turned into a hyperlink linking to https://github.com/frostoov . In the future, if someone uses the @ symbol to mention someone in a GitHub issue or comment, you can find their GitHub profile that way.
We have #933 and #5944 open to add ways to make it easier for users to contact package maintainers via PyPI.
I'm closing this issue since you've now found a contact method to ask the maintainer to publish a new release. If the maintainer doesn't respond and you'd like to request a transfer of ownership, please comment here, reopen the issue, and say so. In that case we'll start on a PEP 541-compliant process and follow up on this issue accordingly.
Best wishes in handling your "GPSes and other navigation-related sensors"! :-)

The issue still remains.  I can not contact @frostoov and he is "releasing" old and broken software that people think is part of the gpsd that does not work with the gpsd project and has no possible use.
reopen, or I create a new issue, until this get fixed.

To reconfirm: I have sent emails to<EMAIL_ADDRESS>and receeive NO response.
PEP 541 is not exactly clear.  Do I need to now wait six weeks for an answer before proceeding?  Seems to me that 6 years without an update should be clear indication of being unmaintained.

How do I demonstrate, for the PEP, my contact attempts?  Who should I copy?

Hi @garyemiller. Please open a PEP 541 issue referring this one and we'll start the due process. This will start with us moderators attempting contact directly. Please understand this is a volunteer ran process so it might take some time.
We do follow the PEP closely so yes, a period of 6 weeks is required until the project is considered abandoned. Please refer to the PEP for more details on what to expect moving forwards.
Thanks for understanding.

Strangely @frostoov has reached out to another member of the gpsd team about this.  Much confusion as he has no clue what is going on, and the assumption is that I was also copied, but I have not been.
So the mess continues, but now a mess in my playpen that I can likely sort out.  Thanks for the help, gpsd is now getting back on track.


GitHub Data Connector issue with KQL query for table GitHubAuditLogPolling CL

Upon installing GitHub Data Connector
The Sentinel Workbook errors and KQL queries looking for a table: "GitHubAuditLogPolling_CL", that is missing.
To Reproduce
Steps to reproduce the behavior:

Go to 'Sentinel Content hub'
Click on 'Search for GitHub
Scroll down to 'Create GitHub data Connector' and configure personal access token (making sure to add Admin.org on the access token for permissions). Once the GitHub content hub has been created, configure the repo organizational name, and add in access token. Refer to these instructions: https://techcommunity.microsoft.com/t5/microsoft-sentinel-blog/microsoft-sentinel-continuous-threat-monitoring-for-github/ba-p/3037154
See error, once you try and run a KQL query when opening the GitHub/Sentinel Data connector

Expected behavior
Audit logs can be pulled by running a Sentinel query using the Log Analytic KQL query, also the Sentinel Workbook for GitHub should work and not error out.
Screenshots
Adding two screenshots to show the error and what looks like a missing table. I also found out that Dicolani is no longer with the Security CXE and was the author for this GitHub data connector in the content hub. Please, help, thank you!
Desktop (please complete the following information):

OS: [Windows 11]
Browser [Edge
Version [21H2]

Additional context
Add any other context about the problem here.

Looks to me like you don’t have GitHub data being ingested and thus those tables don’t exist in log analytics, hence why the queries and workbook will throw out errors.
Based on the screenshots can you confirm that the following tables exist in your log analytics workspace?
Github_CL
Githubauditpolling_CL


[discussion] How to set clusterspec

Now we set the TF_CONFIG to get the cluster spec in the training code, and it follows the idea in Google Cloud Machine Learning Engine (Cloud ML Engine).
In TensorFlow distributed training docs, it uses CLI arguments instead. We can not support those two ways since we want to hide the service discovery layer from the users, so I think we could discuss which one will be supported. Maybe AI engineers could give us more info.
\cc @DjangoPeng @ScorpioCPH

TF_CONFIG is a TensorFlow convention that TF APIs like the EstimatorAPI use to get information about the runtime environment and configure the job appropriately.
If users want to use command line arguments, they can write a launcher script that parses TF_CONFIG and sets the command line arguments as needed. This is what we do in the TFCNN example; see https://github.com/kubeflow/kubeflow/blob/master/tf-controller-examples/tf-cnn/launcher.py
I think the problem with command line arguments is that everyone will use slightly different conventions.

SGTM, WDYT @DjangoPeng


Outlining

This PR adds outlining to common top level code constructs (Types, functions, inner modules, etc.)

Good job. What do you think of adding unit tests?
Either core unit tests like https://github.com/fsprojects/VisualFSharpPowerTools/blob/master/tests/FSharpVSPowerTools.Core.Tests/SourceCodeClassifierTests.fs or Visual Studio unit tests like https://github.com/fsprojects/VisualFSharpPowerTools/blob/master/tests/FSharpVSPowerTools.Tests/HighlightUsageTaggerTests.fs would be great.
I'm sure we will miss some cases in AST traversal, the tests will help a lot in case of ruling out regressions. Thanks.

:+1: for tests.

Looks good to me, but tests for Outlining.visitAst are needed.

Regarding the bug where outlining may not appear when opening files, I can understand the symptom:

When we first checking the project via active document, ActiveDocument might still point to the old document hence no outlining tag is created.

The dependency on ActiveDocument is unfortunate. We need it to retrieve project settings from Visual F# Tools via reflections. In SyntaxConstructClassifier, we bypass this issue by spamming (kind of) doUpdate request asynchronously if the first call doesn't return any tag. See https://github.com/fsprojects/VisualFSharpPowerTools/blob/de5773b851154cb87746fe2f8c89d027878a0488/src/FSharpVSPowerTools.Logic/SyntaxConstructClassifier.fs#L411-L416 .
It isn't ideal but it works well in practice. I think you can do similar things here. We're also open for better solutions :-).

Please, rebase on master.

Multiline do blocks are not handled:

Missing Functionality

[ ] menu page for outlining where you can check which language constructs you want outlined
[ ] open blocks
[ ]  #* blocks
[ ] and for mutually recursive types and functions
[ ] lists
[ ]  arrays
[ ] attributes
[ ] computation expression
[ ] if
[ ] else
[ ] elif
[ ] then
[ ] then in constructors
[ ] with in object expressions
[ ] member in object expressions
[ ] member in type extensions
[ ] match
[ ] lambdas
[ ] list comprehensions
[ ] for
[ ] (* *) comments
[ ] /// comment blocks
[ ] // comment blocks

@cloudRoutine great list :) what about merging this PR to open the door for new ones?

@cloudRoutine great list :) what about merging this PR to open the door for new ones?

@cloudRoutine great list :) what about merging this PR to open the door for new ones?

@cloudRoutine great list :) what about merging this PR to open the door for new ones?

I've noticed you guys don't use feature branches. They seem like they'd be a good improvement to the overall workflow.
You could close this PR, create a new branch off of master called outlining, @dkinitiate could open a new PR onto that branch and then you merge this in. Then PRs could be added against that branch until it's feature complete. But honestly I'd just merge this into master, then create the outlining branch and then have all the PRs be against that feature branch

I'm merging this PR. Please create separate PRs to fix issues in the feature.

https://github.com/fsprojects/VisualFSharpPowerTools/issues/1152


Lucky Die game Added

As described in the issue #425
Added the Dice Game , included the Readme file with the proper
explanation.
here is the demo video in the conversation :
https://github.com/Vikash-8090-Yadav/Future.WebNet/assets/96243602/091b7daf-dcfc-41a5-bec0-0b636bb313e7
I'm a SSOC contributor , please merge my request @shubhisingh184 @Vikash-8090-Yadav

@ayan-joshi please upload the demo video again as it's not uploaded correctly.

Sorry for the previous mistake , I did it now

@ayan-joshi please also attach demo video correctly in README.md file

@ayan-joshi please also attach demo video correctly in README.md file

did it aswell

@ayan-joshi still the video isn't properly attached in README.md file. Please check it and upload it properly.

@ayan-joshi still the video isn't properly attached in README.md file. Please check it and upload it properly.

did the change and I can view the video in the readme and I have also added the video in the project file also

@ayan-joshi the working video is not properly uploaded in README.md file yet, it's on the code part but when you preview it, it doesn't appear. Please check it once carefully.

@ayan-joshi if you want to contribute in SSOC'23, then you should complete it ASAP.

@shubhisingh184 now it's done probably done

@ayan-joshi now it seems fine to me.


⚠ Turnos 24x7 has degraded performance

In c8f2176, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 990 ms

Resolved: Turnos 24x7 performance has improved in b1dfdb5.


🔨 Move export functionality to base anomaly module

What is the motivation for this task?
When exporting, the anomalib engine uses several functions defined in anomalib.deploy.export. A cleaner solution would be to add the export functionality to the base anomaly module, similar to LightningModule.to_onnx.
Describe the solution you'd like
Add the following methods to AnomalyModule: AnomalyModule.to_torch, AnomalyModule.to_onnx and AnomalyModule.to_openvino. This would simplify the export syntax from:
from anomalib.models import Padim
from anomalib.deploy.export import ExportType, export_to_onnx, export_to_openvino, export_to_torch

model = Padim()
export_to_openvino(
    model=model,
    export_root="path/to/export/folder",
)

to:
from anomalib.models import Padim

model = Padim()
model.to_openvino("path/to/export/folder")

Additional context
No response

Hi,
Do you think it would be better to retain the anomalib.deploy.export functions after refactoring or else to move entirely the export functions to AnomalyModule?

@samet-akcay can you assign this task for me pls? I'm working on it right now. Thanks.

All yours :)


Crop dimensions incorrectly scaled when windows UI scaling set

When I have my Windows scale settings set to 150%, the crop dimensions show up as 1.5x the value they are supposed to be.
Repro steps (Windows 10):

Open Cortana/Search (Windows key)
Type "Display Settings" and open the first result
"Change the size of text, apps, and other items:" - change to 150%
Reboot (could just log off and log back on supposedly)
Launch ScreenToGif
Open the editor
Drag/drop a gif onto the editor
Switch to the "Image" tab
Click "Crop"
Drag corners of crop tool to upper left and lower right of image

Expected:

Image extents should be the width & height of the original gif (in my case 367x255)
The image should fill the entire crop extents in the "Preview" (WYSIWYG) area
The entire image should show in the "preview" (text entry) area

Actual:

Image extents are the width & height of the original gif times 1.5 (in my case, 551x383)
The image shows up as ~2/3 the crop extents in the "Preview" (WYSIWYG) area
The entire image shows up in the "preview" (text entry) area - note: this matches expected behavior

Thanks for the feedback.
The viewer is not fully synced with the visual aid (drag elements) and the numerical boxes of the crop feature.
Working on it.

Done, fixed.
Now I need to test if I didn't break anything else. Would you kindly help me out with this task?
I have a 1080p monitor in a 15' screen, it is quite hard for me to work with that DPI factor.
Anyways, I'll attach the executable with this bug fix. (also there's two features ready for the next release)
ScreenToGif 2.7 Preview.zip

It looks like all other features that touch the image got broken with this fix for the crop. It should be easy to fix all.


Fixed typos

Hi @pborreli,
Thanks you fix typos :) !
Can you fix tests please ?

@cedvan done.

:+1:

:metal:


refactor(run.sh): converted run.sh to a rakefile

Merge activity

Jun 8, 7:51 PM EDT: @delta1024 started a stack merge that includes this pull request via Graphite.

#20  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @delta1024 and the rest of your teammates on  Graphite


Error on med sized PR's

When using it to review a med sized PR it threw this error.
json result: {"error":{"message":"This model's maximum context length is 4097 tokens. However, you requested 4788 tokens (2788 in the messages, 2000 in the completion). Please reduce the length of the messages or completion.","type":"invalid_request_error","param":"messages","code":"context_length_exceeded"}}
The documentation from OpenAI mentions that these are hard set sizes and you would need to batch them to get them though.  Would it be possible to batch request over a certain size?

we reduced the max tokens of completion from 2000 to 500, so from now we can use 3597 tokens when review the PRs


Release dotnet-sdk-extensions 1.0.13-alpha NuGet package

What NuGet package do you want to release?
dotnet-sdk-extensions
What is the new version for the NuGet package?
1.0.13-alpha
Auto-generate release notes?
No
Custom release notes?
No response

/retry-nuget-release


nf-core download error

Description of the bug
Hello, there is an error when I want to download nf-core/rnaseq.Can you kindly tell me how to solve it? Thanks in advance.
Here is Error:GitHub API PR failed - got return code 401 from https://api.github.com/repos/nf-core/rnaseq/branches
Command used and terminal output
nf-core download rnaseq

Relevant files
No response
System information
No response

Hi @SNOL2 . This is an issue with nf-core tools and not the pipeline so transferring the issue there.


pr

@Gitprasannag17 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Required validation failed

Hi, I'm trying to set up notifications for my use case and currently can't even get frigate-notify to run in docker.
This is the ever repeating log:
INF Frigate Notify - v0.3.5
INF Starting...
WRN Config file could not be read, attempting to load config from environment
FTL Failed to load config from environment! error="alerts: required validation failed, frigate: required validation failed"

I'm running frigate through docker and WSL (Ubuntu) on a windows machine with Nvidia GPU. Frigate is running without any issues so I'd hope its not my general setup.
Here is my compose file:
services:
  frigate:
    # frigate stuff
    ports:
      - "8971:8971"
      - "5000:5000"
      - "8554:8554"
      - "8555:8555/tcp"
      - "8555:8555/udp"
  frigate-notify:
    image: ghcr.io/0x2142/frigate-notify:latest
    environment:
      TZ: Etc/UTC
    volumes:
      - C:\Users\USER\frigate\frigate_notify\config.yml:/app/config/config.yml
    restart: unless-stopped

And my notify config:
frigate:
  server: localhost:5000
  webapi:
    enabled: true
    interval: 60
alerts:
  general:
    title: test
    timeformat: 2006-01-02 15:04:05 -0700 MST
    nosnap: allow
    snap_bbox: true
    snap_timestamp: true
    snap_crop: false
    notify_once: true

The mounting works fine, I can see and open the config within docker. From the error I would assume, the issue is with the contents of my config? Is localhost not an acceptable address? I tried all kinds of addresses and ports, nothing worked.
I don't know anything about mqtt, so I haven't use that in frigate.
If you need to know anything else, let me know!
Thanks for your help, your work is greatly appreciated.

Hello there & thanks for using this app!
So the key here is the log that states WRN Config file could not be read. The default location for the config file is /app/config.yml, or it can be overriden by using the FN_CONFIGFILE environment variable as shown here.
In the config shared, it looks like you're mounting the config to /app/config/config.yml - so my best guess is that the app can't find it. I would suggest either updating the volume map to /app/config.yml, or use the env var if you like!

@0x2142 Hi, thanks for the quick answer. The solution was indeed that simple. The documentation installation page has it correct, I just failed to see my mistake.
Now im getting:
WRN Cannot reach Frigate server at http://localhost:5000 error="Get \"http://localhost:5000/api/stats\": dial tcp [::1]:5000: connect: connection refused" attempt=1 interval=30 max_tries=5
But I'm already happy seeing a different error. When checking the api/stats in browser, it returns json without issue.
This may also be a very beginner level problem but I hope simple questions will also help others in the future.
Want me to open a separate issue for this or is it also an easy fix?
Thanks again.

That should be a simple fix as well 😊
In the current frigate-notify config, it's using server: localhost:5000 - which is telling frigate-notify to check port 5000 on itself, not the Frigate container.
In your docker-compose.yml, you can set a container_name for Frigate, which can then be accessed via DNS by frigate-notify. So update your compose file for the Frigate container to include that, like shown below:
services:
  frigate:
    container_name: frigate
...

Then, in the frigate-notify config - try setting the server value to frigate:5000 instead & try restarting.
Please note that since you're running both apps in the same compose file, you may still see frigate-notify say it can't reach that at first, since it may take Frigate a moment to start up & the API to be accessible. But once Frigate starts & the API is available, then frigate-notify should connect.

Perfect! Everything is working now, thanks a lot for the help :D
I noticed that the notifications are only sent after an event has concluded, is it possible to sent one as soon as an object is being detected? My use case is kind of like a doorbell so speed is of the essence.
I will close the issue with this comment since my problems have been resolved and this is more of a general question.

Glad to hear it's working for you!
So currently you're set up to use the Web API to check for events, not MQTT. With the Web API, we have to pull events manually from Frigate - which is done on a scheduled interval. By default, this is every 30 seconds but this can be overridden in the config here. That being said the Web API will always be a bit slow because we can't process the event until Frigate processes & publishes it to the API + the lag between that & when frigate-notify runs the next API query to look for events.
If you want instantaneous, you would need to look at using MQTT instead. The way this works, is that there is a separate message broker that frigate-notify subscribes to. Any time there is an event, Frigate will publish the event info to that message broker immediately - and since we're subscribed to those events, that means frigate-notify gets those events as soon as they're published.
The Web API polling method is here since not everyone wants to use MQTT, since it is an additional service to run. However, there are some limitations & speed of detection-to-notification time is one of them.
Hope that helps!

Hi... I have the same error but not the warning:
2024/12/29 01:48:46 +0000 INF Frigate Notify - v0.3.5
2024/12/29 01:48:46 +0000 INF Starting...
2024/12/29 01:48:46 +0000 FTL Failed to load config from file! error="alerts: required validation failed"

here's my compose file
services:
  frigate-notify:
    image: ghcr.io/0x2142/frigate-notify:latest
    environment:
      - TZ= Etc/UTC
    volumes:
      - /srv/dev-disk-by-uuid-aa949b97-12b8-48c8-b6a4-e09a65bc3e34/frigate-notify/config.yml:/app/config.yml
    restart: unless-stopped

I have double check the permissions and even run from the console using:
sudo docker run -v /srv/dev-disk-by-uuid-aa949b97-12b8-48c8-b6a4-e09a65bc3e34/frigate-notify/config/config.yml:/app/config.yml ghcr.io/0x2142/frigate-notify:latest
Still cannot see the error on my config


Add TypeScript Checker

TypeScript Checker for TS 1.5.
There is no simple way to skip code generation for TS1.4-. But at least I hope this can make the best of evergreen TS compilers. Please refer to https://github.com/scrooloose/syntastic/pull/1382 for more info

LGTM, although at some point we will probably need to add a version check (Neomake does not support this yet - but no need to bother with older TS versions). Thanks for the contribution!
@benekastah Should I merge PRs if I reviewed them and found no issue, or do you prefer to review everything yourself?

@Neki In the future, feel free to merge if it looks good to you!


Algorithms page update

Proposed changes
I've updated the Algorithm lists and links on the Algorithm’s main page,  specifically I

Added Space and Time complexity headers with links
Added a Divide and Conquer header with algorithm list
Added an Algorithm design header with the Sliding Window Technique
Added a String Based Algorithms header with algorithm list
Verified that all links on Algorithm page work correctly
Updated links on the following Algorithm subpages:

Backtracking
Branch and Bound
Divide and Conquer
Dynamic Programming
Graphs
Greedy Algorithm
Searching

Fixed header name on word boggle page
Changed algo-rithms to algorithms on sorting page
Fixed spelling error in sliding window algorithm

Types of changes

[x] Proofreading (fix spelling, grammar, punctuation, accuracy) Proofreading
[x] New content (new section, new readme, etc.)
[x] Modifications (modifying existing content)
[ ] Other (please describe)

Checklist

[ ] Tested any code that was added/modified in this PR
[ ] Added external links
[x] Added internal links
[ ] Added images
[ ] Added videos
[ ] Added code snippets

Further comments
I did rearrange the order of the algorithm topics on the algorithms page, but I kept most of the wording the same.  I did proofread the entire algorithm page as well.  The main page has the most changes, and the algorithm topics were only updated to match the algorithm page.
Reviewers:

[ ] @username

I wanted to mention that while working on this issue I found that some algorithms are duplicated.  I don't think it's a big problem right now, but I made a list for those who wanted to know.
Here is a list of all the algorithms that had multiple pages and where they are located.
Duplicated algorithms

Linear and sequential search -> These two pages describe the same topic, and they look very similar.   Linear Search seems more complete.

Algorithms/Searching/LinearSearch/readme.md
Algorithms/Searching/SequentialSearch/readme.md

Fibonacci sequence -> These two pages describe the same topic; Fibonacci Sequence looks more complete.

Algorithms/Dynamic Programing/Fibonacci Sequence/README.md
Algorithms/Dynamic Programing/Nth Fibonaci/README.md

Floyd Warshall’s Algorithm

Algorithms/Dynamic Programing/Floyd Warshall Algorithm/readme.md
Algorithms/Graphs/Shortest Path/FloydWarshallAlgorithm/readme.md

Kruskal's Algorithm
Algorithms/Graphs/Spanning Tree Algorithm/KruskalsAlgorithm/readme.md
Algorithms/Greedy Algorithm/Krushkal's Algorithm/readme.md
Prim's Algorithm

Algorithms/Graphs/Spanning Tree Algorithm/PrimsAlgorithm/readme.md
Algorithms/Greedy Algorithm/Prim's Algorithm/readme.md

Knapsack Problem

Algorithms/Branch and Bound/Knapsack Problem/README.md
Algorithms/Dynamic Programming/Knapsack Problem/README.md

Dijkstra's Algorithm

Algorithms/Greedy Algorithm/Dijkstra's Algorithm/readme.md
Algorithms/Graphs/Traversal Algorithms/Dijkstra'sAlgorithm/readme.md

Subset Sum Problem

Algorithms/Dynamic Programing/Subset Sum Problem/readme.md
Algorithms/Backtracking/Subset Sum/README.md

Bellman-Ford Algorithm

Algorithms/Graphs/Bellman-Ford Algorithm/readme.md
Algorithms/Graphs/Shortest Path/BellmanFord Algorithm/readme.md

Breath first search

Algorithms/Graphs/Traversal Algorithms/BreadthFirstSearch/readme.md
Algorithms/Graphs/ Traversal Algorithms/AStarSearch/readme.md

No worries, thanks for merging.


Remove time-zone information for online events

When an event is online, we show the start date and time in the user time-zone.
When a tz is present, the date and time of the event in that local time zone is also shown to the user, but this is only intended to on-side events: it is intended to help the user better plan their trip.
It currently show confusing information (local build):

With this change, less confusion for an online-event:

Cc @dtaivpp @nateynateynate

@smortex thanks! That explains that. Also, these don't render on firefox it seems like. Web not moved to prod yet because of that. If you don't have time to look at it I'll be digging in tonight.

Also, these don't render on firefox it seems like.

What? I am using Firefox and it works on my computer :confused:.  For completeness, I run Firefox 102.13.0esr (64 bits) on Debian 11.  The process has a TZ environment variable:
romain@desktop-fln40kq ~ % pgrep firefox
2770
romain@desktop-fln40kq ~ % cat /proc/2770/environ | tr '\0' '\n'  | grep TZ
TZ=Pacific/Tahiti

I'll try to fiddle with my conf to see if I can produce unexpected results. Can you provide a screenshot of the disaster and info about the version you are running / TZ on your system?

@smortex okay so I'm not sure if I need to be mad at firefox or our cloudfront but it seems like it was just a caching issue -_-


Wallet - logout when inactive

Logout from wallet when inactivity time exceeds some threshold time.

Closing as new browser-wallet work is in progress


Changes in orbit_from_name in datacom5.py

According to  orbit_from_name  function description
"""Return :py:class:`~poliastro.twobody.orbit.Orbit` given a name.
Retrieve info from JPL DASTCOM5 database.
Parameters
----------
name : str
NEO name.
Returns
-------
orbit : list (~poliastro.twobody.orbit.Orbit)
NEO orbits.
"""

it returns poliastro.twobody.orbit.Orbit class . It actually returns a list of class objects that needs to be indexed while retrieving the orbit from that list in order to plot.
Plotting the orbit without indexing the value returned by orbit_from_name is producing following error. Traceback has been provided below.
Traceback (most recent call last):
File "/Users/pswaldia1/poliastro/src/poliastro/neos/dastcom5.py", line 595, in < module >
frame.plot(orbit_from_name('Atira'))
File "/anaconda3/envs/virtual_platform/lib/python3.5/site
packages/poliastro/plotting/static.py", line 216, in plot
*self.set_frame(orbit.pqw())
AttributeError: 'list' object has no attribute 'pqw'
According to me there is no requirement of sending the list since we are only returning one orbit at a time . This pull request solves this by returning already indexed list containing a single orbit class (in the function iteself).This would be more convenient for the users as it was for me.

Hi @pswaldia, thanks for your contribution! Before reviewing anything else, one question: how do you reproduce this functionality?
halleys = dastcom5.orbit_from_name("1P")

frame = OrbitPlotter2D()
frame.plot(halleys[0], label="Halley")
frame.plot(halleys[5], label="Halley")
frame.plot(halleys[10], label="Halley")
frame.plot(halleys[20], label="Halley")
frame.plot(halleys[-1], label="Halley")

Ok ! So i didn't checked it for all the cases. A mistake on my side. I am closing this pull request.


As an End User, I'd like to access documentation via docs.okteto.com

People's brains work differently when it comes to guessing where docs are for an organization.  e.g. docs.company.com or company.com/docs.  Me personally, I always default to docs.company.com so it get's my vote ;)
Suggest creating a DNS alias 'docs.okteto.com' that redirects to the current 'okteto.com/docs'.
HTH!

Done!


Storage tests failing sometimes on MacOS

I've been seeing this storage test failing intermittently on MacOS.
X UploadRangeAsync_WithUnreliableConnection [1s 634ms] |
-- | --
  | Error Message: |
  | Final progress has unexpected value |
  | Expected: 1048576 |
  | But was:  131072 |
  |   |
  | Stack Trace: |
  | at Azure.Storage.Files.Test.FileClientTests.UploadRangeAsync_WithUnreliableConnection() in /Users/vsts/agent/2.155.1/work/1/s/sdk/storage/Azure.Storage.Files/tests/FileClientTests.cs:line 675

Couple example runs:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=88791&view=logs&j=e44bd064-d8d9-5feb-6343-09b8c49ab1a8
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=86776&view=logs&j=e44bd064-d8d9-5feb-6343-09b8c49ab1a8

Another instance:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=88871&view=logs&j=e44bd064-d8d9-5feb-6343-09b8c49ab1a8
If we keep hitting this I will disable this test until it can be fixed.

This is a dup, so clsoing and linking to the other.


Can't Install on Backbox

sudo bash install.sh
[sudo] password for user1:
Failed to detect the version of Linux
This installer only supports the following Linux distributions:

BackBox
CentOS
Debian
Fedora
Kali
Ubuntu

If the current version of Linux is one of these flavors but it is
not recognized, please open a support ticket and include the version.
└─[~]>cat /etc/*-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=14.04
DISTRIB_CODENAME=trusty
DISTRIB_DESCRIPTION="Ubuntu 14.04.5 LTS"
NAME="Ubuntu"
VERSION="14.04.5 LTS, Trusty Tahr"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 14.04.5 LTS"
VERSION_ID="14.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
└─[~]>cat /etc/issue
BackBox Linux 4.7

waiting ....

The issue has been assigned. It'll be updated when the status has been changed or we have questions.

Currently the tools/install.sh script only supports Backbox versions 4-6. This issue will be addressed in the next release.

Thanks, i guess i have to wait more

The dev branch now has an updated tools/install.sh script to support the newest version of BackBox and future minor releases of BackBox 4.x.

The next release will be this Tuesday, and at that time the changes that are currently in the dev branch will be merged into the master branch.

The 1.6.0 release is now complete.


Upgrade to Sparkle 2

Upgrade from Sparkle 1 to Sparkle 2. Because Sparkle 2 now requires minimum macOS version to be 10.13, we can only use this for normal MacVim builds, but legacy binary (for 10.9-10.12) will need to use Sparkle 1 still. We now have separate Sparkle_1 / Sparkle_2.framework folders and symlink Sparkle.framework to Sparkle_2.framework. When doing legacy builds in CI, we change the symlink to point to Sparkle_1.framework, and set the corresponding compiler ifdef so the code will know what to do.
Also, add support for a new "prerelease" update channel (only available in Sparkle 2) which allows us to deploy pre-release / beta software to users who opt in (in the Advanced settings pane). This helps gather feedbacks during development before we push out full releases.

This relies on #1331 so needs that to be merged first.

Ping @ichizok if you are interested to take a look.

Looks from CI that older versions of Xcode do not like a symlinked Sparkle.framework. Will need to do some investigation. This could be mildly annoying because someone on say macOS 10.13 may have problem building this unless they do some extra steps. Maybe instead of using a symlink I will just make Sparkle.framework the version 2 one, and just do the symlink trick only when building a legacy binary.


Map Remote Rules Support

Description
Hi, First of all, thanks for your excellent work on this repo.
I have a question regarding using Map Remote, in Mac in can use Import Settings > From Proxyman > Map Remote...
I was wondering if Map Remote will be implemented in future releases.
Thanks a lot.

@santicomp2014 We definitely port all features from the macOS to Windows. For now, we're focusing on the UI, the Breakpoint, and Map Local. Map Remote is on our road map 👍


Github-Repository-Resource Service Connection Issue

Have you tried trouble shooting?
Yes
VSTS Type and Version
VisualStudio.com
Account Name
http://jeremytcd.visualstudio.com
What's not working?
I'm having a service connection issue when attempting to use a Github repository as a resource. I am aware of the advisory Troubleshoot Azure Resource Manager service connections and I've read through all of the information on repository resources and service connections I could find (article 1, article 2, article 3). I could not resolve my issue, so I created reproduction repositories:
IssueReproduction.RepositoryResource.Template
This repository has a single file, template.yml, with the following contents:
jobs:
- job: Test
  pool:
    vmImage: 'vs2017-win2016'
  steps:
  - script: echo success

IssueReproduction.RepositoryResource.Consumer
This repository has a single file, consumer.yml, with the following contents:
resources:
  repositories:
  - repository: templates
    type: github
    name: JeremyTCD/IssueReproduction.RepositoryResource.Template
    endpoint: https://github.com/JeremyTCD/IssueReproduction.RepositoryResource.Template.git

jobs:
- template: template.yml@templates

It adds the template repository as a resource using the method documented here and uses template.yml.
Pipeline
After creating those repositories, I created a pipline - IssueReproduction.RepositoryResource.Consumer-CI.
Note, initially, consumer.yml had no endpoint option, as per the documentation. Without it, I got these
errors:
Repository templates requires an endpoint

Parameter name: endpoint,Unable to cast object of type 'Microsoft.TeamFoundation.DistributedTask.Pipelines.Yaml.Templates.JobsTemplateReference' to type 'Microsoft.TeamFoundation.DistributedTask.Pipelines.Yaml.Templates.Job'.

The documentation might need updating with regard to endpoint being required.
With the endpoint, I get these errors:
Repository templates references endpoint https://github.com/JeremyTCD/IssueReproduction.RepositoryResource.Template.git which does not exist or is not authorized for use

Unable to cast object of type 'Microsoft.TeamFoundation.DistributedTask.Pipelines.Yaml.Templates.JobsTemplateReference' to type 'Microsoft.TeamFoundation.DistributedTask.Pipelines.Yaml.Templates.Job'.

Here is an example build result: result.
What I've tried
I noticed that tasks accept service connection options, like azureSubscription. The documentation does not specify any equivalent for repositories. I tried random stuff like githubSubscription but nothing worked.
Some Github issues, such as this one suggested stuff like adding variables and switching branches. These seem to apply more for when .ymls are changed after their initial loading. I tried them anyway and nothing worked.
I've checked my pipeline's service connections. Since the root repository is also on Github, Get Sources already has a valid service connection to Github. I'm not sure if this connection is used for all jobs. Regardless, I could not find any way to add a service connection "for the entire pipline".

@JeremyTCD https://github.com/Microsoft/azure-pipelines-yaml is the new repository for all yaml related question moving forward.
you need to create a github endpoint like this:

resources:
  repositories:
  - repository: templates
    type: github
    name: JeremyTCD/IssueReproduction.RepositoryResource.Template
    endpoint: <name you give when create the endpoint>

jobs:
- template: template.yml@templates

@TingluoHuang Thanks for the quick response!

https://github.com/Microsoft/azure-pipelines-yaml is the new repository for all yaml related question moving forward.

Okay, would be helpful if this was noted in the ReadMe for this repository.
The fix worked, though the documentation here needs to be fixed:
resources:
  repositories:
  - repository: string  # identifier (no spaces allowed)
    type: enum  # see below
    name: string  # repository name (format depends on `type`)
    ref: string  # ref name to use, defaults to 'refs/heads/master'
    endpoint: string  # endpoint for a GitHub repository

I've opened a separate issue in the docs repository, will update it.

Also this documentation should be updated accordingly: https://docs.microsoft.com/en-us/azure/devops/pipelines/process/templates?view=azure-devops#using-other-repositories

Hi @TingluoHuang
What is happening, what is wrong? I have referenced this variables in all ways that i know 😞

Any help its appreciated :+1:

@wkrea I think you need to reference your variables in the template like this: $(varname) (notice the parenthesis)


Fixed a previous change that was causing publications to not be added…

… properly. Also adding back in Pubchem assay ids and patent ids.
Using PUBCHEM.AID: and PATENT: as the prefixes for those supporting documents, but I'm not sure if those are the correct prefixes. Is there a resource to check this?

this was merged into #194


dev-cmd/tap-new: add Brewfile documentation to readme template

It would be helpful for taps to document how they are to be used with Brewfile.
Adding documentation for formulae installation to the generated tap readme would help ensure that new taps have this helpful documentation in their repos from the start.
This is primarily a follow-up to https://github.com/Homebrew/homebrew-cask-fonts/pull/9359 which itself was opened after I spent a significant amount of time trying to figure out how to get a font installed as part of my Brewfile.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Good suggested changes @MikeMcQuaid . I'll update the cask-fonts tap PR to match.

Quick aside question: I haven't frequently made changes to brew itself (it's been a long while).
The PR template requests running brew style, brew typecheck and brew tests. But it's not clear if these are expected to work somehow within the context of a Homebrew/brew working copy, or if they only run against the "installed" version of homebrew. Does brew magically detect it's in a clone of brew itself and run against the current working copy? Or do I need to go to /usr/local/Homebrew | /opt/homebrew and check out my branch to run these commands?
(And is this already clarified somewhere that I should have noticed?)

if they only run against the "installed" version of homebrew.

This 👍🏻

Or do I need to go to /usr/local/Homebrew | /opt/homebrew and check out my branch to run these commands?

This 👍🏻


Update accessing-user-space-memory.md

Updated information that explains how drivers can access and allocate memory in user-mode without requiring a thread to be attached in the user-mode address space.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hi @jasonfish4
Thanks for your contribution.
Unfortunately, we are unable to accept the update. The use of MmCopyVirtualMemory is not supported. Because of your contribution, we have updated the first paragraph to clarify this topic.
Thanks,
WDK content writer


FEATURE REQUEST: Complexity-based weighting

I don't know about you, but a 3-Spirit game with River, Lightning and Shadow is very different from a game with - say - BoDaN, Vengeance and Fractured Days! It's not a difficulty thing per se, more a cognitive load thing. I'd love it if I could set (custom) cognitive load factors for difficulty, based on number of Spirits and Spirit complexity.
I'd like to scale difficulty based on Spirit Count, and I'd like to scale difficulty based on Spirit complexity. My personal factors would perhaps be +⅓ difficulty per Spirit beyond the first and +⅓/-⅓ difficulty above below Medium complexity for Spirits. So a 3 Spirit game with Very Complex Spirits would be 2⅔ difficulty even without an Adversary. I'd like that to be factored in when the setup is created.

Interesting thoughts! Thanks for this (and all the other issues you've raised).
I love the idea, though it might require quite a bit of explanation.
I'm thinking maybe a separate value range would be better than making it part of the difficulty calculation though. Will put some thought into it.

Ah, I see.
This would be an on/off option then. Called something like 'Include spirit complexity in difficulty'.
I think that behind the scenes, it might cause some skewing of probablities, but the effect may not be significant.
Certainly doable though. Will see what I can do.

This option is now added.
Currently, the difficulty adjustment is hard coded as making it configurable would add quite a bit of complexity.
I also tweaked the formula slightly to add more difficulty for subsequent spirits as it felt a bit off to me. The second spirit will be +1/3, but the third will be +2/3, the forth +1, etc.
I'm still not sure the balance of the numbers is quite right, but give it a go and let me know what you think.

Leaving open for feedback on this.

Nice! I like this and it seems to work how I'd imagined. I ran a few generated test cases and it looked like stuff that fitted my expectations. Thanks for this

Glad to hear you like it :)


Javadoc fixes

Current coverage is 96.36% (diff: 100%)

Merging #1603 into master will decrease coverage by <.01%

@@             master      #1603   diff @@
==========================================
  Files            88         88
  Lines         10689      10690     +1
  Methods           0          0
  Messages          0          0
  Branches       1824       1824
==========================================
  Hits          10301      10301
  Misses          245        245
- Partials        143        144     +1

Powered by Codecov. Last update c18ab43...0897801


convert gif files to png

any particular reason to have the query images as gif?
after converting to png and running pngcrush,
images went from 245,423 bytes to 196,856
easy pickings...

Be happy to merge a PR if you want to create one.


Where do I get cmake for z/OS (USS)

Hello,
as already asked in the heading, where do I get cmake for USS?
Thanks
Mike

There is a fork for CMake with z/OS support available at https://github.com/fjeremic/CMake. Unfortunately there are no binaries available, so you will need to build it from source.

Thank you.
Will try this out. :)
Best, Mike

There is a better location.  See https://github.com/ZOSOpenTools/cmakeport.  This is a port of the current version.  I think it is currently 3.18 but more current should be coming.

Thanks.

Thanks :)


catch processing errors and emit them as events

Previously, if the server received a malformed packet, the request processing would throw an uncaught error which can end up killing the node process.
Instead, add a try ... catch handler around the Request / Response processing and trigger an error event which can be caught and handled gracefully. Include an error message, the actual Error object, and the connection info in the event callback so aid debugging and make it possible to log where the bogus requests are coming from.

Thank you for this PR! Unfortunately, I am no longer a maintainer in this GitHub account, but I have merged your branch in my own fork, github.com/jhs/dnsd and I have included it in the v0.9.6 release.

Thanks for the quick turnaround. Will your fork be the authoritative repository for this package going forward in case I have other changes to suggest? I assume so given that you updated the npm metadata as such but I just wanted to be sure.

Yes, my fork is the official one. I started the project when I was in the iriscouch organization however I am no longer there. But I still maintain many of the projects here.


Tracer monitor textual user interface

Apologies in advance if this is not the right place to post this request. It feels more like a "new project" request then a "feature". Please let me know if I'm "going against the grain" here with how I'm reaching out, thank you.
Feature/Project "idea"
A TUI (text user interface) application which can display the rillrate tracer data in a terminal. Imagine something like nmon.
The idea here is that you could view the same information that you would from the embedded web server but instead from the comfort of a "terminal" for example over an SSH session.
This is more of an "epic" than an issue but wanted to bring it up and see if anyone has any feedback and/or existing work that may be related in some way.

That's actually cool idea and initially we did it this way: we really had both types of UI: Web and TUI, but later we've decided to suspend TUI for a while to reduce amount of supported code till we will stabilaze the core.
I'll keep this issue open, because we are still considering implementing this feature.


Update Sui wallet and Sui wallet adapter links

Thank you for making this list awesome! Please make sure that your addition follows the format [project_name](link) - Description. and other contribution requirements such as the Add project_name commit format.

Thanks for spotting this @hyd628! I will merge this :+1:


MM-29933 Fix menu rendering behind input box in RHS.

Summary
Fix menu rendering behind input box in RHS.
Ticket Link
https://mattermost.atlassian.net/browse/MM-29933
Related Pull Requests
Reverts some changes made in https://github.com/mattermost/mattermost-webapp/pull/6920

Looks good to me. I tested the following cases that I know were problematic before:

Post menus open downwards above the comment box and the whitespace below it
The RHS stays at the bottom when another user sends a message
The RHS stays at the bottom when you attach a file
The RHS stays at the bottom when the comment box increases in height

Cherry pick is scheduled.


HDP with Spark2.2

Environment:

Python version [3.6]
Spark version [2.2]
TensorFlow version [e.g. 1.10.1]
TensorFlowOnSpark version [e.g. 1.4.3]
Cluster version [HDP]

Describe the Issue:
While running in YARN cluster mode using spark, we are facing error as per attached.
Logs:
If applicable, add logs to help explain your problem.  Note: errors may not be fully described in the driver/console logs.  Make sure to check the executor logs for possible root causes.
Spark Submit Command Line:
If applicable, add your spark-submit command line.

Environment:
Python version [3.6]
Spark version [2.2]
TensorFlow version [e.g. 1.10.1]
TensorFlowOnSpark version [e.g. 1.4.3]
Cluster version [HDP]
Describe the Issue:
While running in YARN cluster mode using spark, we are facing error as per attached.
Logs:
As per attached.

Spark Submit Command Line:
/usr/hdp/<IP_ADDRESS>-235/spark2/bin/spark-submit --master yarn --deploy-mode cluster --queue default --num-executors 4 --executor-memory 27G --files /etc/hadoop/spark_conf/topology_script.py --conf spark.app.name=TensorflowOnSpark --conf spark.sql.broadcastTimeout=1200 --py-files /data1/home/pythonuser/inception.zip --conf spark.dynamicAllocation.enabled=false --conf spark.yarn.maxAppAttempts=1 --archives hdfs:///user/pythonuser/Python.zip#Python /data1/home/pythonuser/TensorFlowOnSpark/examples/imagenet/inception/imagenet_distributed_train.py --data_dir ${IMAGENET_DATA} --train_dir hdfs:///user/pythonuser/imagenet_train --max_steps 1000 --subset train

That is the driver log... can you look in the Spark executor logs for any other errors?   Otherwise, I'd guess that you may need to specify --train_dir hdfs://default/user/pythonuser/imagenet_train.


Difference in response for same prompt Phi-3 4k model

When i go to https://ai.azure.com/explore/models/Phi-3-mini-4k-instruct/version/2/registry/azureml?tid=72f988bf-86f1-41af-91ab-2d7cd011db47&reloadCount=1 and type "Extract keywords from this text: I am having connectivity issues with my device."
Response is following:
Connectivity issues
Device
Problem
Network
Internet
Troubleshooting
Connection
Service
Support
Technical difficulties
When i download and run the ONNX model from  https://huggingface.co/microsoft/Phi-3-mini-4k-instruct-onnx with same prompt using sample from https://github.com/microsoft/onnxruntime-genai/blob/main/examples/csharp/HelloPhi2/Program.cs and pointing it to downloaded Phi-3 model, i get following response:
Prompt:
Extract keywords from this text: I am having connectivity issues with my device.
Output:
Extract keywords from this text: I am having connectivity issues with my device. The Wi-Fi signal is weak and the internet speed is slow. I have tried resetting the router, but the problem persists. I suspect there might be an issue with the device's Wi-Fi adapter.
Does anybody know if this is expected or i need to tweak some parameter genai_config.json?

Thanks, seems like the models from hugging face have different response, i.e. https://huggingface.co/microsoft/Phi-3-mini-4k-instruct

Actually, it worked in the sample project app, thanks for the chat template! Seems like different user interfaces produce different output.


in_memory_engine: trigger loading pending region with background task

What is changed and how it works?

Issue Number: Close #17561

What's Changed:
Add a new periodical task to trigger loading pending region. We use a new casual message to let apply thread triggers loading pending region(with the same mechanism as `prepare_for_region`).

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

/check-issue-triage-complete

@overvenus @SpadeA-Tang PTAL

/retest

/retest

./retest

/retest

/retest

@overvenus  PTAL

/retest


Cleanup: remove lint warning related to 'try' usage

This change eliminates a compile-time lint warning; the indicated try! macro is removed, and an ? operator is introduced instead.
warning: use of deprecated item 'try': use the `?` operator instead
   --> src/parser.rs:179:29
    |
179 |         let parse_key_res = try!(parse_key(key));

There were the following issues with your Pull Request

Commit: 27e868151cc82d53e7f258ced5055376b06b87ff

Commits must be in the following format: %{type}(%{scope}): %{description}

Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Commit:<PHONE_NUMBER>57caae90894a740f6c4f4ae0ddb850

Commits must be in the following format: %{type}(%{scope}): %{description}

Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Guidelines are available at https://github.com/rustless/queryst/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com


pull request for e2e test at 12/04/2016 01:45:03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added symmetry-breaking init in rbf

The init_params method currently initializes (for example) the rbf kernel deterministically. As such, when using a combination kernel of the same kernel types, the kernels are initialized to be identical. This is problematic as the symmetry between them forces them to stay the same using gradient-based local optimization, preventing e.g.  a sum of multiple lengthscales to be learnt (if not breaking the symmetries in some other way).
This adds a random jitter to the RBF kernels lengthscales to break the symmetry. To resolve the same issue for the other kernels, they would need something similar.
The following PR makes the change to pass a unique key to each kernel within the combination kernel:
https://github.com/JaxGaussianProcesses/JaxKern/pull/41

Hi @bodin-e
Thanks for the PR - this is a good catch that hadn’t occured to me. I wonder if it only makes sense to do this when initialising the lengthscale(s) for a combination kernel?  If so, then I’d prefer this change to be move upstream into the init_params() of the Combination kernel class.
This could be handled through a simple if ‘lengthscale’ in params.keys() which would proliferate the stochastic initialisation through to any kernel that is parameterised by a lengthscale, not just the RBF.

I wonder if it only makes sense to do this when initialising the lengthscale(s) for a combination kernel? If so, then I’d prefer this change to be move upstream into the init_params() of the Combination kernel class.

Yes, I would expect it only being needed when using a combination kernel. It would be needed in all cases where any two (or more) kernels are identical, receive identical inputs and is depended on in an identical way. Which in practice probably would just be in the combination kernel case (at least I cannot think of another case now).
However, it does not only matter for the lengthscale parameter. It would be sufficient to break the symmetry by using different initialization of any parameter in each kernel, such as for example the lengthscale, but all kernels used within a combination kernel may not have a lengthscale. And I think it could end up cumbersome to maintain a set of parameter names {"lengthscale", ...} to include at least one parameter name from each of the library kernels, although not impossible.
I completely agree with the need for a nicer and more maintainable solution than the one in this PR though. The main purpose of this PR I had in mind was to highlight the issue (it would have been better to include an example within the issue ticket instead though).
Yet,  I'm thinking it may be the easiest to maintain and most readable to let the init_params of each kernel initialize itself sensibly, including a stochastic component to it; added to the parameter that it makes sense to add it to for the given kernel (and with a suitable jitter scale for that particular parameter).
For example, in the RBF case we (and a user using the library) can easily read within the function scope that the default lengthscale is 1; and we know that a small jitter constant (say 1e-3 or 1e-4) shouldn't be semantically meaningful for the lengthscale in the RBF (in relation to value 1). In contrast, in the some kernels or setups some parameters can be much more sensitive to its value. If a user uses such as kernel they would easily be able to see how the particular kernel they use is initialised, and can themselves then override the init_params behaviour or use some other way to initialize it.
What do you think of the following? Basically doing exactly what you purpose, which is to reuse initialization logic for the lengthscale (and other parameters when needed). But keeping this logic as separate functions and use them explicitly where needed, to keep high readability and reduce the risk of the user missing what is happening (and getting unexpected outcomes because of it).
Specifically, keeping a file or module with parameter initialize functions somewhere, with functions such as something like: def initialize_lengthscale(key, num_dims, initial_value=1.0, eps=1e-4):, which takes care of both the current initialization logic for that parameter (e.g jnp.array([1.0] * ndims)) but also adding on the stochastic jitter. And then use these parameter initialization functions within each respective init_params functions as and when it is appropriate, such as in matern, rational quadratic, rbf etc. And have and use similar functions for the other parameters and in other kernels. Although it would only be necessary to add stochastic jitter to one of the parameters in each kernel, it wouldn't hurt to add it to more than one either. And if there is a parameter for a kernel somewhere for which it is important no jitter is added its init_params can easily omit doing so.


MobilenetV3 pretrained weights not loading

Hello @zetyquickly ,
I am using densepose_rcnn_mobilenetv3_rw_FPN_s1x.yaml to train a model using MobilenetV3 backbone.
When I kickoff the model training, I see this message in logs:
[03/18 15:12:21] d2.data.build INFO: Using training sampler TrainingSampler **[03/18 15:12:22] fvcore.common.checkpoint INFO: No checkpoint found. Initializing model from scratch** [03/18 15:12:22] d2.engine.train_loop INFO: Starting training from iteration 0 [03/18 15:12:50] d2.utils.events INFO:  eta: 12:19:36  iter: 19  total_loss: 17.218  loss_cls: 1.211  loss_box_reg: 0.045  loss_densepose_U: 4.726  loss_densepose_V: 4.643  loss_densepose_I: 0.967  loss_densepose_S: 4.773  loss_rpn_cls: 0.301  loss_rpn_loc: 0.159  time: 0.3544  data_time: 0.7918  lr: 0.000386  max_mem: 3922M
This indicates that pretrained model weights are not getting loaded properly.
Do you have any suggestions to deal with this error?
Thanks

Hi, @abhaydoke09,
I think this problem occurs because of lack of weights in our repo (whole model, backbones' weights are included in dependencies). One need to train the model from scratch


a little issue

i found time is different in  before16:00 and after , i use moment  when form submit . (fireDate is a field)
code is likelly:
data.fireDate = moment(
form
.getFieldValue('fireDate')
.subtract(-8, 'hours')
.format('YYYY-MM-DD HH:mm:ss')
);
before 16:00  ,i found in network fireDate is right;
but after 16:00 ,  fireDate is next day.
This question is only happend to in  form submit.
My code is wrong ?
thanks all .

Why are you subtracting 8 hours?
You can look at the difference between local mode and UTC mode: https://momentjs.com/docs/#/parsing/


docs: write usage section

Add to invenio_records_rest/__init__.py docstring.

[x] Explanation of what the module is for
[x] Initialization steps

[x] Create Flask app
[x] Dependencies
[x] Minimal required configuration
[ ] Blueprint registration
[x] Add sample data
[x] Index initialization

[ ] Setup permissions
[x] Create/modify/delete a record
[ ] Search a record with filters/sort/facets/mappings
[ ] Using different serializers

Search REST API for Invenio.

Invenio-Records-REST is a core component of Invenio which provides configurable
REST APIs for searching, retrieving, creating, modifying and deleting records.

The module uses Elasticsearch as the backend search engine and builds on top
a REST API that supports features such as:

- Search with sorting, filtering, aggregations and pagination which allows the
  full capabilities of Elasticsearch to be used such as e.g. geo-based quering.
- Record serialization and transformation (e.g. JSON, XML, DataCite, DublinCore,
  JSON-LD, MARCXML, Citation Style Language).
- Pluggable query parser.
- Tombstones and record redirection.
- Customizable access control.
- Configurable record namespaces for exposing different classes of records (e.g.
  authors, publications, grants, ...).
- CRUD operations for records with support for persistent identifier minting.
- Super-fast completion suggesters for implementing Google like instant
  autocomplete suggestions.

The REST API follows best practices and supports e.g.:

- Content negotiation and links headers.
- Cache control via ETags and last modified headers.
- Optimistic concurrency control via ETags.
- Rate-limiting, Cross-Origin Resource Sharing, and various security headers.

The Search REST API works as **the** central entry point in Invenio for
accessing records. The REST API in combination which e.g. Invenio-Search-UI/JS
allows to easily display records anywhere in Invenio and still only maintain one
single search endpoint.

Basics
------

Records
~~~~~~~
- JSON
- JSONSchema

Elasticsearch
~~~~~~~~~~~~~
- Index
- Mapping

- evolution: Alias

Record namespace

Searching is highly impacted by the mapping used.
(doesn't talk about mapping!)

Initialization
--------------

Configuration
-------------
Record namespace
- persistent identifier
- es index alias
- minters
- route specification: pid_value

Exposing multiple namespaces
>>> app.config['RECORDS_REST_ENDPOINTS']['recid']

Installing endpoints
--------------------

Demo data
~~~~~~~~~

Searching
---------
Basic search - query parser, filters, aggregations, sorting, pagination

>> /records/?q=...&size=10&page=1&sort=bestmatch&type=test

Facets: filtering + aggregations
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- facets = filters + aggregations

RECORDS_REST_FACETS

>>> /records/?type=test&type=anoterhvalye
{'aggs':}

Aggregations
++++++++++++
https://www.elastic.co/guide/en/elasticsearch/reference/5.6/search-aggregations.html
bucketing, metric, matrix, pipline

RECORDS_REST_FACETS[:dfd]

Filters
+++++++

- pre/post filters
- many other t

Sorting
~~~~~~~
RECORDS_REST_SORT_OPTIONS
RECORDS_REST_DEFAULT_SORT
Simple, multi-field

/records/?sort=-mostrecent

Ref: https://www.elastic.co/guide/en/elasticsearch/reference/5.6/search-request-sort.html
- order: asc/desc
- mode: min, max, avg, sum
- nested
- missing values
- geo distance
- script based sorting

Query parser
~~~~~~~~~~~~

/records/?q=....

RECORDS_REST_ENDPOINTS['search_factory']
By default the easy query string query parser is used. Features:
- field names + operators
- exists/missing
- ranges
- wildcards, regular expressions
- fuziness
- proximity search
- boosting

Legacy Invenio query parser

https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-query-string-query.html#query-string-syntax

- Replacing the query parser.

Suggesters
~~~~~~~~~~

/records/_suggest?text=...

- Config
- Make sure data is properly indexed.
suggesters[my_url_param_to_complete]

Advanced customization
~~~~~~~~~~~~~~~~~~~~~~
Replace the search factory to do very advanced querying - i.e. exact control over
what is sent to elasticsearch.

Max results

Error handlers

Record class Fetch record from ES instead of database.

Serialization
-------------
Key feature is transforming records from JSON to other formats. E.g. JSON-JSON,
removing sensitive information, enriching, stable output format, or standardized formats like
DataCite, DublinCore,

Content negotiation
~~~~~~~~~~~~~~~~~~~
- Mimetype -> Serializer
- Versioning via mimetypes

Workflow
~~~~~~~~
Preprocess into same format (database vs ES)
Transform record
Serialize data format

Data formats
~~~~~~~~~~~~~~~~~~~~
- JSON: JSON-LD, CSL
- XML: DataCite, DublinCore, MARCXML
- Text: Citation formatting

Transformations
~~~~~~~~~~~~~~~
- JSON-JSON: Marshmallow

Citation formatting
~~~~~~~~~~~~~~~~~~~
Example:
transform to

Tombstones and redirection
--------------------------
error handlers

Access control
--------------
Principles
- search filtering
- factories

impact of es query parser and hidden fields

Factories
~~~~~~~~~

Deserialization
---------------
Create, Update, Delete support
- Loaders
- Minters/Fetchers


mapmatching: findCandidateSnapsInBBox may fail to find relevant tower node snaps.

Describe the bug
In a situation where a GPS trace "overshoots" an intersection as in this example , the relevant tower node of the intersection is not returned by findCandidateSnapsInBBox but only snaps on the irrelevant roads of this intersection.
This leads to detours.
To Reproduce
So far i have been unable to reproduce this error in a smaller datasets as it could even differ over multiple OSM imports. I now suspect that a whole lot depends on the import order.
Potential fix
I fixed this problem in my own modded mapmatching code, by removing seenNodes.add(node)  from line 241 and instead move this to line 252 to only mark nodes as seen that were included in a Snap.
https://github.com/graphhopper/graphhopper/blob/7cbb2484019f76967255aaa15509956266bc3f81/map-matching/src/main/java/com/graphhopper/matching/MapMatching.java#L241

Whoa! This was very helpful, thanks.


new Withings token refresh endpoint breaks OAuthSwift

Description:
Withings recently sent out this announcement to their api users:

Important: Deprecating access and refresh tokens endpoints
As part of our continual improvement, we are migrating the OAuth 2.0 endpoints. The deprecated services will continue to operate normally until September 2021. Please review the article below to get more information on the breaking change.

with more detail here: https://support.withings.com/hc/en-us/articles/360016745358-Deprecating-access-and-refresh-tokens-endpoints
and here: http://developer.withings.com/api-reference#operation/oauth2-refreshaccesstoken
Unfortunately, the new endpoint:

requires a parameter action, with value "requesttoken"
requires parameter grant_type to equal "refresh_token" (not "authorization_code");
and has the response structured differently: the keys of the response are now just body and status, with the former keys, including code, all in an object down below body.

This breaks OAuthSwift. The parameters it sends when getting or renewing an access token are wrong, and so Withings responds with an error. Looking at the source, it's also clear that the new response format would not be correctly parsed.
Handling of the above is all hard-coded in OAuthSwift, with no obvious way to override with e.g. a subclass: while certain functions are open, the functions they themselves call are private, so I can't intercept either the parameter construction or the response parsing. I reviewed #628, and it does not address the problem: non-public methods of OAuth2Swift make replacing the relevant methods not possible.
What should be done here? Is this properly dealt with at the OAuthSwift end or the Withings end? Could there be a strategy for handling this bizarre token-refresh endpoint that I'm not seeing?
I will send Withings a note pointing at this ticket to see if they have anything to contribute.
OAuth Provider:
Withings
OAuth Version:

[ ] Version 1
[X] Version 2

OS (Please fill the version) :

[x] iOS : 14.6
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[ ] CocoaPods
[X] Swift Package Manager
[ ] Manually

Library version:

[X] head
[ ] v2.1.0
[ ] v2.0.0
[ ] v1.4.1
[ ] other: (Please fill in the version you are using.)

Xcode version:

[X] 11.4 (Swift 5.2)

[ ] 11.x (Swift 5.1)

[ ] 10.x (Swift 5.0)

[ ] other: (Please fill in the version you are using.)

[ ] objective c

We are open to code modification if they not break other services
It could be some way to override and some variable to configure for this specific service ( a bool for instance, there is already some, yes because no body was to use same spec 😭 )
Making all public/open could make code less maintenable and have a lot of breaking change. So it depends.
So I prefer new parameter with default value, or configuration var when possible


Add kartevonmorgen

Insert URLs to the project here:
https://github.com/kartevonmorgen/kartevonmorgen.ts

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

Great, thank you *@Ly0n