Feat: Performance Test Auxiliaries

What this PR does / why we need it:
This PR refactors loggings in controller by aggregating functions in ReconcileContext. The new ReconcileContext has the following benefits:

It can easily add checkpoints to track time costs.
It normalizes the logging format in controllers which enables effective logging collection by Loki in monitoring system.
Later controller-level modifications to the context can be easily extended to all controllers.

In addition, several files are added to the ./hack/pressure-test which provides a simple guide for running KubeVela performance test.

Will klog be replaced by ctx?

Will klog be replaced by ctx?

Yes, there are several advantages of using ctx which wraps some extra codes on top of klog, as mentioned in the PR introduction.

ReconcileContext wraps the target object namespace and name in its key/value pair which makes all logs in controller carry this information.
ReconcileContext gives 1 minute time limit for each reconcile.
ReconcileContext automatically extracts controller file location and extract the controller name which is carried for each log as well.
ReconcileContext outputs all logs with key/values described above in fixed format which can be easily collected by logging services such as Loki.
ReconcileContext records time cost for each reconcile by default and can track details optionally.

suggest to make reconcileContext to have the ability to fork

This PR is too large. I will try to split some.


connector should use only one application session

connector should use only one application session
node-opcua is much faster since v0.2.2 and with that we should try to work with less sessions

done with v2.2.0


TP: Provide the ability to view all servers utilized by a ds that is assigned to a topology

I'm submitting a ...

new feature / enhancement request
improvement request (usability, performance, tech debt, etc.)

Traffic Control components affected ...

Traffic Portal

Current behavior:

Right now, topologies servers are not visible on the topologies "edit" page.
Expected / new behavior:

the topology servers view is accessible via a button on the edit topology page.
the topology servers view is accessible via a deep link (i.e. tp.domain.com/!#/topologies/name/servers).
the topology servers view shows all servers utilized by the topology in a tabular format.
the topology servers view provides a method to edit an existing server.

Duplicate of #4559

reopening as this will be closed when #4966 is merged.


fix(@jsii/spec): speed up assembly validation by 20x for large libraries

Switch from jsonschema to ajv to improve assembly validation performance.
Previously, loading the type system for aws-cdk-lib and constructs took around 15-17 seconds on my local development machine, using node v16.8.0. When I ran it with the updated version of @jsii/spec, it only took 750-800ms.
Both ajv and jsonschema are popular npm libraries (we also currently leverage it in the cdk8s-cli). ajv has one downside which is that introduces has a larger bundle size (1.02 MB vs 81.8kB), but I think it's worth it for the performance improvement.
Closes cdklabs/decdk#7

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

🤯


Prevent Issues from being back dated after approval

Describe the bug
Once an Issue/Status has been approved, no UPDATE to this Status/Issue can be "back-dated" and dates before the latest APPROVED UPDATE are greyed out
To Reproduce
Steps to reproduce the behavior:

Go to Issues Tab
Find a current Issue
Try to add an update, make the update IN THE PAST
No save happens, no error message either

Expected behavior
We want to see an error message because UPDATES to an Issue (eg. the Cupcake Issue) should not happen in the past.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

connected to #1729

Solved here https://app.zenhub.com/workspaces/epictrack-63891ea941d309001fa292cf/issues/gh/bcgov/epic.track/1729


Unexpected behaviour of route.start(true)

I have faced with strange behaviour of single page application built with riot.js . Its rendering was delayed until loading of all external js libraries finishes. It includes async externals, iframes content, etc. After investigation it became clear that route.start(true) calls appropriate route handler only after 'load' event browser trgiggers.
It was not trivial to find the reason, a lot of time was spent. I suggest to describe this behaviour in documentation and suggest to explicitly call of route.exec() where it`s possible. My problem was solved as follows:
riot.route.start() // call of start(true) delays route processing before 'onload' event raises
riot.route.exec() // so need call exec() in place

@cognitom comment:

By the way, I think the content should be loaded after routing if you use frontend routing. In that point, I'm not sure your case is general or not.

In my case most of js files and riot templates packed in single js bundle. riot.route is launched after this bundle is loaded.

@pelid the behaviour was introduced for fixing #33.
https://github.com/riot/route/blob/v3.0.2/lib/index.js#L329-L334

@cognitom @pelid it seems a specific issue and not a route one. Riot-route allows you to start the router either automatically or via exec. The autostart strategy waiting the window load is a common behavior used also by other famous html5 routing library i don't see any issue here.

Closing this issue because it's related to an old router version. Please update to the latest @riotjs/route version if you can.


Log endpoint id w/ discovery status

These changes add the observer provided endpoint ids to discovery status logs. Also including a minor discovery mode observer creation setting change* to make sure the ID is set where the host observer expects* it.

    	Messages:   	Error, do this: mount -t proc proc /proc

I'm adding a skip check for arm for now. I've not tracked down what's up but for some reason ps through qemu doesn't work w/ our bundled tools, and I have a feeling it's related to our patchelf use.


revised M15 security slides

Some reorganization; redid duplicate OWASP story to make it about security being a human/management problem.
Updated learning objectives.
Improved animations in first couple of scenarios (I hope you enjoy them).
I will probably make another pass revising the scripts to match updated content

Revisions look good to me. I gave this lecture today, but this will be useful for next time.


CI 캐싱

Describe
tuist fetch 부분을 캐싱하여 CI 속도를 높여야함
Work
No response
Additional
No response

https://github.com/team-xquare/xquare-iOS/commit/09e94440bed2e4f73302680ddbae5ba1f5aac9b3


ENH: Implement Beta negative binomial distribution

Is your feature request related to a problem? Please describe.
This is a distribution I would find useful in the library https://en.wikipedia.org/wiki/Beta_negative_binomial_distribution
I would like to be able to calculate the probability mass function etc. like with other distributions.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

@steppi @WarrenWeckesser : would you have time to review if I submit a PR for this one?
I did a quick experiment using the following Wikipedia sources.
Sampling:

PMF:

import numpy as np
from scipy import special as sc
import matplotlib.pyplot as plt

rng = np.random.default_rng(seed=234346)
a = 100
b = 20
r = 500
p = rng.beta(a, b, size=10000)
x = rng.negative_binomial(r, p)

k = np.linspace(0, x.max(), 500)
logpmf = sc.gammaln(r + k) + sc.betaln(a + r, b + k) - sc.betaln(a, b) - sc.gammaln(k + 1) - sc.gammaln(r)
plt.hist(x, density=True, bins='auto', histtype='stepfilled', alpha=0.2)
plt.plot(k, np.exp(logpmf))
plt.show()

A formula for the CDF is given by Wolfram:

@dschmitz89 Hi, I could take a look at your PR.


Hyprland does NOT launch it automatically at exec-once.

Describe the bug
Hyprland does not launch it automatically after start.
To Reproduce
Steps to reproduce the behavior:
Split config files
Move the user to another one and cancel at some point
Stop SDDM (optinoal)
Do it again and it will give a error
Start SDDM with SystemCTL
Log in.
ags will fail to load a JS file.
Recompile it again
log out and log in
Not launching except kitty (manual, no exec-once.)
Expected behavior
Normally launch
Screenshots
(no screenshot :(((( )
Desktop (please complete the following information):

Distribution: Arch Linux (originally 6.11.4-arch1-1, upgraded to 6.11.5-arch-1-1)
Window Manager/Desktop Environment: Hyprland (latest git)

Additional context
Add any other context about the problem here.

Could you update your exec-once to this please:
exec-once = ags > ~/ags_logs.txt 2>&1

Then provide the logs that show up in your ~/ags_logs.txt file.

@Jas-SinghFSU Got the same issue, here's logs:
(com.github.Aylur.ags:1173): Gjs-Console-CRITICAL **: 23:14:27.913: GLib.SpawnError: Failed to execute child process “bun” (No such file or directory)
proc@resource:///com/github/Aylur/ags/utils/exec.js:11:27
execAsync@resource:///com/github/Aylur/ags/utils/exec.js:77:19
@file:///home/rx3lixir/.config/ags/config.js:15:25
_init/GLib.MainLoop.prototype.runAsync/</<@resource:///org/gnome/gjs/modules/core/overrides/GLib.js:263:34
Same distro, same wm

@rx3lixir Could you verify that bun is installed please? https://hyprpanel.com/getting_started/installation.html#bun
Also if I may have the output of your following command:
which bun

@Jas-SinghFSU yep, bun is installed for sure, thank u for your respond, here's output
❯ bun --version
1.1.33
❯ which bun
/home/rx3lixir/.bun/bin/bun

Same issue here on a fresh install of HyprPanel, with all required dependencies installed beforehand.

@rx3lixir can you re run this command (may require sudo)
sudo ln -s $HOME/.bun/bin/bun /usr/local/bin/bun

And try again? It can't seem to find bun when launching through exec-once.

now we're talking. thanks a lot for your investigation everything works finee

Yay :tada: !!

Came into this requirement myself as well. Bun was not installed as i am not using javascript. Shouldn't that be on the requirements?

@SpyPower actually bun is a key dependency on hyprpanel and moreover this installation script is provided by the docs, I personally don't know how I missed symlink part or mb something went wrong

logging out and in, cancelling my android build.

Logged in and
(com.github.Aylur.ags:281660): Gjs-Console-CRITICAL **: 23:16:00.183: GLib.SpawnError: “bun” alt süreç çalıştırılamadı (Böyle bir dosya ya da dizin yok)
proc@resource:///com/github/Aylur/ags/utils/exec.js:11:27
execAsync@resource:///com/github/Aylur/ags/utils/exec.js:77:19
@file:///home/hyprlanduser/.config/ags/config.js:15:25
_init/GLib.MainLoop.prototype.runAsync/</<@resource:///org/gnome/gjs/modules/core/overrides/GLib.js:263:34
BUN DOES NOT EXIST???
I symlinked bun! I think .bashrc executes too late...

So the fix is should i edit my hyprland config then source bashrc?

(com.github.Aylur.ags:286094): Gjs-Console-CRITICAL **: 23:24:13.092: GLib.SpawnError: “bun” alt süreç çalıştırılamadı (Böyle bir dosya ya da dizin yok)
proc@resource:///com/github/Aylur/ags/utils/exec.js:11:27
execAsync@resource:///com/github/Aylur/ags/utils/exec.js:77:19
@file:///home/hyprlanduser/.config/ags/config.js:15:25
_init/GLib.MainLoop.prototype.runAsync/</<@resource:///org/gnome/gjs/modules/core/overrides/GLib.js:263:34
NO.. sourced bashrc in hyprland and it did not work.
I think it uses fish instead ☹

yay
Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish
hyprlanduser@archuser ~> bash -c
bash: -c: seçenek bir argüman gerektiriyor
hyprlanduser@archuser ~ [2]> bash -c "ags"
Ags with busname "ags" is already running
hyprlanduser@archuser ~> bash -c "ags"
,Ags with busname "ags" is already running
hyprlanduser@archuser ~>

so i think, this is solved.

Solved.
How to solve this in the future
Delete symbolic linked bun
Rerun the install command again
Will your Config stay?
Yes.

proof: (just a empty desktop)


Handle Flagged status=Declined contacts during email campaigns

Since April 19th the batch update contacts is broken on Brevo because of contacts flagged status declined in Brevo and still in our database.
The issue is that the Brevo API is returning 400 Http response if one faulty contact is present in the list, not updating the rest of the contacts.
In consequence, we cannot just ignore the error as a workaround. We could remove the faulty contacts in our database but the issue will raise again once new contacts flagged as declined.

Looks like there is some preventive contact deactivation in the code but doesnt work for some reason. Wondering if contact.emailBlacklisted is the same thing as status: Declined in the UI. looks like its not.


Assignment to map keys and array indexes

[x] Test
[x] Fix/impl

Fixes #16

You are only testing what is expected to work and not the possible error such as a[10+2]

a[10+2] is fine (it's a[12]) but if you mean out of bounds check, sure, though I was wondering if I should make them work at least in term of growing the arrays

Yes, I'm thinking about out of bound, as 10 is not defined in you example of a with a few items.


New Stacked-Slices backend

Description
This PR attempts to bring a new backend to CVXPY that outperforms the current SciPy backend.
It is a major step forward towards tackling the numerous "slow DPP compilation time" issues brought by many users.
Benchmarks have shown that some problems now compile up to 100x faster, but generally, the compilation times range between a 15 to 20% decrease. In addition, a few benchmarks are also much slower when compared to the original CVXCORE backend.
Issue link (if applicable): #1332
Type of change

[x] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[ ] Bug fix
[ ] Other (Documentation, CI, ...)

Contribution checklist

[x] Add our license to new files.
[x] Check that your code adheres to our coding style.
[x] Write unittests.
[x] Run the unittests and check that they’re passing.
[x] Run the benchmarks to make sure your change doesn’t introduce a regression.

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

No objections to merging from me!


WebGLShader has no properties

TypeScript Version:  3.7.3

Search Terms: array element type checking, error type checking, stops checking function argument types
Code
https://github.com/vjancik/dont-find-me-yet/blob/tsc-issue/src/index.ts#L63-L64

fails to infer a possible Error type
and if I set it explicitly (see Playground), fails to enforce it in

https://github.com/vjancik/dont-find-me-yet/blob/tsc-issue/src/index.ts#L146

createProgram doesn't enforce WebGLShader[] at all on the first argument, or seemingly any of the other arguments as well. I can add any variable in there and the array accepts it (also Playground)

Expected behavior:
Types of fn. arguments on createProgram get enforced correctly.
Optional: Return type on loadShader & createProgram gets inferred correctly as ... | Error
Actual behavior:
The issue doesn't resolve even if I specify all return types on all functions manually.
Playground Link:
Playground link
Related Issues:
No, there are multiple problems and I can't pinpoint the cause in a very specific setup.

The root cause here is that WebGLShader is an empty type, which causes all sorts of downstream problems. We should do something with it so this doesn't happen.

Just a thought, but isn't it possible to have a "strict / strong" typing flag that enforces a prototype instead of just the properties (duck typing)?
I suspect there will be more and more opaque types in the future as the Web standard becomes more and more integrated with the OS.

N


Arch Bot added

Have you read the Contributing Guidelines ?
Yes
Description
The project is a chatbot website that utilizes Artificial Intelligence to communicate with a user through a chat channel or interface. The bot will guide or suggest solutions for their various problems and questions.
Fixes #855
Checklist

[ x] I've read the contribution guidelines.
[ x] I've checked the issue list before deciding what to submit.
[ x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
To complete the issue #855. Kindly merge it

@Nidhish-07 Your images are not working , also if it's web dev project then deploy this project and give the link in readme.md file .
Note : If it's not coming under web dev then we will not merge this

@Nidhish-07 Your images are not working , also if it's web dev project then deploy this project and give the link in readme.md file .
Note : If it's not coming under web dev then we will not merge this

So the app must be deployable by docker as will I have to manually deploy it and give a link, or just anyone conditions satisfies the criteria?

@Nidhish-07 Is this a web dev project ? If yes then deploy it .


useless-assert: support bool ops

closes testifylint

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

12 of 16 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 93.959%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/checkers/useless_assert.go
12
16
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
1882

Relevant Lines:
2003

💛  - Coveralls


TC Overview not showing all failures results

Hi,
I am not seeing all failures in the overview.
My TeamCity logfile has them all:
[16:49:39] :	 [Step 4/6] JSHint: app/scripts/JSHintfailures.js
[16:49:39] :		 [JSHint: app/scripts/JSHintfailures.js] app/scripts/JSHintfailures.js: line 3, col 26, Missing semicolon.
[16:49:39] :			 [app/scripts/JSHintfailures.js: line 3, col 26, Missing semicolon.] W033: Missing semicolon.
[16:49:39] :			 [app/scripts/JSHintfailures.js: line 3, col 26, Missing semicolon.] app/scripts/JSHintfailures.js: line 5, col 44, Strings must use singlequote.
[16:49:39] :				 [app/scripts/JSHintfailures.js: line 5, col 44, Strings must use singlequote.] W109: Strings must use singlequote.
[16:49:39] :				 [app/scripts/JSHintfailures.js: line 5, col 44, Strings must use singlequote.] app/scripts/JSHintfailures.js: line 5, col 17, Expected '===' and instead saw '=='.
[16:49:39] :					 [app/scripts/JSHintfailures.js: line 5, col 17, Expected '===' and instead saw '=='.] W116: Expected '===' and instead saw '=='.
[16:49:39] :					 [app/scripts/JSHintfailures.js: line 5, col 17, Expected '===' and instead saw '=='.] app/scripts/JSHintfailures.js: line 11, col 14, Trailing whitespace.
[16:49:39] :						 [app/scripts/JSHintfailures.js: line 11, col 14, Trailing whitespace.] W102: Trailing whitespace.
[16:49:39] :						 [app/scripts/JSHintfailures.js: line 11, col 14, Trailing whitespace.] app/scripts/JSHintfailures.js: line 12, col 3, Expected '{' and instead saw 'window'.
[16:49:39] :							 [app/scripts/JSHintfailures.js: line 12, col 3, Expected '{' and instead saw 'window'.] W116: Expected '{' and instead saw 'window'.
[16:49:39] :							 [app/scripts/JSHintfailures.js: line 12, col 3, Expected '{' and instead saw 'window'.] app/scripts/JSHintfailures.js: line 13, col 3, Expected 'window' to have an indentation at 1 instead at 3.
[16:49:39] :								 [app/scripts/JSHintfailures.js: line 13, col 3, Expected 'window' to have an indentation at 1 instead at 3.] W015: Expected 'window' to have an indentation at 1 instead at 3.
[16:49:39] :								 [app/scripts/JSHintfailures.js: line 13, col 3, Expected 'window' to have an indentation at 1 instead at 3.] app/scripts/JSHintfailures.js: line 17, col 2, Trailing whitespace.
[16:49:39] :									 [app/scripts/JSHintfailures.js: line 17, col 2, Trailing whitespace.] W102: Trailing whitespace.
[16:49:39] :									 [app/scripts/JSHintfailures.js: line 17, col 2, Trailing whitespace.] app/scripts/JSHintfailures.js: line 21, col 3, Missing "use strict" statement.
[16:49:39] :										 [app/scripts/JSHintfailures.js: line 21, col 3, Missing "use strict" statement.] E007: Missing "use strict" statement.
[16:49:39] :										 [app/scripts/JSHintfailures.js: line 21, col 3, Missing "use strict" statement.] app/scripts/JSHintfailures.js: line 21, col 12, Expected '===' and instead saw '=='.
[16:49:39] :											 [app/scripts/JSHintfailures.js: line 21, col 12, Expected '===' and instead saw '=='.] W116: Expected '===' and instead saw '=='.
[16:49:39] :											 [app/scripts/JSHintfailures.js: line 21, col 12, Expected '===' and instead saw '=='.] app/scripts/JSHintfailures.js: line 24, col 19, Expected '===' and instead saw '=='.
[16:49:39] :												 [app/scripts/JSHintfailures.js: line 24, col 19, Expected '===' and instead saw '=='.] W116: Expected '===' and instead saw '=='.
[16:49:39] :												 [app/scripts/JSHintfailures.js: line 24, col 19, Expected '===' and instead saw '=='.] app/scripts/JSHintfailures.js: line 28, col 19, 'notYetDefined' was used before it was defined.
[16:49:39] :													 [app/scripts/JSHintfailures.js: line 28, col 19, 'notYetDefined' was used before it was defined.] W003: 'notYetDefined' was used before it was defined.
[16:49:39] :													 [app/scripts/JSHintfailures.js: line 28, col 19, 'notYetDefined' was used before it was defined.] app/scripts/JSHintfailures.js: line 20, col 38, 'compareNullUsingTripleEquals' is defined but never used.
[16:49:39] :														 [app/scripts/JSHintfailures.js: line 20, col 38, 'compareNullUsingTripleEquals' is defined but never used.] W098: 'compareNullUsingTripleEquals' is defined but never used.
[16:49:39] :	 [Step 4/6] JSHint: app/scripts/controllers/notcovered.js
[16:49:39] :		 [JSHint: app/scripts/controllers/notcovered.js] app/scripts/controllers/notcovered.js: line 3, col 5, Missing "use strict" statement.
[16:49:39] :			 [app/scripts/controllers/notcovered.js: line 3, col 5, Missing "use strict" statement.] E007: Missing "use strict" statement.
[16:49:39] :	 [Step 4/6] JSHint: test/spec/TestJSHintfailures.js
[16:49:39] :		 [JSHint: test/spec/TestJSHintfailures.js] test/spec/TestJSHintfailures.js: line 2, col 3, Missing "use strict" statement.
[16:49:39] :			 [test/spec/TestJSHintfailures.js: line 2, col 3, Missing "use strict" statement.] E007: Missing "use strict" statement.
[16:49:39] :			 [test/spec/TestJSHintfailures.js: line 2, col 3, Missing "use strict" statement.] test/spec/TestJSHintfailures.js: line 5, col 23, Expected '===' and instead saw '=='.
[16:49:39] :				 [test/spec/TestJSHintfailures.js: line 5, col 23, Expected '===' and instead saw '=='.] W116: Expected '===' and instead saw '=='.
[16:49:39] :				 [test/spec/TestJSHintfailures.js: line 5, col 23, Expected '===' and instead saw '=='.] test/spec/TestJSHintfailures.js: line 9, col 23, 'TestnotYetDefined' was used before it was defined.
[16:49:39] :					 [test/spec/TestJSHintfailures.js: line 9, col 23, 'TestnotYetDefined' was used before it was defined.] W003: 'TestnotYetDefined' was used before it was defined.
[16:49:39] :					 [test/spec/TestJSHintfailures.js: line 9, col 23, 'TestnotYetDefined' was used before it was defined.] test/spec/TestJSHintfailures.js: line 1, col 31, 'TestDefinedButNotUsed' is defined but never used.
[16:49:39] :						 [test/spec/TestJSHintfailures.js: line 1, col 31, 'TestDefinedButNotUsed' is defined but never used.] W098: 'TestDefinedButNotUsed' is defined but never used.
but the overview only shows some of them:

Is this a known problem, or is it just something I need to configure?
Thanks

Closing this as we cannot seem to reproduce this. Please open a new ticket with specific set of reproducible steps if this problem persists.


enabling progressive and arithmetic features causes jpegtran to throw an unhelpful error.

Using progressive and arithmetic features together raises this error:
Error: Requested feature was omitted at compile time
Would you accept a pull request to either work around this or to raise a more meaningful error?

Appears to be related to this problem with jpegtran is comiled: http://comments.gmane.org/gmane.comp.graphics.libjpeg-turbo.user/143

:+1: PR welcome :)


How to use bgfx?

This is a brilliant cross-platform rendering library, but I don't know how to use it, even just draw a image pixel data. I know it's pretty easy using sdl2.
I guess if I have the knowledge of OpenGL, then using bgfx would be easier?
Anyone guru could give some advise please!?

its simple dbgPrintText, easy enough to understand. But how about more complex 2d rendering?

Some knowledge of basic concepts like vertex/index buffers, and shaders is required to use bgfx. Try with 01-cubes example, and if you still don't understand, you should find 3D tutorials online and go from there.

https://learnopengl.com/
I’m reading this tutorial. There are concepts like vertex/index buffer and shaders.
Does it mean the way of using bgfx is similar with OpenGL. And if I know some of knowledge of OpenGL, then it would be easy to use bgfx, right?

Yes, there are similarities, for example bgfx uses GLSL-like shader language. Concepts of index/vertex buffers, textures, etc. are easy to translate once you know how they work.
Try creating something with GL and then once it works, try making it work with bgfx. That way you have known reference, and can figure out step-by-step how to translate to bgfx.

Got it. Thank you very much, superman~


Try ahead of time compilation of numba  in setup.py

See https://numba.readthedocs.io/en/stable/reference/aot-compilation.html
Possible in pyodide?

This is unlikely to be possible for a long time cf https://github.com/pyodide/pyodide/issues/621


Demo link

Please have a demo page so we can see without having to clone

Hi, a bit late but here it is https://jsainsburyplc.github.io/react-timelines/

Closing this as answered by @willbamford, and demo link added to repo and README.


Rename test files to remove colons from the file path

They're not being used by any code yet.
Fixes #59

@dbekaert can you check this PR as I need it before pushing the unit tests for checking values. Re #59 the WRF file naming convention the problem is only that we cannot have files with colons in the repo; it is irrelevant for files that a user brings.

@Askaholic independent of a checkout, can the code run with : in the filename on a windows system.

see my comments on #59


Fixing Issue #42

This commit fixes the problem of ecc-vscc crashes when processing
a transaction containing an empty range queries result. This may
happen, for instance, when an auction evaluation is invoked but
no bids are submitted. In detail, a range query is performed but
does not contain any raw reads.
Signed-off-by: Marcus Brandenburger<EMAIL_ADDRESS>

This issue fixes #42

Does fix the crash on both simulator and hw mode. Does not address the somewhat related security/functionality issue identified in issue #42: doesn't have to be fixed here but then we should create an issue separate of #42 for that? Leave it up to you which way to go, so approve here.

ok, will create a new issue for that and this here can be merged


drivers: nrfx: Allow simultaneous use of SPI_0 and I2C_0 on nRF52810

In most Nordic SoCs the SPI and TWI peripherals with the same instance
number share certain resources and cannot be used at the same time.
In nRF52810 there are only single instances of these peripherals and
they are arranged in a different way so this limitation does not apply.
Signed-off-by: Andrzej Głąbek<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9357   +/-   ##
=======================================
  Coverage   52.52%   52.52%
=======================================
  Files         201      201
  Lines       25359    25359
  Branches     5299     5299
=======================================
  Hits        13320    13320
  Misses       9944     9944
  Partials     2095     2095

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 3712fd3...78ea6b0. Read the comment docs.

@ioannisg
Suggested changes applied.
Regarding "other consequences": I don't see any. Technically SPIM0 in nRF52810 is the same what is SPIM1 in other nRF52s, it is just named (or indexed) differently, and TWIM0 is exactly the same in both cases. So when you use SPIM0 and TWIM0 on nRF52810 it is like you would use SPIM1 and TWIM0 on nRF52832 (and it actually is the case when you use the board nr52810_pca10040). The thing is that currently Kconfig does not allow you to do it.


Rename K6 CRD to TestRun

In order to improve readability of our interfaces, we should rename the K6 CRD from:
kind: K6

To:
kind: TestRun

Why? kind: K6 is not very descriptive as from the first glance, it might mean a lot of things. If we add new CRDs (see #191), we should try to remove the confusing names as well.
This is a breaking change for existing setups so consider supporting both names for some period of time.

TestRun CRD has been added as a duplicate of K6 CRD with the latter being marked as deprecated in #286
Missing dependency: #273
We'll support both K6 and TestRun for some time, to give people the opportunity to switch. But afterwards K6 will be removed. IOW, we didn't get to the "breaking" part yet.
I'm leaving this issue open for now. What is left:

[ ] updating documentation
[ ] removing K6 CRD in the future™

K6 CRD is removed in main branch (i.e. the next release) so closing this issue 🎉


Nuke RPC

What does this PR do?
Removes RPC. This is still up for discussion, this PR tests the waters.
Everything was found with  grep -irn "rpc" * --exclude .git
Arguments:

Complicated to use at the current state
Fragile
Potentially to be replaced by fully sharded
Can be re-added when there's a demand for it or the technology evolves.
No public usage found:
https://github.com/search?q=RPCSequentialPlugin&type=code
https://www.google.com/search?q="RPCSequentialPlugin"+filetype%3Apy

cc @justusschock @SeanNaren @tchaton @awaelchli @ananthsub
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[n/a] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Converting to draft until we know everybody is on the same page


Feature Request: Collision Detection System

Would like a collision system for pyglet. Please allow it to detect collisions between sprites, other sprites, and pyglet.shape's.
Maybe:
list_of_items = [item2, item3, ...]
@window.event
def on_collide(item1, list_of_items): #if item1 collides with any item in the list
    print("It has collided")

There are existing physics libraries out there such as pymunk. I think collision detection might be a bit out of scope for pyglet itself, however it is fairly simple to implement yourself. Here is a good article I would recommend using, it contains info on collision detection but you can take it further with resolution too.

I would have to agree that it's a bit out of scope. Pymunk is a good suggestion for complex simulations, and should integrate smoothly without much trouble.
For simple AABB collision detection, it would be fairly easy to roll your own.


feat: infer schema of row if not passed explicitly

Closes #15.
Summary of Changes
The schema parameter of the Row initializer is now optional. If it's not passed explicitly, it gets inferred from the data. The column names are set to col_0, col_1, etc.

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
6
3
0
1.17s

✅ PYTHON
flake8
6

0
0.43s

✅ PYTHON
isort
6
5
0
0.24s

✅ PYTHON
mypy
6

0
1.89s

❌ PYTHON
pylint
6

2
2.28s

✅ REPOSITORY
git_diff
yes

no
0.03s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

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

v0.8.0
GitHub release

Your semantic-release bot :package::rocket:


Stroke Colors

Hey,
I've a set color tokens. Is it possible to use this colors also for Stroke colors?
Maybe a solution is to hold the OPT Key and click on that token? Otherwise I have to set my stroke colors hard but sounds not like a plan :)

Yes, that's possible, you can right-click any color token and you should be able to select Fill or Border — you can then also choose a border width token to go along with it.
I'm not too happy with the right-click action, but I haven't thought about a good alternative just yet, especially as this one is so close to Figma's own behaviour.

Ah! Thanks.
But not possible to use CMD Click or OPT Click for secondary options? :)

Ahh, you mean clicking cmd / opt + to trigger a secondary action? I like that!
Could combine that with another key to trigger a third / 4th option as well 👌

Yeah, sounds great! How many layers are possible atm? :D

Currently the maximum actions we have are 4, on Spacing (All, Horiz, Vertical, Gap) — however these will change to the new auto layout properties (All, Top, Left  Right, Bottom)

Can you elaborate a bit on this? What if you only want this particular color token to be applied as a stroke? Should you tie it to a style in some way?

You can already apply a token to a layer's border by right-clicking and choosing Stroke. Layer needs to have a stroke set in Figma or have a borderWidth token set.
Should probably optimize that by defaulting to 1px borderWidth


ALVR dashboard can't connect to SteamVR unless system has specific libx264 version.

Description
When I'm using portable alvr_streamer_linux package if I press "Launch SteamVR", SteamVR starts, but the Dashboard can't connect to it and says: "SteamVR: Disconnected"/"The streamer is not connected! Clients Will not be discovered". When I look into SteamVR log (.local/share/Steam/logs/vrserver.txt), the following error is present:

[Info] - Unable to load driver alvr_server from /home/player/vr/alvr_streamer_linux/lib64/alvr/bin/linux64/driver_alvr_server.so (libx264.so.163: cannot open shared object file: No such file or directory)
[Error] - ASSERT: "DriverManager alvr_server load error libx264.so.163: cannot open shared object file: No such file or directoryu" at /data/src/common/vrcommon/drivermanager.cpp:439

My system has /usr/lib64/libx264.so.164 library, and /usr/lib64/libx264.so symlink that points to it. Version libx264.so.163 is deprecated and there's no way to install it using package management system.
If I manually create /usr/lib64/libx264.so.163 symlink pointing to /usr/lib64/libx264.so.164, the error changes to:

[Info] - Unable to load driver alvr_server from /home/player/vr/alvr_streamer_linux/lib64/alvr/bin/linux64/driver_alvr_server.so (/home/player/vr/alvr_streamer_linux/lib64/alvr/bin/linux64/driver_alvr_server.so: undefined symbol: x264_encoder_open_163)
[Error] - ASSERT: "DriverManager alvr_server load error /home/player/vr/alvr_streamer_linux/lib64/alvr/bin/linux64/driver_alvr_server.so: undefined symbol: x264_encoder_open_163u" at /data/src/common/vrcommon/drivermanager.cpp:439.
[Error] - Unable to load driver alvr_server because of error VRInitError_Init_FileNotFound(103). Skipping.

If I download libx264.so.163 from rpmfind.net and put it into /usr/lib64 - Dashboard correctly connects to SteamVR.
From what I can conclude that version 163 is more than just and option - it's a requirement for the current ALVR Dashboard.
This situation encourages me to propose to either make the ALVR code x264 version independent or to put the specific version in the requirements.
General Troubleshooting

[x] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR Wikis here and here

Environment
Gentoo (amd64/17.1/desktop/gnome profile)
Wayland 1.22.0
Sway 1.8.1 (despite having Gnome profile, I use Sway for gaming because Gnome + Wayland can't into SteamVR)
Hardware
CPU:
i9-9900K
GPU:
AMD Vega 64
GPU Driver Version:
amdgpu
Installation
ALVR Version:
portable alvr_streamer_linux.tar.gz 20.5.0
portable alvr_streamer_linux.tar.gz 21.0.0-dev00+nightly.2023.12.29
ALVR Settings File:
default
SteamVR Version:
2.2.3
Install Type:

[ ] Packaged (exe, deb, rpm, etc)
[x] Portable (zip)
[ ] Source

OS Name and Version (winver on Windows or grep PRETTY_NAME /etc/os-release on most Linux distributions):
Gentoo Linux 6.1.60

its an issue with the "portable" build not being portable. It is dependent on a specific Ubuntu version I'm pretty sure if you read our docs we don't recommend it. I don't think I got to tell you that in the original help request

either you should compile by source or use appimage(I know packaging on linux sucks)


Crash when clicking "read detailed write-up" for stay healthy street proposal

Under "Community Proposals", for "Convert Lake Washington Blvd to a Stay Healthy Street", selecting "Read detailed write-up" causes the game to crash.

output.txt

Initial inner window size is PhysicalSize { width: 1366, height: 705 }, monitor is PhysicalSize { width: 1366, height: 768 }, scale factor is 1
load map...
Loading map ../data/system/maps/montlake.bin
Reading ../data/system/maps/montlake.bin: 0/4 MB... 0.0001s
Reading ../data/system/maps/montlake.bin: 0/4 MB... 0.2015s
Reading ../data/system/maps/montlake.bin: 1/4 MB... 0.4035s
Reading ../data/system/maps/montlake.bin: 2/4 MB... 0.6036s
Reading ../data/system/maps/montlake.bin: 3/4 MB... 0.8970s
Reading ../data/system/maps/montlake.bin: 4/4 MB... 1.0972s
Read ../data/system/maps/montlake.bin (4)... 1.1793s
create sim...
create sim took 0.0081s
Map and Sim cost ~0 MB
draw_map...
make DrawRoads (491)... 0.1500s
generate thick roads...
generate thick roads took 0.0158s
prepare DrawLanes: 2,347/2,381... 0.2001s
prepare DrawLanes (2,381)... 0.2029s
finalize DrawLanes (2,381)... 0.1794s
make DrawIntersections (371)... 0.1502s
generate unzoomed intersections...
generate unzoomed intersections took 0.0015s
make DrawBuildings: 2,304/3,014... 0.2001s
make DrawBuildings (3,014)... 0.2559s
upload all buildings...
upload all buildings took 0.0272s
make DrawParkingLot...
make DrawParkingLot took 0.0148s
make DrawBusStop (16)... 0.0041s
make DrawAreas (54)... 0.0002s
upload all areas...
upload all areas took 0.0001s
create quadtree...
create quadtree took 0.2849s
static DrawMap consumes 16 MB on the GPU
draw_map... plus 0.2683s
draw_map took 1.5554s
DrawMap cost ~0 MB
load map... plus 0.0406s
load map took 2.7834s
- load map took 2.7834s
  - Read ../data/system/maps/montlake.bin (4)... 1.1793s
  - create sim took 0.0081s
  - draw_map took 1.5554s
    - make DrawRoads (491)... 0.1500s
    - generate thick roads took 0.0158s
    - prepare DrawLanes (2,381)... 0.2029s
    - finalize DrawLanes (2,381)... 0.1794s
    - make DrawIntersections (371)... 0.1502s
    - generate unzoomed intersections took 0.0015s
    - make DrawBuildings (3,014)... 0.2559s
    - upload all buildings took 0.0272s
    - make DrawParkingLot took 0.0148s
    - make DrawBusStop (16)... 0.0041s
    - make DrawAreas (54)... 0.0002s
    - upload all areas took 0.0001s
    - create quadtree took 0.2849s
    - ... plus 0.2683s
  - ... plus 0.0406s
4 notes:
Loading map ../data/system/maps/montlake.bin
Map and Sim cost ~0 MB
static DrawMap consumes 16 MB on the GPU
DrawMap cost ~0 MB
1 warnings:
Parking lane Lane #2178 has no sidewalk!
- load map took 2.7834s
Unknown winit key LWin
Unknown winit key LWin
load_all_objects from ../data/system/proposals...
parse ../data/system/proposals/cap hill superblock.json...
parse ../data/system/proposals/cap hill superblock.json took 0.0008s
parse ../data/system/proposals/simplify burke corliss.json...
parse ../data/system/proposals/simplify burke corliss.json took 0.0003s
parse ../data/system/proposals/stay healthy lake wash blvd.json...
parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0005s
load_all_objects from ../data/system/proposals... plus 0.0008s
load_all_objects from ../data/system/proposals took 0.0023s
- load_all_objects from ../data/system/proposals took 0.0023s
  - parse ../data/system/proposals/cap hill superblock.json took 0.0008s
  - parse ../data/system/proposals/simplify burke corliss.json took 0.0003s
  - parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0005s
  - ... plus 0.0008s
- load_all_objects from ../data/system/proposals took 0.0023s
load_all_objects from ../data/system/proposals...
parse ../data/system/proposals/cap hill superblock.json...
parse ../data/system/proposals/cap hill superblock.json took 0.0007s
parse ../data/system/proposals/simplify burke corliss.json...
parse ../data/system/proposals/simplify burke corliss.json took 0.0003s
parse ../data/system/proposals/stay healthy lake wash blvd.json...
parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0005s
load_all_objects from ../data/system/proposals... plus 0.0006s
load_all_objects from ../data/system/proposals took 0.0021s
- load_all_objects from ../data/system/proposals took 0.0021s
  - parse ../data/system/proposals/cap hill superblock.json took 0.0007s
  - parse ../data/system/proposals/simplify burke corliss.json took 0.0003s
  - parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0005s
  - ... plus 0.0006s
- load_all_objects from ../data/system/proposals took 0.0021s
Unknown winit key LWin
Unknown winit key LWin
Unknown winit key LWin
Unknown winit key LWin
load_all_objects from ../data/system/proposals...
parse ../data/system/proposals/cap hill superblock.json...
parse ../data/system/proposals/cap hill superblock.json took 0.0006s
parse ../data/system/proposals/simplify burke corliss.json...
parse ../data/system/proposals/simplify burke corliss.json took 0.0002s
parse ../data/system/proposals/stay healthy lake wash blvd.json...
parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0004s
load_all_objects from ../data/system/proposals... plus 0.0006s
load_all_objects from ../data/system/proposals took 0.0019s
- load_all_objects from ../data/system/proposals took 0.0019s
  - parse ../data/system/proposals/cap hill superblock.json took 0.0006s
  - parse ../data/system/proposals/simplify burke corliss.json took 0.0002s
  - parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0004s
  - ... plus 0.0006s
- load_all_objects from ../data/system/proposals took 0.0019s
Unknown winit key LWin
Unknown winit key LWin
Unknown winit key LWin
Unknown winit key LWin
load_all_objects from ../data/system/proposals...
parse ../data/system/proposals/cap hill superblock.json...
parse ../data/system/proposals/cap hill superblock.json took 0.0006s
parse ../data/system/proposals/simplify burke corliss.json...
parse ../data/system/proposals/simplify burke corliss.json took 0.0002s
parse ../data/system/proposals/stay healthy lake wash blvd.json...
parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0004s
load_all_objects from ../data/system/proposals... plus 0.0006s
load_all_objects from ../data/system/proposals took 0.0018s
- load_all_objects from ../data/system/proposals took 0.0018s
  - parse ../data/system/proposals/cap hill superblock.json took 0.0006s
  - parse ../data/system/proposals/simplify burke corliss.json took 0.0002s
  - parse ../data/system/proposals/stay healthy lake wash blvd.json took 0.0004s
  - ... plus 0.0006s
- load_all_objects from ../data/system/proposals took 0.0018s
Unknown winit key LWin
Unknown winit key LWin
thread 'main' panicked at 'either event handler is re-entrant (likely), or no event handler is registered (very unlikely)', C:\Users\runneradmin\.cargo\registry\src\github.com-1ecc6299db9ec823\winit-0.22.2\src\platform_impl\windows\event_loop\runner.rs:235:37
stack backtrace:
   0: <unknown>
   1: <unknown>
   2: <unknown>
   3: <unknown>
   4: <unknown>
   5: <unknown>
   6: <unknown>
   7: <unknown>
   8: <unknown>
   9: <unknown>
  10: <unknown>
  11: <unknown>
  12: <unknown>
  13: <unknown>
  14: <unknown>
  15: <unknown>
  16: <unknown>
  17: <unknown>
  18: <unknown>
  19: DefSubclassProc
  20: DefSubclassProc
  21: CallWindowProcW
  22: DispatchMessageW
  23: SendMessageTimeoutW
  24: KiUserCallbackDispatcher
  25: NtUserDispatchMessage
  26: DispatchMessageW
  27: SHSetTemporaryPropertyForItem
  28: Ordinal900
  29: Ordinal99
  30: ShellExecuteExW
  31: ShellExecuteExW
  32: ShellExecuteW
  33: <unknown>
  34: <unknown>
  35: <unknown>
  36: <unknown>
  37: <unknown>
  38: <unknown>
  39: <unknown>
  40: <unknown>
  41: <unknown>
  42: <unknown>
  43: <unknown>
  44: <unknown>
  45: <unknown>
  46: <unknown>
  47: <unknown>
  48: <unknown>
  49: <unknown>
  50: <unknown>
  51: <unknown>
  52: DefSubclassProc
  53: DefSubclassProc
  54: CallWindowProcW
  55: CallWindowProcW
  56: glPushClientAttrib
  57: CallWindowProcW
  58: DispatchMessageW
  59: <unknown>
  60: <unknown>
  61: <unknown>
  62: <unknown>
  63: <unknown>
  64: <unknown>
  65: <unknown>
  66: <unknown>
  67: <unknown>
  68: <unknown>
  69: BaseThreadInitThunk
  70: RtlUserThreadStart
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

The proposal page (https://dabreegster.github.io/abstreet/lake_wash/proposal.html) does open correctly in my default browser (Firefox), though.

I'm using https://crates.io/crates/webbrowser to open URLs. Huh, maybe it interacts strangely with winit (the window management library) strangely on Windows. Looking into it

I think this is the same root cause as https://github.com/rust-windowing/winit/issues/1602 and https://github.com/rust-windowing/winit/issues/1587.

Waiting for https://github.com/rust-windowing/winit/commit/dd866a74a690d37fdd11385741dee06e9f7cd02f to make it into a winit release

We're now pinning to a new winit build that should have the fix. Give https://github.com/dabreegster/abstreet/releases/tag/v0.2.8 a shot and please reopen the bug if it's still happening. Thanks!

The stay healthy street proposal doesn't seem to exist in that version (I don't see any proposals at all), but "See full credits" under About also opens a web page; that one also crashed in v0.2.0d, but now doesn't crash in v0.2.8.  So, yep, it's fixed, thanks!

Great! Indeed the proposals are missing from the release; I'll look into that


Bug fix

bring Spinner to front to solve the problem of Hierarchy mistake when change rootViewController

Thanks for sending this!


feat(*): make multiple plugins support the RPC builder

What this PR does / why we need it:
Other plugins like metrics rely on configuring the middleware for the root path (/) to automatically configure the middleware for all paths which doesn't work for the RPC API style, which requires manual configuration of middleware and descriptor for the RPC style server.
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #382

Special notes for your reviewer:
/cc @xpofei

Release note:

NONE

/lgtm

/approve
/cherrypick release-0.3


Not able to pass values to parent scope!!

You can access in your controller from which you are using this component. From controller i think you can manipulate how ever you need.

Consider this case:
Added one more line in html
 Show Dte
added one function in controller
...
$scope.currentDate = new Date();
...
//new Block
$scope.showDate = function() {
alert($scope.currentDate);
}
here alert always printing new Date only...y?

Its not updating parent controller $scope.currentDate

Heyyy...Its working..  I was trying using $scope.currentDate..
But its working with this.currentDate
Thanks,
Good Job @rajeshwarpatlolla

@rijine how to use the this, to get the value updated in the parent controller?

Thank you @rijine.

Can u help me in this, because it is not showing the selected value.

@anshulklr  for example your html has idate="currentDate"  within tag. In controller declare a variable like { var vm=this;  vm.currentDate=new Date(); } Try this and let me know if u face any problem

@rijine dude i'm still unable to do this,
idate="from_date"
in the controller
var check=this;
check.from_date= new Date();
now i can't see the date which earlier comes

can u share ur code in plunkr?

the code is too big , that i can't share everything, but as explained by @rajeshwarpatlolla  i did the same, i'm getting the date on the view but if i check by console.log(), it gives the right now date ie the value is not updating over there

how many controllers u r using

http://collabedit.com/fb5qw
come to this link

yes..!!

Call back function is added in v0.3.0. Please install this version to get the correct values in to the controller.


Hide WakaTime Status from Status Bar

The Statusbar in Sublime Text is pretty coveted space and many plugins have to fight for space in it.
I love wakatime, but its really passive for me so I don't necessarily care about the status of wakatime at all times.
Is there anyway to hide this? I much rather save the space for my wordcount and git status.

There's a setting for it, as can be seen in Sublime in Preferences -> WakaTime -> Settings - Default. You can add this to your User preference file:
    // Status bar message. Set to false to hide status bar message.
    // Defaults to true.
    "status_bar_message": true

Exactly what I was looking for - thanks!


Reading from the header file to get the location of json

Create a Doc PR in fpm.dev and connect it with this pr @ADITIRAI290698

@Arpita-Jaiswal this needs documentation .

The documentation pull request has been attached sucessfully.

Approved! Code is up to date with the documentation


chore(starknet_consensus_manager): add chain_id to config

This change is

#2433  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Add Dockerfile with required environment for scancode

Purpose
Add Dockerfile with scancode requirements.
Python is downgraded to 3.6 and scancode has been added.
Actions

Moved existing Dockerfile to docker directory
Add new Dockerfile.scancode in docker directory

Related issue.
This has some links with #755
Test
I've created a new docker image philipssoftware/tern:2-scancode for scancode and pushed it to docker hub. ( https://github.com/philips-software/docker-tern )
docker run --rm -v /var/run/docker.sock:/var/run/docker.sock philipssoftware/tern:2-scancode report -x scancode -i alpine:latest > alpine.txt
Unable to find image 'philipssoftware/tern:2-scancode' locally
65b3db15f518: Pull complete
850581be87f3: Pull complete
f355d2a41bb6: Pull complete
a72a02a55e5c: Pull complete
cb0d06c13dc6: Pull complete
7bd231b26968: Pull complete
6625882c9040: Pull complete
f57a6d65b5ce: Pull complete
Digest: sha256:2bd5da481fd346959b9cccedcd140e4558a6df523c6f14dba2dae5d5a37f9358
Status: Downloaded newer image for philipssoftware/tern:2-scancode
2020-09-11 14:21:33,159 - DEBUG - __main__ - Starting...
2020-09-11 14:21:33,159 - DEBUG - run - Setting up...
2020-09-11 14:21:33,179 - DEBUG - container - Checking if image "alpine:latest" is available on disk...
2020-09-11 14:21:33,183 - DEBUG - container - Attempting to pull image "alpine:latest"
2020-09-11 14:21:35,827 - DEBUG - container - Image "alpine:latest" downloaded
2020-09-11 14:21:35,827 - DEBUG - container - Checking if image "alpine:latest" is available on disk...
2020-09-11 14:21:35,830 - DEBUG - container - Image "alpine:latest" found
2020-09-11 14:21:36,019 - DEBUG - rootfs - Running command: tar -tf /root/.tern/temp.tar
2020-09-11 14:21:36,023 - DEBUG - rootfs - Running command: tar -x --exclude=.wh.* -f /root/.tern/temp.tar -C /root/.tern/temp
2020-09-11 14:21:36,035 - DEBUG - rootfs - Running command: tar -tf /root/.tern/temp/fa27ccac46e9a5b7ff1d188e99763db1a5fb45adb34a917c1c482dfde99ad432/layer.tar
2020-09-11 14:21:36,040 - DEBUG - rootfs - Running command: tar -x --exclude=.wh.* -f /root/.tern/temp/fa27ccac46e9a5b7ff1d188e99763db1a5fb45adb34a917c1c482dfde99ad432/layer.tar -C /root/.tern/temp/fa27ccac46e9a5b7ff1d188e99763db1a5fb45adb34a917c1c482dfde99ad432/contents
2020-09-11 14:21:36,081 - DEBUG - rootfs - Running command: chmod +x /usr/local/lib/python3.6/site-packages/tern/tools/fs_hash.sh
2020-09-11 14:21:36,089 - DEBUG - rootfs - Running command: /usr/local/lib/python3.6/site-packages/tern/tools/fs_hash.sh /root/.tern/temp/fa27ccac46e9a5b7ff1d188e99763db1a5fb45adb34a917c1c482dfde99ad432/contents
2020-09-11 14:21:36,332 - DEBUG - common - Reading files in filesystem...
2020-09-11 14:21:36,336 - DEBUG - rootfs - Running command: /usr/local/bin/scancode -ilpcu --quiet --timeout 300 --json - /root/.tern/temp/fa27ccac46e9a5b7ff1d188e99763db1a5fb45adb34a917c1c482dfde99ad432/contents
2020-09-11 14:23:39,379 - DEBUG - executor - Collecting file data...
2020-09-11 14:23:39,398 - DEBUG - generator - Creating a detailed report of components in image...
2020-09-11 14:23:39,399 - DEBUG - rootfs - Running command: rm -rf /root/.tern/temp/fa27ccac46e9a5b7ff1d188e99763db1a5fb45adb34a917c1c482dfde99ad432/contents
2020-09-11 14:23:39,410 - DEBUG - run - Teardown...
2020-09-11 14:23:39,417 - DEBUG - rootfs - Running command: rm -rf /root/.tern/temp/mergedir
2020-09-11 14:23:39,420 - DEBUG - rootfs - Running command: rm -rf /root/.tern/temp/workdir
2020-09-11 14:23:39,426 - DEBUG - __main__ - Report completed.
2020-09-11 14:23:39,427 - DEBUG - __main__ - Finished

output:
$  cat alpine.txt
This report was generated by the Tern Project
Version: 2.2.0

Docker image: alpine:latest:
Generated with ScanCode and provided on an "AS IS" BASIS, WITHOUT WARRANTIES
OR CONDITIONS OF ANY KIND, either express or implied. No content created from
ScanCode should be considered or used as legal advice. Consult an Attorney
for any legal advice.
ScanCode is a free software code scanning tool from nexB Inc. and others.
Visit https://github.com/nexB/scancode-toolkit/ for support and download.

	Layer 1:
	File licenses found in Layer:  mit AND bsd-new, gpl-2.0, gpl-2.0-plus, bsd-simplified, bsd-new, openssl-ssleay, mpl-2.0
	Packages found in Layer:  None
	Licenses found in Layer:  None
------------------------------------------------

###########################################
# Summary of licenses found in Container: #
###########################################
mit AND bsd-new, gpl-2.0, gpl-2.0-plus, bsd-simplified, bsd-new, openssl-ssleay, mpl-2.0

@JeroenKnoops  I've managed to install scancode in a python 3.8 environment using pip install scancode-toolkit[full].

@JeroenKnoops I've managed to install scancode in a python 3.8 environment using pip install scancode-toolkit[full].

That would be awesome, although the site is still saying scancode is not working with python version above 3.6.x
https://scancode-toolkit.readthedocs.io/en/latest/getting-started/install.html#prerequisites

ScanCode currently doesn’t support versions above Python 3.6.x, though support will be added soon.

@JeroenKnoops I've managed to install scancode in a python 3.8 environment using pip install scancode-toolkit[full].

That would be awesome, although the site is still saying scancode is not working with python version above 3.6.x
https://scancode-toolkit.readthedocs.io/en/latest/getting-started/install.html#prerequisites

ScanCode currently doesn’t support versions above Python 3.6.x, though support will be added soon.

Looks like soon is here :)
https://github.com/nexB/scancode-toolkit/issues/2312#issuecomment-722687591

@JeroenKnoops thank you!


[BUG]Logs viewed in Rancher are not showing line breaks

Rancher Server Setup

Rancher version: v2.6.8
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.24
Cluster Type (Local/Downstream): eks

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Describe the bug

rancher view logs:

kubectl logs output

To Reproduce

Result
Expected Result

Screenshots

Additional context

This looks like a specific problem with EKS. Transferring to the UI to triage
@chestarss can you add steps to reproduce?
@kerecsen can you confirm the rancher version?
@chestarss , @skirsch-alethix , @kerecsen it's not clear which logs these are. can you confirm if they're pod, job, helm chart install, etc?

my rancher version is 2.6.8 and the logs are from pod

@richard-cox These are pod logs. Behavior is across the board. All pod logs now appear this way in Rancher immediately after upgrade to 1.24. When you download the log using the Download button under the pod log view, the downloaded log looks good.

It isn't EKS specific either, we have the exact same issue with AKS (Azure Kubernetes Service)

Getting this on Rancher 2.7.1 with a  local cluster running v1.24.11+k3s1

This happens on 2.6.9 and version 2.7.4 and 2.7.5. However, in the 2.6.9 environment, if you view the logs via the old UI, the logs display correctly.

How to switch to old UI ? @wormza

we also have a similar problem
you can deselect the wrap lines option when you want to view logs.
The new trailing logs will come in a proper line by line format, but if we want to view logs for last 12 hrs then the old logs would be somewhere still messy.

How to switch to old UI ? @wormza

Sorry for the delayed reply. If you add /g/clusters to the end of your URL it will take you to the old UI.
eg: rancher.domain.com/g/clusters

Please note accessing the old UI via /g is not supported. There be dragons.

For sure, but for viewing logs until the issue is sorted it’s the only way. I wouldn’t use it for anything else.

Tests pass on  v2.9-16190ecee0aa3fb4aa570aa0fc932a36d9dcd082-head


Add support for archiving GitHub mirrors automatically based on "archived" API parameter on the remote repo

Feature Description
Currently, there isn't an archive option for mirror repositories (and it is available for only regular ones as far as I can remember).
It would be great if Gitea could read the "archived" parameter from the GitHub mirror's remote API configuration during sync. If the archive value is set to true then the mirror can be archived automatically.
Once the mirror is archived, we can continue to query the API alone (while disabling other sync operations) to check if the repo has been unarchived. If the repo has been unarchived, the mirror can also be unarchived (and repo sync can be done as set).

Examples

Example archived repo with archived parameter set to "true" in the API.
Example normal repo with archived parameter set to "false" in the API.

Screenshots
No response

I don't think this will work reliable. The default mirror interval is 8h. If you unarchive, change files and re-archive a repo on Github inbetween this time slot, Gitea never sees a change.
And after the migration we don't know the source type of the mirror anymore because only the git remote data is needed.


Remove master branch and references to the word such as master, slave, blacklist and whitelist.

Words such as master, slave, blacklist and whitelist are not suitable due to the deep history associated with racism.
They can be more accurate and precise than they are currently.
Golang team has already removed the use of those words in their code along with few others - https://www.reddit.com/r/golang/comments/gy9ylr/go_has_removed_all_uses_of_blacklistwhitelist_and/
There are other google projects which have done the same.
Here is a tweet from the github CEO - https://twitter.com/Una/status/1271180494944829441
Rust and python team is planning to do it as well.
There are many more projects moving to remove racism from the code. Would you please consider doing the same for flutter?
PS - Historical context for main - https://www.nakasendoway.com/the-japanese-family-system/
Please consider alternatives such as canary, develop or edge.

Please close this racist issue and ban @ghost

This is so dumb.

Dumbest thing ever.. this is not mature.

https://github.com/flutter/flutter/pull/59120 and https://github.com/flutter/flutter/pull/59484 are already in progress.
I'm not entirely clear on the problem with the word "master" in isolation (i.e. when not used in its colonial sense)? Can you elaborate?

Since the original poster here has gone away, I am going to close this issue since we don't need it to track the work of continuing to apply our code of conduct.

To close the loop on the master branch question, someone pointed me to https://mail.gnome.org/archives/desktop-devel-list/2019-May/msg00066.html which seems like a compelling argument to rename "master". We'll add that to our list of things to do (it being a rather significant and disruptive change, it may take longer than other changes we are making here).


Consider Using Docker Compose for Testing/Development

This is only an idea, so feel free to close this issue right away. After reading your testing.md file, I realized that you're doing the same thing I am with docker-compose. The added benefit of using docker-compose is that the docker commands can be easily shared via a docker-compose.yml file, and you can easily mount your local dev copy into the container. This allows you to both test and develop the nodes at the same time from your host.
version: '3'
services:
  nodered:
    image: nodered/node-red-docker:0.19.5-v8
    ports:
    - 1880:1880
    volumes:
    - .:/local # Your local working copy
    - ./data:/data # Persist configuration between sessions

The following commands accompany this setup:

docker-compose up -d - Bring up the dev environment
docker-compose exec nodered sh -c "cd /data && npm i /local" - Install your local working copy as nodes
docker-compose restart - Restart NodeRED (after npm i and any source code changes)
docker-compose exec nodered bash - Get a shell into the container

/data should probably be added to .gitignore, since its just your local NodeRED installation.
Edit: Corrected "install local copy" command.

interesting, I need to think about this a little more so parking for now but I'll come back to it when I have some more brain capacity


Bootstrap data-toggle for collapse failing

Describe the bug
What went wrong? What did you expect to happen?
The SinopiaResourceTemplate's history function is now failing  https://github.com/LD4P/sinopia_editor/blob/9670178e3e950f50c2c7b4460e70acc7c74bf434/src/components/templates/SinopiaResourceTemplates.jsx#L51-L53
These honeybadger alerts started after Monday's dependencies that updated the jQuery package to 3.5.0.

What is the impact on your work? (can't work / work stoppage; workaround exists; minor issue)
Regression.

What environment are you in? (stage or production)
Both

What OS and browser are you using? (Mac/Windows; Safari, Firefox, etc.)

If you're cataloging, what Resource Template were you using (name or ID)

Please include a screenshot
(Mac: shift-command-4; Windows: ; then drag the image file into the issue)

Looks like https://github.com/twbs/bootstrap/issues/30553

Merging PR #2158, re-opens this bug with corresponding HoneyBadger alerts.

jQuery 3.5.1 should fix it


Creating a database

Do you have a chance to create a separate database for each guild? For example, a channel added to one guild should not be shared in another. I was able to connect the channel system to the database with the command, but I could not create the twitch channels.

I'm not 100% sure I understand your question, but right now the bot only supports sending stream notifications to all the servers it is in, not a per-server configuration.
If v2 ever gets finished, then this will be possible, but not right now basically. :)

I'm not 100% sure I understand your question, but right now the bot only supports sending stream notifications to all the servers it is in, not a per-server configuration.
If v2 ever gets finished, then this will be possible, but not right now basically. :)


[SR-4101] Return early if cached children are present in Syntax nodes

Previous ID
SR-4101

Radar
None

Original Reporter
@bitjammer

Type
Bug

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug, Syntax

Assignee
@bitjammer

Priority
Medium

md5: 13aed5b30928c4c2786db62d5e70c8d5

libSyntax was removed in #62145 and succeeded by swift-syntax.


Fix mutexName inconsistency caused by different PHP binary paths on multiple servers

Problem Description:
Our production environment handles around 200 million requests daily, and we initially had three Ubuntu 22.04 servers running PHP 8.3, where the PHP executable path was:
/usr/bin/php8.3

Recently, we decided to expand the infrastructure and added a fourth server running AlmaLinux 9.5. While the setup was consistent across servers, the PHP executable path on the new server differed:
/opt/remi/php83/root/usr/bin/php

This caused an issue with the mutexName function when using Laravel’s onOneServer()->withoutOverlapping() feature for scheduling commands. The $this->command string varied between servers as follows:

Existing servers (Ubuntu):

'/usr/bin/php8.3' 'artisan' my-command-signature-here

New server (AlmaLinux):

'/opt/remi/php83/root/usr/bin/php' 'artisan' my-command-signature-here

Root Cause:
The mutexName function generates a unique name based on the $this->command. Since the PHP executable paths differed across servers, the resulting mutex name was inconsistent. This led to the same scheduled command being executed on multiple servers simultaneously, despite using ->onOneServer()->withoutOverlapping().
Real-Life Impact:
For example, when scheduling the following command in Kernel.php:
$scheduler->command('my-command-signature-here')->onOneServer()->withoutOverlapping();

The inconsistency in mutexName caused the command to run:

Once on the new server (AlmaLinux).
And once on one of the existing servers (Ubuntu), selected at random.

This behavior defeats the purpose of ->withoutOverlapping(), which was critical in our case (high-traffic production environment).
Solution:
To resolve this issue, I updated the code to strip the PHP binary path from the $this->command. By doing this, the mutexName function will now generate consistent names regardless of the PHP executable's location or version.
For example:

Input command:

'/usr/bin/php8.3' 'artisan' my-command-signature-here

or
'/opt/remi/php83/root/usr/bin/php' 'artisan' my-command-signature-here

After modification:

'artisan' my-command-signature-here

The mutex name will now be derived only from the relevant parts of the command (i.e., 'artisan' my-command-signature-here), ensuring consistency across servers.
Testing Considerations:
Unfortunately, testing this specific scenario programmatically is challenging because it requires:

Multiple servers with different operating systems.
Differing PHP binary paths.

Given these constraints, I was unable to include automated tests for this PR. However, the issue has been thoroughly tested in our production environment, and the proposed change resolves the problem reliably.

Is this going to actually be robust? Not every scheduled command calls PHP for example... Schedule::exec('some arbitrary linux command') ... the explode and string slicing wouldn't seem correct for these cases?
For reference, you can specify a mutex name like this: Schedule::exec('')->createMutexNameUsing(fn () => 'some-mutex-name').

@taylorotwell Hmm, makes sense.
I've never used Linux commands in the Laravel Scheduler before.
You were right.
I noticed something similar in ScheduleListCommand.php, so I added a new method to Event.php and used it in both cases.
I hope this approach is robust enough.


BadValue: need to enable SSL via the sslMode flag when using SSL configuration parameters

Hi Folks,
I'm attempting to spin up a mongo container w/ ssl from the stock 3.4 image with something like this:
docker run -d --name dev-hrs-db -v /home/core/repos/mongodb/conf/dev-hrs-db.conf:/etc/mongod.conf -v /data/dev-hrs-db:/data -v /home/core/certs:/certs -p 27033:27033 mongo:3.4 mongod -f /etc/mongod.conf
mongod fails with this error:
BadValue: need to enable SSL via the sslMode flag when using SSL configuration parameters try 'mongod --help' for more information
Which is strange since all the ssl directives are in the conf file. When I add all the ssl directives directly on the command line, mongod gets a little further but then fails with the following error:
2017-04-25T21:25:36.135+0000 I STORAGE  [initandlisten] exception in initAndListen: 20 Attempted to create a lock file on a read-only directory: /data, terminating
Also, if I just launch the container w/ -it and then manually start mongod -f /etc/mongod.conf in the container, it works just fine, which tells me the conf file is legit and accessible from within the container, and the /data mount is ok as well.
For the life of me I cannot figure out why this is happening; I have a vague suspicion that it might be an issue with permissions on the filesystem in the image relating to the -v mounts, but I really have no idea. It's also worth noting that I've had success previously with this setup, so I'm wondering if something changed in the image?
Any help is welcome!

https://github.com/docker-library/mongo/issues/148 is semi-related -- this is due to the new changes introduced for "initdb" functionality (see also https://github.com/docker-library/mongo/pull/169 and https://github.com/docker-library/mongo/pull/167).  Parsing the --config file to determine that we need --sslMode is very complex. :disappointed: :cry:

Thanks for the swift response tianon; appreciate it!
For now as a workaround I've:

changed the container's internal data path to /data/db
passing all the ssl params on the command line so they don't need to be parsed from the yaml conf file.

Nice, so that was successful for you?

yes, i've got it working now:

our internal db path can be whatever, so it's no problem to change it to the default (/data/db)
it's a bit cludgy to have to push ssl configuration from the conf file to the command line; is this going to be resolved in the future?

i ran into this same issue both with version 3.4.3 and 3.4.4.

@sdanieru possibly -- we're debating whether it's worth trying to parse the YAML configuration, but it's a complicated proposition to do so.
The code currently only checks whether you've specified --sslPEMKeyFile for whether it should set --sslMode to allowSSL or to disabled (https://github.com/docker-library/mongo/blob/ce12e5eba93b6c60c2b19791c7383f03272d7643/3.4/docker-entrypoint.sh#L139), so that's technically the only one you need to "pull out".  We've also considered updating that code to check for --sslMode so that one could simply specify --sslMode on the command line to inform the initdb code that SSL is necessary.

@tianon, then we have to check the value of --sslMode, since they could be setting it to disabled. :disappointed:  sslPEMKeyFile was chosen since if it exists, then the server has to be using ssl.
On the other hand, you can't set sslMode disabled and still have other ssl config, since that would fail to start, so I doubt many users are setting explicit disabled.

I have currently derived a docker container which takes some CLI and initializes the mongod.conf accordingly and then calls the original entrypoint.sh script with mongod --config <conf file> arguments.
It works fine till I enable SSL. The error I get is:
warning: database is not yet initialized, and "--config" is specified
  the initdb database startup might fail as a result!

BadValue: cannot have x.509 cluster authentication in allowSSL mode
try 'mongod --help' for more information

And when I specify all the parameters on the CLI, the sslMode gets set to allowSSL rather than requireSSL.
I went through the docker-entrypoint.sh script but I cannot understand why has it been hacked in the first place! Am I missing something obvious here?

@krish7919 it's for the new initdb behavior -- the MongoDB configuration file is a bit complex to parse via Bash (it's YAML, and doesn't match the CLI flags 1-to-1), but we need to start mongod temporarily with some slightly customized options in order to create the initial user and run any startup scripts
The sslMode parameter is only set to allowSSL for the short period that mongod is started during initdb listening only on <IP_ADDRESS> -- once that's done, mongod should start with whatever parameters are passed, unchanged (possibly with the addition of --auth, depending on whether username/password were specified and --auth was not already supplied).

@tianon : So correct me if I am wrong:
If I start mongod with the default entrypoint script with a MONGO_INITDB_ROOT_USERNAME and a MONGO_INITDB_ROOT_PASSWORD, it would create a default admin user for further access?
And once I log in to the mongo shell using this, I can create other users and roles?


Automated Inference Generation

Context
Now that MLEnabler is able to build and store ECS images in the ECR, as well as to store Image API locations, this PR adds the ability to do the following:

Spin up a custom inference stack (Chip-N-Scale Like)
Send and track chips being run against the inference stack
Save inferences directly to the MLEnabler DB
Destroy the stack upon inference completion

This PR will migrate and heavily modify the Chip-N-scale to live inside the MLEnabler project.
Tasks

[x] Export values from main stack required by substack to facilitate resource sharing
[ ] Create CF Template for creating a prediction ECS task/service
[ ] Add chip-n-scale lambda build to MLEnabler
[ ] Add Lambda to prediction CF Template
[ ] Create UI for creating stack and generating chip lists
[ ] ... more to come ...

cc/ @martham93 @Geoyi @geohacker

A few screenshots of the new Prediction component

@ingalls !!!!!!!!!!!!!!! this is looking spectacular. Thank you! Can we all get a demo sometime?

Ok going to sign off, TODO tomorrow:

[ ] Investigate why tiletanic is only producing 4 tiles for large geometries at z18 (is it only picking up corners?!)
[ ] SQS=>Lambda works (:tada:), but lambda fails to actually do anything useful (unsurprising since I haven't touched this code other than stealing it from chip-n-scale)

the option of having the imagery selection panel is nice. Would it be possible for a user to specify a tile end point? Some projects get specific imagery end points that couldn't always be listed as an option.
cc @ingalls

@martham93 yes, that was in a subsequent PR. You add imagery sources via the Models panel. This lists the imagery sources that have been added to a model


Scope the entire script so it executes in one step

Use brackets to ensure this. Also ensure the script does not require a trailing newline.

@jeffbaumes could you duplicate this PR here?

Sure, but I need push access to girder_worker.

Whoops, you should now have access.

Done https://github.com/girder/girder_worker/pull/2


Can rerun failed tests inside PHPStorm

Describe the bug
Ever since upgrading my repo to using PestPHP v2, I can no longer return failed tests.
Instead I get the following message.
Failed finding any tests to rerun Rerunning support was first added in Pest v0.3.5

To Reproduce
Steps to reproduce the behavior:

Create a simple test in with two or more tests. One should fail.

test('this test should pass', function () {
    $this->assertTrue(true);
});

test('this test should fail', function () {
    $this->assertTrue(false);
});

test('this test should also pass', function () {
    $this->assertTrue(true);
});

Run the entire file

Check the output and click rerun failed test buttons

Note that all 3 tests are rerun and notification pops up

Expected behavior
It should just rerun the failed test only.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac OS Ventura 13.4
IntelliJ Version: PHPStorm 2023.1.2
PestPHP Version: 2.6.1
Plugin Version: 1.10.1

Additional context
Add any other context about the problem here.

This error is shown when filter finds no tests.
There was recently a bug in pest which made filter not work anymore.
Can you try updating your pest in php to the newest version and see if it is still an issue.
If it is, I'll re-open the ticket. 👍

Thanks @olivernybroe,
I've just updated pest to the latest version and can still confirm it is still a problem

I have confirmed that I was able to reproduce.

any news about the fix?

A fix for this will be included in next release.


How to drop role created by pgaadauth_create_principal?

I'm looking for a way to drop the role created by the function pgaadauth_create_principal. In this sense, it's a question already hinted at in https://github.com/MicrosoftDocs/azure-docs/issues/102693#issuecomment-1943429484
Questions:

I'm looking for a way to perform step 6. below, without doing step 7, in other words without removing the Entra admin.
Additionally I'm looking for ways https://github.com/Azure/azure-postgresql/issues/117

To facilitate the reproduction, here are the steps for Entra user creation/deletion, performed with postgres 15.

In Azure Portal, Azure Database for PostgreSQL flexible server | Databases create test database
In Azure Portal, Azure Database for PostgreSQL flexible server | Authentication | Add Microsoft Entra Admins create MyAdAdmin user, using an existing MyAdAdmin AAD group
Connect to the postgres database using MyAdAdmin (due to https://github.com/MicrosoftDocs/azure-docs/issues/102693#issuecomment-1798118261)POSTGRES_USER=MyAdAdmin
export PGPASSWORD=$(az account get-access-token --resource-type oss-rdbms --query "[accessToken]" -o tsv)
PGSSLMODE=require psql -h $POSTGRES_HOST --port=$POSTGRES_PORT --username=$POSTGRES_USER postgres

and create a MyAdReader user, using an existing MyAdReader AAD groupSELECT * FROM pgaadauth_create_principal('MyAdReader', false, false);
            pgaadauth_create_principal
---------------------------------------------------
Created role for "MyAdReader"
(1 row)
SELECT * FROM pgaadauth_list_principals(false);
   rolname  | principaltype |               objectid               |               tenantid               | ismfa | isadmin
------------------------------------+---------------+--------------------------------------+--------------------------------------+-------+---------
 MyAdAdmin  | group         | xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx1 | xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx |     0 |       1
 MyAdReader | group         | xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxx2 | xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx |     0 |       0
(2 rows)

As the regular database Admin (not MyAdAdmin) connect to test databasePOSTGRES_USER=Admin
PGSSLMODE=require psql -h $POSTGRES_HOST --port=$POSTGRES_PORT --username=$POSTGRES_USER test

and create a schemaCREATE SCHEMA "myschema";
CREATE TABLE "myschema"."mytable" (mycolumn text);
INSERT INTO "myschema"."mytable" (mycolumn) VALUES ('Hello, world!');

and add grants to MyAdReaderGRANT USAGE ON SCHEMA "myschema" TO "MyAdReader";
GRANT SELECT ON ALL TABLES IN SCHEMA "myschema" TO "MyAdReader";

As MyAdReader connect to test databasePOSTGRES_USER=MyAdReader
export PGPASSWORD=$(az account get-access-token --resource-type oss-rdbms --query "[accessToken]" -o tsv)
PGSSLMODE=require psql -h $POSTGRES_HOST --port=$POSTGRES_PORT --username=$POSTGRES_USER test

and verify the grantSELECT * FROM "myschema"."mytable";
   mycolumn
---------------
Hello, world!
(1 row)

As the regular database Admin (not MyAdAdmin) connect to test databasePOSTGRES_USER=Admin
PGSSLMODE=require psql -h $POSTGRES_HOST --port=$POSTGRES_PORT --username=$POSTGRES_USER test

and try to drop the MyAdReaderREVOKE USAGE ON SCHEMA "myschema" FROM "MyAdReader";
REVOKE SELECT ON ALL TABLES IN SCHEMA "myschema" FROM "MyAdReader";
DROP ROLE "MyAdReader";
ERROR:  role "MyAdReader" cannot be dropped because some objects depend on it
DETAIL:  privileges for schema myschema

Try to find out the reasonSELECT classid::regclass AS dependent_table, objid AS dependent_object, refclassid::regclass AS referenced_table, refobjid AS referenced_object FROM pg_shdepend WHERE refobjid = (SELECT oid FROM pg_roles WHERE rolname = 'MyAdReader');
dependent_table | dependent_object | referenced_table | referenced_object
-----------------+------------------+------------------+-------------------
pg_namespace    |            xxxx1 | pg_authid        |             xxxx2
(1 row)

In Azure Portal, Azure Database for PostgreSQL flexible server | Authentication | Add Microsoft Entra Admins delete MyAdAdmin, then as Admin, drop the MyAdReader roleREVOKE USAGE ON SCHEMA "myschema" FROM "MyAdReader";
REVOKE SELECT ON ALL TABLES IN SCHEMA "myschema" FROM "MyAdReader";
DROP ROLE "MyAdReader";

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

ID: c454ed0e-944b-2933-ff30-5969ff0de1c9
Version Independent ID: 6f69997e-b534-e573-154d-7116e189d3c9
Content: Manage Microsoft Entra users - Azure Database for PostgreSQL - Flexible Server
Content Source: articles/postgresql/flexible-server/how-to-manage-azure-ad-users.md
Service: postgresql
Sub-service: flexible-server
GitHub Login: @achudnovskij
Microsoft Alias: anchudno

@marcindulak
Thanks for your feedback! We will investigate and update as appropriate.

@marcindulak
I've delegated this to @achudnovskij, a content author, to review and share their valuable insights.

@RamanathanChinnappan-MSFt, @marcindulak  -  you need to be a Microsoft Entra Admin to drop non-admin Entra roles. You cannot drop a Microsoft Entra roles using a regular admin. This is documented and available in the docs page. Did you try dropping MyAdReader using Microsoft Entra Admin

Currently point 6. works for me, I mean both the regular "Admin" and the Microsoft Entra Admin (MyAdAdmin) are able to DROP ROLE "MyAdReader";. Please setup a reproduction case if this is not expected.
I'm not sure it matters for the reproduction, but in my case the user (me) who is issuing the commands above is a member of both MyAdAdmin and MyAdReader AAD groups.

#please-close

@nachoalonsoportillo - why is this issue closed? In my comment above I describe a behavior that differs from the description provided in https://github.com/MicrosoftDocs/azure-docs/issues/119982#issuecomment-1989328589
Can you confirm that both the regular "Admin" and the Microsoft Entra Admin allowed to drop the role created using pgaadauth_create_principal now?


Fixes https://github.com/rv-kip/express-redis-cache/issues/59

Fixes https://github.com/rv-kip/express-redis-cache/issues/59

Looks like the test's failed, I'll take a look later

+1, this is particularly problematic right now as there's no way to conditionally not cache error statuses without this, as referenced in #64. thanks!

Anyone looking at this? @rv-kip

@ckeboss, Sorry. Been pretty busy.  Looks good. Could you add a test for a 0 value on options.expire to ensure it does the right thing?

@rv-kip I have added a test.

@rv-kip when will this get into a published version? Do you want to just publish a patch version or bundle other fixes/features before?

0.4.0 published.


Remove webpack from core-http

core-http had an unfortunate dependency on webpack from the ms-rest days that was cluttering our dependency graph with a huge amount of packages.
This PR removes the dependency on webpack and migrates bundling for browser testing to rollup to match our other packages.
This required restructuring some of the tests into separate browser/node files as the dynamic nature of how we were skipping/selecting tests before did not agree with rollup. No tests have been removed, merely moved around.
This PR allows us to drop webpack and its massive tree of dependencies from our pnpm graph.

Yikes, splitting up those test files looks like it sucked. LGTM!

It wasn't too bad, the only remaining headache is how http mocking makes us mark "fetch-mock" as external in the browser tests even though it isn't used. I was too lazy to figure out how to tease that apart without just duplicating all the client tests...


refactor(api): deprecate TransferRequest assetId

What this PR changes/adds
Deprecates TransferRequest.assetId attribute on management api.
Upgrades api version to 3.0.1.
Remove assetId from the TransferRequest value object and replace its usages with the policy target (that has the same value)
Why it does that
cleanup
Further notes

improve RuntimeExtension a little, to permit better usage of the RuntimePerClassExtension (as shown in the TransferProcessEventDispatchTest)
replaced some EdcRuntimeExtension occurrencies

Linked Issue(s)
Closes #4264
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

Codecov Report
Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 6.03%. Comparing base (7f20ba5) to head (69b557f).

Files
Patch %
Lines

...e/transfer/process/TransferProcessManagerImpl.java
0.00%
4 Missing :warning:

...eclipse/edc/junit/extensions/RuntimeExtension.java
0.00%
3 Missing :warning:

...t/transferprocess/TransferProcessApiExtension.java
0.00%
1 Missing :warning:

...erprocess/validation/TransferRequestValidator.java
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #4269       +/-   ##
============================================
- Coverage     71.74%   6.03%   -65.71%
- Complexity        0     394      +394
============================================
  Files           919    1046      +127
  Lines         18457   20982     +2525
  Branches       1037    1171      +134
============================================
- Hits          13242    1267    -11975
- Misses         4756   19694    +14938
+ Partials        459      21      -438

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


image upload doesn't filter to image files

When clicking the image upload button, the file selection dialog that opens displays all files, and the only option in the filter dropdown is "All Files (.)"
There shuold be options to filter for image types, and the default should be "All Supported Types", as can be seen in the demo of the quill editor

Could this be related to #67?


Bug: Necro cutscene won't remove the letterbox bars when it finishes playing

What happened?
Sometimes when entering Catacombs from the back, the letterbox bars from the cutscene won't be removed. Obviously the game doesn't expect you to enter from that direction. not sure how to address this still, but it's the necro cutscene not playing correctly that causes this.
What were the expected results?
Player should be able to do the fight without having their HUD removed. The HUD stays gone until the player either quits the game or to the menu.
Is this reproducible?
No response

Haven't seen this one for a while so closing it as I'm pretty sure I've fixed it.


Added missing statement about <Plug>(ale_toggle) mapping to the documentation

Also would like to ask why there are no mappings for :ALEEnable, :ALEDisable and on what basis the mappings are added. For all commands? As someone requests them?
Thank you,
Andrew V

Cheers! :beers:

Cheers!


Fix to also work with BSD date.

The -d option specified with date seems to be an option that exists only for GNU date; the date installed by default on macOS is a BSD date, so this command cannot be used.
https://github.com/kenchan/gh-furik/blob/07f151e797065e37ca253e56c6bcf7c5e445989b/gh-furik#L5
(macOS)
$ date +%FT00:00:00%z -d tomorrow
date: illegal time format
usage: date [-jnRu] [-d dst] [-r seconds] [-t west] [-v[+|-]val[ymwdHMS]] ...
            [-f fmt date | [[[mm]dd]HH]MM[[cc]yy][.ss]] [+format]

Instead, I figured that specifying 23:59:59 in the date command would work with either date command.
(macOS)
$ date +%FT23:59:59%z
2022-04-18T23:59:59+0900
$ gdate +%FT23:59:59%z # require `brew install coreutils`
2022-04-18T23:59:59+0900

Thanks for your PR!


Added notebooks of the second session

Hi Riccardo @rmassei ,
here come the notebooks for the second session.
Best,
Robert

Thanks a lot @haesleinhuepf!


Repair watch fedora-ci

After the code restructures, the code about watching fedora-ci needs some care to make it work again.
We need to:

[ ] implement the missing parts in ./packit/bot_api

look on TODOs (sync_fedora_ci_with_fedmsg and sync_fedora_ci)

[ ] implement the logic in SourceGitCheckHelper (.packit/watcher)
[ ] update ./packit/cli/watch_fedora_ci to use ./packit/bot_api

I think we will drop that code since this feature was kind-of de-prioritized; I think I'll remove it with my work for jobconfig


How do I get the id of items that where inserted in mysql

After inserting a row into a table, I would like to know the id that was assigned to it.
In postgres, this seems straightforward:
const returnedValues = await pgDb.insert(pgTable).values(values).returning();
const insertedIds = returnedValues.map(({id}) => id);

But in mysql, I'm not sure this information is available:
const [resultSetHeader, fieldPackets] = await mysqlDb.insert(mysqlTable).values(values);
const { insertId } = resultSetHeader;

Is insertId what I'm looking for? (I couldn't find documentation on it). If I insert multiple items at once, there is only a single insertId returned.

Seems like I can get ids by doing just doing individual inserts. For example:
const inserts = await Promise.all(valuesToInsert.map((value) => mysqlDb.insert(mysqlTable).values(value)));
const insertedIds = inserts.map(([{insertId}]) => insertId)

I trust the API will continue to improve, so maybe I'll go ahead and close this.

Seems like I can get ids by doing just doing individual inserts. For example:
const inserts = await Promise.all(valuesToInsert.map((value) => mysqlDb.insert(mysqlTable).values(value)));
const insertedIds = inserts.map(([{insertId}]) => insertId)

I trust the API will continue to improve, so maybe I'll go ahead and close this.

@jonsmithers I saw this searching for something else in the issues, and had to comment.
Unless there's something I'm not aware of, you should insert all the items in one query and then manually update in code the id - incrementing the insertedId by one for each record.


Generate a default "no transfer" test in execution code, use "accept transfer" to remove it

By default, action should not accept transfers ; the corresponding test should be generated in execution code. Use "accept transfer" if you want a transaction to accept transfer.
A compilation error should be issued if transferred is read while "accept transfer" is not present.
action anaction = {
      called by admin
      accept transfer
     effet ...
}

if transferred is read in effect, then automatically remove the guard.


[sdl2-mixer-ext] add new port

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

The usage and feature has been tested successfully locally.
find_package(SDL2_mixer_ext CONFIG REQUIRED)
target_link_libraries(main PRIVATE $<IF:$<TARGET_EXISTS:SDL2_mixer_ext::SDL2_mixer_ext>,SDL2_mixer_ext::SDL2_mixer_ext,SDL2_mixer_ext::SDL2_mixer_ext_Static>)


fix for thread block

Not sure the exact technical reasons but "MainThreadDispatcher.Instance.Post" -> "DispatchQueue.MainQueue.DispatchSync(action)" was blocking when being called within the lazy load code. Bringing the code out of this into a method did the trick and the thread is no longer blocked.

Thanks for this @alanspires !


support mock versions < 3.0.5

https://github.com/spulec/moto/pull/2776 pinned the mock version, but unfortunately there are other packages that are more strict. Specifically, I cannot install moto + azure-cli.

Coverage remained the same at 94.134% when pulling 0e489a8a287c36af36a9728fe62699b84575ebd4 on steved:master into e2106b7c01f36227e5dcf0e9cc275d548cc27109 on spulec:master.


Web analytics

Please add Matomo tracking code to all pages as documented here:
https://wiki.geant.org/display/UP2U/Web+Analytics

I added it already just forget to close this. Let me know if there is any issue with it.


PyO3 example fails to build

It appears that with the upgrade from 0.19 to 0.20 for the pyo3 dependency, breaking changes were introduced that the library doesn't account for, in particular related to IntoPyPointer.

Hi, @michaeleisel . Happy Chinese New Year! Thanks for discovering the bug. Please check the latest version.


Ignore footnotes when generating icons in editing mode

I like this plugin, but it was bugging me that icons were displayed with footnotes in edit mode. Just isn't visually appealing. I edited the main.js file to remove the footnote icons for myself. I only know enough javascript to get the basic idea of what a script is doing and then make small edits, so my code is usually pretty hacky, but it got the job done for me. Anyhow, maybe it's something you can consider adding to the main plugin

Thanks for the report! I think this should now be fixed. Let me know if there are any other issues.

Awesome, thanks! I'm really inexperienced with Javascript, but it's on my
list of things to learn soon. So just reading your code and editing it
yesterday helped me a lot with understanding some of the quirks with
javascript, and now seeing how you solved the same issue, even if it is a
minor detail, is also a small step forward, so I appreciate it.
On Sun, May 7, 2023 at 4:07 AM Scott Moore @.***> wrote:

Thanks for the report! I think this should now be fixed. Let me know if
there are any other issues.
—
Reply to this email directly, view it on GitHub
https://github.com/scottwillmoore/obsidian-short-links/issues/8#issuecomment-1537353603,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOXIW5USPB42VHSZJPOU5DXE5J5RANCNFSM6AAAAAAXYVPZDY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Shridhar Singh
B.S Chemical and Biomedical Engineering
Carnegie Mellon University | 2018
781-392-4644


add theme generator script

What's been done

Given a color of choice, theme generator creates a new folder in enlighten/assets/stylesheets where Enlighten is tinted to that color

What's yet to be done

enlighten/assets/stylesheets normally contains light and dark themes only. Enlighten has a mechanism for selecting between these two themes. I'm able to preview my theme by temporarily overwriting one of them (dark). For these themes to be selectable, we need a more generic theme selection option.
I would like to generate 5-10 themes as starting points with this, such as yellow, blue, orange (for halloween), xtra dark, etc.

How it works (for fun)
The script copies a whole folder to make a complete theme but it only modifies enlighten.css, which drives at least 95% of Enlighten's colors. I use a HSL-based color find-replace to replace shades of gray from the light theme with shades of a particular target color. This works on embedded HSL values in the stylesheet as well as short and long hex codes (ex: #FFF, #FF0000).
There are a few colors that don't get replaced, such as the Raman button. So theme gen can be used to create a starting point for themes. It may require some additional tweaking to get it just right.
I might want to do a Halloween theme which consists of mostly dark colors with a couple of orange accents added in after the fact

Ready for review.

A theme with a name starting with "dark" will trigger the dark graphs. (For example "dark", "darkblue", "darker")
Contains a tentative fix to #324 copied from Mark's branch
Fail gracefully if a selected theme is removed from disk
Integrate with existing light/mode dark logic. (Most themes are considered light themes.)

To Do (post merge)

fully resolve #108 when switching to and from light and dark themes
use darker ranges of lightness in colorgen for a theme labeled "dark..." (improve appearance of darkblue theme)
themed darkmode toggle*

explanation of "themed darkmode toggle"
Say you had the pink theme selected. With this revision, if you click the dark mode icon (moon) then it will switch to dark. If you click it again it will switch to light. It would be better if toggling twice takes you back to the theme you were on, and if the user can customize (and save) a preference of both light and dark themes.
For example I might use pink theme as my light mode and darkblue as my dark mode. This addition isn't worthwhile until the modified "dark..." themes have their color logic adjusted to actually be darker.

Added more support for custom dark themes: