column for exposed ports

CTOP is the awesome tool.
It would be very useful to have an optional column for port binds.

Hmm, ctop is very cgroup oriented. That said, one way to do it could be to query each detected container engine about the container it manages. This could also pull engine specific data like exposed port for a docker container.
Something similar was contributed a while back for openvz. Would you be willing to open a pull request for that feature ?


tests: text and path glob path matching

Commit Message: Adds additional test cases for path and text glob path pattern matching.
Additional Description: Specifically verifies that =** or ** must be the last operator in the path pattern match.
Risk Level: Low
Testing: Ran new tests locally.
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features:

Please merge the main branch after #33115 is submitted to fix CI breakage.
/wait

Please merge the main branch to fix CI breakage.
/wait

@yanavlasov Updated!

Please merge past 7fdde72e57fb46feac85899dc131fb1152cfef1f Seems like you did not pick up all the necessary updates.
/wait

Please merge past 7fdde72 Seems like you did not pick up all the necessary updates.
/wait

Gotcha, my bad!


Cross-publish stages each Scala-version to a separate staging repository

When I cross-publish with '+publishSigned' the results are staged in multiple staging repositories (one for each Scala-version). Sometimes it is staged in a single repository. This seems ambiguous.
If I run 'sonatypeRelease' it fails when multi-repositories are found, but putting 'sonatypeReleaseAll' in my travis-build is not desired as there can be staging repositories (other builds, other projects).
I looked in the source code and tried some manual testing. It looks like 'sonatypeStagingRepositoryProfile' keeps a reference to a staging-repository. Is it possible to keep multiple references and have 'sonatypeRelease' promote and close this list of staging-repositories?

I guess https://github.com/xerial/sbt-sonatype/pull/77 is related.

I guess #77 is related.
Not sure, but I apologize I forgot to mention similar issues. E.g. #66 #73 and #49 where is reported about multiple parallel publishSigned commands somtimes publish to the same repository.

@ThijsBroersen I've redesigned the release flow. By using the flow described here https://github.com/xerial/sbt-sonatype#multi-module-projects, it should be possible to use the same staging repository to publish.


Update the URL and API desc

Describe Your Changes

Update the URL and API desc

Fixes Issues

Closes #
Closes #

Self Checklist

[ ] Added relevant comments, esp in complex areas
[x] Updated docs (for bug fixes / features)
[ ] Created issues for follow-up changes or refactoring needed

These chat/completion parameters do not work with the TensorRT-LLM engine.
frequency_penalty
presence_penalty
top_p


VSCode shows eslint error with '@vue/airbnb' when using webpack aliases in import paths

Version
3.0.3

Node and OS info
Node 10.7.0
Steps to reproduce
Vscode will throw error, but app running successful.
What is expected?
no error info
What is actually happening?
throw error info

Please share the repository

Build with default configuration @LinusBorg

Do npm run serve and npm run build work?

yes, running successful.

Then this is a problem with the eslint integration of your editor

The same situation，app can work，but error still exist.

So the error is only in your editor? Then I think your editor's eslint integration doesn't play nice with eslint-airbnb ?

Possibly related: https://github.com/Microsoft/vscode-eslint/issues/405

I use Vscode, I don't know. OK, I try to see it.

1 upgrade  vscode editor
2 upgrade vscode-plugin   ESLint to  1.7
3  ok

Same problem with you! When I open the subdirectory only, everything works well. But if I open the large project, vscode shows no-unresolved error
Thanks for that - spent a good hour on this one trying to figure out why it wasn't resolving.

In case anyone else is still struggling with this, I realized the same thing as others that the issue was caused by my front-end code being in a sub-directory of the main project. To fix, add this to your settings.json (change "client" to the name of your front-end sub-directory):
    "eslint.workingDirectories": [
      { "directory": "./client", "changeProcessCWD": true },
    ],


Added Profile Pic to the TEAM.md file

@proful @pranavgoel29 do review these changes & merge the files accordingly. Preview the changes before approving to see the changes made.

@pranavgoel29 are you OK with these changes?

Merging to avoid additional merge conflicts for @Akshay2996
@pranavgoel29 you can improve on top of it.

@proful me & @pranavgoel29 had an offline call & we figured it out.

@proful me & @pranavgoel29 had an offline call & we figured it out.

I just checked the changes Image is still squashed we have to work on it again.


fix:On updating Batta field Total Daily Batta is updated.

Fix:
Update  Total daily batta field when  Batta field is changed.
Solution description
Total Daily Batta is calculated based on days and Batta  and updated when either one is changed.
Output

Is there any existing behavior change of other features due to this code change?
-No
Was this feature tested on the browsers?

Mozilla Firefox

@ZakkiyathLulu Resolve the conflicts


Incompatible with porting lib

Switching to use ModifyExpressionValue should fix the issue
https://github.com/DaFuqs/AdditionalEntityAttributes/blob/1c6cb03cd3b2a2195f8d9274d885cb721d3362a9/src/main/java/de/dafuqs/additionalentityattributes/mixin/common/PlayerEntityMixin.java#L38

Merged!
Will run some final tests and then release an updated jar.


Assign task- Yellow icon user tapping from link, error shows & it goes missing on revisit

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch app
Tap fab
Tap new chat
Enter an email that doesn't have new dot account yet in this specific way
Type email<EMAIL_ADDRESS>and change letters after plus, until it shows yellow profile icon.
When yellow profile icon it displays, select that user
Navigate back to LHN
Tap fab
Tap assign task
Enter any title and tap next
Tap assignee
Select yellow profile icon user mail id which is created in step 6
Tap confirm task
Tap From link in header
Tao back button of "Hhm it is not here page"

Expected Result:
When user taps "confirm task", he must be shown task page properly displayed without refreshing and tapping on "from" link in header, must be directed to conversation page which should not display any error.when user re-visit task page, the header "from link" must not disappear
Actual Result:
When user taps "confirm task", the task page user lands and it gets refreshed after a second. Then tapping on "from" link in header, user directed to conversation page which shows error message " Hhm it's not here" and tapping on back button, the header " From link" disappears in task page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/f467a23a-f40f-42ba-9786-ab232b23ce9c
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Nothing to do with yellow avatar and probably dupe https://github.com/Expensify/App/issues/24387

Agreed, this looks like a dupe.


mac: Failed to build

Reported by @gunyarakun.
# github.com/go-gl/glfw/v3.2/glfw
In file included from ../../go-gl/glfw/v3.2/glfw/native_darwin.go:7:
In file included from ./glfw/include/GLFW/glfw3native.h:92:
In file included from /System/Library/Frameworks/Cocoa.framework/Headers/Cocoa.h:12:
In file included from /System/Library/Frameworks/Foundation.framework/Headers/Foundation.h:10:
In file included from /System/Library/Frameworks/Foundation.framework/Headers/NSArray.h:5:
/System/Library/Frameworks/Foundation.framework/Headers/NSObject.h:19:21: error: expected a type
- (id)copyWithZone:(nullable NSZone *)zone;
…(snip)...
7117 errors generated.

❯ xcodebuild -version
Xcode 8.0
Build version 8A218a

❯ go version
go version go1.7.3 darwin/amd64

❯ sw_vers -productVersion
10.11.6

❯ system_profiler SPSoftwareDataType
Software:

    System Software Overview:

      System Version: OS X 10.11.6 (15G1108)
      Kernel Version: Darwin 15.6.0
      Boot Volume: Macintosh HD
      Boot Mode: Normal
      Computer Name: taskpersonal
      User Name: Tasuku SUENAGA (tasuku)
      Secure Virtual Memory: Enabled
      System Integrity Protection: Enabled
      Time since boot: 3 days 7:35

Is it possible to test this with Xcode 8.1? Mine is Xcode 8.1; Build version 8B62.

In my environment, clang and llvm-* exists in /usr/local/bin and it may prevent building.
❯ clang -v
clang version 3.7.0 (git@github.com:tomboinc/emscripten-fastcomp-clang.git 893219668a1eea241b9fec6deda6009efb9cdfc9) (git@github.com:tomboinc/emscripten-fastcomp.git 3053a86417e2500bba2657586756fecb084388ac)
Target: x86_64-apple-darwin15.6.0
Thread model: posix

After removing these binaries, go get -u github.com/go-gl/glfw/v3.2/glfw works well.

Awesome! Thank you for reporting!


aix: enable openssl support on JDK8/J9

Apparently we haven't been building JDK8 OpenJ9 builds with the OpenSSL support enabled. This PR resolves that.
Testing at https://ci.adoptopenjdk.net/view/work in progress/job/SXA-aix8j9ssl/2/console
Signed-off-by: Stewart Addison<EMAIL_ADDRESS>

@sxa555 I think this PR may have caused jdk8u-aix-ppc64-openj9 to fail. Is --disable-warnings-as-errors supported for aix?
https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-aix-ppc64-openj9/541/
configure: error: unrecognized options: --disable-warnings-as-errors

FWIW, at the OpenJ9 farm we only use --disable-warnings-as-errors on JDK11+
https://github.com/eclipse/openj9/blob/master/buildenv/jenkins/variables/defaults.yml#L227

https://github.com/AdoptOpenJDK/openjdk-build/pull/1520


Show number of InstantSend locks in Debug Console

Targeted for 14.1 instead of 14.0

Added backport-candidate-14.0.x label as this is useful in mainnet right now


Custom Configurations

Hi,
Do you have any plans to add some custom configuration parameter in Parsedown( Extra)??
Because there are some things that can be customized:

Optional to add dot prefix in fenced code block class name
Optional to put class name in <pre> element
Optional to ommit the language- prefix in code block class name
Optional to use class name instead of inline style for table alignment
Optional to set custom classes in footnotes
Optional to create custom footnotes hash pattern
Predefined URL, titles and abbreviations?

Thanks.

