MSID: 89485 Version: 1 DOI: 2<IP_ADDRESS>6984

"msas": "Ecology"
"msid": "89485"
"preprintDoi": "10.1101/2<IP_ADDRESS>6984"
"Reviewed Preprint posted": "2023-09-08"
"Sent for peer review": "2023-06-14"
"Posted to bioRxiv": "2023-07-02" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>6984v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=89485)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89485/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6984)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89485/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89485)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89485

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89485

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89485/v1/89485-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89485)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Reviews are not available yet.
Not quite sure what's up with the corresponding authors here. Both first and last authors are corresponding, but it looks like on the preprint, two emails were provided for the last author and this info is applied to both corresponding authors.

EPP result:

Also, eLocations missing from reference list.

Good spot - thanks James. It also looks like Kevin A. Rucci did the submitting and is marked as corresponding author in eJP, which conflicts with the PDF. Please could you check with the authors, and ask them to update the preprint?

I have reached out to the authors.

The authors have updated their preprint, so that the PDF now shows both corresponding authors' email addresses, but the online view on bioRxiv now only shows the last author as corresponding.
https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>6984v2
Can we work with this? We'd have all the information we'd need to correct this in our version . . .

Nice - thanks James.
We'll need to wait until the MECA package is available to confirm. Hopefully bioRxiv's typesetters capture the info as per the PDF

(MECA package/full text still not yet available)

MECA package now available at s3://transfers-elife/biorxiv_Current_Content/September_2023/11_Sep_23_Batch_1731/0b769684-6ed0-1014-8474-d012260c521e.meca.
I'll load tomorrow

@JGilbert-eLife, I've loaded the latest version for this one - could you take a look when you next have a chance? Thanks

Thanks! Looks good - do we need to update the ticket to indicate the bioRxiv v2? (https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>6984v2)?

Also do the reviews need moving over to the v2 @acollings?

No idea, @fred-atherden, but I've moved them across just in case :-)

@HazalCiplak - do you know why the reviews aren't in the DocMap for this one as well?

(Due to being deleted/reposted, will they appear tomorrow?)

Ah sorry, yeah of course

I have rerun the pipelines to got DocMaps available later today.

Thanks! I think we're good to go with this one.


Investigate stakater/reloader

Since the move to SOPS, some applications have not reloaded automatically on change of configMaps/secrets. stakater/Reloader seems to be a good solution to the problem. Need to investigate installing it and what annotations need to be made to utilize it.

Installed, will expand usage as needed.


Design the educational module page

Created a mockup for the educational module page using Figma.
Included sample lessons and units of Pre-algebra to help with the design.
Below is the link to the Figma page (incorporated with Ella's Login and Registration Mockups):
https://www.figma.com/file/FubKzqMVzbtOXC2u7deWhj/Gamified-Learning-Platform-(Educational-Module)?type=design&node-id=315-324&mode=dev
Please feel free to comment below if you have any suggestions for this mockup and how it can be improved.
Preview image on the current educational module page:

Looks good!


(material-ui@next) MUI InputLabel shrink state is bugged

Steps to reproduce
Link to live example: https://next.mui.com/material-ui/react-text-field/#helper-text
Linked to official documentation since it also happens there (v6.0.0-alpha.8)
Reproduction Steps:

Add a component with an InputLabel (TextField in this case)

Current behavior
InputLabel shrink state is currently bugged (Link stated above):

Expected behavior
Current behavior (Same link but in the latest MUI version):

Context
No response
Your environment
No response
Search keywords: shrink, shrunk

Didn't realize #42345 is related 😅


Syntax highlighting fails when a html tag is inside a data attribute

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:18:03.540Z)
OS Version: Linux x64 4.8.0-56-generic
Extensions:

Extension
Author
Version

theme-dracula
dracula-theme
2.1.2

hopscotch
idleberg
0.1.4

Steps to Reproduce:

Make a new html document
Paste the following html code into the editor

<!DOCTYPE html>

This is a paragraph

    <input data-on=""/>

 This is also a paragraph

You can see from the image above that syntax does not highlight correctly when you have have an html tag inside a data attribute. This is a valid statement as per the HTML 5 reference:

Except where otherwise specified, attributes on HTML elements may have any string value, including the empty string. Except where explicitly stated, there is no restriction on what text can be specified in such attributes.

It works when the charectars are escaped like:
<input data-on="<i class="fa fa-play"></i>"/>

This issue was moved to textmate/html.tmbundle#69


Question: How to properly write a DateTime formatting helper

What would be the right way to write an, e.g. fdate helper to format a DateTime property?
I want to use something like {{fdate myDate "HH:mm"}} to pass HH:mm to toString().
TIA

After some trial-and-error coding, I think this looks good enough:
Handlebars.RegisterHelper("fdate", (writer, context, parameters) => {
    if (parameters.Length > 1)
        writer.WriteSafeString(((DateTime)parameters[0]).ToString(parameters[1] as string));
    else if (parameters.Length > 0)
        writer.WriteSafeString(((DateTime)parameters[0]).ToString());
});

Looks good to everyone?

Hit a snag when I misspelled the property:
How to properly test for undefined parameter? (i.e. when a property is not found)
The debugger shows something named "undefined" but I can't determine what it is.

It is an object of type UndefinedBindingResult, which is internal. Rather than checking for that specifically, I recommend you check for types you can handle (string, DateTime, etc.)

For now, I'm checking that the toString() result is not an empty string, which is fine for DateTimes (they always produce some string). This will not suffice when the wrong property is passed (i.e. a non-DateTime object). I'm going to test the actual object type instead.
Couldn't there be something that is reliably testable (like null or something), just like javascript's undefined?

There's been requests to make UndefinedBindingResult a public type, we may do that in the future. For now, it seems there are a number of viable options at your disposal, including whitelist type checking, or TryParse on the stringified object.


The login page should be converted to a configurable form

To make the application look and feel more configurable, the login page should be turned into a configurable page.
** Acceptance Criteria **

Login page should come pre-configured with Shesha
Login page should be a configurable page
There may be a need to add an AppSetting to specify the log-in page to be able to point to updated one
Same applies for both the admin and public portals
How to article should be added to the documentation

A user will be able to change how the view should be on the login page: https://github.com/shesha-io/shesha-framework/issues/1449 when the login is configurable. I will close item 1449 and will be tracked after the implementation of the login

The requirement "There may be a need to add an AppSetting to specify the log-in page to be able to point to updated one" has been cut out.

Hi @Lukeybooi, there's a requirement for documentation on this item. Can you please complete the documentation before updating the item

@hlayisanimazuze this is resolved. As we discussed I will do the documentation separately.


ai-proxy: support custom openai provider

Ⅰ. Describe what this PR did
A provider supporting the OpenAI protocol with customizable URLs, and it can be used without setting a bearer token.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.03%. Comparing base (ef31e09) to head (92061b6).
Report is 19 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1176      +/-   ##
==========================================
+ Coverage   35.91%   36.03%   +0.12%
==========================================
  Files          69       69
  Lines       11576     9483    -2093
==========================================
- Hits         4157     3417     -740
+ Misses       7104     5750    -1354
- Partials      315      316       +1

see 67 files with indirect coverage changes


When an Application is used by different users LogFiles under c:\ProgramData cannot be overwritten from other user

As told by  @JoasE in #627 (see his comment):
I am using NLog in an WPF application.
Logfiles for any user were stored in ProgramData/Company/App/Logs/LogFile.log. This caused AccessViolationException's when 2 different users use the application.
I solved this problem by creating a separate log folder for each user using the application in the nlog.config: fileName="${specialfolder:folder=CommonApplicationData}/App/Logs/${windows-identity:domain=False}/LogFile.Log"
After this I still experienced issues when the user runs the application in elevated permission. Their windows identity stays the same, but the files created can only be edited by an admin. So the next time the user starts the application without elevation, the user couldn't archive/write to the files.
However, I would still propose a fix for this to be implemented. This could come in the form of a new File target attribute which will make sure any files created by NLog are accessible to all users.
Example:
<target xsi:type="File"
        filePermissions="allUsers | currentUser" />

I opened a new issue as the original one is closed.

For this new duplicate issue, then the reply  remains the same.
Pull-requests are ofcourse welcome that will implement the solution suggested in the reply (or something more elegant).

For this new duplicate issue, then the reply  remains the same.
Pull-requests are ofcourse welcome that will implement the solution suggested in the reply (or something more elegant).


FeatureLayer events not triggering while offline

I created the featureLayer from a featureCollection in order to limit the amount of features rendered. This approach worked well. When I download and disconnect everything is stored properly and displays correctly. The only issue is when I close the browser, open it again, and navigate to the page. The page displays correctly and retrieves all tiles. I’m having an issue with the featureLayer, I can see that the features are stored. The featureLayer events are not being triggered (update, update-end). I created listeners for both these events but they aren’t being triggered when it’s offline. So I can’t call the initOffLineFeaturesMgr function. Any suggestions will be greatly appreciated.

@bsquires16 first, make sure you are on v2.8.1. It includes a bunch of enhancements specifically your workflow.
The How-to doc goes into some detail on the patterns to use.
Also, take a look at the latest version of appcache-features.html beginning at the update-end listener at Line 306.
If you are following a similar pattern and it's still not working, can you post some psuedo-code for me to look at?

