In the navigation mode, open the "Planning" module and other related modules, and click "Start Auto", and the vehicle will not be controlled

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):
Apollo installed from (source or binary):
Apollo version (3.5, 5.0, 5.5, 6.0):
Output of apollo.sh config if on master branch:

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

Supporting materials (screenshots, command lines, code/script snippets):
Problem description：
1.We are manually recorded reference line, and then through the modified navigator.py script sent.We use version 5.5, Ubuntu 16.04.
2.When automatic driving is turned on, the vehicle is always uncontrolled and moves to the right.However, we have no problem in RTK tracing test.
3.I found some strange phenomena. I showed them with pictures.
4.It should be noted that some modules are opened through the terminal.

First of all, there is an extra green line on the map in the upper left corner, This extra green line is always flashing, not always appearing.which does not match the reference line. And the actual test found that the vehicle seems to be looking for this line, not the reference line we want. Secondly, in dreamview, there is an extra group of line segments, which are both white and yellow.

This is our channel information. I don't know if some information is missing.

This is our profile

There are always more things we don't need.
Now, I really need the official help. Thank you. Expect a reply

Is it a bug in apollo's code？

We have a similar problem. Have you solved it?

Hello, brother. Sympathize with you, I also encountered the same problem, can not solve, give up. I thought only I could meet, but you did. Hope you can solve it. Don't forget to inform me. Thank you

Is it a bug in apollo's code？

I don't know, but I'm in trouble.

We have a similar problem. Have you solved it?

NO, and I'm in trouble.

Hello, brother. Sympathize with you, I also encountered the same problem, can not solve, give up. I thought only I could meet, but you did. Hope you can solve it. Don't forget to inform me. Thank you

It's luck, man.

We have encountered a similar problem, have you solved it

We have encountered a similar problem, have you solved it

We have encountered a similar problem, have you solved it

NO, and I'm in trouble.

Supplementary note: after verification, the extra green straight line finally points to the point with zero longitude and latitude.

@liouyx @guoweiwan @JasonZhou404 Would you please help with this issue?  It seems that several developers encounter this problem.

The problem has been solved, thank you.

@yufeilongyuan It seems that you have had success running navigation_mode on a newer version of apollo (5.5). I am having issues sending navigation line data to dreamview. I collect the bag/record file by enabling these modules: relative map, localization, and record and I replay it and the map seems to move along with the speed but the car doesn't move in the replay. Then when I send the navigation line bag I get the error pictured below. I use this command: bash scripts/navigator.sh data/bag/(file). Am I doing everything correctly? Enabling the right modules to collect the bag? I followed the 2.5 tutorial and it is not useful for the newer versions. The smoother tools don't work because they rely on rosbag. How are you doing this process? Collecting and successfully sending the navigation line? What commands and modules need to be enabled? Any help/input on how you do this would be greatly appreciated as I have been stuck on this issue for a long time. I am using LGSVL simlator and version 5.0.

This is the error received when trying to send line data. The replay works fine (it seems) Thank you again


nginx: Format the config file

Motivation for this change
This makes nginx.conf human-readable.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Example output: http://sprunge.us/WKCc

Nice! What about removing some blank lines, like sed 'N; /^\s*\n$/d; P; D' ?

I feel like that should be done in the formatter, not with sed scripts. I like my config files to have some blank lines, and think the current output is probably better than removing all of them.
But in reality, I'm unlikely to write any code to fix that. The config file is readable now.

You are probably right here. nginfmt should remove extra lines (quote, neighbouring empty lines are collapsed to at most two empty lines)

What still needs doing before this can be merged?

perhaps a confirmation from @globin

I haven't had the time to test this and won't have until the weekend, but the code looks good so if anyone confirms he has tested this I'd be very happy to merge :)

I've tested it, of course... guess that doesn't count. :P

I've tested this too and already use on my servers


feat/backend: implement Backend::default()

CLOSE #38

@homu r+

:pushpin: Commit 0d4f0cc has been approved by hobofan

:zap: Test exempted - status


deps: testing new protobuf version 3.18.2 in 2.6.x branch

I believe this is to check compatibility, and we don't want to merge the change.

All tests passing. Closing.


Using Children identifer

Children appears to be a reserved word (requires identifier single quotes).
Parent,Self are like this.
If it's really a reserved word, then intellisense should include the single quotes when it suggests it.

This gives an error in the usage without single quotes, but the usage with quotes works.
With({'Children': 123}, ThisRecord.Children)

This is from list here:
https://github.com/microsoft/Power-Fx/blob/d1931623598e3f557e25fc7ad6fe25065de3b268/src/libraries/Microsoft.PowerFx.Core/Lexer/TexlLexer.cs#L99
Practically - it means intellisense should suggest these words with single quotes to make them identifiers. .


profile.name is buggy

Sometimes I get the following, when the person is open to recruiters:

Do you have to be a recruiter and are you using a recruiter account to see that info?
I don't have a recruiter account to reproduce and solve that error, please Ctrl + S the page that you are trying to scrap and send the HTML file to my email: leonardiwagner at gmail .com
ps: sorry for my late response, I added more volunteer people to handle faster the issues

Ahh, I think I found an exception. This was happening when it was my own profile (which I had set to "open to recruiters")
You can reproduce it, by going into your linkedin job settings and setting "open to recruiters" as true. Then you'll get the same response for your own profile.
Maybe the solution would be to add a gotcha in the docs, instead? This is a minor edge case, can be ignored as well (who would want to scrape their own profile?)


How to set dynamic column name in order function in gorm ?

Suppose my table name is A.
I want to sort the table records on basis of column come from request.
How should i write this in gorm query
I was trying this
Order(A['columnName'] desc)
but it was not working.
Can you please help on this, how to achieve this type of behaviour using gorm?

The document you expected this should be explained

Expected answer
Order(A['columnName'] desc)
How to make dynamic order by clause on column working

Hi @sagg1295-swiggy,
Assuming columnName is a string variable with the value of the name of the column that you want to order by, you can achieve this type of behaviour as follows,
db.Model(&User{}).Order(clause.OrderByColumn{
	Column: clause.Column{Name: columnName},
	Desc:   true,
}).Find(&result)


Fault injection fix connection error hit count

Changes included:

Increased the hit count for connection error rule correctly

each time the rule got triggered, the hit count will increase. So the most reliable way to figure out how many times the rule has applied is by using rule.getHitCount()

Based on the feedback, add log when connection close/reset is being triggered

WARN  com.azure.cosmos.implementation.directconnectivity.rntbd.RntbdRequestManager - Inject Connection close with ruleId {}
WARN  com.azure.cosmos.implementation.directconnectivity.rntbd.RntbdRequestManager - Inject Connection reset with ruleId {}

Try to capture the faultInjectionId in the diagnostics when applicable

When the connection got closed/reset, if there is no in-flight requests pending on the connection, then no requests will be impacted. But if the requests failed and retry due to connection close/reset, then the faultInjectionId will be properly populated in the diagnostics.

API change check
API changes are not detected in this pull request.

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

local test the following tests:
readAllItemsWithDetailMetricsWithExplicitPageSize
readAllItemsWithDetailMetrics
readFeedDocumentsAfterSplit
queryOrderWithTop

/check-enforcer override


VAP Profile

When I import the voice attack profile I get a looping "Not Set" statement. I've looked through the troubleshooting guide and closed issues and everything is where it should be in the apps folder under voice attack
eddi.txt

P.I.C.N.I.C.- Problem in chair not in computer - Issue resolved


infra: GitHub action that will require that any PR has either this tag or "include-in-release",

To reduce noise at the relase notes, we can create a github action that will require or add the tag to PRs ignore-for-release, so the process of adding a PR to a release note is controlled by the user, or create a mechanism that include the PR only if user add include-at-release

/kind enhancement


sticky navbar

Hey there :wave:
I think the nav bar should be sticky i.e with fixed position property on! Why I think this is because when I'm somewhere in the middle of the page and if I have to change theme or font width or something I need to scroll to the top and change it. It's kind of time-consuming and also I get lost when I try to find where I left. I tried to change CSS through dev tools and the text Joy Of Code interferes with the other content on the page but settings and Categories seem perfect! So, I would recommend hiding the Joy Of Code text when the user scrolls down but the rest of the 2 buttons should be visible! By the way, this is my first issue ever opened on Github, so if I did something wrong, I apologize In advance.
Thanks

I agree! I'm going to add it when I have more time because it's more complicated than using position: sticky because of how it's implemented.
Congrats on your first issue! 🎉 I should probably enable GitHub Discussions for these sort of things.
Thank you!

btw position:fixed; width:80% or how much u want with some high z-index value and top :4px or whatever value u like
works like a boss...
i tried using position : sticky but navbar behaved very weird.
well at the end its all up to u


ZEVA - Review and correction of email notifications

Description:
The following issues have been noted in the email notification issued from the ZEVA system.

Analysts and Director incorrectly receive notifications in the following situations

When a comment is added to a credit application and saved.
When a credit application is saved by a policy analyst
When a comment is added to a model year report and saved
When a model year assessment is edited and saved by a policy analyst

BCeID user do not receive email when they are issued a model year assessment or reassessment, even when they have all email option selected.

BCeID users do not get notifications when their ZEV models are approved.

Developer Requirements:

[x] Ensure that notification emails go out only on status changes and not on saving events with no status changes.
[x] On comment save notifications should not be sending out
[x] Fix the bceid notification for model year assessment or reassessment to properly send when those events fire.

Notes:
In principle users should receive notifications when the status of an activity changes. For example when something is submitted into the system, or issued from the system, or recommended to the Director.
Attached are the selection for email available to the IDIR Policy Analyst role and the BCeID role. The ones marked by a red dot are not working currently.

