Get LoPs in Maze?

When I call "/api/runs/maze/"+runId+"?populate=true"
I do not get any LoPs (at least not before I have saved it). Should be 0 when not judged the run before.

LoPs seems to work in maze now...


Existing playlist overwrites with nothing if fetching fails

Will fix it in the next few days

Closes with https://github.com/SCP002/m3u-gen-ttv/commit/c136ad97eb4b1f9a4561ba68d97d94460ceb9417


Color Picker: hex color format settings

Currently, Color Picker can only output colors in uppercase 6-digit format. This may be in conflict with linting rules for a project.
I think this should be configurable through settings:

uppercase/lowercase;
short/long format for colors like #ccc.

#31751


Fixed mobile apps form app_type input if it can't be changed

Fixes #9376
Set the input to disabled in case it cannot be changed, so it won't be sent to the controller.
CR @javitonino

Does this change the form appearance?
Also, now that we are at it, better to also do it for platform.

The form design remains as it was. And it's not necessary change it for platform, as it has already logic to set it as disabled in those cases.

:+1:, ship it then!

✨


DOC: Add note on params to autogreek

Codecov Report

Merging #293 (6cb77a8) into develop (e4880b7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop      #293   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           43        43
  Lines         1456      1456
=========================================
  Hits          1456      1456

Impacted Files
Coverage Δ

pfhedge/autogreek.py
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 e4880b7...6cb77a8. Read the comment docs.


Remove 'reset' secret from emulator for 4.1

See #871
For 4.1, let's just drop the ability to be able to reset/ renew the secret from the emulator. Customers can be pointed to and use msbot cli.

In progress

@vishwacsena This change effects the Getting Started doc too. We should remove:
Choose 'Encrypt keys stored in Bot Configuration' if you would like your .bot file to be encrypted. This is recommended especially if you have keys and other sensitive information in your bot file. CAUTION: If you choose this option, Emulator will encrypt the .bot file and give you a key. You need to keep this key in a secure place (e.g. Azure Key Vault). See here to learn more about bot file encryption.
and update the screenshots and replace with a reference to the MsBot CLI. (We are not adding the reference to msbot cli to the modal.)

Clarification after discussion with Vishwac: We are only removing the single button that says "Generate new secret key" from the bot creating dialog and settings editor.


CI: Travis: ENISuite.TestNodeManagerManyNodes fails

CI failure
level=info msg="Synchronized ENI information" numInstances=100 numSecurityGroups=2 numSubnets=4 numVPCs=1 subsys=eni

----------------------------------------------------------------------

FAIL: node_manager_test.go:563: ENISuite.TestNodeManagerManyNodes
node_manager_test.go:602:

    c.Errorf("Node %s allocation mismatch. expected: %d allocated: %d", s.name, minAllocate, node.Stats().AvailableIPs)

... Error: Node node53 allocation mismatch. expected: 10 allocated: 18

node_manager_test.go:602:

    c.Errorf("Node %s allocation mismatch. expected: %d allocated: %d", s.name, minAllocate, node.Stats().AvailableIPs)

... Error: Node node59 allocation mismatch. expected: 10 allocated: 18

node_manager_test.go:617:
    c.Assert(metricsapi.AllocatedIPs("available"), check.Equals, numNodes*minAllocate)

... obtained int = 1016
... expected int = 1000

...
OOPS: 17 passed, 1 FAILED
--- FAIL: Test (2.60s)
FAIL
coverage: 4.3% of statements in ./...

FAIL	github.com/cilium/cilium/pkg/aws/eni	2.684s
FAIL

Makefile:204: recipe for target 'unit-tests' failed
make: *** [unit-tests] Error 1
The command "./.travis/build.sh" exited with 2.

Done. Your build exited with 1.

https://travis-ci.com/github/cilium/cilium/jobs/335282883
Seen in https://github.com/cilium/cilium/pull/11555

Not so rare for me
https://travis-ci.com/github/cilium/cilium/jobs/336004501

I've seen it twice today.

FWIW:
$ find . -name node_manager_test.go
./pkg/ipam/node_manager_test.go
./pkg/aws/eni/node_manager_test.go

I tried running these locally and they both passed:
$ make unit-tests TESTPKGS=pkg/aws/eni

$ make unit-tests TESTPKGS=pkg/ipam

I'm guessing that there's either an angle here of raciness or an angle of a prior test not cleaning up properly after itself.

FWIW:
$ find . -name node_manager_test.go
./pkg/ipam/node_manager_test.go
./pkg/aws/eni/node_manager_test.go

I tried running these locally and they both passed:
$ make unit-tests TESTPKGS=pkg/aws/eni

$ make unit-tests TESTPKGS=pkg/ipam

I'm guessing that there's either an angle here of raciness or an angle of a prior test not cleaning up properly after itself.

Hmm, there's definitely some raciness going on. I ran the tests with -race enabled, and it found many, many data races. A decent amount of data races are in the TestNodeManagerManyNodes so this might explain this Travis failure. I'm working on a PR to fix some data races under just TestNodeManagerManyNodes, as there are many data races around this code.

Update on this flake: it looks like PR https://github.com/cilium/cilium/pull/11749 ran into this again. What's interesting is that the usual number of errors when this flake occurs has been reduced dramatically, most likely due to https://github.com/cilium/cilium/pull/11685. Bad news, the flake is not fixed. Good news is that https://github.com/cilium/cilium/pull/11685 made progress. It is likely that there are more data races in the test suite or in the ENI code itself.

another occurrence https://github.com/cilium/cilium/pull/11682/checks?check_run_id=718254605

I think the test is compares number of allocations to minAllocate and expect and exact value, from reading the code I believe it shouldn't be < minAllocate.
https://github.com/cilium/cilium/blob/master/pkg/aws/eni/node_manager_test.go#L611-L614
https://github.com/cilium/cilium/blob/05daeef1c29a48928391cb1820c77752675d6bcd/pkg/aws/eni/node_manager_test.go#L627
I'll open a PR and we take it from there :)

If this occurs again which is possible, see https://github.com/cilium/cilium/pull/11773#issuecomment-636153564 for more context.

Here as well: https://travis-ci.com/github/cilium/cilium/jobs/341510221

Occurred in https://github.com/cilium/cilium/pull/11786: https://travis-ci.com/github/cilium/cilium/jobs/341598176

I removed word 'mismatch' in my PR, so those errors are from an unrebased PR build, perhaps?

@pchaigno Any pointers on where this occurred again? We recently removed the primary IP from the allocation pool, might be related?

@gandro I restarted the Travis CI build where it happened. I'll keep it next time and ping here.

Hit again in #16472
https://travis-ci.com/github/cilium/cilium/jobs/513573358
log.txt

Hit in #16865. The full log of the failing test case - https://gist.github.com/brb/ebe4764e17d757e3c87c0bfaf45b6d95 (GH doesn't allow me to attach it).
The failure:
FAIL: node_manager_test.go:574: ENISuite.TestNodeManagerManyNodes

node_manager_test.go:613:
    c.Errorf("Node %s allocation shortage. expected at least: %d, allocated: %d", s.name, minAllocate, node.Stats().AvailableIPs)
... Error: Node node93 allocation shortage. expected at least: 10, allocated: 0

node_manager_test.go:625:
    c.Assert(metricsapi.Nodes("in-deficit"), check.Equals, 0)
... obtained int = 1
... expected int = 0

Interesting part from the log is:
level=warning msg="Could not determine first interface index, maximum allocatable ipv4 addresses will be 0 (unlimited) this could lead to ip allocation overflows if the max-allocate flag is not set" first-interface-index=unknown subsys=eni
level=warning msg="Instance type network adapters limit is lower than the configured FirstInterfaceIndex, maximum allocatable ipv4 addresses will be 0 (unlimited) this could lead to ip allocation overflows if the max-allocate flag is not set" adaptors-limit=3 first-interface-index=4 subsys=eni
level=warning msg="Could not determined instance limits, maximum allocatable ipv4 addresses will be 0 (unlimited) this could lead to ip allocation overflows if the max-allocate flag is not set" adaptors-limit=unknown first-interface-index=0 subsys=eni
``

Checked logs provided in https://github.com/cilium/cilium/issues/11560#issuecomment-860795257, they have the same log message Instance not found! for the failing node (node85).

From checking the code it seems that it might be a race. The # of available IPs is retrieved from https://github.com/cilium/cilium/blob/master/pkg/aws/eni/node.go#L513 which errors. This means that the node didn't have any ENI attached, so it returned 0 of available IPs which made the test to fail.
I am not familiar with the node creation lifecycle in the ENI node, but if ENIs are set asynchronously, then the number can be 0.

Hit in #16258 (https://app.travis-ci.com/github/cilium/cilium/jobs/532499964):
FAIL: node_manager_test.go:563: ENISuite.TestNodeManagerManyNodes
node_manager_test.go:602:
    c.Errorf("Node %s allocation shortage. expected at least: %d, allocated: %d", s.name, minAllocate, node.Stats().AvailableIPs)
... Error: Node node98 allocation shortage. expected at least: 10, allocated: 0

Hit it again https://app.travis-ci.com/github/cilium/cilium/jobs/535750325.

And again: https://app.travis-ci.com/github/cilium/cilium/jobs/536459423

Hit in #18062 https://app.travis-ci.com/github/cilium/cilium/jobs/550242889

Removing @christarazi 's stale assignment here. If you plan on working on this, feel free to reassign it back to yourself.


Failed to login US Gov with a service principal in a VM in global Azure

Describe the bug
When I login US Gov with a service principal in a VM in global Azure, I got the error:
Get Token request returned http error: 400 and server response: {"error":"invalid_request","error_description":"AADSTS90038: Confidential Client is not supported in Cross Cloud request"}

