Rename the ObjectReference field in Destination

Expected Behavior
Currently we put all the fields at the top level of the Destination struct makes validation complex for the user. Similar concerns were expressed in https://github.com/knative/pkg/pull/561. The validation message here considers the three fields apiVersion, kind, and name as a single object distinct from uri, but that's not reflected in the api structure.
Something like this would be backward compatible with trigger and subscription:
type Destination struct {
	Ref *corev1.ObjectReference `json:"ref,inline"`
	URI *apis.URL `json:"uri,omitempty"`
	Path *string `json:"path,omitempty"`
}

and is more obvious that Ref member fields are logically distinct from URI and Path fields.
Actual Behavior
Additional Info
This is related to https://github.com/knative/eventing/issues/1918

/assign @capri-xiyue

Destination has a validation method. Just use that.

There's an ongoing discussion of API changes at https://github.com/knative/eventing/issues/1918. This issue is one suggestion but the decision hasn't been made yet.

Closed since it's a duplicate of https://github.com/knative/pkg/issues/772


Automatics parameters such map_tec_time presenting weird behaviour

Dear
Let me try to explain this issue properly.
I have 4 sub time periods: [SUMMER, SPRING, WINTER, FALL]
and a parent time which is YEAR
I have a technology X that works on the sub-annual time, the seasons however, if I set a relation XR associated to this technology X when IXMP generates the gdx file it creates a map_tec_time and map_tec_act sets automatically but with time year also associated to the technology X.
Therefore, instead of get the result as I was expecting, such as:
X, node, year time=[SUMMER, SPRING, WINTER, FALL]
I got
X, node, year time=[SUMMER, SPRING, WINTER, FALL, YEAR]
If I try to pass those sets to the model manually I get the following error message:
com.gams.api.GAMSException: Symbol map_tec_time exists already
How can I deal with this issue?
All the Best and thanks in advance.

Hi
I would like to emphasize that this issue still happens in the new version 3.0.
I found a work around solution I am creating another relation equation, sets and parameters and passing all info manually however I believe that is not the best solution.
Regards

@DIUANA thanks for opening this issue. Are you trying to use parameter relation_activity  for a scenario with sub-annual timesteps?

Thanks for responding me.
Yes Benham. I am trying to do it but I changed the equation for to ACT take into account the duration time because the equation balance is built only in the year level.
RELATION_EQUIVALENCE2(relation2,node,year)..
    REL2(relation2,node,year)
        =E=
    SUM(tec,
        ( relation_new_capacity2(relation2,node,year,tec) * CAP_NEW(node,tec,year)
          + relation_total_capacity2(relation2,node,year,tec)
            * SUM(vintage$( map_tec_lifetime(node,tec,vintage,year) ), CAP(node,tec,vintage,year) )
          )$( inv_tec(tec) )
        + SUM((location,year_all2,mode,time)$( map_tec_act(location,tec,year_all2,mode,time) ),
            relation_activity2(relation2,node,year,location,tec,year_all2,mode)
            * ( SUM(vintage$( map_tec_lifetime(location,tec,vintage,year_all2) ),
                  duration_time(time) * ACT(location,tec,vintage,year_all2,mode,time) )
                  + historical_activity(location,tec,year_all2,mode,time) )
          )
      )

Actually I changed the new relation that I created in order to be able to correctly pass the values for map_tec_time and avoid this issue.


updated to latest torii

Any chance we can update to the latest torii?
For me, at least, this has an important fix solving a race condition related to postMessage() and closing the popup window for many oauth workflows.

sure


Rebuild website from ground-up with clean architecture, revisit the ui design and rework some section

closes #61
closes #22
closes #60

Linktree page is missing


Rename the partner schools filter option

Context
We have removed the term "Partner schools" for users, instead using the term "Schools I work with"
Changes proposed in this pull request

[x] Rename the filter
[x] Update specs

Guidance to review

Log in as Patricia
Test the "Schools I work" with filter

Link to Trello card
Update wording for the 'partner schools' filter
Screenshots

I'm closing this PR as it has been combined into the overall way finding PR #1071.


Correct Example To Use required_providers

fix: Updates example to use the correct required_providers with KSOC. This was missing and was causing terraform init to not work correctly.

This PR is included in version 1.0.2 :tada:


Default ext install

Summary

the plan is for all extensions to be pulled out of the mainlnbits repo.
however, some LNbits instances expect to have certain extensions installed on start-up

these extensions could be installed manually by the admin, but that would mean a certain downtime

Solution:

this PR introduces a new environment variable: LNBITS_EXTENSIONS_DEFAULT_INSTALL (comma separated extension ids)
all the extensions in this list will be installed when the server starts
the first found release from the manifest.json is picked by default
if no release found, the extension will not be installed and the server will start without it (and try to find it on the next restart)
if an extension from this list is uninstalled then it will be re-installed on the next restart

the extension must be removed from this list in order to not be re-installed.

LGTM


Test workflow

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

Starting Meshery Extensions tests...


Merkle Proofs

Adding further support for the mAPI 1.3 data format, this is a small update though, it covers:

Setting Proof and DsCheck as bools
Adding MerkleFormat property to the Transaction submit request

Codecov Report

Merging #25 (8583374) into master (8308e6b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #25   +/-   ##
=======================================
  Coverage   93.44%   93.44%
=======================================
  Files           8        8
  Lines         290      290
=======================================
  Hits          271      271
  Misses         14       14
  Partials        5        5

Impacted Files
Coverage Δ

submit_transaction.go
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Add NULL check after native call to html method

If the method raises an exception, PyObject_CallObject() returns NULL,
but the code didn't check for that, which led to a segfault.
Fixes #108

It looks like you have Git misconfigured. If you want to, set the user.email to match one associated with GitHub then amend the commit to change the author.

The e-mail is as intended. I used the company e-mail address because markupsafe is a dependency of Lektor, which we use for our web site and documentation. But I use the same GitHub account for both hobby and work.

Tip, you can add multiple emails to github - that way you still get proper credit on your account for the commit. Anyway, up to you of course.

Yeah, probably I should do that.

Cherry picked to 1.1.x branch: eb9c1d434ba7a904410b954426800fc963986d56
Added changelog: d1b3c5adb38bdd27dd575f78f512100d09d499f4


Do not update the tag when data corruption found

Once data corruption has been found, deliberate user action must be required
(restore from backup or manual tag reset). Otherwise, corruption could easily
be missed since a subsequent run of cshatag would return <ok>.

Not sure about this. Not updating the tag means that you will not notice if the file corrupts again later.  Should be at least be activated through a flag.


Push Feature - Merged

Description

For some reason my formatter still changed a ton of white spaces. Any idea @andreybavt ?


Subnets on JDBCs databases

Hi, I'm trying to access some JDBC databases from the template JDBCTOGCS. But it always returns me an connection error. I think it is because of my subnet (I'm using a default one in us-central1 with private Google access).
The problem is that I can't access public databases, like ones in relational.fit.cvut.cz on port 3306. How should I approach that?
That is the error:
Exception in thread "main" com.mysql.cj.jdbc.exceptions.CommunicationsException: Communications link failure

The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.

How can I enable a subnet firewall to accept port 3306?

Hi @sjlva
Thanks for reaching out.
Serverless Dataproc spins up (ephemeral) VM on your GCP VPC and therefore will have access to anything on internet unless your org policy blocks it.
I ran below command and I was able to export data out.
export GCP_PROJECT=shashank-data-eng \
export REGION=us-central1  \
export SUBNET=projects/shashank-data-eng/regions/us-central1/subnetworks/default   \
export GCS_STAGING_LOCATION=gs://shashank-data-eng/dataproc-templates/jdbc2gcs/publicdb \
export JARS=gs://shashank-data-eng/dataproc-templates/jdbc2gcs/jar/mysql-connector-java-8.0.28.jar

bin/start.sh \
-- --template JDBCTOGCS \
--templateProperty 'jdbctogcs.jdbc.url=jdbc:mysql://relational.fit.cvut.cz:3306/financial?user=guest&password=relational' \
--templateProperty jdbctogcs.jdbc.driver.class.name=com.mysql.cj.jdbc.Driver \
--templateProperty jdbctogcs.output.location=gs://shashank-data-eng/dataproc-templates/jdbc2gcs/publicdb/client_export \
--templateProperty jdbctogcs.output.format=avro \
--templateProperty jdbctogcs.write.mode=OVERWRITE \
--templateProperty 'jdbctogcs.sql=SELECT * FROM financial.client'

Please ack if similar works for you or not ?
Reach out to<EMAIL_ADDRESS>if you have more questions.

Hi @shashank-google, thanks for your reply.
But it does not work for me. I think it is probably an VPC error, how should I configure a default VPC to work properly? I already enabled the Private Google Access.
I run the same command you mentioned (just changed the env variables to suit my ones, and using the mysql-connector-java-8.0.30.jar instead).
I get the following error message:
Exception in thread "main" com.mysql.cj.jdbc.exceptions.CommunicationsException: Communications link failure

The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
	at com.mysql.cj.jdbc.exceptions.SQLError.createCommunicationsException(SQLError.java:174)
	at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:64)
	at com.mysql.cj.jdbc.ConnectionImpl.createNewIO(ConnectionImpl.java:828)
	at com.mysql.cj.jdbc.ConnectionImpl.<init>(ConnectionImpl.java:448)
	at com.mysql.cj.jdbc.ConnectionImpl.getInstance(ConnectionImpl.java:241)
	at com.mysql.cj.jdbc.NonRegisteringDriver.connect(NonRegisteringDriver.java:198)
	at org.apache.spark.sql.execution.datasources.jdbc.connection.BasicConnectionProvider.getConnection(BasicConnectionProvider.scala:49)
	at org.apache.spark.sql.execution.datasources.jdbc.connection.ConnectionProvider$.create(ConnectionProvider.scala:77)
	at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$.$anonfun$createConnectionFactory$1(JdbcUtils.scala:64)
	at org.apache.spark.sql.execution.datasources.jdbc.JDBCRDD$.getQueryOutputSchema(JDBCRDD.scala:62)
	at org.apache.spark.sql.execution.datasources.jdbc.JDBCRDD$.resolveTable(JDBCRDD.scala:57)
	at org.apache.spark.sql.execution.datasources.jdbc.JDBCRelation$.getSchema(JDBCRelation.scala:239)
	at org.apache.spark.sql.execution.datasources.jdbc.JdbcRelationProvider.createRelation(JdbcRelationProvider.scala:36)
	at org.apache.spark.sql.execution.datasources.DataSource.resolveRelation(DataSource.scala:350)
	at org.apache.spark.sql.DataFrameReader.loadV1Source(DataFrameReader.scala:274)
	at org.apache.spark.sql.DataFrameReader.$anonfun$load$3(DataFrameReader.scala:245)
	at scala.Option.getOrElse(Option.scala:189)
	at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:245)
	at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:174)
	at com.google.cloud.dataproc.templates.jdbc.JDBCToGCS.runTemplate(JDBCToGCS.java:66)
	at com.google.cloud.dataproc.templates.main.DataProcTemplate.runSparkJob(DataProcTemplate.java:187)
	at com.google.cloud.dataproc.templates.main.DataProcTemplate.main(DataProcTemplate.java:135)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.apache.spark.deploy.JavaMainApplication.start(SparkApplication.scala:52)
	at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:955)
	at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:180)
	at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203)
	at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90)
	at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1043)
	at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1052)
	at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
Caused by: com.mysql.cj.exceptions.CJCommunicationsException: Communications link failure

The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:490)
	at com.mysql.cj.exceptions.ExceptionFactory.createException(ExceptionFactory.java:61)
	at com.mysql.cj.exceptions.ExceptionFactory.createException(ExceptionFactory.java:105)
	at com.mysql.cj.exceptions.ExceptionFactory.createException(ExceptionFactory.java:151)
	at com.mysql.cj.exceptions.ExceptionFactory.createCommunicationsException(ExceptionFactory.java:167)
	at com.mysql.cj.protocol.a.NativeSocketConnection.connect(NativeSocketConnection.java:89)
	at com.mysql.cj.NativeSession.connect(NativeSession.java:120)
	at com.mysql.cj.jdbc.ConnectionImpl.connectOneTryOnly(ConnectionImpl.java:948)
	at com.mysql.cj.jdbc.ConnectionImpl.createNewIO(ConnectionImpl.java:818)
	... 31 more
Caused by: java.net.SocketException: Network is unreachable (connect failed)
	at java.base/java.net.PlainSocketImpl.socketConnect(Native Method)
	at java.base/java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:412)
	at java.base/java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:255)
	at java.base/java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:237)
	at java.base/java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
	at java.base/java.net.Socket.connect(Socket.java:609)
	at com.mysql.cj.protocol.StandardSocketFactory.connect(StandardSocketFactory.java:153)
	at com.mysql.cj.protocol.a.NativeSocketConnection.connect(NativeSocketConnection.java:63)
	... 34 more

This is not dataproc template issue, so at this point it is beyond our scope.
Btw, it is not about private google access, as this mysql is hosted on public Internet.
To debug you can create a vm manually on your vpc and verify if mysql cli client is able to connect (or you are able to telnet host and port). If that works then ideally spark vms launched by dp serverless will also work.

@sjlva bumping up to check if you were able to debug it.

@shashank-google I created a vm, in the us-central1 default network. It runs fine:
mysql -u guest -prelational -h relational.fit.cvut.cz -P 3306 financial -p
mysql: [Warning] Using a password on the command line interface can be insecure.
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

Welcome to the MySQL monitor.  Commands end with ; or \g.
Your MySQL connection id is 50204
Server version: 5.5.5-10.3.15-MariaDB-log MariaDB Server

Copyright (c) 2000, 2022, Oracle and/or its affiliates.

Oracle is a registered trademark of Oracle Corporation and/or its
affiliates. Other names may be trademarks of their respective
owners.

Type 'help;' or '\h' for help. Type '\c' to clear the current input statement.

mysql> SELECT * FROM financial.client LIMIT 5;
+-----------+--------+------------+-------------+
| client_id | gender | birth_date | district_id |
+-----------+--------+------------+-------------+
|         1 | F      | 1970-12-13 |          18 |
|         2 | M      | 1945-02-04 |           1 |
|         3 | F      | 1940-10-09 |           1 |
|         4 | M      | 1956-12-01 |           5 |
|         5 | F      | 1960-07-03 |           5 |
+-----------+--------+------------+-------------+
5 rows in set (0.12 sec)

mysql>

At this moment, I'm running those templates in a dataproc cluster with the command gcloud dataproc jobs submit pyspark because dataproc serverless is failing.
Do you have some step by step network configuration to use dataproc? I followed this https://cloud.google.com/dataproc-serverless/docs/concepts/network, I also tried the default one with Private Google Access enabled. I got no success.
I really can't figure out why It's not running. Why can I run the templates in a dataproc cluster, but I can't in dataproc serverless?

I think I got it.
You will need to create Cloud NAT (for the subnet / region you are passing to dataproc) as mentioned: https://cloud.google.com/dataproc-serverless/docs/concepts/network
LMK if it works after that.

Thank you @shashank-google!
I created a Cloud NAT and it worked!


[clkmgr] Idle signal permanence

This is a tracking issue to investigate how to tweak the idle signal from each transactional block such that they are more fault resistant.
The basic idea here is that if a block is "glitched" to idle, it should be able to recover if that's a temporary condition.
This will need to be brought up during the security working group to be further discussed.

@cdgori
@moidx
@felixmiller

