Bot crashed

My bot crashed, can't respond to commands and trade. Still shows that it's online in pm2 and no logs about its crashing.

Are you 100% sure its not in the logs

yes
On Sat, 3 Aug 2019 at 15:31, ZeusJunior<EMAIL_ADDRESS>wrote:

Are you 100% sure its not in the logs
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Nicklason/tf2-automatic/issues/135?email_source=notifications&email_token=AMLJC2EVA232GYLHH3J7NI3QCU66TA5CNFSM4IJCDNU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3PJ7LQ#issuecomment-517906350,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMLJC2FEHF5QTS7OSXFFEZDQCU66TANCNFSM4IJCDNUQ
.

I did pm2 logs automatic and it only show trade offers.
On Sat, 3 Aug 2019 at 22:37, Colin Saari<EMAIL_ADDRESS>wrote:

yes
On Sat, 3 Aug 2019 at 15:31, ZeusJunior<EMAIL_ADDRESS>wrote:

Are you 100% sure its not in the logs
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Nicklason/tf2-automatic/issues/135?email_source=notifications&email_token=AMLJC2EVA232GYLHH3J7NI3QCU66TA5CNFSM4IJCDNU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3PJ7LQ#issuecomment-517906350,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMLJC2FEHF5QTS7OSXFFEZDQCU66TANCNFSM4IJCDNUQ
.

pm2 log shows error logs seperately. Scroll up and you'll find them

Closing because inactive. If you still need help then reopen the issue.


feat(pkg): Implement package pushing with OCI

Prerequisite checklist

[x] Read the contribution guidelines regarding submitting new changes to the project;
[x] Tested your changes against relevant architectures and platforms;
[x] Ran make fmt on your commit series before opening this PR;
[ ] Updated relevant documentation.

Description of changes

This PR series introduces generic package pushing with initial support for OCI registries.
Pushing to Private repositories is possible and it also enables pulling from private repositories.
Closes: #456

Starting the review, can see that a rebase is necessary also

Pushing a non-existent image results in a nil-pointer exception:
panic: runtime error: index out of range [0] with length 0

goroutine 130 [running]:
kraftkit.sh/cmd/kraft/pkg/push.(*Push).Run.func1({0x21bf5e8, 0xc00020ad80}, 0xc00066d440)
        /go/src/kraftkit.sh/cmd/kraft/pkg/push/push.go:131 +0xf8
kraftkit.sh/tui/paraprogress.(*Process).Start.func2()
        /go/src/kraftkit.sh/tui/paraprogress/process.go:138 +0x21c
github.com/charmbracelet/bubbletea.(*Program).handleCommands.func1.1()
        /go/pkg/mod/github.com/charmbracelet/bubbletea@v0.23.3-0.20230316100943-248eb83001a7/tea.go:248 +0x38
created by github.com/charmbracelet/bubbletea.(*Program).handleCommands.func1
        /go/pkg/mod/github.com/charmbracelet/bubbletea@v0.23.3-0.20230316100943-248eb83001a7/tea.go:247 +0x14d

Can you squash the last 2 commits together? The dependencies are part of this commit


YARA C API scanning running processes OS X

Hi. This is more of a question, then an issue.
Im trying to run yr_scanner_scan_proc on all running processes, but after the first PID, my app exits with:  cant not attach to process (try running as root).
Do I need to spawn a new thread, for every pid? IE. Create a new scanner for each proc?
Thanks in advance.

No, you don't need to create a separate thread for scanning each process, you can scan one process at a time with the same scanner on a single thread.
Does this happens always with the second process independently of which process it is? Or the second process is always the same one?
As the error suggests the problem is that couldn't attach to the process for some reason, maybe because it's a system protected process or some reason.

Thanks for the second set of eyes:)
Made me realize that I exit the program, when an error occurs.
Total brain fart on my behalf.
Thanks


Source for frontend

I have sent an email so I hope I don't overstep by posting an issue too (just in case my email went to spam or something).
As you know in Flow + Blocto ecosystem only some NFTs are displayed. I came across Bitku and this would be a perfect temporary solution to allow NFT users to add an empty collection to their wallet, receive and view an NFT on the site.
I would be very interested in having the source of the frontend to be able to build something similar (it is for testing and in the future internal, non commercial purposes). This would allow my NFT (built with an API that has a contract on Flow mainnet) to be received by other wallets and seen on the site.
I have looked literally everywhere and tried Discord Flow and Blocto servers, even Upwork, but there is nothing as comprehensive as your frontend so I dare to ask.
Thanks in advance.

I found it!


Refactor most of CMakeLists.txt to use modern CMake approach

Hi!
This is meant to be more of a WIP/discussion. I quite like libuv, but its CMake style leaves a lot to be desired. In an ideal world, the minimum CMake version would be bumped to 3.4, but that's a discussion for another time.
Fixes

CMake will not longer look for a C++ compiler when building the project on its own
Project is now safe to be used in an add_subdirectory way.
CTest is included by default, which calls enable_testing and adds the BUILD_TESTING option, but unit tests are locked behind a UV_BUILD_TESTS option that is enabled if BUILD_TESTING is true and is libuv is the root project. This can also be force enabled from the command line, but these defaults fix issues when its included into larger projects.
List of test files is only constructed if unit tests are enabled. This might seem like a micro-optimization, but in a large tree of projects it can affect CMake's performance and memory usage.

New Features

A common 'interface' library for sharing specific settings between the shared and static libraries
Aliases for uv and uv_a to ::shared and ::static. These are available if its used as a subdirectory. Some more work is needed to make sure these are "exported" in a given build
Generator expressions everywhere. These are more efficient as it puts most of the logic that occurs at configure time (which is single threaded) to generation time (which is multithreaded, and can error with more information as needed). They can be a bit confusing, but I think the examples shown here should handle the minimum.

Misc
Please let me know if you'd like changes or additional work done. Some suggestions I'd make:

Add a uv::uv target based on whether BUILD_SHARED_LIBS is true. Alternatively rely on the builtin BUILD_SHARED_LIBS command so that one can only build the static or shared library.
Upgrade minimum CMake version to something very recent, such as 3.4 or even just 3.8.
Add CPack support (This is something I can easily put in as well)
Create uv-config.cmake files for installation.

I'm somewhat ambivalent about this PR. Some of the changes might be improvements but it would've been better it they'd been split into separate commits or pull requests for easier reviewing and discussion.
A change-everything-style pull request like this one also conflicts with every other PR that's currently open and touches CMakeLists.txt.

Fair enough. I'm more than willing to work with any authors of open PRs regarding CMake to get their fixes in, though I did try to incorporate a few, while others are resolved simply by moving to the above.
The massive diff is mostly from moving the setting of test sources to the bottom of the file for performance reasons. These do turn into very hefty allocations when someone isn't building libuv for tests but rather to do an install, and cause very big performance problems in large projects (I've got a small closed source test harness I use for my changes to make sure a CMake project behaves "nicely", and when libuv was attached it added 5 seconds of configure time to a total of 30 seconds. Removing it dropped it back to 22-25 seconds when not building unit tests.

@bnoordhuis I'm going to close this for now and break these up into smaller bite sized PRs. I'd really like to get these changes merged in for usability, speed, and performance reasons. Would it make sense to open an issue for general discussion on minimum CMake needs? I'd love to at least get the CMake minimum required version bumped to 3.4 if possible, as that would make my life easier in submitting smaller patches.


--target updates and RPGM maths

Additional and updated Effects to support latest RPGMaster features
--target multi mode.
--target saving throws.
--target-save and --target-nosave.
--nosave to flag if --target-nosave actually = --target.
Allow RPGM maths for numeric values such as "duration" and "direction" as well as save mods.
Addition of '#' to RPGM maths to represent number of creatures targeted with a --target multi.
Fix support of ^^duration^^ tag in effects.

This pull request must not be merged without Pull Request #1943 also being merged


Add a GPU support matrix in documentation

Request description
Add a GPU support matrix table so that independent software vendors know what is expected to work.
What component(s) does this issue relate to?
Runtime
Additional context
About gpu compatibility with AMD and IREE?
Was curious about https://www.techpowerup.com/gpu-specs/radeon-rx-7900-xtx.c3941 which has 24GB and https://www.amd.com/en/products/graphics/workstations/radeon-pro/w7900.html which has 48GB of video ram.
and.. uh the steamdeck AMD gpu.
Scott Todd — Today at 13:51
What are you curious about specifically? General support for the hardware?
Jakub Kuderski — Today at 14:07
We support both 7900xtx and w7900.
Steam deck is based on rdan2 and should work, but there may be some sharp edges if nobody tried it before
Ben Vanik — Today at 14:12
I tried it ages ago, no clue if it still works though (both CPU and Vulkan)
last year around this time, apparently :P vulkan
iFire — Today at 14:14
Will you claim LLMs run on the steamdeck 😄
Like in a table.
Ben Vanik — Today at 14:15
I claim nothing. YMMV :P
iFire — Today at 14:18
So both AMD 7900xtx and w7900 will have IREE organization support. That's great!
Scott Todd — Today at 14:20
We have CI workflows running unit tests and some model tests/benchmarks on w7900 GPUs using both the ROCm/HIP and Vulkan APIs. That's mostly what we had on hand in sufficient supply for automated testing, but we'd like to expand the test device matrix eventually.
Some performance optimization work is shared across all backends or at least across CDNA and RDNA (via the LLVMGPU / ROCm / HIP code generation pipelines).
iFire — Today at 14:22
Both windows and linux right?
Scott Todd — Today at 14:26
Yep. Related to that, we're evaluating HSA as a backend option for AMD hardware that has many nice features for the runtime to use, like on https://github.com/iree-org/iree/pull/18349, but that (I think?) is currently Linux-only. Definitely pushing to keep the core project fully supported on all operating systems.
Hmm that reminds me, I have a dual boot machine with some w7900s in it now. I should run through our docs etc. on Windows and check that things work out of the box.

Agreed we can do more here.
Current relevant docs:

https://iree.dev/#support-matrix (high level, no specifics)
https://iree.dev/guides/deployment-configurations/gpu-vulkan/#support-matrix
https://iree.dev/guides/deployment-configurations/gpu-vulkan/#compile-a-program (commonly used architectures)
https://iree.dev/guides/deployment-configurations/gpu-cuda/#compile-a-program (commonly used architectures)
https://iree.dev/guides/deployment-configurations/gpu-rocm/#compile-a-program (commonly used architectures)

There's a bug report about AMD 7900XTX card failure. https://github.com/nod-ai/SHARK/issues/2165


