Weird behaviour when multiple arrow keys pressed

Short history:
I try to program my own game engine in C++, that uses OpenGL for rendering, including GLAD and GLFW. So far the game itself has a cube, that can be moved by pressing the arrow keys. I tried to trick my game for exposing bugs by pressing multiple arrow keys. My cube moved weirdly when pressing, holding, releasing keys in a specific order. I found out, that the bug's cause is inside GLFW.
Scenario:
When I press and hold left and right at the same time, my cube doesn't move as expected, so do up and down. I keep holding left and right, now I press up and the cube doesn't move upwards as expected, but if I press down, he goes down. And the most confusing part of this is, that if I hold down, then press and hold up and release down, he continues going down.
The same scenario happens when holding up and down and playing with left and right. In this case right only works and left doesn't.
I already tracked down the cause of the bug inside my game engine and it led me to GLFW. I implemented the error callback, but it wasn't triggered. I tested my theory in a simple code:
#include <GLFW/glfw3.h>
#include <iostream>

bool keys[4];

void errCallback(int err, const char* des) {
	std::cerr << "GLFW " << err << ": " << des << std::endl;
}

void keyCallback(GLFWwindow* window, int key, int scan, int action, int mod) {
	if (key == GLFW_KEY_LEFT)
		key = 0;
	else if (key == GLFW_KEY_RIGHT)
		key = 1;
	else if (key == GLFW_KEY_DOWN)
		key = 2;
	else if (key == GLFW_KEY_UP)
		key = 3;
	if (action == GLFW_PRESS)
		keys[key] = true;
	else if (action == GLFW_RELEASE)
		keys[key] = false;
}

int main() {
	if (!glfwInit())
		return 1;
	glfwSetErrorCallback(errCallback);
	glfwDefaultWindowHints();
	GLFWwindow* win = glfwCreateWindow(1000, 700, "GLFW", NULL, NULL);
	if (!win)
		return 2;
	glfwMakeContextCurrent(win);
	glfwSetKeyCallback(win, keyCallback);
	std::cout << glfwGetVersionString() << "\n";
	while (!glfwWindowShouldClose(win)) {
		glfwPollEvents();
		std::cout << keys[0] << " : " << keys[1] << " \\ " << keys[2] << " : " << keys[3] << std::endl;
	}
	glfwDestroyWindow(win);
	glfwSetErrorCallback(NULL);
	glfwTerminate();
	return 0;
}

Technical information:
OS: Ubuntu 22.04.1 LTS
Compiler: g++ 11.3.0
GLFW version: 3.3.6 X11 GLX EGL OSMesa clock_gettime evdev shared
(g++ uses GLFW 3.3.6 instead of 3.3.8 because I installed it by using Ubuntu's package installer APT. I tried to update it with APT and by compiling from source and installing, however it keeps stuck with 3.3.6)
Errors: /
Architecture: x86_64 (64-bit)
Processor: AMD Ryzen 5 3600 6-Core Processor
GPU: NVIDIA Corporation - GP106 [GeForce GTX 1060 6GB]
Compile command: g++ *.cpp -lglfw

FWIW, I tried to reproduce the issue with your test program and could not. The program behaves like you would expect. All four arrow keys register press and release independently of each other, no matter how many are pressed simultaneously.
This was on Fedora 38 with GLFW 3.3.8

Try this page: https://drakeirving.github.io/MultiKeyDisplay/
If the issue happen here too, the problem might just be your keyboard itself due to ghosting.


How can i modify the payload?

We want to modify something in the payload while ingest from kafka to hdfs. So How can i implement this? Specify the payload-class? Thank u.

are you using the deltastreamer? if so, you can write your own payload class and it will be picked up via a cmd line argument..

OK。thanks

@vinothchandar Is that possible to set my own payload class as DataSourceWriteOptions.PAYLOAD_CLASS_OPT_KEY when using DataSource API to write any dataframe into hoodie managed dataset (like here)?
Plus, I also saw there are some comments in HoodieRecordPayload. These comments mentioned two use cases. Case 1 is using combineAndGetUpdateValue to update counter. I wonder how to add a new counter field into IndexedRecord which store the value of counter. If it was GenericRecord, we could simply call put(key, value). But, I don't know how to do this in IndexedRecord.
Do you have any example code of how to use payload class which I can take a look at?
Thanks a lot!

Maybe I could cast IndexedRecord to GenericData.Record


feat: teach SparseArray maybe_null_indices_iter

@robert3005 all the with_dyn and allocation feels icky; if you think we should close and address later that's fine, but this is in a pile of changes I made while trying to implement pruning.


Get full record

Great project!
Looking at the README and the tests I noticed that all methods (.get(), .getAll(), .query()) only return the records' IDs, there's a way to access the full record object?

There isn't....
The reason why is that mindex doesn't store the full record.
My suggestion would be to keep your records in an object, by ID.
I Just pushed a release that exposes the lower level API. That might make it more obvious how to use it.
insertRecord, updateRecord etc... are actually just convenience methods.

@internalfx I see, using an object with IDs as keys seems good enough


feat(generator): support unified credentials

The generator creates a *Stub decorator to inject the necessary
credentials in the grpc::ClientContext. This decorator is optional, as
many credential types simply initialize the grpc::Channel and never
change the policies after this point.
Fixes #6721

This change is

I think this is too large, I am going to break it down into smaller PRs.


add Blob.Splice()

attn @arv

Can one of the admins verify this patch?

add to whitelist

LGTM
Do you want to do the JS too? Otherwise we should add a bug.

I'll take a swing a the js stuff; fingers crossed that babel doesn't mess me up again.

@arv can you take a look at the js side when you have a sec?

LGTM
@kalman Can you also take a look (TBR)?


made change to README file

You need to include the images in your README.md.  See my sample README.md in Nelson/2 for how to do it.


Validate component instances during policy resolution

If cluster is not set, we should detect and handle this

[error] not sure where components should be deployed, cluster label is not set (dependency 'prod', contract 'twitter_stats', service 'twitter_stats')


avoid using the same name for two APIs of the blockchain service

We were using /transactions to both anchor and fetch Sidetree transactions from the underlying blockchain. This commit fixes the naming scheme.

closing the PR, as discussed.


Require all tests to be defined inside #[cfg(test)] module

Currently, tests will be collected from all files in all directories within a package.
This logic should be changed so tests are only collected from modules marked with #[cfg(test)].
Additionally, each file in tests directory should implicitly be treated as a separate module with #[cfg(test)] added - similarly to how cargo test works.

@mkaput/@maciektr know how to do it FYI

cargo test executes every function marked with #[test] attribute, #[cfg(test)] is just to prevent compiling code used for testing for dev/release builds. We currently support compiling code marked with #[cfg(test)] during tests anyways.


fix: Don't overwrite ldflags in justfile builds

Description
There was a small error that was causing the ldflags to get overwritten (one after another). More or less a go build command error.
This also adds missing build-args for the docker container in the GitHub action. Date and sha might be abit more tricky to get into that action, but for now, the version is good enough
Related Tickets & Documents
Closes: https://github.com/open-sauced/pizza-cli/issues/167 (cc @brandonroberts - I ended up going down a rabbit hole on this one, apologies)
Steps to QA

just build-all
Run ./build/pizza-darwin-arm64 version and see the right version and sha

Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[ ] Tier 3
[x] Tier 4

SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ARG "POSTHOG_PUBLIC_API_KEY") (line 8)

