Notebook won't run due to Kernel dropdown having wrong values

Azure Data Studio Version:  1.35.1
OS Version: Windows 10, build 19044.1586

Steps to Reproduce:
(This is how I ran into the issue, but anyone using notebooks might run into this issue as well.)

Download Azure Arc extension
Go to New Deployments in the top right corner of the left panel
Follow the steps to deploy an Azure Arc data controller
Hit "Deploy"
Error occurs when you try to run the cells in the generated python notebook

Does this issue occur when all extensions are disabled?: Yes/No (Bug occurred while using an extension, so not sure if this applies)

Other details:
After reverting to a commit made on March 11, I no longer saw this issue. When I run a python notebook, the Kernel options are only LogAnalytics and Kusto, but I am supposed to be seeing SQL, Python3, etc... And then ADS forces me to make a Connection, while this should not be the case.

I think it may have to do with this commit @corivera https://github.com/microsoft/azuredatastudio/commit/95980130c87aac610eccee6995e73e0514008e6c


Port POST container attach tests

Porting API POST container attach tests to the integration-cli. Original tests can be found in Issue #12905. Fixes #12905.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "12905-port-container-attach-tests" git@github.com:kostickm/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@runcom I don't believe the tests are quite ready the way they stand, though they pass, but was hoping to get some input :)

LGTM


ImportError: No module named 'openvino'

[Required] Your device (RaspberryPi3, LaptopPC, or other device name):  RaspberryPi3
[Required] Your device's CPU architecture (armv7l, x86_64, or other architecture name):  arm
ありがとうございました
I have already configurated environment in RaspberryPi3 with OpenVINO, but when I ran program of YOLO, this error "ImportError: No module named 'openvino" always occurred. Do you know how to solve this error? Thank you

First, check your version of OpenVINO.
Remember to modify the paths listed in your .bashrc, depending on your OpenVINO version.
2019 R1
$ cd ~
$ nano .bashrc
source /opt/intel/openvino/bin/setupvars.sh

$ source .bashrc

2018 R5
$ cd ~
$ nano .bashrc
source /opt/intel/computer_vision_sdk/bin/setupvars.sh

$ source .bashrc

Thank you for your quick reply.
My version of Openvino is 2019 R1, and I have already cheak my bashrc file. Just like following picture.

But it still doesn't work, like this

$ source ~/.bashrc

Still doesn't work.....

Do not use the super user. (#)

Still doesn't work...... I don't knwo why, and I want to cry.......

$ echo $PYTHONPATH

I'm also met the problem. Have you already solve the problem?

I may already find the bugs, on the 2019 R1 version ,the openvino module is included by the armv7l, should import openvino by from armv7l.openvino.inference_engine import IENetwork, IEPlugin

I also solved this problem as you said, think you very much!

Fixed by 677b5a9a7d23b113c57b863f980541fad87bf69b

I may already find the bugs, on the 2019 R1 version ,the openvino module is included by the armv7l, should import openvino by from armv7l.openvino.inference_engine import IENetwork, IEPlugin

Fixed! Thanks

Hi all, I made a Debian package for OpenVINO RPi, and it can be installed by the following steps:
$ curl -sL https://raw.githubusercontent.com/DT42/BerryNet-repo/master/setup.sh | sudo -E bash -
$ sudo apt install openvino-rpi

For more details, please refer to https://github.com/DT42/BerryNet-repo

Jetson TX2 also met the problem, but can't solve by

Jetson TX2 also met the problem, but can't solve by
from armv7l.openvino.inference_engine import IENetwork, IEPlugin
sudo apt install openvino-rpi
both can't work.
I tryed add PYTHONPATH, but still can't work.
Can anyone help me!

I may already find the bugs, on the 2019 R1 version ,the openvino module is included by the armv7l, should import openvino by from armv7l.openvino.inference_engine import IENetwork, IEPlugin

Fixed! Thanks

I may already find the bugs, on the 2019 R1 version ,the openvino module is included by the armv7l, should import openvino by from armv7l.openvino.inference_engine import IENetwork, IEPlugin

Having a trouble in installing armv7l in my Windows. Tried:
pip install armv7l

but failed
Can you please clarify how you did install armv7l?

I also solved this problem as you said, think you very much!

@SpursLipu  Can you plz clarify how did you get armv7l installed? Thank you.


Improve pylint pre-commit hook

Fix pylint issues in a handful of easy-to-fix python files.

@alainjobart: this suppresses some useless warnings.

LGTM


hw/mcu/nrf: Add common startup code

This adds common startup code to nRF MCU package so it can be reused by
BSP. This avoids multiple duplicates of the same file in all BSPs.
Also proper configuration is added so standard and split startup code
is not included in build at the same time. This fixes issue when link
option -whole-archive is used.

fixed build issue


Scaling a scenario results in incorrect rounding of inhabitants per residence

Any chance you have a scenario ID? Or remember the steps you took to get to this result?

Any chance you have a scenario ID? Or remember the steps you took to get to this result?

Unfortunately this is done by external users and I only got this screenshot to work with...

Okay thanks; will investigate soon. The population slider being 0 is strange: even with number of residences set to 1 (the minimum), the slider shouldn't go below 1.14.
I did find a similar bug however:

Set number of old residences to 0
Set number of new residences to 0
Scale to 1 residence (number probably doesn't matter)

The result is that sliders don't load (due to a "non-numeric GQL result" error) and:

After discussing with @grdw, assigning him.


Link "Security roles and licensing white paper" doesnt work

Link "Security roles and licensing white paper" doesnt work on this page. It gives an error that the page is not valid
This site can’t be reached The webpage at http://download.microsoft.com/download/1/A/4/1A4CA2CF-FE7C-4629-921C-B41CD63FC021/Microsoft_Dynamics_AX_2012_R2_Licensing_Guide-Customer_EditionDec2012.pdf might be temporarily down or it may have moved permanently to a new web address.
ERR_INVALID_RESPONSE

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

ID: 96b540a1-d103-2e12-ddfb-be1447bb8032
Version Independent ID: acbc7f97-31d4-5013-174c-1a2fecaa9db2
Content: Security role reference
Content Source: dynamicsax2012-technet/security-role-reference.md
Product: dynamicsax-2012
GitHub Login: @Khairunj

Thanks for letting us know about the broken link. The link has been fixed!


update mockito and minor test updates

I wanted to use updated mockito in some new tests I was writing. But the version we had was too old. Updated the version and a few minor test changes.

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

https://github.com/NagRock/ts-mockito/issues/117 thanks for the request @DonJayamanne  ;)

@karthiknadig @kimadeline I touched a few core team test here, so looping you in as well.

Codecov Report

Merging #10671 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10671      +/-   ##
==========================================
- Coverage   60.64%   60.62%   -0.02%
==========================================
  Files         583      583
  Lines       31551    31551
  Branches     4487     4487
==========================================
- Hits        19133    19127       -6
- Misses      11446    11450       +4
- Partials      972      974       +2

Impacted Files
Coverage Δ

src/datascience-ui/react-common/arePathsSame.ts
75% <0%> (-12.5%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

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 6438bee...13de2af. Read the comment docs.


Fix tileset urls in Picking development Sandcastle

A couple cases missed from https://github.com/AnalyticalGraphicsInc/cesium/pull/6678

@lilleyse, thanks for the pull request! Maintainers, we have a signed CLA from @lilleyse, so you can review this at any time.
:warning: I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

👍


Could you elaborate on how to evaluate topic segmentation task?

Hi.
Thanks for sharing interesting work with the community 👍
I am trying to understand and see how combination of sentence-transformer embedding and C99 works for topic segmentation.
For example, I get 7 topic segments from running Topic_Segment.ipynb
Segment 1. A: Hi Tom, are you busy tomorrow’s afternoon?
Segment 2. B: I’m pretty sure I am. What’s up?, A: Can you go with me to the animal shelter?., B: What do you want to do?
Segment 3. A: I want to get a puppy for my son., 'B: That will make him so happy., A: Yeah, we’ve discussed it many times. I think he’s ready now.,
Segment 4. B: That’s good. Raising a dog is a tough issue. Like having a baby ;-), A: I'll get him one of those little dogs."
Segment 5.
Segment 6. A: Oh, yes, I took him there last Monday. He showed me one that he really liked., B: I bet you had to drag him away., A: He wanted to take it home right away ;-).,
Segment 7. B: I wonder what he'll name it., A: He said he’d name it after his dead hamster – Lemmy  - he's  a great Motorhead fan :-)))

Could you suggest any tip for evaluating the quality segmentation of above example?
As I understand, you decide the hyperparameter for C99 (window size 4 and std coefficient 1) with qualitative evaluation with several human. Personally, I think that "topic segmentation" is a difficult problem in which people think of the correct segmentation boundary differently.  So if you elaborate on how such qualitative evaluation is done, it would be much helpful for the research community :)

We did not do the qualitative evaluation with several human in a large scale. Instead, I manually checked around 25 segmented results to see if I could summarize meaningful topics from segments and there did not exist large overlaps.


Restructure repository

A src folder is generally a good practice for workflow development. There you can put all files that should be part of the workflow bundle. You don't want to have README in your bundle for example.
It also makes things easier for people that install workflows by symlinking the src directory to the workflows directory in Alfred Preferences which is a good setup for development.

Thanks for the helpful feedback, again! Learning lots of new things 😀. I'll take a look at this soon and merge it in!

Thanks for your assistance again! I tested and changes work great here. I'll keep this structure in mind and symlinking to /src for all my future dev. Going to create a new release with the changes.


please upload barcode files

From the paper supplement table, I cannot seperate the forward barcode from the reverse barcode
acacacac agactatg agcgacta acagcaca gcgtcagc tcagtgtc gtgtacat tgacatca actctgct tatgtcag acatgtgt atatagcg tagtcgca gtacgact ctatgcta tactatac atgatcgc tcgcgctg actagatc acgacgag agcacagt gatcgcga catcagtc tagctagt cgctctcg atcagtca agtgctac gtcgtaga tctactga cgtataca gtcacgtc gatgatct cgagtcgt gactgatg ctgcgtac cacatgat
Can you pls upload the related barcode file to this rep? Thanks

I uploaded the barcode files as requested.
In some cases the same adapters were used for both forward and reverse barcodes, so we added the primer sequence to distinguish between forward and reverse.


[Android] Slow scanning of some labels

In the plugin, there is an issue with slow scanning of certain labels for the Android system. I set the 'speed detection' parameter to 'unrestricted.' I also checked directly with Google's ML Kit (https://github.com/flutter-ml/google_ml_kit_flutter/tree/master), and there the label recognition is very fast. I am attaching sample labels.

Any progress with the issue?

Any progress on this issue? Since 2023?


Question: State vs props

Hi,
I am curious to what the distinction is between state and props and when to use them. Am I assuming correctly that props are for external use and state for internal?
I think that the example in Readme needs to be updated as the props from microcomponent constructor returns undefined in the render function (they are oldProps). I might be totally off here, just thinking out loud 🙃

props are what you call your component with:
${component.render(thoseAreProps)}

State is the component's internal state. You could also just attach state yourself, but it's cleaner to use the already provided object.

Thanks for the clearification. As i mentioned, the example in reader needs to be updated, either pass text in the render method or change props to state in the micro component constructor.


IRGen: attribute correct linkage to Windows DSO handle

Partially address the incorrect handling for the #dsohandle on Windows.  Foremost, we rely on the MS extension of __ImageBase which is a linker pseudovariable that is declared as a
extern const char __ImageBase; as per LLVM.  This is also honoured by LLD and thus we can rely on this extension.  The proper IRGen for this would lower to @__ImageBase = external dso_local constant i8. However, we were previously emitting a local definition for this external constant, and worse yet, not marking the definition for COMDAT. It is unclear what definition would win ultimately (implementation defined), as we had a definition as well as the linker synthesized value.  We can change the SIL linkage for this type to DefaultForDeclaration which will give it available_externally and default visibility and storage which is closer to what we desire. However, because we do not track the LLVM variables and apply heuristics for lowering the SILGlobalVariable, we would attribute it with imported DLL Storage.  This would then cause us to fail at link time (amusingly enough link.exe will report a LNK1000).  Special case the variable and track that we are targeting a windows environment in the UniversalLinkageInfo so that we do not special case this on other platforms.
Partially fixes: #64741

@swift-ci please test

(Frontend/dsohandle-linkage.swift covers enough of this for validation)

=== /Users/ec2-user/jenkins/workspace/swift-PR-macos/branch-main/swift/stdlib/public/Cxx/std/String.swift:128 ===
    ┆
126 │   public init(_ cxxString: std.string) {
127 │     let buffer = UnsafeBufferPointer<CChar>(
128 │       start: cxxString.__c_strUnsafe(),
    │                        ╰─ error: value of type 'std.string' (aka 'std.__1.basic_string<Int8, char_traits<Int8>, allocator<Int8>>') has no member '__c_strUnsafe'
129 │       count: cxxString.size())
130 │     self = buffer.withMemoryRebound(to: UInt8.self) {

Seems like a flake

@swift-ci please test macOS platform

@swift-ci please test macOS platform

@swift-ci please test


Named Operators

I'm working in anonymous operator to named operator for performance improvement and readable stacktrace.
for example stack trace of below code are...
Observable.Return(1000)
    .Take(50)
    .Select(x => x)
    .Subscribe(x => Debug.Log(x));

// improvement version

UniRx.SubscribeObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:147)
UniRx.Operators.SelectObserver:OnNext(Int32) (at Assets/UniRx/Scripts/Operators/Select.cs:79)
UniRx.Operators.TakeObserver:OnNext(Int32) (at Assets/UniRx/Scripts/Operators/Take.cs:62)
UniRx.Operators.ReturnObserver:OnNext(Int32) (at Assets/UniRx/Scripts/Operators/Return.cs:48)
UniRx.Operators.Return`1:SubscribeCore(IObserver`1, IDisposable) (at Assets/UniRx/Scripts/Operators/Return.cs:23)
UniRx.Operators.OperatorObservableBase`1:Subscribe(IObserver`1) (at Assets/UniRx/Scripts/Operators/OperatorObservableBase.cs:40)
UniRx.Operators.Take`1:SubscribeCore(IObserver`1, IDisposable) (at Assets/UniRx/Scripts/Operators/Take.cs:38)
UniRx.Operators.OperatorObservableBase`1:Subscribe(IObserver`1) (at Assets/UniRx/Scripts/Operators/OperatorObservableBase.cs:40)
UniRx.Operators.Select`2:SubscribeCore(IObserver`1, IDisposable) (at Assets/UniRx/Scripts/Operators/Select.cs:48)
UniRx.Operators.OperatorObservableBase`1:Subscribe(IObserver`1) (at Assets/UniRx/Scripts/Operators/OperatorObservableBase.cs:40)
UniRx.ObservableExtensions:Subscribe(IObservable`1, Action`1) (at Assets/UniRx/Scripts/Observer.cs:260)
UniRx.ObjectTest.UniRxTestSandbox:OnGUI() (at Assets/ObjectTest/UniRxTestSandbox.cs:354)

// current assetstore version

