Fix the double leaf node updates in flex_node_system

Objective
If a UI node has a changed CalculatedSize component and either the UI does a full update or the node also has a changed Style component, the node's corresponding Taffy node will be updated twice by flex_node_system.
Solution
Add a Without<Calculated> query filter so that the two changed node queries in flex_node_system are mutually exclusive and move the CalculatedSize node updater into the else block of the full-update if conditional.

@alice-i-cecile should be ready


day_trader_status is bool, not str

account.py {373}
class Account(TastytradeJsonDataclass):

...
day_trader_status: str
...

Should be changed to:
...
day_trader_status: bool
...

According to the official docs it's a string... Could you see what happens if you place a day trade? I'd need to see that, as well as the output for an account >$25k to be sure it's always a bool.

the issue has been resolved. After I ran into a lot of other issues, I realised that not all dependencies had been properly installed.
After installing them everything seems to be ok.

Great!

@Quenos How did you get it resolved? Tastytrade returns day_trader_status = False (which is a bool).
Pydantic TastytradeJsonDataclass rejects it because day_trader_status is set to str.  I can change it to bool but I'm not sure how it's working for others.
@Graeme22 FYI . I'm not sure how it's working for you. I can just directly load this class and test it manually and it failed bool vs str. See your Account class below.
class Account(TastytradeJsonDataclass):
"""
Dataclass that represents a Tastytrade account object, containing
methods for retrieving information about the account, placing orders,
and retrieving past transactions.
"""
account_number: str
opened_at: datetime
nickname: str
account_type_name: str
is_closed: bool
day_trader_status: str
is_firm_error: bool
is_firm_proprietary: bool
is_futures_approved: bool
is_test_drive: bool
margin_or_cash: str
is_foreign: bool
created_at: datetime
external_id: Optional[str] = None
closed_at: Optional[str] = None
funding_date: Optional[date] = None
investment_objective: Optional[str] = None
liquidity_needs: Optional[str] = None
risk_tolerance: Optional[str] = None
investment_time_horizon: Optional[str] = None
futures_account_purpose: Optional[str] = None
external_fdid: Optional[str] = None
suitable_options_level: Optional[str] = None
submitting_user_id: Optional[str] = None
@classmethod
def get_accounts(
    cls,
    session: Session,
    include_closed=False
) -> List['Account']:
    """
    Gets all trading accounts from the Tastyworks platform. By default
    excludes closed accounts from the results.

    :param session: the session to use for the request.

    :return: a list of :class:`Account` objects.
    """

    response = requests.get(
        f'{session.base_url}/customers/me/accounts',
        headers=session.headers
    )
    validate_response(response)  # throws exception if not 200

    accounts = []
    data = response.json()['data']['items']
    for entry in data:
        account = entry['account']
        if not include_closed and account['is-closed']:
            continue
        accounts.append(cls(**account))

    return accounts

Hmm, could you post the output of pip list?

According to the docs it's still a string

And here it's still working.
As you can read from the thread, I installed requirements.txt
That did it for me.

I'm going to close this unless I hear further comments. I believe the problem is related to using pydantic 2 instead of 1, which breaks the type validation.


geo:json for everyone

As suggested by @chicco785 , it would be interesting to have, for the entities expressing location in a non geo-json format (say, with country codes and addresses), an under-the-hood transformation to an equivalent geo-json structure, so that geo-queries can be made across different entities expressing locations in different ways.
I open this issue to keep track of the discussion of this idea.

some pointers that could be useful:

https://github.com/DenisCarriere/geocoder
https://github.com/datasets/geo-nuts-administrative-boundaries/tree/master/data

let's suppose that for some data we can do this in a easier way
for example, for country names and country codes, we could create a separate table
with the associated geo:json, and when running ngsi geo queries, do a merge on the field in the background.
for things like street address, we would definitively need a geocoding service but then, instead of querying the service every time i would cache the data
in relation to NGSI Data Models, when you import the Location-Commons definition in a schema, that schema will have geo:json and/or address, address object includes properties that allows us for some resolution, so if you have only addressCountry, we would do a resolution based on table merge (see above), if it has also an addressStreet, we would use an external service to obtain a geo:point
and cache it.
similar approach for entities with only these fields would be possible:
"addressLocality": {
"type": "string"
},
"addressRegion": {
"type": "string"
},

First version of this already implemented.


ES6 Array methods sample

See https://www.chromestatus.com/feature/6732923508097024

@beaufortfrancois is going to take this one on


Markdown is not rendered in bitbucket editor

Setup
Browser: Chrome Version 65.0.3325.181 (Official Build) (64-bit)
Editor: Sublime text 3
Description
Bitbucket has recently changed their comment editor and now it does no render markdown sent via Ghost text.

Yeah that now has become a plain contentEditable and they hacked on markdown support in it, somehow. GhostText doesn’t inject each keystroke separately so their new editor doesn’t handle them correctly.
To do so GhostText would either have to change its protocol to transmit each character at a time or implement a diff in the browser so it can figure that out without changing all the editor extensions.
Ether way it’s a big undertaking and I don’t have time/desire to handle it.
The good news is that you can now write HTML instead of markdown and since you’re in a text editor HTML shouldn’t be too difficult to write.

Actually it’s even harder than that because if you typed Markdown, it would be converted to HTML by BitBucket and you’d get HTML back in the editor, not Markdown.
Your best bet is #53 but that is also a big task that isn’t being done by anyone 😅

Yeah, I imagined their custom transformation would be hard to integrate, no worries.
#53 sounds great but very ambitious indeed, not holding my breath for it 😄
I'd like to take this opportunity to thank you for this amazing tool, I can't not use it once I discovered it. So thank you!


3.0.0-M3 Simplifed Main Class Name Resolution with Gradle

As Gradle is currently not supported this 'wontfix'.


Can the webpage HTML5 video element be loaded into AR view ?

I was using SceneForm SDK for the AR capabilities on my app. I've tried loading the WebView on ViewRenderable and it worked fine with no issue. I could able to load any HTTP/HTTPS webpages. However when tried to load the webpage that uses HTML5 video element in it, then I just see the black screen instead of the actual video content. BTW, audio content plays fine.
Can the HTML5 video element be loaded into AR view ? Any suggestion on how this can be achieved ?

Interesting. I will have a look on your case.
I send you a collaborator access invitation.
Can you accept it and drag and drop your code here : ../tree/master/samples/shared/webviewrenderable/

Thanks @ThomasGorisse , I could not upload the entire project. However I did upload all the code using which you could reproduce this.


Scene Understanding improvements

This PR adds these features:

Spatial Mapping can now use XR_SCENE_COMPUTE_CONSISTENCY_SNAPSHOT_INCOMPLETE_FAST_MSFT for mesh data.
This results in a smoother and more complete mesh, trading off speed for quality.

Fixes #102

Fixes #101
Support for LOD, defined in https://registry.khronos.org/OpenXR/specs/1.0/man/html/XrMeshComputeLodMSFT.html

Changed SpatialMeshingVolumeSize from being a platform limited variable to being available anywhere, this is to support Holographic Remoting. Also, renamed it to SceneUnderstandingVolumeRadius

New config variables to control the features above
ShouldDoSceneUnderstandingHighQualityMeshing controls the spatial mapping consistency
SceneUnderstandingVolumeRadius renamed and relocated from SpatialMeshingVolumeSize
SceneUnderstandingMeshingLOD this is a interger based value corresponding to the XrMeshComputeLodMSFT definition.

@microsoft-github-policy-service agree


Release numbering