This issue belongs to Extra repository.

https://github.com/erusev/parsedown/wiki/Tutorial:-Create-Extensions#change-element-markup

Endeded up with my own Extension for Parsedown Extra: https://github.com/tovic/parsedown-extra-plugin


The model to consider.
https://huggingface.co/silma-ai/SILMA-9B-Instruct-v1.0
The closest model vllm already supports.
google/gemma-2-9b
What's your difficulty of supporting the model you want?
It is arabic model and may have different way of handling the tokens
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

This model uses Gemma2ForCausalLM architecture without custom model definition, so it can be directly used in vLLM. Have you tried doing so?

Yeah, just now tried it in runpod instance. It works out of the box in v0.6.2. Thanks


Provide a way to move up/down the stack

[X] I have searched the discussions
[X] I have searched the existing issues

Version
0.1.0
Use Case
Looking up my upstream and switching to it can be distracting.  Git finds shortcuts useful enough to have git checkout -.  Providing a way to move HEAD up/down will be a help
Requirements

Move up/down
Moving down handles ambiguities

Possible Solutions
git switch-parent

Implies action through relation to git switch command
"branch" is implied since switching is usually between branches
parent is less ambiguous than upstream

git switch-child

Accept which child on command-line
Prompts when no child selected

Alternative Solutions

depot-tools represents this with special git subcommands, nav-downstream and nav-upstream

Prompts the user on ambiguity

Thanks for the inspiration!
In considering this feature, I'd also love to be able to use this in other commands, like git diff HEAD<..HEAD.  I wish git allowed plugging in additional syntax like that.  A not-very-ergonomic approach is to have a --print argument that says what the prev/next is.  Aliases can make it slightly more ergonomic.

You're probably aware, but for the specific case of getting parent commits, you can use HEAD~ or HEAD^ (https://git-scm.com/docs/gitrevisions). The git-revset command above is nice because it lets you also do children queries.

The difference is I'm not wanting to move by commit but by branch, regardless of how many commits are in it.

You're probably aware, but for the specific case of getting parent commits, you can use HEAD~ or HEAD^ (https://git-scm.com/docs/gitrevisions). The git-revset command above is nice because it lets you also do children queries.

The difference is I'm not wanting to move by commit but by branch, regardless of how many commits are in it.

+1

@searleser97 since you expressed interest in this, could you describe how you see yourself using a command like this and what you would need from it?
I'm looking to understand

Requirements
Level of importance (so far in my git-stack usage, I've not been moving up the stack as much as I'd expected)

@epage I actually just found graphite which is just what I was looking for: https://github.com/screenplaydev/graphite-cli
and the ability to navigate up/down branches is documented here: https://docs.graphite.dev/guides/graphite-cli/navigating-a-stack#moving-up-and-down-a-stack

Oh neat!  When I started git-stack,  I could find very few programs in this space and they seemed pretty esoteric.  Its amazing what you find after you start your own :).  I've now added this to our comparison document.
Would you be willing to share what is it about graphite that makes it what you are looking for so I can have more perspectives to be looking at this from?

I implemented git next --branch/git prev --branch in git-branchless here: https://github.com/arxanas/git-branchless/pull/192
It might also be interesting because you can add the --all option to go to the first or last branch in a stack, or the --interactive option to select between ambiguous branches interactively.

This was implemented in #257


Evaluate if we can remove BlockProcessRecordEvent

Evaluate whether we can replace BlockProcessingRecordEvent with BlockImportedEvent introduced in PR #1064.  See discussion: https://github.com/PegaSysEng/artemis/pull/1064/files#r358519995

Done


Fix SVG shape positioning

One solution is to remove the max width, and have each "box" have width of 100%, with non-blob children being in a separate node with a consistent max width. Then the box can set overflow: hidden.

We'll also want to make sure each of the triangles are centered to the viewport so that the shift doesn't happen when going to smaller screens.

@a-trost halp plz
I think one possible solution would be to make each section have a width of 100%, and a set amount of (media-responsive) padding. So rather than having a page-level container with a max width, the max widths would be section level.
This is a bit non-DRY, but would let us just do overflow: hidden; on the sections, and to put the shapes within the sections themselves. I can take a crack at this as well.


fix(chat): fix compare name generation (Issue #921)

Description:
fix compare name generation
Issues:

Issue #921

UI changes

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review

/deploy-review


Indentation lost when pdf export ?

Hi !
Thanks for this great plugin,
very clean and nice structured appareance of documents with it.
My "request" is : why  it is not "working" when exporting to pdf ?
I mean preview mode is "perfect" but when exporting document to pdf it seems that indentation levels are lost.

hi, thanks for report the issue
i dont figure now if its possible to change export behaviour, but I'm going to analyze it


nbserve: Serve a deny-all robots.txt file

To replace the robots.txt file someone added to the NFS root (which then
broke maintain-dbusers).
Untested, unfortunately.

Related to https://phabricator.wikimedia.org/T334828


Update various dependencies

Issue number: N/A
Description of change
Change logs:

https://docs.djangoproject.com/en/2.0/releases/2.0.6/
https://github.com/jazzband/django-oauth-toolkit/blob/master/CHANGELOG.md
https://docs.pytest.org/en/latest/changelog.html
https://github.com/getsentry/raven-python/blob/master/CHANGELOG.md
http://www.gevent.org/changelog.html
https://github.com/boto/boto3/blob/develop/CHANGELOG.rst

Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

Codecov Report

Merging #972 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #972   +/-   ##
========================================
  Coverage    96.63%   96.63%
========================================
  Files          193      193
  Lines         5443     5443
  Branches       515      515
========================================
  Hits          5260     5260
  Misses         120      120
  Partials        63       63

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 af4ea85...8685e79. Read the comment docs.


[MM-31144] Android Uploads

Summary
Allows android to upload both images and files via multi-part or as a direct stream. Both options read a file input stream directly, and write it to the request (either as a direct stream, or inside a multi-part).
Note:

UploadOptions now optionally allows stream, skipBytes and method; and upload screen updated
Android Icons are fixed 🙌

Ticket Link
Resolves MM-31144

@shazm As discussed in the chats,

I pushed some e2e fixes for iOS. I also disabled upload e2e for Android because of automated file attachment limitation.
Android can only be tested manually for now. Stream upload throws error saying stream is closed. Will only test multi-part for Android
Separate PR for fast image fixes

@migbot For parity, I'm guessing iOS will also implement multi-part and skip-bytes for stream.

@shazm I pushed some new changes so that existing file server can accommodate multipart request. I tried it locally and it's able to receive request from postman. However, from the example app, it times out. Can you please run the android example on your end and the try the file server?

Run under example npm run file-upload-server
Use path: /api/files/multipart

Here are the snapshots from postman showing headers and body and successful upload.

Headers
Body

Here's the snapshot of the android app timeout

Hey @josephbaylon - I think the issues have been resolved, but we've introduced a few new things:

Default upload type is "stream", with multi-part being optional
If multi-part is selected, then you can optionally provide a "key" for the file (@enahum pointed out how MM API calls for files on the file upload endpoint, but image on the user profile picture upload endpoint)
You can then also pass in optional key: string = value: string parts to the multi-part as well
skipBytes should only be for stream uploads, and optional
Uploads should be able to call post / put / patch as required

^^ I think that changes / touches pretty much every test; and I think @migbot is doing the equivalent on iOS

@shazm I'm now getting this error when building android.

@shazm can I ask you that before submitting a PR or a new commit, test it locally and ensure that it works to avoid going back and forth all the time, this will help speed up things.
Tests should be manual and by running the E2E tests we already have.
🍀

@shazm File upload for stream works but for multipart I'm still getting a timeout error. Can you please test locally and use the preset File Upload Server API client and

Run under example npm run file-upload-server
Use path: /api/files/multipart

hey @josephbaylon - I did a lot of digging and the multipartUploadHandler seemed to be crashing for some reason but it wasn't logging out; i wrapped it in a try/catch and changed the for-loop to an Object.entries().map() which seems to work now for android. Will double check if it breaks anything in iOS a bit later today.


Implement pipeline constructor from exec iterator

Reimplentation of https://github.com/hniksic/rust-subprocess/pull/34 which allows to pass a iterator instead of a vector.

Pipeline::from_iter() should check that the iterator produced at least two Execs. For example, Pipeline::popen() expects it, and asserts that self.cmds.len() >= 2.
Note that even without this PR it's possible to create pipelines with a dynamic number of components, it's just less convenient:
let cmds: Vec<Exec> = ...;
cmds.reverse();  // easier to pop from the back
let mut pipeline = cmds.pop().unwrap() | cmds.pop.unwrap();
while let Some(cmd) = cmds.pop() {
    pipeline = pipeline | cmd;
}

Pipeline::from_iter() should check that the iterator produced at least two Execs. For example, Pipeline::popen() expects it, and asserts that self.cmds.len() >= 2.
Note that even without this PR it's possible to create pipelines with a dynamic number of components, it's just less convenient:
let cmds: Vec<Exec> = ...;
cmds.reverse();  // easier to pop from the back
let mut pipeline = cmds.pop().unwrap() | cmds.pop.unwrap();
while let Some(cmd) = cmds.pop() {
    pipeline = pipeline | cmd;
}

I added an assert like the one in Pipeline::popen(). Maybe it would be nicer to introduce a Result instead of panicing but that is probably outside of the scope of this pr.

Thanks. If you don't mind, let's make it an explicit panic!() rather than assert(), which should be reserved for "impossible" conditions which indicate a bug in the crate. For example, the assert() in popen() should never happen simply because there is (prior to the introduction of from_iter()) no public API to create a Pipeline with less than two commands.

I changed the assert into a panic and also added a document example that panics.

Probably should also use a different name for the function as clippy complains: https://rust-lang.github.io/rust-clippy/master/index.html#should_implement_trait

from_exec_iter might be a good choice for the name.
Also, it could take IntoIterator<Item = Exec>, so it can be called with actual Vec (and also with iterators, etc.).

Looks great, thanks for the patience. I've additionally renamed the iterator parameter to iterable. If there're no further changes, I'll merge soon.
At some point it's worth removing the panic for less than two Execs - at least one should be reasonable, and possibly even zero. But that might come in a separate PR.

Thanks for helping with the pull request! I guess its ready to be merged :D


mbedtls: Update to upstream version 2.16.6

Fixes https://tls.mbed.org/tech-updates/security-advisories/mbedtls-security-advisory-2020-04

Cherry-picked for 3.2.2.


fix: Fix invalid value for profile.profiling.debug in Cargo.toml

The  key in the Cargo.toml  file was assigned an invalid value, causing the manifest parsing to fail. This commit updates the value to a valid boolean () to resolve the issue.
Fixes #4677
Linked issue

Closes #4677

Benefits
Installation succeeds with cargo build.
Checklist

[x] I've read CONTRIBUTING.md
[x] I've used the standard signed-off commit format (or will squash just before merging)
[x] All applicable CI checks pass (or I promised to make them pass later)
[x] (optional) I've written unit tests for the code changes
[x] I replied to all comments after code review, marking all implemented changes with thumbs up

debug = true is valid syntax according to the docs. In any case, true is equivalent to "full".

Hello, please follow conventional commits and sign off your commit.


Mod option to highlight new accounts messages

What should be added?
A mod option so new accounts messages get highlited.
Why should it be added?
for mods in trainwrecks chat EleGiggle

Could possibly ask Trainwrecks to enable "New Chatter Announcements" on his TTV settings, then mods would be able to keep an eye on those individuals.

The way I could see this work as a Chatterino solution is:
Each Chatterino client would cache each unique user they've seen, and if it's the first time they've seen the user they would poll the Twitch API for information about the user (e.g. the account age) and store that to be able to do operations like this on the user.
The "reasonable" way to do this is either:
a) Middleware cache users in chatterino/api
or
b) limit requests, and only run requests every 5 seconds if there are users. and ensure we bulk users as well as possible.
USER STARTS CLIENT (assuming 2 users per request is max)

 00:00 -> gempir: xd
 00:01 -> gempir: lol123
 00:01 -> pajlada: hello
 00:02 -> zneix: xd
 00:05 -- REQUEST gempir, pajlada TWITCH ACCOUNT AGE
 00:06 -> gempir (2y): hello
 00:06 -> pajlada (2y): hello
 00:07 -> zneix: xd
 00:08 -> newuserxd: hello
 00:10 -- REQUEST zneix, newuserxd TWITCH ACCOUNT AGE
 00:12 -> gempir (2y): hello
 00:12 -> pajlada (2y): hello
 00:13 -> zneix: (1y) xd
 00:14 -> newuserxd (1d): hello