Testing has shown that users are still not getting email when Assessments are issued by government.

1 looks good but 2 and 3 still seem to be an issue

Can't test emails for an assessment being issued as there is an error preventing the Director being able to issue an assessment.

BCeID should get email when ZEV models are validated. This is not happening.

Not possible to test this, but there has been another card created to properly test. The new card is 1523.


FTL ProcessRPCBlockByHeightTXs: TX cannot be parsed from block 865. Err: unable to resolve type URL /akash.provider.v1beta1.MsgCreateProvider

FTL ProcessRPCBlockByHeightTXs: TX cannot be parsed from block 865. Err: unable to resolve type URL /akash.provider.v1beta1.MsgCreateProvider

The same process described in this comment needs to happen here and in our Lens fork:
https://github.com/DefiantLabs/cosmos-tax-cli/issues/383#issuecomment-1506164709

moving this to the backlog


[hotfix][docs] Fix the misspelling of 'feeback' to 'feedback' in the page 'docs/dev/table/streaming/match_recognize.md'

What is the purpose of the change
Fix the misspelling of 'feeback' to 'feedback' in the page 'docs/dev/table/streaming/match_recognize.md'
Brief change log

Fix the misspelling of 'feeback' to 'feedback' in the page 'docs/dev/table/streaming/match_recognize.md'

Verifying this change
[hotfix] This change is a trivial rework / code cleanup without any test coverage.
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, Kubernetes/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

Hi, @wuchong .
I made a hotfix. Please take a look.
Thank you.

Oh man, this is embarrassing. Thank you! Merging  . . .

fixed in master: 32684a860a85b78da4ccc383fb7e02596b9697e2
and release-1.11; 6a5275b67d881a6e42ee66f655e37aaca139f560


[stable/kube-ops-view] Upgrade redis and also enable when testing

Signed-off-by: David J. M. Karlsen<EMAIL_ADDRESS>What this PR does / why we need it:
The optional Redis child-chart was very outdated - update it and also enable test.
Check in the lock file to ensure reproducability.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

This change is

/assign @cpanato

/lgtm


Theano to Tensorflow backend with correct dim ordering

Current solution in the wiki page for switching the backend for pretrained models is limited to fixed dimension ordering. For an Example, If you trained a model with Theano backend and used 'th' ordering, you have to preserve the dimension ordering as 'th' when you switch to Tensorflow. What is the efficient and correct way to convert model fully to another backend with correct ordering ?
Example:   dim_order = 'th' backend = theano  --> dim_order = 'tf' backend = tensorflow

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps
[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps
[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Update 1 : I am closing this issue until there is a better solution for it. Here is the steps I followed to convert fully a pretrained model.
1 ) Adjust your ~/.keras/keras.json  dim = 'th' backend  = 'tensorflow'
2) load your model and convert it as suggested in wiki link
3) iterate over your layers  and get transpose of the Conv layers with numpy.transpose() and save them.
4) Adjust your ~/.keras/keras.json  dim = 'tf' backend  = 'tensorflow'
5) Load your model with empty weights
6) Iterate over your layers and fill with your transposed weights using model.get_layer(layer.name).set_weights(weight) function
7) save your model with model.save_weights()

@rcmalli This solution works if you only have Convolutional layers and no Dense layer for the final classification.
According to @fchollet (in the google group answer) :
Besides converting (transposing) the kernels, you need to:
- dimshuffle the kernels
- shuffle the rows of the weights of the Dense layer you put after your conv layers, to match the dimshuffle you just did

That's it. That's how Keras applications were generated (in particular the VGG16 and VGG19 models).

However, I can't think of a proper way to actually perform the second step (shuffle the rows of the first dense layer after the last conv layer).
@fchollet, if possible, could you provide a code snippet to demonstrate ? If not, could you provide a gist of the script you used to convert the VGG 16 / 19 weights (from theano to tensorflow or vice-versa) ?


Updated with AutoViz_Pipeline

Rick and Team: As promised, I have created an AutoViz_Pipeline for orchest.io
Please review and approve.
Thanks
Ram

Description

Here is my example!
Checklist

[x] I have read the README file and understand the required format for a contribution.

Please feel free to make changes to it if I have made any mistakes.
Thx
Ram

Really awesome! Merging


Support gliding setpoints for autonomous soaring

Describe problem solved by this pull request
There is currently no way in px4 to enable the vehicle to "glide" along the setpoint.
Describe your solution
This PR enables a gliding mode that can be activated through a setpoint_type mask, which configures TECS to prioritize kinetic energy and thus enabling the vehicle to soar along it's glide slope, while loitering around its setpoint.
This is needed for autonomous soaring applications, where the fixed wing vehicle needs to soar around a setpoint without activating any throttle.
Test data / coverage
A test code can be found in https://github.com/Jaeyoung-Lim/mav_soaring
Video:

Log: https://review.px4.io/plot_app?log=35a0647d-f69f-4a19-bfe1-ea66d099a9a1
Additional context
Upstream PR: https://github.com/PX4/Firmware/pull/14643

The Upstream pull request has been now merged, therefore closing


String 2 Int

https://stackoverflow.com/questions/24115141/swift-converting-string-to-int
let myString = "556"
let myInt = Int(myString)

i see


Card holder empties all card when i put it in a sop backpack

LaserIO version
1.6.7
Minecraft Version
1.20.1
Forge Version
47.1.76
Modpack & Version
dw20 1.9.2
Do you have optifine installed?

Expected behaviour
still have my cards
Screenshots
none
Log files
none
Additional information
none

I've had the odd report of this happening occaisionally, but I can't narrow down a reason. I placed my card holder in a sophisticated backpack plenty of times in my lets play series, and never had this happen. It may be some other mod causing a conflict? Or maybe it'll be fixed in 1.21? Let me know if it still happens in the new minecraft version :)


[9.0] Source-build issue in stage 2 build

Originally created in source-build repo: https://github.com/dotnet/source-build/issues/4641
Stage 2 build is failing in vstest with the following error:
    /vmr/src/vstest/src/Microsoft.TestPlatform.PlatformAbstractions/common/Tracing/PlatformEqtTrace.cs(87,5): error IDE0032: Use auto property (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0032) [/vmr/src/vstest/src/Microsoft.TestPlatform.PlatformAbstractions/Microsoft.TestPlatform.PlatformAbstractions.csproj::TargetFramework=net9.0]

Test build (internal link): https://dev.azure.com/dnceng/internal/_build/results?buildId=2550295&view=logs&j=3121c41d-6088-53a5-4ef6-9a86f3460fe9&t=334539f4-dcef-5cad-a240-02997b3146e7
This can be reproduced in vstest repo builds by updating SDK to the latest 9.0 build.
cc @nohwnd

There's another vstest issue: https://dev.azure.com/dnceng/internal/_build/results?buildId=2551180&view=logs&j=3121c41d-6088-53a5-4ef6-9a86f3460fe9&t=334539f4-dcef-5cad-a240-02997b3146e7
    /vmr/src/vstest/src/Microsoft.TestPlatform.CoreUtilities/Output/ConsoleOutput.cs(15,5): error IDE0032: Use auto property (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0032) [/vmr/src/vstest/src/Microsoft.TestPlatform.CoreUtilities/Microsoft.TestPlatform.CoreUtilities.csproj::TargetFramework=net9.0]

I'm not sure why the build is only reporting some of them at the time.
@nohwnd

pushed fix for the other one as well. Please keep reporting them.


WebSocketDelegate no longer has websocketDid* methods when going to Version 4.0.4

i was using Version 3.1.0 and upgraded to version 4.0.4 (the latest) and WebSocketDelegate no longer has the following methods:

websocketDidConnect
websocketDidDisconnect
websocketDidReceiveMessage
websocketDidReceiveData

The WebSocketDelegate instead has a single didReceive(event:, client:) method.
Were the original methods dropped? Also, there appears to be a fork of this project under HypeHub/Starscream which does have the original methods.
i am using Carthage and my Cartfile has the following:
github "daltoniam/Starscream"

@damacster  yeah, it's right on the README for Starscream:
use .binary case instead of didReceiveData
use .disconnected instead of didDisconnect
use .text instead of did didReceiveMessage
use .connected instead of didConnect
and so on and so forth.
read up on https://developer.apple.com/documentation/network/nwconnection/2998558-betterpathupdatehandler for the .reconnectSuggested case
read up on https://developer.apple.com/documentation/network/nwconnection/2998578-viabilityupdatehandler for the .viabilityChanged case
error should be obvious, not really sure about the cancelled case and how that differs from disconnected and error🤷♂
func didReceive(event: WebSocketEvent, client: WebSocket) {
	switch event {
	case .connected(let headers):
		isConnected = true
		print("websocket is connected: \(headers)")
	case .disconnected(let reason, let code):
		isConnected = false
		print("websocket is disconnected: \(reason) with code: \(code)")
	case .text(let string):
		print("Received text: \(string)")
	case .binary(let data):
		print("Received data: \(data.count)")
	case .ping(_):
		break
	case .pong(_):
		break
	case .viabilityChanged(_):
		break
	case .reconnectSuggested(_):
		break
	case .cancelled:
		isConnected = false
	case .error(let error):
		isConnected = false
		handleError(error)
	}
}

@mmdock Thanks so much!


support bootstrap3?

my project use bootstrap3, the address is //cdn.bootcss.com/bootstrap/3.3.5/css/bootstrap.min.css,
but i

it's not use?
I found that css not have "input-append"

i


No Web Browser Found