I would like to propose that releases in the 2.0.x scheme be used for the manuscript, whereas the 2.1.x line be used for everything thereafter. I have created a 2.0.6 branch in the repo for making the release. I had to revert some changes to process_trace (see https://github.com/sys-bio/tellurium/pull/220). However, I think these changes will cause problems for @matthiaskoenig so I don't intend to merge them into master. They're a temporary fix to get the plots working.
Therefore, please make any new changes in the develop branch knowing they'll go into the 2.1.x series, but won't be included in 2.0.x series.

The current develop branch definitely breaks features in the manuscript, so please don't make any releases off of it right now.

Agreed.
Can hopefully finish my changes next week. These things are not required
for manuscript, i.e. will be things for the 2.1 version.
It basically will resolve the issue with the NA, process_traces, ... which
currently creates problems for us.
I am developing this in the 2.0-sedml-L1V3 branch and try to be up to date
with master, i.e. it should be possible to merge this with master without
any issues when finished.
On Thu, Oct 26, 2017 at 7:09 AM, Kyle Medley<EMAIL_ADDRESS>wrote:

The current develop branch definitely breaks features in the manuscript,
so please don't make any releases off of it right now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sys-bio/tellurium/issues/280#issuecomment-339552954,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA29ui3QvbucufweFZhg6QJDN99l2coeks5swBQngaJpZM4QFSYr
.

--
Dr. Matthias König
Junior Group Leader LiSyM - Systems Medicine of the Liver
Humboldt-University Berlin, Institute of Biology, Institute for Theoretical
Biology
https://www.livermetabolism.com
<EMAIL_ADDRESS>Tel: +49 30 20938450
Tel: +49<PHONE_NUMBER>0

Great, thanks!


epoch fix

This PR fixes a bug in LightEpoch, which caused by memory reorder.
Fix #149

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

Seems to be failing a few unit tests

Seems to be failing a few unit tests

I don't know if it is running too slowly or deadlock.

Seems to be failing a few unit tests

I fixed it.

First of all, thanks for finding the bug and submitting a PR!
If you have more time to work on this:

The use of std::atomic mixed with fences seems overly complicated. By default every operation on atomic should operate under std::memory_order_seq_cst and should do the correct thing for you. Now it is conceivable that we lose some performance due to excessive synchronization, but let's back that up with benchmark results and not prematurely optimize.
Alternatively, consider using std::atomic_signal_fence without the need to turn local_current_epoch into an atomic.
Can you add your test code as a unit test?

Unfortunately, in C++ consistency model, memory_order_acquire only prevents the reorder of load-store and load-load. Thus, the store(memory_order_acquire) and std::atomic_thread_fence (memory_order_acquire) don't work. memory_order_seq_cst is not always doing  the correct thing,  the store(memory_order_seq_cst) is equivalent to store(memory_order_acquire), which
allows the reorder of  store-load. (see https://en.cppreference.com/w/cpp/atomic/memory_order,  and you can verify it  by generate assembly code with g++ -O3 -S).
I have not used std::atomic_signal_fence before. But  it seems to only prevent reorder by compiler rather than reorder by CPU (https://en.cppreference.com/w/cpp/atomic/atomic_signal_fence).

First of all, thanks for finding the bug and submitting a PR!
If you have more time to work on this:
The use of std::atomic mixed with fences seems overly complicated. By default every operation on atomic should operate under std::memory_order_seq_cst and should do the correct thing for you. Now it is conceivable that we lose some performance due to excessive synchronization, but let's back that up with benchmark results and not prematurely optimize.
Alternatively, consider using std::atomic_signal_fence without the need to turn local_current_epoch into an atomic.
Can you add your test code as a unit test?
Unfortunately, in C++ consistency model, memory_order_acquire only prevents the reorder of load-store and load-load. Thus, the store(memory_order_acquire) and std::atomic_thread_fence (memory_order_acquire) don't work. memory_order_seq_cst is not always doing  the correct thing,  the store(memory_order_seq_cst) is equivalent to store(memory_order_acquire), which
allows the reorder of  store-load. (see https://en.cppreference.com/w/cpp/atomic/memory_order,  and you can verify it  by generate assembly code with g++ -O3 -S).
I have not used std::atomic_signal_fence before. But  it seems to only prevent reorder by compiler rather than reorder by CPU (https://en.cppreference.com/w/cpp/atomic/atomic_signal_fence).

Seems like it's worth a comment in the code in your PR.


InlineStyle is not work

Hey @madoka3530
Can you provide more information about this issue or bug that you have discovered?


fix(sql-execution): precision loss when displaying datetime type

What type of PR is this?
type-bug
module-sql execution

What this PR does / why we need it:
This  PR fixes an issue where odc displays datetime type precision loss
Which issue(s) this PR fixes:

Fixes #1388
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: yhilmare:x: zijiacjzijiacj seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Support multi signers in @cosm/stargate

The current sign() function of SigningStargateClient accepts only one signerAddress, but Cosmos basically support multi-signers.
https://github.com/cosmos/cosmjs/blob/06fbc34f72f12c30a396c3ca296f80eca9fa60b0/packages/stargate/src/signingstargateclient.ts#L301-L307
I'd like to suggest to create a new signWithMultiSigners() function that looks like below:
public async signWithMultiSigners(
    signerAddresses: string[],
    messages: readonly EncodeObject[],
    fee: StdFee,
    memo: string,
    explicitSignerDatas?: SignerData[],
  ): Promise<TxRaw>

I've already implemented this function by following the Cosmos' guide, so that I can use it for my Cosmos-based blockchain.
I think this is widely useful. If you think it's reasonable, I'm ready to open a PR :)

Hi @youngjoon-lee,
which of those two use cases do you have?

Multiple signatures because you use multiple messages in one transaction (e.g. Alice sends to Bob and Bob sends to Alice in one tx)
Use a multisig address that has its own account and is controlled by a treshold of pre-defined public keys?

For 1. you just use sign two times. We might need a stargate version of SigningCosmosClient.appendSignature (@cosmjs/launchpad) though. It could be done by hand but requires a lot of knowledge of the transaction structure and protobuf.
For 2. this is implemented and demonstrated in this test case.
The interface you proposed does not seem right to me because a single client would need to hold private keys for multiple signers. This does not seem practical for real applications.

Hi @webmaster128 ,
My use case is none of 1 and 2.
Actually, my use case is similar as 1.
But, my transaction contains only one message and it needs to be signed by two accounts.
As you said, it can be done by SigningCosmosClient.appendSignature in Launchpad, but unfortunately, that approach doesn't work for Stagate as mentioned in the cosmos-sdk doc.
It's because we must gather all SignerInfos first before singing with multiple accounts: https://docs.cosmos.network/v0.42/run-node/txs.html#signing-a-transaction-2
I just created a draft PR to share my idea, but I'm also thinking about:

The interface you proposed does not seem right to me because a single client would need to hold private keys for multiple signers.

My use case is none of 1 or 2. Actually, my use case is similar as 1.

I see. Yeah, I consider it a variant of 1. Multiple messsages is the easiest example but in general the case is: the set of all signers required by all messages.

but unfortunately, that approach doesn't work for Stagate as mentioned in the cosmos-sdk doc.
It's because we must gather all SignerInfos first before singing with multiple accounts: https://docs.cosmos.network/v0.42/run-node/txs.html#signing-a-transaction-2

Yes, right. This is much more pain in Stargate than it was before.

Maybe, we need multiple wallets.

I have no issues writing that code. My issue is if you and me have to sign a transaction, I don't want to hand over my private key such that both signatures can be created on one machine in one call.

I have no issues writing that code. My issue is if you and me have to sign a transaction, I don't want to hand over my private key such that both signatures can be created on one machine in one call.

Yes. Thank you for your reply.
I understand. In Launchpad, a signed tx can be sent to another machine, so that it can be signed by another key.
In my company, two addresses are managed by one person because of some reasons in the medical industry. But, it's absolutely not a general case.
If you think it's not reasonable to deal with multiple wallets on one machine, I am willing to fork your repo for my project. But, I would like to ask you consider supporting this feature again. There might be some use cases that need multiple keys on a machine.

Re-opening as the request is very valid. We can implement a solution that includes the use case described here but supports a multi-machine setup ad well. Many of the changes from #838 make sense, specially looking at the signture of makeAuthInfoBytes.

#854 implements the ability to use different sequences for different signers. While this is not enough to close this ticket, it is an important step that allows you to maintain a much smaller diff in your fork.

Yeah. That is an important first step, as you said. Thank you for handling this issue.


SQL syntax error, expected token is 'EQ', actual token is 'BANG_EQ', literals is '!='."

left join 后添加条件，先是等于条件，然后是!=条件 ，解析失败

For English only, other languages will not accept.

execute sql:"from clyj_goods_order_info as a LEFT JOIN clyj_goods_order_details as b on a.p_trade_no = b.p_trade_no" is not error.But add "and b.order_source != 6" After this statement


[Story][CL]: Pool Manager Module

Background

As a user, I would like the pool management to be unified so that I don't have to reason about additional complexity stemming from divergent pool sources.

Assume the pool management is not unified, then we might have to have prefixes for the pool ids. Not only this is hurting UX but it would also require duplicating the logic that could have been unified.
Therefore, we should implement a single module to handle pool management.
Risks
The problem with this approach is requiring migrations of the gamm module. Additionally, extra refactors in gamm would open up vectors for bugs and exploits.
Suggested Design
To mitigate this, we will incrementally approach this direction by beginning to integrate only the new concentrated liquidity module and pool into the pool manager.
As the final step, we will handle gamm migrations into the pool manager
Messages

CreatePool
JoinPool
ExitPool

We will add pool type field and optional ticks (depending on pool type) to messages that require it
Tasks

https://github.com/osmosis-labs/osmosis/issues/3048

Acceptance Criteria

new module is implemented
follows twap folder structure
utilizes gamm and concentrated-liquidity modules to route pool-creation logic
test coverage is maximized

TODO:

add task for creating the barebones module structure for achieving milestone 2

Note that the message in the pool manager module would route to the concentrated-liquidity module keeper method for the actual CreatePool JoinPool and ExitPool

Replaced by #3051


Created the basic maps.html

I've created a new html file for the basic map functionality and the functionality with entering an address in a search box and re-centering the map to the location of that address. The index.html file is slightly edited, but it should really contain the same code as the initial file (edits were mainly/only whitespace related).

Thanks @sanjayss34. Will review tonight!

It looks like there isn't any indentation in your map.html file. Indentation is really important for readable html and javascript! :bowtie:

Overall, this is a solid first start. You also win the medal for first pull request :trophy:
@nancywong do you mind reviewing this, too? You have experience with front-end and might have some comments.

or*

Also, by "you" I mean Alex

@sanjayss34 @apiatski when you guys are ready for another round of reviews let @nancywong and me know.

Still haven't gotten through the entire PR but looks pretty good

Some of my comments are hidden because they are on "outdated diffs." Make sure you go on the "conversation" page to see them since a few are still applicable.

In the future, you may want to squash similar commits (e.g. "pep8 stuff" 1-4) into one commit using git commit --amend

Over slack, you mentioned that a few functions use addMarkers. Can you explain what they use it for?

Ok here is what I would do.

Keep all the javascript in a separate file like it is now.
Update the addMarkers function to take a javascript object which contains all the data about reports. Instead of addMarkers reading report information from $(<some field>).data it will read that info from the reports javascript object.
You will a slight bit of javascript in the maps.html template which reads the reports data from jinja and converts it to a native javascript object. Once you have this javascript object, call the addMarkers function with it.

Here is a sketch of how this might looks. Note: I will start the serialize function (and you can modify it to meet your needs).
The view
import json

@app.route('/maps')
def my_view():
    data = [report.serialize() for report in IncidentReport.query.all()]
    return render_template('maps.html', data=json.dumps(data))

The template

ACTUALLY ignore that comment.
Here is what you should really do.

Define a new type of object called a MarkerWrapper in your javascript file

    function MarkerWrapper(actualMarker, incidentDate, contentString) {
        this.actualMarker = actualMarker; // actualMarker is of type google.maps.Marker, you should instantiate it outside of this constructor just like you do here https://github.com/hack4impact/clean-air-council/pull/2/files#diff-de047b44d935882c6e5ccd52a76f3ff4R63
        this.incidentDate = incidentDate; // a Date object, like here https://github.com/hack4impact/clean-air-council/pull/2/files#diff-de047b44d935882c6e5ccd52a76f3ff4R75
        this.contentString = contentString; // same as https://github.com/hack4impact/clean-air-council/pull/2/files#diff-de047b44d935882c6e5ccd52a76f3ff4R88
    }
2. Add code like this in your html file to create a list of MarkerWrappers and the map
```javascript

In your javascript files, define a function like this

var globalMarkerWrappers = null; // global variable
var globalMap = null; // global variable
function storeMarkerState(markerWrappers, map) {
    globalMarkerWrappers = markerWrappers;
    globalMap = map;
}

Also in your javascript files, define the dateSlider functions. Now, they should iterate through the list of MarkerWrapper in globalMarkerWrappers. For each one, if wrapper.incidentDate is valid make sure it is ON the map and if it is not valid make sure it is off the map

http://stackoverflow.com/questions/18444161/google-maps-responsive-resize

It looks like this PR is failing on circleci because of pep8 violations. Is flake8 working locally?

Looks pretty good except for a few minor things which I commented on. Once we get the modal working and tests passing, this should be good to merge.
@nancywong can you give this one final review?

Oh also, we should revert the merge from the report-public-private branch. I gave instructions on how to do this on slack.

@kongsally @sandlerben Is this branch ready to be merged with master?

I feel like there is a bit too much padding between the button and the search box. Also, the search box is round while the button is more rectangular. Are we sure we want to make the button gray?
@kongsally what do you think?

@above: I can decrease the padding between the button and the search box, make the button the teal color of the menu bar, and try to make the button round.

Before we merge, I want to verify that this looks right on smaller screens. I'm going to deploy a heroku review app for this PR.

I don't understand the purpose of the geolocation button. 1) I would think that the default should be using current location so that users will find it easier to locate places/incidents close to them, and 2) the user has to click both the new geolocation button and the allow to use location button to use current location. Why is this approach advantageous?

Sanjay we can chat about geolocation logic over slack or in person. I think this flow is preferable but we can easily change it back.


Parser problem in context of setopts

We need to solve the conflict of string replace destination information with method calls, like here for line two:
a$=opts
a$(3,1)=ior(a$(3,1),$02$)
setopts A$

@insafuhrmann
Do you recall where this example comes from? I wonder if 9 in a$9 as a typo.
However second line can be parsed now.

it's a typo. Removed it.

Fixed already. setopts verb is missing and will be handled in  #53


gulp once on linux

Running gulp once on linux  give below logs , seems good now . Please confirm if this expected
builder@rh6-intel64-80-qa-dev-2$ gulp once
[10:29:34] Starting 'once'...
[10:29:35] Starting 'clean'...
[10:29:35] Starting 'middle-tier-start'...
reconfiguring database, starting app server
./gradlew dbInit --no-daemon
[10:29:35] Finished 'clean' after 15 ms
[10:29:35] Starting 'bower-files'...
[10:29:35] Finished 'bower-files' after 629 ms
[10:29:35] Starting 'build'...
[10:29:36] processing files
...[10:29:38] saw 221 files
[10:29:40] Finished 'build' after 4.57 s
...........................................
./gradlew dbTeardown --no-daemon
.........................................
./gradlew dbInit --no-daemon
.............................................
./gradlew dbConfigure --no-daemon
.................................................................
./gradlew test --no-daemon
.......................................
/home/builder/.gradle/caches/modules-2/files-2.1/org.springframework/spring-context/4.1.4.RELEASE/5c66d0962e89ee577627a526b6b299a5337072be/spring-context-4.1.4.RELEASE.jar(org/springframework/context/annotation/PropertySource.class): warning: Cannot find annotation method 'value()' in type 'Repeatable': class file for java.lang.annotation.Repeatable not found
1 warning............
/home/builder/.gradle/caches/modules-2/files-2.1/org.springframework/spring-context/4.1.4.RELEASE/5c66d0962e89ee577627a526b6b299a5337072be/spring-context-4.1.4.RELEASE.jar(org/springframework/context/annotation/PropertySource.class): warning: Cannot find annotation method 'value()' in type 'Repeatable': class file for java.lang.annotation.Repeatable not found
1 warning....................................................
./gradlew dbLoad --stacktrace --no-daemon
.................................................
./gradlew bootrun --no-daemon
...................................................................................................................................................
[10:33:21] detected middle tier started
preparations complete
[10:33:21] Finished 'middle-tier-start' after 3.77 min
[10:33:21] Starting 'selenium-start'...
Skipping task selenium-start because there is no desktop environment
[10:33:21] Finished 'selenium-start' after 93 μs
Skipping task e2e because there is no desktop environment
[10:33:21] shutting down middle-tier
Samplestack configuration OK!
[10:33:21] Finished 'once' after 3.78 min

Yes.


Build kernel for all numbertypes for ApproximateEqual Op

"The Op ApproximateEqual registered for numbertypes as per kernel registration here.
https://github.com/tensorflow/tensorflow/blob/dadeddbce13b7d422c73ed872e386c56dd41a478/tensorflow/core/ops/math_ops.cc#L781-L783
But Kernel built for float and double. Proposing a fix for same.
Might Fix #64256 .

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!

Hi @rohan100jain, Can you please review this PR ? Thank you!


rkt install doesn't create /etc/rkt

$ sudo rkt install
rkt directory structure successfully created.

$ rkt run coreos.com/etcd:v2.2.3
rkt: using image from file /usr/share/rkt/stage1-coreos.aci
rkt: searching for app image coreos.com/etcd
rkt: remote fetching from URL "https://github.com/coreos/etcd/releases/download/v2.2.3/etcd-v2.2.3-linux-amd64.aci"
prefix: "coreos.com/etcd"
key: "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg"
gpg key fingerprint is: 8B86 DE38 890D DB72 9186  7B02 5210 BD88 8818 2190
        CoreOS ACI Builder<EMAIL_ADDRESS>Trusting "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg" for prefix "coreos.com/etcd" without fingerprint review.
rkt: error adding keys: Error adding key for prefix "coreos.com/etcd": mkdir /etc/rkt: permission denied
rkt: downloading signature from https://github.com/coreos/etcd/releases/download/v2.2.3/etcd-v2.2.3-linux-amd64.aci.asc
Downloading signature: [=======================================] 819 B/819 B
run: openpgp: signature made by unknown entity

This is the error:
rkt: error adding keys: Error adding key for prefix "coreos.com/etcd": mkdir /etc/rkt: permission denied

rkt run can only run as root. However, the error above is not user-friendly.
I have the same error as you when I use rkt fetch (which is supposed to run without being root)
$ rkt fetch coreos.com/etcd:v2.2.3
rkt: searching for app image coreos.com/etcd
rkt: remote fetching from URL "https://github.com/coreos/etcd/releases/download/v2.2.3/etcd-v2.2.3-linux-amd64.aci"
prefix: "coreos.com/etcd"
key: "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg"
gpg key fingerprint is: 8B86 DE38 890D DB72 9186  7B02 5210 BD88 8818 2190
	CoreOS ACI Builder<EMAIL_ADDRESS>Trusting "https://coreos.com/dist/pubkeys/aci-pubkeys.gpg" for prefix "coreos.com/etcd" without fingerprint review.
rkt: error adding keys: Error adding key for prefix "coreos.com/etcd": mkdir /etc/rkt: permission denied
rkt: downloading signature from https://github.com/coreos/etcd/releases/download/v2.2.3/etcd-v2.2.3-linux-amd64.aci.asc
Downloading signature: [=======================================] 819 B/819 B

I'm not sure it is a good idea to make /etc/rkt writeable by the rkt group. I think non-root users should not be able to trust new keys. @robszumski what do you think?

Seems like we just need to explain what's going on and die immediately.
$ rkt fetch coreos.com/etcd:v2.2.3
...snip...
rkt: error adding keys: Error adding key for prefix "coreos.com/etcd": mkdir /etc/rkt: permission denied
Only privileged users can add new keys to the key store

I feel like its not correct to say "root". Is "privileged" clear enough?
cc: @joshix

I think you're right that rkt run should exit immediately when encountering this, because it is fatal.
I'm not sure we need to adjust the error output at all, however. permission denied is the most well-known, and truest, thing we can print here. Print that, and die.

After thinking this over a bit, I think we need to move away from saying anything about the user, and @joshix agrees. What about:
$ rkt fetch coreos.com/etcd:v2.2.3
...snip...
rkt: error adding keys: Error adding key for prefix "coreos.com/etcd": mkdir /etc/rkt: permission denied
Permission denied when writing to the key store

As robzsumski knows from IRL chat, I would prefer to print the standard shell permission denied and exit, but will accept the extra friendly \nCould not write to key store.

can we hint at an action at least? "Could not write to key store. Ensure you have suitable permissions."

Sounds good to me. @joshix opinions on phrasing?

This guy with the beard says "permission denied" is that hint, @jonboulle.
But if we must, maybe:
[...]: mkdir /etc/rkt: permission denied
Cound not write to key store. Check permissions on $keystore.
where $keystore is usually /etc/rkt/, but is probably configurable, so should be variable in the err output.

sgtm

oh, oops, fixed by #2081


Decide whether to use phpredis

https://github.com/phpredis/phpredis

The answer is yes. It's being implemented.


Add placeholders to make the etcd cluster run in secure TLS mode

I wanted to update the docs but it turns out that the cf-stub comes from cf-release, hence my Pull Request here. I tried to keep this change as concise and small as possible, and I am sorry in advance if I might have missed something.

cf-release v241 release notes tell us:
[..]
This release introduces official support for running the etcd cluster
(shared by several components such as Routing API and the loggregator
subsystem, but not Diego which uses its own secure cluster) in secure
TLS mode. Upgrading an existing deployment with an insecure etcd cluster
to a secure one with minimal downtime is non-trivial. Instructions and
additional information for this procedure can be found here. If you are
using the manifest generation scripts included within the cf-release
repo to generate manifests, you're strongly recommended to upgrade to a
secure etcd cluster at this point. The instructions above assume you are
upgrading to a secure etcd cluster from a pre-v241 Cloud Foundry
deployment and will not apply as smoothly if you later attempt to
upgrade a post-v241 non-TLS etcd cluster to a TLS cluster within the
Cloud Foundry deployment.
[..]
See: https://github.com/cloudfoundry/cf-release/releases/tag/v241

Did you see my new PR above @staylor14?


update qemu to v8

update qemu from v7 to v8.
i think it should compile, once the flake is updated to a nixpkgs that uses qemu 8.
this qemu version should have libvfio-user patches as well as new support for 40G intel NIC-like device emulation.
This is still to be tested in practice.

with more recent nixpkgs: build fails because meson tries to check out git submodule for vfio-user

qemu builds. Uses fd interhiting orchestration py script from oracle/elena. With our python script at least, libvfio in the remote process (vmux) throws failed to receive version: Bad address

that qemu branch has new commits :)


why do you want to develop this tool?

https://github.com//PeachScript/vue-infinite-loading
There's already one fully developed tool vue-infinite-loading, why do you still want to develop this one?
Any way, I respect your work.

This project is probably older than the one you mentioned.

@zzzgit that's the beauty of open source: Everyone can build something just because they feel like it. Consisted the world not of those people but your kind only, we couldn't even dream of all the amazing software we have around now.
Your issue adds nothing, but disrespects the author and every contributor to this library grreatly. Sometimes it might be wiser to say nothing at all if you don't have anything to say.


Terminal Window does not parse line number for IAR compiler errors

Issue Type: Bug
Using an IAR cross-compiler, if you click on the error, VS Code always navigates to line 1 of the file instead of the line with the error.  Here is some example syntax:
  myFunction();
             ^

"C:\repos\file.c",80  Error[Pe165]:
too few arguments in function call
  pVar = Var;
       ^

"C:\repos\file.c",81  Error[Pe513]:
a value of type "uint32_t" cannot be assigned to an entity of type
"uint32_t *"
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:59:35.033Z)
OS version: Windows_NT x64 10.0.18362

https://github.com/microsoft/vscode/commit/183deef5f43d77dc0c13e8accc41f5dff492a434


Nested tabs inside stacknavigator with swipe enabled crashes app

Current Behavior
Navigating from route A to route B at stackNavigator where route B screen is tabsNavigator (so tabs is nested inside stack) causes app to crash on IOS  without no clear error message in logs. Bug is ONLY happening when either swipeEnabled=true OR animationEnabled=true in TabNavigator config AND it only happens when I first have visited inside Tabs and go back to top level stackRoute and when trying to go again to Tabs --> crash. I wish this could be easier to explain. Any tips are highly appreciated!
Here's my navigators:

const AppTabNavigator = TabNavigator(
    {
        TabOne: { screen: TabOne },
        TabTwo: { screen: TabTwo },
        TabThree: { screen: TabThree },
    },
    {
        swipeEnabled: true, //setting this to false prevents app crashing
        animationEnabled: true, //and this too
        tabBarPosition: 'bottom',
        tabBarOptions: {
            showLabel: true,
            showIcon: true,
        }
    },
);

const AppStackNavigator = StackNavigator(
    {
        Tabs: { screen: Tabs }, //into this one
        Foo: { screen: Foo }, //app crashes when navigating from this screen ^^
    }
);

export default class Tabs extends Component {
    static propTypes = {
        dispatch: PropTypes.func.isRequired,
        tabNav: PropTypes.object.isRequired,
    };

    static navigationOptions = {
        header: null
    };

    render() {
        const { dispatch } = this.props;
        return (
            <View
                style={{ flex: 1 }}
            >
              <AppNavigator navigation={addNavigationHelpers({ dispatch, state: this.props.tabNav })} />
            </View>
        );
    }
}
@connect(
  state => ({
      stackNav: state.stackNav,
      user: state.auth.user
  }),
  dispatch => ({ dispatch }),
)
export default class AppWithNavigationState extends Component {
    static propTypes = {
        dispatch: PropTypes.func.isRequired,
        stackNav: PropTypes.object.isRequired,
    };

    render() {
        const { dispatch, stackNav } = this.props;
        return (
          <View
            style={{ flex: 1 }}
          >
            <AppStackNavigator navigation={addNavigationHelpers({ dispatch, state: stackNav })} />
          </View>
        );
    }
}

Expected Behavior

Your Environment

software
version

react-navigation
beta.11

react-native
0.44

node

npm or yarn

i am also experiencing this with a similar setup. The js error is:
navigation.dispatch is not a function. (In 'navigation.dispatch(_NavigationActions2.default.back({ key: backFromScene.route.key }))', 'navigation.dispatch' is undefined)

Anecdotally, I am seeing something similar using StackNavigator, go to a Component that renders a WebView. When doing the swipe from left to right gesture to dismiss it off the top of the stack, once the animation is done and the previous card is showing we get this error. Only on a real iOS device, not on the simulator, both running a Debug build.
[error][tid:com.facebook.react.JavaScript] navigation.dispatch is not a function. (In 'navigation.dispatch(_NavigationActions2.default.back({ key: backFromScene.route.key }))', 'navigation.dispatch' is undefined)

@jslz 你应该是自定义的router吧，使用redux自定义就是有这个问题，没办法只有把手势设置为false，貌似手势是写在里面的，不知道能不能继承，或者是自定义。

Pinging OP @ovaris since this issue is quite old and related to an old version of the lib. If no answer I'll close in 7 days.

Hi there @ovaris ,
In an effort to cleanup this project and prioritize a bit, since this issue had no follow up since my last comment I'm going to close it.
If you are still having the issue with the latest version (especially if it's a bug report) please check the current open issues, and if you can't find a duplicate, open a new one that uses the new Issue Template to provide some more details to help us solve it.

I know this issue is closed but I'm still getting the same problem with the same scenario.

Did anybody find a solution for this? @ovaris @xiaoyongchen @jslz

can someone who would like to see this issue resolved please create a new issue with a reproducible example on https://snack.expo.io or in a github repository, with a minimal amount of superfluous code? (please don't post your entire app, just a small one to reproduce the problem). thanks!


pagination meta-data support

Fetch gateways may return pagination meta data along with the data itself

enough by https://github.com/Mazuh/resource-toolkit/commit/d5c7471f4e0373701eb55695276f70e7d7e4a388


bugfix in recv to read full length

Currently, recv() fails if the line to be received is greater than 65535 bytes, irrespective of the size value. This fix makes sure that recv() reads size bytes.

Hi any update on this pr? The error in travis is only for Go tip because location of go vet has changed. and has nothing to do with my change.
Without this change, a any job with 100s of inputs will fail is the list of inputs exceed 65535 bytes.


Stop / Cancel command?

Any plans to add a stop or cancel command for stopping a deployment that is already in progress, similar to the Stop-AzureRmResourceGroupDeployment PowerShell command?
https://docs.microsoft.com/en-us/powershell/module/azurerm.resources/stop-azurermresourcegroupdeployment?view=azurermps-6.13.0
The delete command cannot delete a deployment if it's still active.

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

ID: de233487-a2e5-00ab-30e9-f68866735957
Version Independent ID: 2cd3ed9f-113f-a6c2-6a36-2925a044edf8
Content: az group deployment
Content Source: src/azure-cli/azure/cli/command_modules/resource/_help.py
Service: azure-resource-manager
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@Juliehzl Please help with this issue.

add to S167.


JSI function for transcribe audio buffer

Provide JSI function for transcribe audio buffer, so we can use library like react-native-audio-pcm-stream or from another source, and we can manage recorded audio samples on JS without writing platform specific code.
Compare to native bridge, JSI can convert buffer from JS in high performance.

Any progress on this one? :)

Hi, bumping this up as well. Looking forward to integrating the live transcription into https://github.com/deeeed/expo-audio-stream/blob/main/packages/expo-audio-stream/README.md .

Hi @jhen0409 ,
After implementing non-live transcription in my audio playground, I'd like to discuss approaches for live transcription integration between expo-audio-stream and whisper.rn. I see two main paths:

JavaScript-Level Integration (Currently Implemented)
Approach:

Using expo-audio-stream's base64 PCM data stream
Interfacing with whisper.rn's transcribeData API
Managing buffering and transcription state in JavaScript
Pros:
Simpler to implement initially
More flexible for different use cases
Platform-agnostic implementation
Cons:
Multiple base64 conversions
Higher memory usage
JavaScript bridge overhead

Native-Level Integration (Proposed)
Approach:

Direct PCM data handling at native layer
Add new native methods in whisper.rn for streaming PCM
Implement efficient buffer management between expo-audio-stream and whisper.rn
Implementation Suggestion:
Add startRealtimeTranscribeWithAudioInput method to handle streaming setup
Implement receiveAudioDataChunk for direct PCM data processing
Connect directly to whisper.cpp's audio processing pipeline
Use circular buffers for efficient memory management

Questions:

Would you be open to adding these new streaming-focused methods to whisper.rn? This would allow direct PCM handling without base64 conversion overhead.

Should we extend the existing realtime API or create a new pathway specifically for external audio sources?

I believe the native integration path would provide better performance for real-time use cases, but I'd appreciate your thoughts on this approach. Happy to contribute PRs once we align on the best path forward.
Looking forward to your feedback!

I'd like to extend current transcribeRealtime implementation to support other audio sources, it may like this:
transcribeRealtime({
  /** [NEW option] Choose audio source (custom: put by yourself in JS or native side */
  source: 'built-in' | 'custom',
  // ...
}): Promise<{
  /** Stop the realtime transcribe */
  stop: () => Promise<void>
  /** Subscribe to realtime transcribe events */
  subscribe: (callback: (event: TranscribeRealtimeEvent) => void) => void
  /** [NEW method] Put audio buffer (Buffer or base64 encoded string) for `custom` source */
  pushAudioDataChunk: (data) => void
}>

The pushAudioDataChunk native method will implement by cpp/JSI directly, it may use the same method with react-native-blob-jsi-helper. It would be better if we could move context pool & jobs into JSI, so we don't need to have to use Blob module and avoid JNI costs on Android, but that would probably a big refactor.
Also, we can expose a static method for put audio data to a realtime-transcription job in the native side, so that can use by a custom audio stream native module.
For the transcribeData method, we will also support array buffer by use the same method with blob-jsi-helper, this is the main purpose in this issue.


Update CrawlTool to always require a user agent name and webpage

Remove the default UserAgent settings from the CrawlTool so that a user is required to provide the agent name, web address and email address.

Although this isn't required for the Flink Forward talk, I feel like I should take this on. I did some initial work in this area, and @vmagotra is on vacation this week.

I handled this via https://github.com/ScaleUnlimited/flink-crawler/commit/0efd2e71ced07e0fc4c3aac80338005fd75d586b#diff-2a070fbbece0a7e69b81a5240a997585


Notes from the Python Cryptography class

Hi, here are some notes from the pyclass.
Thanks!

Cool - can I submit the binary if it's a pdf? I added a pdf extension.

removed the empty flashcards directory


Fix auto-scroll in chat box

If the user clicks inside the chat box, it will stop auto-scrolling until they refresh the profile. It should begin auto-scrolling again after they de-select the box instead.

Resolved in 700d9108a8c15e2a5aefc8210ed0de09ac364e26.


CRS-1: create MVC classes and html pages

Create MVC classes and html pages
Acceptance criteria:
MVC classes and html pages created.

MVC classes and html pages created.


full documentation?

where i can find full documentation? so short and ununderstandable here

Hi @shady11, to be honest this project is currently mostly used internal and therefore has no full documentation, sorry about that.
I'll keep this open to track we might want to add some documentation in the future which should make it possible for everyone to use this package.


Add a physical input for saving magnetic Calibration

@DanielG86 also suggested having a physical switch to trigger the saving of a magnetic calibration would be useful, in addition to the command-based software method.

Implemented using digital input, debounce transform, and lambda consumer. See examples/example_physical_switch.cpp, which has comments on how to use and how to modify.
Tested on ESP32 and ESP8266.

Implemented using digital input, debounce transform, and lambda consumer. See examples/example_physical_switch.cpp, which has comments on how to use and how to modify.
Tested on ESP32 and ESP8266.


Fix #193

Hi @mattiasd - I'm going to close this PR because that's generally not how I return values for null/empty checks.
I usually do a check BEFORE I call and handle it there.
So i'll update the code, now.


make apex

When I use the "make apex" command to install apex, it cannot be installed, and then when I use "make apex.done" to install, the following error occurs:

How can I solve it? Thank you.

Maybe apex support only the latest python.
Could you try with python 3.9+ & latest pytorch?

Yes.
I installed python 3.9 later, but I still have this error, and the error also occurs when training with DDP. In the end, I modified some codes myself so that the DDP method can run normally.
Thanks for your reply! Wishes.


Late bind CopyUpToDateMarker

PR #2180 caused a performance-test scenario internal to Microsoft to
start failing. The root cause was that the projects in the test
customized $(IntermediateOutputPath) very late, after importing
CSharp.targets (and transitively Common.targets). That meant that the
path stored in $(CopyOutOfDateMarker) was no longer under
$(IntermediateOutputPath), and that the directory wasn't created before
the new code attempted to touch the marker file.
Ideally, projects would set IntermediateOutputPath before importing
common.targets, but that hasn't been strictly required in all cases
before so requiring it now is an unacceptable regression.
The late customization was handled for other intermediate outputs by
defining their paths with items (expanded after all properties are
expanded) rather than properties (expanded in order).
There are other properties defined by using $(IntermediateOutputPath),
$(_GenerateBindingRedirectsIntermediateAppConfig) and
$(WinMDExpOutputPdb). This PR doesn't change those since they weren't
recently regressed.

@panopticoncentral This will impact the FUTD check implementation. I'll add some notes in https://github.com/dotnet/project-system/pull/2414.
@jcouv FYI, related to ref assemblies--when I implemented the marker code, I imitated things that were subtly broken in some customized projects. This implementation imitates the things that work.


Could not find a declaration file for module 'eslint-plugin-import'

Using latest eslint with flat config in a TS project.  Need type declarations.

We don't currently ship any (and they're not required to be able to use the plugin)

Try this:
npm install eslint-plugin-import --save-dev

as part of https://github.com/import-js/eslint-plugin-import?tab=readme-ov-file#installation

Yeah, so the issue it that typescript-eslint tells you to put //@ts-check on your config file which then produces a slew of errors due to missing type declarations.  To be fair this is not the only package that is missing type defs, so yeah.  For now I am just removing that, really no need to be on the config tbh.
Would still be nice to have them, but probably not a priority.

have the same type checking problem, but the plugin itself works


Model vs Schema

Model and Schema are very confusing topics for beginners. Hope this will help them

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@Sibinjoseph111 it looks like this was copied directly from a Stack Overflow answer - we can't accept plagiarism in the Guide.


Overriding configuration key-value pairs only works for built-in stages

Having an external stage
[external-stages.circt]
location = ...
exec = ...
toplevel = "main"

it is currently not possible to manipulate the toplevel through the -s parameter of fud exec. Only built-in stages (i.e. stages with a [stage.${stagename}] header in the config file) can have their key/value pairs overridden.

Thanks for catching this @mortbopet! If possible, could you provide a gist of the external stage you were trying to add so that we can debug it directly?

Sure! I'm working on https://github.com/circt-hls/circt-hls/blob/main/stages/CIRCT/stage.py where I'd like to run with different values for the circt.toplevel variable:
fud exec "Polygeist/mlir-clang/Test/aff.c"  \
  --from c                                  \
  --to mlir-calyx                           \
  -s circt.toplevel "kernel_deriche"`

however, I'm having to resolve to using a configuration like the following:
[stages.circt]
toplevel = ""

[external-stages.circt]
location = "/home/morten/Work/circt-hls/stages/CIRCT/stage.py"
bin_dir = "/home/morten/Work/circt-hls/../circt/build/bin/"
llvm_bin_dir = "/home/morten/Work/circt-hls/../circt/llvm/build/bin"

with toplevel being listed under a [stage.circt] header, instead of [external-stages.circt]. As far as i can tell, the -s argument parsing assumes that the provided configuration key is listed as a stage and does not consider external-stages.

Oh, fantastic! This looks like a super cool way to use fud. I'm going to tag @EclecticGriffin & @cgyurgyik if they get the time to look at the bug before I do.

So far I'm really liking the driver! ... I'm only wondering about why it's nested within Calyx and not a separate project - something like fud could be really nice for a general lightweight/standalone pass manager infrastructure.

Largely because it started off as a way to run Calyx-related tools and we don't have any external users yet 😅. I think it makes a lot of sense to move it into a separate project in the long-term because it supports a lot of tools now. We'd need to document it better and generalize the code but if this sounds like something you'd be interested in helping with, we can create an issue to get started!


Display node values

From https://github.com/ricklupton/ipysankeywidget/issues/10

This is now implemented in v0.7.0: see for example this gist


Cache expiry

When we set the package for auto cache it downloads and cache the ads. But after a while the app is becoming slower and slower on some devices it’s crashing.
Once the ad is displayed is the cache of that ad gets deleted ? How to handle his?

If you want to hide the banner from all activities and clear the memory, call the code below:
Appodeal.destroy(Appodeal.BANNER);
I’m talking about destroy function it’s not working on your package

According to Appodeal's documentation the ads that are auto-cached are automatically freed from memory after they are used. As far as I know there's no way to cache the same ad type more than once. My plugin simply uses the Appodeal code, but I didn't change anything on how the cache works.
In any case, I might have some time to work on the implementation of the destroy function in the next couple of weeks, but you're welcome to open a PR also if you want things to move a little quicker 😃

Please add support for smart banners and a method to destroy banner ads. How do I show different banners on different screens?


Risk objects require fields be present

Details
Added a field to two detections so that risk objects could be created.
Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

Looks great!


Open Preview in own Tab should set correct url

Sometimes we get from the content managers urls like the following /admin/content/preview/74638779-ad74-4b02-a889-dae4428b0695/r..
where we need to do changes. This Urls come from the preview (when it is opened in new window/tab).
It would be good when the preview would update the url to the correct one for example with history.pushState. So the shared links by the content managers works.

@alexander-schranz Can you check in 0.16, please?

@berndhep it is still the temp url(/admin/content/preview/746...) after clicking on

instead of /de/... tested on version 0.16.1

Now I know what this is about... @wachterjohannes are you sure this is an easy pick?

@danrot @wachterjohannes Is it a preview bug? Them we should fix it.

It's not really a bug. The only thing is that if you open the preview in a new window, it doesn't show the real URL of the page.
Compare the URL in this window (preview):

And in this one (the real page):

Alex wants them to be the same. But I am not sure if this is easily possible at all.

Ok. Then let's keep it in mind but we don't plan it for a near future release.

@berndhep i will do this in a free minute it is not a big thing!


for "all time", time series API tends to generate one extra point at the end of the series

It appears somewhere in the pipeline, we're adding a day. IT coudl be in the API, or it could be somewhere in the dashboard code.

Backend work is resolved here: https://github.com/rilldata/rill-developer/pull/1757

According to Eric, this doesn't require further frontend work.


Add missing components to Button theme

Description
The Button and BC Gov themes don't have exactly the same components. This issue is about creating the missing ones in each theme.
User Story
As a developer
I want the same components to be available in both themes
So that I can switch between themes and have more components available
Acceptance Criteria

Given
When
Then
Pass/Fail (TBD by reviewer)

precondition/initial context/state of the system. Ex: I am on Product page
event/trigger. Ex: I click Buy Now
output/action. Ex: I am redirected to Payment page

Research + Rationale Summary
Why should this feature be added/modified?
Prototype (if applicable)
Developer notes

[ ] open this repo, change this file

@BCerki and @mikevespi , Can we add the checkbox list of missing components in the description?

@eazeredo I'm going to change this card to be more of an exploration because I think we're going to need designs before building anything missing

Button theme is missing:
-[ ] Alert
-[ ] Callout
-[ ] Card
-[ ] Fieldset
-[ ] HeroImage
-[ ] Link
-[ ] Modal
-[ ] Navigation
Gov theme is missing:
-[ ] Notification
Also noticed:

Button theme doesn't have any of the Getting Started introduction that the bcgov theme has
Button theme doesn't have explanatory intros (*.stories.mdx files) that bcgov has
file structure is different between themes
what exactly are the two "themes"? It looks like the Button and bcgov folders each have their own Accordion etc. file and story. Are the the components from each theme connected at all?

Here is the ticket that meets the AC: https://github.com/button-inc/service-development-toolkit/issues/482


"Tags" should not be displayed as per the Desktop Entry specification

The Desktop Entry spec for the "Keywords" entry states the following:

The values are not meant for display

For many applications, these values are inconsistently capitalized, and as such they should probably not be displayed on the website.
At the very least, they should be displayed in all-caps, so we avoid having gnome, Gnome and GNOME.

Uppercasing should be something we can do.
Not displaying them is not really an option as far as I know. As if you want them to enable searching, we have no other way to expose them to search engines (outside of our own search)


Nix: remove wayland-protocols overlay

wayland-protocols 1.35 already merged, the overlay can be removed now @fufexan

Thanks. I'm at work now, I'll do it later if no one makes a PR in the meantime.


🚧 (LTI) introduce deep linking

Purpose
Introduce content selection through deep linking request with Moodle.
Proposal

This pull request primarily addresses backend setup, followed by the introduction of a straightforward single-input React form. This form guides users to a designated Demo page upon navigation.
Then, a second pull request will be created, featuring frontend options for selecting either a dashboard or a plot. The following implementation of conditional lazy routing will ensure the appropriate resource is displayed in the frontend.
Please note, django-lti-toolbox needs some updates, and this PR will still be a draft until the next release of the toolbox Python library.

[X] Solve the allowed host TODO item in FastAPI conf.py.
[x] Try locally PR#8 of the Django LTI Toolbox lib (TODO item).
[X] Extract content type message to a proper Enum (TODO item).
[X] Pass parameter to _do_on_success method (TODO item).
[X] Documentation.
[X] Reorganize Django Apps.
[X] Minimal frontend Content-Item selection form.
[X] End-to-End deep linking request flow (from the TC back to the TC).
[ ] Access token while passing form data to the frontend  (TODO item) and back to the server (TODO item).
[X] Fix app's test.

Documentation
Useful links:

In Marsha, docs/lit.md (requests payload examples).
Deep Linking Specifications here.

Moodle setup
Start by creating a free trial Moodle cloud instance using the following link: Moodle Cloud Instance. Alternatively, you can utilize the developer sandbox available here: Developer Sandbox.
To make your local Django server accessible, employ ngrok for tunneling.
While configuring Moodle, remember to include a legacy LTI tool. Pass the URL to the /lti/config.xml file. Additionally, access the Warren LTI tool configuration and enable deep linking. For receiving the Content-Item Selection LTI request, use the /lti/select route.
You can find detailed instructions for these steps in the Marsha documentation, specifically in the docs/moodle.md file.
Ensure to update the privacy settings of the Warren LTI tool. This step is crucial as it allows the user's identity to be transmitted to the LTI Provider during requests.
Lastly, within your profile settings, set a User ID. This extra configuration is necessary for proper validation of the user form by the LTIRequestView.
Next Steps

Handle authentication.
Handle LTI role checks.
Improve UX selection.

Until PR#8 in django-lti-toolbox is merged, please update the dependency under pyproject.toml:


[Bug] Unicode characters not displayed properly

Insomnia Version: 6.0.0
Operating System: Windows 10 x64 v1709

Details

In the response preview panel, Unicode characters are not displayed properly. For one character, it remove the one preceding it. For example, arrivée become arriée. And if multiple Unicode character are written, it bugs and show some part of the Unicode code (example: arrivéééée become arrié\u00eé\u00e9e).
The response is correctly received from the server (as shown by the Raw data panel). It only appear when the response is viewed in the Preview panel and the Source code panel.
The issues wasn't there in older version (my colleague still has the 5.16.6 version and doesn't have this problem).

Same as #1071

Duplicate of #1071 and #1071


Template update process on Copier 9.3.0/9.3.1 is slow/indefinite on Windows compared with 9.2.0 for repos with lots of .gitignores (I suspect a new copytree operation)

Describe the problem
My template https://github.com/blakeNaccarato/copier-python is used for my project https://github.com/softboiler/boilercv, among others. Since Copier 9.3.0, I notice behavior where the cached repo contents in %TEMP% during e.g. copier update --vcs-ref=HEAD is 17GB+, and includes even .gitignored content such as .venv, and in this case, a .dvc/cache directory containing ~2000 files.
I think that this manifests as Copier trying to update from template and somehow starting to make an insanely large diff against the 17GB+ directory copied to %TEMP%. It causes a long-lasting hang during the template updating phase, which was at least fifteen minutes long before I cancelled the operation.
In Copier 9.2.0 and prior, I see the cached repo contents over in %TEMP% is only ~1GB, as expected, and doesn't venture into .gitignored territory. In comparing diffs across your tagged releases, I see the following section where a copytree operation seems to take place, only ignoring the .git folder. My suspicion is that this may be involved. See below for the permalink to the potentially offending code snippet.
By updating the template with Copier 9.2.0 instead of 9.3.0 or 9.3.1, I get the relatively fast template update operation that I expect.
https://github.com/copier-org/copier/blob/26c811f2f3152549c9f5d908cba79f142c0eccd6/copier/main.py#L936-L948
Template
https://github.com/blakeNaccarato/copier-python
To Reproduce

Potentially, be on Windows, but this may be OS agnostic
Have a repository with lots and lots of .gitignored files (perhaps "lots" by number of files, or by size on disk, or both)
Run copier update on this repository, with Copier 9.3.0 or 9.3.1
Wait forever, copier update will not finish

Logs
No response
Expected behavior
Template updating should behave as in Copier 9.2.0, with no major slowdown and no massive copytree operation sending ~17GB to %TEMP%.
Screenshots/screencasts/logs
No response
Operating system
Windows
Operating system distribution and version
Windows 11
Copier version
9.3.0
Python version
CPython 3.11
Installation method
pipx+pypi
Additional context
No response

Thanks for reporting this, @blakeNaccarato! It's a performance regression, you already traced it back correctly. I'll look into it asap. Sorry for the inconvenience.

Now that you mention it, I noticed this slowness too on Linux since 9.3.0.

No worries @sisp, glad I could point in the right direction! Regarding inconvenience, I think Copier must have saved me hundreds of hours of inconvenience over the past couple years, so I'm not too worried about a little inconvenience caused. Thanks!


bugfix fixes #1191

removes declaration for unsupported modules
adds exclusions for unsupported modules

Coverage remained the same at 100.0% when pulling a656cc946e28f380f0d6a846b247394288e70353 on baywet:bugfix/pnpJSunsupported into 04754dbedc05e8504c1b616b095f58eb1a14c8e7 on pnp:dev.

Thanks @baywet! ❤

@waldekmastykarz done!

Awesome! Appreciate the quick turnaround 👏

@baywet , @waldekmastykarz , would it be a good idea to include a test-project similar the spfx-updgate test projects so we can more quickly do end 2 end testing. To test pnpjs, I have to spin a new project with it inside, If we had this under test project I could do npm i and test it manually.

I'm already piggy backing on the sample projects of  the upgrade command in unit test if you look at it.
The reason I'm re-using it is to stay lightweight. We could simply add a few dependencies there and be good to go.

We should keep the SPFx test projects vanilla because we're using them for testing the upgrade logic of SPFx project. We should avoid adding anything custom that could introduce side-effects to the upgrade process. If having a separate test project would make it easier to run e2e tests, then we should definitely do it. There is very little overhead in having these test projects and we're not shipping them with the packaged version anyway.

ok, I'll include a copy of the test project as part of this PR and remap my unit tests.

done!

Merged manually. Thanks!


docs(dart/toh): add intro & toh-1 pages

A quick conversion of the first two TOH pages to Dart, using #700 as a basis for toh-1.
(Thank you, @kasperpeulen!)

I'm going to merge these now.


Multi word commmands / subcommands without quoting.

Is your feature request related to a problem? Please describe.
In recent years it's become more and more common to split the usages of a program into multiple sub commands, i.e. cargo has cargo run, cargo build, ... each with their own, specific help. But halp doesn't support multi word commands without quoting.
Describe the solution you'd like
It would be great if one could just do halp cargo build and get help for the build subcommand.
Describe alternatives you've considered
It's currently possible to get that help by simply quoting the command halp "cargo build", therefore it might not be worth the effort to support multiple words further than escaping.

It seems like a reasonable feature, but I took a fast look at the current implementation for the CLI arguments parsing and I can't find an inexpensive way  to implement this feature


Codeaction for exporting unused top-level bindings

This codeaction lets you export unused top-level functions, data types, newtypes, type synonyms, type classes, type families and pattern synonyms.

When exporting data types/newtypes/typeclasses/type families, we export all (e.g. Foo(..)).
We don't support extracting the data constructor/fields when the corresponding type is already exported. e.g.

module X(Foo) where
Foo = Foo {foo :: ()}

This is done because it would require us to find the corresponding type in the export list.
Fixes https://github.com/haskell/haskell-language-server/issues/157

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

I did that because it was easier to implement but now that I think about it, it would be just as easy to append it at the end (in the same line as the closing bracket). I'll update the PR.

I just realized there is an edge case when appending to the export list -module X(foo,) where and module X(foo) where. We'll have to look for that last comma.


[bug]: Using input/textarea inside of Dialog which is triggered by a DropdownMenu

Describe the bug
<DropdownMenu>
        <DropdownMenuTrigger asChild>
          <Button variant="ghost" className="h-8 w-8 p-0">
            Open menu
            <MoreHorizontal className="h-4 w-4" />
          </Button>
        </DropdownMenuTrigger>
        <DropdownMenuContent align="end">
          <DropdownMenuLabel>Actions</DropdownMenuLabel>
          <DropdownMenuItem onSelect={handleFullscreen}>

              <Eye className="h-4 w-4" /> View

          </DropdownMenuItem>
          <Dialog>
            <DialogTrigger asChild>
              <DropdownMenuItem onSelect={(e) => e.preventDefault()}>

                  <CheckCheck className="h-4 w-4" /> Review

              </DropdownMenuItem>
            </DialogTrigger>
            <DialogContent className="max-w-2xl">
              <DialogHeader>
                <DialogTitle>Review content of collaborators</DialogTitle>
                <DialogDescription>
                  Choose the final status for this content
                </DialogDescription>
              </DialogHeader>
              <ContentReviewStatus
                initialStatus={data.status!}
                contentId={data.id}
              />
            </DialogContent>
          </Dialog>
          <ConfirmModal
            header="Delete this content?"
            description="This will delete this content completely"
            disabled={false}
            onConfirm={onDelete}
          >
            <DropdownMenuItem onSelect={(e) => e.preventDefault()}>
              <Trash className="mr-2 h-4 w-4" /> Delete
            </DropdownMenuItem>
          </ConfirmModal>
        </DropdownMenuContent>
      </DropdownMenu>

The code of ContentReviewStatus component:
<form onSubmit={handleSubmit} className="space-y-4">
      <Select
        value={status}
        onValueChange={(value) => setStatus(value as ReviewStatus)}
        disabled={isPending}
      >
        <SelectTrigger className="w-full">
          <SelectValue placeholder="Select status" />
        </SelectTrigger>
        <SelectContent>
          <SelectItem value="pending" disabled>
            Pending
          </SelectItem>
          <SelectItem value="accepted">Accepted</SelectItem>
          <SelectItem value="declined">Declined</SelectItem>
        </SelectContent>
      </Select>
      {status === "declined" && (
        <Textarea
          placeholder="Add your review comments here..."
          value={comment}
          onChange={(e) => setComment(e.target.value)}
          disabled={isPending}
          className="min-h-[100px] w-full resize-none"
          rows={5}
          maxLength={1000}
        />
      )}
      <Button
        type="submit"
        disabled={isPending || status === initialStatus}
        className="w-full"
      >
        {isPending ? "Submitting..." : "Submit Review"}
      </Button>
    </form>

If I move the Dialog inside of DropdownMenu, then when the Dialog is closed, I can click on the DropdownMenuItem, but cannot type anything in the Textarea.
But if I move the Dialog outside of DropdownMenu, then I can type in Textarea, but then when Dialog is closed, I cannot click on DropdownMenuItem.
How to fix this issue? Many thanks and appreciation for your help.
Affected component/components
Input, Textarea, Dialog, DropdownMenu
How to reproduce
Dialog inside of DropdownMenu

Click on the DropdownMenuItem
The dialog pop up
Cannot type in the Textarea

Dialog outside of DropdownMenu

Click on the DropdownMenuItem
The dialog pop up
Can type in the Textarea
Dialog closed
Cannot click on DropdownMenuItem

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
Browser

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

NVM, I just found a solution: https://www.youtube.com/watch?v=JwNf4ujqsFU&ab_channel=HosnaQasmei
Even though it's kinda need more steps to create a state to check for Dialog open or not, but in general it works.


'None' type being returned from valid Boxscore URI instantiation

Describe the bug
Trying to get data for a Boxscore (I have tried multiple valid URI id's for NBA games
To Reproduce
!pip install git+https://github.com/davidjkrause/sportsipy@master
import sportsipy
from sportsipy.nba.boxscore import Boxscore

game_str = "201806080CLE"
boxscore = Boxscore(game_str)

Expected behavior
I would expect a valid Boxscore object to be returned.
Screenshots

Appears to be a setup or environment issue, as this is working for me.  Output:
>>> import sportsipy
>>> from sportsipy.nba.boxscore import Boxscore
>>> game_str = "201806080CLE"
>>> boxscore = Boxscore(game_str)
>>> boxscore.away_players
[<sportsipy.nba.boxscore.BoxscorePlayer object at 0x7feab07386a0>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea88087f40>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880ea500>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9c90>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9e70>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9ea0>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9d80>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9db0>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9c30>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9de0>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9f60>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880ea020>, <sportsipy.nba.boxscore.BoxscorePlayer object at 0x7fea880e9e40>]

One other person who was trying to use the fork ran into an issue where they had already installed sportsipy from pip, and then installed the fork afterwards, which causes issues.


Edit service UI issue

Describe the bug

Steps to reproduce

Create a service using https://github.com/unitong/test-variable version 0.0.7.
Edit and change to version 0.0.8.

Result

Expected behavior

Display properly.
Environment

UI version: dev-e6fb7f8

Additional context
Add any other context about the problem here.

Environment

Walrus version: main 8fc1ee8f23f152bbeaf1aec685c03d15ffbccae1
UI version: dev-199faf2

Test Result: No empty option after switching the version. But it does not show the default values https://github.com/seal-io/walrus/issues/1656.


Technologies Used?

Question or Support Request
Describe your question or ask for support
I would love to help out, but I don't exactly know what technologies are going to be used. Could you provide a list so I could study them? There's lots of new stuff I have encountered ever since I started contributing, so this would be extra helpful.

The website would be developed in NextJS 14 App Router due to its decent SEO

You can take a look at the dev branch, I've setup the project there.
Still unable to setup tests with Jest though, if you know a way, that would be great.
Will merge to main in a few hours

Whoops, I didn't know there was a dev branch. Makes a lot more sense now


All duplicated checkboxes contain same value on save

How to reproduce the bug:

Create a check-box control
Provide two options: Yes, No
Duplicate the check-box control
in the second check-box, provide two different options: Average, Good

On saving the form, both check-boxes contain the same values: Average, Good
If I change the 1st check-box, 2nd check-box will now have the same value as the 1st one.
Note: On the viewer UI checkboxes appear to have different value, but in the edit panel both checkboxes have the same values.

@ajb Could you pleasehelp me out to achieve multi column functionality so that I can drop multiple elements in a single row.
Thanks for an awesome plugin.


Added my name to Contributors.md

First pull request

@DB2104 congrats on getting your first PR merged :tada:


[Docs] “Proxy” components don’t include proxied props in documentation

Description
When looking at the documentation of a “Proxy” component (a component which “just” passes props to another one exploiting its functionalities) “inherited” proxied props are not listed.
Example
A clear example is MenuItem which looks like it doesn’t supports a disabled prop, while actually it does (since it comes from ListItem).

I have forgotten to add // @inheritedComponent ListItem for the MenuItem.js.
Do you want to add it? This automatically adds an Inheritance paragraph in the docs of the API.

Hold my beer™.

Also, If you have a better wording in mind for the documentation, let us know!

(Sorry for so many tentatives in getting the commit [namespace] right 😅 )

Also, If you have a better wording in mind for the documentation, let us know!

I believe having the list of props directly in place (along side the reference to the inherited component) would be easier to understand.

I believe having the list of props directly in place (along side the reference to the inherited component) would be easier to understand.

It was simpler to implement this way. For anyone else looking into this issue, here is an example.


Is it in the goals building and running other C programs directly from Gramado?

G'day to Gramado developers.
I've been following this work since it started (I think since 2020, if I'm not mistaken) and I was thinking if it would be possible to build other C programs written to UNIX on it.
I haven't looked at the source code and documentation yet, but I'm asking this because I thought about building the Heirloom NG set of utilities on it and see if it can run/the toolset that I maintain is portable enough.

I have some few programs using only the libc. Take a look at the folder gramado/core/commands/  ... you're gonna see some programs using only the libc ... the problem is that this kind of application is printing in a raw mode ... not inside the client area of a terminal window yet, but  in the fullscreen ... those are ring3 programs ... stdout is a tty ... when you send data to stdout you're gonna print out in the foreground kernel console.

Yes. it is. The plain is create unix-like commands using the libc and client-side gui apllications using the libgws. Take a look at  gramado/core/commands/  and gramado/desktop/apps/. :)

I have some few programs using only the libc. Take a look at the folder gramado/core/commands/ ... you're gonna see some programs using only the libc ... the problem is that this kind of application is printing in a raw mode ... not inside the client area of a terminal window yet, but in the fullscreen ... those are ring3 programs ... stdout is a tty ... when you send data to stdout you're gonna print out in the foreground kernel console.

I've took a look at some of the commands and saw a little about the C library. I'd need to study Gramado more before talking anything about the C library, by the way.
I would propose that Gramado had some sort of compatibility with POSIX, but I don't know if "being an UNIX-compatible" is part of the goals of Gramado, so maybe it could be just a "compatibility layer".


Add test function for clearing the last used search and substitute patterns

Use the new test function to add additional tests after clearing the last used search/substitute patterns.

Codecov Report

Merging #5579 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5579      +/-   ##
==========================================
+ Coverage   83.01%   83.04%   +0.03%
==========================================
  Files         137      137
  Lines      151977   151984       +7
==========================================
+ Hits       126157   126222      +65
+ Misses      25820    25762      -58

Impacted Files
Coverage Δ

src/evalfunc.c
91.62% <ø> (ø)
:arrow_up:

src/search.c
82.29% <100%> (+0.17%)
:arrow_up:

src/testing.c
90.78% <100%> (+0.08%)
:arrow_up:

src/if_xcmdsrv.c
86.17% <0%> (-0.18%)
:arrow_down:

src/term.c
81.55% <0%> (-0.06%)
:arrow_down:

src/normal.c
83.58% <0%> (+0.03%)
:arrow_up:

src/window.c
88.56% <0%> (+0.03%)
:arrow_up:

src/register.c
91.53% <0%> (+0.07%)
:arrow_up:

src/edit.c
85.68% <0%> (+0.07%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


[MINOR][TESTS] Fix appname from sql.protobuf.functions to sql.connect.protobuf.functions

What changes were proposed in this pull request?
This PR proposes to fix appname from sql.protobuf.functions to sql.connect.protobuf.functions
Why are the changes needed?
For consistency
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
Manually checked.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @zhengruifeng

Merged to master.


GH container registry 503 error

Hey, I get a 503 error trying to fetch https://ghcr.io/v2/jidicula/clang-format/manifests/15 wanted to bring it to your attention 👍🏼

Our builds just started failing this morning as well with the following...I don't see 503 explicitly, but I assume it's the same issue:
Run<EMAIL_ADDRESS>  with:
    clang-format-version: 10
    check-path: src/EnergyPlus
    fallback-style: llvm
Run "/home/runner/work/_actions/jidicula/clang-format-action/v4.10.1/check.sh" "10" "src/EnergyPlus" "llvm" "" ""
  "/home/runner/work/_actions/jidicula/clang-format-action/v4.10.1/check.sh" "10" "src/EnergyPlus" "llvm" "" ""
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
Unable to find image 'ghcr.io/jidicula/clang-format:10' locally
docker: Error response from daemon: Head "https://ghcr.io/v[2](https://github.com/NREL/EnergyPlus/actions/runs/4427506300/jobs/7765193171#step:3:2)/jidicula/clang-format/manifests/10": Get "https://ghcr.io/token?scope=repository%[3](https://github.com/NREL/EnergyPlus/actions/runs/4427506300/jobs/7765193171#step:3:3)Ajidicula%2Fclang-format%3Apull&service=ghcr.io": net/http: request canceled (Client.Timeout exceeded while awaiting headers).
See 'docker run --help'.

If it's not the same then I can open a separate issue.

@Myoldmopar I'm not able to reproduce this in https://github.com/jidicula/test-clang-format-action/actions/runs/4428271920/jobs/7767104502 . Can you try triggering a re-run on the failed run and see if it happens again?
The 503 issue could be an intermittent issue with GHCR.

Looks like it is working again, just a fluke!  Thanks for the reply, and for this GH action, we really appreciate it!

@aea7 could you retry your workflow run as well to see if it was just an intermittent 503 being returned by GHCR?

I suspect this failure was related to this incident with Packages a few hours ago.

I'm sure you are right, and I'm also sure you weren't the only random repo owner getting harassed by users! :D
Thanks again

I'm sure you are right, and I'm also sure you weren't the only random repo owner getting harassed by users! :D
Thanks again

@aea7 could you retry your workflow run as well to see if it was just an intermittent 503 being returned by GHCR?

yep, it's working now, apparently ghrc was down looking at the incident that you mentioned. @jidicula


Installer Defaults

This may be more of a question but if you "winget install vscode" do you get no opportunity to configure it like a traditional installer wizard? how do you guys determine what the defaults would be? Things like do I want "Open in Code" to appear in the Windows Context Menu. Or is the philosophy that you would first install a software and then configure it post installation?
Thanks

I would like to see custom install arguments for each app when I do:
> winget install Microsoft.VisualStudio --help
> --python - Installs python
> --Node - Installs Node
> etc....

I would also like a guided installation where the installer UI shows up and I can personally select options and manually hit next, next, next, finish.
> winget install Microsoft.VisualStudio --guided

This has been frustrating to stumble upon. Do you know how to configure VS Code after the fact?
I might just have to uninstall and re-install from the website 😞

Winget CLI has two additional command line options for these use cases:

Option
Description

-i, --interactive
Runs the installer in interactive mode. The default experience shows installer progress.

--override
A string that will be passed directly to the installer.

Source: https://docs.microsoft.com/en-us/windows/package-manager/winget/install#options
However, the --interactive option does not seem to work properly at the moment. The parameters specified via --override completely replace the original installer parameters from the manifest which should allow you to run the installer with parameters you want.

Thanks @megamorf! The part I don't understand is why the default behavior is different. It's not intuitive. So far the 2 times I've tried to install an app with winget, I haven't had any success.
How do I discover what string to pass to the installer via the --override switch?

The problem is that there are countless installer formats, e.g. InstallShield, Wise InstallMaster, Inno Setup, Nullsoft Scriptable Install System, Ghost Installer, msix, appx, custom exe files, etc. All have varying command line parameters (some even lack command line parameter support). Some setups have additional custom parameters. All of those are hard to discover so ther's no way for winget to be able to list them on the fly.

So, I have to know how to run the installer? Which presumably I would have to download the installer or visit a website to understand, at which point I'll just download from there. I get that maybe once I know it, I could script the install with winget, which could be useful.
What about my original question about why the winget install of VS Code doesn't have the same UI as the regular VS Code install? Why do I need to pass the --interactive switch to get normal behavior?

So, I have to know how to run the installer? Which presumably I would have to download the installer or visit a website to understand, at which point I'll just download from there. I get that maybe once I know it, I could script the install with winget, which could be useful.

Correct, that's the way you'd do it.

What about my original question about why the winget install of VS Code doesn't have the same UI as the regular VS Code install? Why do I need to pass the --interactive switch to get normal behavior?

Winget is a packager manager. Package managers install packages silently which means they'll pass a set of default parameters to the package's installer. That's why you'll see a progress bar or nothing at all. Having package managers run installers interactively is not a typical use case.

Having package managers run installers interactively is not a typical use case.

Right, but from my experience I've also never installed something through a package manager, found that the installation doesn't meet my needs, uninstall it, and re-install it from the website. I totally get that the amount of installers that exists makes this difficult and the fact that we're playing catch-up makes this a difficult problem to solve. But I don't think "this is as good as it can get" is a suitable response. Surely there must be something apps can do so that they have the proper install experience on winget regardless of installer?

It's probably a long road ahead and not sure if Microsoft wishes to head to this in the future, but in my ideal world winget would work similar to installing packages on Linux. That means that in a way there's no need for "installers".
However that means the software prolly needs to be bundled differently and unified via the winget way, so I think that's why winget currently relies on external installers (but hopefully this changes in the future).

@nbarkhina the -i "interactive" install has been fixed for a few releases. Does that satisfy your request here?

very cool i will give that a try thanks!

Is there a way to make installs interactive for all installs?
I couldn't find a setting for it at https://learn.microsoft.com/en-us/windows/package-manager/winget/settings.


build: approve builder

优化builder性能，所有与libuv有关的fsevents全部采用同步API

Coverage decreased (-0.5%) to 75.047% when pulling f131ab33b1fadd2304b0f236d6f18ab264169a50 on xtx1130:approve-compile into 87112ea456af07832963a02c0459ae0dc7de7968 on lavas-project:feature-entry.


nixos/speechd: init module

Motivation for this change
Closes https://github.com/NixOS/nixpkgs/issues/39500
This might be a start. I need a test system next to install nixos from this PR.
I'm not sure yet how speechd works at all.
Things done

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

[ ] 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"
[ ] 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
[ ] Fits CONTRIBUTING.md.

@jtojnar would you implement the options just like the package?
services.speechd.withFlite = true;
services.speechd.withEspeak = true;
services.speechd.withPico = true;

Do we need more options like config file?

I think it should be sufficient to provide package option where you would pass the configured package.

oh no

oh no


Handling multiple imports in the same line when using aliases

Fix for #13.
Added tests for the new method get_import_name_without_alias

Coverage remained the same at 100.0% when pulling ea2db2c29a5e38ec9814c5ae91fe7299d5cb8a8d on tiagofmcosta:master into 5989c3b109926702eb9bd96857859408c3cef403 on bndr:master.

Coverage remained the same at 100.0% when pulling ea2db2c29a5e38ec9814c5ae91fe7299d5cb8a8d on tiagofmcosta:master into 5989c3b109926702eb9bd96857859408c3cef403 on bndr:master.

Thanks!


Recreate the functionality of plot_mpr.R

Describe the New Feature
point_stat_ST4_240000L_20190731_120000V_mpr.txt
mpr_plots.pdf
plot_mpr.R.txt
The output of this script is a multi-panel PDF with four plots for each case:
-forecast and observation histograms
-scatter plot
-Q-Q plot
-forecast and observation and wind errors wind rose
Acceptance Testing
List input data types and sources.
Describe tests required for new functionality.
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the new feature down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label
[ ] Select milestone to next major version milestone or "Future Versions"

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdatadb, METviewer, METexpress, METcalcpy, METplotpy

New Feature Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding source.
[ ] Fork this repository or create a branch of develop.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project(s), Milestone, and Linked issues
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

Wind rose is implemented as a separate plot and included into the mpt plot grid


Microsoft.Xaml.Interactivity - strongly-named assembly

At the risk of exposing my self as a complete and utter noob, what do I need to do in order to install the template 10 version that does not give me the "strongly-named assembly" error with the "Microsoft.Xaml.Interactivity" assembly?
Thanks,
Bjarne

At the moment the only way to get this without strong name is to get the source, turn off strong name and compile your self, and not rely on nuget for the package at all.  Also requires you to add the reference manually to your project

Ouch.. When will we see a nugget package?

Next week.

The current NuGet preview resolves this. Please help by testing.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/386 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit fd553a2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Complete official Docs

I am new to learn C# and it's better to prepare the complete in one place to learn C# language. The complete and in once place documents can make me clear how to learn C# and easy to understand then a messy as professional looking.

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

ID: 756e4425-16b0-abfd-1e5a-927f07a8af69
Version Independent ID: 040f0373-5f32-5132-2e26-adf03e4d6100
Content: Introduction to C# - interactive tutorials
Content Source: docs/csharp/tutorials/intro-to-csharp/index.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@suloveoun Thanks for the feedback. We agree, and we'll be continuously adding more learning materials here.
I'm going to close this. I agree with your feedback, and we'll keep adding content, but this issue doesn't represent a task-level work item.


Added submodules

Not sure if you would want all these changes, so this PR is more of a suggestion... review and change as needed or reject... increases the size of the download considerably but at that point the download has everything you could need...

I'll add the new links, however keeping the submodules always up to date probably isn't worth the effort when everything is linked.