This corresponds to the stuff we talked about earlier today right?  Assuming so... I'll elaborate:
We want the idle/ok-to-clock-gate signalling to be as "expansive" as possible.  Something like "state == IDLE" is kind of ideal (but not always possible) - with sparse state we'll get an alert if the state gets glitched, and if the ok-to-gate indicator is glitched, it should at most suppress one clock edge when the indicator gets re-clocked/latched.
(Tim and I talked about having some lag in the response of clkmgr to actually doing the clock gating, since we need at least one clock edge to do this re-latching - and in reality a few cycles before gating probably doesn't hurt anything.)

this hasn't really gained any traction, so i will just close it down for now.


Adds argo-site repo

/cc @nate-double-u
Signed-off-by: Patrice Chalin<EMAIL_ADDRESS>

Any idea why the new repo isn't yet showing up? https://clomonitor.io/projects/cncf/argo/argo

Hi @chalin
Yes, we haven't deployed this change yet. We'll do it a bit later today.

Deployed!


Over 24 hours and still dSYMs files missing

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.6 (11E708)
Firebase SDK version: NA (Refer to below components)
Firebase Components & Verison:
FirebaseCore - Version: 6.7.0
Firebase/Messaging - Version: 6.24.0
Firebase/Analytics - Version: 6.24.0
Firebase/Crashlytics - Version: 6.24.0
Component versions: Above
Installation method: `CocoaPods

[REQUIRED] Step 2: Describe the problem
dSYMs are getting uploaded successfully with the below upload script from terminal but not reflecting on Firebase Crashlytics console. Tried uploading the zip file with all dSYM files and individually uploading the specific one dSYM file we're not being able to have insights into our crashes.
Steps to reproduce:
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.
Step 1: Downloaded dSYMs from AppStore connect.
Step 2: Run below upload script, upload is successful.
Step 3: Firebase Crashlytics console still shows dSYMs is missing even after more than 24 hours
Relevant Code:
dSYMs upload script:
/Users/xxxx/WorkSpace/ProjectRootDir/Pods/FirebaseCrashlytics/upload-symbols -gsp /Users/xxxx/WorkSpace/ProjectRootDir/xxxx/Firebase/Dev/GoogleService-Info.plist -p ios /Users/xxxx/dsyms/appDsyms.zip
Extra description:
Having the exact same issue as:
https://github.com/firebase/firebase-ios-sdk/issues/5327
At the end, the referenced issue was closed indicating a temporary delay but for our team crashes are still not being reported and it's been over 24 hours, even after trying uploading the zip file with all dSYM files and individually uploading the specific one dSYM file we're not being able to have insights into our crashes. This is a high priority for our team... We opened a Firebase support ticket but no response yet.
Could anyone assist on this?

Hey all, the 24-hour delay was related to an outage and should be resolved now. If your dSYMs are still not listed as available or if you're having other migration troubles, please file a support issue so our staff can look at your project.
Closing this issue since it's not easily resolvable in GitHub comments; please file a support ticket instead of continuing to comment here.
Originally posted by @morganchen12 in https://github.com/firebase/firebase-ios-sdk/issues/5327#issuecomment-616699688

Hey @hIUG, sorry to hear you're still having trouble. Please follow the link in my original comment to file this issue with Firebase Support instead of on GitHub.


Added math documentation

All functions and constants are accounted for.

I just caught a spelling error in the code that was merged - chech was used instead of check in some places - it's also been copied into these docs. It's an easy thing to miss, so I'm correcting the code right now.
As for these docs, I'll give them a once-over so the text flows a bit better - but next time, remember to put a period at the end of each sentence (some are missing one).

OK, I've made the tweaks (I prefer full sentences for documentation - otherwise what you wrote was fine).
I've also added the link to the math docs to the main page: https://toylang.com/


Implement joined source for UPDATE statement

    UPDATE odoc SET osmlineid = s.osmlineid
              FROM odoc o INNER JOIN osmline s ON o.uniqueid = s.uniqueid;

Hvad created feature request so people can vote
http://feathub.com/agershun/alasql/+24

Any Update on This

Not that I know of...


Compatibility check azd 0.7.0-beta1 - Issue 50

Purpose

This PR contains the necessary changes in the repository to make the project compatible with azd version 0.7.0. The changes in the new release do not affect the project, so mainly the documentation was updated.
In the regression test a mismatch between the Terraform provider version and the Node.js version showed up. This was fixed by upgrading to the latest minor version of the Azure terraform provider. The  CAF provider fetched the latest version based on the restrictions and does not need an upgrade.

Does this introduce a breaking change?
[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?
[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code

- execute azd provision or up (adjust the azure.yaml for Terraform)
- execute azd infra delete or down (adjust the azure.yaml for Terraform)

What to Check
Verify that the following are valid

Infrastructure gets set up/destroyed successfully
if azd up/azd down is used, in addition to the infrastructure the apps gets deployed/undeployed successfully

Other Information
See: https://devblogs.microsoft.com/azure-sdk/azure-developer-cli-azd-march-2023-release/ for details about the azd version 0.7.0

Fixes #51

INFO: Opened a issue https://github.com/Azure/azure-dev/issues/1716 as well as a PR https://github.com/Azure/azure-dev/pull/1717 in the Azure Developer CLI repository to fix this.

should we wait for azd update of tf provider version 3.47? TF deploy with AZD is not supporting this version at this point, right?

should we wait for azd update of tf provider version 3.47? TF deploy with AZD is not supporting this version at this point, right?

The azd does not put constraints on the provider version. The templates delivered with azd contain the outdated version. So we can already go for the latest provider version and do not need to wait for azd to upgrade. I would not expect the adjustment to land before the April release of azd


HDDS-11325. (addendum) Intermittent failure in TestBlockOutputStreamWithFailures#testContainerClose

What changes were proposed in this pull request?
HDDS-11325. (addendum) Intermittent failure in TestBlockOutputStreamWithFailures#testContainerClose
Please describe your PR in detail:

Removed @Flaky annotation  from TestBlockOutputStreamWithFailures#testContainerClose and TestBlockOutputStream#testWriteMoreThanMaxFlushSize
Relaxed the assertions in the test to match the current behavior.
Verified there's no longer intermittent test failures in TestBlockOutputStreamWithFailures#testContainerClose and TestBlockOutputStream#testWriteMoreThanMaxFlushSize

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11325
How was this patch tested?
Flaky-test-check:
https://github.com/jojochuang/ozone/actions/runs/10532241331
TestBlockOutputStream#testWriteMoreThanMaxFlushSize[HDDS-11325-take2]-10x10
https://github.com/jojochuang/ozone/actions/runs/10565349000
TestBlockOutputStreamWithFailures#testContainerClose[HDDS-11325-take2]-10x10

It failed in CI at a later location. This one might indicate product issue actually.
Error:  Tests run: 24, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 29.049 s <<< FAILURE! - in org.apache.hadoop.ozone.client.rpc.TestBlockOutputStream
Error:  org.apache.hadoop.ozone.client.rpc.TestBlockOutputStream.testWriteMoreThanFlushSize(boolean, boolean)[3]  Time elapsed: 0.053 s  <<< FAILURE!
org.opentest4j.AssertionFailedError: expected: <250> but was: <50>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:166)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:161)
	at org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:632)
	at org.apache.hadoop.ozone.client.rpc.TestBlockOutputStream.testWriteMoreThanFlushSize(TestBlockOutputStream.java:539)

Thanks @jojochuang for the patch, @smengcl for the review.


routing: move call to PruneGraph outside of the collating loop

This commit moves the call to PruneGraph outside of the loop that collates all of the spentOutputs in syncGraphWithChain. With this change, if a node has been offline for a long period of time, resyncing with the chain no longer takes up as much memory (1MB vs 200MB in some cases) or time. Previously, PruneGraph was called for every block and allocated a very large map further down in the pruneGraphNodes function. Now, pruneGraphNodes is only called once.
I found this by profiling lnd and noticed that it allocated a huge amount of memory in the pruneGraphNodes function.  This was after my node was offline for several hundred blocks.  The syncGraphWithChain function also took ~20 seconds.
ROUTINE ======================== github.com/lightningnetwork/lnd/channeldb.(*ChannelGraph).pruneGraphNodes.func1 in /Users/nsa/go/src/github.com/lightningnetwork/lnd/channeldb/graph.go
  185.88MB   185.88MB (flat, cum) 15.81% of Total
         .          .    934:			return nil
         .          .    935:		}
         .          .    936:
         .          .    937:		var nodePub [33]byte
         .          .    938:		copy(nodePub[:], pubKey)
  185.88MB   185.88MB    939:		nodeRefCounts[nodePub] = 0
         .          .    940:
         .          .    941:		return nil
         .          .    942:	})
         .          .    943:	if err != nil {
         .          .    944:		return err

After moving PruneGraph out of the loop, initial sync of ~300 blocks took ~1 second and only ~1MB on the heap :)
ROUTINE ======================== github.com/lightningnetwork/lnd/channeldb.(*ChannelGraph).pruneGraphNodes in /Users/nsa/go/src/github.com/lightningnetwork/lnd/channeldb/graph.go
         0     1.05MB (flat, cum)  0.13% of Total
         .          .    924:
         .          .    925:	// We'll use this map to keep count the number of references to a node
         .          .    926:	// in the graph. A node should only be removed once it has no more
         .          .    927:	// references in the graph.
         .          .    928:	nodeRefCounts := make(map[[33]byte]int)
         .     1.05MB    929:	err = nodes.ForEach(func(pubKey, nodeBytes []byte) error {
         .          .    930:		// If this is the source key, then we skip this
         .          .    931:		// iteration as the value for this key is a pubKey
         .          .    932:		// rather than raw node information.
         .          .    933:		if bytes.Equal(pubKey, sourceKey) || len(pubKey) != 33 {
         .          .    934:			return nil

2019-06-27 13:24:50.588 [INF] CRTR: Prune tip for Channel Graph: height=1565711, hash=000000000000000f871f41667bd527f58e7be40f1d3cc6fd91e98e3fb555130e
2019-06-27 13:24:50.588 [INF] CRTR: Syncing channel graph from height=1565711 (hash=000000000000000f871f41667bd527f58e7be40f1d3cc6fd91e98e3fb555130e) to height=1566009 (hash=00000000000004f825026610b2cc7757c0eaf1d5fd49f0e94cb9c092bec4edc5)
2019-06-27 13:24:51.648 [INF] CHDB: Pruned unconnected node 023d6f31cb13a37c4ede2cd6e0aa119d933ca18fa15244778a72e8ca0232f53539 from channel graph
2019-06-27 13:24:51.648 [INF] CHDB: Pruned unconnected node 0291f510a28f04c76cf18d2c48096f7d5332157443de46e94c188a5e30d51014d4 from channel graph
2019-06-27 13:24:51.648 [INF] CHDB: Pruned unconnected node 030f933df635a4b7ce68cd36f8f25521babe015d1f611775de997d5b35618030ae from channel graph
2019-06-27 13:24:51.649 [INF] CHDB: Pruned 3 unconnected nodes from the channel graph
2019-06-27 13:24:51.683 [INF] CRTR: Graph pruning complete: 4 channels were closed since height 1565711

I think here, since the changes are rolled back since the last prune tip via DisconnectBlockAtHeight, the chain filter should also be updated to this previous tip.  Otherwise, if there's a re-org, we would only prune the graph for any new blocks and forget about old changes since the last tip.
https://github.com/lightningnetwork/lnd/blob/f4feb66a1fe1a933e798352458a32db1844e5477/routing/router.go#L934-L953

Comments addressed @cfromknecht

@Crypt-iQ changes lgtm, ready for squash!

@cfromknecht squashed


Question on the experimental setting of frozen model.

The CLIP backbone is not frozen by setting UNIFORMERV2.FROZEN = False. The gradient will still be backward into the CLIP model, since the temporal_cls_token acts as a query to attend all CLIP image patches.
The CLIP backbone parameters have also changed at the checkpoints you provided, such as Frozen UniFormerV2-L/14@336.

Thanks for your finding! I have ignored adding the code about frozen when merging code. I will add it soon.

Note that frozen means prompt tuning, thus the gradient of new parameters will be backward.
I have checked the backbone parameters, and they are the same as the original visual encoder. Maybe you use different packages or environments (FP16?), thus the extracted weights of the visual encoder are a little different.

The frozen UniFormerV2:

The original CLIP visual encoder:

I have added the code here.
https://github.com/OpenGVLab/UniFormerV2/blob/aab529237f69bd420d00f43ab0f780661a593720/slowfast/models/uniformerv2.py#L88-L104

Thank you very much for adding the code!
On the reason why CLIP parameters have changed, I found it comes from the difference in the CLIP model. I used CLIP from hugging face, lol.


Constrain uninitialized memory to 0

Missing constraints in the Memory Stark.
We do not check that memory is initialized at 0, that is to say that a read at a new address without a prior write must have a value of 0.
An immediate exploit is to bypass JUMPDEST analysis in case of a jump outside of the contract code, and subsequently arbitrary code execution in the uninitialized memory. In https://github.com/topos-protocol/plonky2/tree/code_injection, the code of the contract is a jump to address 0x80 which should raise an invalid jump exception, but the transaction is accepted and the proof is valid.

@nbgl Assigning you, since I presumed you'd be interested in this exploit!


LDAP (RFC 7612) sharing/browsing support

(Referencing corresponding ippsample sharing and ippsample browsing issues.)
Implement LDAP (RFC 7612) sharing/browsing support in cupsd, with cupsEnumDests using a new CUPS operation to search for shared printers.
Changes include:

[ ] Add OpenLDAP schema (LDIF) file for printers (installed to /etc/openldap/schema)
[ ] Add LDAPBindDN, LDAPPassword, LDAPServer, and LDAPDN directive support to "cupsd.conf"
[ ] Add ldap calls in dirsvc.c
[ ] Add vendor extension operation to cupsd to search for LDAP printers and return a list to clients (locally only). Working name: CUPS-Search-Printers (which could also do the Bonjour lookups as well)
[ ] Add request in cupsEnumDests.

CUPS-Search-Printers request:
Group 1: Operation Attributes:
    "attributes-charset (charset)"
    "attributes-natural-language (naturalLanguage)"
    "printer-uri (uri)" or "system-uri (uri)"
    "requesting-user-name (name)"
    "requested-attributes (1setOf keyword)"
    "limit (integer(1:MAX))"
    "max-time (integer(0:MAX))" - timeout in milliseconds
    "printer-location (text)"
    "printer-type (enum)"
    "printer-type-mask (enum)"
    "printer-uuid (uri)"

CUPS-Search-Printers response:
Group 1: Operation Attributes:
    "attributes-charset (charset)"
    "attributes-natural-language (naturalLanguage)"

Groups 2-N: Printer Attributes:
    Printer attributes as defined in [Appendix D of RFC 8011](https://tools.ietf.org/html/rfc8011#appendix-D)

Example cupsd.conf directives:
LDAPBindDN uid=cupsd,ou=people,dc=example,dc=org
LDAPPassword <ldap password>
LDAPServer ldaps://ldap-server.example.org/
LDAPDN dc=example,dc=org

This no longer seems likely.


else if redundancy

#patch#
Remove redundancy. This else if is identical to the one before it

Visual Acceptance tests:
No changes


HTML Converter: Images position

The docx to Html Converter takes the image and adds a img element inside a span element on the html generated, but all the spans have white-space: pre-wrap (because of this setting : GeneralCss = "span { white-space: pre-wrap; }"; ) and because of that all the images are shifted to the right  when the image is on the left side of the document, and shifted down when the image is on the right side of the document. If the white-space: pre-wrap CSS is removed from all the spans then the html has no spaces between all elements.
I fixed that by adding a new css class which overwrites the white-space: pre-wrap with white-space: nowrap for the spans that have an img element inside with jquery after the html has been generated.

Do you think that the default css as shipped should have white-space: nowrap?

Only the spans that have an img tag inside. I fixed it by replacing the General Css with span { white-space: pre-wrap; } .nowrapwhitespace {white-space:nowrap !important;} and then with jquery $('img').parent().addClass ("nowrapwhitespace"); so I can add that css class to the spans containing an image.

Closing all issues as this repo is being archived and will no longer be maintained by Microsoft. The project is licensed for continued use and development by forking to your own repo.


Cannot deactivate virthualenv inside screen/tmux

If I activate a virthualenv in a shell and then start screen or tmux, the virthualenv persists and I can use it as normal, as I would expect. However, I cannot deactivate it as the deactivate function is not exported by bin/activate. I would recommend exporting that function, but I don't know if that would have other undesirable consequences, and I haven't yet had a chance to test it out.
I think we should either find a way to safely and robustly deactivate from within screen/tmux or at least make it explicit in the documentation that it can't be done.

I can say that adding export -f deactivate somewhere after the definition of the function does make it available inside each screen/tmux window and appears to deactivate only within that shell.

I wrote a simple wrapper for tmux command:
# virtualenv does not work correctly when tmux is run inside virtualenv.
# this function is a wrapper around tmux command that deactivates virtualenv
# before running tmux and reactivates it again. --cenkalti
tmux()
{
    local current_env=""
    if [ "$VIRTUAL_ENV" != "" ]; then
        current_env="$VIRTUAL_ENV"
        deactivate
    fi

    command tmux "$@"
    local ret=$?

    if [ "$current_env" != "" ]; then
        workon $(basename $current_env)
    fi

    return $ret
}

It seems commands after tmux execution is executed after the tmux subprocess is detached or exited.
So I wrote that part in bashrc directly instead.
The following code works for me:
(write in ~/.bashrc, for instance)
if [ "$current_env" != "" ]; then
    workon $(basename $current_env)
    unset current_env
fi

tmux()
{
    current_env=""
    if [ "$VIRTUAL_ENV" != "" ]; then
        current_env="$VIRTUAL_ENV"
        deactivate
        export current_env
    fi

    command tmux "$@"
}

The same way is available for screen, too.


outputDir config not working

Version
3.0.4
Reproduction link
https://github.com/oswaldofreitas/vuecli_outputdir_issue
Node and OS info
Node 10.11.0 / npm 6.4.1 / macOS Mojave 10.14
Steps to reproduce

$ vue create project
$ vue ui
run build task (not sure if this step is mandatory)
change outputDir to "dist2"
run build task

What is expected?
built files created in the new output folder
What is actually happening?
files are still created in the default "dist" folder

I tried changing outputDir value to:

"dist2"
"../htdocs/js/client-prod-new"
path.resolve('../htdocs/js/client-prod-new')
and none worked

hmmm, now trying to build on vue ui it uses the default dist value again, so I guess issue is only in the gui

I can confirm that it outputDir is not taken into account when using the UI and does work when doing npm run build.

Next to the button to execute the build command there's a settings dialog which contains the output dir. Clear that value and outputDir from vue.config.js will be used.

changing it within vue ui created the file below which started working with custom directory. Haven't debugged yet which one helped.
module.exports = {
    publicPath: undefined,
    outputDir: '../public/site/dist',
    assetsDir: undefined,
    productionSourceMap: undefined,
    parallel: undefined,
    css: undefined
}

I just had the same problem. It's indeed happening when using the default build task on the UI.
The reason is that this task has actually an output directory dist as default parameter. It will always override the value vue.config.js, by using the --dest argument of the command line.
My fix so far is to just set that parameter to an empty string so that the build task use the value from vue.config.js.

I thought this parameter under build was confusing as well, shouldn't it be empty as default in stead of overriding as default?

Wait, this seems crazy to me. @LinusBorg Shouldn't vue.config.js be the source of truth? Or at least if you're building through the UI, the parameters should be read/copied from vue.config.js?
You can literally set the `outputDir' (or some other configuration) one screen up in the UI, and it's ignored if you run the tasks via the UI.

Wait, this seems crazy to me. @LinusBorg Shouldn't vue.config.js be the source of truth?

I would say it's the other way around, because those params in the gui are indeed comandline parameters so they are meant to override main config on the fly.

Or at least if you're building through the UI, the parameters should be read/copied from vue.config.js?

This however makes sence because it's what normally should be expected, values should extend from vue.config if any and fallback to "default values" only if there is not settings yet.

What I expected when I encountered this issue is that vue.config.js was used to determine the output dir. The GUI shouldn’t have a default value set to /dist. It’s confusing because:

it’s the same value as the default for vue.config.js yet it’s in fact an override. I found out about the outputDir when i modified the outputDir in config file and had different behaviour between the CLI and the GUI.
it’s hidden in parameters and can easily be missed in the output log.

An empty outputDir for the GUI seems to be a more sensible default imo, fallbacking this to vue.config.js

If anyone else has this problem I solved it by using npm run build in the root of my directory Instead of  vue build. This runs the vue cli module installed in the project directory after running  vue create application_name.  I was using the globally installed before vue cli and it wasn't workin.g

I don't understand what the point is in the VUE CLI vue build command if it ignores what you setup in  vue.config.js and uses what is set in parameters of the build command where the value is actually read from package.json or fallsback to dist, it is not read from vue.config.js according to the description in the UI modal this is confusing!
This is the description in the CLI Tools when you click on parameters button on the build command and read the option description of ``:

If enabled, hard-coded arguments in your package.json file will be replaced with the values defined below...

Note that it specifically says package.json not vue.config.js...
https://cli.vuejs.org/config/#outputdir does not mention anything about the config value for this parameter having to be set in package.json.
This sub topic is under the main topic of https://cli.vuejs.org/config/#vue-config-js
HOWEVER

Override hard-coded arguments on/off has no impact on any combination of settings.

Deleting any content out of the Output Directory field in the build parameters screen DOES make it fallback to your settings in vue.config.js

For my GitHub templates this bug in the build parameters screen for production build is a breaking setting.
I have to inform other developers of these extra steps to fix this bug.
I don't know why @LinusBorg closed this issue and not address it?
Clearly these is some inconsistency between the documentation and what is actually happening vs what is expected to happen.

I don't know why @LinusBorg closed this issue and not address it?

Good question. I can'T answer it as that was over a year ago.
Just a quick note:

Note that it specifically says package.json not vue.config.js...

That's likely just a small inaccuracy in the docs.
Just like the UI (like the CLI itself) will write changes to the config to either vue.config.js or a vue field in package.json (depending on which variant you are using), it will also read from both.
Sidenote: From readint he following, izt seems to me that you are unaware of that second option:

https://cli.vuejs.org/config/#outputdir does not mention anything about the config value for this parameter having to be set in package.json.
This sub topic is under the main topic of https://cli.vuejs.org/config/#vue-config-js

But at the very top of that page it say (highlighting mine):

vue.config.js is an optional config file that will be automatically loaded by @vue/cli-service if it's present in your project root (next to package.json). You can also use the vue field in package.json, but do note in that case you will be limited to JSON-compatible values only.

Concerning your actual issue: Unfortunately I have a bit of a problem following your explanation of what exactlky does and doesn't work, so I'll try to repeat:
The problem, according to what you experience, is basically that the "override hardcoded arguments" option is deactivated in the UI, but nonetheless works as if it's active, that means: the build command that the UI runs will use what is defined in the "Output Directory" field event though it should not, as the oltion is deactivated?

The problem, according to what you experience, is basically that the "override hardcoded arguments" option is deactivated in the UI, but nonetheless works as if it's active, that means: the build command that the UI runs will use what is defined in the "Output Directory" field event though it should not, as the oltion is deactivated?

Correct sir.
The deactivated override is overriding the value of my setting of outputDir: 'public/app' in vue.config.js with the default value of dest, defiantly incorrect behavior.

Got it, thanks.

By they way you just mentioned that the settings in that Build Parameters screen should be written to  to package.json or config.vue.js?
It is not doing that, it actually saves all of those settings to %HOMEPATH%/.vue-cli-ui/db.json

No, I mean the actual config values, under the "Configuration" page, not the setting that you set fopr your task paramters.

Understood, Yes those work as expected then, thanks for clarity on this.

FYI the "override hardcoded arguments" is about arguments in the scripts in package.json, for example vue-cli-service build --dest ./out.


Rawgit is deprecated

this project uses rawgit to host the install script, but the rawgit.com site says their site is deprecated and would close down anytime.
Shouldn't you just use the script hosted at the repo instead?

Very true, thanks for pointing it out. I've now updated all these to use the direct github URL, thanks!


Added Defaro DAT-101 and more ProductID for Fibaro FGFS101

PR description here

@zwave-js-bot fix lint

@zwave-js-bot fix lint

Thanks - I had a few more things, but I did them myself.

Thanks a lot for your help.


[feature-request] Add font selection

If we use HTML why don't make it able to select a font for render?

The HTML output should use whatever your browser's default monospace font is. So just set your browser to use the font you want.

Depending on your system, that may rely on the configuration at ~/.config/fontconfig/fonts.conf.

There's now an example fontconfig file (which is used by the current deployment) at app/.fontconfig/fonts.conf.


Customize format for last-commit badge

:clipboard: Description

Can we add a query parameter, say f which will send to moment.format(f), to control the date format? Maybe not only last-commit badge, but also other date related badges.
My real use-case is a little special. There is a repository, Bogdanp/awesome-advent-of-code, which collects solutions for a programming puzzle activity. Because the activity is only hold in December every year. It's not easy to distinguish last-commit date among those solution repositories.

relative date badges

should always be absolute date, to get a timeless view (archive friendly)
the relative date can still be visualized as color (green - yellow - red)

Hi everyone, since this issue is still open, here we bump.
We really like the age coloring, but it would be useful to have customizable date format too (to have costant size badges, in our case).
To get an idea, we need to generate badges like these:

Or these:

Thanks in advance.


consider increase  the maximumpoolsize of the addConnectionExecutor when network has delay

the  corepoolsize  and the maximunpoolsize  of the addConnectionExecutor is always 1 . that is  is Serial create the connection.
It usually just take 20-30 ms to create a new Connection， this case is OK  when the  network is  normal. 50 connections  just take 1-2 seconds.
 ThreadPoolExecutor executor = new ThreadPoolExecutor(1 /*core*/, 1 /*max*/, 5 /*keepalive*/, SECONDS, queue, threadFactory, policy);
the  following case:
the  minIdlenum  is   5, the  maxpoolsize is  50.
But when the network has delay， every connection  need take 3 seconds ,  A lot of requests came，achieve 50 connections  need  150 seconds (2.5 min) ， because  this  is  serial  create the connection，Why not run multiple threads at the same time, increate the number of the  maximumpoolsize ?
in  this  case of  network  has  delay , the  hikariCP  performance is  not as good as the others, for example：
everytime  when we getConnection，we need check the connection is alive, this  is almost check  everytime  ，
the  value of the param （ aliveBypassWindowMs ）  is  500ms， the code as following, because the  network has delay , the sql execute  time  becomes longer ， many run time exceed 500ms， this will also call the  method to check  the  connection is alive，
the  method valid(connection) has two ways （sql  or  ping） ,if use the  type( select 1) ，also need take a few seconds ,   further increase getConnection  time,
so do we  need consider enlarge the value  aliveBypassWindowMs ?
if (poolEntry.isMarkedEvicted() ||(elapsedMillis(poolEntry.lastAccessed, now) > aliveBypassWindowMs && !isConnectionAlive(poolEntry.connection))) {  }
when the  network has delay , It is difficult to reach the maximun value quickly, and it need to  verify the connection is alive , increase the borrow time ,so the performance degradation, can we consider imporvement from the above points ?

increase the maximumpoolsize of the addConnectionExecutor ， increase one , it will be twice  ，but  the max number connection  may be max+1 . I think this is acceptable


Remove XML functional constructors

Purpose

Disallow compatibility for XML functional constructor syntax.

Fixes #25874
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Two tests of XMLTypeDescContextTest were disabled in this PR and #26189 was created to add the necessary changes.


Fixes #619 by adding a new command: generate-dkim-domain

This command can be used for generating DKIM key for a domain which is not included in any configuration files. For example, when using SQL database for transport maps.

Could you rebase on the new master and push again? There have been improvements on the stability of the tests.
I don't see why the other mail related tests should fail with your changes.

I think it's done and this time build passed.


Sql templates with deployment scripts failed to execute azd up on China Cloud

Describe the issue:
Test todo-csharp-sql/todo-csharp-sql-swa-func templates on China Cloud. Run azd up, get error as follow：

Repro Steps:

Run azd config set cloud.name AzureChinaCloud
Login in with command: azd auth login.
Execute command: azd init -t todo-csharp-sql --branch pr/2897.
Run azd up.

Environment:

Cloud: China.
Template: todo-csharp-sql, todo-csharp-sql-swa-func.
Azd version : 1.6.0-beta.1-pr.3436205 (commit 79ee6dfcae682672d0808d2590b93d298fcba844)

Expected behavior:
Run azd up can pass.
@danieljurek for notification.

This appears to be a quota-related issue, as well.

Same issue about template todo-csharp-sql in china cloud.

https://github.com/Azure/azure-dev/pull/3481

@rajeshkamal5050 Retest with https://github.com/Azure/azure-dev/pull/3481, this issue still exsits.


fix: allow FloatingCard wrapper to have layout modified

Overview

The wrapper of this component has a set display: inline-block which is unfortunate when you want the card to fill an entire width.  Adding the ability to change the display property on the wrapper with prop containerDisplay

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: DISC-653
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

PR Links and Envs
You will need to turn the feature flag on: https://tengu.codecademy.com/admin/test_user?prefill_id=62a3add47b25f875df069e01
Then visit a hub page with https://tengu.codecademy.com/catalog?PR_ENV=portal-app-pr-2108

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.145-alpha.9c823b.0@codecademy/gamut-labs@32.0.8-alpha.9c823b.0@codecademy/gamut@48.0.4-alpha.9c823b.0@codecademy/monaco@0.6.53-alpha.9c823b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62ab838c033a9c21a63a7022--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.145-alpha.c6ee29.0@codecademy/gamut-labs@32.0.8-alpha.c6ee29.0@codecademy/gamut@48.0.4-alpha.c6ee29.0@codecademy/monaco@0.6.53-alpha.c6ee29.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.145-alpha.3c99b6.0@codecademy/gamut-labs@32.0.8-alpha.3c99b6.0@codecademy/gamut@48.0.4-alpha.3c99b6.0@codecademy/monaco@0.6.53-alpha.3c99b6.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62acc256efede208b4817425--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.145-alpha.344187.0@codecademy/gamut-labs@32.0.8-alpha.344187.0@codecademy/gamut@48.0.4-alpha.344187.0@codecademy/monaco@0.6.53-alpha.344187.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.145-alpha.2a3a2b.0@codecademy/gamut-labs@32.0.8-alpha.2a3a2b.0@codecademy/gamut@48.0.4-alpha.2a3a2b.0@codecademy/monaco@0.6.53-alpha.2a3a2b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62acca185e99580b459ed42c--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://62b1f085e6a1806768dab019--gamut-preview.netlify.appDeploy Logs

@Codecademy/gamut @dreamwasp can I get a review?  We'd like to ship the associated portal-app PR soon... thanks!

@Codecademy/gamut @dreamwasp can I get a review? We'd like to ship the associated portal-app PR soon... thanks!

@j10sanders code lgtm - can you link the figma? not sure what visual changes i'm checking for on the portal-side

oops, forgot to push that change!  just did that

The "Related Resources" section needs to fill to the end of the column.

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.145-alpha.bb0e22.0@codecademy/gamut-labs@32.0.8-alpha.bb0e22.0@codecademy/gamut@48.0.4-alpha.bb0e22.0@codecademy/monaco@0.6.53-alpha.bb0e22.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.146-alpha.ecffbe.0@codecademy/gamut-labs@32.1.1-alpha.ecffbe.0@codecademy/gamut@48.0.4-alpha.ecffbe.0@codecademy/monaco@0.6.53-alpha.ecffbe.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62b311006e49e709beb18c62--gamut-preview.netlify.appDeploy Logs


Person: Add followers info in person meta

This was present before for person meta but upon recent version release, it was removed.

Nevertheless, this is present for organization

Any update on this?


key assignment feature

Splits up keys to require separate key file for each chain ID
Changes --keyfile flag to --key-dir, as there can be multiple keys. Keys must be named {chain_id}_share.json for threshold signer mode, or {chain_id}_priv_validator_key.json for single signer mode.
Breaks up cosigner RSA keys into separate rsa_keys.json key file. RSA config is one private key per cosigner, but ed25519 config is potentially many key private key shards per cosigner.

Sharding a priv-validator key is now done with horcrux create-ed25519-shares
RSA keys must be created for the cluster separately with horcrux create-rsa-shares

Adds horcrux config migrate command to migrate cosigner share files and config file from v2 to v3:

Splits up share.json into {chain_id}_share.json (using chain ID from v2 config) and rsa_keys.json.
Removes chain-id from config and writes new config.
Removes old share.json file.

Resolves #135

I'm taking a look. I'll have some comments. Give me a little bit because this is a large PR.

I’m switching to slog for all my new work, since it is on track to be part
of the standard library.
On Tue, May 2, 2023 at 6:14 PM Andrew Gouin @.***>
wrote:

@.**** commented on this pull request.
In cmd/horcrux/cmd/cosigner.go
https://github.com/strangelove-ventures/horcrux/pull/144#discussion_r1183097323
:

@@ -117,21 +117,26 @@ func startCosignerCmd() *cobra.Command {
return err
}

	var (

		// services to stop on shutdown

		services []cometservice.Service

		logger   = cometlog.NewTMLogger(cometlog.NewSyncWriter(os.Stdout)).With("module", "validator")

	logger := cometlog.NewTMLogger(cometlog.NewSyncWriter(os.Stdout)).With("module", "validator")

It would be great to swap out logging to slog or zap in horcrux. I created
#146 https://github.com/strangelove-ventures/horcrux/issues/146 as
follow up
—
Reply to this email directly, view it on GitHub
https://github.com/strangelove-ventures/horcrux/pull/144#discussion_r1183097323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACG6V6SXHECQIOPD4OVNLTXEGBLLANCNFSM6AAAAAAXSNT6ZA
.
You are receiving this because your review was requested.Message ID:
@.***>

I missed this, but we save keys into 1 dir and name files with a chain-id suffix?

Sharding will create a dir for each cosigner. The files can be copied directly to the relative cosigners without renaming any files now.
$ ls
priv_validator_key.json

$ horcrux create-ed25519-shares --chain-id cosmoshub-4 --key-file priv_validator_key.json --threshold 2 --shares 3
Created Ed25519 Share cosigner_1/cosmoshub-4_share.json
Created Ed25519 Share cosigner_2/cosmoshub-4_share.json
Created Ed25519 Share cosigner_3/cosmoshub-4_share.json

$ horcrux create-rsa-shares --shares 3
Created RSA Share cosigner_1/rsa_keys.json
Created RSA Share cosigner_2/rsa_keys.json
Created RSA Share cosigner_3/rsa_keys.json

$ ls -R
.:
cosigner_1  cosigner_2  cosigner_3 priv_validator_key.json

./cosigner_1:
cosmoshub-4_share.json  rsa_keys.json

./cosigner_2:
cosmoshub-4_share.json  rsa_keys.json

./cosigner_3:
cosmoshub-4_share.json  rsa_keys.json


Updated Public IP SKU details

Updated Public IP SKU to say only Standard is supported.

@ashjain : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


purge gitter mentions

as per the agreement at CC meeting on 8/3/22, I have purged all references I could find to Gitter.

I would actually point to slack for anything developer related.
I would also add references to the community collaboration sessions on Fridays, where appropriate.

I'm happy for you to add anything you want, I was just trying to action the removal of gitter, that is all this PR is meant to do

I'm happy for you to add anything you want, I was just trying to action the removal of gitter, that is all this PR is meant to do

Well, if we're already doing it, imo, let's do it "right" and funnel new contributors towards slack primarily.

I'm happy for you to add anything you want,

Do you mind accepting the suggestions? They replace discord by slack whenever developer content is the context.

oops, something is broken now, let me check

?? seems like there is no error message, readthedocs build just didn´t start. Just restart it?


How is sprite tinting implemented and can I calculate a tint from an original and expected value?

I took a look at SpriteRenderer and saw that the tint value gets added to the buffer, but I don't understand what this means functionally.  Any hints or links helping me understand what is happening in that class would be great, but also, can anyone point out how the tint is being applied?
More specifically, I am trying to make a "brighten effect" by using a reverse tint.  For example, I have a sprite that is of color 0x00838f.  In a graphics editor make the sprite "20%" brighter, so the color is now 0x339CA5.  In pixi, I then want to apply a tint to the sprite, of value X, so the resulting tinted color is the original 0x00838F.  I'll remove the tint when I want to make the sprite brighter.
Is it possible to do this?  If so, how can I calculate what X would be?  If it's not possible, is it possible to calculate it the other way round, where I know the tint value before hand I use that to calculate what color I should make the brightened sprite?

there's no "20% brighter" tint, it can only go down.

I'm aware of that, that's why i'm proposing to do it in reverse...

Tint is applied in the shader: https://github.com/pixijs/pixi.js/blob/master/src/core/sprites/webgl/generateMultiTextureShader.js#L14
The tint color is added to the buffer, uploaded as an attribute, and passed to the fragment shader via a varying. We then multiply the texture color by the tint color.
To figure out what you should pass as the tint, first think about what happens when you multiply colors in glsl. First of all colors are represented as normalized vec4s. SO lets do that version real fast:
// note: I am rounding a bit
0x00838f = vec4(0.00, 0.51, 0.56, 1.00);
0x339CA5 = vec4(0.20, 0.61, 0.65, 1.00);

Now you can see what we do is a vec4 * vec4 which is a component-wise multiplication. So, the formula you are solving is:
originalColor * tintColor = finalColor;
tintColor = finalColor / originalColor;

So,
// 0x339CA5 (final) / 0x00838f (original)
vec4(0.20, 0.61, 0.65, 1.00) / vec4(0.00, 0.51, 0.56, 1.00);

As you can see, there is a green and blue value that will make this work; but there is no red value you can pass to tint your 0x00 red channel to be 0x33. Obviously, anything multiplied by 0 is 0. But hopefully this gets you on the right path with how to do calculate your required tint color.
You can achieve your desired effect by changing your original color to more of a 0x01838f, so you have a red channel to multiply against. That being said, ColorMatrixFilter might be more what you are looking for so I would experiment with that as well.
Hope this helps!

Most of the problem I was having was due to not really understanding the vector math, or what component-wise multiplication was.  I ended up writing a pen to calculate either the tint or the bright colour required.

This also led me to determine that the bright colour we were specifying as a 20% brighter shade was being mixed differently to how pixi would have applied a tint (ie; the presence of the red channel).
Extremely helpful @englercj, most appreciated!


Update splinter setup.py on pypi

Needs to update setup.py dependencies on splinter version that is stored at pypi.
That version on pypi is requiring selenium==2.5.0
O.o
Thanks.

Just got hit by this too.  Selenium < 2.12 doesn't support Firefox 8.  I upgraded some packages and ended up downgrading my Selenium from 2.13.1 to 2.5.0 because of the splinter requires.txt.  Can it be updated to say >=2.5.0?

In master branch, I updated selenium support to 2.13.1. I will make the splinter release with this.

We'll try to release splinter next weekend :)

fixed in the release 0.4.1

Awesome, thanks guys!

Wow, nice news.
Thanks!


enhance information output

Even with debuglevel=debug, it doesnt say a whole lot about what is going on. This issue is to track improving the normal on screen information that is output, as well as adding to the debug output.

does debuglevel=trace give you the information that you are looking for?
What info are you looking for in particular?

Speaking from my own experience, I think a lot of the debug/trace lines should be info.  It's not at all clear right now what calculations/decisions are being performed without using trace.

Information should be shown to the user about the amount of funds being spent on transaction fees.

I see "[TRC] TKBY: All tickets have been purchased, aborting further ticket purchases".  This could be standard output or at least debug, and it should give further information about how many tickets were purchased and the total targeted.

[TRC] TKBY: Scaled fee is 0.4100305345454546, but max fee is 0.05; using max
[DBG] TKBY: Mean fee for the last blocks or window period was 0.4100305345454546; this was scaled to 0.05
These are redundant.

[INF] TMGR: Inserting unconfirmed transaction d....d
[INF] WLLT: Successfully sent SStx purchase transaction d...d
It tells me that transactions are being sent, but says nothing about the fees being used.

TKBY: Purchased ticket d...d at stake difficulty 24.75565885 (0.08343431 fees per KB used)
Would be helpful if it also showed the actual fee paid and not just the fee per KB.

[TRC] TKBY: All tickets have been purchased, aborting further ticket purchases
This message should be changed because as soon as new funds come into the wallet, it will again start purchasing tickets. So the msg should be more like, 'Not enough funds to purchase more tickets.'

TKBY: Aborting purchasing of tickets because our balance after buying tickets is estimated to be 10.99135837 but balance to maintain is set to 0
Thats not even a valid msg. 10.9 dcr isnt enough to buy a ticket.

It should show information about the pre-sstx tx that makes the outputs the right size. Show information such as, that it is doing it and how much is is spending on tx fees.

When it says
[INF] TKBY: Purchased ticket xxxxxxxx at stake difficulty 43.54 (0.04432 fees per KB used)
I would like it to show the actual fee used in addition to just the fees per KB.

[TRC] TKBY: Aborting purchasing of tickets because our balance after buying tickets is estimated to be 0.08572371 but balance to maintain is set to 0
That does not make any sense. The correct message should be 'not enough funds to make another purchase'


fix: delete an empty line in mod list

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

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

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 54.347%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
9096

Relevant Lines:
16737

💛  - Coveralls


Show fancy field name in same rule

Add a basic rule that matches the value of the current field with the value of another field

ok I realized there is a same rule with is exactly that, the bad thing here is that only the currently-tested-field shows a good fancy name/label but the field to test against shows the name. both fields should show a fancy name.


Leaderboard UI

Update style.css file
Update README.md file
Run linter for errors

Thank you @frank1738


Update DG

Fix issues in #150 and #163
Updated all the sequence diagrams to be consistent

Codecov Report
Base: 70.20% // Head: 70.20% // No change to project coverage :thumbsup:

Coverage data is based on head (601aaf8) compared to base (e11c7f2).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #212   +/-   ##
=========================================
  Coverage     70.20%   70.20%
  Complexity      642      642
=========================================
  Files           100      100
  Lines          2014     2014
  Branches        221      221
=========================================
  Hits           1414     1414
  Misses          531      531
  Partials         69       69

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add ability to disable anti-aliasing.

You should add a button to deactivate the anti-aliasing, for small images/pixel-arts, it can really mess up the rendering.

intentionally, I did not do this function, I will try to turn it off

intentionally, I did not do this function, I will try to turn it off

If possible, I will add to the quick lens adjustment menu (image context menu) so far only in the settings

If possible, I will add to the quick lens adjustment menu (image context menu) so far only in the settings


Add .editorconfig file

The .editorconfig file  is compatible with most development environments and is used to setup project-based consistent text editing style. For this repo, gofmt already handles this for the .go files. The .editorconfig file is for consistency in the the config.json file. This would solve the tabs/spaces problem (see #714) automatically for anyone with an editorconfig aware environment.
Closes #722

This is relevant: https://github.com/exercism/configlet/issues/18

Ok let's kill this for now and I will add it  to .gitignore so if people want to use it locally they can.

only solves the problem for those who use it, which I don't,

@tleen re-reading this, I realise my comment sounds awful, as if I'm saying I don't use it therefore it's no good! This isn't what I meant, sorry! More that I'd like to see a solution for the repo rather than for a subset of it's users.

LOL that's not how I read it I get what you were saying. Agreed it would probably be best checked/reformatted at the configlet level for consistency. I'm just terribly embarrassed I submitted tab indenting to a space indented file! I've become over-reliant on things like editorconfig and don't think about it much anymore. We can put a note in the README for maintainers about the preference and point them in the editorconfig direction. If we somehow get more stuff needing formatting then it may make sense to revisit.
All in all this keeps showing why gofmt is so awesome!


adding a loaded callback so I can set an initial scence

@yawetse thanks for this PR. I have released the new version, and this no longer needed.


Disassembly does not show until call stack is clicked

Since some time, when "Show disassembly" is configured to "always", the disassembly listing tab does not show until I explicitly click on some call stack function. Stepping in also does not show the disassembly listing tab. When I step in an instruction that performs a call, the disassembly does not switch to the callee disassembly, and again, the disassembly listing tab only shows when I explicitly click on the newly-called function in the call stack.
The previous, correct behavior, was that disassembly listing tab appeared the moment I switch "Show disassembly" to "always". Stepping into a call instruction also switched the disassembly listing to the callee. All without me touching the call stack view.
OS: Debian trixie.
VSCode version:  1.90.0
CodeLLDB version: 1.10.0
Compiler: rustc 1.78.0
Debuggee: x86_64-linux-gnu
log.log

Fixed in v1.11.1

@vadimcn thank you!


🐛 BUG: Wrangler2 does not load dist/_worker.js unless you are running wrangler directly in dist directory

What version of Wrangler are you using?
2.0.23
What operating system are you using?
Linux
Describe the Bug
I copied the documentation _worker.js into a dist directory:
export default {
  async fetch(request, env) {
    const url = new URL(request.url);
    if (url.pathname.startsWith('/api/')) {
      // TODO: Add your custom /api/* logic here.
      return new Response('Ok');
    }
    // Otherwise, serve the static assets.
    // Without this, the Worker will error and no assets will be served.
    return env.ASSETS.fetch(request);
  },
};

When running wrangler pages publish ./dist --project-name polykey-docs, the worker does not get uploaded.
Whne running wrangler pages dev ./dist the worker also does not work.
However if I run cd dist && wrangler pages publish ./ --project-name polykey-docs or cd dist && wrangler pages dev ./ then the _worker.js actually does work.

Hi there. Trying this on the latest version of wrangler (2.0.24), I am unable to reproduce this.
Could you provide a minimal reproducible sample if you still experience this on the latest version?

Stopped using pages and went back to workers.


Add example SEDCMD to mask data in position of passwords

The Sysmon data set picks up a lot of passwords in CommandLine and ParentCommandLine fields.   It is very difficult to define a regular expression which identifies passwords with 100% accuracy.  Instead, a Splunk admin can implement a strategy of replacing text in known positions of passwords.   Add an example of use of SEDCMD in props.conf to mask data in known positions of passwords.

I don't think this should be in the default settings of the app. You might
want your analysts to know what passwords someone might be using to run
scripts, as it could assist with attribution, response, and legal action in
the even of misuse or fraud.
I don't mean to get off on a rant here, but...
Instead of masking the passwords used in your scheduled tasks, etc. , you
should probably find a better way of running your scripts. Once an
adversary pops one of your boxes, they'll get that password when they
enumerate scheduled tasks or when they search through the logs to find if
the sysadmins are that sloppy.
Also, I hope that the password you're concerned about is not the local
admin password that's used throughout the environment. Because that's day
one stuff.
On Dec 12, 2016 10:47 PM, "dstaulcu"<EMAIL_ADDRESS>wrote:

The Sysmon data set picks up a lot of passwords in CommandLine and
ParentCommandLine fields. It is very difficult to define a regular
expression which identifies passwords with 100% accuracy. Instead, a Splunk
admin can implement a strategy of replacing text in known positions of
passwords. Add an example of use of SEDCMD in props.conf to mask data in
known positions of passwords.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/splunk/TA-microsoft-sysmon/issues/17, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AJuwfT2ABRtzPXtlb9ok9MUhMrSYh53cks5rHhU_gaJpZM4LLT0k
.

Different organizations will have different policies on how to treat sensitive data in Splunk.
For those organizations, which don’t want their Splunk analysts to come across it so easy, it would be nice to have an example of how to handle the situation.   Perhaps a compromise would be to include an example but to have it commented out by default.
I don’t foresee a single entry (regular expression) which would handle all passwords, so providing a sample transform is mostly about getting folks started.
David
From: MikeKemmerer<EMAIL_ADDRESS>Sent: Monday, December 12, 2016 11:58 PM
To: splunk/TA-microsoft-sysmon<EMAIL_ADDRESS>Cc: dstaulcu<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [splunk/TA-microsoft-sysmon] Add example SEDCMD to mask data in position of passwords (#17)
I don't think this should be in the default settings of the app. You might
want your analysts to know what passwords someone might be using to run
scripts, as it could assist with attribution, response, and legal action in
the even of misuse or fraud.
I don't mean to get off on a rant here, but...
Instead of masking the passwords used in your scheduled tasks, etc. , you
should probably find a better way of running your scripts. Once an
adversary pops one of your boxes, they'll get that password when they
enumerate scheduled tasks or when they search through the logs to find if
the sysadmins are that sloppy.
Also, I hope that the password you're concerned about is not the local
admin password that's used throughout the environment. Because that's day
one stuff.
On Dec 12, 2016 10:47 PM, "dstaulcu" <notifications<EMAIL_ADDRESS>wrote:

The Sysmon data set picks up a lot of passwords in CommandLine and
ParentCommandLine fields. It is very difficult to define a regular
expression which identifies passwords with 100% accuracy. Instead, a Splunk
admin can implement a strategy of replacing text in known positions of
passwords. Add an example of use of SEDCMD in props.conf to mask data in
known positions of passwords.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/splunk/TA-microsoft-sysmon/issues/17, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AJuwfT2ABRtzPXtlb9ok9MUhMrSYh53cks5rHhU_gaJpZM4LLT0k
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/splunk/TA-microsoft-sysmon/issues/17#issuecomment-266641569, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGgv2rOSgAsm8EigNQWQZf6fA5zHufRMks5rHiXUgaJpZM4LLT0k.

Good discussion. Can't thank you both enough for your ongoing contributions and discussion.
Every org and situation will be different. We'd never be able to include such config directly in the add-on by default, however we could certainly add some directions in the README to serve as a jumping off point for those interested. We could also refer to some existing documentation on the broader topic of anonymizing data in Splunk. Finally we could even write a Splunk blog post on this topic showing how we modified stock Sysmon TA per instructions in README to attempt to mask passwords.
@dstaulcu can you propose appropriate changes to the README?

Sounds good.
Problem is only going to get worse as folks adapt to thishttps://www.microsoft.com/en-us/download/details.aspx?id=52630.  Search for “command line”.  I like the idea of providing a per-source type (Sysmon, Security log, etc) approach and then let the community contribute to a library of knows.
From: Dave Herrald<EMAIL_ADDRESS>Sent: Wednesday, December 14, 2016 4:56 PM
To: splunk/TA-microsoft-sysmon<EMAIL_ADDRESS>Cc: dstaulcu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [splunk/TA-microsoft-sysmon] Add example SEDCMD to mask data in position of passwords (#17)
Good discussion. Can't thank you both enough for your ongoing contributions and discussion.
Every org and situation will be different. We'd never be able to include such config directly in the add-on by default, however we could certainly add some directions in the README to serve as a jumping off point for those interested. We could also refer to some existing documentation on the broader topic of anonymizing data in Splunk. Finally we could even write a Splunk blog post on this topic showing how we modified stock Sysmon TA per instructions in README to attempt to mask passwords.
@dstaulcuhttps://github.com/dstaulcu can you propose appropriate changes to the README?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/splunk/TA-microsoft-sysmon/issues/17#issuecomment-267169277, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGgv2jJVnr5_OThFmsqFxn85mbrtXDpVks5rIGX3gaJpZM4LLT0k.

This is where the MITRE CAR data model (
https://car.mitre.org/wiki/Data_Model) might come in handy for inspiration.
As long as you have a specific action on an object (like process create)
you just need to alias the field names to fit the CIM or your own data
model and develop all of your analytics to your data model. It doesn't
matter what your sources are. Whether it's security logs, sysmon logs, or
carbon black logs, your analytics will work and you can share then with
others.
On Dec 14, 2016 7:37 PM, "dstaulcu"<EMAIL_ADDRESS>wrote:
Sounds good.
Problem is only going to get worse as folks adapt to this<
https://www.microsoft.com/en-us/download/details.aspx?id=52630>. Search for
“command line”. I like the idea of providing a per-source type (Sysmon,
Security log, etc) approach and then let the community contribute to a
library of knows.
From: Dave Herrald<EMAIL_ADDRESS>Sent: Wednesday, December 14, 2016 4:56 PM
To: splunk/TA-microsoft-sysmon<EMAIL_ADDRESS>Cc: dstaulcu<EMAIL_ADDRESS>Mention <
<EMAIL_ADDRESS>Subject: Re: [splunk/TA-microsoft-sysmon] Add example SEDCMD to mask data
in position of passwords (#17)
Good discussion. Can't thank you both enough for your ongoing contributions
and discussion.
Every org and situation will be different. We'd never be able to include
such config directly in the add-on by default, however we could certainly
add some directions in the README to serve as a jumping off point for those
interested. We could also refer to some existing documentation on the
broader topic of anonymizing data in Splunk. Finally we could even write a
Splunk blog post on this topic showing how we modified stock Sysmon TA per
instructions in README to attempt to mask passwords.
@dstaulcuhttps://github.com/dstaulcu can you propose appropriate changes
to the README?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://github.com/
splunk/TA-microsoft-sysmon/issues/17#issuecomment-267169277>, or mute the
thread<https://github.com/notifications/unsubscribe-auth/AGgv2jJVnr5_
OThFmsqFxn85mbrtXDpVks5rIGX3gaJpZM4LLT0k>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/splunk/TA-microsoft-sysmon/issues/17#issuecomment-267201585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJuwfZG5HYtIYmQoGMdJYFiz-U8K4AOSks5rIIvegaJpZM4LLT0k
.


Configurable texture filtering in materials asset pipeline

I have some pixel art textures I'm loading using the materials pipeline (a la the image example) but they're drawn with linear filtering by default. Can we add the option for nearest-neighbor filtering in the material pipeline?

One for you to tackle if you have some free time, @VinhTruongAmbient ?
I think you'd have to extend this with an option for a sampler:
https://github.com/AmbientRun/Ambient/blob/ae54f3e0fbff4b6a7b7c093ebd10441d6cb226a2/crates/build/src/pipelines/materials/mod.rs#L115-L152
and then wire it up here:
https://github.com/AmbientRun/Ambient/blob/ae54f3e0fbff4b6a7b7c093ebd10441d6cb226a2/crates/renderer/src/materials/pbr_material.rs#L443
@marceline-cramer would a simple Linear/NearestNeighbor sampler choice be sufficient, or would you want full control over the sampler? The latter is better in the long run but it might require more work with asset keys/the cache.

This should be a simple thing to add. While we are at it we should also add wrap modes to be complete.

@marceline-cramer would a simple Linear/NearestNeighbor sampler choice be sufficient, or would you want full control over the sampler? The latter is better in the long run but it might require more work with asset keys/the cache.

It would be sufficient, but there's no rush to get this in. I'll need it eventually but in my opinion it's better to prioritize the long run.

My #440 solved this in the most simple way possible. I created a follow-up issue #443 that tackles arbitrary texture-sampler pairs, which is going to be a more involved project.


Network and Fetch domain activation

Currently, the network domain is incorrectly enabled.
The network domain should be enabled via the CDP client/target, only when the browsing context has...

...subscriptions to network. events, and/or
...global subscription to the network namespace

It's important to not do Network.enable globally, otherwise we'd emit too many unnecessary CDP events, when the client may not be interested in them.
Edit(2023-09-11): @Lightning00Blade: It seems that if we never enable interception we always call Fetch.disable on target create, I think it would be good it we omit that that way its a little bit faster for WPT and other test.
cc @Lightning00Blade

cc @sadym-chromium FYI as this slightly relates to supporting multiple BiDi/CDP sessions.

ack

Fetch is partially resolved via #1409.

This should be resolved and we should be enabling both Network and Fetch only when needed.


Update Private Cloud documentation for new release

For release at the same time as #5126

Approved by Nidhi over Slack.

Publication agreed with Nidhi


Ambigous toggle buttons in Email Settings

Issue Description
Inorder to disable or enable notification emails, the user is expected to toggle the options in the Email Settings page. However the toggle buttons are ambigous and does not clearly convey whether the On or Off implies that the emails are currently enabled or disabled. The fact that the unsubscribe links from emails errors out when not logged in does not help matters much either.

Suggested Solution
Buttons with text like Enabled/Disabled with suggestive colors might be the more user friendly way to go.

I'll take up this.

@erictleung How about using Yes/No instead of Enabled/Disabled?

@prateekgoel yes/no sounds more intuitive :+1:
cc/ @QuincyLarson tagging you for comment on UI/UX changes :smile:

@erictleung Great. 😄
@QuincyLarson Let me know if you have any thoughts regarding this.

@erictleung @prateekgoel I agree that this will be more clear.
I like how Medium does this. It's monochrome and would jive with freeCodeCamp's visual design. It also allows for more than an on/off state.

@prateekgoel would you be interested in updating this on staging? Also, could you Reactify this page? We'd like to overhaul our profile and settings pages to use components instead of Pug views.

@prateekgoel With Yes/No or On/Off, the user will not be able to understand the current state. If we go with one of those, it wouldn't be clear whether On implies Notifications are currenly enabled or clicking will enable notifications.
Hence I went for something like Enabled (if the button implies that notifications are currently enabled).  And if the button changes state to enable notifications, then the text should be  Enable
@QuincyLarson My way sucks. This would be the right way to go!

@QuincyLarson I wanted to help but I am new to React. So i think it would be good if someone else take this.

@QuincyLarson I will take this up. I will start by converting the email settings page to react components and change the buttons to an On/Off toggle setup if that is okay

@ZacharyKearns you can check for ToggleButtonGroup and ToggleButton in react-bootstrap.
Note: we are currently using react-bootstrap v0.30.5 but ToggleButtonGroup and ToggleButton have been exported for public use only in v0.31.2

@vs1682 v0.30.5 does not have ToggleButtonGroup or ToggleButton. React-bootstrap would need to be upgraded or a custom component would have to be made.

@ZacharyKearns right, that is what I mentioned in my earlier comment.

tagging @BerkeleyTrue for thoughts on being able to upgrade react-bootstrap?

@dhcodes that version is currently within our semvar range.

Submitted a PR to update react-bootstrap 🎉 . You'll need to add ToggleButtonGroup and ToggleButton to the import on the Settings.jsx file when you go to make this change.

Nice.
@ZacharyKearns your turn.

@vs1682 @dhcodes Okay great. I am just doing a coding challenge for a potential new job. I can start on this right after that.

I have started working on this. Here a screenshot of what I have so far.

They do break at certain screen widths

Still working on making them functional

@ZacharyKearns Awesome! Would <ToggleButtonGroup> styles have a display: block fix the breaking issue?

@dhcodes I tried add block={ true } to the ToggleButtonGroup component with no luck. It appears that it's column gets to small which causes it to break. I am experimenting with different column layouts.

@ZacharyKearns You can also try changing bsSize on <ToggleButtonGroup> for smaller screen widths.
Note: All buttons on this page have large size. Changing bsSize will make <ToggleButton> appear smaller than others. We should confirm if we can do that.

@vs1682 Changing them to a smaller size did not stop them from breaking unfortunately.
@BerkeleyTrue Here are screenshots for the final version as well as the css that was added to the main.less file and one of the ToggleButtonGroup's. The new code passed all the tests.
Just want to note that there seems to be a major issue with the layout of the settings page.
Because the content is not wrapped in a traditional bootstrap container the negative margins on the rows are causing the elements to be 30px wider than the width of the viewport which is causing horizontal scrolling.
.email-settings {
  @media (max-width: 768px) {
    text-align: center;
  }
}

.toggle-btn-group {
  float: right;
  @media (max-width: 768px) {
    float: none;
    .btn-toggle {
      margin-top: 15px;
      margin-bottom: 25px;
    }
  }
}

<ToggleButtonGroup
  className='toggle-btn-group'
  name='montly-email'
  onChange={ toggleMonthlyEmail }
  type='radio'
  >
  <ToggleButton
    bsSize='lg'
    bsStyle='primary'
    className={
      classnames(
        'positive-20',
        { active: sendMonthlyEmail },
        'btn-toggle'
      )
    }
    type='radio'
    value={ 1 }
    >
    On
  </ToggleButton>
  <ToggleButton
    bsSize='lg'
    bsStyle='primary'
    className={
      classnames(
        'positive-20',
        { active: !sendMonthlyEmail },
        'btn-toggle'
      )
    }
    type='radio'
    value={ 2 }
    >
    Off
  </ToggleButton>
</ToggleButtonGroup>

This view is for tablets and desktops

This view is for smartphones

If there are any problems please let me know before I issue a pull request.
This is my first contribution so be gentle haha.

@ZacharyKearns Nice work! If adding a wrapping makes the layout better, I'd say you might as well make that change in your PR too.

Thanks @dhcodes
Adding a div with the container class seems to fix the horizontal scrolling so I will add it to the PR.
There is still work to be done on this page.

The job search settings button is also broken. Maybe that should be it's own issue?


[EventRequest] Codeunit 781 "Sales Pipeline Chart Mgt.".DrillDown

Please add new events
procedure DrillDown(var BusinessChartBuffer: Record "Business Chart Buffer"; var SalesCycleStage: Record "Sales Cycle Stage")
var
    OppEntry: Record "Opportunity Entry";
begin
    if SalesCycleStage.FindSet then begin
        SalesCycleStage.Next(BusinessChartBuffer."Drill-Down X Index");
        OppEntry.SetRange("Sales Cycle Code", SalesCycleStage."Sales Cycle Code");
        OppEntry.SetRange("Sales Cycle Stage", SalesCycleStage.Stage);
        //-------------------------------OnBeforeDrillDown:BEGIN
        OnBeforeDrillDown(OppEntry);
        //-------------------------------OnBeforeDrillDown:END
        PAGE.Run(PAGE::"Opportunity Entries", OppEntry);
    end;
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeDrillDown(var OppEntry: Record "Opportunity Entry")
begin
end

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Minor update in README

This p/r is to do a minor update in root and DSS README files.

Merged. Thanks!


Sword texture is backwards when blocking

In both the 1.16 and 1.17 versions, the sword texture is flipped when blocking.
You can see this easily by looking at which side of the sword is light vs dark, and it's even more apparent with certain modded swords.

Vuld sword from Hellish Materials. You can see the glob of stuff is sticking out of the wrong side when blocking:

Interestingly, this problem is also present in the Sword Blocking mod.

Compare this to 1.7, where the dark side of the sword stays on the right:


License Key: where get it?

Hi Guys,
On DOCs says: "Before develop Badaso, please get BADASO_LICENSE_KEY by register on Badaso or contact badaso core team. This key must be included in the laravel project's .env. Steps for registering and getting a license on Badaso Dashboard can be found on Badaso Docs."
But, i don't find it on site OR how to get it.
How to get the LICENCE KEY?
Thanks!

Now you don't need a license key to use badaso


Smalltalk expansion

see:
https://github.com/hpi-swa/smalltalkCI/issues/106
also waiting for:
https://github.com/travis-ci/travis-build/pull/689
https://github.com/hpi-swa/smalltalkCI/pull/113

@BanzaiMan this is also now ready to merge. Thanks!


Can write Markdown with rich text like GitLab issue?

Related to Improve_websites_thanks_to_open_source/issues/370.
As a workaround can use MarkText.
+112
Feature Description
Not read.
Screenshots
No response

Related to Benjamin_Loison/discord/issues/12.


Add support for Gradle version catalog to determine ktlint version

Would love to be able to programmatically set our ktlint version based on a value in our dependencies.toml file (version catalog)

Closing. Configuration of the plugin should not be mixed with configuration of other build tools. It would also introduce complexity in case the plugin would not be used with a Gradle build.


Basic Treatee CRUD

I want to be able to perform CRUD operations on a treatee

The following Stories belong to this epic:
https://github.com/tuco80/healthacta/issues/2


New functionality for DSU fabric, batch screen

PL user story: n/a
Description:
Changes to the user Interface for DSU fabric:

Add the following check marks to the screen (new functionality):

Acceptance criteria's:
positive tests

create a batch with SN, but mark no serial number check. Verify if the barcode with the wrong serial number creates no error message, because system should deactivate the SN check
repeat for the other check marks

negative tests / error path testing
n/a
Process scenario:
tbd
Additional information:

@steinso2 this references step 5 but step 5 is not described. Suggest to include the complete set of screen shots/ descriptions so that the developers see the bigger picture.

@maherpa2 -

Should we keep all the 7 & 8 options checked by default. These checks are pretty much mandatory?
Should differentiate 7 & 8 on the product status file with below error messages. The message can be shown when the user clicks on the product status.
for 7 - "The expiry date on the scanned drug pack is not invalid and not as per the expiry date set by the Manufacturer."
for 8 - "The scanned drug pack is expired".


Version 7.31 breaks REST::Neo4p

Mojolicious version: 7.31
Perl version: 5.18.2 (Ubuntu) / 5.10.1 (CentOS 6)
Operating system: Ubuntu 14.04.5 LTS (trusty) / CentOS release 6.8 (final)

Steps to reproduce the behavior
perl -MCPAN -e"install REST::Neo4p" without Mojolicious installed (pulls 7.31 as required)
Expected behavior
REST::Neo4p should install without errors.
Actual behavior
Reading '/u0/arobinso/.cpan/Metadata'
Database was generated on Tue, 23 May 2017 12:41:02 GMT
Running install for module 'REST::Neo4p'
Running make for M/MA/MAJENSEN/REST-Neo4p-0.3012.tar.gz
Checksum for /u0/arobinso/.cpan/sources/authors/id/M/MA/MAJENSEN/REST-Neo4p-0.3012.tar.gz ok
CPAN.pm: Building M/MA/MAJENSEN/REST-Neo4p-0.3012.tar.gz
Server for live tests: [http://<IP_ADDRESS>:7474 ]http://<IP_ADDRESS>:7474
Username: [ ]Pass: [ ]Created MYMETA.yml and MYMETA.json
Creating new 'Build' script for 'REST-Neo4p' version '0.3012'
Building REST-Neo4p
MAJENSEN/REST-Neo4p-0.3012.tar.gz
./Build -- OK
Running Build test
t/0010_load.t ............... ok
t/0011_parse.t .............. ok
t/002_agent.t ...............
Dubious, test returned 255 (wstat 65280, 0xff00)
All 17 subtests passed
t/003_objs.t ................ ok
t/0041_idx_create_unique.t .. ok
t/004_idx.t ................. ok
t/0051_handle.t ............. ok
t/005_db.t .................. ok
t/006_query.t ...............
Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 1/43 subtests
t/007_accessors.t ........... ok
t/008_batch_agent.t ......... ok
t/009_batch.t ............... ok
t/010_batch_synopsis.t ...... ok
t/011_neo4p_synopsis.t ...... ok
t/020_constraint.t .......... ok
t/021_validation.t .......... ok
t/022_constrain.t ........... ok
t/023_constrain_synopsis.t .. ok
t/030_idx_escape.t .......... ok
t/040_v2_labels.t ........... ok
t/041_v2_txns.t .............
Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 19/29 subtests
t/050_v2_schema.t ........... ok
t/097_parse_errors.t ........ ok
t/098_pod.t ................. skipped: Test::Pod 1.00 required for testing POD
t/099_error.t ............... ok
t/agent_mod.t ............... ok
t/issue_5.t ................. ok
t/rt_80150.t ................ ok
t/rt_80196.t ................ ok
t/rt_80471.t ................ ok
t/rt_81128.t ................ ok
t/rt_90072.t ................ ok
t/rt_91640.t ................ ok
t/rt_91748.t ................ ok
t/rt_92135.t ................ ok
t/rt_96847.t ................ ok
Test Summary Report
t/002_agent.t             (Wstat: 65280 Tests: 17 Failed: 0)
Non-zero exit status: 255
Parse errors: No plan found in TAP output
t/006_query.t             (Wstat: 65280 Tests: 43 Failed: 1)
Failed test:  30
Non-zero exit status: 255
t/041_v2_txns.t           (Wstat: 65280 Tests: 10 Failed: 0)
Non-zero exit status: 255
Parse errors: Bad plan.  You planned 29 tests but ran 10.
Files=36, Tests=1868, 28 wallclock secs ( 0.27 usr  0.05 sys +  9.80 cusr  1.11 csys = 11.23 CPU)
Result: FAIL
MAJENSEN/REST-Neo4p-0.3012.tar.gz
./Build test -- NOT OK
//hint// to see the cpan-testers results for installing this module, try:
reports MAJENSEN/REST-Neo4p-0.3012.tar.gz
Running Build install
make test had returned bad status, won't install without force
[root@vector REST-Neo4p-0.3012-D4I_ep]#
RESOLUTION
Use cpanm to remove Mojolicious 7.31 and install version in Ubuntu repository (4.63)

There were some changes that CPAN modules had to make given deprecated/removed functionality around this time. Those changes were made per the Mojolicious deprecation policy. Therefore I don't think this is a bug in Mojolicious and I recommend filing a bug on the module in question.