Aggressively prune any unmaintained links

Very aggressively prune a large majority of projects/links if matching the following:

Unmaintained (last commit 1 year ago+)
Link is dead
Vue 2 only

Additionally:

Remove leftover empty categories & their header link
Update some repo redirects (e.x. if repo renamed)
Update to updated version of repo
Add official solutions (e.x. Nuxt)

Awesome! I'll look at it in detail tonight.
This needed to be done badly, the only thing is that I was planning on moving the vue2 links to a readme.vue2.md for historical purposes. Maybe I'll use your work to still do that.

My two cents - while I believe there is indeed a lot of outdated junk in the list, the "last commit more than 1 year ago" criterion may be too strict - some projects may just be minimally utilitarian, just work and don't require additional maintenance.
Hopefully some criteria were used in addition, like "there is an unanswered issue" or "0 downloads from NPM" and so on.
Also some projects may be abandoned but still used as a source of inspiration or starting point to write a new solution if needed, but it's hard to formalise how to differentiate between "some abandoned junk" and "some abandoned project that might make sense for something"
Great useful work either way, good job 👍

I agree with your first point, but not entirely with the second. It's an awesome list, so the listed projects should be awesome, and in my view that means more than a source of inspiration/starting point.
That said, I've been not too strict about merging, but maybe I should after this cleanup.


website link metadata

i know we've discussed this before but i believe an optional website link should be included

There is already the url field for the package. generally this will link to the website of the package and author.
So I don't think a separate link is needed


Updating dependabot to support Javascript, Typescript and docker

Hi @WadeBarnes , if this looks good , I can squash and merge.

Thanks @WadeBarnes I have made changes as per your suggestion.

Thanks @WadeBarnes , I have added the deleted one now.


Remove moment.js dependency and fix UTC time string conversion to local time date object

Fix #7

@lookfirst
So sorry I didn't run a build.

Weird, I don't see the change to the .ts file.

Not sure what happened but i just made the changes should be good now.

@harry-sm thanks man!

@lookfirst
No problem.


Kronecker matrix utilities

Need utilities for Kroneckers for Gaussian processes.  I don't know much about it but want to get it down as an issue and point to Ben's R example:
https://groups.google.com/forum/#!msg/stan-users/iddShgJVwTQ/QgcDEP3h9JQJ
Talk to Ben to figure out how this should look in Stan.

I think the most immediate need is to have a user-facing function called something like quad_form_kronecker that takes 3 arguments (matrix X, matrix Y, and vector or matrix Z) and outputs the equivalent of Z' * kronecker_product(X, Y) * Z. Algebraiclly, that is a lot of subvector * scalar * matrix * subvector but with a smaller number of unique matrix * subvector expressions that could be performed once each internally.
It is often the case that either X or Y is diagonal or otherwise sparse, in which case there could be some more optimizations. I'm not sure we need any more than that because it is rare in statistics to see a Kronecker product that is not embedded in a quadratic form.

