[BEAM-2253] Use resource transformer plugin to use versions defined in root pom.xml within maven archetype poms

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[X] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[X] Make sure tests pass via mvn clean verify.
[X] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @kennknowles @tgroh

Coverage decreased (-0.02%) to 70.652% when pulling a380dd3a78cfc45f49c6802fb381f6683cbe097f on lukecwik:beam2253 into 49245080ad6393b5678077caa047f7a6ef0efffc on apache:master.


Enhancement(web): Support onDrag/detectDragGestures Modifier on Web Platform

It seems that onDrag/pointerInput detectDragGestures doesn't work with Web Platform, here's a simple example
var offset by remember { mutableStateOf(Offset.Zero) }
Image(
  painter = painterResource(Res.drawable.logo),
  contentDescription = null,
  modifier = Modifier.offset {
    IntOffset(offset.x.toInt(), offset.y.toInt())
  }
    .size(48.dp)
    .pointerInput(Unit) {
      detectDragGestures(
        matcher = PointerMatcher.Primary
      ) {
        offset += it
      }
    }
)

Thanks for reporting!
It's rather a bug, since it's available for using it in the code but it doesn't work as expected :)
The feature was not implemented internally yet.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Suggestions for new styles

Permanent issue so people can suggest new message styles.
One inspiration: movie title sequences. Like those: https://www.artofthetitle.com/feature/top-10-title-sequences-of-2018/

A marquee style where the text would fit vertically and slowly scroll would be useful.


Update guidance for writing samples

@heaths commented on Mon Mar 23 2020
https://azure.github.io/azure-sdk/dotnet_introduction.html#dotnet-samples is not up to date about how to use markdown files with source files.

@willmtemple


The arrow head is small

could you modify the arrow size to the half length of the direction,

I would recommend making a copy of the Arrow struct and modifying it to suit your needs. I may add an arrow with a customisable length in a future release, but the current arrowhead size suits my use cases, and I don't want to add another field to it to make it customisable.


style(listing managment): move enabled filter

Moves enabled filter to below status in sidebar
Closes #AMLNG-822
How to test
Go to listing management, Enabled should be located beneath Status filter in sidebar on left.

:+1: listing mgmt sidebar re-arranged as desired

:+1:


Watch roles_map_file when started in watch mode

At the moment watch mode only watches config files. In our use case we frequently change the role map during development.
If the role map was inlined in the config that would already work, but we want to use the feature to make the role map its own file.
Please watch those files as well, so we don't have to manually restart couper.

Thanks @kevinsandow. #747 has been merged a while ago. Could you please check if the edge/master version works as expected in your case?

See also https://github.com/coupergateway/couper/issues/697. Actually, I don't remember which attributes are affected by the fix implemented there...

@kevinsandow Looks like this is already implemented in master, at least if you change the contents of an already existing file referenced by roles_map_file (and not change between roles_map_file and roles_map).


How to set SQL Azure firewall policies with Pulumi?

Whoops, hit enter before I had a chance to type in my issue.
How do you set Azure Firewall policies in Pulumi? At the moment, my SQL Azure database isn't accessible from within my AKS cluster and I would like to go about changing that so all of those resources appear on the same network.

Edit: I should note that I am using the C# provider to do this

Have you had a look at https://www.pulumi.com/docs/reference/pkg/azure/sql/firewallrule/ by chance? It has an example as well.

No I haven't! That is perfect - let me go try that out.

Yep, that helped a lot! Creating a VNet rule and a ServiceEndpoint for the same subnet that my AKS cluster uses did the trick.


Expose an API to log out the current account and clean up any related cached credential

Related to https://github.com/Azure/azure-sdk-for-net/issues/31498

We decided not to add API for this because we can't guarantee it would work as expected and there will be workarounds for clearing the cache. See https://github.com/Azure/azure-sdk-for-net/issues/32048#issuecomment-1485930145 for more information.


Fixed #304 Made title for event archive

Coverage remained the same at 100.0% when pulling 8ff56da1fc52e4f8b0c39d981c5ee98732a1cf96 on 304_archive_label into 227ef678311fe9fa13991651077a06529a8a6e5f on dev.

Coverage remained the same at 100.0% when pulling 8ff56da1fc52e4f8b0c39d981c5ee98732a1cf96 on 304_archive_label into 227ef678311fe9fa13991651077a06529a8a6e5f on dev.


INSPIRE: Fix abbreviation mappings for type code

Originally on 2011-09-08
e.g.,
Indeed it works. Only problem is that 980 has not all conf tags.
  fin tc c*165,340*
  980:conference*165,340*
  690c:"conference paper"**247,933*

This is no consistent behavior. For others - like review or book - it's the
other way around.
So some of this is probably in the data, e.g., we need to be consistent in the contents of 980 tags.  And that may be enough.  But we may also want to address whether for example c maps to conference or conference paper or whether one maps to the other, in the type code kbr.

Moved into INSPIRE internals


Test coverage improvements

