✨ group deprecated

Description
New option printDeprecated with the following supported values default | group | skip.
This PR only implements the group option for #735.
When printDeprecated is set to group:

deprecated types are grouped under a section deprecated in the sidebar
deprecated parameters are grouped in a collapsed section deprecated at the end of the Parameter section
deprecated types in relation are grouped in a collapsed list deprecated at the end of each relation section

Checklist

[ ] My changes follow the contributing guidelines of this project.
[ ] I have performed a self-review of my code.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my changes work.
[ ] New and existing unit tests pass locally with my changes.

Coverage: 94.776% (+0.07%) from 94.706% when pulling 7b61f8806d2378194f51c134accccdf30a714e12 on feat/735-groupDeprecated into 05770c87b4040a656f0c4b0616efe4f42559bd75 on main.


Downloadable ISO Files

Hi Guys,
thanks for your work, but may someone can explain why you offer only img.bz2 files for download and not simple iso/img files for direct work with (e.g. in proxmox environments)
May you can think about additive simplestupid iso releases. :-)
merci

Too much bandwidth, no verification. External (cloud) services will just use it to provide VMs to users which is not good.
Cheers, Franco

Hi,
so are you and your mirros that poor of bandwith? i mean other tools also distribute as plain iso?
Thanks

first of all bandwidth is not the issue. traffic volume on our main mirror is the issue for an open source project. also the traffic volume given by our mirror provides is not for others to benefit from.
if you want just host your own iso...


resolve current directory for processes

can you explain to me why this was needed?

I was trying to fix the issue @grosu raised. I'm wasn't sure windows correctly recognizes the "." as the current directory. As the issue was the extension, this is likely not a fix.

Yeah, that's not the issue.
The problem is when you are trying to run a command line from a java process. On linux and macs you can write kast a.txt, but on windows, you have to write kast.bat a.txt.

@radumereuta, thanks! I know, I saw the other issue.

For documentation purposes.

@dwightguth, please either merge or delete this PR.

It's possible this change might affect the PWD or CD environment variables but it would likely not have any other effect since the working directory of new processes would be effectively the same both before and after. Given this, and the fact that any effect if one exishs would be subtle and potentially non obvious, I'm going to close this without merging. If some aspect of the current behavior can be found to be undesirable and more accurately traced back to this, we will reevaluate


Swift3

Refactored All APIs for Swift 3

Hi @JourneyWind,
We won't be merging into master until the Xcode 8 GM drops. At that time, we'll be releasing both the swift2.3 branch as AF 3.5.0 and the swift3 branch as AF 4.0.0. This is our current plan, but we're trying to squeeze in some pretty big changes for AF 4.0 that may cause us to delay slightly.
Cheers. 🍻


 Great repo!

Thanks!


2.0.2 Failed to log-in with OAuth.

Steps to reproduce
const { Innertube, UniversalCache } = require('youtubei.js');

(async () => {
  const yt = await Innertube.create({
    // required if you wish to use OAuth#cacheCredentials
    cache: new UniversalCache()
  });

  // 'auth-pending' is fired with the info needed to sign in via OAuth.
  yt.session.on('auth-pending', (data) => {
    console.log(`Go to ${data.verification_url} in your browser and enter code ${data.user_code} to authenticate.`);
  });

  // 'auth' is fired once the authentication is complete
  yt.session.on('auth', ({ credentials }) => {
    console.log('Sign in successful:', credentials);
  });

  // 'update-credentials' is fired when the access token expires, if you do not save the updated credentials any subsequent request will fail
  yt.session.on('update-credentials', ({ credentials }) => {
    console.log('Credentials updated:', credentials);
  });

  // Attempt to sign in
  await yt.session.signIn();

  // ... do something after sign in
  console.log(yt.getLibrary())

  // You may cache the session for later use
  // If you use this, the next call to signIn won't fire 'auth-pending' instead just 'auth'.
  await yt.session.oauth.cacheCredentials();
})();

Failure Logs
/home/stas/Sync/Documents/yt-demo/node_modules/youtubei.js/dist/src/core/OAuth.js:185
            return __classPrivateFieldGet(this, _OAuth_session, "f").emit('auth-error', new Utils_1.OAuthError('Could not obtain user code.', { status: 'FAILED', error: err }));
                                                                                ^

OAuthError: Could not obtain user code.
    at OAuth.<anonymous> (/home/stas/Sync/Documents/yt-demo/node_modules/youtubei.js/dist/src/core/OAuth.js:185:89)
    at Generator.throw (<anonymous>)
    at rejected (/home/stas/Sync/Documents/yt-demo/node_modules/youtubei.js/dist/src/core/OAuth.js:6:65)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  info: {
    status: 'FAILED',
    error: TypeError: fetch failed
        at Object.processResponse (/home/stas/Sync/Documents/yt-demo/node_modules/undici/lib/fetch/index.js:199:23)
        at /home/stas/Sync/Documents/yt-demo/node_modules/undici/lib/fetch/index.js:928:38
        at node:internal/process/task_queues:140:7
        at AsyncResource.runInAsyncScope (node:async_hooks:203:9)
        at AsyncResource.runMicrotask (node:internal/process/task_queues:137:8)
        at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
      cause: ConnectTimeoutError: Connect Timeout Error
          at onConnectTimeout (/home/stas/Sync/Documents/yt-demo/node_modules/undici/lib/core/connect.js:131:24)
          at /home/stas/Sync/Documents/yt-demo/node_modules/undici/lib/core/connect.js:78:46
          at Immediate._onImmediate (/home/stas/Sync/Documents/yt-demo/node_modules/undici/lib/core/connect.js:119:9)
          at process.processImmediate (node:internal/timers:471:21) {
        code: 'UND_ERR_CONNECT_TIMEOUT'
      }
    }
  },
  date: 2022-09-09T10:38:53.626Z,
  version: '2.0.2'
}

Node.js v18.8.0

Expected behavior
User's library to be printed.
Current behavior
A crash.
Version
Default

For some reason, it worked on the second try. Should I close this issue?

There was probably a small disturbance on your network, given Undici reports a timeout.

Ok. Thanks!


[INLONG-4471][Manager] Fix json parse exception in Client

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-4471][Manager] Fix json parse exception in Client

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes #4471

Motivation
Modifications
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

This exception may be fixed by #4461, please re-check.

fixed by https://github.com/apache/incubator-inlong/pull/4461, close it.


Integration privacy changes

Codecov Report

Merging #854 into websocket-application-factory will decrease coverage by 0.05%.
The diff coverage is 99.06%.

@@                         Coverage Diff                         @@
##             websocket-application-factory     #854      +/-   ##
===================================================================
- Coverage                            99.38%   99.33%   -0.06%
- Complexity                            2124     2570     +446
===================================================================
  Files                                  322      361      +39
  Lines                                 6214     7399    +1185
  Branches                               319      426     +107
===================================================================
+ Hits                                  6176     7350    +1174
  Misses                                   5        5
- Partials                                33       44      +11

Impacted Files
Coverage Δ
Complexity Δ

...om/quorum/tessera/partyinfo/ResendRequestType.java
100% <ø> (ø)
1 <0> (?)

...action/exception/NoRecipientKeyFoundException.java
100% <ø> (ø)
1 <0> (?)

...va/com/quorum/tessera/p2p/TransactionResource.java
100% <ø> (ø)
5 <0> (ø)
:arrow_down:

...ava/com/quorum/tessera/enclave/PayloadEncoder.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...ava/com/quorum/tessera/sync/PartyInfoEndpoint.java
100% <ø> (ø)
11 <0> (ø)
:arrow_down:

...om/quorum/tessera/client/GrpcP2pClientFactory.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

.../tessera/transaction/TransactionRequesterImpl.java
100% <ø> (ø)
8 <0> (ø)
:arrow_down:

...a/com/quorum/tessera/partyinfo/ResendResponse.java
100% <ø> (ø)
4 <0> (?)

...va/com/quorum/tessera/partyinfo/ResendRequest.java
100% <ø> (ø)
7 <0> (?)

...java/com/quorum/tessera/data/StagingRecipient.java
100% <100%> (ø)
11 <11> (?)

... and 104 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 b236197...7324805. Read the comment docs.


[tools] Improve cf flink tool code

Related issue(s): #128
Description of changes

Check-list
Please make sure to review and check all of these items:

[ ] Are the updated classes tested?
[ ] Are the updated classes documented & properly formatted?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.

Pull Request Test Coverage Report for Build 875

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 94.34%

Files with Coverage Reduction
New Missed Lines
%

codefeedr-core/src/main/scala/org/codefeedr/pipeline/Pipeline.scala
5
92.75%

Totals

Change from base Build 870:
0.0%

Covered Lines:
800

Relevant Lines:
848

💛  - Coveralls


Forward headers to grpc hook java

I have an angular app which has an upload file feature. On the frontend side I'm using tus-js-client, the tusd server is v2.4.0 and the hook is grpc. From the frontend, I upload the file with the Authorization header but the tusd doesn't seem to pass them to the grpc server that I have written in Java. Is there an option that I should enable in order for the Authorization token to be passed to my java app?
Is there an option to enable more logging in tusd in order for me to see also the headers that are send from tus-js-client to it?
In my java app, I can see that on the grpc hook I have no headers.
Setup details
Please provide following details, if applicable to your situation:

Operating System: Linux
Used tusd version: v2.4.0
Used tusd data storage: disk store
Used tusd configuration: --hooks-grpc <IP_ADDRESS>:7333 --hooks-grpc-retry 5 --hooks-enabled-events "pre-create,post-create,post-receive,post-terminate,post-finish,pre-finish"
-upload-dir=/opt/tusd/uploads/
Used tus client library: tus-js-client