On Ubuntu 16.04 I get the following:

You can ignore that. Just open a webbrowser and go to http://localhost:8888

@louwers Awesome, thanks didn't realize it would work regardless.


Why ViewModel has no access to presenter? How can I access to individual label?

This is ridiculous, I have spent TWO WEEKS because I'm stuck with this framework of not being able to customize many component. Such as DYNAMICALLY changing the textmessagebubble based on whether the incoming message belongs to certain parties.
Why do you make such framework which is so hard to access individual component?
If is so, then why can't you make simple stuff like that customizable?
Even changing the AvatarImageView I had to spend 2-3 hours to get it done because the Demo sample do not provide DIRECT access to it, you actually have to copy and paste some class and then modify it.

@shungz89 Imagine, that incoming messages from one party should have blue bubble and borders, incoming messages from the second party should have red bubble and no borders and incoming messages from the third party one shouldn't have bubble and borders at all. Instead of trying to use the same type of message to display messages from certain parties you can try to use different type of messages for every party involved. At this point all the difference will be at configuration point. So, in this case you may have three message types with three presenter builders that only differs in styles that were used.
Let me show you an example:
override func createPresenterBuilders() -> [ChatItemType: [ChatItemPresenterBuilderProtocol]] {

    let textMessagePresenter = TextMessagePresenterBuilder(
        viewModelBuilder: DemoTextMessageViewModelBuilder(),
        interactionHandler: DemoTextMessageHandler(baseHandler: self.baseMessageHandler)
    )
    textMessagePresenter.baseMessageStyle = BaseMessageCollectionViewCellDefaultStyle(
        colors: BaseMessageCollectionViewCellDefaultStyle.Colors(
            incoming: .blue,
            outgoing: .blue
        )
    )

    let anotherTextMessagePresenter = TextMessagePresenterBuilder(
        viewModelBuilder: DemoTextMessageViewModelBuilder(),
        interactionHandler: DemoTextMessageHandler(baseHandler: self.baseMessageHandler)
    )
    anotherTextMessagePresenter.baseMessageStyle = BaseMessageCollectionViewCellDefaultStyle(
        colors: BaseMessageCollectionViewCellDefaultStyle.Colors(
            incoming: .red,
            outgoing: .red
        )
    )

    return [
        DemoTextMessageModel.oneChatItemType: [textMessagePresenter],
        DemoTextMessageModel.anotherChatItemType: [anotherTextMessagePresenter],
    ]
}

In this example I've registered two message types: one of them displays text with blue bubbles, another one displays text with red bubbles. Your model is responsible for defining its type taking into account certain parties. This is the main idea. As you can see, you can do dynamic behaviour in different ways.

Why do you make such framework which is so hard to access individual component?
If is so, then why can't you make simple stuff like that customizable?

There should be strict layer separation that is a good engineering practice and gives your lots of benefits. A View shouldn't know about anything and provide a way to customize itself through ViewModel. In case of this framework ViewModel doesn't have a business logic and it's stores data for a View. A Presenter is responsible for cell configuration and having business logic. It configures a cell through setting up a ViewModel with specific values based on your business logic. Answering your questions from the title, you shouldn't try to access a Presenter from a ViewModel because it inverse relationships between them, also you shouldn't access UI elements from a Presenter because your jump between layers in this case. Instead, you should  cusomize your ViewModel in a Presenter to configure a View.

Okay, is a good idea on the creating two items type, but how do you create two types of item?
For my requirements, I would need to access individual message label because I need to change it into html attributed text. Without access to the UILabel it is not easily doable.

@shungz89

Okay, is a good idea on the creating two items type, but how do you create two types of item?

Imagine you have a model class in your project that represents a message. It must confirm to the most basic ChatItemProtocol that defines a type variable. So, value of this property defines a type of the message. This value can be dynamic and depend on other properties of you message. For example:
class Message: ChatItemProtocol {
    let isSystemMessage: Bool

    // MARK: - ChatItemProtocol
    var type: String {
        if self.isSystemMessage {
            return Message.oneType
        } else {
            return Message.anotherType
        }
    }

    static var oneType: String = "oneType"
    static var anotherType: String = "anotherType"
}

After that you can register two different presenter builders for Message.oneType and Message.anotherType. This will allow you to display messages with different types using the same model.

For my requirements, I would need to access individual message label because I need to change it into html attributed text. Without access to the UILabel it is not easily doable.

The issue with attributed text was described here: https://github.com/badoo/Chatto/issues/462
TLDR: Currently, attributed strings are not supported, but you can add support in your project on your own by copy-pasting and modifying classes responsible for displaying text messages (https://github.com/badoo/Chatto/issues/462#issuecomment-372590898)


fix(production): Updating to docker container v1.8.4

:tada: This PR is included in version 1.8.5 :tada:
Your semantic-release bot :package::rocket:


Rework server extension interface to Jupyter Server

One of the changes in Jupyter Server is the way server extensions are discovered and installed.  Although there is sufficient backward compatibility, the plan will be to eventually deprecate the use of things like _jupyter_server_extension_paths - which Elyra uses - and we should adopt the correct approach.

This issue should be resolved in the jupyterlab3 branch via commit https://github.com/elyra-ai/elyra/commit/25a7c1895982e802ecd1c702f7990a254b39637e


Independent numer/denom for ratio measures

Summary
Addresses https://github.com/projecttacoma/fqm-execution/issues/182 (single IPP ratio measure numerator/denominator populations should be treated independently in fqm-execution).
New behavior
Fqm-execution currently does not align with HQMF in the fact that ratio measure calculation should always treat numerator and denominator as independent populations. In fqm-execution, this currently only happens when the measure has multiple IPPs.
Fqm-execution now handles the numerator and denominator populations as independent populations for all ratio measures (so, including single IPP).
For example, we should now be able to achieve the following populationResults, meaning the numer is not automatically set to be false if the denom is false:
 "populationResults": [
          {
            "populationType": "initial-population",
            "criteriaExpression": "ipp",
            "result": true,
            "populationId": “ipp-id”
          },
          {
            "populationType": "numerator",
            "criteriaExpression": "numer",
            "result": true,
            "populationId": “numer-id”
          },
          {
            "populationType": "denominator",
            "criteriaExpression": "denom",
            "result": false,
            "populationId": “denom-id”
          }
        ]

Code changes
Overall change: in ClauseResultsBuilder and DetailedResultsBuilder, check that the measure is a ratio measure (rather than checking that it has multiple IPPs) in spots where there is an assumed dependence between the numer and denom populations.
Note that the scoring code can occur at both the group and measure root level, so we should check for the presence of a scoring code in both spots. The measure scoring code is passed in to these functions as a fallback in case the scoring code does not appear at the group level. Open to feedback on a better way to pass in the measure scoring code… the only spot that I could see was passing it in through calculate() since the ClauseResultsBuilder/DetailedResultsBuilder  take in the measure group as input rather than the entire measure.
Code changes to focus on:

Update buildPopulationRelevanceMap to check if measure is a ratio measure when deciding whether to set NUMER/NUMEX to false if DENOM is false
Update buildPopulationRelevanceMap to check if measure is a ratio measure when deciding whether to se NUMER/NUMEX to false if DENEX is true
Update handlePopulationValues to do similar checks based on scoring code as those in buildPopulationRelevanceMap
Add unit test in DetailedResultsBuilder for checking numer/denom independence for single IPP ratio measure (plus some rearranging of unit tests - sorry the code diff is really ugly)
Update unit tests in ClauseResultsBuilder to take in the new argument for measure scoring code

Testing guidance

Run unit tests
Run detailed results against patient/measure bundle in https://github.com/projecttacoma/fqm-execution/issues/182 - on the main branch the numerator should false out, but on this branch the numerator should stay as “true” in the populationResults
Run detailed results against the same patient/measure bundle but change the scoring code to “proportion” instead of “ratio” - the numerator should false out since it is dependent with the denominator (since this only applies to ratio measures)

One overall thing I noticed though was there don't seem to be any unit tests for testing numer/denom independence when there is only 1 ipp and its a ratio measure. This should maybe be something we add unless it's there and I just missed it.

They are there (they probably got overshadowed by the rest of the code diff since I made a lot of little changes surrounding them) - the unit tests are in a describe block at the bottom of DetailedResultsBuilder.test.ts: 'handlePopulationValues for group with single IPP - ratio measure'


Upgrade to Truth 1.4.2.

This requires one code change to address a newly ambiguous
call.
It's also a good opportunity to remove the dep on
truth-java8-extension, which has been unnecessary since
1.3.0 moves its
classes into the main Truth artifact.

We don't build off forks anymore (which means we can't merge either since the tests never run).
It LGTM any chance you can push it to a branch on the main repo and create a new PR?

Thanks.
When you talk about pushing to the main repo, are you talking about to verygoodsecurity/starlarky? I don't think I have permissions for that, at least based on what I'm seeing from gh and git at the command line.

Ah, I was afraid of that. Let me check what process we want and I'll get back to you.

I was able to move it here https://github.com/verygoodsecurity/starlarky/pull/463

Merged into master via #463

Thanks! I don't know if my PR actually made things any easier than they would have been if I'd just left it to you to make the one-line change yourself... :)

It was on my list to look at. We use Dependabot to keep the deps updated but that one was failing. Annoying when there are breaking changes for sure.

Annoying when there are breaking changes for sure.

Yeah, I can identify with that. We won't be making a habit of it for Truth.


[SPARK-41781][K8S] Add the ability to create pvc before creating driver/executor pod

What changes were proposed in this pull request?

