fix several defects found by cppcheck

just 3 commits, I will not add more commits soon

@freerdp-bot test


Self-reference doesn't match document location when validating feed

Hi all, I'm a new user to jekyll-feed, and I suspect the behavior I'm seeing is simply due to a misconfiguration on my part. However, I'm having a hard time finding the misconfiguration. When validating the Atom feed (you can see the validation results at http://feedvalidator.org/check?url=http:%2F%2Fblog.scottlowe.org%2Ffeed.xml) it reports "self-reference doesn't match document location", and I note that it's reporting the document at "http://blog.scottlowe.org//feed.xml". Any suggestions on how I can fix this to report the correct document location? Note that the plugin is generating a valid feed at http://blog.scottlowe.org/feed.xml, just the location is being reported incorrectly.

In your _config.yml, you set baseurl to /.
Don't do that.

TL;DR: Use baseurl when you are building a site that doesn’t sit at the root of the domain.

@pathawks Thanks. I included baseurl because I migrated to GitHub Pages from WordPress and was having difficulties maintaining the 1500+ permalinks from the previous site. I seem to recall at the time that I couldn't make it work properly without setting baseurl, but I'll try it again and see what (if anything) it breaks.

If anything does break, please ping me on talk.jekyllrb.com and we can dig in deeper.
baseurl caused me a fair number of headaches when I migrated my personal blog to Jekyll as well.


Is there a plan to process data in sft format

sft dataset support.

do you mean a finetuning dataset? there is no reason why it shouldn't be supported currently. Can you give an example?

Hi,
I suppose the question implied the need of handling a slightly different data schema, in cases where a dataset does not necessarily have a text field.
Some SFT instruction datasets usually have a couple of fields such as inputs, targets, and does not actually have a single text field.
Currently, the framework supports specifying a different text_key for the readers, in case there is no text field, but maybe it would be useful to have a text_keys field, accepting a list of field names, thus internally handling the concatenation of those fields, in place of a issing unique text, for use cases where this is needed (for example: in order to create an artificial unique id, based on the contents).


feat: enable preferring attributions in attribution view

Summary of changes
In this PR we enable preferring attributions in the attribution view. If an attribution has multiple resources, a warning is shown and the user is asked for confirmation.
This PR is the second PR on a stack (only look at 2nd commit on this one).
Context and reason for change
In #2270 preferring attributions globally was enabled. Consequently, it should be possible to prefer attributions in attribution view as well.
Closes #2335.
How can the changes be tested
added two e2e tests

fixed test names


Presenter: Unable to switch to the next slide. This issue may be caused by caching.

Describe the bug
When using presenter mode, I utilize the right arrow on my keyboard to navigate to the next page. Initially, the URL reads "http://localhost:3030/presenter/1". Once I press the right arrow, the URL changes to "http://localhost:3030/2". However, the current area still displays page 1, while the next area displays page 3. If I continuously press the right arrow, the URL eventually reads "http://localhost:3030/x", with the current area still showing page 1 and the next area displaying page x+1.
On the other hand, using the button in the lower left corner to switch to the next page works without any issues. Additionally, clearing the browser cache helps resolve the problem.
I vaguely recall encountering an error on the page due to a coding mistake. After making the necessary modifications and restarting vite, the problem with the presenter mode occurred. I am uncertain if the two incidents are related.
To Reproduce
This problem occurred randomly and has not recurred since. Sorry :(
Desktop (please complete the following information):

OS: macOS 13.2.1 (22D68)
Browser: 114.0.5735.198
Slidev version: 0.42.5

Same issue here, but using incognito seems to solve the problem

Same issue here, but using incognito seems to solve the problem

Yes, this is the same principle as I mentioned about clearing the browser cache

A guess, this problem may be caused by the hot start of vite

It should be fixed in the latest version


chore(java): mockito 3.5.13

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: fd7e7f79819f05ab0f50e8645da274ff84c2fc4f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@dependabot squash and merge

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 0f0f40ecfd6eaa5fbb2869a2577891ea54b04338
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: e87083665e81cc78f71edbf4ce23e4a47c15ab26
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Are MouseDown and MouseUp events implemented?

I can't get them to work, so I was wondering if they are supposed to work in the first place? If they aren't I'll definitely turn this into a feature request, since those 2 simple events expand the customizability of the library by a lot.
I was trying this on a FormsPlot.
EDIT: I know that MouseDownOnPlottable exists, but that's not what I need.

Hey @LogDogg I may have fixed this with #390 but it's also worth noting that the FormsPlot has some custom mouse events under the "Misc" category. Let me know if this doesn't solve your issue! 👍


Scriptaculous redirect infinite on jquery load page

Hi when i load a page with jquery containing scriptaculous, after being loaded the page goes into and infinite redirection to itself. Do you have a fix for that ?

Do you have sample code that recreates the same behavior?
Do you have the latest version of PrototypeJS loaded?

What order are you loading the script files? We do this without any problems, but we always load jQuery first as this page says that jQuery will work if we do that: http://learn.jquery.com/using-jquery-core/avoid-conflicts-other-libraries/
However, that method does require us to access jQuery using jQuery() instead of $(), which is not a problem for us (given that we only use jQuery for a few functions).

So, we load them like this:

@everybody
Yes i have latest version of prototype js and my pages works well with jquery noconflict and i have loaded each librairy working well but when i go to another page and try to use the jquery function that load pages .load() to load and entire page into another then i get the infinite redirection

Ok, but that means you are loading prototype first, correct?

Can you put together a sample file that illustrates the problem?
That way we can replicate and tinker with it to try and find the problem and solve it

Please visit http://usefaith.voiedusucces.net/ and log in using username justfortest

Please connect to http://usefaith.voiedusucces.net/standard_a.php using
username  justfortest and password justfortest
and after that you are logged in click on charger_tout  and it will load normally a page that has scriptaculous and after that it will be going through some infinite load. This is the image of where to click to test after that you have logged in.

OK I think I know why you are having a problem - the main scriptaculous.js file is mainly a loader file that creates script tags in the head for the supporting libraries (effects.js, dragdrop.js, etc)
it looks like when a user clicks on that link you are loading a whole other page in via ajax, and when the second scriptaculous.js file loads it triggers the loading of the supporting libraries.
Here is a solution that I use on most of my sites. Combine all the needed Scriptaculous files into one file and then load that file as scriptaculous.js
Also because Prototype lives in the global scope, loading a second copy on the page via ajax might cause problems or errors

Ok Thanks


initial pass at resolving ruff issues

Tons of changes, many of them small (like resolving unused imports or variables) but great code cleanup.
This effort is motivated by the following PR to add ruff support in CI/CD: https://github.com/splunk/contentctl/pull/254
However, because there are so many changes, maybe they should be made after the PR backlog has been cleared out?
This does not begin to address formatting issues, just things like bad imports, unused variables, etc.

I would agree- clearing the PR backlog first will make this easier so we won't have to spend even more time potentially clearing conflicts in every open PR.
I am open to doing this in two stages where we enforce the linter first and then fast-follow w/ the formatter if that means getting the first one of these changes (this one) in faster.

I agree.
The takeaway is that this PR only took maybe 30 minutes to throw together, so the initial pass of changes will be pretty easier!
Easier than keeping this PR up to date as we merge changes. I am closing this out - we will reopen a PR later with all the changes rather than continuously update this one.


mac chrome 126 report error

Make sure nm_nodejs.json is pointing to the absolute path of nm_nodejs.js and nm_nodejs.js is executable.

@warmhug I updated the main branch to use code that works on Mac with node version 21+, see https://github.com/guest271314/native-messaging-nodejs/issues/2.


bugfix: bad dynamicconfig filter/string mapping

This logic desperately needs to be refactored, it's incredibly error-prone :
We should probably just use enumer's codegen tbh.  Or something similar.

Pull Request Test Coverage Report for Build 019070d4-f837-4f0d-bc9f-a372952d8109
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
56 unchanged lines in 17 files lost coverage.
Overall coverage decreased (-0.03%) to 71.494%

Files with Coverage Reduction
New Missed Lines
%

service/history/queue/timer_queue_processor_base.go
1
77.66%

common/task/weighted_round_robin_task_scheduler.go
2
88.56%

service/matching/tasklist/db.go
2
73.23%

common/peerprovider/ringpopprovider/config.go
2
81.58%

common/persistence/sql/sqlplugin/mysql/task.go
2
73.68%

tools/cli/admin_db_decode_thrift.go
2
70.51%

service/matching/tasklist/matcher.go
2
90.91%

common/persistence/sql/sqlplugin/mysql/db.go
2
79.49%

service/history/task/transfer_standby_task_executor.go
3
87.45%

service/history/task/task.go
3
84.81%

Totals

Change from base Build 0190705f-c927-4d1b-b54d-772b615f64e2:
-0.03%

Covered Lines:
105277

Relevant Lines:
147252

💛  - Coveralls


GitHub Commit Feed

A "social" feed which shows the latest commits to CompSoc projects on GitHub; may involve using a hook of some kind to inform the webserver of commits as they happen.

Investigated; takes quite a lot of setup and data storage on a per-commit basis. Probably not worth it.


solution for issue #55

Fixes #55 completely.
Changes Made

Changed indentation as per this issue
Changes were made to the dynamo_pandas.py file where incorrect indentation was changed.

NOTE

Please consider this PR as a submission towards Hacktoberfest 2021 and add the hacktoberfest-accepted label to it.
Hacktoberfest 2021 has started as of now globally due to a bug in the timing algorithm so you can merge the PR now and it will still count. Cheers.

Looks good. Thank you @sreyan-ghosh!


[WASM] Delete API for shared data

With the current state of proxy-wasm, shared data has only get_shared_data() and set_shared_data() methods. It'll be really useful to have a delete method so that shared data can be used for things like implementing a cache, having a simple lookup table etc. Is there any specific reason for not to have a delete method for shared data ?
https://github.com/proxy-wasm/spec/tree/master/abi-versions/vNEXT#proxy_get_shared_data
https://github.com/proxy-wasm/spec/tree/master/abi-versions/vNEXT#proxy_set_shared_data

@PiotrSikora @mathetake

@NomadXD it's going to be included in the next iteration of the ABI:
https://github.com/PiotrSikora/spec/tree/review/abi-versions/vNEXT#proxy_remove_shared_kvstore_key
For the time being, you can set an empty value. This isn't perfect, since it leaks key memory, but it might work for some use cases with limited number of keys.

Sorry to bother you, is there any progress for this api change ?

@moonheart No. Proxy-wasm is not actively developed now I guess.


Decentralized storage - xhr error

Linux Mint 18.3
Firefox
jsipfs daemon running
with http://localhost:8000/
tab indicator is green
Error msg is 'failed to fetch' in Chromium
How can I get past this error and get 'Save text to storage' working?

This was solved by issue 655


Use value-equal instead of deepEqual

Fixes #378

By the way, why leave node_modules out of .gitignore in your projects? It adds an annoying step for me to contribute.

@mjackson here you go. Aside from maybe size is there an advantage to valueEqual over lodash.isequal?

Thanks, @aaronjensen! Looks good.

By the way, why leave node_modules out of .gitignore in your projects? It adds an annoying step for me to contribute.

I configured my global .gitignore a long time ago and never looked back. This way I find I only have stuff in .gitignore that is relevant to that project and not to every project I ever work on.

Aside from maybe size is there an advantage to valueEqual over lodash.isequal?

Size and simplicity are the main benefits. I don't need all of the guarantees lodash.isequal makes. Just a simple "are these two values equal?" is good enough in most cases.

I configured my global .gitignore a long time ago and never looked back. This way I find I only have stuff in .gitignore that is relevant to that project and not to every project I ever work on.

Gotcha, I figured that may be what you did. I still adhere to the philosophy that a project's .gitignore are for files that it generates naturally and my global is for stuff my OS or other personal apps generate. I can see the argument either way.

@mjackson could you release a new history version when you get a chance please so we can start using state? Thanks!

Done! Released in version 4.4.0


disable tcell-mouse until tui-go supports mouse events

Not sure if you're willing to merge this, since you're not working much on tui-go anymore. However.. since it's doubtful mouse events will be passed through anytime soon (maybe I will work on a PR someday to add this), this will disable it so users can at least allow selectable text, as currently this isn't possible.

Thanks for this! I'll still review pull requests and help out in any way I can so feel free to keep them coming! :)


Could not locate the bindings file

→ /Users/maris/personal/crul-converter/node_modules/deasync/build/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/build/Debug/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/build/Release/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/out/Debug/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/Debug/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/out/Release/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/Release/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/build/default/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/compiled/16.19.1/darwin/arm64/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/addon-build/release/install-root/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/addon-build/debug/install-root/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/addon-build/default/install-root/deasync.node
→ /Users/maris/personal/crul-converter/node_modules/deasync/lib/binding/node-v93-darwin-arm64/deasync.node
at bindings (/Users/maris/personal/crul-converter/node_modules/bindings/bindings.js:126:9)
at Object. (/Users/maris/personal/crul-converter/node_modules/deasync/index.js:30:31)
at Module._compile (/Users/maris/personal/crul-converter/node_modules/v8-compile-cache/v8-compile-cache.js:192:30)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1245:10)
at Module.load (node:internal/modules/cjs/loader:1069:32)
at Function.Module._load (node:internal/modules/cjs/loader:904:12)
at Module.require (node:internal/modules/cjs/loader:1093:19)
at require (/Users/maris/personal/crul-converter/node_modules/v8-compile-cache/v8-compile-cache.js:159:20)
at Object. (/Users/maris/personal/crul-converter/node_modules/parcel-bundler/src/utils/syncPromise.js:1:79)
at Module._compile (/Users/maris/personal/crul-converter/node_modules/v8-compile-cache/v8-compile-cache.js:192:30) {
Node : 14

I have no clue what you're saying here. I don't know what version of curlconverter you're trying to use, how you're building it or how to reproduce this error so I'm going to close this. If you want to provide step-by-step instructions for me to also encounter this error I can try to help at least understand what's going on.


Issues in px-backup resources

kubectl get secrets
NAME                                                  TYPE                             DATA   AGE
all-icr-io                                            kubernetes.io/dockerconfigjson   1      69m
amb-px-backup-1a3eab5711ccbbb89961bba6a37cd948-0000   kubernetes.io/tls                2      62m
backup-d50cfd1-1638f9f-default                        Opaque                           8      10m
location2-bc220da                                     Opaque                           9      3m41s
ambikanair@Ambikas-MBP demo % kubectl get secret -o yaml location2-bc220da

kubectl get secret -o yaml location2-bc220da
apiVersion: v1
data:
  accessKeyID: xxx=
  disableSSL: ZmFsc2U=
  encryptionKey: ""
  endpoint: czMudXMuY2xvdWQtb2JqZWN0LXN0b3JhZ2UuYXBwZG9tYWluLmNsb3Vk
  password: azBwMUAjR0I=
  path: dGVzdGJhY2s=
  region: dXMtZ2Vv
  secretAccessKey: xxx
  type: czM=
kind: Secret
metadata:
  annotations:
    stork.libopenstorage.org/skip-resource: "true"
  creationTimestamp: "2023-08-16T10:03:02Z"
  name: location2-bc220da
  namespace: default
  resourceVersion: "7542"
  uid: 3d507724-771d-40f6-a8cf-eed39d32f5fe
type: Opaque

 kubectl get secrets
NAME                                                  TYPE                             DATA   AGE
all-icr-io                                            kubernetes.io/dockerconfigjson   1      82m
amb-px-backup-1a3eab5711ccbbb89961bba6a37cd948-0000   kubernetes.io/tls                2      75m
backup-d50cfd1-1638f9f-default                        Opaque                           8      23m

This is internally known and discussed before as well. As of now, there was no pressing need for using vault to store the secrets and hence was not taken up. Please provide more justification if there is requirement from customer and Px can consider it based on PM priorities.

@trenukarya-px , This is regarding making our platform secure for user this is very much ask by every financial customer, this may be not reported by user because px-backup is not used much but going forward it will be and that time it will be very critical requirement, can you please plan accordingly

Case ID: CS0394019


vue 部分没有提示，保存文件反应慢

Issue Type: Feature Request
vue 部分没有提示，存文件反应慢
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:35.169Z)
OS version: Windows_NT x64 10.0.18362