⚠ ClamAV has degraded performance

In a787139, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 6e47656.


Add RabbitMQ High Availability connection support

This is an attempt to add basic support for RabbitMQ High Availability (see #13).
Limitations / TODOs for future PRs:

Prefetch values are currently ignored
No SSL support
Documentation enhances

@mihaitodor Feel free to update the update the deps, and we will be able to merge this PR then.

@AlexisMontagne Sorry for the long delay. I finally managed to look into this and updated the dependencies as you requested.
However, I'm not 100% sure it went well, because I had a lot of trouble figuring out which version of Godeps you are using in combination with Go 1.4... The latest version fails to update the 3 dependencies from sensu-go, so I tried to use an older version and to re-create the whole Godeps folder from scratch via godep save, but that messed up the imports completely. I ended up hacking it, so I hope I got it right.

LGTM


Optional components

Additional section in the components guide should be considered, as people have ordered all the parts from this page and not noticed the optional pieces on the schematics. Suggested formatting below:
Optional components:
There are several components that can be added when building a tracker that allow for additional features. A 'Battery sense' resistor can be added to allow the server to determine battery level, while a pair of diodes can be added to allow the tracker to be used while charging and as an added safety measure against accidental damage due to charging while powered on. While both these additions are extremely low cost, they are considered optional and can increase the complexity of the schematics.
Recommended resistor is: 180k Ohm 1/4W Metal Film Resistor
Recommended diodes are: 1N5817 SCHOTTKY Diode

It seems this has since already been implemented into the Docs.
Going to close this issue unless further changes have to be made.


Email HELOs with machine short name and not FQDN

Please see "<IP_ADDRESS>  Extended HELLO (EHLO) or HELLO (HELO)" in https://www.ietf.org/rfc/rfc2821.txt
Many SMTP daemons drop connections for short named HELO/EHLO.

uhhhh sure...what exactly is the problem?

vCheck-vSphere has the option to send an email with the results.  When it sets up the email transfer, the first stage is to identify itself to the other end (ie the SMTP server).  You can try it yourself, here I am connecting to a GMail server:
$ telnet gmail-smtp-in.l.google.com. 25
Trying <IP_ADDRESS>...
Connected to gmail-smtp-in.l.google.com.
Escape character is '^]'.
220 mx.google.com ESMTP n190si27562671wmg.90 - gsmtp
helo me
250 mx.google.com at your service
I typed in the line "helo me" where I claim to be a system called "me".  Many mail systems do not allow a short helo and will drop the connection, considering it as likely spam.  What I really should have put was "helo me.example.co.uk".  In this case GMail is not that fussy but many are.
All that needs changing is to get the part of vCheck-vSphere that determines the hostname of the system it is on is to also gather it's fully qualified (DNS) domain name and use that in the emailing routine.

Not that easy I'm afraid. This is actually a limitation of the SmtpClient class of the .net framework, which we obviously have no control over.
Look like support for this was added, but depends on .net version as the property was renamed at some point. Needs further investigation...
https://msdn.microsoft.com/en-us/library/system.net.configuration.smtpnetworkelement.clientdomain(v=vs.110).aspx
On 10 May 2016 10:07 PM, Jon Gerdes<EMAIL_ADDRESS>wrote:vCheck-vSphere has the option to send an email with the results.  When it sets up the email transfer, the first stage is to identify itself to the other end (ie the SMTP server).  You can try it yourself, here I am connecting to a GMail server:
$ telnet gmail-smtp-in.l.google.com. 25
Trying <IP_ADDRESS>...
Connected to gmail-smtp-in.l.google.com.
Escape character is '^]'.
220 mx.google.com ESMTP n190si27562671wmg.90 - gsmtp
helo me
250 mx.google.com at your service
I typed in the line "helo me" where I claim to be a system called "me".  Many mail systems do not allow a short helo and will drop the connection, considering it as likely spam.  What I really should have put was "helo me.example.co.uk".  In this case GMail is not that fussy but many are.
All that needs changing is to get the part of vCheck-vSphere that determines the hostname of the system it is on is to also gather it's fully qualified (DNS) domain name and use that in the emailing routine.
—You are receiving this because you commented.Reply to this email directly or view it on GitHub


Add logic to refine research output options

We should be able to reevaluate the metadata we receive that feeds the repository, license and metadata standard lists.
For example:

we might be able to make use of the selected ‘research domain’ (e.g. Biological sciences), the selected ‘research output type’ (e.g. Software) to filter the repositories list by tags.
We might be able to make use of the DMP owner's organizational affiliation to highlight institutional repositories.
Based on the deleted repository(ies), we should then be able to refine the license and metadata standard options

For an example of re3data metadata see: https://www.re3data.org/repository/r3d100012384
For example, the CaltechDATA repository could be ‘preferred’ for researchers from the California Institute of Technology; the Ecosounds could be ‘recommended’ if the selected research domain is ‘Biological Sciences’ and the output type is ‘Audiovisual’

We should port this ticket over to the new system.


No 'children' property on PIXI.DisplayObject

Hello,
using latest 4.0.3 release in a webpack setup (appended to bundle as is from vendor folder).
I noticed that at some point PIXI removed child display objects from PIXI.DisplayObject.
However after passing a single DisplayObject to renderer.render(...) function am getting a TypeError from PIXI when it attempts to to access a null 'children' property.
Rendering PIXI.Container works as expected, as it should I guess.

is it assumed that DisplayObjects are not passed to renderer directly and only Containers used at root level?

it can be removed in destroy(). Dont destroy parent in a event, i suppose :)

I don't think we ever "removed" the children property from DisplayObejct. Container has children, but DisplayObject does not (and never has).
This error is probably because interactiveTarget has a interactiveChildren property that defaults to true. When mixed into DisplayObject this means that it will hit this check. This is just a bug, we need to do one of the following:

Have interactiveChildren default to false in DisplayObject constructor, OR
Add a check to ensure children exist in processInteractive. if (interactiveChildren) -> if (interactiveChildren && children)

#3132 done.
Yep, you are right Chad, I thought I used a DisplayObject in my probe project a year ago but that in fact was a Container as I just double checked :)
And only now I realized that documentation specifically mentions not using abstract DisplayObject (maybe renaming it into AbstractDisplayObject would be a good idea in the future too, but thats my ocd talking)
Nevertheless, here is a pull request with a fix you suggested.
Thanks for quick tips.


about-me3: Add Maker and Model

Is your feature request related to a problem? Please describe.
I wanted to quickly check the model of a raspberry pi and could not via the "About me" service.
Describe the solution you'd like
The command cat /sys/firmware/devicetree/base/model provides

Raspberry Pi 3 Model B Plus Rev 1.3

and could be incorporated into the about-me output.
On PC there exists the command # dmidecode | grep -A3 '^System Information, for which I get:

System Information
Manufacturer: Hewlett-Packard
Product Name: HP EliteDesk 800 G1 SFF
Version: Not Specified

Not sure if there exists an abstraction that would provide the information for all Linux devices and not just for raspberry pi.
Alternatives
No response
Additional context
No response

Please try.


Enable Compute + CCL Data Movement Overlap (Fuse All Gather + Matmul)

High level issue to track the initial path-finding/clearing effort to build an all-gather + matmul fused operation that overlaps compute + data movement.

[ ] #10416

so the matmul can get full chip input chunks at a time

[ ] Support all-gather targeting sub-region of worker grid with offset
[ ] Add new test test mode for fusable-all-gather (all-gather portion)

All gather fuses with datacopy op.
Datacopy op expects 1/ring_size input each timestep. Compare output tensor for datacopy to ensure it alternates ring_index inputs

[ ] Add new test test mode for fusable-all-gather (Matmul portion)

matmul fuses with datacopy op
datacopy op reads input tensor in chunk order that will match all-gather (alternating chip indices)
These are the activations fed in to matmul - matmul must update weight read order accordingly and produce correct output

Adding notes from a duplicate issue that @xuncaiTT  opened
At each t, matmul can be performed on the recieved chunks on idle cores, which can significantly improve utilization and latency of device. Note that the order does not matter as long as the output accumulates. See below proposed scheme:

A pseudo code is shown below:

FYI @cglagovich @xuncaiTT @avoraTT

@xuncaiTT I assume you are aware of this, but I didn't see it mentioned so I wanted to say it explicitly that the chunk schedule will be unique per chip. We will also need to coordinate a little because there is a bit of coupling between the CCL and compute because you would prefer to start computing the first chunk from a specific direction based on how the all-gather works because one direction will end up sending more chunks than the other by one.

We can close this now, right?


NOt works with nextjs

im impliment it with next js but it didn't works, no media display

please fix for nextjs

Could either of you provide reproducible testing steps for this issue? Right now we have nothing to run off of so there is almost no chance of this getting fixed as-is.
P.S. We are always taking PRs!

In case this is helpful to anyone, I noticed that with Next.js router changes from one url to another (i.e. different router.asPath values) but which use the same dynamic path (i.e. router.pathname), the scripts aren't executed.
I added a key prop with a unique identifier to the component, which forces React to re-mount it when there's a change. Now it works perfectly fine.
Example:
<EmbedContainer key={page?.id} markup={page?.content}>

</EmbedContainer>

Thank you to the author for a very useful component.


Scarlet Crusade Bishop's Sanctify ability

If the Bishop uses the Sanctify ability on itself , the effect becomes permanent

Duplicate of #2842 so I'll re-open that one (as it indeed wasn't fixed) and close this one.


Working tested backend added

Fixes Issue
Closes #1998

Changes proposed

Screenshots

Note to reviewers

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Updated API routing to include versioning for improved management and future updates.

Improvements

Enhanced consistency in the VigyForm component's code for better readability and maintainability.
Streamlined handling of form inputs and submission logic in the VigyForm.

Bug Fixes

Modified OTP request endpoint in the Admin component to point to a local server for development purposes.

@codervivek5 please see current resolved commit


use streaming response to output llm results

Related to this question in #30
Enable the possibility to use the  langchain4j's StreamingChatLanguageModel  instead of ChatLanguageModel

Hi, is there any progress on this issue? Or, if a non-serializable object needs to be injected into a node in Worflows, such as WebSocket, SseEmitter, etc., which are all non-serializable, but the content needs to be pushed to the outside in real time during the execution of the node, is there any other solution?

Hi, is there any progress on this issue? Or, if a non-serializable object needs to be injected into a node in Worflows, such as WebSocket, SseEmitter, etc., which are all non-serializable, but the content needs to be pushed to the outside in real time during the execution of the node, is there any other solution?

Currently  to implement such feature we have to consider that :

