Failure to attempt to change Data Formats

System information:

Operating system (distribution) and version: Win10 Pro v.2004
DBeaver version: Version <IP_ADDRESS>006211844
Java version: build 1.8.0_241-b07
Additional extensions: none

Connection specification:

Database name and version: Oracle 10G
Driver name: oracle.jdbc.OracleDriver
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
Trying to change data formats and locale but doesn't save the changes or just reload the global settings after "apply" and "apply and close".

Steps to reproduce, if exist:

Window/Preferences/DBeaver/Editor/Data Formats

Manage Profiles
1.1 New Profile
1.2 Language: pt-BR
1.3 Contry: Brazil
Format (date): dd-MM-yyyy
Format (timestamp): yyyy-MM-dd HH:mm:ss
Apply
Apply and Close
Repeat step 1 and you probably will see the glosal settings instead the new settings

Thank you for the report. It is actual for all connections.

@CarlosDias01 A littler investigating was needed.
The described behaviour is expected.
The profile is needed to select it during Export.

You can set custom settings for your connection in Edit connection>Data viewer>Data Formatting

Please open new ticket if you have any other questions.

Hi @kseniiaguzeeva
I didn't know about the profile to export but all the preferences are the same, including the Global settings.

To add profile to database settings will be implemented as feature request.
You can set your settings in global profile: window>preferences>dbeaver>editors>data editor>data formats. They are saved.


TEM2015 Test 02:  There are two versions of the Coastal Relief Model

There are two versions of the Coastal Relief Model.  One is owned by ReleasedIntel and the other is owned by admin.  The ReleasedIntel points to the develeopment server while the admin service points to the Test server.
To reproduce search for Coast Relief.

The portal item owned by "admin" is the publisher of the service on the test ArcGIS Server. The portal item owned by ReleasedIntelligence is the owner of the service on the dev ArcGIS Server. The RemapIDsOnServices.py script should modify the guid stored in the service json so that the service is remapped to the original owner. The fact that the ReleasedIntelligence item still references the dev server url should explain the issue #1298. At this point I'm not sure why the url of this item references the dev server because the server name on other portal service items were modified to reference the test server.

@NatalieCampos @conklinbd I'll manually edit the service json to point to the item owned by "ReleasedIntelligence" and delete the item owned by "admin".

@NatalieCampos @conklinbd Finished the modifications as well as editing the URL on the portal item. Assigning back to verify

The fix has been verified.


HLS spec : allow #EXT-X-MEDIA URI to be optionnal

Have you read the FAQ and checked for duplicate open issues?
yes
Is your feature request related to a problem? Please describe.

in our HLS manifest the first #EXT-X-MEDIA do not have an URI
in current shaka HLS implementation that means it will be dropped
as per HLS spec URI is optional for #EXT-X-MEDIA

When its URI attribute is omitted, the EXT-X-MEDIA tag can indicate that the media described is included in the URI of the EXT-X-STREAM-INF tag.

source : https://developer.apple.com/documentation/http_live_streaming/example_playlists_for_http_live_streaming/adding_alternate_media_to_a_playlist
Describe the solution you'd like

as per the spec if for a EXT-X-MEDIA the URI is missing, then we should look into the #EXT-X-STREAM-INF to get the URI
Describe alternatives you've considered

Additional context

more info and discussion in #1850

@fadomire Is it a case of a TS stream carrying both audio and video?

@ismena hmm yes in our case the TS stream carry both audio and video
but this feature request is more generic than that i think

Yeah, sorry, I was just trying to understand the concrete use case :)
Do you have stream for us to test?
I looked through the comments in #1850, but I don't think there was a stream there.

yes i got the authorization to share the stream :
https://ios-q1-ssl.tf1.fr/2/USP-0x0/53/75/13615375/ssm/13615375.ism/13615375.m3u8

Hello @fadomire ,
Thank you for sharing the stream!
For this file, Shaka is not loading the video successfully not merely because of dropping #EXT-X-MEDIA tags without URI. The playlist claims separate audio and video streams, however there's audio and video together in the video streams in TS, and that's where we get inconsistent information and fail to load the video.
Do you have a more clear content that we can test with?

Hello @michellezhuogg
what do you mean by more clear content ?
we have this stream too
https://ios-q1-ssl.tf1.fr/2/USP-0x0/18/80/13631880/ssm/13631880.ism/13631880.m3u8
also i'm not sure i understand what you mean by "The playlist claims separate audio and video streams" how can i check that in the manifest ?
we dont do anything special when packaging. We use this packager "Created with Unified Streaming Platform(version=1.8.4)"

Hey @fadomire
How familiar are you with the HLS manifest structure? There's something in the first piece of content you provided (we haven't looked at the second one yet, but will do) that totally confuses the player and it's a bit tricky to explain.
I will do it (explain), but I don't want to annoy you with a lecture on HLS spec if you're pretty familiar with it already.
Let me know and I'll walk you through what's up with that manifest!

UPD: Your second piece of content has the same pattern! :)

hey @ismena i'm not familiar enough with HLS spec so an explanation would be great !
basicly, we just use Unified Streaming Platform command line to create our manifest (dash and hls) and expect them to respect the specs :)

Ok, there we go: HLS spec has media tags and variant tags.
Variant tags describe combinations of media, something like "you can use this video with this audio, here are the codecs you will need and this is how much bandwidth it will take". They look like this (from your content):
#EXT-X-STREAM-INF:BANDWIDTH=282000,CODECS="mp4a.40.2,avc1.42C01E",RESOLUTION=416x234,FRAME-RATE=25,AUDIO="audio-aacl-64"
13615375-audiodescription=64000-video=200395.m3u8?

// Use the video from the uri "13615375-audiodescription=64000-video=200395.m3u8?" with any audio from audio group "audio-aacl-64" (I'll explain about audio groups in a sec).
There are also media tags that describe specific media, like "This is an audio playlist, it's in French and it has two channels".
Audio tags look like this:
#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID="audio-aacl-64",NAME="English",LANGUAGE="en",CHANNELS="2",URI="13615375-eng=64000.m3u8?"
There might be several of them with the same group id (same audio in different languages, for example).
Now, the original reason for this issue was that we did not take multiplexed streams into account and assumed all media tags will have a uri. Not true. If video and audio are carried in one stream, one of them will have a media tag (usually audio), but no uri.
So, here's the deal with your content.
It has several audio media tags:

Audio in English, has it's own uri
Audio in French, has it's own uri
Audio in French, does not have it's own uri, is multiplexed with the video.

The Variant tag says,  we can pair the video with any of these audios.
But we can't! There is no way we can separate audio and video multiplexed together and play the video with another audio stream. Safari might be able to do it cause they're not working through Media Source, but not us.
Even if we stop looking for the URI tag where none exists, what happens is we try to treat a video+audio stream like a purely video stream and it's not working.
The only thing we can actually play is the multiplexed stream of video and audio together.
I think, the best we can do is try and discard all the audios that we can't play and proceed with that single multiplexed stream. I'll try and schedule this work for our 2.7 hls-focused release.
In the meantime, do you have any control over the manifest of your content? I can do a quick-ish fix to not require the uri, but to get your content to play both audio and video (the multiplex stream that we can feasibly play), you'll need to cut the other audio variants from the manifest (for now, until we can work around this structure). Is this something you can do?

Sorry for the wall of text, let me know if you have any questions.

Please check with Unified Streaming to see if you are running the latest software from them.  If not, try updating first to see if this is an issue they have already dealt with.

ok i will check on their side
i will also read the detailed reply from @ismena (thanks), read it quickly but need more time
but one thing is that for e.g hls.js allow us to choose between all the variants we expect as you can see in their demo with our manifest
https://video-dev.github.io/hls.js/demo/
is there something special they do ? not meaning to say it is better over there, i choosed shaka for many reasons and happy with the choice

@fadomire Maybe they are better at some things at hls.js, nothing wrong with that ;)
We'll take a look, see if we can learn from them how to handle this case better!

UPD: I've taken a look at hls.js. My best bet is, they're handling transmuxing differently.
If given a multiplexed ts stream, they demux it to two separate streams, one video and one audio.
We mux it back to a single mp4 stream (or rather mux.js does for us).
It is the limitation of our current architecture and something we're not planning to change in the near future (not this year at least).
This might change if it turns out to be a vital use case for the industry, but for now it is what it is :(
If there's any chance you can encode your content to be consistent in terms of multiplexing/not multiplexing streams, it would be awesome. Sorry about that!

thanks for taking a close look
i will see with our backend if something can be done

The original bug us still relevant: we should not require the URI for all media tags :)

Nooope, this is still an issue!


fix(cli): windows EOL

Please verify the following:

[ ] yarn test jest tests pass with new tests, if relevant
[ ] README.md has been updated with your changes, if relevant

Describe your PR

Closes #2551
Not every dev machine on Windows has the unix2dos utility installed via git apparently, so we changed it up to just change line endings more manually
Also added a try/catch around this region specifically, since it doesn't need to kill the entire new command

:tada: This PR is included in version 8.11.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update preview example of the action

Use branches on pull_request_target
Mention limitations of the preview action

See: https://progress.opensuse.org/issues/158236

Hum. The branch is on the wrong repo. I don't know how since my config uses my fork. Will try again.


Webpack HMR is broken

Bug Report
Always have to manually reload the page after each change.
Error:
GET http://localhost:3000/__webpack_hmr net::ERR_CONNECTION_RESET 200 (OK)
Steps to Reproduce
Didn't change any default config
Other Details
Payload v.1.3.4

Hey @johannesschaffer — this will happen if you change server-side code, like adding a new field or adding a new collection. If you are updating admin-only components like .tsx, .scss, or .jsx files, the hot reloading will work.
This is because the server relies on nodemon and actually does restart. However, there are a few discussions happening on Discord about opening up a way to use HMR on the server as well which would be awesome. I will convert this to a discussion regarding server HMR and we can track progress accordingly!