The main.py module had been left mostly untested as far as coverage showed, so this PR adds a test to see that the main bits in there don't smoke an awful lot when plugged in.
There was needless duplication in the multiprocessing code, which is now neatly DRYed out.
In addition, use of concurrent.futures (3.2+) was replaced with multiprocessing (2.6+). (Not that it matters an awful lot with #782 pending, but still, it felt cleaner like this.)

This is great! Thank you!


Update all.json from Allure Security findings

These are the new sites for today and the corresponding top-level domains added to all.json.
Please keep in mind that the script checks against all.json from the master branch. If there are open PRs, there may be duplicates.
Also, please check if the domains added are any well-known that shouldn't be blacklisted at the top level (eg. netlify, ddns, etc) and change these to include the subdomain.
Please check them for eligibility and liveness, add the additional information (urlscan, screenshot) were applicable, and separate them in eligible, non-eligible and dead.
The curator should only copy the "eligible" ones into the spreadsheet.
Below are 50 new scam sites, along with 57 domains added to all.json:

Site
Domain

correct.zealtm.group
N/A

dapp.activationaccess.com
N/A

connect.ico-sale.com
N/A

support.bitvestnetwork.info
N/A

upgrade.synchronisedata.com
N/A

dapp.ibarachoir.com
N/A

dapp.easyseverconnect.com
N/A

sales.kubecoln.org
N/A

app.mainsecuretools.com
N/A

bot.dappwalletconnet.click
N/A

connect.sale-ico.com
N/A

wallcon.wdapwallreceivecondapp.live
N/A

connect.wallreceivecondapp.click
wallreceivecondapp.click

wallletuserconect.company
wallletuserconect.company

validationport.com
validationport.com

app.webswalletconnect.com
webswalletconnect.com

main.mainconnectedwallet.com
mainconnectedwallet.com

integrals-dexs.net
integrals-dexs.net

connect-mainnet01.ddns.us
ddns.us

bankk-11.cf
bankk-11.cf

integral-dex-bridges.net
integral-dex-bridges.net

fastautolinked.com
fastautolinked.com

sync.cryptoflips.net
N/A

lopezgarciaabogados.com
lopezgarciaabogados.com

walldmconecttkprecieve.live
walldmconecttkprecieve.live

connect-mainnet.ddns.us
ddns.us

vnwerolivh-3w4erhgv-38ih49-ghwehgfv-3w4g.obs.na-mexico-1.myhuaweicloud.com
myhuaweicloud.com

integral-dexs.com
integral-dexs.com

aliancrypto.com
aliancrypto.com

web.portalsync.live
portalsync.live

walletconnecti.com
walletconnecti.com

apps-syncs.net
apps-syncs.net

q2rv.com
q2rv.com

defisolutionprotocol.online
defisolutionprotocol.online

wallet.appsregistry.io
appsregistry.io

wcvalidate.dataconnectted.com
dataconnectted.com

m26securiter.info
m26securiter.info

wlt-register.live
wlt-register.live

trustwallet-verifyid-16.ml
trustwallet-verifyid-16.ml

connectiwallets.com
connectiwallets.com

wcsynchprotocols.com
wcsynchprotocols.com

mobilewalletsdapp.com
mobilewalletsdapp.com

connectwallet.waconlinkks.click
waconlinkks.click

app.connectdappsmove.com
N/A

ajudacef.com
ajudacef.com

naskizz.com
naskizz.com

migratev2finance.teachablei.com
teachablei.com

integrals-dex.com
integrals-dex.com

walletdappvalidator.io
walletdappvalidator.io

syncrvalidation.pagekite.me
pagekite.me

Eligible

Site
Urlscan
Screenshot

Non-eligible

Site

Dead

Site

@ical10 This is the list of sites for today. Some of the first ones might also be in the previous two days, but I think I've fixed the script to not check those in the future.

@ical10 Sorry, scratch that. I made a mistake. I'll close this and try again


chore: remove unused dependencies

Due to the refactor into ipld-explorer-components there are
dependencies in the package.json which are no longer direct
dependencies.

Superceeded by #47


[Fix #479] Add support for yearly tasks with custom month values

Right now there is no way to specify the month value for yearly tasks. In fact, yearly tasks are just handled as a special case of monthly tasks. This PR adds dedicated support for them, along with custom at values.
This should fix https://github.com/javan/whenever/issues/479.

Apologies, it seems there is already an open PR for this from 2 years ago: https://github.com/javan/whenever/pull/522.
That one used a different approach, further extending the code responsible for monthly tasks, while I thought it'd be more fitting to have a dedicated piece of code handling yearly tasks.
Is there any reason that one hasn't been merged in the meantime? Should I close this one?


Made it possible to have a property named "$ref"

For a few weeks we've had a test from the json-schema common test suite
failing on master. The test was checking that it was possible to
validate a property named "$ref" (and that it wasn't reserved or
interpreted as a json-reference).
The problem was that whenever we had a property name that has special
meaning in json-schema (ie. one that matches the keys of the
Schema::Validator#attributes hash), we'd skip a level in the schema
object while validating, which would cause errors. To fix the problem
I've made sure we always descend one level at a time (see
attributes/additionalproperties.rb line 17).
Fixing this caused new tests to fail, in test/extends_nested_test.rb. It
turns out that these tests were depending on the old incorrect
behaviour.
What's more, the same tests were using an incorrect form of the extends
property. According to the json-schema spec, extends should take a
json-schema, or an array of json-schemas. However, these tests were
passing a string to extends. It seems that although this is incorrect,
we had code in attributes/extends.rb which would automatically correct
this incorrect syntax (ie. from {"extends": "foo.json"} to {"extends:
{"$ref": "foo.json"}}). I've removed that workaround, as well as
correcting the tests.
Fixes #357

@RST-J could you please take a look at this when you have time? It fixes test suite failures on master, so we can start merging pull requests again

Awesome! Thanks for the fast review!


Create a SAT 2.4.11 release that includes cray-sat 3.19.2

Summary and Scope
Create SAT 2.4.11 release. The most notable changes are:

Update the cray-sat container image to 3.19.2 to include sat bootprep updates to handle the name change of the hpc-shasta-software-recipe to hpc-csm-software-recipe.
Adds Snyk scanning.
Adds license-checker workflow on this repository rather than at the organization level.
Includes a new 3.3.0 version of cfs-config-util that is used by update-mgmt-ncn-cfs-config.sh. We don't use the new functionality, but it has been adequately tested and is low risk to pull in.

Issues and Related PRs

Required for CRAYSAT-1552
Addressed CRAYSAT-1399

Testing
Tested on:

surtur

Test description:
The release distribution built by Jenkins from this branch will be downloaded from artifactory and installed on surtur as a sanity check.
Risks and Mitigations
All low risk changes.
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

Correct. And I also noticed that commit message issue and fixed it up.

It looks correct to me except for the cherry-picking issue (I think you just forgot to put in the call to render_templates.sh, right?).
Also, the header of the commit message on the most recent commit should say 2.4.11 rather than 2.4.10, I think.

Correct. And I also noticed that commit message issue and fixed it up.

I think you should be able to fix the builds now by re-applying the commit(s) that did the git vendor update release.

Install test output here: https://gist.github.hpe.com/ryan-haasken/dda70f18ee038fe28955945704689bf1
Note that the management NCN components were in a failed state in CFS, so I had to follow the procedure to apply the CFS configuration manually from our documentation, which worked smoothly.


Feature Proposal: Ability to search based on table and column level badges

Enabling searching of tables by table and/or column level badges.
Expected Behavior or Use Case
User wants to look for all tables which have certain table and/or column level badges from the UI.
Currently backend supports searching by table level badges, and I am not sure if search by column level badges is supported in backend. If a table level badge is clicked on UI, all tables with that table badge are displayed. But user can't specify badges (table or column level) in the advanced search form.
Service or Ingestion ETL
amundsenmetadatalibrary
amundsenfrontendlibrary
Possible Implementation

Example Screenshots (if appropriate):
Context
https://amundsenworkspace.slack.com/archives/CHQERT0D7/p1607966876006300?thread_ts=1606856618.396600&cid=CHQERT0D7

cc @allisonsuarez


Include selenium Keys with py?

I saw the following in the docs for type and was wondering if Keys could just be part of py so it doesn't need to be imported.  It could be accessed by py.Keys or something similar.  It'd be a small but nice quality of life improvement.
# import the Keys from selenium
py.get('#search').type('puppies', Keys.ENTER)

Also, is it possible to silence the webdriver manager output?
====== WebDriver manager ======
Current google-chrome version is 90.0.4430
Get LATEST driver version for 90.0.4430

For runs that have many tests I see this throughout my output and it gets very noisy.
Thanks!

That should be a pretty quick add to the Pylenium class. Good idea!
As for the webdriver manager, I don't know how to suppress those logs cuz I've never had a need to. Usually you want that kind of info for traceability so we've always included it

Thank you!  Looking forward to the next release!

@chingc It's already released! Check it out with v1.12.4

I figured out a way to silence the webdriver logs with an automatic fixture.  :)
@pytest.fixture(autouse=True)
def no_wdm_logs():
    """Silence webdriver-manager logs.

    This is the stdout output from Pylenium that shows the use/download
    of browser drivers.  Change "autouse" to False to display its logs.
    """
    os.environ['WDM_LOG_LEVEL'] = '0'


Bl thick devel

This is a coupled solver based on FVM. Have a quick look before our next meeting. I have included a instruction file inside the steadyNACA0012 test-case directory

Codecov Report

Merging #35 into bl_thick_devel will decrease coverage by 1.2%.
The diff coverage is 0%.

@@               Coverage Diff                @@
##           bl_thick_devel    #35      +/-   ##
================================================
- Coverage             7.5%   6.3%   -1.21%
================================================
  Files                  12     17       +5
  Lines                2198   2619     +421
================================================
  Hits                  165    165
- Misses               2033   2454     +421

Impacted Files
Coverage Δ

src/thickCoupled/matu_cyl.jl
0% <0%> (ø)

src/thickCoupled/correlate.jl
0% <0%> (ø)

src/thickCoupled/IBLThickCoupled.jl
0% <0%> (ø)

src/thickCoupled/IBLFV.jl
0% <0%> (ø)

src/thickCoupled/runScr.jl
0% <0%> (ø)

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 e421159...3f76187. Read the comment docs.


Forward attributes on type declaration to definition

This'll allow things like #[derive(Clone)] or #[derive(Debug)] to control
traits for these types.

Er -- nvm, these are imports of JS things not exports of Rust things.


[Bug] [Module Name] Bug title After deploying dinky-release-1.19-1.1.0.tar.gz for the first time, there is no data. When I click on the workbench, an exception is reported.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Caused by: org.springframework.jdbc.BadSqlGrammarException:
Error querying database.  Cause: java.sql.SQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')) AND tenant_id = 1' at line 1
The error may exist in org/dinky/mapper/JobInstanceMapper.java (best guess)
The error may involve defaultParameterMap
The error occurred while setting parameters
SQL: SELECT id, tenant_id, name, task_id, step, cluster_id, jid, status, history_id, error, create_time, update_time, finish_time, duration, failed_restart_count, creator, updater, operator FROM dinky_job_instance WHERE (id IN ()) AND tenant_id = 1
Cause: java.sql.SQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')) AND tenant_id = 1' at line 1
; bad SQL grammar []; nested exception is java.sql.SQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')) AND tenant_id = 1' at line 1
What you expected to happen
不出现报错，兼容无数据的情况，下面版本选不了1.1.0，下载的地址为：https://github.com/DataLinkDC/dinky/releases/download/v1.1.0/dinky-release-1.19-1.1.0.tar.gz
How to reproduce
下载包https://github.com/DataLinkDC/dinky/releases/download/v1.1.0/dinky-release-1.19-1.1.0.tar.gz，部署后，登录，点击工作台查看日志
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

目前无法复现, 请提供详细复现流程步骤

已复现, 已修复


April P2 tickets part 3 and 4

Closed Tickets
Part 3
#400
#402
#405
#410
#412
#421
#422
#424
Part 4
#429
#433
#435
#436
#446
#447
#449
#453
#458
#459
#460
Programs added
modinfo
webpack
crew
grubby
zypper
getent
gpasswd
pwdx
wg

Tickets Closed
#442
#319
#320
#323
#327
#329
#331
#333
#358
#359
#365
#373
Programs added
peaclock
unimatrix
lolcat
fluxbox
recode
tldr
clib
jstat
jmap
firewall-cmd

Correction.. April P3 tickets Part 1**


Replace strangeness

I am struggling to understand what replace is actually doing.  I am trying to debug my test code in chrome dev tools,  running polymer serve and calling the test.html file at its correct url.
I am replacing elements in an overall fixture which contains my top level element pas-app.  This has some top level elements (which I am replacing with mocks) alongside an app-header-layout, and within that an app-toolbar.
The issue appears to be that whilst replace is updating the object model of the dom, the dom itself is incomplete.  In particular looking at the elements tab in the chrome dev tools at my fixture I can see the pas-app element, with the correctly replaced top level elements and the app-layput element.  But within that I can only see the div id="contentContainer", but NOTHING within that div. Specifically the app-toolbar is missing.
However the $ collection from pas-app shows the ids of all the elements not showing in the dom, and manipulating them via code works. (specifically, calling click() on them fires events/observers that manipulate the properties of contained elements that then propagate via databinding up to my pas-app element. So  my tests pass.  The only ones that don't are trying to tranverse the dom using the $$(selector) call.

I pulled down the master version from github as opposed to the version bower was giving me and this problem is fixed.


feat: an option to throw error of result

Closes #119

Looks great but I recon we replace throw_error_of_result: boolean with something like error_handling and the following enum:
/// The mode which the error handling is done in the bindings.
pub enum ErrorHandlingMode {
   /// Errors will be thrown
   Throw,
   /// Errors will be returned as a Result enum
   Result,
}

so in the future when we solve #89 we can just add a variant for it.
Also feel free to ignore CI, happy to merge with it failing cause that's all problems on main.

I have reimplemented with ErrorHandlingMode enum

Thanks!


📃: Bubble Game

🔴 Title : Bubble Game
🔴 Tech stack : Html, Css, Javscript, Three js, Canvas
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name :
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.): GSSOC, Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

Can you add the gssoc-1,2,3


Added all sorting algo in one folder

Added all sorting algo in one folder

Excellent work @deytulsi18 Thanks a lot for making this PR.

Welcome


Is it possible to move the position of node labels from the right hand side to the top of the node?

Right now, the default label for nodes is on the right hand side.
My problem is that when i connect two nodes the edge goes through the label making it hard to see what the label actually says.
Is it possible to move the label to the top position of the node?

Take a look at this pull request, it adds label alignment settings which also allows you to place labels just above nodes.

I tried to set 'defaultLabelAlignment: "top"' but to no avail, is that the right value for setting labels above nodes or I am missing something?

Ah nevermind, so what I need to do was basically copy the pull request to my own cloned source, build it myself and deploy the javascript.


Revert "Experiment to use regular import instead of lazy loading. Try to fix …"

Reverts vlakyi/invoice-app#11

Regular import fixed issue. Will investigate lazy loading with netlify in the future.


Fix missing immediate wire to ALU control unit in pipelined CPU

Thanks! Not sure how I missed that. I'll write down 5 points extra credit for you on the assignment since you both found and fixed the error.

Wouldn't I get 10 points based on this page?
https://github.com/jlpteaching/ECS154B/blob/master/extra-credit.md

:)
When I said "bug" I meant in the code. I would consider this a documentation bug, which isn't worth quite as much.
But sure. This time I'll do 10, next time I'll be more clear about documentation vs. code.