NodeAction should be able to return an  AsyncIterator.
NodeAction should accept another argument (ie RunnableConfig) to decide if or not use streaming as result.
connects the AsyncStream to the LLM streaming result, that in case of langchain4j is StreamingResponseHandler.
propagate the node streaming result over graph streaming execution

It is not so simple and, potentially, could break backward compatibility so I'm evaluating the best strategy before start implementation

@bsorrentino Thanks, looking forward to this feature

up

HI @TyCoding, @sharfy The LLM streaming support for langchain4j has been added to last 1.0-SNAPSHOT release
A reference implementation is here. Please take the opportunity to try it and let me know, in the meanwhile I'll document the new feature


Renderer2D can't render more than Renderer2DData::MaxTextureSlots different textures

When trying to render more than Renderer2DData::MaxTextureSlots different textures using Hazel's Renderer2D the TextureSlots array overflows. This only occurs when you render these textures in one batch.
Probably the solution to this issue is to check if all texture slots are already used when submitting a new textured quad. If that is the case the renderer should flush and start a new batch via the Renderer2D::FlushAndReset()method. The mentioned check has to be implemented in the two methods that actually submit a textured quad (normal and rotated). Maybe just replace
if (s_Data.QuadIndexCount >= Renderer2DData::MaxIndices)
    FlushAndReset();

with
if (s_Data.QuadIndexCount >= Renderer2DData::MaxIndices || s_Data.TextureSlotIndex >= Renderer2DData::MaxTextureSlots)
    FlushAndReset();

This should make sure that the array doesn't overflow and also that all textures are rendered correctly.

That would not be the best idea to implement it:

Assume you have already 31 textures rendering (including the flat color one).
You request a new quad with new texture, so it will fail your if statement, not flushing, adding the 32nd texture and render that quad.
You request a new quad with an existing texture, it will pass your if statement and flush already, while it could still add this quad to the existing batch.

Instead, we should check the number of texture slots when it tries to add a new texture instead. So instead of adding it where you said, adding it to the place where the new texture is added to the batch:
		float textureIndex = 0.0f;
		for (uint32_t i = 1; i < s_Data.TextureSlotIndex; i++)
		{
			if (*s_Data.TextureSlots[i].get() == *texture.get())
			{
				textureIndex = (float)i;
				break;
			}
		}

		if (textureIndex == 0.0f)
		{
			if (s_Data.TextureSlotIndex >= Renderer2DData::MaxTextureSlots)
				FlushAndReset();

			textureIndex = (float)s_Data.TextureSlotIndex;
			s_Data.TextureSlots[s_Data.TextureSlotIndex] = texture;
			s_Data.TextureSlotIndex++;
		}

Oh, your right, apparently I didn't really think this through. Your solution seems pretty good.


Need UI adjustments and beautification for the refresh button

Please use material design and icons to make the refresh button look appealing

I would like to work on this. Please assign me.

Sure, please raise a PR after you are done working, I will review

You haven't assigned me though :) Is it alright?

This works?

Please review my PR. Let me know any changes.

Please resolve conflicts

This issue has been closed now


cli: install dependencies of experiment

we could leverage poldek or something alike that is
platform-agnostic

see #37


how to Hyperparameters of the AdamW optimizer? How to use global vectors?

when I train the ICAR-ENSO dataset,I found the hyperparameter settings of AdamW in the paper are different from the parameter settings in the earthformer_enso_v1.yaml file, so which one should be used？If I want to use global vector, should I modify those parameters？ Thank you.

Thank you for bringing up this issue. Your observation is correct regarding the default config on ICAR-ENSO, which slightly differs from that of SEVIR, N-body MNIST, and Moving MNIST. We have noticed that using lr=1e-3 and num_global_vectors=8 for training is rather unstable. Therefore, we have released a more stable config for better reproducibility.

Thank you.May I ask, when num_global_vectors=0 in earthformer_enso_v1.yaml, the global variable is not applied?The corresponding indicators should be:
，correct？How to use global vector to train ENSO dataset,I can get the second row of results？

Thank you for your question. Yes, num_global_vectors=0 indicates not using global vectors.
The training of Earthformer with global vectors (num_global_vectors=8) on ICAR-ENSO is rather unstable. To help alleviate this, I would recommend initializing some parameters to zero, similar to what we did in our PreDiff that used Earthformer.

Thank you.


[WIP] Improve OSM XML file saving