[FLINK-33263][bugfix][table-planner] Update the flink shaded guava version used to 32

What is the purpose of the change
Bugfix for FLINK-33263. Update the flink shaded guava version used in SourceTransformationWrapper to 32.
Brief change log

Update the flink shaded guava version used in SourceTransformationWrapper to 32.

Verifying this change
This change added tests and can be verified through CI progress.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

Hi @libenchao , this PR is a hotfix for the wrong version usage of flink shaded guava and change the version to 32. Please review this commit. Thanks a lot.


preventDefault:true option overwrites method

I have a listener like this:
<input @keydown.enter.prevent="enter">

And dispatch the event like this:
wrapper.trigger('keydown.enter', {
  preventDefault: true
})

And get the following error
Uncaught TypeError: $event.preventDefault is not a function

It iterates over all options and set them:
https://github.com/vuejs/vue-test-utils/blob/dev/src/wrappers/wrapper.js#L533
preventDefault should not be set to true. Suggest either changing the option name to a non conflicting one, or removing it from the iterated keys. Or set it to a noop function.
I know I am doing preventDefault twice, but it shouldn't fail anyway imo.

Thanks for the bug report, I agree that it shouldn't error.
Would you be able to make a PR to fix this?

I can do that.

I wonder though, why is that option there in the first place? Isn't it the code under test that has the responsibility to prevent the default action? Now you can have a test that passes since the test itself prevents the default, but it is no guarantee that the code under test prevents the default action.
Aren't triggering events meant to simulate user interactions? The user can't (AFAIK) prevent the default action to happen.

You're correct, I don't think that option should be there.
I believe it was added in response to a user request, but I can't find the issue. I'm going to remove the code that runs preventDefault.


Failed to load weights

Hello,
I have just cloned the github repository and was trying to check that it works by predicting the test/test.mp4 video. I am running the predict.py script like so:
%run predict.py --HEIGHT 720 --WIDTH 1280
However, I get the following error:
Fail to load weights, please modify path in parser.py --load_weights Load csv file successfully
I have tried modifzing the parse.py file to try one of the two files that are in the weights folder (weights/TrackNet_final.index and weights/TrackNet_final.data-00000-of-00001), since the one by default (weights/TrackNet) does not exist.
Is there anything I am doing wrong?
Thank you for your help

I've correct the name of two files. Please try it again.
(Use original setting "weights/TrackNet" in parser.py)
If any question raised further, please let me know, Thanks.
Regards,
Max

Closed the issue.


docs/en: Replace "have in mind" with "keep in mind"

"Keep in mind" is how you would express this in an imperative form.
First pass was done on macOS 10.15 with:
LC_ALL=C find docs -type f -exec sed -i '' 's/Have in mind/Keep in mind/g' '{}' +
LC_ALL=C find docs -type f -exec sed -i '' 's/have in mind/keep in mind/g' '{}' +
git restore docs/en/docs/img

📝 Docs preview for commit 5805b1051464b2b9a0824243535fe43b251141d8 at: https://75293bc4.fastapitiangolo.pages.dev

As a non-native speaker I wondered if Have in mind is wrong, or what the difference is between the two.
The definition of have in mind:

To be considering someone or something as a candidate or possibility for something. In this usage, a noun or pronoun can be used between "have" and "in."

The definition of keep in mind:

Used to introduce something that one wants to emphasize or be remembered.

So I think both versions are valid, but the later fits the situations it is used in the documentation better.
Another random discussion in a language forum:

I think there is no difference in meaning between the three: "While doing AAA, (have/keep/bear) in mind BBB." All of them say "for the period of time you are doing AAA, also think about BBB, because it affects your understanding of AAA." All are good AE usage.
bear in mind is the most common. Here bear literally means carry. It is easy to imagine carrying BBB while doing AAA.
keep in mind is also common in this use. It has the concept of holding BBB while doing AAA.
have in mind is okay, but it could be a little bit confusing, only because to have something in mind (meaning "to already have your own ideas about the current topic") is such a common phrase.

Based on that information, I support that change.

📝 Docs preview for commit d9f8b017eabc2c4a7f854c9e765e664c52a591bc at: https://6975e493.fastapitiangolo.pages.dev

Thank you! @malicious 🤓
And thanks for the help too @gregmuellegger 😸


How can I get the current size of expired cache?

I want to show the expired cache size, but I haven't find some function supporting this. Who can show me the correct way to handle this? Thanks.

Does calculateDiskCacheSizeWithCompletionHandler work for you?

There is a calculateDiskCacheSizeWithCompletionHandler method to show you the size of the whole disk cache. But now there is no method or property exposed for the expired cache size. The cache system is more likely a black box and generally speaking you could use it without worrying this. It is really a rare need to show the expired cache size (since they will be cleaned out from disk when the user switch you app to background).


The program will terminate after the error occurs

I developed a desktop client with golang. According to the incoming EXCEL, open multiple browsers and call golang functions in batches to process tasks (each task is automatically clicked and submitted). When I manually close one of the browsers, my The desktop client can no longer call new golang functions. Depending on when the browser is closed, different errors will be displayed in the command line, for example:

"page load error net::ERR_ABORTED
logout
...copying shared history...
...saving history...truncating history files...
...completed.
[Process completed]
"
or
"context canceled
logout
Saving session...
...copying shared history...
...saving history...truncating history files...
...completed.
[Process completed]
"
How to receive these errors, How to continue normal operation of other logic after the error occurs，currently these errors will cause my desktop to be unable to call the new browser to perform tasks

@ZekeLu God save me

open multiple browsers

Maybe it's better to open multiple tabs in a single browser.

When I manually close one of the browsers

Why not run the browser in headless mode? And why you want to close the browser manually? It should be better to manage the lifecycle of the browsers (or tabs) with the context.Context.

How to receive these errors, How to continue normal operation of other logic after the error occurs

In golang, the error is just a value, it won't force your process to exit (as far as I know). You should have got the error returned from chromedp.Run. And I guessed you have called something like log.Fatal(err) when err != nil, which will exit the process. If you think that the error is expected and can be ignored, just handle it differently.
I can not give you more advices without seeing the source code. Good lucks!

Thanks！

It is a good idea to run the browser in headless mode, but it may not be allowed in terms of requirements, especially when logging in sometimes requires manual dragging of pictures to the specified location for verification. .
The purpose of opening multiple browsers is to avoid the anti-crawl mechanism by specifying different user-data-dir and user-agent. I haven't tried to open multiple tabs for the time being.
The reason for turning off one of the browsers is that ordinary users may do this. .
In fact, none of the above problems is a problem. As long as the program can run normally after receiving the error, I can continue to open the next browser.
Code:

func (c *guiFunction) upload(uploadArgs uploadParams) string {
	var execPath string
	execPath = uploadArgs.ExecPath
	if *brwoserPath != "" {
		execPath = *brwoserPath
	}
	logs.AllowFileLogs()
	tik := &params{
		ctx:         nil,
		isLogin:     false,
		isClose:     false,
		userAgent:   uploadArgs.UserAgent,
		userDataDir: `./port/`,
		port:        uploadArgs.Port,
		videoPath:   uploadArgs.UploadPath,
		topicFirst:  uploadArgs.TopicFirst,
		topicSecond: uploadArgs.TopicSecond,
		topicThird:  uploadArgs.TopicThird,
		content:     uploadArgs.Content,
	}

	tik.ctx, tik.cancel = chromedpEngine.NewExecCtx(
		chromedp.ExecPath(execPath),
		chromedp.UserAgent(tik.userAgent),
		chromedp.UserDataDir(tik.userDataDir+tik.port),
	)
	defer tik.cancel()
	if err := chromedp.Run(tik.ctx, tik.uploadTasks()); err != nil {
		logs.Fatal(err)
		//tik.cancel()
		logs.Println("fail")
		return "fail"
	}
	//tik.cancel()
	logs.Println("success")
	return "success"
}

Understood. Saw logs.Fatal(err) in your source code. Comment out that line and it should work as what you expected.

明白了 看到logs.Fatal(err)了您的源代码。注释掉该行，它应该可以按预期工作。

Great, it's okay


pull request for e2e test at 01/12/2017 09:56:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore(package): update grunt-browserify to version 5.3.0

Closes #26

Coverage remained the same at 22.17% when pulling 0b5943888807f1252e1738dd44a087876d279b03 on greenkeeper/grunt-browserify-5.3.0 into e81bea253112c96c4fe18a74eb18c3681cbb49cd on master.


Update WALLY to Node 15

Describe the task
To update WALLY to Node 15.
Purpose
Due to Platform Services image tag prune, we have lost Node 10 in artifactory. Upgrade to Node 15 to continue using BC Gov artifactory or find Node 10 in a public repository.
Acceptance Criteria

[ ] Do upgrade locally and successfully test locally. If successful, move to OpenShift.

Additional context

Expecting updating to Node 15, there will be extra debugging and updates required.

We will be updating to the next supported version of Node


Add https://images.search.yahoo.com/ to image section

Add https://images.search.yahoo.com/ to image section

Hey! Can you assign this issue to me?

of course!

Done!


Fix the Nuget feed.

removing retired feed from the list

@schellap please have a look.

LGTM

the failures looks because network connectivity issue. so I'll merge the change to unblock compiling corert


fix(store): fix inferrence of array types

Previously produce would wrongly infer all arrays from a store as readonly.
This fix makes the return type of produce StoreSetter instead, fixing the issue.
This change has a few caveats; namely, produce is less strictly typed: it should return a function that takes and returns DeepReadonly<T>, but now it returns all types contained in StoreSetter<T>. This has no consequence for normal usage other than better type inference.
Additionally to facilitate this change, DeepReadonly is no longer applied to the entire SetStoreFunction, but only where it is consumed (the setter and filter functions).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.454%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1197