s3 doesn't populate connection details in secret

What happened?

When creating s3 bucket with "writeConnectionSecretToRef" the secrets gets created but it is empty. Expected result: to have id (bucket name).
How can we reproduce it?

cat <<EOF | kubectl create -f -
apiVersion: s3.aws.upbound.io/v1beta1
kind: Bucket
metadata:
  generateName: upbound-test-bucket-
spec:
  writeConnectionSecretToRef:
    name: test-bucket
    namespace: default
  forProvider:
    region: us-east-1

After the bucket becomes available
kubectl get buckets.s3.aws.upbound.io
NAME                        READY   SYNCED   EXTERNAL-NAME               AGE
upbound-test-bucket-szdjn   True    True     upbound-test-bucket-szdjn   17m

The secret has no data:
kubectl get secret
NAME          TYPE                                DATA   AGE
test-bucket   connection.crossplane.io/v1alpha1   0      19m

What environment did it happen in?

Crossplane Version: 1.13.2
Provider Version: 0.40.0
Kubernetes Version: 1.27.4
Kubernetes Distribution: EKS

There was a similar issue with sqs which got fixed here: https://github.com/upbound/provider-aws/issues/730

Hi @candonov,
Thank you for raising this. We would be happy if you would like to contribute, this guide will help you.

Fixed in https://github.com/upbound/provider-aws/pull/951


Fixed watch command on Windows platform.

Windows equivalent of 'which' is 'where', so just used the appropriate command. I haven't tested this fix except on Windows, but I'll trust you can double-check it :) I had problems getting tests to run; they just seemed to die without message after a few tests, so I haven't added an appropriate test for the fix. I have only used rspec/riot for testing and life is too short to try to work out what is wrong - Sorry!
Originally patched master. Just a repeat on the dev branch.

I ran into this too; hacked my way around it; and forgot all about reporting it. Thanks for the patch.
However I think it misses the more fundamental issue that Windows has no widely-used equivalent to either 'growlnotify' or 'notify-send.'
As written, your patch will discard the notifications for most Windows users.
In my opinion, Notifier.notify by default ought to just emit 'message' to std out, which is what I hacked it to do.  On windows, I run 'start nanoc watch' which then gives me a new console window with the notifications scrolling by.  That's plenty for me.

It will still just fail if you don't have a notifier installed, yes, but it won't work any worse than before and will give a more meaningful error message (saying that there isn't a notify installed rather than saying that "which" isn't installed). ! I installed http://www.growlforwindows.com/gfw/ and tweaked the code so nanoc would find it, which works fine for me now. I didn't have to do anything odd when starting the watching; just started it in a standard cmd window or via Rubymine and it output fine.
I did wonder whether using something like http://rubygems.org/gems/ruby-growl might not be a better cross-platform solution though, but didn't want to try to hack that in when I imagine that OS X and Linux are managing quite well and Windows can manage with a simple installation of a couple of files.

I like your fix; yes it's an improvement. I was just commenting on the general idea.
Incidentally, do you feel like you get a lot of value out of growlforwindows?  ...I've always dismissed the idea without giving it much of a chance.  Am I missing much?

I have never used it before, but it seems to work correctly and instantly; I was worried that for Windows it would take a long time to notice any change. watch seems a bit unstable, but I think that is more about nanoc itself than the notifier it uses (it sometimes crashes out when it fails to compile).
Ideally I would be using autocompile rather than watch, but that takes 10x longer to serve every page access than watch does to recompile on each change, so I assume there is something fundamentally broken in there, though I don't know if that is a Windows-specific problem.


fix: equality of iterable with symbols

Summary
Fixes #13936
Test plan

https://github.com/jestjs/jest/releases/tag/v30.0.0-alpha.2


Comparison Should include Azure Stack Edge

this Comparison Should include Azure Stack Edge for completeness.

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

ID: afda862d-5084-45c0-43d6-fa1cf6723e2a
Version Independent ID: 12cb954d-fd9d-0877-6437-31bdd292abb2
Content: Differences between global Azure, Azure Stack Hub, Azure Stack HCI - Azure Stack Hub
Content Source: azure-stack/operator/compare-azure-azure-stack.md
Service: azure-stack
GitHub Login: @Justinha
Microsoft Alias: justinha

@SumeshP Thanks for the feedback.
Assigning content owner @Justinha for visibility, review and document enhancement consideration as appropriate.

All 3 Edge options. Edge zones, private edge zones, carrier edge zone
https://azure.microsoft.com/en-us/blog/microsoft-partners-with-the-industry-to-unlock-new-5g-scenarios-with-azure-edge-zones/

@KrishnaG-MSFT @Justinha any news/plans?

@sethmanheim

@sethmanheim

Hi @KrishnaG-MSFT, can you please assign this issue to me? Thanks!

Hi @KrishnaG-MSFT, can you please assign this issue to me? Thanks!

@SumeshP Thank you for reaching out and sorry for the slow response. We do have this on our list to update. If you still need assistance, please open a new feedback request. #please-close


Smaller dependency count

After some analysis I found that the two largest dependencies we have are web3 and ipfs. Collectively they add 317 dependencies to the project with a total of 438 dependencies.
Most of the features they are providing are fancy wrappers around http get requests. It wouldnt be that hard to create our own smaller abstraction to use instead of this.
We should remove npm from packages.
bignumber.js and mocha can be moved to devDependencies

Looks ethers has a much smaller dependency count and a lot of cool features. I think we should switch eventually


There is a problem in training

unsupported format string passed to Tensor.format

And， how to save this model as .h5 file? please help me,thanks.


Avoir voting button overlapping with question text

Good catch! Thanks for the fix. LGTM. I'm going to merge this and then deploy.


link under "For more information, see Profiles in the Adobe Target Admin and Profile API guide." - profiles link is going to old developer doc portal

Issue in /src/pages/before-implement/methods-to-get-data-into-target/profile-api-settings.md

Fixed


Expand-Archive does not return a System.IO.FileInfo nor System.IO.DirectoryInfo