const upload = new tus.Upload(file, {
  endpoint: "http://<IP_ADDRESS>:1080/files",
  retryDelays: [0, 3000, 5000],
  metadata: metadata,
  onError: (error) => {
	  reject(error);
  },
  headers: {
	  Authorization: `Bearer ${metadata['authToken']}`,
  },
  onShouldRetry: function (err: any, retryAttempt, options) {
	  // omited
	  return true;
  },
  onProgress: (bytesUploaded, bytesTotal) => {
	  const percentage = (bytesUploaded / bytesTotal) * 100;
	  progressCallback(percentage.toFixed(2));
  },
  onSuccess: () => {
	  resolve(upload);
  },
});

upload.start();

Headers from the incoming requests are not attached to HTTP/2 requests for gRPC. Such an option to forward HTTP headers is only available for HTTP hooks (see the flag).
However, the headers from the incoming requests are also provided in the payload provided via gRPC: https://github.com/tus/tusd/blob/15f05c021c9770dc0cd2818aa4ef44b9e361f5b2/pkg/hooks/grpc/proto/hook.proto#L98-L99

Indeed. I've created a PR to handle those shortcomings. Please see https://github.com/tus/tusd/pull/1113
Only this line need to be modified:
https://github.com/tus/tusd/pull/1113/files#diff-02dbee1afbbc2d3019a64b4337548b6408b150fdafd5d9960145f692df7295b7R91


Array default values with shadow blocks

Lets you specify the inner shadow block for array shadow blocks. For example, in the animation API that @dannytech and @dharinirajah have been making the default value in the array should be images:

To use this, all you need to do is set the defl for the parameter on the API to the block ID of the shadow. For example,
    //% blockId=run_image_animation
    //% block="%sprite=variables_get(mySprite) animate frames %frames=lists_create_with with interval %frameInterval=timePicker ms"
    //% group="Animate"
    //% frames.defl=screen_image_picker
    export function runImageAnimation(sprite: Sprite, frames: Image[], frameInterval?: number): void {
}

(note the frames.defl line)

Also, I put two items in the array because three took up a lot of space and two is more obvious than one


ts-node version upgrade when using 'ng update @angular/core'

Versions
Using ng update @angular/core to update Angular 5.2 to Angular 6.0
Repro steps

Step - Using ng update @angular/core to update Angular 5.2 to Angular 6.0

Observed behavior
After the update, the ts-node is still "ts-node": "~4.1.0",
Desired behavior

I tested with the a totally new application created with ng new, the result is "ts-node": "~5.0.1", so I think when using the 'ng update @angular/core', the corresponding package version should also be updated.
Mention any other details that might be useful (optional)

It's true that a new project will have a new version of ts-node. We don't really update non Angular packages, such as ts-node. Those packages can have different public APIs and break in unexpected ways.


Add deprecated scope resolving

Adds the ability to register deprecated scope resolvers for controlling the deprecated scope.
Fixes #64

Thank you, this is nice. I'll release it when PHPStan 1.11 is released because the default branch here already requires it.

Did a few more things: https://github.com/phpstan/phpstan-deprecation-rules/compare/7d7840458a6df74a8890a0f1bf192c682148912d...27397cd9b1aff47824d5ccfe7e0c2a4ada8eb6d2

Woah! Thanks!

Just released: https://github.com/phpstan/phpstan-deprecation-rules/releases/tag/1.1.4

You are my hero 😀 ❤


Navigate through command history in command box

Added features:

Using up/down arrow keys to navigate through command history (similar to shell)
Created classes related to CommandHistory akin to AddressBook
Load past command and  history is saved at data/commandhistory.json

Note: constructor for ModelManager has been changed to include an immutable copy of the command history (public ModelManager(ReadOnlyAddressBook, ReadOnlyUserPrefs, ReadOnlyCommandHistory))
Tasks left:

[ ] Negative test cases for ModelManager needs to updated for CommandHistory
[ ] Tests for new classes created for this PR is still missing

Will resolve the possible conflicts from my side and merge this PR only upon merging #49, #51, #54.


multivariate API and dependency on nalgebra

We support multivariate distributions and nalgebra is part of our API. We'll be merging #209 and this will be a breaking change. The new methods clearly rely on nalgebra, but it might be more implicit with other methods. I think we can either make it clear that args and outputs for/from these distributions will need the dependency or find some way to opt out of using the same or any nalgebra.
Options I can think (in order of what I think is increasing complexity)

clearly state the dependency in multivariate documentation
non-default feature for multivariate, would also generally address not compiling a linear algebra library
write API using standard types, this may not be possible without removing the benefits of #209
write API to support different numeric algebra backends, nalgebra is broadly used, but there are others

See #199 #209

non-default feature for multivariate, would also generally address not compiling a linear algebra library

FYI: Dirichlet currently depends on nalgebra::DVector and Multinomial has nalgebra types in its public API (e.g. Multinomial::mean). So those would need to be addressed too.
One more issue I'd like to raise is one of developer ergonomics. There are a few functions which take nalgebra types as parameters (e.g. new_from_nalgebra), but library users cannot easily use these without both

adding nalgebra as a dependency to their own project and
figuring out any nalgebra version mismatches on their own, or asking for a version bump in statrs

It might make sense to reexport some nalgebra types from statrs to make this less of a problem.

write API to support different numeric algebra backends, nalgebra is broadly used, but there are others

If it's just for the public API, this could be done with extension traits, hidden behind a feature flag or maybe in a different crate.

It might make sense to reexport some nalgebra types from statrs to make this less of a problem.

This doesn't sound like a heavy lift, but I don't know how much we'd need to reexport. Would be good to look into.
A question I have: would that mean the syntax in user code would look like this?
use statrs::distribution::MultivariateNormal;
use statrs::appropriate_module_name::{...}; // whichever may be needed
use nalgebra as na;

let mvn = MultivariateNormal::new_from_nalgebra(statrs::appropriate_module_name::vec!(_), statrs::appropriate_module_name::matrix!(_));

Do you have ideas for what you'd choose for traits we'd create to connect to a general linear algebra library type? If not, I think we start small and see what mileage we get with re-exporting. Hopefully most users (and us) will be using an updated nalgebra.

This doesn't sound like a heavy lift, but I don't know how much we'd need to reexport. Would be good to look into.

Well, it could be a minimal reexport (vector/matrix types plus maybe convenience macros) or just a re-export of the entire nalgebra library. This depends on the other API decisions too IMHO.

A question I have: would that mean the syntax in user code would look like this?
...

Generally, yes. I don't think there's a "right" way of doing these re-exports, but I could think of
use statrs::distribution::MultivariateNormal;
use statrs::nalgebra::{DMatrix, dvector}; // or whatever type it may be in the end

let mvn = MultivariateNormal::new_from_nalgebra(dvector!(...), DMatrix::from_column_slice(...));