Hi Andy,
Thanks for the response. I went through the documentation and made a few tweaks and everything works well on my desktop. Now I have to figure out a few things on the tablet.

Awesome. Let me know if you have any suggestions (e.g. documentation, samples, etc) on improvements based on your experience.

@bsquires16 any luck on the tablet?

Hi Andy,
Sorry for the late reply, I was on vacation. I was able find out why it wouldn't work on the tablet. The "verifyOnline" function was failing because the req.status returned 0 while the req.responseText was actually a value with a correct response. So I had to include (req.status === 0 && req.responseText !== "") in order for it proceed as expected.

@bsquires16 No worries. Interesting. What OS version and browser are you using? Just in case anyone else runs into the same issue.


Smoke test: Failed to create autodetected launch.json

See TFS build id 30914 for details.
Failed test:
3) Smoke Test Suite Debugging JavaScript autodetects program attribute for launch.json
Other tests impacted by this failure:
4) Smoke Test Suite Git verifies current changes are picked up by Git viewlet
Seems like Node.js debug environment configuration failed to create launch.json file. I've tried to manually perform test steps, and I wasn't able to get even to debug environment selection on Windows (pressing 'Configure' gear button in Debug viewlet did not give any result with the following error in Chromium console:
TypeError: Cannot read property 'openConfigFile' of undefined.

Dup of https://github.com/Microsoft/vscode/issues/31435
Good catch smoke test!


Allow some special reaction

It could be nice to allow some special reactions, like :qotd:

Actually, no!


[DOCS] Add **kwargs params description for models

📖 Documentation improvement
Hi! Looking through the documentation of some models, I noticed that some of them don't have a description of **kwargs parameters. I think we should fix it.
A list of model that don't have **kwargs argument description:

[ ] CatBoostMultiSegmentModel (docs link)
[ ] CatBoostPerSegmentModel (docs link)
[ ] ElasticMultiSegmentModel (docs link)
[ ] ElasticPerSegmentModel (docs link)
[ ] LinearMultiSegmentModel (docs link)
[ ] LinearPerSegmentModel (docs link)

Additional context
No response

Usually, these kwargs mean that these additional parameters are added into the model constructor. It looks like a good idea to point this out in documentation.
And may be we should give a link to the place there user could see all the parameters. I don't think we should list all the possible parameters, because they could change in the future.

Yes, I meant that we should add a mention that these parameters are added into model constructor and add link to the models. So, I am going to do it.


Screen goes blank when tapping outside image area.

Once the recaptcha image is visible, if you tap on an area near the edge of the screen.  The screen goes blank.  This was reproducble on all devices tested using iOS 11.x.

I have not tried matching the div size to the webView size, we currently have the webview edge to edge.  I think it might be a chicken/egg, if we decrease the webview frame, the div might decrease as well, but I will try it.

So, swing-and-a-miss, decreasing the size of the webView didn't help.  Tapping around the edges still makes the recaptcha image disappear.

@xmidi could you check if the branch fix/div-dismiss fixes it for you?

@ssathy2 that's nice to know.
@xmidi if you can confirm this fixes it for you as well, I may start working on releasing this fix

Excellent, works for me too, thanks!


Feat: Add ConfigMap annotations for all addons

Allows additional annotations to be added to the ConfigMap resources for all addons in the vela-core helm chart.
Fixes #2559
I have:

[X] Read and followed KubeVela's contribution process.
[X] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[X] Run make reviewable to ensure this PR is ready for review.
[X] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
I've successfully deployed this version of the vela-core helm chart against our cluster with additional annotations passed in via Helm.

@kaiju I think it works, can you help sign the cla

@wonderflow sounds good, I'll sign the CLA bit once I get the legal bits sorted out on my side.

@wonderflow Finally got everything sorted on my side and signed the CLA. Thank you for your patience! :)

It looks like there's been some major changes in the mean time. I'm closing this PR and will revisit if it's still an issue.

@kaiju Thanks for your contribution. We have migrated the whole addon directory to https://github.com/oam-dev/catalog/tree/master/addons. You can contribute addons there.  Thanks!


added visual feedback.

Sorry it took so long! Ran into some stuff around the house that took precedence. If you've got any input or would like any changes, let me know.

Thanks a lot for this, Jacob.
This looks good for the most part - though in the preview it doesn't seem like the input box's ring colour is updating as you intend it to?

I'll check again in a bit. It was a quick green for correct and a quick red for incorrect.

@chowder Will dig into this deeper. I'm not sure how Netlify deployments work. All is well locally, but those colors aren't updating on the deployed version.

@chowder there we go! conditionally applying the color to the class wasn't working. damn that took too long for such a small issue.

Thanks Jacob!
I'll likely make some small touches to the styles, out of personal preference. But this looks good to me. :)
Thanks again for the contribution!

No problem! I guess I should have asked first. Never worked on anything open source before. Just personal and work stuff.


fix(material/list): ensure leading icon scales with token

Correct the font size of leading icon in List component. Set font-size of leading icon to same value as its width and height. Ensure Ensure icon scales to "list-item-leading-icon-size" token.
Fix icon with cut-off appearance when "list-item-leading-icon-size" is less than 24px.

Closing in favor of #28567


Add department association

The department association of a publication is stored in the header/setSpec
To get the meaning of the values of the setSpec it is necessary to get the sets from dspace using the /oai/request?verb=ListSets endpoint.
This gives a long list of records (at ZHAW some 146 or so), so to requests are needed. These records hardly ever change for out needs.
We need to build a lookup table from these records in order to set the departmental affiliation.

alternatively, we could have a manual lookup table. At least a lookup table with the name - label association is needed.


Add Sahmyook university

Adding Sahmyook University, Computer Science and Engineering to the JetBrains SWOT repository.
University Name: Sahmyook University, Computer Science and Engineering
Official Website: https://www.syu.ac.kr/
Mail Domain : syuin.ac.kr
Please review and merge if everything is in order. Thank you!

@NOOJU Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.

Thank you, Have a Good Day!


[CLI] Flag to create a new Swift project