Relevant Lines:
1278

💛  - Coveralls


fix(lint/noUselessElse): ignore else clauses of if that doesn't break

Summary
Fix #924
Fix #920
Test Plan
I added a test and updated the documentation.

@net-tech
i take the opportunity to address #920 by updating the documentation with your suggestion.


feat(rules): detect httpx for S113

Summary
Bandit now also reports B113 on httpx (https://github.com/PyCQA/bandit/pull/1060). This PR implements the same logic, to detect missing or None timeouts for httpx alongside requests.
Test Plan
Snapshot tests.

Not sure a missing timeout is a problem since httpx uses 5s as default. The rule is about preventing the client from waiting indefinitely, which it never will in the case of httpx and a missing timeout. Setting timeout to None is still problematic though.

httpx has default timeout， this would be a false positive

Not sure a missing timeout is a problem since httpx uses 5s as default. The rule is about preventing the client from waiting indefinitely, which it never will in the case of httpx and a missing timeout. Setting timeout to None is still problematic though.

Indeed, really sorry for missing that obvious information, I should have better checked that.

false positive fix https://github.com/astral-sh/ruff/pull/12213


remove tap command for homebrew

No longer needed, it's already in core formula

Codecov Report

Merging #149 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   26.12%   26.12%
=======================================
  Files          13       13
  Lines        1129     1129
=======================================
  Hits          295      295
  Misses        821      821
  Partials       13       13

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 8970352...c764c9e. Read the comment docs.

Codecov Report

Merging #149 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   26.12%   26.12%
=======================================
  Files          13       13
  Lines        1129     1129
=======================================
  Hits          295      295
  Misses        821      821
  Partials       13       13

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 8970352...c764c9e. Read the comment docs.

Yes, we are aware of this, but idk if it's a good idea to completely not mention the tap.
I mean it's already nicely configured and updates are almost instant, where in the core one must to manually submit a PR and wait for maintainers.
So I propose to mention that the tap exists and can be used to get updates quicker.

I agree with @dawidd6 :)

@dawidd6 @jesseduffield I've updated my commit after your valuable comments.

Nice, thanks man.

@dawidd6 you're welcome:)


feat(core): Upgrade API kubernetes 1.28

Ref https://github.com/apache/camel-k/issues/5307
Upgrade to API kubernetes 1.28

Release Note
feat(core): Upgrade API kubernetes 1.28

@gansheer is this good to merge?

@gansheer is this good to merge?

At last it is ready :+1:


As a moderator, I want to create and edit related links in wagtail so that I can associate them with projects and people.

testing notes
fyi - "related links" were previously called "resources"; we've renamed them to be a bit more clear. they allow you to add a particular link to a person or project, which will show up on the corresponding page or profile.
the "website" related link is special - if you add a website to a person, clicking on them on a list page will take you to that website (unless they already have a profile on the CDH site). we do this often for faculty, who have profile pages on departmental websites that we want to link to instead of making them a profile on our site. if you add a website to a project, the website will be displayed in a special way on the project's page, showing its URL.
as a moderator
projects

go to the admin site and edit any project. scroll down to the bottom until you find the "related links" section.
add a new related link of any type. if the project doesn't have a website link, add a website link too.
visit the project's page by clicking "preview". you should see the link you added, as well as the website link, and the website link should show the URL you entered.

people

go to the admin site and edit any non-staff CDH person. scroll down to the bottom until you find the "related links" section.
add a website link for this person.
find the person on whichever person list page they will show up (e.g. students) and click them - you should go to the website URL you added.
go back and edit any CDH staff member, repeating step 1 above.
add any kind of related link for this person you like, including a website link.
visit the person's profile by clicking them on the staff list page. this should take you to their profile, not the website URL you added (we only use that URL if you don't have a profile on our site).
you should see any other related links you added on the person's profile, shown on the left.

dev notes
This is ready to be worked on once #268 is merged.
background
n.b.
You will see the term "Resource" used throughout the codebase, but the title of this issue uses the term "Related Link"; this is the term we want to use going forward because it's more descriptive. Thus related links for people will be renamed from PersonResource to PersonRelatedLink, etc.
We currently allow associating arbitrary URLs (related links) with two models in cdh-web: Person and Project. This is used, for example, to display a list of relevant links on a person's profile page:

More importantly, it lets us link to profile pages hosted on external sites, such as departmental websites. We often do this for faculty who already have a profile page elsewhere; see the link below for Marina which goes to nes.princeton.edu. This functionality is powered by creating a PersonResource (a related link) with a ResourceType of website. When a Person does not have a ProfilePage internally (on the cdh site), this URL is used instead.

Part of the utility of these related links is that you can edit them directly ("inline") when changing a Person or Project in the admin backend, rather than needing to individually create links one-by-one and then associate them with the Person or Project (see below screenshot). We want to preserve this functionality, but Wagtail has a different way of supporting it, which forms the basis for this issue.

implementation
the wagtail docs on inlines specifically address our use case, and we can use their example with only slight modifications. the idea is that we have an abstract model RelatedLink that describes relating a link to anything, and then concrete models extending it that describe relating links to particular things (Person and Project, in our case).
the first and simplest thing we want to do is to rename the ResourceType model to RelatedLinkType for more clarity, since the name "resource" doesn't mean very much. take a look at cdhweb/resources/migrations/0008_userresource_to_personresource.py for an example of a migration that does this with migrations.RenameModel. you can rename the model in resources/models.py and save the file, then create the migration manually using python manage.py makemigrations resources --empty and edit the resulting file to add the RenameModel command. make sure to give the generated migration file a descriptive name.
next we will want to copy the wagtail docs RelatedLink example into resources/models.py. we just need to change the title field to a ForeignKey to RelatedLinkType, since that's where that information is stored. we also call the url field url instead of link_external, and don't allow it to be blank. so it would look like:
class RelatedLink(models.Model):
    type = models.ForeignKey(RelatedLinkType, on_delete=models.CASCADE)
    url = models.URLField()

    panels = [
        FieldPanel('type'),
        FieldPanel('url'),
    ]

    class Meta:
        abstract = True

you can see how this model now abstracts away the repeated parts of PersonResource and ProjectResource (type and url). create another migration using python manage.py makemigrations resources; django should populate it correctly automatically.
now we just need to create concrete versions of RelatedLink for Person and Project - PersonRelatedLink and ProjectRelatedLink. here's an example for Person:
class PersonRelatedLink(Orderable, RelatedLink):
    person = ParentalKey(Person, on_delete=models.CASCADE, related_name="related_links")

instead of storing them in resources, let's put them in the apps to which their "related" model belongs: people/models.py for PersonRelatedLink and projects/models.py for ProjectRelatedLink. then we can again create migrations for each of those apps, which should be generated automatically.
finally, we need to make these related links inline-editable on their respective models. in the wagtail example, that's BookPage - for us it will be Person and Project. for Person, all we need to do is add another panel to the available panels in the admin, e.g.:
class Person(ClusterableModel):
  # ...

  content_panels = Page.content_panels + [
    #....
    InlinePanel("related_links", heading="Related Links"),
  ]

Project isn't yet a wagtail Page model, however, so we can't do the same there until #182 is completed.
there's not much to test here, since all the logic is managed by Django in the form of database fields/models. the only thing to update is to rename/update the test for ResourceType in resources/tests/test_models.py, since it got renamed. you could also look for lingering instances of "resource" to rename across the codebase.
if you want to check out how things looks, fire up a dev server and visit localhost:8000/cms. if you select a person in the wagtail admin, you should see new fields for adding and removing related links.
that's everything this issue covers - the actual migration of PersonResources and ProjectResources into PersonRelatedLinks and ProjectRelatedLinks will be #249.
checklist

[x] migration to rename ResourceType to RelatedLinkType
[x] create RelatedLink model and migration
[x] create PersonRelatedLink model and migration
[x] create ProjectRelatedLink model and migration
[x] make PersonRelatedLink inline-editable on Person
[x] make ProjectRelatedLink inline-editable on Project
[x] update tests & check for any remaining references to "resource"
[x] verify things look right in wagtail admin

FWIW, django may be able to detect the model rename and generate the migration for you. (IIRC it asks if you if you did a rename)

FWIW, django may be able to detect the model rename and generate the migration for you. (IIRC it asks if you if you did a rename)

Testing notes:

[ ] Go into the admin interface and add and remove a related link from a project and make sure the changes are reflected in the page.
[ ] Go into the admin interface and add and remove a related link from a profile and make sure the changes are reflected in the page.

Testing notes:

[ ] Go into the admin interface and add and remove a related link from a project and make sure the changes are reflected in the page.
[ ] Go into the admin interface and add and remove a related link from a profile and make sure the changes are reflected in the page.

can't test this for projects yet since they're not migrated to wagtail.

can't test this for projects yet since they're not migrated to wagtail.

Under Projects, I tried adding links to Philly Wireless Project and Indigenous DH Working Group and "received the chicken"
Under People, I successfully added website links to both non-staff (Ipsita Dey) and staff (Sierra) members.

The link to Philly Wireless Project now works.

The Working Groups still do not show up.

Ipsita Dey's email address does not appear on her page of the test site like it does in the current production site. Also, when I add a website link to Ipsita, I am not directed there when I click on her on the list page; instead her website appears to the left of her photo.

working groups issue should be tested/fixed on #280.
@manawinters can you test and see if Ipsita's email appears now? if so, this is ready to close.


[Bug] Curve TWAP appears to have failed. No errors.

It appears that another TWAP from user failed:

https://etherscan.io/tx/0x9756d93e33d83c58efc11d5c48cd669de40647547ef2ee4d8218d39cdcb0ec83  this bot shows up in Discord as:
a.  — 05/14/2024 1:08 PM
New Bot! - Bot Name: juice, Token Input: eth, Amount Input: 525.89, Token Output: crvusd, Amount Output: 2475.46, Chain Name: ETH
b. — 05/14/2024 4:02 PM
New Bot! - Bot Name: squeeze, Token Input: eth, Amount Input: 525.17, Token Output: crvusd, Amount Output: 4947.84, Chain Name: ETH

Problem: It appears that the TWAPs failed and that there are 53 trades remaining with an interval.
Question: where is the logs of the failures?

From @wc117

The user created bot but the TWAP count was 55. So he requested 55 times TWAP. I think there is some bug in calculating the twap count. His bot is 11.65ETH worth but there is no reason to divide it 55 TWAP.
However, his TWAP didn't work. That means there is a bug in backend service, too.

from @rootedbox

May 21 14:31:08 bots-server start.sh[189172]: executeWithdraw(withdraws) []
May 21 14:31:10 bots-server start.sh[189172]: processWithdrawEvents(withdrawn_events)
May 21 14:31:11 bots-server start.sh[189172]: getNextBotArgs {"reason":"unknown account #0","code":"UNSUPPORTED_OPERATION","operation":"getAddress"}
May 21 14:31:11 bots-server start.sh[189172]: {"reason":"unknown account #0","code":"UNSUPPORTED_OPERATION","operation":"getAddress"}
May 21 14:31:11 bots-server start.sh[189172]: cycleDeposits [{"id":830,"deposit_id":0,"token0":"0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2","token1":"0xf939e0a03fb07f59a7331>
May 21 14:31:11 bots-server start.sh[189172]: Health updated successfully.
May 21 14:31:11 bots-server start.sh[189172]: id health health_threshold:  830 4549.993063435179 200

Recovered users funds. Now superseded by https://github.com/VolumeFi/paloma/issues/1601


Fix #2345: Upgrade Jackson dep to 2.14.1 (from 2.13.4)

What this PR does:
Updates Jackson dependency to 2.14.1 (from 2.13.4); forces use of newer SnakeYAML (1.33 from 1.32)
Which issue(s) this PR fixes:
Fixes #2345
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CLA Signed: DataStax CLA

@maheshrajamani This would resolve same CVE that base image has (wrt JDK 17 base), but for our API dependencies.


Rewrote todos app(examples/todos) to maximize unit testability.

This is PR for discussion in #1458. Please have a look the changes under src/components and test/components, ignoring other parts for now.

Hi!
I appreciate the work you put into this but ultimately I don’t think this is something we want in the official examples. Our position is that testing functions is easier than testing classes. Factories also add quite a bit of unnecessary, in my opinion, indirection, whereas shallow rendering lets you abstract away implementation details of children in what I find to be a more satisfactory way without diverging from React conventions.
That said, I’m glad you found something that works better for you, and I’m happy to discuss specific problems you have with our current suggested approaches in #1458.
Thanks!

@gaearon , I have made a new PR #1488, in which I have removed all unrelated parts. The topic I focused on here was to show a potential better way to make an app more unit testable, which is a known way in OOP world. The approach, which was already explained in the beginning, is a general idea, which can be applied regardless of the use of classes or functions.
Now that we have a comparable PR(#1471), let me try to point out the key different between them below:
a. Component creation is top down tree structure. Child components are created by parents where required in original approach.
Whereas, in the approach I propose the component creation is aninverted tree. There are only two places where component creation happens, which are application entry and factories. Application entry consists of three top level containers(they have to stay at very top) and their role is composing or assembling the app. And a function, displayTodo, plays factory role for infinite instance components, regardless its form, a function or a class.
b. #1471 has to depend on shallow rendering to accomplish kind of unit testing. The other approach does not.
Let me try to point out key consequences and observations with the two approach:
a. Unit test, or a better name Focused test, is an isolated test against a component or object under test. It is to test whether a component behaves correctly assuming all its collaborators are correct.
In the original top-down approach, as components are created directly in place, there is no way to mock out. In other words, it mixes component creation logic and business/ui logic.
b.The need of a special tool shallow rendering for testing may be a hint that components are not written in a test-friendly way.Take TodoList.spec.js for example, it will become an integration test without shallow rendering, as it contains children or collaborator code.
Depending on how well shallow rendering is, such tests may have more reasons, than it should be, to fail.
Having said that, I feel this may be better to post to react, which I will do.
Cheers, Cheng


change error wrapping fmt.Errorf -> xerrors.Errorf

now logger when prints errors does not print  a stacktrace.
fmt.Errorf does not works with trace properly ("%v", "$+v", "%w" are not fit)
The offer: change fmt.Errorf to  xerrors.Errorf and logging with "%+v"


Set horizontal padding

Hi
Is there a way to set horizontal padding?  It's too big even with yLabels.

+1

+1
Having the same issue.

I am facing the same

I have same problem but with linear chart, in the example below the x-labels are so much disaligned

facing the same problem

Yes I do want to remove that padding on the left.

Any update?

+1
Having the same issue.

Hey! I got a solution for this.
<Linechart
// ..some props
withHorizontalLabels={false}
style={{ paddingRight: 0 }}
/>

I think that area you want to remove is like 30, I've made a LineChart width 300 and added paddingLeft: 0 and paddingRight:30 to the style of the LineChart, then wrapped the LineChart into a view with marginLeft: 30, alignItems: 'center'.


Added rotation lambda for external API keys.

Issue #, if available:
Description of changes:
Added another example of rotation lambda for external API keys. Wanting to publish here so I can reference this in my upcoming blog post. Already past bar raiser review just needing to host my code here as it's 374 lines.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for the pull request. We will discuss this as a potential feature enhancement.

Any update on this?


input_pill: Identify pills by DOM nodes, remove random IDs

Merged, thanks for the cleanup, thanks @andersk!


Fix bug where events service tries to unsubscribe to non-existent callbacks

undefined error being thrown by events service due to a component unsubscribing from an event before it has subscribed. This can be rectified by simply doing a check on the callbacks to check whether they're undefined.

Merged into develop.


Quilt Compatibility

The latest 1.18.2 release utilises net.fabricmc.fabric.impl.biome.TheEndBiomeData, which is not available on Quilt. This leads to an incompatibility.

Please provide logs.

Please provide logs.

crash-2022-10-02_13.29.12-client.txt


Excluding non-interactive login shells from being parsed from /etc/pa…

…sswd
Overall Review of Changes:
Excludes non interactive shells listed in /etc/shells from being registered in the rhel7stig_passwd_file_audit variable
Issue Fixes:
https://github.com/ansible-lockdown/RHEL7-STIG/issues/462
Enhancements:
N/A
How has this been tested?:
tested code in separate task, results were as expected

I forgot to use a feature branch goign to close this one, sorry if this isn't very tidy


Not picking up on right device SwapChainPanel

0:052> .scriptload C:\Users\a\Downloads\D3DDred.js
JavaScript script successfully loaded from 'C:\Users\a\Downloads\D3DDred.js'
0:052> !d3ddred
@$d3ddred()                 : [object Object] [Type: D3D12_DEVICE_REMOVED_EXTENDED_DATA2]
    [<Raw View>]     [Type: D3D12_DEVICE_REMOVED_EXTENDED_DATA2]
    DeviceRemovedReason : Unknown exception [at D3DDred (line 248 col 13)]
    AutoBreadcrumbNodes : Count: 0
    PageFaultVA      : 0x0
    ExistingAllocations : Count: 0
    RecentFreedAllocations : Count: 0

I'm using a SwapChainPanel, so I think it's not picking up on the right Direct3D12 device. As can be seen by the

DeviceRemovedReason : Unknown exception [at D3DDred (line 248 col 13)

Anyway, this has been frustrating. I'm just going to write a printer for D3D12_DRED_AUTO_BREADCRUMBS_OUTPUT1. Perhaps you should supporting an official printer.

I managed to get something working by sticking the linked list from GetAutoBreadcrumbsOutput1 in a vector, then copy pasting the commands in this repos README (replacing with the pointer values with those in the vector)


error in scenario loading

Hi, @nsubiron
I have tried run the scenario_runner.py script, the command is:
python scenario_runner.py --scenario FollowLeadingVehicle.
The output is
Configuration for scenario FollowLeadingVehicle cannot be found!
After jumped into your code, I found that the function named "find_scenario_config" in file "./srunner/tools/config_parser.py" may have some errors. It parses the xml file to some attributes and compares them with the input scenario's name. But the attributes are named like "FollowLeadingVehicle_5", "FollowLeadingVehicle_6", etc.. No such an attribute named "FollowLeadingVehicle".
Should I input with other arguments to specify the scenario?

I switch the branch carla_challenge into master branch, the problem has gone.
It seems that the xml files is in a wrong form on the carla_challenge branch.

But I always meets an timeout error. The log is:
Preparing scenario: FollowLeadingVehicle ScenarioManager: Running scenario FollowVehicle timeout Failure! No more scenarios .... Exitin

I see. After the log of ScenarioManager: Running scenario FollowVehicle, must I add a ego car to following the car.
Thanks a lot to create the scenario module, it may helps a lot.

Hello @Johnnyswong  I am facing the same problem of:-
Preparing scenario: FollowLeadingVehicle
ScenarioManager: Running scenario FollowVehicle
Failure!
No more scenarios .... Exitin
I am not able to resolve this problem from your answer, Will you please elaborate a little more?
Thanks!!

Hi @ravishk1 After the log:
Preparing scenario: FollowLeadingVehicle
ScenarioManager: Running scenario FollowVehicle
You must run an agent to connect the scenario sever in 10 seconds, otherwise it will cause a timeout. It seems that the default timeout setting is 10 seconds.
Try python manual_control.py after the log printed.

After I checkout the carla_challenge branch into the master branch, the problem has gone.
It seems that the xml file is in a wrong form on the carla_challenge branch.

Hi @ Thanks for your post and I met the same issue. However, switching to master branch does not solve my problem. Do u have any other tips? I am using CARLA0.9.5, Python3.5 on Ubuntu 18.04

After I checkout the carla_challenge branch into the master branch, the problem has gone.
It seems that the xml file is in a wrong form on the carla_challenge branch.

Hi! Thanks for your post and I met the same issue. However, switching to master branch does not solve my problem. Do u have any other tips? I am using CARLA0.9.5, Python3.5 on Ubuntu 18.04

Well, I figure it out by changing in config_parser.py around Line 220
# if set_attrib(scenario, 'name', None) == scenario_name:
if scenario_name in set_attrib(scenario, 'name', None):

After I checkout the carla_challenge branch into the master branch, the problem has gone.
It seems that the xml file is in a wrong form on the carla_challenge branch.

Hi! Thanks for your post and I met the same issue. However, switching to master branch does not solve my problem. Do u have any other tips? I am using CARLA0.9.5, Python3.5 on Ubuntu 18.04

Well, I figure it out by changing in config_parser.py around Line 220
# if set_attrib(scenario, 'name', None) == scenario_name:
if scenario_name in set_attrib(scenario, 'name', None):

same for Line 186 in config_parser.py
# if new_config.name == scenario_name:
if scenario_name in new_config.name:

I think they have pushed the wrong code to the carla_challenge branch and the master branch....


Revise footer

Codecov Report

Merging #774 (01dcad9) into develop (497ee7b) will decrease coverage by 10.91%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #774       +/-   ##
============================================
- Coverage    97.46%   86.54%   -10.92%
============================================
  Files          233       12      -221
  Lines        12545      446    -12099
  Branches        66       66
============================================
- Hits         12227      386    -11841
+ Misses         318       60      -258


Ship module furniture not disabled on link

In the Taiyo and HopeTech Captain's Cabin modules the single seat couch (next to the lavatory) does not disable when connecting the module by the fore connection point causing significant clipping.

Details
Cells: 003D05564 and 000165DC
Object Reference: Taiyo [REFR:003D0671] and Hopetech [REFR:0004E1EA] - both [FURN:0002F42D]
To Fix this I added a new Linked Ref to both the object references with the keyword: (SnapBehaviorDisableLinkedRef01Fore [KYWD:002BBDC2])
For Taiyo I linked it to: [REFR:003D05C7]
For HopeTech I linked it to: [REFR:00002227]
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Install the Taiyo and HopeTech Captain's Cabin ship modules with the connection point at the fore of the module. The single seat couch (next to the lavatory) will not disable and will clip significantly with the door when it opens.

UPDATE
This error is also present in the following cells:
PackInShipPISMODHabTaiyoCptQuarters2L1W1HTopBIntStorageCell [CELL:000165D3]
PackInShipPISMODHabTaiyoCptQuarters2L1W1HTopAIntStorageCell [CELL:000165D5]
PackInShipPISMODHabTaiyoCptQuarters2L1W1HBtmBIntStorageCell [CELL:000165D7]

EDIT
The reference to the Hopetech Captain's Cabin module is incorrect. It actually refers to: PackInShipPISMODHabTaiyoCptQuarters2L1W1HBtmAIntStorageCell [CELL:000165DC]


fix(common): type conversion from str for a custom resource

Description of change
We're missing the Custom resource type conversion from str.
How has this been tested? (if applicable)
Reproduced when I deployed the custom-resource/pdo example.
TODO: test the project deployment again after the fix.

I was surprised to see the error too. Not sure what we've fixed the last time, or if we introduced a regression in the meantime. I haven't digged into it, but it might not be worth doing it.


[Enhancement?] Allow SFOS:Chum GUI app to ignore vendor changes when updating packages

@nephros discovered that packagekitd ignores a vendor change if the method install is called for updating packages (instead of the method update).  See that aforelinked thread for rationale and details.
This is easily done for a single-package update by changing update to install in line 398 of src/chum.cpp, but I do not know if this is the best way to implement this change and how to adapt that for updateAllPackages.
I would appreciate much, if @rinigus and / or @piggz retraces the rationale and replies here, if this consideration is sound in your opinion.
Implementing this change should be a minor task for any experienced C++ programmer, when we reached consensus that this change constitutes an enhancement.

Vendor was added to allow users to choose whether they use Chum or some other repository for particular package.
Let's say you prefer Chum for package A that is also available in Jolla Store and OpenRepos. Without vendor, you would be on mercy of version-release differences between streams. For example, Jolla Store could have the same version but with larger release.
Same goes for opposite - I may prefer OpenRepos for some package to Chum.
So, vendor allows to pin the source of the package - as it is expected. While it causes issues for those who want to jump from one source to another, it prevents the issues that we will get by changing the source of the package automatically.
I consider it a feature and the needed one.

I consider it a feature and the needed one.

This is definitely the case for OpenRepos (and hence Storeman) as I documented recently.
But the assessment is different for the SailfishOS:Chum repository, hence I wrote this suggestion above.
Actually there seems to be some misunderstandings in details:

"Without vendor, you would be on mercy of version-release differences between streams."

This suggestion is not about eliminating vendor tag, which also is impossible for other "streams" (repositories), e.g. the Jolla Store.
Merely evaluating the version-release differences for a package in SailfishOS:Chum (compared to the same package in other repositories) might be wanted behaviour for some.

"For example, Jolla Store could have the same version but with larger release."
For this case the change suggested here would make no difference at all, because the SailfishOS:Chum GUI app does not access the Jolla Store rsp. its repository.
"So, vendor allows to pin the source of the package - as it is expected."
Though the first half of this statement is absolutely true, it is not what some people expect: They want the newest version, wherever it comes from.
"While it causes issues for those who want to jump from one source to another, it prevents the issues that we will get by changing the source of the package automatically."
Which issues do you mean to address here?  This is exactly the reason why my assessment for SailfishOS:Chum is different than for OpenRepos:

"System package"-superseding packages are not allowed in SailfishOS:Chum.  Such packages are then principal reason why ignoring the vendor tag when updating with packages from OpenRepos is a "pretty bad idea"™.
All packages in SailfishOS:Chum are built in a fully traceable manner from their source, vastly reducing the risk of malicious software.
All packages in SailfishOS:Chum undergo manual approval twice: The first time fundamentally, weh a package is accepted into SailfishOS:Chum:Testing, and the each time when a package is submitted to SailfishOS:Chum proper.

"I consider it a feature and the needed one."
It definitely is a feature, but not a required one: Before the introduction of "vendor stickiness" about a decade ago, Linux distributions worked well without it.

After some pondering, I do concur that ignoring the vendor tag when the SailfishOS:Chum GUI app updates packages (implicitly only with ones in the SailfishOS:Chum repository) should be optional, because the current behaviour is well established and desirable for many.

I would back the view that the use of vendor stickiness is useful, and designed to cause less issues when updating.  If it was made optional, perhaps there should be a dialog/info of packages that would change vendor?  It may also be the case that users dont appreciate the concept of a vendor change, so perhaps a description or link out to some docs would be useful?

For example, Jolla Store could have the same version but with larger release.

For this case the change suggested here would make no difference at all, because the SailfishOS:Chum GUI app does not access the Jolla Store rsp. its repository.

Let me explain then on example: you can have Pure Maps from Chum and Jolla Store. They would have the same version but different release. Let's say Chum one has 3.2.0-1.jolla.1234 and Store has 3.2.0-2. In the store, release was set to 2 for one or another reason. When users will update some package at Chum (not necessarily Pure Maps, but anything else), pkcon in its great wisdom will update all packages it can (we had discussion regarding it already). And, if without vendor tag, Pure Maps will be "updated" to Jolla Store version.
That's a situation, vendor is designed to protect against and it is used by us accordingly.

They want the newest version, wherever it comes from.

No, not really. Pure Maps in Jolla Store is significantly degraded when compared to Chum version. If user has selected a channel, that user should expect that the software updates would come from that channel as well.
Then again, there could be software that is distributed via Jolla Store and preferred by the user. Through vendor tag, we give a choice to the user and respect it.
I hope that my examples illustrated usefulness of vendor tag.

I hope that my examples illustrated usefulness of vendor tag.

Yes, they do.  I missed to take issue #67 (originally reported as Storeman issue #96 into account.
Sorry to have bothered you, sometimes I loose oversight of what I know.
⇒ Closing.


 Also a bit of clean up on the users index page.
Staged at https://www.openmicroscopy.org/site/support/omero5.3-staging/users/index.html and https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/index.html and various pages linked.

If PostgreSQL version 9.4 or later is required then also update sysadmins/version-requirements.txt?

We have 9.3 marked as deprecated already @mtbc it was the postgres page that hadn't been updated not version-requirements

Surely "9.3 deprecated, 9.4 recommended" is inconsistent with "9.4 required"? I may simply be just completely misunderstanding what's going on here though!

We already had this state of affairs after @jburel 's update the postgres page for 5.2.3, I can amend the wording to 9.4 is recommended if you prefer though, to my knowledge, 9.3 isn't actually broken.

If anything I vote for dropping 9.3

@hflynn: would much prefer "recommended", thanks, unless @manics manages to get 9.3 dropped!

Happy to make 9.3 dropped on the requirements page instead but that's not a docs decision - I'm adding a comment to the postgres card on the requirements scoping list -https://trello.com/c/A4nhbaE8/8-have-sql-upgrade-scripts-require-pg-9-3

technically it works with pg 9.3. This is the default version installed on ubuntu 14.04
As @hflynn pointed out during 5.2.3, we have adjusted omero-install and the generated walkthrough so they, at least, match what was mentioned as recommended/dropped etc. (omero-install scripts only install 9.4 and 9.5)
The page is confusing for many people (topic for another day)
The changes made in this PR look fine

One last commit to make clear that Java 7 is required but Postgres 9.4 is only recommended @mtbc

Thank you. :smiley:

Staging rebuilt and everything green, going to merge tomorrow to get this out of the way unless there are any vetoes by then.

No vetoes received, merging. We will no doubt be revisiting version issues before 5.3.0 anyway, I still need to implement splitting up the page and simplifying it.

--rebased-to #1504


Translate Code-Splitting

Thanks for the opportunity.
BTW, I cannot find the nice translation for code-splitting so I leave it as it is. I've tried pemecahan kode and pembagian kode but it didn't felt right. Finally left it as code-splitting also thinking that should someone wants to dive deeper, using the term code-splitting as a search keyword will make the search easier.
Please comment if you have suggestion :bowing_man:

Terimakasih untuk reviewnya.
Updated, minta tolong bantuannya untuk dicek ulang @resir014


⚠ Secret Website 3 has degraded performance

In e113534, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 4346 ms

Resolved: Secret Website 3 performance has improved in 7019f5e.


RT-2.2 open sourcing

Co-authored-by: Dan Lepage<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.178%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
651

Relevant Lines:
1047

💛  - Coveralls

Few of the functions in Assert and Session are now failing staticcheck.

Yes, I plan to follow up with a PR after this to clean up.


Add page to explain the benefits of prototyping in code

Think it might be nice to have a page which outlines some of the benefits of prototyping in code with the NHS prototype kit (as well as some of the downsides).

I think the content should be written, however it may actually be best in the service manual? It could be that we write this with an expectation that it moves, but if the Service Manual and Standard had any guidance about why this is important it could be a really useful lever. One that needs to be sense checked with the Service Manual team anyway

Good point and worth checking with the team.
I think it’d be ok for it to be part of the Prototype Kit website for now (and contain some content specific to it), and be linked to from service manual. But it could move in future.
This GOVUK page is a good inspiration: https://prototype-kit.service.gov.uk/docs/prototyping

… a page that I did not know about until today! 🙈 (I now realise it's way down the home page of the GOV.UK prototype kit). So that may at least mean thinking about how it's found in wherever it ends up living.

Another reference is how DfE mentions this in their design manual https://design.education.gov.uk/design-system/prototype-kit

Realistic
Create a service that looks and behaves like the real thing, using components and styles from the NHS.UK website.
Accessible
Create or test designs with people who use assistive technology (for example, screen reader users).
Free and open source
Use common tools like Git and Github for collaboration and project history.


Bump pyodide dev version and allow peer installs up to current latest

Prior to this change, the langchain package's peer dependency on pyodide was restricted to ^0.24.1. Because this is an unstable release, this causes peer dependency conflict warnings in user code that depends on the latest version (0.26.2).
This change updates the range in the peer dependency to allow peer versions from the previously-defined version until the current latest minor version (allowing for patch updates), and sets the dev dependency version to the caretted latest version, to ensure that tests still pass with this version.
I also audited the Pyodide Changelog for breaking changes that might affect the PythonInterpreterTool (the only direct use of pyodide that I can find). While there weren't any logic changes that would break the direct use of pyodide in LangChain code, there was one breaking change to the input params object of the loadPyodide function in 0.25.0.
With respect to whether these changes cause any breakages in the LangChain API, I think that's a bit open to interpretation, and it would depend on how you classify breaking changes to experimental tools, and whether you regard interface structures that are "forwarded" from a peer dependency to be part of the LangChain public API.
I mention this because PythonInterpreterToolParams is defined as a type intersection with the loadPyodide params object, meaning that breaking changes to that type do effectively cause breaking changes to PythonInterpreterToolParams. That said, even setting aside that PythonInterperterTool is an experimental tool and this likely doesn't matter for that reason, I'd argue that by setting the version range to continue to allow versions that matched the old range, these changes aren't actually breaking, as the consumer of LangChain can decide whether or not they want a version of pyodide from before or after these breakages.
For good measure I also checked comments for links to pyodide docs (e.g. in examples/src/tools/pyinterpreter.ts) and those are still valid and live.
Finally, sorry for closing & reopening this - my original PR & branch mentioned "pydantic" in a bunch of places when I meant "pyodiode." 😊

Thank you very much!
I'll merge this for now, we could also just use a tilde if we think the maintainers won't make any heavily breaking changes until 1.0


sidebar

http://nimb.ws/pdLeK5

I think he ate it.

try cleanup localStorage

pleas test on a different screen resolution. 1060 and less.

针对手机模式 暂不进行支持，等待之后版本。（以后的 issue 请注明环境） 也欢迎 PR

closed in #10 https://github.com/sendya/ant-design-pro-vue/commit/2ddf9e9d839c63cdee54d8015ce88fa6533212c5


Due date extension nam

I was in a hurry to get this done. So, I coded the backend. It's not possible for the instructor to allocate extension days through the backend. Requesting by the student is not possible through the frontend though at the moment.
So, this fixes half of #9

If this is ok, let's merge and then I can comment on what I think needs to be done on the student's end.

@recluze Bilal is working on this... he is currently testing his code.

No problem. He can continue working on the front-end part. You two review this code to just to see if you can find a problem.

He has completed back-end and front-end part... he is pushing the code now.
However, your code looks fine to me... but let see if @BilalZaib have something for us.

Sure. If he has completed and it works fine, it should be ok to merge that.


Fix including xsd from another xsd while using inline xmlns

@e6c31d plz get the build to pass

@jsdevel I don't think the build failure is related to this PR. It builds successfully for me on Node.js 14 and 16, but not on 18. The build also fails on the master branch on Node.js 18.

@jsdevel I created another PR to fix the build on Node.js 18: #1206

Hi @e6c31d,
I rebased you branch to check if it still works, I recently released new version with some major changes.
I will merge it and include into next release.


Contributors/justin p barnett/unity project

Added new Unity project with settings, folder structure, and XR packages as a starting point.

 Add new Unity project to repo


GRUB - missing link for reference

"Why it’s sda and hda ? refer here for more." - does not point to any link for reference, add a refer link or remove the statement instead.

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

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

Thanks for contributing to the freeCodeCamp repository. Congratulations on your first PR! 🎊
Look forward to more contributions from you! ✈


GSK-857 Create a better error message for when a user keeps the target in the feature_names

Issue Type
Performance
Source
source
Giskard Library Version
N/R
Giskard Server Version
N/R
OS Platform and Distribution
No response
Python version
No response
Installed python packages
No response
Current Behaviour?
f"The following columns are not found in the dataset: {', '.join(sorted(column_names))}"
E   ValueError: The following columns are not found in the dataset: target

Standalone code OR list down the steps to reproduce the issue
put the target in the feature_names list

Relevant log output
No response

fixed


3.8 Algorithm development and pilot use case testing for search on the KG

Corresponds with https://github.com/nih-cfde/IDG/issues/48.
Progress on this issue should be logged via comments, including specific tasks, and related deliverables including code and data.

I've created a local neo4j instance using the dump file shared via Chiltepin.  Currently exploring the content of this KG and learning basic of cypher


Feature | Update mysql_db module to use terraform-aws-rds latest version (v4.2.0)

What?

Updating Binbash MySQL_DB module to use AWS RDS Terraform module latest version (v4.2.0)
(Terraform module which creates RDS resources on AWS.)

How?

le-tf-infra-aws/apps-devstg/us-east-1/databases-mysql --/db_mysql.tf
source version = "github.com/binbashar/terraform-aws-rds.git?ref=v3.3.0"
target version = "github.com/binbashar/terraform-aws-rds.git?ref=v4.2.0"

Why?

Keeping Leverage Reference Architecture up to date.

@rodriguez-matias Took care of this upgrade. It was released at https://github.com/binbashar/le-tf-infra-aws/releases/tag/v1.7.2
CC: @binbashar/leverage-ref-architecture-aws-dev @binbashar/leverage-ref-architecture-aws-admin


Iceberg writes for map columns with fill_missing_columns_in_df fails due to type mapping error

Describe the bug
I'm not entirely sure if this is a bug or an incorrect usage pattern, but from a brief scan of the source code it looks like a legitimate bug.
Scenario: An iceberg table is create with custom dtypes to have a map type column (map<string, string>)
When: You attempt to write new data to the table using a dataframe that is missing the map type column, and:

mode=overwrite_partitions (and possibly append too, haven't tested, but looks like that would have the same problem)
fill_missing_columns_in_df=True
schema_evolution=True

Then: The call to to_iceberg write will fail with:
Unsupported Athena type: map<string,string>
--> 419     df[col_name] = df[col_name].astype(_data_types.athena2pandas(col_type))
    421 schema_differences["missing_columns"] = {}

awswrangler/_data_types.py:382, in athena2pandas(dtype, dtype_backend)
    380 if dtype == "geometry":
    381     return "string"
--> 382 raise exceptions.UnsupportedType(f"Unsupported Athena type: {dtype}")

This appears to be because the athena2pandas method used to fill the missing column on the DF is looking for the type to be exactly 'map' and won't handle a type like 'map<string, string>'
https://github.com/aws/aws-sdk-pandas/blob/cb63a18d17f2b16731955facdb67c3340f766d49/awswrangler/_data_types.py#L378
    if dtype in ("array", "row", "map"):
        return "object"

How to Reproduce
# Example DataFrame creation
data_with_map_col = {
    'partition': [1, 1, 2, 2],
    'column2': ['A', 'B', 'C', 'D'],
    'map_col': [{'s':'d'}, {'s':'h'}, {'i':'l'}, {}]
}

data_missing_map_col = {
    'partition': [2, 2],
    'column2': ['Z', 'X'],
}

df_with_col = pd.DataFrame(data_with_map_col)
df_missing_col = pd.DataFrame(data_missing_map_col)

glue_dtypes: dict[str, str] = {
    "partition": "int",
    "column2": "string",
    "map_col": "map<string, string>",
}

write_args = {
    "dtype": glue_dtypes,
    "partition_cols": ["partition"],
    "mode": "overwrite_partitions",
    "database": "data_warehouse",
    "table": "bug_demo",
    "table_location": deps_container.config.data_warehouse_s3_path(),
    "temp_path": deps_container.config.data_warehouse_s3_path() + '_tmp',
    "boto3_session": boto3_session,
    "schema_evolution": True,
    "fill_missing_columns_in_df": True,
    "keep_files": False,
}

# Write the dataframe with the map col
wr.athena.to_iceberg(**{**write_args, "df": df_with_col})

# FAILS: Write the dataframe with the missing col
wr.athena.to_iceberg(**{**write_args, "df": df_missing_col})

Expected behavior
The missing column in the dataframe should be properly filled with the appropriate NA type and the write should succeed
Your project
No response
Screenshots
No response
OS
Mac
Python version
3.11
AWS SDK for pandas version
3.7.1
Additional context
No response

Thanks @LeonLuttenberger!


Module LightsOff

I have a contribution to Omri Cohen's Modular Fungi collection, the module is called "Lights Off". Enjoy!

I use the gitflow workflow methodology.   So I would branch off develop to create a feature branch, work in there. When it's stable I would pull it into the develop branch.
Finally I would pull develop into master ONLY when I was ready for a new release.

Thanks for the advice. I have to organize my development better in this regard!


Update information about C# 7 packages and ValueTuples

I believe there are currently no released versions of .Net Framework/Core/Standard or VS which would include the packages by default, so I think it doesn't make sense to mention any versions in these notes.
cc: @BillWagner

I'd like to get @richlander 's opinion on this set of changes.
.NET Framework 4.7 just shipped, which (I think) has ValueTask and ValueTuple. I haven't verified yet, because my Win10 machine hasn't gotten the update yet.
How would you want to phrase this now?

I am looking at this topic for the first time, so will give feedback above and beyond the PR.

[!NOTE] Tuples were available before C# 7 as an API, but had many limitations. Most importantly, the members of these tuples were named Item1, Item2 and so on. The language support enables semantic names for the fields of a Tuple.

This feels like an apples to orangutan comparison.  You can compare type to type and language support to no language support. ValueTuple and Tuple both have Item1 and Item2,  so they don't feel different in that regard. If Tuple was a struct, I doubt the team would have created a new type. I think that the two changes are: language support is now available for tuples and is done so through a new type, created for performance reasons. It's possible that there are back-compat reasons, too, but I don't know that. Semantic names is a good description of the language support, however, I think of deconstruction as a separate additional feature.

Tuples were available before C# 7 as an API, but had many limitations.

Do we have a good sense of what the limitations where? It's not useful to make this claim unless they are given. The Item1, Item2 thing wasn't really a limitation as a type, for the reasons given above. Also, I'd call the old Tuple a "type" or "class". That's more descriptive, IMO.

That assignment creates a tuple whose members are Item1 and Item2, following the existing @System.Tuple syntax.

I don't think this is a useful comparison, making specific reference to "Tuple syntax". You could say "which you can use in the same way as System.Tuple". There is no such thing, IMO, as "Tuple syntax", however.

You can modify that assignment to create a tuple that provides semantic names to each of the members of the tuple:

I don't think of this as "modifying the assignment" but more about changing the C# syntax to take advantage of a language feature.

The new tuples features require the System.ValueTuple type.

This shows up in a odd spot in the topic. It should show up as the first bit of content.
To the question that @BillWagner raised, I'd suggest wording like the following:

You need the TypeA type in order to use Language Feature B. It is included in .NET Framework x.y. When targeting earlier .NET Framework versions, you need to install NuGet Package C.

I like the idea, which was suggested by @GuardRex, to focus the text on the type. I think people will appreciate that

(string Alpha, string Beta) namedLetters = ("a", "b");

I get the use of "named" in "namedLetters". That said, some people might think this is guidance. We don't actually suggest that people start naming variables this way. I'd just keep with "letters".

The namedLetters tuple contains fields referred to as Alpha and Beta.

This is very subtle. I think we need to explain that the field names are convenience names using some wording, as opposed to actual field names. I'd suggest "virtual name" but I think "virtual" already has a well established meaning. We might also need to explain that they are fields and not properties.

The language allows you to specify names for the fields on both the left and right-hand side of the assignment:

I think you can more simply say: "You can specify names ..."

@svick Rich's comments go beyond what you've submitted here. I've created issue #1889 to capture those comments. Let us know if you want to expand the scope of what you're doing here. If not, we'll take this PR once you've updated it per the other feedback and we'll leave the other issue open.

@BillWagner I have addressed most of the comments.
@richlander Can you review if the changes are what you requested? Some comments follow.

I think of deconstruction as a separate additional feature

How should that be reflected in the text? Especially considering that deconstruction is not really specific to ValueTuple.

Also, I'd call the old Tuple a "type" or "class".

It's not just a single type, and I think "tuple types" would be confusing, so I left it as "tuples".

I'd suggest wording like the following:

You need the TypeA type in order to use Language Feature B. It is included in .NET Framework x.y. When targeting earlier .NET Framework versions, you need to install NuGet Package C.

How would that work for .Net Core and .Net Standard, when they include ValueTuple in the future? Should it say something like "It is included in .NET Framework 4.7, .Net Core 2.0 and .Net Standard 2.0"? And even now, are people going to understand that .Net Core 1.x and .Net Standard 1.x are "earlier .NET Framework versions"?
Because of that, I made it vague:

You must add the NuGet package System.ValueTuple in order to use it on platforms that do not include the types.

I get the use of "named" in "namedLetters". That said, some people might think this is guidance. We don't actually suggest that people start naming variables this way. I'd just keep with "letters".

I think it's useful to clearly differentiate letters from the first snippet with namedLetters from the second snippet, so I think the names should stay different.

I think we need to explain that the field names are convenience names using some wording, as opposed to actual field names.

I think the main reason why people should know this is that the names are not really preserved at runtime, so that's how I explained it.

We might also need to explain that they are fields and not properties.

Why would this need a further explanation? It does say they are fields, and I don't think that's something that would be confusing.

Wow.  Thanks for all the work @svick
I reviewed the changes and I'd be happy to merge.   @richlander What's your opinion?

Inspired by https://github.com/dotnet/docs/pull/1962, I have also added links to the packages on nuget.org to the notes.

Rebased on top of current master due to conflict caused by #1962.

is this ready to merge @GuardRex and @richlander?

Merging now. This is a major improvement, and already addresses some of the comments now in #1889. Once merged, we can examine those that still need work.


IS there a plan to support Astra Pro+ used in ROS2 Humble?

how I can make Astra Pro+ works under ROS2 Humble now?

We can't??!!!


[WIP] evm:status displays all columns

This is option 3 for fixing this BZ (base ref #18248)
There was some confusion on the best approach.
I coded up all 3
Users were confused that redundant columns were hidden
Now displaying all columns
concern:
Columns Version, Heartbeat (and probably Status and Started) are expected to be the same. Those columns were added because they are useful, but may make the screen too wide
https://bugzilla.redhat.com/show_bug.cgi?id=1651256

#18259 was merged instead


Feature #126: Onboarding improvements

This PR:

Implements #126
Add QR Code icon to assets
Dep bump
Few eslint fixes
Rewrite GnoStepper with hooks (I ran into in issue with react-final-form trying to use it in a certain way but it didn't worked, it used to work in another component written with hooks - thought this is the case. This wasn't the case, but I decided to keep the component like that)
Add mutators prop to GnoStepper
Update tests to match new create safe form
Fix create safe form bug:
It was impossible to remove the second owner, if you remove it and then click 'Add another owner' it would show the inputs with pre-populated values from previously removed second owner. This was because the updateInitialProps(initialValues)(https://github.com/gnosis/safe-react/pull/138/files#diff-0dda9ebbe54f4857bdcd1a0ce78bc037R96) was called with the props equal to the props we initially passed to the form, and if they're equal, react-final form won't recalculate the form values ( https://github.com/final-form/react-final-form/blob/master/src/ReactFinalForm.js#L139). This is why in some places it now calls form.reset(). I think it's pretty hard to explain what happened from technical side, so feel free to contact me for more detailed explanations.

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Weird, after adding a second owner and removing it, it still showed 1 oo 2 confirmations required. Clicking "review" would then show the owner that I just removed. 🤔

@mikheevm can we please make the sentence "Any transaction requires the confirmation of:" non-bold and maybe have some more spacing between that instruction and the dropdown field?

When Uploading a Metamask QR code, it decodes also the prefix "ethereum:". Ideally we automatically detect & delete this.

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com

Travis automatic deployment:
https://pr138--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr138--safereactstorybook.review.gnosisdev.com


Image caption formatting

https://www.servicenow.com/blogs/2022/data-augmentation-intent-classification.html

https://www.servicenow.com/blogs/2022/multi-label-iterated-learning-images.html
vs
https://main--aemeds--servicenow-martech.hlx.live/blogs/2022/multi-label-iterated-learning-images

What I'm thinking at the moment is that for structuring the image caption we could do the following:
If we have text in the same paragraph as the image, we style it as a caption. To have text in the same paragraph and author would need to do:

Add image
Shift + Enter
Add text

If the author does

Add Image
just Enter
Add Text

Then that's a new paragraph and the text looks like a normal paragraph.
WDYT?

@benpeter ok. let's do it like this. but let's make sure we keep in mind the image with link case when testing: https://github.com/hlxsites/servicenow/blob/main/scripts/scripts.js#L362
would you like to take this one?

There are cases where the caption in on the left and where are code is looking correct, so the caption style is not uniform.
https://www.servicenow.com/blogs/2021/why-project-portfolio-management-matters.html
https://main--aemeds--servicenow-martech.hlx.live/blogs/2021/why-project-portfolio-management-matters

Closing for now as multiple blogs use different formating. The one from my comment above has the same formatting that we currently have in EDS. In case the customer would like a different specific one we can reopen and discuss.


ARROW-5967: [Java] DateUtility#timeZoneList is not correct

Related to ARROW-5967.
Now timeZoneList in DateUtility belongs to Joda time.
Since we have replace Joda time with Java time in ARROW-2015, this should also be changed.
TimeStampXXTZVectors have a timezone member which seems not used now and its getObject returns Long(different with that in TimeStampXXVectors which returns LocalDateTime), should it return LocalDateTime with its timezone?
Is it reasonable if we do as follows:

remove Joda timezoneList in DateUtility
add method like getLocalDateTimeFromEpochMilli(long epochMillis, String timezone) in DateUtility
Not sure make TimeStampXXTZVectors return LocalDateTime？

My vote would be to either leave as is or would be to have TimeStampXXTZVectors return ZonedDateTime or something similar.  CC @BryanCutler who I think did the initial migration to get his thoughts.

Yeah, a lot of from DateUtility I was not sure of the usage, so I left most as is. It would be nice to clean it up and (1) and (2) sound reasonable to me. I think changing the vector API in (3) might be better brought up as a separate issue. Could someone from Dremio verify what is being used in this utility and what could be cleaned up? cc @pravindra

@praveenbingo @pravindra Mind take a look and give some comments about TimeStampXXTZVectors API change? thanks

sorry for the delay, @tianchen92 - will get back to you on this by tomorrow.

This patch requires only a minor change in dremio, and I agree this list doesn't belong here. So, I'm fine with 1 and 2.
For 3 - dremio doesn't support the TZ variants yet. If there is no immediate use case, I'd prefer to leave it as it is.

Thanks a lot for your efforts

thanks @tianchen92 for fixing this.


Improve Docker Build and Deployment for Multi-Arch Support (arm64)

Add multi-arch support for Docker images: The previous implementation did not handle multi-arch correctly. This PR updates the Docker build process to support multiple architectures (linux/amd64 and linux/arm64) using the docker/metadata-action and docker/build-push-action.
Tested the changes on an arm machine and verified the build with the GitHub Container Registry.
Fixes: #25
Additional Note: The beeper/docker-retag-push-latest action does not seem to handle multi-arch correctly, which is a separate issue that needs to be addressed.
Verification: Please verify the changes by checking the Docker image on the GitHub Container Registry: https://github.com/slikie/bridge-manager/pkgs/container/bridge-manager

@tulir Sorry to bother you but would you mind merging this pull request? Thank you! 🙏


Why does yabai --load-sa require SIP to be turned off after --install-sa

I had once turned off SIP and ran sudo yabai --install-sa and then sudo yabai --load-sa and booted into recovery mode and enabled SIP why do I still need to have SIP disabled on every boot to have extra features

See https://github.com/koekeishiya/yabai/issues/798#issuecomment-767084471


Simplification of GetHttpMethod and httpMethod in WebCmdlets

PR Summary

Simplification of GetHttpMethod

Simplification of setting httpMethod

Remove ParameterSetName = "StandardMethodNoProxy";"CustomMethodNoProxy"  --> added [Proxy-NoProxy] to ValidateParameters()

(At the moment WebRequestMethod does not contain the CONNECT  Http request method is this ok?)
(After this rewrite the difference between Method and Custom Method seems arbitrary, should we look into uniting them?)

PR Context
Improve readibility of the WebCmdlets

@CarloToso Of course cmdlets with lots of parameters and parameter sets look complicated nevertheless I don't see the benefit of removing these parameter sets.

I am somewhat surprised that parameters are declared as virtual and we override them. We can't make them not virtual without PowerShell Committee approval but we could remove the overrides.

@iSazonov I'll add back the parameters set, is the rest of the code ok?

I propose we change the ParameterSetName in the Proxy and NoProxy region from [StandardMethod, CustomMethod -->Proxy] and [StandardMethodNoProxy, CustomMethodNoProxy --> NoProxy]

I don't see value in the change. (Also it can break scripts working with meta data.)

At the moment WebRequestMethod does not contain the CONNECT Http request method, should we do something about it?

At the moment WebRequestMethod does not contain the CONNECT Http request method, should we do something about it?

Please open new issue with investigation why we should add.

@CarloToso Please update the PR description.


Lagrange basis aurora speedup

Addresses #28.
I believe all feedback has been accounted for, but if not I will be happy to revise.

I think my latest change actually broke lincheck--I'll look into what's going on and commit fixes once I find them.


Focus input is visible

The focus input is visible :

I need to reintegrate the following rule from chosen original css :

to make it work as expected

Thanks for picking this up. I released a new version that should fix the issue.

thanks !


testground/daemon: cluster-autoscaler improvements

ATM, the daemon is starting up the pods and throwing errors during scaling up of the cluster's instances to accommodate the load given to it.
This can result in unstable behaviours like:

Some of the test pods might be evicted from run causing the whole test run to hang
Sync-service throwing context deadline error that cause the network to be never being properly initialised

In order to fix both issues above, it might be beneficial for the daemon to know upfront when the scaling has been complete to start spinning up pods for test run

https://www.notion.so/celestiaorg/DevOps-Berlin-May-June-mini-onsite-da1c4795d16d4bedba741b93e6e41d0c#bf200c5ae55a43a2aba9f954c7b8eb24


Keep connection between activities?

Hi,
straight to example:
i have an Activity A -> this activity is the about selecting a device from devices (in recyclerview) and establish a connection when selected, i have done this part successfully. and then after the device is selected it goes to Activity B where all the main menu is there, but i want to keep the connection between that activities. so my questions is how can i make that happen ?
please help me, i've been struggling to find how to do this.
Thanks before :)
i've post this question in stackoverflow also, this is the link :
https://stackoverflow.com/questions/44353812/android-keep-bluetooth-connection-between-activities

You probably want the Bluetooth actions occurring in a Service, and that Service using broadcasts for various activities, fragments, etc to listen in on.

Hello @baadillahnabil
This is a general android / rxjava architecture question — not an issue or a feature request with the library.
I have seen that you have already posted the same question on stackoverflow (https://stackoverflow.com/questions/44353812/android-keep-bluetooth-connection-between-activities) which is a better place to ask.
Closing.

Hey,
May you please help me with how you fixed the issue?
I mean how you kept connections between activities
I couldn't find any solution for that :(

@9otFound There are many different approaches to that (eg. Android Service, a singleton object) and every one of them boils down to something that's lifecycle is longer than a single Activity.


微调模型和代码发布 chatglm2-lora model released

we actually released our chatglm2 lora model on https://huggingface.co/silk-road/Chat-Haruhi-Fusion_B
the simple version of inference code is in cleaning, but any developer tried glm lora before should be able to run it easily.

其实论文报告里说的挺清楚的-o- 要不再努力看一下

Let me closed the issue


[Bug] Fix reconciler panic on restart

Fixes https://github.com/coinbase/rosetta-sdk-go/issues/67.
Changes
Update to v0.3.2 of the rosetta-sdk-go
Related PRs
https://github.com/coinbase/rosetta-sdk-go/pull/68

Pull Request Test Coverage Report for Build 1597

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.868%

Totals

Change from base Build 1582:
0.0%

Covered Lines:
455

Relevant Lines:
760

💛  - Coveralls


Use `negotiator like library for accept-language and accept-encoding

Currently we are parsing both accept-language and accept-encoding, the parse of accept-encoding is not reliable, so it's better to use a library instead, but as we are using accept-language library in the i18n.js we might want to keep the usage of accept-language when we create a separate package from it. Though meanwhile using the negotiator might be a right choice.

Negotiator implementation requires passing the whole request object, each time it doesn't come with a simple implementation as accept-language, there is another package https://www.npmjs.com/package/accept but says that have an insecure dependency.

Currently accept-language seems to be the best option.

I think having current non 100% reliable parsing should be fine meanwhile. Let's keep current implementation and come back to it, if an issue is noticed.


Added puppeteer exec path as env variable and send run number on event

Partially closes #37
I also added the PUPPETEER_EXEC_PATH as an environment variable when runing the tests on each action

This PR can be closed


fix: migrate to assets-pkg

https://autoricardo.atlassian.net/browse/CAR-8227
generating assets based on our conventions

Thanks, I missed those! Will move them as well and ping you again for review


upgrade go and other dependencies

chore: upgrade go and other dependencies
also regenerating docs and fixed linting
related to jenkins-x/jx#8670

/approve

/hold
secretfacade isn't upgraded to go 1.22


Styled-Nav-Bar-12-16

Description
This is a styling fix to update the app to look more modern. I have re-sttyled the Nav Bar, added a new Logo & Flavicon, and removed the background color.
Type

[ ] This is a bugfix.
[ ] This is a new feature.
[x] This is part of a feature.

Organization

[ ] Have you linked this pull request to a Trello card?
[ ] Have you named the branch in the format of feature/describe_feature_or_change OR bug/describe_bug_this_is_fixing

Review

[x] Is this a work in progress?
[x] Is this ready to be reviewed?
[x] Have you added two reviewers to this pull request?

 Restyle NavBar, Create Flavicon

The AWS preview successfully deployed, after it's initial fail, however I don't know how to get GitHub to reflect that.