UniRx.AnonymousObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:55)
UniRx.AnonymousObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:55)
UniRx.<Select>c__AnonStoreyDE`2:<>m__1E0(Int32) (at Assets/UniRx/Scripts/Observable.cs:39)
UniRx.AnonymousObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:55)
UniRx.AnonymousObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:55)
UniRx.<Take>c__AnonStoreyA0`1:<>m__1A0(Int32) (at Assets/UniRx/Scripts/Observable.Paging.cs:28)
UniRx.AnonymousObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:55)
UniRx.AnonymousObserver`1:OnNext(Int32) (at Assets/UniRx/Scripts/Observer.cs:55)
UniRx.<Return>c__AnonStorey71`1:<>m__183() (at Assets/UniRx/Scripts/Observable.Creation.cs:121)
UniRx.ImmediateScheduler:Schedule(Action) (at Assets/UniRx/Scripts/Schedulers/ImmediateScheduler.cs:26)
UniRx.<Return>c__AnonStorey70`1:<>m__D2(IObserver`1) (at Assets/UniRx/Scripts/Observable.Creation.cs:119)
UniRx.AnonymousObservable`1:Subscribe(IObserver`1) (at Assets/UniRx/Scripts/Observable.Creation.cs:46)
UniRx.ObservableExtensions:Subscribe(IObservable`1, Action`1, Action`1, Action) (at Assets/UniRx/Scripts/Observer.cs:173)
UniRx.<Take>c__AnonStorey9F`1:<>m__EB(IObserver`1) (at Assets/UniRx/Scripts/Observable.Paging.cs:23)
UniRx.AnonymousObservable`1:Subscribe(IObserver`1) (at Assets/UniRx/Scripts/Observable.Creation.cs:46)
UniRx.<Select>c__AnonStoreyDD`2:<>m__10A(IObserver`1) (at Assets/UniRx/Scripts/Observable.cs:27)
UniRx.<Subscribe>c__AnonStorey11F:<>m__210() (at Assets/UniRx/Scripts/Observable.Creation.cs:42)
UniRx.InternalUtil.ScheduledItem:Invoke() (at Assets/UniRx/Scripts/InternalUtil/ScheduledItem.cs:44)
UniRx.Trampoline:Run(SchedulerQueue) (at Assets/UniRx/Scripts/Schedulers/CurrentThreadScheduler.cs:122)
UniRx.CurrentThreadScheduler:Schedule(TimeSpan, Action) (at Assets/UniRx/Scripts/Schedulers/CurrentThreadScheduler.cs:91)
UniRx.CurrentThreadScheduler:Schedule(Action) (at Assets/UniRx/Scripts/Schedulers/CurrentThreadScheduler.cs:69)
UniRx.AnonymousObservable`1:Subscribe(IObserver`1) (at Assets/UniRx/Scripts/Observable.Creation.cs:42)
UniRx.ObservableExtensions:Subscribe(IObservable`1, Action`1) (at Assets/UniRx/Scripts/Observer.cs:158)
UniRx.ObjectTest.UniRxTestSandbox:OnGUI() (at Assets/ObjectTest/UniRxTestSandbox.cs:526)

Please wait a moment for easy to debug!
rest of operators list---

[x] Scan
[x] Multicast
[x] Publish
[x] PublishLast
[x] Replay
[x] RefCount
[ ] Wait
[x] Concat
[ ] Merge
[ ] Zip
[ ] CombineLatest
[ ] Switch
[x] WhenAll
[x] StartWith
[ ] Synchronize
[ ] ObserveOn
[ ] SubscribeOn
[ ] DelaySubscription
[ ] Amb
[x] AsObservable
[x] ToObservable
[x] Cast
[x] OfType
[x] AsUnitObservable
[x] Create
[x] Empty
[x] Never
[x] Return
[x] Throw
[x] Range
[x] Repeat
[x] RepeatSafe
[x] Defer
[ ] Start
[ ] ToAsync
[x] Select
[x] Where
[x] SelectMany
[ ] ToArray
[ ] Do
[ ] Materialize
[ ] Dematerialize
[ ] DefaultIfEmpty
[ ] Distinct
[ ] DistinctUntilChanged
[ ] IgnoreElements
[ ] ForEachAsync
[ ] Finally
[ ] Catch
[ ] CatchIgnore
[ ] Retry
[ ] OnErrorRetry
[ ] FromEventPattern
[ ] FromEvent
[ ] FromAsyncPattern
[x] Take
[ ] TakeWhile
[ ] TakeUntil
[ ] Skip
[ ] SkipWhile
[ ] SkipUntil
[ ] Buffer
[ ] Pairwise
[x] Last
[x] LastOrDefault
[x] First
[x] FirstOrDefault
[x] Single
[x] SingleOrDefault
[x] Interval
[x] Timer
[x] Timestamp
[x] TimeInterval
[ ] Delay
[ ] Sample
[ ] Throttle
[ ] ThrottleFirst
[ ] Timeout
[x] FromCoroutine
[ ] SampleFrame
[ ] ThrottleFrame
[ ] ThrottleFirstFrame
[ ] TimeoutFrame
[ ] DelayFrameSubscription
[ ] RepeatUntil***

More Improvment StackTrace.

This stacktrace indicates Where().Select().Take().Subscribe()
Extremely clear and gets more performance!

I've done all.
I'll release next version(5.0) soon.

New version is very fast.

Check Subscribe Speed, 10000 subject with the operators .Where(x => x % 2 == 0).Select(x => x * x).Take(10).Subscribe()
Check OnNext Speed, 10000 ReactiveProperty change the value with the operator .Where(x => x % 2 == 0).Select(x => x * x).Take(10).Subscribe()

Old:
Subscribe = 457ms
OnNext = 11ms
New:
Subscribe = 38ms
OnNext = 2.6ms

Release UniRx 5.0.0.


Add back button for mobile

On mobile, the editor takes up the entire screen and there's no way to go back. It would be awesome to have a back button, or keep the top bar visible.

Yes, that's messed up.
I'm wondering if there should be even support for mobile since this is a very technical integration.
Anyway, the UI aspect of this integration is pretty basic. I didn't investigate how to make it right since it essentially works. Anyone with experience on frontend coding would be welcome to join in an implement it the proper way.

Makes sense! Is there a simple way to disable this on mobile? It’s easy to accidentally tap it 😄

For now, the way I do it is disabling the integration at the integrations page and re-enable it in the rare cases I need it. It's pretty fast to do it, it handles the mobile issue and, most importantly, it keep the sidebar less crowded.
But yes... mobile "support" is todo.

Ah makes sense. Thanks, will do!


connect to API

In this milestone of the project, I've

[x] used the Bookstore API as the backend
[x] Fetched books from the Bookstore API after the component has rendered
[x] Refactored add book and remove book features to persist your changes in the server

Hi @binhussen, Thanks for approving the PR😊.


Add support for mapping task names to multisource keys

Fixes #60
Depends on https://github.com/spring-cloud-stream-app-starters/core/pull/99
Remove ListFileRotator and add SftpSourceRotator  configured for all multi source flows.  This replaces some functionality of the list rotator, providing SFTP connection information if needed, and sets the selected server key as a message header.
The latter is used by SftpMultiSourceTaskNameMapper to configure the task name based on properties like sftp.multisource.task-names.one=foo, sftp.multisource.task-names.two=bar.

@garyrussell

Re Artem's comments, we can't do as he suggests. See the javadoc here https://github.com/spring-cloud-stream-app-starters/sftp/pull/61/commits/2b61d98da70e8c4c77175eda8002c975a60810cf#diff-9359cf8d76348761cff528fd31b01c28R56.  I forgot to push that yesterday.
I'm puzzled about the build issue, this change depends on the corresponding PR in core, but I'm not sure if that error is related.  But you would get compile errors if you didn't install task-launcher-common first.
Re import order, there is no checkstyle on these projects. We should have one, but if everything were to conform to standard rules, it would likely create a diff everywhere, so I'm reluctant to get into it here. IMO - there should be a separate issue for that.

Thanks @artembilan @garyrussell for the excellent feedback.  I will have you review all my PRs from now on. LMK if there is anything else on this.


Add Windows build to CI

It seems like AppVeyor is either not building the PR or taking a much longer time than normal to do so.

Looks like the Windows build is getting stuck on NavigationTests.MultiLineJaggedTextTests.pressingDownMovesCaretToNextLine. My workaround for insuring the TextFlow was properly set up to have 3 lines seems to be the culprit.

I have observed this earlier with some other tests running on Windows - the root cause is that
waitForMultiLineRegistration() NEVER returns, means it does not even time out after five seconds in WaitForAsyncUtils.waitFor(5, TimeUnit.SECONDS, textFlowIsReady);. A workaround I tries earlier was to explicitly terminate the while() loop, something like
int count = 0;
while (area.getParagraphLinesCount(0) != 3 && count++ < 500) {
   sleep(10);
}

With that , the test still fails, but at least it does not block forever.

Why do we need that "MultiLineRegistration" workaround at all? Is it because of different font sizes/different layouts on different window systems (Mac vs. X11 vs. Windows)? During my earlier analysis, I also simply made the text longer so that it is spread over 3 lines, and the tests also passed ...

It never returns or times out? Very strange....
The workaround is needed because in my initial tests on Linux, the tests would always fail because TextFlow wouldn't return the correct paragraph count immediately after it was set up. So, by waiting for the expected count to be the value returned, then the tests will work. Perhaps we should try running the test without this workaround to see which OS needs it and which do not.

Hmm... seems it works even on Mac now without that method....
I wonder if splitting up the navigation tests into their respective classes also removed an interference between tests that I didn't know about.

Since the font is never specified in the test, I'm guessing the tests pass on Linux/Mac because their default fonts are the same but that Windows uses a different default font. Thus, more/less content is being displayed per line than expected, resulting in a failed test.

It seems the referenced issues are currently blocking the last commit I've pushed to this PR...

An update to JUnit fixed the problem. Nice.


StackExchange rollover text shows "StackExhange"

The StackExchange rollover text on any elementary site shows "StackExhange" instead of "StackExchange".

 Completely my fault, I made this typo in #784. I'll submit the fix right away.


duplicate column name

Describe the bug
I upgraded the database and added a new column named speed to a table. However, I encountered an intermittent error on Windows when attempting to add a new row to the table:
SqliteException(1): while executing, duplicate column name: speed, SQL logic error (code 1)
Causing statement: ALTER TABLE "file_contents" ADD COLUMN "speed" INTEGER NOT NULL DEFAULT 0;, parameters:
I am using:
drift: ^2.15.0
sqlite3_flutter_libs: ^0.5.21

Can you show the migration? Is it the latest one? Are you doing anything except adding the new column?
My best guess (but this requires more information either way) is that something in the migration goes wrong after that ALTER TABLE statement already ran. In that case, drift won't store the new version number and will try to re-run the migration afterwards, causing the error you're seeing. Do you see any other drift-related errors on the same devices that might indicate something with the migrations going wrong?

Can you show the migration? Is it the latest one? Are you doing anything except adding the new column? My best guess (but this requires more information either way) is that something in the migration goes wrong after that ALTER TABLE statement already ran. In that case, drift won't store the new version number and will try to re-run the migration afterwards, causing the error you're seeing. Do you see any other drift-related errors on the same devices that might indicate something with the migrations going wrong?

Thank you for your help. I've identified the issue causing the problem. My database's latest version is 4, but I reinstalled an older version of my app and kept the database file, which downgraded the database version to 3. However, the 'speed' column wasn't deleted during this process. When I reinstalled the newest version of the app, it tried to add the 'speed' column again. To fix this, I should either add a statement for downgrading or check if the column exists before attempting an upgrade.

Wondering, how can we check if the column exists before executing migration? I got the same issue the other day

This PR contains code to check which columns currently exist in a database table. I'm not sure about merging it as I think the check should ideally be unnecessary in well-written migrations, but perhaps it's helpful as a pointer if you need that logic.

Thanks, that would be amazing to have! I came up with something similar but it seems to be now wrong compared to your solution (didn't know about readNullable)
extension on Migrator {
  Future<bool> columnExists(String name, String table) async {
    // check if column exists
    final tableInfo = await database.customSelect('PRAGMA table_info(${table})').get();
    final column = tableInfo.firstWhereOrNull(
      (e) => e.read<String>('name') == name,
    );

    return column != null;
  }
}

To check if a column exist in a certain table I use this function (adapted from above)
Future<bool> hasColumn(String tableName, String columnName) async {
  // PRAGMA doesn't support parameter binding, need to escape table name
  final escapedTableName = tableName.replaceAll('"', '""');
  final tableInfo = await customSelect(
    'PRAGMA table_info("$escapedTableName")',
  ).get();

  return tableInfo.any((row) => row.read<String>('name') == columnName);
}

@simolus3 the use case for this is that I have a python process that might also migrate the database. Would be useful to have such a method in the drift api.


Add HTML entity replacing for & and '

It is observed that the character '&' is output by other LLMs as HTML entity ("&\amp;"), just like '<' and '>'. e.g. here and here
This commit is to add "&" and possibily "'" (since " is there) to the workaround list.

Thank you @zjuwyz and @sohaieb for the PR 🙏
Note to reviewers: we'll need to compare this with https://github.com/cline/cline/pull/975 and try to get both branches merged in.


Add dial timeout by default

This sets dialing timeout on SSH calls. It could happen that if the
remote part doesn't shutdown the SSH connection properly, the connection
on golang could sit on ssh.Wait().
Setting a Dial timeout allows the client to poke with keep-alives and if
the remote part doesn't answer in a timeframe the connection is closed.
Signed-off-by: Ettore Di Giacinto<EMAIL_ADDRESS>

Nice catch


Environment Not Easily Reproducible on Other Machines

We currently use the packages specified in requirements.txt to create the virtual environment for running all OPS code. While this works on the Whitehead Institute (WI) HPC that has some packages and programs initialized, I have been unable to easily recreate and use this virtual environment on 2 other machines that are outside of WI HPC.
To make the use and development of this software accessible from other machines, we should consider updating the venv or moving to a different env system that makes set up and use of the environment easy on other machines.
Let's use this thread to track our approach to this problem!

This blog post discusses when we one would benefit from using different package management solutions.

Conclusion
Choosing the right environment management tool depends on your needs. If you need a simple, easy-to-use tool, venv might be the best choice. If you’re dealing with complex dependencies, Conda env is the way to go. If you need to switch between different Python versions, consider pyenv or virtualenv.
The best tool is the one that fits your project’s needs and makes your development process smoother. Experiment with these tools and choose the one that suits you best.

Given that this project will likely involve complex dependencies, conda seems like a good possible option for our environment management. I was able to create a environment file for conda that make this repo accessible on other machines in https://github.com/cheeseman-lab/OpticalPooledScreens/pull/4.

The Snakemake documentation also mentions that conda is the recommended way to install Snakemake, because it also enables Snakemake to handle software dependencies of the workflow. This would be particularly helpful if we have multiple workflows that require different environments.

Added a ops_dev_requirements.yaml file in #4 that I was able to use to run 1_sbs example code. This environment will live in the dev branch and be updated as we refactor code. Hopefully, the work to migrate the code to Python 3.11 is minimal and we don't have large issues with changing package versions.


Raises and possibly addresses a bug in the multilevelprojection class.

- Since the interpolation functionals can return positive or
 negative values it is possible rbcSum can be zero after
 completion of the loop even if the interpolationfunctional
 return non-zero numbers.
- This can cause issues in line 6617 when rbcSum is used
 to scale the rbc values.
- I believe the best solution is to make rbcSum the sum
 of positive numbers and avoiding the scaling in the corner
 case where all interpolation values are zero.
- As I'm not familiar with the multilevelprojection algorithm,
 this solution may not be appropriate.

Thanks, I think this should be fine for now. I'll revisit when I get around to finishing #162


[Bug]: M1 GPU (mps) support

Describe the bug
It looks like the Speechbrain library does not support the M1 GPU (mps backend). The error is raised when trying to use the MPS backend on a pre-trained model (at least, this is the case I found, I don't know if it happens also in other situations, but I guess it does) and in particular the error is:
{ValueError}invalid type: 'torch.mps.FloatTensor'
The error is caused by this line in dual_path.py (file in the Speechbrain library, line 1066):
        if gap > 0:
            pad = torch.Tensor(torch.zeros(B, N, gap)).type(input.type())

And it is caused by the fact that input.type() returns torch.mps.FloatTensor but such value is not a valid Tensor type.
Such problem has been already reported in PyTorch (here: https://github.com/pytorch/pytorch/issues/82296) and looks like it is on its way to be fixed.
However, it looks like Speechbrain will need to upgrade its PyTorch dependency (from the PyTorch discussion it looks like they're gonna include the fix in Torch 2.0) or find a workaround with the datatype in the meanwhile 🤔
Expected behaviour
Being able to use the MPS backend on a M1 Mac to run Speechbrain models
To Reproduce
from speechbrain.pretrained.interfaces import SepformerSeparation
import torchaudio
import torch

separator = SepformerSeparation.from_hparams(source="speechbrain/sepformer-wsj02mix", savedir="./pretrained-sepformer-wsj02mix",  run_opts={"device": "mps"})

s1, fs = torchaudio.load('./my_file.wav') # Just insert here any wav file you want
resampler = torchaudio.transforms.Resample(fs, 8000)

s1 = resampler(s1)

est_sources = separator.separate_batch(s1)

Versions
0.5.13
Relevant log output
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[49], line 1
----> 1 est_sources = separator.separate_batch(s1)

File ~/Desktop/personal_git/voice-assistant/.venv/lib/python3.10/site-packages/speechbrain/pretrained/interfaces.py:1976, in SepformerSeparation.separate_batch(self, mix)
   1974 mix = mix.to(self.device)
   1975 mix_w = self.mods.encoder(mix)
-> 1976 est_mask = self.mods.masknet(mix_w)
   1977 mix_w = torch.stack([mix_w] * self.hparams.num_spks)
   1978 sep_h = mix_w * est_mask

File ~/Desktop/personal_git/voice-assistant/.venv/lib/python3.10/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File ~/Desktop/personal_git/voice-assistant/.venv/lib/python3.10/site-packages/speechbrain/lobes/models/dual_path.py:1017, in Dual_Path_Model.forward(self, x)
   1012     x = self.pos_enc(x.transpose(1, -1)).transpose(1, -1) + x * (
   1013         x.size(1) ** 0.5
   1014     )
   1016 # [B, N, K, S]
-> 1017 x, gap = self._Segmentation(x, self.K)
   1019 # [B, N, K, S]
   1020 for i in range(self.num_layers):

File ~/Desktop/personal_git/voice-assistant/.venv/lib/python3.10/site-packages/speechbrain/lobes/models/dual_path.py:1097, in Dual_Path_Model._Segmentation(self, input, K)
   1095 B, N, L = input.shape
   1096 P = K // 2
-> 1097 input, gap = self._padding(input, K)
   1098 # [B, N, K, S]
   1099 input1 = input[:, :, :-P].contiguous().view(B, N, -1, K)

File ~/Desktop/personal_git/voice-assistant/.venv/lib/python3.10/site-packages/speechbrain/lobes/models/dual_path.py:1067, in Dual_Path_Model._padding(self, input, K)
   1065 gap = K - (P + L % K) % K
   1066 if gap > 0:
-> 1067     pad = torch.Tensor(torch.zeros(B, N, gap)).type(input.type())
   1068     input = torch.cat([input, pad], dim=2)
   1070 _pad = torch.Tensor(torch.zeros(B, N, P)).type(input.type())

ValueError: invalid type: 'torch.mps.FloatTensor'

Additional context
No response

Hello,
Thanks for raising this issue.
I think a workaround would be to replace:
if gap > 0:
    pad = torch.Tensor(torch.zeros(B, N, gap)).type(input.type())

by
if gap > 0:
    pad = torch.Tensor(torch.zeros(B, N, gap)).type(input.dtype).to(input.device)

If seems to work.
I will do a PR now to fix this.


[FLINK-15902][python][doc] Improve the Python API doc adding the version an API introduced

What is the purpose of the change
Currently it's not possible to know in which version a Python API is added. This information is very useful for users and this PR will address this problem.
Brief change log

Add a since decorator
Add since decorator for the the APIs introduced in 1.10

Verifying this change
This change is a documentation work without any test coverage and have checked the Python doc manually.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@WeiZhong94 Thanks for the review. Good catch and have updated the PR.

@dianfu Thanks for your update! LGTM, +1 to merge.

@WeiZhong94 Thanks for the review. Will merge the PR once the travis turns green.

@flinkbot run travis

@flinkbot run azure

Merged as the travis has passed.


Provide an in-cloud pollinate random number seeding service for Ubuntu images

TIL that Ubuntu cloud images call out to a remote web service to seed their random number generators, and it makes me really uncomfortable. Firstly it gives Canonical really good telemetry about how many Ubuntu instances are out there in clouds, even private ones. Secondly, how can we actually trust someone to not own that service and hand out deliberately bad random number generator seeds?
Now, its been around for a decade, but I can find basically zero discussion of its security implications online. I think that means people probably don't know it exists.

The client: https://git.launchpad.net/pollinate/tree/pollinate
The server: https://bazaar.launchpad.net/~kirkland/pollen/trunk/view/head:/pollen.go

Am I being unreasonably concerned here?

Actually, we should just remove it from the Ubuntu images. It turns out libvirt is already using virtio to seed the RNG based on the host's RNG.


Add to logic documentation and other doc updates

Changes:

Updates to logic DG section
AddressBookParser -> InputParser
Some leftover seedu fixes
Made handle import/export private

Codecov Report

Merging #63 (a751b3b) into master (98be97e) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master      #63   +/-   ##
=========================================
  Coverage     72.04%   72.04%
  Complexity      475      475
=========================================
  Files            79       79
  Lines          1481     1481
  Branches        153      153
=========================================
  Hits           1067     1067
  Misses          372      372
  Partials         42       42

Impacted Files
Coverage Δ

src/main/java/teletubbies/ui/MainWindow.java
0.00% <ø> (ø)

src/main/java/teletubbies/logic/LogicManager.java
75.00% <100.00%> (ø)

...ain/java/teletubbies/logic/parser/InputParser.java
100.00% <100.00%> (ø)

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 98be97e...a751b3b. Read the comment docs.


Add numbers strategy for Gson to prevent changing int to double

Fixes issue described here: https://github.com/ably/ably-flutter/issues/461

@ikbalkaya Hey, this issue still exists in latest version because code with fix that I've provided is not on main branch


Past Incident

Incident created with starttime: Tuesday, 23 January 2024 17:30:00 GMT+05:30 and endtime: Tuesday, 23 January 2024 17:35:00 GMT+05:30 [start_datetime:1706011200] [end_datetime:1706011500]

Incident no 36


Update docs to use new collapsing TOC

Also include docs refactoring similar to https://github.com/spring-cloud/spring-cloud-dataflow-server-kubernetes/commit/9455d62b3a55d9cdddfee36c98c4a0d0c0373acf

This has already been implemented (but isn't getting published apparently)


added bn.js types fixed #1711

added bn.js types into devDependencies

now the typing for BN works as expected

resolves #1711


Facebook: logging in shows peers as offline until monitoring kicks in

If Alice + Bob are friends on Facebook, and Alice is already online with Facebook in uProxy, then Bob logs into Facebook, Alice will immediately see Bob as online, but Bob won't see Alice as online for about a minute.
Current understanding of this is that:

Bob see's an onClientState for Alice with ONLINE, sends his instance message, which successfully reaches Alice.
Alice see's an onClientState for Bob with ONLINE (as well as Bob's instance message) and sends her instance message.  However for some reason Alice's instance message never reaches Bob.
After a minute, Bob will detect that he knows Alice is ONLINE, but doesn't have her instance message.  He will then send an instance_request (3003), which Alice gets and this time Alice is successfully able to send her instance message back to Bob.  No idea yet why Alice's first instance message doesn't make it to Bob, but her second instance message does...

One quick fix (hack) if we can't figure this out in the next few days is to change the monitoring back to every 5 seconds for Firebase networks only

I have a fix for this, https://github.com/freedomjs/freedom-social-firebase/issues/16


Set Python user-agent string

I'm surprised that we set the user-agent string on the service-side of the RPC; I had only expected it to be set on the invocation-side. Is there a requirement that we set it on the service-side, or is it good practice, or courtesy, or... anything else?

Looks good; waiting on test results.

Jenkins mostly green. Dunno why Travis isn't running.


Feature Request: Allow two different elements with the same styles

I have a Button component (in React) which can either be a button or an a element, depending on if a href prop is passed to the component. Something similar to below:
const Button = ({ children, href, onClick }) => {
    if(href) {
        return {children};
    }

    return <button type="button" onClick={onClick}>{children}</button>;
};

I previously used Sass to style these components, but am now attempting to move over to styled-components. However, I have come across an issue where these two elements require the same styles, but the syntax of styled-components would require me to create to separate variables - styled.button and styled.a, with duplicated styles for each.
I would propose being able to pass a function with an element's props as the first argument, to allow a developer to decide which element should be used. Something similar to this:
const StyledButton = styled((props) => props.href ? 'a' : 'button')`
    ...
`;

An important feature of CSS is that styles and elements are not tied together. The example above shows that styled-components currently does have this issue, but I believe that my solution (or something similar) would allow for separation of element and styles.

You could use as prop: https://styled-components.com/docs/api#as-polymorphic-prop
Or use className prop:
const Button = ({ className, children, href, onClick }) => {
    if(href) {
        return {children};
    }

    return <button type="button" className={className} onClick={onClick}>{children}</button>;
};

...

const StyledButton = styled(Button)`
    ... styles
`;


Throw error when canvas background is used without sample buffer

Fixes: https://github.com/godotengine/godot/issues/16226
edit: I have updated this PR with a completely different approach. After spending some more time with it, I realized that the render_empty_scene call is needed to actually render the canvas background. So the real problem is that in order to use a canvas background, by nature you have to sample the main buffer. So using "2D - No Sampling" shouldn't be allowed. My new solution is to throw an error and to exit the render_scene function.

Note that render_empty_scene becomes unused after this PR, so it should also be removed.
From @reduz's comment it's not clear whether it's the right solution though:
14:36 <Akien> reduz: OK with #31751 (bugfix)?
14:36 <IssueBot> #31751: Remove `render_empty_scene` when Viewports cant draw 3d | https://git.io/fjxCk
14:36 <Akien> It's not clear why this render_empty_scene was used in the first place
14:37 <reduz> it exists for when you use an environment with a canvas BG
14:39 <Akien> Maybe the bool was meant to be `can_draw_3d` instead of its negation?
14:39 <Akien> See https://github.com/godotengine/godot/issues/16226#issuecomment-520125123
18:50 <clayjohn> reduz: Akien: I removed render_empty_scene because it was causing a crash in GLES3 and it caused the screen to go black in GLES2. Is it just being used to clear the background and optionally draw a sky?

Thanks!

Cherry-picked for 3.1.2.


API not working with BeamNG.drive

The documentation mentions that BeamNGpy should have basic support for BeamNG.drive:

The regular Steam release of BeamNG.drive is compatible to an extent as well. Certain sensors like the simulated LiDAR or camera will not work, but most of the functions that are not exclusive to a Tech build will likely work.

But I cannot get it to work. It seems that it fails to load the util_researchGE extension and with that the game will not respond on the API port 64256. I tried to follow earlier reported (now closed) issues on how to manually set up a workspace, but it does not seem to work for me. Is that the reason why the util_researchGE extension does not work? What am I missing?
BeamNG.drive console output:
000.623|I|Paths|User path: C:\Users\quaid\AppData\Local\BeamNG.drive\0.24
002.941|I|GELua.core_input_bindings.bindings|Controller connected: XBox Controller 1 (xinput0/0xXIDevice)
002.977|I|GELua.print|*************LightRayPostFX 2 set
002.982|I|GELua.print|********PostFXSSAOGeneralEnabled 2 set
003.033|I|reloadLanguages|Switched from '' language to 'en-US' (as fallback, after trying 'sv-SE')
003.382|W|GELua.core_levels.|info.json missing: /levels/GridMap/info.json
003.383|E|GELua.core_levels.|No entry point for level found: /levels/GridMap. Ignoring level.
003.733|E|GELua.extensions|extension unavailable: "util_researchGE" at location: "util/researchGE"
Python output:
python .\west_coast_random.py
2022-04-15 01:20:39,475 |INFO     |beamngpy                      |Started BeamNGpy logging.
2022-04-15 01:20:39,475 |WARNING  |py.warnings                   |C:\Users\quaid\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\beamngpy\beamng.py:69: DeprecationWarning: The use of beamng.setup_logging is deprecated and will be removed in future versions. Use beamngcommon.set_up_simple_logging or beamngcommon.config_logging instead.
create_warning(warn_msg, DeprecationWarning)
2022-04-15 01:20:39,475 |WARNING  |py.warnings                   |C:\Users\quaid\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\beamngpy\beamng.py:69: DeprecationWarning: The use of beamng.setup_logging is deprecated and will be removed in future versions. Use beamngcommon.set_up_simple_logging or beamngcommon.config_logging instead.
create_warning(warn_msg, DeprecationWarning)
2022-04-15 01:20:39,681 |INFO     |beamngpy.BeamNGpy             |Opening BeamNGpy instance.
2022-04-15 01:20:39,683 |INFO     |beamngpy.BeamNGpy             |Started BeamNG.
2022-04-15 01:20:42,006 |INFO     |beamngpy.BeamNGpy             |Terminating BeamNG.tech process.
2022-04-15 01:20:42,051 |INFO     |beamngpy.BeamNGpy             |Started BeamNG.
2022-04-15 01:20:44,323 |INFO     |beamngpy.BeamNGpy             |Terminating BeamNG.tech process.
2022-04-15 01:20:44,362 |ERROR    |beamngpy.beamng               |No workspace set up at userpath: <C:\Users\quaid\AppData\Local\BeamNG.drive>. Setup is required prior to mod deployment.
2022-04-15 01:20:44,365 |INFO     |beamngpy.BeamNGpy             |Started BeamNG.
2022-04-15 01:20:54,367 |INFO     |beamngpy.BeamNGpy             |Connecting to BeamNG.tech at: (localhost, 64256)
2022-04-15 01:20:56,390 |ERROR    |beamngpy.BeamNGpy             |Error connecting to BeamNG.tech. 25 tries left.
...

Hi,
I followed the steps in https://github.com/BeamNG/BeamNGpy/issues/157#issuecomment-1015330607 for the latest Steam version of the game (<IP_ADDRESS>) and the latest version of BeamNGpy (1.22).
It seems that the mod deployment part did not work, in case of a successful mod deployment, there should exist a BeamNGpy.zip file in the C:\Users\<USERNAME>\AppData\Local\BeamNG.drive\0.24\mods\ directory. Does it exist in your case?
Please try to double-check that you are doing the same steps as I did.

Hi,
I checked and indeed I do not have the BeamNGpy.zip file in the ..\0.24\mods\ directory. How do I get this file? I tried to follow the steps in the issue you linked to, to regenerate the BeamNG user folder, but that does not add the BeamNGpy.zip file to the folder? Which step have I missed?
Thanks for your help!

Have you also created the researchHelper.txt file? If so, BeamNGpy.zip should be generated with the first call to the open method of the BeamNGpy object:
bng = BeamNGpy('localhost', 64256, home=r'C:\Program Files (x86)\Steam\steamapps\common\BeamNG.drive')
bng.open() # here, the BeamNGpy.zip should be automatically created in the mods folder

If that doesn't happen, as a workaround you can try to use this already generated BeamNGpy.zip, move it manually into the ..\0.24\mods folder, and try to run your BeamNGpy code again.
Be aware that the features of BeamNGpy are quite limited for BeamNG.drive. If you are an academic/non-commercial user and have an interest in BeamNG.tech, you can try to apply for a BeamNG.tech license at https://register.beamng.tech/.

Thanks, it works now with the zip-file you provided.


Add FileCollection overload for bootClasspath

Answering #2747 I realized that we have no good way to lazily resolve the bootClasspath of a compile task. This can be useful if a JDK is fetched from a repository. In general it's just odd that a classpath property would only take a String.
We should add a FileCollection overload and possibly deprecate the String version.

When we add the FileCollection version let's make sure to also fix https://github.com/gradle/gradle/issues/2619.

This is now merged as #3002 and will ship with Gradle 4.3.


@synonymdev/slashtags-docid not found in NPM registry

Root-level npm run install fails with:
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@synonymdev%2fslashtags-docid - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)

The url package depends on @synonymdev/slashtags-docid which is not found in NPM.
The wallet demo in demo/auth/wallet similarly fails on npm install, as its dependency @synonymdev/slashtags-url expects @synonymdev/slashtags-docid.

npm run install works now that slashtags-docid has been published, but the root-level package-lock.json is looking for a local packages/docid folder that doesn't exist.
"@synonymdev/slashtags-docid": {
  "version": "file:packages/docid",
  "requires": {
    "@synonymdev/slashtags-common": "^0.1.0",
    "multiformats": "^9.4.6"
  }
},

Root-level node_modules looks like:

Probably need to delete and regenerate package-lock.json so it pulls docid from npm?

Thanks, @Entryist for the report and sorry for all that confusion, fwiw, docid is indeed deprecated, and I am working on updating slashtags-url to remove docid as a dependency, but currently the demo for slashtags-accounts doesn't depend on slashtags-url either, but it will after updating it, and refactoring some of the logic out of slashtags-actions and slashtags-accounts to be in slashtags-url.

please consider change slashtags-url to slashtags-uri for a more flexible  naming in regards to any future protocol adoptions ...


No library dir is defined

If this is supposed to be used as a library project you also need to specify the QuickButton folder as an included dir: https://www.defold.com/manuals/libraries/#_setting_up_library_sharing

Thanks. Another file transfer mishap.


river: init

:fish: What?

Added a new package;

:fishing_pole_and_fish: Why?

(1) This new package is interesting to myself;

:fish_cake: Pending

[ ] Complain with linter;
[ ] Remove some temporary fix;
[ ] Publishing to news' channel.

:whale: Extras

I like chocolate cake.

Can you rebase and apply linter suggestions? (nix fmt and statix)


Add JsonListProcessor

Adding the ability to pack a list of JSON objects in a single Kinesis message.
This will increase compatibility with other consumer such as Kinesis Firehose.
I feel everything is correct except this line.   I didn't want to change the code too much, but it is a little awkward. :sweat_smile:
https://github.com/hampsterx/async-kinesis/compare/master...jmcgrath207:master#diff-d6252472eb8deafe21520a24da3f5e3aR134
Testing for me works, but if the functionally looks good to you, I will write the units, update the benchmark, and Readme.

hi @jmcgrath207 looks good thnx! Yes if you can add a couple of test and update the readme that would be great. You can try the benchmark, the results did vary a little bit between runs so yeah, not too worried there~

Hey @hampsterx  sorry this took so long. Here are the unit test and a update to the doc.


New AWS autoscaling groups

AWS recently introduced the option of using autoscalinggroups and combine multiple instances.
https://aws.amazon.com/blogs/aws/new-ec2-auto-scaling-groups-with-multiple-instance-types-purchase-options/
For us this would be a great feature.
Is someone already working on this or should we create a PR?

Pr merged


"Flip comma" assist in parameter list should update callers

fn f(a: u32,$0 b: &str) {}

fn g() {
    f(0, "");
}

Applying "Flip comma" results in:
fn f(b: &str, a: u32) {}

fn g() {
    f(0, "");
}

Ideally it would update the call in g to update the argument order. (The same should probably happen when applying "Flip comma" in f(0,$0 ""))

I'm not sure... Is it so obvious that when I do "Flip comma" in e.g. f(0, 1), I want the parameters exchanged everywhere? Maybe we should make the parameter order refactoring a separate code action :thinking:

Yeah, that would work too

I guess this ties into https://github.com/rust-analyzer/rust-analyzer/issues/6499

As usual, IntelliJ gets this right:
https://user-images.githubusercontent.com/1711539/116228961-a2801700-a75e-11eb-8857-87d107d290b3.mp4
Doing this is going to  be pretty challenging:

we need to figure out how to expose this UX given LSP / editor UI constraints
this adds a new dimension of state to rust-analyzer, managing that is going to be hard and fun


Change Submit button Color  because both the background and the button are yellow

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots


Improve XAML Diagnostics docs

@ahmed605 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ahmed605 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ahmed605 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ahmed605 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ahmed605 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Is there any updates on this?

Hey @jwmsft, is there any updates on this? It's been over 3 months since I opened this PR

Thanks for your contribution. At this time the APIs covered in this PR are not intended to be documented.
If the situation changes, we'll re-open the issue.
-John

mfw the

Not documenting this API is anti-competitive behavior: it prevents competing IDEs (like CLion and Rider) from implementing XAML Live VIsual Tree and XAML Hot Reload.


Add log when tappers cannot be started

I don't think we need this after https://github.com/up9inc/mizu/pull/1022


Finds network interface without configured IP address

Problem: finds network interface name but not IP address.
Reason: network interface (device) isn't configured for IP address.
Solution: sudo dhclient -v $(dev) can be added on the second condition level on the line: https://github.com/jschmid1/gopro_as_webcam_on_linux/blob/be1d881e6f826922ad5341fdf7d229718d30961f/prepare_webcam.sh#L123

Hi @dmitry, that script is now deprecated. Nevertheless, thanks for the suggestion!

Hi @dmitry, that script is now deprecated.

Actually, this entire project seems deprecated, with the amount of comments here and elsewhere indicating this doesn't work, and has never worked, except in very narrow cases of devices and highly customized environments.
The top-level README.md should be updated to reflect this, so more users don't find this repo linked from various forums and blog posts, and end up similarly frustrated with these dead-ends.


Minor updates to the paper

Generally speaking, I think it looks good.
I went mostly through the grammar and split sentences to improve the readability. However, I did not come up with a nice solution for the sentence starting in l. 38. I do not really like the way we have it written right now as it is a bit convoluted and really long. That makes it hard to understand.
One suggestion is
TimeStruct simplifies the writing of key equations in optimization problems through separation of the indexing sets and the equation. Consequently, equations unaffected by the the chosen time structure, e.g., simple deterministic operational or stochastic programming models, must not be adjusted when switching between different time structures. Hence, it simplifies both model development and subsequent switching between different time structures.
What are your thoughts? Is that paragraph still capturing the essence of he previous paragraph?

I think your paragraph is an improvement. Minor adjustments: plural both for index sets and equations? s/must not/need not/.


How to add a user to the sudoers

Hello.
I'm using "NixOS KDE 16.09".
I have created a user, but I am in trouble I do not know the way that I want to add to the sudoers.
Please professor how to add a user to sudoers.
Steps to reproduce
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
Nix version: (run nix-env --version)
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

add the user to the wheel group.

@usuha You should not ask such questions on the issue tracker. Questions are for sites like http://unix.stackexchange.com/ or the IRC channel. The issue tracker is essentially for bugs.

Thanks a lot tohl.
Thanks a lot Zimmi48.
Many thanks.

We have an example of adding a user to sudoers here: https://nixos.org/nixos/manual/#sec-user-management see that their extraGroups includes "wheel".

Closing as this seems to have all necessary information :)

Adding a "wheel" group in the user declaration failed to create the "wheel" group during nixos-rebuild build.


Mention Tags not showing up on desktop app

I see we have mention tags enabled and I have created a few and added my team members.  However, I cannot do any @ mention of them from within the desktop app and my general team channel.  I understand these don't work for private channels at the moment, but the general channel should still work correct?  We are able to make this work over the mobile app, just not the desktop app.

@GaryCoopATL thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


[Enhancement/功能请求]

Checks/检查

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting./我确认在报告之前我已经搜索了现有的问题或者拉取请求，以避免重复报告。
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly./我确认我已经检查，如果我不按照说明进行操作，该问题可能会被直接关闭。

Description/描述
A quick suggestion that I recall a version of this mod previously had, could you make it so that the boots, when equipped as part of the full set, automatically step you up onto the next block as you run up to it? Not an auto-jump, but a step up. If yes, potentially it could step up more than 1 block, possibly up to 3?

That feature has been removed due to poor reception.


Refactor Filtering class

Description
Refactor Filtering and create as many filtering classes as method in it
Additional information
MeanFiltering is a good example to follow

TODO

[ ] update base class as abstract classes
[ ] create new classes from 4 different methods of the Filtering class:

[ ] remove_data_without_metadata
[ ] remove_rows_without_relevant_info
[ ] selecting_rows
[ ] deleting_only_zeros_rows

[ ] Replace usage of Filtering in main.py and differentialanalysis.py part
[ ] Delete Filtering


Retrieving resources

Is there a way to retrieve page's resources? Which I can see on the network tab in DevTools.

See #105. This keeps coming up, so I might add an example.


Build Runner Change Listeners now get their own class

Subclass BuildRunnerChangeListener and implement done_listening? and
row_received

BuildRunner now cleans up the list of BuildRunnerChangeListeners when needed

Implemented WebSocketBuildRunnerChangeListener

Fixes #826

[x] I have run rspec and corrected all errors

[x] I have run rubocop and corrected all errors

[x] I have tested this change locally and tried to launch the server as well as access a project, and with that at least one build

[x] If there is an existing issue, make sure to add Fixes ... as part of the PR body to reference the issue you're solving

I'm going to merge this and get it running on CI


Split buttons

I have added split buttons like these

I'm not to sure about the color of the "arrow" button when pressed. They match the pressed color of the dropdown buttons. I have previously asked @markdotto about this color.
I think that the color of the pressed dropdown/split buttons should be the same as the color of the regular pressed buttons.

very nice, I like @brennanmceachran's border color better

dang this is awesome!!!

+1 @brennanmceachran's change.
Would be nice to see a version with text centered in its own area to see if it looks odd.

NICE

Looks nice!

@pokonski as the markup for the split looks like this
Success

you can simply reverse them like so
Success

Sweet :)

+1

+1

Good stuff. What about also leaving the button's style, when clicked, as in a focus state, like the arrow?

@zrod I think it wouldn't be split button but a dropdown button, if the button syle was pressed as well.

+1

+1

+1

+1

+1

what is the plan for the kasperp-dropdown-btn-dev branch, will it be available in 1.5 or 2.0?

+1

+1

+1

In the process of updating the 2.0-wip docs for our new button groups, I realized we had effectively built 75% of what we needed for split buttons with dropdowns. I just spent the last couple hours overhauling the CSS to support this and document it in the 2.0-wip branch.
Here's what we're looking at:

Going to close this pull request since it's for a branch off master and one that I'm not maintaining anymore. It took us long enough, but we've got the split buttons in :).

@markdotto those look great. I'm excited to finally get these in (even if they don't use my code ;-)).
Do you provide an option for dropdown button as well (without the split), as proposed in my original pull request https://github.com/twitter/bootstrap/pull/573 ?

Not yet, not explicitly. Might be able to add that with the state 2.0-wip is in soon.

I can also give it a stab if you prefer.

any update on the non-split buttons for 2.0?

@kbond actually @markdotto's implementation also allows you to create "non-split" buttons, simply remove the "split" part of the button html, so you end up with something like this.

    Action

Action

Another action

Something else here

Separated link

@markdotto I guess all thats missing is a doc entry for "non split" dropdown buttons ;-)


:zap: Mt 882 add h4 tag title card article

Added the class h4 to the h4 tag after the design review


Verbosity/synchronization challenges

Both URL band and source text band solutions require potentially large duplication of these attributes. This is a challenge as it increases shipping size for web, which is the primary target of this proposal, and means keeping all location of the attributes in sync. Additionally, if the attributes are in multiple locations due to using a solution that allows/requires such, it should clarify how mismatches are resolved.
Example given a URL band solution:
// x.mjs
import a from 'import+json://foo.json';
import b from 'import+javascript://foo.json';
import c from 'import://foo.json';

// y.mjs
import d from 'import+json://foo.json';

Could all 3 in x.mjs succeed? They have separate request URLs in web spec terms, so it seems like they would resolve independently.
I think solving for the verbosity impacts synchronization of these in non-trivial ways. E.g. SRI being defined per source text would mean invalidating the whole dep graph when an SRI changes. Out of band solutions/single location solutions could preserve parts of the dep graph that were unchanged still.
I think having more attributes and their use cases discussed would increase the visibility of this concern.

I suppose the type JavaScript would be the default, only new modules would require a type attribute. Repeating the same attributes many times in the file will compress well.
An advantage of the out-of-band format is that the type would be specified only once per file, as opposed to every file that imports it.

@xtuc the update for every file that imports it would apply to any attribute that would require updating. Changing the format of a file for type or changing the content of a file for integrity. This means updating dependencies and transitive dependencies would require updates to all dependent source files if done in band for every operation that needs to synchronize with an attribute.

The SRI concerns that @bmeck mentions are exactly why we don't focus on inline SRI in this repository, and why there's a good chance it would never happen at all. We could discuss technical details more in #8 . On the other hand, for module type, I think it's important to be in-band and not out of band because that is indeed the place to do the update where appropriate. Finally, if you want to discuss the form of putting metadata inline in the module specifier, let's discuss this in #11 .

@littledan regardless of where it is located and what the attribute key/value pairs are, the issue here is the duplication and potential mismatches of attributes. Other attributes outside of SRI are interesting here if we don't want to discuss SRI, like language, no-top-level-await, and/or mobile. I can think of many varieties of useful attributes, and do not want this proposal to only focus on essentially specifying a single accept: header key/value pair.

The appropriate place to determine what type a module is is in the module itself, not in its consumers, which is why i also prefer out of band.

@ljharb How would you compare the approach of syntax within a module specifier to the one in this proposal?

Sorry, my last comment got a bit off topic. Let's centralize the discussion about out of line attributes in https://github.com/littledan/proposal-module-attributes/issues/13


Add how to install this project in server

add how to install this project in server :
example

[ ] - what is requirement to install this project.
[ ] - how to clone project.
[ ] - how to install this project

harus bahasa inggris kah?


Unit test check environment for device_ver

Change unit test check environment parameter from cuda_ver to device_ver, which is more compatible with hardware platforms such as MPS, NPU and more upcoming hardware platforms.

@microsoft-github-policy-service agree

@tjruwase Would you please launch CI for me and review this commit. Thanks :)


[website] fix: pulsar website cli docs build error

Signed-off-by: LiLi<EMAIL_ADDRESS>
(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #13452

(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[x] doc
(If this PR contains doc changes)

@Anonymitaet @tuteng PTAL, thanks.

/pulsarbot run-failure-checks


redirect code is invalid

on this page
const destinationURL = "https://example.com"
const statusCode = 301

async function handleRequest() {
  const { pathname } = new URL(request.url)
  return Response.redirect(destinationURL, statusCode)
}

addEventListener("fetch", async event => {
  event.respondWith(handleRequest())
})

this should be
const destinationURL = "https://example.com"
const statusCode = 301

async function handleRequest(request) {
  const { pathname } = new URL(request.url)
  return Response.redirect(destinationURL, statusCode)
}

addEventListener("fetch", async event => {
  event.respondWith(handleRequest(event.request))
})

would be great if we could set up some way to run tests on the example code we have in the docs, maybe integration tests w/wrangler dev


Is it nearly impossible to write a Port Forwarder that works?

So I wrote a port forwarding application using Simple Tcp.  I test it with a custom client connecting to the port forward app, which forwards packets to a custom server (also for testing).  When the server receives a packet, it echos the data back to the port forward app which relays it back to the originating client app.  This way I can verify that the data receives and comes back as the same data.
It works.  But when I try to use this with real applications (such as Sql client to connect to a Sql Server), I get spotty results.  It half-works but some applications throw errors about unexpected data/messages and other things like that.
Am I wrong in thinking that port forwarding is as simple as putting a middle-man application in between to 2 endpoints to marshal packets from A to B and from B to A?  Is there something else necessary to make this work?

Hi @pha3z in theory it should work but that's not always the case. I suggest looking into how firewalls that proxy TCP do this. Often they have to do fix ups or use NAT/virtual IPs to emulate the actual client or server to maintain protocol semantics. Some app protocols also are sensitive to placement of TCP segments and which ones have/don't have the PSH flag set (I dealt with this extensively in a past life with WAN acceleration and also with security). The correct answer is you have to test app by app and dig into their protocol semantics to see why it may or may not be working.
Cheers
Joel

Thanks, Joel.

No problemo. I think doing what you want would demand a transparent TCP proxy, ie even though there is a man in the middle, the proxy listens on the IP of the server, and acts on behalf of the client while emulating the client IP and port. We did this within a product at Cisco and it literally either required a) network configuration gymnastics to handle the routing (ie diverting traffic from its original destination) or b) deployment using an inline bypass card that punted packets to a user space packet processor. That way you can preserve IPs and preserve TCP semantics (ie when there is/isn't a PSH and other flags).
Another option if protocol dissection isn't of interest would be to build something at the IP layer instead of TCP. That also has challenges and some protocols require fix ups (such as FTP).

Wow.  That is very helpful!  Sounds like its just too involved unless the use case absolutely demands it -- which mine doesn't demand that level.
Thank you, sir!


State Serialization

A State object should be serializable; for replays, saving/loading from disk, and other uses.

Just as an aside, we can potentially avoid needing to completely serialize/deserialize states for the sake of replays, at a cost of forward-incompatibility.
The PRNG in #55  and the map save/load feature in #56 means we can save/load replays by serializing the map and the list of actions in a battle. With this (and the matching software version) one can recreate all the states of the battle. The downside is that these will be less future-proof since changes in the game mechanics are more likely to break old replays.


Add the black formatter GitHub Action

Add the black formatter GitHub Action
Overview
Currently, there is no Code Formatter used in flet-cli, Our Goal is to add a Black code formattingr in flet-cli.
Description
Describe the solution you'd like - Add a GitHub action that checks a PR for coding style using black. The GHA should be able to format the code and commit it to the PR if the PR is not following the coding style.
Expected outcomes
A uniform coding style across flet-cli, that Autoformat code on every PR.
Skills
GitHub Actions, Python
Possible mentors
Kanishk Pachauri, Anjuman Hassan
Expected project size
50 hours.
Difficulty
Hard.

#Add the black formatter GitHub Action
My Background and Technical Skills
I have been into coding stuff since class 8th trying my hands on Python and its libraries like Pandas etc. Right now i'm handy with WebD, Python, C++ and in designing I have worked on Figma.
What problem are you trying to solve and Your Approach?
So I am trying to add the black formatter through github action. Black formatter is python libraries which auto formats the code like indentation and other things. Which ultimately saves user time who will push there messy code.
I have also created a sample github action in one of my repo which uses super-Linter to check format of code.
linnk to it : https://github.com/Srijan-SS02/ActionDemo
Your Timeline to solve the problem.
19th Sep 2022 : get the project assigned by Mentors and try to interact with mentors
20th Sep 2022 : trying to solve the issue and trying to get helped by the mentors
21st Sep 2022 : wrapping up the issue and fixing the minor bugs if any
@Mr-Sunglasses @AnjumanHasan tagging maintainers for better reach

@AnjumanHasan @Mr-Sunglasses  please review this PR and let me know if any changes are required


Refactor label font-size and margin-top to allow label resizing

Closes #325
Allows the lablels to resize dynamically with cellSize. I moved the font-size controls into the d3 code and updated the margins to allow for the larger dynamic labels.
 Makes me think I should also implement the changes to the int node table soon too

Already on it! See #328


Define enroute decorator

class TicketQueryService(QueryService):
    """TODO"""

    @enroute.broker.query(topics=["CreateTicket"])
    @enroute.rest.query(url="tickets/", method="GET")
    @enroute.rpc.query(method="createTicket")
    async def get_tickets(self, request: Request) -> Response:
        ...

    @enroute.broker.event(topic="TicketAdded")
    async def ticket_created(self, request: Request) -> NoReturn:
        ...

Proposed base structure:
class BrokerCommandEnroute:
    @classmethod
    def __new__(cls, *args, **kwargs):
        fn = args[0]
        print("broker command")
        return fn

class BrokerQueryEnroute:
    @classmethod
    def __new__(cls, *args, **kwargs):
        fn = args[0]
        print("broker query")
        return fn

class BrokerEventEnroute:
    @classmethod
    def __new__(cls, *args, **kwargs):
        fn = args[0]
        print("broker event")
        return fn

class BrokerEnroute:
    command = BrokerCommandEnroute
    query = BrokerQueryEnroute
    event = BrokerEventEnroute

class RestCommandEnroute:
    @classmethod
    def __new__(cls, *args, **kwargs):
        fn = args[0]
        print("rest command")
        return fn

class RestQueryEnroute:
    @classmethod
    def __new__(cls, *args, **kwargs):
        fn = args[0]
        print("rest query")
        return fn

class RestEnroute:
    command = RestCommandEnroute
    query = RestQueryEnroute

class Enroute:
    broker = BrokerEnroute
    rest = RestEnroute

enroute = Enroute

@enroute.rest.query(url="tickets/", method="GET")
@enroute.rest.command(topics=["CreateTicket"])
@enroute.broker.query(topics=["CreateTicket"])
@enroute.broker.command(topics=["CreateTicket"])
@enroute.broker.event(topics=["CreateTicket"])
async def fn():
    ...


Make preferences menu interactive while retaining scrollback

Right now an interactive prompt appears when the Recipe Robot script is first run.
During this prompt, it would be excellent to "refresh" the screen while retaining scrollback. Right now the act of toggling recipe types on/off causes the entire list of recipes to appear again.
I believe the right way to do this is using the curses module, although I'm not familiar with it yet.
This will not affect the Mac app, which will have a separate interface for prompting for preferences.

curses can do it.
However, should it?
I think it may appear heinous to the eyes of gui users... But they're going to be using the GUI. As a die-hard CLI user, I wouldn't expect my menus to redraw the screen on me.

True, the people who care a lot about pretty menus will use the Mac app.
So you think the script prefs interface is good to go for 1.0?


$attachment->getExtension() return wrong file name extension

Hi,
I'm not sure if this is:

A bug
An intended Behavior
A documentation error
A wrong named function

The problem from my point of view is that the function return the MIME type of the file and NOT the real file name extension.
As example, I've a case where I try to get all csv file attachment from a message but my function return nothing, this happen because the file name extension is csv but the extension  type returned by getExtension() is txt (because the mime type of csv return txt) so my check fail.
I did something like this:
$attachmentList = $currentMessage->getAttachments();
foreach ($attachments as $attachment){
  if($attachment->getExtension() === 'csv'){
   //some actions
  }
}

The mime type obviously is correct because a csv file is a simple txt file with a ".csv" extension.
So, technically the result is correct (mime_content_type return "text/plain" as result) but  the file extension is not txt... so at this point I'm back to my initial consideration.. I don't know if this is really a bug or something else.
As reference for other people that can find this problem, I solved in an "old fashion way" :D
    function getAttachmentFileNameInfo(\Webklex\PHPIMAP\Attachment $attachment): array
    {

        $result = [
            'dirname' => '',
            'basename' => '',
            'extension' => '',
            'filename' => ''
        ];

        $attachmentInfo = $attachment->getAttributes();
        if (
            !array_key_exists('name', $attachmentInfo)
            && (trim($attachmentInfo['name']) !== '')
        ) {
            return $result;
        }
        return pathinfo($attachmentInfo['name']);
    }

//code that check for attachments
$attachmentList = $currentMessage->getAttachments();
foreach ($attachments as $attachment){
  if (
         !array_key_exists('name', $attachmentInfo)
          && (trim($attachmentInfo['name']) !== '')
      )
     {
                continue;
      }
  $fileNameInfo = getAttachmentFileNameInfo($attachment);
  if($fileNameInfo['extension'] === 'csv'){
   //some actions
  }
}

Hi @mdemori ,
many thanks for your report. Please update to v5.1 and give it another try.
If you are currently using an older version below v5.0, please read the breaking changes leading up to v5.1 before upgrading.
Best regards,


[stb] Update to 2021-09-10

Updates stb to 2021-09-10

Which triplets are supported/not supported?

Yes

Thanks for the update!


VAE allows only CGaussian

Can we allow VAE for AbrstactPDF and AbstractCPDF? As it is now, I can not use SharedVarCGaussian in VAE.

Would it still be a VAE if it was an AbstractCPDF? We also have:
abstract type AbstractCPDF{T<:Real} end
abstract type AbstractCGaussian{T} <: AbstractCPDF{T} end
struct CGaussian{T,V<:AbstractVar} <: AbstractCGaussian{T} end
# same for SharedVarCGaussian

so we could also make it a AbstractCGaussian. I can change it to either one.

closed with #36


Fuzzy-finder window is too small

I have pretty long paths and looking for the right file when they are prefixed the same is super hard with the current window fuzzy-finder width.

I'll close this out as its a duplicate/related to https://github.com/onivim/oni2/issues/1482, but if you have any further input on how it should look, that would be appreicated!
Also make sure to +1 the issue!

Thanks! Sorry for duplicate


How to use with explicit many-to-many models (error in Edit view due to Input)

My schema has a few models that are related via many-to-many relations. I'm using the explicit form of the relation in my prisma schema by defining the intermediate model exactly as described in the prisma docs.
As requested in the docs for ra-data-prisma, I'm calling addCrudResolvers on all 3 models (both sides of the relation and the intermediate model).
On the frontend, I'm also defining resources for all 3 models. My issues start when trying to build out the Edit page for the intermediate model. The List view works fine, but when trying to show the Edit view I'm getting a graphql error that says Error: Variable "$where" got invalid value "MODEL_ID" at "where.id"; Expected type StringFilter to be an object. This happens with both the EditGuesser and when trying to define a basic SimpleForm with only a single TextField for the id. (MODEL_ID actually shows up as the string id of the instance it's trying to fetch).
It seems that the Input type expected in the Edit view for the intermediate model is different from the Input type expected by other Edit views, including those for my one-to-many relations. The intermediate model is looking for a ${typeName}WhereInput, whereas the other models are looking for a ${typeName}WhereUniqueInput. This is causing the error since the dataProvider is passing just the id for both Inputs, but ${typeName}WhereInput seems to want a StringFilter.
Sorry for the long explanation. I'm not even sure this is the right package to be creating the issue for. If the explanation isn't clear, I'd be happy to try to create a minimal test repo that demonstrates the issue.
Thank you again for your help.

And as is so often the case, after hours of debugging I find the issue right after submitting the github issue.
It seems that the problem stems from the way Prisma recommends naming intermediate many-to-many models. If the related models are Post and Category, the recommendation is to name the intermediate model CategoriesOnPosts. Then in the backend package for ra-data-prisma, the queryAllName is auto-generated using the pluralize package. But when pluralize runs on a model name like CategoriesOnPosts, the result is ... CategoriesOnPosts! So the queryAllName clobbers the queryName, resulting in no available query for a singular object, and a graphql endpoint that thinks the Edit view is asking for many objects when it's only asking for a singular object.
I can get around this by changing the name of my model. Now that I know what's causing it, I'm wondering what the right solution is though. It seems like perhaps queryAllName should be guaranteed to be different from queryName, even if queryName is plural by default?
The challenge of "naming things" strikes again...

And as is so often the case, after hours of debugging I find the issue right after submitting the github issue.

haha, story of my life.

But when pluralize runs on a model name like CategoriesOnPosts, the result is ... CategoriesOnPosts! So the queryAllName clobbers the queryName, resulting in no available query for a singular object, and a graphql endpoint that thinks the Edit view is asking for many objects when it's only asking for a singular object.

So it should be CategoriesOnPostss ? with two s? I think that should be an easy fix.

@bennettrogers so we need to wait until that is resolved in nexus-plugin-prisma

Yes, exactly! Sorry, that's what I should have said.

upstream PR is open, not sure if it will be merged soon https://github.com/graphql-nexus/nexus-plugin-prisma/pull/804


test wasm-pack-template in this repo

related: https://github.com/rustwasm/wasm-pack-template/issues/12

plan:

install cargo-genearte
run cargo-generate --git https://github.com/rustwasm/wasm-pack-template/issues/12
run cargo build
run cargo fmt

this also makes me realize that i really want the template to live in this repo

i think this is not a priority and since we test the template in the template repo that seems fine for now.


FreeBSD support

During boot, the following crash occurs:
panic: bad APIC LVT delivery mode: 0xffffdfef
Tested today, November 21, 2016, with FreeBSD-11.0-RELEASE-i386-bootonly.iso

This works now, took some time to update the website. Also: http://copy.sh/v86/?profile=freebsd

@copy Awesome, thanks!


⚠ Samsung has degraded performance

In 492a6e2, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2554 ms

Resolved: Samsung performance has improved in 267554e.


⚠ Fetchers has degraded performance

In 7c9fc91, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 322 ms

Resolved: Fetchers performance has improved in 6110733.


Return existing PR from sync_release

This makes the implementation consistent with the docstring meaning that
a PR is returned if create_pr=True. The usage of such implementation is
easier in a sense that if create_pr is set to True and no exception was
raised, the caller can expect a valid PR to be returned which was not
the case previously and lead to surprising tracebacks.
Also make the typing stricter by using overloads for the literal
create_pr values.
Signed-off-by: František Nečas<EMAIL_ADDRESS>TODO:

[x] Write new tests or update the old ones to cover new functionality.
[x] Update doc-strings where appropriate.
[x] Update or write new documentation in packit/packit.dev. (not relevant)

Fixes https://github.com/packit/packit-service/issues/1504
(lets see if github can close issues in another repo)

Yes, sure, I will try to explain. Overloads allow you to make the type-checking system aware of relations between input arguments and return values. Previously, the return value of sync_release was expected to be Optional[PullRequest], regardless of the arguments. So if you used something like
pr = api.sync_release(<args>, create_pr=True)
print(pr.url)

mypy, would (or at least should) shout at you that the return value of sync_release may be None and that your code contains an error. However, with the changes, if create_pr=True, the return value will never be None. Hence, we add an overload which specifies this relation -- if a True literal is passed in as an argument, the return value will always be a valid PullRequest, not None. This typing is more reflective of how the method actually functions. The method variants with an ellipsis only exist to add the typing information.
The most common usage for overloads is either Optional or Union. An example of the latter can be seen here: https://adamj.eu/tech/2021/05/29/python-type-hints-how-to-use-overload/ , quite a nice article also explaining this topic. Mypy's documentation contains some more examples: https://mypy.readthedocs.io/en/stable/more_types.html#function-overloading

Thanks, I didn't notice they differ in the return type, I checked only the arguments.
Could we add a simple docstring to those overloaded methods saying that they are there just for mypy and they differ in the create_pr type arg and return type? (in your words)
Or a link to your previous comment would do as well.


Consider allowing extension types to implement non super type

Currently extension types can only "implement" super type of representation or representation type itself.
This renders example at https://github.com/dart-lang/sdk#2727 as invalid.
I think this restricts a powerful tool to just a way to hide members from a type, other than that it seems more like a simple extension.
Most obvious usage if restriction would be lifted is to use sealed hierarchy to achieve compile-only union of different types with full static analysis rather than dynamic.

If extension types are compile time only, then they cannot soundly be allowed to implement any interface that their representation object doesn't implement.
Extension types are not able to implement union types, because they are erased at runtime, and there are no union types they can be erased to which are still sound.

Agreeing with @lrhn, I'd like to add a little bit of background information.
@Zekfad wrote:

Currently extension types can only "implement" super type of representation or representation type itself.
This renders example at https://github.com/dart-lang/sdk#2727 as invalid.

I guess you meant https://github.com/dart-lang/language/issues/2727?
Thanks for catching that! The IdNumber example in dart-lang/language#2727 was indeed obsolete because the relevant rules have changed (several times ;-) since this example was written. I've just updated it such that it is again working code.
A clause like implements T in those older proposals did support values of T that weren't supertypes of the representation type. For example, we could have IdNumber ... implements Comparable<IdNumber>. That is no longer possible, because the representation type int isn't a subtype of Comparable<IdNumber> (and it is not even a subtype of the erasure Comparable<int>).
The crucial difference between the old proposals where implements Comparable<IdNumber> was allowed and the current specification where it is an error is that we have a subtype relationship in the current specification. The older proposals did not give rise to that subtype relationship, and this means that you wouldn't be able to do things like this in the old proposals:
// According to old rules.

void f(Comparable<IdNumber> comp) => ...;

void main() {
  IdNumber id = ...;
  f(id); // Compile-time error!
}

In the old proposals, implements was simply used to require that a bunch of members exist and have the expected signature. In the current specification, we actually have a subtype relationship:
extension type IdNumber(int i) implements Comparable<num> {
  bool operator <(IdNumber other) => i < other.i;
  int compareTo(IdNumber other) => i - other.i;
  bool verify({required int age}) => true; // TODO: Implement.
}

void main() {
  IdNumber id = ...;
  Comparable<num> comp = id; // OK!
}

Note that implements T creates a subtype relationship in class declarations, mixin declarations, enum declarations, etc., and we're just returning to normal business by using the new (and stricter) rules that allow us to have this subtype relationship for extension types as well. Here's the reason why we must use stricter rules:
When there is an actual subtype relationship, it must be sound to have a situation where the value of a variable like comp is the object denoted by id. But at run time the object denoted by id is an int (that's the main motivation for having extension types in the first place: the apparent wrapper object does not exist). The subtype relationship int <: Comparable<num> holds, so it's perfectly fine to specify that we also have the subtype relationship IdNumber <: Comparable<num>.
In the current specification, extension type E2 ... implements ... E1 ... has a completely different meaning when E1 is another extension type. This again creates a subtype relationship, but the main purpose of that kind of declaration is that it allows E2 to "inherit" the members of E1 that E2 doesn't redeclare (we say "redeclare" when E2 declares a member whose name is also the name of a member of E1, or some other superinterface of E2).
So, lots of things happened with implements in extension type declarations over time, and given that we want to maintain the subtype relationship from an extension type to each of its superinterfaces, there's no way we could allow a non-extension type superinterface if it isn't sound, and that property is ensured by insisting that it must be a supertype of the representation type.

I am going to close this given @lrhn and @eernstg answers.