I reject that suggestion 🕵🏻♂ the posthog public key isn't a secret thankfully, otherwise, yes, we'd want to handle it more securely

SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ARG "POSTHOG_PUBLIC_API_KEY") (line 8)

I reject that suggestion haha 😂  the posthog public key isn't a secret thankfully, otherwise, yes, we'd want to handle it more securely

Oh yeah, I guess it has no way of knowing if it's a public key or not. 😅


EG_KERNEL_WHITELIST brackets

Enterprise Gateway environment variable EG_KERNEL_WHITELIST will possibly no longer include brackets in new release
https://github.com/jupyter/enterprise_gateway/pull/894

Contribution was merged, brackets can be dropped in the latest version of EG.


Ensure control plane is deployed immediately when the target namespace is created

@luksa: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

e2e-kind_sail-operator_main
e7aea5b46b87139bb19eca81a60e95fbddae907a
link
true
/test e2e-kind

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


[Amplitude] Replay AssistantCreated and DataSourceCreated

Description

Risk

Deploy Plan

Trusting your judgment. read-only migrations are inherently safe :+1:

Migration was run successfully. Closing this PR.


Documentation for sbt-datatype

Cool, TIL a bunch. Looks good to my noob eyes.

@dwijnand Thanks! I hope there are not too many grammar errors?

Didn't spot any.


WIP: feat: uv pip list --outdated

Summary
This PR adds support for uv pip list --outdated
https://pip.pypa.io/en/stable/cli/pip_list/#cmdoption-o
Fixes https://github.com/astral-sh/uv/issues/2150
WIP
This PR is currently a work in progress, and I am seeking feedback on the approach 🙏
I have left some comments in the code and would appreciate feedback on both the large scale approach and also any small-scale implementation details.
I have got to the point where running the command correctly identifies those packages which have an update available.
➜  uv git:(feat/pip-list-outdated) cargo r --bin uv -- pip list --outdated
Resolved 370 packages in 2.33s
Package                           Version
--------------------------------- --------------
aenum                             3.1.11
aiodns                            3.0.0
...

[ ] The output formatting needs updating in order to match pip

➜  uv git:(timl/pip-list-outdated) pip list --outdated
Package                           Version        Latest         Type
--------------------------------- -------------- -------------- -----
aenum                             3.1.11         3.1.15         wheel
aiodns                            3.0.0          3.1.1          wheel
...

Test Plan
WIP

One small general though on this one. Feel free to ignore.
I believe this is one command that could use an enhancement over what pip currently has.

Supporting uv's --resolution modes on this,  I think would be fantastic to leverage this pre-existing functionality.

Right now I only see support only for highest.

Supporting semver-like matching, I think we can leverage the existing versioning spec for this. (see below)

For point #2, I'm thinking something like --target major,minor,patch very similar to what pip-check-updates does. These could be even incorporated into --resolution itself in the future if it's deemed stable enough.
Rationale, I often find myself wonderwing what are the "patch", "minor", or "major" updates available (for packages that follow semver conventions). In many cases, I might want to update to the latest minor even if there's a major available. In the same vein, I also find myself often wanting to know this answer to direct dependencies over trasitive so direct support plays a major role in such situations.

It sounds like this deserves its own dedicated command — it doesn't really feel like it deserves to be tacked on as a pip list feature.

@samypr100 I like the --target idea, could you open a new issue describing that functionality? I don't see why we wouldn't add that to pip compile and pip install. We should probably discuss it in general.

It sounds like this deserves its own dedicated command — it doesn't really feel like it deserves to be tacked on as a pip list feature. I feel weird about adding all the resolver options to command intended to list the current environment.

I agree. The sketch of that approach in this PR I think highlights how out of place this command is under pip list. I think the overall use case still stands. I tried a few of the things you suggested and they come close, but have slightly different semantics to what I'm after (I want to know what the latest version of my packages are, whether or not those packages are consistent with my current pyproject.toml/requirements.txt).
I'll not take this PR any further for now. @zanieb Not sure of the protocol, but feel free to close the PR when appropriate.

@samypr100 I like the --target idea, could you open a new issue describing that functionality? I don't see why we wouldn't add that to pip compile and pip install. We should probably discuss it in general

@zanieb Good point, I'll re-post as an enhancement issue once I'm able, currently struck down with the flu 😷

It sounds like this deserves its own dedicated command — it doesn't really feel like it deserves to be tacked on as a pip list feature. I feel weird about adding all the resolver options to command intended to list the current environment.

@zanieb I agree, for the current approach, where it's reusing the package resolver.
The real way this is supposed to work however, it does make sense for it to be a part of pip list, since you are not using a resolver at all (you may be reusing the part of the resolver that talks to the package index however, pip is calling this a PackageFinder internally), you are listing the currently installed version of all the packages in the environment alongside the latest version available on the package index (ignoring all pins/overrides), unless the package is already up to date. It's thus just a different way to list the current environment.
This is important information for evaluating your current dependencies. Some of this information would be available in the compiled requirements.txt, but it's easier to work with (especially for humans) with output from this command.
At first glance, it looks like here it might be possible to leverage DistFinder from uv-resolver to accomplish something similar, although it would need a different, more simplistic select strategy, that just takes a small list of preferences into account (like e.g. whether you want to prefer wheels over sdists, or whether you want to allow wheels/sdists at all or whether you want to allow pre-releases or yanked releases or want to ignore requires-python).

What is the progress so far? ❤

Looking forward to this getting merged

Please just upvote the issue if you are interested in this feature. Comments should be reserved for substantive discussion.
As we've noted in the discussion here, we're not sure this is a good path forward for this feature. We will need to do design work to determine a new way to fulfill this use-case.

Going to close for book-keeping. Thanks for the poc! I'll be working on a design for this in the future.

Going to close for book-keeping. Thanks for the poc! I'll be working on a design for this in the future.

please be sure to post link when you start working on this one. i'm really interested in this feature and i don't want to miss any progress on this. ;-) thanks!


create pfInfoStatusCard

We have pfAggregateStatusCard, but what if you just want to display a few lines of text or whatever!!
Introducing, the pfInfoStatusCard 👍
Should look something like the two cards in the upper right of this img:

a closer look

As requested by @serenamarie125

@jeff-phillips-18 @dtaylor113 Can someone assign this to me and add enhancement as a label? (pretty please!!!)

@AllenBW   Assigned to you and tagged.


[BUG] Faucet twitter login error

Description
Cannot login with twitter with the latest faucet deployment.
Screenshot

Closed in https://github.com/webb-tools/webb-dapp/pull/1724.


Train model to translate language

How to fine tune model for langugae translate  with this dataset:
{ "en": "Did you find something?", "fr": "This way!" }
{ "en": "Article 6", "fr": "Article 6" }