Translation: Vue does not have code suggestion/auto-completion，and saving a file is slow.

Auto completion for vue is provided by this extension https://marketplace.visualstudio.com/items?itemName=octref.vetur. If there are issues with this, then please report the issue against this extension.


Build MixedRealityToolkit-Unity-releases v1.2017.2.0 failed in Visual Studio 2017 Pro

Overview
I installed Unity 2017.2.0p1-MRTp4, downloaded MixedRealityToolkit-Unity v1.2017.2.0. I open HoloToolkit-Examples/Sharing/Scenes/SharingTest in Editor and build it in Build Settings with SDK=10.0.16299.0. Then I open the sln in Visual Studio 2017 Pro, set it as Release-x86-Remote Machine(give IP of my HoloLens device). When I click on Build, it failed because of an error saying that one of dll not found. I checked why the dll is not found and see that its path string is incorrect because of one letter of one digit-number missing with no any reason. For example of SharingTest, the error message is
System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Public\Documents\UnityProjects2017\HoloLens*MRT-Unity-Releases-2017125*\App-SharingTest\MRT-Unity-Releases-20171205\UnityEngine.Networking.dll'.
In the path string, my project name is MRT-Unity-Releases-20171205, but in error message it shows "MRT-Unity-Releases-2017125", missing "0".
I tested Examples/InputScenes/KeyboardTest too. The error is also missing a letter as this:
System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Public\Documents*UntyProjects2017*\HoloLens\MRT-Unity-Releases-20171205\App-KeyboardTest\MRT-Unity-Releases-20171205\UnityEngine.Networking.dll'.
In this test, "UnityProjects2017" becomes "UntyProjects2017", missing letter "i'.
Expected Behavior
Should pass the build.
Actual Behavior
Build failed
Steps to reproduce
download this MixedRealityToolkit-Unity: https://github.com/Microsoft/MixedRealityToolkit-Unity/releases

As I said in Overview, simply load Examples/Sharing/Scenes/SharingTest into Unity 2017.2.0p1 Editor
build it with SDK = 10.0.16299.0.
Load the solution into Visual Studio 2017 pro
Set to: Release, x86, Remote Machine (type the HoloLens IP address)
Click on Build
you can the build error in Output window

Unity Editor Version
Unity 2017.2.0p1
Mixed Reality Toolkit Release Version
v1.2017.2.0

Seems rather arbitrary. What's the line, and in which class is the directory not found exception being thrown?

I can't reproduce this with the steps provided.
Is there a step you may have missed when building in the Unity editor?

I did one more time. the project is v1.2017.2.0: this folder "MixedRealityToolkit-Unity-1.2017.2.0"
Build Settings in Unity is

The solution is in this folder: C:\Users\Public\Documents\UnityProjects2017\HoloLens\MixedRealityToolkit-Unity-1.2017.2.0\App-SharingTest
Open MixedRealityToolkit-Unity.sln in Visual Studio 2017 Pro
Settings in Visual Studio 2017 Pro is (Release x86 Remote machine)