When I login US Gov with a service principal in a VM in Azure US Gov. It works.
I think, it's an expected behavior. Can you please confirm it?
However, I didn't find an official document to describe this error in Azure doc.
Environment summary
Install Method (apt-get) / CLI version (2.0.44) / OS version / Shell Type (bash)

@sauryadas can you respond? @yugangw-msft is OOF for a couple weeks.

It is about the Cross-cloud federation. WW -> Sovereign is supported, but not vice versa. So yes, this is expected.
This is not a CLI question, rather on AAD.  So closing ...


feat: add parquet file reading support for s3fdw

What kind of change does this PR introduce?
This PR is to add Parquet file reading support for S3FDW. It also support (gzip, bzip2, xz, zlib) compressed Parquet file. Fix #97 .
What is the current behavior?
Parquet file is not supported currently.
What is the new behavior?
S3FDW will support Parquet file reading.
Additional context
N/A

Does it push filters / projections down? I.e. only read the relevant sections of the parquet file by filtering / projecting via its metadata?

Does it push filters / projections down? I.e. only read the relevant sections of the parquet file by filtering / projecting via its metadata?

Pushdown isn't supported yet, nested data is also not supported, it just do simple column projection at this moment.


Consistent use of BSD 3-Clause license header

Coverage remained the same at 91.72% when pulling 3771c282b2619344d4656cf69d16196c1ecee58c on consistent-license-header into 8912a4f19e06b808ab0be9c4f0cafdf40abf689f on master.


Add user-focused flows principle

Code Graph PMs were recently discussing how we coordinate across teams within our org when it comes to things like implementing cross-cutting features, which will be a thing that happens more and more especially with use-cases which are designed to force that kind of thinking.
I put a stake in the ground here with this PR, but feedback and changes/improvements are truly welcome. Please take a look and share your thoughts.

@rrhyne maybe you have some ideas on this one also.

@anna-mmikhova @bcreager tagging both of you since this includes a blurb on how we think about collaboration with Cloud. Feedback welcome, and not just on that bit.

Merging as there hasn't been any further feedback. Thanks everyone!


Update pipeline job names and dependencies

Rename job from Publish to Build
Rename job from Audit to Analyze
Make Analyze and Test depend on Build

See https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki?wikiVersion=GBwikiMaster&pagePath=%2FAzure DevOps Pipeline Guidance&pageId=2&anchor=build-pipelines-jobs

Fixed with https://github.com/Azure/azure-sdk-for-js/pull/1342


Utilities: Added script to remove LISAv2 old files.

This script cleans DriveLetter:\LISAv2 directory based on the file age.
Default cleanup age is 7 days.
This script iterates through all the drives available in current machine.

I have create a cleanup pipeline here  (in linuxpipeline)
Execution is being verified.
https://XXXXXXXXXXXXXX.xxxxxxx.cloudapp.azure.com/view/Utilities/job/tool-cleanup-lisav2-files-from-nodes/14/console

I'd recommend to not clean up the folders before @LiliDeng @lizzha @chvalean and other pipeline users agree things can be deleted.


Annotation Page Refactor for Phase 2 Store Refactor

Now that we are into the second phase of the store refactor - moving onto the annotation and download pages - it will be necessary to refactor the annotation page itself before its dependent components can be fully tested manually and via our page-level e2e tests for the page.
Below is a list of the refactor work that needs to be done to the annotation page:

[ ] Tabs now must look at new store configuration for info on their construction (previously in store field annotationDetails
[ ] Column unlinking from categories must be updated - includes a) transforming a column to its original unannotated state and b) the actual unlinking of a column from its previously assigned category
[ ] Removing all old tool group functionality
[ ] Altering how missing values are added and removed to accord with the store refactor
[ ] Altering how the annotations are 'saved' now that changes are being made to dataDictionary.annotated instead of the previous dataTable.annotated

Once this work is complete, then the e2e test for the annotation can be re-enabled for the github testing workflow. At this point, the e2e tests for the annotation page will likely need to be updated as well:

[ ] Re-enable annotation page e2e test (See #341.)
[ ] Update annotation page e2e test until all tests pass

In light of the possible complexity of this page refactor, I will spin off this work into several PRs.

Blocked by #386

This seems to be unblocked / finished. @jarmoza / @rmanaem could you check + close?

I believe this is finished @jarmoza could you confirm?

@surchs / @rmanaem Confirmed. Closed.


Change old notify.js to bootstrap-notify

I need to update something along with this commit so I close this pull request


fix(HMS-2034): Unify missing source empty state

Unifies the delimiter of DirectProviderLink.
Moves delimiter out of the DirectProviderLink for other usage (No Permissions).
GCP:

AWS:

@adiabramovitch any reason we had the OR delimiter just for GCP?

@MariSvirik can you give it a look and let us know what you think? :)

Yep, using "or" makes sense in this case.
"or" is just supposed to be in lowercase and not bolded. Also, apply bigger padding (I'm not sure what we are using now, but it looks stuck together)

(I've already used this screenshot in a different Jira)

Thank you! New screenshots:

/retest

Why doesn't the region drop down appear in the new screenshots?

Why doesn't the region drop down appear in the new screenshots? And I guess the or is supposed to be in both cases.

You've gotten me scared a bit, that there's some magic, but I just took the screenshot too fast while regions were still loading, updated it.


Font family issues

in some part the font family is not proper.

I want to contribute in this part -> GSSoC'23.

It has different font to make it different from the content. Thanks for stepping up ,u can contribute on other issues.
Thank you!!


change parameters of function moveObstacle.

moveObstacle ("box/base_link_0", position) made it impossible to move objects of any other name.

Done!
Anna


realsense_viewer with automatic control

Hello Everyone,
As I am a new in field of programming and c++, all what I want is, what's the differents changes to make in order to open realsense_viewer programm cpp with an fixed control and record ( to save a bag) ( without interacting by users).
Thanks in advance.

I don't really see the point of opening the RealSense Viewer if you don't want to change the parameters, so I assume you mean "rendering a stream with fixed parameters" instead, am I right ?
I would recommend you check out the examples codes :
https://github.com/IntelRealSense/librealsense/tree/master/examples
Maybe try combining :
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/opencv/imshow/rs-imshow.cpp
with
https://github.com/IntelRealSense/librealsense/blob/master/examples/save-to-disk/rs-save-to-disk.cpp
to render the stream and save a file (again, I'm assuming that's what you want to do).
And if you are beginning, take a look at :
https://github.com/IntelRealSense/librealsense/wiki/API-How-To

Hi @TAREK-ELOUARET
Thank you for your input. Writing dedicated C++ / python code is probably better long term solution.
Opening the Viewer to limited automation is an interesting idea (that we also explore in need for better testing). We will consider it.

Hi @AntoineWefit
I made some modifications in record-playback file so I could resolve my problem, instead, I have tried to implement opencv with realsense ( as libraries ) but the result is less reliable (very slow in record and streaming ) than using and exploit realsense parameters.
So thank's for your help Sir.
@dorodnic  I would like to appreciate your time and consideration, you helped me in lot of issues, so thanks' in advance

[Realsense Customer Engineering Team Comment]
@TAREK-ELOUARET
If the issue is solved and no further question, please help to close the ticket, thank you.

@RealSense-Customer-Engineering
I'd like to keep this is open as enhancement request.


Spark Thrift Server cannot use external metastore

Describe the bug
I am running standalone spark version 2.3.2 in docker with ranger-hive-plugin 0.5.3. I use a extend MySQL database for store hive metastore data.
I found out that Spark Thrift Server cannot use external metastore for excuting SQL statements after enabling spark configuration: spark.sql.extensions=org.apache.ranger.authorization.spark.authorizer.RangerSparkSQLExtension .
If I remove the above configuration Spark Thrift Server can use external metastore but the ranger-hive-plugin is not enabled.
This issue does not happen when using spark-sql and spark-shell.
To Reproduce
Steps to reproduce the behavior:
copied ranger*.xml conf files to spark_home/conf, copied ranger-hive*.jars to spark_home/jars along with spark-authorizer jar. gave full permissions to all xml and jar files. Modified conf xml files as advised.
Environments are spark standalone 2.3.2, with ranger-hive-plugin 0.5.3, ranger-admin 0.5.3.
Tried running show databases command in beeline.
Expected behavior
Spark Thrift Server can use external metastore after enable ranger-hive-plugin.
Additional context
Gist for configuration files: Spark-SQL-Ranger

Spark Thrift Server is not guaranteed you may try kyuubi https://github.com/yaooqinn/kyuubi


Code duplication

Issue #19
Eliminated code duplication. The logic for constructing the SQL statement is encapsulated in the constructSQLStatement function, reducing code duplication and making your code more modular.

Sorry about the unstashedcommits 🤦♂


feat: fixed for the issue (#168)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #172 (e838420) into main (ea12b0b) will increase coverage by 5.60%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
+ Coverage   84.81%   90.41%   +5.60%
==========================================
  Files          74        8      -66
  Lines         553       73     -480
==========================================
- Hits          469       66     -403
+ Misses         84        7      -77

see 66 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


when a e2e case finished, the pod was not deleted in time

kubectl get po --all-namespaces
NAMESPACE       NAME                                            READY     STATUS    RESTARTS   AGE
kube-system     kube-dns-659bc9899c-q2vdh                       3/3       Running   0          3h
poseidon-test   firmament-scheduler-7788b7d89b-9cwnx            1/1       Running   0          6m
poseidon-test   poseidon-86796db96-chslx                        1/1       Running   0          6m
poseidon-test   restricted-pod                                  0/1       Pending   0          28s
poseidon-test   test-nginx-deploy-4039455774-7bbd79f9c6-p2tx2   1/1       Running   0          6m
poseidon-test   test-nginx-deploy-4039455774-7bbd79f9c6-svn8x   1/1       Running   0          6m
poseidon-test   test-nginx-job-1879968118-dv7j6                 1/1       Running   0          5m
poseidon-test   test-nginx-job-1879968118-srtpq                 1/1       Running   0          5m
poseidon-test   test-nginx-rs-2854263694-czx47                  1/1       Running   0          6m
poseidon-test   test-nginx-rs-2854263694-ppfnw                  1/1       Running   0          6m
poseidon-test   test-nginx-rs-2854263694-v9rw5                  1/1       Running   0          6m

I0524 17:24:23.474194  107687 deployment_controller.go:573] Deployment poseidon-test/test-nginx-deploy-4039455774 has been deleted
I0524 17:24:23.479394  107687 deployment_controller.go:573] Deployment poseidon-test/test-nginx-deploy-4039455774 has been deleted
I0524 17:24:23.496520  107687 replica_set.go:477] Too few replicas for ReplicaSet poseidon-test/test-nginx-rs-2854263694, need 3, creating 3

