Feature/testing spike

Adding a workflow to github actions that runs tests. In this case also adding Playwright as an e2e testing framework

So this is in no way complete - but figure we mind as well get this committed and being used - even if there are not many tests yet.
So this enables 4 levels of testing in github

a Typescript compile
unit tests (no dependence on anything)
db tests (needs a running edgeql but nothing else)
e2e (a UI test suite that runs against the full running solution)

I had to make some. minor (non testing) code changes in order for the typescript compile to succeed


Fix min margin error

Description
Fixes an issue where attempting to open a position below min margin results in an error
closes #2339

Good catch, thanks. Now successfully displays the error message informing user of the min 50$ requirement.


Avatar image circle cropping is slow

I'm seeing 10–22 msec of work being done when I call + (JSQMessagesAvatarImage *)avatarImageWithImage:(UIImage *)image diameter:(NSUInteger)diameter for an avatar image for a cell. This is on an iPhone 5s on iOS 8.3.
This work is:

Happening on the main thread
Taking longer than 16 msec

which all but guarantees that we drop a frame while scrolling.
Bookmarking this so I can come back and do some perf work on it.

While there may be room for improvement, you should not be doing this as you scroll anyway. You should create JSQMessagesAvatarImage cache as you open your conversation view (or even before), and then store them and reuse them as needed.

That's what I did to mitigate it.
If you all want that to be official, I'll add some documentation to that effect for that particular API and open a PR.

Perhaps even better would be caching inside the framework.

Hey guys — the JSQMessagesAvatarImageFactory class doesn't make any claims regarding on which thread it should be used.
You definitely should not be calling these methods on every dataSource/delegate callback. And probably not on the main thread. Note that the demo creates these once and caches them, though it does do this on the main thread — that's just because it is a demo app.
There are plenty of great caching libraries available that you could utilize — caching is not a goal of this library.


Click doesn't work on iPad - version 1.0.2

Hello, I'm testing our application on iPad, but the elements in left menu doesn't seem to be working. Clicking doesn't respond at all. Only description shows.
This is happening with Vaadin v 8.6.1 and App layout v 1.0.2.
We use LEFT_RESPONSIVE_SMALL_NO_APP_BAR setting and we are adding elements via addClickable method in builder.

@lipisak cannot reproduce with 1.0.2. Are you using Tooltips for any chance?

@appreciated yes We are using it. Right now I turned them off on touch devices, as they are useless anyway. Since then it works, so I'm closing this issue.

Maybe open an issue on the Vaadin?

Actually it works eg. for buttons but there has to be "double tap". First shows tooltip, second fires click event. In app layout menu it doesn't work at all for click event. It only shows tooltip. But it can be vaadin issue anyway. I will try to find it in vaadin github issues. Thx.

Actually it works eg. for buttons but there has to be "double tap"
🤦♂


Fix BROOKLYN-271: ConfigKey card shows object hashCode of default values.

Fixing BROOKLYN-271
Avoiding object references in catalog ConfigKey cards, for example:

Last picture shows that default value of softwareProcess.lifecycleTasks config key does not contain the object hashcode @XXXXX.

Note for other reviewers - seem comments in BROOKLYN-271 before merging.

@sjcorbett  thanks for your message. Yes, probably this PR will be closed soon

Taking in account the comments in BROOKLYN-271 issue, this PR is replaced but two different PRs:

https://github.com/apache/brooklyn-library/pull/38
https://github.com/apache/brooklyn-server/pull/146


Implement DPC++ sparse gemm and gemv primitives

Following functionality was implemented:

sparse_matrix_handle class - a wrapper to oneapi::mkl::sparse::matrix_handle_t.
set_csr_data(...) primitives allowing to fill the internal CSR data structure of the sparse_matrix_handle with the data provided as memory blocks or oneDAL csr_table.
gemm(...) and gemv(...) primitives accepting ndviews for dense input and result and sparse_matrix_handle as sparse input.
tests for sparse gemm and gemv primitives.

/intelci: run

Re-run CI:
http://intel-ci.intel.com/eeda108b-cbb8-f1e1-a791-a4bf010d0e2e

Please, also add sparse_blas to primitives BUILD file here https://github.com/oneapi-src/oneDAL/blob/main/cpp/oneapi/dal/backend/primitives/BUILD#L82
Also add file named sparse_blas.hpp with necessary includes to the directory backend/primtives https://github.com/oneapi-src/oneDAL/tree/main/cpp/oneapi/dal/backend/primitives

/intelci: run

/intelci: run

/intelci: run

/intelci: run

/intelci: run

/intelci: run

/intelci: run


Edit Biergarten preset

Remove outdoor_seating:
Amenity=biergarten is itself an outdoor seating so this tag doesn't make sense.
Remove air_conditioning:
amenity=biergarten+air_conditioning: 1 feature
Remove building:
Although there are 1166 features on OSM amenity=biergarten+building, I don't understand how can an open-air area be a building or inside a building. One excludes the other.
Wiki: "A biergarten is an open-air area with benches ..."
See also photos there.
I think features with amenity=biergarten+building can come from people who confused biergarten with a pub.
Add cuisine to More Fields.
amenity=biergarten+cuisine: 276 features
amenity=biergarten: 10132 features
amenity=biergarten+building: 1166
amenity=biergarten+outdoor_seating: 546
amenity=biergarten+cuisine: 276
amenity=biergarten+air_conditioning: 1

I would suspect a lot of those biergarten tags on buildings are in the US. It's pretty common to have indoor shared table seating pubs with German beer here. People in the US would consider that to be a biergarten even if it doesn't strictly meet the guidelines of the wiki.

@maro-21 The extra fields are there to accommodate biergarten variations that may exist around the world. In the US, for example, alcohol generally cannot be consumed in public spaces, so biergartens may indeed be indoor "beer halls" or be covered by a building=roof.
That said, I'm okay with moving the Building field to moreFields.

What about removing outdoor_seating?

What about removing outdoor_seating?

I think it's okay for people to add this. Imagine a POI app that has an option to filter by "outside seating". This tag keeps the data model consistent with cafes, restaurants, etc. without making the app developer do more work.

So either every biergarten should have it or none. Because now it looks like a world is divided into biergartens with an outdoor seating and biergartens without an outdoor seating...


🌱 Syncer gen patience2

Summary
This PR is an alternative to #1275 and makes syncer-gen follow the pattern shown in https://eng.d2iq.com/blog/service-account-tokens-in-kubernetes-v1.24/#how-does-this-affect-you if a token Secret for the syncer's ServiceAccount does not show up within a limited amount of time. This also adds a second wait for a token to appear in the Secret. This behavior should work for kcp v0.11.0, Kubernetes 1.23 and later, and OpenShift (which is unlikely to be relevant as a space provider).
This is part of the campaign in #1274
This is built on #1273 and will be rebased if that merges first.
Related issue(s)
Fixes #

The force-push to cc7340c99 fixes a bug (tokenSecretName was left empty if the Secret had to be created).

/cc @yana1205
/cc @pdettori
/cc @ezrasilvera
/cc @waltforme

/lgtm

/approve


Prevent boost 1.72 in conda

Should fix https://github.com/Bonelab/Bonelab/issues/18.
In the future, we can update to 1.72. We will need to modify CMakeLists.txt as well.

I think I’ve submitted the pull requests now correctly. Shall I accept them or is it better for you or another person to accept and then build the conda libraries again. Steve
--
Steven Boyd
PhD, Professor, University of Calgary
Director, McCaig Institute for Bone and Joint Health
Full Signaturehttps://ucalgary.ca/people/steven-boyd/contact
On Apr 8, 2021, at 3:14 PM, Bryce Besler<EMAIL_ADDRESS>wrote:
[△EXTERNAL]
Hi Steve,
if you look at the change I made in this pull request (one line) you can do the same w/ n88util and pqctio. Once merged to master and build successful, do the process to upload to anaconda as before (for n88util, aimio, and pqctio - no need to do vtkbone/n88tools).
https://github.com/Numerics88/n88util/wiki/Azure,-Conda,-and-all-that
Bryce
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Numerics88/AimIO/pull/5#issuecomment-816206954, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFAQITQNK2ZEIX36AKNUWZDTHYMBXANCNFSM42TVVU2A.

Yes, these looks good! I can see the artifacts: https://dev.azure.com/babesler/n88/_build/results?buildId=1347&view=artifacts&pathAsName=false&type=publishedArtifacts

Now that the changes for pQCTIO and n88util are also merged, do we need to build new artifacts before uploading?
Steven Boyd
PhD, Professor, University of Calgary
Director, McCaig Institute for Bone and Joint Health
Full Signaturehttps://ucalgary.ca/people/steven-boyd/contact
On Apr 9, 2021, at 11:15 AM, Bryce Besler<EMAIL_ADDRESS>wrote:
[△EXTERNAL]
Yes, these looks good! I can see the artifacts: https://dev.azure.com/babesler/n88/_build/results?buildId=1347&view=artifacts&pathAsName=false&type=publishedArtifacts
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/Numerics88/AimIO/pull/5#issuecomment-816829866, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFAQITQLRDIKFTMUFPVIKOTTH4Y27ANCNFSM42TVVU2A.

Once you merge into master, the artifacts will be built automatically. Find them, upload them is all.

Ok, I was just surprised I didn’t get a message from Azure saying the builds had happened again. I’ll look at the Azure page.
--
Steven Boyd
PhD, Professor, University of Calgary
Director, McCaig Institute for Bone and Joint Health
Full Signaturehttps://ucalgary.ca/people/steven-boyd/contact
On Apr 9, 2021, at 12:16 PM, Bryce Besler<EMAIL_ADDRESS>wrote:
[△EXTERNAL]
Once you merge into master, the artifacts will be built automatically. Find them, upload them is all.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/Numerics88/AimIO/pull/5#issuecomment-816868263, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFAQITQCREKTSST5PVLFI4DTH477HANCNFSM42TVVU2A.


swiper.params.control bug

in the demo:
Swiper-3.3.0\demos\23-thumbs-gallery.html
open it in your PC,
use mouse slide to 8th image,
Up and down two pictures, not the same.
pls fix it.
Chrome	47.0.2526.106  m 32bit

Can't see such issue, everything is on the place

I stay in China, maybe the images not loaded success / or in loading, cause the bug.
I can reproduce the bug.

you can use chrome dev tool , simulate the situation, make image load slowly.


Updated labs service with alpha feature option

no refs

Extends labs.isSet method with an alpha option to determine if an alpha feature is active.
Alpha feature needs both labs flag as well as developer experiments flag enabled

As new alpha features get added, this allows us to verify if they are enabled via a single method instead of checking both labs.isSet and config.get(enableDeveloperExperiments) at each place in the code, and also make it easy to remove the dev exp check when feature goes into beta.
Its inspired by similar option in Admin here that allows marking a feature as alpha and needs a single change to switch it to beta.

Closing as we updated the code to handle alpha features with explicit lists that take automatically handle the isSet method to work properly.


[Feature] DevTools should be 3rd party extensible with custom data and logic

DevTools should support custom data and logic. For example:

inspect state of my internal app database
run custom debug commands in my app, and see their results