Targeting the forthcoming OSMnx v2.0.0 release (see also #1106), this PR wholly refactors the _osm_xml module and fixes several bugs, significantly speeds up the save_graph_xml function, and streamlines the  function's signature.
The PR does the following:

[x] Speed up the save_graph_xml function runtime by a factor of 5-10x depending on graph size.
[x] Fix a bug where edge merging grouped edges into ways on the "id" column, but this column was not the way OSM ID, so each group just contained a single edge.
[x] Fix a bug where the old topological sort only worked when the entire way was a single cycle (such as is the case with a roundabout way), but could not sort other ways merely containing a cycle (or multiple cycles, such as here and here) within them. See also #970.
[ ] Add missing "bounds" sub-element to the XML file (as seen in this example).
[x] Remove the node_tags, node_attrs, edge_tags, edge_attrs, api_version, and precision function parameters in favor of using the settings module directly for the first 4 and hard-coded sensible defaults for the last 2. See also #917 and #961
[x] Change the save_graph_xml function's data parameter to G and its argument type to nx.MultiDiGraph instead of nx.MultiDiGraph | tuple[gpd.GeoDataFrame, gpd.GeoDataFrame (that is, the function no longer accepts a tuple of GeoDataFrames: you can only pass a MultiDiGraph).
[x] Change the save_graph_xml function's edge_tag_aggs parameter to way_tags_agg and its argument type to dict instead of tuple.
[ ] Create an XSD file to validate the XML output against (maybe?)
[x] Require that the graph be unsimplified when saving with merge_edges=True, because ways have a one-to-many cardinality with edges but graph simplification makes the cardinality many-to-many instead, which breaks the grouping of edges into ways.

To-do separately, in the v1 branch: deprecate these now-removed function params with FutureWarning. We may deprecate a couple more... see questions below.
A couple questions... perhaps @mxndrwgrdnr could provide some answers.

Do we really need the save_graph_xml function's oneway parameter? What is the use case for this ever being equal to True? I propose we remove this parameter and instead just hard-code False in the one line it's used.
Do we really need the save_graph_xml function's merge_edges parameter? Is it ever valid for it to be False, and therefore to generate 1 XML way (with fake way ID) for each edge in the graph? I don't see how that could be valid... I propose we remove this parameter and instead always operate as though merge_edges=True.

As a tangential aside, it would be nice someday to replace all the OSMnx XML read/write functionality with PBF read/write functionality instead and outsource it to a dedicated third-party library. See also https://github.com/HTenkanen/pyrosm/issues/228.

@gboeing I think its safe (and advisable) to hardcode oneway=False, and to operate as if merge_edges=True. Both settings were parameterized to maximize flexibility but they're probably not necessary. You might consider just defining a ONEWAY = False constant at the top of the module and using that within _save_graph_xml()  rather than hardcoding the value into the method, just to make the assumption more explicit.

Thanks @stl-maxgardner . Did this change above also make sense:

Require that the graph be unsimplified when saving with merge_edges=True, because ways have a one-to-many cardinality with edges but graph simplification makes the cardinality many-to-many instead, which breaks the grouping of edges into ways.

If we make the function always operate as though merge_edges=True, users moving forward will be unable to save a simplified graph as XML. But I believe this would be appropriate given the nature of the data.

Yes I think so!

@stl-maxgardner one more question for you.
In 54170cd I proposed that we remove the settings module's osm_xml_node_attrs, osm_xml_node_tags, osm_xml_way_attrs, and osm_xml_way_tags settings. The reasoning is as follows...
The osm_xml_node_attrs and osm_xml_way_attrs settings are unnecessary because the  node and way XML elements should have a consistent set of attributes as defined in the OSM XML spec. That is, the same attributes are needed every time so this shouldn't be configurable. These attributes are "changeset", "timestamp", "uid", "user", "version", "visible" and "id" for both nodes/ways, plus "lat" and "lon" for nodes.
The osm_xml_node_tags and osm_xml_way_tags settings are unnecessary because they are redundant with the settings modules useful_tags_node and useful_tags_way settings. That is, the latter are used to retain desired tags from the raw Overpass data and lets the user specify what data the graph nodes and edges should have. OSMnx should just save to XML the graph that the user has created, rather than offering another set of settings for changing what gets saved. If the user wants to only save a subset of these tags, they can just change the useful_tags_node and useful_tags_way settings, or just edit their graph with 1-2 lines of code prior to calling the save_graph_xml function.
Does that all make sense and seem reasonable to you?

Some timings comparing this xml branch to the main branch with a small graph and a large graph... Given a code snippet like:
import osmnx as ox
ox.settings.all_oneway = True
G = ox.graph.graph_from_place(place, network_type="drive", simplify=False)

%%timeit
ox.io.save_graph_xml(G, filepath="./test.xml")

The %%timeit magic gives the following timings:
# place = "Piedmont, CA, USA"
# main branch: 1100 ms ± 4.94 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
# xml branch:   167 ms ± 411 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)
# ~6.6x speed-up

# place = "Los Angeles, CA, USA"
# main branch: 113.0 s ± 790 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
# xml branch:   26.2 s ± 82.1 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
# ~4.3x speed-up

An XSD schema file was added in 06edb3d and successfully validates a variety of test OSM XML files saved by OSMnx, exported from Overpass turbo, and exported from OpenStreetMap directly.

Added quality-of-life improvements, warning, and additional tests for saving projected/consolidated graphs as OSM XML in ca9744b.

The first pre-release OSMnx v2 beta has been released. Testers needed! See #1123 for details.


Trying to merge multiple sheets into one.

Hello,
I'm trying to parse through a giant string that has multiple tables in it.
The string might look something like: '

Table One

Table Two

'. This is a very simple explanation, but I can elaborate if I need to.
My question is, how can I retain the merged cells in each of these table headers/footers? I know the above example string doesn't have headers or footers...
I have tried doing something like the following:

Break the string into multiple strings representing each individual table
Use XLSX.read(tableString, {type: 'string'}) for each table string, then run each through sheet_to_json like the following:

tableStringArray = tableStringArray.concat(XLSX.utils.sheet_to_json(XLSX.read(tableString, {type: 'string'}).Sheets['Sheet1'], {header: 1});
When I'm done building this array, I can run the following:
let newSheet = XLSX.utils.json_to_sheet(tableStringArray, { skipHeader: true });
let newWrkBook = XLSX.utils.book_new();
XLSX.utils.book_append_sheet(newWrkBook, newSheet, 'Report Data');
XLSX.writeFile(newWrkBook, reportName + '.xlsx');
This is all well and dandy and allows me to actually combine my tables into one spreadsheet, but sheet_to_json strips out the merged cells!!! We can't have that happen.
I know I can simply write just one of the tables say tableStringArray[0],
XLSX.write(tableStringArray[0], { bookType: 'html', type: 'array' });
and that will come out just fine in excel. However, it seems that there isn't a way to combine multiple tables into a single write method.
XLSX.write(tableStringArray, { bookType: 'html', type: 'array' }); ????????
Is there another approach to getting the merged cells to actually show up in the final excel file? I have poured through the docs and cannot find anything about what I'm trying to solve. Thank you.

create an HTML DIV, set the innerText to your HTML string, and use sheet_add_dom with the option origin: -1 to write the table below existing data in a worksheet.  That can be used multiple times to add multiple tables.
If you want the tables in separate worksheets, use table_to_sheet to generate each sheet and book_append_sheet to add them to a shared workbook


Fix utility page code example Markdown

Current issue: On the v4 Utility classes page, the code isn't being rendered properly with Markdown.
Screenshot:

What this PR does: Adds an extra blank line above the code block so that Markdown recognizes the code block separately.
I know it's tiny, but still :smiley:

Rad. Thanks @mdo!

Derp, 9a35ccb properly fixes this.

Redeployed docs to fix that. Thanks for bringing it to attention!

🙌


Change URLs for MR data

Both:
https://github.com/CCPPETMR/SIRF-Exercises/blob/2e52fb8071bb401bdb113a6b542474d9e717b091/scripts/download_MR_data.sh#L49
and
https://github.com/CCPPETMR/SIRF-Exercises/blob/2e52fb8071bb401bdb113a6b542474d9e717b091/scripts/download_MR_data.sh#L57
should be changed to not rely on DropBox.

What seems like throttling from Dropbox slows/stops machine builds for Azure.

closing as this is now only for the "old" notebooks which we're normally not using


Could not find 'X'. The program may not be installed. on Windows 10

Description
When open atom with right click menu, it's always get "Could not find 'X'. The program may not be installed. ", But when open atom with shortcut it's work find.
Debug
Here is a link to the debug.md Gist:
https://gist.github.com/anonymous/27d26ff481383bae07f7971a1bcc2f97

This appears to be a duplicate of #1052

I really want to focus on improving the installation experience for users. I have created a new Issue, #1687, to target this problem. Please provide your feedback! Thanks in advance.


Files using != None syntax rather than is Not None

Not a game-breaking issue, just a formatting issue with the syntax

Also adding general refactoring


[WIP]Re-introduce support of TLS 1.3

This is to confirm how to introduce support of TLS 1.3

Closing in favor of #293


Wireless Charger forcing chunks to stay loaded when powered

When a Wireless Charger has power in its buffer, it forces its chunk to always stay loaded. This is regardless of any chargable machinery or items in its vicinity, or if a redstone signal turns off the wireless charger.
This causes surrounding chunks to load by other means such as conduits and machinery from other mods, which then become unloaded during auto-save and immediately loaded again by the Wireless Charger keeping the chunk alive.
Details
Discovered on

Direwolf20 1.7.10 1.5.0 and 1.6.1 live server
Forge <IP_ADDRESS>8
EnderIO <IP_ADDRESS>1

Reproduction
Testing environment
Edit: Re-tested without Thermal suite

Forge 1.7.10 <IP_ADDRESS>8
EnderIO <IP_ADDRESS>3_beta
EnderCore <IP_ADDRESS>_alpha
ChickenChunks <IP_ADDRESS> & CodeChickenCore <IP_ADDRESS>

Steps

Create a flatland creative world with cheats enabled
Execute /tp 0 8 0 and then /setworldspawn (keeps spawn chunks away from testing areas)
Execute /tp 1000 8 1000
Place an EnderIO wireless charger on the ground, with a Creative Capacitor Bank next to it
Execute /tp 0 8 0
Press ESC to open the main menu and force a chunk purge
Resume game and execute /chunkloaders. A new window will open under Minecraft
Observe that chunk with the Wireless Charger is kept loaded

Workarounds
Wireless Chargers only chunkload if they have power in their buffer. Therefore, drain the charger of any stored energy to prevent it from chunk loading. Note that applying a redstone signal to disable the charger has no effect.
Notes
I discovered this bug as part of a long running investigation as to why chunks are mysteriously loading and unloading at random on my server. It has since been found that other mods and bugs are contributing to this problem; this one in particular.
To determine the existence of this bug, I took an offline copy of an affected base and tore it down machine by machine (e.g. reactor, turbine, EnderIO conduits, etc) and teleported in and out, until I was able to unload the chunk.
Because of time constraints and lack of experience with the Ender IO codebase, I am unable to offer specific technical advice or pull requests. However, if it helps, please see this similar bug report I had filed for AE2:

Multi-block ME controllers forcing chunks to stay loaded #1656
Fix by yueh for the above bug

First of all - you don't need anything from CoFH at all. EnderIO has creative capacitor banks.
I'll take a look at the wireless charger.

You're right; my bad. I have re-tested without CoFH mods and simplified the above reproduction.
Thanks for looking into this!

Looks like the redstone checks keeps the chunk loaded:
https://github.com/SleepyTrousers/EnderIO/blob/master/src/main/java/crazypants/enderio/machine/wireless/TileWirelessCharger.java#L161

That code change did nothing, blockExists is already checked by isPoweredRedstone

However, the relevant code change in EnderCore may have fixed this issue


Should use the GitRepository.onDidChangeStatuses event to be notified of changes instead of polling

In GitTabStatus._setupWatchConditions() you start a polling process to execute every half-second to check for updates. It would be, in my opinion, better if you used the GitRepository.onDidChangeStatuses event to be notified when anything changes. This way you aren't polling for no changes, which is most likely the common case. Additionally, the tabs' status should be updated at the same time as the tree view and gutter information ... creating a more unified experience :grinning:
Ref atom/atom#6561

I agree with you. The interval is a temporary solution. This project previously has used events to determine status, however, those events have historically not been reliable enough in previous betas and caused odd results.. I'll take a look again and see if it has improved.
Jake
Sent from my iPhone

On Apr 29, 2015, at 9:46 AM, Lee Dohm<EMAIL_ADDRESS>wrote:
In GitTabStatus._setupWatchConditions() you start a polling process to execute every half-second to check for updates. It would be, in my opinion, better if you used the GitRepository.onDidChangeStatuses event to be notified when anything changes. This way you aren't polling for no changes, which is most likely the common case. Additionally, the tabs' status should be updated at the same time as the tree view and gutter information ... creating a more unified experience
—
Reply to this email directly or view it on GitHub.


Error when install.sh is executed

When I run install.sh as the tutorial recommended, but following errors rises.
Installing Lua version: LUAJIT21
CMake Error at exe/luajit-rocks/luajit-2.1/CMakeLists.txt:156 (MESSAGE):
Cannot compile a useful lua.
Function dlopen() seems not to be supported on your platform.
Apparently you are not on a Windows platform as well.
So lua has no way to deal with shared libraries!
And I can't find solution from googling, anyone can help please?
SYSTEM: Ubuntu 14.04 LTS server 64bit

./clean.sh
./install.sh
that should fix it.

@soumith It does not fix it . The commands did nothing but repeat the warning.

Actually in my case @soumith 's answer fixed my problem.


[CockroachDB] No difference between integer and bigInteger

Environment
Knex version: 0.95.15+
Database + version: CockroachDB v21.1.13
OS: macOS Monterrey
Bug
Both integer and bigInteger both result in a int8 column being created in CockroachDB, whereas Postgres creates them as int4 and int8 respectively. This becomes an issue, as int8s are always returned as a String by the pg connection library, causing unexpected incompatibilities between the database vendors.
knex.schema.createTable('example', (table) => {
  table.increments();

  table.integer('example_int');
  table.bigInteger('example_bigint');
});

Note how both columns are created as an int8 in CockroachDB.
I believe this is due to the fact that CockroachDB defaults to int8 when using .. ADD COLUMN example INTEGER, which is the default from here (AFAICT as the PG dialect doesn't have a specific integer in the columncompiler).

Would you consider sending a fix for this? Should be fairly straightforward to override this for Cockroach

Sure thing!

Are there any current tests I can enhance to check against the type creation @kibertoad @OlivierCavadenti? I noticed the schema specs under integration2 here:
https://github.com/knex/knex/blob/master/test/integration2/schema/index.spec.js
but I wasn't able to spot any test that confirms that a .integer() actually creates an integer.
Alternatively, we might also be able to catch this by adding some check against integer fields created by knex under integration2/query/select 🤔

@rijkvanzanten I don't think we have tests to validate integers being integers. Probably the closes thing would be unit tests which validate SQL being generated

@kibertoad I added a proof of concept "type consistency" spec file to the integration2 tests. Is that useful to have, or a bit overkill? (https://github.com/knex/knex/blob/9ea1ae98e3d8f4ce38a10971e25768b68105b8d0/test/integration2/schema/type-consistency.spec.js)

@rijkvanzanten It looks great!

@kibertoad I added a proof of concept "type consistency" spec file to the integration2 tests. Is that useful to have, or a bit overkill? (https://github.com/knex/knex/blob/9ea1ae98e3d8f4ce38a10971e25768b68105b8d0/test/integration2/schema/type-consistency.spec.js)

Quite useful, for example on work binary uuid I did !

The same applies to .increments() and .bigIncrements()

It looks like .increments() currently creates a int8 with default value unique_rowid(), which works very differently from Postgres' int4 with nextval() on a sequence. This matches the default behavior as per the CockroachDB docs.
If we'd want to make this consistent across databases, I guess we should have .increments() create a int4, create a sequence, and rely on nextval() instead of the current default behavior.
It looks like CockroachDB has a session flag that might be able to do this automatically:

Which makes we wonder if this is something Knex should handle, or if that's something we should point out in the docs and have the user configure it properly.
I think this comes back to the discussion we've had a couple times before: How much abstraction should knex have to align the different database vendors. We already implemented a bunch of extra custom logic to allow SQLite to play nice with the others, but in this case, a simple set of SET statements in the pool connection callback would solve it. However, in that case the default behavior for CRDB is going to be pretty confusing / unexpected 🤔 There's no point in having integer vs bigInteger (and their increments counterparts) if they both do the same thing..
Thoughts @kibertoad @OlivierCavadenti?

Seems like the following configuration should align Cockroaches behavior with how you would expect knex to behave by default:
const knex = require('knex')({
  client: 'pg',
  connection: {...},
  pool: {
    afterCreate: function (conn, done) {
      conn.query('SET serial_normalization = "sql_sequence";', function (err) {
        if (err) {
          done(err, conn);
        } else {
          conn.query('SET default_int_size = 4;', function (err) {
            done(err, conn);
          });
        }
      });
    }
  }
});

This makes me question whether we should be doing any overrides of default CRDB behaviour to begin with. Forcing it to behave like pg sounds too opinionated for a query builder.
And if we keep increments an int8, then integer probably should stay one too, so that it is at least consistent with itself.
What are the implications of current misalignment between pg and crdb? numeric values need to be handled differently? wonder if it should be handled via documentation instead.

What are the implications of current misalignment between pg and crdb? numeric values need to be handled differently?

As of right now, it means that (for CRDB):

There's no difference between integer/bigInteger and increments/bigIncrements
.increments doesn't actually use auto-incrementing integers, like the name suggests
All integers are returned as Strings at all times

I agree with @rijkvanzanten! Maybe we could add a CRDB-only "postgres compat" option that sets those options and essentially makes it behave more like pg and the other vendors? The question then becomes if this should be enabled by default.

@rafiss Maybe you could comment on this issue and advise on whether preserving status quo or adding overrides would be better from CockroachDB perspective?

@nickrum I like that idea, but I wonder how intrusive that would be. Maybe it would be easier to achieve not as an option, but as a separate dialect, extending vanilla cockroachdb one. I would still like to hear @rafiss perspective first, because maybe we would be sabotaging crdb by overriding defaults that might be there for a good reason.

I'm personally worried that the API of knex itself becomes too unexpected/unreliable between dialects 🤔 There's currently already quite a lot of "It works like X, except for in VENDOR where it does Y".
We could also add a new .uniqueID() (name TDB) method that would allow to explicitly use Cockroach's default row identifier, which allows knex to keep a consistent set of methods for the above mentioned problems? For example:

.integer() would use int4
.bigInteger() would use int8
.increments() would use serial_normalization = "sql_sequence" with int4
.bigIncrements() would use serial_normalization = "sql_sequence" with int8
.uniqueID() Use Cockroach's unique_rowid based int8 serial primary key

Hi - what would also be possible is to force for integer to create an INT4 - https://www.cockroachlabs.com/docs/stable/int.html
This could be done by setting the session variable to use int4 for integer


[WIP] fix dse ci to use the latest builder image version

Trying to see if 1.0.4 will work..
Fails with 1.0.4 as well, mabe the problem was introduced in #733 this change, when we updated the DSE version to the 6.8.5 to 6.8.9.
And seems that there was already a revert from 1.0.3 to 1.0.4 in https://github.com/stargate/stargate/commit/f8073debd799b8bcabd2cc029b6dfb3a69ae3f18.

Closing in favor of #1155


Update typo on README.adoc

@nbompetsis Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@nbompetsis Thank you for signing the Contributor License Agreement!


Refactor cleanup_procs_and_nuke used by p-s-c/c-s-c uninstall or cleanse

Stop flailing at the runsvdir process tree and shut it down properly.
If initctl doesn't properly shutdown the process tree then there is
a bug that needs to be identified and fixed.

Closing per request from @jeremiahsnapp


fix: .net 7.0.12 patch

:white_check_mark: Build TheIdServer 7.4.4-ci.1+82.build.3328 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/6c24f1c595 by @aguacongas)


App force delete

If a user creates an app with a delete job, there could be a situation where the user pins themselves into a corner. If their delete job is misconfigured somehow and is unable to complete successfully, then their app will never be cleaned up properly.
There needs to be a way for a user to be able to skip cleanup and force delete their apps.

The changes for this have been merged. The easiest way I have found to test this is to deploy an app with following Acornfile:
containers: nginx: {
	image: "nginx:latest"
	ports: "80/http"
	files: "/usr/share/nginx/html/index.html": "Version 2"
}

jobs: "my-job": {
	image:   "busybox:latest"
	command: "sleep 600"
	events: ["delete"]
}

Then, in one terminal, delete the app with acorn rm -af APP_NAME. The delete will hang (for an hour). In another terminal, run acorn rm --ignore-cleanup APP_NAME. Once this second command completes, the command in the other terminal will also complete and the app will be removed.
Note that this requires the latest version of the CLI.

Tested with acorn version - v0.7.1-52-g63cbf08a+63cbf08a

Deployed app using the following Acornfile:

containers: nginx: {
	image: "nginx:latest"
	ports: "80/http"
	files: "/usr/share/nginx/html/index.html": "Version 2"
}

jobs: "my-job": {
	image:   "busybox:latest"
	command: "sleep 600"
	events: ["delete"]
}

once app is deployed successfully , attempt to delete the app.
App is stuck in Waiting for app mytestserive to be removed

 % acorn rm -af  mytestserive
Waiting for app mytestserive to be removed...

In another terminal , attempted to force delete the app:

acorn rm --ignore-cleanup mytestserive
Removed: mytestserive

This resulted in app getting deleted . Also delete that was stuck in "Waiting for app to be removed..." state proceed further and attempts to delete volumes and secrets and exits as expected.
acorn rm -af  mytestserive
Waiting for app mytestserive to be removed...
Removed: mytestserive
  •  No volumes associated with mytestserive
  •  No secrets associated with mytestserive

Also tried the following scenario where we were previously unable to delete the apps.
Able to delete app successfully using --ignore-cleanup option.
Deploy app that uses AWS resources which will fail to deploy successfully. Attempt to delete the app.
acorn run -n mytest ghcr.io/acorn-io/aws/rds-aurora-serverless:v0.0.2
%acorn apps
NAME      IMAGE                                               HEALTHY   UP-TO-DATE   CREATED   ENDPOINTS   MESSAGE
mytest    ghcr.io/acorn-io/aws/rds-aurora-serverless:v0.0.2   0         0            43s ago               [jobs: [cdk-cfn-render: [pending create], context-generator: [pending create]]] [services: rds: [pending create]] [secrets: [acorn-service: [waiting: [job not complete]], admin: [waiting: [job not complete]], aws-config: [missing: [context-aws]], cdk-context-json: [waiting: [job not complete]]]]
% acorn rm -af mytest
Waiting for app mytest to be removed...
^CRemoved: mytest
  ✗  ERROR:  context canceled
 % acorn apps
NAME      IMAGE                                               HEALTHY   UP-TO-DATE   CREATED   ENDPOINTS   MESSAGE
mytest    ghcr.io/acorn-io/aws/rds-aurora-serverless:v0.0.2   0         0            78s ago               [removing] [secrets: [acorn-service: [waiting: [job not complete]], admin: [waiting: [job not complete]], aws-config: [missing: [context-aws]], cdk-context-json: [waiting: [job not complete]]]]
% acorn rm --ignore-cleanup mytest
Removed: mytest
% acorn apps
NAME      IMAGE     HEALTHY   UP-TO-DATE   CREATED   ENDPOINTS   MESSAGE


modules connections

There are modules in the code:
SSD1306,
ADS1X15,
HMC5883 , MPU6050,
but I can't find my configuration and pin connections, unless the code is incomplete

The modules you referenced all use I2C for communication. The I2C pins are defined in include/deviceconfig.h.
I can section in the readme for connection with a diagram


Plan for renaming asm! macro to llvm_asm!

The change to rename the asm! macro to llvm_asm! was just approved in https://github.com/rust-lang/rust/pull/68404, so I expect that it will be in tomorrow's nightly. As far as I understand it, the old asm! will continue to work for now, but deprecated soon.
To avoid breakage, I think it makes sense to make it a minor release to avoid breakage for people that pinned an older nightly. The deprecation period should then be enough time for users to switch to the new version

The asm macro was now deprecated in https://github.com/rust-lang/rust/pull/71007.


Cannot Resolve Module

yarn add stampery yields not found modules:

fs
net
tls

I am not sure if this is an error on my site, but shouldn't yarn handle the installation of the dependencies?

Maybe an issue with my Webpack configuration.
I could not even add the missing package fs without changing my webpack confiuration:
node: {         fs: 'empty',     }


Add more Unity UI based examples to Interactables Scene

Overview
Expanded the Unity UI Interactables canavs in the InteractablesExample scene to also include a toggle button, radial buttons and checkbox buttons leveraging Unity UI images and interactables, and existing intractable themes to drive visual effects.
Changes

Fixes: # .https://github.com/microsoft/MixedRealityToolkit-Unity/issues/5998

Added a Unity UI based 3D toggle button to the scene

Added  Radial button group and checkbox button prefabs to the scene.

Updated the description of the panel in the scene.

The example also demonstrates how to setup layouts on Unity UI canvases.

Verification

Verified that the new added buttons work well with near field interaction as well as far field interaction.

/azp run

I did some scene/layout polish and pushed.
I am getting the error messages related to the Unity editor. could you please check?
Also, I don't see proper shape for the Unity UI toggle switch and radio.

NullReferenceException: Start Dimension Index: SerializedProperty is null
UnityEditor.EditorGUI.BeginPropertyInternal (UnityEngine.Rect totalPosition, UnityEngine.GUIContent label, UnityEditor.SerializedProperty property) (at C:/buildslave/unity/build/Editor/Mono/EditorGUI.cs:5471)
UnityEditor.EditorGUI.BeginProperty (UnityEngine.Rect totalPosition, UnityEngine.GUIContent label, UnityEditor.SerializedProperty property) (at C:/buildslave/unity/build/Editor/Mono/EditorGUI.cs:5446)
UnityEditor.EditorGUI+PropertyScope..ctor (UnityEngine.Rect totalPosition, UnityEngine.GUIContent label, UnityEditor.SerializedProperty property) (at C:/buildslave/unity/build/Editor/Mono/EditorGUI.cs:5434)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.RenderGeneralSettings () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:360)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.RenderCustomInspector () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:96)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.OnInspectorGUI () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:85)
UnityEditor.InspectorWindow.DoOnInspectorGUI (System.Boolean rebuildOptimizedGUIBlock, UnityEditor.Editor editor, System.Boolean wasVisible, UnityEngine.Rect& contentRect) (at C:/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:1625)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
Scope was not disposed! You should use the 'using' keyword or manually call Dispose.
UnityEngine.Scope:Finalize()

I did some scene/layout polish and pushed.
I am getting the error messages related to the Unity editor. could you please check?
Also, I don't see proper shape for the Unity UI toggle switch and radio.

NullReferenceException: Start Dimension Index: SerializedProperty is null
UnityEditor.EditorGUI.BeginPropertyInternal (UnityEngine.Rect totalPosition, UnityEngine.GUIContent label, UnityEditor.SerializedProperty property) (at C:/buildslave/unity/build/Editor/Mono/EditorGUI.cs:5471)
UnityEditor.EditorGUI.BeginProperty (UnityEngine.Rect totalPosition, UnityEngine.GUIContent label, UnityEditor.SerializedProperty property) (at C:/buildslave/unity/build/Editor/Mono/EditorGUI.cs:5446)
UnityEditor.EditorGUI+PropertyScope..ctor (UnityEngine.Rect totalPosition, UnityEngine.GUIContent label, UnityEditor.SerializedProperty property) (at C:/buildslave/unity/build/Editor/Mono/EditorGUI.cs:5434)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.RenderGeneralSettings () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:360)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.RenderCustomInspector () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:96)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.OnInspectorGUI () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:85)
UnityEditor.InspectorWindow.DoOnInspectorGUI (System.Boolean rebuildOptimizedGUIBlock, UnityEditor.Editor editor, System.Boolean wasVisible, UnityEngine.Rect& contentRect) (at C:/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:1625)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
Scope was not disposed! You should use the 'using' keyword or manually call Dispose.
UnityEngine.Scope:Finalize()

