fix: truncate step name + max height for modal content

Fixes

Max height for the template modal with scrollbar
Truncate step name if too long

Screenshots

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Perhaps we should show the name and description of the template on the top - above the steps?

Perhaps we should show the name and description of the template on the top - above the steps?

I'll make another step showing all template information, that will be the first one and then you would click "Start" to show the first form


Use etcdctl v3

The version of etcdctl on our nodes is the default CoreOS version...which is v2.
However, we run etcd v3 for our cluster.
We frequently ssh to an etcd node and run etcdctl.  This results in etcdctl v2 querying etcd v3.
Some etcdctl items needed for etcd v3 are not there. (e.g. backup/snapshot is completely different).
We need to make sure we run the etcdctl binary that comes with the etcd release we are running.  This means we should be running etcdctl via a container that has the current etcd version.
e.g.:
sudo rkt run --net=host --volume ssl,kind=host,source=/etc/etcd/ssl,readOnly=true --mount volume=ssl,target=/etc/etcd/ssl --set-env=ETCDCTL_API=3 coreos.com/etcd:v3.2.5 --exec etcdctl -- --cacert=/etc/etcd/ssl/client-ca.pem --cert=/etc/etcd/ssl/client.pem --key=/etc/etcd/ssl/client-key.pem --endpoints=https://<IP_ADDRESS>:2381  member list

(this isn't quite correct, but you get the idea).

I don't think this has any relative impact yet.  However, for backup/recovery this has to be fixed.

If you run etcdctl from k2-tools, the Dockerfile already sets this for you. Interestingly though, if you read the etcdctl source, some options are still implemented using the etcd2 protocol. It is not clear to me which features of etcd3 we require from the command line. Backup and recovery make sense.

Yes, but we ssh to the nodes to test status...i.e. we don't run the k2-tools version.
Also you have to set ETCDCTL_API=3 for v3 etcdctl to run as v3...
Note: v3 has different arg names and commands.

NAME:
	etcdctl - A simple command line client for etcd3.

USAGE:
	etcdctl

VERSION:
	3.2.6

API VERSION:
	3.2

COMMANDS:
	get			Gets the key or a range of keys
	put			Puts the given key into the store
	del			Removes the specified key or range of keys [key, range_end)
	txn			Txn processes all the requests in one transaction
	compaction		Compacts the event history in etcd
	alarm disarm		Disarms all alarms
	alarm list		Lists all alarms
	defrag			Defragments the storage of the etcd members with given endpoints
	endpoint health		Checks the healthiness of endpoints specified in `--endpoints` flag
	endpoint status		Prints out the status of endpoints specified in `--endpoints` flag
	watch			Watches events stream on keys or prefixes
	version			Prints the version of etcdctl
	lease grant		Creates leases
	lease revoke		Revokes leases
	lease timetolive	Get lease information
	lease keep-alive	Keeps leases alive (renew)
	member add		Adds a member into the cluster
	member remove		Removes a member from the cluster
	member update		Updates a member in the cluster
	member list		Lists all members in the cluster
	snapshot save		Stores an etcd node backend snapshot to a given file
	snapshot restore	Restores an etcd member snapshot to an etcd directory
	snapshot status		Gets backend snapshot status of a given file
	make-mirror		Makes a mirror at the destination etcd cluster
	migrate			Migrates keys in a v2 store to a mvcc store
	lock			Acquires a named lock
	elect			Observes and participates in leader election
	auth enable		Enables authentication
	auth disable		Disables authentication
	user add		Adds a new user
	user delete		Deletes a user
	user get		Gets detailed information of a user
	user list		Lists all users
	user passwd		Changes password of user
	user grant-role		Grants a role to a user
	user revoke-role	Revokes a role from a user
	role add		Adds a new role
	role delete		Deletes a role
	role get		Gets detailed information of a role
	role list		Lists all roles
	role grant-permission	Grants a key to a role
	role revoke-permission	Revokes a key from a role
	check perf		Check the performance of the etcd cluster
	help			Help about any command

OPTIONS:
      --cacert=""				verify certificates of TLS-enabled secure servers using this CA bundle
      --cert=""					identify secure client using this TLS certificate file
      --command-timeout=5s			timeout for short running command (excluding dial timeout)
      --debug[=false]				enable client-side debug logging
      --dial-timeout=2s				dial timeout for client connections
      --endpoints=[<IP_ADDRESS>:2379]		gRPC endpoints
  -h, --help[=false]				help for etcdctl
      --hex[=false]				print byte strings as hex encoded strings
      --insecure-skip-tls-verify[=false]	skip server certificate verification
      --insecure-transport[=true]		disable transport security for client connections
      --key=""					identify secure client using this TLS key file
      --user=""					username[:password] for authentication (prompt if password is not supplied)
  -w, --write-out="simple"			set the output format (fields, json, protobuf, simple, table)```

vs

./etcdctl
NAME:
etcdctl - A simple command line client for etcd.
WARNING:
Environment variable ETCDCTL_API is not set; defaults to etcdctl v2.
Set environment variable ETCDCTL_API=3 to use v3 API or ETCDCTL_API=2 to use v2 API.
USAGE:
etcdctl [global options] command [command options] [arguments...]
VERSION:
3.2.6
COMMANDS:
backup          backup an etcd directory
cluster-health  check the health of the etcd cluster
mk              make a new key with a given value
mkdir           make a new directory
rm              remove a key or a directory
rmdir           removes the key if it is an empty directory or a key-value pair
get             retrieve the value of a key
ls              retrieve a directory
set             set the value of a key
setdir          create a new directory or update an existing directory TTL
update          update an existing key with a given value
updatedir       update an existing directory
watch           watch a key for changes
exec-watch      watch a key for changes and exec an executable
member          member add, remove and list subcommands
user            user add, grant and revoke subcommands
role            role add, grant and revoke subcommands
auth            overall auth controls
help, h         Shows a list of commands or help for one command
GLOBAL OPTIONS:
--debug                          output cURL commands which can be used to reproduce the request
--no-sync                        don't synchronize cluster information before sending request
--output simple, -o simple       output response in the given format (simple, extended or json) (default: "simple")
--discovery-srv value, -D value  domain name to query for SRV records describing cluster endpoints
--insecure-discovery             accept insecure SRV records describing cluster endpoints
--peers value, -C value          DEPRECATED - "--endpoints" should be used instead
--endpoint value                 DEPRECATED - "--endpoints" should be used instead
--endpoints value                a comma-delimited list of machine addresses in the cluster (default: "http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001")
--cert-file value                identify HTTPS client using this SSL certificate file
--key-file value                 identify HTTPS client using this SSL key file
--ca-file value                  verify certificates of HTTPS-enabled servers using this CA bundle
--username value, -u value       provide username[:password] and prompt if password is not supplied.
--timeout value                  connection timeout per request (default: 2s)
--total-timeout value            timeout for the command execution (except watch) (default: 5s)
--help, -h                       show help
--version, -v                    print the version

ugh, the flags are different, so if i wanted to use cluster-health it would require a change of environment variable and flags, sigh.

I haven't dug into potential incompatibilities, but for as far as cluster-health goes, I don't think it's a problem.
core@ip-10-0-175-130 ~ $ etcdctl --version
etcdctl version 2.3.7
core@ip-10-0-175-130 ~ $ ETCDCTL_API=2 etcdctl --endpoint https://<IP_ADDRESS>:2379 --ca-file=/etc/etcd/ssl/peer-ca.pem --cert-file=/etc/etcd/ssl/peer.pem --key-file=/etc/etcd/ssl/peer-key.pem cluster-health
member 133ef08f66f8d5fc is healthy: got healthy result from https://<IP_ADDRESS>:2379
member a35104fa78e6be0a is healthy: got healthy result from https://<IP_ADDRESS>:2379
member b185e28292711e2e is healthy: got healthy result from https://<IP_ADDRESS>:2379
member cee4cb6c9c90611e is healthy: got healthy result from https://<IP_ADDRESS>:2379
member e478ebcd8a7b53fc is healthy: got healthy result from https://<IP_ADDRESS>:2379
cluster is healthy
core@ip-10-0-175-130 ~ $ ETCDCTL_API=3 etcdctl --endpoint https://<IP_ADDRESS>:2379 --ca-file=/etc/etcd/ssl/peer-ca.pem --cert-file=/etc/etcd/ssl/peer.pem --key-file=/etc/etcd/ssl/peer-key.pem cluster-health
member 133ef08f66f8d5fc is healthy: got healthy result from https://<IP_ADDRESS>:2379
member a35104fa78e6be0a is healthy: got healthy result from https://<IP_ADDRESS>:2379
member b185e28292711e2e is healthy: got healthy result from https://<IP_ADDRESS>:2379
member cee4cb6c9c90611e is healthy: got healthy result from https://<IP_ADDRESS>:2379
member e478ebcd8a7b53fc is healthy: got healthy result from https://<IP_ADDRESS>:2379
cluster is healthy
core@ip-10-0-175-130 ~ $

@davidewatson yes that works, but look at the flags for v3, i cant just change environment variable and run a command.

Work on  kraken (cli) will allow for the right version of etcdctl and effectively fix this issue. I suggest we close this ticket @mikeln , is this ok?

yes, when the PR for this is merged.

https://github.com/samsung-cnct/kraken/pull/176


Modify AdminLTE 2 | Invoice Title

How to modify The "AdminLTE 2 | Invoice " Title on print screen.  I want to Change it to "adminLTE my order print "
Please anyone help.

Did you mean change page title? You can change it in title tag.
<title>AdminLTE 2 | Invoice</title>
in
<head>...</head>

I already did it. Thanks
On Wed, Feb 20, 2019, 1:18 AM heemza<EMAIL_ADDRESS>wrote:

Did you mean change page title? You can change it in title tag.
AdminLTE 2 | Invoice
in
...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/almasaeed2010/AdminLTE/issues/2059#issuecomment-465271821,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkyovP8QL9M4CRVUpCzDQj2rPBuPzQeGks5vPE4KgaJpZM4a5aMd
.


#10574: Fix lerp_bw and update golden function

Issue : #10574 , Tracking Issue : #10573
Fixed ternary lerp_bw
https://github.com/tenstorrent/tt-metal/actions/runs/10055753475 - passed

https://github.com/tenstorrent/tt-metal/actions/runs/10061785998 - passed


📃: Adding Bert Quiz application in using Reactjs

:red_circle: Title :  Bert Quiz App
:red_circle: Aim : Develop an online quiz platform with multiple-choice questions, scoring, and real-time feedback.
:red_circle: Tech Stack Used : Reactjs ,Node js
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Maheshwari Love
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.)GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney can you please assign this


[FEAT] TSDuck Donation via Wise

Is your feature request related to a problem?

I had an issue trying to send a TSDuck donation via PayPal ( https://tsduck.io/donate/ ).

Ref:
https://www.paypal-community.com/t5/Products-Services/Donation-to-this-recipient-aren-t-supported-in-this-country/td-p/2334804
Proposed solution:

Support/Accept TSDuck Donation via Wise. As a bonus, the transfer fee is cheaper than PayPal.
Alternatives:

Publish Name & IBAN of the bank account.
Additional information:

@lelegard
I sent related email on 1st November 2022 to your git commit email.

Not currently possible, sorry.


KAFKA-9286: Optimize RecordAccumulator

https://issues.apache.org/jira/browse/KAFKA-9286

For producer, the Sender call RecordAccumulator#ready to collect all ready nodes. Then it call RecordAccumulator#drain to iterate all ready nodes's topic partition to check if there is data to send.

This process is redundancy. When we collect all ready nodes, its ready topic partition can also be collected.
In our case, the producer only produce data to part(about 20) of all partitions(1024), with this patch, the performance can be increased ~3.4%.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

If i understand correctly, one of the difference in behaviour provided by this PR is to avoid redundant calls to partitionsForNode?

Yes


New syntax-highlighters for CSS, JS, JSON

Some examples using the new "Local View" (see coming experimental feature, which I will post to an issue).

LGTM
@sbcgua any comments for the CSS changes

Wow, new cool parsing stuff :)) Nice, nice ! Tested, works fine.
Colors:

I would remove red coloring from brackets (operator) - they seem popping up too much from the rest of the code
maybe change color of values to green - to conform with strings in abap (arguable - violet looks fine, just out of consistency considerations) - this supposes creating a selector instead of using attr_val
if yes for the green above then probably selector should be changed from green as well for consistency. E.g. to violet ?

Anyway, a matter of preference. I'd just recommend removing red brackets :) Otherwise looks very cool and works.
abap

css

js

I removed the brackets since they where causing also too much workload. Also changed the text style as suggested. Should we change the attr_val style (for xml/html) to green as well to be consistent?

Test repo: https://github.com/mbtools/abapgit-syntax-highlight
You can switch between branches to get diffs to render.

I'm done with this one. Anymore wishes regarding syntax highlighting?

@sbcgua any additional comments/suggestions?

Sorry, have been a bit busy. The syntax CSS looks fine. But one thing. I'd better move
cab09cf (hover style changes) to another PR. The change is not related to syntax highlighting and, I'd say, it is very controversial ... there are better ways to emphasize active element. Let's separate into another PR. Otherwise - OK from me.

P.S. zcl_abapgit_file_status reordering: looks like no changes ... is it related to syntax highlight ?

Removed hover.
PS zcl_abapgit_file_status was just a pretty print which reordered the method (which is a different issue).


Add gcm_sender_id prop on ManifestOptions

Just adding gcm_sender_id prop on ManifestOptions
related: https://github.com/arthurbergmz/webpack-pwa-manifest/issues/46

gcm_sender_id is legacy stuff and shouldn't by used

@frlinw could you please provide some documentation on that?
I am following this doc in order to setup Push Notifications on my PWA. And it says:

For Chrome to route FCM messages to the correct service worker, it needs to know the Sender ID. Supply this by adding a gcm_sender_id property to your app's manifest.json file.

ps: it also says:

Note: The gcm_sender_id is required for Chrome prior to version 52, Opera Android, and Samsung Internet.

Since this config is legacy, but it is still working and documented, I think this could be accepted for now. What do you think? I will add a comment saying it is legacy (adding the documentation you'll recommend on that) in the code...

comment added here for information https://github.com/arthurbergmz/webpack-pwa-manifest/issues/46#issuecomment-370541516
Chrome 52 is a release from 2016/07, focus on last 2 (or 3) versions for evergreen browsers is a best practice.
If you want to start with web push, try this one https://web-push-book.gauntface.com/ (gauntface is a googler working on Workbox)
Anyway, custom properties in pwa-manifest options are added as-is in the output manifest.json

@frlinw thank you very much for the references! =)
I'm still reading https://web-push-book.gauntface.com/ .
It is very well written.
Well, so it doesn't make sense to add gcm_sender_id. Therefore I'll close this PR and open another one with the publicPath prop.
Is this ok for you?

@ghashi do not hesitate to bookmark the service worker cookbook from mozilla too https://serviceworke.rs/
And a lil useful tool to generate VAPID easily https://web-push-codelab.glitch.me/
About the PR, I let @arthurbergmz answer

Not adding gcm_sender_id, but I updated our typing with publicPath. Thanks!


Integration tests should use real block stores (e.g. GCS, S3)

The reason is that there's a non-trivial amount of code that only gets exercised when we use a real block store.  It has happened a few times where bugs in those code were caught only in production.

duplicate. referenced in aggregation issue


Implement Key.get decently

Currently, if you call it before command line parsing, it's going to alway returns the default values.
I see two ways to fix it:

Add a way to check that command line parsing at occurred and raise an exception if it's not the case.
Introduce a fully abstract datatype needed by Key.get and that is only provided inside the configure and connect methods.

One uses exception, one is type-crazy, I'm not sure which one I prefer ...

Don't know anything about the context but if this is considered a programming error then just raise Invalid_argument: it doesn't seem like a constraint that is tricky to enforce by the client.

It's trickier than that. There are multiple phases during configuration and all the methods of a configurable are not needed at the same phase. Some before command line parsing, some after. It's clearly possible to get it wrong. Arguably, Key.get should not be used to begin with, but it's still a bit used in mirage (I would need to rewrite even more portions of it and I didn't).
@samoth: maybe a solution is to make connect and configure returns a Key.value ....

an exception should be enough. Both will cause a compilation error of config.ml anyway.

maybe a solution is to make connect and configure returns a Key.value

if that make the Key.get disappear  that might indeed be a good idea.


Google maps

Please send your PR's the develop branch.
Don't forget to add the change to changelog.md.

Does the pull request solve a related issue? [yes | no]
If so, can you reference the issue?
What does the pull request accomplish? (please list)
If it includes major visual changes please add screenshots.

This PR is probably sent by mistake?


Feature/make classes abstract

Closes #85
I stumbled upon this awesome project during the Hacktoberfest. It would be great if you could label this PR according to their recent update

Hi @bofalke, thank you for your contribution!
As stated by Digital Ocean in that link:

Submitted in a public repo AND (
The PR is labelled as hacktoberfest-accepted by a maintainer OR
Submitted in a repo with the hacktoberfest topic AND (
The PR is merged OR
The PR has been approved
)

This repository already has the "hacktoberfest" topic, so it counts :smiley: worry not

I have fixed the branch name because it was erroneously targetting a tag.
Please change the merge target to release-1.3.0

@bofalke no need for those classes to be declared final too, remove that please :+1:
You gave me some ideas on how we should improve the contributing guidelines to have these sorts of things explained. Better than to then have to look out for IDE and editor default :grin:

Updated the classes to not be final

@all-contributors please add @bofalke for code
Thank you very much!


Generated RTC & RTM Token not working in RTCClient

Hello,
Based on your sample code, I integrated the RTC and RTM token generation in my web application which is served via express.js for server side. I successfully arrive to generate and retrieve the tokens, but when I pass the RTC or RTM in the RTCclient.join channel function, it doesn't work.

I used this endpoint with the function generateRTEToken. (I also tried the others functions but result is same)

app.get('/rte/:channel/:role/:tokentype/:uid/', nocache , generateRTEToken);

In my script "webvr-broadcast-client.js", I improved the function JoinChannel()
with the token request.
function joinChannel() {
 fetch("https://example.com/rte/web/publisher/uid/44/").then(function(response) {
return response.json();
}).then(function(data) {
token = data.rtcToken; (RTC or RTM are well retrieved)

rtcClient.setClientRole('audience', () => {
  console.log('Client role set to audience');
}, (e) => {
  console.log('setClientRole failed', e);
});

rtcClient.join(token, channelName, 0, (uid) => {

    console.log('User ' + uid + ' join channel successfully');
    localStreams.uid = uid;
    createBroadcaster(uid);   // Load 3D model with video texture
    createCameraStream(uid);  // Create the camera stream
    joinRTMChannel(uid);      // join the RTM channel
}, (err) => {
    console.log('[ERROR] : join channel failed', err);
});

}).catch(function() {
});
}

The error returned is :
"Get server node failed [NO_AUTHORIZED]" – "https://webrtc2-ap-web-2.agoraio.cn/api/v1" – "NO_AUTHORIZED"
I confirm the code structure work, by using a Temp RTC Token it works very well.
But when I pass a generated RTC or RTM token as parameter, it doesn't work.
At this stage you can reproduce the scenario by replacing the joinChannel() function and creating a "token" variable ( var token = "";) on top of the file.
We will be happy to know how to resolve that:)

I think you're calling the join method wrong. This is the function signature:
join(appid: string, channel: string, token: string | null, uid?: UID): Promise<UID>

The first parameter should be your Agora App ID

I think you're calling the join method wrong (assuming you're using v4 of the web SDK). This is the function signature:
join(appid: string, channel: string, token: string | null, uid?: UID): Promise<UID>

The first parameter should be your Agora App ID

Hello, thanks you for your answer.
I'am using previous version, I confirm you the join function work because I can pass a temp Token.
I finally solve the problem. Within the URL fetch request, I try to fetch the uid 44, and in the join function I request the uid 0 (mismatch)
Finally I had to put the uid 0 in my url request.
Thanks you for your help.

You're right, the UID mismatch was causing the token issues.
However, using 0 as your UIDs isn't the best approach as you wouldn't be able to uniquely identify users joining your channel. This can be a potential issue if you want to kick/ban users or revoke access from your backend later. Since 0 is used as a "wildcard" token - a user with a token for UID 0 can rejoin the channel after their old UID is banned (as the SDK will automatically assign a new UID when using 0 as the UID).
I'd recommend using some part of the user data as the UID. For example an integer based primary key, if that's an option.


error with cppad

Hi!, when I started the MPC_Node setting target point, the car could not start, and reported an error. I reinstalled ipopt several times according to the tutorial and it did not work. Do you know how to solve this problem:
cppad-20160000.1 error from a known source:
yq = f.Forward(q, xq): a zero order Taylor coefficient is nan.
Corresponding independent variables vector was written to binary a file.
vector_size = 158
file_name = /tmp/filer6SLLw
index = 82
Error detected by false result for
if( CppAD::isnan( yq[ (q+1) * index + 0 ] )
at line 224 in the file
/usr/include/cppad/local/forward.hpp
MPC_Node: /usr/include/cppad/utility/error_handler.hpp:207: static void CppAD::ErrorHandler::Default(bool, in
t, const char*, const char*, const char*): Assertion `false' failed.
[MPC_Node-6] process has died [pid 13466, exit code -6, cmd /home/sz/racecar/devel/lib/art_racecar/MPC_Node /
move_base_node/NavfnROS/plan:=/move_base/NavfnROS/plan __name:=MPC_Node __log:=/home/sz/.ros/log/78ff706a-a63
9-11e9-8c68-00e15c683339/MPC_Node-6.log].
log file: /home/sz/.ros/log/78ff706a-a639-11e9-8c68-00e15c683339/MPC_Node-6*.log

Hi @001SCH ,
I think for the issue of cppad, you should ask "COIN-OR" directly.
For testing, if you can not compile sucessfully on your environment, I suggest you use our released VirtualBox image first.

@001SCH I also encountered this issue when in my own MPC work. Have you figured out the fix to this  bug? I would appreciate so much on your help for this!

@haochihlin  @001SCH  @mustang66ytz
Hello, everyone~
Do you have a solution now?     Thank you

@haochihlin  @001SCH  @mustang66ytz
Hello, everyone~
Do you have a solution now?     Thank you

@Chuan701 @mustang66ytz @haochihlin @001SCH any updates on the issue? Have the same problem


Prevent tooltip from going offscreen

Description

When the cursor hovered over dates near the edge, the tooltip tended to go offscreen.  I fixed that by putting the tooltip to the left of the cursor when the cursor was too close to the right edge.
Motivation and Context

I could not read the whole tooltip because it was cut off.  This fix solves the issue of the tooltip going offscreen.
How Has This Been Tested?

I hovered over multiple days across both sides of the heatmap to check that the tooltip showed properly.  I tested it on chrome on gentoo linux as well as on windows with chrome and edge. The only thing my code affects is how the tooltip is shown.  I used various window sizes while testing it.
Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] Code maintainance (non-breaking change that makes existing code better)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have updated the changelog accordingly.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

Thanks! It works flawlessly now.


RFC0593: suggest we make options a JSONLD object such that it can be extended

The options object of the<EMAIL_ADDRESS>format is a plain old JSON object. It would be better if this object is actually a JSONLD document with @context and type such that it can be extended by different implementations. Of course, the definition of a core Aries JSONLD options type would still be useful and part of the JSONLD profile subtarget for interop. Making options a JSONLD document would also align perfectly with the data formats involved in this RFC.
The primary motivation for this is we would like enable the use of P curves with this attachment format, and we'd like to use JsonWebSignature2020 for it.
Note: specifying the verification method's type (JsonWebKey2020) is insufficient as the underlying key could be anything. We'd like to be able to specify the exact P curve to use (P256, P384, P512).
cc @TimoGlastra

I agree. As we're already in JSON-LD land it makes sense to utilize its features
How is DIF PE specify the key type? Seems like it is also missing this. Or is it expected that you support all underlying key types if you support the proof type?
Maybe we should have taken the same route as the VC HTTP API where you specify the verificationMethod instead of the proofType. Which is sufficient to determine the key type.


certs: Sync with Mozilla bundle as of Jul 19, 2022

https://github.com/bagder/ca-bundle/commit/7f33e7eb8472dbcf31fdcf50cd216c89a282825d

Cherry-picked for 3.5.


How to embed client info in Apollo Client call, which can further be tracked on Apollo Studio

We're using Maven based Java project, and using Apollo Client 2.5.9 to perform query/mutations.
After performing operations, on the studio in the Clients tab it says Unidentified clients for the calls originating from Apollo Client.
See below:

Got this link from @jesse-apollo https://www.apollographql.com/docs/android/advanced/client-awareness/ and was able to make things work.


unable to properly pipe commands with conf.d/__fasd_run.fish

when __fasd_run.fish is in effect, piping from aws resulted in the command being piped not the content
debian@debian10 ~/.c/f/conf.d> aws --version > /t/c
debian@debian10 ~/.c/f/conf.d> cat /t/c
aws --version
debian@debian10 ~/.c/f/conf.d> file (which aws)
/home/debian/.local/bin/aws: Python script, ASCII text executable

it works fine after I remove the file.
debian@debian10 ~/.c/fish> rm conf.d/__fasd_run.fish
debian@debian10 ~/.c/fish> fish
Welcome to fish, the friendly interactive shell
debian@debian10 ~/.c/fish> aws --version > /t/c
debian@debian10 ~/.c/fish> cat /t/c
aws-cli/1.18.211 Python/3.7.3 Linux/4.19.0-13-amd64 botocore/1.19.51
debian@debian10 ~/.c/fish>

I'm not sure if this is a problem with my configuration or something.
I've a hunch that it might be related to the fact that aws is a python script.
Anyway, I'm running in tmux under byobu in debian10 in a virtual machine with vmplayer.
it's a relatively new machine.
debian@debian10 ~/.c/fish> fish --version
fish, version 3.0.2

I took a look but I am not sure how to reproduce this - I am not sure what__fasd_run.fish is, and my aws is not a Python script. Can you please provide steps to reproduce, or narrow it down on your end?

I took a look but I am not sure how to reproduce this - I am not sure what__fasd_run.fish is, and my aws is not a Python script. Can you please provide steps to reproduce, or narrow it down on your end?

I am not sure how to reproduce this. Does type python or type aws change when __fasd_run.fish is active?

I am not sure how to reproduce this. Does type python or type aws change when __fasd_run.fish is active?

nope. they do not change.
# when __fasd_run.fish is not active
debian@debian10 ~> type aws
aws is /home/debian/.local/bin/aws
debian@debian10 ~> type python
python is /usr/bin/python
debian@debian10 ~> aws --version > /t/c
debian@debian10 ~> cat /t/c
aws-cli/1.18.211 Python/3.7.3 Linux/4.19.0-13-amd64 botocore/1.19.51

# when __fasd_run.fish is active
debian@debian10 ~> type aws
aws is /home/debian/.local/bin/aws
debian@debian10 ~> type python
python is /usr/bin/python
debian@debian10 ~> aws --version > /t/c
debian@debian10 ~> cat /t/c
aws --version
debian@debian10 ~>

I realise that it's nothing to do with aws or python apparently; pwd have the same behaviour.
for this test I'm not using tmux.
# when __fasd_run.fish is active
debian@debian10 ~> pwd > /t/c
debian@debian10 ~> cat /t/c
pwd
debian@debian10 ~> pwd
/home/debian
debian@debian10 ~> exit
debian@debian10:~$ rm .config/fish/conf.d/
aliases.fish     __fasd_run.fish  fzf.fish
debian@debian10:~$ rm .config/fish/conf.d/
aliases.fish     __fasd_run.fish  fzf.fish
debian@debian10:~$ rm .config/fish/conf.d/__fasd_run.fish
debian@debian10:~$ fish
Welcome to fish, the friendly interactive shell
debian@debian10 ~> pwd > /t/c
debian@debian10 ~> cat /t/c
/home/debian
debian@debian10 ~>

nope. they do not change.
# when __fasd_run.fish is not active
debian@debian10 ~> type aws
aws is /home/debian/.local/bin/aws
debian@debian10 ~> type python
python is /usr/bin/python
debian@debian10 ~> aws --version > /t/c
debian@debian10 ~> cat /t/c
aws-cli/1.18.211 Python/3.7.3 Linux/4.19.0-13-amd64 botocore/1.19.51

# when __fasd_run.fish is active
debian@debian10 ~> type aws
aws is /home/debian/.local/bin/aws
debian@debian10 ~> type python
python is /usr/bin/python
debian@debian10 ~> aws --version > /t/c
debian@debian10 ~> cat /t/c
aws --version
debian@debian10 ~>

I realise that it's nothing to do with aws or python apparently; pwd have the same behaviour.
for this test I'm not using tmux.
# when __fasd_run.fish is active
debian@debian10 ~> pwd > /t/c
debian@debian10 ~> cat /t/c
pwd
debian@debian10 ~> pwd
/home/debian
debian@debian10 ~> exit
debian@debian10:~$ rm .config/fish/conf.d/
aliases.fish     __fasd_run.fish  fzf.fish
debian@debian10:~$ rm .config/fish/conf.d/
aliases.fish     __fasd_run.fish  fzf.fish
debian@debian10:~$ rm .config/fish/conf.d/__fasd_run.fish
debian@debian10:~$ fish
Welcome to fish, the friendly interactive shell
debian@debian10 ~> pwd > /t/c
debian@debian10 ~> cat /t/c
/home/debian
debian@debian10 ~>

hey, it look like this issue is the same / related to #17

hey, it look like this issue is the same / related to #17

since I would imagine a lot of complaints if this is a generic problem. :

you are welcome to provide a PR.

since I would imagine a lot of complaints if this is a generic problem. :

you are welcome to provide a PR.

I've tested  #20 and it does resolve the issue I had.
Thanks!

@pekcheey thank you, did merge.


Dictionary keys in fromt matter having blanks: How to access?

Let’s assume someone has:
---
start time: 2021-04-24T14:00
end time: 2021-04-24T15:30
---

in her front matter and wishes to use these values with dataview:
`= this.start time`

obviously doesn’t work, so how to access keys from YAML that have blanks?

= this["start time"]
I will add field name canonicalization so you can also access using the "cleaned up" form start-time.


feat: handle watermark barrier for map vertex

TODO

I know that pr is WIP, was just understanding the code, so left a few comments.


💡 [FEAT] remove Menu + icon and Todos+Icon.

💡 [FEAT] remove Menu + icon and Todos+Icon.
In my humble opinion it is clear as ice what the left and right side are supposed to be.
So that would free screen estate for the list and topics itself.

Yes ig you may be right... That was just for aesthetics.
I will push an update in which you can set to show them or not in the config file


User interface unresponsive

Strapi version: 4.14.3
Publisher plugin version: 1.5.0
Nodejs version: 18.18
The UI for selecting a (un)publish date and time is very slow. Also after setting a publish and/or an unpublish date the entire UI becomes very very slow. It seems this plugin is intercepting all kinds of interaction events causing the entire interface to slow down.

It's possible some update to the design system has caused an issue again. This plugin does not listen for any interaction evens that should cause the UI to slow down.
The datepicker is from the strapi admin UI so performance is somewhat dependant on them.
What version of the strapi design system are you on? I will see if I can reproduce this issue.

I have tried to isolate the issue. I have created a new Strapi project with npx create-strapi-app@latest my-project --quickstart. Version 4.14.4.
I have added the publisher plugin. Version 1.5.0.
The lag is there. @strapi/ui-primitives is at version 1.12.2.

Something is up with the Strapi DatePicker. If I increase the step size the performance issue is decreased. I will try to investigate further today or tomorrow.

Yes, the more items you lad in for the time selection the worse the performance will be. I believe the actual time picker component from the design system has this issue itself.
Generally it's recommended to increase the skip to 30 or higher. You will see a performance increase if you do.
The default of one was to prevent a breaking change when this setting was introduced.

Yes. I will follow this up in the Strapi project. Thank you for your quick communication.

This is related to https://github.com/strapi/design-system/issues/1170.


chore(main): release 0.9.1

:robot: I have created a release beep boop
0.9.1 (2024-11-26)
Bug Fixes

ProportionalCAN: add set_state return value (b43c14b)
ProportionalCAN: change default in get_by_id (87a4071)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.9.1
:sunflower:


[WIP] Editor

Added Scrumpys TipTap editor which is based on Prosemirror.

Needs backend on branch "editor"
Related with https://github.com/Human-Connection/Nitro-Backend/pull/136

Todo`s:

[x] edit posts
[ ] create posts

[x] title
[x] content
[ ] tags
[ ] categories

[ ] fix broken integration test as now you cant report your own contributions or your self as a user
[x] always add target="_blank" to the rendered links
[ ] render all content with the editor-content component
[ ] Stylings

[x] bold
[x] italic
[ ] strike

[ ] Elements

[x] links

Embeds are comming later

@appinteractive can you fix CodeFactor please? We can merge afterwards

@appinteractive can you fix CodeFactor please? We can merge afterwards

@roschaefer I disabled it, will not be shown on next build. So merging that branch?


Fix OpenVPN makefile so that it properly works on Windows

subj.

@oleg-nenashev  Can you correctly rebase on staging, and merge to staging

It was done in another PR
On Wed, Nov 13, 2019, 08:36 Olblak<EMAIL_ADDRESS>wrote:

@oleg-nenashev https://github.com/oleg-nenashev Can you correctly
rebase on staging, and merge to staging
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jenkins-infra/openvpn/pull/51?email_source=notifications&email_token=AAW4RIAIOW2DUTNEQLJFAUTQTOU7RA5CNFSM4JME6IRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED5FRVY#issuecomment-553277655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW4RIE72FYN6OTUIPOHHN3QTOU7RANCNFSM4JME6IRA
.

Oh, will fix. Maybe you should just make staging a default branch
On Wed, Nov 13, 2019, 08:41 Oleg Nenashev<EMAIL_ADDRESS>wrote:

It was done in another PR
On Wed, Nov 13, 2019, 08:36 Olblak<EMAIL_ADDRESS>wrote:

@oleg-nenashev https://github.com/oleg-nenashev Can you correctly
rebase on staging, and merge to staging
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jenkins-infra/openvpn/pull/51?email_source=notifications&email_token=AAW4RIAIOW2DUTNEQLJFAUTQTOU7RA5CNFSM4JME6IRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED5FRVY#issuecomment-553277655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW4RIE72FYN6OTUIPOHHN3QTOU7RANCNFSM4JME6IRA
.

Oh, will fix. Maybe you should just make staging a default branch

Very good suggestion, I just changed it staging


[WIP] Investigation around unexplained timeouts in Elasticsearch tests

Don't merge this please, I just want to trigger CI builds.

Jenkins, retest this please

Jenkins, retest this please

Jenkins, retest this please

Superseded by #1348


The CONTRIBUTING.md needed to be updated.

I'm not very familiar with what are the things needed while sharing a Facebook Spark AR Project. If anyone who has worked with this. Kindly share your thought, even better raise a PR updating the CONTRIBUTING.md.

hi @abhisekmazumdar I am willing to work on this issue, here are the things I think should a spark are project have :-

project itself
trial link
preferably some screenshots/gifs showcasing their ar filter.

Yes sure. Raise a PR our team will review it.


how to create OAuth Authorization Server

e.g.
var OAuthServerOptions = new OAuthAuthorizationServerOptions
{
AllowInsecureHttp = true,
TokenEndpointPath = new PathString("/token"),
AccessTokenExpireTimeSpan = TimeSpan.FromMinutes(30),
Provider = container.Resolve(),
RefreshTokenProvider = container.Resolve()
};
// Token Generation
app.UseOAuthAuthorizationServer(OAuthServerOptions);
app.UseOAuthBearerAuthentication(new OAuthBearerAuthenticationOptions());
how to do this in asp.net 5 (vnext).
references:
http://www.asp.net/aspnet/overview/owin-and-katana/owin-oauth-20-authorization-server
http://bitoftech.net/2014/06/01/token-based-authentication-asp-net-web-api-2-owin-asp-net-identity/
http://aspnetguru.com/adding-authentication-to-asp-net-web-api/
http://blogs.msdn.com/b/webdev/archive/2013/09/20/understanding-security-features-in-spa-template.aspx
http://odetocode.com/blogs/scott/archive/2015/01/15/using-json-web-tokens-with-katana-and-webapi.aspx

You can't, the OAuth2 authorization server has not been (and won't be) ported to ASP.NET 5: http://stackoverflow.com/questions/29055477/oauth-authorization-service-in-asp-net-mvc-6

IdentityServer3 works fine on ASP.NET 5 (but not on the CoreClr)
https://github.com/IdentityServer/IdentityServer3.Samples/tree/master/source/AspNet5Host

@leastprivilege This line of code is busted:
https://github.com/IdentityServer/IdentityServer3.Samples/blob/master/source/AspNet5Host/src/IdentityServer3/Startup.cs#L21
It should be using IApplicationEnvironment

thanks! Will update.

For anyone still looking for the original OAuth Authorization Server in ASP.NET 5, I have ported the code and the original sample here:
https://github.com/XacronDevelopment/oauth-aspnet
The port includes backwards compatibility to allow ASP.NET 4.x resource servers to read the access tokens created by the authorization server.
The nuget packages are here:
https://www.nuget.org/packages/OAuth.AspNet.AuthServer
https://www.nuget.org/packages/OAuth.AspNet.Tokens
https://www.nuget.org/packages/OAuth.Owin.Tokens

@Xacron thanks, will try.


Don't add admins to projects they create

When admins create projects, they're usually not doing it for themselves, so we shouldn't add them to the project.
Bonus:

Perhaps we should add them if they create a project from their "My projects" page?
Perhaps we should make it visible and/or configurable on the "Create project" page who will be added?

I like the idea of making it visible on the create project page (only to admins), but I'm less sure about the create project page acting differently based on where they came from, it might be confusing.

As discussed, we're going to:

Never add admins to a project that they create (so projects they create can initially be empty
automatically add the requesting user as a project manager (and display that on the create page)
nothing else!

Why?

Because that likely covers 99% of use cases
Ultimately we'd like to change how requesting a project works. We'd like requested projects to be visible to all admins in the UI (e.g. as draft projects). That's a pretty big amount of work and not super high priority, but we don't want to invest more effort than necessary in a different direction.


Fix bugs with isSameContact and editc command

Codecov Report

Merging #201 (f6bbb65) into master (5ea5584) will increase coverage by 1.75%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master     #201      +/-   ##
============================================
+ Coverage     57.58%   59.34%   +1.75%
- Complexity      569      599      +30
============================================
  Files           113      113
  Lines          2181     2189       +8
  Branches        259      260       +1
============================================
+ Hits           1256     1299      +43
+ Misses          832      794      -38
- Partials         93       96       +3

Impacted Files
Coverage Δ

...eedu/address/logic/commands/AddContactCommand.java
100.00% <ø> (ø)

...address/logic/parser/EditMeetingCommandParser.java
0.00% <0.00%> (ø)

...edu/address/logic/commands/EditContactCommand.java
94.52% <75.00%> (-2.59%)
:arrow_down:

...main/java/seedu/address/model/contact/Contact.java
97.87% <100.00%> (+0.19%)
:arrow_up:

...address/logic/parser/FindMeetingCommandParser.java
35.71% <0.00%> (+35.71%)
:arrow_up:

...edu/address/logic/commands/FindMeetingCommand.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...s/model/meeting/MeetingOccursOnDatesPredicate.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...model/meeting/MeetingNameHasKeywordsPredicate.java
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Audit app/views/layouts/application.html.erb for bootstrap and CSS

Audit the view in the title for bootstrap grid implementation.
Views should use the bootstrap grid system for display.
Resources

Bootstrap Grid System
Here's the Difference Between Flexbox, CSS Grid & Bootstrap

Acceptance criteria

[ ] Implement the bootstrap 12-column grid for top-level panels in the view
[ ] Use flex for all layers under the top-level panels
[ ] Review the CSS assigned to the elements in the view; remove CSS that is no longer used, and restyle CSS still in use to use flexbox appropriately
[ ] Include a screenshot of the view in the browser as part of the PR

This one should be done before the other view audit tickets as it will impact them all.  I am setting it as "high priority" in Zenhub to indicate this.


double check if things exist

prevent the case where a datasource ends up with dimensions and measures of the same name

💧


Error generating ZoneId on Java 21

Describe the bug
Instancio.create(ZoneId.class) generates one of the subclasses, either ZoneOffset or ZoneRegion. ZoneOffset is generated correctly. ZoneRegion was also generating correctly in version 4.7.0 , but in version 4.8.0 invalid ZoneRegion is generated.
Expected result
Valid non-null ZoneRegion
Actual result
Invalid ZoneRegion with null id and rules fields
Additional information
It may require a few tries to reproduce as Instancio can generate either ZoneOffset or ZoneRegion implementation.
Link to a minimal sample project reproducing the issue or runnable sample code
Mini-project reproducing the issue https://github.com/bulatnig/instabug/blob/main/lib/src/test/java/org/example/InstancioTest.java
package org.example;

import org.instancio.Instancio;
import org.junit.jupiter.api.Test;

import java.time.ZoneId;

import static org.junit.jupiter.api.Assertions.assertNotNull;

class InstancioTest {
    @Test
    void zoneIdTest() {
        ZoneId zoneId = Instancio.create(ZoneId.class);
        assertNotNull(zoneId.getId());
        assertNotNull(zoneId.getRules());
    }
}

Java version: 21.0.3 (Eclipse Adoptium 21.0.3+9-LTS) , reproduced on Amazon Corretto 21.0.3 as well
Instancio version: 4.8.0

@bulatnig thank you for reporting this bug and the sample project! I was able to reproduce quickly using the sample. I released 4.8.1 with the fix.


fix svc names created by cp-helm-charts chart

Eh, I missed that you've added -kafka in your install name.  I'd rather change that then too... having kafka appear twice in the name is a bit obnoxious.

Can we hold off on this until this change to master until we actually switch over to your terraform-based deployment? Angelo and I are working on a cluster based on this original set up, so it doesn't make sense to change our docs from under us before we change the cluster itself.


No Docker Image

There is no docker image built for this. I will work off of the current codebase and try to create one. If I haven't posted back in a few days consider that I failed :)

Was about to add this - dockerise!
I second this!

#What I have so far
#make a folder for the docker bits
mkdir youtube-dl-front
cd youtube-dl-front
#pull a nice premade make script
wget  https://raw.githubusercontent.com/mvanholsteijn/docker-makefile/master/Makefile
wget  https://raw.githubusercontent.com/mvanholsteijn/docker-makefile/master/.make-release-support
#make our Dockerfile
echo < Dockerfile
FROM alpine
WORKDIR /
#copy run script
COPY run.sh /run.sh
#install dependencies
ENV APK_ADD="nodejs npm git bash"
RUN apk add --no-cache ${APK_ADD}
RUN git clone https://github.com/Enriquito/youtube-dl-front
WORKDIR /youtube-dl-front/server
RUN npm install
#setup ports and volumes (volume tbd), startup script
EXPOSE 3000
VOLUME ["/youtube-dl-front"]
ENTRYPOINT [ "/run.sh" ]
EOF
#Make our run.sh script
echo < run.sh
#!/bin/bash
cd /youtube-dl-front/server
node app.js
EOF
#now run make
make
It should build the container.  Still testing, as i need to setup mapped folder for downloads.

Thanks for looking into this! I dont have that many experiance with docker. I will test it out soon.

Not having fun(tm) yet -
cd /youtube-dl-front/server
node app.js
Is failing with -
internal/modules/cjs/loader.js:968
throw err;
^
Error: Cannot find module '/youtube-dl-front/server/app.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:965:15)
at Function.Module._load (internal/modules/cjs/loader.js:841:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}

Did you install all the npm modules? like this? Then you can copy the node_modules folder.
cd /youtube-dl-front/server
npm install

Ok, seems to work, although I need to ideally map database.json
alternately if you can stick config files into a separate folder, and I can map that at runtime?
I'm assuming you'll make the docker files in your account, the above is enough to get started.  you'll need to setup an account on docker.io then publish.
This is the script i used - has some instructions.
https://binx.io/blog/2017/10/07/makefile-for-docker-images/
Let me know if this makes sense or not.

For now, I'm manually creating a database.json file in my /opt/youtube-dl-front/database.json
with this content
database.json
{"videos":[]}
and mapping to /youtube-dl-front/server/database.json
With:
youtube-dl-front:
image: root/ydf
container_name: ydf
ports:
- 3000:3000
volumes:
- /opt/youtube-dl-front/videos:/youtube-dl-front/server/videos
- /opt/youtube-dl-front/music:/youtube-dl-front/server/music
- /opt/youtube-dl-front/database.json:/youtube-dl-front/server/database.json:rw
labels:
- "traefik.http.routers.ydf.rule=Host(ydf.DOMAIN)"
- "traefik.http.routers.ydf.middlewares=test-auth@docker"
networks:
- public
Anyone have a better suggestion how to do this?
(Or @Enriquito - if you can move config files to a separate folder under server/config ?  then we can map that folder in docker more easily)

@csloz Many thanks so far! You have been a great help.
The config file can me moved that should not be a problem.
I will look into this tomorrow
cheers.

Thanks :)
What needs to be done:
change config to use a separate folder.
ideally create database.json at runtime if not exist, and stick in that folder.
Then.
You register an account at dockerhub.
you setup make, Dockerfile and run.sh as i've noted above, and create the docker container on dockerhub, and push to dockerhub.
Then put the Dockerfile, run.sh, and a sample (working) docker.yml onto your git folder, so people know how to use it!
Will also need to look at automatic docker builds with travis, but for now that should be a good start.

I've never dockerised anything before so I just got a container that was somewhat functional before I decided to check out this thread. I'm definitely going to check this out and try to add some things.
Have you thought about building this off the node:current-alpine container? It adds a few extra things installed by default, might make it easier in the future.

@csloz I merged the docker files. And will try to push the docker image tomorrow!

Hi @csloz, @TheBros35 ,
So i manged to upload it to docker hub. Could you guys test it?
https://hub.docker.com/repository/docker/enriquek/youtube-dl-front
Cheers!

I need to make a few changes to files.  Can you add me as a contributor?  then i can pull current stuff and push, Or I can email you changes later today?

@csloz

I need to make a few changes to files. Can you add me as a contributor? then i can pull current stuff and push, Or I can email you changes later today?

Were you able to push the changes?

Docker image is ready.


About the original

Can you run the original code to the accuracy of the original?
Why do I always reach the precision of baseline

@xieyuhaoli sorry for a late reply. I remember I had a similar issue; I eventually solved it but maybe forgot to push into upstream. Anyone who fixes this, please make a PR.


Floating element doesn't reposition when reference is repositioned on page

So looking through the code it seems like a floating element will only repositions with autoUpdate when:

a scrolling/overflowing ancestor is scrolled or resized
when the reference element itself is is resized

In the video below none of those are true. Things around the reference that aren't scrollable or boundaries in any way get resized, which repositions the reference, but does not trigger the floating element to update. I could use animationFrame but that seems really heavy for what seems like it should be default behavior
https://user-images.githubusercontent.com/1192452/195693045-2227865e-5e65-4015-a20d-56a0898d5c28.mp4

Figured out a way to fix it on my side. Whenever something inside the grid is resized I fire:
 window.dispatchEvent(new Event('resize'));

Which triggers update functions

The platform doesn't yet provide a way to know this scenario in an efficient way, you'd have to attach a bunch of ResizeObservers to all parent containers.
For transient elements that are open only for a short time, animationFrame: true is fine perf-wise, unless there'll be tons of them open.

Figured out a way to fix it on my side. Whenever something inside the grid is resized I fire:

That's the same as calling the update function yourself manually when the user is resizing:
autoUpdate(reference, floating, update);

const onPaneResize = () => {
  update();
};

True. Not directly using the library though so I can't directly access that function

Thanks!


squid animation not working

Fixed as of 5a2e5f02f1f9e5e40cca78954425aeb784731153, though the tentacles might not work (since Minecraft thinks, the player is moving wrongly, which isn't fixable in an instant).

I can confirm the tenticles aren't working


bug: "value is unknown" after update to v1.20.0+

Describe the bug
After updating from tfsec v0.19.1 to v1.20.0 or higher, a "value is unknown" error is happening.
To Reproduce
Steps to reproduce the behavior:

Run tfsec on our Terraform project

Expected behavior
Run without an issue like tfsec v0.19.1
Screenshots/Output
[debug:eval][root] Finished processing 299 submodule(s).
[debug:parse][root] Finished parsing module 'root'.
panic: value is unknown

goroutine 163 [running]:
github.com/zclconf/go-cty/cty.Value.AsString({{{0x13d3c48, 0x400012a0ea}}, {0xec0f40, 0x1ef42a0}})
	/home/runner/go/pkg/mod/github.com/zclconf/go-cty@v1.10.0/cty/value_ops.go:1262 +0x150
github.com/aquasecurity/defsec/pkg/terraform.(*Reference).KeyBracketed(0x400176a5a0)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/terraform/reference.go:147 +0xe4
github.com/aquasecurity/defsec/pkg/terraform.(*Reference).String(0x400176a5a0)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/terraform/reference.go:110 +0x198
github.com/aquasecurity/defsec/internal/types.(*Metadata).ToRego(0x40095e0cc8)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/internal/types/metadata.go:27 +0x6c
github.com/aquasecurity/defsec/pkg/rego/convert.StructToRego({0x103ef20, 0x400b955080, 0x99})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/struct.go:55 +0x620
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0x103ef20, 0x400b955080, 0x99})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:34 +0x3c0
github.com/aquasecurity/defsec/pkg/rego/convert.StructToRego({0x1026020, 0x4010cc8718, 0x199})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/struct.go:41 +0x3d0
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0x1026020, 0x4010cc8718, 0x199})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:34 +0x3c0
github.com/aquasecurity/defsec/pkg/rego/convert.SliceToRego({0xe593c0, 0x400b92fc98, 0x97})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/slice.go:28 +0x2e8
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0xe593c0, 0x400b92fc98, 0x97})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:36 +0x40c
github.com/aquasecurity/defsec/pkg/rego/convert.StructToRego({0x102f620, 0x400ddaff70, 0x199})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/struct.go:41 +0x3d0
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0x102f620, 0x400ddaff70, 0x199})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:34 +0x3c0
github.com/aquasecurity/defsec/pkg/rego/convert.SliceToRego({0xe59380, 0x4006b9ecf8, 0x97})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/slice.go:28 +0x2e8
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0xe59380, 0x4006b9ecf8, 0x97})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:36 +0x40c
github.com/aquasecurity/defsec/pkg/rego/convert.StructToRego({0xf6ce40, 0x4006b9ece0, 0x99})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/struct.go:41 +0x3d0
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0xf6ce40, 0x4006b9ece0, 0x99})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:34 +0x3c0
github.com/aquasecurity/defsec/pkg/rego/convert.StructToRego({0x105e920, 0x4006b9e700, 0x99})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/struct.go:41 +0x3d0
github.com/aquasecurity/defsec/pkg/rego/convert.anonymousToRego({0x105e920, 0x4006b9e700, 0x99})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/anonymous.go:34 +0x3c0
github.com/aquasecurity/defsec/pkg/rego/convert.StructToRego({0xebbba0, 0x4001a91c00, 0x16})
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/rego/convert/struct.go:41 +0x3d0
github.com/aquasecurity/defsec/pkg/state.(*State).ToRego(0x4001a91c00)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/state/state.go:31 +0xdc
github.com/aquasecurity/defsec/pkg/scanners/terraform/executor.(*regoJob).Run(0x400e47d8e0)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/scanners/terraform/executor/pool.go:162 +0x3c
github.com/aquasecurity/defsec/pkg/scanners/terraform/executor.(*Worker).Start.func1(0x40009b2f98, 0x40048d52c0)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/scanners/terraform/executor/pool.go:281 +0x34
github.com/aquasecurity/defsec/pkg/scanners/terraform/executor.(*Worker).Start(0x40048d52c0)
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/scanners/terraform/executor/pool.go:286 +0xac
created by github.com/aquasecurity/defsec/pkg/scanners/terraform/executor.(*Pool).Run
	/home/runner/go/pkg/mod/github.com/aquasecurity/defsec@v0.53.2/pkg/scanners/terraform/executor/pool.go:55 +0xc4

System Info

tfsec version: v1.20.2
terraform version: v1.1.9
OS: macOS 12 & Docker Container

Example Code
Code base is quite big; if we can figure out which module is the source of problem, I can share the code for sure.
Additional context

I'm having this issue as well. Here is a minimal example needed to reproduce. It appears this is related to handling the variables in the strings when also using toset. Removing the lines with ${variable} in them causes the issue to go away in the first example.

Make a new directory and cd to it
Paste this in as main.tf
run tfsec .

locals {
  gcr-push = [
    "serviceAccount:${data.terraform_remote_state.foo.outputs.deploy-gsa}",
    "serviceAccount:${data.terraform_remote_state.foo.outputs.general-gsa}",
<EMAIL_ADDRESS><EMAIL_ADDRESS>  ]

  gcr-pull = [
    "serviceAccount:${module.foo.cluster-sa}",
<EMAIL_ADDRESS><EMAIL_ADDRESS>  ]
}

resource "google_project_iam_member" "gcr-push" {
  for_each = toset(local.gcr-push)
  member   = each.key
  role     = "roles/storage.admin"
  project  = var.gcr_project_id
}

resource "google_project_iam_member" "gcr-pull" {
  for_each = toset(local.gcr-pull)
  member   = each.key
  role     = "roles/storage.objectViewer"
  project  = var.gcr_project_id
}

Also, testing a single string with a variable in it without toset does not cause an error.
resource "google_project_iam_member" "gcr-push" {
  member   = "serviceAccount:${data.terraform_remote_state.foo.outputs.deploy-gsa}"
  role     = "roles/storage.admin"
  project  = var.gcr_project_id
}```

#1731 similar symptom, though in our case, seems to be a slightly different cause.


About the quality of the paper

For relatively new conference papers like ICRL2020, are you sure about the quality? I think there are a lot of articles recently added. After all, the title is a must-read thesis. I hope to control it and only keep the essence.thanks

Thanks for your suggestion. We will try to control the quality of the papers.


Run network-related deployment tests in travis

Might as well; we have the mock drivers we need.
//cc @henn, @knikolla, @shwsun

@zenhack I like it a lot! I think it is great that we can run tests on mock switches, and in the long run we should do this for all of our mock drivers.
@henn @SahilTikale @knikolla Anyone interested in giving one more review?

@henn, I'm assigning you as a reviewer. This PR isn't terribly complex, but it's open for almost two months. Please have a look.

@shwsun I'm interpreting your comment as being a +1, and now that @knikolla has approved this, we ought to be able to merge. Am I interpreting that correctly?


feat(tokenfactory): execute msgs for CreateDenom, ChangeAdmin, UpdateModuleParams

Includes queries, transaction messages, and CLI commands for CreateDenom, ChangeAdmin, and UpdateModuleParams
Part of #1584

I'm encountering something strange here where it says the perp protos are breaking and I'm not even editing those files.


Update to 1.21.x (1.21.1-1.21.3)

Hi!
Can you please update this mod to latest version? No new functions, just keep the mod on the line.
I'm Ukrainian too, so I can understand if you do not have much time to maintain. Still, would GREATLY appreciate an update because there are effectively no mods with similar features at the moment (talking about those pesky channels feature).
Have a good day! And thank you

Thanks a lot for the kind words, and I hope you're doing ok!
As for the mod itself, I'm slowly getting back into things. So, I do have plans to properly update this mod, along with several others, and perhaps even port it to NeoForge and whatnot :)


[AutoTVM] Support to pass additional compiler options

Currently, AutoTVM doesn't support to pass additional compiler options to builder. It works fine on most cases. However, if we use NDK CC, some times we want to specify additional compiler options. For example, another sysroot.
After this PR, the user could do like this:
os.environ['TVM_NDK_CC'] = /path/to/ndk_cc
options=['--sysroot=/path/to/sysroot', '-shared', '-fPIC']

tuning_option = {
    // ...
    'measure_option': autotvm.measure_option(
        builder=autotvm.LocalBuilder(
            build_func='ndk' if use_android else 'default', options=options),
       ...
    ),
}

@merrymercy @tqchen @ajtulloch would you like to help review it? Thanks.

We should not put options in MeasureResult. It is not result (output). It is input.
If you change MeasureInput / MeasureResult, you should also update serialization for them in python/tvm/autotvm/record.py::encode, decode. You should be very careful about the backward-compatibility.
If you add options to records, why don't you also add build_func?

Currently, I think not serializing them is fine. Or we can embed options into MeasureInput.task and serialize them carefully.

Likely you can create a build function(via functor that captures these additional options) instead of directly passing ndk

Try this:
from tvm.contrib import cc, ndk

build_func = cc.cross_compiler(
   ndk.create_shared, options)

Try something along the following line:
from tvm.contrib import cc, ndk

build_func = cc.cross_compiler(ndk.create_shared, options)

tuning_option = {
    // ...
    'measure_option': autotvm.measure_option(
        builder=autotvm.LocalBuilder(
            build_func=build_func if use_android else 'default', options=options),
       ...
    ),
}

Perhaps it is a good time to document these usecases of cross_compiler(e.g. get target triple from compile_func if it has the attr)

Yes. Your way is good and don't change anything. Do you mean this ?
from tvm.contrib import cc, ndk

build_func = cc.cross_compiler(ndk.create_shared, options)

tuning_option = {
    // ...
    'measure_option': autotvm.measure_option(
        builder=autotvm.LocalBuilder(
            build_func=build_func if use_android else 'default'),
       ...
    ),
}

No options needed in the builder.

We should not put options in MeasureResult. It is not the result (output). It is input.
If you change MeasureInput / MeasureResult, you should also update serialization for them in python/tvm/autotvm/record.py::encode, decode. You should be very careful about the backward-compatibility.
If you add options to records, why don't you also add build_func?

Currently, I think not serializing them is fine. Or we can attach options into MeasureInput.task and serialize them carefully.

Yeah. I agree with you. I think this PR is not one elegant solution for this problem. The nicer way is @tqchen 's suggestion, using custom build func. I will close this PR. Thanks.


[WIP/LoadableByAddress] Fix debug info holes when expanding alloc_stack.

We should also skip debug info as they don't really are lowered
to anything real. Add an helper so that we can use it in passes
as well.
@adrian-prantl I'm dealing with a bot failure, but I wanted to put this online so you can take a look and let me know if this is fine. I'm going to add a test & run the tests.
This is the last failure we have before enabling the verifier for debug info, so my plan is to flip the switch today (crossing fingers).

@adrian-prantl now with the correct patch, after coffee.

@swift-ci please test

@adrian-prantl addressed all your comments. Please take another look.

Build failed
Swift Test Linux Platform
Git Sha - 64e8189182faa94c22d5ff1b5593370956f1546e

Build failed
Swift Test OS X Platform
Git Sha - 64e8189182faa94c22d5ff1b5593370956f1546e

Sure!

@swift-ci please test and merge

@swift-ci please test and merge


[1.12.2] Hopper chest exploit

Lostsouls-1.12-1.1.4.jar
LostCities - 1.12-2.0.10
In a haunted building you can simply place a hopper below a chest to extract it's loot.
That way you can skip clearing the building entirely. That's probably not intentional.

yes I'm aware of this. However, I see no way to avoid this

I wonder, is it at all possible to put loot in the chests AFTER the monsters have been slain, rather than when they spawn? You know how monsters track which building they came from? Maybe you could do the same for chests and populate them with loot when the last monster is slain.

That also has some problems. If players are aware of that they can just fill an entire floor with chests before killing the monsters and get TONS of loot that way

I meant attaching special NBT tags to chests that binds them to the building (like youbdo for the mobs) to ensure that only those chests get loot.
Alternatively, you could add your verybown chest item that looks exactly like a vanilla chest but cannot output items until the place is clear. But the mod currently does not add new blocks/items, so Imagine you might want to keep it that way.

I don't know if it's a general purpose solution, but I found a workaround was to use LootR, the chests don't populate till they are opened and are per-player, so things like hoppers and crafting stations don't avoid the haunted mechanic. Sorry if this doesn't help the mod creator resolve the issue.


Unable to dynamically compile Java from source.  tools.jar is missing from classpath

I have a very simple Gosu program:
var myHM = new HashMap<String, String>();
myHM.put("a", "bob");
myHM.put("b", "john");
myHM.eachKeyAndValue( \k, val -> print(k + val) )

When executing, it does print abob & bjohn; however, I also get the following printed out in my console:

!! Unable to dynamically compile Java from source.  tools.jar is missing from classpath.

What am I missing?

Fixed by adding the tools.jar dependency to the project.
Located at JDK_HOME/lib.


Node type Embedded in chrome do not work with localhost IP and cannot use webui

Describe the bug
I try to use ipfs companion with bode type Embedded in chrome, which is experimental, but I have expected that it should still work without local ipfs node and daemon, but it seem not the case. Especially the link in welcome page which specifically try to open localhost
To Reproduce
Steps to reproduce the behavior:

Install IPFS companion in chrome

Change Node Type to Embedded

Open the extension welcome page
3.1 Such as at the IPFS icon in the extension

Click on any button below to open webui

It would be opened into something like this

I could open https://webui.ipfs.io/ but then it said Could not connect to the IPFS API

Not sure this was a problem of companion itself or the webui
Expected behavior
Should the chrome companion also work as a daemon by itself? I think it should be able to detect the ipfs node of the companion and change the link of extension to be the link of the embed node, or did I was misunderstand and the embed node still require daemon anyway?
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 96.0.4664.110

Additional context
I am very new with IPFS and I still don't want to install ipfs things into my machine. I don't even create my node or upload anything yet. I wish I could run IPFS node and daemon directly inside chrome browser. And so I would leave my OS environment out of anything related ipfs. I would like to sorry in advance if I wrongly did any step I need to

Thank you for reporting these broken links – these should be hidden when embedded experiment with js-ipfs  is used.
If you are looking for a Chromium web browser with full IPFS node, give Brave a try: https://brave.com/brave-integrates-ipfs/

@lidel Thank you for your suggestion but I just want to use chrome on daily basis, I wish full IPFS node would be working the same to every browser
Hope this could be possible soon


Decrease the pageController Hieght

Its a Good Control and iam not able to customise i want to decrease the height  of the PageView Controller and I want the titles Tab but i want to show it  Half of the Screen Like Around  y Position should be from 200 i tried but now able to decrease the Frame of the Pagecontroller  can u tell me How to DO it thanks in Advance

Hi,
Currently you can't do it but as a work around please provide y position on line 180 in place of options.tabViewHeight in ViewPagerController.swift file.
I will try to incorporate all these changes and provide it in next update asap.

Hi,
This feature has been provided in support for swift 3. All relevant code has been updated in master branch. Use viewPagerPosition option from ViewPagerOptions class. Provide its y value 200.


NIOSingleStepByteToMessageDecoder reentrancy safety

Motivation:
NIOSingleStepByteToMessageDecoder calls out part way through its processing step to a user-provided closure which can cause re-entrant behavior which violates the assumption made in the code that if the buffer is non-empty at the start that will be true later in the method.
Modifications:
NIOSingleStepByteToMessageDecoder no longer assumes a non-empty buffer in its final phase of buffer management.
Further changes to protect against re-entrancy shouldn't be necessary because the outside call to messageReceiver is the only one which is permitted to be re-entrant (decode and decodeLast are not).
Result:
NIOSingleStepByteToMessageDecoder can handle re-entrant processing calls.

Resolves https://github.com/apple/swift-nio/issues/2639


Add documentation for member ordering presets and alphabetize option

PR checklist

[x] Addresses an existing issue: #4051
[x] New feature, bugfix, or enhancement

[x] Includes tests

[x] Documentation update

Overview of change:
Add a description to the member ordering rule presets, document alphabetize option
Is there anything you'd like reviewers to focus on?
Is it appropriate to link to the source/is there a better way of describing the orderings?

CHANGELOG.md entry:
[no-log]

Thanks for your interest in palantir/tslint, @grantwwu! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

I have rebased and fixed up my commits to align with the changes introduced in 85d87d1f12f9204dca957dfb8e99f74c99db8e3c.
I noticed that running npm run prettier still introduces some changes, incidentally.  @JoshuaKGoldberg is this intended?

running npm run prettier still introduces some changes

No ☹ not intended. What changes are you seeing?

Just added some additional clarification regarding the presets.
@JoshuaKGoldberg this is what I'm seeing on my branch after npm run prettier (it seems 85aa8a70f6aeedb0bfb9f3d6c78443a332d69ba7 enabled fixing for orderedImportsRule.ts but didn't commit them?):
diff --git a/README.md b/README.md
index cdc4f034..fb51ef62 100644
--- a/README.md
+++ b/README.md
@@ -65,12 +65,12 @@ yarn test

 ## Creating a new release

-1. Bump the version number in `package.json` and `src/linter.ts`
-2. Add release notes in `CHANGELOG.md`
-    - Use `./scripts/generate-changelog.js` (after building it with `tsc -p scripts`) to generate the changelog diff. This script expects a [Github.com personal access token](https://github.com/settings/tokens) to exist at `~/github_token.txt` with "repo" permissions.
-3. Commit with message `Prepare release <version>`
-4. Push your branch to GitHub and make a PR
-5. Once your PR is merged, wait for the tests to pass on CircleCI for develop
-6. Create a "Release" on GitHub with the proper tag version and notes from the changelog.
-    - The tag should be identical to the version in `package.json`
-7. Run `yarn run publish:local`
+1.  Bump the version number in `package.json` and `src/linter.ts`
+2.  Add release notes in `CHANGELOG.md`
+    -   Use `./scripts/generate-changelog.js` (after building it with `tsc -p scripts`) to generate the changelog diff. This script expects a [Github.com personal access token](https://github.com/settings/tokens) to exist at `~/github_token.txt` with "repo" permissions.
+3.  Commit with message `Prepare release <version>`
+4.  Push your branch to GitHub and make a PR
+5.  Once your PR is merged, wait for the tests to pass on CircleCI for develop
+6.  Create a "Release" on GitHub with the proper tag version and notes from the changelog.
+    -   The tag should be identical to the version in `package.json`
+7.  Run `yarn run publish:local`
diff --git a/src/rules/orderedImportsRule.ts b/src/rules/orderedImportsRule.ts
index 9e2733c1..005283bd 100644
--- a/src/rules/orderedImportsRule.ts
+++ b/src/rules/orderedImportsRule.ts
@@ -182,7 +182,8 @@ function parseOptions(ruleArguments: any[]): Options {
         "import-sources-order": sources = "case-insensitive",
         "named-imports-order": named = "case-insensitive",
         "module-source-path": path = "full",
-    } = optionSet === undefined ? {} : optionSet;
+    } =
+        optionSet === undefined ? {} : optionSet;
     return {
         groupedImports: isGrouped,
         importSourcesOrderTransform: TRANSFORMS.get(sources)!,

😕 I'm not seeing those changes locally after modifying orderedImportsRule.ts. Is your prettier version up-to-date as 1.14.3?
It probably makes sense to include them in this PR if they keep coming up.

Changes no longer appearing after rerunning yarn install:thumbsup:
By the way, npm i --package-lock-only && npm audit states
found 17 vulnerabilities (15 low, 1 high, 1 critical)
  run `npm audit fix` to fix them, or `npm audit` for details

😱


Move nbformat into services

Also upgrade to nbformat 4.1.
Uses nbformat where applicable in the kernel message interfaces.
Allows users to access the interface without depending on JupyterLab (and soon @jupyterlab/notebook).

👍 . I don't have permission to review or merge on this repo anymore, though.

👍 . I don't have permission to review or merge on this repo anymore, though.

Invitation sent!

Thanks, @jasongrout!


gen_snapshot and dart snapshot sometimes crash on ios flutter buildbot bot

from https://ci.chromium.org/ui/p/flutter/builders/prod/Mac Production Engine Drone/422092/overview logs
[1641/6217] ACTION //flutter/third_party/dart/runtime/observatory:build_observatory(//build/toolchain/mac:ios_clang_arm)
FAILED: gen/flutter/third_party/dart/runtime/observatory/observatory/web/main.dart.js
vpython3 ../../../flutter/third_party/dart/build/gn_run_binary.py compiled_action ../../../flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart --dfe=../../../flutter/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot compile js -o gen/flutter/third_party/dart/runtime/observatory/observatory/web/main.dart.js --packages=../../../flutter/third_party/dart/.dart_tool/package_config.json ../../../flutter/third_party/dart/runtime/observatory/web/main.dart --minify
Command failed: /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart --dfe=../../../flutter/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot compile js -o gen/flutter/third_party/dart/runtime/observatory/observatory/web/main.dart.js --packages=../../../flutter/third_party/dart/.dart_tool/package_config.json ../../../flutter/third_party/dart/runtime/observatory/web/main.dart --minify
output:
===== CRASH =====
si_signo=Segmentation fault: 11(11), si_code=SEGV_MAPERR(1), si_addr=0x1011c5112e0
version=3.6.0-2.0.dev (dev) (Tue Jul 2 21:02:51 2024 -0700) on "macos_x64"
pid=79469, thread=4099, isolate_group=main(0x7faa8a808200), isolate=(nil)(0x0)
os=macos, arch=x64, comp=no, sim=no
isolate_instructions=114735000, vm_instructions=1082d75a0
fp=700003156dd0, sp=700003156d90, pc=1085b15ad
  pc 0x00000001085b15ad fp 0x0000700003156dd0 /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart+0x30d5ad
  pc 0x00000001085b1c49 fp 0x0000700003156e30 /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart+0x30dc49
  pc 0x00000001085b1859 fp 0x0000700003156ea0 /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart+0x30d859
  pc 0x0000000108587927 fp 0x0000700003156f20 /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart+0x2e3927
  pc 0x0000000108587bb3 fp 0x0000700003156f50 /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart+0x2e3bb3
  pc 0x0000000108507a3e fp 0x0000700003156fb0 /Volumes/Work/s/w/ir/cache/builder/src/flutter/third_party/dart/tools/sdks/dart-sdk/bin/dart+0x263a3e
  pc 0x00007ff81976818b fp 0x0000700003156fd0 _pthread_start+0x63
  pc 0x00007ff819763ae3 fp 0x000070

Seemingly related is how gen_snapshot crashes without any diagnostic.
from https://ci.chromium.org/ui/p/flutter/builders/prod/Mac Production Engine Drone/428094/overview
logs
from https://ci.chromium.org/ui/p/flutter/builders/prod/Mac Production Engine Drone/423461/overview logs
[4823/6580] ACTION //flutter/lib/snapshot:generate_snapshot_bin(//build/toolchain/mac:ios_clang_arm_sim)
FAILED: gen/flutter/lib/snapshot/vm_isolate_snapshot.bin gen/flutter/lib/snapshot/vm_snapshot_instructions.bin gen/flutter/lib/snapshot/isolate_snapshot.bin gen/flutter/lib/snapshot/isolate_snapshot_instructions.bin
vpython3 ../../../build/gn_run_binary.py clang_arm64/gen_snapshot --snapshot_kind=core --enable_mirrors=false --vm_snapshot_data=/Volumes/Work/s/w/ir/cache/builder/src/out/ci/ios_debug_sim_arm64/gen/flutter/lib/snapshot/vm_isolate_snapshot.bin --vm_snapshot_instructions=/Volumes/Work/s/w/ir/cache/builder/src/out/ci/ios_debug_sim_arm64/gen/flutter/lib/snapshot/vm_snapshot_instructions.bin --isolate_snapshot_data=/Volumes/Work/s/w/ir/cache/builder/src/out/ci/ios_debug_sim_arm64/gen/flutter/lib/snapshot/isolate_snapshot.bin --isolate_snapshot_instructions=/Volumes/Work/s/w/ir/cache/builder/src/out/ci/ios_debug_sim_arm64/gen/flutter/lib/snapshot/isolate_snapshot_instructions.bin /Volumes/Work/s/w/ir/cache/builder/src/out/ci/ios_debug_sim_arm64/flutter_patched_sdk/platform_strong.dill

[4824/6580] ACTION //flutter/third_party/dart/runtime:generate_version_cc_file(//build/toolchain/mac:clang_x64)

cc @rmacnak-google @zanderso @a-siva

Manually symbolizing the first one:
0x30d6d0 dart::MarkingVisitorBase<true>::MarkingVisitorBase
0x30de80 dart::MarkingVisitorBase<false>::MarkingVisitorBase
0x30d930 dart::MarkingVisitorBase<true>::DrainMarkingStackWithPauseChecks
0x2e3b40 dart::ThreadPool::Worker::Main
0x2e3bf0 dart::ThreadPool::OnEnterIdleLocked
0x263ac0 dart::OSThread::CreateThreadLocal

Symbols might be a bit off, since DrainMarkingStackWithPauseChecks does not (even indirectly) call MarkingVisitorBase constructor.

The second crash mentioned in the issue has been fixed, Closing issue, if we see the crash again please reopen.


error running tests

I just cloned the repo and ran the tests, I am getting this weird error
ReplyError: Ready check failed: NOAUTH Authentication required.
Looks like that means that Redis is started up, but it needs auth. Looks like auth is provided in the form of certs and not username/password. Any idea why the certs might not be good?
Would like to get the tests running, thanks, I am on the latest version of this lib and node.

here's the actual stack trace
ReplyError: Ready check failed: NOAUTH Authentication required.
	    at JavascriptReplyParser.Parser.returnError (/Users/Olegzandr/WebstormProjects/node_redis/index.js:193:31)
	    at JavascriptReplyParser.run (/Users/Olegzandr/WebstormProjects/node_redis/node_modules/redis-parser/lib/javascript.js:135:18)
	    at JavascriptReplyParser.execute (/Users/Olegzandr/WebstormProjects/node_redis/node_modules/redis-parser/lib/javascript.js:112:10)

The stack trace is not relevant and has no connection to the calling function currently. More relevant would be what tests fail first.
Auth is always provided with a username and password and the certs are needed for tls. You might have to install stunnel to run the tests.

Ah, stunnel, will let you know if I figure it out thanks


chore: tag all releases with semver

Fixes https://github.com/dagger/dagger/issues/7760
Instead of just using commit hashes and directory IDs, which produces nearly incomprehensible versions (which is more of a pain, now that engineVersion is actually important). This makes it a bit clearer when and where a non-tagged build came from by using changie to get the last release we built.
Frustratingly, it seems non-trivial to do version comparison on these, so we keep skipping these. In the future, we should work out on compromise. For an example of why this is hard: suppose that we add a MinVersion of v0.12.0 - the next pre-release will end up being something like "v0.11.10-dev-240709191414-1ccb8772510a". So this version will fail these checks.
We could add a little piece of code to note what the next release will be - which would make these comparison a lot simpler (essentially, we'd treat pre-releases of vX as equivalent to X). But this would require too much extra maintainance for now - but maybe we could dupe this with MinVersion numbers in version.go today.

You get a semver tag!

This follows the idea in https://go.dev/doc/modules/version-numbers.

One thing I just noticed, we construct the default runner host to connect to based on Version: https://github.com/sipsma/dagger/blob/b06d45a7e0e026bbe557994314afb1f09282cf8e/engine/consts.go#L26-L26
This will continue working for tagged releases, but I think the change here would create a disconnect for CLIs published off main in that now we publish the engine image using just the git commit SHA and Version is more involved.
Getting CLIs built off main to automatically provision the correct engine image built off main by default was a pretty nice property to have, so it would be nice to maintain. It's not an utterly crucial requirement though

Getting CLIs built off main to automatically provision the correct engine image built off main by default was a pretty nice property to have, so it would be nice to maintain. It's not an utterly crucial requirement though

Oof, just missed this till now - I've split this behavior out to populate Tag separately, so we can preserve this.

Marking as draft while I rework this a bit.

So I think this is getting much closer to ready - this changes some release things, so I'll keep an eye on main once it's merged. However, the major changes are:

Every build gets semver tags
All builds cut directly off of main pretty much keep working as they do today
All dev builds will happily work with each other (as long as they share the same major/minor/patch) part of their build - we all roughly know what we're doing.
The compat mode for a module with engineVersion vX.Y.Z-<anything> is always vX.Y.Z

Note: we should still strongly advise against doing this - imagine the following scenario:
We add breaking change 1 to dagger
You generate a module with a version with this engineVersion
We add breaking change 2 to dagger
Your module is still requesting the same api version - and so your module breaks.
This is expected! There's really no way to avoid this without having every single breaking change in dagger introduced with a timestamp (I thought about this for a good while, I don't really like it, but we could always return to it if it's a sticking point)
The only other option that I thought of is that vX.Y.Z-dev could serve the schema vX.Y.(Z - 1) - but then this means that builds cut off of main won't work with new features (making it harder to pick up), you'd need to manually specify --compat=vX.Y.Z - that feels fiddly - and would probably some weird test hackery.

Future work:

Because we add a new link-time variable, we need to update everywhere we do builds of dagger:

The downstream homebrew formula: https://github.com/Homebrew/homebrew-core/blob/master/Formula/d/dagger.rb#L28-L33
@jpadams are there others I need to be aware of? It would be nice to not break folks.

Daggerverse should have some way to ensure that modules with pre-release engine versions are carefully rejected - or at least highlighted.

As mentioned above, we really really want to make sure that users aren't really using these for general use.


Marketplace pages that are not responsive

Below is a list of pages that need to be made responsive so that the marketplace is fully operable by desktop and mobile devices:
/account

[ ]  assignUserInclude.xhtml
[ ]  errorMessages.xhtml
[ ]  importUser.xhtml
[ ]  ldap.Settings.xhtml
[ ]  organizationUdaDetails.xhtml
[ ]  paramtable.xhtml
[ ]  subscriptionAministratePanel.xhtml
[ ]  subscriptionUdaDetails.xhtml
[ ]  units.xhtml

/subscriptions

[ ]  parameterConfigurations.xhtml
[ ]  status.xhtml

/templates

[ ]  errorcontent.xhtml

/terms

[ ]  imprint.xhtml
[ ]  privacypolicy.xhtml
[ ]  terms.xhtml

others:

[ ]  accessDanied.Page.xhtml
[ ]  changePassword.xhtml
[ ]  changePasswordInclude.xhtml
[ ]  completePwdRecovery.xhtml
[ ]  confirm.xhtml
[ ]  confirmStartPwdRecovery.xhtml
[ ]  enterpriseLandingpage.xhtml
[ ]  errorPage.xhtml
[ ]  index.jsp
[ ]  index.xhtml
[ ]  insufficientAuthorites.xhtml
[ ]  loginPage.xhtml
[ ]  logoutPage.xhtml
[ ]  publicLandingpage.xhtml
[ ]  recoverPassword.xhtml
[ ]  registration.xhtml
[ ]  registrationOk.xhtml
[ ]  serviceProvider.xhtml

The impact is not clear. Let's group and prioritize this list according the user experience.
e.g.

Password recovery - Priority P2

Let's turn this into an epic and add issues from this list with respective priority.

Let's first merge the UI and test those pages.


release notes of v0.7.0 and deprecation warnings

Coverage increased (+0.008%) to 84.703% when pulling 42760c426382627c0c399f51bc92956ffb22de5d on INM-6:release/070 into c818613dd8e24b52285f13dcd83807fc66a476a2 on NeuralEnsemble:master.


Roll Flutter Engine from ecb513689262 to c33eae133e21 (1 revision)

https://github.com/flutter/engine/compare/ecb513689262...c33eae133e21
2022-10-27<EMAIL_ADDRESS>Macos re-enable TestSetMenu (flutter/engine#37058)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Extension causes high cpu load

Environment:

OS: Ubuntu 22.04
VS Code version: Any recent version
Extension version: v10.11.0

Steps to reproduce:

Open VS Code with Git Blame extension
Open a very large file with 20000 lines of text or so (eg: package-lock.json). In my case, the file has been part of more 300 commits ($ git rev-list --count HEAD -- package-lock.json), not sure if that is also relevant for this issue.
Monitor the CPU usage via htop
When the file was opened, in htop, it can be observed initially a git blame operation runs for a few seconds (eg: between 2 to 7 seconds) and after that, code consumes 100% CPU and that number never comes down.
Disabling the extension, followed by "Restart Extensions" releases the CPU consumption

Suggested solution:
In case it is tough to solve the issue due to the file size/history being too large, kindly consider providing manual and/or automatic turn off feature for this extension based on the file name and/or content length.
Similar issues:

https://github.com/Sertion/vscode-gitblame/issues/146
https://github.com/Sertion/vscode-gitblame/issues/97
https://github.com/Sertion/vscode-gitblame/issues/157

Hey webextensions! Thanks for the detailed bug report!
Do you know if running git gc mitigates the issue?
I still agree that there is value in a setting to not blame files that are too large but its very difficult to know how character count, line count, number of commits the file is part of, or number of commits in general influence the resource requirements.

@Sertion
Thank you for the response.
It seems that large file + longer history both are playing certain parts.
For a particular package-lock.json file, it works and fails randomly as tried with multiple attempts. In case the following helps to get the picture of the file-size/command-time:
$ time git blame -C --incremental -- /path/to/project/package-lock.json | wc -l
23056

real	0m7.143s
user	0m6.946s
sys	0m0.169s

For another use case where the file contains more than 100,000 lines and part of over 1000 commits, the CPU usage never comes down.
Probably a way to configure excluding some specific files would also be a decent workaround.
PS: GitLens seems to work fine even for the file with 100,000 lines as well (might take a few seconds to effectively load/render, but not too much considering the scale).

I've spent some time trying to figure this out and have made a pre-release version with a few changes. It would mean a lot if you could try it out and report back if it solved the issue!
It would also be helpful to know what state (the S column in htop) the process is in when it locks at 100% if the pre-release version does not resolve the issue.
gitblame-10.11.1-pre-release.zip

@Sertion
Thank you for the update.
The git blame operation runs with R and the code instance (/usr/share/code/code --type=utility --utility-sub-type=node.mojom.NodeService --lang=en-US ...) which is eating up the CPU also runs with R.
I tried out the pre-release version, but it didn't seem to help much. It is still working intermittently only for the file with 20000+ lines (out of 10 attempts, around 7 times it did not load and CPU usage stayed 100% for a code instance). When it works, the CPU usage gets back to normal after the load/rendering is complete. Otherwise, the CPU usage of code instance stays at 100% till when that particular instance of code is not killed or extensions aren't disabled.

Thank you for your help but I have to admit that I'm stumped. I've spent a few hours trying to reproduce the issue with this quite large file (25764 lines, 1678 commits) and have not been able to do so. I've made another pre-release version with some very minor changes from the last but I am unsure if it will solve anything.
gitblame-10.11.1-pre-release-2.zip

Thank you for trying further.
While I am yet to try the next version of pre-release, I attempted to establish reproducibility further.
I was also unable to reproduce it with next.js project in the few attempts I tried :+1:
But, https://github.com/facebook/create-react-app/blob/main/package-lock.json seems to be failing to work for my machine 3 to 6 times out of 10 attempts. It has more than 50k lines of contents.

@Sertion
gitblame-10.11.1-pre-release-2.zip doesn't seem to help. The problematic behavior is still the same.

The pre-release below adds gitblame.maxLineCount: a setting that prevents files with more lines than the setting declares from being blamed at all.
gitblame-11.0.0.zip

@Sertion
This solution / workaround is helping the situation. Thank you :smile:
Marking closed :+1:


fix: do not throw an error, revert to log and continue

This was a bug that was inadvertently added during support for the experimental feature groups feature. For now this is being reverted back to warn and continue as it should have been.
A note has been placed in the code to try and prevent this change from happening again in the future without future error handling at the proper levels.
Version 4 of aws-nuke may want this behavior as to not support incorrect filters, special compensation will have to be made for __global__

:tada: This pull request is included in version 0.21.1 :tada:
The release is available on GitHub release :rocket:


RealSense Viewer not starting / no video rendering in SDK

Required Info

Camera Model
D400

Firmware Version
Should be 5.9.11

Operating System & Version
Windows 10

Platform
PC

SDK Version
SDK 2.0 (Build 2.12.0)

Issue Description
I cannot launch the RealSense viewer nor any other application which renders a video.
To be more specific: When starting the RealSense viewer, only a white window appears. Similary, I cannot see any video when after building and running the examples in VS2017. Only two applications work: "Hello-world" and "save-to-disk".
Are there any additional requirements that could prevent my computer from starting these applications?
The device manager shows the RGB and the Depth camera (as expected since two applications work fine).
Do you have any suggestions?

[Realsense Customer Engineering Team Comment]
hi @Julz-s,
would like to confirm if pre-built realsense viewer can work on your PC? or only from your build?

[Realsense Customer Engineering Team Comment]
Hi @Julz-s,
if the pre-build viewer executable download from GitHub also have the same issue,
please see if your graphic card driver already update to the latest and support openGL.
We use some opengl api for image rendering.
best

[Realsense Customer Engineering Team Comment]
Hi @Julz-s,
please discard previous comment about openGL support,
if the pre-build viewer executable download from GitHub also have the same issue,
please see if upgrading your graphic card's driver can help or not
We have customer has similar issue and issue fixed after upgrading the graphic card driver.
best

Hi,
thank you for your replies.
The .exe has the same issue. It does not load the GUI.
I've just checked the Intel driver update tool telling me that all drivers (except one Bluetooth driver) are up to date.
Best

I don't believe this can be related to drivers.
Do you have additional USB3 ports on your platforms (perhaps back / front ports)?
Can you run rs-enumerate and post the output?

[Realsense Customer Engineering Team Comment]
Hi @Julz-s,
are you able to run the viewer now on either on same PC or another PC?
or provide the output of rs-enumerate?

Hi,
I use another PC now (and a Jetson Board) and it works fine now.
There is/was no issue with connecting the camera (since I was able to use the "save-to-disk" and the "Hello world" examples). However, all examples and tools using video streams don't work at all.
I have no idea why, but I'm with with my current setup.

[Realsense Customer Engineering Team Comment]
Hi @Julz-s
(1)are those two png files created by save-to disk looks valid? (like what you could see on viewer)
rs-save-to-disk-output-Depth.png
rs-save-to-disk-output-Color.png
(2) please see if the setting for permission of app to access camera is on or off
setting >privacy->Camera ->let apps to use my camera


feat(main menu): changes to the main menu (DEV-1451)

resolves DEV-1451

There's some top and bottom padding that should be removed and the menu item is highlighted if you are on the respective route which should be removed. This may also overwrite what @Vijeinath has done in #944 but he will have to confirm that.

I am very aware of the top and bottom padding. I could change it locally for this menu only - or change it for every menu within the app. The first option is imho wrong. The second option is out of the scope of this PR.


Error finding hashed API

Hey there. Great job and thanks for sharing.
Actually it works, there's just an error message with finding hashed function SecureZeroMemory:
Process hollowing successful. Error retrieving the function address.
I use Win11 version 23H2

@hawaii67 just don´t use this project, it is just nuts...
look at the discussion on X about the other project of this guy called "EchoStrike"
https://x.com/passthehashbrwn/status/1833316078430130573
Use something realy advanced and stealthy like ChaiLdr or DarkWidow combined with minimalistic FUD Shellcode like my own CelestialSpark Project

but to add a real solution, just uncomment line 291 https://github.com/stivenhacker/GhostStrike/blob/23a6b54c8299882e6e8f0536e1ca0fbf326b3229/GhostStrike.cpp#L291
I don´t see the actual problem, but the author wants so badly dynamically resolve the function names for whatever reason, he did include the windows.h header anyway, so you can just use the function SecureZeroMemory without dynamicaly resolving it first. No Security product will flag your Program as bad for using this function...