Click on Build, then I see Build error:
2>  System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Public\Documents\UnityProjects2017*HolLens*\MixedRealityToolkit-Unity-1.2017.2.0\App-SharingTest\MixedRealityToolkit-Unity\UnityEngine.HoloLens.dll'.
This time, HoloLens becomes "HolLens", missing "o".

I think it happens when running AssemblyConverter.

Attach lines from Output and Error windows.
MRT-VS2017Pro-Failed-Error-Messages.txt

the right project folder is
C:\Users\Public\Documents\UnityProjects2017\HoloLens\MixedRealityToolkit-Unity-1.2017.2.0
But when Build in Visual Studio Pro, it becomes
C:\Users\Public\Documents\UnityProjects2017\HolLens\MixedRealityToolkit-Unity-1.2017.2.0
This happens randomly in a line.

Try using the build and deploy window.

Also, try using the default build directory name 'uwp' when making a build from unity.

where is the build and deploy window?
BTW, I build solution into uwp, then Build in visual studio 2017 pro is ok. And then deploy is also ok.
Why have to use uwp folder?

I also tried to build solution into a folder named "App", it has no problem to build in Visual Studio. But I try to build the solution into a folder named "App-SharingTest2", when I build in Visual Studio pro, the build failed again. This time the error message said
Running AssemblyConverter...
2>  System.IO.PathTooLongException: The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.

So the solution for this problem is: DON'T make the path or file name too long. Have to follow what is said in the error message:
The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.

I close this one myself. But I still think it is a problem that there is a character or digit missing in the file path string.

We should report this issue to Microsoft. ;)

I really forgot what is this issue? I reported it?
Best,
Zhenping Guo
On Thursday, March 29, 2018 03:56:01 AM CST, Josh Wittner<EMAIL_ADDRESS>wrote:

We should report this issue to Microsoft. ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

I found the cause of this issue: The total lenght of the argument to the assemblyconverter is above 8192 characters.
From the example in comment above: https://github.com/Microsoft/MixedRealityToolkit-Unity/issues/1472#issuecomment-349211784
The character at position 8192 is removed from the argument.

This major increase in assemblyconverter argument size happens after the Unity 2017.2 update: "Scripting: UnityEngine.dll is now split into separate dlls for each UnityEngine module". All unity engine components now repeats the entire path for each UnityEngine module.
It is possible to work around this limitation by limiting path length because this is repeated for every DLL, but for every non-unity DLL added to the project this argument list will increase in size by "path+name.dll" characters.

Experienced same issue recently. Shortening the path resolved the issue.


Adding cache to server table doesn't add corresponding entry to servercheck table

Loading 'Server Checks' UI results in a datatables error.

Tested in 1.1.3. Problem still present. While we were provisioning more Mid Caches, the Health->Server Checks failed to display. After a Ping check was working, it finally rendered. I guess just add a dummy Mid-Cache to reproduce.

Huh! I'll take a look.

@dangogh put in a(nother) fix in 1.1.5. @smalenfant please retest when 1.1.5 is released.

Reopen if still an issue after 1.1.5 is released.


panic: runtime error: index out of range , when there are ~ 9k elements in array.

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.7.3
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/usr/local/opt/go/libexec/bin:/usr/local/go"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.7.3/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.7.3/libexec/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/d6/lnrxqp194zn2c1g3_02j1pdh0000gn/T/go-build196018249=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
source code --
http://tinyurl.com/hnzm7nd
What did you expect to see?
https://www.hackerrank.com/challenges/ctci-array-left-rotation
All test case passed.
What did you see instead?
failed in test case 6, 8, 9
[Error Messages]
panic: runtime error: index out of range

goroutine 1 [running]:
panic(0x9a840, 0xc42000a110)
	/usr/local/Cellar/go/1.7.3/libexec/src/runtime/panic.go:500 +0x1a1
main.main()
	/Users/raliclo/work/@Netbeans/HackerRank_Practices/TestNG-1/src/main/golang/CrackingTheCodingInterview/Arrays_LeftRotation.go:25 +0x5ff
exit status 2

@ralic the error you are getting seems to come from your code, according to the stack trace ie
on line 25, which is from https://github.com/ralic/HackerRank_Practices/blob/master/TestNG-1/src/main/golang/CrackingTheCodingInterview/Arrays_LeftRotation.go#L25
line2 := strings.Split(lines[1], " ")

could you confirm that you actually get more that one line as a sanity check in your code?

I have the same issue with @ralic
@odeke-em With test cases, the second line always exists .

P/S sorry, that was my mistake. I found the issue :)

@namKolo, so you found the issue in your code? Does this issue still stand then?
@ralic and @namKolo could y'all perhaps reproduce this by making a repository with the inputs that caused the failure? Otherwise we won't be able to diagnose what's going on easily.
Thanks.

@Emmanuel,
Input :
https://github.com/ralic/HackerRank_Practices/blob/master/TestNG-1/src/main/golang/CrackingTheCodingInterview/Input_for_Array_Left_Rotation.txt
Output :
https://github.com/ralic/HackerRank_Practices/blob/master/TestNG-1/src/main/golang/CrackingTheCodingInterview/Output_for_arrays_left_rotation.txt
Failed Code for this large array :
https://github.com/ralic/HackerRank_Practices/blob/master/TestNG-1/src/main/golang/CrackingTheCodingInterview/Arrays_LeftRotation.go
Working Code for the same input :
https://github.com/ralic/HackerRank_Practices/blob/master/TestNG-1/src/main/golang/CrackingTheCodingInterview/Arrays_LeftRotation_2.go
------------------------------------------------------------------------
---
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>------------------------------------------------------------------------
---
--
On Tue, Nov 15, 2016 at 8:00 PM, Emmanuel T Odeke<EMAIL_ADDRESS>wrote:

P/S sorry, that was my mistake. I found the issue :)
@namKolo https://github.com/namKolo, so you found the issue in your
code? Does this issue still stand then?
@ralic https://github.com/ralic and @namKolo
https://github.com/namKolo could y'all perhaps reproduce this by making
a repository with the inputs that caused the failure? Otherwise we won't be
able to diagnose what's going on easily.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/17910#issuecomment-260847848, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGbukq4AUZEJPRYbQ06ehJXGQfwi7k1_ks5q-n_EgaJpZM4KxC7X
.

@ralic thank you for the repro.
Okay so the problem is that bufio.Scanner returns a bufio.ErrTokenTooLong error after that scan loop, yet that error wasn't being checked for early enough (it was checked way too late down the program) hence the runtime panic because you only got one line instead of two. The diff below
catches the error
--- orig.go	2016-11-16 20:08:58.000000000 -0800
+++ fixed.go	2016-11-16 20:23:51.000000000 -0800
@@ -6,6 +6,7 @@
 	"os"
 	"fmt"
 	"strings"
+	"log"
 	"strconv"
 )

@@ -21,6 +22,10 @@
 		lines = append(lines, scanner.Text())
 	}

+	if err := scanner.Err(); err != nil {
+		log.Fatalf("err reading standard input: %v", err)
+	}
+
 	line1 := strings.Split(lines[0], " ")
 	line2 := strings.Split(lines[1], " ")
 	//fmt.Println(line1)
@@ -37,11 +42,6 @@
 		}
 	}

-	if err := scanner.Err(); err != nil {
-		fmt.Fprintln(os.Stderr, "reading standard input:", err)
-	}
-
-
 	// Performance Report
 	//elapsed := time.Since(start)
 	//fmt.Println("Time Elapsed:", elapsed)

and the example on bufio.Scanner also shows how to catch that error https://golang.org/pkg/bufio/#example_Scanner_lines.
To get your program to deal with the multitude of elements that are in the single line, I instead used bufio.NewReader and then Reader.ReadString('\n') like this https://play.golang.org/p/WGfyqmVk79.
I don't see anything more that we can do on the Go side, except encourage checking errors early, otherwise this isn't a bug in the Go stdlib packages.

Hi Emmanuel,
Thank you so much for catching my need, I would also start using
bufferedReader (newReader) in most cases, instead of newScanner for
data array that may have unknown size.
If the case of fixed size array, it seems to be a good workaround using
make() to initiate it and then get each value by fmt.Scan() , which works
very well to replace the scanner with the flexibility to define the data
input type either to be double/string ... for the large array as well.
Sincerely,
Ralic
--
On Wed, Nov 16, 2016 at 8:44 PM, Emmanuel T Odeke<EMAIL_ADDRESS>wrote:

Closed #17910 https://github.com/golang/go/issues/17910.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/17910#event-862233522, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AGbukvookgbC3y7c0q34mm2QJ1LDs7Kdks5q-9ulgaJpZM4KxC7X
.


Do not update timestamp for empty changeset

Fixes #731.

Thanks @antipax!
I have been thinking about this. The issue is that we do not guarantee the timestamp callbacks run last... so another callback can change something which will update the database without modifying the updated_at column (because the timestamp callback was already executed).
So I think a better solution is to not even execute callbacks if there are no changes. This way we avoid all problems upfront. We could have a :force option to give to Repo.update! that would bypass such check. @ericmj thoughts?

Sounds good :+1:

Cool, thanks @josevalim and @ericmj. I'll look into implementing that soon.


Popularity calculation optimizations (Matview refresh)

Start Date
ETA
Project Lead
Actual Ship Date

2023-03-13

@stacimc
TBD

Description

Currently, re-calculating popularity values during data refresh takes a long time. We need to optimize the calculations to make data refresh shorter. ~This is currently scheduled for December, but if some data refresh threshold is crossed, we might have to re-prioritize this.~
Documents