or (if you're already importing types from another nalgebra)
use statrs::distribution::MultivariateNormal;
use statrs::nalgebra as statrs_na;

let mvn = MultivariateNormal::new_from_nalgebra(statrs_na::dvector!(...), statrs_na::DMatrix::from_column_slice(...));

Do you have ideas for what you'd choose for traits we'd create to connect to a general linear algebra library type?

The way I've seen this in the wild is often just a 1:1 extension trait per type. And in this case, I guess one trait per type per linalg library. So roughly:
// mod nalgebra_ext
pub trait MultivariateNormalNalgebraExt {
  pub fn new_from_nalgebra(...) -> MultivariateNormal;
  // ...
}

// ...

// mod ndarray_ext
pub trait MultivariateNormalNdarrayExt {
  pub fn new_from_ndarray(...) -> MultivariateNormal;
  // ...
}

// ...

and then their respective implementations. With some structuring and/or a prelude, this could allow users to
use statrs::ndarray_ext::*;
use statrs::distribution::MultivariateNormal;

MultivariateNormal::new_from_ndarray(...);

This would still mean that there's a dependency on nalgebra/etc. at some point, but I'm not sure I see a better way at the moment.


Why I still can't pass test.py test when I use 80GB A100, it will still explode memory.

This GPU does not run any programs

please check this issue https://github.com/longzw1997/Open-GroundingDino/issues/7
This PR fixed this issue https://github.com/longzw1997/Open-GroundingDino/pull/10

Hi, @lanhas is your model training running normally? I would like to know the current status of the latest code.

Hello， @BIGBALLON Thank you very much for your work! The current code works fine, and it successfully trains on our small dataset. Of course, this is just a running test, without evaluation. I will contact you in time if there are any questions during the follow-up training.
The environment information is pytorch2.1.0-cuda12.1, node=1, gpus=8.


Skip duplicate lines

hexdump will skip duplicate lines and output a single * for any number of duplicate lines. Would you accept a PR that implements that behaviour?
0000280 04 00 00 00 2a 00 00 00 00 00 00 00 00 00 00 00
0000290 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
*
0000300 00 00 00 00 00 00 00 00 ac 0f 00 00 01 00 00 00
0000310 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
*
0000330 0c 00 00 00 38 00 00 00 18 00 00 00 02 00 00 00
0000340 01 fa e4 04 00 00 01 00 2f 75 73 72 2f 6c 69 62

That feature was merged with #59 or are you thinking about something else?

Whoops! I'm on version 0.3.0, that was the problem. Thanks :)


chore: bzlmod support

Sorry, just a draft to get a CI status - needs a bunch more work...

I'm also happy to push to this PR to get it green if you like. I have some bzlmod-ifications for this that I've been meaning to bundle up.

Thanks, I had it partly done last night though - I think it's ready.


Spike: How do we ensure access keys are rotated every 90 days or less

User Story
As a security engineer
I want to ensure AWS access keys are rotated every 90 days or less
To reduce the likelihood that they are exploited
https://docs.google.com/document/d/1ZOrGgOjApNo61SD2WAqAJgQScC7feoqP/edit
User Type(s)
Security engineer
Value
Reduce risk around credentials.
Questions / Assumptions / Hypothesis
This is currently monitored in security hub, but we could add something to security baselines to enforce this.
The credentials in the platform now are IAM users, member CI/CD users and the testing user.
Can we implement this through the guidance provided here? https://docs.aws.amazon.com/prescriptive-guidance/latest/patterns/automatically-rotate-iam-user-access-keys-at-scale-with-aws-organizations-and-aws-secrets-manager.html
Alternatively, is this more appropriate for us? https://aws.amazon.com/blogs/mt/managing-aged-access-keys-through-aws-config-remediations/
Definition of done

[ ] Spike timeboxed
[ ] Options discovered
[ ] Options discussed with team
[ ] Decision reached
[ ] Story raised (if appropriate)

Reference
How to write good user stories

@dms1981 to refine

I think the need for this will reduce as we increase the use of OICD, but some bespoke additions like SES email users will still be in scope.

Checked, this still needs to be implemented, although access keys for superadmins are deleted they are not for collaborators (only console access)

https://docs.google.com/document/d/1OU4mwCVqUFN-MwHeDXCaxc17I3ytOVWDFfG3IOEEpZg/edit?usp=sharing

From what I can tell there isn't a solid conclusion from the linked document. I couldn't see any evidence of a discussion with the team in Slack, nor does this issue have any comments on outcomes or related stories for implementation.

So a conclusion has been added, I didn't want to assume route 1 2 or 3 without a team discussion, and I was off sick before that could happen. Hoping after stand up today there is time, and then I can edit the conclusion if needed and create a ticket from what is agreed.

Its actually already done here - https://github.com/ministryofjustice/modernisation-platform/blob/main/scripts/iam-monitoring/collaborators-inactivity-monitoring/disable_user_credentials.sh


Feature: Manage remote docker

It would be awesome to be able not only to connect to localhost, but to remote hosts as well.
I understand that this is a bit unusual and not an intended use for kitematic, but it already uses remote docker daemon for container management. Thus it would be very handy to be able to run local containers as well as remote ones using the same interface.
Limitations I can think of:

Shared folders. Would require NFS / SSHFS / some other remote file system
Probably some hacky container management if any is used

We've discussed this functionality as well, but as you mentioned a lot of the 'localhost' functionality would need to be hidden/updated to handle the remote hosts.
I'm open to suggestions on how this would be done.

There are two possible ways:

List of nodes with only one active node. You can switch the active node, but you see only containers of active node. Similar behaviour is used by Docker CLI.
Multiple active nodes in the same time. It's more complicated, but it's also more powerful. You see all containers that can belong to different nodes.

One can start by the first way, keeping second approach in mind. Kitematic should have some screen for nodes management (list + CRUD operation). Interface should have some reference which node is currently active.

I want this Feature, so I searched and entered in this issue.
I develop some data program in Win7, using remote jupyter server(Toree/Scala/pySpark/pandas). So I need this Feature to manage remote Docker Engine and container.
Please add this Feature, thank you very much.

I also want this feature.
I am about to release a really easy to use docker container.   A JSON wiki.
https://pythonlinks.info/json-wiki
My problem is that there is a shortage of easy-to-use docker hosting facilities.
Most everyone is into large company orchestration.  I want an app store.  Small company hosting of docker apps.  Sloppy.io was really easy to use, but now they seem to be more targeting complex container orchestration.  Maybe azure has a simple solution.
Then I found this product.  Look at the demo.  Install it, click on the container you want.
I went to the docs, and Docker.com now recommends some new service.  Read the docs and we are back to a stone age teminal app.
I am about to start offering Pug.js trainings.   For my classes, everyone has a Mac, they can use this for learning.  But what about production.  They need to host on a server.
So something like this for managing a hosting service would be just awesome.  Plus there is a business model to drive it.   Give this open source software for free, but charge for the hosting
Is there any chance that any hosting provider would offer this as a user interface.  That would be beyond awesome.


Spike/dockerizing

This PR aims to dockerize the UKHPI app. It already has a lot of suggestions previously made for https://github.com/epimorphics/ppd-explorer/pull/134

This is currently on pause and waiting for changes on how the DSAPI gem handles the JSON response coming back from SAPINT


install via brew bug.. on yosemite mac

iMac-2:dist 2 greg$ consul
-bash: consul: command not found
iMac-2:dist 2 greg$ brew install caskroom/cask/brew-cask
==> Installing brew-cask from caskroom/homebrew-cask
Error: brew-cask-0.47.0 already installed
To install this version, first brew unlink brew-cask' iMac-2:dist 2 greg$ brew cask install consul ==> Downloading https://dl.bintray.com/mitchellh/consul/0.4.1_darwin_amd64.zip Already downloaded: /Library/Caches/Homebrew/consul-0.4.1.zip Error: no implicit conversion of nil into String Please report this bug:     https://github.com/caskroom/homebrew-cask/issues /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/digest.rb:44:in initialize'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/digest.rb:44:in open' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/digest.rb:44:in file'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/digest.rb:29:in file' /usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/download.rb:41:in block in _check_sums'
/usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/download.rb:39:in each' /usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/download.rb:39:in _check_sums'
/usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/download.rb:32:in perform' /usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/installer.rb:72:in download'
/usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/installer.rb:49:in install' /usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/cli/install.rb:20:in block in install_casks'
/usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/cli/install.rb:17:in each' /usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/cli/install.rb:17:in install_casks'
/usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/cli/install.rb:6:in run' /usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/cli.rb:81:in run_command'
/usr/local/Cellar/brew-cask/0.47.0/rubylib/cask/cli.rb:121:in process' /usr/local/bin/brew-cask.rb:42:in <top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require'
/usr/local/Library/brew.rb:58:in require?' /usr/local/Library/brew.rb:142:in '
iMac-2:dist 2 greg$ brew cask install consul

You’re using an old version of homebrew-cask. Please brew update && brew upgrade brew-cask && brew cleanup && brew cask cleanup and try again.
Please always do this before opening new issues.
See also #10224.

thank you Vítor


Wrong layout generated by render() to text

Bug description
The default OCR model works very well, but the render() algorithm which converts coordinates to text positions is very buggy.
This causes lines originally placed at the top to be positioned between other lines at the bottom, making the overall result unusable for LLM inference.
I wonder if you have considered reusing the algorithm implemented in Tesseract. They probably solved the same problem many years ago.
And I also wonder why the Tesseract team is not integrating the doctr engine into Tesseract :-)
Good job! You are leading the OCR leaderboard.
I attached a sample .PDF file and a snippet to reproduce the problem.
I checked other similar inactive issues, so I'm afraid rendering to text is currently not a hot topic :-(
...but how are we suposed to feed our hungry LLMs?
Code snippet to reproduce the bug
import argparse
import os
import json

from doctr.io import DocumentFile
from doctr.models import ocr_predictor

def convert_pdf_to_txt(input_pdf, output_txt):
  """
  Converts a PDF file to a text file using DocTR OCR.

  Args:
      input_pdf (str): Path to the input PDF file.
      output_txt (str): Path to the output text file.
  """

  print("Load pre-trained OCR model")
  model = ocr_predictor(pretrained=True)

  # Ensure input PDF exists
  if not os.path.exists(input_pdf):
    raise ValueError(f"Input PDF file '{input_pdf}' does not exist.")

  # Load the PDF document
  try:
    doc = DocumentFile.from_pdf(input_pdf)
  except Exception as e:
    raise ValueError(f"Error loading PDF '{input_pdf}': {e}")

  # Perform OCR and extract text
  try:
    result = model(doc)
    #exp = result.export()
    #text = json.dumps(exp)
    text = result.render()
  except Exception as e:
    raise ValueError(f"Error performing OCR on '{input_pdf}': {e}")

  # Write extracted text to output file
  with open(output_txt, 'w', encoding='utf-8') as f:
    f.write(text)

  print(f"PDF '{input_pdf}' converted to text file '{output_txt}'.")

if __name__ == "__main__":
  parser = argparse.ArgumentParser(description="Convert PDF to text using DocTR OCR")
  parser.add_argument("input_pdf", help="Path to the input PDF file")
  parser.add_argument("output_txt", help="Path to the output text file")
  args = parser.parse_args()

  convert_pdf_to_txt(args.input_pdf, args.output_txt)

Error traceback
No error
Environment
Linux, conda, python 3.9
Deep Learning backend
Default model.
test-ocr.pdf

Hi @kripper :wave:,
Thanks for reporting :)
The issue here is that page 2 & 3 contains small rotations could you give it a try with passing assume_straight_pages=False to the ocr_predictor instance ? :)

Predictor initiated with:
model = ocr_predictor(pretrained=True, assume_straight_pages=False)
But the probelm persists on page 1:
Notario y Conservador de Bienes Raices Licanten Vilma Beatriz Navarro
<--- "Reyes" SHOULD GO HERE
Certifico que el presente documento electronico es copia fiel e integra de
CERTIFICADO otorgado el 26 de Abril de 2024 reproducido en las siguientes

Reyes <-------- BUT WAS PLACED HERE

paginas.

Also note that the OCR'ed page (page 1) is a clean PDF page.
The second page is an image and assume_straight_pages could help here.

From what I have also seen, sometimes the models predict lines in the wrong block, even though their coordinates are correct. This is why the render() method returns the text mixed up, as it is only a bunch of nested for loops going over all the pages, blocks, lines and words. To get over it I did this, although it kind of messes up the line breaks, it preserves the order:
result = model(doc)
text = ""

def sort_by_coordinates(element):
    return (element.geometry[0][1], element.geometry[0][0])

for page in result.pages:
    line_list = []

    for block in page.blocks:
        line_list.extend(block.lines)

    sorted_lines = sorted(line_list, key=sort_by_coordinates)

    for line in sorted_lines:
        for word in line.words:
            text += word.text + " "
        text += "\n"

    text += "\n"

@kripper Have you already tried to disable block and/or line resolving ?
https://mindee.github.io/doctr/using_doctr/using_models.html#two-stage-approaches
resolve_blocks=False
resolve_lines=False

@kripper Have you already tried to disable block and/or line resolving ? https://mindee.github.io/doctr/using_doctr/using_models.html#two-stage-approaches
resolve_blocks=False resolve_lines=False

It's now mixing blocks multiple times per line.
What about taking a look at Tesseract's implementation?

@kripper Have you already tried to disable block and/or line resolving ? https://mindee.github.io/doctr/using_doctr/using_models.html#two-stage-approaches
resolve_blocks=False resolve_lines=False