Here's a summary of the main points from the Stan dev meeting on this topic (17 February 2015):
Eigen now has (experimental) support for Kronecker products: http://eigen.tuxfamily.org/dox-devel/unsupported/group__KroneckerProduct__Module.html and Eigen also has a fair bit of supported sparse types (some experimental) e.g.: http://eigen.tuxfamily.org/bz/show_bug.cgi?id=275 http://eigen.tuxfamily.org/dox/group__SparseQuickRefPage.html
http://eigen.tuxfamily.org/dox/group__Sparse__chapter.html
http://eigen.tuxfamily.org/dox-devel/group__SparseCholesky__Module.html
We discussed ways of implementing, a type kronecker_matrix (which would also be useful for new types tridiagonal_matrix, sparse_matrix and toeplitz_matrix and symmetric_matrix) which would be similar to cholesky_factor_cov.
Important design decision
If K = K1 ⊗ K2 ⊗ K3  and K1, K2, and K3 are different sizes (let's say n x n, m x m, and o x o) we need a new data structure to be able to handle this. There was a discussion of using view or map in C++/Eigen, with a vector of matrices all sized max(n,m,o). @bob-carpenter needs to weigh in on this, and others should make their cases.
Once we get that worked out, the main wish list for Kronecker matrix functions is simply everything in section 34.7 of the manual, Linear Algebra Functions and Solvers, e.g.:
row_vector operator/(row_vector b, kronecker_matrix A)
row_vector mdivide_right_kronecker(row_vector B, kronecker_matrix A)
vector eigenvalues_sym(kronecker_matrix A)
We also need matrix-vector multiplication (not sure the naming convention here):
row_vector matrix_multiply_vector(kronecker_matrix A, vector B)
row_vector vector_multiply_matrix(row_vector B, kronecker_matrix A)
I guess we need a function (although like inverse it shouldn't be the first choice for a user):
matrix kronecker_product(kronecker_matrix A)
Also very useful will be:
y ~ multi_normal_kronecker(vector mu, kronecker_matrix A)
Relatedly, the matrix normal distribution is already implemented, but not exposed. It should be exposed. (@mbrubake may have thoughts.)

As a start, we might think about exposing a Triplet
http://eigen.tuxfamily.org/dox/classEigen_1_1Triplet.html
That would make it possible for a C++ function to take a Triplet and do
sparse stuff internally. In addition, that would be useful for
(non-)missing data.

This issue was moved to stan-dev/math#60


Create FIRST_TIMERS.md

Fixes #724
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation about the functionality in the appropriate .md file
[ ] I have added in line documentation to the code I modified

Short description of what this PR does:

This PR adds the FIRST_TIMERS.md file that explains how to contribute on the first time, as explained in #724

Hello @qligier,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Fixed by https://github.com/sendgrid/sendgrid-php/commit/fce001ddfb2a8d4539a0fa8f01ee6083c20fe434


How to import an ejabberd dump?

Hi,
Is it possible to load my dump created from my previous installation with ejabberdctl dump?
Also, is it possible to store the whole configuration in my host, for better portability?
Thank you.

Hi,
to store your own configuration you can use this image as  base-image and overwrite the config-template-files:
FROM rroemhild/ejabberd
ADD ./ejabberd.yml /opt/ejabberd/conf/ejabberd.yml.tpl
ADD ./ejabberdctl.cfg /opt/ejabberd/conf/ejabberdctl.cfg.tpl

To restore from your previous installation you have to copy the dumpfile to a running container with docker cp by hand and restore from the webgui or with the ejabberdctl tool, but I've not tested this yet.
I can add an startup restore script to automate this. Maybe by pulling the dumpfile with curl from an URL defined in the environment. Would this be helpful?

If add a Maintenance section to readme for backup and restore. This should also work for dump and load, but I've not tested this yet. Feel free to reopen this issue. Regards.

I didn't have the opportunity to try this, but I wanted to thank you. :bow:


Aptly reports conflict on addition for identical (architecture, name, version) but different filename

Hello,
I've come across a behaviour which I found puzzling at the beginning, but I think I've now found the source of the issue. using prepackaged aptly 0.9.7 on Ubuntu Trusty.
I use aptly in an automated fashion, where my CI system creates DEBs and adds them to aptly by invoking its command line; in this automated systems, a random suffix is prepended to DEB filename [see later why].
As an example, for my apt-current_1.7-27-jessie_all.deb deb file, the filename is tmpIwAWHDapt-current_1.7-27-jessie_all.deb. The package metadata has still apt-current as a name and the repo is handled properly by apt clients (i.e. apt-get install apt-current )
What does NOT seem to work is the "same (architecture, name, version)" check.
If I do:
aptly repo add jessie-apt-current-v1 /tmp/tmpIwAWHDapt-current_1.7-27-jessie_all.deb
aptly repo add jessie-apt-current-v1 /tmp/tmpIwAWHDapt-current_1.7-27-jessie_all.deb
aptly repo add jessie-apt-current-v1 /tmp/tmpIwAWHDapt-current_1.7-27-jessie_all.deb
aptly repo add jessie-apt-current-v1 /tmp/tmpIwAWHDapt-current_1.7-27-jessie_all.deb

repeatedly, aptly properly says
Loading packages...
[+] apt-current_1.7-27-jessie_all added

and exits with 0 every time.
Now, If I do:
/tmp$ mv /tmp/tmpIwAWHDapt-current_1.7-27-jessie_all.deb /tmp/CHANGEDapt-current_1.7-27-jessie_all.deb
/tmp$ aptly repo add jessie-apt-current-v1 /tmp/CHANGEDapt-current_1.7-27-jessie_all.deb
Loading packages...
[!] Unable to add package to repo apt-current_1.7-27-jessie_all: conflict in package apt-current_1.7-27-jessie_all
[!] Some files were skipped due to errors:
  /tmp/CHANGEDapt-current_1.7-27-jessie_all.deb
ERROR: some files failed to be added

The file is EXACTLY the same, just the filename changed, but aptly doest not properly detect that.
I noticed, as well, that within apt repos created by aptly the original filename is retained within the pool.
I would suggest (but I'm not sure if Debian/Ubuntu dictates anything different on the topic) that either an invalid package filename is rejected at the first upload, or that just the package metadata is employed and the filename is disregarded and ignored entirely.
About the random prefix that I employ:
AFAIR (but I'd need to verify, and I will if you tell me that I should since it could be an aptly bug) I had some issues when I wanted to use aptly to store some debs that don't have the "distro version suffix" within their version, i.e. let's suppose that I have two apt-current_1.7-27.deb , one is built for jessie, the other for squeeze, so they're different; I had some error reported (which I imagine were caused by the identical filename), I don't remember if it were on aptly repo add or snapshot publishing/switching.

Package filename shouldn't matter that much (or shouldn't matter at all).
More on conflicting packages: https://www.aptly.info/doc/feature/duplicate/
aptly supports conflicting packages just fine in different repositories.
btw, there's clear bug when handling multiple duplicate package in single aptly invocation

See #552 for a similar issue, but that one is about updates while this one is for add and manually changed file names.
With my Debian Maintainer hat on: the canonical name for a package file is more-or-less defined by the metadata: ${name}_${version}_${architecture}.${type}, where $version is without the Epoch and $type is deb, but also can be udeb for µ-Debs used by the Debian Installer. There even is the tool dpkg-name to rename your files to the canonical naming.
So just renaming a file does not change the meta-data and the next dpkg-name run will undo the rename.
I don't know if aptly explicitly stored the provided file name or uses above described naming convention, but just renaming a package file is at least not a good idea.
(There is a second naming convention used by APT when it downloads files with an epoch in the version number, where that is included and separated by an :, which is hex-encoded as used in URLs.)


Allowing for Unpkg to search for scoped packages

Fixes Issue #352

I'm going to assume JSDelivr has the same problem.  Can you double check?

@jimmylewis JSDeliver completion for scoped packages seems to be fine

Going to break this into multiple PR's to address:

Fixing Issue 352 specifically
Fixing version completion for Unpkg / JsDelivr
Fixing up the error shown when package version is missing when users hit "install"


chore: patch tokio for Sleep instrumentation

Signed-off-by: Zahari Dichev<EMAIL_ADDRESS>

closed in favor of #166


Infer does not respect javac_jar in .buckconfig

Rules that set javac_jar or if it is configured in .buckconfig, do not invalidate the buck cache for infer as it always uses a javac from path. This leads to different build behavior in buck and infer and creates cache key duplication for the network cache. Can we support running on top of a javac_jar that buck supplies in it's config?

I am not sure I understand. Using Infer will create infer specific cache entires anyway that should be separated from the cache entries of the normal build.
Infer can use the javac_jar to compile the code by adding the option:
"java-jar-compiler":
"path/to/javac.jar"
in the .inferconfig file.

Was not aware of that option. Because there are two configuration options one in .buckconfig and one in `.inferconfig

Closing the issue. @kageiit: please re-open it is there is more to be done.


Cognito OAuth invalid logout redirect URI

Hi,
I have setup kafka-ui with Cognito for Oauth. Login seems to be working fine but on logout redirect url seems to be invalid.
it should be https://kafka-ui.xxxxx.com/logout but instead it is http://kafka-ui.xxxxx.com/logout
I have highlighted same in the following screenshot

configuration for cognito Oauth looks like
auth:
  type: OAUTH2
  oauth2:
    client:
      cognito:
        clientId: <client-id>
        clientSecret: <client-secret>
        scope: openid
        client-name: kafka-ui
        provider: cognito
        redirect-uri: <redirect-uri>
        authorization-grant-type: authorization_code
        issuer-uri: https://cognito-idp.eu-west-2.amazonaws.com/<pool-id>
        jwk-set-uri: https://cognito-idp.eu-west-2.amazonaws.com/<pool-id>/.well-known/jwks.json
        user-name-attribute: username
        custom-params:
          type: cognito
          logoutUrl: https://<domain>.auth.eu-west-2.amazoncognito.com/logout

In AWS Cognito I have configured https://kafka-ui.xxxxx.com/logout as one of allowed logout urls. Since it is coming as http there is error on the UI. Even though error says missing redirect_uri but here error is because of mismatch of redirect_uri. It works fine with localhost.
Can someone tell me why is this happening? Is there any configuration missing here?

Hi, do you forward x-forwarded-proto header?

No, I dont. I have not done any changes to the kafka-ui. All I have done is added config for Cognito OAuth and setup same in AWS.
Will there be any config in AWS for x-forwarded-proto?

Are you running on EKS or EC2? Anyway if you're behind a reverse proxy you should take a look at your HTTP headers.
I suggest checking these out:
Headers for a classic LB: Here
Headers for ALB: here

@VikramAhuja I faced with the same issue. did you manage to fix it ?

Why would a redirect URL use any protocol other than https? Using http is insecure. And for an authentication process, we really should be using secure protocols.


The results of failed tests are deleted if many tests are run in sequence

STF creates a unique directory for each test.  After 10 (by default) directories have been created, the first is deleted before creating the 11th.  This prevents results from using too much disk space (the resutls files may be very large), but retains sufficient history for normal desktop use.
Full regression tests such as test runs within Jenkins build pipelines run many tests in sequence.  If most tests pass but a few fail there is currently a good chance that the results for the failed tests were deleted before the end of the test run, requiring that the test be rerun in isolation to investigate the failure.
If STF had an option to delete the results directory of a test if it passes, then only the failed tests results would exist at the end of the run, which could then be examined without a rerun.

Sounds like this is complete, after the changes I made. The makefile option to delete the results from passed runs is:
RM_PASS=1


[Bug] An error 'The query did not generate a result set' is reported when Hive Engine executeQuery INSERT OVERWRITE DIRECTORY

Code of Conduct

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

Search before asking

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

Describe the bug
Using the Java JDBC client to execute 'INSERT OVERWRITE DIRECTORY' on the Hive engine through executeQuery results in an error. However, it works when using the Spark engine.
Using stmt.execute to execute 'INSERT OVERWRITE DIRECTORY' returns false.
Regardless, from the results, this SQL is always executed successfully.
Exception in thread "main" org.apache.kyuubi.jdbc.hive.KyuubiSQLException: The query did not generate a result set!
at org.apache.kyuubi.jdbc.hive.KyuubiStatement.executeQuery(KyuubiStatement.java:462)
at com.zyb.sqlservice.localtest.TestKyuubiHive.main(TestKyuubiHive.java:26)
Affects Version(s)
1.9.0
Kyuubi Server Log Output
The error is the client

Kyuubi Engine Log Output
None

Kyuubi Server Configurations
None

Kyuubi Engine Configurations
None

Additional context
jdbc:hive2://<IP_ADDRESS>:10009/default?;kyuubi.engine.type=HIVE_SQL;
Are you willing to submit PR?

[ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
[ ] No. I cannot submit a PR at this time.

What's Hive runtime version? Any specific configurations?


InvalidParameterValue

Hello there
Whit this link (and others also):
https://www.amazon.it/gp/aw/d/B08QJPMM2J?pf_rd_m=A11IL2PNWYJU7H&storeType=ebooks&pf_rd_p=889e88dc-a023-41fa-8871-b0fc0a00d2b4&pf_rd_r=V50DCEG5YY1HAE7AG1JC&pf_rd_s=mobile-hybrid-3&pf_rd_t=30901&ref_=dbs_f_r_shv_ts_889e88dc-a023-41fa-8871-b0fc0a00d2b4_16&pf_rd_i=1728160031
it returns:
InvalidParameterValue","Message":"The value null provided in the request for ItemId is invalid."}]}

in get_product function.

Hi,
You are not allowed to pass the product link in the get_product function.
Please see the simple example and provide the asin:
from amazon.paapi import AmazonAPI
amazon = AmazonAPI(KEY, SECRET, TAG, COUNTRY)
product = amazon.get_product('B01N5IB20Q')
print(product.title)

In get_product then use the function “get_asin” which tries to infer the asin like this
# Extract ASIN from URL searching for alphanumeric and 10 digits
have_asin = re.search(r'(dp|gp/product)/([a-zA-Z0-9]{10})', url)
return have_asin.group(2) if have_asin else None

In your case the Link does not match the regex, so null is returned and that explains your exception.

But this link is provided directly by Amazon. So how can i solve?

Thanks for your help, I've already merged your pull request. This change will be available in future updates via pip.

Nice to help you


Update changelog workflow

Update mskelton/changelog-reminder-action to v3 which removes the include option. Now we must define that option in GitHub's path option. This requires us to split the docs out into a separate workflow file.

As far as I can tell, the permissions option is being ignored here, which is resulting in the action failing to even run. Not sure what to make of this.

I am closing this in favor of #1373.


ListItem 2.0 (part 3): PropertyContent column auto-sizing

What
This PR:

Implements column auto-sizing for PropertyContent. The current heuristics is rather simple and will likely need improvement when it's battle tested. Currently, the content requests a minimum width of 200px (there really isn't a way to properly deal with two-column-plus-button neatly all the way to 0 width), and allocate a maximum of 70% of the available width to the label column.
Improves the way ListItem handles content capturing the hover state (which happen for interactive content). There is no longer a requirement for ListItemContent::ui() to return an Option<Response>, which simplifies a bunch of things.
Updates re_ui_example accordingly

Note: still no change applied to the actual viewer code base so far.

Part of #6075
Follow-up to #6174

Limitations and todo

Improved heuristics for column sizing, when we have a better idea of the need.
More helpers are needed for various kinds of values.
There can be only 0 or 1 action button. This should be extended by using a … button with some kind of popup with all available actions in a future PR.
Right gutter space is reserved for the action button even if no list item in scope use them. The list_item_scope could track this and skip reserving that space if it's never used (e.g. component list in entity path selection panel): https://github.com/rerun-io/rerun/issues/6179

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using examples from latest main build: rerun.io/viewer
Using full set of examples from nightly build: rerun.io/viewer

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!

PR Build Summary
Recent benchmark results
Wasm size tracking

To run all checks from main, comment on the PR with @rerun-bot full-check.

All review comment addressed in #6183 (except the suggested postpone ☝🏻)


Standarize tabs styling

 2.0 FE Bug: Strange Tab Behavior Connecting Manage Shipments tab style with the tab style inside one of BoxView components

@fhenrich33 , after looking at this, I'd like to change the Move/Transfer tabs back to Pascal case. That does keep the text case different from QR Scanner but that is OK. I think those internal tabs will end up being on a case-by-case basis.
If you're planning/already putting the large page tabs (e.g., SENDING/RECEIVING) into a global component I'm OK with having them all caps and setting that as default.

I don't think it's worth having this as a component just for that since we use wildly varying props and conditionals throughout the app with the tabs components. We would be abstracting just for the sake of abstraction and losing flexibility.

 2.0 Section Headers in the Sidebar Should be Capitalized #tiny


Updates required for a stable deployment

Reference Issues or PRs

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[x] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

Hi @aktech just refactored the changes to avoid the linting. May I add your changes from amit-deploy here as well?

Hi @aktech just refactored the changes to avoid the linting. May I add your changes from amit-deploy here as well?

I have added upgrade changes to it as well, which is not ready for merge. I would say skip that for now.


Add a Two-Pointer important question for DSA

Add resources not codes
Add resources like courses, certifications, roadmaps, notes, cheat sheet, etc

ok Have uploaded a road map u can cheak


Is this still up to date for Big Sur and Monterey?

Is this still up to date to decode the csr-active-config values for Big Sur and Monterey?
Because if I enable all the values I get 0x7ff, hex-swapped FF070000 which was used in macOS Catalina to disable SIP)
But in Big Sur 0x867 or 67080000 hex-swapped is used instead.
So I am wondering if there have been new entries added to that list. If so, can you update the tool?
Thanks.

I haven't kept this script updated - I ended up expanding the functionality to include a number of other common bitmasks and writing another script called BitmaskDecode.  That's updated for Big Sur, but I haven't looked into Monterey settings yet (it's still in beta, and I doubt any of the source for the csr values is available yet).
-CorpNewt

Thanks. I didn't even know this Tool existed!


Proposed SIG-Docs meeting agenda for 2024-07-03

Meeting Details

Date/Time: July 3, 2024 @ 10:30 CDT/11:30 EDT/16:30 CEST
Location: Discord SIG-Docs Voice Room
Moderator: @RoddieKieley
Note Taker: @RoddieKieley, @ShaunaGordon

The SIG-Docs Meetings contains the history past calls, including a link to the agenda, recording, notes, and resources.
SIG Updates
What happened since the last meeting? - The creator of this issue is responsible for providing a brief summary here to be used as an update to start any important discussions.
Naomi made Roddie maintainer of o3de.org repo, which sets us up to start doing more.
Open action items

[x] Start bringing development and main into parity - In progress
[ ] Triage PRs and Issues
[ ] Clean up existing PRs as needed
[ ] Run through broken links issues
[ ] Start stabilization process
[ ] Run through other PRs/Issues

Newly resolved items

[x] Roddie gets maintainer permissions

Initial Agenda
The pre-approved agenda for this meeting is as follows:

Review current needs to start ramping up
Next steps

Meeting Notes
Facilitator: Roddie Kieley (@RoddieKieley)
Notes taker: Roddie Kieley (@RoddieKieley), Shauna Gordon (@ShaunaGordon)
SIG Updates

First recurring meeting of revived SIG group
Naomi reported between meetings that Roddie has maintainer access, Shauna should have reviewer and can start going through process to gain maintainer
Jan Hanca (@JanHanca) of Robotec joined in and called out his PR bringing dev into parity with main

Group looked at PRs since late December 2023 and found only a handful that might need added to Jan's PR

Reviewed repo settings to see how to lock down main so that no more commits go to it without going through dev/stabilization
Shauna has Hugo experience and so can easily get docs spun up locally to test Jan's PR when it's ready

Action items

Roddie

double-check Shauna's reviewer perms, then other maintainer perms to prune maintainers
notify sig-docs-community list and discord about process change so everyone with merge to main perms knows what's going on

Jan

update parity PR

Shauna

start reviewing and triaging PRs/Issues
take point reviewing Jan's PR when ready


Add list of supported keystores.

Adding a list of supported keystores.
TLS contexts using PEM types keystores cannot be selected when the secret groups' target is of type Mule.

Confirmed.


Getting error with simple config on existing Express4 app

I have an express4 app and I add the following to my app...
require('express-ws')(app);
app.ws('/ws', function(ws, req) {
  ws.on('message', function(msg) {
    ws.send(msg);
  });
});

Then I run the app. However, when I try to connect to the websocket I get the following...
    at ServerResponse.OutgoingMessage._send (_http_outgoing.js:138:38)```

Your stacktrace seems to be incomplete. Can you provide the full error message and stack trace, as well as the Node.js version that you're experiencing this issue on?


Export excel - missing decimals

Hello,
I would like some help with a simple conversion. I have an excel file and I would like to remove the 1st row. When I do that, the output file does not show the decimals in column 3. Is there a way to keep the decimals? Using the -nonumberconversion doesn't seem to work as expected.
Import-Module importExcel
sl C:\test
Import-Excel test.xlsx -NoHeader -StartRow 2 |
Export-Excel test11.xlsx -NoHeader -NoNumberConversion *

please post the excel file you are working with

testxx.xlsx

As output I get this
test11.xlsx

That data is formatted in Excel, the Import-Excel pulls only the value.
Try this:
$excel = import-excel testxx.xlsx -StartRow 2 | Export-Excel ./test11.xlsx -AutoSize -PassThru

Set-ExcelRange -Worksheet $excel.Sheet1 -Range "C:C" -NumberFormat "#,##0.00"
Close-ExcelPackage $excel -Show

Thank you so much! I knew you were my only hope. I really appreciate your help!!!


Brokered: cannot create topic: user null does not have permissions to 'send'...

branch: issue_784
after enabling authz in brokered address space, topic cannot be created.
23:08:44,184 WARN [org.apache.activemq.artemis.protocol.amqp.proton.ProtonServerReceiverContext] AMQ119032: User: null does not have permission='SEND' on address 4b6c0f47-1fa8-4f63-a57c-2fe466f4ef77: ActiveMQSecurityException[errorType=SECURITY_EXCEPTION message=AMQ119032: User: null does not have permission='SEND' on address 4b6c0f47-1fa8-4f63-a57c-2fe466f4ef77]
automated test:
make MULTITENANT=true SYSTEMTEST_ARGS='brokered.SmokeTests#testAddressTypes' systemtests

Fixed


Implement and parse the admonition node

We have a Note node https://github.com/pythonecuador/lira/blob/f05975aa442089a6859e36c929f935575be830a4/lira/parsers/nodes.py#L137-L143
but we actually want an Admonition node with different types (note, warning, tip).

Or maybe we are fine having a different class?

This is implemented, we need to parse it now.


Api client integration

Seems to have broken the InfoDatabanksTest(). It is returning a 500 error.

This looks good Ryan. I have added a couple of comments/questions that are more for you to consider than resolve.

Thanks for the feedback Jason.


Testing on my own dataset

Hi,
it's me again. I would like to test the trained model on my own dataset. I would greatly appreciate it if you tell me the step by step. Thank you in advance.

If you want just to test the trained model, you need to change path(to your dataset) and mode. It can be found at test part in "README.md".
If you want to train the model on your dataset, your dataset should be replaced with tuSimple dataset form, because our dataloader works only on tuSimple dataset. After this step, follow train part in "README.md".

if i have lane instance segmentation images，is there a method to generate json format like Tusimple?thans in advance.

Sorry, we have no module to convert instance segmentation images to tuSimple form. In my opinion, you can find related module from other lane detection method based on segmentation like LaneNet. Their method predict segmentation image first, and estimate points to evaluate on tuSimple dataset.

thanks, the problem that dasetsets format should be converted to Tusimple-style has been  solved based on your suggestion.And from the test result on my own datasets after re-train the model is pretty good,even in some complex case. besides the idea is worth learning.thanks for your wonderful job again.

thanks, the problem that dasetsets format should be converted to Tusimple-style has been solved based on your suggestion.And from the test result on my own datasets after re-train the model is pretty good,even in some complex case. besides the idea is worth learning.thanks for your wonderful job again.

How do you convert dataset format to Tusimple-style? Could you tell me some details. Thanks a lot
@pandamax

thanks, the problem that dasetsets format should be converted to Tusimple-style has been solved based on your suggestion.And from the test result on my own datasets after re-train the model is pretty good,even in some complex case. besides the idea is worth learning.thanks for your wonderful job again.

Hello，I have some problems when using these codes. I am a Chinese too. Is it convenient to leave an email or Wechat for communication？ Thanks a lot.

thanks, the problem that dasetsets format should be converted to Tusimple-style has been solved based on your suggestion.And from the test result on my own datasets after re-train the model is pretty good,even in some complex case. besides the idea is worth learning.thanks for your wonderful job again.

Hello，I have some problems when using these codes. I am a Chinese too. Is it convenient to leave an email or Wechat for communication？ Thanks a lot. @pandamax

Sorry for not responding in time .Briefly explain my thoughts for reference,my own datasets was labeled in instance segmention form,in other words,each lane line in the image has an id.so I just take points at equal intervals along the y direction for each lane line,like this
https://raw.githubusercontent.com/TuSimple/tusimple-benchmark/master/doc/lane_detection/assets/examples/lane_example.jpg
For each lane line, if there is an intersection, get its x value, if not, use -2.you can choose the appropriate interval according to the image size and the length of the marked lane.(that's what I understand,maybe not very accurate),finally saved it in tusimple data format.
here is a reference.
https://stackoverflow.com/questions/53189712/generating-binary-segmentation-file-for-lane-detection-in-python-tensorflow-tus

Sorry for not responding in time .Briefly explain my thoughts for reference,my own datasets was labeled in instance segmention form,in other words,each lane line in the image has an id.so I just take points at equal intervals along the y direction for each lane line,like this
https://raw.githubusercontent.com/TuSimple/tusimple-benchmark/master/doc/lane_detection/assets/examples/lane_example.jpg
For each lane line, if there is an intersection, get its x value, if not, use -2.you can choose the appropriate interval according to the image size and the length of the marked lane.(that's what I understand,maybe not very accurate),finally saved it in tusimple data format.
here is a reference.
https://stackoverflow.com/questions/53189712/generating-binary-segmentation-file-for-lane-detection-in-python-tensorflow-tus

Hello，first of all, thank you for your efforts.  I 'm trying to mark  my images myself. But I found it difficult take points at equal intervals along the y direction for each lane.(I can only mark the lane line without fixed y coordinate by labelme)  And I don't know how to generate rhe json having the format(lanes,h_samples,raw_files) like tusimple dataset. Can you show me a detailed way? thank you again @pandamax

thanks, the problem that dasetsets format should be converted to Tusimple-style has been solved based on your suggestion.And from the test result on my own datasets after re-train the model is pretty good,even in some complex case. besides the idea is worth learning.thanks for your wonderful job again.

How do you convert dataset format to Tusimple-style? Could you tell me some details. Thanks a lot
@pandamax

@chenmeng12   Hello，I don't know if you've found a way to convert the dataset format to tusimple style. I 'm trying to mark my images myself. But I found it difficult take points at equal intervals along the y direction for each lane.(I can only mark the lane line without fixed y coordinate by labelme) And I don't know how to generate the json having the format(lanes,h_samples,raw_files) like tusimple dataset. Can you show me a detailed way if you have made it. thank you a lot

@koyeongmin hi~
From the test results, it seems that each lane line predicted as far as possible extends to the bottom of the image, although there may be no lane lines in this area in the labeled image, any suggestions on model training strategies?for example, Increase this value constant_exist?


Testing with op ver fix

fix problem with testing old version of operator with OP_VER specified.
if you specify non existing version you get error :
cluster is running and ready for installing olm   [  OK  ]
Find kube config                                  [  CONTEXT: operators  ]
CSV file for version 1.545.15 not found test cannot continue

/retest


AWS EC2 set environment variable during installation process

Problem description
PATH=/usr/local/bin:$PATH cannot be set permanently by running ./install.sh. Without PATH variable during installation, users still can complete the installation correctly. However, gridlabd command cannot be performed directly under gridlabd folder after installation. Users have to find it in the build folder.
Expected behavior
None
System information

GridLAB-D version:
Operating system: AWS ec2

Supporting information
none

This can be done by modifying the shells' .rc files.

@catzzz, can we close this or is this still something we need to fix?

Yes. this issue had been fixed.

Close issue


fix(ToDoItem): bind the click event to the outer div

For a better experience, bind the click event "goToIssuePage" to the outer div.

thanks for your contribute !!!


feat: cleanup log spew

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3660   👈

This comment was auto-generated by Graphite.


Create e2e test image for Prow

This change will allow Prow to run all HNC e2e tests as postsubmits or
periodic tests. Prow configs will be submitted in a followup commit in
the appropriate repo.
Tested: 'make run' successfully runs all HNC e2e tests (excluding the
NetworkPolicy test, which is expected) in a Kind cluster triggered from
the container image that will run in Prow.

/assign @rjbez17
/assign @GinnyJI
/cc @yiqigao217

/assign @rjbez17
/assign @GinnyJI
/cc @yiqigao217

/lgtm
/approve
We'll need to finish https://github.com/kubernetes/test-infra/pull/19856 after this pr is merged.


iCloud support, smarter clipboard detection, branding tweaks.

The word list can now be stored on iCloud (it still defaults to local storage.) When you change to "Use iCloud" (using the Settings app), you will be prompted to merge the local words in a list stored in iCloud.
To enable iCloud, you'll need to provision your App ID using the developer portal. The brief instructions are in Setup_iCloud.txt.
I also tuned up the clipboard detection so that it won't prompt you more than once for each new word or for text that has newlines in it.
Moved some color into the navbar, too.

The real test here is if @gruber will successfully navigate the developer portal and add the iCloud entitlements to his app :-)

Thanks so much, @chockenberry!
I'm in the middle of some local changes related to pasteboard (I'll write up more details of what I'm attempting to do in issue #43), and there are some conflicts with your PR, so I'll merge this one later tonight (Japan time).
This is a huge and generous contribution, CHOCK, THANK YOU!! ❤

Thanks. This pull request has been integrated manually.


HTML output - visioalization, comments, bugs

graph title should not be "NCA Graph" Change to something like "VPC Connectivity Diagram"
https://github.com/np-guard/vpc-network-config-analyzer/issues/360

Hi @zivnevo @kyorav @adisos
HTML output is now supported.
please try it, and comment here whatever you thing will improve it
tnx

Not only it is supported, it was already demoed to clients today 😃 That's how this issue was born.
Hopefully, clients will play with it (we made a pre-release version for that) and provide more feedback.


Restore CI to passing

Change test_aks_create_service_no_wait to live only.

@marstr will merge once all CI stages pass except tests.


Remove external references from package meta data - codeload.github.com

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.11.5
Current Behavior
Hey Team, we are trying to include wagmi package in our application. While installing this package in our network we are getting network timed out error as it is trying to access one of it's transient dependency from codeload.github.com which is restricted in our network.
Error:
npm ERR! code ETIMEDOUT
npm ERR! network request to https://codeload.github.com/ethereumjs/ethereumjs-abi/tar.gz/ee3994657fa7a427238e6ba92a84d0b529bbcde0 failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
npm ERR! network This is a problem related to network connectivity.
Expected Behavior
We want to install wagmi and enable wallet connect in our application.
As we analyzed, it is because of this following line: https://github.com/MetaMask/eth-sig-util/blob/1.x/package.json#L24 that codeload.github.com reference is hardcoded into the package meta data.
We believe, one possible solution for this issue is, if we can upgrade wagmi to have all external dependencies within the package instead of using external github references.
Steps To Reproduce
If we try to install wagmi with common command, npm i wagmi within our ecosystem we are getting this error.
Error:
npm ERR! code ETIMEDOUT
npm ERR! network request to https://codeload.github.com/ethereumjs/ethereumjs-abi/tar.gz/ee3994657fa7a427238e6ba92a84d0b529bbcde0 failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
npm ERR! network This is a problem related to network connectivity.
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

Looks like this is from an upstream dependency we don't control. Best to raise an issue over there.


Keep remote and local Redfish resources in sync

When we issue a request to the Redfish service that modifies some resources, we should also update the resource's local representation with a fresh set of data and invalidate the cache entries.

Basically we need to fully invalidate cache everytime we execute action, right? Because we can never know what the action is about; they are all POST.

I did not think about this much yet, but this how I would handle various modifications:

PATCH: refresh the local resource (this also refreshes the cache).
DELETE: remove an entry from the cache.

POST is a bit more complicated because we can use it in two different situations:

when creating a new resource and
when executing an action.

When we execute an action, we do not have enough information to do anything meaningful. Creating a new resource, on the other hand, also modifies the item's parent collection, which means that we need to invalidate the cache entry for it.


Blog Overview missing middle Blog in 3 on Logistics page

Go to https://www.netcentric.biz/our-industries/logistics
Check the Blogs further down the page.
There should be 3 blogs in a row showing, but the middle one is not showing at all.

My suspicion is that the URL is not complete, but I might be wrong.

Notified Livia to check if it's a simple authoring fix.

Made the fix myself. Seems the old blog had changed names, therefore link changed.
Updated now to show all three.


GLMakie not supporting Mac M1 yet?

I tried to install and compile GLMakie on mac M1.
It seems that GLMakie does not work on the M1 chip.
julia> using GLMakie [ Info: Precompiling GLMakie [e9467ef8-e4e7-5192-8a1a-b1aee30e663a] ┌ Warning: GLFW.GLFWError(GLFW.PLATFORM_ERROR, "Cocoa: Failed to find service port for display")

Well, that's somewhat expected :D I guess the first step is, that GLFW supports M1...

Well, that's somewhat expected :D I guess the first step is, that GLFW supports M1...

This happened for me on a non M1 iMac for me

With the exact same error?

It also happened to me – with exact the same error. If I can provide any other helpful information, let me know.

weird, when I google for that exact error, I get only m1 issues as far as I can tell...
In any case, this isn't really Makie specific, so anyone should feel free to try to debug why GLFW fails on their mac ;)
In https://github.com/MinecraftForge/MinecraftForge/issues/7558#issuecomment-751265857, I see that Apple deprecated some functions... So maybe this isn't m1 specific but specific to the newest OSX version or something?

With the exact same error?

Yes, same exact error

Can you be more specific about the OSX version + hardware?

Sure.I am running macOS Big Sur (11.2.2) on an MacBook Pro (13", M1, 2020, Model ID: MacBookPro17,1), 16 GB Ram.
If you need any other information, feel free to ask.

If anyone can try googling this, and see what issues apply to their problem, it'd be of great help..
I just tried it, and that error seems to pop up in a variety of circumstances, and it's hard to tell, which ones apply to the issue at hand.

I just found, that upgrading glfw may fix this:
https://github.com/glfw/glfw/pull/1833
https://github.com/glfw/glfw/releases/tag/3.3.3
https://github.com/glfw/glfw/issues/1841
I'm trying to update the binaries right now:
https://github.com/JuliaPackaging/Yggdrasil/pull/2705
When that's merged, we can upgrade  GLFW.jl.

I am using it right now on my M1 Mac. Works totally fine.


Fix incoming webhook PLR match with generateName

fixed an issue in PAC when pipelinerun name is defined via genetateName field it wasn't matching pipelinrun.
https://issues.redhat.com/browse/SRVKP-6808
Changes

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

/test


Variable expansion with environments

pass additional configurations for variable lookup
add support for variable expansion callbacks used if path has not been found
in config itself or in additional configurations passed for lookup
add ResolveEnv option using os.Getenv for variable expansion if path can not
be resolved

I'm trying to get my head around this one and I'm not 100% sure yet what the consequences of this PR are. Could you give an example of a yml config option that is possible after but isn't possible now?
Code change wise I got the impression that we pass opts *options in way too many places. Couldn't we perhaps simplify some of the code and passing params by making it part of some objects. I don't have a specific suggestion, this is more a question.

The options kind of acts like evaluation environment/context telling ucfg how to process merging and unpacking. Due to variable expansion (not being based on regex of input file) adding some minimal of evaluation supports (by nature), we need to pass the run environment down to the bottom nodes of our computations. Some new fields in options are now required by path evaluators (see path.go) iterating the config given some path. Due to most structures requiring the environment being precomputed, the environment is passed through.
Having options available in all places will allow us to add options for more strict/weak type checks during conversions (e.g. make support for int/float->time.Duration optional).
Normally it's up to the ucfg user (in this case libbeat) to decide on common settings being passed to Merge and Unpack. Not passing options, but storing them within the config can be difficult in the presence of merging configs with multiple other configs + requires rather big refactoring to ucfg in general.
Some (extreme) use-case for ucfg variable expansion with support for environments is https://github.com/urso/ljtest . ljtest computes the config environment dynamically per test, merging testsuite + test + service configs in one common config.
Sample use case: environment variable lookup (not possible with ucfg right now)
    config.Unpack(&to, ucfg.ResolveEnv)

ResolveEnv installs a callback into options.resolver, looking up variables if name is unknown to ucfg.
Difference to beats implementation is: variable expansion in libbeat is based on file-content treated as raw string. expansion can happen at any place potentially breaking the file. Using ResolveEnv variable expansion is limited to config values only.
The generalization is custom name lookup callbacks:
    config.Unpack(&to, ucfg.Resolve(customLookup))

customLookup can potentially do anything users want (e.g. read configs via etcd or zookeeper, use predefined patterns for common names).
Sample use case: predefined application specific variables during lookup
In addition to custom resolvers one can pass additional environments for lookup to ucfg (field envs in options). The environments are of type *ucfg.Config themselves.
Advantages of environments over merging configs:

No need to implement custom merge logic, Just pass additional environments for lookup with predefined variables.
Config access recording (not yet implemented) records all fields accessed giving ucfg users the chance to report all fields not being read. If common predefined names were to be merged into the config for lookup, they would also be subject to error reporting.
compute/pass custom environments based on computation context. E.g. used by ljtest to pass testsuite + test + service configs + full active test config (via _active variable) in ljtest without having to rewrite the global config every so often.

e.g. passing common application paths in environments (beats use-case)
    pathsEnv, _ := ucfg.NewFrom(map[string]string{
        "path.home": ... ,
        "path.config": "${path.home}/config",
        "path.data": "${path.data}/data",
    }, ucfg.PathSep("."), ucfg.VarExp)

    config.Unpack(&to, ucfg.PathSep("."), ucfg.Env(pathsEnv), ucfg.ResolveEnv)

In config file one can use:
    registry_file: ${path.data}/registry.json

This is currently handled in beats by automatically prepending ${path.data} in case of registry.json being a relative path (by default ${path.home} might be the current working directory). The configs should be explicit about what happens to configs instead of doing some magic.
Another potential use case regarding multiline support (and regex) in beats:

common multiline scenarios environment config:

multiline.scenarios.java-stack-trace:
  pattern: ...

multiline.scenarios.python-stack-trace:
  pattern: ...

multiline.scenarios.rabbitmq-log:
  pattern: ...

and filebeat config:

    multiline:
      - ${multiline.scenarios.java-stack-trace}
      - ${multiline.scenarios.python-stack-trace}

This works due to ucfg (already) treating single references on right hand side as references (no string replacement). If any string-splicing happens supported types are limited to numbers and strings and ucfg will generate errors on type mismatch.
All features are opt-in and disabled by default. It's up to the user of ucfg to decide which features will be used.
Already support by ucfg, there are different types of variable expansion operators (inspired by bash):
${VAR}: try to expand VAR. Error if VAR being unknown
${VAR:DEFAULT}: try to expand VAR. Use DEFAULT if VAR is unknown
${VAR:+ELSE}: expand to ELSE if VAR can be resolved. Otherwise expand to empty string
${VAR:?custom error message}: Generate custom error message if VAR can not be resolved.
In contrast to libbeat current variable expansion support, unknown variables are not expanded into "", but an error message will be generated unless one explicitely asks for empty expansion via ${VAR:}.

Merging as this feature is off by default and it is definitively worth having it in ucfg. We can continue the discussion on enabling or not when updating ucfg in beats.


Native Notebooks - Julia notebook not recognized?

Steps to reproduce:

Create new notebook
Switch kernel to julia
Add some cells and save
Send to someone else to open OR close vs code and reopen notebook

Expected behaviour
Notebook opens with cell language as Julia
Actual behaviour
Notebooks opens up with cell language as Python

@claudiaregio
You'll need to run a cell, and then the cell information is saved in the notebook.
Lert me know what you think.
Note: Please install latest VS Code Insiders (todays build) to test this out (not stable).

@claudiaregio
You'll need to run a cell, and then the cell information is saved in the notebook.
Lert me know what you think.
Note: Please install latest VS Code Insiders (todays build) to test this out (not stable).

At least for me this was not what's happening when using a notebook created with the current version of VS Code and Jupyter plugin and then opened with the Insiders version of both, see https://github.com/microsoft/vscode-jupyter/issues/5237#issuecomment-804561372

@claudiaregio this should be fixed in the latest Insider Build.

Validated. Scenario looks good for me now.


Output line numbers with gcode

In my gcode sender, I have an editor where I can select lines and have those lines highlighted in a visualizer. Internally I do this by generating a simple CSV intermediate file: "original line, processed line, parsed comment, line number". The visualizer uses this intermedia tagging each line segment with a line number from the file.
If there was a way to output the line number which generated each gcode command I could create a GSharp editor with a visualizer that displayed lines as the script is edited.
One way to implement this might be a new option to the cli: g#2g <input> <output> <format>, where format might look like: %line_number%, %g_code%.
In the screenshot below I've highlighted all the lines which make up the spiral

Just to clarify which line numbers you want to be returned.
Let's take more simple example of regular polygons. The G# code will look something like this (e.g. octagon, line numbers are for the reference only):
(1) #1=8 (sides)
(2) #2=[360/#1] (angle step)
(3) #3=25 (circumference radius)
(4) o100 while [#1 GE 0]
(5) G1 X[#3*cos[#1*#2]] Y[#3*sin[#1*#2]]
(6) #1=[#1-1]
(7) o100 endwhile

The output of the interpreter after calling Step() 9 times until finished will be:
(1) G1 X25 Y0
(2) G1 X17.678 Y-17.678
(3) G1 X0 Y-25
(4) G1 X-17.678 Y-17.678
(5) G1 X-25 Y0
(6) G1 X-17.678 Y17.678
(7) G1 X0 Y25
(8) G1 X17.678 Y17.678
(9) G1 X25 Y0

Each line in the output was created from a single line (5) of the input, and this number can be retrieved at each step (essentially it will be the same number 5).
Which lines would you like to highlight in the editor - from the input or the output? If it's the output, then there should be no difference to what you already have now. Lines (1-9) correspond to the complete polygon. And they all could be referenced back to line 5 of the input.
If it's the input, the interpreter has no knowledge of the output until executed. So I guess you could feed the highlighted part of the code into the interpreter, run it, get the output related to the highlighted block and use it in visualizer. But that could be dangerous, for example if the user highlights only lines (2-7), then parameter 1 will be lost and the number of sides will be zero. And what if the highlighted block uses a variable which has been defined somewhere far away?
Actually you may have already solved this issue in your code. What if G91 (relative coordinates) was used somewhere before and it is not highlighted? You probably keep the current state for each part of the code. I guess we could use similar idea here, although it's impossible to keep the state of all 5000 parameters for each output line. It's easier just to run the G# program from the beginning to the end, checking if the returned line number is present in the highlighted block and then feed it into the visualizer. It's very fast anyway.
Will this work? I can already see some minor issues, but I'm frankly opened to suggestions.
(strictly speaking line 1 of the output does not belong to the polygon, but that doesn't change the logic)

The comment above is if you want to integrate the library into your own code.
If you simply are after changing the output of the "example.cpp", it's fairly easy to do. I'll have a look what is the best command line options (I don't like including C-type string format for code security reasons).

I was going to run the lines through your program in the background. So
using your example the user would see the 7 lines of G# and the visualizer
shows the 9 lines of Gcode. If they highlight line 5 everything in the
visualizer would be marked.
To be more precise, each line of the interpreter output would need to have
"5," added to the front as a link from script to gcode.
In a larger script with multiple paths you could select a subset of the
lines which create output and just the gcode generated by that output would
be marked.
I parse the gcode and normalize it into absolute coordinates and mm units
so that I don't need to keep the state of each point in mind while
rendering each frame.
For a first pass I was just going to run your cli app in the background.
Maybe making a Java binding layer will be useful in the future.
On Friday, June 10, 2016, Night Road Software<EMAIL_ADDRESS>wrote:

Just to clarify which line numbers you want to be returned.
Let's take more simple example of regular polygons. The G# code will look
something like this (e.g. octagon, line numbers are for the reference only):
(1) #1=8 (sides)
(2) #2=[360/#1] (angle step)
(3) #3=25 (circumference radius)
(4) o100 while [#1 GE 0]
(5) G1 X[#3cos[#1#2]] Y[#3sin[#1#2]]
(6) #1=[#1-1]
(7) o100 endwhile
The output of the interpreter after calling Step() 9 times until finished
will be:
(1) G1 X25 Y0
(2) G1 X17.678 Y-17.678
(3) G1 X0 Y-25
(4) G1 X-17.678 Y-17.678
(5) G1 X-25 Y0
(6) G1 X-17.678 Y17.678
(7) G1 X0 Y25
(8) G1 X17.678 Y17.678
(9) G1 X25 Y0
Each line in the output was created from a single line (5) of the input,
and this number can be retrieved at each step (essentially it will be the
same number 5).
Which lines would you like to highlight in the editor - from the input or
the output? If it's the output, then there should be no difference to what
you already have now. Lines (1-9) correspond to the complete polygon. And
they all could be referenced back to line 5 of the input.
If it's the input, the interpreter has no knowledge of the output until
executed. So I guess you could feed the highlighted part of the code into
the interpreter, run it, get the output related to the highlighted block
and use it in visualizer. But that could be dangerous, for example if the
user highlights only lines (2-7), then parameter 1 will be lost and the
number of sides will be zero. And what if the highlighted block uses a
variable which has been defined somewhere far away?
Actually you may have already solved this issue in your code. What if G91
(relative coordinates) was used somewhere before and it is not highlighted?
You probably keep the current state for each part of the code. I guess we
could use similar idea here, although it's impossible to keep the state of
all 5000 parameters for each output line. It's easier just to run the G#
program from the beginning to the end, checking if the returned line number
is present in the highlighted block and then feed it into the visualizer.
It's very fast anyway.
Will this work? I can already see some minor issues, but I'm frankly
opened to suggestions.
(strictly speaking line 1 of the output does not belong to the polygon,
but that doesn't change the logic)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NRSoft/GSharp/issues/1#issuecomment-225322629, or mute
the thread
https://github.com/notifications/unsubscribe/AAHqRWw7SdX8lKC4pW7wBdwv2OXwFWpvks5qKfqwgaJpZM4Iyj2i
.

Ok, I think I know what's the best way to proceed.  In this case the change is minimal, just replace line 105 of "example.cpp" and recompile:
-            file_out << str << endl;
+            file_out << r.GetCurrentLineNumber() << ", " << str << endl;

That should suit your specific purpose and will not forfeit the purpose of the example.
At this moment I'm a bit reluctant to add generic format parameter because 1) it'd require more time and thinking and 2) I haven't got any requests from other people so far. I'd better wait for higher demand and broader requirements.
Yes, I agree, Java binding may be good to add in the future. Maybe you would be able to help here. See how it goes and let me know.


WIP #83 Auth Resource Tests

This is WIP and it's based on my previous Java 15 fixes (https://github.com/stargate/stargate/pull/242, https://github.com/stargate/stargate/pull/241). When they are not merged before I'm done here I'll rebase to the upstream master.

Contributes to #83.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 7

Complexity increasing per file
==============================
- auth-api/src/test/java/io/stargate/auth/api/AuthResourceTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 7

Complexity increasing per file
==============================
- auth-api/src/test/java/io/stargate/auth/api/AuthResourceTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- auth-api/src/test/java/io/stargate/auth/api/AuthResourceTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- auth-api/src/test/java/io/stargate/auth/api/AuthResourceTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- auth-api/src/test/java/io/stargate/auth/api/AuthResourceTest.java  1

See the complete overview on Codacy


Boost linking error

Problem
I noticed the recent changes with regard to how boost is set up for the project and really appreciate that the process in clearer now. However, building the latest release version of the SDK (release/v4.4.2), I still face issues with boost during the build process using your updated build instructions.
I did setup the project and resolved dependencies like described in the latest BUILDING.md notes
git submodule update --init --remote --recursive

curl -o boost_1_77_0.zip -L https://sourceforge.net/projects/boost/files/boost/1.77.0/boost_1_77_0.zip/download
unzip boost_1_77_0.zip
ln -s boost_1_77_0 libs-external/boost
cd boost_1_77_0
./bootstrap.sh
./b2 headers
cd ..

As no issues occurred during the setup process and the symlink was verified to be correctly set, I tried to build the project as
python swigpp-objc.py --profile standard
python build-ios.py --profile standard --use-metalangle

which results in the following type of errors
/mobile-sdk/libs-external/sqlite3pp/sqlite3pp.h:29:10: fatal error:
      'boost/utility.hpp' file not found
#include <boost/utility.hpp>

The problem occurs on Mac OS.
Solution
As the build instructions above did not work and the issue was clearly related to the symlink not working, I tried to use a relative path like (after unpacking boost in the project root folder) as
cd libs-external/
ln -s ../boost_1_77_0 boost

which solved the linking issue. I am not sure if the ln command does handle things differently under Linux, but for Mac OS, I would propose to at least specify this behavior in the build instructions and/or provide an example.

Thanks! I have updated the instructions, as you suggested.


Tyrogue > Hitmonlee, Hitmonchan, Hitmontop suggestion

As the evolutions of all 3 of tyrogue's evos seem to be bugged, would it be possible to change the evo requirements for the moment to something like levelling up with a power bracer held for hitmonlee, levelling up with a power belt for hitmonchan and levelling up to 20 with no item for hitmontop

Need this reported to Pixelmon, but in the meantime,

Updated Tyrogue evolution method to Hitmonlee by trading while holding a Power Bracer.
Updated Tyrogue evolution method to Hitmonchan by trading while holding a Power Belt.


[REQUEST] - Add new component

Description
add component to the existing components
Basic Example
No response

hey @bieefilled i would like to work on this issue but would you please elaborate me as i am new to open source

okay @bieefilled understood , will you please assign me this

Can you assign me , as team we work together


wrote stage damage with uncertainty tests

I have written a test that has two purposes:

Replace the damage with uncertainty tests on stage damage
Serve as test for profiling stage-damage compute

The test is passing for 1 of 2 impact areas. Making the other tests pass is being tracked in #856.
I attempted a different way to make sure that test values are within a reasonable tolerance. Doing the assertion for multiple cases that have different orders of magnitude is complicated. If you see a better way, let's do it.

This PR Fixes #768. I forgot to include that in the commit message.

The good news is that we've confirmed that damage with uncertainty in 2.0 reasonably matches damage with uncertainty in 1.4.3 given that the function is calculated using a graphical stage-frequency function.

When we refactor stage-damage as planned, this test will assure that we do not mess up uncertainty bounds - which has been our concern about sampling the set of occupancy types once for a given stage-damage realization.

all of my tests appear to pass but the test run is being aborted

failure message: "vstest.console process failed to connect to testhost process after 90 seconds. This may occur due to machine slowness, please set environment variable VSTEST_CONNECTION_TIMEOUT to increase timeout."

tests are now passing but I will leave my comments above for reference

Was there an issue with your tests that caused the abortion that you fixed or was it just random?

I think it was random


added end point to accses external server

The original installer misses endpoint management options as in the Nuget publisher task.
I added this option as the publisher's task

Hi @sergienko4, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Hi @sergienko4, thanks for your contribution. We expect that people will add a nuget.config containing their package sources. This way, both the build machine and on a dev box, you have access to the same sources. No API key is needed to restore packages.


Better error output for toHaveBeenCalledWith

First, great project!  I'm sure you saw my tweet of praise:

https://twitter.com/ericclemmons/status/681615901343399936

toHaveBeenCalledWith
Anyway, having replaced sinon with this, I've noticed that this very readable call...
expect(this.say).toHaveBeenCalledWith("foo");

...yields an error with only half of the picture.  There's no telling what it was called with, or never called at all:

Error: spy was never called with [ 'foo' ]

toEqual
However, explicitly using toEqual is much more useful:
expect(this.say.calls[0].arguments).toEqual(["foo"]);

[
-  "bar"
+  "foo"
]

Proposal
I'd recommend mimicking the output of toEqual.
When the spy was called & matched.
No changes.
(Patch) When the spy was never called.

"say" was never called

(Patch) Show all calls.

"name" was called N times without ["foo"]:
1 with ["bar"]
2 with [undefined]
...

(Breaking Feature) Expect the assertion to always reference the last call (e.g. spy.calls[spy.calls.length -1].
This is a suggestion for a possible future release that's largely unrelated to this issue.

"name" was last called with ["bar"], expected ["foo"].

What I like about this approach is that it works great for repeated assertions:
const query = function(params) {
  return axios.get("/api", { params });
};

var spy = expect.spyOn(axios, "get");

query({ foo: "foo" });
expect(spy).toHaveBeenCalledWith("/api", { foo: "foo" });
query({ bar: "bar" });
expect(spy).toHaveBeenCalledWith("/api", { bar: "bar" });

I only list it here because improving the usability of toHaveBeenCalledWith will lend itself to more use :)

What do you think @mjackson?

Hey @ericclemmons! Thanks for the input. I'm not sure if you saw #48 or not, but it seems you're not the only one who wants better error messages from toHaveBeenCalledWith. I like your suggestions.
Re: only referencing the last call - I think that makes a lot of sense. I can't think of any real scenario where I call a method more than once and then want to assert that one of the calls had a certain set of args. I'd be fine with making that the default behavior. One big upside is that by simplifying we can avoid printing the arguments that were used in multiple calls, which is nice. And we can use the same mechanism as toEqual for printing the diff between the actual and expected args.
Yes, this would be a breaking change. But I'm ok with that. We've been on 1.x for a while now, and this behavior is subtle enough that I doubt it'll break many people's tests.

Whoops!  I apologize for not catching #48.
How would you like this to be tackled?  Does #48 still apply, or does this subsume it with a new PR?

I'd say let's get a new PR going. You want to take a shot @ericclemmons?

@mjackson I'll take a stab at it.  Thanks for being so receptive!

Taking a stab at this today, but wanted to OK having a separate toHaveBeenCalledWith-test.js file for these.
Rational being, from a cursory inspection, createSpy and spyOn tests should largely ensure that spy objects have been created successfully.  From that point on, it's really enforcing the behavior of the individual assertions within Expectation.js.
If you'd prefer I not do this, I'll bake the tests into createSpy-test.js and spyOn-test.js.
Let me know, thanks!

Ya, a separate file would be great.
On Wed, Dec 30, 2015 at 1:39 PM Eric Clemmons<EMAIL_ADDRESS>wrote:

Taking a stab at this today, but wanted to OK having a separate
toHaveBeenCalledWith-test.js file for these.
Rational being, from a cursory inspection, createSpy and spyOn tests
should largely ensure that spy objects have been created successfully. From
that point on, it's really enforcing the behavior of the individual
assertions within Expectation.js.
If you'd prefer I not do this, I'll bake the tests into createSpy-test.js
and spyOn-test.js.
Let me know, thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/mjackson/expect/issues/60#issuecomment-168078555.

Hmm, looks like you already have an instance of Karma running. It may be confused.

Hmm, looks like you already have an instance of Karma running. It may be confused.

Hi, thanks for the great library, wondering if a fix for this issue is on the horizon?

If there were a way to run the tests purely in Node (and let CI do the integration tests), I could pick this back up.
Me thinks that it doesn't like that I have both Chrome & Chrome Canary, but I also haven't used Karma since my despicable Angular 1 days :)

fwiw, I have both Chrome stable and canary installed and npm test works fine.

Can you take over this then?
On Tue, Mar 15, 2016 at 11:34 AM Jordan Harband<EMAIL_ADDRESS>wrote:

fwiw, I have both Chrome stable and canary installed and npm test works
fine.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/mjackson/expect/issues/60#issuecomment-196911135

--

Eric Clemmons

@ericclemmons could you link me to the PR you have in progress (or create one if there isn't one yet)?

Your proposal looks good.
I have been using expect my branch and it serving me good. Although I agree that most times we want to see only last call, I would like to see a clear information that it is checking only the last call. Perhaps even a different name e.g. toHaveBeenLastlyCalledWith.
I have updated #48 so that it is in sync with master and linting passes. I've been using expect from this branch for a while. (did not test after merging master).

This is something I would use everyday, and that for now makes me go back to sinon just for that.

I think this is fixed in expect@21 (maintained by Jest). : )


Drop a check on MatchAlias in TypeComparer

Looks from the test cases that this isn't needed?

I agree we could get a matchtype. But why is that weird?

It's not a pattern I would expect to see normally. Normally, we have an AppliedType, then we do a tryNormalize on it, then we get the redux. We never get a top-level match type. So without a good reason I'd not change this here.


The LocalVector remove() API was renamed to remove_at(), it breaks godex

The LocalVector remove() API was renamed to remove_at(), it breaks godex
https://github.com/godotengine/godot/commit/e078f970db0e72bcc665d714416e6fc74e8434a6

Fixed


Fix CCM test to make sure that load balancer service LB is deleted before cluster delete kicks in

What happened:
CCM tests gets stuck as CCM load balancer is not deleted, and hence cluster deletion gets stuck
What you expected to happen:
Cluster delete should happen only after LB corresponding to LB service is deleted
How to reproduce it (as minimally and precisely as possible):
Run CCM test
Anything else we need to know?:
Environment:

CAPOCI version:
Cluster-API version (use clusterctl version):
Kubernetes version (use kubectl version):
Docker version (use docker info):
OS (e.g. from /etc/os-release):

this was not the original issue, the original issue was a CCM bug. Closing this as we cannot do anything in CAPOCI


Fail LocalWebApp build if viewer not found to embed

The LocalWebApp will fail to start if the viewer isn't available when it gets built.
That's not obvious when e.g. spawning the LocalWebApp from inside our platform.bible extension.
So, failing early seems like a good idea.

🤣 I didn't think of that!


[GOBBLIN-916] Make ContainerLaunchContext instantiation in YarnService more efficient

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-916

Description

[ ] Here are some details about my PR, including screenshots (if applicable):
Handle NPE in containerCompleteAction and change the excutor of contextLaunch to be scaling thread pool excutor

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Test on test cluster to make sure the wait time for initialize contextLaunch will not accumulate.

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@sv2000 can you take a look at this minor change?

Codecov Report

Merging #2770 into master will increase coverage by 0.01%.
The diff coverage is 2.56%.

@@             Coverage Diff              @@
##             master    #2770      +/-   ##
============================================
+ Coverage      45.3%   45.32%   +0.01%
- Complexity     8846     8858      +12
============================================
  Files          1892     1894       +2
  Lines         70752    70861     +109
  Branches       7773     7790      +17
============================================
+ Hits          32054    32116      +62
- Misses        35740    35784      +44
- Partials       2958     2961       +3

Impacted Files
Coverage Δ
Complexity Δ

...lin/hive/metastore/HiveMetaStoreBasedRegister.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...main/java/org/apache/gobblin/yarn/YarnService.java
14.92% <100%> (+0.08%)
3 <0> (ø)
:arrow_down:

...in/java/org/apache/gobblin/cluster/HelixUtils.java
32.71% <0%> (-6.55%)
11% <0%> (-2%)

...lin/elasticsearch/writer/FutureCallbackHolder.java
61.42% <0%> (-1.43%)
4% <0%> (ø)

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
64.35% <0%> (-0.47%)
28% <0%> (ø)

...ement/copy/UnixTimestampCopyableDatasetFinder.java
100% <0%> (ø)
2% <0%> (?)

...nt/copy/UnixTimestampRecursiveCopyableDataset.java
85% <0%> (ø)
8% <0%> (?)

...blin/service/modules/orchestration/DagManager.java
79.29% <0%> (+0.29%)
12% <0%> (ø)
:arrow_down:

...service/modules/orchestration/DagManagerUtils.java
84.81% <0%> (+0.81%)
34% <0%> (+2%)
:arrow_up:

...e/gobblin/runtime/locks/ZookeeperBasedJobLock.java
64.44% <0%> (+1.11%)
16% <0%> (+1%)
:arrow_up:

... and 2 more

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 b92f516...08823cb. Read the comment docs.

Any chance we can actually verify that onStartContainerError() and handleContainerCompletion() are both invoked? Is it possible to subscribe to the eventbus defined in YarnService and handle the events emitted in each of these method calls?

We have already emitted "HelixInstanceCompletion" event in handleContainerCompletion() and "ContainerStartError" event in onStartContainerError(). And the eventbus is used to handle container-related requests, which seems not related to this.


Add fix for NavViewItems in footer crashing

Description

Add null check to handle when NavigationViewItems are not rooted in the NavigationView menuitems.
Motivation and Context

Closes #2388
How Has This Been Tested?

Add API test and add NavigationVIewItem to footer of a testpage.
Screenshots (if appropriate):

/azp run

Pushed to the wrong branch, but everything is correct now. Sorry about that. Tests also all ran successfully now.


Pricing Overview

The Pricing endpoints provides the capability to retrieve the pricing definition for a specific entity.
So far, the tariff which is applied combines pricing definitions coming form different levels in the organisation hierarchy.
We can define prices at the tenant level (in the pricing settings), at the site level, and at the charging station level. This flexibility is confusing for the person responsible for maintaining the prices. We therefore need to provide a new UI to display an overview of the prices, showing all pricing definitions whatever the org level is.
The pricing router and the pricing service already provides some flexibility to retrieve all prices in one shot. The caller simply needs to pass null as entity type and entity ID to the end point shown below.

To do:

Enhance the UI displaying the list with the possibility to show all "Pricing Definitions" -  A kind of "overview" mode.
When this mode is enabled, the list must show two additional columns to clearly show the entity the price is attached to.
The list is in read-only mode. The user can view the price definition details, but cannot modify it or create new ones.

To be discussed: We may have a new entry in the navigation panel to display the list directly in this mode (instead of changing the behaviour of the current pricing settings).


VnV Plan 4.7: Distance Formula

Kind of a nitpicky point. As you said this is only 2D, you should specify $n$ is the dimensions or just replace it with a 2.

Since you did not mention the section, I'm not sure if we are on the same page. But n is the number of joints. For example for a triangle truss n=3. Please feel free to open a new issue, if I misunderstood your point


Support install on YARN context

Added two new variables yarn_deploy and is_edge to determine if the installation is on a hadoop cluster, with a YARN deployment.
When yarn_deploy is 1, installation only happens on the edge node. You still need to fill in the other hosts, and the node number for the configuration part.
Only works for dynamic mode (ha=1).
TODO: Make it work for ha=0

Abandoned feature


need custom ID support

OIDs are a common commercial data property
typegoose and mongoose are creating them implicitly under the hood
I would like to emulate my own custom ID number of range [0..n] and have it be autoincrement on newly created models.
therefore
/api/user - returns all users
/api/user/10 - returns user of ID 10
and makes for simple REST urls

One solution (not the best, but working) is to ignore the fields _id and id. Instead, use a custom field 'userID' where you can set your values as you wish (make them unique). Then you can search für Users with specific userID
Regards.

You can disable the _id already using the schema options passed to mongoose:
See http://mongoosejs.com/docs/guide.html:
var childSchema = new Schema({ name: String }, { _id: false });


Fixes Components Not Recognized with Multiple project.razor.json Files

Used projectSnapshot.GetDocument instead of .GetDocumentFilepaths as the former uses a hashmap to locate the document and the latter would require (a likely less performant) Collection.Contains operation.

Problem Description:
Ex. Debug/project.razor.json + Release/project.razor.json => we may take and use the Release/project.razor.json for our understanding of the project (because it's the most recently updated file), but we may be operating under the Debug/project.razor.json environment.
So if you add a file, Debug/project.razor.json would be updated, however Release/project.razor.json wouldn't have the file and consequently, the file would be put into the __MISC_RAZOR_PROJECT__. This results in a significantly degraded experience (ex. no completions) as that file is interpreted as not being part of the project.
Thanks @NTaylorMullen, @ajaybhargavb, @ryanbrandenburg!

Fixes: https://github.com/dotnet/aspnetcore/issues/24500

Is there some way we can ensure that the "correct" project.razor.json will always be chosen? That seems like a more widespread fix. Or maybe I'm misunderstanding this PR and that's what you're up to?

That would be ideal, but from what I gathered that isn't currently possible as we do not have access to the configuration mode (debug/release) currently chosen. Consequently, we can't determine which is the "correct" project.razor.json


embed: gracefully close peer handler

Passing canceled context to close existing idle connections only once and to not delay the server shutdown. And make embed.Etcd.Peers non-exported (need release note).
c.f. https://github.com/coreos/etcd/issues/6174

Apparently this breaks EXPECT_DEBUG=1 go test -v -run TestCtlV3AuthMemberRemove in e2e. Will investigate.

All fixed (it had to start rafthttp layer first).
PTAL.
32-bit test seems broken in Semaphore.


Jump to Bottom button doesn't work when using Damage Log

Foundry v12 build 331
D&D5e 3.3.1
Damage Log 1.21.1
Reproduction steps.

Disable Show limited damage info to players and Allow players to view the damage log, and enable Use separate damage log tab.
Make sure that the damage log has an entry for the player character.
As a player, have enough messages in chat to scroll up, then scroll up.
Try to click Jump to Bottom.

Expected result: After clicking Jump to Bottom your chat scrolls to the bottom.
Result: Jump to Bottom button doesn't do anything until you remove all the damage logs affecting the player character.
Module settings:

f3bc39fa05011debc52f6f4c1009fdd4e19bffba

Fixed in v1.22.2


Add @AliasFor declaration for @RefreshScope

Closes https://github.com/spring-cloud/spring-cloud-commons/issues/1148

Should this change be made in the 3.1.x branch?

I don't think so if you look at #1148 it's an issue in framework 6


feat: expose Period::new by introducing a PeriodUnit enum

The Perdiod::years and Period::months may feel redundant now, but keeping them to not break any public API.
We're building an interface at work. Doing this change upstream removes the need for us (and probably others) to create a data type that solely acts as a bridge.

It's unclear to me what the benefit of this change might be. This change definitely already breaks the public API, so it doesn't make much sense to retain some parts of it.
Also note that this crate is currently being maintained on a volunteer basis. If you're going to start depending on it for business-critical purposes, I suggest we should have a conversation about commercial support.

Did I "break" the public API? My intent was to only extend it. Perhaps our definitions differ 😊
Good point. I'll send you a email later today after having a conversation with my bosses. Where would you prefer us to send it to<EMAIL_ADDRESS>I have some fairly uncontroversial PRs lined up that I wish to send in during the meantime. Hope that that's ok by you.

Did I "break" the public API? My intent was to only extend it. Perhaps our definitions differ 😊

Ah, sorry -- I was mistaken. Would still be good to discuss why this is an improvement for you.

Good point. I'll send you a email later today after having a conversation with my bosses. Where would you prefer us to send it to<EMAIL_ADDRESS>
Yup, that should work!

I have some fairly uncontroversial PRs lined up that I wish to send in during the meantime. Hope that that's ok by you.

I'll take a quick look and will decide whether I want to review as a volunteer per PR. 👍

FWIW, I don't think "deprecate" is a great description for just killing API. :)

Heheh, yeah. Should probably just have written "remove".


extends can-run-as support for permitted-user checks

Changes proposed in this PR

extends can-run-as support for permitted-user checks

Why are we making these changes?
Makes the permitted-user check equivalent to run-as-user checks by using the can-run-as support for the entitlement manager.

This may have unintended consequences, e.g. perhaps a service owner would prefer a request to fail unless the request is made by the actual role account, instead of also allowing it from users that can become that role account.

We ignore this concern while making run-as-user checks e.g. during token registration or while launching services - mainly for convenience of the end user. This PR causes the permitted-user check to have the same behavior.

What is our motivation for this change?

Apart from bringing the check closer to how run-as-user checks works, we have at least one user who has requested for a more robust permitted user check, e.g. for groups or role accounts.

more than one users who have requested for a more robust permitted user check, e.g. for groups or role accounts.

Maybe we should add a new field with support for more robust checks instead of changing the semantics of the current one?

@pschorf  That would defeat the purpose of this PR which intends to bring how we treat permitted-user closer to how we treat run-as-user. We have made similar changes for run-as-user before with end user convenience in mind.

We decided to not introduce additional burden of authorizing requests and delegate the responsibility to the backend instance.


Change build platforms for better configuration and multiple build targets

This repo needs a build system that can handle the growing requirements. In particular, I think we need the following:

Multiple build targets. In particular, I would like to support:

a full build (i.e. roughly what's produced today),
a debug build (no mangling, easy to understand issues in dev tools debugging),
one or more reduced builds, e.g.

a JS animation only build without the CSS support #173)
a build with minimal performance overhead, e.g. skips observers for size changes.

I suspect the easy way we get different subsets of features is with different top level targets and modular includes. E.g. an include could install hooks for conformance code which would be no-ops when not installed.

Reserved name support. See #200 for a particular example where due to mangling we are failing tests.

Typescript? Not strictly required but I think having strict typing will make it easier to ensure correctness and make working on the polyfill easier.

@johannesodland @bramus WDYT? Does this seem good, anything else we need?

(Stretch?) Auto publish tagged releases to NPM


Uncaught DOMException: Failed to execute 'removeChild' on 'Node'

Steps to Reproduce:

Make changes in an input that has an oninput event.
Immediately navigate to another page using the browser's back or forward button.

Environment: Mithril 1.0.1 on Chrome 56.0.2924.87 (Windows 10).
Previous Related Issue: #358
Sample Code:
module.exports =
  { oninit:
      vn => {
              const id = Number( vn.attrs.id )
              const stream = M.loadItem( id )
              vn.state.data = { id, stream, initial: stream() }
            }
  , view: vn =>
    <div>
      <label class="label">
        Last Name
        <input
          class="input"
          type="text"
          placeholder="Last Name"
          onchange={ X.setStreamPropToValueAttr( vn.state.data.stream )( 'lastName' ) }
          value={ X.getStreamProp( vn.state.data.stream )( 'lastName' ) }
        />
      </label>
    </div>
  }

@kurtmilam Just to be sure, does this happen when you navigate to another page, or on routed components, and you navigate back to a previous route?

Also, @kurtmilam, could you verify it also repros with v1.1.0 (the latest release)?

@pygy This occurs on routed components when I navigate to a previous route using the browser's back or forward buttons. I believe it's due to the fact that the action that triggers the input's onupdate event also causes the input to be removed from the DOM.
@isiahmeadows I have just verified that the described behavior occurs with mithril v1.1.0 .

@kurtmilam One last thing: could you edit your repro down to something that doesn't require external functions?

@isiahmeadows I've just confirmed that the behavior persists when the component in question is stripped to bare bones:
import m from 'mithril'
module.exports =
  { view: vn =>
      <input onchange={ console.log } />
  }

I've also just confirmed that the behavior persists when I swap out the JSX syntax for javascript:
import m from 'mithril'
module.exports =
  { view: vn =>
      m( 'input', { onchange: console.log } )
  }

@kurtmilam JSX is okay. I just needed a minimal workable repro.
How are you redrawing the component?

@isiahmeadows I wanted to eliminate all possible variables.
The component is drawn on a specific route. The 'app' is just an experiment based on an extended version of the simple application (from the mithril docs). The behavior occurs when I change an input on my '/edit/:id' route, then navigate back to the '/list' route while focus is still in the input.

So finally, a fix incoming in #2286.

I'd be happy to work up a repro, but I'm not sure it'll work in flems or code sandbox (just tried in both), since I only saw the error when I used the browser's forward or back button to navigate to a different route in my mithril application while my input still had focus.

@kurtmilam If you have to, drop a debugger statement in the view of an offending component and try stepping through it. That might help you in isolating a repro case.
Alternatively, you could do input.focus() + location.hash = ....

@kurtmilam you can post it as a gist then?
Filename: index.html, load Mithril through<EMAIL_ADDRESS>then we can load it with bl.ocks.org which is just a gist viewer.

@kurtmilam Do note that iframes do still work with hash navigation and history navigation.

@kurtmilam Found anything reproducible yet? At this point, we're not even looking for anything minimal, just self-contained (even if it means slapping together a full repo we have to clone).

PSA: I don't care if it's a website at this point - as long as you can point me to where the functionality breaks, I'm willing to reverse engineer minified JS and place strategic breakpoints if I have to to isolate this bug.

I was able to reproduce this on Chrome 53 (probably the version I was using when I initially reported this issue), but have not been able to reproduce it on Chrome 70 or the latest versions of Firefox or Safari.
I reproduced it using this flems experiment.
Steps to reproduce:

Go to the flems.
Click on any user's name to open the /user/{id} route.
Type a few characters into the input.
While the input still has focus, use the browser's back button to navigate back to the /list route.

et voila!
Screen shot (a little messy because I was running a virtual copy of Chrome 53 on turbo.net):

Apologies that it's taken so long to help track this down.

@kurtmilam Yeah, that reads like a browser bug we can't really work around, especially if setting the hash directly doesn't trigger identical behavior. Given it takes a seriously old Chrome version for it to repro, I'm not sure we can fix it. BTW, we only support the last 2 versions of Chrome, so you need to create a repro that works on one of those.


quote instead of delete

Deleting some characters lead to shitty looking descriptions.
Illegel characters can be reduced further to
"'`

only

Thanks for removing the +-... characters. Those definitely didn't look very good.
@tfoote Would you prefer the illegal characters be escaped, or would you prefer the illegal characters be removed?
My opinion here is that, since we only get 80 characters, I'd rather have more of the actual content than spend characters escaping the one's that aren't supposed to be in that field (note: they will still appear in the metadata.xml file's longdescription).
But if I'm the only one with that opinion, I'm open to just escaping them.

I think escaping will be easier to read, especially as the number of characters goes up. Packing in characters by dropping whitespace will only gain a marginal benefit of more characters and lose readability.


fix: apply first version of eslint rules

:tada: This PR is included in version 1.4.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


App Packing: Eel & Selenium

Hi. I built a small app using Eel and selenium (among others) but when I pack it using pyinstaller it simply closes. Here is what my app uses:
from selenium import webdriver
from selenium.webdriver.common.keys import Keys
from selenium.common.exceptions import NoSuchElementException
import pandas as pd
import random
import time
import math
import os
import sys
import subprocess
import eel
import gspread
from gspread_dataframe import set_with_dataframe

In order to pack the app I am using the following command (on mac):
python3 -m eel raportarismart.py web --noconsole --onefile --icon=icon.icns
Question: Could you please let me know what I am doing wrong and maybe how I can pack the app? Thanks!
P.S. Since I am using selenium, I am guessing that I should somehow include the chromedriver in PyInstaller, but I don't really know how. Same should apply to the json file used for gspread.
Thanks in advance!
Later edit:
This is how the directory looks like, should this help:
|____icon.icns
|____.DS_Store
|____updateDB.py
|____driver
| |____chromedriver.exe
| |____chromedriver
|____web
| |____index.html
| |____.DS_Store
| |____css
| | |____bootstrap.min.css
| |____img
| | |____.DS_Store
| | |____bg-img
| | | |____a3ca141c-event-arrows-bg.png
| |____main.js
| |____ExportZilnic.txt
| |____style.css
|____MyApp.py
|____README.md
|____setup.py
|____credentials.json
|____MyApp.spec

Hi @mihneamanolache - sorry for the slow reply. Have you fixed this yet? Try building it without the --noconsole and --onefile flags first and you should be able to understand the errors better. Including the chromedriver executable feels likely, yes. Have a look at PyInstaller docs directly for how to include extra files.

Thanks, @samuelhwilliams! I will try and let you know what the outcome was.

Are you blocking with .init and .start in your main? It is likely that the last 2 lines of your main statement will be eel.init('folder') and eel.start('file'). That is the basic arena.
I use selenium and eel with pyinstaller without issues. You don't need a pyinstaller reference for Selenium, but you do have to point at the driver file on the users machine. There are various ways to package the chromedriver into pyinstaller but I highly recommend just having your software download the latest version from a remote because it updates all the time whenever chrome updates and they are not backwards compatible. (software no longer works every new chrome version)
For compiling eel with pyinstaller the eel documentation is exactly right... if you begin with the example you should be fine.


Infinite calls to api/v1/orchestrations/extractors/tap-gitlab/configuration when the Connect button is open

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1605
Originally created by @mbergeron on 2020-01-24 19:47:23

Found while testing !1382
If you click on Connect on any Data Source, there is a succession of calls made to the api/v1/orchestrations/extractors/tap-gitlab/configuration endpoint for no apparent reasons (other than the initial call).


Problem with symlink

When I ran node-waf configure build I got the lib/binding.node symlink wrongly pointing to ../build/default/... instead of ../build/Release/... directory. I guess it depends on what build target you chose or something like that, but I looked up how waf works and used the ctx object in wscript to get the path instead, so neither Release nor default must be specified.

Thanks!
Although I haven't merged this pull request this should be fixed by now.
Is it ok if I put your clear name in the list of contributors?