Editor errors are due to absence of fix for https://github.com/microsoft/MixedRealityToolkit-Unity/issues/6161. I merged latest mrtk_dev into this branch so that error has gone away.
The shader rendering mode setting was not pushed as part of the PR causing the circle to appear as square. This is fixed now.
However, in Scene view this is what you will see -

But when you enter play mode, it goes to this view

Is this acceptable?
I feel the theme colors for the radio button are not visually discernible for the 2 background rings . Is there some property in the material that can be tweaked to make it better? This is not an area of expertise for me, so would be great if i could get some help for it.

Some final things but looking good

The folder issue
The TextureTheme example was deleted and needs to be brought back in (see png)
The button status in the Unity UI panel causes the button layout to shift oddly (see gif)
Test that the panel text is readable on device (Text => "Various types of blah blah") because it looks very small, especially in editor

Thank you @Troy-Ferrell .
For #1 - I am ok doing that but would defer to @cre8ivepark  since i think the material that i have may or may not be the right one to push to a shared folder. But we should definitely have a ugui one in the standard assets folder .
For #2 - i think @cre8ivepark removed that. Unsure if this was a conscious choice.
For #3 - Will do. With the scene polish changes hiding it is no longer required.
For #4 - Yes. This is pending on my side.

I think I accidentally deleted the TextureTheme example. I'll add it back to the scene!