Of course, this code would be highly custom to my app, and can only be achieved if DevTools is extensible by 3rd parties.

@chipweinberger I suspect this issue should be opened here

@iapicca is right. Transferring to the devtools repo.


TensorflowOp Module+Plugin to handle TensorFlow Custom Ops

Request description
My company is interested in experimenting and potentially adopting OpenXLA/IREE in our model inference stack. But one blocker is that all our major models have at least one TF custom op, so we can't experiment IREE easily. I hope a TensorFlowOpModule can be formally supported and a TF model with custom ops can be compiled and executed with IREE out of the box. Below I list out all the impacts of this project from my perspective:

Unblock quick experiments. This is very critical for new users to onboard IREE. If our company get good results from experiments and decides to onboard, there will be a whole team building our stack on top of IREE, and we will contribute a lot to the source code if needed. With this, the impact is actually much bigger than this feature alone.
Easily land models that are not latency-critical. I understand using custom op may harm the performance. But there are always some small models that don't care much about latency. Since there are already custom ops in TF, people won't spend much time migrating the custom ops to native IREE ops.
More friendly to ML engineers. In my company, only optimization team touches model compiler. However, models are developed and maintained by modeling teams. The ML engineers are more familiar with TF APIs and it's hard for them to onboard IREE custom modules or MLIR. It's not always the case that the optimization team has the bandwidth to immediately help them develop the IREE custom module, so it's very useful if they can use TF custom ops as a temporary workaround.
Better TensorFlow integration. In classic XLA world, TF developed a feature "autoclustering" to handle ops that are not supported by XLA. However, that's not very ideal. A model is broken up into tens of sub XLA engines, thus it's not even possible to visualize the XLA graph or use XLA tools to debug the whole model. A TensorFlow module would be much better, if there is plan to replace classic XLA with OpenXLA.

What component(s) does this issue relate to?
No response
Additional context
No response

Thanks for the query. Since there's no one on the other side of this issue, we're closing for now. Feel free to open a new issue if you'd like to bring it up again.

We can leave this open for now. I am trying to connect it to someone internally.
(But as mentioned on Discord, the IREE team is not planning to do work on this in 2023)

Sorry I was using a wrong account so I closed that account. This account is the POC for this FR.


WIP views: don't explode on test connection

WIP: have no clue on why mysql fails.
If we don't have the database saved.
Props to LittleQ (sqj597) for the fix.
Fix #3799

Coverage decreased (-0.1%) to 71.13% when pulling 1968448daec00f02e37ec54151911e22922a91fc on xrmx:fixuptestconn020 into 21aee9862278de4277546371f235a37691def763 on apache:0.20.

Coverage remained the same at 71.271% when pulling e5bb085bced77a05b8700d55960795f8f21aa9f7 on xrmx:fixuptestconn020 into 21aee9862278de4277546371f235a37691def763 on apache:0.20.

Coverage remained the same at 71.271% when pulling e5bb085bced77a05b8700d55960795f8f21aa9f7 on xrmx:fixuptestconn020 into 21aee9862278de4277546371f235a37691def763 on apache:0.20.

Coverage remained the same at 71.271% when pulling e5bb085bced77a05b8700d55960795f8f21aa9f7 on xrmx:fixuptestconn020 into 21aee9862278de4277546371f235a37691def763 on apache:0.20.


[ENH] - Add ansible playbook for dev

Feature description
Add ansible playbook for setting up a dev environment for nebari
Value and/or benefit
Easier onboarding and getting started
Anything else?
No response

Does any other open source python project do this?  It seems a bit of pain requiring ansible to run a few shell commands to create a conda environment and run pip install -e .[dev].

It's been quite a few months since we discuss this, but my main takeaway is that the playbook would be useful to create a local dev cluster so we can create a dev AMI (potentially using packer). The AMI would be then used to run tests again an "existing" cluster, without the need of deploying a cluster every single time we want to run the local integration tests.
I think this would be highly beneficial but we would need to think when deploy/destroy tests are actually run. Some options are:

Every time a PR is merged
Every time we cut a release (candidate)

Ahh, I took environment to mean "conda environment".  I see what you mean now. Thanks for clarifying.

as @marcelovilla said the main use case would be the AMI setup, and I also think we could merge #2554 and use that in our benefit to reduce the complexity of the ansible tasks

So, I've done some testing around this, and here are my main points:
PROS:

Using Pecker with the playbook was an incredible tool to create the instance with a running nebari on it;
CONS:
Running Nebari commands internally within the playbook works, but its a nightmare to properly instepec the logs from Terraform, as Ansible puts all of them inside a dictionary (also only shows up once the task concludes)

So my takeaway on this would be:

Have the makefile PR merged without changing any of our current tests, so that we can further enhance it if needed;
Use the Ansible playbook to create a suitable environment within the EC2 instance without running any nebari commands
Explore a way to run the nebari deploy within the VM outside of the ansible playbook and catch the messages in a more visible format (maybe an ssh exec)

BTW, Packer's license changed recently along with Terraform's so I assume we'll need to use a version of Packer prior to the license change to BSL in August 2023.

https://stackoverflow.com/questions/18794808/how-do-i-get-logs-details-of-ansible-playbook-module-executions
@viniciusdc it seems like adding the -v option might allow for streaming logs from the playbook steps.


docs: index learn more link with id

Codecov Report

Merging #1132 (5413934) into master (542ae35) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1132      +/-   ##
==========================================
- Coverage   47.30%   47.27%   -0.03%
==========================================
  Files         185      185
  Lines       14464    14464
==========================================
- Hits         6842     6838       -4
- Misses       6724     6726       +2
- Partials      898      900       +2

Impacted Files
Coverage Δ

catalog/mvcc/cataloger_create_entry.go
94.73% <0.00%> (-5.27%)
:arrow_down:

catalog/mvcc/cataloger_create_repository.go
59.25% <0.00%> (-3.71%)
:arrow_down:

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 542ae35...5413934. Read the comment docs.


🦺  Make sure code for fetching details pages handles missing information gracefully

Describe the bug.
We recently had a bug where get_table_details threw a KeyError that prevented the page from loading.
This is now fixed, but in general if expected metadata is missing, it would be better to continue to render the rest of the page (perhaps with a warning about the missing data)
To prevent this from happening again, we should ensure that our GraphQL response parsing code (mainly graphql_helpers.py) handles any missing data by returning some kind of empty value (e.g. None, empty list, empty dictionary).
When we parse nested values (e.g. ownership.owners.owner) we should make sure that we catch any KeyError on any of the intermediate keys, and return the empty value in that case.
When these errors happen, we should make sure we log them as errors so we can still debug the issue (see https://github.com/ministryofjustice/find-moj-data/issues/216)
In the frontend templates, we can use conditionals or the default filter to change what we display if data is missing.

Will be better doing this post refactor now that has started
And this will also look to address bug where methods return results even when a given urn does not exist

Actually this is now all likely to be covered within the wider refactor

https://github.com/ministryofjustice/find-moj-data/issues/327 fixing this bug is the parent problem of this particular bug

After review we decided to close ticket as graphql_helper is defensive enough. Is any KeyError is raised and seen on alerts, we can fix those particular instances.


style(runloop/events): remove unnecessary set_workspace

Summary

set_workspace() is useless in CE, we should remove this line.
NOTICE: in EE we MUST have this line.
Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

If this line is harmless, maybe leave it for now to reduce the discrepancy between OSS and EE.
You can add a comment signaling this is no-op in OSS.

It seems that Adding a comment does not make any sense, let's keep it in the codebase.


Please add CHANGELOG.md about changes :)

Hi!
Unfortunately, we don't have a public changelogs at now.
According to CONTRIBUTING.md, all work on tramvai is done in a private repository, where we made untranslated commits, which can contain sensitive information.
So, we use Copybara to mirror all releases to Github, and Copybara squash all private commits to one public release commit.
I only can update some release tags manually, if there are a important changes, for example 1.8.3 release
At the future, we will improve public releases processes and will try to automatically generate changelog.


Add REXML to gem dependency

Summary
REXML gem has been bundled gem since Ruby 2.8.0-dev (Ruby 3.0).

https://bugs.ruby-lang.org/issues/16485
https://github.com/ruby/ruby/commit/c3ccf23d5807f2ff20127bf5e42df0977bf672fb

This change causes the following error on the Ruby master:
% ruby -v
ruby 2.8.0dev (2020-01-14T00:46:52Z master 440013b2fa) [x86_64-darwin17]
% ./script/test
(snip)

Traceback (most recent call last):
        11: from -e:2:in `<main>'
        10: from -e:2:in `each'
         9: from -e:2:in `block in <main>'
         8: from -e:2:in `load'
         7: from test/xml_test.rb:1:in `<top (required)>'
         6: from test/xml_test.rb:1:in `require'
         5: from
/Users/koic/src/github.com/jnunemaker/crack/test/test_helper.rb:3:in
`<top (required)>'
         4: from
/Users/koic/src/github.com/jnunemaker/crack/test/test_helper.rb:3:in
`require'
         3: from
/Users/koic/src/github.com/jnunemaker/crack/lib/crack.rb:7:in
`<top (required)>'
         2: from
/Users/koic/src/github.com/jnunemaker/crack/lib/crack.rb:7:in `require'
         1: from
/Users/koic/src/github.com/jnunemaker/crack/lib/crack/xml.rb:1:in
`<top (required)>'
/Users/koic/src/github.com/jnunemaker/crack/lib/crack/xml.rb:1:in
`require': cannot load such file --
rexml/parsers/streamparser (LoadError)

This PR adds REXML to gem dependency.
Other Information
The ruby-head of gems that depends on crack has failed.
e.g. https://circleci.com/gh/rubocop-hq/rubocop/81817
I hope a release that includes this change. Thank you!

Looks fine to me.  I wonder if it should have a gem 'rexml' declaration at the top?

When can we expect a 0.4.5 release? Ran into this when testing my gems on Ruby 3.0.

I just released 0.4.5 off of master. (PR is https://github.com/jnunemaker/crack/pull/67 just ironing out some asymmetrical ownership rights...)


Distraction free mode: hide comments

Today most lines in code are comments - about 70-80%? It becomes harder and harder to view the real things, scrolling and eye-filtering increases more and more. Because comments themselves are recognizable in every language - would it be possible to become a switch like hide comments in VS Code? As every IDE would need that, it would be fantastic if VS Code becomes the first one that implement this :1st_place_medal:

You can fold all comments with the Fold comments command.
https://github.com/Microsoft/vscode/issues/40338 is about being able to define an initial folding state.

Thats not what i want. Look at UWP apps - between every line is now a folded comment line. Thats not nice :-1:
Whats the problem to hide comments?


decode fails with 'none' algorithm and verify

When providing a token that has the algorithm 'none' to the decode method, a decode error is raised from line 87 with the error message 'Not enough or too many segments'.
This can be suppressed by disabling verification when decoding, but that has knock-on effects of other claims not being verified.
I believe the correct behaviour should be that, when the algorithm is 'none' and the token is otherwise valid, the decode method should return successfully.
It is worth noting that this behaviour provides (accidentally, I believe) protection against certain attacks. To provide support for this without exposing users to additional vulnerabilities, I would suggest also adding some flag (that defaults safely) that allows the developer to 'turn on' acceptance of the 'none' algorithm.

Does your none-token have a . (dot) as the last character?

it does indeed.
Generated using JWT.encode(payload, 'some key that gets ignored', 'none', my_header)
I believe the RFC calls for the trailing '.' in the case of the 'none' algorithm (see section 6.1), so I believe the token is correct. This is to indicate that the signature is actually the empty string.

I just needed to know this. This is a bug. Thanks for reporting. :)