The docs (https://msdn.microsoft.com/en-us/powershell/reference/5.1/microsoft.powershell.archive/expand-archive) say that Expand-Archive should return a System.IO.FileInfo or System.IO.DirectoryInfo but it has no output

This was fixed in MicrosoftDocs/PowerShell-Docs#3696 on Feb 12. You need to run Update-Help -Force to get the new help files.

This is a duplicate of issue #9.


Small typo in AWS Deploy doc

user -> users

@googlebot I signed it!

@googlebot I signed it!

/approve

/approve


Create visualstudio

vs1

This is not a valid PR. Looking at the history of this account, it looks like a spam account.


[FR] Exclude todos below a certain header

Hi! I want to start using your plugin, I think it might be really useful.
I have one problem though. I also put my habits and trackers in my daily note, and I don't check all of them. But I don't want them to rollover into the new daily note. Thanks!

Fixed as soon as #79 is merged


file not found

In the foundation project there are some reference like
/// <reference path="../../foundation.core.htmlclient/foundation.core.d.ts" />
/// <reference path="../../foundation.core.htmlclient/foundation.core.d.ts" />
/// <reference path="../../foundation.viewModel.htmlclient/foundation.model.context.d.ts" />

to some files, but those file does not exist.
and also the foundation.core project does not exist too.
/// <reference path="../foundation.core/typings.d.ts" />

Hi
its about your path.
every "../" means a folder above from original path.
please check your path.

dear mo-kml
I have installed all of those projects. The files foundation.core.d.ts , foundation.core.d.ts ,... and the project foundation.core does not exist, i search these files in The whole of project,
Are these files created automatically?

@miladra Yes, those files are being generated automatically, foundation.core.d.ts is a result of building foundation.core.csproj for example. You can build bit solution using instructions provided in read me file. In case you encounter any problem, you're welcome to submit an issue right here :)

https://github.com/bit-foundation/bit-framework/blob/master/README.md


ensures computed protocol is HTTP/2.0 instead of HTTP/2

Changes proposed in this PR

ensures computed protocol is HTTP/2.0 instead of HTTP/2

Why are we making these changes?
The Waiter code expects the client protocol to be HTTP/2.0 while processing http/2 requests. This change ensures that we compute the protocol correctly as the rest of the Waiter codes expects HTTP/2.0 as the value for the client protocol.

merge when green


Temporary files for the classpath are never cleaned up

Version 4.2.0 and earlier.
After successfully executing SpotBugs a couple of times, the temporary directory is cluttered with files produced by
https://github.com/spotbugs/spotbugs-gradle-plugin/blob/466f322c0608ef69d66c10a3528571cf8ff4caf4/src/main/groovy/com/github/spotbugs/snom/internal/SpotBugsRunner.java#L143
After executing the task, they should be cleaned up.

I've noticed the same thing and I made some changes that will contribute to this issue so hopefully this can be fixed.

Task#getTemporaryDir() is a better place for this file. API doc.
Just note that this cannot be relied upon by downstream tasks:

There are no guarantees that the contents of this directory will be kept beyond the execution of the task.


fix(foundryup): warn macos users without libusb

should close https://github.com/gakonst/foundry/issues/498


Unset pull policy to enable always pulling latest

To better support development use cases where we want to always pull development images, but then also only pull production images once we can leverage the behaviour described at this location [1]
[1] https://spacelift.io/blog/kubernetes-imagepullpolicy#image-pull-policy

/lgtm
/approve


Upload metrics to metrics center from test runner

DeviceLab test metrics have been uploaded to metrics center by calling cocoon from test runner. Since metrics center migration has been finished, it makes sense to port the logic into test runner.

[x] Add support to upload metrics from test runner
[x] Recipe change to support uploading metrics from test runner
[x] Validate changes and clean up old cocoon logic.

Related: https://github.com/flutter/flutter/issues/88296

https://flutter-review.googlesource.com/c/recipes/+/17200 to start uploading metrics from test runner.

Tests are failing:

when ResultData is null

type 'Null' is not a subtype of type 'Map<String, dynamic>' in type cast

metrics center token is not working in cocoon API authentication

Successfully uploaded metrics to metrics center
AuthenticatedClientError:
  URI: https://flutter-dashboard.appspot.com/api/update-task-status
  HTTP Status: 401
  Response body:
null is not allowed

Reopen as this hasn't been fully implemented.

Test runner has been uploading metrics successfully. Closing.


Package the nodejs code with babel instead of webpack

Fixes https://github.com/lightstep/lightstep-tracer-javascript/issues/67
cc @bcronin
I don't have strong feelings about the directory structure or build rule structure. I did the minimum viable thing to put it into practice

Thanks, Nick! I'm a bit swamped at the moment, but will check this out as soon as I can!

(To be clear, "as soon as I can" means by early tomorrow...I do want to get this in!)

Nick, I got a change to try this and it looks great. Thanks.
Looks like you're done the hard part. Per your point of "I don't have strong feelings about the directory structure or build rule structure", I might make a few minor (easy) changes to be more consistent with our other client libraries. I'll try to get this taken care of tomorrow.

@nicks , I'm closing this and merging #74, which is built off of these changes. Thanks for doing this!


Touch application choices if a candidate updates their email

Context
Came from a provider testing updating Candidate emails and that not reflected in the API using the since parameter
Changes proposed in this pull request
Touch application choices if a candidate's emails is updated
Guidance to review
Probably should scope this to current recruitment cycle only
Link to Trello card

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

@stevehook oops I missed this, I'll raise a separate pr

@stevehook https://github.com/DFE-Digital/apply-for-teacher-training/pull/5719


fix: Make the add page responsive

Summary
Adding Responsive Layout to add-blog page
Description
add-blog page is not responsive. Now, adding responsiveness to add-blog page according to Laptop, Tablet, Mobile.
Images
Mobile:

Tablet:

Laptop:

Issue(s) Addressed

Closed #15

Review Checklist
Please make sure to go through this checklist before submitting your review.

[X] Make sure the PR title is of format tag: Issue Title

tag should be one of fix,feat, docs, chore, refactor
Ex: feat: Added responsive design for the home page

[X] Ensure the code follows our coding standards and guidelines

Ex:  naming conventions should be of form file-name.ts not fileName.ts

[X] Ensure that you have staged only the required commits and raised the PR and not included all the files which would be otherwise required for local setup
[X] Check if the PR is rebased to the latest main/development branch.
[X] Make sure there are no unnecessary files or debugging artifacts in the PR.

Hi @SankalpHaritash21 , i still dont think header and hint texts are 22px and 16px respectively, please check the same.

I updated text-base which is accurately 16px. Now values are not hard coded


Added a block argument to at_exit

This feature becomes able to check an error in a at_exit block. It is useful to run your code when main script finished. See kemal.cr for example. It uses at_exit to starts a HTTP server after main script. It is good. However, it starts the server even if main script failed. I think this behavior is wrong. If this PR is merged, I will fix like following code.
at_exit do |status|
  if status == 0
    # starting server process here
    # ...
  end
end

Looks good. How would you handle the same situation in Ruby?

See the entry point of sinatra. Ruby has $! special variable which is the last raised error. This special variable is useful in only two cases: first is such a case, the another is that I want to get an error in posfix rescue. The problem of this implementation does not consider the case to end program with exit method.

Thanks for the explanation. Passing the status code to the callback is good, we have that info and we are not using it :-)


Issues with background colors of Lf_hl_cursorline and Lf_hl_match

vim or neovim?

[x] vim
[ ] neovim

Output of vim --version or nvim --version:

VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Apr 17 2020 13:03:59)
Included patches: 1-587
Modified by Solus
Compiled by Solus
Huge version with GTK3 GUI.
If I set Lf_hl_match's background to NONE and Lf_hl_cursorline's background color to, say, Green, I would expect that all characters in the current cursor line (including matches and non-matches) should have the same backgournd color--Green. In reality, the non-matching characters have a green background. However, the matching characters have some background that I am not sure where they are from.

As can be seen above, the matching characters DSI have different backgrounds than other characters in the red cursor line. This should not have happened since I set the background color of Lf_hl_match to NONE.
I should also point out that if I set Lf_hl_cursorline's background color to NONE, then all characters in the cursor line (matching or non-matching) have the same background color.
Expected behaviour
If Lf_hl_match's background color is set to NONE, then the matching characters in the cursor line should have red background in the above screenshot.

I don't know how to fix it. They both use matchadd to highlight, can you achieve your expected behavior?
For example, there is text like aaabbbaaa in vim,
hi aaa guifg=White guibg=Red
hi bbb guifg=Yellow guibg=NONE
call matchadd('aaa', 'aaa.*aaa')
call matchadd('bbb', 'bbb')

Yeah the result seems not as expected:

Do you think it would work if we do the following way?

Define internal highlight groups (invisible to users) for matching characters in the cursor line, one for each of Lf_hl_match, Lf_hl_match0, Lf_hl_match1, ... The colors of these groups mostly derive from the corresponding Lf_hl_matchX, except for the background color.