see https://github.com/hiyouga/LLaMA-Factory/tree/main/data#supervised-fine-tuning-dataset


Rotate around look-at target

Project Type
3D
Feature Description
at the moment there is no simple way to just let the camera rotate around the target with the positional offset.
Use Cases
One example would be to have a 2D game in 3d space which simply follows the players orientation. While keeping the other features like framing and dampening.

Importance
Low - there are workarounds where the proposed feature would just simplify things
Usage
All the time - the feature can be very useful for a varity of different projects
(Optional) Proposed Solution
This could be implemented as a simple bool in addition to the look at modes to just be something like "rotate around target". Then the offset position needs to be rotated by the rotation the look at target gets. Mainly useful for simple and glued.

Think this is a very good proposal and can definitely see its usefulness.
Framed Follow already has an orbit-like rotation behavior, so can't imagine it'll be too difficult to add this as well.

Would this be similar to Orbital Follow mode in CineMachine?

Think this could also be achieved using Third Person follow?
Would just need to apply the rotational setter methods as the character turns.

Closing this issue per the above suggestion.
Think anything beyond that will creep into a fairly complex solution that effectively just simplifies what Third Person follow can already do.
Feel free to suggest reopening this if there are other opinions, and we can continue the conversation.


How to integrate Omnicore in exchange?

I'm exploring omnilayer and I'm wondering about it's integration in exchange.
Let's say I've set up node and created addresses for A, B and C users. A and B user deposited 100 Omni to their address and C wants to withdraw 150 Omni to some other wallet on exchange XYZ.
I understand there is a method omni_send which will use for simple send functionality but as it requires from_address, do we need to sweep out tokens from user A and B's address to some other central address CW(let's say cold wallet address) through which withdrawal will happen? If yes, then can I use BTC from my cold wallet(CW) to pay for fees while transferring omni from user's address to CW?
Can you provide me some reference for such integration for exchange? I'm already done with deposit part and researching on withdrawal options.
Thank you,
Vimal

I'm also interested in this. If I try omni_getbalance I get 100 omnis each for user A and B, and 0 omni for user C. So user is unable to withdraw.
I can try to do several omni_sendfrom A and B to C but would require btc transaction fees
For bitcoin, I am able to do above with HD address generation but not sure about process for Omnicore
Thanks

Omni will work same as ethereum. So you have to fetch coins from user's address to hot wallet and withdrawal will work from hot wallet.
Hope this answers your doubt.
Thanks,
Vimal

I can try to do several omni_sendfrom A and B to C but would require btc transaction fees

Right. Omni transactions use Bitcoin transactions, so you may have to fund destinations, before they can move tokens.
However, I recently merged #658, which should make things easier.
For a few hints regarding an integration, check out the following post:
https://github.com/OmniLayer/omnicore/wiki/Platform-Integration-of-Omni-Core


Extra plugins not showed and don't work

Extra plugins not showed and don't work

EnableSafeDefaultPlugins must be zero for non-default plugins

EnableSafeDefaultPlugins must be zero for non-default plugins

Ok, thanx!

*EnableDefaultSafePlugins

@0x4E69676874466F78
The name changed when the plugin interface had breaking changes. Old plugins will crash newer versions of the program, so it's recommended you don't disable the safe plugins.

@dmex After minor edits, almost all extra-plugins work without problems for me.

@0x4E69676874466F78
Yep, that's how it's supposed to work 😄
The problem is when you're using older plugins and update to newer versions. PH/SI will try load the plugins on startup and immediately crash if there's an outdated plugin installed.
Renaming the setting force resets this option back to the default for everyone (disabled) so when they install the newer version it won't instantly crash. If you re-enable the option and immediately start crashing then you'll know there's an outdated plugin causing it and not the update 👍

@dmex now I understand.


IS KM does not include configuration management feature

Identity server 5.7.0 does include configuration management feature, but not with Identity server 5.7.0 as key manager.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


cannot change listen http port via commandline?

Am I correct that it is impossible to change the listen port for incoming web requests, and it always binds on port 80?
I don't want to create a config file just to change the listen port for web requests. It's even more weird that you do specify a commandline parameter to change the port for the web-admin interface (--web.address), but not for the actual web port itself...
I don't see any parameter in traefik --help that always me to complete this simple straightforward task...
I'm want to run traefik as a k8s ingress controller, as a daemonset using host networking, but I need to specify another port.

You can specify the listening port along the --entryPoints parameter. For instance,
--entryPoints='Name:http Address::8080'

will cause Traefik to listen on port 8080 for HTTP traffic.
If you have further or other user-related question, please use either our Slack channel or StackOverflow. We try to limit the usage of Github issues for feature requests and bug reports. Thank you!


Performance issue during rendering for 3200row X 110col

I have a performance issue during rendering for 3200 rows and 110 columns. I am using lazy load but jexcel trying to append all html content at once and taking around 6-10 seconds to render.
For below code, if i limit to append 100 elements only , it renders very fast. is there a way to make it working like lazyLoading style ? It can append during scrolling, will it be slow in this case ?

I am happy to discuss that on Skype.
My skype account: paul.hodel

I have one year record. Which have 365 days as rows and it have formulation in some cells. It slow down the loading of grid. Web service respond in quick time but page grid load takes about 15 second time to load. If i switch to month. Than grid loads 31st days record quickly. What can be the solution?

Are you using the most recent version?

I am using 3.6.2. is it the latest one?

Ok I update it from 3.6.2 > 3.6.3. I changed the jexcel dist files css and js with updated one and replace the package.json as well. Still the issue exist.

Alright i have checked one more thing. If i load without calculation it loads very quickly. The one with calculation takes time to load for 365 rows. Thanks

Mansoor, can you create a jsfiddle with your example? I pretty sure there is room from improvement.

Mansoor, can you create a jsfiddle with your example? I pretty sure there is room from improvement.

Hi. Thanks for response. Please check the jsfiddle link
https://jsfiddle.net/cx20ze7v/2/

Mansoor.
I have checked your example. I debug to see if there is anything, in this case, to make it fast. But, unfortunately, your spreadsheet creates a chain of 600+ formulas to be executed if you change the second row. So, basically, JS is slow enough to impact the performance of the task. I have a check for duplications or optimizations, but jExcel is only waiting for all formulas to be executed, and in this case, should wait for the result of one formula to trigger the next since it is a chain of executions, therefore there is no much I can do in jExcel side.
Sorry, but this is a browser limitation. You should avoid cascade formulas.
Best regards,


Update weather-data Notebook

Updated the weather data notebook a bit to a more interesting example

Hi @fred-sch, sorry about the late reply.
Due to the addition of pre-commit hooks this MR now has a tiny merge conflict. If you could please resolve that (since I cannot modify your fork), I'll merge the proposed change. If there will be any conflicts with the pre-commits i'll fix them myself.

Hey @zigaLuksic, I did as you requested. Let me know, if I need to change anything else.

Hey @zigaLuksic, I did as you requested. Let me know, if I need to change anything else.

It's perfect, thanks :D


Hypoglycemia White Paper Ready to Share