Thanks, Ritika! I have polished the size of the checkbox and radio buttons.
@Troy-Ferrell I have added TextureTheme back to the scene.

/azp run

just testing this and seeing that the label that updates the button / radio button / checkbox interaction on the right panel is not updating the text properly and also not fitting into the panel

when i click the non toggle button it still says i clicked the toggle button

just testing this and seeing that the label that updates the button / radio button / checkbox interaction on the right panel is not updating the text properly and also not fitting into the panel

when i click the non toggle button it still says i clicked the toggle button

(edit: checkbox and push button won't update text, radial button and toggle button will update text - is this how it's supposed to be?)

also the far left panel is not properly updating the button status in the label below - seems to only work for right hl1 button

unity ui example looks good to me now 🚀 👍 !
But I'm seeing errors when interacting with the panel on the left while pressing one of the hololens 1 button - also all the square buttons don't update the label underneath - animator button doesn't update the state as well

yeah i looked into the other buttons, seems like this has always been the case. I am going to spend some time to try to wire up the status of the other buttons and clean up the functionality in that panel but if it is too time consuming, then i will open a separate github issue for it and fix it in a separate PR

unity ui example looks good to me now 🚀 👍 !
But I'm seeing errors when interacting with the panel on the left while pressing one of the hololens 1 button - also all the square buttons don't update the label underneath - animator button doesn't update the state as well

yeah i looked into the other buttons, seems like this has always been the case. I am going to spend some time to try to wire up the status of the other buttons and clean up the functionality in that panel but if it is too time consuming, then i will open a separate github issue for it and fix it in a separate PR

@thalbern  - i have wired up the Button status label to correctly reflect the status of all buttons. Personally i feel that the sample is not built ideally since there is no way to call out which button's status is being reported in the TextMesh label. But in order to achieve that there might be more script changes required (mostly cleanup and refactor of the whole InteractableReceiever logic) which i want to avoid doing in this PR.

/azp run


Add email monitoring

Reason for this pull request
Qsub job status monitoring via email is missing in the existing fc_app code. Adding pbs job monitoring via email would help in monitoring the PBS jobs in an efficient way.
Proposed solutions
Update the fc_app.py to notify via email whenever a pbs job is successfully executed or failed to execute or aborted.

[x] Tests passed (no update to tests required, as the test scenarios were already written as part of dea-env testing)

Coverage decreased (-0.9%) to 35.492% when pulling 0d9d6d825bec271a60938e0c846f7746fc5911fa on add-email-monitoring into 427233e7efec2cb1b66afc8ed7f739ef38742efe on master.


test(plonky2): Signed comparisons tests

Added some tests checking comparisons between signed integers. Since signed integers aren't supported yet, they are put in plonky2_prove_failure.

This still waits for a review, after the new test.
Additionally given #81 exists, I think it may be better to wait for it to be merged and afterwards to add tests, since there is a chance (especially the new test) would fail when it shouldn't. And it makes the merge workflow a bit more fluent (nickysn doesn't have to update his local branch and fix the tests).

This still waits for a review, after the new test.
Additionally given #81 exists, I think it may be better to wait for it to be merged and afterwards to add tests, since there is a chance (especially the new test) would fail when it shouldn't. And it makes the merge workflow a bit more fluent (nickysn doesn't have to update his local branch and fix the tests).

It's actually better to merge this first. Then #81 will move the tests from "failure" to "success" to demonstrate how it changes the behavior of the compiler.


Ensure SNI flag set on multi-site hostname listener (Azure#426)

This PR add support for RequireServerNameIndication field for V1 gateway.
This fields is needs to provide the correct certificate bound to the listener/hostname/port on V1.

This is a highly desired change for us. Thanks team!


missing description for -with-extras version

Hi!
Thanks for keeping this fork updated. I have a question – What is the difference between regular and -with-extras packages?

The with extras was some work that ducksboard did to add a 8 new functions for a TV display.  I think they were considered experimental, and never moved into the main baseline.
You can see the additional functions here: https://github.com/dsmorse/gridster.js/blob/master/src/jquery.gridster.extras.js

@dsmorse thx for info. I'm also wondering about max_size_x which states:
The maximum number of columns that a widget can span.
I've tried to change this value and well it doesn't seem to affect anything.

You can see a working example on the demo page:
http://dsmorse.github.io/gridster.js/demos/resize-limits.html
On Tue, Jun 23, 2015 at 6:24 AM, pySilver<EMAIL_ADDRESS>wrote:

@dsmorse https://github.com/dsmorse thx for info. I'm also wondering
about max_size_x which states:
The maximum number of columns that a widget can span.
I've tried to change this value and well it doesn't seem to affect
anything.
—
Reply to this email directly or view it on GitHub
https://github.com/dsmorse/gridster.js/issues/17#issuecomment-114473748.


Add guard against calling setEditingState in detached mode

This closes #16

@danvick ?

Hi @kengu,
Thanks for the PR. I'll review and merge as soon as I can.


CI/test: add race detection check

Description:
Adds race detection for Go unit tests.

Affected area:

Functional Area

Tests
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Codecov Report

Merging #4157 (0aa8180) into main (2d2be96) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4157      +/-   ##
==========================================
- Coverage   69.81%   69.78%   -0.04%
==========================================
  Files         211      211
  Lines       11463    11463
==========================================
- Hits         8003     7999       -4
- Misses       3407     3411       +4
  Partials       53       53

Flag
Coverage Δ

unittests
69.78% <ø> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/crdconversion/crdconversion.go
71.42% <0.00%> (-4.09%)
:arrow_down:

pkg/certificate/rotor/rotor.go
84.37% <0.00%> (-3.13%)
:arrow_down:

pkg/providers/kube/client.go
71.68% <0.00%> (+0.88%)
: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 2d2be96...0aa8180. Read the comment docs.


[libgd] Fix static build

Fixes #6306 by reverting portfile changes made by  050e71d

Thanks for the PR!


Convert components to classes.

This makes the library work with more recent React versions.

Hi @hawkrives
There is a little bit of JSX here: https://github.com/hawkrives/react-list-select/blob/master/List.es6#L193
This causes my build to fail, adding babel-preset-react resolved that for me.

Ah, so there is JSX. Good catch.
If you revert your version number bump commit, I'll gladly merge and release this 😄

Awesome :)
I've reverted the version bump.

Well, that cascaded a bit. If v0.4.0 doesn't work for you, please let me know.
I've added a changelog to the repo, so I'll direct you there.

(@svenvandescheur)

@hawkrives Thanks, Yes 0.4.0 works perfectly, thanks for helping out.


Feature Request: useIntersectionObserver

Describe the feature in detail (code, mocks, or screenshots encouraged)
A utility function which provides a way to track visibility and position of a DOM element within the viewport or a container element. Under the hood it will use Intersection Observer API so it would expect to receive IntersectionObserver's root, margin, and threshold options and would return an instance of IntersectionObserverEntry
What type of pull request would this be?
None
Provide relevant links or additional information.

https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API

Closed by #73


License

Hello, please assign a license to these contracts, i.e. are we free to modify them and deploy our own?

I can see this was resolved in #112. Closing.


Directed BFS sampling

The purpose of ticket #464 is to create a version of breadth-first graph sampling that can work on directed graphs, as a prelude to implementing directed GraphSAGE. There is a document which covers BFS and GraphSAGE on the shared drive under (approx.) IA > Team folders > ML > Graph ML library. This document explains the choice of data structures.

My feedback is mostly in the form of questions, because I don't have much of a picture of what the overall goal/design is here.
I haven't looked at the tests at all.

Firstly, did you read the google doc I wrote? If not, that should give you a better idea of what slots are and why they are needed, and what data structures I've designed.
Also, all the tests that say, hey this must be a list - they are not my tests! I agree with you, most things should be iterables not concrete lists. We plan to rationalise the library, and I would like such things to form part of the design.

Another question:

explorer.py, line 165: why can't we have n_size=0? Setting n_size=0 is our way to easily switch-off neighbour sampling in GraphSAGE and HinSAGE!

Another comment:
explorer.py, lines 673 and 777: you reuse GraphWalk._check_parameter_values method here, but pass len(n_size) as length argument. This is somewhat confusing, as in the declaration of GraphWalk._check_parameter_values, length is defined as "Maximum length of walk measured as the number of edges followed from root node" - which is completely irrelevant to what we are in fact passing to it here. One way to fix this would be making the docstring in GraphWalk._check_parameter_values more general, to cover all possible meanings of the arguments. Another way would be to revert to walker-specific methods for checking parameter values - less code reuse but easier to understand.

@youph if I reworded "Maximum length of walk measured as the number of edges followed from root node" to be something like "Length of walk; the maximum number of hops from the root node", would that be suitable? Or "The path length n for n-hop walks"? Or give me your own wording.
I agree that "number of edges followed from root node" is misleading; my intent was that a single walk only follows one edge at a time, even if the sampler is allowed to amalgamate multiple walks into one structure.

The build is getting this error: E       AttributeError: module 'keras.backend' has no attribute 'tf' Runs fine on my tests, is this an issue with buildkite - the maligned test is tests/layer/test_graph_attention.py.

Will merge as recommended by @PantelisElinas.


use FEVM-Hardhat-Kit deploy failed when wallaby reset

I use FEVM-Hardhat-Kit to deploy smart contracts. Everything works fine before the network is reset, but after wallaby is reset, the following error will be returned:

Only when I use another address to deploy can it succeed, what is the reason?

Could you provide the rest of the error? Most likely what is happening is that hardhat is looking through your deployments folder and looking for a contracts at the addresses stored there, but since Wallaby has been reset those contracts no longer exist on chain. If that is the case you can run the command yarn hardhat deploy --reset


Recreating vim's newline at EOF logic

Hello all:
I know I am entering a minor quagmire by asking about vim "adding" a newline at the EOF when saving a file. I'll mention that I know the POSIX standard states that a line is "A sequence of zero or more non-  characters plus a terminating  character." (from: http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_206). This isn't another post asking why vim writes the newline on save.
The background for my question is that I am trying to submit a pull request for the vim-lsp plugin (https://github.com/prabirshrestha/vim-lsp), which provides Language Server Protocol support to vim. I was getting some linting errors about "no newline at end of file" when using this plugin in vim, but not when sending the file directly to the linter and skipping vim in the middle.
I tracked the issue down to how vim-lsp is sending the contents of the file to the Language Server. The Language Server specification has a "didChange" message that expects you to send the contents of the file/buffer inside of the message. I'm pretty sure you send the contents of the buffer instead of just a file path so that you can have linting performed on your buffer even if it hasn't been saved to disk.
Ultimately the issue comes down to how we can grab the full contents of a buffer. The original code in vim-lsp looks like this:
function! s:get_text_document_text(buf) abort
    let l:buf_fileformat = getbufvar(a:buf, '&fileformat')
    let l:line_ending = {'unix': "\n", 'dos': "\r\n", 'mac':"\r"}[l:buf_fileformat]
    return join(getbufline(a:buf, 1, '$'), l:line_ending)
endfunction

getbufline is grabbing all of the contents of the buffer and returning a list of the lines. Each one of the items in the list does not contain the newline character that would be found in the file. This is why the join function is used, to insert a fileformat-dependent newline in between each of the lines. As-is, the join doesn't append a newline on the last item, which led to the linter complaining.
My pull request to fix this attempts to inspect and respect the eol, binary, and fixeol options to recreate what would be found on disk:
function! s:requires_eol_at_eof(buf) abort
    let l:file_ends_with_eol = getbufvar(a:buf, '&eol')
    let l:vim_will_save_with_eol = !getbufvar(a:buf, '&binary') &&
                \ getbufvar(a:buf, '&fixeol')
    return l:file_ends_with_eol || l:vim_will_save_with_eol
endfunction

function! s:get_text_document_text(buf) abort
    let l:buf_fileformat = getbufvar(a:buf, '&fileformat')
    let l:eol = {'unix': "\n", 'dos': "\r\n", 'mac': "\r"}[l:buf_fileformat]
    return join(getbufline(a:buf, 1, '$'), l:eol).(s:requires_eol_at_eof(a:buf) ? l:eol : '')
endfunction

After staring at :help eol, :help fixeol, and :help binary, I think I'm recreating the logic that vim uses when deciding to put the newline in the file. Basically, eol is set if the file contained the eol on load. The user can unset this if they want, but they need to be aware of fixeol as well. Vim will attempt to fix a file missing eol at EOF unless the file is loaded with binary or the user has set nofixeol. If the user has disabled fixing eol or loaded a file as binary, but the file contained the eol at EOF then vim will obviously put the eol back.
Does this logic match what we expect vim to do? Is there a different way to get the contents of the buffer that includes the newline characters?
Adam

Ideally it would be good to have a function like getbufcontent() which automatically does this. But for old vim we would still need to understand the algorithm so it can continue to work.

FYI, you need to convert encoding too.

@mattn Did you mean this https://github.com/nsf/gocode/blob/9d1e0378d35b0527c9aef0d17c0913fc38d88b81/vim/autoload/gocomplete.vim#L6-L19?
fu! s:gocodeCurrentBuffer()
	let buf = getline(1, '$')
	if &encoding != 'utf-8'
		let buf = map(buf, 'iconv(v:val, &encoding, "utf-8")')
	endif
	if &l:fileformat == 'dos'
		" XXX: line2byte() depend on 'fileformat' option.
		" so if fileformat is 'dos', 'buf' must include '\r'.
		let buf = map(buf, 'v:val."\r"')
	endif
	let file = tempname()
	call writefile(buf, file)
	return file
endf

@adamatom  does this one work?

@prabirshrestha I don't mind adding similar logic to pass the buffer contents through iconv to force the utf-8 encoding. I think this would be best done in a new pull request though, and I can't pretend to be an expert on encodings. For instance, I can't say for sure if always converting to UTF-8 for the didChange message is the best thing to do.


overlapping 2 bar in same stack

Is there any way to overlap 2 bar in same stack? Like this image

Can you share with me how you to plugin waterfall?
I have follow the step(https://github.com/everestate/chartjs-plugin-waterfall), but still cant work forme.
my chart.js version: "2.7.2" && chartjs-plugin-waterfall version: "1.0.3"
This is my chart plugin. Any wrong in my plugin??

@CoolM1234  what error you are having? console.log(vm.data) and paste the output here

@rassemdev  Okay. I not open up a new issue is because I see all the issues are not buddy to reply, so hopefully you can give an answer.
This is my dataset (vm.data):


time-comp Timestamp conversion wrong by one day.

Describe the bug
I am using the node time-comp to parse timestamps and convert them to a different format. I was facing some offset problems in the converted timestamps. I simplify my flow to simple have a UNIX timestamp as an input and then pass it directly to the output (see attachments). The result is one day ahead.
Screenshots

Expected behavior
The output shall be the same value as the input.
System information (please complete the following information):

Node Version: node-red-contrib-sun-position 2.1.1
Node-Red Version:  v1.2.9

Additional context
Add any other context about the problem here.

This is fixed in the latest version.
Since this version has undergone some changes internally, I have this but still released as beta. That means you can't update it via Node-Red, but only via the command line.

(closed automatically), I will close Issue when version released regular.

Installation guide can be found in the release notes:
https://github.com/rdmtc/node-red-contrib-sun-position/releases/tag/2.2.0-beta


Show ceteribus paribus in a plot

In problem set 2, there is an exercise where we have to show that age and education are ceteribus paribus.  Online I found the function ceteris_paribus
ceteris_paribus(
explainer,
observations,
y = NULL,
variable_splits = NULL,
variables = NULL,
grid_points = 101
)
I am not sure if this is correct, and if so, what would be the explainer and what would be the observations?

@gwism: hi Gian, thanks! Could you please post the package and some context?
In any case: no external functions are needed to solve PS2.

The library is also called library (ceterisParibus).But I think this is not relevant for PS2

I'm having difficulty in understanding the task:
build plots to show that the ceteris paribus assumption holds true for age and education.
Can somebody rephrase what we are supposed to do? Thanks!

@marcograf1: are treated and non-treated similar? Or could there be some selection bias?


Packaging for embassyOS

Have you considered packaging Bitcart as a service that can run on embassyOS?
embassyOS is a Linux distro that streamlines the installing and managing of self-hosted services. Users can one-click install Bitcoin core and a dozen lightning layer services - and BTCPay has been in the marketplace for a while.
Might want to check it out. They've got thousands of users - might be a shot in the arm for the project. I think package Dev is pretty minimal - seems to be writing a Dockerfile and some trivial manifests.
https://start9.com/
https://docs.start9.com/latest/developer-docs/packaging

Thanks, I will check it!
Ideally though it would be cool if users of their OSes/Package managers/etc managed it, but I will look at it

I can tell you that the embassyOS team has said explicitly that, while they have packaged a few dozen services thus far, they prefer a future where most packages are produced and managed by the service's team. (A la Apple doesn't produce all the 3rd party apps in their AppStore)

I see, thanks
Makes sense indeed


Flatlist doesn't invoke renderItems when data passed is array of nulls

Description
When numColumns field is added to display Flatlist as gridView, the flatlist's renderItem method doesn't invoke when we pass array of nulls as data.
Version
0.67.2
Output of npx react-native info
System:
OS: macOS 12.3.1
CPU: (8) arm64 Apple M1
Memory: 122.77 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.13.0 - /usr/local/bin/node
Yarn: 1.22.17 - /usr/local/bin/yarn
npm: 8.1.0 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
Android SDK:
API Levels: 17, 23, 28, 30, 31
Build Tools: 29.0.2, 30.0.0, 30.0.1, 30.0.2, 31.0.0
System Images: android-29 | Google Play ARM 64 v8a, android-30 | Google APIs ARM 64 v8a, android-31 | Google APIs ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_311 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.67.2 => 0.67.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Use a flatlist and pass data to flatlist as [null, null] you'll see that it render the list but when you change the same flatlist to gridView by passing numColumns the same code stops working, the renderItem method is never invoked.
Snack, code example, screenshot, or link to a repository
Flatlist code with array of nulls that invoke renderItem method and displays item.
 <FlatList
                            data={[null, null]}
                            keyExtractor={(item, index) => index.toString()}
                            renderItem={({item, index}) => this.renderItem(item, index)}
/>

Adding the numColumns field doesn't call renderItem method. Below code doesn't work
<FlatList
                            data={[null, null]}
                            keyExtractor={(item, index) => index.toString()}
                            renderItem={({item, index}) => this.renderItem(item, index)}
                            numColumns={2}
/>

I opened a PR to fix this issue


Fix/make tier processing consistent between copy and sync

Without this, there'll be a bug in that it will impossible to, for instance, sync between a standard blob storage account and a premium block blob account (because they don't have the same access tiers).
Note that there are two commits in this PR. Only the first is technically necessary to fix the bug. But the second reduces code duplication and should improve maintainability as we add more properties in future.  The second basically just takes common.NewCopyTransfer and makes it a method of storedObject so we don't have to pass heaps of parameters invididually

Should comfortably reduce some code dupe between sync and copy.


reformatting fix for WSAS-1853 from Nipuni

Refer [1] for changes
[1] https://github.com/wso2/carbon-deployment/pull/103


sad :(

💀

issue by having big font size, this would require layout change based on font, kinda complicated to just do qwq

please update the description so the issue could be found by search


[Mailman2Bridge] fix message separation and improve "From_ lines" disambiguation

This PR mainly fixes separation of messages in the monthly mbox archives, which rendered the bridge broken without it.
"From_ lines" disambiguation is also improved a bit to make erroneous message splits less likely.

thanks


Allstar review for outreachy

Thanks @Twhite2, please see https://github.com/open-telemetry/sig-security/pull/29 and work with @EjiroLaurelD to ensure your work and findings don't overlap

Alright then. Will do.👍🏾


Fix description of parameters "AWS Access Id" and "Secret Key" in User Profile > EC2 Connectors.

Fix description of parameters AWS Access Id and Secret Key in User Profile, in configuration sections of EC2 connectors:

In the current implementation of the help-hints there is no point in having HTML into the text, because they have a hover behaviour, i.e. the help hint popover disappears as soon as the mouse leaves the ? icon.
Therefore, the suggested solution is to remove HTML from the descriptions.

HTML content has been removed from info text.
New help dialogs:

:smiley:  :+1:

As discussed yesterday with @st, now those popovers correctly parse HTML, but only for formatting purposes, i.e. active elements ( tags, <button>s and other) are not allowed because the popover disappears when the mouse leaves the ? icon.
Therefore, in the current popover, we can use a  tag to wrap the corresponding words, instead of using single quotes, i.e. secret key instead of  'secret key'.
More over, I'd suggest to remove the (required to submit to EC2) part from the name.
See PR referenced above.


Questions about the implementation of GLU

This is my first issue post, so I apologize if this is not the right place to discuss what I'm posting.
I have a question about the GLU used in layers such as ResidualBlock in "athena/layers/commons.py".
In the ResidualBlock implementation, the output tensor of the same convolution layer is used for the main signal and the gate signal of the GLU. Is this a mistake in the implementation? Or my misunderstanding?
   def call(self, x, c):
        x = self.conv_1(x)
        x = self.cin_1(x, c)
        x = gated_linear_layer(inputs=x, gates=x)

If my understanding is correct, the tensor given to the GLU is the output of different convolutional layer instances with the same structure.
   def call(self, x, c):
        x_main = self.conv_1_main(x)
        x_main = self.cin_1_main(x_main, c)
        x_gate = self.conv_1_gate(x)
        x_gate = self.cin_1_gate(x_gate, c)
        x = gated_linear_layer(inputs=x_main, gates=x_gate)

I'm a beginner, so I apologize if I'm wrong.

@sanaperiod Your implementation is right！


Colab Notebook?

This is an amazing method. Is there a colab notebook (especially for video) planned?
Thank you! :)

Here's the notebook I'm using:
https://colab.research.google.com/drive/1x0NyS3zXO8b68IRfNjYEn0AMcsqPEekI?usp=sharing


replace online help with sphinx docs

Interim:
Provide read the docs link.
Desired Final State:
Sphinx docs distributed with visit releases.

We should also add the release notes to the Sphinx documentation.

Do we want to only replace the "VisIt User's Manual" section and keep everything else as is? The other secionts are:

Visit home page
Release notes
Command line arguments
Frequently asked questions
Copyright
Visit Contributors
Visit UltraWrapper

In the spirit of @cyrush remark about the final desired state, all of that other content is used/needed in multiple places but our goals should be

having it all available as part of a release
maintaining only a single source for all of it

I think that means that all the sections you mention eventually need to migrate either into the manual (most of it probably makes sense there) or maybe onto the new web site.
For this ticket, maybe just adding the http link to the on-line manuals and leaving the other sections as is.

I agree with Mark, only replace the "VisIt User's Manual" section until we figure out where things should eventually live.

It sounds like, in the future, we'll want to run the sphinx documentation builder for each new release and use that build as the manual in the help menu. For now, though, how should we handle building the sphinx documentation? I see two options (maybe there are more?):

Have VisIt generate the sphinx documentation during the build process. This would ensure the most up-to-date version of the documentation is being displayed, but it also would require that the python VisIt is using has the sphinx-build package installed. This would probably require updating the build scripts to include that python package.
We can manually generate the sphinx build ourselves and update the documentation whenever needed. This would bypass the python requirements (sphinx-build is installed on LC and also easy to get through pip/conda on laptops), but it would put responsibility on developers to update the documentation build.

Thoughts?

It sounds like, in the future, we'll want to run the sphinx documentation builder for each new release and use that build as the manual in the help menu.

Agreed.

Have VisIt generate the sphinx documentation during the build process.

I would skip that for resolving this issue. It might be worthwhile in the near future but maybe not too.

We can manually generate the sphinx build ourselves and update the documentation whenever needed.

We are effectively doing this now via commits that change docs. ReadTheDocs automatically tracks that and updates the live documentation shortly thereafter. While that works great, it doesn't help air-gapped networks or other similar logistical hurdles for hosting VisIt documentation.
So, we will definitely need the ability to build the sphinx documentation and package it with a release. Once we can effectively do that, I honestly think we should just get rid of trying to render our documentation inside a Qt widget in a running VisIt session (same with release notes) and instead simply have VisIt launch the default browser given either a file:/// or http:// url to the docs to render.

@markcmiller86 It sounds like you are in favor of option 2? I'm not entirely clear on how the ReadTheDocs tracks and updates its VisIt documentation build. Could we leverage something similar for the Qt display build?
Ideally, we would have a sphinx doc build located and updated in the resources/help/en_US directory so that it is consistent with the location of the rest of the Qt help docs. That's where I've currently built, and it requires only a few changes to the Qt code (it doesn't look as nice as the browser display, but it seems to all be there).

@markcmiller86 It sounds like you are in favor of option 2?

Ok, sure.

Ideally, we would have a sphinx doc build located and updated in the resources/help/en_US directory

That makes sense.
I just don't think we should maintain code we don't need to display that stuff. Today, it doesn't look as nice. Next Qt update, it won't render png images to correct sizes or some other funky Qt thing will happen. Update after that, something else may go wrong with Qt support for it....All I am saying is I see no reason to continue to maintain the Qt display of our docs if we can just launch a user's (default) browser to display it. That is what most tools do today.
I am NOT saying you or this issue is about doing that last Qt-elimination step. I am just saying that is probably something we ought to shoot for in the future.


docs: useLatest

:tada: This issue has been resolved in version 7.26.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


tBTC dApp redeem function - Interface freezes at step 5

Description:
Minor interface bug happens at the last step of Redeem function - when the process is over and I can see that I received testnet BTC to my wallet, interface freezes at step 5 without showing the "Complete" screen at step 6.
Steps to reproduce the behavior:

Go to 'https://dapp.test.tbtc.network/redeem'
Pass steps 1-4 successfully
dApp is getting stuck at step 5

Expected behavior:
Passing to the last step 6.
Environment details:
OS: macOS Mojave 10.14.6 (18G5033)
Web Browser: Google Chrome, 83.0.4103.106 (Official Build) (64-bit)
Wallet: Metamask (Chrome extension), Version 8.0.4
Screenshots

What does "freezes" mean? Also, can you share the URL of the dApp in this case?

@Shadowfiend "Freezes" in a sense that it's just getting stuck on this page. Normally I would expect see the last step, but it's just keeps loading step 5, while I already have my testsnet BTC on my wallet. If I will encounter this issue one more time, I will update you with URL. Thanks


Extension packages with hidden option.

@swc-node/jest seems like a good idea and I think such support for some libraries worth it.
But supporting library means a maintenance hazard. What should happen if version of a library is bumped, or abandoned?
It will require bumping major version of swc, which is not a good idea.
So I think the hidden option, intended to be used only by packages in the @swc namespace, can be a solution.

How about expose API in @swc/core to allow people use native plugin in NodeJS.
For example:
const { createTransformSync } = require('@swc/core')
const jestHoistPlugin = require('@whatever-namespace/jest')
const swcImportPlugin = require('@other-namespace/import-plugin')

const transformSync = createTransformSync(jestHoistPlugin, swcImportPlugin)

transformSync(code, swcoptions)

I think the way you suggested is the best way to go, but I'm not sure how about the way to do it.
Long ago, I tried implementing native plugin system, but I failed because of platform issue (deployment) and neon did not support such thing.
Does napi itself supports such operation? I mean, loading other .node  file and invoking it?
Also, what should be done when swc made some breaking changes? This was one of the main problems while my experiment.

Does napi itself supports such operation? I mean, loading other .node file and invoking it?

Let me try it in @swc-node project.


Upgrade Groovy to 2.4.10

There seems to be an issue with binary compatibility between 2.4.7 and 2.4.9 that is fixed in 2.4.10: https://issues.apache.org/jira/browse/GROOVY-8107
+review REVIEW-6487

Successful stage 5: https://builds.gradle.org/viewLog.html?buildId=2565904&tab=buildResultsDiv&buildTypeId=bt46
I think we're good.

@lptr Can you please close the issue when you think it's ready?


Gradle: always use latest intellij version

I think it would be good to automatically test on the latest (release) version so when builds suddenly fail it might be that intellij has updated with backwards incompatible changes and we will be notified.

Doesn't work for me with 2018.2: IJ doesn't accept the plugin build like this


Needs upgrading...

RaisedButton is replaced with ElevatedButton, etc, I tried to upgrade it myself, but was a string that kept unravelling...

I upgraded it to Flutter 3.16.5 and get 4.6.6 here:
https://github.com/clovett/flutter_samples/tree/main/getx_todo_list