This patch aims to make PVCs as pre-resources and setup them before driver/executor Pod creation. After this patch, the workflow looks like this:

setup pre-resources (including PVCs) before Pod creation
create Pod
refresh all pre-resources' owner references as created Pod
make the created Pod an owner of other resources and create them

Why are the changes needed?

These pre-resources are usually necessary in Pod creation and scheduling, they should be ready before Pod creation, should be deleted when user delete Pod, the lifecycle of these resource is same with Pod.
If PVCs are created after Pod creation, we will have warning event in Pod as follow:
error getting PVC "spark/application-exec-1-pvc-0": could not find v1.PersistentVolumeClaim "spark/application-exec-1-pvc-0"

Does this PR introduce any user-facing change?

No.
How was this patch tested?

Add new UTs.

Can one of the admins verify this patch?

@dongjoon-hyun Sorry for my late reply. I'm working on minimize intrusive code :)

@dongjoon-hyun
I'd like to discuss two options with you :)

First: Refer to the idea of SPARK-37331, I try to move PVC from resources to pre-resources, and set up before driver and executor creation. In future, we could set ConfigMap as pre-resource because there is same warn information about resource not found from k8s scheduler. (This is how this PR works.)
Second: Leave PVC as resource, filter PVC from resources and setup it before pod creation.

I prefer the first option. First option has more scalability, we could set other k8s resource as pre-resource if need. And I agree with you about code intrusiveness. I think it's worth setting PVC as pre-resource.
Any suggestion is greatly appreciated.

@dongjoon-hyun

According to the updated PR description, is the only benefit of this refactoring is to remove warning K8s event messages?

This PR brings two benefit:

Remove unnecessary warning k8s event messages. We collect events as alarms, and it is currently impossible to determine whether this event is caused by insufficient storage resources or resource creation order.
In some schedule scenarios, the problem that webhook cannot find PVC based on Pod Spec can be fixed, because old logic is to create PVC after Pod creation.

Does this PR work with the existing feature like SPARK-41515 ?
I'm working on it. Let me check

Thank you for the reviews @dongjoon-hyun , I believe I've addressed your comments! Tomorrow is also the Chinese New Year, I wish you a happy Chinese New Year.

gentle ping @dongjoon-hyun :)

@cometta Hi, this PR is not related to the problem you're having. Could you create an issue in JIRA, attach your spark configuration? I will follow up :)


Ignore more directories

Is your feature request related to a problem? Please describe.
Currently some directories that shouldn't contain any Ruby files aren't ignored by default:

log
public
storage

These may contain a large number of subdirectories and files (especially storage) which can drastically slow down RuboCop.
Describe the solution you'd like
I think we should ignore these directories by default because it's time consuming to understand what's causing the slowdown.
Describe alternatives you've considered
Alternatively a warning message should be displayed when RuboCop scans more than X non Ruby files in a directory.
Additional context
I can work on a PR.

That makes sense. I've opened #797.


An attempt was made to access a socket in a way forbidden by its access permissions