/assign


[Question] Where are the setting files saved?

Before Snowy ver 1.9,  the setting files are saved in the following folder.
[system drive]\Users[user name]\AppData\Roaming\SnowyImageCopy\
Please let me know where is the setting files in Snowy Plus 2.2.4?

In general, the executable files and created files of a Store app are not supposed to be accessed by an user. They are handled by the OS and even a developer does not necessarily need to know their actual location.
Said that, the most part of Store apps' files are located in the following folder:
C:\Users\[user name]\AppData\Local\Packages
I am not sure but if you dig into the folders, you may be able to find it.

Thank you...I've found it.
C:\Users[user name]\AppData\Local\Packages\10186emoacht.SnowyImageCopy_0q7myvhtpbc7w\LocalCache\Roaming\SnowyImageCopy


Why is my codebase referenced in this dataset?

https://github.com/ahabegger/CodeQuality4DevGPT/blob/09c313353528da31a8b1fd0160e1ff101b699ebd/Output/javascript_Human_code_objects.csv#L75

Genuinely curious.


setup pkg documentation in readthedocs.io

once all features completed (ecg, ppg)

The readthedocs has been configured and the branch merged. Please double check in your readthedocs account that the project has compiled properly, if not please do share the log with the errors (I can't see them). Please do also share the readthedocs URL, it should be of the format "https://.readthedocs.io/en/latest". I have tried the following but I don't findit:
https://vital_sqi.readthedocs.io/en/latest/ [not working]
https://vitalsqi.readthedocs.io/en/latest/ [not working]
Note: Changing the vital_sqi repository from private to public converted my forked repository into a private one. Thus, I decided to use it as a previous check for both gh-pages and readthedocs and they work fine (see PS below).
PS: https://bahp.github.io/vital-oucru-clinical/
PS: https://vital-sqi-rtd-test.readthedocs.io/en/latest/

@bahp, https://vitalsqi.readthedocs.io/en/latest/ should work now.


Allow multiple methods in ActiveRecord callbacks

Fix for https://github.com/robotdana/leftovers/issues/17

oh thanks :)
this looks good i’ll merge and make a release some time this weekend


Roll Engine from 95a40884c738 to 3b6a6001b4b1 (9 revisions)