[x] Project Proposal
[x] Implementation Plan

Issues

Milestone: https://github.com/WordPress/openverse/milestone/11

Prior Art

One thought I had while thinking about this for the iNaturalist data refresh complications is that we could potentially split this up from a single matview into a separate table that gets joined on in a non materialized view which is used during the data refresh. Right now, the popularity matview is the one that the data refresh uses to copy data from the catalog to the API. This means that if the matview is not refreshed, data that exists in the underlying table will not make it to the API. This makes the popularity calculation a prerequisite to any new data being added to the API, which seems like it might not be necessary.
Instead, we could have the image_view be a standard view which is a left join between image and image_popularity. The latter would be a materialized view which would contain popularity values, but the difference here is that it could be refreshed independently of the data refresh itself! This would free us up from depending on the popularity calculation for bringing new data into the API.

I thought that part of why we don't, for example, have a separate tags table, is because we don't want to join image to anything. Is the idea that this would be faster because image_popularity is much smaller?

I thought that part of why we don't, for example, have a separate tags table, is because we don't want to join image to anything. Is the idea that this would be faster because image_popularity is much smaller?

That's a really good point I had overlooked, I ran a query and found that the standardized popularity field is way more populated than I had initially expected...Perhaps a join like that will have a higher impact than is desirable 😕
deploy@localhost:openledger> select count(*) from image_view where standardized_popularity is not NULL;
+-----------+
| count     |
|-----------|
| 523503895 |
+-----------+
SELECT 1
Time: 1725.256s (28 minutes 45 seconds), executed in: 1725.251s (28 minutes 45 seconds)

Update 2023-04-11

The project proposal has been posted and is in the Clarification round.
Done

[x] Project proposal drafted

Next

This week I will be making revisions to the project proposal and putting it into the Decision round. I am working on prototyping/testing some ideas which will be detailed in the implementation plan.
Blockers

None.

Update 2023-04-27
The implementation plan is in the Clarification round and has already received some good feedback.
Because the image data refresh was last successfully run 2 months ago, and the most recent attempt at a run timed out after 21 days trying to refresh the materialized view, I have also initiated a new strategy for getting an image data refresh to complete. The recreate_image_popularity_calculation DAG has been triggered, which will drop and fully recreate the matview instead of attempting to refresh it. This plan can be tried in the background while we work on the IP and subsequent tickets.

Update 2023-05-23
All issues are available in the milestone created for this project.
Work is well under way. The bulk of the implementation for this project is in two tickets:

Calculate popularity at ingestion
Create the popularity refresh DAG factory

Most of the remaining issues are for allowing DAGs to run in the background (to backfill data), or small cleanup steps. The first of these issues has been completed, and standardized_popularity is now a column on the media tables which is being calculated at ingestion!
I am working on creating the popularity refresh DAG, and hoping to do so in such a way that we can reuse the logic for other batched updates.
There was a catalog mini-meet-up Monday, May 22nd at which we began investigating some hypotheses for improving catalog performance, including potentially speeding up the batched updates. That investigation is still under way, with notes to come soon.

Update 2023-06-05
Calculate popularity at ingestion was merged, so standardized popularity is now being calculated at ingestion 🎉  What remains is to refresh existing records.

I am working on creating the popularity refresh DAG, and hoping to do so in such a way that we can reuse the logic for other batched updates.

This work is being resumed after a delay to investigate performance issues in the catalog. We will be moving forward with the batched update DAGs as planned and potentially iterating further, pending an analysis of their performance.
A batched_update DAG which can be configured for popularity refreshes as well as arbitrary updates will be the first step, followed by a second PR to generate the popularity DAGs themselves.

Work has been paused on this while some other investigations, including catalog performance testing, continue.
However the batched_update DAG was merged in #2331 🎉
I've added a new issue to the milestone (#2507), after realizing that we'll need to make the batched_update DAG resilient to failures caused by catalog deployments, as it's unrealistic to wait to deploy the catalog until long-running batch updates are finished. Other than that, all that remains is to generate the popularity refresh DAGs themselves.

Update: Before leaving for a week of AFK @stacimc produced two PRs which are ready for review:

#2570
#2592

The batched_update DAG was updated to be resilient to mid-update failures (this allows it to resume safely after deploying the catalog, for instance), and the popularity refresh DAGs were also merged. The popularity refreshes were both triggered. The audio popularity refresh completed in about 5 minutes! The image popularity refresh has completed for all providers except Flickr, which is currently 45% done. Based on its current rate of progress, I hope that it will finish sometime this weekend, and take about 9 days total.
I have a PR as well for some quick fast-follows for issues identified during these initial runs (mostly adjusting poke_intervals and updating logging and message formatting).
New update: In local testing, I realized something that was missed during the implementation planning -- the audioset_view is created from the audio_view, which means that we cannot simply drop the matview without addressing that. Work is currently halted for this week and likely next week due to other team obligations, but I'll need to do a little bit of work to clarify this before we can continue with cleanup steps. I do not anticipate this being a large concern.

All planned code for this project has been merged, including a fix to the audioset_view issue mentioned in the previous update. All that remains for code changes is a final cleanup issue for removing dead code/etc.
The project was delayed by an unanticipated performance degradation in refreshing the popularity constants views, addressed in this quick fix. The PR adds popularity constants to the metrics tables instead of having a separate constants view, and updates the popularity refresh DAGs to update the metrics table directly. This is an alteration from the original plan and needs to be monitored closely.
An audio data refresh, decoupled from popularity, was successfully run from the audio table!
What is left:

Run an image data refresh (potentially blocked by a catalog deploy)
Code cleanup
Merge the constants fix and try running the popularity refreshes again, using the new workflow

I believe this project can be closed!
A late, unplanned addition was made to the popularity constants due to an (unrelated) issue, so we're now running an additional image popularity refresh to ensure this all works.
This project is complete, but will be considered definitely successful once an additional audio and image popularity refresh run (fully testing the constants work and refactors).

Update:
We're currently running an image popularity and data refresh concurrently, to observe whether there is any performance degradation to either batched updates or the copy data step caused by running at the same time.
The copy data step for the image data refresh completed in 6 hours and 40 minutes. That’s very similar to past results (6 hrs 33 mins was the last one). The batch update time for the concurrently running Flickr update also did not seem to suffer; I checked in periodically over the day and they were consistently taking ~15 seconds for 10k records, which is what we’ve seen in previous runs. So it looks like if there is any performance degradation, it’s negligible.


Use lower case license name in image

Fixes #42
Description

Use lower case license name in images
Other information

Checklist:

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[x] I added tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin
Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

@ritesh-pandey, the change looks good. I am merging this.
A side note - Please do not mark all the checkmarks in the PR checklist if they are not applicable, for eg: here the 4(I added tests for the changes I made) and 5(I added or updated documentation) are not applicable.


Add arch instructions to the readme

I added tget to the aur
https://aur.archlinux.org/packages/nodejs-tget/

Thank you!


Clarification to the activity_occurrence column

In the specification you state that:

The column activity_occurrence represents the number of times a given entity has done that activity, at the time that activity occurred (starting with 1). activity_repeated_at is the timestamp of the next time that same activity occurred for that entity. Both should be NULL if another activity has not yet occurred.

I think that there is a discrepancy between above statement and proposed usage of the activity_occurrence column in the examples section:

They can be used to easily get the first time each customer did an activity (activity_occurrence = 1) and the last time (activity_repeated_at = null). Since these two expressions each return only one row per customer, they're also a very efficient way to get every unique customer that has done an activity.

I would appreciate a clarification to this issue. Is it a bug in the spec or something that I do not understand?

I'll further clarify how I interpret the situation. When building an activity schema implementation:

Both fields are NULL when we run transformation SQL queries and insert the results into the activity stream
When we scan the activity stream and fill in the activity_occurrence and activity_repeated_at columns

Increment the value of activity_occurrence. Set to 1 if there is only one activity of this kind for the customer
Set the ts value of next activity of the same kind to activity_repeated_at leaving the last activity with a NULL value


[DOC] update documentation

Describe the bug
Right now the doc is nonexistence.
To Reproduce
All the steps are needed it.
Media proof
No response
Describe the solution you'd like
No response
Additional context
No response
Code of Conduct

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

Done!


Create a uniform representation for function type isolation

Not quite NFC because apparently the representation bleeds into what's accepted in some situations where we're supposed to be warning about conflicts and then making an arbitrary choice.  But what we're doing is nonsense, so we definitely need to break behavior here.
This is setting up for isolated(any) and isolated(caller).  I tried to keep that out of the patch as much as reasonably possible, though.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test


Currently change indicatorClass has no effect.

Problem
Setting custom indicator class via PagingViewController.setter:indicatorClass has no effect. PagingViewController keeps using default PagingIndicatorView.
Source of problem
Custom indicatorClass is getting registered in collection view in PagingCollectionViewLayout.registerDecorationViews, but this function is called only via PagingViewController.configureCollectionViewLayout from PagingViewController.init and from PagingViewController.didSet:menuLayoutClass.  All the other option changes, including indicatorClass, have to effect.
Proposed solution
Currently, temporally, as hotfix hack, one can force registration of indicatorClass by resetting PagingViewController.menuLayoutClass which in didSet causes call of PagingViewController.configureCollectionViewLayout and PagingCollectionViewLayout.registerDecorationViews.
But proper solution should make PagingViewController.setter:indicatorOptions to cause  PagingCollectionViewLayout.registerDecorationViews directly.

Thanks for the detailed report! I'll look into getting that fixed as soon as possible 👍

Oh, borderClass setter may require to fire same update.

Should be fixed on #191 now 🙂 Will release a new version soon!


gollvm: building fails due to missing header llvm/ADT/None.h

What version of Go are you using (go version)?
Not applicable.
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?

Operating system: Ubuntu Linux (kernel <IP_ADDRESS>)
Architecture: ARM64
Compiler: clang 14.0.0
cmake v3.22.1
ninja v1.10.1

What did you do?
I followed the directions from this page on building gollvm: https://go.googlesource.com/gollvm
I was at the step to run this command: ninja gollvm
What did you expect to see?
I expected all the files to be compiled successfully.
What did you see instead?
I saw this error message:
llvm-project/llvm/tools/gollvm/passes/GoNilChecks.cpp:28:10: fatal error: 'llvm/ADT/None.h' file not found
#include "llvm/ADT/None.h"
^~~~~~~~~~~~~~~~~
1 error generated.
Additional info:
None.h is located here on my system: /usr/include/llvm-14/llvm/ADT/None.h

The commit of gollvm I am using is 0f6ef9bc2a14be0a5fdb0d05fc6bb2d5474bf0c6.

CC @ianlancetaylor, @thanm.

I think it's because the latest llvm-project doesn't include None.h.
https://github.com/llvm/llvm-project/commit/397f2e9ebee8d8e45547e90dd05228d7f965df67


jax.lax.pcollective_broadcast

This is only a draft PR, and shouldn't be seriously reviewed until after https://github.com/openxla/xla/pull/8968 is merged into openXLA.
Add a new jax.lax op pcollective_broadcast that calls the new CollectiveBroadcast StableHlo op.
This new operation is intended to be able to lower directly to ncclBroadcast. https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/api/colls.html#ncclbroadcast.
Example usage of this operation:
@jax.jit
@partial(shard_map, mesh=...,
    in_specs=P('i', None), out_specs=P('i', None), check_rep=False)
def f(a):
  return jax.lax.pcollective_broadcast(a, 'i', 3)

x = jnp.arange(64).reshape((8, 8))
print(x)
print(f(x))

This prints
Array([[ 0,  1,  2,  3,  4,  5,  6,  7],
       [ 8,  9, 10, 11, 12, 13, 14, 15],
       [16, 17, 18, 19, 20, 21, 22, 23],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [32, 33, 34, 35, 36, 37, 38, 39],
       [40, 41, 42, 43, 44, 45, 46, 47],
       [48, 49, 50, 51, 52, 53, 54, 55],
       [56, 57, 58, 59, 60, 61, 62, 63]], dtype=int32)
Array([[24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31],
       [24, 25, 26, 27, 28, 29, 30, 31]], dtype=int32)

as the row x[3] == [24, 25, 26, 27, 28, 29, 30, 31] is broadcast to all devices.

@mattjj If you want to start looking at it go for it. All of the meat is there.

I need to add tests which requires backend support, but beyond that this is ready for full review.

I added a test.

Yes, I will add much more test coverage.
Do we wish to also discuss the name pcopy? Now is the time to do so. Pinging @nouiz who had opinions to share.

If you prefer to use the name pbroadcast, I'd be open to renaming our other pbroadcast! Should be easy since I expect there are ~no users of the existing pbroadcast.
Reasons to call the thing in this PR pbroadcast:

it corresponds to the HLO name
it corresponds to the traditional MPI name

Reasons not to:
3. nothing, nada, zilch
WDYT?

Good enough reasoning for me.

@mattjj should be g2g now

I removed the test in shard_map_test.py

I think the next step is for Google to review/approve.
It would be great to have it merged Monday/Tuesday as we will present that at GTC Wednesday.

I'll make sure this gets into at least main by Wednesday!


Dynamic Install Path

With newer versions of the Raspberry Pi OS you can't count on the default pi user being the path for the home folder anymore. This PR attempts to modify the install procedure and the install script to insert the current user name in places where a full path is needed. For the install.sh script this is pretty simple. I just replaced any hardcoded paths from /home/pi to /home/$USER.
The one piece that was a little tricky was the slowmovie.service file. This needed to be updated on the fly rather than a static file with hardcoded paths. I utilized the envsubst shell function to generate the service file from a template and then move that file to the correct location.
Limitations
This is a 1:1 replacement for the current install procedure. It doesn't try and account for more dynamic path options like someone having a completely different path for the SlowMovie install other than as a subdirectory in the user's home folder (/home/$USER/SlowMovie). That could probably be added based on these modifications but that wasn't the intent here.

/home/$USER can be shortened to $HOME or ~

Good thought - I was going for simplest solution (literally did a find replace) but using $HOME makes more sense.


Next release

Hi. What we need for next release or maybe 1.0 version? Maybe some help needed

Hi @Jesterovskiy! Thank you so much for asking to help. I have been sick and busy (not a good combo), but I'm pushing out 1.0rc1 later today! I'll post some issues with a "need help" label if there's anything you or someone else could help with :D


[2022-08-01] Fix Linter Errors

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Hi @solankisamir ,

As there are still go/.NET/python SDK generation pipeline error. Can you fix it? Thanks

Hi @solankisamir ,  As there are still go/.NET/python SDK generation pipeline error. Can you fix it? Thanks

@zzvswxy all sdks are compiling now

We can merge your branch to Azure:release-apimangement-2022-08-01 and see if the errors in origin PR has been fixed.

Thanks @zzvswxy! Lets do that.


Lombok installer does not work with SpringSource Tool Suite 2.1.0.RELEASE

Migrated from Google Code (issue 22)

Rename the file "STS.EXE" to "sts.exe" under ../sts-bundle/sts.app/Contents/MacOS/.
And then run java -jar lomobok.jar and select the STS.ini file under ../sts-bundle/sts.app/Contents/Eclipse
This worked for me.

Rename the file "STS.EXE" to "sts.exe" worked for me. Thanks.


Fix typo: synomymous -> synonymous

Found a small typo while I was looking for the variable to get the source checkout directory.

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

@nightlark Thank you for for fixing this. I am merging this PR and it should be live soon.


Color grid parsing

This is complete for new Clone server schema, will fail if you try to fetch old assignment JSON.

@quaiquai I touched several files during this, and added some driver code under tests to make quick and dirty tests a bit faster to prototype. If it looks good, go ahead and pull, didn't want to create any conflicts on your end, if you still have code yet to be pushed.

I have created a new pull request with these changes and renaming of modules to be snake_case.
https://github.com/krs-world/bridges-python/pull/22


OpenAI null collection fix

This is my first contribution to this project so I know it's not great right now - I'm struggling to get the test environment working without forcing things through in Debug mode. This fix relates to https://github.com/Azure/azure-sdk-for-net/issues/40810 though - I've put some more details in that issue.

Thank you for your contribution, @HiltonGiesenow!  I'm not sure this is the approach that we would want to use for addressing #40810, as all Azure SDK models should have any collection property initialized and a null value should not be possible.  This seems like the OpenAI library may be violating that constraint.
@joseharriaga:  Would you please take a look at the OpenAI side of this and offer thoughts?

Thanks @jsquire for looking into this. Of course, totally happy if you think we should toss the PR, but maybe some background would at least help explain what I was seeing. I have an overload that takes in a List of Functions, but sometimes I don't want to pass Functions, so I have it as an optional parameter, defaulting to null. Unfortunately, the check inside of IsCollectionDefined was returning true, so as a result when the serialization was trying to iterate over the Functions list, it was throwing a null reference exception.
An alternative could be to modify the serialization code to check for null, e.g.:
if (Optional.IsCollectionDefined(Functions) && Functions != null)

but then the same check should be added on lines 72, 87, etc.

Hello, @HiltonGiesenow. Thank you for bringing this to our attention! @jsquire is correct that all collection properties must be initialized, and a null value shouldn't be possible. The root cause of this issue is that the Functions property has a setter. Collection properties must not be settable as per our guidelines. More info here:
🔗 https://learn.microsoft.com/en-us/dotnet/standard/design-guidelines/guidelines-for-collections#collection-properties-and-return-values
For the case when you don't want to pass any functions as part of your request, you don't have to set the Functions property. It is initialized as an empty collection, and you can just leave it like that. 🙂
I have prepared this PR to remove the setter of the Functions property:
🔗 https://github.com/Azure/azure-sdk-for-net/pull/41152

At the end of the day, you guys will implement as you see fit, so this is just my two cents, but I'd really suggest NOT going this route. Removing the setter is the easiest approach but actually the worst from an end developer perspective. Here's my current scenario:
private async Task CallLlm(..., IEnumerable<FunctionDefinition> functions = null)
var chatCompletionsOptions = new ChatCompletionsOptions(openAiDeploymentName, messages)
{
    Functions = functions,
    Temperature = 0.4f,
};
...

Being that the parameter is nullable, I can pass it or not pass it as needed, based on my scenario. However, the code above currently does NOT work (hence this discussion), because the serialization fails, as described. It's failing because, even though the code is currently checking if this collection is initialized (as well as other collections), it's doing so in a way that doesn't cater for this scenario above correctly.
If we ignore my scenario entirely for a moment, and assume I always DO have a value, the change you're proposing (removing the setter), would make even the above simple code more annoying, as follows:
private async Task CallLlm(..., IEnumerable<FunctionDefinition> functions = null)
var chatCompletionsOptions = new ChatCompletionsOptions(openAiDeploymentName, messages)
{
    Temperature = 0.4f,
};
chatCompletionsOptions.Functions.AddRange(functions);
...

I now need, essentially, *3* steps to initialize `ChatCompletionsOptions` (the normal constructor, the additional properties, and extra code). Incidentally, you can see it's already an issue for me because how I'm composing my `messages` collection - I have a function that returns me a 'standard' message history in some cases as an IEnumerable, but I can't pass that in the initializer because `Messages` already has no setter!

In an ideal world, I'd simply be able to write the following:

var chatCompletionsOptions = new ChatCompletionsOptions
{
DeploymentName = openAiDeploymentName,
Messages = messages,
Functions = functions,
Temperature = 0.4f,
};
and the `Write` method that's throwing the exception would be doing better checks.

Incidentally, the 'guidelines' link you pointed to refers exactly in that paragraph to "replacing" the contents of the collection, which could definitely be better with AddRange. In this case we're talking about **Initialization** (i.e. Constructor), which I would suggest is a different scenario. A better answer altogether, for both `Functions` AND `Messages`, might actually be to use [the `init` accessor](https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/object-and-collection-initializers#object-initializers-with-the-init-accessor) - this is exactly what the keyword is designed for. Curiously, that same section of the guideline, in the last line, says "The general rule is that null and empty (0 item) collections or arrays should be treated the same.", and to be frank, this entire discussion is happening because the code is NOT treating null and empty collections the same.


DSEGOG-128 Image centroids/crosshairs

Description
Adds the ability to go into "crosshairs" mode. When switching into this mode, it will display the centroid for the image and the corresponding intensity plots. The user can click on the image to reposition the crosshairs. Resetting the view resets back to the centroid.
It should work when switching via the thumbnail selector, we always default to the centroid though.

Testing instructions
Add a set up instructions describing how the reviewer should test the code

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage
[ ] Play around with the crosshairs mode

Agile board tracking
Closes DSEGOG-128

Should an exported images also include the crosshairs ??

@joshuadkitenge no it shouldn't (I think - they're not exported in eCat) - if they do it's an oversight on my part (although I wouldn't think they would as I would have thought we download the raw image itself, not the canvas overlay where the crosshairs are drawn...)


Reduce code duplication between String.charAt and charCodeAt

JerryScript-DCO-1.0-Signed-off-by: Mátyás Mustoha<EMAIL_ADDRESS>

This reduces the binary size by about 80 bytes.
Also Vera has some troubles detecting the inline ifs, which is why the indentation looks wrong in some places.

Rebased to master and added the empty lines before the two ifs.


VSCODE - Improve how the VSCode writer works and handles crashes

Previously there were  issues with mismatched types that were causing crashes and format exceptions. Now the code is a bit cleaner to follow and understand especially in terms of handling named configs.
Wrapped writeConfig logic in a try/catch block to handle possible comment failures and provide information for any crashes.
If there is a crash, it returns the original file content.

@ibrahim-mubarak Could I get some eyes on this PR? Thanks

I believe this will not preserve the arguments which were not part of dart defines,
such as --profile or --vmservice-out-file=detective_connect.txt
I am unable to test this out myself as I do not have access to my main work PC right now


Request: Smart URL for Device Detection

Hello,
looking for a way to smartshorten the url
which has a Device Detection.
For Examle bla.de/shorturl == redirects to Google Playstore (if android) or to appstore (if IOS).
Would be a killer feature.
Thanks

I'm going to guess this is possible with a plugin that detects the useragent. Probably relatively complex, on account of needing to store multiple URLs, but doable.

You can try to use the UA Parser lib in a plugin.
Then, may be add consecutively the different URLs (eg: sho.rt/blah_ios & sho.rt/blah_android) then use this database records.
Short version:
get sho.rt/blah => UA Parser => redirect (expand sho.rt/blah_[device selection]).
But I'm sure there are prettiest way!


Problems with AWS Keyspaces

Hello,
I have a PHP third-party application, which needs to access a Cassandra keyspace and it does not support TLS connections; I have set up a Cassandra keyspace in AWS Keyspaces, which requires TLS connections: therefore I am trying to use this proxy.
After that I have followed these instructions, I managed successfully to connect to my Cassandra keyspace on AWS Keyspaces locally via cqlsh.
Before configuring your proxy in a Kubernetes container (where the PHP third-party application is running), I am trying to run the cql-proxy image for some local tests:
docker run -p 9042:9042 \
  --rm datastax/cql-proxy:v0.1.3 \
  --contact-points="cassandra.eu-west-1.amazonaws.com" --username="${CASSANDRA_USERNAME}" \
  --password="${CASSANDRA_PASSWORD}" --port="9142" --bind="<IP_ADDRESS>:9042" \
  --proxy-cert-file="/path/to/sf-class2-root.crt" --debug

Unfortunately this simply returns the following error without further explanation:
cql-proxy: error: unable to connect to cluster context deadline exceeded

Am I missing something in the parameters passed for my test? Has somebody else tested this proxy solution  with  AWS Keyspaces already? Thanks in advance.

cql-proxy doesn't currently support  Proxy <--> Cassandra TLS. It only supports Client <--> Proxy and Proxy <--> Astra TLS. PRs welcome.

@vittoriocanilli I'm in a similar situation, did you find a solution ?

Here's a PR that support Proxy <--> Cassandra TLS: https://github.com/datastax/cql-proxy/pull/103. It's not tested yet, but please give it a try!

@kamek-pf sorry for the very late reply; eventually I have managed to achieve my goal by using a container with HAProxy instead of CqlProxy.


Name Thered WebSocket Candlesticks