What version of Dapr?
v1.1.1
Actual Behavior
Everything worked like a charm after applying the latest windows updates.
I'm using dapr locally on my dev machine. If I try to start dapr on a specific port e.g.
"tasks": [
  {
    "appId": "my-service",
    "appPort": 6969,
    "httpPort": 51002,
    "grpcPort": 52002,
    "placementHostAddress": "localhost:6050",
    "componentsPath": "./dapr/components",
    "metricsPort": 9090,
    "label": "daprd-appcontext",
    "type": "daprd",
    "logLevel": "info"
},

I get the following exception level=fatal msg="listen tcp4 :51001: bind: An attempt was made to access a socket in a way forbidden by its access permissions." app_id=kratos-admin-api instance=GreenHornet scope=dapr.runtime.http type=log ver=1.1.1 and dapr terminates.
Actually I don't think it's a Dapr issue in general but more related to a Windows kernel update or the Go standard lib implementation. But I'm not an expert on either.
Steps to Reproduce the Problem
Install KB5001330, KB5000802 on a Windows Kernel v. 2004.
We've encountered a similar issue within kubernetes Win2004: kube-proxy is not creating HNS endpoints, I'm not sure if it's the same root cause, but somehow it seems related to me.

Hi @quickstar it seems hyper-v already reserves that port and you could refer to this solution
https://github.com/docker/for-win/issues/3171#issuecomment-459205576

I didn't change any thing in particular... it just startet to work again 😕


Use bsf instead of tzcnt where mask != 0

For when we know we aren't going to put in a mask of 0, we can use this and get better codegen.
There actually isn't anywhere we actually care about mask == 0 as we already check for that case to end the loop ourselves.

This kind of looks like it's changing something for the sake of changing something, I don't really like functions with undefined behaviour in some cases in the first place and saving two instructions in these places won't really affect performance.

Then can we change tzcnt to use the builtin with a check instead of inline asm? That gets optimized away on GCC.

Can't we just implement a proper iterator rather than doing some weird hacks? Like https://gcc.godbolt.org/z/v7M6oY4M6

here's a branch that uses @doitsujin's iterator https://github.com/doitsujin/dxvk/compare/master...DadSchoorse:bititer


Please provide script for Testing

In README.md, it is written to use bash test_en_de2.sh for testing. But no such file is there

Hi, thanks for your attention.
Now, the evaluating scripts for ende and enzh directions are uploaded.


Iframe cross origin issues on Firefox

I've been testing the preview box on Firefox, and it shows the following warning and the preview doesn't work.

I'd suggest to use Chrome before making further progress.


Add "Did You Mean This" Extension

Replacing the previous PR https://github.com/Azure/azure-cli-extensions/pull/1536
New extension (in preview) which uses the Aladdin service to generate and show failure recovery recommendations on command failure. This extension makes use of the hook added to parser.py in CLI version 2.4.0. For more information, see this CLI PR.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[X] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/christopher-o-toole/azure-cli-extensions.git@thoth-extension#subdirectory=src/$EXT&egg=$EXT"

add to S170

/azp run


Question: How do I send an update to a DNS server to update a CNAME and Arecord ?

m := new(dns.Msg)
r, _ := dns.NewRR("foo.company.net. 3600 A <IP_ADDRESS>")
m.WHATDOIDONOW(r)

I'm at a lost, I tried most of the functions on dns.Msg and can't find anything that doesn't end up in an error. Real newbie here (re dns not Go) so if you just want to point me to a document that explain DNS that would be fine,

RFC 2136 and then the Dynamic Updates section of the docs would be a good place to start. Let us know how you get on.

Ok I've read a bit and understand a tiny bit more
m := new(dns.Msg)
m.SetQuestion("platform.company.net", 3600)
r, err := dns.NewRR("foo.platform.company.net. 3600 A <IP_ADDRESS>")
m.Ns = []dns.RR{r}
resp, _, err := c.Exchange(m, "int-ddns01.resource.company.net:53")
w.Write([]byte(resp.String()))

(error handling omited for brievety)
I'm getting a domain must be fully qualified. My problem might be in the Question (which is zone when doing dynamic updates) but I can't tell what's wrong

ok first problem resolved, I need to add a period after the fqdns

Ugh, apparently it'll take a while for my $WORK DNS to update, can someone confirm that the code looks correct ?

The status: FORMERR in the response means the server rejected the query. In this case that's probably due to:

m.SetQuestion("platform.company.net", 3600)

The second parameter to SetQuestion is a Type and must be dns.TypeSOA for an Update query.

m.Insert([]dns.RR{r})

Keep in mind that this is additive and does not replace existing records.
Regarding update times, it's unusual for the server that accepts Updates to not immediately reflect changes. That's not to say that all the authoritative servers will immediately reflect the change, just that the one that accepts the Update will generally reflect the change immediately.

changing for TypeSOA gives me a slightly different result but still Format Error (and not Forme RR as I originally thought)
;; opcode: QUERY, status: FORMERR, id: 2825
;; flags: qr rd; QUERY: 1, ANSWER: 0, AUTHORITY: 0, ADDITIONAL: 0

;; QUESTION SECTION:
;platform.company.net.	IN	 SOA

I tried with nsupdate(8) and that worked just fine

Ok eventualyl figured out everything, ty


fix(healthcheck): Don't fail if there is no healthcheck for a proctype

Current coverage is 5.36% (diff: 0.00%)

Merging #173 into master will increase coverage by <.01%

@@            master      #173   diff @@
========================================
  Files           46        46
  Lines         3286      3281     -5
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           176       176
+ Misses        3088      3083     -5
  Partials        22        22

Powered by Codecov. Last update 55247cf...4f2d342


Allow supressing _id for subdocuments

Fixes #89

Coverage increased (+0.1%) to 87.009% when pulling b60d4312f3d690d04d4d501e75768e848767b87c on allow-suppressing-ids-for-subdocuments into 2ffe774a1fd357fd35a1ed79322fa581c262b866 on master.

Coverage increased (+0.1%) to 87.009% when pulling b60d4312f3d690d04d4d501e75768e848767b87c on allow-suppressing-ids-for-subdocuments into 2ffe774a1fd357fd35a1ed79322fa581c262b866 on master.


Rewrote tool a little bit

This was mostly done in response to #14, I took the time to add IPv6 support as well

The one issue that is left to address is the fact it holds a lot of file handles open at once if the pcap file contains a lot of connections.

I think the easiest solution is to "batch" writing to each handle in streaming::write_pcaps(), where e.g. it opens the file for writing once every 10/100 packets (and of course, once we're done reading from the source)  before it once again closes it.

 @Pommaq let me know if you have any issues with it. thank you for the improvement PR! 🎉


chore: bump chart versions

Is your feature request related to a problem? Please describe.
Since we have done the initial commits, several of the charts we depend on have gone to newer versions. We need to bump and test these.
Describe the solution you'd like
Update to chart versions that we use as defaults.
Describe alternatives you've considered
None.
Additional context
None.

Closed by #89


In mode "all" combat should not be engaged unless there are PokéBalls available.

Expected Behavior
Don't engage combat unless the Pokémon could be caught.
Actual Behavior
Endless loop (Until pokemon despawn i presume)
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] Something rustles nearby!
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] Something rustles nearby!
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] Something rustles nearby!
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] Something rustles nearby!
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] A Wild Drowzee appeared! [CP 131]
[x] Out of pokeballs...
[#] Something rustles nearby!

Steps to Reproduce
Run the bot for long enough for it to run out of PokéBalls, then it will enter a loop.
Other Information
OS: Ubuntu 16.04
Git Commit: 345ccb8f7b1e9609a768b9d0fc7d2754b0d86f92
"mode": "all"

same problem here with "mode": "farm" ,  dev commit: 6199d49
[#] A Wild Pidgey appeared! [CP 60] [x] Out of pokeballs... [#] A Wild Pidgey appeared! [CP 60] [x] Out of pokeballs... [#] Something rustles nearby! [#] A Wild Rattata appeared! [CP 125] [x] Out of pokeballs... [#] A Wild Rattata appeared! [CP 125] [x] Out of pokeballs... [#] Something rustles nearby! [#] A Wild Pidgey appeared! [CP 60] [x] Out of pokeballs... [#] A Wild Pidgey appeared! [CP 60] [x] Out of pokeballs... [#] Something rustles nearby! [#] A Wild Rattata appeared! [CP 125] [x] Out of pokeballs... [#] A Wild Rattata appeared! [CP 125] [x] Out of pokeballs...

Closed in https://github.com/PokemonGoF/PokemonGo-Bot/pull/333


Fix 10bit video display issue

This change is to correct plane number for
P010 format. The expected plane number should
be 2, but we got 3, which caused no image created
for P010 display.
Tracked-On: OAM-100284
Signed-off-by: Chen, Tianmi<EMAIL_ADDRESS>

@feijiang1 , @renchenglei , @dyang23 , @YuanjunHuang , could you please have a review?
Thanks,

@TianmiChen
Can you help add FLAG "INTERNAL:" to tiltle? Plz refer to:
https://github.com/projectceladon/external-mesa/pull/40/commits

@TianmiChen Can you help add FLAG "INTERNAL:" to tiltle? Plz refer to: https://github.com/projectceladon/external-mesa/pull/40/commits

Updated. Thanks for the heads up. Please help review again.

Thanks @TianmiChen
LGTM!


[JTC] Change clock_type for tests and further bugfixes

I extracted changes to the JTC tests from #558 and #609, which are not directly linked to the behavior changes being introduced there.

While implementing the action tests for #558, I struggled with different time sources and c++ exceptions of the type  can't compare times with different time sources. Now consequently RCL_STEADY_TIME (introduced with #427) is used everywhere for the tests, also as clock source for the node itself. I had to change the trajectory class as well because I found no other way to get the correct time source for trajectory_start_time_ for every case in the tests -> ABI breaking
has_active_trajectory etc will be used in tests of #558 and #609 later on.
I think the logic behind test_ignore_partial_old_trajectory was wrong: It should accept the new goal and end up with the last of the new points.

CI tests are failing due to the admittance controller issues.

@christophfroehlich , I am sorry that I could yet find time to work on my own attempt of fixing the clock.
I encountered the compare time issue when trying to move to time+dt sampling, because at new goal, there must be an initialization of time (start_time is set to the time sent by the update in the first call to sample), but this could be a problem if the previous initialization of the trajectory_msgs was with a different clock.
The compare time issue was happening there
On my side I fixed it with a time initialization AFTER set_point_before_trajectory_msg is called. Then the comparison error disappeared (without changing clocks), because the time_before was then initialized with the time passed by update
I obviously did not dig into your issue, maybe my info gives you hints.
Personally, I believe one should not need clocks to be passed around. The only time one needs to compare time that might be from different clocks is when a new goal is validated (https://github.com/ros-controls/ros2_controllers/blob/148687d51d6651ef4c9879110a627b5444fe1a0a/joint_trajectory_controller/src/joint_trajectory_controller.cpp#L1301) to be in the future or in the past isn't it ? but not in update, when we go in update, the start_time is using the time past from update (at first sample and next ones)
Where exactly did your compare time issues occur ? (what line causes that exception ?). I had to use some log error to find where, because I did not match to set breakpoints when called with pytest.

@AndyZe introduced RCL_STEADY_TIME for good reasons I think. The current master branch is ok, but adding new tests always introduces problems with time sources.
The last one which I could not fix other than passing it to the trajectory::update method was:
If time_from_start is zero, the update method sets trajectory_start_time_ from sample_time having the correct clock type.
https://github.com/ros-controls/ros2_controllers/blob/8436ed549becce0b694dfe23cfa053436f2b6d42/joint_trajectory_controller/src/trajectory.cpp#L80
But if it is set to a specific time, e.g. with test_ignore_old_trajectory, it is set with the operator= of rclcpp::Time
https://github.com/ros-controls/ros2_controllers/blob/8436ed549becce0b694dfe23cfa053436f2b6d42/joint_trajectory_controller/src/trajectory.cpp#L56
and there the clock source is RCL_ROS_TIME causing the c++ exception here:
https://github.com/ros-controls/ros2_controllers/blob/8436ed549becce0b694dfe23cfa053436f2b6d42/joint_trajectory_controller/src/trajectory.cpp#L98

thanks for clarifying, it is exactly at the same spot, and indeed, I did not try for a time different than zero. Instead of passing the clock type to update, isn't there a way to pass it once only, and use it for the rest of the existence of JTC ? clock types are usually at start thing (sim or walltime typically).
In any case, I read too fast yesterday night and you don't pass the clock but the clock type which is probably fine and in theory, this should still permit me to maybe get rid of the steady_clock completely (now needed due to the way state_msg is passed around that only works with steady_clock)

I wanted to give a hint why I had to change the clock_type of the node, which introduced the error described above with the tests from test_trajectory_controller.cpp. I reverted the changes, and guess what? No exceptions this time... I don't know how much time I spent yesterday on that with clean rebuilds etc. Now it just works without.
If I can't reproduce it, I'll close this PR and make a minimal one without the clock stuff.

 I feel your frustration. Could it be that you were in a situation with some race condition that made the first point (the one before msg) to not be initialized yet when doing the comparison ?

I will really try to reserve some time soon to progress on a PR for the state_msgs to not be used (except in one test which evaluates that everything it passed fine there).

I can't reproduce the c++ exceptions now, which made the changes to the clock type necessary. I tried rebuilds with different build types (Release, Debug,..), but it does not occur any more. Saving this PR for later, if it occurs again.


Duplicated product id when copying new products

Describe the bug
Currently, it is possible to create products with the same product id (see PM_10).
To Reproduce
Steps to reproduce the behavior:

Create a new product
Copy the new product
Change e.g. the price
Observe that the copied product has the same id

Expected behavior
The product id should be unique
Additional context
Make the id also unique in the database as additional fail safe

 Duplicated product id when copying new products


nextinspace: init at 1.0.4

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

You can just update your old PR and do not need to open a new one every time.


chapter6-added

- Summary

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Added Chapter 6


[WIP] Star in nested function

Description
WIP for * in nested function
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Needs to be reworked. Will follow up with another PR.


Plotting : plottly, not breeze

Plottly-scala easy to use from ammonite, Jupyter

Not really an issue


Description wasn't clear on the purpose of slots

The description on the purpose of Slots wasn't so clear at the beginning of the "Slots" section. I changed it a bit, with some examples, to explain the benefits of slots. How we can use them instead of using attributes or props for passing HTML content to other component.

Thanks for the effort, but I'd rather keep the current version which (IMO) is more precise and to the point.


Stepper Reorg (W/ Templates)

Make the flow of the steps be:

What kind of DAO would you like to make?

Open Community
Membership Club
Worker Co-op
Token Holder

Do you have members?

"No, just me. I'll add other members once it goes live"
"Yes, seed from CSV file"
"Yes, seed from token holder snapshot"

Give your DAO a name
Customize (optional)
Review & Deploy

Dependent on this: https://github.com/dOrgTech/DAOcreator/issues/241

Closing this as we've addressed this UX in UI V2


How to add a space between the comment delimiter

hi,
This is not a issue.
I want to add a space between comment delimiter, but I don't know how to do that.
For example,
int version;

after comment it with  <leader>cc
it will become
//int version

But what I want is
// int version

There is one space after the delimiter
Here is a another example
/*int version*/

But what I want is
/* int version */

There is a space before and after the delimiter.
Is it possible to configure that in nerdcommenter?
Thank you!

This is supported and explained in the docs.
https://github.com/scrooloose/nerdcommenter/blob/master/doc/NERD_commenter.txt#L668-L684

@rosshadden
awesome!!!
Having let NERDSpaceDelims=1 would do the trick.
Thank you @rosshadden  :)


Message text colour not customizable

Pretty much everything has a colour customization option, so another one for .msg-text would just be consistent.

I can give this a shot

PR: https://github.com/izzy/stream-chat/pull/18


Just doesn't appear in my menu bar?

I run the app, nothing exists in my menu bar ... no sign of the app anywhere.
MacOS Sonoma 14.5, M1 Macbook Air

Seems to be bartender weirdness - wasn't showing icon yet it was then when i used the search ui. Closing.


Local llama model not working - for instance Wizard-Vicuna-7B-Uncensored.ggmlv3.q5_1.bin via llama-cpp-python[server]

Description
console output:
USER: hello
{'id': 'cmpl-30ab8a26-9d60-4a60-b1d1-8325826f3da9', 'object': 'text_completion', 'created':<PHONE_NUMBER>, 'model': 'F:/LLAMA/AGiXT/agixt/models/Wizard-Vicuna-7B-Uncensored.ggmlv3.q5_1.bin', 'choices': [{'text': '', 'index': 0, 'logprobs': None, 'finish_reason': 'stop'}], 'usage': {'prompt_tokens': 34, 'completion_tokens': 1, 'total_tokens': 35}}
2023-06-16 18:17:46,329 | INFO | Error: object str can't be used in 'await' expression
2023-06-16 18:17:46,329 | INFO | PROMPT CONTENT: None
You are LOCAL, an expert at answering the users question.  Use available context help answer the users questions.
USER: hello
2023-06-16 18:17:46,329 | INFO | TOKENS: 29
2023-06-16 18:17:46,329 | INFO | Failed to get a response 5 times in a row.
2023-06-16 18:17:46,334 | INFO | Response: None
2023-06-16 18:17:46,334 | INFO | Response: None
2023-06-16 18:17:46,334 | INFO | Response: None
2023-06-16 18:17:46,334 | INFO | Response: None
INFO:     <IP_ADDRESS>:12512 - "POST /api/agent/LOCAL/prompt HTTP/1.1" 200 OK
Steps to Reproduce the Bug

run server llama-cpp-python[server] with the model Wizard-Vicuna-7B-Uncensored.ggmlv3.q5_1.bin
run AGiXT
setup new agent  - Set Select Provider to llamacppapi , Set Select Embedder to llamacpp , Set AI_PROVIDER_URI to http://localhost:8000 ( like llama server shows )
save it
test with chat

Lllama server getting data from AGIXT
output:
llama_print_timings:        load time =  1992.45 ms
llama_print_timings:      sample time =     0.21 ms /     1 runs   (    0.21 ms per token)
llama_print_timings: prompt eval time =     0.00 ms /     1 tokens (    0.00 ms per token)
llama_print_timings:        eval time =   176.22 ms /     1 runs   (  176.22 ms per token)
llama_print_timings:       total time =   200.36 ms
←[32mINFO←[0m:     ::1:12527 - "←[1mPOST /v1/completions HTTP/1.1←[0m" ←[32m200 OK←[0m
Expected Behavior
should works?
Operating System

[ ] Linux
[X] Microsoft Windows
[ ] Apple MacOS
[ ] Android
[ ] iOS
[ ] Other

Python Version

[ ] Python <= 3.9
[X] Python 3.10
[ ] Python 3.11

Environment Type - Connection

[X] Local - You run AGiXT in your home network
[ ] Remote - You access AGiXT through the internet

Runtime environment

[ ] Using docker compose
[X] Using local
[ ] Custom setup (please describe above!)

Acknowledgements

[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of AGiXT.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

To whoever might make an attempt to fix this error, possible causes could be:
Incorrect or Compatibility Syntax Issue with the 'await' expression and 'str' object.

any help with that issue?

Sorry this one was overlooked when doing async changes.  Should be all set now.  Thanks!


Update csharp protobuf to 310

moves https://github.com/grpc/grpc/pull/8240 to master branch

test this please

test this please

Out of date.


cargo run with L072CB doesn't work

Hi,
I have used L072 before but in a smaller package.
When trying the blinky example with the L072CB I see no log message at all and it seems the firmware doesn't work.
The firmware is flashed via cmsis-dap (picoprobe, I use it all the time and it works great normally) but afterwards nothing happens.
The USB-Bootloader works, though, when booting the µC with my boot button pressed. And dfu flashing works too, but the flashed firmware doesn't seem to work at all.
I'm using the cargo.toml feature stm32l072cb and for probe-rs --chip STM32L072CBTx
Has someone an idea what this could be?

Found the problem, I didn't switch the instruction set to thumbv6m-none-eabi because the code was built for an L151 before. :man_facepalming:


contributor docs: Add guidance on claiming a second issue.

Please feel free to tweak the wording.
This seemed like a reasonable place to put the question, but I could also imagine having it higher up in the bulleted list.

This is great; I also added a link to the "Make sure your pull request is reviewable" -- since that's usually a better next step than starting a next project.

Oops, forgot to squash my changes before merging; actual version merged is 1176f1e674663c6d13acad843edc6adf1d083804.


MD037  Not Detecting Error Consistently

Hi @DavidAnson
I cannot seem to get this line of code to clear the test indicated in the title of this issue.  Funny thing is, the exact line of text is used earlier in the document, but seems to avoid detection.  I literally copied and pasted this line lower down in the document.   The MTL logo is italicized and the whole line is bold face.  I am not sure how to correct his.
Line earlier in the document that is not detected

Line in question that violates rule MD037

As shown in GitHub Actions

It's hard to tell from the screenshots and I don't seem to have access to that repository (https://github.com/mtl/blue), but what seems likely is that there is another '' somewhere earlier that is unmatched and therefore skewing the results. My guess is that it's the one in data_ui. By spec, that should not be relevant for this rule, but this rule exists to identify scenarios that are outside the spec and so it can get confused by things like that. If you find and escape (with '') the relevant '', that should address the issue.

It's hard to tell from the screenshots and I don't seem to have access to that repository (https://github.com/mtl/blue), but what seems likely is that there is another '' somewhere earlier that is unmatched and therefore skewing the results. My guess is that it's the one in data_ui. By spec, that should not be relevant for this rule, but this rule exists to identify scenarios that are outside the spec and so it can get confused by things like that. If you find and escape (with '') the relevant '', that should address the issue.

The full code is below.  I'm sorry I can't share the repo with you, it doesn't belong to me, I'm just a contributor.  I checked for an unpaired quote elsewhere in the table, but was unable to locate any.  As you can see with the code below, that same line of code exists all over the place in the file.
Code Follows

title: "MTL Session 2 - Learner See"
author: "Team PSD"
date: "March 2020"
release: "MTL 2.0"
output:
github_document: default
html_document: default
pdf_document: default
word_document: default
ioslides_presentation: default
slidy_presentation: default
powerpoint_presentation: default

Note: If you are a self-directed learner, then some of the details in the guides may not apply to you. These guides were developed for facilitated Modeling to Learn Live team meetings.
MTL Live Session 02
Today we're modeling to learn how to check our patient data and team trends
Done and Do (15 minutes)

 Done
 Do

  We identified our Team Vision, selected our Team Lead and set a standing team meeting time. We logged in to mtl.how/data to look at the facility data.   MTL 2.0. For MTL 1.7, click here
 We will confirm and review Team Data for MTL relevant to our Team Vision.  MTL 2.0. For MTL 1.7, click here

Learning Objectives

Describe the decisions your team made in producing your team data table.

Test out whether your expectations about team historical trends are displayed in the "viz" tabs.

Apply your clinical expertise to identify new information about a team patient in the "data" tabs.

In-session Exercise (30 minutes)
[](data UI says what is and sim UI asks What if - illustration?)
MTL on BISL
1. Open an internet browser window. Go to mtl.how/data. First select your VISN and then select your facility

MTL 2.0. For MTL 1.7, click here
2. Explore the facility data

To view facility data click on the [...] next to the file "facility.xlsb" and click "Edit" to open in Excel.
Team files are in team folders. If you do not see a team folder, request a team folder above.

MTL 2.0. For MTL 1.7, click here

Click through the tabs on the bottom of the Excel spreadsheet. What options are there and what do they mean? Hints:

Data: Sortable report of data
Diag: Diagnostic data
Enc: Encounter types of visits
HF: Health Factors data related to visits
Meas: Measures or flag names
Viz: Visualization of trends

What filtering options are available?

Filter to your clinic or division to see trends for the last two years.

New for MTL 2.0

You no longer need to use only Internet Explorer or Edge to open mtl.how/data. You can use any browser.

View facility data via the "facility.xlsb" Excel sheet instead of the splash page.

vizDiag shows unique visits by primary diagnosis on the visit of PTSD, Dep, OUD, or AUD.

Patient SSID is no longer asterisked out in the facility data. It has been completely removed. Please go to the data UI to see Patient SSID information.

Your Team Data Folder
3. Scroll to your team folder at the bottom of the page. Open the data_ui folder and open your data_ui file in Excel

Click on the 3 dots /[/./././] next to the data_ui file.
Click on "Edit" to open in Excel, then click "Edit Workbook" at the top of the sheet.

4. Go to the ClinicSelection tab. Use columns C-H to select the clinics that make up your team

You can sort and filter by Clinic Name, Division Name, Physical Location, Primary Stopcode, Secondary Stopcode, and Default Provider.
Note: This will pull all clinics used in the last two years (including de-activated clinics: denoted by "ZZ"). For de-activated clinics, you can find the date of de-activation in column I.
Follow the instructions in Box A2.
After filtering, click the "Add All" arrow in column B (Cell B6). Fine-tune your selections by double-clicking on the clinic name in column A (to remove) or column C (to add).

New for MTL 2.0

In the future, if you want to quickly get the same list of clinics again, you can select the grey "Get previous clinic list from last click of Get Patient-level Data" arrow in Column B (Cell B3).

For MTL 1.7 click here.
5. To view your team patient data and your team trends click "Get Patient-level Data." We will not do this in-session today

You already have a fresh data UI file to work with in your team data_ui folder.
We will learn about the "Create Team Data Table" button in our next team meeting.
To query patient and clinic level data using the data_ui, you will need LSV permissions. To check which facilities you have LSV access for, click here (VA intranet link to access verification).
If you don't have access to facilities that you believe you should, please verify:

CPRS access/permissions at your station (i.e COR tab access for CPRS).
Your network username recorded in VistA.
To have your network username recording in VistA, please follow the instructions outlined here (VA intranet link to Vista).

New for MTL 2.0

"Create Team Data Table for Sim UI" is now called "Create Team Data."

For MTL 1.7 click here.
NOTE: It takes some time to run a query from your team data UI to the VA Corporate Data Warehouse.

On average (depending on the size of your team) it may take 15 minutes or so for your team data UI to pull in fresh data, and Microsoft Excel will be unresponsive until the data UI has finished pulling in your data.

MTL 2.0. For MTL 1.7, click here
6. Click to view the "viz" tab, which show team trends

There are team trends for diagnoses, encounters, health factor data (e.g., suicide plans, evidence-based practice templates), and measures from Mental Health Assistant.
What stands out to you?
What is most important to you to check out first?
What is most surprising?

MTL 2.0. For MTL 1.7, click here
New for MTL 2.0

Click on any trend line and then "Quick Explore" (look for the magnifying glass). In the "Explore" box, pick the item you want to explore and click "Drill To." Keep using the Quick Explore to drill until you reach the data you want. To return to the original settings, click on the gray "Reset Pivot Chart."

The viz tabs will also return to default selections, when you click "Get Patient-level Data" for new data pulls.

Right click on a trend line and filter to hide a selected item or only keep the selected item.

vizDiag in the data UI shows a drill down by diagnoses or combination of diagnoses.

For MTL 1.7 click here.
7. Click to view the "data" tab, which show your team's individual patient information

Patients who have requested restricted access to their information have asterisks /(/////) in Columns F & G. If you are a VA provider, you can still identify patients from Column H.
Patient information corresponds to the same categories as the team trends: diagnoses, encounters, health factor data (e.g., suicide plans, evidence-based practice templates), and measures from Mental Health Assistant.
Providers can filter to find specific patients, or produce reports.

New for MTL 2.0

dataDiag in the data UI has an additional column after "Diagnoses of Interest," called "Primary Diagnoses", specifying which diagnosis is primary.

For MTL 1.7 click here.

What data tab would you use to find out how many current patients on the team are engaged in a specific evidence-based psychotherapy? What column shows you the session number (EBP template) that the patient is on?
What viz option would you use to see what the most common service encounters or visits are?
Are there services that have been increasing over time? Are there services that have been decreasing over time?

With the team trends (viz) and team patient (data) information in the data UI, your team can efficiently use team meetings to focus on the interrelated issues of care coordination and team quality improvement.
Done and Do (15 minutes)

[](Done icon) Done
[](Do icon)) Do

 We confirmed the clinics that make up our team for the Team Data UI.  MTL 2.0. For MTL 1.7, click here
 Review the HF, Diag, Enc and SP tabs in Team Data UI to find a patient (zoom in) and find a team trend (zoom out). Log in to mtl.how/data and look at the two team folders: data_ui and team_data_table.  MTL 2.0. For MTL 1.7, click here

The full code is below.  I'm sorry I can't share the repo with you, it doesn't belong to me, I'm just a contributor.  I checked for an unpaired quote elsewhere in the table, but was unable to locate any.  As you can see with the code below, that same line of code exists all over the place in the file.
Code Follows

title: "MTL Session 2 - Learner See"
author: "Team PSD"
date: "March 2020"
release: "MTL 2.0"
output:
github_document: default
html_document: default
pdf_document: default
word_document: default
ioslides_presentation: default
slidy_presentation: default
powerpoint_presentation: default

Note: If you are a self-directed learner, then some of the details in the guides may not apply to you. These guides were developed for facilitated Modeling to Learn Live team meetings.
MTL Live Session 02
Today we're modeling to learn how to check our patient data and team trends
Done and Do (15 minutes)

 Done
 Do

  We identified our Team Vision, selected our Team Lead and set a standing team meeting time. We logged in to mtl.how/data to look at the facility data.   MTL 2.0. For MTL 1.7, click here
 We will confirm and review Team Data for MTL relevant to our Team Vision.  MTL 2.0. For MTL 1.7, click here

Learning Objectives

Describe the decisions your team made in producing your team data table.

Test out whether your expectations about team historical trends are displayed in the "viz" tabs.

Apply your clinical expertise to identify new information about a team patient in the "data" tabs.

In-session Exercise (30 minutes)
[](data UI says what is and sim UI asks What if - illustration?)
MTL on BISL
1. Open an internet browser window. Go to mtl.how/data. First select your VISN and then select your facility

MTL 2.0. For MTL 1.7, click here
2. Explore the facility data

To view facility data click on the [...] next to the file "facility.xlsb" and click "Edit" to open in Excel.
Team files are in team folders. If you do not see a team folder, request a team folder above.

MTL 2.0. For MTL 1.7, click here

Click through the tabs on the bottom of the Excel spreadsheet. What options are there and what do they mean? Hints:

Data: Sortable report of data
Diag: Diagnostic data
Enc: Encounter types of visits
HF: Health Factors data related to visits
Meas: Measures or flag names
Viz: Visualization of trends

What filtering options are available?

Filter to your clinic or division to see trends for the last two years.

New for MTL 2.0

You no longer need to use only Internet Explorer or Edge to open mtl.how/data. You can use any browser.

View facility data via the "facility.xlsb" Excel sheet instead of the splash page.

vizDiag shows unique visits by primary diagnosis on the visit of PTSD, Dep, OUD, or AUD.

Patient SSID is no longer asterisked out in the facility data. It has been completely removed. Please go to the data UI to see Patient SSID information.

Your Team Data Folder
3. Scroll to your team folder at the bottom of the page. Open the data_ui folder and open your data_ui file in Excel

Click on the 3 dots /[/./././] next to the data_ui file.
Click on "Edit" to open in Excel, then click "Edit Workbook" at the top of the sheet.

4. Go to the ClinicSelection tab. Use columns C-H to select the clinics that make up your team

You can sort and filter by Clinic Name, Division Name, Physical Location, Primary Stopcode, Secondary Stopcode, and Default Provider.
Note: This will pull all clinics used in the last two years (including de-activated clinics: denoted by "ZZ"). For de-activated clinics, you can find the date of de-activation in column I.
Follow the instructions in Box A2.
After filtering, click the "Add All" arrow in column B (Cell B6). Fine-tune your selections by double-clicking on the clinic name in column A (to remove) or column C (to add).

New for MTL 2.0

In the future, if you want to quickly get the same list of clinics again, you can select the grey "Get previous clinic list from last click of Get Patient-level Data" arrow in Column B (Cell B3).

For MTL 1.7 click here.
5. To view your team patient data and your team trends click "Get Patient-level Data." We will not do this in-session today

You already have a fresh data UI file to work with in your team data_ui folder.
We will learn about the "Create Team Data Table" button in our next team meeting.
To query patient and clinic level data using the data_ui, you will need LSV permissions. To check which facilities you have LSV access for, click here (VA intranet link to access verification).
If you don't have access to facilities that you believe you should, please verify:

CPRS access/permissions at your station (i.e COR tab access for CPRS).
Your network username recorded in VistA.
To have your network username recording in VistA, please follow the instructions outlined here (VA intranet link to Vista).

New for MTL 2.0

"Create Team Data Table for Sim UI" is now called "Create Team Data."

For MTL 1.7 click here.
NOTE: It takes some time to run a query from your team data UI to the VA Corporate Data Warehouse.

On average (depending on the size of your team) it may take 15 minutes or so for your team data UI to pull in fresh data, and Microsoft Excel will be unresponsive until the data UI has finished pulling in your data.

MTL 2.0. For MTL 1.7, click here
6. Click to view the "viz" tab, which show team trends

There are team trends for diagnoses, encounters, health factor data (e.g., suicide plans, evidence-based practice templates), and measures from Mental Health Assistant.
What stands out to you?
What is most important to you to check out first?
What is most surprising?

MTL 2.0. For MTL 1.7, click here
New for MTL 2.0

Click on any trend line and then "Quick Explore" (look for the magnifying glass). In the "Explore" box, pick the item you want to explore and click "Drill To." Keep using the Quick Explore to drill until you reach the data you want. To return to the original settings, click on the gray "Reset Pivot Chart."

The viz tabs will also return to default selections, when you click "Get Patient-level Data" for new data pulls.

Right click on a trend line and filter to hide a selected item or only keep the selected item.

vizDiag in the data UI shows a drill down by diagnoses or combination of diagnoses.

For MTL 1.7 click here.
7. Click to view the "data" tab, which show your team's individual patient information

Patients who have requested restricted access to their information have asterisks /(/////) in Columns F & G. If you are a VA provider, you can still identify patients from Column H.
Patient information corresponds to the same categories as the team trends: diagnoses, encounters, health factor data (e.g., suicide plans, evidence-based practice templates), and measures from Mental Health Assistant.
Providers can filter to find specific patients, or produce reports.

New for MTL 2.0

dataDiag in the data UI has an additional column after "Diagnoses of Interest," called "Primary Diagnoses", specifying which diagnosis is primary.

For MTL 1.7 click here.

What data tab would you use to find out how many current patients on the team are engaged in a specific evidence-based psychotherapy? What column shows you the session number (EBP template) that the patient is on?
What viz option would you use to see what the most common service encounters or visits are?
Are there services that have been increasing over time? Are there services that have been decreasing over time?

With the team trends (viz) and team patient (data) information in the data UI, your team can efficiently use team meetings to focus on the interrelated issues of care coordination and team quality improvement.
Done and Do (15 minutes)

[](Done icon) Done
[](Do icon)) Do

 We confirmed the clinics that make up our team for the Team Data UI.  MTL 2.0. For MTL 1.7, click here
 Review the HF, Diag, Enc and SP tabs in Team Data UI to find a patient (zoom in) and find a team trend (zoom out). Log in to mtl.how/data and look at the two team folders: data_ui and team_data_table.  MTL 2.0. For MTL 1.7, click here

I can't do much with the rendered Markdown. If you put it in a code block, GitHub should preserve the original text.

I can't do much with the rendered Markdown. If you put it in a code block, GitHub should preserve the original text.

Code block as requested - thank you.

Code block as requested - thank you.

Thanks, that confirms its due to data_ui in the text like I suggested above. Changing to data\_ui avoids the warning.

Thanks, that confirms its due to data_ui in the text like I suggested above. Changing to data\_ui avoids the warning.

I will check it out. Thank you.

I will check it out. Thank you.

Yes - that corrected the problem.  Thanks for your help @DavidAnson.

Yes - that corrected the problem.  Thanks for your help @DavidAnson.


Allow project initialization via GitOps

Is your feature request related to a problem? Please describe.
Initializing a keptn project today requires a empty GIT repository which is in many use-cases too imperative vs declarative configurations which is used by other tools.
Describe the solution you'd like
Be able to declarative configure both Projects and Service's using CRD's in the keptn cluster instead of the CLI directly.
Preferably by the keptn-operator would be watching for new Project resources and creating them in keptn on the user's behalf and keptn would be able to initialize itself based current state from the GitOps repository.
This would allow easier use/integration with certain other tools like Flux V2 and potentially other tools that often has per-existing files in the repository.
It is also a benefit when it comes to mobility / disaster recovery of keptn Projects etc since if keptn get's a problem then you can simply re-create the Project | Services etc from a existing git repository.
Describe alternatives you've considered

Have keptn create project ... be able to onboard a non-empty repository potentially where existing service definitions are present somehow where the repository already has a shipyard.yaml file, having stage structure etc present already.
The  the keptn-gitops-operator kicks in and watches either the repository or CRD's it does a analysis and triggers keptn actions behind the scenes via the API or alike.

Great idea @ekarlso, thanks for posting it!
While the CLI command keptn create project needs to be addressed on the Keptn core side for allowing non-empty repositories, I would like to ask @thschue: if the keptn-gitops-operator is already capable of bootstrapping the project then?

No, the project has to be initialized and the git repository connected. We could create the Keptn Project using the KeptnProject resource, but this is a thing that has to be discussed with the Keptn Core team, as it is not clear for us what Keptn is doing when creating a Project ...

As a new user, I have to say that this is precisely the behavior I expected to encounter. I was very surprised that I had to create an empty git repository, which simply did not feel right.
Ideally, I would like to store Keptn configurations in the same repository as my application.

hey, if theres any way i can help, i would like to work on this @thisthat

Sure @utibeabasi6, you can have a look. You might want to look into https://github.com/keptn-sandbox/keptn-gitea-provisioner-service to see if you can automate the provisioning of the project

alright, will do

@utibeabasi6 can we close this?

Since Keptn V1 has reached EOL on December 22nd, 2023, this issue has been closed.


Recongize .xhtml files

Visual Studio Code does not provide any IntelliSense, not even syntax highlighting, for .xhtml files. If IntelliSense for .html file cannot be used because of the possible conflict between the compact xhtml syntax and html syntax, I think .xhtml file should at least be treated as an XML file, which has syntax highlighting and bracket/quotes matching.

This will be in the March release https://github.com/Microsoft/vscode/issues/3521


Citation

How exactly a reference to DynaSim should appear in a manuscript?
Up-to-date citation:
Sherfey JS, and Contributors (2016) DynaSim toolbox for modeling and simulating dynamical systems. https://github.com/DynaSim
or
Sherfey JS, & Contributors (2016) DynaSim toolbox for modeling and simulating dynamical systems. https://github.com/DynaSim

What I am using right now:
Sherfey JS (2016) DynaSim toolbox for modeling and simulating dynamical systems. https://github.com/DynaSim

Don't sell yourself short Salva! I would put Sherfey and "Contributors"
since you also wrote code that went into DynaSim.
IIRC, oftentimes science software projects don't list every single person
who's contributed to code in the "authorship", but instead have a single
CONTRIBUTORS.md file in the repo that lists everyone who's contributed.
--
Austin E Soplata
Graduate Program for Neuroscience,
Boston University
L - linkedin.com/in/austinsoplata
http://www.linkedin.com/in/austinsoplata
P -<PHONE_NUMBER>
T -  @austinsoplata https://twitter.com/austinsoplata
On Mon, Sep 19, 2016 at 3:31 PM, kupiqu<EMAIL_ADDRESS>wrote:

What I am using right now:
Sherfey JS (2016) DynaSim toolbox for modeling and simulating dynamical
systems. https://github.com/DynaSim
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DynaSim/DynaSim/issues/33#issuecomment-248097970, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAoWtTIa2Z5aKzlXz66eQgPDwqrIy0oHks5qruMbgaJpZM4KA3Gz
.

It sounds good to me. I'll update the citation above, and will modify it according to what we agree on...

There's actually a big paper that just came out on software citation in research https://peerj.com/articles/cs-86/

What got decided for this?

Jason recently told me to cite it as the most recent poster presentation of it:
Sherfey JS. (2016). DynaSim: a Matlab toolbox for rapidly building and exploring neural models. 2016 Janelia conference on Collaborative Development of Data-Driven Models of Neural Systems, Ashburn, Virginia.
Where the poster itself is at here
This is probably the best choice until the DynaSim paper itself comes out.

Since the paper is imminent, I'm gonna close this.


tests: replace Equal on proto with protoassert.

Description
This is continuation of job done with eg.
This time manually fixed issues.

https://github.com/stackrox/stackrox/pull/11503

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

#11531
#11533
#11503
#11507


UI text example

This doesn't look right for the "UI text" example: "Hmm … looks like that’s a broken link."

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

ID: d28c9871-f3a6-2dc1-1494-93f981f50238
Version Independent ID: 8e95f7bc-399a-7648-36bd-8dd7dc90f347
Content: Formatting common text elements - Microsoft Style Guide
Content Source: styleguide/text-formatting/formatting-common-text-elements.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

What about it seems wrong? "UI text" can refer to a number of things, including button labels, error messages, notifications, steps in a wizard, and so on. What would you expect to see here instead? Thanks!

Perhaps more examples than that one as in developer text and instructions:
https://docs.microsoft.com/en-us/style-guide/developer-content/formatting-developer-text-elements
https://docs.microsoft.com/en-us/style-guide/procedures-instructions/formatting-text-in-instructions
Otherwise, instead of looking like an example, it just looks like someone meant there to be a link.

You closed this without making any changes or replying to my comment?

Oh, my mistake. I was thinking of a different thread. Thanks for the additional info. This is on the list of revisions to be made.

Thanks!


Use standard default vault client configuration

Instead of checking for env vars, prefer the standard configuration and check for the use of a tokenhelper (ie existing token at ~/.vault-token).

good job! I will add one commit for updating the docs and then merge and releaseit with the other PR


DO NOT MERGE! Updates html tag to include lang

@coreyd303 it looks like you have submitted a PR with the changes from #120 in it. I have locked this pull request, unless you would like to submit a clean PR without those changes in it. Thanks!

Picking up where this left off...


Issue without password

The greeter automatically loads when a user has no password. I want it to wait for me to manually click the 'Login' button before loading.

Try disabling auto login, password is required for users.

This is LightDM's issue, not theme's issue i think.

Closed.


[Question] would it be possible to show a trajectory (moving object) with Osmnx (stream plotting)

plotting in osmnx is on images, but is it possible to plot a trajectory (for example a car) that is dynamically moving ?
if it is possible what is the efficient approach ? thank you

Please reread the contributing guidelines and new issue template for suggestions on where to ask how-to usage questions.

could you please share the contributing guidelines link.
thank you.


County flag changed when type number

Hello Dear
I have used below code for phone input
I am getting issues while enter input,  I have select country canada and enter number, Flag become change to US.
 <PhoneInput                                 style={styles.enterMobileNumberInputStyle}                                 ref={ref => { this.phone = ref; }}                                 initialCountry='in'                                 allowZeroAfterCountryCode={true}                                 textStyle={{ fontSize: normalize(16), color: Colors.black, height: normalize(44)}}                                 textProps={{ placeholder: 'Enter Mobile Number' }}                                 pickerBackgroundColor={Colors.appColor}                                 pickerButtonColor={Colors.white}                                 autoFormat={true}                             />
I have attached video link here
https://drive.google.com/file/d/1NWSjk5bL9cbhXBKmgxlfvtmSAIf8e7wR/view
Thanks,
Umesh

I have the same issue, is there a solution to that?

Same issue is there for countries with dial code, is there a solution to that?


upstream: allow subset_lb to be configured to respect locality weights

Adds a flag to the subset lb config which will cause the subset
lb to respect the locality weights of the original host set. This allows
subset matching on clusters that are configured with locality weights.
Risk Level: Low, new optional feature
Testing: Unit tests
Docs Changes: Added documentation to proto definition
Release Notes: Added note about new configuration option to release notes
Fixes #3123

@snowp can you merge master to pickup https://github.com/envoyproxy/envoy/pull/3840?

@htuch done! should be good to merge now