It's now mixing blocks multiple times per line.
What about taking a look at Tesseract's implementation?

Sure :)
Do you have a direct reference to the code or algorithm ?

Do you have a direct reference to the code or algorithm ?

No, but I will research tomorrow.

Have you tried existing tools to convert doctr's HOCR output to text? There are many. Tesseract probably is also using some of them.

Have you tried existing tools to convert doctr's HOCR output to text? There are many. Tesseract probably is also using some of them.

Yeah you can use doctr's XML/hocr output to create PDF/A files for example with OCRmyPDF

sometimes the models predict lines in the wrong block

The synthesized page looks fine. Identifying lines shouldn't be that difficult IMO.

sometimes the models predict lines in the wrong block

The synthesized page looks fine. Identifying lines shouldn't be that difficult IMO.

Depends on the documents layout ^^ And there is a lot of difference (rotated, block text, etc.)

Especially with rotations all other open source tools (paddleOCR / tesseract / easyOCR) fail also:
Possible way to go: https://arxiv.org/abs/2305.02577 --> investigate


Align to new Scala 2.13 collections

See lampepfl/dotty#818

Here is a first implementation of Martin Odersky: https://github.com/lampepfl/dotty/blob/master/src/strawman/collections/CollectionStrawMan1.scala

Martin Odersky wants to re-design Scala's views, as stated in a recent talk.
The according dotty issue can be found here.
This Scala debate is a side-track of re-designing the collections: https://groups.google.com/forum/#!topic/scala-debate/Z1YH_0Hgu5A
Bill Venners slides presented at ScalaWorld 2015: http://www.slideshare.net/bvenners/collections-prettyslow
Daniel Spiewak wrote down a first idea of redesigning Scala's collections in his Collections Redesign Proposal gist.
Lex Spoon's thoughts: http://blog.lexspoon.org/2015/10/initial-input-on-scala-collections.html
Paul Phillips idea of bringing implicit methods into scope (e.g. map, flatten => flatMap): https://github.com/paulp/psp-std/blob/master/api/src/main/scala/Trait.scala

The discussion on the dotty Github repo has ended. The discussion now takes place here:

Scala Debate > Scope of redesigned collections
Mirror on QnaList (contains useful comments)

Beside discussion there has not been made much progress on this topic (my subjective view).
See https://groups.google.com/d/msg/scala-debate/Z1YH_0Hgu5A/ZsswMjg9BAAJ
I will close this ticket for now. We will see if there is anything to do for us when a new Scala collection library is released. Until then we just observe...

See SCALA 2.13 COLLECTIONS REWORK

This should be reopened as mentioned on Twitter.
The full migration guide for scala is available now here: https://docs.scala-lang.org/overviews/core/collections-migration-213.html
Some key takeaways for Vavr (not a complete list!):

Traversable no longer exists (-> Iterable)
Stream is now LazyList
new type ArraySeq (sequence backed by array)
new types MultiSet, SortedMultiSet, MultiDict, SortedMultiDict
groupMap and groupMapReduce on collections
distinctBy and partitionMap

The mentioned document could/should be scanned for further issues with regard to vavr

I like the idea to rename Stream to LazyList. Stream can be a bit confusing to many Java developers because of the overlap with the completely different type in Java.

In Scala, Stream & LazyList exist both, whereas Stream was deprecated.
I'm not sure if the head can be easily made lazy in Java (without significantly increasing the mem footprint) because Java has no notion for lazy evaluated parameters.
I will reopen this issue and postpone it to 2.0 because we already re-designed the type hierarchy. Additions like new collections or new functions do not need to be part of 1.0.

Isn't it just the matter of accepting a supplier instead of a concrete value for the head element?

Maybe, that was also my first thought. On the one side, it is more heavy weight. On the other side I think we experimented with it a few years ago and came to the conclusion that it was problematic to some extend. I can’t remember atm.


fix: incorrect battery percentage on Pinebook Pro

As discussed on #1098 , little fix for pinebook pro laptops.

This breaks the battery display on my system. Now it's showing half the percentage it should be. That's weird, because I looked at the code changes and there's no way it should have affected the battery calculation with my battery.


Experiment with adding source-build infrastructure to a repo with Arcade

This issue tracks doing some exploratory work on moving the low level infra into Arcade, in a way that feeds into our goal of validating source-build and producing a tarball during the Microsoft product build and repo PR validation.
The main topics I want to investigate:

Adding source-build functionality to the Arcade SDK.

Can we jump to the point where instead of cloning source-build, clone the target repo (Core-SDK) at the desired commit and do some "build from source" or "create a tarball for me" gesture?

Taking inputs in a way that's naturally compatible with Darc/Maestro++/BAR.
Packaging outputs in a way that can be cached and used in downstream source-builds.
Maintaining a potential way to have per-repo patches.

Ideally the Microsoft build will respin when source-build doesn't work. However, can we still add/maintain patches if we're unable to do this? Is there an analogous flow for emergencies?

Notes:

Having source-build's darc clone manually manipulate .git/modules to store Git data is probably ok to ensure git clean -xdf doesn't make you have to redo a ton of cloning time, however I don't think it's reasonable to roll that out to users that don't normally interact with source-build due to large size and cleanup difficulty.
Similar to signing, build, etc., Arcade can implement the basics, but repos need to add eng/SourceBuild.props/targets for the custom stuff we're handling in repos/*.proj today.
Patching could be represented by a separate branch in the repo. We can force the commit hash to match the official build, if need be with something like git checkout --detach; git reset --soft <official>. This is effectively the same as a patch, but raises awareness and is more intuitive to work with as a repo contributor.
darc clone doesn't quite do what we need here. Right now, source-build has a dependency for every single repo (recursively) necessary to source-build, and relies on auto-updates to "fix up" the info based on coherent parent dependencies. We can copy these dependencies to the target repo, however that's a significant amount of unnecessary info. darc clone should ideally be able to figure it out instead.
The "bootstrap Arcade" is the version that the repo depends on. While source-building the repo, you can download a source-built Arcade from intermediates, and that works fine for the upstream repos, however for the current repo it's too late to switch the version of the tooling (MSBuild, SDK, Arcade) being used at that point because we're already running the process. We need some workaround: recursive build?
We may want to consider doing some housekeeping on the source-build side to make it easier to push the way source-build orchestrates things into Core-SDK/Arcade. In my prototype, there are some common themes: stuff is running without the Arcade SDK, so some properties aren't available;   artifacts are mostly in bin, not artifacts, so it's not trivial to redirect; there is a lot of old code; and some info is set up in the build scripts rather than in MSBuild logic. The work would also likely be trivial to apply to 3.1 servicing to help out there. (There are also less trivial things like incrementality and isolation that could be improved.)
global.json resolution throws a wrench in some things. For example, when using Arcade's Build.proj, that's located in the package cache. I think that therefore, the SDK looks for the global.json in the hierarchy of the package cache, so it matters where it's located. What I observed was that two versions of Arcade are used at once in this scenario, which causes problems.

I added a target that copies the global.json into the package cache dir before each subrepo build for now, and we can probably use the source-build SDK resolver to forcefully resolve this situation if necessary.

I hit a decent milestone and took a snapshot branch:
https://github.com/dotnet/core-sdk/compare/v3.1.102...dagood:sb-proto-sb-in-eng
This command builds upstreams using the mostly ordinary source-build infra I copied into eng/source-build, then builds Core-SDK using those assets (including source-built Arcade):
./build.sh -c Release /p:FromSource=true /p:BuildUpstreamRepos=true /bl:artifacts/msbuild.binlog /nr:false /p:KeepUpstreamGitDataInDotGit=true

Source-built outputs end up in artifacts/source-build/repo/packages/Release/Shipping.
The Core-SDK-side source-build is the most interesting bit for this prototype, and I have the targets somewhat separated into different eng/SourceBuild*.[props/targets] files. All the pre-build infra is driven through eng/Build.props. I think this approach could be migrated to Arcade cleanly.
To get the source-built SDK loaded, it uses the recursive build approach I have a discussion thread about at https://github.com/dotnet/arcade/issues/5116. It's similar to (1) at https://github.com/dagood/source-build/blob/sb-in-official-doc/Documentation/planning/arcade-powered-source-build/in-arcade.md#supporting-source-built-sdks but doesn't require recloning the entire Core-SDK repo (the current checked out sources are used).
A scenario that might still call for a fresh clone is if the repo doesn't properly put its intermediates in artifacts/ (pollutes the source dir) and a dev expects source-build to work from an unclean repo state.

The next big rock I'm looking at to let us actually reasonably merge this stuff into Core-SDK is teaching darc clone how to clone the dependency graph with forced coherency. (https://github.com/dotnet/source-build/issues/1495.) The prototype copies the hand-crafted expanded graph from source-build into core-sdk, which would not be good to actually use for 5.0.
The last big thing looming that hasn't been prototyped yet is the intermediate nupkg structure and infra. ("Packaging outputs in a way that can be cached and used in downstream source-builds" above.) This is key for reducing build time so source-build can reasonably happen during validation builds.

I'm going to close this issue as the Core-SDK prototype is completed, and track remaining work (some I'm working on now) in new issues:

teaching darc clone how to clone the dependency graph with forced coherency. (#1495.)
the intermediate nupkg structure and infra: https://github.com/dotnet/source-build/issues/1543


Compatibility with different serializers and their underlying schemas.

Recently many frameworks and projects started to use custom transport that come with strongly typed schema. The main ones out there:

Avro (Used in big data shops)
Protobuf

Avro is an popular data serialization system (with many benefits over the JSON)
https://avro.apache.org/docs/current/
Protobuffers it an binary format and competitor with Avro.
https://developers.google.com/protocol-buffers
What can be done
Clarify how Rapid can coexist with this format and avoid specifying schema twice.

CC @ralfhandl


Update docstrings

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

Project coverage is 100.00%. Comparing base (c78617e) to head (f6f2013).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #84   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           49        49
  Lines          897       897
  Branches        71        71
=========================================
  Hits           897       897

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


aws-cpp-sdk does not support IPv6 type address for S3 endpoint

Describe the bug
Background: I am using MinIO( S3 compatible storage ). The address for endpoint is in IPv6 format:
https://[[240b:c0e0:102:553e:b4c2:2::]]:3000
This does not work. Error message: Invalid DNS Label found in URI host
However when I use the DNS name for it: uhn7klr2crbct001.rmnkiba.local the functionality works as expected.
Expected Behavior
The sdk should work with IPv6 type addresses.
Current Behavior
Using IPv6 address for S3 endpoint, like:
https://[[240b:c0e0:102:553e:b4c2:2::]]:3000
Throws error message:
Invalid DNS Label found in URI host
Reproduction Steps
Any IPv6 address used as S3 endpoint.
Possible Solution
No response
Additional Information/Context
No response
AWS CPP SDK version used
1.9.220
Compiler and Version used
gcc (GCC) 4.8.5 20150623
Operating System and version
Linux 3.10.0-1160.31.1.el7.x86_64

Hello @kv83821-yb ,
Thank you very much for your submission.
Unfortunately the behavior you are experiencing is expected as the aws-cpp-sdk does not offer support for IPv6 address in S3.
There is a feature-request open here for IPv6 address support in S3. You can add a thumbs up or a comment to it to showcase your interest in this feature which will help us with feature-request prioritization.
I will also bring this up for discussion with the team.
If that is ok with you, I will close this issue as a duplicate and invite you to follow status update for this feature in #1857 to prevent duplicate submission.
Thank you very much again for your submission, your feedback is extremely valuable to our team.
Best regards,
Yasmine


color changes

Changed some colours in the css file
Issue no; #41
Add some more good colours to the website to increase UX. #41

@YadavAkhileshh please help me to raise a good Pull request so that it can be merged into main branch

Please update the label so that It can be counted in hactoberfest

Please update the label so that It can be counted in hactoberfest


ntr: abolished protein localization to meiotic kinetochore during meiosis I   + .....

term: abolished protein localization to meiotic kinetochore during meiosis I
def:
A cell phenotype in which the localization of a protein to the kinetochore of a chromosome is abolished during the first meiotic nuclear division.
descendent of FYPO:0001375

just spotted
decreased protein localization to kinetochore during meiosis I
so maybe need a grouping term if  "abolished' can't be a descendent of 'decreased'

also
normal protein localization to meiotic spindle pole body during meiosis I
descendent of
FYPO:0003541	normal protein localization to meiotic spindle pole body

abnormal protein localization to kinetochore during meiosis FYPO:0004762
abolished protein localization to kinetochore during meiosis I FYPO:0004763
normal protein localization to meiotic spindle pole body during meiosis I FYPO:0004764
I won't add any other grouping terms for now, because there may be some relief on the abolished/decreased front in PATO ... at least, they were considering it when I asked a while back. I've prodded again to see if they've made a decision/forgotten about the question/etc.
https://github.com/pato-ontology/pato/issues/46

edit file: 71c1d4c3495fa0675989a8523d3bcec4357fa04f
release: 7cda3fe634db6b3967c97dd04819d944aab4629b


Breakpoints not getting hit.

Hi Martomo
I am using sublime3 and its built in xdebug plugin. The zf2 application code stepping happens only in the initial bootstrapping module but doesnt stop in controllers.
Any reason why this could be ?
I also see client connection timeout errors in xdebug-debug.log file
Followed all steps as mentioned by you including path_mapping etc.
thanks

I can't make it too, but last time I can. So weird.


New options to filter forms in Equipment slots

There is this option to filter for specific forms to allow or deny in the equipment displays.
Suggesting new options for it. The original suggestion was to use Keywords.

There already is a condition filter for last equipped, but temporary forms are ignored in general. They can be deleted at any point, their ids assigned to another form - too risky and problematic to deal with. It won't let you display temp forms or detect them in inventory (maybe this last part could be changed at some point)
On December 1, 2022 2:34:02 AM GMT+01:00, GiraPomba @.***> wrote:

Hey. Just wondering.
Could this system for filtering items be implemented in the custom displays, along the Last equipped option?
I was trying to make a display for player crafted potions and I couldn't make a link with the potions in the inventory, afaik the form ids are created in the savefile.
--
Reply to this email directly or view it on GitHub:
https://github.com/SlavicPotato/ied-dev/issues/16#issuecomment-1333022486
You are receiving this because you commented.
Message ID: @.***>


Test a simple replicaset setup

This PR add an additional set of server-spec tests.
Like the ones for replica, but they test the simplest replica setup using just the mongodb::globals, mongodb::server and mongodb::client classes. These simulate the use of PuppetDB by fully setting up the first node, then expanding the set with the new member.
Tests are performed with and without authentication enabled.

Not sure how to get the test to actually run...

If you look at https://github.com/voxpupuli/puppet-mongodb/blob/master/spec/acceptance/replset_spec.rb#L8 the if test ensure that the test have to be run in an environment ( beaker controlled ) with more than one host.
At https://github.com/voxpupuli/puppet-mongodb/blob/5d80bfa2dcaa65dcad6e3c8038bde540d21a0179/spec/acceptance/replset_spec.rb#L47 there is code to run puppet apply on all hosts.
I suspect the entire file has been used with a beaker hosts file at some time.
As gha now auto generates host files the multihost beaker set file may have been deleted at som time.
( its not supported in gha I think )
I think we need to figure out how to create such a beaker set file to at least be able to run these tests locally.

I get the if, I started out with a copy of the acceptance tests for a replicaset, but these new ones doesn't seem to be processed. Currently no clue on how to proceed.

There is some tutorials here https://github.com/voxpupuli/beaker that may get you started on beaker :)

Started looking at nodeset definitions. See #743

Closing, see https://github.com/voxpupuli/puppet-mongodb/pull/729#issuecomment-2061175472


Adds Android support

This PR adds Android support for all of the documented Orientation methods.
Tested with react-native 0.16.0.

@yamill If you can take a quick look. We're looking to add this for our own RN app. :)

@jaysoo great work :+1:

@skv-headless Thanks :)

Thanks will push to npm shortly :+1:
good job @jaysoo !


Handlers delegation in polymer 1.0

Hi,
Could you please have a look:
http://stackoverflow.com/questions/30997153/handlers-delegation-in-polymer-1-0
Best regards, Eugene.

Responded: https://stackoverflow.com/questions/30997153/handlers-delegation-in-polymer-1-0/31010099#31010099


cofactor function

Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #55
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings
cofactor function added

you are correct , when this submatrix is passed to the determinant function then we will get the cofactor right

So you haven't calculated or called determinant in this right? You could add that

i have called cofactor, which is giving out the submatrix inside determinant , which in turn is used to calculate determinant , there by optimising the determinant function also

I think it's the other way round mathematically but since you have gotten the determinant optimized and the cofactor correction is very minor I'll take this

!bounty 40


Jakarta namespaces

Thanks for the contribution! This is something, I had on my mind as well.
What about the "Temporarily disable import order Checkstyle rules" commit? Can it be enabled again or is it due to the new namespace?

The problem is javax.xml.bind -> jakarta.xml.bind automatic replacement breaks import order and I have no idea yet how to fix order without manual editing of 60+ files :)


Spark Driver Pod Getting Stuck in init state due to no driver configmap found