Hello friends,
When running WebSocket Candlesticks .. The name of thred gets (OkHttp https: //stream.binance.com: 9443 / ...)
Is there any way to change this name when running the CandlesticksCacheExample class?

..


Instagram mobile website as login method

Can we use instagram mobile login web page as a login method ?

I can help you with testing and developing the feature. My fork is also a Carthage compatible. We can talk about it more if you have a discord channel

Sure, contact me by email<EMAIL_ADDRESS>

Hi,
Did you find any solution or do you have any idea for this problem?

@flamme0011  I'm working on it, but it's not stable yet.

@flamme0011  I'm working on it, but it's not stable yet.

Hello, it is great repository. Did you finish this feature ? Are you planning to publish it ?

@freeman4706  Hello,
The feature works fine separately, but adding this feature to library requires a lot of changes.
a few functions and structures needs to be recreated.
If I release this feature it wouldn't be stable and may causes unexpected behaviors.
So fixing this issues takes time.

@TheM4hd1 I would like to maintain if you publish under different branch and beta version

@mzgnr sure, I will create a new branch when I added base structures. for now it's just a separate project. I didn't still merge codes with library.

@mzgnr sure, I will create a new branch when I added base structures. for now it's just a separate project. I didn't still merge codes with library.

Great! A new branch is good idea. When will you publish it ? Thank you.

Thanks to @freeman4706 for implementing this feature.
It's still in beta version and may contains bug. test it and let us know if you found anything.
Usage example:
override func viewDidAppear(_ animated: Bool) {
    super.viewDidAppear(true)

    loginWebView = InstagramLoginWebView(frame: self.view.frame)
    self.view.addSubview(loginWebView!)
    loginWebView?.loginDelegate = self
    loginWebView?.loadInstagramLogin(isNeedPreloadForCookieSync: true)
}

extension ViewController : InstagramLoginWebViewDelegate {
    func userLoggedSuccessfully() {
        print("User Logged Successfully")
    }

    func webViewFinishedToLoadUser(sessionCache: SessionCache, handler: APIHandlerProtocol) {
        print("Loading user with sessioncache is successful")
    }
}

It looks well thanks for your all effort, but after login how can we redirect our page instead of instagram main page?
And how will cache mechanism work? (I mean after first login next time user should not be login again)

webViewFinishedToLoadUser called when a user loggedIn successfully with a cache.
you can remove your InstagramLoginWebView from superView here. Save the cache in your database, every time a user opens the app, you should check for a cache, if cache exists, then you call handler.login(cache: SessionCache)

working great thanks. just also trying to work out how to handle the cache. save to database?

Can you merge it to master please? Because it is not coming with pod update

@podycust SessionCache supports Codable, it means when you encode it, it will give you Data as output, you can save this output to your Local Storage or UserDefaults or any other databases like Realm.
Also it's possible to convert Data to String by using String(data: data, encoding: .utf8) too.
@flamme0011 merged.

Ok thanks will have to google that as codable is new to me lol

@podycust just dont save it to UserDefaults for security reasons.

@mzgnr wasn't planning on doing that! I just save the sessioncache somewhere to speed up future logins when my app starts lol

@TheM4hd1 Thanks for this great library.
I have a question about to store SessionCache in DB.
How can I convert SessionCache to String for storing DB and when I read from db to login system how can I convert it to SessionCache?
Thanks

@arda0011 use JSONDeceoder to convert Data to SessionCache and JSONEncoder to convert SessionCache into Data
to convert Data to String: String(data: data, encoding: .utf8)
to convert string to data: “string value”.data(using: .utf8)

Thanks @TheM4hd1 you are awesome

@TheM4hd1 How long does it take for a session to expire?

@RealOlympusDev 90 days as I know

Ok, thanks!

thanks .
do you help me for save SessionCache to file and use again after run app?

hi mehdi
do you have login in weblogin method with saved cache ?
like this code
loginWebView.loadInstagramLogin(cache: sessionCache)

@freedevcoder
Hi, please use Siwa for login.
this new framework uses web api for login.

ok
but I would login to instagram in weblogin method with cookie that saved before

no there is no method like this and it's not necessary
when you have the saved cache, you have to use handler.login(cache) method.


Add ignore pattern to FixedField

Adding a regular expression to ignore field content.
As an example, I have a field that is supposed to contains a date as "yyyyMMdd", but it occurs to be filled with "0" when no date is specified. With this new attribute, I can define "0{8}" as a pattern to ignore.
PR follows.

Merged, thank you!


fix: (chart) fix nodeDrainPolicy key

Removing a space between the key and colon.

@mergify backport v1.5.x v1.4.x v1.3.x

cc @PhanLe1010


.cmd files do not exist anymore, 'Running Protractor without a plugin' section won't work

README updated


Roll Flutter Engine from 9448f2966c11 to a80741c39acd (1 revision)

https://github.com/flutter/engine/compare/9448f2966c11...a80741c39acd
2023-01-31<EMAIL_ADDRESS>Roll Skia from 3c4cd77c5e5d to fc31f43cc40a (1 revision) (flutter/engine#39278)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Fix broken docker build

I still have issues with https://github.com/cantino/huginn/pull/1204, but wanted to have newer security updates. This is a minimal change that fixes the existing docker build.

@dsander ^
On Sat, Jan 23, 2016 at 3:45 PM Danny Su<EMAIL_ADDRESS>wrote:

I still have issues with #1204
https://github.com/cantino/huginn/pull/1204, but wanted to have newer
security updates. This is a minimal change that fixes the existing docker
build.
You can view, comment on, or merge this pull request online at:
https://github.com/cantino/huginn/pull/1240
Commit Summary

Fix broken docker build

File Changes

M docker/scripts/setup
https://github.com/cantino/huginn/pull/1240/files#diff-0 (4)

Patch Links:

https://github.com/cantino/huginn/pull/1240.patch
https://github.com/cantino/huginn/pull/1240.diff

—
Reply to this email directly or view it on GitHub
https://github.com/cantino/huginn/pull/1240.

Thanks @dannysu!


[BUG] ... Live on Last months income dissapeared

Describe the bug
"Live on Last Month's Income" disappeared from the budget inspector. I have been using this feature for a  while and now it is gone. I have tried turning off everything but this and it's still not there.
To Reproduce
Steps to reproduce the behavior: I'm not sure I can reproduce the behavior  since I only use the feature once a month when planning next months budget. I tried removing every other setting except this one

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. OSX, Windows]
Browser [e.g. chrome, firefox]
Toolkit Version [e.g. 2.22.0]

Setting Export
Please replace the text 'REPLACE_ME_WITH_SETTINGS' below with your exported Toolkit settings. You can export these by going to the Toolkit Options page, click the Import/Export Settings button at the top right and copying the text in the modal which appears.
Note! DO NOT replace the leading and trailing ``` characters as they are required formatting characters.
REPLACE_ME_WITH_SETTINGS

Desktop: OSX 12.6
Browser- Chrome
Toolkit Version 3.3.6

Duplicate of #2956, fixed in #2975. Will go out in next release.


Fix query params

Update only seems to work when linking separately, which works fine, but when I try to link to show courses for a department, this happens:

Turns out I forgot to add brackets around queryParams. Will merge commit to master ASAP because tests are passing


⚠ Punto de Fuga has degraded performance

In 8083216, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 11521 ms

Resolved: Punto de Fuga performance has improved in 97369d7.


A simple tutorial for Carla

I am Frank from China and very interested in Carla Simulator.
Thanks for Carla team hard work.
I want to do some self-driving car simulation in Chinese traffic scenes based on Carla.
From my side, a simple tutorial is appreciated for a new user to know:
1, how to set start points
2, how to set goal points
3, how to use map data
4, a simple straight path planning using existing API
I spent some time on documents and python client code but still have no idea to integrate my modular pipeline self-driving car algorithm into Carla.
These tasks may be very basic but really important for a beginner.
This simple tutorial will benefit for a beginner a lot to know how to use Carla soon.
Please consider this suggestion.
Thanks a lot.

Hey @xfqbuaa.
We are working on improving the documentation.
Thanks for the feedback, we will take this into account
Cheers

@xfqbuaa makes a very good point. When I first built Carla, I edited the original maps and created my own; however, I noticed that during simulation the cars that would spawn would just go around in circles. Are these cars part of Carla's framework or are they part of the UE4? It would be great to figure out how to control their behavior.
Thank you!

Hi @xfqbuaa, the vehicles are part of CARLA. They follow the road map generated by the Carla Map Generator, but you need to trigger the map generation after editing the road by clicking at "Trigger Road Map Generation". The road map created it's serialized together with the road to speed up loading the map. If there is no map, the cars can't find the road and drive in circles. Their behaviour is coded in the AWheeledVehicleAIController C++ class.

@nsubiron Thank you for your response.
Given that the vehicles are part of CARLA and follow the map generated by the Carla Map Generator, if I wanted to completely customize my own roads and wanted to use vehicle models, do you think it would be better to design my own roads using UE4 and then to edit AWheeledVehicleAIController class, or would it be better to edit the Carla Map Generator directly?

@AftermathK If you just want to change the meshes or textures of the road you can reuse our Map Generator. But otherwise this class is not very flexible, it is not going to be easy to add curves, elevation and such.
A workaround for now to use your own roads with the AWheeledVehicleAIController is to set RoutePlanners along your roads (you do so by dragging a spline). When the vehicles enter the trigger box they get a route assigned that they'll try to follow. (RoutePlanner is used for instance for intersections).
For something more complex I'm guessing modifying the AWheeledVehicleAIController is still the easiest.

@nsubiron Very helpful; thank you.
I noticed that on your latest release, it is mentioned that the basic planner is a temporary solution until the map generator is upgraded. What kind of upgrades are you planning on doing to the generator? is this going to be part of the next release or the idea still being developed?

@AftermathK We plan to do major changes to the map generator; curvy roads, elevation, better customization. But this is something to do in the long run as it requires a considerable amount of work.

@AftermathK, @felipecode and @nsubiron
Before formal document tutorial, can you share some hints or idea about:
1, python client to set start point and end point
2, how to transfer map data to waypoints during path planning.
Thanks  a lot.
My goal is to reproduce modular pipeline method in Carla and only 1 scenario with set start point and end point.

Hi,
is there any update on how managing trips (start/goal points) and eventually to provide a playback of a registered trip in CARLA?
Thanks for your support.
Alberto

The driving benchmark module already provides a way to set goal and start points.
Please, check out the new documentation http://carla.readthedocs.io/en/latest/benchmark_start/
We are still improving it. I am sure a way to set start points in command line would be also cool for some quick tests.

Hi Felipe,
thanks for your reply. I have a question (not directly) related to
benchmarking.
My goal is to define some customized hazard-scenarios (i.e. pedestrian
being hit by the player) to use Carla as a test-bench for ROS algorithms.
In this case I need to keep the other variables (like a pedestrian or
non-player vehicle path) to be always the same to accomplish this task. I
have read in your documentation that Carla does not provide trajectory or
carries any geometric info (like for GPS navigators). Does it mean that
they will always follow a random path or can we preset a pose (start-end
tuple) for a non-player agent in the scenario?
Thanks in advance for your availability and support
Regards,
Alberto
2018-04-23 11:50 GMT+02:00 Felipe Codevilla<EMAIL_ADDRESS>
The driving benchmark module already provides a way to set goal and start
points.
Please, check out the new documentation http://carla.readthedocs.io/
en/latest/benchmark_start/
We are still improving it. I am sure a way to set start points in command
line would be also cool for some quick tests.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/carla-simulator/carla/issues/194#issuecomment-383519117,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkFtLncxJudn3ulxkVXt2XOhOHHBTRdcks5traPugaJpZM4R3GCx
.

The driving benchmark module already provides a way to set goal and start points.
Please, check out the new documentation http://carla.readthedocs.io/en/latest/benchmark_start/
We are still improving it. I am sure a way to set start points in command line would be also cool for some quick tests.

I am not able to access this page. Is the page removed?

@xfqbuaa It was moved to its own repo https://github.com/carla-simulator/driving-benchmarks

I'm a bit confused on how to use CARLA for training and validation. The best turoail I found was this one: https://medium.com/asap-report/introduction-to-the-carla-simulator-training-a-neural-network-to-control-a-car-part-1-e1c2c9a056a5
Are there plans to release an 'official' tutorial?

@xfqbuaa  how can I set routes for a vehicle ? I am beginner in Carla.

All the problems mentioned in this thread are solved in the current version of CARLA. Please read the documentation for further information: https://carla.readthedocs.io/en/latest

All the problems mentioned in this thread are solved in the current version of CARLA. Please read the documentation for further information: https://carla.readthedocs.io/en/latest

I still have the issue with local_planner.py  written for the latest version which is the command can't move the ego car. Have any idea why? the controller is working but It can't move the car at all! I really appreciate your help!


Visible Items Should Reflect Actual Cluster State

Describe the bug
Sometime over the last two weeks a Lens update seems to have caused issues presenting the actual current state of the cluster. To view the accurate current state of Pods, for example, I now have to click a different menu item then click Pods again to come back and see what I was looking at before was incorrect. I first noticed this problem using watch and seeing that Lens is consistently out of date when left viewing just one screen.
To Reproduce
Steps to reproduce the behavior:

Launch a Pod with K3d
Give the Pod an Init Container
View Pods in Lens
Restart Init Container from CLI and observe Lens Pod status go stale.

Expected behavior
A more accurate representation of my cluster state, even if this is only a K3d issue.
Screenshots
n/a
Environment (please complete the following information):

Lens Version: 4.0.6
OS: Darwin
Installation method (e.g. snap or AppImage in Linux): AppImage

Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
Your logs go here...

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
your kubeconfig here

Additional context
Add any other context about the problem here.

happens in GKE as well multiple times a day

happens in GKE as well multiple times a day

@balibebas Is this still a problem in 4.1.3? There were some fixes to how Lens pulls data in the 4.1 release.
Note: this still requires watch permissions.

I'm AFK but let's assume it was fixed and let the issue tracker do its thing. Thanks to the authors for this great project.


Fix warning in Xcode 12

…#328

Checklist

[x] I've checked that all new and existing tests pass
[x] I've updated the documentation if necessary
[x] I've added an entry in the CHANGELOG to credit myself

Description
Fix warning in Xcode 12. The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.0.99.
Motivation and Context

To fix warning in Xcode 12

#328

Runned tests in iOS lib and build Examples/SwiftPackageManager.

Fixes #328


Try to recover if we reach the end of the stream when searching for the EI marker of an inline image (issue 8798)

Fixes #8798.

/botio test

/botio-linux preview

/botio makeref

Thank you for fixing this!


My addition to the story

An addition with many words and little content. Do as you wish.

Hi @daing100
You can attempt to complete this task at any time, otherwise I will keep the grade that is on canvas.

Closing this as changes will never be made.


Issue/369/goldenspike pipeline

For #369.

Fixes broken pipeline file; we can now run goldenspike from command line
Consolidates data references to the examples_data directory (while shifting pipeline & configs to goldenspike_examples directory--it's a pipeline example)
Creates documentation in the goldenspike_examples/README.md; explains how users can run this pipeline
Adds base_catalog.pq, the parquet file created in from pretrained_flow.pkl in the early stages of goldenspike.ipynb (I don't love this, but pointing to pretrained_flow created complaints from tables_io that pkls are not among the supported input types)

(Also: fixed some outdated goldenspike.ipynb paths I'd missed in an earlier PR)

I just saw your comment about the base_catalog.pq file versus the pretrained_flow.pkl  file.
These are actually very different things.   The pretrained_flow file is a model that gets made by the FlowModeler stage called something like flow_creator_train I think.
The base_catalog.pq is an input file that gets used to by flow_creator_train to make a new version of the model , i.e., a not_so_pretrained_flow.pkl  file.
Does that help clarify things?

Sort of--but the pre-existing pipeline uses
inputs:
  flow: examples/goldenspike/data/pretrained_flow.pkl

while the pipeline output by running goldenspike.ipynb uses
inputs:
  input: <mypathtorail>/examples/goldenspike_examples/data/base_catalog.pq

What should be used as the input?
If I'm following, I'd guess we would want the model file pretrained_flow.pkl, not the new version of the model that I'm assuming is generated by running goldenspike (is that what's happening here?), but then that brings us to the pkl problem. Replacing this PR's pipeline's input to point to input: input: examples/goldenspike/data/pretrained_flow.pkl results in flow_modeler.out:
Traceback (most recent call last):
  File "/Users/orl/miniconda3/envs/freshrail/lib/python3.10/site-packages/tables_io/types.py", line 206, in fileType
    return FILE_FORMAT_SUFFIXS[fmt]
KeyError: 'pkl'
...
KeyError: 'Unknown file format pkl, supported types are{list(FILE_FORMAT_SUFFIXS.keys())}'
Executing stage: FlowModeler @ 2023-05-09 22:45:47.753948
Inserting handle into data store.  input: examples/goldenspike/data/pretrained_flow.pkl, flow_modeler
Stage failed: FlowModeler @ 2023-05-09 22:45:47.753998 after 0.00 minutes

Ok, so, the inputs block is the global inputs for the pipeline.   Basically this means any inputs to any stages that aren't connected to the outputs of other stages.
The original version of the pipeline started with a FlowCreator, which takes a model file that it refers to as 'flow', so that the inputs block looks like this:
inputs:
  flow: examples/goldenspike/data/pretrained_flow.pkl

The new version of the pipeline starts with a FlowModeler, which takes a data file that it refers to as "input" and creates the model file that it then passes to the FlowCreator, so in that case the global inputs looks like this:
inputs:
  input: <mypathtorail>/examples/goldenspike_examples/data/base_catalog.pq

So, for the new version of the pipeline we will want something like above.


updated to swift 5

Made changes relating to the changes in naming convention in swift 5 UIKit .Now compatible with swift 5

error setting the dispatch time for the button


refactoring, add types

fix 19
Changes:

Add typings
Change names of some names of variables
Add docstrings to some methods
Add remarks/TODO

send #24

Reopen


Add test coverage for the Pulsar IO Batch connectors

Is your enhancement request related to a problem? Please describe.
The Pulsar Batch IO classes do not have any unit tests for them
Describe the solution you'd like
I will be adding some unit tests for these classes
Describe alternatives you've considered
Not implementing unit tests.
Additional context
N/A

Closed as stale and no one worked on it. Please open a new issue if it's still relevant to the maintained versions.


CEL: make validations have to match all elements in a list

I want to create a CRP that allows multiple CR-dnsNames to match multiple RegExes.

All dnsNames have to be validated by (at least) one validation
One successful validation ought to be enough to validate each single dnsName
a DNSname has not to successfully validate against all validations

As I find it difficult to find the precise words to describe what I want, I'll do with an example.
given the following CRP (excerpt):
spec:
  allowed:
    dnsNames:
      required: false
      validations:
        - message: only URIs matching the CR's namespace or *.example.com are allowed in dnsName
          rule: self.matches( '(\\.' + cr.namespace + '\\.svc(\\.cluster\\.local)?) || (.*\\.example.com)' )

☑ When creating a Certificate in namespace 'foo' with single or multiple dnsNames out of bar.foo.svc, bar.foo.svc.cluster.local, or foobar.example.com the Certificate Request is approved, the policy works as expected.
🚫 When creating a Certificate with the following spec:
spec:
  dnsNames:
    - test.foo.svc.cluster.local
    - github.com

the Certificate Request is approved as well - meaning that as long as a single dnsName is validated, the Certificate is created.
This is obviously not what I intend.
Other variants I tried:

having multiple validation rules: doesn't work, as then all of the dnsNames have to match all of the rules (the other extreme - all n:m validations have to succeed)
multiple CRP: not tested, because if a single one validates a CR, others won't be tested

I havent' many more ideas, except maybe using OPA with some allOf/anyOf matching. Surely there must be a way of achieving the same with CEL?
Thanks!

@erikgb OK, that looks really nice - I wasn't aware I could combine rules with || and thought this was only possible inside the self.matches() stanza. Seems to work for me, I'll do some further testing in the coming days (weeks).
My CRP doesn't allow github.com, but I could create a (self-signed) Certificate with it as long as a valid (according to at least one other regex validation) dnsName was included alongside. It was just an example here on github.com why it would be a bad idea to use this construct :)


feat(ext): provide direct access to database tx

Description
Provide a way to access read-only database transaction directly from CLI extensions.
This is useful for overriding or modifying different kinds of provider behavior.

This is useful for overriding or modifying different kinds of provider behavior.

such as?


rs / gemmi installation issue

Hey team,
I recently created a new conda environment and installed careless into it. careless successfully installed the reciprocalspaceship dependency, but did not install gemmi. I eventually solved the issue by explicitly installing reciprocalspaceship==0.9.16, which brought gemmi==0.4.9 along with it.
I am raising this issue here, rather than on reciprocalspaceship directly, because I was unable to reproduce the issue by creating a fresh environment and installing reciprocalspaceship. It seems that however rs (and @JBGreisman) is handling version control and staying up to date with gemmi, careless was unable to interface with it properly?
I think that any examples I try to provide will be inevitably poisoned by what is cached in my local and/or cluster conda installations, so I will not do that and instead just leave it to others to get to the bottom of this.

Can you provide the commands you used to install careless? Does this happen from a fresh miniconda installation?
I was not personally able to reproduce this issue.

Any thoughts about this @JBGreisman or @dennisbrookner? My inclination is to close this issue. I think it is most likely something pathological in @dennisbrookner 's conda install.

(or pathology in the cluster filesystem)

I'm also unable to reproduce any issues with reciprocalspaceship and gemmi when installing careless. I think this is likely a specific environment issue--perhaps there were some packages installed into the base environment or cached that were causing issues?

Yeah this can definitely be closed, I don't think this is a real issue. I'll circle back if I learn anything more concrete.
This probably wasn't helping...
==> WARNING: A newer version of conda exists. <==
  current version: 4.12.0
  latest version: 22.9.0


docs: init project readme

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1 (a597d5d) into main (27ffc47) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a597d5d differs from pull request most recent head 5884a89. Consider uploading reports for the commit 5884a89 to get more accurate results

@@            Coverage Diff            @@
##              main        #1   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            3         3
=========================================
  Hits             3         3

Continue to review full report at Codecov.

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


Cannot create cloudlet in a restricted mode

When you try to create a cloudlet with a restricted access, you get an error: "Cloudlet key {} not found", which seems to come from node access api.

Fixed.


Keep flag in unknown NANP option

Try to solve #108.

Coverage decreased (-19.0%) to 72.648% when pulling 6396a085dafc3588fe862afe0a6cea7542dab7b2 on keep-flag-in-unknown-nanp-option into 33c1f05c08ea70a605ebb31fd111cac9d1c4883b on master.


Turndown travis deployments and enable mtime randomization

This change removes travis deployments of the tc-rule based script-exporter VM. This functionality is now succeeded by the ndt5-client based e2e monitoring in the prometheus-federation GKE cluster.
As well, this change also restores the mtime randomization used to cache success statuses in cache_exit_code.sh used by the ndt5-e2e configuration.

This change is

https://github.com/m-lab/dev-tracker/issues/574


⚠ Samsung has degraded performance

In 75d00d9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2099 ms

Resolved: Samsung performance has improved in 07a943b.