when I create a new project by running the running the react-native-cli
react-native init AwesomeProject
from the quick start guide
the newly created project is an objC project.
which means I need to be working with the swift auto generated header (`#import "AwesomeProject-Swift.h") - assuming I want to write mostly in Swift, it's a pain.
is there way to create a new (preconfigured) swift project using the CLI? e.g react-native init --swift AwesomeProject
this is somewhat of a follow up question to - https://github.com/facebook/react-native/issues/1669
since I was trying to solve this issue by creating a new Swift project and then install ReactNative from CocoaPods. (which didn't work as well...)

At Facebook they are only using Objective-C with React Native at the moment, so this would need to be a community effort. That said, I think this would be an awesome idea to reduce the friction of creating a Swift project. Any interest in tackling it?

Any interest in tackling it?

honestly, I wouldn't know where to begin with this... :-/

@yelled3 - fair! maybe someone else will pick it up now that I've tagged it, cc @robertjpayne

I managed to create a swift version of the default project created by init and got everything working properly.
Next step is to make it a template and mimic the yeoman generator stuff in the CLI. I should have a PR for this in the next day or two.

@nsimmons interesting - did you install RN from cocoapods or as an external lib?

External lib. The trickiest part is generating the bridged header, but I have a script that will create this file when you init the project.

@nsimmons amazing - do share :-)

I will try to have a PR up for this tonight.

Looking forward to this!  Doing it by hand is a bit tedious.

I'm super close. Just finishing up the tests. Haven't been able to dedicate as much time as I would have liked this week. Will have a PR up this weekend for sure.

:fireworks:  :+1:

PR is up!

Status of PR?

Not actively working on ReactNative anymore. The solution I had was working but will require a rebase and probably major rewrite.

Closing due to inactivity.
See comments in PR #2648.


Remove duplicated methods

The duplicated methods from ActiveForm::Base and ActiveForm::Form are placed in the ActiveForm::FormHelpers.

I think you need to update your branch first with the latest changes, it seems a bit messy.
It looks like you pushed straight to rails/activeform and are PRing to your repo, iirc we were going to be opening PRs on the rails side, weren't we?

@carlosantoniodasilva That's correct, my branch has become messy.
You're correct about opening PRs on the rails side, my bad. As soon as I update my repo, will start to send PRs.


Increment version for messaging/azservicebus releases

Increment package version after release of messaging/azservicebus

/check-enforcer override

(failing dep check for azidentity, not relaed to this version increment)


Proposal: Add --version flag for kubectx and kubens

When having multiple versions of kubectx installed it is currently not possible to differentiate between the versions except with looking into the code and comparing the recent changes (or checking the git tags).
As commented in https://github.com/ahmetb/kubectx/pull/124 hardcoding the values is error-prone. This issue is for discussion how to implement this in a good way :)

I thought about some automated variants of this, imho a git hook would be the most elegant version, as you can not forget updating the version. In my head a pre-commit and a serverside post-receive are reasonable.

pre-commit hook: This can be calculated on client-side and automatically update a variable in the scripts. git does not know about the future hash though, so that you would run into an off-by-one error (imho you can only use the last existing commit hash). Also tags do not trigger a pre-commit hook.
post-receive hook/webhook: This is more powerful, but needs to be hosted somewhere and needs push access to the repo then to update the hashes

Please feel free to enlighten me, if there is another (better) solution which is not hardcoded :)

I recommend we hold off on this until we do #115 (Go rewrite). Once we start distributing in Go, we will be able to publish binaries from the release environment, which will give us ability to "stamp" the binaries on "tag commits" where we can just read the version from git describe --tags --always easily.

Bump for this now we have the go rewrite - and I think you are using goreleaser which has native functionality to write the current git-sha into a golang global at link-time, which would be enough for now if reported via the --version command.
Human readable version numbers in the --version response would be nice also.

Following.
This is also an issue for the Webi installer https://webinstall.dev/kubectx https://github.com/webinstall/webi-installers/pull/175

Following.
This is also an issue for the Webi installer https://webinstall.dev/kubectx https://github.com/webinstall/webi-installers/pull/175

Since you're using goreleaser, this would be a good fix:
var (
	name    = "gitdeploy"
	version = "0.0.0"
	date    = "0001-01-01T00:00:00Z"
	commit  = "0000000"
)

func ver() string {
	return fmt.Sprintf("%s v%s (%s) %s", name, version, commit[:7], date)
}

func main() {
	// Support [--]version and -V
	if len(os.Args) > 1 {
		if "version" == strings.TrimLeft(os.Args[1], "-") || "-V" == os.Args[1] {
			fmt.Println(ver())
			os.Exit(0)
			return
		}
	}

	// ...
}

Since you're using goreleaser, this would be a good fix:
var (
	name    = "gitdeploy"
	version = "0.0.0"
	date    = "0001-01-01T00:00:00Z"
	commit  = "0000000"
)

func ver() string {
	return fmt.Sprintf("%s v%s (%s) %s", name, version, commit[:7], date)
}

func main() {
	// Support [--]version and -V
	if len(os.Args) > 1 {
		if "version" == strings.TrimLeft(os.Args[1], "-") || "-V" == os.Args[1] {
			fmt.Println(ver())
			os.Exit(0)
			return
		}
	}

	// ...
}

Hi, could this be implemented now that the go rewrite is being built and released?

Thanks all, I made patch #295, it'll be available in the next release.


IORedis.Cluster doesn't throw error when connection can't be estabilished

Details:
When provided with a malformed host, the Cluster constructor doesn't throw any error just silently fails.
How to reproduce:
Use the following snippet and run the code.
import IORedis from 'ioredis';

new IORedis.Cluster([{host:"redis://invalidUrl:8782"}])

Expected behaviour:
An error should be thrown like:  [ioredis] Unhandled error event: Error: getaddrinfo ENOTFOUND.
Actual behaviour:
Nothing is thrown, the ENOTFOUND exception is seemingly caught and not propagated in any way.

facing the same issue, any solution?

We wrapped the redis object, and put some validators on it. Not the best solution, but it works.

We wrapped the redis object, and put some validators on it. Not the best solution, but it works.

would share some code sample


GCC error: 'for' loop initial declaration used outside C99 mode

We are currently porting the code to Linux with GCC compiler and hit a number of issues during compilation. A workaround of this would be to add the CFLAGS parameters -std=c99 after the gcc command, however we intend to make the code more robust across various major compilers, together with a bit refactoring in the code itself.

fixed


Brand new hydrogen project issue with VITE import of Tailwindcss v4

What is the location of your example repository?
https://github.com/IngoVals/drangey
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
2024.7.7
What version of Remix are you using?
2.12.1(react-dom@18.3.1(react@18.3.1))(react<EMAIL_ADDRESS>Steps to Reproduce
Fetch repo, need to have PNPM, possibly any package manager has same error, not checked.
pnpm i
pnpm shopify hydrogen link
pnpm dev

Route to localhost:3000
All seems fine but error reported in console.
Unable to resolve `@import "tailwindcss"` from /home/ingovals/projects/drangey/app/styles
10:11:43 AM [vite] Internal server error: [postcss] ENOENT: no such file or directory, open 'tailwindcss'
  Plugin: vite:css
  File: /home/ingovals/projects/drangey/app/styles/tailwind.css?direct:undefined:NaN
      at async open (node:internal/fs/promises:639:25)
      at async Object.readFile (node:internal/fs/promises:1242:14)
      at async Object.load (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:36797:25)
      at async loadImportContent (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:844:19)
      at async Promise.all (index 0)
      at async resolveImportId (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:800:27)
      at async parseStyles$1 (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:708:5)
      at async Object.Once (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:965:22)
      at async LazyResult.runAsync (/home/ingovals/projects/drangey/node_modules/.pnpm/postcss@8.4.47/node_modules/postcss/lib/lazy-result.js:261:11)
      at async compileCSS (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:36860:21)
      at async TransformPluginContext.transform (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:36141:11)
      at async PluginContainer.transform (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:49033:18)
      at async loadAndTransform (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:51866:27)
[vite] Internal Server Error
[postcss] ENOENT: no such file or directory, open 'tailwindcss'
    at async open (node:internal/fs/promises:639:25)
    at async Object.readFile (node:internal/fs/promises:1242:14)
    at async Object.load (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:36797:25)
    at async loadImportContent (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:844:19)
    at async Promise.all (index 0)
    at async resolveImportId (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:800:27)
    at async parseStyles$1 (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:708:5)
    at async Object.Once (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-CdRA0jgn.js:965:22)
    at async LazyResult.runAsync (/home/ingovals/projects/drangey/node_modules/.pnpm/postcss@8.4.47/node_modules/postcss/lib/lazy-result.js:261:11)
    at async compileCSS (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:36860:21)
    at async TransformPluginContext.transform (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:36141:11)
    at async PluginContainer.transform (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:49033:18)
    at async loadAndTransform (file:///home/ingovals/projects/drangey/node_modules/.pnpm/vite@5.4.7_@types+node@22.5.5_lightningcss@1.27.0/node_modules/vite/dist/node/chunks/dep-DG6Lorbi.js:51866:27)

Expected Behavior
Brand new install of Hydrogen, no changes, using tailwind v4 Alpha given in CLI options.
Run project and everything works
Actual Behavior
The tailwind.css file reports error, VITE doesn't seem to be able to resolve @import 'tailwindcss';

It seems to be  an issue with missing<EMAIL_ADDRESS>package.
@tailwind/vite does not include it as a peer dependency, but tries to resolve like it is a regular dependency. I'm thinking that since Yarn and NPM hoist all packages up it works, but in the case of PNPM it does not.
Since I think tailwind will probably became a peer dep in a future release anyways the fix here is just to add "tailwindcss": "4.0.0-alpha.17" under devDependencies in the template.


Email Extractor

Project title
Email Extractor
Project Description
A web app to extract emails from a given texts.
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[x] Html
[x] CSS
[x] JavaScript

Assigned u @AnIkeT126


Update to release v0.1.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Mistake : needs to be made from the fork


release plans?

it has been quite a while since we had a release and a lot of changes pilled up in git.
any blockers for a release?

Duplicate of #74


update_attributes is not updating the field under certain situation

Look at this example ruby snippet
class XConfig
  include Mongoid::Document
  field :samples, :type => Hash
  field :summary_fields, :type => Array
  field :filters, :type => Array
end

xc = XConfig.create
xc.update_attributes(
  :samples => {
    "food.ingredient" => "example"
   },
   :summary_fields => [ "collection", "description", "inactive" ],
   :filters => [{ :new => true }]
  )

  xc.reload.samples
  xc.reload.filters

So after updated the XConfig instance attributes, I reload it
What I expect: The hash passed to the update_attribute
What I actually get: nil and nil

You're probably getting an error on the MongoDB side if you persist this in safe mode... It should be:
xc.update_attributes(
  :samples => { "food" => { "ingredient" => "example" }},
  :summary_fields => [ "collection", "description", "inactive" ],
  :filters => [{ :new => true }]
)


[Bug]: traefik does not detect port in docker compose gives 404

Error Message and Logs
when I try to run a docker compose file with traefik that file does not detect a port and yet it's just giving 404
Steps to Reproduce

use docker compose to start a server
2.do not have proxy in the docker compose
try to access next server after running the server (port 3000)
you will not access it and you will got 404

Example Repository URL
No response
Coolify Version
v4.0.0-beta.347
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
No response
Additional Information
No response

When I started to debug this I got to the next results:

I had to check the traefik logs which was not provided out of the box from coolify (it would be great to see them in the ui)
in the logs I noticed that traefik detects my container but does not detect the port giving the next error: port not found
searching on the internet I got the next:

https://community.traefik.io/t/port-is-missing/4422
https://stackoverflow.com/questions/68627212/docker-compose-traefik-error-port-is-missing-but-network-mode-is-in-host-mode
https://github.com/traefik/traefik/issues/5732

My setup:
Turbo repo with different next apps, here is the files
docker compose:
version: "3"

services:
  public_web:
    image: public_web
    container_name: public_web
    build:
      context: .
      dockerfile: ./apps/public_web/Dockerfile
    environment:
      - NODE_ENV=production

Dcokerfile:
FROM node:18-bullseye AS base

# This Dockerfile is copy-pasted into our main docs at /docs/handbook/deploying-with-docker.
# Make sure you update both files!

FROM base AS builder
# Check https://github.com/nodejs/docker-node/tree/b4117f9333da4138b03a546ec926ef50a31506c3#nodealpine to understand why libc6-compat might be needed.
# RUN apk add --no-cache libc6-compat
# RUN apk update
# Set working directory
WORKDIR /app
RUN npm install turbo --global
COPY . .
RUN turbo prune public_web --docker

# Add lockfile and package.json's of isolated subworkspace
FROM base AS installer
# RUN apk add --no-cache libc6-compat
# RUN apk update
WORKDIR /app

# First install the dependencies (as they change less often)
COPY .gitignore .gitignore
COPY --from=builder /app/out/json/ .
COPY --from=builder /app/out/pnpm-lock.yaml ./pnpm-lock.yaml
COPY --from=builder /app/out/pnpm-workspace.yaml ./pnpm-workspace.yaml
RUN npm install pnpm --global
RUN pnpm install
# Build the project
COPY --from=builder /app/out/full/ .
COPY turbo.json turbo.json

# Uncomment and use build args to enable remote caching
# ARG TURBO_TEAM
# ENV TURBO_TEAM=$TURBO_TEAM

# ARG TURBO_TOKEN
# ENV TURBO_TOKEN=$TURBO_TOKEN

RUN pnpm turbo build --filter=public_web...

FROM base AS runner
WORKDIR /app

# Don't run production as root
RUN addgroup --system --gid 1001 nodejs
RUN adduser --system --uid 1001 nextjs
USER nextjs

COPY --from=installer /app/apps/public_web/next.config.js .
COPY --from=installer /app/apps/public_web/package.json .

# Automatically leverage output traces to reduce image size
# https://nextjs.org/docs/advanced-features/output-file-tracing
COPY --from=installer --chown=nextjs:nodejs /app/apps/public_web/.next/standalone ./
COPY --from=installer --chown=nextjs:nodejs /app/apps/public_web/.next/static ./apps/public_web/.next/static
COPY --from=installer --chown=nextjs:nodejs /app/apps/public_web/public ./apps/public_web/public

CMD PORT=3001 HOSTNAME=<IP_ADDRESS> node apps/public_web/server.js

adding label to the docker compose file solved the issue:
docker compose:
version: "3"

services:
  public_web:
    image: public_web
    container_name: public_web
    build:
      context: .
      dockerfile: ./apps/public_web/Dockerfile
    environment:
      - NODE_ENV=production
+    labels:
+     - 'traefik.http.services.public_web.loadbalancer.server.port=3001'


Use 32bit integer to enforce byte alignment while parsing wav sources.

This changes the binary mask used to discard the 1-bit while seeking through wav files from 0xfffffe (which casted to 32bit is 0x00fffffe) to 0xfffffffe. This prevents a serious issues which may arise when parsing large files.
This mask is used in addition with a offset of 1 to pad the chunk size to even numbers.
Without this adjustment, files which force the parser to seek beyond this limit would end up seeking to a quasi-random position of the file, try to read chunk ID and size from there and continue. This, leads to the parser basically jumping randomly within the file, interpreting random data as chunk size and chunk id.
One could argue that a 64 bit integer should be used for this operation? Though wav itself uses 32bit integers to determine chunk sizes.

Superseded by #65. Closing. Thanks a lot, @wro-ableton, for providing the fix!


Add AWS Templates page styling

@TimonVS could you please rebase off develop?  There are some conflicts I don't feel comfortable resolving.

Hi guys, did this get dropped or are we still working on it?

I think this has gone stale so I'm closing it. Let me know if you still want it merged.


[consensus][easy] rename stop_consensus to sync_only

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@bors-libra r=aching

:pushpin: Commit cc865af has been approved by aching

:hourglass: Testing commit cc865afdc1683149cc8d65bd976df11f0a7c424e with merge d024369f7347570f7b9953c2cb5609af638e8c7d...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: aching
Pushing d024369f7347570f7b9953c2cb5609af638e8c7d to master...


exclude some fields from history models.

Solves the first part of issue 165.
Open to suggestions of needed tests.

Current coverage is 96.68% (diff: 90.00%)

Merging #236 into master will decrease coverage by 0.16%

@@             master       #236   diff @@
==========================================
  Files             9          9
  Lines           445        453     +8
  Methods           0          0
  Messages          0          0
  Branches         56         58     +2
==========================================
+ Hits            431        438     +7
  Misses            7          7
- Partials          7          8     +1

Powered by Codecov. Last update 6580dde...c5e3d8c

Closing in favor of #238


Remove obsolete navigation link

Actually, the Prefixer still exists, the link is just wrong. It should be #prefixer, not #prefixer-settings.
Can you change to
Prefixer
 and I’ll merge it in?
Thanks!

Actually, the Prefixer still exists, the link is just wrong. It should be #prefixer, not #prefixer-settings.
Can you change to
Prefixer
 and I’ll merge it in?
Thanks!

Sorry, it's late here and I might be sleepy, but isn't that link already there? In the commit is the line just below the removed line. I was initially going to fix the link to point to #global-prefixer, but I saw that was already below in the "Settings" section.

Sorry, it's late here and I might be sleepy, but isn't that link already there? In the commit is the line just below the removed line. I was initially going to fix the link to point to #global-prefixer, but I saw that was already below in the "Settings" section.

Ah, yes! You’re correct. Thanks!

Ah, yes! You’re correct. Thanks!

El Friday 02 January 2015, Tyson Gach escribió:

Ah, yes! You’re correct. Thanks!

Thank you for Bourbon. :)


com.apple.Safari.plist under Containers folder is not analyzed

The latest mac_apt (1.3.1) does not analyze com.apple.Safari.plist under Containers folder (/Users/*/Library/Containers/com.apple.Safari/Data/Library/Preferences/com.apple.Safari.plist).
In addition, it is missing both AppExtensions (/Users/*/Library/Containers/com.apple.Safari/Data/Library/Safari/AppExtensions/Extensions.plist) and WebExtensions (/Users/*/Library/Containers/com.apple.Safari/Data/Library/Safari/WebExtensions/Extensions.plist).

I will have to add them. Thanks for reporting it.

Added in v1.4.0.


Is there a way to purge?

Hi, This is more like a question...
Is there any way to fully restart the information saved?
For example if a have an App and the user log-out I would like to reset all stored values, actually I have to go through every value in order to delete it.

I guess the way I'd do it is:
for (key, _) in Defaults.dictionaryRepresentation() {
            Defaults.removeObjectForKey(key)
}

but yeah, it wouldn't be too bad to have a shortcut for that in SwiftyUserDefaults.

https://github.com/radex/SwiftyUserDefaults/pull/47
here it is

I added Defaults.removeAll() in 1ac8701. Hope this solves it.

This is not documented anywhere. Added this in https://github.com/radex/SwiftyUserDefaults/pull/94/commits/39e122d828d7937a3a82f1c6d199d06191f9a6c3.


fix(datepicker): disable pop-up in read-only mode

This adds ability to disable datepicker pop-up in read-only mode.
Fixes #5017

Fixed and ready for review. @mmalerba Input and button already had the "disabled" attribute, so I didn't need to add it to datepicker-input and datepicker-toggle.

Ready for review again :)

Changes made. Ready for review

caretaker note: build files need to be updated to reflect datepicker dependency on cdk


When using this plugin, if it is a full-screen large picture, a white screen will appear.

How to keep the element from disappearing before leaving the viewable area, which attribute controls this function

Do you have a code sample?


Add reactive kafka support

Depends on https://github.com/spring-cloud/spring-cloud-stream/issues/519

Closing this spike as a lot of things have changed since the time this issue was created. Also, reactor-kafka is a separate effort that is not directly part of the core Kafka binder.


Make getu AD friendly

Is your feature request related to a problem? Please describe.
Currently getu returns an anonymous function to calculate the symbol -> index once and cache the results.
During AD, we are likely to do something like the following:
uf = getu(sol, sys.x)
gradient(sol, prob) do sol, prob
  uf(prob)
end

Here we lose tracking of gradients back to sol for the next step of our analysis. The anonymous function loses this tracking.
Describe the solution you’d like
Potential solution could be for getu to return a callable struct which can be called similarly to how it is carried out right now and have it plumb the gradients back to this struct.
Additional context
We need to do reverse-mode AD here, and this needs to also match the interface for the batched interface.

@ChrisRackauckas can you assign @AayushSabharwal and I to this issue instead

oh I see, I thought this was the AD of it. If it's just the struct change, @AayushSabharwal should do that for all anonymous functions in the package here.

Example for BatchedInterface AD:
@variables x(t) = 1.0 y(t) = 1.0 z(t) = 1.0 w(t) = 1.0
@named sys1 = ODESystem([D(x) ~ x + y, D(y) ~ y * z, D(z) ~ z * t * x], t)
sys1 = complete(sys1)
prob1 = ODEProblem(sys1, [], (0.0, 10.0))
@named sys2 = ODESystem([D(x) ~ x + w, D(y) ~ w * t, D(w) ~ x + y + w], t)
sys2 = complete(sys2)
prob2 = ODEProblem(sys2, [], (0.0, 10.0))

bi = BatchedInterface((sys1, [x, y, z]), (sys2, [x, y, w]))
getter = getu(bi)

p1grad, p2grad = Zygote.gradient(prob1, prob2) do prob1, prob2
    sum(getter(prob1, prob2))
end

@test p1grad.u0 ≈ ones(3)
testp2grad = zeros(3)
testp2grad[variable_index(prob2, w)] = 1.0
@test p2grad.u0 ≈ testp2grad

getp is due a major rewrite, so we will need to work out what code paths AD will hit post the changes. As of right now, @AayushSabharwal is aware of the issues. I think we can close this issue if there aren't any other concerns


pixel(0,0)

Checklist

[ ] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[ ] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

pixel(0,0)


Addition of conditional branching of Tasks

Pull request for adding the functionality of branching of tasks based on a condition.
The following changes are made -

With the already existing implementation, a join operation is checked if there is an extra parameter passed to the function. But a join task for split-or takes only one input, the input from the branch being run. Hence another function traverse_fix_split_or is run after split_or to fix the graph.
Conditions are added for scheduling of out tasks of split-or and the scheduling of split-or join.

@savingoyal  I have a few use cases which I'm currently using. Let's take the case of model invocation. Suppose if the result of model-1 is +ve then I may want to run some more steps, like maybe invoking some other models. But if the result of model-1 is -ve I want to move to the end.

@savingoyal I have a few use cases which I'm currently using. Let's take the case of model invocation. Suppose if the result of model-1 is +ve then I may want to run some more steps, like maybe invoking some other models. But if the result of model-1 is -ve I want to move to the end.

Hi @thesillystudent
Do you have any information about merging this PR? I'm creating a Data pipeline where I need conditional branching, it will be good to have this feature in the nearest future.

@hovhanns I don’t have any plans as such. I can complete it over some weekends if enough people ask for it and I get a goahead from the devs

@thesillystudent I am converting this PR to draft since there is no immediate need for this functionality. Please reach out to us at slack.outerbounds.co if you would like to work on this PR.


enable rule line_only_punc?

Any thoughts on enabling https://rules.abaplint.org/line_only_punc/ ?
there are 170 occurrences which can be quick fixed, https://abaplint.app/stats/SAP/abap-file-formats-tools/disabled_rules
its part of the style guide, https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#close-brackets-at-line-end

Enable 😃


fix(markdown-serializer): Override Turndown escaping behaviour with custom rules

Overview
We've detected a few issues and inconsistencies with how our Markdown serializer escapes Markdown syntax, and this PR aims to fix that. The escaping is done by a bunch of aggressive built-in rules (based on regular expressions) implemented in Turndown, and this PR overrides the escape function with a custom implementation with custom rules.
We've been discussing this internally (ref), and this is the best solution I could come up with. However, I'm going to keep this in a draft state until we do a live test directly in production 🔥 (no better way to test this, really).
PR Checklist

[x] Pull request title follows the Conventional Commits Specification
[x] Added/updated unit test cases and/or end-to-end test cases

Test plan
Since this will be tested directly in production, and given that unit tests have been written to make sure the Markdown serializer outputs exactly what's expected, a code-review should suffice.

A patch of this PR has been in production for at least 3 weeks now, and no major bugs have been reported. I believe this fix to be in an optimal state to be merged as-is, and will do just that once all statuses checks are green.


🎁 Deployment using Docker and docker-compose

What would you like to be implemented?

Create a Dockerfile for the deployment of the backend.
Create a Dockerfile for the deployment of the frontend.
Create a docker-compose file for the overall deployment of the project.
Write deployment instructions in the README file.

Is this a duplicate issue?

[X] I confirm that this is not a duplicate issue

Code of Conduct

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

Closed by #13 .


Remove carto.js from share options

Closes #9750

Removed mobile SDK sharing option.
Disabled carto.js sharing option.

Please @CartoDB/design review the design changes and @brittanymicek could you take a look at that This sharing option is currently unavailable text? :)
CR @xavijam

I'll code-review after the design ok? Just in case we need to change anything.

👍

Applied design changes requested by @josecruz cc @urbanphes

Hey @MariaCheca !
A few copy changes:
Change: "Get the link" to "Share"
"Send this link to your colleagues and friends, or post to your social
networks."
Change: "Embed it" to "Embed"
"Insert your map into your blog, website, or application. Get a full
screen map."
copy for CARTO.js module is perfect
"Interested in sharing within your organization?"
"Contact us to try one of our Enterprise plans."
Thanks!
On Fri, Oct 14, 2016 at 12:00 PM, María Checa<EMAIL_ADDRESS>wrote:

Applied design changes requested by @josecruz
https://github.com/josecruz cc @urbanphes https://github.com/urbanphes
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/pull/10194#issuecomment-253843672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJboLkDx8H2NwbNO0kby4zcgrh-_w1Oxks5qz6cEgaJpZM4KW3KF
.

--
Brittany Micek
Content Writer
CARTO - Predict through Location
201 Moore Street, Brooklyn, NY 11206
T: + <%2B34695497365>19174633232
W: www.carto.com

For the CARTO.js option I'd say something like
Your plan doesn't allow you to do so. Read more (only if the user has the engine enabled). If not, the whole module should dissapear.

After talking with @saleiva offline, we should remove the whole CARTO.js block, it doesn't make any sense to leave it if it is not possible to enable it, although you upgrade your account, so 🔥  @MariaCheca.

Could you replace screenshots from description please?

Sure!

@josecruz, could you take a look to the new disposition of the sharing options? After talking with @xavijam maybe we could make the inputs wider now they have more space around them, and take the embed option "Get a simple URL" link to a new line.

@MariaCheca @xavijam agree on making the inputs wider and move "Get a simple URL" to a new line!

Done! @josecruz :)

Let's go then!

🚢 🇮🇹


List of TF ops that tf.vectorized_map supports

Where can I get this information?
Which tf operation is a vectorized operation? and which one is not?

@Suzan009
Please refer this link  and let us know if ir helps?This is not a bug or feature request,for any further queries please post this issue in TF discussion forum as there is a larger community to help.Thanks!

It seems that is not available so it Is a valid feature request.
Also, If you are looking instead for XLA supported ops, Markdowns tables are not updated by years as you can find in the related ticket https://github.com/tensorflow/tensorflow/issues/14798#issuecomment-1047796247
/cc @wangpengmit @rohan100jain

The solution should be similar to https://github.com/tensorflow/tensorflow/issues/14798#issuecomment-1047796247, an auto-gen tool that dumps tf.vectorized_map's internal registration table to a markdown file.

The solution should be similar to https://github.com/tensorflow/tensorflow/issues/14798#issuecomment-1047796247, an auto-gen tool that dumps tf.vectorized_map's internal registration table to a markdown file.

Yes but we also need to find an internal orchestration owner cause orchestration It is not opensoruce/visibile to the community

update?

Is there any pages listed down the vectorized operation list

CCing @ishark .

The solution should be similar to https://github.com/tensorflow/tensorflow/issues/14798#issuecomment-1047796247, an auto-gen tool that dumps tf.vectorized_map's internal registration table to a markdown file.

For that one I have now a PR
https://github.com/tensorflow/tensorflow/pull/56510


Add support for @starting-style rule

The @starting-style rule will be shipping with Chrome 117 stable next month and will be tremendously helpful against a minor but irritating CSS headache. It will finally allow you to use CSS transitions with elements that are going to or from display:none. No more hacks needed!
Spec: https://drafts.csswg.org/css-transitions-2/#defining-before-change-style-the-starting-style-rule
Chrome notes: https://chromestatus.com/feature/4515377717968896
Example1: https://www.bram.us/2023/05/24/the-yellow-fade-technique-with-modern-css-using-starting-style/
Example2: https://codepen.io/callmetimm/pen/ExOqbBe

Update: With the release of Chrome 117, @starting-style is now supported in Chrome Stable for desktop and mobile as well as MS Edge. Please add it to your list.


Add BSD-3 license

Before we accept any more contributions here we need to add a license. It looks like setup.py mentions this is currently MIT licensed, but there is no LICENSE file. It would probably be best to update this to use the same BSD-3 license we use for the rest of Dask.
@rajagurunath @TomAugspurger you are the only two contributors here so far. Could you please confirm you are both happy to license this project under the standard Dask BSD-3 open-source license?
xref #12

Yep, +1 to BSD-3 for any past and future contributions.
Tom

From: Jacob Tomlinson @.>
Sent: Friday, May 26, 2023 10:59 AM
To: dask-contrib/dask-deltatable @.>
Cc: Tom Augspurger @.>; Mention @.>
Subject: [dask-contrib/dask-deltatable] Add BSD-3 license (PR #15)
Before we accept any more contributions here we need to add a license.
@rajagurunathhttps://github.com/rajagurunath @TomAugspurgerhttps://github.com/TomAugspurger you are the only two contributors here so far. Could you please confirm you are both happy to license this project under the standard Dask BSD-3 open-source license?
xref #12https://github.com/dask-contrib/dask-deltatable/issues/12

You can view, comment on, or merge this pull request online at:
https://github.com/dask-contrib/dask-deltatable/pull/15
Commit Summary

e1bd787https://github.com/dask-contrib/dask-deltatable/pull/15/commits/e1bd78738d9594260a1756b7f73ee7cf156db6bf Add license

File Changes
(1 filehttps://github.com/dask-contrib/dask-deltatable/pull/15/files)

A LICENSEhttps://github.com/dask-contrib/dask-deltatable/pull/15/files#diff-c693279643b8cd5d248172d9c22cb7cf4ed163a3c98c8a3f69c2717edd3eacb7 (31)

Patch Links:

https://github.com/dask-contrib/dask-deltatable/pull/15.patch
https://github.com/dask-contrib/dask-deltatable/pull/15.diff

—
Reply to this email directly, view it on GitHubhttps://github.com/dask-contrib/dask-deltatable/pull/15, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAKAOIXKC2HABVE66EWBCQLXIDHOLANCNFSM6AAAAAAYQNGTME.
You are receiving this because you were mentioned.Message ID: @.***>

Happy to license this project as Dask BSD-3 open-source license !! ❤
Gurunath

Awesome thanks both. CI failures here are unrelated and are being caused by CodeCov due to the migration. I'll merge this and then look into that separately.

Codecov seems happy on main now so I guess the migration has finished.


Fix bugs for the backing image manager

https://ci.longhorn.io/job/private/job/longhorn-tests-regression/195/


Fix JOIN without Alias

Issue
gluesql> CREATE TABLE User (id INT, name TEXT, deptId INT);
Table created

gluesql> CREATE TABLE Dept (id INT, name TEXT);
Table created

JOIN with Alias works

gluesql> SELECT * FROM User U JOIN Dept D ON U.deptId = D.id;
| id | name | deptId | id | name |

JOIN without Alias gets error

gluesql> SELECT * FROM User JOIN Dept ON User.deptId = Dept.id;
[error] parsing failed: ParserError(
    "Expected end of statement, found: .",
)

Cuz it is ParseError, it might be a issue of sqlparser.
Is there any way to handle within our project?

I think this is not related to join alias.
sqlparser handles User not as a normal table.
gluesql> CREATE TABLE User (id INTEGER);
Table created

gluesql> SELECT User.id FROM User;
[error] parsing failed: ParserError(
    "Expected end of statement, found: .",
)


Wheel connection intermittent

The wheel will show connected and work fine for 10 minutes and then show as not found in the program and stop working.

What OS (windows 11/10) are you running on, and what version?


UP-4956 French documentation - step 1

Checklist

[x ] the individual contributor license agreement is signed
[ ] commit message follows commit guidelines
[ ] tests are included
[x ] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

French documentation for uPortal 5

Be indulgent, I translated the docs during the evening and the weekend, when my child fell asleep. there must be some spelling mistakes, I'll look at this tonight. 😄

Hello @jgribonvald,
tu pourrais jeter un petit coup d'oeil 👀 sur la doc française ?

Why there is a iframe html tag on docs/Accessibility.mb ? this isn't rendered into github view.

@jgribonvald the <iframe> is visible a https://jasig.github.io/uPortal/ACCESSIBILITY.html

Merci @jgribonvald. You rock!


BIGTOP-3556. Upgrade Bigtop-ambari-mpack aligned with Ambari 2.7.5.

https://issues.apache.org/jira/browse/BIGTOP-3556
This PR containing minimum changes for bumping the version.

Thanks @iwasakims .
But I'm working on it.
Agreed with you to makea minimum changes for bumping Mpack version.
In PR #803:

minimum changes for bumping
Upgrading Packages repo to 1.5.0 as some guys ask us to do that in the maillist.


[FLINK-10181][rest][docs] Add anchor links to rest requests

What is the purpose of the change
This PR adds anchor links to every request listed in the REST API documentation.
This makes it easier to guide users to specific parts of the docs.
Brief change log

modify generator to format every rest request url as a heading
regenerate docs

Verifying this change

manually verified

+1 to merge.


chore: actualize dependencies

What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[ ] Bug fix
[ ] New feature
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)
Is there anything you'd like reviewers to focus on?

Codecov Report

Merging #15 (65659e0) into main (3d7efae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   91.20%   91.20%
=======================================
  Files           8        8
  Lines         125      125
=======================================
  Hits          114      114
  Misses         11       11

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 3d7efae...65659e0. Read the comment docs.


Starts and quits

I installed with Choco command. Rebooted. When I go to c:\tools\lxrunoffline and run the exe, it just ghost opens a quick outline of a window and then closes. I don't see any processes running.
https://github.com/DDoSolitary/LxRunOffline/assets/147220784/48eb7969-7dbd-483c-818a-dfbb0b6f9e43

Let me know if you need any more info from me. Thanks!


Direct component template loading in AOT

Currently within the CLI, a full webpack child compilation is created and executed to load a template via the raw-loader in AOT.  The end result is equivalent to reading the file directly.  This PR eliminates the overhead of the extra webpack compilation for the CLI by adding an option to the @ngtools/webpack plugin (defaulting to false to retain existing behavior) that reads the HTML component templates directly from the compiler host if enabled.  This option is then enabled for the CLI.

Updated the readme with the option and information about when not to enable it.

It seems to be hard for a customzied/extend Angular cli builder to turn this option off.


lodash  ^3.0.0 API :: _.any = _.has

When running the JsFiddle with newset lodash 3.x, I had to set
_.any = _.has;
or it crashed.

Apologies for the trouble (at long overdue reply). This doesn't appear to be happening any longer (though I've certainly had my share of headaches with v3 vs v4 of lodash :-) )


travis.yml: add arch s390x target

s390x support comes to Travis CI!
https://community.ibm.com/community/user/ibmz-and-linuxone/blogs/elizabeth-k-joseph1/2019/11/12/s390x-support-comes-to-travis-ci
Corresponding travis doc:
https://docs.travis-ci.com/user/multi-cpu-architectures

Let's see if it works ..

YES!

Let's see if it works ..

Looks like it does!

merged.
master: https://github.com/openssl/openssl/commit/985412f8c14853b9936852bc7ef4d9438db27b88
1.1.1: https://github.com/openssl/openssl/commit/f3d32d3cbaf3edaed3c12ff4dbba5d23cb4519d7


feat(core): Add all currencies in  Currency Enum

Feature Description
Add missing currencies to Currency enum in enum.rs, since we want descent coverage of the currencies. Reference - https://en.wikipedia.org/wiki/List_of_circulating_currencies
Possible Implementation

Add all the missing currencies.
Run the migrations (Can refer to this pr - https://github.com/juspay/hyperswitch/pull/1670/files#diff-b90f2b50c9b9e664a9f6b840c1590342362713cffb3546db779813106242772f)

Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to submit a PR?
None

@prasunna09 going to take a look at this by the evening, and will submit it in morning

Then I'll assign this to you! @tusharxoxoxo

what should be done for these currencies whose iso code is missing?

In this issue, do we only need to add the missing currencies in pub enum Currency or do we also need to implement them with their respective iso_4212 code in the very next function and also the other parts such as pub fn is_three_decimal_currency(self) -> bool { and pub fn is_zero_decimal_currency accordingly??

Yes, you have to add all the missing currencies in pub enum currency and have to add their corresponding iso_4217 codes and also include them in other is_three_decimal_currency(self) -> bool and pub fn is_zero_decimal_currency.

there r some currencies which are mentioned in this Wikipedia article List_of_circulating_currencies which r not present in ISO_4217 like, for example, SVC is not present in the list of currencies, but is present in ISO_4217 list of currencies and vice versa CNY is present in the list of currencies but is not present in the ISO_4217 list

HRK is also not present in ISO_4217 list

There r some currencies which are mentioned in this Wikipedia article List_of_circulating_currencies which r not present in ISO_4217 like, for example, SVC is not present in the list of currencies, but is present in ISO_4217 list of currencies and vice versa CNY is present in the list of currencies but is not present in the ISO_4217 list.

my bad CNY is present in the list

some of the currencies go to the 4 decimal digits so for them should I create a new function or leave them as it is
CLF

there r a lot of currencies that don't have anything in the place decimal row like it's blank


Quantum-resistant crypto

per https://github.com/vector-im/element-web/issues/8889
see also https://github.com/matrix-org/matrix-doc/issues/3520
see also https://github.com/matrix-org/matrix-doc/issues/3516

Wire has done this: https://wire.com/en/blog/post-quantum-resistance-wire/
Disclaimer: I have no idea what this entails.

Okay, so I'm a quantum information guy, and I can chime in:
The main threat to security is that the discrete log problem is no longer hard for quantum computer, and computing the discrete log is only slightly more expensive than a modular exponentiation. So RSA and ECDH, along with most signature schemes, are no longer safe, in the absolute worst sense that the hardness assumption is completely broken and you can easily compute the private key from the public key. In general, fault tolerant quantum computers have an exponential speedup compared to classical computers for any instance of the hidden subgroup problem.
For the time frame, assuming current trends for IBM (others may be slower or faster) and assuming that they can maintain their recent trend of scaling up in a Moores law exponential trend, their quantum computers should be assumed to be able to do this by the end of the decade, with a very boring "scale up and put it in a large building" approach to achieving that target.
For specific protocols that may be used, I would suggest starting by taking a look at the post quantum cryptography wikipedia article. Most of them do something similar to Diffie-Hellman key exchange, but necessarily do so in a structure that is not an abelian group (for example, with matrix multiplication). NIST is currently running a competition between various candidates but hasn't picked a winner yet.
Lattice based cryptography seems to have the most mature understanding of their hardness assumptions with regards to quantum algorithms. The variants that pick weaker hardness assumptions to compress key sizes are also very fast, and the protocols support a very wide variety of features. Typically they are based on the LWE problem, where you can reduce the problem for a random key, to the worst case complexity problem for a lattice problem (typically finding the shortest vector in a high dimensional lattice assuming that a short one exists), so that it has an average case safety guarentee rather than just a worst case complexity guarentee.
In terms of having the most compact keys, protocols based on isogenies of elliptic curves (i.e. replacing exponentiation within a curve with a more general map between curves) seem to have the best performance but are much more complex to understand than classical ECC. I'm currently learning more about elliptic curves in order to properly grok SIDH.

@saolof thanks for the in-depth explanation, do you know anything about any prominent IETF drafts or RFCs that would standardise PQ IANA-designated cipher suites and the likes? Specifically that could be a good yardstick for encryption imo.
Furthermore, I'm tagging @uhoreg into this, to maybe shed light on how MegOLM/MLSv2 could take steps to prepare itself for PQ, or what mitigations should maybe happen here.

I mentioned some of this in the #e2ee room, but to keep the discussion here: From a technical perspective, olm, megolm, and MLS could be easily adapted to use other encryption algorithms.  We basically just need something to do signatures, and something that to do key agreement.  (Aside from olm/megolm/MLS, we also need to worry about server signatures, 3PID invites, etc. but it's basically the same story there too.)  However, switching things will involve a migration plan, because it will affect many things.  e.g. we'll need to find a good implementation of the new algorithms for clients to use, people will need to generate new keys, etc.
I think that we will likely wait until the NIST competition is done before trying to transition to any post-quantum algorithm, before trying to migrate to something post-quantum.

OK, in that case, the general recommendation that I have for anyone getting into this right now is:

The NIST competition will not deliver all recommendations at the same time. It will decide to standardize some of the currently more mature algorithms at the end of round 3 (very near term, as in imminent), and will hold further rounds over time to downselect the other options that are a bit more bleeding edge in order to provide a wider diversity of algorithms over time.

The options that will be selected in round 3 are definitely "good enough" to use in Matrix as a standardized selectable option imho.

For signatures, DiLithium is imo the most practical default option for a general-purpose post-quantum digital signatures algorithm that is likely to be certified near term.

The runner-up is Falcon, which is somewhat complementary and allows for smaller key sizes and ID-based encryption when compared to DiLithium but is much more complex to audit implementations of and has a significant key-leaking footgun if your Gaussian distributed RNG isn't perfectly Gaussian and perfectly cryptographically secure (while DiLithium only needs uniformly distributed integer RNG in a power of 2 range, which is a rather widely available cryptographic primitive).
For political reasons, only one of DiLithium and Falcon will be standardized at the end of round 3, but there is a significant push from the lattice community to standardize both. However, if given a direct choice between the two and given the constraint to avoid fallout in the event of a faulty implementation, I'm think they'll downselect to DiLithium as the immediately standardized option while keeping Falcon in later rounds.
Sphincs+ (hash based) is also fairly likely to get standardized soon because it is an extremely conservative scheme that only assumes that SHA-3 is safe and it has few implementation gotchas, at the expense of having really terrible performance (i.e. operations that are over a thousand times slower than the lattice based ones) and a bunch of other practical limitations.

For key exchange, it's harder to predict what will happen at the end of round 3, other than that Classic McEliece based on Goppa codes will be standardized as the slower backup option, while one of the three competing fast LWE-based options (NTRU, Saber, or Crystals-KYBER) that each rely on some special case of the LWE problem will be standardized. Among those three, afaik it's not obvious to tell which of the three will be selected. But any work on understanding classic McEliece or on general LWE theory is very unlikely to be wasted.

(opinionated) The runner ups that come later still have a good chance of getting long-term adopton

Supersingular isogeny crypto is both incredibly cool and has the best performance by a number of metrics and is my preferred long term option, but it has the issue that specific curves can have backdoors and this matters even more than for standard ECC, so even when an implementation is standardized I would wait until parameters become trusted before really suggesting mass adoption, while the lattice based cryptos just use random parameters and are fairly immune to this.
I personally think that adding optional support for whichever lattice option will be standardized at the end of round 3 (which is very soon) is a perfectly defensible choice. Those standards will be perfectly good enough for most purposes, should be able to provide forward secrecy for any conversations stored from now until FTQCs are widely available, and will have first-mover advantage with regards to adoption.

(note) There are quantum attacks on Hashes and Symmetric key crypto but those are far longer term since the speedups there are square root, and only really just means that the quantum version of Moores law for doing brute force attacks is roughly twice as steep than would be suggested by graphing physical performance measures like number of logical ops per second per dollar. They'd need many decades to catch up to classical computers here so this is not a near term concern, unlike public key crypto where it's as dramatic as "if a quantum computer big enough to perform a single modular exponentiation is built, then all our crypto is instantly broken".


Setup/Teardown run as subtest using provided name

What do you want to see?
At present, when multiple Setup or Teardown steps are defined, it can be challenging to identify which specific step has failed if the proper logs were not configured initially. This results from the fact that while Assessment steps are executed in dedicated subtests, Setup and Teardown steps are not. Actually, names for the Setup and Teardown steps are totally ignored, which I don't think was the desired behaviour.
I think it could be useful to execute all steps in dedicated subtests.
Extra Labels
No response

Ping @vladimirvivien @harshanarayana given that I had your attention for the other issue 😬

+1 to this. This will be a very useful change.

@phisco Do you happen to have a possible format for these setup/teardowns in mind in terms of naming them ? If so, please share that here ?

@phisco I think it's a good idea to take a closer look how setup and teardown are put together. I don't see any harm in running them as subtests similar to other steps like "Assess".

Cool @vladimirvivien, I can give it a look over the holidays and try opening a PR for it!

/remove-lifecycle stale

/remove-lifecycle stale

I really need to find the time to work on this 😩 if anyone else wants to jump in, please do!

This is still relevant and needed.


Delaborator for "not an element" doesn't check for mdata

The delaborator delab_not_in (in Mathlib/Utli/Delaborators.lean) doesn't check for mdata, and as a result it gives the wrong result in certain situations.  These examples illustrate the problem:
import Mathlib.Tactic

example (A : Set Nat) : 0 ∈ A := by
  by_contra h

example (A : Set Nat) : 0 ∈ A := by
  have h1 : 0 = 0 := by rfl
  by_contra h

In the first example, after the by_contra tactic, the display of h in the tactic state is h : 0 ∉ A.  But in the second example, it is h : ¬0 ∈ A.  The reason for the difference is that in the second example, the type of h has the form:
(app (const Not []) (mdata [(noImplicitLambda, true)] (app (app (app (app (app (const Membership.mem ...

It is the presence of the mdata that causes the delaborator to fail to recognize this as the negation of a membership statement.  In the first example, the mdata isn't there, and the delaborator works correctly.
It seems like adding a call to consumeMData should fix the problem.

This change seems to fix the problem:
diff --git a/Mathlib/Util/Delaborators.lean b/Mathlib/Util/Delaborators.lean
index c872f89c38..c36a246291 100644
--- a/Mathlib/Util/Delaborators.lean
+++ b/Mathlib/Util/Delaborators.lean
@@ -159,8 +159,14 @@ open Lean Lean.PrettyPrinter.Delaborator

 /-- Delaborator for `∉`. -/
 @[delab app.Not] def delab_not_in := whenPPOption Lean.getPPNotation do
+  let go := do
+    guard <| (← SubExpr.getExpr).isAppOfArity ``Membership.mem 5
+    let stx₁ ← SubExpr.withNaryArg 3 delab
+    let stx₂ ← SubExpr.withNaryArg 4 delab
+    return ← `($stx₁ ∉ $stx₂)
+
   let #[f] := (← SubExpr.getExpr).getAppArgs | failure
-  guard <| f.isAppOfArity ``Membership.mem 5
-  let stx₁ ← SubExpr.withAppArg <| SubExpr.withNaryArg 3 delab
-  let stx₂ ← SubExpr.withAppArg <| SubExpr.withNaryArg 4 delab
-  return ← `($stx₁ ∉ $stx₂)
+  SubExpr.withAppArg do
+  if f.isMData
+  then SubExpr.withMDataExpr go
+  else go

However, it seems a bit clunky to me. I wonder how many other delaborators potentially hit mdata problems, and whether there is a more elegant way to address the problem.

This looks reasonable to me, but I don't know enough about SubExpr to know if this is the best way to do it.
My original thought was to use consumeMData.  Comparing consumeMData to SubExpr.withMDataExpr, it looks to me like consumeMData repeatedly removes mdata if necessary, whereas SubExpr.withMDataExpr just removes one mdata.  Would it ever be necessary to remove more than one?  Seems unlikely, but whoever wrote consumeMData seems to have been assuming that it might happen.


Is there a way to extract geographical data, i.e. long & lat in this model?

Is there a way to also extract geographical data, i.e. long & lat using this model? Where is long & lat data hidden? Neither .lbl or .hdr files seem to have it and .img file seem to have 438 bands. Thanks!

No, geographical data is part of metadata and they are not available in the HDR or LBL files. They are however available on the CRISM website. For the example image is this:
http://crism-map.jhuapl.edu/details.php?data=hrl_webmap_polygons&shape=5308&x=-165.07382&y=-29.10805
And you can see the coordinates both in the URL and in the summary table.
To find the page for a given file you need to extract the first part of the code for the file name, for example for hrl00016cfe_07_if181l_trr3 you need to select 00016cfe, and search it in the CRISM map.
The CRISM files have indeed 438 bands, but we select a subset. See the crism_to_mat function to see what kind of format is expected by our toolbox.


Can't open the generated .py file

Hey there, I just was trying to use this custom node, but when I was trying to run the generated .py file, I got this error.
Maybe can someone help me with that? I am using comfyUI in a linux cloud computer.

Were you able to solve this problem @spacecodee?

Were you able to solve this problem @spacecodee?

Hello, no, I wasn't, I haven't tried it again since then.


CASE Statements with multiple conditions

Case()
     .when(
       ((AliasedQuery("aa").user_type == 'dormant')) , True)
        .else_(False)
        .as_('dormant_previous')
                  )

The case statement works if there is a single condition to it. Pypika will generate out query
        CASE
        WHEN "aa"."user_type"='dormant' THEN true
        ELSE false
        END "dormant_previous"

However, if there are 2 conditions to it.
    .when(
        ((AliasedQuery("aa").user_type == 'dormant') and (AliasedQuery("bb").user_type == 'dormant')) , True
         )
         .else_(False)
         .as_('dormant_previous')
)

It does not capture all the conditions
        CASE
         WHEN "bb"."user_type"='dormant' THEN true
        ELSE false
 END "dormant_previous"

The documentation/ examples do not show anything related to multiple conditions in a WHEN statement. Attempts to use AND to link conditions failed as shown in the code above.
Any way to resolve this?

It would be useful to have using & in the tests and docs
This issue could be reopened to track adding this in


How to override table name

I am deploying an app to different environments and have a table per environment with the env in its name. How do I go about changing the name dynamically at runtime?
I've seen the extension WithTableName that I can add to every single query i make, but when i create an instance of the DynamoDbContext why can't I set the table name at that time? I am now going to have an app with the repository data pattern, 7-8 repo's with 4-9 functions a piece each with a table name that has to be appended and injected in. There has to be a better way lol

Sorry for the late response. I was able to work out something using the information that you provided that worked for me. Thank you very much!


MudDatePicker Doesn't Allow Dates After 11/30/9999

Bug type
Component
Component name
MudDatePicker
What happened?
If I use the Editable MudDatePicker on the MudBlazor Component documentation page, I can enter 11/30/9999, but if I drop the calendar down, and click the forward arrow, it throws the exception below. Is this a known bug? I didn't find it when I searched Issues.
System.ArgumentOutOfRangeException: ArgumentOutOfRange_BadYearMonthDay
at System.Globalization.GregorianCalendar.GetAbsoluteDate(Int32 , Int32 , Int32 )
at System.Globalization.GregorianCalendar.DateToTicks(Int32 , Int32 , Int32 )
at System.Globalization.GregorianCalendar.AddMonths(DateTime , Int32 )
at MudBlazor.MudBaseDatePicker.GetMonthStart(Int32 month)
at MudBlazor.MudBaseDatePicker.GetMonthName(Int32 month)
at MudBlazor.MudBaseDatePicker.<get_PickerContent>b__141_3(RenderTreeBuilder __builder4)
at Microsoft.AspNetCore.Components.Rendering.RenderTreeBuilder.AddContent(Int32 , RenderFragment )
at MudBlazor.MudPickerContent.BuildRenderTree(RenderTreeBuilder __builder)
at Microsoft.AspNetCore.Components.ComponentBase.<.ctor>b__6_0(RenderTreeBuilder builder)
at Microsoft.AspNetCore.Components.Rendering.ComponentState.RenderIntoBatch(RenderBatchBuilder , RenderFragment , Exception& )
Expected behavior
I should be able to enter or scroll to any allowable date in C#.
Reproduction link
https://mudblazor.com/components/datepicker#basic-usage
Reproduction steps

Go to https://mudblazor.com/components/datepicker#basic-usage
Enter 11/30/9999 in the "Editable with Placeholder" sample under "Basic Usage"
Click the calendar icon to open the calendar
Click the forward arrow
...

Relevant log output
No response
Version (bug)
6.2.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Hello @skincer357 ,
You exceeded DateTime.MaxValue.
Here are the microsoft docs.

Thank you for the reply.  Perhaps I’m not understanding you.  I don’t see how anything beyond 11/30/9999 exceeds DateTime.MaxValue.  For example, the following code yields the output shown.
DateTime dateA = new DateTime(9999, 12, 31);
Console.WriteLine($"dateA = {dateA.ToString("MM/dd/yyyy HH:mm:ss")}");
DateTime dateB = DateTime.MaxValue;
Console.WriteLine($"dateB = {dateB.ToString("MM/dd/yyyy HH:mm:ss")}");
Output
dateA = 12/31/9999 00:00:00
dateB = 12/31/9999 23:59:59
The Microsoft docs you link to also appear to indicate that values in December of 9999 are valid – “this constant is equivalent to 23:59:59.9999999 UTC, December 31, 9999 in the Gregorian calendar, exactly one 100-nanosecond tick before 00:00:00 UTC, January 1, 10000.”
Thanks again.
Steve
From: Tyme-Bleyaert @.>
Sent: Thursday, April 20, 2023 11:07 PM
To: MudBlazor/MudBlazor @.>
Cc: skincer357 @.>; Mention @.>
Subject: Re: [MudBlazor/MudBlazor] MudDatePicker Doesn't Allow Dates After 11/30/9999 (Issue #6714)
Hello @skincer357 https://github.com/skincer357  ,
You exceeded DateTime.MaxValue.
Here are the microsoft docs https://learn.microsoft.com/en-us/dotnet/api/system.datetime.maxvalue?view=net-8.0 .
—
Reply to this email directly, view it on GitHub https://github.com/MudBlazor/MudBlazor/issues/6714#issuecomment-1517309846 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A5CECOL7YFBGDKMOUFYWH23XCIPYPANCNFSM6AAAAAAXGEVL6Q .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/A5CECOIZENE5LQETEH4KNEDXCIPYPA5CNFSM6AAAAAAXGEVL6SWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS2OBHZM.gif Message ID: @.*** @.***> >


Muscles don't change color when playing animation from states

Encountered when testing the "Strength of Simulation" example on Confluence, Part I, Section C (using AppVeyor artifact OpenSim-dd7dc47b-2018-02-11.zip). To reproduce:

open leg6dof9musc.osim
run Static Tool (Motion = Swing\leg69_IK_swing.mot); muscles change color as Static solves
play animation; muscles change color as expected
run Forward simulation (using either :running_man: button or Forward Tool); muscles remain blue (even if excitations were nonzero; see #520)
in Navigator, double-click "states" generated by Static to switch back to previous results
play animation; now muscles do not change color, even when playing results from Static

Possibly related to #520.

Same bug found in 3.3.

While unexpected this isn't new behavior, as confirmed by @tkuchida . The issue is specific to SO and the main cause is that SO doesn't produce states, instead it produces separate activations and kinematics that the GUI puts together in ad-hoc way.

A holistic fix here would start by requiring the Tool to produce a StatesTrajectory (for model display) and not have the MotionDisplayer be overwhelmed with complex logic for trying form a complete state for visualization/animation of Tool generated results. Analyses do not and should not affect the state of the active model in the GUI.
In this case we need a StaticOptimizationTool that is responsible for updating the activation states based on the SOAnalysis that it runs internally (secretly).

@aseth1 Feel free to take on the holistic approach on the API side and the GUI will follow. I agree it's hacky to try to marry kinematics from the AnalyzeTool to activations from StaticOptimization but that seems beyond the scope of this issue. Thoughts? Note that results from FD and other tools that produce states always playback with correct colors.

The behavior is the same as 3.3 so will leave for now and address in a future release.


Patch for Hierarchical Search

Attempt to implement Hierarchical Search along with junit unit tests to automate tests with proper documentation.

Hi
I have compiled it successfully using jdk 8 ,eclipse and all Junit 4.11 unit tests were also successful also passed ,but here it says all checks failed. Is there something else that i need to do to pass compilation test here. Please point it out if any.
Thanks
Subham

Hi @smsubham, as you can see from the Travis-CI log
1> You have created two Problem.java classes:

aima-core/src/main/java/aima/core/search/nondeterministic/HierarchicalSearch/Problem.java
aima-core/src/main/java/aima/core/search/nondeterministic/problem.java

2> The issue is with the second path, where the filename is "problem.java", but is declared as "public class Problem" (Case Sensitive)
3> I'd recommend you use the already existing class for problem defined in:
package aima.core.search.framework;

Thanks for pointing it out. Just made an silly mistake while commiting files. Will fix it soon @anurag-rai


Fix 'PageData.LanguageBranch' is obsolete: 'Use ILocalizable.Language…

… instead' with the html element on _Root.cshtml

@kmukku Then we'll close the PR :) If you disagree let us know.


Fix parameter check constant

Codecov Report

Merging #82 into develop will decrease coverage by 0.19%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           develop      #82     +/-   ##
==========================================
- Coverage    60.16%   59.96%   -0.2%
==========================================
  Files            5        5
  Lines          605      607      +2
  Branches       129      130      +1
==========================================
  Hits           364      364
- Misses         217      218      +1
- Partials        24       25      +1

Impacted Files
Coverage Δ

petab/lint.py
40.74% <0%> (-0.44%)
:arrow_down:

Continue to review full report at Codecov.

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


[GR Phase 3] Step 5 - navigation, default data, logic and API call to save it

Design doc section: https://docs.google.com/document/d/1Y4P7G9h0Easjcea7wyk52gKrNB9o1yTjpmJFYkrHDW4/edit?tab=t.0#bookmark=id.mdmothqmscoy

Hi! I'm Bartek from Callstack - expert contributor group. I’d like to work on this task.

Will look into this when I'm back from OOO