@danleyden If you find some time you can checkout the rspec-test-refactoring branch and test the updated code.

Thanks @excpt - that looks like it fixes the bug.
However, that does open the door for third parties tampering with the token in the default use case..
Take the case where I send a valid, signed (not encrypted) token with e.g. RS256. A man in the middle could intercept that token, remove the signature and alter the alg claim to 'none' and send it on as a valid token. In this case, unless the implementer has set the algorithm when calling verify (a slightly buried option) the token will be accepted.
Given that, by fixing this, we open the door to that attack vector (albeit through the developer not using the library fully), I would be tempted not to merge that fix in until there is a better story around the none algorithm, such as forcing the developer to specifically enable its use.

Thanks for testing. I will remove that fix and the documentation is already updated.


pivccu-modules-dkms build failed - Error! Bad return status for module build on kernel: 4.9.35-v7+ (armv7l)

HI dear maintainers,
I am having trouble making piVCCU up and running.
I followed the setup guide https://github.com/alexreinert/piVCCU/blob/master/docs/setup/raspberrypi.md but the procdedure fails.
I searched around and found the ticket https://github.com/alexreinert/piVCCU/issues/130.
The i tried to reinstall the packages  raspberrypi-kernel-headers pivccu-modules-dkms but it didn't help.

root@raspberrypi:~# apt install raspberrypi-kernel-headers pivccu-modules-dkms --reinstall
Reading package lists... Done
Building dependency tree
Reading state information... Done
0 upgraded, 0 newly installed, 2 reinstalled, 0 to remove and 0 not upgraded.
Need to get 0 B/15.3 MB of archives.
After this operation, 0 B of additional disk space will be used.
Preconfiguring packages ...
(Reading database ... 234407 files and directories currently installed.)
Preparing to unpack .../pivccu-modules-dkms_1.0.36_all.deb ...

------------------------------
Deleting module version: 1.0.36
completely from the DKMS tree.
------------------------------
Done.
Unpacking pivccu-modules-dkms (1.0.36) over (1.0.36) ...
Preparing to unpack .../raspberrypi-kernel-headers_1.20170703-1_armhf.deb ...
Unpacking raspberrypi-kernel-headers (1.20170703-1) over (1.20170703-1) ...
Setting up pivccu-modules-dkms (1.0.36) ...

Creating symlink /var/lib/dkms/pivccu/1.0.36/source ->
                 /usr/src/pivccu-1.0.36

DKMS: add completed.

Kernel preparation unnecessary for this kernel.  Skipping...

Building module:
cleaning build area....
make KERNELRELEASE=4.9.35-v7+ ARCH=arm all......(bad exit status: 2)
Error! Bad return status for module build on kernel: 4.9.35-v7+ (armv7l)
Consult /var/lib/dkms/pivccu/1.0.36/build/make.log for more information.
Created symlink from /etc/systemd/system/multi-user.target.wants/pivccu-dkms.service to /lib/systemd/system/pivccu-dkms.service.
Created symlink from /etc/systemd/system/debmatic.service.wants/pivccu-dkms.service to /lib/systemd/system/pivccu-dkms.service.
Created symlink from /etc/systemd/system/pivccu.service.requires/pivccu-dkms.service to /lib/systemd/system/pivccu-dkms.service.
Setting up raspberrypi-kernel-headers (1.20170703-1) ...
run-parts: executing /etc/kernel/header_postinst.d/dkms 4.9.35+
Error! Bad return status for module build on kernel: 4.9.35+ (armv7l)
Consult /var/lib/dkms/pivccu/1.0.36/build/make.log for more information.
run-parts: executing /etc/kernel/header_postinst.d/dkms 4.9.35-v7+
Error! Bad return status for module build on kernel: 4.9.35-v7+ (armv7l)
Consult /var/lib/dkms/pivccu/1.0.36/build/make.log for more information.
root@raspberrypi:~#

Looking the make.log:
root@raspberrypi:/usr/src# more /var/lib/dkms/pivccu/1.0.36/build/make.log
DKMS make.log for pivccu-1.0.36 for kernel 4.9.35-v7+ (armv7l)
Sun 22 Sep 10:20:44 CEST 2019
make -C /lib/modules/4.9.35-v7+/build M=/var/lib/dkms/pivccu/1.0.36/build modules
make[1]: Entering directory '/usr/src/linux-headers-4.9.35-v7+'
  CC [M]  /var/lib/dkms/pivccu/1.0.36/build/eq3_char_loop.o
  CC [M]  /var/lib/dkms/pivccu/1.0.36/build/plat_eq3ccu2.o
  CC [M]  /var/lib/dkms/pivccu/1.0.36/build/generic_raw_uart.o
/var/lib/dkms/pivccu/1.0.36/build/generic_raw_uart.c: In function ‘generic_raw_uart_get_gpio_pin_number’:
/var/lib/dkms/pivccu/1.0.36/build/generic_raw_uart.c:726:12: error: implicit declaration of function ‘dev_fwnode’ [-Werror=implicit-function-declaration]
     struct fwnode_handle *fwnode = dev_fwnode(dev);
            ^
/var/lib/dkms/pivccu/1.0.36/build/generic_raw_uart.c:726:36: warning: initialization makes pointer from integer without a cast
     struct fwnode_handle *fwnode = dev_fwnode(dev);
                                    ^
cc1: some warnings being treated as errors
scripts/Makefile.build:299: recipe for target '/var/lib/dkms/pivccu/1.0.36/build/generic_raw_uart.o' failed
make[2]: *** [/var/lib/dkms/pivccu/1.0.36/build/generic_raw_uart.o] Error 1
Makefile:1490: recipe for target '_module_/var/lib/dkms/pivccu/1.0.36/build' failed
make[1]: *** [_module_/var/lib/dkms/pivccu/1.0.36/build] Error 2
make[1]: Leaving directory '/usr/src/linux-headers-4.9.35-v7+'
Makefile:23: recipe for target 'all' failed
make: *** [all] Error 2

I have the doubt that my kernel (4.9.35-v7+)  is not supported.
Can you please help with it?

You need at least kernel 4.14. This kernel is used in Raspbian Stretch and Buster. Maybe you have an old installation which has the Jessie kernel?

Hi Alex, Ouch. Yes it is a Jessie Installation. Let me update to Stretch or Buster. Will update you soon.

Do you still have any questions or can this be closed?

I am still updating to buster. The process is quite slow on the Raspberry
Pi.
Let me confirm that everything is working properly.
On Wed, Sep 25, 2019 at 10:55 AM Alexander Reinert<EMAIL_ADDRESS>wrote:

Do you still have any questions or can this be closed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alexreinert/piVCCU/issues/185?email_source=notifications&email_token=AA3NM4AS2NIDWLFOYRHWWA3QLMRRXA5CNFSM4IZDR6L2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7REQLI#issuecomment-534923309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3NM4DPFOTJDJTV6EK7YXTQLMRRXANCNFSM4IZDR6LQ
.

Okay now I am able tp install everything but something is wrong.
piVCCU version: 2.47.15-53
Kernel modules: Available
Raw UART dev:   Available
Rasp.Pi3 UART:  Not assigned to GPIO pins
HMRF Hardware:  unknown
HMIP Hardware:  unknown
Board serial:   unknown
Radio MAC:      unknown
SGTIN:          unknown
State:          RUNNING
PID:            3042
IP:             <IP_ADDRESS>
IP:             2003:d1:df0f:a6f1:a869:99ff:fe30:d31b
IP:             fd85:6fd0:5f94:1:a869:99ff:fe30:d31b
CPU use:        15.77 seconds
BlkIO use:      3.76 MiB
Memory use:     53.80 MiB
KMem use:       2.48 MiB
Link:           vethpivccu
 TX bytes:      7.35 KiB
 RX bytes:      73.54 KiB
 Total bytes:   80.89 KiB

When I access the CCU gui I get "ccuNotReady"
My module is not recognized.
Sep 26 16:16:22 raspberrypi.home.local systemd[1]: Starting piVCCU...
Sep 26 16:16:49 raspberrypi.home.local piVCCU[4042]: HMRF hardware was not detected
Sep 26 16:16:49 raspberrypi.home.local start_container.sh[3968]: <12>Sep 26 16:16:49 piVCCU: HMRF hardware was not detected
Sep 26 16:16:49 raspberrypi.home.local piVCCU[4043]: HMIP hardware was not detected
Sep 26 16:16:49 raspberrypi.home.local start_container.sh[3968]: <12>Sep 26 16:16:49 piVCCU: HMIP hardware was not detected

I reinstalled a clean Buster installation and now everything works properly:
root@raspberrypi:~# pivccu-info
piVCCU version: 2.47.15-53
Kernel modules: Available
Raw UART dev:   Available
Rasp.Pi3 UART:  Assigned to GPIO pins
HMRF Hardware:  HM-MOD-RPI-PCB
HMIP Hardware:  HM-MOD-RPI-PCB
Board serial:   PEQ2215944
Radio MAC:      0x6bdd19
SGTIN:          3014F711A061A7D8A9AB3808
State:          RUNNING
PID:            2004
IP:             <IP_ADDRESS>
IP:             2003:d1:df0f:a6f1:389b:24ff:fe2d:df85
IP:             fd85:6fd0:5f94:1:389b:24ff:fe2d:df85
CPU use:        177.55 seconds
BlkIO use:      136.00 KiB
Memory use:     97.30 MiB
KMem use:       4.25 MiB
Link:           vethpivccu
 TX bytes:      23.54 MiB
 RX bytes:      4.79 MiB
 Total bytes:   28.33 MiB

Thanks Alex for the support.


Add WhatsApp sharing service

Subj.
Maybe this page on Stackoverflow could be useful for implementaion.

Check out #135

Created a PR for that. #135


Simplify to get kids references in DefaultPages

Let's simplify to get kids references in DefaultPages.

Closed by 1149a2e.


Develop

Codecov Report

Merging #3 into master will increase coverage by 9.5%.
The diff coverage is 51.35%.

@@           Coverage Diff            @@
##           master      #3     +/-   ##
========================================
+ Coverage   30.89%   40.4%   +9.5%
========================================
  Files           4       4
  Lines         178     198     +20
========================================
+ Hits           55      80     +25
+ Misses        109      97     -12
- Partials       14      21      +7

Impacted Files
Coverage Δ

main.go
20% <0%> (-1.06%)
:arrow_down:

filewatcher.go
30.64% <53.84%> (+30.64%)
:arrow_up:

jsonutils.go
59.37% <62.5%> (+19.98%)
:arrow_up:

Continue to review full report at Codecov.

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