Hello,
Spark Application driver pod get stuck in init state due to driver configmap not found issue; what could be possible reason for the same and this results in Spark Operator stopping as process id (mentioned below) responsible for SparkSubmit also gets stuck. This is not consistent, encountered by sample Pi Spark application that has run successfully on number of times in the same environment.
sparkxpix75ba3dfb-driver             0/1     Init:0/1          0          47m
Warning  FailedMount  19s (x20 over 31m)    kubelet            (combined from similar events): MountVolume.SetUp failed for volume "spark-conf-volume" : configmap "sparkxpix75ba3dfb-1657310554583-driver-conf-map" not found
Process ID invoked by SparkSubmit is also stuck
/opt/tools/Linux/jdk/openjdk<IP_ADDRESS>2_<IP_ADDRESS>_x64/bin/java -cp /opt/spark/conf/:/opt/spark/jars/* org.apache.spark.deploy.SparkSubmit --master k8s://https://<IP_ADDRESS>:443 --deploy-mode cluster --conf spark.kubernetes.container.image.pullPolicy=IfNotPresent --conf spark.executor.memory=512m --conf spark.driver.memory=512m --conf spark.network.crypto.enabled=true --conf spark.driver.cores=0.100000 --conf spark.io.encryption.enabled=true --conf spark.kubernetes.driver.limit.cores=200m --conf spark.kubernetes.driver.label.version=3.0.1 --conf spark.app.name=sparkxpix75ba3dfb --conf spark.kubernetes.submission.waitAppCompletion=false --conf spark.executor.cores=1 --conf spark.authenticate=true --conf spark.kubernetes.driver.label.sparkoperator.k8s.io/launched-by-spark-operator=true --conf spark.kubernetes.namespace=abc-watch --conf spark.kubernetes.container.image==test:1 --conf spark.kubernetes.executor.label.sparkoperator.k8s.io/launched-by-spark-operator=true --conf spark.kubernetes.driver.label.sparkoperator.k8s.io/submission-id=e7075bf4-c30d-4d53-b924-0d2011555ce1 --conf spark.kubernetes.executor.label.sparkoperator.k8s.io/app-name=sparkxpix75ba3dfb --conf spark.kubernetes.executor.label.sparkoperator.k8s.io/submission-id=e7075bf4-c30d-4d53-b924-0d2011555ce1 --conf spark.kubernetes.driver.pod.name=sparkxpix75ba3dfb-driver --conf spark.kubernetes.authenticate.driver.serviceAccountName=spark-driver-abc-watch --conf spark.executor.instances=1 --conf spark.kubernetes.executor.label.version=3.0.1 --conf spark.kubernetes.driver.label.sparkoperator.k8s.io/app-name=sparkxpix75ba3dfb --class org.apache.spark.examples.SparkPi --jars local:///sample-apps/sample-basic-spark-operator/extra-jars/* local:///sample-apps/sample-basic-spark-operator/sample-basic-spark-operator.jar
These process ids blocked increase overtime and Spark Operator stops processing new SparkApplications.

I seem to hit these when there are too many old spark applications in my cluster. If we keep them below a couple thousand we seem to be ok.

same too

We identified the bottleneck of this performance issue is to do with spark-submit functionality is in JVM based scala code. We have ported the spark-submit functionality in golang, and see 4X performance gains. Code base is in https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/compare/master...gangahiremath:spark-on-k8s-operator:master

@liyinan926 Any thoughts about this ^? We've shared the same information in the k8s slack but didn't get any reply from you yet. I'm porting the messages to here for visibility. Please check and share how you think about this effort, and we can discuss about how we can contribute this back to OSS. Thanks!

Ganga H 12:12 PM
Hi [@liyinan926]
@Shiqi Sun and I work in a team in Salesforce. We provide managed Spark service(Spark and Spark Operator on K8S) to internal customers. Our customers faced issue with Spark Operator performance. At the max, it is able to process 80 Spark Application submission in a minute. We did deep-dive of Apache Spark and Spark Operator code base, and understood the bottleneck is spark-submit step. We have ported and re-written spark-submit within Spark Operator golang code base. With this, we see performance improvement of 4-5X (300 to 380 Spark Application submissions in a minute). We wanted to contribute this back to open source in a way both Spark and Spark Operator communities are open to it and embrace it as a default mechanism. Please guide how to go about the same. We are happy to share additional information, code base regarding the same. Thank you.
@Shiqi Sun, please add/correct.

Shiqi Sun 12:59 PM
@liyinan926 The performance was a problem to us since one of our customers submits a lot of batch jobs in tens of namespaces to our cluster, and with the existing Spark Operator performance (i.e. 80 SparkApps per min) we couldn’t keep up with their peak request, and that bumps up the Spark Operator internal queue a lot, which caused both the latency issue and a lot of job failures. We couldn’t resolve the issue by scaling up the Spark Operator, both vertically (giving Spark Operator pods more cpu/memory/controller-threads) and horizontally (spinning up more Spark Operator pods, since there is only one leader pod doing the work anyways). The bottleneck we found was in the spark-submit, where the spin-up of the JVM takes lots of resources (cpu/mem) and time, which significantly impacted the throughput. All this heavy JVM does is to run the cluster manager process in the Spark code to spin up driver pod and service objects, which doesn’t make much sense to us since we shouldn’t have needed a JVM to do that - golang is the cloud-native way of doing that and why not just let spark operator do that using golang? Therefore, @Ganga H worked on rewriting the driver pod/svc creation code into golang in the Spark Operator (vs. the existing Scala code in Spark source code), and we gained a lot of performance gain as Ganga mentioned above. This also decoupled the Spark Operator from including the Spark inside it, which alleviated the maintenance pain of different version of Spark and Spark Operator. However, one issue we can see of this golang code rewriting is that, we constantly need to translate the feature added to the Spark Scala code into golang. We have some ideas of how to solve this problem, and we can discuss more about it. Anyway, we benefitted a lot from this effort that @Ganga H did, we are thinking about contributing this back to the OSS Spark Operator project so that it can help solve the scalability issue when other Spark or Spark Operator users encounter it. But since this is relatively a big change, we wanted to provide the context and talk with you about the path forward before we make the PR. Let us know if you have any questions or concerns, thanks!

Hi, we also got an issue with spark operator slowness , but we found that the real reason for the slowness is scale is the controller delayed queue mutex.
We saw that even with controller threads up to 10000, and a very large machine, the CPU usage was under 5% percent, due to the fact that all threads are using one single mutex, even though it does not seem necessary.
We've decided to create a branch with a very different approach - queue per spark application, so all events of the same always happen in order, but different apps are not blocking each other .
We've tested it in large scale (creating up to 1000 apps at once) and got to a latency of less than 20 seconds at most (and our 96 cores Instance got to 100% CPU usage)
This is our branch:
https://github.com/kubeflow/spark-operator/compare/master...bnetzi:spark-operator:queue-per-app?expand=1

We identified the bottleneck of this performance issue is to do with spark-submit functionality is in JVM based scala code. We have ported the spark-submit functionality in golang, and see 4X performance gains. Code base is in master...gangahiremath:spark-on-k8s-operator:master

@bnetzi , I do not see momentum on the proposed PR to get merged with required documentation and unit test coverage added. Please share thoughts on the future plan for this work.

@bnetzi , I do not see momentum on the proposed PR to get merged with required documentation and unit test coverage added. Please share thoughts on the future plan for this work.

So, I actually I presented this PR in the last spark operator community meeting and it seems like it's on a roll. However, as it is a lot of new code, it would take some time for it to get in.
In the meantime you can try it on your env, it runs in ours for a while now without issues


Dealing with incorrect data length in descriptor's value

Not really an issue, much more a support request.
Env:

kernel 4.2.0 32bit (Ubuntu 15.10 Linux distro)
bluez 5.39 (built from sources, with the --enable-experimental and --enable-library options)
tinyb 0.4.1 (built from sources, with the -DCMAKE_BUILD_TYPE=DEBUG and -DBUILDJAVA=ON options)
OpenJDK 1.8.66

Using a simple Java main program, I'm connecting to a GATT server peripheral (specifically a Xiaomi MiBand bracelet, latest version) exposing the Heart Rate Service (0x180D).
Inside the Hart Rate Measurement characteristic of the service (0x2A37) I'm trying to access the CCC descriptor (0x2902): I get the BluetoothGattDescriptor instance with no problem, but the following error occurs while trying to read the current value of the descriptor:
code is simply de.getValue() (where "de" is the BluetoothGattDescriptor object), and the outcome is

(process:9020): GLib-CRITICAL **: g_bytes_get_data: assertion 'bytes != NULL' failed
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc

Using an external tool (nRF Master Control Panel on Android 5.0) I have found that the value field of that CCC descriptor has an "incorret length": nRF says "wrong size, 16 bits were expected (0x00)".
It seems that the manufacturer put a 8 bits value in the CCC descriptor instead of 16bits.
May that be the cause of the error reported above? I'm not a C++ developer, but it seems that "bad_alloc" deals with allocation, so maybe is a matter of size.
For completeness: the MiBand device also seems to have a problem with the pairing process, since bluez5 asks for the SC (Secure Connection) option and the bracelet fails the handshake with an "invalid parameter" response. Here is an excerpt of the btmon tool output:

< ACL Data TX: Handle 64 flags 0x00 dlen 11
SMP: Pairing Request (0x01) len 6
IO capability: NoInputNoOutput (0x03)
OOB data: Authentication data not present (0x00)
Authentication requirement: Bonding, MITM, SC, No Keypresses (0x0d)
Max encryption key size: 16
Initiator key distribution: EncKey Sign LinkKey (0x0d)
Responder key distribution: EncKey IdKey Sign LinkKey (0x0f)

ACL Data RX: Handle 64 flags 0x02 dlen 6      SMP: Pairing Failed (0x05) len 1
Reason: Invalid parameters (0x0a)
< HCI Command: Disconnect (0x01|0x0006) plen 3
Handle: 64
Reason: Authentication Failure (0x05)

I cannot say if this lack of support of the SC feature may be related to the read error or not...
I hope my questions can be of any help and not just a bothering.
Thanks for the very good code you are delivering to the community.

Thanks for the report. It looks like an issue with the data received from BlueZ, so it's likely that it's the same issue that nRF reports, but I will try replicating to see if there's something I can fix on my side.

Actually, I remembered seeing this error before. There is also the possibility that the device is not really connected or disconnects during operation and BlueZ returns an array of length zero. Running malloc(0) returns null, hence the failed assert.
Unfortunately, at this time it's not possible to get disconnection events, but I plan on adding this feature.

Well, it may be, but if you may see the test running you may find it unlikely, for two reasons:

the whole mini-test executes in less than 5 seconds and the "suggested" reconnection time of the MiBand should be around 39s
if I replace that specific instruction (ch.getValue()) with a different one (like srv.getCharacteristics(), where "srv" is the service) it works: in case of disconnection also getCharacteristics should fail, makes sense?

Try using readValue instead. getValue returns a cached value, but since it might be the first read there would be no cached value. If this is the issue I'll catch the exception and return null instead of crash.

I've just tried readValue(), I don't get the value, this is the outcome:

A fatal error has been detected by the Java Runtime Environment:

SIGSEGV (0xb) at pc=0x84cdfad0, pid=12137, tid=3058088768

JRE version: OpenJDK Runtime Environment (8.0_66-b17) (build 1.8.0_66-internal-b17)
Java VM: OpenJDK Server VM (25.66-b17 mixed mode linux-x86 )
Problematic frame:
C  [libtinyb.so.0+0x5cad0]  gatt_descriptor1_call_read_value_sync+0xe1

Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again

An error report file with more information is saved as:
/home/iooota/workspace/com.iooota.tests.tinyb/hs_err_pid12137.log

If you would like to submit a bug report, please visit:
http://bugreport.java.com/bugreport/crash.jsp
The crash happened outside the Java Virtual Machine in native code.
See problematic frame for where to report the bug.

Here is the quoted log file.
hs_err_pid12137.log.zip

Ok. Thank you. I was able to replicate this and found some additional errors with getValue.

It should be fixed in a99cba2755935b01c80cbd7b83eba3a6d788d5d6 and readValue should work as expected. Could you please confirm?
I will add an exception when a cached value for getValue is not available yet.

It seems fine, readValue causes no errors now, the returned value is just an empty array, so I assume that no value at all is present inside the descriptor: the smartphone app that I'm using as a comparison gives the following output

Incorrect data length (16bit expected): (0x) 00

Maybe the value is considered "incorrect" since it is totally absent, while 16bits were expected.
Using getValue I get the same errore as before

(process:25408): GLib-CRITICAL **: g_bytes_get_data: assertion 'bytes != NULL' failed
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc

but I understand that this is still expected at the moment, right?

Great, I'll close the issue then.


Provides a "col-all-1..12" class

Instead the use of:
div class="col-lg-3 col-md-3 col-sm-3 col-xs-3"
Provides a class to use with:
div class="col-all-3" that set 3 to all screen sizes!
Thanks.

You can just use .col-xs-3. It'll apply to every viewport.

Additional docs: https://github.com/twbs/bootlint/wiki/E029


UWP support is broken on Unity 2017.2.0b7

I am seeing following issues when trying to build for windows store:
Assets\Tools\Zenject\Source\Install\Contexts\Context.cs(249,69): error CS0121: The call is ambiguous between the following methods or properties: 'System.Linq.Enumerable.Append(System.Collections.Generic.IEnumerable, TSource)' and 'ModestTree.LinqExtensions.Append(System.Collections.Generic.IEnumerable, T)'
Assets\Tools\Zenject\Source\Main\DiContainer.cs(2054,35): error CS0121: The call is ambiguous between the following methods or properties: 'System.Linq.Enumerable.Append(System.Collections.Generic.IEnumerable, TSource)' and 'ModestTree.LinqExtensions.Append(System.Collections.Generic.IEnumerable, T)'
The solution was easy in my case:
I just had to rename the ModestTree.LinqExtensions.Append extension to something else...
I am using:
Unity 2017.2.0b7
Scripting Runtime Version: .Net 3.5 equivalent
Api Compatibility Level .Net 4.6
I noticed with different setup I get the other error that is mentioned in issue #334

I couldn't reproduce this but I'll take your word for it, so I removed the append method


Unhandled 'error' event on non default host:port

Hi. I don't understand what's happening, hour ago everythings was working fine.
I got Unhandled 'error' event when I run npm run dev. Changed with
  "config": {
    "nuxt": {
      "host": "<IP_ADDRESS>",
      "port": "3333"
    }
  },

Ok, this sh*t happens when you didn't restart PC for a week :smile:


Link with demo is broken

In the plugin description
http://goo.gl/uWhbzg

see: https://range-touch.herokuapp.com/ (thanks for reminding me!)


SAA-2198: Add endpoint to get multiple appointments

So that a user can mark attendance for multiple appointments

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

Project coverage is 84.62%. Comparing base (ad0bd09) to head (4d6cb42).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1214      +/-   ##
============================================
+ Coverage     84.60%   84.62%   +0.01%
- Complexity     3768     3772       +4
============================================
  Files           385      385
  Lines         12791    12799       +8
  Branches       1102     1103       +1
============================================
+ Hits          10822    10831       +9
+ Misses         1459     1458       -1
  Partials        510      510

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


material Snackbar only showed once when Koalaplot is added to project

First of all thanks for the great work on Koalaplot, i love it!.
i've had this wiered behaviour with snackbar when koalaplot library is added to the project, the snackbar showes only once and cannot be shown the second time calling 'showsnackbar' function.
effected targets:

iOS (Material 2 & 3)
Desktop (Material 3)

here are the steps to reproduce the issue:

Create a new compose multiplatform project using: https://kmp.jetbrains.com/
Replace the code in App.kt in 'commonmain'

import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.fillMaxWidth
import androidx.compose.material.*
import androidx.compose.runtime.Composable
import androidx.compose.runtime.getValue
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.runtime.rememberCoroutineScope
import androidx.compose.runtime.setValue
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier
import kotlinx.coroutines.launch

@Composable
fun App() {
    MaterialTheme {
        val snackbarHostState = remember { SnackbarHostState() }
        val scope = rememberCoroutineScope()
        var x by remember { mutableStateOf(1) }
        Scaffold(modifier = Modifier,
            snackbarHost = { SnackbarHost(snackbarHostState) }) {
            Column(Modifier.fillMaxWidth(), horizontalAlignment = Alignment.CenterHorizontally) {
                Button(onClick = {
                    scope.launch {
                        snackbarHostState.showSnackbar("Snackbar shown this many times: $x")
                        x++
                    }
                }) {
                    Text("Show snackbar!")
                }
            }
        }
    }
}

Add Koalaplot library to build.gradle.kts and sync
If you run the app on one of the effected targets snackbar will only be shown the first time.

if you remove the koalaplot library and sync then run the snackbar will be shown normally.

I tried this on Desktop (I don't have any Apply devices to try iOS), and I don't see the behavior you describe.

I tried this on Desktop (I don't have any Apply devices to try iOS), and I don't see the behavior you describe.

did you try Desktop with material 3?

I just tried with that as well, and it will popup the snackbar more than once. If you click the button multiple times quickly it doesn't increment the count correctly but if you wait for the snackbar to go away, then click the button again, it shows with the correct total count of clicks. But I don't think that is the issue you are reporting (that behavior was the same with or without Koala Plot).
I'm using Desktop on Windows 10. Are you on a Mac or Linux? Could be a platform dependent issue?

This is a known issue in compose multiplatform 1.6.0-alpha01 (i guess Koala plot uses this version) this should be fixed by upgrading to 1.6.0-beta01 as stated here.
closing this as its not an issue with Koalaplot

Thanks for tracking that down. I made a new release 0.5.2 that depends on 1.6.0-beta01.


Keep favorite and album assignment when deleting an asset

I think it might be useful to have something like this implemented:
https://github.com/agross/immich-duplicates/blob/master/README.md?plain=1#L147-L150

We might think about adding a feature that checks if the 'duplicate' we're planning to remove holds a spot in an album and enjoys favorite status. Should it turn out that the 'original'—the one we're keeping—isn't a favorite, we could then promote it to become the new favorite within the album

@agross what wrote is what you had in mind? Just to implement something or close it

what wrote is what you had in mind

Yes.

Has this been implemented since then, or not yet? It's still open, hence my question.


Vanguard: Get Outbound Network Dependency

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[x] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Hi @solankisamir , would you please have a check on the spellcheck issue.

@changlong-liu I have fixed that. Can you take a look?

Hi @ArcturusZhang , would you help to confirm whether the azure-sdk-for-go failure blocks this PR?

/azp run

@changlong-liu it should be caused by a bug in the pipeline and now it should have been resolved. I just retriggered the pipeline and the error should be gone.

Hi @solankisamir , now we are waiting for the ARM feedback

@changlong-liu can we please merge this?


Fix #277 - Switch from updateOne to replaceOne.

Fix #277 - Switch from updateOne to replaceOne to match the changes semantics in the MongoDB 3.0 driver.
Created tests for the basic DB operations.
Updated MongoDB Inc. Driver version to 3.0.2.

Thanks for the fix!


updated swagger readMe

Changed the swagger source

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-callautomation


Question about the stucture picture

Hi, I want to generate the whole body. How can I get some whole body stucture picture in densepose like those in tsetset? Are they generated by Densepose code?

We provide a large set of densepose images here (refer to densepose.zip). You can also generate densepose maps using Densepose.


Use reusable changelog workflow

The meeseeks backports shouldn't have to have a no-changelog-entry-needed label for their CI. This PR solves this by moving to the old workflow that was used by astropy for its changelog and is still used by jdaviz. This workflow happens to include skips for meeseeks.
Note the original ASDF workflow was a copy of the astropy one from before it was refactored into the reusable workflow, being used here. So they should operate exactly the same, except for the additional meeseeks logic.

You are welcome to use it... 😬

Label check CI is expected to fail because I fixed the milestone name, this is a pull_request_target issue.


Exclude generated yarn.lock files

We deliberately don't mark these as generated and our test has caught your attempt to change that 😁. See https://github.com/github/linguist/issues/4348 for more details as to why.
Closing.

Oops! Missed that discussion, sorry about that. But it clearly is generated... I guess the purpose of linguist is more to identify what should be shown in diffs than to categorize code?


Make particles capable of picking a part of the particle texture

So far the particles will pick the entire texture:
https://github.com/BabylonJS/Babylon.js/blob/master/src/Particles/babylon.particleSystem.ts#L390
(check third and forth parameter)
It could be great to add an option where a particle can be pick a specific subset. This subset could be constant or could evolved over time to create an animation

Will start to investigate and working on this.

Hi, I have a few suggestions here:
animationType:
WholeSheet: Animate over the whole texture sheet from left to right, top to bottom.
SingleRow: Animate a single row in the sheet from left to right. that is, each row of the sheet represents a separate Animation sequence.

numTilesX: Defines the tiling of the texture in the X axis.
numTilesY: Defines the tiling of the texture in the Y axis.
cycleCount: Specifies how many times the animation will loop during the lifetime of the particle.
enabled: Enable/disable the Texture Sheet Animation module.
useRandomRow: Use a random row of the texture sheet for each particle emitted. This option is only available when Single Row is selected as the Animation mode.
rowIndex: Selects a particular row from the sheet to produce the animation This option is only available when Single Row mode is selected and Random Row is disabled.
startFrame: Allows you to specify which frame the particle animation should start on (useful for randomly phasing the animation on each particle).
8.easingMode : specifies how the frame of animation increases as time progresses.

I hope it can help you :P

@deltakosh
So I'm going to rephrase what I understood :)
we need to enable the emitter to specify a subset of it self to emit the particles (doesn't minEmitBox and maxEmitBox in ParticleSystem do this job ?) ?
and "could evolved over time to create an animation" means we need to give the developer an option to specify how this subset can animate\move while emitting ?
One more thing third and forth parameters are the xOffset and yOffset in method _appendParticleVertex ?

@wilson-yong thanks for your suggestions :) will consider it in the implementation.

Not the emitter. Just the part of the texture used by particles over time. For now they all use the entire texture.
The idea here is too pick a part of it and change the part used over time.

aha :) thanks for clarification.

Closing it (only Doc and samples are missing:))


Template helper

Closes https://github.com/stamen/mapbox-gl-style-build/issues/14
This PR alters existing custom scripts to set up the build system into a general helper script. To run the script in a repo after installing the package, see "Implementation" section in README:
create-layer-templates
    --in-dir=styles
    --out-dir=templates
    --base-path=styles/base-style.json

Thanks @ebrelsford ! Just made updates, back to you!


Local file seeking is broken

Seeking backwards and then forwards in local files is broken.
After spending some time with a debugger, I tracked the issue down to the incorrect variable being used to track the current file position for files with a pStream.
For example here, the code uses hf->dwFilePos:
https://github.com/ladislav-zezula/StormLib/blob/7b3aadd4999c4c150c7ac8ceed5586b75efc46a1/src/SFileReadFile.cpp#L703-L706
However, for files with pStream, the field that tracks the actual position is within the stream implementation. The code above should use FileStream_GetPos instead of hf->dwFilePos.
Fixing just this does not fix the issue entirely, I'm guessing there are a few other places where the wrong file position tracker field is used.

Fixed by https://github.com/ladislav-zezula/StormLib/commit/e482d3847e175a99ba264f421e86df016570ff6d


correct location for install-config?

TASK [common : Generate the ignition manifests] *************************************************************
fatal: [localhost]: FAILED! => {"changed": true, "cmd": ["openshift-install", "create", "manifests", "--dir=/ocp/ocp4-vsphere-upi-automation/install-dir"], "delta": "0:00:00.106394", "end": "2020-04-09 14:31:55.273405", "msg": "non-zero return code", "rc": 1, "start": "2020-04-09 14:31:55.167011", "stderr": "level=fatal msg="failed to fetch Master Machines: failed to load asset \"Install Config\": invalid \"install-config.yaml\" file: pullSecret: Invalid value: \"{\\\"auths\\\": \\\"...\\\"}\": json: cannot unmarshal string into Go struct field imagePullSecret.auths of type map[string]map[string]interface {}"", "stderr_lines": ["level=fatal msg="failed to fetch Master Machines: failed to load asset \"Install Config\": invalid \"install-config.yaml\" file: pullSecret: Invalid value: \"{\\\"auths\\\": \\\"...\\\"}\": json: cannot unmarshal string into Go struct field imagePullSecret.auths of type map[string]map[string]interface {}""], "stdout": "", "stdout_lines": []}
My install is suffering a fatal error as seen above. I'm executing from the /ocp/ocp4-vsphere-upi-automation directory and my /root/ocp4/install-config.yaml is as per the directions for the helper node. Where does this need to be?

We will be using helper only for all the services (DHCP, DNS, Load Balancer, Web Server, TFTP) it provides but not for installation. As mentioned earlier the only files you need to touch are the staging, group_vars/all.yml and ansible.cfg. In the group_vars/all.yml, line 8, you have to get the pull secret from Red Hat and add it as a line. The install-config.yml is auto-generated from a template. The link to pull secret is given in the README.

Great, that worked. I'm going to make a commit and merge request with some updates to your docs that will hopefully make them easier to read. Thank you!


Release 0.2.1

Description

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make unit_testing and make integration-testing successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Do we use branches instead of tags to mark versions?
We need to merge into 0.2.1 or tag 0.2.1, not 0.2.

We use tags to mark the releases and branches to manage the minor versions.

Do we use branches instead of tags to mark versions?
We need to merge into 0.2.1 or tag 0.2.1, not 0.2.

We use tags to mark the releases and branches to manage the minor versions.

Make sense?

Do we use branches instead of tags to mark versions?
We need to merge into 0.2.1 or tag 0.2.1, not 0.2.

The tag comes in the make command: make new_release


Publish to Maven Central

Gradle build scripts to generate all the required metadata
Manage optional external dependencies (required for SQL drivers)
Access and keys for publishing

https://maven.apache.org/repository/guide-central-repository-upload.html
https://central.sonatype.org/pages/requirements.html#sufficient-metadata
Accenture Atom is already published to Maven Central.
https://github.com/Accenture/atom

This was addressed in #152 and #155. Also see #170 where we started publishing binary distribution packages via GitHub releases.


Integrate Keycloak for Identity and Access Management Services

Implement identity services using Keycloak to provide robust authentication mechanisms, including token extraction, user information retrieval, and token validation.

Secure Authentication

Implement secure login and authentication using Keycloak
Develop service methods to handle access and refresh tokens.

Token Management

Manage access and refresh tokens including extraction and validation.
Create utilities to extract tokens from authentication requests.
Develop methods to validate token integrity and expiration.

User Data Handling: Retrieve user details securely from the token.

This issue has been solved in

https://github.com/NASA-IMPACT/veda-auth-central/commit/b879f98381d87ecb5fbf6f32ae533d03ef11f301
https://github.com/NASA-IMPACT/veda-auth-central/commit/8fe633bf6431348afb95f8dca1d25f7b22cf8a90


issue template: run podman info --debug

Ask users to run podman-info with --debug to the exact git commit,
compiler and go version.
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

LGTM
On Mon, Feb 25, 2019, 05:57 OpenShift CI Robot<EMAIL_ADDRESS>wrote:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: vrothberg
https://github.com/containers/libpod/pull/2425#
The full list of commands accepted by this bot can be found here
https://go.k8s.io/bot-commands?repo=containers%2Flibpod.
The pull request process is described here
https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process
Needs approval from an approver in each of these files:

OWNERS https://github.com/containers/libpod/blob/master/OWNERS
[vrothberg]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/2425#issuecomment-466968060,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHYHCDS2L7OLF6fqWN0Xv3O-cTq5EAdTks5vQ8GVgaJpZM4bPm92
.

Tests are failing, can't imagine it's related?  @mheon is this a known issue atm?
[Fail] Podman pod create [It] podman pod container share Namespaces
/var/tmp/go/src/github.com/containers/libpod/test/e2e/pod_pod_namespaces.go:41

I restarted it manually. Thanks a lot for checking, @TomSweeneyRedHat :pray:

I've seen a lot of flakes on that test. Not 100% sure what's going on with it.

Actually, might not be that test specifically, but another pod test...

/lgtm


Can't read file from private Google Cloud Storage Bucket

Hello,
I'm trying to read a Cloud Optimized Geotiff file from a private GCS bucket using the googleapis link:
`
URL = "https://storage.googleapis.com/storage/v1/b/test-oneoftwo-prove-riotiler/o/dem_my_COG.tif?alt=media"
from rio_tiler.io import Reader
@router.get("/")
async def get_tiles():

 with Reader(URL) as cog:
     stats = cog.statistics()
     vmin = stats["b1"].model_dump()["min"]
     vmax = stats["b1"].model_dump()["max"]
     print(cog.statistics())

return {response: 'ok'}

`
Since the bucket is private I need to access to that file using something like API Key provided by gcloud and setting the API Key as a Bearer token to the authorization header like "Authorization: Bearer [TOKEN]", otherwise I get:

httpx.HTTPStatusError: Client error '401 Unauthorized' for url 'https://storage.googleapis.com/storage/v1/b/test-riotiler/o/dem_COG.tif?alt=media'

Is there a way to pass headers to the Reader() or just authenticating to googleapis in any other way?
Thank you

🤔 the httpx error message doesn't really make sense because rio_tiler.io.Reader doesn't use httpx
about using private bucket in GCP, you can have a look at https://gdal.org/user/virtual_file_systems.html#vsigs-google-cloud-storage-files and then use https://github.com/rasterio/rasterio/blob/f424fce16e885d932bb0c9a02dbeb7d27e77b720/rasterio/session.py#L410
import rasterio
from rasterio.session import GSSession
from rio_tiler.io import Reader
with rasterio.Env(session= GSSession()):
     with Reader(URL) as cog:
         ...

🤔 the httpx error message doesn't really make sense because rio_tiler.io.Reader doesn't use httpx
about using private bucket in GCP, you can have a look at https://gdal.org/user/virtual_file_systems.html#vsigs-google-cloud-storage-files and then use https://github.com/rasterio/rasterio/blob/f424fce16e885d932bb0c9a02dbeb7d27e77b720/rasterio/session.py#L410
import rasterio
from rasterio.session import GSSession
from rio_tiler.io import Reader
with rasterio.Env(session= GSSession()):
     with Reader(URL) as cog:
         ...

Thank you @vincentsarago this was really helpful! Actually the HTTPX error came from another trial I previously did with riotiler STACRead so do not consider it.
Also, since I found out that GDAL doc may not be extremely help at a first glance I put my code with comment here, hoping it would be helpful for other:
   import rasterio
   from rasterio.session import GSSession
   from rio_tiler.io import Reader

   URL="gs://bucket-name/file-name.tif" # Use 'gs://... ' notation

    with rasterio.Env(session=GSSession()):

      # GSSession use the Google application-default-credential standard path
      # Then if you are locally developing it would be helpful just like any other implementation
      # of gcloud api

        with Reader(URL) as cog:
            stats = cog.statistics()
            print(stats)


Ability to edit "behavior_preferences" options

I'd like to be able to edit the behavior_preferences file/options.

PREF_PASS_FILTER_MANUFACTURER_ARRIVE="Apple"
#ARRIVE TRIGGER NEGATIVE FILTER(S)
PREF_FAIL_FILTER_ADV_FLAGS_ARRIVE="NONE"
PREF_FAIL_FILTER_MANUFACTURER_ARRIVE="NONE"
#If true, this value will cause monitor to report a 'home' or 'not_home' message to ... /device_tracker conforming to device_tracker mqtt protocol.
PREF_DEVICE_TRACKER_REPORT="true"
#This value is either true or false and determines whether monitor publishes when a scan begins and when a scan ends
PREF_MQTT_REPORT_SCAN_MESSAGES="true"
Thank you!

You can edit /home/pi/monitor/behavior_preferences
Those are the options available:
https://github.com/andrewjfreyer/monitor#fine-tuning


wrong hashagg results

Describe the bug
This is a rare case of hashagg over multiple columns, as the keys are combined into one "row" like String, these three rows will be counted into one group
To Reproduce
group by col_a, col_b

col_a
col_b

"ab"
"ab"

"a"
"bab"

"aba"
"b"

Expected behavior
fix the hash conflict in the above case
Additional context
N/A

duplicated w/ #1104