When deciding the background colors (let's call it internal_bg) for these internal highlight groups, first check if the background color of Lf_hl_macthX (let's call it match_bg. If match_bg is something other than NONE, then internal_bg should match match_bg.

If internal_bg is NONE, then we check the background color of Lf_hl_cursorline (we call it cursorline_bg). If cursorline_bg is something other than NONE, then internal_bg is set to cursorline_bg.

If cursorline_bg is also NONE, we set internal_bg to the background color of Lf_hl_popup_window, no matter it is NONE or not.

I don't know whether it is useful, but I got some of these ideas from here: https://stackoverflow.com/questions/17728343/word-under-cursor-highlighting-and-multi-highlighting-in-vim
If Lf_hl_match's background is NONE, then matching characters in the cursorline should have the same background as Lf_hl_cursorline. If Lf_hl_cursorline's background color is also NONE, then `

Do you think it would work if we do the following way?

Define internal highlight groups (invisible to users) for matching characters in the cursor line, one for each of Lf_hl_match, Lf_hl_match0, Lf_hl_match1, ... The colors of these groups mostly derive from the corresponding Lf_hl_matchX, except for the background color.
When deciding the background colors (let's call it internal_bg) for these internal highlight groups, first check if the background color of Lf_hl_macthX (let's call it match_bg. If match_bg is something other than NONE, then internal_bg should match match_bg.
If internal_bg is NONE, then we check the background color of Lf_hl_cursorline (we call it cursorline_bg). If cursorline_bg is something other than NONE, then internal_bg is set to cursorline_bg.
If cursorline_bg is also NONE, we set internal_bg to the background color of Lf_hl_popup_window, no matter it is NONE or not.
Use these internal highlight groups to render the matching characters in a cursor line.

the matching characters are not highlighted line by line, they are highlighted only by, for example,
call matchadd('bbb', 'bbb').
This method is too tricky.  In my opinion, if you want to have color background of cursorline, you can define the color of CursorLine.

In my opinion, if you want to have color background of cursorline, you can define the color of CursorLine.
Setting CursorLine would also affect the look of texts in the vim buffer is the cursorline option is set, but I only want to change the look of the popup window. Originally I thought that setting Lf_hl_cursorline will do the job for me, but it seems that Lf_hl_cursorline controls the look of text in the cursorline, and CursorLine controls the background of the whole cursor line, including the part that does not have text.

Is it possible to have another highlight group to control the background of the cursor line in the popup window, rather than let CursorLine in control of that?

Lf_hl_cursorline.

Yeah I tried that and that only affects the texts in the cursorline, not the whole cursorline. I mean the overall background of the cursorline is set by CursorLine. Is it possible to create a highlight group in LeaderF to control the line-wise background of the cursor line?

I'm afraid there is no way.

I see. Thanks for your quick reply and this awesome plugin.


Flashlight spawner

TEST PLAN

Put this server script on any collisionless, non-grabbable item (spawner item).
A flashlight should appear that can be grabbed
When the flashlight is moved more than 0.1m from the position of the spawner item, a new one appears within 5 seconds.

Testing.

Testing complete.
Put this server script on any collisionless, non-grabbable item (spawner item). Pass
A flashlight should appear that can be grabbed Pass
When the flashlight is moved more than 0.1m from the position of the spawner item, a new one appears within 5 seconds. Pass
All Pass :)

@misslivirose @RebeccaStankus A few questions with this PR. While the flashlights are re-spawning / you can grab them, i am not seeing them with a "put down decay" timer, so could we end up with a domain full of flash lights? also they do not attach to the player light an avatar item (see the Material gun or Rainbow Cannon in the spot.) like i would expect? So people on desktop can NOT use them and you have to hold on to them the entire time you use them.
Is this intentional? or should i expect them to function like the material gun in the spot?

@RebeccaStankus did we get the scripts adjusted much like you did with the rainbow gun in the spot for the flashlights in haunted? Think im going to send this Salinas way if you have?

@AndySeattle - The forest domain isn't intended to be a desktop domain given the darkness anyway - I think these are a pass given that they're deployed in there already and have lifetimes on them. Given that we're more than halfway through October I'd like to merge this.


problems when using optical_flow in iGibson 1.0

Hi, I encountered several problems (possibly bugs) after I added optical_flow to sensors.
I conducted my experiments with iGibson 1. 0 (commit: 893fcda1a1d01ce2da5102d500d2d16f35a62162)
First, I think there is a bug in https://github.com/StanfordVL/iGibson/blob/893fcda1a1d01ce2da5102d500d2d16f35a62162/gibson2/sensors/vision_sensor.py#L108
return raw_vision_obs['optical_flow'][:, :, :3]
should be
return raw_vision_obs['optical_flow'][:, :, :2]
Second, I find if I want to use optical_flow, I have to change optimized to True in MeshRendererSettings, otherwise self.trans_data and self.rot_data in MeshRenderer will be None, leading to error for updating dynamic positions with need_flow_info=True. Then, vertices and textures will be optimized when the scene is first loaded. Is it necessary to do this optimization process for rendering optical flow?
Third, I always encounter a  CUDA error after even 1 million frames or more, while this error never occurred when I only used RGB and depth as inputs.
/pytorch/aten/src/THC/THCTensorRandom.cuh:166: void sampleMultinomialOnce(long *, long, int, T *, T *, int, int) [with T = float, AccT = float]: block: [1,0,0], thread: [0,0,0] Assertion `THCNumerics<T>::ge(val, zero)` failed.
/pytorch/aten/src/THC/THCTensorRandom.cuh:166: void sampleMultinomialOnce(long *, long, int, T *, T *, int, int) [with T = float, AccT = float]: block: [1,0,0], thread: [1,0,0] Assertion `THCNumerics<T>::ge(val, zero)` failed.
/pytorch/aten/src/THC/THCTensorRandom.cuh:166: void sampleMultinomialOnce(long *, long, int, T *, T *, int, int) [with T = float, AccT = float]: block: [1,0,0], thread: [2,0,0] Assertion `THCNumerics<T>::ge(val, zero)` failed.
...
RuntimeError: CUDA error: device-side assert triggered

And I find it's because there are some nan values in the optical flow. How can I fix this problem?
I'm looking forward to your reply and thank you so much!

As an alternative method of calculating optical flow, the intrinsic matrix of the realsense camera in the simulator is helpful. If the above issues are difficult to fix, can you provide this info for a quick solution?
Thanks!

@fxia22 , has this been addressed in the newest versions? Should we close this pending issue?

This issue has been fixed.


337 copywright validation in macedonian

Creative part of the copy remains unchanged & the designated team should have a second look at it.

@y0vche not sure if you used --no-verity when commiting this, but you have a lot of prettier errors, and husky shouldn't have let you to commit this.
runnpm run build in the terminal to see errors

I will close this PR since another team member already finished this


Allow disabling the "preserveUnknownFields" option in CRD

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
n/a

License
Apache 2.0

What's in this PR?
This allows installing the operator on Kubernetes 1.14 by optionally disabling the preserveUnknownFields flag in the CRD manifests.
Why?
Because I am still running 1.14 and want to use the operator 😆
Checklist

[x] Implementation tested
[x] User guide and development docs updated (if needed)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution, can we please use helm feature called Capabilities.
Using that, helm can dynamically decide whether to use this field or not.
https://helm.sh/docs/topics/charts/

Thanks for the contribution, can we please use helm feature called Capabilities.
Using that, helm can dynamically decide whether to use this field or not.
https://helm.sh/docs/topics/charts/

Ooh cool, I didn't know that was a thing. Will update the PR!

Sorry, I completely forgot about this! I did have a go at using .Capabilities.KubeVersion but I couldn't find a nice way of comparing the version string in the template. Any thoughts on how to do that cleanly?

I just checked the official helm chart repo and I think a nice thing to do is like here in the Traefik chart:
https://github.com/helm/charts/blob/24df78eb1835f1b6a605c0cccb487990d6f8dbf4/stable/traefik/templates/rbac.yaml#L8

Huh, interesting. semverCompare does not appear in the helm docs at all, or even the go template docs!

https://golang.org/pkg/text/template/
Will give that a go though! Will be some time next week as I am moving house this weekend.

Yeah because helm using not just the go template functions but the Masterminds sprig lib as well.
http://masterminds.github.io/sprig/

Hi @aaron-trout any update on this? Do you need any help from our side?

Hey, sorry for the slowness on this! It's been a busy few weeks and we have actually upgraded to 1.15 now so less of an issue for us. Do you still want to add this or will you drop support for kubernetes <= 1.14?

Okey then I think we can close this, as kubernetes 1.14 is so old, so we can safely drop the support.


cmd/go: default SUMDB to 'off' or 'direct' when GOPROXY is a file URL

I believe this is the root cause of the symptom reported in #32216.
(*modfetch.dbClient).initBase attempts to derive the GOSUMDB setting from the GOPROXY setting. GOPROXY may be set to a file:// URL, in which case the call to web.GetBytes here will fail.
Since the sum database requires an HTTPS transport, we should probably default it to either direct or off if the only GOPROXY values are file URLs.
CC @FiloSottile @bradfitz @jayconrod

@bcmills I'm new here, can I send a PR for this?
base on your suggestion, I believe having a check like this should work
if strings.HasPrefix(proxyURL, "file://") {
	break
}

@pyjac, the behavior here is pretty subtle, so I'd rather have @rsc clarify the intended behavior before we move forward. Thanks!

To clarify, it's not trying to derive the GOSUMDB setting so much as the connection transport for getting to it.
I think we do want file:// proxies to be able to provide GOSUMDB files, since the checksum database is (or can be) a static file tree too. But when it does try the file URL, it needs to get a 404 from the sumdb/supported endpoint, so that it can fall back to the next thing it would normally try (probably a direct connection).
So I think we just need to make the checksum fetches use the same access mechanism that the proxy fetches do, so that file:// is implemented during that fetch, and we also need to make sure that the response is a not-found, but that seems like it will work OK, since it is already using errors.Is(err, os.ErrNotExist).

Worth remembering to check that  tags can't point at file:/// if you make the general web library handle those.


Request to stream a chat completion results in buffered response instead of stream

What Happened?
I tried running the gateway locally using npx @portkey-ai/gateway, node run dev:node and node build/start-server.js (Node 18). I noticed that even though I was specifying the "stream" parameter in the config the response was buffered and returned all at once to the client instead of streamed back. I also noticed that this is somehow related to the "compress" middleware step. After removing the following middleware, the response was streamed correctly:
// app.use('*', (c, next) => {
//   const runtime = getRuntimeKey();
//   if (runtime !== 'lagon' && runtime !== 'workerd') {
//     return compress()(c, next);
//   }
//   return next();
// });

What Should Have Happened?
The response should be streamed to the client.
Relevant Code Snippet
code on the client side

const run = async () => {

    const client = new OpenAI({
        apiKey: "xxx",
        baseURL: "http://<IP_ADDRESS>:8787/v1",
        defaultHeaders: {
            'x-portkey-config': JSON.stringify({
                "provider": "openai",
                "api_key": "xxx",
            })
        },
    });

    const stream = await client.chat.completions.create({
        model: "gpt-3.5-turbo",
        messages: [{ role: "system", content: "You are a helpful assistant." }, { role: "user", content: "Write a short story about Mumin" }],
        stream: true,
        max_tokens: 254,
    });

    for await (const part of stream) {
        console.log(part.choices[0]?.delta?.content || "", new Date());
    }
}

run()

Your Twitter/LinkedIn
No response

Issue can be closed

@au-re thank you for raising the issue and @mohankumarelec for fixing this!! This is amazing. Thanks to @flexchar & @arbi-dev as well.


"Protobuf gencode version" warning

Just bumped from 0.16.0 to 0.18.1, and I'm getting these warnings now, looks like fallout from https://github.com/authzed/authzed-py/pull/191.
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/core.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/error_reason.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/experimental_service.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/permission_service.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/debug.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/schema_service.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
/.venv/lib64/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.3 is older than the runtime version 5.28.0 at authzed/api/v1/watch_service.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(

I don't think there's a real breaking change, so I'm just gonna filter the warning out, but it seemed worth noting.

Hmm... This test should theoretically catch that. Gencode being older than runtime code should also be generally okay, as long as semver boundaries aren't being crossed.
That said, the intended use case is that the version of protobuf used to generate the code is the same as the version used to run the code. I think the vision is that anyone consuming a gRPC service is doing their own code generation, so us doing the generation and providing the lib is breaking that expectation somewhat. We also don't want to be too prescriptive as library authors and provide too narrow a dependency range that then breaks someone's usage who needs a different version of protobuf for whatever reason.
I think the fix for you in the meantime would be to explicitly put protobuf==5.27.3 into your dependency management file. I can add that to an FAQ or something if that ends up working.

I hear you, and I agree with you that the dependency range should be broad. After some digging, it turns out this is a new warning as of protobuf==5.28.0, so I'm gonna close this issue as they hash this one out.

https://github.com/protocolbuffers/protobuf/issues/18096
https://github.com/grpc/grpc/issues/37609

Honestly, I don't think it's uncommon for a product to provide the gRPC client to its customers, so this warning is a little weird, especially for a minor version difference. If I happened to use two different gRPC clients in one project, chances are slim that they'd match protobuf gencode versions perfectly. For now though, filtering the warning or locking the protobuf version both work well for me.


Not compiling in Android

The app is not compiling for both platforms.
flutter --clean does not solve the problem.
This is the issue for IOS: (https://github.com/flutter/flutter/issues/53776)

Logs (Android)
Output
Launching lib/main.dart on Android SDK built for x86 in debug mode...
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
Note: /Applications/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore-0.13.4+2/android/src/main/java/io/flutter/plugins/firebase/cloudfirestore/CloudFirestorePlugin.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugResources'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
   > Path '/Users/juliansanchez/Documents/Proyectos/go_agenda/build/flutter_statusbarcolor/intermediates/compiled_local_resources/debug/out' is not a readable directory.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 33s
Gradle task assembleDebug failed with exit code 1
Exited (sigterm)

flutter run --verbose
flutter-verbose.txt

flutter analyse
No issues found! (ran in 16.3s)

flutter doctor -v
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.15.3 19D76, locale en-AR)
    • Flutter version 1.12.13+hotfix.8 at /Applications/flutter
    • Framework revision 0b8abb4724 (8 weeks ago), 2020-02-11 11:44:36 -0800
    • Engine revision e1e6ced81d
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/juliansanchez/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.9.1

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] Android Studio (version 3.4)
    • Android Studio at /Users/juliansanchez/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/183.5692245/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.2)
    • IntelliJ at /Users/juliansanchez/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 39.0.4
    • Dart plugin version 192.7761

[✓] VS Code (version 1.43.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.9.0

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

! Doctor found issues in 1 category.

Hi @julisanchez
Can you post your pubspec.yaml ?
Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem
Thank yo


added support for https and changed listening port to 443

Both react as well as uvicorn support HTTPS. This eliminates the need for an SSL reverse proxy outside of the container.

I guess my concerns here, besides which port we should be using, is that I am not sure the application should be handling the logic involved with SSL. I understand that this project is a little different because we are communicating with a device we have no control over that requires SSL, yet it still seems vastly more user friendly to just ship a sanely configured NGINX container in the compose file versus deal with it all at the application layer.
That being said, I understand the pull here. It would be nice to have everything "just work" out of the box. The more I think about this, the more I think this should just be a configurable option.
Run USE_CERTS_FOR_HTTPS=true python -m feeder and we run Uvicorn with the certs. Otherwise, we just use them for the MQTT Broker.
FYI @fbretmounet, we can achieve this by adding a use_certs_for_https class variable to the Settings class in feeder/config.py. This will automatically read from the environment variable. We just need to handle the logic of checking that variable from the instantiated settings object and calling Uvicorn with the correct parameters.
To be clear, I still don't think it should be enabled by default, yet I do think it is useful to have and to document.

@mbillow , I hear your point. I am very technical and used to be a developer. I'm also a hacker, so I'm used to cobbling things together.... yet it took me 3 evenings just to get the container functional. I had to figure out how to run a reverse proxy on my NAS, had to figure out how to manually configure it. Thank god I ignored the readme regarding the need to change iptables and just put the container in my existing address space because it would have been a few more evenings of head banging.
So, what I am saying is that in its current implementation, the average petfeeder customer will not be able to use this solution unless we seriously simplify its deployment and config requirements.

@fbretmounet Please don't get me wrong... I'm not under some kind of delusion. This project is no where near the state it would need to be in for even an experienced user to just stand up.
Breaking the current install instructions out of the README and into a segmented, easy to follow Wiki is one step in the right direction though; which I hinted at in your last PR. I also really like @kruton's idea of just shipping an RPi image OS image with everything just in place.
All that said, do you think your experience would have been easier if we shipped the config's necessary to just start an nginx container (or really any kind of reverse proxy) on your NAS?

the rpi image is an interesting one. While it may make it easier for the general public, I think the container flexibility is very valuable for the more advanced users.
We already have the requirement for pihole. Most running pihole will already have it running on an rpi that is underutilized. So documenting a howto add a docker image to it might be the compromise?

I imagine for local development you could just use the HTTP port. If you specified HSTS on another port of your development server, just use the IP address :)

Closing this since I think we have made some decent effort in making setup simpler. Still a long way to go, but including NGINX containers in the compose file and shipping known-working configs for it are a step in the right direction.

Closing this since I think we have made some decent effort in making setup simpler. Still a long way to go, but including NGINX containers in the compose file and shipping known-working configs for it are a step in the right direction.


modules

menu.json has a lot of modules (.gendsp) from version 0.3.x.
how to represent them in vr?
so far, they're able to be represented. this is from menu.json.
how to represent them in the mischmasch document?
at the moment, I think just their inputs, outputs, and knobs are represented, not the interior ops.
how to represent them in genish?
if we can solve the mischmasch doc, this should work too
which of them are compatible with genish? (which has a reduced set of operators from gen~)

[ ] go through oldMenu.json, filter out the abstractions into a new file
[ ] crawl through each associated .gendsp file using code from ops2menu.js, and determine which abstractions are compatible with genish operators
[ ] those that are compatible, add them to the palette
[ ] decide on a way to represent them in the mischmasch document (the vr localGraph only cares about the abstraction-level), but the mischmasch document will probably need to have the entire .gendsp as a graph of nodes and connections, with inlets, outlets, and params (all ops) for the abstraction included


RubyGems shows min Ruby as 1.8.7, but ActiveSupport/i18n requires 1.9.3

https://rubygems.org/gems/mailcatcher shows 1.8.7 as the min, but trying to install it with that gives:

ERROR: Error installing mailcatcher: ==> default: i18n requires Ruby version >= 1.9.3.

(Via Chassis/mailcatcher#2)

Fixed in 134a99c9c1a2f75cbe1c94f75cbb58959a0176dc.


Wrong topic / labl

This repo should not be labeled as progress-bar. Thx

The label was not removed… please review.

not removing


[bug] Manual WebView resizing does not work correctly

Describe the bug
Manually resizing the WebView(set_size) results in incorrect positioning of the WebView.
However, if the WebView coordinates are set to (0., 0.), it works correctly.
https://youtu.be/02gJWTb1Hjg
Reproduction
https://github.com/kanatapple/tauri-manual-resize
Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 125.0.2535.92
    ✔ MSVC:
        - Visual Studio Professional 2017
        - Visual Studio Professional 2019
    ✔ rustc: 1.77.2 (25ef9e3d8 2024-04-09)
    ✔ cargo: 1.77.2 (e52e36006 2024-03-26)
    ✔ rustup: 1.27.0 (bbb9276d2 2024-03-08)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.11.0
    - yarn: 1.22.11
    - npm: 10.2.4

[-] Packages
    - tauri [RUST]: 2.0.0-beta.22
    - tauri-build [RUST]: 2.0.0-beta.17
    - wry [RUST]: 0.40.1
    - tao [RUST]: 0.28.1
    - tauri-cli [RUST]: 1.2.3
    - @tauri-apps/api [NPM]: 2.0.0-beta.13
    - @tauri-apps/cli [NPM]: 2.0.0-beta.20

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - bundler: Vite

Stack trace
No response
Additional context
No response

If the initial position is (1, 1) and scale_factor is 1.5 (150%), the position becomes (2, 2) in set_bounds .
In bounds.position.to_physical it becomes (1.5, 1.5), but it is cast to (2, 2).
After that, the position seems to increase from (2, 2) -> (3, 3), (3, 3) -> (5, 5), and so on.

Now that I know the cause, I'll send a Pull Reauest later.

I have a similar issue with my app. Whatever values I use, my Webview fills the bottom half of the app window.
I reported this and @FabianLars responded that it might be related to this issue.
So I cloned the repo from the OP. In that case the Webview fills the whole app Window whatever I do with the commented out code.
No matter what I do I don't seem able to affect the position / size / behaviour of the Webview in the OP example or my app. This fits with my not understanding how the multiwebview example is achieving it's layout. I don't see how that can divide the vertical space equally between (four I think) Webviews, nor how it results in them taking up the full width.
A good place to start for me would be for the multiwebview example to be explained (and fixed first if indeed it is buggy, which seems to be the case).

I have a similar issue. Updating webview size using set_size causes cumulative positioning on Windows devices where it works perfect on Mac devices. Following here..

@happybeing @hattapoglu
wry is fixed, so this issue should be fixed soon(Not sure if it's the same cause... 😅 )

Yeh, thanks. I suspect not based on the symptoms and the strangeness I see in the example - how does that work? I shall take a another look before long.


change import for github.com/googleapis/gnostic/OpenAPIv2

client-go is dependent on
https://github.com/googleapis/gnostic
The recent PR merged into the master of googleapis/gnostic/. Is flaking client-go at
k8s.io/client-go/discovery/discovery_client.go:30:2: cannot find package "github.com/googleapis/gnostic/OpenAPIv2"
The above import is renamed to
"github.com/googleapis/gnostic/openapiv2"

Also, I can raise PR addressing the same.

See resolution in https://github.com/kubernetes/client-go/issues/741#issuecomment-580957445
Using a specific version of client go will correctly fetch compatible versions of dependencies

git config core.ignorecase false --local to set git don't igrnore alpha case and then run go mod to re-gen the vendor


Get UTC/GMT time

I use:
var currentDate = new Date(new Date().getTime() + 24 * 60 * 60 * 1000);
var day = currentDate.getDate(); var month = currentDate.getMonth() + 1; var year = currentDate.getFullYear(); var startDate = year + '/' + month + '/' + day;

$('#when').datetimepicker({dayOfWeekStart : 1, lang: 'fr', step: 30, defaultDate: startDate, defaultTime: '12:00', beforeShowDay: function(date) { if (date - new Date().setHours(0,0,0,0) < 0) { return [false, ""] }; return [true, ""]; }});
$('#when').datetimepicker({format:'l d F Y à H:i'});

# Example: mardi 13 mars 2018 à 15:00

How is it possible to make that
$('#when').val();

returns the same datetime  but in UTC / GMT (instead of local timezone, which is GMT+2), or UNIX timestamp?

@josephernest I think you were meant to be tagged instead 😄

@tunnes Yes but
$('#your-selector').datetimepicker('getValue')
// returns -> Fri Apr 20 2018 00:00:00 GMT-0300 (-03)

doesn't return UTC/GMT time, it returns local-timezone time (here GTM-3).
How to get UTC (GMT+0) timezone instead?

@josephernest You're right! My bad sorry, but did you try to convert the date?
let date = new Date( $('#your-selector').datetimepicker('getValue') );
date.toUTCString();
// returns -> Fri, 20 Apr 2018 03:00:00 GMT

Thanks @tunnes, you're right, since it's a Javascript Date object, we can use the standard Javascript methods.


self.web_client.channels_list

Hi there,
Great to see the move towards slackclient v2 - thanks for all the work on the new version.
After debugging a KeyError relating to my tests with the new version (in which I was testing in a DM and a private channel) it seems the use of self.web_client.channels_list in machine/clients/singletons/slack.py may be causing it, due to it not including the same level of conversation coverage as conversations.list.  At the moment, this appears to prevent the bot responding to private channel/DM messages.
Regards,
Kris

Hi @smoothimpact, this was fixed in the latest version. You can upgrade to v0.20 to get this fix.


Fix error due to Ruby version being < 2.0.0.

When installing building my Jekyll page on Ubuntu 16.04 it crashes due to:
-----> Building Jekyll site
       Fetching gem metadata from http://rubygems.org/...........
       RubyGems 1.8.23 is not threadsafe, so your gems will be installed one at a time. Upgrade to RubyGems 2.1.0 or higher to enable parallel gem installation.
       Fetching rake 12.3.0
       Installing rake 12.3.0
       Gem::InstallError: rake requires Ruby version >= 2.0.0.
       An error occurred while installing rake (12.3.0), and Bundler cannot continue.
       Make sure that `gem install rake -v '12.3.0'` succeeds before bundling.

       In Gemfile:
       rake

To me this seems to be because the system version of Ruby is used and not the rbenv version.
As a fix I just added rbenv global <version> and that fixes the issue for me.

Makes sense! Thanks!


Error in default.rb commets in line 12

Hello. Is that a typo or something else in default.rb line 12: # ~FC033
Regards,
Alexei.

Hi @akuzmenkov,
Sorry, I totally missed your question. It's used to suppress a Foodcritic rule, which is used in some of the later Learn Chef modules.


Add unit test for Node controller

Which issue this PR addresses:
https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/10227470/

What this PR does / why we need it:
Adds some coverage to the Node controller to ensure we're adding and reading our annotations properly. Also adds a const.go file for the node package since it's standard practice among other operator controllers.

Test plan for issue:
make test-go
go test -v

Is there any documentation that needs to be updated for this PR?

No

@m1kola @troy0820 I agree we should test via Reconcile, and I had previously attempted to do it unsuccessfully, but I can work on it more. I have addressed the other comments and squashed commits, so we can either merge with followup issue to change to Reconcile, or else I will continue to work on it via this PR - whatever is preferred.


How to load .framework on MacOS with golang

$ go SDK version 1.13
$ Platform MacOS 10.14

I have tried "plugin-Open" function, but it warns ".framework is not a file..." thanks ...

Please see https://github.com/golang/go/wiki/Questions. Thanks.

thanks~


chore: mark version v3.0.0

This LGTM, but we’ll probably want to merge your Node.js PR, then @mjzffr’s patch, and land a Chromium roll first.

I was going to merge this first, and rebase other PRs on top of it and merge them, but your plan actually sounds easier.
For Chromium roll, I can definitely roll a new version in, but it seems like it won't be Chrome 81 yet. The latest stable is 80.0.3987.132.

It seems like Chromium 81 will be released on March 17th. I'll wait on 81 to be released and roll that version in.

It seems like Chromium 81 will be released on March 17th. I'll wait on 81 to be released and roll that version in.

^ /cc @digitarald

Since M81 stable release has been delayed until further notice, I will roll the newest M80 in the next few days, which is released today, and publish 3.0.0.

It seems like Chromium don't publish snapshots for versions other than X.Y.Z.0, so will skip rolling a new Chromium in for 3.0.0

Chrome 81.0.4044.92 is out: https://chromereleases.googleblog.com/2020/04/stable-channel-update-for-desktop_7.html


Scoped token allocation

Problem
Privacy and security are major concerns with consumers of the go-amizone API.

While on their own, you'd expect consumers to implement at-rest encryption, it is not a primary implementation concern. Even amibot, the largest user today, stores credentials in plaintext. Tokens allocated by go-amizone will reduce the need for at-rest encryption for consumers (but it'd still be recommended)
Scoped tokens can restrict what a consumer can do, reducing the need for trust. For example if a user wants to use Amibot just to check their class schedules, they should be able to generate a token that's restricted to that use case on Go-amizone's end.

Solution

RPC:

Accepts a user credentials, a set of token usage scopes and an optional expiry date.
Yields a user token for the credentials and the scope set requested. This token must be in a JWT or Paseto form.

Web page in the vain of the GitHub token generation page

Reference: asetalias/amibot: Discussion about the privacy of the bot

We can go with a Paseto based token authentication. I recently worked with this go package.
I can implement it but would like to know what kind of payload should we go for. I believe username and password should be enough.


Add foundationdb-simulation crate

This crate aims to provide a framework to easily test in the Foundationdb simulator Rust layers built with foundation-rs.
To do so, Rust workloads that implement the RustWorkload trait, are "registered" using the simulation_entrypoint macro and compiled as a "cdylib", will be recognized as ExternalWorkload by the fdbsever. More info in foundationdb-simulation/docs/README.md.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
108 unchanged lines in 7 files lost coverage.
Overall coverage increased (+1.3%) to 73.946%

Files with Coverage Reduction
New Missed Lines
%

foundationdb/src/directory/mod.rs
2
92.56%

foundationdb/src/transaction.rs
3
86.87%

foundationdb/src/directory/directory_partition.rs
9
75.0%

foundationdb-bindingtester/src/main.rs
11
88.3%

foundationdb/src/database.rs
11
60.43%

foundationdb/src/tuple/pack.rs
35
86.21%

foundationdb/src/tenant.rs
37
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
1.3%

Covered Lines:
4141

Relevant Lines:
5600

💛  - Coveralls

Codecov Report
Patch coverage has no change and project coverage change: +2.30% :tada:

Comparison is base (3445ccc) 80.13% compared to head (c6c7665) 82.43%.

:exclamation: Current head c6c7665 differs from pull request most recent head a29ce91. Consider uploading reports for the commit a29ce91 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
+ Coverage   80.13%   82.43%   +2.30%
==========================================
  Files          27       23       -4
  Lines        5200     4846     -354
==========================================
- Hits         4167     3995     -172
+ Misses       1033      851     -182

see 14 files with indirect coverage changes

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


[New Sniff] Check for hardcoded URLs

Rule:
WARNING Verify that no hard-coded URLs are found. Manual verification necessary. An exception is made for the Author URI and (Theme) URI as set in the style.css header as well as links to wordpress.org. Links in the text of PHP/JS comments should be excluded from this check.
Ref: https://make.wordpress.org/themes/handbook/review/required/theme-check-plugin/#info
Theme check file covering this rule:
https://github.com/Otto42/theme-check/blob/master/checks/links.php
To Do:

[ ] Create unit tests
[ ] Create new sniff

The sniff in the PR corresponds to the rule, but I consider that the rule is not precise enough. The reason is that the automation should alleviate the work of the reviewer. But the current rule and implementation only assists the reviewer, and does not reduce the workload.
Hardcoded URLs are related to several issues, these come to mind:

Scripts and styles that are hardcoded into the templates, instead of being enqueued.
JavaScript files that are being loaded from CDNs instead of local.
CSS files that are being loaded from CDNs instead of from local.
Links that are placed in the templates.
URLs that are included in JavaScript files.

All of these need a different sniff:

For the first issue, I think we would have to resolved to parsing the templates with regular expressions to look for <link> and <script> tags that link to these types of assets.
For the second issue, we would need to look for all occurrences of wp_register_script(), and wp_enqueue_script(), and ensure that no URL of any kind is passed to those functions.
For stylesheets, one would have to do something similar, but here one must consider whitelisted URLs, such as those pointing to the Google Fonts CDN.
Links that might be spam, here we can use regex again, and warn reviewers about these links.
For JavaScript, I think we would just flag all URLs in scripts. It is perfectly possible to inject malicious code this way, and most people skip JavaScript files during review, so having this is very important.

I'd recommend that we look at the validity of the cases outlined above, and that @grappler or someone more familiar with the rules can give an opinion on whether these are valid according to the review rules.
If so, we would need a dedicated issue for each of the cases, with a dedicated sniff.

Totally agree with the additional sniffs. We want to check all of the URLs added in theme to check that there are no spam links. Maybe we need to rename the sniff to make it clearer.

The NoHardCosedUrls sniff currently identifies urls in php, css and js files, and contains some white listed strings to exclude some urls, such as Google Fonts. So I believe your bottom three points are looked after in this sniff.
Checking for specific urls contained in links in templates (comment 4 above) is more problematic because quite often urls are contained in variables or constants. The NoHardCodedUrls sniff only looks for urls and identifies many more urls then are found in the current Themecheck sniff, which focuses on urls in links.
Issue #20  covers discouraging creating CDN's. It could be switched from WARNING to ERROR and re titled to say NoBlacklistedCDN or something like that.
A separate sniff should be developed to detect urls in links and script tags and nor allow them.

I consider that it is important to keep in mind what the sniffs should do, and that is reduce the amount of human reviewer time needed to review a theme. A sniff that outputs all hardcoded URLs found in a theme has no advantage over using grep.
I think that having a sniff for detecting spam links is very useful, and we should focus on that for this ticket, and implement more precise and automated checks for the issues I mentioned above.

Just some quick ideas.  We could whitelist some URLs:

wordpress.org(.*?)
fonts.googleapis
Theme URI
Author URI

We don't want to block something like a link to theme docs, for example.  It seems like we could get the Theme URI and Author URI domains and check other links against those.  If the Theme URI is example.com, we'd allow for other URIs like example.com/docs (this should still be manually checked).
Other URIs that don't match probably shouldn't exist.  But, there's probably some edge cases.

Would that preclude links in the admin area to a support forum or where to go for premium versions?
Or comments in code or CSS giving credit to original authors? An example is the Meyer's reset CSS or the GPL licensing.

Support and premium versions links should be fine under the above.
I don't really think a whitelist is realistic, I was just giving a quick thought about a foundation to build from if we were to go such a route.  Hardcoded links are tricky, so I want us to exhaust all of our options.

Just my two pennies on a few points mentioned above:

A sniff that outputs all hardcoded URLs found in a theme has no advantage over using grep.

It does up to a point - in that the theme reviewer does not have to use a number of different tools, but gets all (most) the issues which can be checked in a semi-automated way in one place using just one tool.

I think that having a sniff for detecting spam links is very useful

We could whitelist some URLs

Both of these would change the current rule. Similarly, a number of other points brought up are related, but not the same issue. So maybe the whole spectrum of rules which relate to urls should be clarified first ?

Is this something we want to work on? Any info is appreciated 🙂


Adds the ability to pass included/ excluded tags to Locust.

Description of changes:
Adds the ability to pass included/ excluded tags to Locust.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @pobtastic, thank you for the patch! Overall I'm okay with this change, but at the same time I feel it might overload properties in the future because Locust CLI has a lot of options.
What do you think about creating a more generic property like additionalArguments: string[] instead, allowing users set arbitrary options? e.g.
command.push(...(additionalArguments ?? [])),

Sure!  Good plan, can confirm it works nicely:

Thank you for taking the time to review.

Cool, thank you so much!


AI can't building,What's wrong?

AI can't building,What's wrong? I run the project and FPS is to low when AI collect resources!

There have been a few upgrades since working on the upgrade to 5.3.
If you still run into these issues when all is merged, please let me know


Update Data Source Descriptions

Issue name: “Update data source descriptions to account for other feeds”
Summary
As a note for the next update cycle, we may want to revisit the Feed Data Source Object descriptions to ensure they are also applicable for the smart work zone device feed (SwzDevice).  Most descriptions refer to work zone data or the authoritative/responsible source.  This could be general enough for the devices but could be clarified to provide more direction or guidance.
This isn't explicitly included anywhere but I think a business rule could also be established that the data source for devices should refer to the contractor performing the work and not the device manufacturer.  This will ensure that DOT or others consuming the data have direct contact to the contractor responsible for the device.

I would support knowing both the contractor and the data provider source. The reality is that most general contractors are responsible, but do not know much if anything about the data, thus the agency would still need to get to source of the information, thus having both may streamline the effort. I understand the contractual approach, but we often experience a different reality. Most contractors are happy to not get caught in the middle. My two cents.

This is tangential, and I have said it before, but data source also needs to include information on data rights and ownership.  Otherwise the entire industry will fight this.


How to get the framerate of flutter desktop app?

How to get the framerate of flutter desktop app?

Details

Logs

The performance overlay should work the same way on desktop that it does on mobile.
If you are having issues making it work on desktop, please file a new bug that includes details of exactly what steps you are trying and what is/isn't happening.

@stuartmorgan  Thanks for you help. I have tried it on my desktop app and works fine.


In TensorRT 7.1 use TensorRT's Resize for align corners cases

Right now we use the ATen library to run the align corners case of resize. TRT 7.1 should have a fix for the align corners issue. This will give use better performance in this case.

This has been implemented


[Model Deprecation]: io.catenax.serial_part /1.0.0/SerialPart

Update Reason
Deprecation due to an updated model version: SerialPart 1.0.1
MS1 Criteria

[ ] The model that should be deprecated exists and is not deprecated already
[ ] The respective model does not affect existing use cases (aspect is either used in higher versions or replaced by a different aspect)
[ ] A shared aspect, which will be deprecated, is not used by any released aspect model

modeling_team


Add a package manager

As the distribution grows, it may stop making sense to package all apps into it. What kinds of package management solutions make sense for this project?
My preliminary thoughts are:
Packages are maintained in their own repo by their respective maintainers. There is one central repository that hosts package metadata (maybe JSON?) of 'approved' packages. The package manager syncs to the master list (with the option of using other package lists?) and uses the metadata to resolve dependencies, clone and install applications, etc.

(in case it wasn't clear, with input from the community and further study of how the distribution works, I may be willing to write the package manager)

You could just host a Snakeware-specific PyPI server; then you can simply use pip to install stuff from it.

I think using the existing PyPI is the right approach here, and very inline with the Snakeware philosophy

It would be cool to only ship stock Python (and pip) with snakeware releases, and then use pip to install required packages over the internet on first boot. We'll have to get networking and "persistence" fleshed out first though

On Thu Jun 4, 2020 at 1:41 PM +03, Josh Moore wrote:

It would be cool to only ship stock Python (and pip) with snakeware
releases, and then use pip to install required packages over the
internet on first boot.

That might not be a good idea. I would recommend snakeware's root to be
fully read only, and have the ability to easily be switched out.
That way, if Pygame, for example, broke after an update, you can always
download an older version and load it into a live USB or whatever, and
keep the OS working.
Also, since I managed to get snakeware down to a ~50MB live USB while
doing some experiments, space shouldn't be an issue at all.
That said, users should definitely be able to download modules
themselves, so a read-write user partition will be needed.

We'll have to get networking and "persistence" fleshed out first though

A basic curses-like wifi/ethernet connection manager would be more than
enough I assume. For the actual hard work, we can delegate to dhcpcd (or
Busybox's equivalent) and wpa_supplicant.


[breadboard-ui] Deprecate old UI controller

This deprecates the old UI controller because it's not needed. I didn't remove it because I want to give it a once over tomorrow and just check that all the functionality has been brought over to the new UI controller.

Oh yeah, there's the node information. Not sure if it's actually useful or not though 🤔