https://github.com/flutter/engine/compare/95a40884c738...3b6a6001b4b1
2021-07-28<EMAIL_ADDRESS>Empty commit to apply latest LUCI config (flutter/engine#27767)
2021-07-28<EMAIL_ADDRESS>[ci_yaml] Add autoroller builder (flutter/engine#27731)
2021-07-28<EMAIL_ADDRESS>Roll Skia from c7218f57add1 to 222c1c16317c (1 revision) (flutter/engine#27762)
2021-07-28<EMAIL_ADDRESS>Roll Dart SDK from e4fa78b5025a to fec06792b6e3 (1 revision) (flutter/engine#27761)
2021-07-28<EMAIL_ADDRESS>Roll Dart SDK from c9f1521fd3fe to e4fa78b5025a (1 revision) (flutter/engine#27759)
2021-07-28<EMAIL_ADDRESS>Roll Skia from 5def25af3f9a to c7218f57add1 (1 revision) (flutter/engine#27758)
2021-07-28<EMAIL_ADDRESS>Roll Skia from 77292ac4a12f to 5def25af3f9a (1 revision) (flutter/engine#27756)
2021-07-28<EMAIL_ADDRESS>Roll Skia from 2527fd0b8d5e to 77292ac4a12f (1 revision) (flutter/engine#27754)
2021-07-28<EMAIL_ADDRESS>Roll Skia from dc409946e9f0 to 2527fd0b8d5e (1 revision) (flutter/engine#27753)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Sometimes modifications are not reflected in the account list

Steps to reproduce the behaviour

Change the name of an account.
Press SAVE

Expected behaviour
The account is listed with the new name.
Actual behaviour
Sometimes the account appears with the old name.
I just reproduced it once, but the user (reported through User Voice 152) claims it happens every time for him.
Software specifications

GnuCash Android version: 2.1.3
System Android version: 5.0

Strange one this is...


Feature request: enable duckdb httpfs extension

I'd like to run a duckdb query against http hosted csv files.
This is one of my favorite features of duckdb. (and hopefully for others too)
For example:
sql: covid_jp_daily is ||
  select *
  from 'https://raw.githubusercontent.com/kj-9/covid19jp/master/data-raw/dist/jp_daily.csv'
  limit 10
;;

Running this on malloy extention throws the error:

lookup.message.includes is not a function

Seems like to do this, we need to enable httpfs extension by INSTALL and LOAD statement .
P.S.
I enjoy using Malloy and exploring repo. Thank you for developing and open-sourcing this repo!

I looked at this before, but found the fetching unreliable (which caused problems).  We can take a look at this again...

One of the main challenges for us using extensions is that we are currently using development versions of duckdb to have the latest features and bug fixes, but extensions often don't seem to be available for these releases (the extensions are tied to a specific git commit hash, not just a major/minor version). If we are able to land on a release version of duckdb we can consider what extensions we want to support.


Bug 1116467 - Beef up E4X compatibility

Looks good if AVM2 tests pass.
/botio test

@mbebenita  do we test only the AVM2 interpreter in test_avm2_quick ? do you see anything that can break the compiler execution?

@mbebenita do we test only the AVM2 interpreter in test_avm2_quick ?

We should go through and enable all the Tamarin tests that we currently pass. If we don't run them in reftest, the slowdown that'll introduce seems acceptable.

do you see anything that can break the compiler execution?

I tested with both compiler and interpreter locally.


Unable to require openssl, install OpenSSL and rebuild ruby (preferred) or use non-HTTPS sources

I'm attempting to install jekyll and I've encountered an error. I'm running Mac OS X 10.11.4 (El Capitan).
$gem install jekyll
ERROR : While executing gem ... (Gem::Exception)
      Unable to require openssl, install OpenSSL and rebuild ruby (preferred) or use non-HTTPS sources
$gem source -l
https://ruby.taobao.org
$which openssl
/usr/local/bin/openssl

I want to know that how to resolve this error.

Suggest you ask the jekyll support area, this isn't an openssl issue.  To download and install openssl, go https://www.openssl.org


SA1513 reported when I wouldn't expect it

I'm getting a violation of SA1513 (ClosingCurlyBracketMustBeFollowedByBlankLine) reported when I'm not expecting it. It's possible it's what you guys expect, but on the off chance it's a bug I thought I'd report it :)
var target = isIFrame
    ? new Dictionary<string, string>
        {
            { "target", "_parent" }
        } // <-- SA1513 Violation reported here
    : new Dictionary<string, string>();

Note: The following code is not raising the issue, but personally I think it's an unnecessary new line that isn't helping readability.
var target = isIFrame
    ? new Dictionary<string, string>
        {
            { "target", "_parent" }
        }

    : new Dictionary<string, string>();

What do you think?

I wouldn't expect SA1513 here. I think the readability of the sample code could be improved, but the problem isn't due to a missing newline after the }.
Marked :bug: up for grabs. :smile:

Grabbing this


i18n example is 404

Bug report

What is the current behavior?
this page: https://github.com/webpack/webpack/tree/master/examples#i18n
i18n example is 404

send a PR and remove the link

send a PR and remove the link

so we do not support i8n now?

so we do not support i8n now?


Reverting back source codegen

@fjtirado Hi, i added some important fixes&improvements in the scanner-part of the extension
(see https://github.com/quarkiverse/quarkus-asyncapi/pull/105 and https://github.com/quarkiverse/quarkus-asyncapi/pull/102)
I'd like to create a new minor-version v 0.2.0 for that, but is saw that you are currently working on your side.
Is the new version ok for you?

@fjtirado Hi, i added some important fixes&improvements in the scanner-part of the extension (see #105 and #102) I'd like to create a new minor-version v 0.2.0 for that, but is saw that you are currently working on your side. Is the new version ok for you?

Yes, but lets wait till I have finished

@fjtirado Hi, i added some important fixes&improvements in the scanner-part of the extension (see #105 and #102) I'd like to create a new minor-version v 0.2.0 for that, but is saw that you are currently working on your side. Is the new version ok for you?

Yes, but lets wait till I have finished

I will go on holiday tomorrow for 3 weeks.  But my colleagues are waiting for my fixes...
If you need longer, could you do me a favor and make the new version, when you are finished?

@ChMThiel sure, Ill do it tomorrow. Just commit your changes and I will be publishing 0.2.0 as soon as possible


:city_sunrise::crocodile: GNN Ntupling (feat. setup script for UF HiPerGator)

Added the following:
MiniDoublets: dphichange is now being read out (changes in the SDL/Event.cu)
Hits: detid is now being read out (changes in the SDL/Event.cu)
bin/sdl:
added -G option to enable GNN input variable branches to the output
AccessHelper:
Added some helper function for hitTypes/Index retrieval in one go
AnalysisConfig.h:
added gnn_ntuple boolean flag
trkCore:
minor helper function
write_sdl_ntuple:
GnnNtuple create/set/fill branches
setup_hpg.sh:
University of Florida HiPerGator cluster setup

From A100 on HiPerGator

@sgnoohc Can you please also run the master on the same machine and post the timing for comparison?
Also, since this PR involves the addition of a new variable, can you please perform the checks documented here to make sure that nothing has been missed?
I will take a more detailed look at the code by our meeting tomorrow.

@sgnoohc Can you please also run the master on the same machine and post the timing for comparison?
Also, since this PR involves the addition of a new variable, can you please perform the checks documented here to make sure that nothing has been missed?
I will take a more detailed look at the code by our meeting tomorrow.

Total Timing Summary
Evt    Hits       MD       LS      T3       T5       pLS       pT5      pT3      TC       Event      Short           Rate
avg      3.4      2.2      1.0      3.5      3.1      0.7      1.1      0.8      1.9      17.8      13.6+/-  2.9      18.9   explicit_cache[s=1]
avg      4.8      4.3      2.2      5.2      4.7      1.6      2.8      2.3      4.4      32.5      26.0+/-  9.7      18.9   explicit_cache[s=2]
avg     10.2      5.0      4.1     13.4     14.5      2.5      4.9      4.4      4.9      63.9      51.1+/- 20.6      19.0   explicit_cache[s=4]
avg     15.5      6.7      4.9     20.1     21.7      5.1      6.8      2.2      8.3      91.4      70.8+/- 35.2      19.1   explicit_cache[s=6]
avg     30.1     10.5      6.9     27.8     29.2      3.6     10.7      5.5     13.4     137.7     104.0+/- 49.5      19.3   explicit_cache[s=8]
The memcheck returned 0 error


Request for help with harmony companion dimmer rocker

Hi,
I've had problem using the rocker dimmer switch on the harmony companion remote so added a shell script to debug out parameters.  The rocker dimmer sends a continuous stream of adjustment requests.  So we end up making x number of request to HASS or others, all of which take some time so end up in odd states.
I'm thinking I would like to implement something so I can press the rocker once, and it increases / decreases the dimmer by 30%.
Something like

Get current value of brightness and new value into bash script
See if we are increasing or decreasing
Send request to Hass to adjust by 30% either way
Ignore all subsequent requests for 1000s

Can anyone help with what I need for 1 and 3?  Or any other pointers much appreciated.
Thanks,
Alex

An update in case anyone else want to get harmony companion remote rocker (brightness up down controllers)  - this is doable with home assistant and ha-bridge.
Rough outline: add webhook in home-assistant triggered by dim action in ha-bridge into which we pass brightness intensity, use home assistant data_template to set brightness accordingly.
(actual implementation a bit messier:
-- We need to store last passed brightness from ha-bridge in a new input_number, to see if we are going up or down and set dimness accordingly
-- For Tuya devices groups created in harmony or home-assistant turn off each light one by one.  To avoid this set up each dim state required as a scene in Smart Life / Tuya.
)
Works fine, I have 3 dim states to cycle between on up and down, could add more but seem redundant right now.
Config available if anyone want it.


[actions] add arm-gcc-{9,10.3} for macOS

Also, move lint check to new workflow for better organization

@jwhui Could you update the required status checks to use the new job names?


Controls Not Rendering

I see the volume rendered but no controls:

I have run:
jupyter nbextension enable --py --user ipyvolume
jupyter nbextension enable --py --user widgetsnbextension

$ pip freeze | grep widg
ipywidgets==7.0.1
widgetsnbextension==3.0.3

$ pip freeze | grep jupy
jupyter==1.0.0
jupyter-client==5.1.0
jupyter-console==5.1.0
jupyter-contrib-core==0.3.3
jupyter-contrib-nbextensions==0.3.1
jupyter-core==4.3.0
jupyter-highlight-selected-word==0.0.11
jupyter-latex-envs==<IP_ADDRESS>
jupyter-nbextensions-configurator==0.2.7

$ pip freeze | grep note
notebook==5.0.0

This seems to fix the issue:
from ipyvolume.widgets import quickvolshow
quickvolshow(...)

Ah, actually, I've replaced most controls with the buttons you see on the top left corner. quickvolshow should do the same as volshow actually, I'll open this, since it does seem like inconsistent behaviour.

If by "controls", you mean the sliders which control the transfer function, then I also experience the same thing - quickvolshow generates a VBox with the controls and the figure, while volshow generates only the figure.

Looking back at this, volshow(..) returns the current figure (since there is no difference between the two at the moment). If you want the controls, call ipv.show()


WIP Bit more idiomatic async

As discussed with Laurent and Damian
https://twitter.com/damovisa/status/1129314638561079296?s=19

Looks good to me!


Azure Synapse Analytics (formerly SQL DW) is no longer available under Register sources

When I register sources in Purview, the Azure Synapse Analytics (formerly SQL DW) option is no longer available

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

ID: 1e292340-fde7-24cf-922e-c852643c7dfb
Version Independent ID: 194ae394-7145-8b01-63c5-ee4159f56773
Content: How to scan Azure Synapse Analytics - Azure Purview
Content Source: articles/purview/register-scan-azure-synapse-analytics.md
Service: purview
Sub-service: purview-data-catalog
GitHub Login: @viseshag
Microsoft Alias: viseshag

@v-jheichman Thanks for your feedback! We will investigate and update as appropriate.

We'll need to update the docs. Here's the tile to click on:

#assign:viseshag to update the right pages.

Thanks @JasonWHowell.

#please-close I hope the screenshot helps. We'll update the article to better explain things.


"ended" event does not fire at the end of the media item

Sometimes the "ended" event does not fire when a media item ends.
This means that the player will continue to show the Chromecast Tech even after the item has ended, instead of closing the Chromecast session after the SESSION_TIMEOUT is up.
This seems to be caused by this if statement. The PlayerState is sometimes null after an item ends.

Closed by #10


Clean up enum Outcome

A small drive-by PR to clean up enum Outcome in the backtrace module.

bors r=BriocheBerlin


Update README.md

Fix incorrect variable naming in README.md

Hey @Beeta I'll merge this PR in but FYI rewriting the entire README has been on my to do list so it might not last for long :) Thanks for the edits though!


Improve dashboard tab menu

Change dashboard tab menu to be openable on right click plus via ellipses on the right of each panel to replace the Settings and Close icons.
Change dashboard tab menu items to make it clearer what they relate to:
Rename 'Table 1'
Settings for 'Table 1'
Hide 'Table 1'
Duplicate 'Table 1'
Duplicate All Tabs
Remove 'Table 1'
Remove All Tabs

Also disable (Duplicate|Remove) All if there is only one tab.


:art: Minor refactoring

Pull Request
My patch is about

[ ] :bug: Bugfix
[ ] :arrow_up: Improvement
[ ] :pencil: Documentation
[ ] :heavy_check_mark: Tests
[ ] :sparkle: Feature

Checklist

[ ] I accept that my PR will be distributed under the MIT license.
[ ] Test cases added for changed code if necessary.

Describe what you have changed in this PR.

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

Comparison is base (598676b) 90.42% compared to head (9037e11) 90.42%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   90.42%   90.42%
=======================================
  Files           8        8
  Lines        1326     1326
=======================================
  Hits         1199     1199
  Misses        127      127

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


Remove Upstreamed and Backported Recipes

Removes:

uhubctl (backported)
smemstat (upstreamed)
linux-serial-test (upstreamed)

@hnez Oh, haven't noticed this appeared while collecting my changes, sorry. Thanks for merging! :wink:


Add a set of APIs for colour filling with alpha gradient

Filling-colour-with-gradient-opacity is a valuable tool when creating visual elements; it is commonly used in drawing backgrounds for panels, widgets etc.
The following features should be taken into consideration when adding this set of APIs:

Support 2/3/4 points gradient mode
Have variants to support mask (on the target side)
Have variants to support an extra opacity

The following API variants  have been added:

fill colour with 4pts alpha gradient
fill colour with 4pts alpha gradient and opacity
fill colour with 3pts alpha gradient
fill colour with 3pts alpha gradient and opacity
fill colour with horizontal alpha gradient
fill colour with horizontal alpha gradient and opacity
fill colour with vertical alpha gradient
fill colour with vertical alpha gradient and opacity

The following variants have been added:

fill colour with 4pts alpha gradient and mask
fill colour with 4pts alpha gradient mask and opacity
fill colour with 3pts alpha gradient and mask
fill colour with 3pts alpha gradient mask and opacity
fill colour with horizontal alpha gradient and mask
fill colour with horizontal alpha gradient mask and opacity
fill colour with vertical alpha gradient and mask
fill colour with vertical alpha gradient mask and opacity


Add the ability to swap current with next / previous item

Hi! thanks for the awesome plugin.
Could you add the ability to swap current item with next / previous without showing keystrokes? This is how it works in vim-swap.

This is already implemented in nvim-treesitter-textobjects.

I'm not sure if nvim-treesitter-textobjects allows you to swap things like list elements etc. At any rate, that would be more suitable for a feature request for that particular plugin. Thanks for checking out this plugin though!


XCS: management messages

Closes: #4156
What is the purpose of the change
Adds management messages to XCS so the contract owner can modify the config.
Brief Changelog

Added management messages

Testing and Verifying
Added tests for the new management messages. All old tests pass.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Can we update some docs for this?
TBH a little bit confused why its the XCS contract that maintains the channel registry

TBH a little bit confused why its the XCS contract that maintains the channel registry

This was added for a few reasons:

To be able to send to a receiver without having to specify the channel
To be able to limit which channels were used. Sending to an address with a bad bech32 prefix would fail transactions at the IBC level and if that error happened, the tokens would be stuck on osmosis. This doesn't matter anymore, as we have IBC lifecycle callbacks, but we can still fail early instead of sending the tokens and having to deal with the callback
This should actually belong in the registry/router contract, but since the router is going to have to deal with unwinding and is a bit more complex, having it here for now allows us to launch XCS without having to wait for the registry/router.


Deep Zoom Image view keeping image tiles on google drive.

As my dzi_files are kept in google drive (public folder), is it possible to show the deep zoom image with the following code like this--
https://stackoverflow.com/questions/16451256/how-do-i-get-started-using-openseadragon/16452141#16452141
It worked for GitHub repository, but as my tilesource folder crossed 1GB quota, I had to keep my files in google drive.
I tried this option too, but it seems like the webpage cannot fetch the google drive link.
My google drive link where the files are kept - https://drive.google.com/drive/folders/13zgWeO88OJWumktB42URK_UOwIgeilgV?usp=sharing

The html code that I am trying-
`

`

I don't know enough about Google Drive to know if there's a way to get a "raw" file URL from it. A link like https://drive.google.com/drive/folders/13zgWeO88OJWumktB42URK_UOwIgeilgV?usp=sharing just gives you a page that is user-facing, but won't serve the files like they are needed for code. Perhaps there's some way to ask Google Drive to serve the files like a regular web server? If not, maybe it's possible with Dropbox or box.com or something?
I know it's definitely possible with Amazon s3, and there are lots of other web hosting options that would allow it as well.

I found a solution using google drive as web server using Drive to Web.
Just need to organize the files in a public folder in google drive like this, courtesy to @iangilman, and it is ready to go.

Lovely! Good to know that service exists :)


Error while Building the Samples

i am trying to build the samples following the instructions that you provided here, but it is throwing the following errors:
Consolidate compiler generated dependencies of target webgpu
[ 15%] Built target webgpu
Consolidate compiler generated dependencies of target hello_triangle_minimal
Consolidate compiler generated dependencies of target clear_screen
Consolidate compiler generated dependencies of target clear_screen_sync
Consolidate compiler generated dependencies of target hello_triangle_verbose
Consolidate compiler generated dependencies of target vertex_buffer
[ 21%] Linking CXX executable clear_screen.html
[ 31%] Linking CXX executable clear_screen_sync.html
[ 31%] Linking CXX executable hello_triangle_minimal.html
[ 36%] Linking CXX executable hello_triangle_verbose.html
[ 42%] Linking CXX executable vertex_buffer.html
em++: error: Invalid parameter (do not use "=" with "--" options)
em++: error: Invalid parameter (do not use "=" with "--" options)
em++: error: Invalid parameter (do not use "=" with "--" options)
em++: error: Invalid parameter (do not use "=" with "--" options)
em++: error: Invalid parameter (do not use "=" with "--" options)
make[2]: *** [CMakeFiles/clear_screen.dir/build.make:101: clear_screen.html] Error 1
make[2]: *** [CMakeFiles/vertex_buffer.dir/build.make:101: vertex_buffer.html] Error 1
make[2]: *** [CMakeFiles/hello_triangle_verbose.dir/build.make:101: hello_triangle_verbose.html] Error 1
make[1]: *** [CMakeFiles/Makefile2:125: CMakeFiles/clear_screen.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:203: CMakeFiles/hello_triangle_verbose.dir/all] Error 2
make[2]: *** [CMakeFiles/clear_screen_sync.dir/build.make:101: clear_screen_sync.html] Error 1
make[1]: *** [CMakeFiles/Makefile2:229: CMakeFiles/vertex_buffer.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:151: CMakeFiles/clear_screen_sync.dir/all] Error 2
make[2]: *** [CMakeFiles/hello_triangle_minimal.dir/build.make:101: hello_triangle_minimal.html] Error 1
make[1]: *** [CMakeFiles/Makefile2:177: CMakeFiles/hello_triangle_minimal.dir/all] Error 2
make: *** [Makefile:91: all] Error 2
For reference i am using an older version of emsdk i.e., 2.0.6

This is due to old Emscripten version. Try updating to the latest version to fix.

The print "Failed to create WebGPU Context Provider" is something that is not printed from the code in this repository, but probably comes from Chrome browser internally.
Try visiting some WebGPU sample pages on the web: https://webgpu.github.io/webgpu-samples/samples/helloTriangle to see if any of them work?

@juj thanks for the help, Found out webGPU is not currently supported by chrome browser on Linux, had to build on mac and its running perfectly. And also I would like to reach out to you for further understanding of this repo, can you share a way of contact, if you dont mind?
Actually I am also trying to build a webGPU application using WASM, I will be grateful for your help.

Best way to discuss this repo is definitely via this tracker, feel free to open non-bug report related tickets.
Another place to discuss would be the gpuweb repository Discussions tab, which has been quite lively for a while now: https://github.com/gpuweb/gpuweb/discussions . There are a lot of developers there who are much more familiar with WebGPU programming itself - I largely develop only the bindings to Wasm, and have not actually developed any WebGPU based renderers yet.

Closing resolved issues.


Triton CPU submissions missing from NV system_descriptions.tsv

https://github.com/mlcommons/inference_results_v1.0/blob/master/closed/NVIDIA/systems/system_descriptions.tsv
@tjablin can you take a look?

Whoops, disregard


make it work with class "light" instead of "theme-light"

Is it possible to make it work with the default tailwind light and dark classes?
I currently use class="dark" and class="light"

Well, I could make the className prefix a configurable option, however having the theme className equal to the theme name would also result in the creation of the variants light and dark that would conflit with the dark variant of tailwind.
Bear in mind that with tw-colors you probably don't need to use variants at all if the only thing that change between a theme and another are the colors.
Can you provide a code sample (html or jsx) that shows what you are trying to achieve? This way I'll get a better idea of your issue

Exposed a new getThemeClassName option in the latest version that lets you customize the theme classNames

Thanks for the update, I haven't noticed your previous reply. I'll test and report.


Limit Access to Cards

Identify the Mentor who can access several cards versus the person who can only see their own card.
Should we store that somewhere else, perhaps AirTable? Or can we store it on the cards somewhere?

Used Custom Fields on the cards themselves to store the end-user emails.
If the auth'd user matches the board email, then they're an admin.
Solved via 8acfe582080ec1943cdd5cc09861d3ebb9b9129b


Corrections, and improve nuspec minClientVersion ref

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Dependency-Resolution.md
:white_check_mark:Succeeded

docs/Quickstart/Create-and-Publish-a-Package.md
:white_check_mark:Succeeded

docs/Schema/nuspec.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Geometry Shader Samples

The following legacy DirectX SDK samples were written for Direct3D 10 / FX10 and demonstrate the geometry shader stage. They should be ported to Direct3D 11, perhaps using FX11, for reference to GS techniques:

CubemapGS
ParticlesGS
PipesGS
Skinning10
ShadowVolume10

Note that ParticlesGS is mostly for historical value. If you read this blog post, you'll see in most cases not using the GS for point-sprites is actually faster depending on your target hardware.

The Direct3D 10 versions can be found on directx-sdk-legacy-samples.


Does not work if anchor has children elements

The script does not seem to trigger if I have inner elements inside the anchor.
Common usages of that would be adding glyphicons (or similar) as icons.

Strange. I will look into it and try to get a fix in this weekend.
Thanks for reporting this!

This bug is now fixed in version 1.0.4.


Potential new datetime issues

Describe the issue
See CI failures

https://github.com/ietf-tools/datatracker/actions/runs/3899998769/jobs/6660186805
https://github.com/ietf-tools/datatracker/actions/runs/3900011570/jobs/6660211516

There is also this one, which is different but possibly related:

https://github.com/ietf-tools/datatracker/actions/runs/3900217767/jobs/6660629415

CC @jennifer-richards
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

I didn't encounter these during the recent release builds - are they transient or OBE?

Men neither. Transient.

I think it's probably real, the test is using a UTC date to compare with the deadline computed in DEADLINE_TZINFO (still PST8PDT). I'd have thought there was enough fuzz in the comparison, but it's easier to fix this than think super hard about that. Will do that now.


DevOps best practice

As this is the product documentation for CI/CD using Azure DevOps I'm concerned that this tutorial skipped the build process as well as creating an artifact. This starts customers off with a poor understanding of DevOps best practices and will lead to more problems as a result. Instead the process should be to create an artifact with a build pipeline first and then deploy that into the staging environments using release pipelines. I have a demo on my GitHub at https://github.com/davedoesdemos/DataDevOps/blob/master/Data_Factory/ADFDevOps.md which shows how this should be done. While the page does show a working process, I wouldn't recommend any customer implement it that way so I think the documentation should be changed to reflect the best practice approach.
Cheers
Dave
[Enter feedback here]

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

ID: 1e22e4d6-366f-fbf8-e141-8905ca7ab7ae
Version Independent ID: 5be2acf2-66e7-cadf-6666-5a85a65b9771
Content: Continuous integration and delivery in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

@davedoesdemos - Thanks for sharing valuable inputs. As per my understanding, usually build branch may change in every development sprint and to avoid updating the trigger filter across all release pipelines for every change it is recommended to change the default branch in the build pipeline and build artifact is referred in release pipeline. This does not apply to ADF as ADF_Publish branch would be always the default for a project. Hence, standard approach for ADF deployment using DevOps does not include build steps in the documentation. Please suggest your thoughts if my understanding is incorrect and I can share it with content author for further discussion and updates.

Hi thanks for the quick response. The build branch wouldn't usually change at this level since we're talking about the collaboration branch rather than feature branches - these map well to ADF. It would still be highly desireable to create a real and immutable artifact in the store when dealing with CI/CD. The documented method works fine for demo purposes but in real world scenarios breaks down quickly because you would have a build process and unit tests for ADF, Databricks, SQL DW etc. and a release pipeline which takes the tagged and approved artifacts from these and deploys as one stable version of the platform. I have a customer doing this at scale right now and using Git as a repository for the artifact is not suitable at enterprise scale. Happy to chat about that use case internally (alias is dalusty) if you like. Managing versions between Azure DevOps artifacts and Git commits would be exceptionally hard outside of the very simple scenario of standalone ADF deployment.

@davedoesdemos - Thanks for the detailed response.
@djpmsft - Can you please take a look and share your thoughts?

Hey @davedoesdemos ,
In ADF, the build artifact is the ARM template for each release and in DevOps it is real easy to trigger creation of a release so that each new version of the 'adf_publish' branch becomes a real and immutable release. For example, if I ever want to go back to an older version of the published environment, I can.

Now there is a valid feature gap that we cannot currently 'automate' the publishing process and have a true continuous integration that automatically builds on updates to the collaboration branch. This is something we are addressing in the current semester and should be live hopefully by the end of the year.
I highlighted the current process in detail during a live stream I did earlier this year: https://www.youtube.com/watch?v=QzAtK_pFPYI
Feel free to message me if you have any questions!
Best,
Daniel

Going to close this issue as I don't believe any doc changes are required. Feel free to keep commenting or message me internally.
#please-close

While I agree that what you describe is true, I think it's irresponsible pushing customers down a non-standard way of using Azure DevOps for one use-case. We've had to battle to get data people to understand DevOps in the first place and this is just adding to the confusion. What you're missing is the integration with the wider environment and other products, all of which use artifacts within Azure DevOps as the standard way to store the deliverable. I'm not suggesting a change to the ARM generation (although that's also very odd!) but copying it to a proper object repository for versioning alongside the other artifacts should be our standard approach. Almost every one of our UK customers use Databricks within their pipelines so need to manage the two artifacts as a single unit.
It's disapointing that you're unwilling to add this to the core documentation as it leaves us in the UK sub pointing customers to third party documentation and recommending against our own docs.

Azure Data Factory is not a CI/CD tool, we just simply integrate with existing DevOps services. I would argue we are following a standard practice and using a specific state of the 'adf_publish' branch as the artifact to maintain history and state of the ARM template artifact used in deployment.
We have many customers that have successfully implemented this model and integrated it with Azure Databricks. I don't think this is a UK-specific thing at all and would highly recommend that you attempt to implement using the steps outlined in our documentation.
If you need to create a custom build task that is always possible, but for the majority of our customers it is not a best practice.

It might be useful to get feedback from the Azure DevOps team on this one. The issue isn't on the ADF side but rather what constitutes best practice within Azure DevOps, especially when working on a wider project with multiple workstreams. Obviously you're not happy to take this feedback on board so I'll drop the issue here, but we'll continue recommending Azure DevOps best practice over this documentation out in the field as it's the right thing to do and prevents customer confusion.


Add crd-install job

Upstream has added new CRDs, which due to Helm limitations don't get properly updated when this app already exists. This moves CRD installation to a separate pre-install job in order to apply new CRDs when applying on top of an existing installation
Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

Important: After this PR is tested and approved, ensure you "Squash and Merge" unless you are updating a subtree. The release automation in use on this repository relies on squashing, but git subtrees will be lost if squashed. This repo allows both, so you may need to change the merge type when merging.

Going with a ping, holler if you see trouble brewing :cloud_with_lightning:


dvc: add overrides for scmrepo and grandalf

Description of changes
Fixes #175885

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 175945 run on x86_64-linux 1

2 packages built:

dvc
dvc-with-remotes

Result of nixpkgs-review pr 175945 run on x86_64-linux 1

2 packages built:

dvc
dvc-with-remotes


AssetManager appendTimestamp not working if you use registerJsFile()

Timestamp is appended only if you create your own bundle. This is too complicated if you just want to register on JS file on certain page.
registerJsFile() creates bundle inside, but does not fill the basePath. I guess this is the reason timestamp is not appended.

:+1:

It you're feeling convenient enough, feel free to send us a PR.

My pull request
https://github.com/yiisoft/yii2/pull/10297

#10429

Resolved by dc23a6d3d8a9cd937cbbc64652db903ee4ff5598

Please reopen. Because #10429 (comment)

For those who is intersted in (temporary) solution:
Instead of registerJsFile() we can create new AssetBundle on the fly. Replace $this->registerJsFile($file) in your view with this code:
$file = '/js/page.js';
$asset = new \yii\web\AssetBundle([
    'js' => [ltrim($file, '/')],
    'basePath' => '@webroot',
    'baseUrl' => '/'
]);
$this->getAssetManager()->bundles[$file] = $asset;
$this->registerAssetBundle($file);

@cronfy thank you for a good suggestion. I'm closing this issue

@cronfy @SilverFire
if there is ten special static resources,i need registerAssetBundle ten times or create ten AssetBundles?


autobahn 17.5.1 breaks buildbot unit tests

buildbot.test.unit.test_mq_wamp
  WampMQ
    test_forward_data ...                                               [ERROR]

===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/Users/ptardy/dev/bb/buildbot/.venv/lib/python2.7/site-packages/twisted/internet/defer.py", line 1301, in _inlineCallbacks
    result = g.send(result)
  File "/Users/ptardy/dev/bb/buildbot/master/buildbot/test/unit/test_mq_wamp.py", line 135, in test_forward_data
    yield self.mq._produce(('a', 'b'), 'foo')
  File "/Users/ptardy/dev/bb/buildbot/master/buildbot/mq/wamp.py", line 62, in _produce
    return self.master.wamp.publish(self.messageTopic(routingKey), _data, options=options)
  File "/Users/ptardy/dev/bb/buildbot/master/buildbot/test/unit/test_mq_wamp.py", line 70, in publish
    details = EventDetails(1, topic=topic)
exceptions.TypeError: __init__() takes at least 3 arguments (3 given)

investigating..

@oberstet looks like it is coming from c2ba9312c83622319168ff3c40f3f2248f937e81
I would prefer this could be done in a backward compatible way.
from autobahn.wamp.types import EventDetails
is public api, and I would like my test to be able to run on older version of autobahn too.

Ok, there are 2 things here: the concrete issue, and the general issue ("public API").
The constructor has changed: https://github.com/crossbario/autobahn-python/blob/master/autobahn/wamp/types.py#L482
def __init__(self, subscription, publication, publisher=None, publisher_authid=None, publisher_authrole=None, topic=None, retained=None, enc_algo=None):

Likely, your test will work if doing EventDetails(None, 1, topic=topic).

Rgd "public API": the class itself is marked public, but not the constructor.
This marking is intentional: all the properties of EventDetails are considered public, but NOT the ctor (there is no user need to construct it).
However we fail in two aspects here:

The docs do not reflect this properly: the documentation makes no distinction between instance attributes being consider public (as is the case of all of wamp.types.*  classes) versus the constructor method itself being public or not
There are classes in wamp.types.* where the constructor itself must be public, eg SubscribeOptions - that is, all classes supposed to be instantiated by the user.

To sum up, I take respons. for breaking stuff (sorry:( - and I would like to improve on that;) However, how exactly .. see above .. needs discussions.
What I would like to avoid is a general "every change must work with existing code" policy. This is holding back refactorings, making things more ugly etc.
So I would like to have a crystal clear way of communicating what is public and what is not, and only for the former adhere to a strict policy of not breaking code.

Thanks for detailed answer.
It looks like those @public annotation are fairly recent, and my code is fairly older.
My broken code is an attempt to make a fake WampComponent which does a naive routing itself.
https://github.com/buildbot/buildbot/blob/master/master/buildbot/test/unit/test_mq_wamp.py#L69
so if I cannot construct an EventDetails, the only thing I can see is to actually construct a FakeEventDetails of my own. do you think it is reasonable?

Yeah, this @public stuff is quite new, and part of our efforts to make Autobahn more dependable and predictable and "professional" now it is used in production apps people rely on.
In a way, I feel Autobahn move into a new phase, as it is "feature complete" in general (of course there are still many gaps, but it has reached the boundaries of its original scope.
And FWIW, for this phase, I seem to miss some of the language features in strictly/statically typed languages - like enforced privacy etc., and the @public decorator is an attempt to fill this.

..make a fake WampComponent

Ah, I see. So it is a unit test within your app (BB) that uses a mock WAMP transport. And since objects of EventDetails are provided to user code in the real WAMP transport, a fake transport needs to construct it.
I think this busts my naive thinking: "there is no user need to construct it" =)
IOW: in this particular instance, I'd say we should make the constructor public too (EventDetails.__init__ should be marked @public) and be bound with the responsibility that comes with that.

As we could not merge anything in our mainline, I chose to use the simple fake object.
https://github.com/buildbot/buildbot/pull/3164
I like the @public decorator. In buildbot, we have the policy of keeping compat for everything that is documented. We have a separated, non-autogenerated documentation for that.
This is often painful to keep backward compatibility, but generally not that much impossible.
Python being a dynamic language also helps on that :)
What we usually do is to only add keyword arguments and we don't modify positional arguments.
In the case of EventDetails, I think this would have been pretty simple to do it, at the price of a bit more verbosity for the caller.
We don't use positional as way to mark arguments mandatory, but rather make an assertion that its not configured with the default value.
m2c

sorry @public to mistakinly include you in this discussion. Not that a good idea for a github nick :)


NIL scoring for mobile apps

@GinaAbrams  Thanks for suggesting to start this issue.
What is the problem you are seeing? Please describe.
As confirmed by @larrysalibra, NIL currently is unable to score mobile and native apps on use of the sandbox, cookies and third party requests. Because of the current limitation, mobile apps are given a blank which are effectively counted as 0s for these parameters, severely affecting the total NIL score for the mobile apps.
How is this problem misaligned with goals of app mining?
The current scoring method though unintentionally but unfairly penalizes mobile apps in the app mining program. This creates a non level playing field for mobile apps compared to web apps. Web apps already outnumber mobile apps in app mining program by a huge margin and this will further disincentivize mobile app developers to participate in the program.
What is the explicit recommendation you’re looking to propose?
Either of the following two options could be chosen,

Take only the avg of the scores of parameters being reviewed. Or,
Postpone implementation of new NIL rules till NIL comes up with a review mechanism for mobile apps.

What is the dry run period (if any)
The dry run has already been concluded for December and the effect can be seen. Any one of the recommendations can be implemented right away.
Describe your long term considerations in proposing this change.
At this stage where mobile apps are already too less in blockstack ecosystem, the current scoring mechanism will further discourage developers to venture into mobile app development significantly affecting number of mobile apps in the ecosystem. Mobile functionality is essential for adoption by a wider user base and will be critical for achieving milestone 2 (2020 Q1-Q2) of reaching first million users on the blockstack network. Creating a level playing field for mobile apps in the app mining program will encourage new mobile developers, bring more awareness to mobile users thereby bringing new set of blockstack users to the community.

Mobile functionality is essential for adoption by a wider user base and will be critical for achieving milestone 2 (2020 Q1-Q2) of reaching first million users on the blockstack network.

The web has a wide user base including those using browsers on mobile, so native mobile is actually not critical nor essential for reaching a million users and beyond.

Mobile functionality is essential for adoption by a wider user base and will be critical for achieving milestone 2 (2020 Q1-Q2) of reaching first million users on the blockstack network.

The web has a wide user base including those using browsers on mobile, so native mobile is actually neither critical nor essential for reaching a million users and beyond.

71% of America's internet traffic is coming from mobile devices that is further divided into equal share between mobile web-browsers and mobile apps. [Read more here] This article is focused on e-commerce and retail, users of long-term brands such as Facebook and Twitter should have much higher percentage of users coming from the mobile apps. This is just the US, mobile-penetration is higher in 3rd world countries.
The web is predominantly on mobile now, apps serve users better. This is only blockchain and crypto community that is trying to re-invent the web by sticking to the browsers.
Also, it fails to amaze me now how Blockstack implements policies without any regard to which subset of apps these policies are going to affect or how future-proof they are even if it's necessary to implement solutions like Can't be evil sandbox to evolve and roll-over to the next phase of check and balance. I totally agree we needed things like open source and detecting if sites redirect users to somewhere else but things like detecting "cookies" and them holding a number on the points table - how do you plan to implement that for mobile apps or does Blockstack consider mobile-apps a very small subset of apps for the foreseeable future?
Arguably it's not that hard for mobile-apps as well if they are open source and even with closed source you can easily detect which URLs they are pinging at the backend. Open sourcing them could allow see to see which 3rd party packages they are using such as Facebook analytics.
By continuing to allow Digital Rights Reviewer to not support mobile apps, Blockstack is essentially discouraging mobile apps from joining the program.

This is only blockchain and crypto community that is trying to re-invent the web by sticking to the browsers.

I strongly disagree! Two companies benefit most from app distribution. There is less and less distinction between native apps and PWAs. Microsoft Store is embracing PWAs from the beginning. Google Play has published some tools for PWAs. And even Apple is moving forward.

This is only blockchain and crypto community that is trying to re-invent the web by sticking to the browsers.

I strongly disagree! Two companies benefit most from app distribution. There is less and less distinction between native apps and PWAs. Microsoft Store is embracing PWAs from the beginning. Google Play has published some tools for PWAs. And even Apple is moving forward.

This article is just suggesting to use PWAs to circumvent iOS app-store reviews. Lets assume I buy the argument that PWAs are better for decentralized internet (Apple is no way against it by the way but is susceptible to remove the app on certain government pressures and of course you are subject to some government reporting if you are using encryption in your app) - lets say we all should just build PWAs and somehow find our way to a user's home-screen. Now, shouldn't Blockstack should just stop supporting Android and iOS native SDKs and publicly state that you should just build a web-app and not native app if you want to be a part of app-mining? Instead of making them go on disadvantage silently without giving clear guidelines of what is favoured here?

I agree that either:
a) There's an official communication saying native apps are not recommended on Blockstack.
b) Native apps need to have an equal footing in the app mining program.
I would prefer b). Although mobile apps on app stores depend too much on good will from monopoly companies, it's still a huge market and an important growth source. Example: 4 of the top 10 apps on theblockstats.com have native apps.
And it's not like we can't monitor network traffic on native apps. We might not be able to enforce a sandbox mode like on the web, but we can check if they're not being evil. Although NIL decided to start with checking web apps, I believe native apps should not get a 0 score until a tool is develop to check them as well.

I guess nobody in Blockstack PBC is denying the importance of mobile apps for the ecosystem or bringing in new users into the network. It's just that there isn't any communication on how mobile apps are going to be reviewed by NIL on the new NIL policies. Will request @GinaAbrams , @hstove or @larrysalibra to update on how scores are going to be administered this month so that we can have clarity on this issue.

@hammadtq The Mobile web vs Mobile apps article is outdated and biased; It is from 2017 and posted by a company that promotes their mobile app builder and services.

cc @larrysalibra to please add additional comments 🙏

I actually agree with most of the comments here.
Native mobile apps are great and provide the optimal user experience for a wide range of applications.
Web is wide reaching and works very well on mobile as well.
We can put software between web apps and the rest of the world to get the massive data/privacy leakage that happens on the web under control.
The same problems exist in native mobile apps - to many extents it's even worse, but only the operating system manufactures are really in a position to do anything about it.
If we're scoring solely on how Can't Be Evil an app is - we can tell what a web app is doing. We can expose that behavior to the user and provide some sorts of guarantees about what the app will or will not do.
We can't tell what a native mobile app is doing and can't give the user any guarantees about its behavior.
Is that fair? Of course it's not. But we don't operate in a fair world - we operate in a competitive adversarial world where operating system vendors and hardware manufacturers (the same company in the case of Apple) wield enormous power to determine what we can and cannot do.
In the context of Can't Be Evil - it absolutely puts native mobile apps at a disadvantage. And that's really unfortunate, because native mobile apps deliver some of the best user experiences and this handicaps them.
Having said that, I think that it's not fair that I didn't give a clear answer to how we are going to deal with the scoring of mobile apps.
This problem regarding constraints that native mobile apps bring is only going to get worse in the future as we introduce apps distributed by blockstack names ("Stacks Publishing") and other uses of our technology stack.
I propose that this month, native mobile apps are exempt from the Can't Be Evil-related items and rated only on Gaia and Auth. In the future, in the absence of significant changes to app mining, I'd propose that apps that can't be rated on a particular metric because of the choice of form factor or technology get a zero in that category.

@larrysalibra
Some questions:
a) What about those who have both Web and Mobile apps for the same product?
b) And although there's no way to enforce "Can't be evil" on native apps, what about scenarios where we can check that they are not being evil? For example, an open source Android app with an APK self-published? Or monitoring a native app traffic during usage?

a) We always check web if an app has both web and mobile
b) We need to have a way to not only monitor but also prevent the behavior. I'm not sure how you can do that on mobile platforms without rooting devices. Would love suggestions though.

I don't have a suggestion for enforcing it for mobile/native apps.
But if a native app:

is open-source
has a predictable compilation/packaging/building process
is self-published and includes a verification hash

Couldn't that ensure that they aren't evil, even without enforcement?

At f-droid.org, they have a monitoring and enforcement of certain behaviors
for Android apps established. It might be possible to do extend these for
can't be evil behavior
Sérgio Santos<EMAIL_ADDRESS>schrieb am Mo., 13. Jan. 2020,
17:06:

I don't have a suggestion for enforcing it for mobile/native apps.
But if a native app:

is open-source
has a predictable compilation/packaging/building process
is self-published and includes a verification hash

Couldn't that ensure that they aren't evil, even without enforcement?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/blockstack/app-mining/issues/216?email_source=notifications&email_token=AALBYWNYXDRCSOJDZZWMS3LQ5SGRDA5CNFSM4KCVRTNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIZI6GA#issuecomment-573738776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALBYWOQX24WTQFRXSPGCRLQ5SGRDANCNFSM4KCVRTNA
.

I just realised that the extension allows for any request to the same domain. And, of course, we're not monitoring the server-side code, let alone enforcing anything. That sounds way worse than mobile apps, where at least we can monitor what they're doing.
Isn't saying on the extension that it's preventing evil misleading for web apps with a custom server?
Just want to clarify that I'm all for "Can't be evil" apps, that at least I can verify to be trustworthy. Enforcement is nice to have, but writing-off mobile apps that have nothing to hide seems harsh.


Cosim mismatch in some tests with PMP enabled using QuestaSim

Observed Behavior

I am getting the following error when running the tests using the default configuration (OpenTitan).
# UVM_FATAL ibex/dv/uvm/core_ibex/common/ibex_cosim_agent/ibex_cosim_scoreboard.sv(168) @ 50673967:
uvm_test_top.env.cosim_agent.scoreboard [uvm_test_top.env.cosim_agent.scoreboard] Cosim mismatch Synchronous trap was expected at ISS PC: ffffffff8000360a but the DUT didn't report one at PC 80003606

riscv_debug_instr_test
riscv_debug_wfi_test
riscv_interrupt_wfi_test
riscv_debug_single_step_test
Other tests are matching the behavior in Ibex Regression Results

Expected Behavior

It was expected for these tests to pass with the default configuration as of here Ibex Regression Results
When I set the PMPEnable to 0 in ibex configuration file it passes gracefully.
Steps to reproduce the issue

make SIMULATOR=questa ITERATIONS=1 TEST=riscv_debug_wfi_test

The test passes when PMPEnable is set to 0 in ibex_configs.yaml
My Environment
EDA tool and version: Questasim 2023.1

Operating system: Linux-64 CentOS 7.9

Version of the Ibex source code: df88055aa38868451ce90d7fcc2bbf172a2dc8a0

Hmm, this might be a problem with Questa support. At lowRISC, we don't currently have Questa access so it's rather hard for us to help.
Can you reproduce the problem with another simulator (Xcelium / VCS / Verilator) as well? If not, I think you might have to do a bit more digging to figure out what's coming unstuck.

Unfortunately, I can't run using another simulator at the time.
I have pinpointed the issue location to this code section.
vendor/google_riscv-dv/src/riscv_asm_program_gen.sv
    if (riscv_instr_pkg::support_pmp) begin
      str = {indent, "j main"};
      instr_stream.push_back(str);
    end

If I comment it out the test passes.
Any ideas here about what does this code section do or what can be a possible cause here?

Another follow-up is that the traces between ISS and RTL are erroring out at only the last instruction where on ISS it is giving the instruction "wfi" while on the RTL side, it is not giving anything the last instruction is the one before it.

trace_core
rtl_trace

8000353c	58dd	c.li	x17,-9	 x17=0xfffffff7
8000353c,c.li,a7:fffffff7,,58dd,,"c.li	x17,-9","a7,-9"

8000353e	0199c2b3	xor	x5,x19,x25	 x19:0x5dba8bcc x25:0x0000000e  x5=0x5dba8bc2
8000353e,xor,t0:5dba8bc2,,0199c2b3,,"xor	x5,x19,x25","t0,s3,s9"

80003542	10500073	wfi
---

The code you've pinpointed is part of riscv-dv: a random instruction generator that we use as a dependency. Looking at it briefly, this seems to tell the generated binary to branch to "main" at the start. We probably won't do anything interesting if that's commented out :-)
Your note about the traces makes sense. I suspect that this means the "there was a WFI instruction" message isn't getting to the DV code properly. @GregAC knows lots more about this area. Greg: any ideas?

All tests mentioned don't generate the "WFI" instructions in the rtl_trace.


Investigate google/web-starter-kit

https://github.com/google/web-starter-kit
Because it's maintained by great Sindre Sorhus.

I recently moved from web-starter-kit to este.
Migrating from wsk on a medium scale project to a new project that i've started.. i haven't really gotten anything out of wsk.
you can view the gulpfile and see that it's very outdated from today's workflow standards.

I think embedding a css framework is  brobably better than web-starter-kit for este.
+1 https://github.com/primer/primer

Thank you guys. Good to know.


android studio cannot create flutter project  (version 3.5.3)

Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter --no-color create --template app --description A new Flutter application. --org com3.example --ios-language objc --android-language java flutter2
exception
FileSystemException: FileSystemException: Creation failed, path = 'C:\flutterdemo\flutter2\android\app\src\main\java\com3' (OS Error: 目录名称无效。
, errno = 267)

Hi @leoche1984
Change your package name com3.example on com.example or some another variant, com3 seems to be not valid

Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it.
Thank you


How to change axis and grid colors?

I installed ng2-nv3 using this method in order for it to work on Angular 2 final. I'm wondering how I would changed the styling for the graph so that I could change the axis lines and grid (tick) colours?
Not sure how to change stylings in the component where I import nvd3 since its installed as a module and modifying the .css of the component where I use nvd3 selector tag in the HTML doesn't do anything...

I have the same problem, the style doesn't change.

I have been targeting the charts with CSS on the pages I am using them by using this selector
.nvd3-svg{ }
This question is pretty old now so I assume you've already figured it out or moved on but thought i'd leave a comment.


HMM2.0

I was hired by @bvisness to work on the list of changes for HMM2.0. The tracking issue for all of the changes is: https://github.com/HandmadeMath/HandmadeMath/issues/147.
Changes
Here are the changes I made:

Renaming. Types and functions have been renamed as described in the 2.0 issue: https://github.com/HandmadeMath/HandmadeMath/issues/147. This affects almost every file in the project.
Angles. A consistent unit is used for angles both as inputs to functions and internally. The default is radians for both, but can be changed. To change the input unit to all HMM functions, define one of HMM_USE_DEGREE_INPUT or HMM_USE_TURN_INPUT. There are macros, HMM_AngleRad, HMM_AngleDeg, and HMM_AngleTurn that will convert an angle to the input representation, such as passing an FOV in degrees: HMM_Perspective_RH(HMM_AngleDeg(120), ... other args ...). There are also functions to convert angles in the input representation to any of the other units, HMM_ToRad, HMM_ToDeg, HMM_ToTurn.
Configuration. Added HMM_PROVIDE_MATH_FUNCTIONS, removed individual checks for each math function being defined. Users now define this and all of the HMM_SINF and other trig functions, or accept the default crt implementations. The atanf and atan2f functions were unused and have been removed. If users override the functions, they can also use a different unit for the angles in their functions by defining both of HMM_ANGLE_USER_TO_INTERNAL and HMM_ANGLE_INTERNAL_TO_USER.
HMM_PREFIX macro has been removed. Everything now uses an uppercase HMM_ prefix, so it should be easy to find+replace on your local copy.
Handedness. All functions that depended on the handedness of the coordinate system have both a left and right handed version, marked by a "_LH" or "_RH" suffix respectively.
2x2 and 3x3 Matrices. Basic operations (Add, Sub, Mul, Div, Transpose) are implemented for each size.
Matrix Inverses. There are determinants and general inverses for each size of matrix. For 4x4 matrices there are inverses specific to each type of matrix we have a constructor for (Orthographic, Perspective, Rotation, etc). New tests were added for the inverses.
Overloads and _Generic. There are overloads for the new matrix sizes and the determinant and inverse functions. There is also a C11 _Generic macro for each function that was overloaded before.
FastNorm is default. The FastNorm functions are now just the Norm functions. Precision has declined in several places from this, but is still on par with the accuracy of GLM. Per: https://github.com/HandmadeMath/HandmadeMath/issues/87
SLerp. The Slerp implementation was not robust in cases when the quaternions were very close. It has also been changed to always take the shortest path between angles. https://github.com/HandmadeMath/HandmadeMath/issues/135
HMM_LinearCombineSSE is now HMM_LinearCombineV4M4. It will use the SSE code when available but fallback to a slower implementation. This reduces the amount of special cases needed for various Mat4 functions, which previously used HMM_LinearCombineSSE when available and then each had their own implementation.

Update Tool
I also created a tool that can be used to update user's code to 2.0 automatically, which you can find in a separate repository here: https://github.com/dev-dwarf/HMM2.0UpdateTool. The tool I created does the following:

Renames types and functions to use the new naming conventions from: https://github.com/HandmadeMath/HandmadeMath/issues/146
It wraps arguments that were previously taken in degrees to convert them to radians, per: https://github.com/HandmadeMath/HandmadeMath/pull/107
Adds the "_RH" suffix to functions that depend on handedness, since now both left-handed and right handed functions are available: https://github.com/HandmadeMath/HandmadeMath/issues/75
Removes ambiguity with matrix functions now that there are multiple sizes of matrices: https://github.com/HandmadeMath/HandmadeMath/issues/123

The tool is written in one .c file that compiles as C or C++, so any of our users should be able to compile it easily. The tool takes one or more filenames as arguments and updates their contents to 2.0. There are also .bat and .sh scripts to recursively run the tool on all the c/h/cpp/hpp files in a directory. The tool changes the files it touches so it should be used with source control. I can provide compiled executables for windows and linux, but someone else would have to handle mac.

Please take a look and let me know anything that needs changes. Sorry for the huge PR, didn't know a good way to split things up. Happy to answer any questions or make additional changes. Thanks for the opportunity!

Per offline feedback from Ben I've added some commits so that test coverage for the additions is comprehensive. Additionally I've added a configuration option so that projection matrices can use a Z range of [0, 1] for NDCs, this will be helpful for supporting graphics APIs other than OpenGL (DirectX and Metal use this convention that I know of).

Just took a look through this, and everything looks great. Thanks so much for doing this!!!


All URL generation parameter

Please update all URL generation parameters and include raw as well. See node.js docs for updated list

Fixed in 4.0.0


Skipped task keep in pending status

Hi.
I think that we're missing something in the configuration, so I dont know if our case is a bug or not.
By the way, we need help, so... could you help us?
We are creating a pipeline that has some conditionals in the tasks.
For example:
    #########################################
    ## TASK: get dockerfiles and artifacts ##
    #########################################
    - name: set-comerzzia-envs-dockers
      taskRef:
        name: set-comerzzia-envs-dockers
      runAfter:
        - maven-deploy
      params:
      [...]
      workspaces:
        - name: source
          workspace: shared-workspace
          subPath: $(params.git-revision)
      when:
        - input: $(params.git-customer-project)
          operator: in
          values:
            - BO

So if ´params.git-customer-project´ is not BO, the task is skipped. That is working, but... in the task status, it stays as pending:

Staying as pending results that when the task timeout happens, the pipelinerun changes from complete to pipelineruntimeout.

What are we missing?
I've seen a discussion about using "when" conditionals (https://github.com/tektoncd/pipeline/discussions/4185) and we have set the feature-flag to true, but i think is not affected.

closing by related #6622


modify for BIO_new_file() error return value

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Please rebase that on a fresh master checkout, then git push -f


Fix manual generation on Unix (1.1.0)

Same as #2863, but for the 1.1.0 series.

@richsalz, I came to the conclusion that possible future backporting makes this PR worth the effort.

Merged.  b7a9c3fcb5fc11cc102c4520514e01c09a62a1d8 and 6542af0571ccf491022c8d3494871c73c2127389