Adding docs for the class directive

Fixes #350

Thanks for writing some docs to go along with your new feature! I've added a few comments about things I think should be tweaked.

@jacwright
#371
The demo of the shorthand of class binding has "Error compiling component".
Link:
https://svelte.technology/repl?version=2.14.2&demo=bz2ba6
Error:
Computed property name 'is-selected' is invalid — must be a valid identifier such as is_selected
12:     // Because shorthand relfects the var name, you must use component.set({ "is-selected": true }) or use a computed
13:     // property like this. It might be better to avoid shorthand for class names which are not valid variable names.
14:     "is-selected": ({ isSelected }) => isSelected
        ^
15:   }
16: }


[bug]: Linux installer 2.2.3 refers to non-existent directories.

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
No response
What happened?
file install.sh contains in lines 124-127 a reference to the directory "source_installer", but such a directory does not exist.
This causes the invoke.sh and update.sh character links to fail
ln -sf ./source_installer/invoke.sh.in ./invoke.sh
ln -sf ./source_installer/update.sh.in ./update.sh
chmod a+rx ./source_installer/invoke.sh.in
chmod a+rx ./source_installer/update.sh.in

Screenshots
No response
Additional context
I use install.sh first time on ny pc
download link https://github.com/invoke-ai/InvokeAI/releases/download/v2.2.3/invokeAI-src-installer-2.2.3-linux.zip
Contact Details
No response

The 2.2.3 installer will no longer work due to changes in the directory structure. Please either use the 2.2.4 installer to install 2.2.4 (recommended), or use one of the manual methods to install 2.2.3.
Sorry for the inconvenience!

For anyone else coming here from searching, you'll want to pay close attention to the "Everything now lives in the invokeai directory" section in the 2.2.4 release note here:
https://github.com/invoke-ai/InvokeAI/releases/tag/v2.2.4


PoC: Introduce File System Access API Helper / Adapter

It is work in progress.
I'll propose unified FS APIs. I'm trying implementing.
Motivation
JavaScript runtimes have different APIs to access file system. So Hono has wrapper in serveStatic and SSG Helper. They should be merged I think. Therefore I'll propose it.
Hono is just Web application framework but I'm sure that middleware creators can create more powerful middleware.
Web Standard
Hono follows Web Standard APIs. This PR will use WHATWG File System Access API (Spec, MDN).
However File System API is formulated for browsers, features for using local filesystem is sometimes missing. (e.g., Get absolute path from entry) So I'm going to extend API interface.
API Plans/Ideas
Open dir/file:
import { openDir, openFile } from 'hono/deno'

const dir = openDir('./public') // It's FileSystemDirectoryHandle but extended
const dir = openFile('./public/example.txt') // It's FileSystemFileHandle but extended

Serve static:
import { openDir, openFile } from 'hono/deno'
import { serveStatic } from 'hono/serve-static'