The hypoglycemia white paper is ready to share.  The attached document contains my edits in response to SME feedback, with tracking turned on.
20180524REVISED-CDSK_CCWP_B3B33B45B66Hypogly_final_508_copyright_KBS_APPROVED_LLWedits.docx

@linophth We don't actually have the ability to release white papers by themselves: only as attachments to artifacts. Are the artifacts also ready?

I will change my comments to indicate that only the white papers are ready for release, since I am not doing anything with the other artifacts - Franklin Cuello is handling that.


[WebToolsE2E][Aspire] Running the Aspire project, the browser page is not localized.

REGRESSION INFO: Repro on Aspire 8.0.0-preview.3.24105.21
INSTALL STEPS

Clean machine: Win11 x64 22h2 CHS
Install Dev17.10 Preview 2 from [[34629.198.main] (Includes Aspire 8.0.0-preview.4.24128.4)
Apply NuGet Feeds

REPRO STEPS

New Project > .NET Aspire Starter Application > .NET 8.0 > Create
Build and F5

ACTUAL
The browser page is not localized.

EXPECTED
Browser pages should be localized.

Localization infra onboarding in progress: https://ceapex.visualstudio.com/CEINTL/_workitems/edit/948961/
@adamint  please verify (post 3/15)  and close end of preview 5

These pages are now localized 😄


Fix random test failure in test_mul_within_range

If gen_biguint() generated a 0, then the sub(1) will cause the test to fail.
See https://github.com/hyperledger/solang/actions/runs/3044527275/jobs/4905014572

I've been running:
$ while cargo test --test substrate --  --exact  substrate_tests::expressions::test_mul_within_range ; do : ; done

For a few hours now, no failures any more


[top_englishbreakfast] Follow-up tasks for auto-generated top-levels

Since #4329 we now have basic support for different top-level configurations and a second top-level called top_englishbreakfast. For this top-level we mainly check in the hjson config files. All the auto-generated sources are generated at build time through FuseSoC. Some parts of the framework are far from perfect if not horribly ugly but we needed to move on to unblock the SCA work and get ready for future additions to the top-level that will definitely outgrow the NexysVideo FPGA board.
In the process of the PR review, rebase, and debug we came across several points that need to be investigate/solved in the future. The purpose of this issue is to track those points.
Documentation:

[x] See e.g. slides available on the OT team drive.

Project:

[ ] Define how to proceed with top_earlgrey/top_jasmine as the NexyisVideo becomes too small. Most likely, the NexysVideo will no longer be the main platform and the corresponding top-level will become a secondary configuration. Most of the CI and DV resources will be focused on the main platform. See also #5397
[ ] Define up to which level we are going to support differences in the various top-levels.

Software:

[ ] Find a better and cleaner way to build software for different top-levels. See also #5397

Hardware:

[x] Add support for a bigger FPGA board and switch over to that for main development platform. See also #5397, see #6614
[x] Cleanup of old and no longer used/usable Earl Grey targets: top_earlgrey_cw305, top_earlgrey_artys7-50

Tooling

[ ] Find a better way to deal with multiple implementations of the same core in FuseSoC. We are currently using fileset_x flags but this is not a scalable and long-term solution.
[ ] Allow topgen to generate software only, or register files only, see #4512
[ ] Add lint support for secondary top-levels (dvsim would need to call the FuseSoC generator for that).

CI:

[ ] Add the ChipWhisperer CW305 FPGA board to the lowRISC CI center and perform an AES test capture in CI.
[x] Execute a "hello world" on the English Breakfast Verilator simulation in CI. See #7582

This is related to #295 .

Adding the reviewers from #4329 @tjaychen , @eunchan , @msfschaffner , @imphil as well as @tomroberts-lowrisc , @lenary , @mateenfaisal and @gkelly for visibility.

Thanks for summarizing and listing all these tasks, @vogelpi.
In the interim, until we actually have a candidate FPGA board available, we could switch CI synthesis to the following part: XC7K410T-2FBG676C. If I am not mistaken, this is going to be the part that we are plan to use for the boards.

This also related to #4069.


[Feature] Make the --quiet flag behavior default, and instead have a --verbose flag

[x] Create a new flag
[x] Update documentation

[x] Change docs detailing flag usage
[x] Detail that previously prefilled games will not show up if they are up to date.

Released in v1.6.0


fix: Make sure updatable connections come back with decrypted tokens

Description
When going through REST API updatable connection flow (which is for every authenticated ds execution), we were using a method that does not decrypt values after saving the configuration in DB. This was causing requests to fail since the encrypted data would not be expected to make sense to these endpoints.
This was affecting anything marked as @Encrypted in a datasource configuration. Hence includes client credentials as well as self signed certificates in its scope.
Fixes #24910
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[x] Manual - with a server using a self signed certificate as well as one that uses OAuth2 (client creds)

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

Trying to figure out if we can write JUnit tests for this flow to avoid such an error in the future.

/ci-test-result

/build-deploy-preview skip-tests=true airgap-enabled=true

Tested this issue with Self Signed Certificate [Appsmith PR branch setup on local, SelfSigned Cert created locally with the help of this Documentation]

Tested with Authentication type set to None
Tested with Authentication type set to OAuth2.0 with self-signed certificate

Both the scenarios work fine.
Found a couple of issues with regards the Edit DS page, which will be logged separately, as they are related to functioning of UI components.


Set limit on how many windows in special workspace

Describe your PR, what does it fix/add?
Fixing point 1 of #2596
Added config option misc:isolate_special INT
For windows in special workspace > INT the new window is send to the active workspace (behind special).
Disabled on 0.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Possible bug : Assign negative number to config
Maybe somehow later make it workspace specific and not for all specials ?
Is it ready for merging, or does it need work?
Tested on a nested Hyprland Session and works as expected.
Future PR if all goes well:
Toggle workspace when isolate limit reached. ( ex. isolate_special=1 Open file manager, open file toggles automaticaly)

probably better to implement and use getVisibleWindowsOnWorkspace()
also this as a workspace rule would be more flexible (not just for special workspaces)

probably better to implement and use getVisibleWindowsOnWorkspace()

Can you give some more details ? Isn't this just
 g_pCompositor->getWindowsOnWorkspace(m_iWorkspaceID)

also this as a workspace rule would be more flexible (not just for special workspaces

I sort of agree with you on this but on the other hand , what would happen to the windows that dont fit on the workspace any more , that would also need configuring  ( direction maybe) .
I think this feature is more suited for special workspaces.
Another concern is that this isn't as simple to do anymore

Future PR if all goes well:
Toggle workspace when isolate limit reached. ( ex. isolate_special=1 Open file manager, open file toggles automatically)

probably better to implement and use getVisibleWindowsOnWorkspace()

Can you give some more details ? Isn't this just

check if window isn't hidden, things like groups and swallowed windows would count as only 1

I do think this is a feature though , for example i have my terminal in a special workspace open mpv with swallowing , i wouldnt want mpv to spawn behind the special.
Now for groups i would imagine the same usability ( haven't used groups yet)
I think of it as , If i explicitly went out of my way to group/swallow windows , i clearly want them together.

that's what I meant, the current behavior does the opposite

that's what I meant, the current behavior does the opposite

Oh, you are right , i was testing my setup with isolate_special=2
Will try tomorrow to create the g_pCompositor->getVisibleWindowsOnWorkspace() method

there isn't any, you have to copy the getWindowsOnWorkspace() and check if window isn't hidden as well to create it

Did the workspace rule implementation in https://github.com/LamprosPitsillos/Hyprland/pull/1
Also did the VisibleWindows on Workspace but that doesn't work since window gets inside group later on
Will see how that can be handled

but that doesn't work since window gets inside group later on Will see how that can be handled
This is what i noticed last night, and was thinking of ways to solve it.

Great ! Only one small request , can "maxsize" be changed to "maxclients" ?

implementation is kinda meh but should be fully working after https://github.com/LamprosPitsillos/Hyprland/pull/2


broken doc link

the link is broken. please fix https://aka.ms/AnomalyFinderPrivatePreview

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

ID: 3bf09f20-c06f-0d5a-fdd8-c20279786534
Version Independent ID: 8606c464-8ca9-636e-ab5f-70e8db1c3752
Content: What is Anomaly Finder? - Microsoft Cognitive Services
Content Source: articles/cognitive-services/Labs/Anomaly-Finder/overview.md
Service: cognitive-services
GitHub Login: @TonyXing
Microsoft Alias: tonyxing

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

@sammym1982 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks Tony. I will reach out to this alias.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Tony Xing<EMAIL_ADDRESS>Sent: Monday, November 12, 2018 6:14 PM
To: MicrosoftDocs/azure-docs
Cc: sammym; Mention
Subject: Re: [MicrosoftDocs/azure-docs] broken doc link (#18342)
@sammym1982https://github.com/sammym1982 sorry about the experience. The project anomaly finder entered private preview in Oct with many enhancements and slightly different api signature from the lab api. Due to the nature of "private" preview, the broken link you mentioned is actually access controlled. If you are interested in trying the preview API, please reach out to kenshoteam<EMAIL_ADDRESS>In the mean time, we will think a better way on the "broken link" issue, probably remove that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/18342#issuecomment-438103786, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK7CiJZZO2fdBXZjaNdH0Hsp0n1Q4g4cks5uuisCgaJpZM4YRjcz.

@sammym1982
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


OCPBUGS-17171: Update OLM catalog image tags

What this PR does / why we need it:
OLM catalog images should reference images that correspond to the release
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #OCPBUGS-17171
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

/jira refresh

/override "Pipelines as Code CI / hypershift-operator-main-on-pull-request"

/test e2e-kubevirt-aws-ovn

kubevirt error looks like a CI registry flake:
error: failed to push image registry.build05.ci.openshift.org/ci-op-tlncpj0d/release:latest: unable to convert the image to a compatible schema version: received unexpected HTTP status: 500 Internal Server Error

/test e2e-kubevirt-aws-ovn

It looks like prow timed out trying to delete HCs issue again.
{"component":"entrypoint","error":"process timed out","file":"k8s.io/test-infra/prow/entrypoint/run.go:84","func":"k8s.io/test-infra/prow/entrypoint.Options.internalRun","level":"error","msg":"Error executing test process","severity":"error","time":"2023-08-04T08:23:37Z"}

/test e2e-aws


Lancer sheet update

Changes / Comments
Adds functionality for npcs and gives players more options that were lacking in current version
Note this shouldn't affect any player sheets as doesn't change the core values past ones you would changing a lot in play.
The exact changes are one, added more sizes for players to use as previously it didn't go up to the maximum npcs can get to.
Added more accuracy/difficulty to the choice select so players with specific builds can utilize there accuracy properly
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[x ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Shouldn't have anything else to add so ill wait till this is merged or denied.


linux-47 lost external connection from phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-47 lost external connection from phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting phone device to restore connection.


Input channels

Some observations:

Recent experiments show that NARS itself can be effective for perception processing:
https://groups.google.com/forum/#!topic/open-nars/pba2Mfyti0I
Currently all input events enter event bag and are processed using the systems general purpose temporal inference control strategy.

One one hand it should remain that all input comes from addInput(string); method,
but on the other hand not all input modalities can be processed by the general policy of event bag very effectively. That raises the question:
How to allow modality-specific control strategies, "custom channels" which only report their highest priority results into event bag, while being addressed by addInput without complicating the interface?
Idea:
Allow certain sensory input terms to be registered to go into a certain input channel.
These then don't enter NARS directly, but enter the custom channel instead, and are processed in that channel using a custom policy. The best results will be retrieved from the NARS instance automatically for transition to event bag.

https://github.com/automenta/narchy/blob/skynet5/nal/src/main/java/nars/control/CauseChannel.java#L17
https://github.com/automenta/narchy/blob/skynet5/nal/src/main/java/nars/NAR.java#L1646
https://github.com/automenta/narchy/blob/skynet5/nal/src/main/java/nars/control/Deriver.java#L57
https://github.com/automenta/narchy/blob/skynet5/app/src/main/java/nars/NAgentX.java#L192

I don't think that ensembles what we want besides that you also called it "Channel". Thanks anyways.

for reference

Already implemented. And the vision channel is already in development, not much missing anymore.


[CP][stable channel] Bug-fix for bug in CastMap

commit(s) to merge: 5b64d1233b1d8c12df6ec320a237b89ee738b5f6
merge instructions: clean merge
What is the issue: A cast in CastMap was incorrectly doing as V instead of as V?. This made otherwise valid code throw in some situations.
What is the fix: Changed to as V?, which allows null to be a value in the cast map.
Why cherrypick: It's a very simple and painless fix for a reported bug (meaning someone has already hit it).
Risk: None. Only difference is that some code which would incorrectly throw will now work correctly.
Link to original  issue(s): https://github.com/dart-lang/sdk/issues/45473
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @franklinyow

LGTM

lgtm

We are approving this for 5/21 window

Description:
Fix an incorrect behavior in CastMap, see #45473 for detail

cc: @kf6gpe

Merged to stable 375a2d7c66c8eb5d40872bdb57dd664b5f850d94 (2.13.1).


Fix empty instance_config.yml produced by header filter

Filter 'header' was previously 'molecule_header'. This filter should
prepend molecule header to the input content. But it was buggy since
probably move to community.molecule.
ISSUE TYPE

Bugfix Pull Request

Here are a few more details:

header filter is used especially in delegated driver. There is an example in issue ansible-community/molecule#1292

Without the fix, content of file molecule/instance_config.yml is bad (only one line) :

# Molecule managed

With the fix, content is ok:
# Molecule managed
- {address: <IP_ADDRESS>, connection: winrm, instance: inst-win-000, password: my_password,
  port: '5986', user: Administrator, winrm_server_cert_validation: ignore, winrm_transport: ntlm}

Please let me know if something more is useful. Thanks

Dear @ssbarnea, @gundalow
Please let me know if I can do something to help. thx

@jpmat296 Sorry for the delay, next time ping me on irc if I do not respond in one or two days.


Safari on iOS mobile - data binding issue

Bug Report
System Information

Operating System (Including distribution name and version number)
AMP version and build date (Always use the version number, 'latest' is not valid!)
Which AMP release stream you're using (Mainline, Nightly or FastTrack)

I confirm:

[ x] that I have searched for an existing bug report for this issue.
[ x] that I am using the latest available version of AMP.
[ x] that my operating system is up-to-date.

Accessing the panel on iOS keeps giving data binding issues which replicates the instances and does not allow us to select unless page is refreshed. This only happens on Safari as far as I know.

Can you verify if this is an issue on <IP_ADDRESS>?

It's still an issue. I get it on my phone. Android 14 chrome

Good to know about Android. I can test that more easily.

Caused by #962 - using that issue instead.


Dart2js fails when running on Dart VM with fixed-size integers

Reproduction:
DART_VM_OPTIONS=--limit-ints-to-64-bits tools/test.py -m release -c dart2js -r chrome -a ia32 --dart2js-batch --reset-browser-configuration dart2js_extra/21579_test

Typical failure:
tests/compiler/dart2js_extra/21579_test.dart:15:20:
Internal Error: '1000000000000000000000': FormatException: Invalid radix-10 number
1000000000000000000000
  a.sort((a, b) =><PHONE_NUMBER>000000000000 * a.compareTo(b));
                   ^^^^^^^^^^^^^^^^^^^^^^
The compiler is broken.
When compiling the above element, the compiler crashed. It is not
possible to tell if this is caused by a problem in your program or
not. Regardless, the compiler should not crash.
The Dart team would greatly appreciate if you would take a moment to
report this problem at http://dartbug.com/new.
Please include the following information:
* the name and version of your operating system,
* the Dart SDK build number (build number could not be determined), and
* the entire message you see here (including the full stack trace
  below as well as the source location above).
The compiler crashed: Internal Error: '1000000000000000000000': FormatException: Invalid radix-10 number
1000000000000000000000
#0      CompilerDiagnosticReporter.internalError (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/compiler.dart:1242:5)
#1      NodeListener.handleOnError (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/parser/node_listener.dart:371:14)
#2      NodeListener.handleLiteralInt.<anonymous closure> (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/parser/node_listener.dart:377:46)
#3      LiteralInt.value (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/tree/nodes.dart:1298:13)
#4      ResolverVisitor.visitLiteralInt (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:3507:66)
#5      LiteralInt.accept (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/tree/nodes.dart:1302:38)
#6      CommonResolverVisitor.visit (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/resolution_common.dart:32:54)
#7      ResolverVisitor.visitExpression (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:1093:31)
#8      ResolverVisitor.handleUserDefinableBinary (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:1370:37)
#9      ResolverVisitor.handleOperatorSend (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:1698:20)
#10     ResolverVisitor.visitSend (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:3025:14)
#11     Send.accept (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/tree/nodes.dart:716:38)
#12     CommonResolverVisitor.visit (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/resolution_common.dart:32:54)
#13     ResolverVisitor.visitReturn (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:3601:5)
#14     Return.accept (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/tree/nodes.dart:1609:38)
#15     CommonResolverVisitor.visit (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/resolution_common.dart:32:54)
#16     ResolverVisitor.visitFunctionExpression (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:605:5)
#17     FunctionExpression.accept (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/tree/nodes.dart:1209:38)
#18     CommonResolverVisitor.visit (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/resolution_common.dart:32:54)
#19     ResolverVisitor.resolveArguments (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:711:33)
#20     ResolverVisitor.handleDynamicAccessSemantics (file:///b/swarming/w/ir/cache/builder/sdk/pkg/compiler/lib/src/resolution/members.dart:2287:11)

/cc @sigmundch @rakudrama

Fixing with https://github.com/dart-lang/sdk/issues/33286


addAsteriskPlugin not compatible with repeater

Reproduction
https://formkit.link/574b84f69f2b205d11fdc10e24b41933
Describe the bug
The repeater is visible but the inputs/labels are not. It is fine when the plugin is removed. This is using the basic example code from https://formkit.com/inputs/repeater

Environment
• OS: Windows 10
• Browser Chrome 119.0.6045.16

Looks like the plugin is not properly accounting for the repeater input having a legend section. This updated version should handle it. https://formkit.link/79d26284f4f28177d9a2a7c5e250a6bb

Fixed the example to work with some inputs that would also fail like transferlist and the repeater.


F1945 lambda secret refs

This PR adds support for references to existing secrets to this module.
If a user sets an env var to the value "{{ secret(arn) }}", we were just discarding this env var. With this PR, the env var will now be included and have a value set to the arn passed in. We also add permissions for the app to be able to the secret from secrets manager.

Please update CHANGELOG.md


The menu map of Rio is glitched.

Since the last update, the menu map of Rio is now glitched like in the picture below, with those squares moving around as I move the map. This doesn't happen in the other cities, and the minimap isn't affected.
I'm using Beta 2.

I also want to thank you for your time for addressing those issues!

Oh, it does happen when bilinear filtering is off...

Fixed in 772a676


Added information about compatibility with OpenVPN Connect (iOS)

VPN is working fine with OpenVPN Connect on iOS devices.

Thanks a lot for testing; much appreciated!


Set the pointer passed to free() to NULL

This is permitted by the spec
( https://man7.org/linux/man-pages/man3/free.3p.html ),
and makes sure that dereferencing the pointer will
result in a crash.

This won’t work because it’s only setting the local copy of the pointer to NULL. In order to support this via the API we need to create a exported function that takes a pointer to a pointer, makes a local copy, NULLs the original pointer (after dereferencing it) and then passes the local copy to iso_free().


Add PR size labeler workflow

Added config to label PR size

@prxjwal It seems to be correct.
Please could you change the tags so that instead of starting with size/, the tags start with Size:
Example: Size: XS
And a question, why the comment “Changed to pull_request_target”?
Thank you.


Support for toDataURL

Using canvas with an image that refreshes very often (showing simulation real time) I am using a library that requires an url to the image. Therefore I would love a canvas.toDataURL() function.
Leaflet.ImageOverlay.Rotated.js ->
var overlay = L.imageOverlay.rotated(...        });
// routine that updates image
overlay.setUrl(canvas.toDataURL());

I agree! Would be nice, if this library supports the toDataUrlfunctionality out of the box. The workaround I am currently using goes like this:
index.html
<script>CanvasDataUrl = (canvas, dataType) => { return canvas.toDataURL(dataType); }</script>

MyCanvas.razor.cs
[Inject]
public IJSRuntime JSRuntime { get; set; }

protected BECanvasComponent _beRef;

string myCanvasDataUrl = "";

public async Task StoreCanvasImage()
{
   this.myCanvasDataUrl = await this.JSRuntime.InvokeAsync<string>("CanvasDataUrl", this._beRef.CanvasReference, "image/jpeg");
}


Unable to Invoke

this issue has been discovered for new v1.0.10

this issue will be fixed at next version.

check v1.0.11


Failed to sign package; error was<PHONE_NUMBER>

Getting the following when I try to publish new packages on a Windows Server 2019 WSUS server. I have been able to publish such packages previously (I've recently upgraded to v1.4.2011.23 so it could be related?) - the package will look like it is about to publish before failing and say "Failed to sign package; error was<PHONE_NUMBER>"
Any ideas

I can confirm if I roll WPP back to an older version e.g. 1.4.1904.29 then I can deploy packages fine so me thinks there is an issue with the newer version?

I can confirm if I roll WPP back to an older version e.g. 1.4.1904.29 then I can deploy packages fine so me thinks there is an issue with the newer version?

Hi, I can publish to WSUS 2019 with the latest version of WPP. Can you look into the publisher log (%LocalAppData%\Temp) to see if any error message appears?

Also, ensure your WSUS server can access Internet to use the timestamping option.


remove duplicate selectors

I accidentally noticed that you have repeating selectors in output css. Looks like source is also containing duplicates.

I signed it!

Hey @Jabher, thanks for opening a PR :) The effort is greatly appreciated.
However, we probably cannot merge this. Those duplications are most on purpose to increase the specificity of the rules. It would actually break the behavior of the grid.
Regardless of whether they are technically necessary, we can’t change it at this point anymore since it would be a breaking change. Imagine some site’s CSS rules suddenly being applied when they didn’t before because the specificity of our  rules set decreased in between releases.

@surma could you explain what's the difference between .myclass and .myclass.myclass?
I'm guessing the browser detects it's the same class so it doesn't look for an element with the same class twice...

@tleunen CSS Selector specificity is an important concept whenever you are writing CSS. It determines when a CSS rule takes precedence over another.
Take a look at this codepen. All three rules apply to the <p>. The text appears blue, because the class selector (.example) gives rule 2 a higher specificity than the type selector (p) of rule 1. To be exact, rule 1 has a specificity of 001, while rule 2 has a specificity 010. If you comment in rule 3’s body, the text will appear red, because the number of class selectors is higher than in rule 2, giving it a specificity of 020.
CSS rules are applied in order of their specificity. I.e: Highest specificity comes last, overwriting all the previous rules.

I think your example is wrong and should be this http://codepen.io/anon/pen/KpYzRP
Anyway it's just a detail here.
Didn't know about this trick, but it seems it kind of a hack in this case because if your CSS is well organized, you know the last rule will be applied (if they have the same specificity).
It might make more sense to write something like .mdl-cell--#{$i}-col..mdl-cell--#{$i}-col-phone instead.

I think your example is wrong and should be this http://codepen.io/anon/pen/KpYzRP
if your CSS is well organized, you know the last rule will be applied

No. Order is relevant if and only if the two rules have the same specificity.
The only thing you changed is the order in which rule 2 and 3 appear. They have different specificity, so the order does not matter. As you can see, the text appears red in both your and my example if you remove the comments.

@surma Got it, thanks, forgot about this behavior, though it's misprint.
But why then not to use body .mdl-cell--#{$i}-col-phone instead of .mdl-cell--#{$i}-col-phone.mdl-cell--#{$i}-col-phone ? It will give more compact output code while beating with specificity (well, 011 instead of 020, but it's still bigger than 010).
Rough calc gives me 462 chars saved, not very much, but still useful.

Using types in selectors, even more so if they are not related to the component being styled, is generally considered bad practice. @sgomes will yell at you :wink: You can tell that we almost never have types
in our selectors.
Secondly, as we’re trying to overwrite our own rules from earlier in that file, adding a type selector is not necessarily sufficient.
In any case, as I said before, we can’t do anything about this now, anyways, because it’s a breaking change.

Rough calc gives me 462 chars saved, not very much, but still useful.

Gzip doesn’t care :computer:

Yelly @sgomes reporting for duty! Just kidding :)
Here's a bit of a rundown of why we have this pattern so much in our code (well, comparatively speaking).
Using body before a rule is a generally accepted hack, but it's still a hack :) It'll get you one extra level of specificity, but at the cost of referencing something that really has no relevance to your rule. Also, remember that the body tag is actually optional (styling should still work if you don't add it, but suddenly that CSS becomes even less semantically meaningful). Still, it's one solution.
Another option is to use other elements in the selector, such as div or button or something along those lines, and appending them to the class, e.g. button.mdl-button. That's sometimes undesirable, since it locks your implementation to a specific element; we wouldn't be able to have MDL buttons on <input>, <button> and <a> as easily if we had done that, for example.
Another common scenario is to use hierarchical knowledge, such as adding a selector for a parent you know will be there (e.g. .container .child). That's also an option, but it needs to be used carefully, as you could be locking yourself into a very specific hierarchical structure, particularly when you have multiple levels of that. That may or may not be a good thing, depending on what you need. And again, if you're simply doing it for bumping up a rule's specificity, you're sort of breaking semantics anyway, because it's not relevant to the rule that your target is inside that specific parent.
Mind you, repeating the class isn't a perfect solution either. It's an unusual pattern, which makes things slightly harder to grok if you haven't come across it before, but generally its benefits fit very well with a CSS library like MDL.
Hope this clears things up! :)

It was also brought up:

Why not use .mdl-cell--#{$i}-col.mdl-cell--#{$i}-col-phone?

We shouldn't do that since it then locks you into needing to define the first class when you don't need to. You can specify just the phone, tablet, and desktop columns and be fine without the general one.


Fixes #2609 - Upgrade Sonar github action version

Fixes https://github.com/medplum/medplum/issues/2609
Using recommended config from here: https://github.com/SonarSource/sonarcloud-github-action
We pinned the version due to previously failed upgrade: https://github.com/medplum/medplum/pull/1893
We'll see if this works.

coverage: 94.292%. remained the same when pulling 2e0f3b3ed84bd54e6390e2b7bb317532ad29d298 on cody-upgrade-sonar-github-action into 43e494a0b54de13f79218ce3daeb8564ce86b773 on main.


buildbot: 1.1.1 -> 1.2.0, fix build

Motivation for this change
#41894 broke buildbot again by updating buildbot-pkg but not the rest of the packages. This PR fixes this by updating the rest of the packages to 1.2.0.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build buildbot buildbot-worker buildbot-pkg buildbot-ui buildbot-full

Success on x86_64-linux (full log)
Attempted: buildbot, buildbot-worker, buildbot-pkg, buildbot-ui, buildbot-full
Partial log (click to expand)

buildbot.test.unit.test_www_endpointmatchers.ViewBuildsEndpointMatcherBranch.test_build
-------------------------------------------------------------------------------
Ran 5272 tests in 81.175s

PASSED (skips=237, successes=5035)
building '/nix/store/l54df0mfgyxb007i17ddgw1v5xk7m5yw-wrapped-buildbot-1.2.0.drv'...
building '/nix/store/yqpsvnmbxj1mx0xpbqgq338ypq5yjfff-wrapped-buildbot-1.2.0.drv'...
'/nix/store/zl17wkldwc2zn6r0cfnnmw7dvcapjhjj-wrapped-buildbot-1.2.0/lib' -> '/nix/store/9lhaj26ca9jjvsg4vjbjsm7kns4in2xm-buildbot-1.2.0/lib'
'/nix/store/gq6k54aqwb58xvx6zvxlnsrqha5cr6sv-wrapped-buildbot-1.2.0/lib' -> '/nix/store/9lhaj26ca9jjvsg4vjbjsm7kns4in2xm-buildbot-1.2.0/lib'

Failure on aarch64-linux (full log)
Attempted: buildbot, buildbot-worker, buildbot-pkg, buildbot-ui, buildbot-full
Partial log (click to expand)
running=OS_STDOUT_CAPTURE=${OS_STDOUT_CAPTURE:-1} \
OS_STDERR_CAPTURE=${OS_STDERR_CAPTURE:-1} \
OS_TEST_TIMEOUT=${OS_TEST_TIMEOUT:-60} \
${PYTHON:-python} -m subunit.run discover -t ./ .  --load-list /build/tmp7xbbRB
Ran 165 tests in 161.371s
PASSED (id=0)
cannot build derivation '/nix/store/2qj1q97pglaafynkshnmhsxw453d81a7-buildbot-1.2.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/bbrcf5hwhissc7lvcqmn0nlyjy228f68-wrapped-buildbot-1.2.0.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/x2c05gk3qickrclcc5iqc8ys70in3hkr-wrapped-buildbot-1.2.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/2qj1q97pglaafynkshnmhsxw453d81a7-buildbot-1.2.0.drv', '/nix/store/bbrcf5hwhissc7lvcqmn0nlyjy228f68-wrapped-buildbot-1.2.0.drv', '/nix/store/cwgabggday3afy04mwmm9zxl5ggcvc21-buildbot-worker-1.2.0.drv', '/nix/store/x2c05gk3qickrclcc5iqc8ys70in3hkr-wrapped-buildbot-1.2.0.drv' failed


fix(editor): icon was not visible for lit Elements

The icon registered for lit-elements was defined as polymer.
Renamed it to litElement.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature (please, look at the "Scope of the project" section in the README.md file)
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

There no icon when switching between to the lit framework
Issue Number: N/A
What is the new behavior?
The correct icon is shown when switching between frameworks
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

All good to me.


Runtime exception from NSKeyedUnarchiver unarchivedObjectOfClass:fromData:error

Integration Method: CocoaPods
Xcode Version: Version 13.2.1 (13C100)
Library Version: 4.1.2
Platform: iOS
Language: Swift
Description: I recently hit a runtime error in our project in the method + (id)unarchiveFromFile:(NSString *)filePath asClass:(Class)class.
Expected Behavior: App should not have this runtime exception.
The error in question
*** -[NSKeyedUnarchiver validateAllowedClass:forKey:]: NSSecureCoding allowed classes list contains [NSObject class], which bypasses security by allowing any Objective-C class to be implicitly decoded. Consider reducing the scope of allowed classes during decoding by listing only the classes you expect to decode, or a more specific base class than NSObject. This will become an error in the future. Allowed class list: {(
    "'NSObject' (0x10e5222e8) [/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS.simruntime/Contents/Resources/RuntimeRoot/usr/lib]"
)}

The offending method
+ (id)unarchiveFromFile:(NSString *)filePath asClass:(Class)class
{
    id unarchivedData = nil;
    @try {
        if (@available(iOS 11, macOS 10.13, tvOS 11, watchOS 4, *)) {
            NSError *error = nil;
            NSData *data = [NSData dataWithContentsOfFile:filePath];
            unarchivedData = [NSKeyedUnarchiver unarchivedObjectOfClass:[NSObject class] fromData:data error:&error];
            if (error) {
                MPLogError(@"%@ got error while unarchiving data in %@: %@", self, filePath, error);
            }
        } else {
            unarchivedData = [NSKeyedUnarchiver unarchiveObjectWithFile:filePath];
        }
        // this check is inside the try-catch as the unarchivedData may be a non-NSObject, not responding to `isKindOfClass:` or `respondsToSelector:`
        if (![unarchivedData isKindOfClass:class]) {
            unarchivedData = nil;
        }
        MPLogInfo(@"%@ unarchived data from %@: %@", self, filePath, unarchivedData);
    }
    @catch (NSException *exception) {
        MPLogError(@"%@ unable to unarchive data in %@, starting fresh", self, filePath);
        // Reset un archived data
        unarchivedData = nil;
        // Remove the (possibly) corrupt data from the disk
        NSError *error = NULL;
        BOOL removed = [[NSFileManager defaultManager] removeItemAtPath:filePath error:&error];
        if (!removed) {
            MPLogWarning(@"%@ unable to remove archived file at %@ - %@", self, filePath, error);
        }
    }
    return unarchivedData;
}

@pouriaalmassi this issue has already been fixed in the latest version, v4.1.3, please update and let us know if there are any other issues.


daemon: fix typo in policy trigger log

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.08%) to 42.017% when pulling 22ef280750f3039a61105311e7ddb9b4ebc5053c on pr/ianvernon/fix-typo-trigger into 69037fe562f3e66e242424ca3461c872054ddf79 on master.

test-me-please


Add Code for Maximum Subarray Sum

Add code for Maximum Subarray Sum using Kadane's Algorithm

Assigning this to you @abhishekaman09, let the PR rain :tada:

@abhishekaman09 any issues?


contributor account for: Ayumu Tsubosaka

Please create an account for a contributor.
Contributor details:

Name: Ayumu Tsubosaka
Email<EMAIL_ADDRESS>
Request Details:

Requested by: William Sullivan

@willrockout can you please confirm here that the contributor has received and can use the account? Thanks!

I've actually gotten no response from the contributor in regards to the account or spreadsheet.


Corresponding to new chatwork page document

ChatWork seems to change some lines declaring variables.
from
var ACCESS_TOKEN = '**************************';
var myid = '*******';

to
var ACCESS_TOKEN               = '**************************';
var myid                       = '*******';

I changed the regular expression for new ChatWork document.
Please refer to #7 RuntimeError no token.

@swdyh Hi, please check this PR.
thanks.

@hirokishirai Thank you for your PR. :octocat:


Implement Tutorial Group Commands

Implement the following commands for TutorialGroups:

viewTG
addTG
findTG
deleteTG

Issue fixed by the linked pull request


TP - Adds confirmation dialogs when unlinking ds/server and standardizes all ds tables

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/227/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/185/


42: Create and upload release 0.8.19

resolve #39
resolve #40
resolve #42
Task-Url:
https://github.com/IQNOMY/IQNOMY_Magento_extension/issues/issue/42

Pay attention installing this version if you are using an old version of the IQNOMY_Magento_extension first fully uninstall that version. Otherwise you get errors like:
a:5:{i:0;s:120:"SQLSTATE[42S02]: Base table or view not found: 1146 Table 'shopname_ma2.mage_techtwo_iqnomy_templates' doesn't exist";i:1;s:5402:"#0 /var/www/vhosts/kopenkopen.nl/httpdocs/magento8/lib/Varien/Db/Statement/Pdo/Mysql.php(110): Zend_Db_Statement_Pdo->_execute(Array)

tested and fixes work