This is, however, a lot of work both to write and for each client to do when the feature is implemented for a very small use case, so take my theorizing with a big heap of salt.

Extending https://github.com/chatterino/api to cache some user info, like timestamp of account creation sounds like an excellent idea to me. Might even contain more stuff if we'd plan to cache more user data in the future.

I don't think that a system like this is feasible.


fix(dashmate): port marks as closed if ipv6 is not disabled

Issue being fixed or feature implemented

Issue https://github.com/dashpay/platform/issues/2100
What was done?

Forced HTTP client to use IPv4

How Has This Been Tested?

Running locally
Breaking Changes

None
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
[x] I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

Summary by CodeRabbit

New Features

Enhanced port status checking with improved HTTP request handling using the https module.
Integrated error handling and response size management for better robustness.

Bug Fixes

Removed outdated requestText method to streamline functionality.

@shumkov can you address the security issues found?

@shumkov can I remove the suggestion to disable IPv6 now? It was added in https://github.com/dashpay/docs/pull/390


update(build): update crates.io metadata to point to the new repo location

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area ci

/area event

/area event_derive

/area plugin

/area plugin_api

/area plugin_derive

/area plugin_tests

What this PR does / why we need it:
There were still links to my private repo and the generated docs, so update them to point to the new SDK home. Also, falco_plugin_runner was missing metadata completely and could not be published to crates.io.
Sadly, I only noticed it after publishing the other crates so 0.4 is stuck with my old metadata. 0.5 will be fine.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c0462f9 udpate(build): update crates.io metadata to point to falcosecurity github
53c3b56 fix(build): add crates.io metadata for falco_plugin_runner

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 55acb8ac3ba35a6669718f03ea13f7c17ef14a48


How to read the rotation and position of the camera

Is it possible to access the rotation and position of the camera in a similar way to a-frame
https://aframe.io/docs/1.3.0/components/camera.html
looking for a way to read the rotation constantly
thanks

Have you found a work around for this?


How to view Flow View List data without using Navigation menu?

How to view Flow View List data without using Navigation menu?

Hi @mdaniyalkhan,
Could you elaborate?


Broken link to PAM50 on OSF

https://github.com/neuropoly/spinalcordtoolbox/blob/266bfc88d6eb6e96a2c2f1ec88c2e185c6f88e09/scripts/sct_download_data.py#L93
the link should be: https://osf.io/u79sr/download

I added a pull request for this (#2671).
This caused me some issues installing the toolbox (I am a new user), since the install script would hang at this line, never getting to configurations, editing the .bashrc, etc., so none of the commands worked. Thank you for raising the issue as the change got it working.

Thanks for this @htp2! It should fall back to the second mirror, but I noticed that our second mirror was also hanging. I wonder if what happened is actually that this link never worked and it was always using the second mirror, but that has become overloaded in the last couple weeks.


Generate sample does not handle well not keyword

Nuget Package
JsonSchema.Net.DataGeneration
Package Version
online
Operating System
None
.Net Target (if relevant, please specify the version in the description)
None
A clear and concise description of the bug.
give the schema below the data genereation generate data that are not valid if i pass them to the validator
here i want to express that you must provide either A , or B but never both , and still be extensible ...
so leveraging not keyword
looks not working as generating data with A , and B ...
then if i take back the value generated to the Validate data it mentionned that data are not valid
according to the documentation
https://docs.json-everything.net/schema/datagen/schema-datagen/#schema-datagen-capabilities
it should be supported
generation

validation

What did you expect?
to not generate unvalid data
ie never have A and B
Please add test code or steps to reproduce the behavior.
{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "type": "object",
  "additionalProperties": true,
  "oneOf": [
    {
      "required": ["A"],
      "properties": {
        "A": {
          "type": "string"
        },
        "B": {
          "not": {}
        }
      }
    },
    {
      "required": ["B"],
      "properties": {
        "A": {
          "not": {}
        },
        "B": {
          "type": "string"
        }
      }
    }
  ]
}

Is there any other information you'd like to share regarding this bug?
No response
Code of Conduct

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

Thanks for reporting this.
What the generator does in the case of not is it inverts any requirements that are on the subschema.  In your case, you have an empty schema {} which means there are no requirements to invert.
Realistically, {"not": {}} is just false, and I recommend using that instead.

the idea behind the scene was to say strictly A or B exclusive , without sacrifying extensibility of the object
ans so leveraging not was a way to forbid usage of the other field .

Yes, but by titling the issue "Generate sample does not handle well not keyword", you're saying that you think your usage of not is what's causing the problem, and it is part of it.
Please try with a false schema instead of the not: {}.

not 100% confortable with the syntax
tested with

or  "B" :  {false}  gives invalid schema

Okay, yeah. That demonstrate an issue with oneOf.


[RESTEASY-2083] Fix issue when SseBroadcasterImpl.notifyOnCloseListeners removes wrong sink from the outputQueue

Due to GenericType.equals implementation currently (as per 3.6.2.Final) SseBroadcasterImpl always removes first item from the outputQueue in notifyOnCloseListeners method when client closes connection. Resulting in other clients stop receive events.
Test project that reproduces the issue and suggest workaround: link

@Ingvord thanks for the contribution. @jimma will review the PR; in the mean time, we require that all contributions are made under the terms of the ASL 2.0 License: http://www.apache.org/licenses/LICENSE-2.0
Do you agree to these terms?

@asoldano

we require that all contributions are made under the terms of the ASL 2.0 License: http://www.apache.org/licenses/LICENSE-2.0

sure, no objections from my side!

@Ingvord  Appreciate your detailed analysis and patch for this issue.  This looks OK to me. It would better that we have a test for this important fix. Can you please add your reproducer/test to this PR ?

@jimma test case has been added in 35ffb25
While looking through the test class I have noticed the following code snippet:
sseBroadcasterImpl.onClose(ses -> {
   Assert.fail("Close listeners should not have been notified");
});

AFAIK fail will never happen because onClose is invoked from async thread and exceptions thrown there are ignored. Could you please correct me if I am wrong?


Created skills endpoint

Added get and delete methods
Added test_036
Passed test and added evidence
modified:   back-end/onet_app/urls.py
modified:   back-end/onet_app/views.py
modified:   back-end/tests/test_views.py
new file:   doc/tests/test_036.png


Handle overlaps when compiling static fonts

Overlaps produce cutouts when drawn with evenodd fill. Apparently older versions of macos and maybe some printers do this.
When compiling a static (non-variable) font do at least one of:

At least set OVERLAP_SIMPLE (only if there are overlaps?), https://learn.microsoft.com/en-us/typography/opentype/spec/glyf
Actually remove the overlaps.

No Rust implementation equivalent ot Skia pathops appears to exist, would likely have to call into pathops
@simoncozens https://github.com/simoncozens/rust-font-tools/blob/main/fonttools-cli/src/bin/ttf-remove-overlap.rs does this via https://docs.rs/crate/skia-safe/latest

The importance of actually removing the overlaps is a bit unclear to me. It used to be done for macos but now we have the overlap simple flag. Rumor has it that old printers might need actual removal but I don't have specific examples.
References:

https://github.com/fonttools/fonttools/issues/2059 discusses FreeType and Microsoft handling

Courtesy of @simoncozens , "There are many experienced designers out there sharing their perspective on this topic why it’s not needed to outline your text anymore but it’s still the go-to method for most designers whenever they’re preparing their files for print. " (https://www.jukeboxprint.com/blog/is-it-still-necessary-to-outline-your-fonts)


Scaler performance

Codecov Report

Merging #305 (50f9d1d) into master (3361f52) will increase coverage by 0.04%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master     #305      +/-   ##
==========================================
+ Coverage   67.19%   67.23%   +0.04%
==========================================
  Files         120      120
  Lines       10561    10569       +8
==========================================
+ Hits         7096     7106      +10
+ Misses       3465     3463       -2

Impacted Files
Coverage Δ

assemblyline/remote/datatypes/queues/priority.py
76.19% <87.50%> (+0.93%)
:arrow_up:

assemblyline/odm/randomizer.py
96.05% <0.00%> (+0.87%)
:arrow_up:

assemblyline/common/heuristics.py
84.37% <0.00%> (+1.56%)
:arrow_up:

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 3361f52...50f9d1d. Read the comment docs.


Securing Your Clusters - iap-ingress privateGKECluster parameter config

https://www.kubeflow.org/docs/gke/private-clusters/
The following instructions are wrong as the structure is not different. There is no iap-ingress.yaml and also I cannot find the privateGKECluster parameter in iap-ingress/kustomization.yaml (on any yaml within the iap-ingress directory)

Update iap-ingress component parameters:
cd ${KF_DIR}/kustomize
gvim iap-ingress.yaml

Find and set the privateGKECluster parameter to true:
privateGKECluster: "true"

Then apply your changes:
kubectl apply -f iap-ingress.yaml

Duplicate of #1881


Potential cdc url xrefs and NTRs

Scraped the CDC Diseases and Conditions A-Z index for first level urls and mapped them to Wikidata QIDs and MONDO IDs where possible. For items with no MONDO IDs, filled in details for NTR whenever reasonable. https://docs.google.com/spreadsheets/d/1yRzWksvL3t--gYFG0EQPOK3SwRKleuXs0Se2_8VcK6g/edit?usp=sharing

Hi @gtsueng - thanks for this ticket. I'm unclear what the action item is.
Apologies for the long delay.

I think those were just potential xref urls that could be added if MONDO were so inclined.

Ok, thank you! We'll look into this further as soon as possible.


osd-17784: changing schema validation script

Changed the command for the schema validation using the new image.
And removed the 'success' message as it is already provided by the command output.

/lgtm
/approve


Update quota manager to reduce zk access

This PR updates quota manager to reduce zk access.
Currently when a broker resource gets changed, all the tables which contains qps quota will be re-calculated. This PR reduces the zk reads.

Codecov Report

Merging #4979 into master will increase coverage by 21%.
The diff coverage is 32.43%.

@@             Coverage Diff              @@
##             master    #4979      +/-   ##
============================================
+ Coverage     36.35%   57.35%     +21%
- Complexity        0       12      +12
============================================
  Files          1174     1174
  Lines         62244    62268      +24
  Branches       9148     9150       +2
============================================
+ Hits          22627    35713   +13086
+ Misses        37596    23877   -13719
- Partials       2021     2678     +657

Impacted Files
Coverage Δ
Complexity Δ

...quota/HelixExternalViewBasedQueryQuotaManager.java
61.98% <32.14%> (+24.56%)
0 <0> (ø)
:arrow_down:

...ache/pinot/broker/queryquota/QueryQuotaConfig.java
57.14% <33.33%> (+57.14%)
0 <0> (ø)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
57.31% <0%> (-4.88%)
0% <0%> (ø)

...r/filter/predicate/PredicateEvaluatorProvider.java
52.17% <0%> (-4.35%)
0% <0%> (ø)

...ot/controller/helix/core/util/HelixSetupUtils.java
88.75% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...impl/dictionary/DoubleOnHeapMutableDictionary.java
36.58% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ces/PinotSegmentUploadDownloadRestletResource.java
61.34% <0%> (+0.61%)
0% <0%> (ø)
:arrow_down:

.../impl/dictionary/LongOffHeapMutableDictionary.java
64.51% <0%> (+1.07%)
0% <0%> (ø)
:arrow_down:

...e/io/writer/impl/MutableOffHeapByteArrayStore.java
85.71% <0%> (+1.09%)
0% <0%> (ø)
:arrow_down:

...g/apache/pinot/common/request/InstanceRequest.java
28.83% <0%> (+1.14%)
0% <0%> (ø)
:arrow_down:

... and 586 more

Continue to review full report at Codecov.

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

I still dont understand how this reduces ZK access. Correct me if I am wrong with the following description.
We used to fetch the propertystore table config all the time. Now, we do a Stat followed by a fetch if the stat does not match.
So, we have effectively increased the number of read calls to zk.
Yes, we have reduced the overall volume of reads from ZK, but I am not sure that is a bottleneck we are facing. Our problem with ZK is on the write side.

OK, the resolution is that most of the time we don't need to fetch the table config. On clusters with 1000s of tables this change will reduce the volume of data pulled from zk each time we restart a broker.


Support RFC2617 stale

RFC 2617 defines the "stale" server option that indicates to clients that the nonce was stale, and that the client should just re-encrypt the password with the new nonce.  This should be done when the nonce was found to be missing, so that a restart doesn't cause all users to be challenged again.

Thanks for reporting Erik, I added stale support with 1.1.8 version.

Awesome!


feat(jupyter): Use global venv and update requirements from template

Added a run-jupyter-notebook target to the Makefile
Install dependencies from requirements_template per default
Update depdendencies automatically during container startup
Use a global venv instead of --break-system-packages
Change ownership of pip packages from root to techuser

Resolves #178

Nice!
_OLD_VIRTUAL_PATH looks a bit hacky, but this is better that --break-system-packages for sure.

I theory, we could also drop it, but with _OLD_VIRTUAL_PATH the user can still disable the venv with the official deactivate script.


Bug: dimr_config for parallel run is incorrect

Fixes: #562

Have a look at the SonarCloud issue please! :)


Add ATL dependencies setup steps for windows

It's for people that do not have full Visual Studio Professional installed and have the basic tools flutter installed. Took me ~15 min to find the issue.
Open Visual Studio Installer, press on Modify for VS 2019, go to Individual components, search for ATL and install last version for your processor architecture.

i am using vs community edition 2019 installed all ATL for (ARM and X86/X64) still its showing atlimage.not found
please help!


Extract Cli parsing argument from Cli

This is for #114.

This pull request #172 is assigned to @vatavuk/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@g4s8 ping

Codecov Report

Merging #172 into master will increase coverage by 3.47%.
The diff coverage is 83.72%.

@@             Coverage Diff              @@
##             master     #172      +/-   ##
============================================
+ Coverage     42.58%   46.05%   +3.47%
- Complexity       86       93       +7
============================================
  Files            32       33       +1
  Lines          1031     1051      +20
  Branches         44       44
============================================
+ Hits            439      484      +45
+ Misses          586      560      -26
- Partials          6        7       +1

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/artipie/rpm/Cli.java
23.80% <60.00%> (+23.80%)
1.00 <1.00> (+1.00)

src/main/java/com/artipie/rpm/CliArguments.java
86.84% <86.84%> (ø)
4.00 <4.00> (?)

...ain/java/com/artipie/rpm/StandardNamingPolicy.java
87.50% <0.00%> (+87.50%)
2.00% <0.00%> (+2.00%)

Continue to review full report at Codecov.

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

@g4s8 please remove this issue from the scope

@g4s8 ping

@0crat out

@sereshqua/z please review this job completed by @vatavuk/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat quality good


Implement processor for generating ContentValues.

this is a pull request to implement #3 - it adds an annotation processor which generates code for generating ContentValues using a Builder pattern. The Builder is modeled as an interface, similar to the technique used by AutoValue. The @Table annotation is not used right now in and of itself (the processor looks directly for @Table.Builder), but i kept it there based on the assumption that this would expand as time goes on to support some of the other tickets.

Mostly just stylistic comments. This is a great start to the processor!

I realized now that I jumped the gun a little on this and submitted this request without fully understanding the vision of the project (which I now have a better sense of). In the future, I will propose a solution and get approval on the direction before actually submitting a pull request. I apologize for any inconvenience, and thank you for the review and for providing the community with this library!


Feat: Save for later

Introduce a new feature selective cart.
Breaking changes: cart detail component, cart item list component, cart item component.
Closed #5720.

@maggielin0 did you intend to add someone else as reviewer?

@maggielin0 did you intend to add someone else as reviewer?

hi @sstadelman I added you as one of the reviewer if you're available. the design was discussed before with you.

@maggielin0 I’m Stan Stadelman on the iOS SDK team... I don’t recognize this Angular code.

sorry, remove you from reviewer,thanks let me know


python310Packages.breathe: 4.31.0 -> 4.32.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.breathe is: "Sphinx Doxygen renderer"
meta.homepage for python310Packages.breathe is: "https://github.com/michaeljones/breathe"
meta.changelog for python310Packages.breathe is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
Warning: no invocation of /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0/bin/breathe-apidoc had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0/bin/.breathe-apidoc-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 4.32.0 with grep in /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0
found 4.32.0 in filename of file in /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.breathe
python39Packages.breathe
wireplumber

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.breathe https://github.com/r-ryantm/nixpkgs/archive/6ef5dc851031a7015c5a59c7da33b9a2167cf8fd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0
ls -la /nix/store/iw124mi94xbqv92xjih6yj1nn3d15p5p-python3.10-breathe-4.32.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

python310Packages.breathe
python39Packages.breathe
wireplumber

Maintainer pings

Result of nixpkgs-review pr 157485 run on x86_64-linux 1

3 packages built:

python310Packages.breathe
python39Packages.breathe
wireplumber


Test against (part of) the JSON Schema Test Suite

See #16.
This adds the JSON Schema Test Suite repository as a submodule in the tests/ directory. It also adds another binary target which reads the JSON files containing the tests, extracts the schemas to separate files (placed in tests/test_suite/schemas/<test_name>_<index>.json), and generates a .rs file which generates types from the schema and checks whether the sample data does or does not parse.
The code generator, in src/generate_tests.rs, can be run with cargo run --bin generate-tests. It contains a function called is_blacklisted(), which is used to skip the majority of the tests in the test suite.
I think it would take quite a lot of work to get schemafy to pass the test suite, but hopefully this provides at least a bit of value in terms of test coverage, and helps people trying to add new features.

Note: If you don't run git submodule update --init and then cargo run --bin generate-tests, then cargo test will fail. It would be easy to change things around so that running the test suite is optional.

Nice! Thanks!


Auto label PR with Hacktoberfest

Added workflow labeler for automatically labeling the PR's
https://github.com/Opentek-Org/bookquotes/issues/39

@AnirudhPanda please review now. Hope everything is good:)