app.use('*', serveStatic({
  root: openDir('./public')
})
app.get('/favicon.ico', serveStatic({
  file: openFile('./assets/favicon.ico')
})

SSG:
import { toSSG } from 'hono/ssg'
import { openDir } from 'hono/deno'

toSSG(app, openDir('./dist'))

Progress
Currently it's almost an idea.
Runtimes

[ ] Deno

[x] readDir (without file access)
[ ] readFile

[ ] Bun
[ ] Node.js

Features

[ ] SSG
[ ] Serve-Static

I want forward compatibility.
The author should do the following, if applicable

[ ] Add tests
[ ] Run tests
[ ] bun run format:fix && bun run lint:fix to format the code
[ ] Add TSDoc/JSDoc to document the code

@nakasyou
Can we refer to these?

https://github.com/mjackson/remix-the-web/tree/main/packages/file-storage
https://github.com/edgefirst-dev/r2-file-storage

@yusukebe
You can implement adapter for those.
Or, do you mean to make an API similar to this?


Fix generation of hash sets to actually use std::collections::HashSet

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

You need to be sure this is what you actually want–we didn't use hashsets
for a few reasons, mainly they produce nondeterministic serialization.
On Mon, Dec 13, 2021, 7:48 AM github-actions[bot] @.***>
wrote:

A new generated diff is ready to view.

No codegen difference in the AWS SDK
Server Test
https://d2luzm2xt3nokh.cloudfront.net/codegen-diff/f9e5d64a15ee2eebbf5d68a8a86e713da6a3c037/b6b8042cc6350055125b36d43771a63d0e0f9149/diff-server-test.html
(ignoring whitespace
https://d2luzm2xt3nokh.cloudfront.net/codegen-diff/f9e5d64a15ee2eebbf5d68a8a86e713da6a3c037/b6b8042cc6350055125b36d43771a63d0e0f9149/diff-server-test-ignore-whitespace.html
)

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/smithy-rs/pull/962#issuecomment-992444632,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYKZZ42OK7ACH32UI7SGDUQXTTRANCNFSM5J6BHIJA
.

ah yeah you won't be able to do this unless you do it only for the server. We made it a Vec for a few reasons:

backwards compatibility. Because floats aren't hashable if a float key was added in the future we would lose derive Hash. There are ways around this (eg. manual hash derivation)
deterministic behavior: if you store values in a hashset there isn't a practical way to serialize them in a deterministic way. This makes dealing with protocol tests much tricker

ah yeah you won't be able to do this unless you do it only for the server. We made it a Vec for a few reasons:
* backwards compatibility. Because floats aren't hashable if a float key was added in the future we would lose derive Hash. There are ways around this (eg. manual hash derivation)

* deterministic behavior: if you store values in a hashset there isn't a practical way to serialize them in a deterministic way. This makes dealing with protocol tests much trickier

Maybe the outcome of this PR should be to add comments to that effect in RustTypes.kt?

Interesting. I think there's an even stronger reason to use Vec. I've just learnt that the Smithy spec says "Sets MUST be insertion ordered." The only programming languages whose stdlibs provide insertion-ordered set types that I know of are JavaScript and Java's LinkedHashSet.

Alternatively, we could use something like IndexSet

yeah we considered IndexSet but we wouldn't want to expose that publicly (we would've needed to create a wrapper in aws-smithy-types (which is still not out of the question).
I don't remember the insertion ordered requirement @david-perez but TIL!

closing, a comment was added clarifying the behavior


does your pod support resign responder

Yes since it inherits from UITextField it supports everything that a UITextField will do including resigning first responder


Optional Extras (Signals) for NonUnity

It would be nice to have the new Signals (optional extras) also in the non-unity project. Are there plans to do that?
Regards

Yeah we can do that

That would be great! Thanks!

Ok I've added this and will be in the next release


Implement a delete command

This command should read the .cogs/spreadsheet_id.txt file created in #2 and delete the Google Sheet with the matching ID. Then it should delete the .cogs/spreadsheet_id.txt file.

Since we changed init to create the sheet as well, I'm thinking that delete should working like this:

read .cogs/config.tsv to get the Sheet details
delete the Google Sheet
delete the .cogs directory
do not touch any local paths specified in sheet.tsv

Sounds good.


Just export/copy last response from ChatGPT

Is there anyway we can have an option to Just export/copy last response from ChatGPT?

Hmm I think we can put it around the 👍 👎  buttons. This will be useful for copying long response from ChatGPT.

That would be awesome!!!!

Large screen(right side)

Medium and Mobile

Large screen(put it to the next line so that it won't shrink the width of text content)  Medium and Mobile

Looks complicated with the code blocks (in mobile)


openbmc,Hwmon Sensors, Fan diaplay web problem

Hi,
I have a problem  need your help,
And
I set it  like this:

Linux has achieved the normal control of PWM Fan:

use ipmitool sensor can display the fan information:

but web Unable to display the fan:

so need your give me some guidance,Thank you very much!
Thanks.
Best Regards

This is not the appropriate place to ask for support on OpenBMC overall...  Please use the mailing list or Discord.


Bug by reusing Set with the same prefix

In the following line the contract reuses the Set, by using the same prefix for all users
https://github.com/near-examples/NFT/blob/bbafd2195d4f6253d84a8cced560fda10fd51d35/contracts/rust/src/lib.rs#L81
The prefix should uniquely identify the collection. Right now it uses the same prefix for access for all users.
One option is to use prefix like like access-<account_id_hash>

In the near-sdk FT implementation the contract uses the account-id-hash without prefix:
https://github.com/near/near-sdk-rs/blob/b8277e009e1d7d8a95faa0b77d99ded0b8afc81d/examples/fungible-token/src/lib.rs#L36

fixed since implementation has changed


Install php-amqp for messagin development

Could you please include the  extension for Shopware 6? So that you can also develop messaging with e.g. RabbitMQ.

was already released according to source codes


"source.organizeImports" is reordering imports and comments in polyfills.ts

Issue Type: Bug
Given a simple angular project, created with ng new angular-repro, update .\.vscode\settings.json to:
{
  "editor.codeActionsOnSave": {
    "source.organizeImports": true
  }
}

Then edit the polyfills.ts file to either uncomment some of the imports or document why some polyfill imports are needed. For example, I added this at the end of the file:
/***************************************************************************************************
 * Zone JS is required by default for Angular itself.
 */
import 'zone.js/dist/zone'; // Included with Angular CLI.

/***************************************************************************************************
* APPLICATION IMPORTS
*/
// Needed to use dynamically compiled components in AOT/prod builds: https://github.com/angular/angular/issues/27584#issuecomment-446462051
import 'core-js/es7/reflect';

When I save the file, the text is reordered to:
/***************************************************************************************************
 * Zone JS is required by default for Angular itself.
 */
/***************************************************************************************************
* APPLICATION IMPORTS
*/
// Needed to use dynamically compiled components in AOT/prod builds: https://github.com/angular/angular/issues/27584#issuecomment-446462051
import 'core-js/es7/reflect';
import 'zone.js/dist/zone'; // Included with Angular CLI.

Which makes the comments less than useful, and is frustrating. The only ways to save the file without triggering this reformatting are:

Disable the line in settings.json (requires developers to know that this is causing the reformatting)
or close VS Code without saving.

VS Code version: Code 1.30.0 (c6e592b2b5770e40a98cb9c2715a8ef89aec3d74, 2018-12-11T22:29:11.253Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz (12 x 2904)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.74GB (3.95GB free)

Process Argv
--folder-uri file:///c%3A/src/ease/WebUi

Screen Reader
no

VM
0%

Extensions (24)

Extension
Author (truncated)
Version

ng-template
Ang
0.1.11

vscode-svgviewer
css
1.4.7

dart-code
Dar
2.21.1

flutter
Dar
2.21.1

githistory
don
0.4.4

xml
Dot
2.3.2

gitlens
eam
9.2.0

EditorConfig
Edi
0.12.5

json-tools
eri
1.0.2

todo-tree
Gru
0.0.109

svg
joc
0.1.4

docomment
k--
0.1.2

vscode-json5
mrm
1.0.0

csharp
ms-
1.17.1

PowerShell
ms-
1.10.1

vscode-typescript-tslint-plugin
ms-
0.2.0

debugger-for-chrome
msj
4.11.1

debugger-for-edge
msj
1.0.6

vscode-jest
Ort
2.9.2

tsimporter
pmn
1.2.14

partial-diff
ryu
1.4.0

code-settings-sync
Sha
3.2.4

stylelint
shi
0.47.0

quokka-vscode
Wal
1.0.174

Issue is that comments are not being moved along with imports. However there also cases where you really do not want the comments moved with the imports, such as with copyright header comments
Moving upstream to get more feedback

The file leading comment should be moved with the import unless it's a "pinned" comment (a comment which starts with /*!).

See #22731. I am not in favor of regressing this as I  do not see many codebases using pinned file headers

@DanielRosenwasser @amcasey your opinions?

If there were a way to easily disable reformatting a file, that would work here as a workaround; but this would still be an issue, and developers would have to discover the workaround.
Note that every angular project has a polyfills.ts file of this format. The expected use is that devs will comment out, or uncomment, imports based on their anticipated needs.
Leaving this as is guarantees frustration for any angular developer who has this feature enabled, and tries to edit the polyfills.ts file.

My preference is: Don't re-order imports if comments are co-mingled between the imports.

I'm not sure if this issue is just about re-ordered imports loosing their comments... I think the more important issue is that these imports must not be re-ordered at all because polyfills may need to be loaded in a specific order.
So while import organizing is a pretty useful feature for every other file there must be a way to prevent import reordering on this specific file.
Some suggestions:

Add a new setting (i.e. source.organizeImports.exclude) with which we can specify file globs which are then excluded from import organizing.

Accept comments for grouping imports and only reorder imports within the groups. So this:
// Base polyfill
@import "core-js";

// Other polyfills
@import "foo";
@import "bar";

// Modules
@import "b";
@import "c";
@import "a";

becomes this:
// Base polyfill
@import "core-js";

// Other polyfills
@import "bar";
@import "foo";

// Modules
@import "a";
@import "b";
@import "c";

Don't organize imports when there is a tslint:disable:ordered-imports comment in the file which is most likely there anyway to disable the corresponding tslint rule.

Implement your own marker comment to disable import organizing. Maybe a tripple-slash-directive?
/// <disable-organize-imports />
@import "core-js";
@import "foo";
@import "bar";

Found a stinking workaround for the problem which may cause some trouble with some tools but so far it seems to work at least with tsc, vscode, tslint, webpack and webpack-dev-server:
import "01/../core-js";
import "02/../hammerjs";
import "03/../raf-polyfill";
import "04/../zone.js";
import { enableProdMode } from "@angular/core";

Issue is that comments are not being moved along with imports. However there also cases where you really do not want the comments moved with the imports, such as with copyright header comments

This might be a bit outdated now. Today we do move comments when we organize imports, with the exception of the first comment because it could be the header. We could maybe be a little smarter with this by looking at blank lines so that we know the comment on the import here is separate from the header:
before organize
/*******************************************
* This is the header!
* I know the header is separate from the 'zzz' comment because there is a blank line between them!
*/

/**
* zzz
*/
import "zzz"

/**
* aaa
*/
import "aaa"

after organize
/*******************************************
* This is the header!
* I know the header is separate from the 'zzz' comment because there is a blank line between them!
*/

/**
* aaa
*/
import "aaa"

/**
* zzz
*/
import "zzz"

Otherwise it seems like you would not want the setting turned on in the first place if you need your imports to happen in a certain order, but configuring that on a per file basis would be a VSCode change.
@mjbvz Aside from this template, are there other cases where you would want to organize imports within separate sections?

No, I have not see other reports of this against VS Code

@mjbvz Thanks. It looks to me like the ask here is for the ability to toggle organizeImports at the file level which is probably just a VSCode thing. Should we move this issue back to VSCode?

This would be a new concept for TypeScript as well. We only have compiler directives right now, not language service directives.
Our view on this at the moment is that a refactor request should not be sent if a refactor is wholly undesirable as it is in this case. We have a lot of refactors and quick fixes so allowing directives for them might quickly add up to a lot of not very meaningful requests to the server.
It would be good to get more feedback on the proposed idea of grouping imports though. Maybe we can do something clever with regions?
//#region group1
import "z"
import "y"
//#endregion

//#region group2
import "b"
import "a"
//#endregion

//#region group1
import "y"
import "z"
//#endregion

//#region group2
import "a"
import "b"
//#endregion

I've integrated organizeImports into a prettier plugin (see mentioned issue 👆) so that it runs as a preprocessor of the formatter. We've added support for an // organize-imports-ignore comment which stops prettier from running organizeImports on files that contain the comment. So that solves the whole "per-file basis" problem of this issue for us.
But there's still a problem with the order of comments:
// comment on top of line 1
import foo from "foo";
// comment on top of line 2
import bar from "bar";

console.log(foo, bar);

becomes
// comment on top of line 1
// comment on top of line 2
import bar from 'bar';
import foo from 'foo';

console.log(foo, bar);

It seems like the "line 1" comment is considered a file header (first comment of the file) and never moved?
import abc from 'abc';
// comment on top of line 1
import foo from "foo";
// comment on top of line 2
import bar from "bar";

console.log(abc, foo, bar);

👆 this one works as expected (comments are moved with the imports):
import abc from 'abc';
// comment on top of line 2
import bar from 'bar';
// comment on top of line 1
import foo from 'foo';

console.log(abc, foo, bar);

We could maybe be a little smarter with this by looking at blank lines

That's pretty much what I mean. I'm not saying it's wrong that the first comment doesn't get moved but i think that should only be the case if it's followed by a blank line. That's pretty common I would think.
/cc @mohsen1 have a look at https://github.com/microsoft/TypeScript/issues/29028#issuecomment-607520158

Here's another standard angular case (eg the file is created by ng new) where the organizeImports behavior (again, in VS Code only) combined with autoformatting on save is causing major problems:
Given a test.ts file like:
// This file is required by karma.conf.js and loads recursively all the .spec and framework files
import 'zone.js/dist/zone';
import 'zone.js/dist/zone-testing';

import { getTestBed } from '@angular/core/testing';
import {
  BrowserDynamicTestingModule,
  platformBrowserDynamicTesting
} from '@angular/platform-browser-dynamic/testing';

Upon saving, the imports become:
// This file is required by karma.conf.js and loads recursively all the .spec and framework files
import { getTestBed } from '@angular/core/testing';
import {
  BrowserDynamicTestingModule,
  platformBrowserDynamicTesting
} from '@angular/platform-browser-dynamic/testing';
import 'zone.js/dist/zone';
import 'zone.js/dist/zone-testing';

This reordering causes tests to fail with an error message that is difficult to debug. This StackOverflow post exactly describes the problem that is caused when imports are re-ordered.
I would be happy if there were a comment or region text that could be used to disable re-ordering for a file, or portion of a file.
In short, organizeImports is placing ES6 imports

Of the proposals in this issue, I would love to see something like:
/// file:disable-organize-imports

(obviously to skip organizing imports for the whole file)
and
/// <disable-organize-imports>
...
imports that won't be touched,
and imports before and after this block won't be moved before it or after it.
...
/// </disable-organize-imports>

I do really like @kayahr 's proposal:

Don't organize imports when there is a tslint:disable:ordered-imports comment in the file which is most likely there anyway to disable the corresponding tslint rule.

But since tslint is on its way out, I don't think it makes much sense to tie the syntax to tslint. Maybe an equivalent using an existing eslint directive would be the way to go.

Disabling organize imports (and similar behaviors like formatting, see #18261) in a specific region makes the most sense as a TypeScript feature.
Disabling at the file level is something that should be implemented as a editor setting, as TS does not organize imports unless asked to by an editor. I don't believe VSCode even organizes imports on save unless you explicitly enable the feature via the editor settings. I imagine something like this:
{
  "editor.codeActionsOnSave": {
    "source.organizeImports": true,
    "ignore": [
        "polyfill.ts",
    ]
  }
}

Closing this as the original issue is a VSCode settings limitation issue.

Hi all,
I'm having the same issue with my 'polyfill.ts', I've read the responses and I think any number of the suggested would be welcome.
This is in now way a solution or even ideal, but since 'polyfill.ts' is a file that changes so rarely - this workaround may get some of us by in the meantime:

Copy the 'polyfill.ts' file to a location outside of the project directory
Open the copied 'polyfill.ts' in a new window
Organise it appropriately
Save it (outside of the project directly)
Drag it back into the project (overwriting the 'polyfill.ts' that already exists)
Now when you open the file it will be organised as required - DON'T save

Notes:

If you add a new polyfills you will need to do this again
If your 'organizeImports' is set globally (not just at the project level) you will need to use an alternative text editor to correct the order
If you run any commands throughout the project or have some git hook that does this for you, this approach may not work for you. If this is the case, it may be wise to alter your commenting layout to complement the grouping of the imports.

It looks like the TypeScript team decided not to fix this issue, b/c it's a "VSCode settings limitation issue".
After wasting more time on bugs caused by automatic (but incorrect) import ordering, I've found a workaround that works nicely:

Disable VSCode + typescript organize imports on save
Add an eslint plugin that correctly orders imports, and configure the eslint to run on save

Here are the changes:
In .vscode/settings.json :
// .vscode/settings.json

  "editor.codeActionsOnSave": {
- (remove):  "source.organizeImports": true,
- (add):    "source.fixAll.eslint": true
  },

In .eslintrc enable the simple-import-sort plugin (the eslint default sort-imports plugin doesn't work in typescript):
// .eslintrc

{
  "plugins": ["simple-import-sort"],
  "rules": {
    "simple-import-sort/imports": "error"
  }
}

After making these changes, run eslint --fix to update your imports, then you should be good to go.


fix: multipart request exception issue

Description

Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Codecov Report

Merging #10 (4141a29) into master (9e83960) will decrease coverage by 0.17%.
The diff coverage is 54.54%.

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

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   84.02%   83.85%   -0.18%
==========================================
  Files          36       36
  Lines        1271     1276       +5
==========================================
+ Hits         1068     1070       +2
- Misses        203      206       +3

Impacted Files
Coverage Δ

lib/src/helpers/shared_preferences_manager.dart
100.00% <ø> (ø)

lib/src/http/chucker_http_client.dart
90.00% <ø> (+4.28%)
:arrow_up:

lib/src/interceptors/chopper_interceptor.dart
100.00% <ø> (ø)

lib/src/interceptors/dio_interceptor.dart
89.06% <54.54%> (-7.31%)
:arrow_down:

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 9e83960...fc29298. Read the comment docs.


Hassio in docker onto RPi4

Hi all, not directly a issue, more a request.
Currently have a Hassio running in Docker on a RPi3b
Any chance we can get a how to migrate to RPi4 document compiled.
Understand the easiest might be do to a Raspbian upgrade in place, but Raspberry community is implying rather do a full fresh install, which imply if thats the route, new SD card then what's all involved to migrate from SD to new SD...
G

Take a snapshot and restore it on new system. One of the amazing feature of Hass.io that you can almost restore everthing on different Hardware.

take note I don't have a bare metal Hassio deployment. see my  one above, this is Hassio inside docker on Stretch.
is there a document/link that defines how to do this.
G

That's not an official way to run Hass.io and there is also no support. You should next time simple run the installer script and you are fine with any movments ;)

hmmm, actually it is a method laid out by Hass.io community on their site as an alternative...
, but not to worry.
thanks
G

Guys,
Probably slightly off-topic but I don't want to mess around with other issues being created. I'm new to hassio and wanted to install it on my fresh Rpi-4 however it's not yet listed on https://github.com/home-assistant/home-assistant.io/blob/current/source/hassio/installation.markdown
I could just run the container Docker version of it but I think it might be more logical to run the standalone full OS version of it as I will have a dedicated Rpi.
Any timeline or suggestion about achieving so?
Thanks!

As a new build, pretty easy, Deploy Buster (Rpi4 OS), deploy Docker, and then follow the Hass.io Docker deployment guide on the Home-Assistant site.
G

PS: took me about a hour.
G

Thanks for your prompt response @georgelza
I'll give it a go in the coming days, I need to find time though! Any special care while running this application in a Docker container apart from inherited Docker thumb rules?
Cheers

Biggest thing is to get docker installed...
2nd thing.. check your swop space allocated to the OS. I kept on running out, based on default build, I increase the swop to 1GB, I'd say if you can, when building the OS, look at increasing it, before installing any software.
G

I bet you can increase the SWAP partition at any time by modifying this variable in the files mentioned in the URL below.
https://wpitchoune.net/tricks/raspberry_pi3_increase_swap_size.html

True/Correct, you can, I just mentioned do it now, before installing anything, so that you don't forget, end with a slow down, HA freeze later and wonder why,
G

Wonderful - Will give it a go then. I just burnt the latest Buster image in my SD, but I'll start from scratch in case I regret it later :)
Cheers @georgelza


Update DAOS blueprints to use google-cloud-daos v0.5.0, slurm v6

DAOSGCP-182

Bump version of DAOS modules to v0.5.0 which install DAOS v2.4

Modify community/examples/intel/hpc-slurm-daos.yaml to use Slurm v6 modules

Add temporary fix to community/examples/intel/hpc-slurm-daos.yaml to work around issue with missing lustre-client 8.8 repo

Update community/examples/intel/README.md to account for changes in DAOS v2.4

/gcbrun

/gcbrun

/gcbrun

LGTM now. Merging it.


MySQL Database Backup

This PR adds a new script to backup a MySQL database completely, including schema, data, triggers and procedures.
Also, this script does not lock tables and uses single transaction.
It does not creates a database and you must create it manually on import.

TY @dexfs to show me this project.

@wandersonwhcr your PR is fantastic ❤
Thanks for your contribution.


Bump conditional authentication functions version.

Purpose
$subject
Related PR

https://github.com/wso2-extensions/identity-conditional-auth-functions/pull/112

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3512134959

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3512134959
Status: success


New step-functions-workflows-collection - fileprocessor workflow

Issue #, if available:
Description of changes:
This sample project shows handling of file processing workloads following the eventdriven microservice pattern. Deploying this sample project will create 2 S3 buckets, Eventbridge rules, one stepfunction 3 lambdas and a couple of roles for working successfully
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

There are over 4,000 files on this pull request, please review, and make a new PR with only the files needed to deploy the workflow.


Does not connect a second time if it closes the connection ver. 10.0.17763.771

Does not connect a second time if it closes the connection
After restarting the computer, the connection is successful
If I choose another authentication method, click the apply button, the connection is successful. BUT! If I close the connection and try to open it again, it gives an error
https://www.screencast.com/t/BYPXGY9bj7O
This means:

The admin has ended this session.
An error occurred while establishing the connection.
There was a network problem.

ver. 10.0.17763.771
my conf
https://www.screencast.com/t/CnJ0M43mI

[10.0.17763.771]
LocalOnlyPatch.x86=1
LocalOnlyOffset.x86=AFEB4
LocalOnlyCode.x86=jmpshort
SingleUserPatch.x86=1
SingleUserOffset.x86=4D7F5
SingleUserCode.x86=nop
DefPolicyPatch.x86=1
DefPolicyOffset.x86=4BFF9
DefPolicyCode.x86=CDefPolicy_Query_eax_ecx
SLInitHook.x86=1
SLInitOffset.x86=5B30A
SLInitFunc.x86=New_CSLQuery_Initialize
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=77AD1
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=1339C
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=18025
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=1ACDC
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.17763.771-SLInit]
bInitialized.x86 =CD79C
bServerSku.x86 =CD7A0
lMaxUserSessions.x86 =CD7A4
bAppServerAllowed.x86 =CD7AC
bRemoteConnAllowed.x86=CD7B0
bMultimonAllowed.x86 =CD7B4
ulMaxDebugSessions.x86=CD7B8
bFUSEnabled.x86 =CD7BC
bServerSku.x64 =ECAB8
lMaxUserSessions.x64 =ECABC
bAppServerAllowed.x64 =ECAC4
bInitialized.x64 =ECAB4
bRemoteConnAllowed.x64=ECAC8
bMultimonAllowed.x64 =ECACC
ulMaxDebugSessions.x64=ECAD0
bFUSEnabled.x64 =ECAD4

Thanks @s0nic9!! This solved it for me.

update : one line registry then reboot

reg add "HKLM\SOFTWARE\Policies\Microsoft\Windows NT\Terminal Services" /v fEnableWddmDriver /t reg_dword /d 0 /f


Enable asynchronous data copy to get a better performance

What does this PR do?
Improve Habana HPU performance, especially the host side performance by enabling
asynchronous data copy from host to device with arg non_blocking_data_copy.
Before setting non_blocking_data_copy to True

After setting non_blocking_data_copy to True

Here is the overall performance before and after enabling non_blocking_data_copy (take Swin as an example):
Gaudi2

non_blocking_data_copy
1x
8x

True
374
1894

False
354
1857

Gaudi

non_blocking_data_copy
1x
8x

True
183
1060

False
183
1061

Both 1x and 8x performance get improvement on Gaudi2 but on Gaudi, none of them get any improvement. The reason is that the device computing on Gaudi2 is faster than Gaudi, when asynchronous data copy is enabled, the host can continue to execute other tasks like graph building, then we can get a better pipelining between the host and the device.  But on Gaudi, the device executing time is much longer which already hides the host time, so this optimization is more appropriate for Gaudi2.

[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @regisss, is there anything left I need to do for this PR?

Hi @regisss, is there anything left I need to do for this PR?

Ah I thought you would add the paragraph above to the documentation. But it's okay, I can do it in another PR.
I'm just going to run some tests locally because for now GitHub cannot trigger them when the PR comes from a fork. And I let you know when it's good.

Hi @regisss, is there anything left I need to do for this PR?

Ah I thought you would add the paragraph above to the documentation. But it's okay, I can do it in another PR.
edit: I already ran the tests before and they all passed, so it's good to merge 🙂

Oh, sorry, I misunderstood. Thanks!


Add: Keras reproducibility guide in the call

@fchollet I have some preliminary work done in this: https://bit.ly/34V5ZNz. I wanted to extend it and include a call for contribution for this particular guide.
I have some pointers in mind:

Fixing seeds
Ensuring development infrastructure unification
tf-determinism for using deterministic CUDA kernels to trade away performance

With these, it's possible to at least some amount of reproducibility. But I am of course open to any suggestions you may have. I think it's time we had a guide on this topic.

Thank you @fchollet.

A TF API for enabling fully-deterministic computation at the cost of a performance degradation, tf.config.experimental.enable_op_determinism()

Sounds really good. We might want to even include a friendly warning message on the performance degradation part.


Set config.server.keepAliveTimeout to 0 should disable keep-alive

It seems setting config.server.keepAliveTimeout to 0 doesn't disable keep-alive, but using the default value (5000ms according to the nodejs doc).
https://github.com/verdaccio/verdaccio/blob/e40083a6be4abf1a3539b776d3a4053f484e0002/src/lib/bootstrap.js#L51

Hi @toddwong,

server.keepAliveTimeout to 0

This evaluates to false as it seems.

@DanielRuf
Yes, and leaves the webServer.keepAliveTimeout untouched.  But I want setting it to 0 instead.

Please see the FIXME line and https://github.com/verdaccio/verdaccio/blob/master/conf/default.yaml#L69

The fallack seems to be the NodeJS default when it is 0.
Otherwise we have to do a typeof === "undefined" check.
https://github.com/verdaccio/verdaccio/blob/e40083a6be4abf1a3539b776d3a4053f484e0002/src/lib/bootstrap.js#L52

Default is 5000 (= 5 seconds). https://nodejs.org/api/http.html#http_server_keepalivetimeout

I'll propose a PR based on

A value of 0 will disable the keep-alive timeout behavior on incoming connections. A value of 0 makes the http server behave similarly to Node.js versions prior to 8.0.0, which did not have a keep-alive timeout.

Please check out https://github.com/verdaccio/verdaccio/pull/1353

Thanks @DanielRuf
Very looking forward to the release

If you need it earlier you can use patch, git apply or patch-package with https://github.com/verdaccio/verdaccio/pull/1353.patch or https://github.com/verdaccio/verdaccio/pull/1353.diff

https://github.com/verdaccio/verdaccio/releases/tag/v4.0.4


Don't use build and config gradle caches in RC building workflows

Goal

Testing

Release Notes

WHAT:
WHY:

#1241  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite

Merge activity

Aug 13, 4:41 PM EDT: @bidetofevil started a stack merge that includes this pull request via Graphite.


PULL REQUESTS WILL BE MERGED

PULL REQUESTS WILL BE MERGED
Thankyou so much everyone for contributing in the repository.
As i am the only one to manage the repo so after some simple checks I will surely merge your pull request.
Happy Coding! and contributing to open source.
These contributions are really valuable.

issue#156  ,can you plz explain it ?

This is just to tell the contributors that all pr will be merged


Incorrect normalization factor in attention

Tutorial: 6
Describe the bug
The normalization in scaled_dot_product() doesn't use the correct dimension and is thus in general wrong.
To Reproduce (if any steps necessary)
This can be seen by printing d_k.
Expected behavior
The normalization quantity d_k should in fact be the "hidden" dimension of the queries or keys.
Thus, instead of
def scaled_dot_product(q, k, v, mask=None):
    d_k = q.size()[-1]

we should have
```python
def scaled_dot_product(q, k, v, mask=None):
    d_k = q.size()[-1]

Hi, the normalization ensures that the variance of the dot product between one query and one key does not blow up and stay in the expected range of 1. The number of elements in the sum depend on the number of hidden dimensions/dimensionality size of the keys and queries, i.e. d_k. This is independent of the sequence length/number of keys/values and size of the value vectors. Hence, it can be determined by q.size()[-1] or k.size()[-1], but can't be inferred by the values. Does this answer your question, or can you specify the bug more, e.g. by giving an example?

You are correct. I just realized this 20 minutes after your response. Sorry for the false alert!


Add scripts to modify "Low" and "Very Low" ScenarioMIP/SSP scenarios

This PR adds my scripts that I developed during the ScenarioMIP submission, to modify the "Low" and "Very Low" scenarios.
This PR is associated with the following:

The branch ssp-dev
The PR: #235
The issue: #234

The changes made to the "Low" (SSP1 & SSP2) scenarios were:

Removed growth_activity_lo and initial_activity_lo for bof_steel
Changed growth_activity_up for dri_gas_steel, dri_h2_steel, eaf_steel from 0.02 to 0.075
Changed initial_activity_up for dri_gas_steel  and dri_h2_steel from 0.1 to 1
Added balance equalities

The changes made to the "Very Low" (SSP1 & LED) scenarios were:

The same changes made to the "Low" scenarios above
Disallow the revival of loil_rc and biomass_rc in residential & commercial sector by changing the bound_activity_up
Set the UE share of gas in transportation to 0.05 and the UE share of electricity in transportation to 0.9 (starting in 2035)
Replace the tax_emission from 2080 onwards

I've uploaded the UE shares Excel files I use to message_ix_models/data/ue-shares, but please let me know if that would be a problem.
How to review
TBD
PR checklist
TBD

Hi @measrainsey — @glatterf42 and I were just discussing the various SSP-related branches, issues, and PRs.
What I'd suggest here:

Let's merge this branch into ssp-dev, as-is.
Then, ssp-dev can serve its purpose as being the "single point of reference" for @OFR-IIASA and others, containing all SSP-related code in message-ix-models.
That does not mean that this code will next get merged to main as-is. Per the description of #225, we can later choose how/whether to clean up, rearrange, etc. the code.
It also does not preclude you making a further branch from ssp-dev, or commits directly to ssp-dev, to improve the scripts for the "Low" and "Very Low" scenarios, as we probably will need to do while preparing the next round of submissions.

We can imagine that perhaps @ywpratama @yiyi1991 and @macflo8 have similar code or scripts. It would be better to collect these on the same, single branch, rather than have them sit separately on multiple branches; in the latter case we won't achieve our goal of having that single point of reference.

hi @khaeru and @glatterf42 -- that works for me, thanks! would you like to go ahead and do the merging, or should i? (or should i open up this PR for review and we go through the process of review, approve, merge)?

Please just go ahead! The usual process only applies for PRs into main. For other target branches, as long as "All checks have passed" you can merge.

Reclarrify here that my examinations on the M scenario (multiple alternative NPi_forever pathways) were not making more sense than the original version so no specific updates from my side (and sorry for not being able to reply earlier).


Deserialize polymorphic open class

Hey,
i'm currently trying to deserialize a list of a polymorphic open class, but this fails with Kind OPEN is currently not supported., which seems like some limitation? But i wonder why, since it's also possible to generate a schema for it, which you also cover within your tests.
A solution is to make the BaseEvent sealed, which will probably work for me but prevents me to build a package which can be used by my other packages.

Here an example code:
package example

import com.github.avrokotlin.avro4k.Avro
import com.github.avrokotlin.avro4k.io.AvroDecodeFormat
import com.github.avrokotlin.avro4k.io.AvroEncodeFormat
import kotlinx.serialization.Serializable
import kotlinx.serialization.modules.SerializersModule
import kotlinx.serialization.serializer
import org.apache.avro.generic.GenericData
import org.junit.jupiter.api.Test
import java.io.ByteArrayOutputStream
import kotlin.test.assertEquals

@Serializable
abstract class BaseEvent {
    abstract val id: Int
}

@Serializable
data class SubEvent(
    override val id: Int,
    val value: String,
) : BaseEvent()

@Serializable
data class AnotherEvent(
    override val id: Int,
    val value: String,
) : BaseEvent()

@Serializable
data class History(
    val events: List<BaseEvent>
)

val module = SerializersModule {
    include(Avro.defaultModule)
    polymorphic(BaseEvent::class, SubEvent::class, serializer())
    polymorphic(BaseEvent::class, AnotherEvent::class, serializer())
}

class SerializationTest {

    @Test
    fun `serialize and deserialize list of polymorphic open classes`() {
        val history = History(
            listOf(
                SubEvent(
                    1,
                    "Foo"
                ),
                AnotherEvent(
                    2,
                    "Bar"
                )
            )
        )

        val historySerializer = History.serializer()
        val avro = Avro(module)
        val historySchema = avro.schema(historySerializer)
        val buffer = ByteArrayOutputStream()

        avro.openOutputStream(historySerializer) {
            encodeFormat = AvroEncodeFormat.Binary
            schema = historySchema
        }.to(buffer).write(history).close()

        avro.openInputStream {
            decodeFormat = AvroDecodeFormat.Binary(historySchema)
        }.from(buffer.toByteArray()).iterator().forEach {
            val reconstructedHistory = avro.fromRecord(historySerializer, it as GenericData.Record)

            assertEquals(history, reconstructedHistory)
        }
    }
}

Thanks :v:
PS: The code in this repo is locked to kotlinx.serialization 1.4.0-RC, do you think it makes sense to update to latest version in order to give people more up2date examples? I could open a PR if you like.

Just to give you a heads up, updating the deps is unfortunately not so easy because some tests fail with kotlinx.serialization 1.4.1.

kotlinx.serialization 1.4.1 has a bug that does not allow us to use it. Therefore we'll only update to 1.4.0.
I've added the missing test cases to the code base and fixed the problem. The current main should be able to handle open classes in lists/maps. Fixed with 57946b59011df2ea16233dec572f53047b37f523. I'll prepare a release.

Thanks for the fast response and fix. :muscle:


[SMALLFIX] Factor out TProtocol generation code

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18549/
Test PASSed.


mu: 0.9.11 -> 0.9.12

This release contains quite a few fixes & improvements. Works for me. Note that upstream didn't change the version string, so mu4e etc still report 0.9.11.
@the-kenny @antono

Thanks. I have create a PR that updates the version number djcb/mu/pull/605.


Extension mechanism is broken

Just want to add "x-ms-summary" (which ironically is Microsoft-based) to a Parameter but the AddExtension mechanism requires me to write a whole IOpenApiExtension class and go through a writer.  I just want to add custom key/value pairs.

This is how it should be done for reference: https://github.com/tavis-software/Tavis.OpenApi/blob/master/src/OpenApi/Model/Parameter.cs

Can't you just do .AddExtension("x-ms-summary", new OpenApiString("blah"))?

I'm going to assume this is the fault of me not actually writing any documentation for this library.  I really should do that.  I'm going to close this issue and @Cloudmersive  can you let me know if this is still an issue for you?


Login problem

Issue Type: Feature Request
where can i login in visual studio
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.17134
Restricted Mode: No

/question
See https://code.visualstudio.com/docs/editor/settings-sync


Multiple L2s

let transactions: Vec<TransactionSigned> = decode_transactions(&meta.txList);
After this, you'll have to filter on the tx chain_id to only build a block for the transactions with matching chain_id for that specific l2 node.

Oh and can you also update the readme: https://github.com/taikoxyz/gwyneth?tab=readme-ov-file#use
at least for another L2 so the pattern is clear, and maybe specify formula there as well (or link to the code).


Build failing on rpi3 "CMake Error at /usr/share/cmake-3.6/Modules/FindPkgConfig.cmake:424 (message):   A required package was not found"

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
I run the setup.sh as is and the setup still fails (on a fresh install) for my raspberry pi3
What is the expected behavior?
SampleApp gets built
What behavior are you observing?
Here's the full log:
cmake /home/pi/avs/avs-device-sdk -DSENSORY_KEY_WORD_DETECTOR=ON -DSENSORY_KEY_WORD_DETECTOR_LIB_PATH=/home/pi/avs/third-party/alexa-rpi/lib/libsnsr.a -DSENSORY_KEY_WORD_DETECTOR_INCLUDE_DIR=/home/pi/avs/third-party/alexa-rpi/include -DGSTREAMER_MEDIA_PLAYER=ON -DPORTAUDIO=ON -DPORTAUDIO_LIB_PATH=/home/pi/avs/third-party/portaudio/lib/.libs/libportaudio.a -DPORTAUDIO_INCLUDE_DIR=/home/pi/avs/third-party/portaudio/include -DACSDK_EMIT_SENSITIVE_LOGS=ON -DCMAKE_BUILD_TYPE=DEBUG
-- The CXX compiler identification is GNU 4.9.2
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
Creating the build directory for the AlexaClientSDK with build type: DEBUG
WARNING: Logging of sensitive information enabled!
Creating AlexaClientSDK with keyword detector type: Sensory
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.28")
-- Checking for modules 'gstreamer-1.0>=1.8;gstreamer-app-1.0>=1.8'
--
CMake Error at /usr/share/cmake-3.6/Modules/FindPkgConfig.cmake:424 (message):
A required package was not found
Call Stack (most recent call first):
/usr/share/cmake-3.6/Modules/FindPkgConfig.cmake:597 (_pkg_check_modules_internal)
build/cmake/MediaPlayer.cmake:14 (pkg_check_modules)
build/BuildDefaults.cmake:28 (include)
CMakeLists.txt:10 (include)
-- Configuring incomplete, errors occurred!
See also "/home/pi/avs/CMakeFiles/CMakeOutput.log".
Provide the steps to reproduce the issue, if applicable:
Just follow the github instructions on a raspberry pi3
Tell us about your environment:
AVS version: v1.6.0 released 03/08/2018
Tell us what hardware you're using:

[ ] Desktop / Laptop
[x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):
PRETTY_NAME="Raspbian GNU/Linux 8 (jessie)"
NAME="Raspbian GNU/Linux"
VERSION_ID="8"
VERSION="8 (jessie)"
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Hi amounir86,
It looks like you are running in to trouble with step 3 of Setup and Run.  Those steps are for Raspbian Stretch, not Raspbian Jessie.
This link has some steps for upgrading to Raspbian Stretch.
Thank you,
-SWH

Hi @amounir86,
Closing this due to inactivity, but please reopen if you're still having issues.
Thanks,
Kenneth


Bosch Heatpump with K30RF Module

Is your feature request related to a problem? Please describe.
I am using a Bosch K30RF module with my Bosch heat pump. Currently, this module doesn't seem to be supported by the Bosch Thermostat integration for Home Assistant. This results in my inability to monitor or control the heat pump's data (such as temperature, operational status, etc.) in Home Assistant. It would be really helpful if this module could be integrated to allow easy access to relevant data.
Describe the solution you'd like
I would like the K30RF module to be fully integrated into the Bosch Thermostat component for Home Assistant, enabling me to view and control key parameters of the heat pump (e.g., current temperature, operating modes, energy consumption) through the Home Assistant interface. The module should be configurable and functional in the same way as other Bosch heat pump modules.
Describe alternatives you've considered
An alternative would be to manually query the data through Bosch's API, but this is cumbersome and not user-friendly in the Home Assistant environment. Another approach would be using a separate application to control the heat pump, but centralizing all smart home devices in Home Assistant is the preferred solution.
Additional context
The K30RF module is used to control and monitor the heat pump. I’m attaching a screenshot of the heat pump and the module for clarification on the device that needs to be integrated. Full support for this module would significantly improve usability and flexibility for Bosch heat pump owners within Home Assistant.

Yes totally the Same as I saw now . Then maybe I should close this one an keep looking at #335


TS Server confusing files in monorepo with same package id due to multiple package.jsons with same name

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:

Version: 1.74.0 (Universal)
Commit: 5235c6bb189b60b01b1f49062f4ffa42384f8c91
Date: 2022-12-05T16:43:37.594Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.6.0

版本: 1.74.1
提交: 1ad8d514439d5077d2b0b7ee64d2ce82a9308e5a
日期: 2022-12-14T10:35:03.979Z
電子: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
作業系統: Linux x64 5.4.0-135-generic
Sandboxed: No

OS Version: MacOS BigSur 11.7, Linux Mint 20.1

Steps to Reproduce:

Clone the reproduce repository and install volar
Run Volar (Debug): Write Virtual Files command
Disable Volar Extension in the workspace and reload vscode (or just open by code --disable-extensions)
Inspect src/Hello/World/World.vue.js & src/Bye/World/World.vue.js

There is a simple video to describe the issue:
https://raw.githubusercontent.com/FlandreDaisuki/reproduce-volar-syntax-highlight-bug/master/demo.mp4

Thanks @yume-chan! Confirmed this with TS 4.9.4

Wow, this is bananas. I thought about it for a minute and think I know the cause, so I retitled it to emphasize the package.json similarity and the totally corrupted state.

This seems like intended behavior. In order for packages to be deduplicated and included only once, we identify them by name in 'package.json' and version. Thus, mentioning the path or any other information may make that impossible. Additionally, where should the line be drawn in path to incorporate the package name.
Based on repro if  a/Foo and b/Foo need to be distinct then you need to indicate so in package.json name so they are not confused with one another.

@sheetalkamat Thanks for your answer, and I think it should at least be classified as a "design limitation" instead of "working as intended".


Update xkablevif() function

xkablevif() was putting out different results from vif() when the regressors were not in alphabetical order. The table() function reordered the names of the coefficients to alphabetical order but the values that were being assigned to vifs object were still in the order they were input into lm(). I added one step to sort the vif results by the coefficient names before setting the values.

This is something I noticed when working on our final technical_analysis document in DATS 6101. I noticed that the VIFs for one of our variables looked different depending on whether I used xkablevif or vif. I hope this is helpful!

Thanks Mark!


Drop support for elixir < 1.15

Contributor checklist

[ ] Bug fixes include regression tests
[ ] Chores
[ ] Documentation changes
[ ] Features include unit/acceptance tests
[ ] Refactoring
[ ] Update dependencies

🚀 Thank you for your contribution! 🚀


Escaping issue

I've ran into an issue where I get this
unable to parse 'apachecombined-access clientip=\"X.X.X.X\",request=\"/XXX/XXX/99999/XXX/1?XXXXX=[\\\\\"XXXXX\\\\\"]&XXXX=0\",response.integer=200,bytes.long=703,response_time.double=0.065,time=1443528858': invalid number\n

The original event request was like this (I could not find the EXACT same but I've got enough to be sure there are all the same)
/XXX/XXXX/99999/XXXX/1?XXXX=[\"XXXX\"]&XXXX=0

I guess the already escaped " caused an issue and made influxdb think the IP was not surrounded by quotes.
~|⇒  irb
irb(main):001:0> def escape_value(value, quote_escape)
irb(main):002:1>       val = value.
irb(main):003:1*         gsub(/\s/, '\ ').
irb(main):004:1*         gsub(',', '\,').
irb(main):005:1*         gsub('"', '\"')
irb(main):006:1>       val = %("#{val}") if quote_escape
irb(main):007:1>       val
irb(main):008:1>     end
=> nil
irb(main):009:0> escape_value('/XXXX/XXXXX/99999/XXXX/1?XXXX=[\"XXXX\"]&XXXXX=0', true)
=> "\"/XXXX/XXXX/99999/XXXX/1?XXXXX=[\\\\\"XXXX\\\\\"]&XXXX=0\""

I got the same error. This patch to lib/influxdb/point_value.rb fixed it.
     def escape_value(value, quote_escape)
       val = value.
+        gsub("\\"){ "\\\\" }.
         gsub(/\s/, '\ ').
         gsub(',', '\,').
         gsub('"', '\"')

This should be fixed in #121, I believe. Thanks!


Update remaining physics exclude parameters to use TypedArray

Also update the documentation to refer to the get_rid() method on collision objects.
Follow-up to https://github.com/godotengine/godot/pull/69017

Thanks!


[5.5] Adds inline eloquent factory states

Frequently with eloquent factory states, you only need to change one attribute to a scalar value. You often don't actually need to use Faker at all. This change allows you to simply pass an array as the 3rd argument instead of a closure.

Note: this is simply an addition of functionality. Closures still work as expected.

I'm mostly indifferent on this, but personally I don't see enough of a benefit for all the code changes and additions.
BTW, you don't need to pass in Faker, you can do a simple closure if it's not used.

Sure, it's mostly a nice to have. When you have a factory that has multiple states, they add up quickly.

this might interest you.
https://github.com/Stydenet/Factory
in regards to this PR
https://github.com/laravel/framework/pull/18829

Once the short closure syntax is available in PHP, this will no longer be a concern.

@jerguslejko While you're not wrong, considering the RFC is still 'under discussion', IF it actually makes it into the language, it's still multiple years out. I'm not holding my breath.

Seems nice to me.


fltk: 2-alpha -> 1.3.3

FLTK 2 is abandoned (http://www.fltk.org/software.php#OLD). Moreover, the URL in the nix expression is not valid anymore.
This PR just resets default FLTK to 1.3.3.

Makes sense. Added a few notes you might want to address, seeing as you're touching the file anyway.

Was this just lost by accident?

Yes and no: I created a new branch for fltk and a new PR : #15622 (I don't know how to reuse the PR).

AFAIK github doesn't allow to switch branch of an open PR. You can only force-push into the same branch, edit description, etc.


Max: Allows user to select material duplicates option during loading max scene object

Changelog Description
This PR is to add the options/dialogs to allow user to select material duplicates options when loading max scene via Loader.
(More Info to be updated)
Additional info
The material duplicate option would be at the default setting promptMtlDups when 3dsmax is at headless mode
Testing notes:
Load...

Launch 3dsMax
Go to AYON -> Load...
Right Click load Max Scene and you will find memo icon

Click that memo icon, you can select your preferred material duplicate options.

Click Ok and it would load the max scene objects with your choice on material duplicate option.
Updating/Switching Version...
Go to AYON -> Manage...
Set Version with the max scene product type
After finishing to choose your version, it will pop up a dialog to ask to choose material duplicate types.
Click Ok button
The scene will update and load the objects with the chosen material duplicate type.
If clicking cancel button, the scene will still update and load the objects with the default material duplicate type.

It's hard for me to really understand whether this is the best design approach. You might have 10 max scenes loaded. I'd personally not want to click through 10 popups on update.
So can't really decide on what the best approach is for this use case but it does feel like an anti-pattern from the outside.

I am thinking about adding as loader setting into 3dsmax addon, when talking about the best approach.

All seems working fine in multiple occasions and circumstances...did test different scenarios with material and all options seems working as expected.
LGTM

Can we please improve the labels on those drop down menu entries? :) What is promptMtlDups? Is there no way to improve readability on those labels?

All seems working fine in multiple occasions and circumstances...did test different scenarios with material and all options seems working as expected.
LGTM

Can we please improve the labels on those drop down menu entries? :) What is promptMtlDups? Is there no way to improve readability on those labels?
Also @LiborBatek have you also tested the prompt/dialog on Update instead of Load?

@BigRoy Yes I did, I loaded asset multiple times and in different and also same version...juggling with the options a lot speaking of those material settings. Trying different circumstances and how this PR behaves under conditions like duplicate shaders etc.
All seems working nicely and without any glitches (predictably).

All seems working fine in multiple occasions and circumstances...did test different scenarios with material and all options seems working as expected.
LGTM

Can we please improve the labels on those drop down menu entries? :) What is promptMtlDups? Is there no way to improve readability on those labels?
Also @LiborBatek have you also tested the prompt/dialog on Update instead of Load?

I edited the code to be more descriptives and artist friendly in terms of the choosing options.

All seems working fine in multiple occasions and circumstances...did test different scenarios with material and all options seems working as expected.
LGTM

Can we please improve the labels on those drop down menu entries? :) What is promptMtlDups? Is there no way to improve readability on those labels?
Also @LiborBatek have you also tested the prompt/dialog on Update instead of Load?

I edited the code to be more descriptives and artist friendly in terms of the choosing options.

Much better - so what is Prompt Material? Is that like a Pick & Choose dialog?

All seems working fine in multiple occasions and circumstances...did test different scenarios with material and all options seems working as expected.
LGTM

Can we please improve the labels on those drop down menu entries? :) What is promptMtlDups? Is there no way to improve readability on those labels?
Also @LiborBatek have you also tested the prompt/dialog on Update instead of Load?

I edited the code to be more descriptives and artist friendly in terms of the choosing options.

Much better - so what is Prompt Material? Is that like a Pick & Choose dialog?

According to the documentation, it means prompting the user when duplicated materials are found. But I think it's too long as the option's name.

I see.
How about "Prompt on duplicate materials" - and I'm not sure what the "prompt" offers for the user. What happens with the prompt itself - is that like a 'decide what to include or skip' dialog?


Cannot install SharePoint Yeoman Generator

Category

[ ] Question
[ ] Typo
[X] Bug
[ ] Additional article idea

Expected or Desired Behavior
Expected behaviour is that the yeoman generator installs successfully.
Observed Behavior
Has the git repository been moved? I've followed the guides on https://dev.office.com/sharepoint/docs/spfx/toolchain/scaffolding-projects-using-yeoman-sharepoint-generator
24 error git clone git@github.com:microsoft/generator-sharepoint Cloning into bare repository
24 error git clone git@github.com:microsoft/generator-sharepoint Host key verification failed.
24 error git clone git@github.com:microsoft/generator-sharepoint fatal: Could not read from remote repository
Steps to Reproduce

Install the latest version of Node JS x64 for Windows
Open command line and run
npm install @microsoft/generator-sharepoint -g

Attached Files - Logs for
npm-debug.zip

Could you try using Node LTS (v6) with npm@3 and see if you still get the error? Your log mentions you're using Node v8 with npm@1, whereas SPFx has been tested with Node LTS.
Also, it's quite odd that your system tries to pull the generator from GitHub instead of npmjs. Did you change any configuration related to packages registry?

Hi, thanks for your quick response. I ran it again with the Node LTS v6 installed and experienced the same issue. Looking at the logs however, I noticed that the following lines were referencing an old  chocolatey path for npm which wasn't desired.  This appeared to be some left over configuration from a previous install.
'npm verb cli [ 'C:\Program Files\nodejs\node.exe',
npm verb cli   'C:\ProgramData\chocolatey\lib\npm\tools\node_modules\npm\bin\npm-cli.js',
'
Once I uninstalled this version, I was able to run the below successfully
'npm install @microsoft/generator-sharepoint -g'
I tried re-running the command against Node v8.5, npm 5.3.0 and it worked correctly as described, so it looks like this issue was caused by some incorrect configuration on my machine.
Thanks for your assistance anyway!

Great to hear you were able to fix it!