Great!!


Consider bundling tests with Browserify

Pros

Removes need to update the runner.html file when new tests are added
May require adding watchify if the library/test suite grows large
Tests could use require statements to load individual pieces of the library
Boilerplate becomes easier to adapt to larger, client-side apps

I'm going to close this for now, as it doesn't seem too necessary. I can revisit this later if there's any interest.


Updated pod labels to include all labels

PR Description
Updates the _pod.yaml file to include the same labels used in the DaemonSet/StatefulSet/Deployment to help improve traceability throughout the application.  The new labels include the selectorLabels which it was previously using so this should only be adding additional labels and not removing any.
Which issue(s) this PR fixes
https://github.com/grafana/alloy/issues/2070

 #issue_id
Notes to the Reviewer
PR Checklist

@petewall Is there anything specific I need to get this PR to continue its workflow? It's been stuck for a couple of days now at the continuous-integration step and when I click Details it just says to contact the repo admin.  :(  If there's something I'm missing from my PR, I'm more than happy to address that.  Thanks!!!

I approved and kicked off the build.

Thanks @mattdurham!!! It looks like I needed to run a script to update the tests.  I'm guessing that's why it failed at the previous step, but I think it should be good now.

@petewall Any chance you or someone else can take a look at this PR please?


UE4/Android ClientConfiguration destructor crash

I've built aws-sdk-cpp as shared library to use .so, because that's what best supported in UnrealEngine 4. Packaging process doesn't return any errors, but when I successfuly instantiate GameLiftClient with custom configuration it crashed after ClientConfiguration destructor is called. What am I doing wrong? Do I need to pass another argument?
What platform/OS are you using?
OS: MacOS Mojave 10.14.6
Target: Android
Which version of the SDK?
3.15.2
What compiler are you using? what version?
LLVM/Clang/libc++ (clang-1<IP_ADDRESS>)
What are your CMake arguments?

@KaibaLopez Which part of the code do you want me to share? Build.cs or loading module?

Hi @AlexGroszq, both would be helpful, but since this is not an error on build, if you can provide the simplest code sample to reproduce this error, that would save us a lot of time.

@KaibaLopez Here are code examples of what I am doing.
Fragment of Build.cs for GameliftLibrary:

GameLiftClientSDK_UPL:

Same thing is done for AWS Core part. No brainer here.
Here is the part, which causes the fatal error:

I have already debugged it. It turns out, that method from line 38th is called successfuly (I checked all the lines in Internal_InitGameLiftClientSDK). After GC is collecting Aws::Client::Configuration class it crashes with fatal error.

Hi @AlexGroszq, this seems to be an implementation and not an sdk problem, so not much I can help you with here...
But just in case, can you try to just do:
delete GameLiftClient;
and see if you get the error on that destructor call?

Hello @KaibaLopez I tried what you told me to do, and I am also getting the same crash as before.

@KaibaLopez One week later and no response from you?

@KaibaLopez Hello, do you know anyone who would not ignore me? I know you (as amazon) are not officially supporting Epic's engine in terms of client API, but at least you could response to my comments in this issue.

I'm sorry for the lack of response last week @AlexGroszq , the truth is I have not made significant progress with your issue so I have no real udpate for you. From the looks of it, this is not a problem with the SDK itself, so it is lower priority for us.
Now, because this is a kind of specific issue to your implementation you're having. I'd say try forums like stackoverflow or Unreal Engine.
While this is an open source project and ideally we'd have contributors from the community help out when possible, you'll probably find more community support/responses from one of those sites.
As of now, all I can recommend for you is trying not to use standard pointers, as they can be tricky to delete.

Just a hunch, are you calling Aws::InitAPI() anywhere in your code?

@JonathanHenson
Hello Jonathan,  I am calling Aws:InitAPI(). Otherwise I was getting an error in Aws::Utils::Crypto::CreateSha256Implementation()

@AlexGroszq , I'm truly sorry I was never able to solve your issue, but because it is not really reproducible, I don't have anything else to do about it so I'll be closing it, again I recommend you take this to forums in hopes to get more community support for implementation kind of issues.

Hey, I had some similar issue with a crash in destructor of Aws::Client::ClientConfiguration. It seems to be solved by making Aws::Client::ClientConfiguration static or global... go figure.


Page Extension Triggers vs. Event Subscribers

What is the design phylosophy behind supporting both Page Extension Triggers and Page Event Subscribers.
They seem to do the same, execute code after the standard Microsoft code is executed.

Events came first, but we felt that with the new extension objects we could provide an easier and better way. We have no plans to remove the event support, but we expect that developers will prefer the trigger model over the events.
In the current implementation the new triggers are called "closer" to the original trigger than the events. This is an implementation detail that could changed and shouldn't be relied upon.


Errors while creating Knowledge base

when I run this line to create a kb
create_op = client.knowledgebase.create(create_kb_payload=create_kb_dto)
I receive this error:
create_op = client.knowledgebase.create(create_kb_payload=create_kb_dto)
Traceback (most recent call last):
File "<ipython-input-21-cbda6ca88672>", line 1, in <module>
create_op = client.knowledgebase.create(create_kb_payload=create_kb_dto)
File "C:\Users\shchitt.NORTHAMERICA\AppData\Local\Continuum\anaconda3\lib\site-packages\azure\cognitiveservices\knowledge\qnamaker\operations\knowledgebase_operations.py", line 389, in create
request = self._client.post(url, query_parameters, header_parameters, body_content)
File "C:\Users\shchitt.NORTHAMERICA\AppData\Local\Continuum\anaconda3\lib\site-packages\msrest\service_client.py", line 193, in post
request = self._request('POST', url, params, headers, content, form_content)
File "C:\Users\shchitt.NORTHAMERICA\AppData\Local\Continuum\anaconda3\lib\site-packages\msrest\service_client.py", line 108, in _request
request = ClientRequest(method, self.format_url(url))
File "C:\Users\shchitt.NORTHAMERICA\AppData\Local\Continuum\anaconda3\lib\site-packages\msrest\service_client.py", line 155, in format_url
base = self.config.base_url.format(**kwargs).rstrip('/')
KeyError: 'Endpoint'

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

ID: f424197c-decf-8490-2117-949537325fca
Version Independent ID: 298fe24b-0a1f-a629-63a9-fa2958a45eea
Content: Quickstart: QnA Maker client library for python - Azure Cognitive Services
Content Source: articles/cognitive-services/QnAMaker/Quickstarts/python-sdk.md
Service: cognitive-services
Sub-service: qna-maker
GitHub Login: @diberry
Microsoft Alias: diberry

@ShivaKumarChittamuru
Thanks for the feedback! We are currently investigating and will update you shortly.

#in-progress

@ShivaKumarChittamuru Please review this text, does it make sense:
Ignore spelling mistakes
If you don't want to use the Bing Spell Check API v7 service, you need to add the correct and incorrect spelling.
Two solutions are:

Label example utterances that have the all the different spellings so that LUIS can learn proper spelling as well as typos. This option requires more labeling effort than using a spell checker.
Create a phrase list with all variations of the word. With this solution, you do not need to label the word variations in the example utterances.


Is it possible to use Live Metrics on a static generated Site?

I'm using Gatsby to generate a static site, and have App Insights working there. My build pipeline sends the static files to an Azure Storage account and it is serving via the static website info there with an Azure CDN as well.
Like I said, I get telemetry and everything seems to work, but no live metrics. What needs to be done in addition to the basic just to track page views to get Live Metrics working?

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

ID: f464520a-395e-47a6-216d-348444145911
Version Independent ID: 51943e10-d024-8bc0-9e36-0fe196cb7ca9
Content: Diagnose with Live Metrics Stream - Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/live-stream.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@Schenz Thank you for your query. We will further investigate and get back to you at the earliest.

@Schenz Thanks for the query. Live Metrics for static generate site should not be restricted, however one possible reason I could think off is related to firewall restrictions. Could you please help with some additional information on whether your application is in protected network or not. Live Metrics Stream uses different IP addresses than other Application Insights Telemetry. Please find the list of IP addresses that should be open to the firewall here.
Could you please help validate this , so that we can navigate through next steps. Thank you

@Schenz Hope the above information helps to further isolate the issue. For now, will go ahead and close the issue. Please validate from your side if any further queries please feel free to tag us in your reply and will be more than happy to reopen the thread and continue the conversation. Thanks


Spinner does not support dark mode

Description
Spinner does not seem to have any support for dark mode AFAICT. Here's what it looks like:

Steps to reproduce
<ThemeProvider colorMode="dark" nightScheme="dark_dimmed">
  <Spinner />
</ThemeProvider>

Version
35.14.2
Browser
Chrome

Hi!
Spinner gets it's foreground colors from the parent, so along with ThemeProvider, you would need to wrap your app in BaseStyles as well
import { ThemeProvider, BaseStyles, Spinner } from '@primer/react'

<ThemeProvider colorMode="dark" nightScheme="dark_dimmed">
  <BaseStyles>

    <Spinner />

  </BaseStyles>
</ThemeProvider>

Sandbox: https://codesandbox.io/s/magical-franklin-41yrk7?file=/src/App.tsx:484-519

Closing this issue as answered, but feel free to comment back if this doesn't help and I'll open it again :)

Thanks so much @siddharthkp ! What does BaseStyles do exactly? Is there documentation for it anywhere? I was only aware of the existence of ThemeProvider

BaseStyles are responsible for injecting global styles for primer/github
Reference: https://primer.style/react/getting-started#basestyles
Source: https://github.com/primer/react/blob/main/src/BaseStyles.tsx
(It's opt-in because some applications at github inject globals throught a different library)

Ahhh, got it! Ok, thank you so much @siddharthkp !!


Large memory allocations which are not tracked

Description
In Gluten there is one common issue is "killed by yarn", the root cause is that some memory allocation usually the std::vector which bypasses memory pool track. Some of the std::vector can be very large like a commonly used per row std::vector<char*>. If row size is 1G, the vector size can be as large as 8G.
The ideal solution is to avoid the per row vector as much as possible. If we have to use it's better to track in memory pool by using std::vector<char*, memory::StlAllocator<char*>> and std::allocated_shared.
Here is a umbrella track of such allocations.
Link to Gluten issue: https://github.com/apache/incubator-gluten/issues/6947

#11077 remove the per row vector allocation

https://github.com/facebookincubator/velox/blob/9e1280a7223fa324b8307d379605bcfc89f8447a/velox/exec/SortBuffer.cpp#L121
Per row memory allocation

https://github.com/facebookincubator/velox/blob/9e1280a7223fa324b8307d379605bcfc89f8447a/velox/exec/SortBuffer.cpp#L320
Potential per row allocation. Spill is already triggered, we should allocate it using spill_memorypool.


Features are not found by ReqnrollFeatureFiles/ReqnrollGeneratedFiles when the feature files are in a shared project that is referenced by Content Include in csproj

Reqnroll Version
2.0.3
Which test runner are you using?
NUnit
Test Runner Version Number
4.1.0
.NET Implementation
.NET 8.0
Test Execution Method
ReSharper Test Runner
Content of reqnroll.json configuration file
{
"$schema": "https://schemas.reqnroll.net/reqnroll-config-latest.json",
"language": {
"feature": "en-US"
}
}
Issue Description
We are using Appium for UI testing our app. We need a seperate project for Android and for iOS.
The shared project has the following csproj:
<Project Sdk="Microsoft.Build.NoTargets/3.7">
    <PropertyGroup>
        <TargetFramework Condition="'$(TargetFrameworkOverride)' == ''">net8.0</TargetFramework>
        <TargetFramework Condition="'$(TargetFrameworkOverride)' != ''">TargetFrameworkOverride</TargetFramework>
        <GenerateAssemblyInfo>False</GenerateAssemblyInfo>
        <GenerateMSBuildEditorConfigFile>False</GenerateMSBuildEditorConfigFile>
        <GenerateTargetFrameworkAttribute>False</GenerateTargetFrameworkAttribute>
        <RootNamespace>App.UiTests.Shared</RootNamespace>
    </PropertyGroup>
    <ItemGroup>
        <None Include="Features\**\*.feature" />
        <Compile Include="**\*.cs" />
        <None Include="**\*.json" />
        <None Remove="obj\**\*" />
        <None Remove="bin\**\*" />
    </ItemGroup>
    <ItemGroup>
        <None Update="appsettings.json">
            <CopyToOutputDirectory>Always</CopyToOutputDirectory>
        </None>
    </ItemGroup>
</Project>

It is a NoTargets project, so this project won't be compiled.
The C# and Feature files are referenced into the other assemblies with the following csproj:
<Project Sdk="Microsoft.NET.Sdk">
    <PropertyGroup>
        <TargetFramework>net8.0</TargetFramework>
        <ImplicitUsings>enable</ImplicitUsings>
        <Nullable>enable</Nullable>
        <IsPackable>false</IsPackable>
        <IsTestProject>true</IsTestProject>
        <RootNamespace>App.UiTests.Android</RootNamespace>
    </PropertyGroup>
    <ItemGroup>
        <PackageReference Include="Appium.WebDriver" Version="5.0.0" />
        <PackageReference Include="coverlet.collector" Version="6.0.2">
            <PrivateAssets>all</PrivateAssets>
            <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
        </PackageReference>
        <PackageReference Include="Microsoft.Extensions.Configuration" Version="8.0.0" />
        <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
        <PackageReference Include="NUnit" Version="4.1.0" />
        <PackageReference Include="NUnit.Analyzers" Version="4.2.0">
            <PrivateAssets>all</PrivateAssets>
            <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
        </PackageReference>
        <PackageReference Include="NUnit3TestAdapter" Version="4.5.0" />
        <PackageReference Include="Reqnroll.NUnit" Version="2.0.3" />
        <PackageReference Include="Reqnroll.Tools.MsBuild.Generation" Version="2.0.3" />
        <PackageReference Include="Shouldly" Version="4.2.1" />
    </ItemGroup>
    <ItemGroup>
        <Using Include="NUnit.Framework" />
    </ItemGroup>
    <ItemGroup>
        <Content Include="..\App.UiTests.Shared\Features\**\*.feature"
                 Link="Features\%(RecursiveDir)%(Filename)%(Extension)"
                 CopyToOutputDirectory="PreserveNewest" />

        <None Include="..\App.UiTests.Shared\*.json"
              Link="%(Filename)%(Extension)"
              CopyToOutputDirectory="PreserveNewest" />

        <Compile Include="..\App.UiTests.Shared\**\*.cs"
                 Link="Shared\%(RecursiveDir)%(Filename)%(Extension)" />
    </ItemGroup>
    <ItemGroup>
        <Content Include="Features\*.feature" />
    </ItemGroup>
</Project>

If I add normal unit tests to the shared project, they can be run when running the Android or iOS project. However, ReqNRoll does not "see" the .feature files nor compiles them.
Can you help us out? I want to avoid real symlinking, and want to just include the files to the Android/iOS project.
Steps to Reproduce

Create a shared project with the first CSPROJ from the issue description. Add a folder "Features" with one .feature file in it, with one scenario. Also add a folder "Steps" and add one step implementation to that feature file in it.
Create two NUnit test projects with the second CSPROJ from the issue description. If you are using an IDE like rider, two folders will show up: Features and Shared (with the Steps folder in Shared)
Build one of the NUnit projects
Notice that the feature file is not compiled
Open the test runner and notice that the feature is missing

Also, the output is not listing the features:
0>------- Started building project: App.UiTests.Android
ReqnrollFeatureFiles:
-> Using reqnroll.json
ReqnrollGeneratedFiles:

Link to Repro Project
https://github.com/maurictg/repro-reqnroll-sharedprojects

It seems to be a Rider issue.
dotnet build in the Android/iOS folder followed by dotnet test works just fine.

If it is a rider issue it most likely had to do with the resharper build engine and their heuristics to determine a particular project is up to date. If you manually include files in a nocontent project you will have to specify the uptodatecheckinput https://github.com/dotnet/project-system/blob/main/docs/up-to-date-check.md


Can't login with a new account...

Hi,
I used TVHGoogleMapi with another account on a computer before, and we switch the user on this computer but we can't login with the new user account on tvhgooglemapi, even with the good password...
I really don't know why i can't login...i tryed to reset the password and we always get the invalid password message...
Can you help us please?
Thanks

Your answer is probably here: https://lists.exim.org/lurker/message/20150831.183913.650bb6ba.en.html

Hi,
I tried your solution but we keep getting the wrong password message.

Does this help? https://support.google.com/accounts/answer/6009563?vid=1-635790400077101136-2917816429
I assume you have enabled imap?

Hi Jan,
It's working now!
Thanks a lot for your help.
Your tool is very usefull.
Alexis Routhier
Intégrateur Logiciel
T  1-418-956-5922 / 1-514-266-9774
________________________________________________________________
1788, chemin de la Canardière, Québec (Qc) G1J 2E3
1, Place Ville Marie, bur. 2001, Montréal (Qc) H3B 2C4
2015-09-28 8:29 GMT-04:00 Jan Keirse<EMAIL_ADDRESS>
Does this help?
https://support.google.com/accounts/answer/6009563?vid=1-635790400077101136-2917816429
I assume you have enabled imap?
—
Reply to this email directly or view it on GitHub
https://github.com/jankeirse/tvhgooglemapi/issues/45#issuecomment-143729131
.


Update publish.command.js

Steps to Reproduce:

Create and execute the config.js using the testbeats package with cypress or other testtool.
Observe the error message indicating ReferenceError: config is not defined.

Root Cause:
The variable config is not defined in the scope of the #processConfig method. The correct variable that should be referenced is processed_config.
Solution:
Replace config.reports with processed_config.reports in the #processConfig method to reference the correct variable

#199 should work fine after this update.


Overriding dispatch in runSaga method

Hi,
There is a use case where we want to override "dispatch" while passing to runSaga (https://github.com/redux-saga/redux-saga/blob/master/packages/core/src/internal/runSaga.js#L13). Out dispatch  method is run time function of props.
But the dispatch method is bind initially in middleware (https://github.com/redux-saga/redux-saga/blob/master/packages/core/src/internal/middleware.js#L14) which is stopping us to pass our dispatch.
Can this be done ?

@Andarist Any thought(s) on this ?

The builtin middleware is a thin wrapper around runSaga API - so you can just built you own version on top of the latter API.
Even if the dispatch would not be bound it probably still wouldn't work for you, because the dispatch which gets passed into middleware is actually not the same store.dispatch which you use in your code. See here:
https://github.com/reduxjs/redux/blob/85024d431ca765c843561be6792edeb7890d5fce/src/applyMiddleware.ts#L72


Add true support for vector drawables

@samtstern I didn't know this at the time, but we were never actually getting the true benefits of vector drawables because the build tools would just convert them to pngs at the end of the day. I went through the pains of figuring out how to enable true vector drawable support for my app and I'm bringing this knowledge over to FUI. Yay! 😄
Turns out we actually save quite a few kbs considering how few drawables we have:

Old
New
Diff
% of original

496KB
468KB
28KB
94%

Oh nice!  I'll take another look at this early next week (and your other PRs, I have been slacking on this repo).  I remember when we went with vectors for Santa Tracker there were a bunch of gotchas and I just want to make sure they don't apply here (I think we're OK).

@samtstern SGTM! (I also have another PR coming up 😊)

Very cool to see this change! #vectorallthethings

@samtstern Was this closed by accident or did you decide against it?

@SUPERCILEX accident!  Sorry haha.

@samtstern 😄👍

LGTM!


feat: add balm-ui resolver

Description
balm-ui resolver
Linked Issues
[Feature Request] Support unplugin-vue-components
Additional context

Thanks for your contributions. We are sorry we no longer accept new resolvers:
https://github.com/antfu/unplugin-vue-components/blob/main/src/core/resolvers/_READ_BEFORE_CONTRIBUTE.md


This app is blocked

When I tried to authorize via browser (run skicka init and then skicka ls without setting any value), I got the following error after clicking my e-mail address:

This app is blocked
This app tried to access sensitive info in your Google Account.  To keep your account safe, Google blocked this access.

Why did it happen and how can I workaround?

I got the same error

This is the right fix

thanks @mmp :)

Same error aswell

I get ```skicka: attempting to launch browser to authorize.
(Re-run skicka with the -no-browser-auth option to authorize directly.)
skicka: error with OAuth2 Authorization: oauth2: cannot fetch token: 400 Bad Request
Response: {
"error": "invalid_request",
"error_description": "client_secret is missing."
}
when attempting to use this method.


Broken attribute highlighting when using tag name interpolation

- var tag='a'
// Broken
#{tag}(
    @click="doSomething",
    :href="url",
)
// Broken
#{tag}(
    click="doSomething",
    href="url",
)
// These work fine
a(
    @click="doSomething",
    :href="url",
)
a(
    click="doSomething",
    href="url",
)

Ok, so the issue seems to be with the interpolation subpattern having the same name: https://github.com/genoma/language-pug/blob/f1f1610222a9f14f0a95fdffcd579a6e4cb64b42/grammars/pug.cson#L1096
It looks like this prevents the tag_name from ending correctly, because if I log the cursor scope at ( (with the cursor in front of it), it still says "source.script.jade entity.name.tag.pug", whereas without the interpolation, it says "constant.name.attribute.tag.pug".
Renaming it to "source.script.jade entity.name.tag.pug.interpolated" fixes the highlighting for the attributes (the tag name ends correctly at } now), but the highlighting of the interpolation doesn't work correctly (it doesn't seem to work that well anywhere else either).


Execute stages in a dag

This changes enhances the yip execution flow such as now allows introspection.
The PR changes the inner logic of the executor to just generate operations. The operations are loaded into a DAG which is executed.

A new flag --analyze is introduced, can be used as such to print the execution dag of a set of cloud config:

❯ ./yip -s network --analyze ~/_git/kairos/examples
INFO[0000] 1.
INFO[0000]  <init> (background: false) (weak: false)
INFO[0000] 2.
INFO[0000]  <Default deployment-network-Setup k3s> (background: false) (weak: true)
INFO[0000] 3.
INFO[0000]  <Default deployment-network-1> (background: false) (weak: true)

A new field after is available in the stages to allow nested dependencies in the same stage, across different files by refering to it with the name of yip file and the block concatenated by a . (note this is not allowing dependencies from different stages!):

file 1
stages:
 rootfs:
 - after:
   - name: foo.action

file 2
name: "foo"
stages:
  rootfs:
  - name: "action"

maybe this should be named after instead of depends. it is not introducing any logic to block the dependants to not fire.

after doesn't really seem to work, marking as draft

could this generate infinite recursion?
file 1
name: "foo"
stages:
  rootfs:
  - after:
    - name: bar.action
  - name: "action"

file 2
name: "bar"
stages:
  rootfs:
  - after:
    - name: foo.action
  - name: "action"

could this generate infinite recursion?
file 1
name: "foo"
stages:
  rootfs:
  - after:
    - name: bar.action
  - name: "action"

file 2
name: "bar"
stages:
  rootfs:
  - after:
    - name: foo.action
  - name: "action"

nope, should badly panic as for now (needs to be dags)

good point indeed, maybe this should go under 1.x so we signal the changes to the core


Add docs for static layer param footprint_clearing_enabled

Docs for https://github.com/ros-planning/navigation2/pull/4282

On hold until main code PR is merged


module 'malaya.lexicon' has no attribute 'sentiment'

Hi, I tried to run the following lines:
sentiment_lexicon = malaya.lexicon.sentiment
sentiment_lexicon.keys()
and get error message as below:
AttributeError: module 'malaya.lexicon' has no attribute 'sentiment'
Please advice, thanks!

Alright, will look into it, thanks!

Bumped to 4.0.1,
pip install malaya -U

Reopen if still got the same problem.

Solved! Thanks 💯


Edits

updates

Docs Build status updates of commit 514975d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/importing-data.md
:white_check_mark:Succeeded
View

Community/large-lists-large-libraries-in-SharePoint.md
:white_check_mark:Succeeded
View

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


Feature/server side data factory

Hi, I just exposed one interface for Table which is responsible for Providing Data to results, Also I added small tweaks to existing TheGridDataFactory just for getting usage of new IDataProvider interface.

Thank you!


Fix empty search results logic

Closes #722

Current coverage is 53.99% (diff: 0.00%)

Merging #730 into master will not change coverage

@@             master       #730   diff @@
==========================================
  Files           105        105
  Lines          5523       5523
  Methods           0          0
  Messages          0          0
  Branches        664        664
==========================================
  Hits           2982       2982
  Misses         2427       2427
  Partials        114        114

Powered by Codecov. Last update 6d78d22...1da2c0e

@patrys fixed


Test: Do not set response headers after the response has started

TestServer is being updated to more closely mimic a real server, see https://github.com/aspnet/Hosting/pull/1256. Response headers can't be set after the response has started. This filter is fine for a few tests that avoid writing to the body, and none of the others check for this header.
This has recently been a source of test flakiness when it tries to modify the headers while the test client is enumerating them on another thread.

Also disabled broken anti-forgery tests due to https://github.com/aspnet/Mvc/issues/7040


chore(deps): upgrade numpy 1.18

I'd appreciate this merged. numpy-1.23 was from Dec 2022 and there are already packages requiring numpy-1.24, which is causing version conflicts.

It seems that there were some misalignment here, since we already support numpy>=1.16.0,<1.24.
https://github.com/ydataai/ydata-profiling/blob/develop/requirements.txt#L8
I will close this PR, since the requirement is already satisfied, we may open a new one to increase the upper limit.


Feature: Automatic recording of date of last status change of task

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
To be able to query on tasks of which the status was recently changed (not only completed, but also e.g. canceled, in progress, ...) add the option to automaticaly include/update a 'StatusLastUpdated' date to the task.
(Relates to question asked in Obsidian Forum: https://forum.obsidian.md/t/dataview-query-on-all-tasks-with-modified-status-possible-or-not/56105?u=nils72)
✔ Solution
In my daily note I have a basic dataview query that shows a list of all tasks that I completed on that day. However … during the day I also change the status of tasks to (e.g.) ‘In progress’ or ‘Canceled’. I would also like to also see those changes in my daily ‘Modified tasks’ overview.
Since no date is recorded for changes other than setting the status to ‘Completed’, It is not possible to get list of recently updated tasks. Or at least ... not without adding anual steps/macro's etc.
What I would like to be able to do is use a query like this:
TASK WHERE StatusChangeDate= this.file.day

(Where, obviously, the StatusChangeDate attribute currently does not exist)
❓ Alternatives
Some work-arounds/solutions were mentioned by other users. These involve using Templater, QuickAdd, regex and custoim dataview attributes. Could work, but adds steps to the workflow (and could be prone to errors)
📝 Additional Context
No response

@Nils-72 this seems to be an awesome idea. Could you elaborate on the issue you are trying to solve? Let's say you can query by the status change and see those changes ‘Modified tasks’ overview. What then?

This comment on that Forum post made me smile:

Since there appears to be no easy solution for this, and because I think others could also benefit from having this functionality, I have made a Feature request for the Tasks plug-in.
https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1746

It isn't easy in the Tasks plugin either! 😄
All the comments in that dataview Forum post apply to tasks, about Obsidian only tracking changes at the file level, not the line level.
And there are so many that users can modify their task lines, that Tasks will not know about.
Someone recently added the ability to track Created date via Tasks, but it will only work in very specific circumstances, where is the Tasks plugin that actually edits the line.
The same limitations would apply to this facility, if it were added.
I would like to wait and see how much user-support traffic the created field adds, before considering whether to add this field as well.

I would like to wait and see how much user-support traffic the created field adds, before considering whether to add this field as well.

This seems to be a very good point.

Hi @ilandikov and @claremacrae, this would be mainly be to show how 'alive' certain tasks are. Being able to sort, group, filter on the date of the last status change (either in directly via the Tasks plug-in or via Dataview) helps to see which tasks are actualy worked on even though they are not yet completed. Only having Created and Completed dates results in kind of losing the work done one a task in between those states, even though the new custom statuses make more detailed tracking already possible.
(And being able to show all those daily changes to tasks would help with my daily dose of Dopamine .... ;-))

Thanks a lot for this honest answer. So let's step out a bit from the Tasks plugin context. First of all, you wouldn't have had this issue if some of your tasks are were "alive", right? So the issue is here is that some tasks are "dead".
You can change the wording which may help (at least it does for me) like "do the thing" -> "do that thing arrrrggggg" (emotional formulations help getting things done).
But most importantly, you don't need to see the the "alive" tasks, you need the "dead" ones, the one you have not been doing. And I'm pretty sure the "last change" is not the solution here. At least because simply updating the task's last changed date will provide the sensation of getting things done, but we know the truth =)
Also in the next release you will have the created date. So if (today - created date > X day) { task is dead...} you see where it is going ;)
I hope that this philosophical spin-off gave you some helpful thoughts.

At least because simply updating the task's last changed date will provide the sensation of getting things done, but we know the truth =)

I had the exact same thought.

I currently don't feel that there is a very strong case for adding this field.
I'll leave this issue open for a little while to see if any extra motivations are suggested, though.

PS Thank you both for the very helpful discussion.

@claremacrae, thank you for your for looking at this idea and indeed having a good discussion. And @ilandikov, thank you for helping to make the idea a bit more crisp.
I am very much OK with the outcome. Might be that others have stronger use-cases which they can add in the future (thank you for keeping the discussion open for now). I fully acknowledge that my own task-tracking-needs might differ from others. This is - luckily - not only about getting my daily Dopamine-fix ;-). This is also about keeping track of (e.g.) tasks that were forwarded to others or that were canceled. Specifically for the forwarded tasks, knowing the forwarded-date will help to know when to start chasing colleagues. Looking at a KanBan way of working, the change-date is what could be used to track how long something has been in a specific column on a KanBan board (if those columns are state-based).
I want to thank you for all the hard work you put in the Tasks plug-in and for the open way of discussing any thoughts and ideas others - like myself - might have.

My feeling is that Obisidian (and Tasks as a derivative) were build without multi-user approach (which is the case for tools Notion & coda). So adding "delegation" functionality will require a tremendous effort. Maybe you can have it done with a combination of regex and tags. The final decision is not mine, but if you ask me, I would not even think to go in that direction at least right now.
As for the initial request, now we will have the created field, so maybe we can think about and option in the settings that this field is updated each time the task is edited. This seems to be an easier solution if needed at all.
I'm not sure how you use kanban in your workflow, I have setup something simple with "due, not scheduled", "scheduled later", "scheduled today or due today" and "done" using Tasks (Let's not debate the columns/states, this is a choice). If something is too long overdue you can see it be diff between today and due/scheduled date. You can also consider the start date field.

I currently don't feel that there is a very strong case for adding this field.
I'll leave this issue open for a little while to see if any extra motivations are suggested, though.

I still feel the same: this is not high enough priority to justify the considerable extra complexity that it would add to the Tasks code.
Closing as 'wont do'.


tested and verified with PCF 2.1.x

updated documentation to reflect support for PCF 2.1.x.

@shahramk Thanks!


Support for vite.config.js

Is your feature request related to a problem?
Some developers do not use TypeScript.
And some developers develop on their laptops but run inside containers.
We are a team of developers (a company) that do not use TypeScript and we do not have Node.js installed on our machines.
We develop using VS Code, but we run inside containers.
For us, vite.config.ts is full of errors. If we change it to vite.config.js as we do for our React projects, errors would disappear, but then we get this error:
container | > start
container | > vite --mode ssr
container |
container | ✘ [ERROR] Could not resolve "/Projects/SiteQwik/vite.config.ts"
container |
container | failed to load config from /Projects/SiteQwik/vite.config.ts
container | error when starting dev server:
container | Error: Build failed with 1 error:
container | error: Could not resolve "/Projects/SiteQwik/vite.config.ts"
container |     at failureErrorWithLog (/Base/Qwik/node_modules/esbuild/lib/main.js:1566:15)
container |     at /Base/Qwik/node_modules/esbuild/lib/main.js:1024:28
container |     at runOnEndCallbacks (/Base/Qwik/node_modules/esbuild/lib/main.js:1438:61)
container |     at buildResponseToResult (/Base/Qwik/node_modules/esbuild/lib/main.js:1022:7)
container |     at /Base/Qwik/node_modules/esbuild/lib/main.js:1134:14
container |     at responseCallbacks.<computed> (/Base/Qwik/node_modules/esbuild/lib/main.js:671:9)
container |     at handleIncomingPacket (/Base/Qwik/node_modules/esbuild/lib/main.js:726:9)
container |     at Socket.readFromStdout (/Base/Qwik/node_modules/esbuild/lib/main.js:647:7)
container |     at Socket.emit (node:events:513:28)
container |     at addChunk (node:internal/streams/readable:324:12)

Describe the solution you'd like
No solution yet. We just tolerate a lot of red squiggly lines.
Describe alternatives you've considered
We should be able to not use TypeScript at all.
We should be able to have vite.config.js instead of vite.config.ts
Additional context
No response

The package.json is pointing to vite.config.ts if you change the filename, update package.json scripts to point to the different file!


Reverse engineering PostgreSQL sequence-driven IDs

First, some details on PostgreSQL and auto-increment. PostgreSQL implements auto-increment columns via sequences (i.e. there's no equivalent for SqlServer's identity). A shorthand syntax for creating auto-increment columns is supported: if, when creating a table, a column's type is specified serial, a sequence with the name __seq is created, and the column's default value set to nextval('foo_a_seq'::regclass) (simple function call which gets the next value of the sequence).
 Much like the Sqlite provider, Npgsql currently checks OnAdd in NpgsqlMigrationsAnnotationProvider, and passes a Serial annotation to NpgsqlMigrationsSqlGenerator, which in turn creates the column with type serial.
However, when reverse engineering things get a bit tricky. Unlike with Sqlite and SqlServer, where identity/autoincrement columns are clearly marked in the table/column metadata, in PostgreSQL all you get is a column with a default value, which happens to get the next value of a sequence; there's no simple way to distinguish between id/OnAdd columns and columns which simply have a default value.
My current Npgsql reverse engineering provider parses the column's default expression to check whether it happens to correspond to the nextval('<table>_<column>_seq'::regclass), and if so, it sets ValueGenerated to OnAdd and removes the default value. This feels extremely hacky but achieves the expected results. I expect that reverse engineering an SqlServer table using sequences as an identity strategy will create similar issues.
Am I misunderstanding/missing something? Does the hack above seem reasonable?

Without seeing code, the hack seems reasonable. (Could you provide link to your branch?) If you find yourself needing to undo too much of what RelationalScaffoldingModelFactory provides, we can probably adjust this base implementation.
In SQL Server, I don't believe we can reverse engineer sequences directly into value generation on properties because sequences could be used by a number of columns and tables. (e.g. we won't be generating PropertyBuilder.ForSqlServerUseSequenceHiLo()).

Thanks for the feedback @natemcmaster. I just merged the reveng code to the Npgsql develop branch, the relevant code is here. Please feel free to comment!
Note that this code targets EF7 rc1, I will be syncing up with rc2 soon as well.

@roji curious, are additional steps needed to manage the sequence object on the database? E.g. what would happen if a table that depended on a sequence gets dropped and then recreated? Won't the sequence remain after the table is dropped and then cause a failure on the table creation? Or is PostgreSQL smart/resilient to this. Other than possibly (depending on PostgreSQL's behavior) having danglingb sequence objects in the database, the shortcut and the hack sound great.

@divega, PostgreSQL is smart about this: when a sequence is implicitly created as part of a serial column, the sequence is owned by the table; this causes the sequence to be automatically dropped when its owning table is dropped. This prevents dangling objects and failure on table drop/recreate. I'm not sure if the notion of a sequence being owned by a table is PostgreSQL-specific or not. You can read about serial and sequence in the docs, section 8.1.4.
Thanks for the feedback on this guys, I'll go ahead and close this but please don't hesitate to continue the conversation here.


fix: Show "interesting" doc relations on HTMLized page

Fixes #5344

Holding this for discussion at codesprint

Bah - we missed talking about this Saturday - help watch for a chance to talk about it while here this week

We should talk about this during 117.

Converting to draft until we have a chance to talk.

closing this as asleep - it can be pulled back out later if needed.


how to play video automatically?

set delegate:
    `videoPlayer.delegate = self`

then implement this
func playerReady(_ videoPlayer: YouTubePlayerView) {                 videoPlayer.play()             }


Check Package Manager for system updates

Describe the solution you'd like
It would be ace if LNXLink could you show whether systems updates are available (yum/apt).
Additional context
No updates are available:

APT  (> 2): apt list --upgradable | wc -l

If anyone knows the other package managers how to get the updates, let me know.

yay
That's all that has to be type to update EndeavourOS, based on Arch Linux to do a full system update.

Actually I need the commands that will show if there are updates or not which will be running every several seconds.
The result is a binary sensor that informs the user if there are updates or not.

checkupdates
That seems to be what you're looking for.

How about an option regarding how often it checks? Every several seconds sounds excessive. I wouldn't want it check too many times in one day.

That's great news
The yum/dnf version is:
yum -q updateinfo list updates | wc -l

Added this feature on the latest release 2023.3.1.


Add new credit card link in checkout not working

For poster v2.1
1 log in account with saved credit card info
2 go to checkout
3 go to payment
-->issue: cant use the add new credit card link in checkout

not present anymore as of posters v2.2