Text Display is broken after update

Text in Radial Menu become invisible after recent update.

[Error  : Unity Log] NullReferenceException: Object reference not set to an instance of an object
Stack trace:
TMPro.MaterialReference..ctor (System.Int32 index, TMPro.TMP_FontAsset fontAsset, TMPro.TMP_SpriteAsset spriteAsset, UnityEngine.Material material, System.Single padding) (at <9baa8119dbcd498f902c1a92f10843c1>:0)
TMPro.TextMeshPro.SetArraySizes (TMPro.TMP_Text+UnicodeChar[] unicodeChars) (at <9baa8119dbcd498f902c1a92f10843c1>:0)
TMPro.TMP_Text.ParseInputText () (at <9baa8119dbcd498f902c1a92f10843c1>:0)
TMPro.TMP_Text.GetPreferredValues (System.Single width, System.Single height) (at <9baa8119dbcd498f902c1a92f10843c1>:0)
TMPro.TextMeshPro.Rebuild (UnityEngine.UI.CanvasUpdate update) (at <9baa8119dbcd498f902c1a92f10843c1>:0)
TMPro.TMP_UpdateManager.DoRebuilds () (at <9baa8119dbcd498f902c1a92f10843c1>:0)
UnityEngine.Canvas.SendWillRenderCanvases () (at <c6e732ee43764427b2b923952ecc7fa9>:0)

Thank you for this stack trace! Let me take a look.

Would there be any relevant logs before the one you shared? It doesn't actually help a ton, to be honest. There wasn't any type of error previous to this message?
Do you think there might have been another mod conflicting with this? Could be hard to guess though.

Nope. No single error while loading and hosting, apparently maybe there was other mod that tried to load and caused TooManyEmotes to cause error later. MonitorLabels 1.3.0 caused this issue after little check and i reported that as well to creator.

Sorry for the late reply. It sounds like it's not a specific issue related to my mod. Would it be fine if I closed this since there's not much I can do?


Tizen ACR

[ ] Add tizen feature check
[ ] Add tizen privilege check

:octocat: cibot: Thank you for posting issue #419. The person in charge will reply soon.

[ ] TCM
[ ] TCT

migrated this to #259


Paginate.Callbacks.onLoadMore() method is invoking two times

Hello  @MarkoMilos,
First of all, thanks a lot for creating such a wonderful API for pagination. It works smooth for me.
But this is having an issue, if invoking onLoadMore() method on activity onCreateView(), REST API is invoking two times. This is happening because of below code:
private final RecyclerView.OnScrollListener mOnScrollListener = new RecyclerView.OnScrollListener() {         @Override         public void onScrolled(RecyclerView recyclerView, int dx, int dy) {             checkEndOffset(); // Each time when list is scrolled check if end of the list is reached         }     };
Method RecyclerView.OnScrollListenervoid onScrolled (RecyclerView recyclerView, int dx, int dy):
Source[https://developer.android.com/reference/android/support/v7/widget/RecyclerView.OnScrollListener.html#onScrolled(android.support.v7.widget.RecyclerView, int, int)]
Callback method to be invoked when the RecyclerView has been scrolled. This will be called after the scroll has completed.
This callback will also be called if visible item range changes after a layout calculation. In that case, dx and dy will be 0.
So could you please add some condition to prevent onScrolled() method invocation on view creation.
This will be helpful.
Thanks,
Chandan Kushwaha

@ckushwaha207 Hi, I'm having some troubles to understand the this issue.
So correct me if I'm wrong.. :)

You are setting up Pagination on RecyclerView in Acitvity onCreateView method
onLoadMore gets called
You started your data loading (network or something)
onLoad more gets called again for the 2nd time ?

What did you return in hasLoadedAllItems callback method?

Hi,
Returning false for hasLoadedAllItems callback method.

@ckushwaha207 Can you copy/past stripped code snippet of that activity/fragment/view?

@ckushwaha207  Please write you decision, I catch double call onLoadMore too.

Cannot reproduce this and no example is provided :(
Closing due to inactivity.


for simplicity we use  biult in group model as teams and roles and , we use group in workflows

done


Add support for TP_PING message in C++ SDK

This PR adds support to the C++ SDK for responding to TP_PING requests from the validator.

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

To test locally (YMMV, but this is what worked for me in Vagrant VM on Windows):

build_all (or if you are impatient build_all -l cxx
In a shell, start up validator:

sawtooth admin keygen
sawtooth config genesis -k ~/sawtooth/keys/validator.priv -o config-genesis.batch
sawtooth admin genesis config-genesis.batch
sawtooth-validator -vv --endpoint tcp://localhost:8800 --bind component:tcp://<IP_ADDRESS>:4004 --bind network:tcp://<IP_ADDRESS>:8800

In another shell, start up C++ intkey TP:

./sdk/cxx/build/bin/intkey_cxx tcp://localhost:4004

In another shell, start up any other TP (e.g., settings-tp -vv tcp://localhost:4004)

If you kill (i.e., Ctrl-C) the TP in step 4, you should see the intkey C++ TP emit a log message that it received a TP_PING message and the validator should emit log messages that it both sent a TP_PING and recieved a TP_PING_RESPONSE.

Quick rebase before merge


weird error

when i try to import pmag in a jupyter notebook, i get this error:
ImportError                               Traceback (most recent call last)
 in ()
----> 1 import pmag as pmag
2 #import ipmag as ipmag
3 #import ipmagplotlib as ipmagplotlib
4 import numpy as np
5 import pandas as pd
ImportError: bad magic number in 'pmag': b'\x03\xf3\r\n'
any ideas?

Shouldn't the import be:
import pmagpy.pmag as pmag

Might that be the problem?

You are right - but that wasn’t the problem. I had to delete all the .pyc
files for some reason.
On Sun, Mar 4, 2018 at 9:20 PM Nick Swanson-Hysell<EMAIL_ADDRESS>wrote:

Shouldn't the import be:
import pmagpy.pmag as pmag
Might that be the problem?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/PmagPy/PmagPy/issues/313#issuecomment-370311602, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKrDUEk0inTo5on7G-rZ_U_1MD_Ko2Gks5tbMkQgaJpZM4SbfTW
.

Interesting. That's a bit odd, but I am glad that fixed the issue.


add editorconfig

This is purely for local editor convenience. .editorconfig is widely used for setting workspace settings.
My editor often doesn't infer tab size correctly, esp for new files. I guess I could use a .vscoode/config.json, but I thought this may help new contributors, too.
I've also added some other reasonable defaults.

Why not all files to have the same rules?


Update to latest examples-ui in most examples

Description
The latest UI package no longer uses next-transpile-modules so the next.config.js has been removed from most places.
Don't merge until all deployments are working!
Type of Change

[ ] New Example
[x] Example updates (Bug fixes, new features, etc.)
[ ] Other (changes to the codebase, but not to examples)

I had to write a script to update some settings in every Vercel project and in order to update every deployment I'll open this PR again.


remove deprecated usage of com.netflix.zuul.message.http.Cookies

Addresses #432

@carl-mastrangelo is this being addressed in a different PR?


Correction for raw affiliation Arts et Métiers Institute of Technology, CNAM, DynFluid, HESAM Université, F-75013 Paris, France

Correction needed for raw affiliation Arts et Métiers Institute of Technology, CNAM, DynFluid, HESAM Université, F-75013 Paris, France
raw_affiliation_name: Arts et Métiers Institute of Technology, CNAM, DynFluid, HESAM Université, F-75013 Paris, France
new_rors: 042949r55;018pp1107;0175hh227;059jx7q58
previous_rors: 042949r55;018pp1107;0175hh227
works_examples: W4389137880
contact: 4a50bd91bbd7590395a895c1afe9b41b:2a078f65d2d1c01fb8402262ba @ ensam.eu

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


etcd controller shall not ignore ghost etcd member in health check

Environmental Info:
K3s Version:
Node(s) CPU architecture, OS, and Version:
Cluster Configuration:
k3s cluster with embeded etcd cluster
Describe the bug:
Currently, when there is a ghost etcd member (the member which don't belong to any k3s node), etcd controller choose to ignore it.
https://github.com/k3s-io/k3s/blob/4adcdf8db32a4be6569c65ede782526c102a0e52/pkg/etcd/etcd.go#L1115-L1117
This could be dangerous. Suppose there is a 3-controlplane nodes k3s cluster with embeded etcd, and 1 ghost member which may already offline (may caused by unclean node removal), then the cluster is actually in unhealthy state and cannot bear another cp node offline.
We are working on k3s cluster-api provider, when we do rollout update on k3s cluster, we encountered issue that node not removed cleanly and have etcd member left, and later it causes quorum loss in the middle of rollout.
Although it can be solved by leveraging annotation etcd.k3s.cattle.io/remove, it's better to have a 2nd guard to ensure no ghost etcd member eixists before doing any cluster managment operation (capi default controlplane implementation have this check). See previous discussion https://github.com/k3s-io/k3s/discussions/9841
Steps To Reproduce:

create 3 controlplane node k3s cluster
shutdown 1 controlplane machine and delete that node by kubectl delete node

Expected behavior:
node EtcdIsVoter condition shall be false with message showing the ghost member
Actual behavior:
node EtcdIsVoter condition shows as true
Proposed fix
PullRequest https://github.com/k3s-io/k3s/pull/11232
Additional context / logs:

It is normal for a node to be added to etcd before the kubelet comes up and creates a Kubernetes node object.  This should be handled by member promotion, and on the other side, the finalizer on node objects should prevent nodes from being deleted without being removed from etcd.
Can you provide steps to reproduce this?

It is normal for a node to be added to etcd before the kubelet comes up and creates a Kubernetes node object.

Good point, CAPI may consider this as abonormal state, and hold any mgmt operation until cluster to be stable (all etcd member mapped to node)

This should be handled by member promotion, and on the other side, the finalizer on node objects should prevent nodes from being deleted without being removed from etcd.
Can you provide steps to reproduce this?

It seems that the issue we were facing is due to stop VM before removing node, not etcd member leak. When we remove etcd first, and then stop VM and delete node, rollout seems to be ok.
Well, we do hope there is way for CAPI to know the etcd member list as another security check, but currently etcd controller doesn't surface such info.
Another idea is to expose the etcd member list as a node annotation, what do you think @brandond ?

It seems that the issue we were facing is due to stop VM before removing node, not etcd member leak. When we remove etcd first, and then stop VM and delete node, rollout seems to be ok.

You should be able to delete the node from the cluster (both the kubernetes node, and the etcd member) even when the node is offline. It may take a bit longer if you're waiting for the etcd controller lease to transition to another node to trigger the cluster member remove and cleanup of the finalizer on the node object... but it should succeed. Can you provide steps to reproduce whatever problem you're running into here?

Another idea is to expose the etcd member list as a node annotation

I'm not sure this makes sense to me. You'd have global etcd cluster state, stored as an annotation on individual node objects? If for some reason you need to check etcd cluster membership, I'd probably recommend just querying that directly from etcd (if you must), or via the supervisor API - which is available anonymously if accessed via localhost:
https://github.com/k3s-io/k3s/blob/917761ce54524b2156fd31ae33350ff19080b981/pkg/etcd/etcd.go#L701-L703
root@systemd-node-1:/# curl -kS https://<IP_ADDRESS>:6443/db/info
{"members":[{"ID":4682922252190157995,"name":"systemd-node-1-103abf0d","peerURLs":["https://<IP_ADDRESS>:2380"],"clientURLs":["https://<IP_ADDRESS>:2379"]}]}

You should be able to delete the node from the cluster (both the kubernetes node, and the etcd member) even when the node is offline. It may take a bit longer if you're waiting for the etcd controller lease to transition to another node to trigger the cluster member remove and cleanup of the finalizer on the node object... but it should succeed. Can you provide steps to reproduce whatever problem you're running into here?

steps:

create a 2 cp nodes cluster.
stop 1 VM
delete node

problem is: If stop VM first, then cluster will lose quorum directly, and there is no chance for deleting node. That's the reason why we need delete etcd member first before stop VM.
But if there is ghost member exists for some reason, then delete etcd member first may not be safe as well. However, i'm not able to reproduce the condtion which has ghost member now.

I'm not sure this makes sense to me. You'd have global etcd cluster state, stored as an annotation on individual node objects? If for some reason you need to check etcd cluster membership, I'd probably recommend just querying that directly from etcd (if you must), or via the supervisor API - which is available anonymously if accessed via localhost:

Indeed, that seems not quite right to surface etcd cluster level info on node.
Access etcd directly is the solution we take right now for capi k3s provider, see ref https://github.com/k3s-io/cluster-api-k3s/issues/75, but this is also not an ideal way. it requires manage etcd ca cert on mgmt cluster, and creating etcd proxy pod on target cluster, which is too heavy, besides, it bypasses k3s etcd controller to manage etcd cluster, that seems not quite right from design perspective.
Well, if etcd controller can expose etcd membership info, we shall be able to avoid the etcd direct access from capi mgmt cluster.

create a 2 cp nodes cluster

You should never have a 2 node cluster. As you noted, loss of either node will cause a total cluster outage, and you'll need to reset the etcd cluster to a single node to recover. This is what the --cluster-reset flag is for.

that's the reason why we need delete etcd member first before stop VM.

You can either delete the node object, or add the pre-delete annotation to the node to remove it from the cluster without deleting the node resource itself. This annotation is how Rancher prepares nodes for deletion.

You should never have a cluster with 2 etcd nodes. As you noted, loss of either node will cause a total cluster outage, and you'll need to reset the etcd cluster to a single node to recover. This is what the --cluster-reset flag is for.

2 cp node cluster is the intermediate state when we test scale down from N cp nodes to 1 by capi.

You can either delete the node object, or add the pre-delete annotation to the node to remove it from the etcd cluster without deleting the node resource itself. This annotation is how Rancher prepares nodes for deletion.

Thanks, we are using etcd.k3s.cattle.io/remove annotation. What else does pre-deletion do than removing etcd member?

That's all it does.
If you're scaling down to 1, you might consider just running a cluster-reset instead of stepping through all the intermediate node counts one by one.


Spanner Table are not roll backed when a transaction is aborted

Hello,
I am getting this behavior when I wrap the creation of a table and insertion inside a transaction.
The spanner transaction is aborted  but the the table  is not removed only the row inserted is removed.
Environment details

OS: Windows 7
.NET version: .Net 6.0
Package name and version:  Google.Cloud.Spanner.Data Version=3.15.1

Steps to reproduce
Run this code:
`
                        using (var trans = connection.BeginTransaction())
			{
				string createTableStatement =

				@"CREATE TABLE Venues (
                                        VenueId INT64 NOT NULL,
                                        VenueName STRING(100),
				) PRIMARY KEY (VenueId)";

				var cmd = connection.CreateDdlCommand(createTableStatement);

				cmd.Transaction = trans;

				cmd.ExecuteNonQuery();

				var dmlCommand = connection.CreateDmlCommand("INSERT INTO Venues(VenueId, VenueName) values(1, 'Test')");
				dmlCommand.Transaction = trans;
				dmlCommand.ExecuteNonQuery();

				throw new Exception("Something bad happened");

				trans.Commit();
			}`

Spanner transactions do not cover DDL operations, so the behaviour you observed is expected.
Google.Cloud.Spanner.Data let's you use SpannerCommand for DDL operations as a convenience, but we understand this gives the false impression of transactions being applicable to DDL commands. I'll send a PR shortly to document this behaviour clearly.

Thank you @amanda-tarafa for the clarification.


Added installation guide for Anugular 2.x and greater.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Discord Webhook to follow GitHub activities

As in SWE574-Fall2023-Group1/SWE574-Fall2023-G1#4
we may have Discord Webhook to follow GitHub activities for mobile repo too.

Done in discord 🤖

Cool ✨ thank you 👏🏼


Bump to Bevy 0.14.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Listbox, Accordion showing extra grey rect re-draw on state change in Chrome.

Current Behavior
Bingers found an issue on Chrome - more obvious in mobile sizes, but has been replicated at larger sizes. There is a grey rect drawing on state changes of items in the components.
https://user-images.githubusercontent.com/52020637/229342184-288ffcee-3a44-48b5-8a30-dea4ecd2a73f.mp4
Steps To Reproduce
No response
Anything else?
No response

Not a bug, this is a focus selection on mobile:
https://www.skeleton.dev/elements/core#mobile-focus
But we might be able to make show only on the element you select.

@niktek I can't seem to replicate the way this is being shown. I've noted my findings here:
https://discord.com/channels/1003691521280856084/1091997311607001099

@niktek I forgot to setup a PR so this was committed straight to the dev branch here: https://github.com/skeletonlabs/skeleton/commit/2cdf844537cfd852b492f5948541c51ce9d849b0
This was an odd one though - something about setting cursor-pointer -outline-offset-[3px] on the parent was causing this. I'm still not sure why. These styles were intended for the child items though, so I adjusted this. I also nixed an extra Label tag in the docs page that served no purpose.
As stated above though, the quick white flash should always be present on mobile as this is how touch devices handle the focus event. We just style it from Chrome's default blue to a more neutral white tone.


Share research?

I would like to try reversing the Linux library's private key, Would you like to share the research for reversing the windows dll?
For example methods used for detecting the code containing key, de obfuscation process etc...

Here are some resources that might help you.
Key is obtained via applying DFA on AES


What is the default font for hubpress?

Hi,
While I use chrome on both windows 7 and ubuntu 14.04,  my hubpress page rendering in different font.
:-)

Merriweather Light

Thankyou

No problem :grinning:


Add health checks

Changes

Sets up a handler for readiness/liveness probes to be enabled on demand in sharedMain. Moves work here from:
https://github.com/knative/serving/pull/13563
Checks if start up work has been done.

@dprotaso @evankanderson gentle ping for review.

We should not have the same problem as the controller-runtime issue, because we always start controllers, even if they don't win the leader election.
I agree that the check should start up and run independently of the controller startup, but it seems like we shouldn't return healthy until informer caches have been filled (which, for example, can be blocked by incorrect RBAC permissions).

I'd also advocate for having health endpoints always enabled.  If you don't want to use them, they should be minimal overhead.
Ideally, we could provide this same server to the Prometheus metrics implementation and share a port, but I don't want to force this behind refactoring that big ball of mud.

We should not have the same problem as the controller-runtime issue, because we always start controllers, even if they don't win the leader election.

We do but they dont own any bucket right? That technically means they are not ready no?

/assign @evankanderson

Will update the PR.

@evankanderson pls review. Updates:
a) Added a mechanism to register only specific probe types.
b) made the health handler easy to override
c) added unit tests
Not sure how queue proxy could utilize this because drainer is used as part of a handler chain. The scope of this for now is for regular deployments of Knative control plane but probably we can refactor further.

Yeah, I'm not deeply concerned about using this for queue-proxy or activator, which will both be a little funny, but I just didn't want to over-specialize and then have to walk it back.
At the same time, it would be nice if we could use this for e.g. eventing brokers, and have them fail readiness or liveness if they can't reach their storage backends.  That probably needs that we need something like an ability to register health checks in the injection.ControllerConstructor, which probably means adding the healthcheck registration(s) to the context before calling the constructors.

(Reviewing now)

I like the idea of enabling health probes -- is there any reason not to have them on all the time (it might simplify the code)?

The probes are on by default, the default ones are used.

I'd also sort of like to be able to say:

func NewController(ctx context.Context, cmw configmap.Watcher) *controller.Impl {
  ...
  checker := func (r http.Request) error {
    // pretend we have appropriate locking
    return impl.connectionError
  }

  // internally, calls getReadinessRegisterFunc(ctx).Register(checker)
  injection.AddReadiness(ctx, checker)

  return impl
}

Now you can say:
ctx = WithOverrideHealthCheckHandler(ctx, &HealthCheckHandler{
			ReadinessProbeRequestHandle: testHandler(),
			LivenessProbeRequestHandle:  testHandler(),
		})
networkpkg.ServeHealthProbes(ctx)

@evankanderson I simplified the impl. Pls review.

{"level":"error","ts":1675945698.819293,"logger":"fallback","caller":"consumergroup/reconciler.go:302","msg":"Returned an error","targetMethod":"FinalizeKind","error":"unable to delete the consumer group my.group.id: kafka server: The client is not authorized to send this request type","stacktrace":"knative.dev/eventing-kafka-broker/control-plane/pkg/client/internals/kafka/injection/reconciler/eventing/v1alpha1/consumergroup.(*reconcilerImpl).Reconcile\n\t/home/runner/work/pkg/pkg/downstream/control-plane/pkg/client/internals/kafka/injection/reconciler/eventing/v1alpha1/consumergroup/reconciler.go:302\nknative.dev/pkg/reconciler/testing.(*TableRow).Test\n\t/home/runner/work/pkg/pkg/downstream/vendor/knative.dev/pkg/reconciler/testing/table.go:160\nknative.dev/pkg/reconciler/testing.TableTest.Test.func1\n\t/home/runner/work/pkg/pkg/downstream/vendor/knative.dev/pkg/reconciler/testing/table.go:389\ntesting.tRunner\n\t/opt/hostedtoolcache/go/1.18.10/x64/src/testing/testing.go:1439"}

@evankanderson gentle ping.

@evankanderson pls review

/unhold

/hold is it ok to unhold?

Yes, I hadn't even realized there was still a hold.
I'll typically use a hold if I want to give the author a choice between merge now and change based on optional feedback.

@evankanderson @dprotaso pls take a look. Addressed the latest comment.
For the use of the PR check here: https://github.com/knative/serving/pull/13563.
One scenario not addressed here is what default-domain job uses:
	// Start an HTTP Server
	h := netprobe.NewHandler(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
		w.WriteHeader(http.StatusOK)
	}))
	//nolinlt:gosec https://github.com/knative/serving/issues/13439
	server := http.Server{Addr: ":8080", Handler: h, ReadHeaderTimeout: time.Minute}
	go server.ListenAndServe()

This is a special case as we need to modify the path, have only a common handler and use only the "/" as net-kourier probes that path only, any other will not work. This is left for a separate PR.

I think it's fine to add the option to change the path (register additional paths?) as a follow-up.
/lgtm

@dprotaso is it ok to unhold?

/unhold


Update what-is-text-to-speech-avatar.md for TTS Avatar support on East US 2

Avatar is enabled on East US 2.

#sign-off

#please-close


IOS ProgressBar Notifications are creating repeatedly

I am showing the download progress with following implementation:
AwesomeNotifications().createNotification(
              content: NotificationContent(
                id: 5,
                channelKey: 'download_status',
                progress: ((count / total) * 100).toInt(),
                title: "Downloading Video",
                notificationLayout: NotificationLayout.ProgressBar,
                locked: true,
              ),
            );

Everytime progress updates, it is showing notification again. It is only happening in IOS but working properly in android.

This is the current behavior for iOS, because iOS does not have a native progres bar notification.

I think it would be better to avoid showing repeating notifications in IOS and instead, show some message in debug console regarding it.
The reason is that, I didn't knew about this behavior in iOS and spent hours thinking about implementing it incorrectly.

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


FHIR Resource  - Storage

How will we be storing the FHIR resources

[ ] Repo
[ ] Sync from serverside
[ ] In-app

Related and being tracked in issue #55


Wrap ForStatement in a block for const decls

See https://github.com/jlongster/prettier/pull/169#issuecomment-272530265

👍

... How is it possible we have a flaky test


[Form - possible enhancement] Reset undefined fields in 'save' server response

I realize this is probably really marginal but I have this one expected behaviour that I'd like to be supported... eventually.
When a form is saved, Focus sends the entity to the server, gets the updated one from the server back and updates the form fields. In my case, according to one field value, some other fields may be deleted in the save action, and my server only sends back the actual existing fields.
The form only updates the fields it finds in the response and leaves the other be. That leads to impossible situations on the client side that can lead in server errors if I try to edit and save again.
One possible solution could be to store the form state in a dedicated state node (that would surely help for other things I guess), that can be entirely replaced easily.
(I found an 'acceptable' workaround for this and it's far from critical for my case.)

Ok, I agree with you. Thanks for your input.
We are currently working on the form to ease its use and separate references, store related informations (previous and new state), validation.

@JabX  Maybe the server should send null for a property which is no longer filled .

Linked to #708


Add kernel generator cast operation

Summary
Adds typecast operation to kernel generator and uses it in distributions instead of forcing type promotions by adding zero.
Tests
Added tests for cast. Reminder of this PR is just refactor.
Side Effects
None.
Release notes
Adds typecast operation to kernel generator.
Checklist

[ ] Math issue #(issue number)

[ ] Copyright holder: Tadej Ciglarič
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[ ] the code is written in idiomatic C++ and changes are documented in the doxygen

[ ] the new changes are tested

This is still waiting for a review.

@SteveBronder This PR is still waiting.


AuthenticationException occured when creating a SSHClient with user elyra

Use standard ansible scripts to setup the environment on a brand new stack/cluster with 4 nodes based on Redhat 7.3. Here are the steps from @sxguo to setup the environment on the master node:

Installing Ansible on RHEL

curl -O https://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm
rpm -i epel-release-latest-7.noarch.rpm
yum update -y
yum install -y ansible

Updating Ansible configuration

Add(read uncomment) the following configuration in /etc/ansible/ansible.cfg
[defaults]
host_key_checking = False
hash_behaviour = merge

Clone https://github.com/lresende/spark-cluster-install
Zip up and upload to master node
Edit hosts-fyre with node names/ips for your cluster
ansible-playbook --verbose setup-ambari-cluster.yml -i hosts-fyre

Once this is done, start Enterprise Gateway as shown below:
$ cd /opt/elyra/bin
$ start_elyra.sh

This will result in the following exception:
[E 2017-10-13 15:10:28.215 EnterpriseGatewayApp] Exception 'AuthenticationException'
occurred when creating a SSHClient connecting to '<IP_ADDRESS>' with user 'elyra',
message='Authentication failed.'.

Note that EG_REMOTE_USER is set to elyra in the /opt/elyra/bin/start_elyra.sh. Change the vaue of EG_REMOTE_USER to root in /opt/elyra/bin/start_elyra.sh and run it again. This time, it will launch Enterprise Gateway successfully.

The aforementioned steps requires changes from @kevin-bates's PR.
Till that ER is merged, you can instead use the attached tar and NOT perform the Cloning and Zipping/Uploading steps. Instead, you can copy the attached tar archive to the master node and just untar it as shown below:
$ mv ansible.zip ansible.tar
$ tar xvf ansible.tar

And, then continue with editing the hosts-fyre file with the names/ips of your cluster.
ansible.zip

The mentioned PR has been merged

Thanks @lresende!
This means that to be able to reproduce the issue, now one can use the steps listed in the original description as-is without needing the attached ansible.zip.

What user are you using when you try to start elyra ? I can think of two options :

You did not su to elyra user initially and some folders (e.g. log) are owned by root and elyra can't start
You might not have, for some reason, passwordless ssh configured.

Anyway, let me try to install a cluster from scratch and see what results I get.

@lresende -- @sxguo and I tried the steps that are specified in the original issue description. If we have a missed a step in that process, it would be good to know. We can also check our documentation to ensure that all the steps are specified.

I tried the steps listed in the original description on a pristine cluster/stack again without using the temporary ansible.zip as the PR has been merged. As @lresende correctly pointed out, the missing step is to do su elyra before starting the Enterprise Gateway.
Here are all the steps for firing up the Enterprise Gateway on a cluster:

SSH into the master node as root.
Installing Ansible on RHEL - Execute the following steps on the master node:

curl -O https://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm
rpm -i epel-release-latest-7.noarch.rpm
yum update -y
yum install -y ansible

Updating Ansible configuration on the master node

Add(read uncomment) the following configuration in /etc/ansible/ansible.cfg
[defaults]
host_key_checking = False
hash_behaviour = merge

git clone https://github.com/lresende/spark-cluster-install on your local machine
Zip up spark-cluster-install folder on local machine and upload the archive to the master node
Unzip the archive to create spark-cluster-install folder on the master node
cd spark-cluster-install on the master node
Edit hosts-fyre-spark with node names/ips for your cluster on the master node
Execute ansible-playbook --verbose setup-ambari-cluster.yml -i hosts-fyre-spark on the master node

The previous step will take sometime to install necessary bits on the all nodes of the cluster, set them for SSH login and such. Once ansible-playbook has been executed on the master node as root, the Enterprise Gateway should be started a shown below:
[root@master-node ~]# su elyra
[elyra@master-node root]$ cd /opt/elyra/bin
[elyra@master-node bin]$ ./start-elyra.sh

I will keep this issue open till we make sure that our documentation contains all the steps.


PHP-Webshell

Hello,
thanks for providing the scanner to the public.
In a customer projects regarding exploited netscalers, we found evidence of the creation of a PHP-webshell. The webshell was located at /var/vpn/themes/admin.php and simply eval'd $_POST[1]:

The attacker could thus simply POST commands to https://[server]/vpn/themes/admin.php.
Best regards

?

The original webshell posted above should be caught by the following pattern:

As @sebnerz pointed out, the best way to handle this detection is via whitelist; unfortunately, we don't have access to a comprehensive list of expected files and their hashes across all versions. During initial development, we explored this option, but decided against it because we couldn't test against a large enough fleet of prod netscalers. This remains the case :-(


Add papis: 0.5.0

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Need help here. It does not find configparser though the dependency is clearly in scope.

BTW it was surprising to discover that in Nixpkgs lib.platforms.unix == lib.platforms.all. As a matter of style, I think all is preferred in all cases.

Hi, I just realized this issue was open, #34761 solves it, can be closed


device registry hotfixes

Is your feature request related to a problem? Please describe.

[x] Add country, district, and region to the sites summary endpoint
[x] Having this error when I use v2 to get running devices

NetworkSchema
airqlouds: [
      {
        type: ObjectId,
        ref: "AirQloud",
      },
    ]

airqloudSchema
    networks: [
      {
        type: ObjectId,
        ref: "Network",
      },
    ],

@Codebmk , closing this one. Transferring all unfinished ones to the new card


Dictate DispatchQueue Choice at Init

Was trying to use this in a server-side swift project but unfortunately we don't have an option with Swifter right now to dictate which thread is being used...
Right now it's hardcoded to .main:
DispatchQueue.main.async {
  let session = URLSession(configuration: .default, delegate: self, delegateQueue: .main)
  self.dataTask = session.dataTask(with: self.request!)
  self.dataTask?.resume()

  #if os(iOS)
  UIApplication.shared.isNetworkActivityIndicatorVisible = true
  #endif
}

I promise we init Swifter with the ability to choose which DispatchQueue is used

I guess this was just always the case since Matt wrote the original Swifter 6 years ago (wow! 6). I don't see why we can't change it to allow for different queues. Feel free to put up a PR


Program Committee

We are looking for volunteers to serve as the program committee! Everyone is welcome!
It would be great if we could get at least one member of the program committee from each of the Nordic-Baltic countries with active Carpentries instructors/trainers/community managers.
If you would like to become part of the team, please indicate below as a "comment" @ yourusername
To learn more about CarpentryConnect events and how they are planned, organized, and run, take a look at the CarpentryConnect handbook

@naoe-tatara will be a part of PC :)
The first Nordic RSE conference was postponed to May 2021 in Stockholm, and it will be great if we could make this as a joint event so that effort needed for logistic part of planning could be reduced.

I am interested being part of this @wna-se


Separate graph identifier from graph driver

The graph holds identifiers used to reference images.
Each of these images contains chain of layers which are each referenced by the graph with content cached on disk by the graph driver.
Using a separate ID for the graph driver allows the IDs of the images referenced by the graph to change without modifying the on disk content.
The graph driver IDs should be considered private and never exported regardless of the identifier used by the graph.
Note:
This patch keeps the 1-1 relationship between graph directories and what is stored in the graph driver. Collisions are no more likely than a collision with any other identifier, and their relationship could only be broken through manual intervention.

I don't think this is relevant anymore. Please let me know if I'm mistaken.


chromedriver,{ungoogled-,}chromium: 127.0.6533.119 -> 128.0.6613.84

Description of changes
https://chromereleases.googleblog.com/2024/08/stable-channel-update-for-desktop_21.html
This update includes 38 security fixes. Google is aware that an exploit
for CVE-2024-7971 exists in the wild.
CVEs:
CVE-2024-7964 CVE-2024-7965 CVE-2024-7966 CVE-2024-7967 CVE-2024-7968
CVE-2024-7969 CVE-2024-7971 CVE-2024-7972 CVE-2024-7973 CVE-2024-7974
CVE-2024-7975 CVE-2024-7976 CVE-2024-7977 CVE-2024-7978 CVE-2024-7979
CVE-2024-7980 CVE-2024-7981 CVE-2024-8033 CVE-2024-8034 CVE-2024-8035
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on x86_64-linux 1
31 packages built:

aws-azure-login
chromedriver
chromium
chromium.sandbox
fast-cli
flaresolverr
gotenberg
grafana-kiosk
mermaid-cli
mermaid-filter
percollate
playwright-test
puppeteer-cli
python311Packages.pytest-playwright
python311Packages.pytest-playwright.dist
python311Packages.shiny
python311Packages.shiny.dist
python312Packages.pytest-playwright
python312Packages.pytest-playwright.dist
python312Packages.shiny
python312Packages.shiny.dist
selendroid
selenium-server-standalone
sharedown
single-file-cli
sitespeed-io
undetected-chromedriver
ungoogled-chromium
ungoogled-chromium.sandbox
vhs
wayback


20779 Added the alternate name interface

Issue #: /bcgov/entity#20779
Description of changes:

Moved the alternate name interface to the shared interfaces

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the business-filings-ui license (Apache 2.0).

@severinbeauvais Sev, let me know if you want me to change anything. Will open another PR in that case.

@severinbeauvais Sev, let me know if you want me to change anything. Will open another PR in that case.

Looks great!


fix: Fix for ubuntu runner

#8  👈
main

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

Join @tazarov and the rest of your teammates on  Graphite


Import statement moved while files got merged

Commit: https://github.com/gwtproject/gwt/commit/e0dda9f61b7c409944c4734edf75b108e0288f59
File: MessageDigest.java
Import statement Moved from one file to another and the original file got merged into the new file.

[ ] Fix the issue
[ ] Updating the oracle

Also found: https://github.com/brettwooldridge/HikariCP/commit/1571049ec04b1e7e6f082ed5ec071584e7200c12
File: HikariPool.java

Also https://github.com/brettwooldridge/HikariCP/commit/cd8c4d578a609bdd6395d3a8c49bfd19ed700dea
File: ClockSource.java


Copy-paste is broken

http://hackety-hack.com/forums/clubhouse/feature_requests

We maintain our own clipboard. This is the root of the issue.


Variable substitution replaces all variables, defined or not

When using the kustomize variable substitution with envsubst, kustomize will replace values as empty if they are not defined, which might be on purpose.  Using the substitution to set some values but not others can break workloads, including the example cronjob to sync ecr credentials:
# kustomization
  ...
  postBuild:
    substitute:
      cluster_env: dev

But then the workload that should have the following command:
- command:
  - /bin/sh
  - -ce
  - aws ecr get-login-password --region ${REGION} > /token/ecr-token
  env:
  - name: REGION
    value: us-east-1

ends up being created in the cluster as:
- command:
  - /bin/sh
  - -ce
  - aws ecr get-login-password --region > /token/ecr-token
  env:
  - name: REGION
    value: us-east-1

Please read the docs, this behavior has been documented and we made suggestions on how to change your configs: https://toolkit.fluxcd.io/components/kustomize/kustomization/#variable-substitution


Fixes the regex to allow for dashes in usernames.

Currently the - in the below regex doesn't actually work. It needs to be escaped to register.

Thanks! Pushed as 1.1.7.


Fix ntfy upgrade banner in dark mode

Codecov Report
Patch and project coverage have no change.

Comparison is base (64ac111) 65.85% compared to head (d838790) 65.85%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #792   +/-   ##
=======================================
  Coverage   65.85%   65.85%
=======================================
  Files          53       53
  Lines        7646     7646
=======================================
  Hits         5035     5035
  Misses       1775     1775
  Partials      836      836

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Change x

Some of my friends want to ask about changing or replacing their x accounts, because many have been suspended and I tried it myself but it doesn't work via mobile.

Please ask about Renaissance in the Discord channel: https://discord.com/channels/1143699396500934796/1242322885633118218.

Thanks


Allow specifying buffer size when creating AKAmplitudeTap

This PR makes the bufferSize in AKAmplitudeTap configurable at creation time. This is analogous to AKFFTTap; however, unlike the solution there which takes an fftSize: AKSettings.BufferLength argument, my proposal allows specifying arbitrary bufferSizes.
This can be useful when you want to track amplitude in a specific time interval, for example, every 10ms.

Thanks!


Create NFTs examples with related docs

Description
Create examples for the NFTs section. It should contain:

[ ] Burn
[ ] Transfer
[ ] Transfer with Storage Deposit Return and Expiration
[ ] Claim
[ ] Claim under conditional Transfer
[ ] Introduction
[ ] Mint
[ ] Create NFT collections

Are you planning to do it yourself in a pull request?
Yes, someone from out team will work on it.

I'll try my hand here

[ ] Read NFT metadata
[ ] Query all NFTs of a collection
[ ] Query addresses owning a certain NFT collection

Status update on this:

Rust/NodeJs bindings are done.
Ani is currently figuring out the problems he has with running/using the Python bindings.


Laravel 5.4 issue

I recently upgraded to laravel 5.4 and when I try to load a page, I get the following error:
L A R A V E L  E C H O  S E R V E R

version 1.2.4

⚠ Starting server in DEV mode...

✔  Running at heroes.derek.app on port 6001
✔  Channels are ready.
✔  Listening for http events...
✔  Listening for redis events...

Server ready!

[10:13:57 PM] - JdcgyEqh9AWXkv4rAAAA authenticated for: private-App.Notify.1
undefined:1

SyntaxError: Unexpected end of JSON input
    at Object.parse (native)
    at Request._callback (/usr/lib/node_modules/laravel-echo-server/dist/channels/private-channel.js:45:34)
    at Request.self.callback (/usr/lib/node_modules/laravel-echo-server/node_modules/request/request.js:186:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/usr/lib/node_modules/laravel-echo-server/node_modules/request/request.js:1081:10)
    at emitOne (events.js:96:13)
    at Request.emit (events.js:188:7)
    at IncomingMessage.<anonymous> (/usr/lib/node_modules/laravel-echo-server/node_modules/request/request.js:1001:12)
    at IncomingMessage.g (events.js:291:16)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Please help, I have done tons of googleing and gotten nowhere. Has anyone else experienced this issue?

There is a probably a problem with your Laravel application.
Can you POST to the authentication endpoint /broadcasting/auth?

@tlaverdure the response back from the server when laravel-echo-server posts all the data is a 200 response with an empty body. I can post that response output if you'd like?

Is it private or presence?
Presence expects an array with info like user ID and name.

It is a private channel I am attempting to access. I'm using Echo.private()

Here is a screenshot of me POSTing with vue:

But again, the response is blank.

What does your broadcast check look like for that channel, are you returning true? Are you sure it's called?

I am using the default broadcast check:
Broadcast::channel('App.User.*', function ($user, $userId) {
        return (int) $user->id === (int) $userId;
 });

And I'm also getting the same error with a fresh installation of laravel 5.3

And it reaches that statement? Eg. When you add a log statement inside the check, it's triggered?

That may be your issue,private-App.Notify.1 and private-App.User.1 are not the same channel. You would need  separate check for that private channel.
I would check out the section in the docs on Authorizing Channels

@tlaverdure neither of the channels are working, I switched to using the default to see if it made a difference
@barryvdh  - no, it doesn't reach my log message

I suggest some higher logging then, to see which request is actually sent.

My BroadcastServiceProvider:
...
class BroadcastServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        Broadcast::routes();

        /*
         * Authenticate the user's personal channel...
         */
        Broadcast::channel('App.User.*', function ($user, $userId) {
            Log::info("Yes, we are here");
            return (int) $user->id === (int) $userId;
        });
    }
}
...

My bootstrap.js:
...
import Echo from "laravel-echo"

window.Echo = new Echo({
    broadcaster: 'socket.io',
    host: window.location.hostname + ':6001'
});

My initiation of the code inside a vue component:
<template>

Example Component

                        I'm an example component!

</template>

Auth check in 5.4 also doesn't use * per the upgrade guide.
https://github.com/laravel/laravel/blob/master/routes/channels.php#L14

Sorry, this is 5.3 now as aforementioned, and for private channels, it still utilizes the * in 5.4

I tried to rollback and see if the version change was the issue and I'm still getting the same error.

I can teamview if you'd like to take a look at it. It's baffled me for about 3 days now.

You;re sure you fully reverted to 5.3 (composer, compiled file, cache etc)? Did you try the alternative syntax?
I suggest logging the entire request to determine where it goes wrong and then re-try that with Postman or something.
I think it's on the Laravel side, unless the authentication url is not correctly passed.

@barryvdh to be safe, I completely started with a fresh install of Laravel 5.3:
Here is the example code with a fresh installation of laravel 5.3: https://github.com/dpollard/test_laravel_echo_server
It is also productionized js, so all that needs done is a new laravel-echo-server init in the repo, if you'd like to test it out

@barryvdh how would you suggest I test Laravel? I don't really know where to begin

You could add log statements in your middleware or routes files. Or something like github.com/barryvdh/laravel-debugbar and view the request with the browse button

So I converted over to pusher, and got it to work with pusher. Is that indicative of anything?

Any ideas, kind of desperate...I have installed debugbar but really, the connection doesn't stay open long enough to debug anything

Also getting this problem.

Are you both using the Redis broadcaster? Or pusher?

I am using redis

@dpollard can you try installing 1.2.5.
This may or may not fix it, but hopefully so. https://github.com/tlaverdure/laravel-echo-server/commit/ec4277a383755d4141c72c3c9d6eb107f83c56cf

Hot diggity damn, it works!

You guys are awesome!

Good to hear!


Epoll: Avoid redundant EPOLL_CTL_MOD calls

Motivation
Currently an epoll_ctl syscall is made every time there is a change to the event interest flags (EPOLLIN, EPOLLOUT, etc) of a channel. These are only done in the event loop so can be aggregated into 0 or 1 such calls per channel prior to the next call to epoll_wait.
Modifications
I think further streamlining/simplification is possible but for now I've tried to minimize structural changes and added the aggregation beneath the existing flag manipulation logic.
A new AbstractChannel#activeFlags field records the flags last set on the epoll fd for that channel. Calls to setFlag/clearFlag update the flags field as before but instead of calling epoll_ctl immediately, just set or clear a bit for the channel in a new bitset in the associated EpollEventLoop to reflect whether there's any change to the last set value.
Prior to calling epoll_wait the event loop makes the appropriate epoll_ctl(EPOLL_CTL_MOD) call once for each channel who's bit is set.
Result
Fewer syscalls, particularly in some auto-read=false cases. Simplified error handling from centralization of these calls.

@netty-bot test this please

@normanmaurer have pushed a commit addressing your comments. I made the Runnable lazily constructed, not sure if that's what you had in mind or just a final. And still uncertain about the change since it's an extra object per channel and I guess repeated calls to setAutoRead(false) for a given channel aren't common?

@netty-bot test this please

@netty-bot test this please

@netty-bot test this please


Lacking Documentation on how to use library

Brian Just started reading through your paper and decided to take a look at the code (rather than just read a paper that describes the code.r)
Your documentation (README) is a bit lacking on how to use your library. While an experienced developer can grok the code and make heads or tails of it; your best bet would be to include code samples of usage and explanation of your design patterns.
This is a great example of the level of detail (while somewhat overkill) for a good project:
http://kenwheeler.github.io/slick/
https://docs.angularjs.org/api
EDIT: I had a second look. I noticed you commented the code (which I think is fantastic) however, most practitioners would dive into the read me and clearly (markdown) readable usage.

Thanks for your interest in this package.
Is there something specifically unclear about the examples in the documentation?
Note: documentation is a great way to contribute to open source, and I'd be happy to accept improvements via pull request.

Actually, that should do it! (I missed out on the docs icon on the page, my
mistake!)
thanks!
On Wed, Dec 16, 2015 at 2:58 PM, Brian McFee<EMAIL_ADDRESS>wrote:

Thanks for your interest in this package.
Is there something specifically unclear about the examples in the
documentation http://muda.readthedocs.org/en/latest/?badge=latest?
Note: documentation is a great way to contribute to open source, and I'd
be happy to accept improvements via pull request.
—
Reply to this email directly or view it on GitHub
https://github.com/bmcfee/muda/issues/23#issuecomment-165225951.

Happy to help.  I'll close this one out then.


prev state vs next state

createReducer infers the next state from the actions instead of using the previous state, and I wonder why that is, because it creates weird types on the State interfaces.
For example, when I try this
For example, with the following sample code:
import { createReducer, createAction } from 'deox';

interface TestType {
    foo: string;
    bar: number;
}

interface TestState {
    [s: string]: TestType;
}

const initialState: TestState = {};

export const addTest = createAction('ADD_TEST', (resolve) => (id: string, type: TestType) => resolve({ id, type }));

const testReducer = createReducer(initialState, (handleAction) => [
    handleAction(addTest, (state, action) => ({
        ...state,
        [action.payload.id]: {
            ...state[action.payload.id],
            ...action.payload.type,
        },
    })),
]);

type CombinedState = ReturnType<typeof testReducer>;

I get this as CombinedState (when inspecting via vscode):
type CombinedState = {
    [x: string]: TestType | {
        foo: string;
        bar: number;
    };
}

And this gets worse with more reducers and actions.
Should the initialState not have the same type definition as anything that comes out of the reducer? Why the separation of both?
Obviously, I can create a wrapper function for this:
function createReducerClean<TPrevState, THandlerMap extends HandlerMap<TPrevState, any, any>>(defaultState: TPrevState, handlerMapsCreator: (handle: CreateHandlerMap<TPrevState>) => THandlerMap[]): (state: TPrevState | undefined, action: InferActionFromHandlerMap<THandlerMap>) => TPrevState {
    return createReducer(defaultState, handlerMapsCreator);
}

But aside from this being additional maintenance work, I wonder why this is done in the first place.
Btw, thanks for deox, it helps a lot.

createReducer infers the next state from the actions instead of using the previous state

In fact, createReducer infer next state type from handlers map which in each handler the next state should be extended from the previous one. So if we make the relation simpler the next state type produces by createReducer should just extend the previous state in Deox. But why not just <State>(state: State, action) => State? Is there any difference between the previous state type and next state type?
The previous state that given to a reducer can be:

mutable - in this case, the next state just can be mutable and type signature is <State>(state: State, action) => State.
immutable - in this case, the next state can be mutable or immutable so the type signature becomes <State>(state: Immutable<State>, action) => State | Immutable<State>.

In the second case, you can see the next state type can be different from the previous state type.
But there is an issue with the type signature in the second case. Imagine the following example:
type NamesState = DeepImmutableArray<string>

const defaultNamesState: NamesState = Object.freeze([])

const namesReducer = createReducer(defaultNamesState, handleAction => [
  handleAction(actionB, (state) => state) // DeepImmutableArray<string>
])

namesReducer(...) // DeepImmutableArray<string>

What if the reducer type in the above example was <State>(state: Immutable<State>, action) => State | Immutable<State>? The next state type instead of being DeepImmutableArray<string> will become string[] | DeepImmutableArray<string> and that was wrong. The TypeScript, in that case, will lie to the developer. So unlike other type signatures for reducer which have been defined in Redux or other projects, Deox considers the difference between the previous state and next state but they have relation, the next state should extend the previous one.
BTW, the whole history behind this can be found in #55 and #58.

Thanks for the explanation.
I tried reproducing your example, but I keep getting DeepImutableArray<string> for CombinedType here:
import { createAction, DeepImmutableArray, Immutable } from 'deox';
import { HandlerMap, CreateHandlerMap, InferActionFromHandlerMap, createHandlerMap } from 'deox/dist/create-handler-map';
const merge = <T extends {}>(...objs: T[]): T =>
  Object.assign({}, ...objs)

export function createReducer<
  TPrevState,
  THandlerMap extends HandlerMap<TPrevState, any, any>
>(
  defaultState: Immutable<TPrevState>,
  handlerMapsCreator: (handle: CreateHandlerMap<TPrevState>) => THandlerMap[]
) {
  const handlerMap = merge(...handlerMapsCreator(createHandlerMap))

  return (
    state: Immutable<TPrevState> = defaultState,
    action: InferActionFromHandlerMap<THandlerMap>
  ): TPrevState | Immutable<TPrevState> => {
    const handler = handlerMap[action.type]

    return handler ? handler(<any>state, action) : state
  }
}
type TestState = DeepImmutableArray<string>

const initialState: TestState = Object.freeze([])

export const actionA = createAction('ACTION_A', (resolve) => (foo: string) => resolve(foo));
export const actionB = createAction('ACTION_B');

const testReducer = createReducer(initialState, (handleAction) => [
  handleAction(actionA, (state, { payload }) => [...state, payload]), // string[]
  handleAction(actionB, (state) => state) // DeepImmutableArray<string>
]);

type CombinedState = ReturnType<typeof testReducer>;

The original createReducer does show DeepImmutableArray<string> | string[] instead, which is wrong according to your comment.. or did you mix up that sentence?

Tried simplifying this even further by removing all referenced to Immutable* in createReducer and it still returns the same type:
import { createAction, DeepImmutableArray } from 'deox';
import { CreateHandlerMap, InferActionFromHandlerMap, createHandlerMap, HandlerMap } from 'deox/dist/create-handler-map';
import { Handler } from 'deox/dist/types';
const merge = <T extends {}>(...objs: T[]): T =>
  Object.assign({}, ...objs)

export function createReducer<
  TPrevState,
  THandlerMap extends HandlerMap<TPrevState, any>
>(
  defaultState: TPrevState,
  handlerMapsCreator: (handle: CreateHandlerMap<TPrevState>) => THandlerMap[]
) {
  const handlerMap = merge(...handlerMapsCreator(createHandlerMap))

  return (
    state = defaultState,
    action: InferActionFromHandlerMap<THandlerMap>
  ): TPrevState => {
    const handler = handlerMap[action.type]

    return handler ? handler(<any>state, action) : state
  }
}
type TestState = DeepImmutableArray<string>;

const initialState: TestState = Object.freeze([])

export const actionA = createAction('ACTION_A', (resolve) => (foo: string) => resolve(foo));
export const actionB = createAction('ACTION_B');

const testReducer = createReducer(initialState, (handleAction) => [
  handleAction(actionA, (state, { payload }) => [...state, payload]), // string[]
  handleAction(actionB, (state) => state) // DeepImmutableArray<string>
]);

type CombinedState = ReturnType<typeof testReducer>; // DeepImmutableArray<string>

Your other sample from #55 throws an error like expected with the above createReducer:
  handleAction(actionB, (state) => Object.freeze(state)) /* throws error
                                                            because of that input/prev state
                                                            defined as a mutable array
                                                            but the output/next state is a `ReadonlyArray` */
//...
type CombinedState = ReturnType<typeof testReducer>; // string[]

The next state type instead of being DeepImmutableArray will become string[] | DeepImmutableArray and that was wrong.

I meant here that the correct type is DeepImmutableArray<string>. So string[] | DeepImmutableArray<string> is wrong. Sorry if it is confusing.
Your 1st comment after mine:

The original createReducer does show DeepImmutableArray | string[] instead, which is wrong according to your comment.. or did you mix up that sentence?

This is a correct type in the tested case because you are really producing string[] or  DeepImmutableArray<string> in handlers. But even your handlers only return DeepImmutableArray<string> the reducer's return type still become string[] | DeepImmutableArray<string> which is wrong.
Your 2nd comment after mine:

Tried simplifying this even further by removing all reference to Immutable* in createReducer and it still returns the same type:

When the default reducer state is mutable there is no problem. But if the default reducer state you give to the reduce is immutable you become obligated to return immutable data structure in handlers. To clarify, you can test the following example:
type NamesState = DeepImmutableArray<string>

const defaultNamesState: NamesState = Object.freeze([])

const namesReducer = createReducer(defaultNamesState, handleAction => [
  handleAction(actionB, (state) => [...state]) /* throws error
                                                  because the expected return type of handler
                                                  is `DeepImmutableArray<string>`
                                                  but it's returning `string[]` */
])

namesReducer(...) // DeepImmutableArray<string>

I meant here that the correct type is DeepImmutableArray<string>. So string[] | DeepImmutableArray<string> is wrong. Sorry if it is confusing.

OK, so I read it correctly the first time.

This is a correct type in the tested case because you are really producing string[] or DeepImmutableArray<string> in handlers. But even your handlers only return DeepImmutableArray<string> the reducer's return type still become string[] | DeepImmutableArray<string> which is wrong.

Actually, no.. my latest sample does have a return type of DeepImmutableArray<string>. That's what confused me in the first place, as the latest version of deox (2.1.0) returns  string[] | DeepImmutableArray<string>, which you just said is wrong. Here you can see that the simplified reducer returns the correct type:

When the default reducer state is mutable there is no problem. But if the default reducer state you give to the reduce is immutable you become obligated to return immutable data structure in handlers. To clarify, you can test the following example:

That sample throws no error for me:

@SantoJambit I updated the comment.

That sample throws no error for me:

This is because you are using the new version of Handler type:
https://github.com/thebrodmann/deox/blob/65e4c1bc11c098fac499eca5b6e2d4a1d868d172/src/types.ts#L38-L42

I provided a codesandbox for testing the return type of reducer.

Thanks for the code sandbox, that helped to make it click. I get the following types with 2.1.0:
type MixedReducerReturnType = ReturnType<typeof mixedReducer>; // string[] | DeepImmutableArray<string>
type MutableReducerReturnType = ReturnType<typeof mutableReducer>; // string[]
type ImmutableReducerReturnType = ReturnType<typeof immutableReducer>; // DeepImmutableArray<string>

Could not find a solution that creates better return types and still works with the above :\


Properties Descriptor finder bug fixes and a test

Adding parsing of plugin.description and plugin.requires to the properties finder, and added a test

Coverage increased (+2.7%) to 17.875% when pulling 047b8d9c75ca3e403d062a4da3aab2dbe17aa532 on cominvent:properties-descriptor-finder into e9126c4d358ace2922382ea0829df493595b29e4 on decebals:master.

Coverage increased (+2.7%) to 17.875% when pulling 26cbed53d9b4d3f293f178c01e173bdab374545e on cominvent:properties-descriptor-finder into e9126c4d358ace2922382ea0829df493595b29e4 on decebals:master.

Thanks!


Deployment errors

I created a separate docker container with only the aqueduct environment.
Then i uploaded the source code to the server, ran it in the container through the mapped directory, and got the following error

It looks like it's a pub-cache path error.
".pub-cache" directory should be (/root/.pub -cache) in container
How to solve this problem,tks

Is the Dockerfile located here not sufficient: https://github.com/stablekernel/kubernetes/tree/master/aqueduct/docker?

Thank you for reply!
But that image is not what I want.
My understanding of the ADD. /app command is that it adds the project files in the current directory to the image.
When I make changes to the file, I need to rebuild an image.
What I want is to specify the project directory by VOLUME ["/app"]
So when I make a change to the file, I just restart the container.
My problem I think I can solve slowly by manually adding dependent libraries to the appropriate directory.
If there is no better way, you can close this question,Thank you!


As a server admin, I want to host multiple authors on my server

 Issue by kdhaywar
Tuesday Jan 19, 2016 at 17:16 GMT
Originally opened as https://github.com/shawnwanderson/cmput404-project/issues/13

This one is done


Takes so many saves to work

Maybe I'm doing something wrong? I click the collection I want to import, click N - Upload. Sometimes I'll pick to upload to my personal account, sometimes to our game group. Either way, it takes 10-30 minutes to upload. By then, I'm out of patience and time, so I give up and try again another day, just to still struggle. No errors. Nothing. Just never shows up. Yes, same issue with the latest release. I even change file name, and still have the same issue.

Updated to Blender 4, and now it's talking about bones?.. I'm thinking this plug in may be too outdated now, since Blender keeps updating so often. Was amazing for the few times I was able to use it.

Updated to Blender 4, and now it's talking about bones?.. I'm thinking this plug in may be too outdated now, since Blender keeps updating so often. Was amazing for the few times I was able to use it.

https://github.com/Roblox/roblox-blender-plugin/issues/32

Can you download the latest release (which fixes the 4.0 issue) and confirm if you are still having problems?
https://github.com/Roblox/roblox-blender-plugin/releases/tag/v1.0.2

Someone else encountered the same 'No module named roblox-blender-plugin-1 issue. Let's track that one here: https://github.com/Roblox/roblox-blender-plugin/issues/39
Please add to that issue if you're experiencing that new problem.
Closing this issue since the other issues seem to be resolved by updating the plugin to v1.0.3.


Drag of link should discard initial popup hovered by that link

Right now if I try to drag a link from a source that doesn't require ctrl to hover, the popover appears during the drag and obscures the underlying (main) editor pane that I'm trying to drag to.  Probably watching for a dragstart event on the element that triggered the hover-link event would allow detecting that as a case to ditch the initial popover.

An additional note: this could probably be done by checking for a mousedown outside the popup, since either a click or drag outside an unpinned popup should close it.

Hmm, I already have a mousedown handler doing this but you have to click before the trigger delay (200ms by default but customizable in settings). Where are you seeing this happen specifically?
I suppose I could make the click handler auto hide the popover if a click is detected within a certain amount of time to make it even less likely to accidentally show a popover.

What I'm seeing happen is: I try to drag a link from a source that doesn't require the ctrl key to hover.  No matter how quickly I attempt to drag, the popup shows up and doesn't go away until I finish or cancel the drag, because mouseover, mouseenter, etc. events don't fire during drag-drop: only dragover, dragenter, etc.  So the popover hangs around because it doesn't see a mouseover until after the drag is finished.
In any event, it seems to me that if a popup isn't pinned and you mousedown anywhere outside of it, it should always go away, no matter how long it's been since you began the hover.  If you need to drag something into the popover, you can always pin it first (or turn on auto-pinning), but if you're dragging something elsewhere, there is no way to get the popover out of the way.
(To be fair, I have checked and it appears Obsidian's builtin popovers actually have this same problem...  but then this would just be another way this plugin is superior to the builtin popovers.  :wink:)

Hmmm, yeah, that's fair. I suppose I can keep the abort click handler around for the life of the popover... or until it gets pinned.
If you're seeing this on QE then part of the problem is that my existing click handler is on the popover targetEl which I don't believe you're setting... so it doesn't even try to do the logic I was talking about above. I can switch the handler to being on the document body though to solve the targetless popover issue.

No, I hadn't actually tried it from QE, this was just dragging a link from one markdown view (in reading mode) to another (in editing mode).  But that sounds like a good idea.
As far as I know, the target element is indeed optional by design in Obsidian core: Obsidian checks on all code paths that it is a truthy value before using it for any dom stuff.  The reason QE omits it is because otherwise the popovers close immediately when you are using the keyboard.  (As the mouse is not necessarily on the targetEl when triggered.)
I see you've implemented this in 0.3.5, so I will close this.


Uncaught Exception: listen EADDRINUSE <IP_ADDRESS>:5672

FYI: GPMDP can't find any audio devices and won't play any songs.
An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.1.0
Time: Sat Apr 09 2016 14:24:26 GMT-0600 (Canada Central Standard Time)
Error:
listen EADDRINUSE <IP_ADDRESS>:5672
Error: listen EADDRINUSE <IP_ADDRESS>:5672
    at Object.exports._errnoException (util.js:856:11)
    at exports._exceptionWithHostPort (util.js:879:20)
    at Server._listen2 (net.js:1234:14)
    at listen (net.js:1270:10)
    at net.js:1379:9
    at doNTCallback3 (node.js:476:9)
    at process._tickCallback (node.js:382:17)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #325


VisData messages queued in webworker aren't cancelled when changing trajectories

Description
It is possible to have queued agent data from a previous trajectory still arriving (and rendering!) AFTER changing trajectories.
I already diagnosed this:  the POINTS TEST returns data on a very fast interval, and each message gets queued in a webworker.  Because so many get queued, and yet the webworker has to do a lot of processing, the worker just keeps chugging away long after you change trajectories, and when the worker finishes, it sticks its data in the VisData frame cache ready for updating in the scene.
Expected Behavior
Changing trajectories should completely forget about the previous one.
Reproduction
Load example viewer.
Select the POINTS TEST and click Load model.
Press Play.
Press Pause.
Fairly quickly, change to "Medyan test" and Load model.
See the rendered display continue to draw frames from the POINTS TEST.

Ooh, wonder if fixing this would take care of this bug? https://github.com/allen-cell-animated/simularium-website/issues/160


Update bspacks README instructions

I just followed errors & updated README when I successfully build the thing.
Well almost, now I have proper error during compilation, not error because I cannot even build the packed version :)

Hey @MoOx, I added some comments on your diff. Would be good to get those fixed before this is merged.
Additionally, the tests are failing for totally unrelated reasons. This is not important because you only changed a markdown file, but if you want to have a green build you can rebase on master where all of it is fixed.

Tell me if that's better now :)

I think #2495 supersedes this one now.


Add new docs Versions page

Fixes #26883.

Looks Good!

Tests seem to fail.

@mdo: is this ready from your side?

@XhmikosR Thanks for the squash and the ping! <3

@mdo: shouldn't the link here be {{ site.baseurl }}/docs/{{ site.docs_version }}/versions/? Or you plan to move this to the root? I just have a feeling this isn't right.

@XhmikosR If can be that. I was copy-pasting stuff across multiple versions and was being lazy with the hard coding.

All right, I thought so too. I added a patch in #27408


Next button is disabled when trying to invite person to workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Regression from https://github.com/Expensify/App/pull/15672
BUG ‘Next’ button is disabled on back navigation even if there is a selected user; as a result user cannot continue to invite the selected user after navigating back
Action Performed:
Click on ‘Profile’ avatar
Click on ‘Workspaces’ link
Open an existing workspace
Click on ‘Manage members’ link
Click on ‘Invite’ button
Enter email address > select user > click on ‘Next’ button
Click on back navigation icon
Expected Result: Next button shouldn’t be disabled as long as there is a selected user
Actual Result: Next button is disabled even thought there is a selected user
Issue 2
Alluded to by @bondydaa https://github.com/Expensify/App/pull/15672#issuecomment-1533840472
Action Performed:
Log in with account that has access to the policy-expense-chat beta
Go offline by navigating to [profile icon] > Preferences > and enable Force offline
Create a new workspace.
Invite some people to the workspace. Verify that workspace chats for each member appear optimistically.
Go online by navigating to [profile icon] > Preferences > disable Force online.
Expected Result:
Workspace chats created for each members are updated accordingly
Actual Result:
??? Something else ???
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number:
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @natha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1683077956552549
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01200c513fe2717ddb
Upwork Job ID:<PHONE_NUMBER>073346048
Last Price Increase: 2023-05-04

Triggered auto assignment to @strepanier03 (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Job added to Upwork: https://www.upwork.com/jobs/~01200c513fe2717ddb

Current assignee @0xmiroslav is eligible for the Internal assigner, not assigning anyone new.

Working on this now.

For issue 1, I cannot recreate the behavior. On staging it works just fine for me, the Next button doesn't get disabled after clicking the back navigation icon.

For issue 2, I'm working on testing this but need to ensure my accounts are part of the beta. Will update once I'm done.

Correct this should be now fixed in staging 🎉

@abdulrahuman5196 @0xmiroslav this was a regression from https://github.com/Expensify/App/pull/15672, I believe in this case we should only pay the C+ for help on the PR. Would this be $1000 too as internal PR? I think we can proceed with the payment on this one

You're correct, internal issues are paid to the C+ at the flat $1000 rate.
I see that you listed @Natnael-Guchima as the bug reporter in the OP comment and I wanted to confirm they get a bug report bonus of $250 with you since you only mentioned paying the C+. Should I whip up a bug report job for Natnael?

@strepanier03 correct @Natnael-Guchima should also be paid out $250

Gotcha, thank you @mountiny, will take care of setting everything up today.

@0xmiroslav - I have hired you to the job in UpWork, please take a look at the BZ checklist and leave any necessary comments and I'll update the OP checklist with your responses.

@Natnael-Guchima I've also created and hired you to a bug reporting job, please accept and I'll pay out tomorrow as well.
Thank you both!

Everything is done here, I've finished paying and closing contracts as well as leaving feedback. Thank you all for the diligence and effort 🙌 .


Backend Software Idea: CiviCRM

Maybe explore any alternatives as well:
CiviCRM and CiviCRM Bitnami installer

Old issue from when this was more non-profit focused


Add examples, test, and documentation

Platform needs more examples and documentations. As of now, Platform Config Documentation is out of date https://github.com/opentdf/platform/blob/main/docs/configuration.md all of these services need to be updated for the new configs. Maybe requiring updating the readme when configs are changed is a good idea.
There is not enough examples even if tests are considered examples. Our current examples consist of the following

simple encrypt
simple decrypt with client
i. we need examples with jwt identities
no subjectmapping examples.
i. i dont think these are really tested even
creating simple attributes
the authorization example
i. maybe i did something wrong but this example didn't work for me

Certain parts of Platform seem to be untested.
One example of is:
This PR https://github.com/opentdf/platform/pull/740 had a nilpointer bug on May 7th. Everything Greenlights. Even though any RPC call to authorization would cause a panic. And didn't get resolved until https://github.com/opentdf/platform/pull/791 May 13th
I spent about two weeks getting changing the old backend to the new platform, for my project. It was a pain to setup platform. Logging statements and reading the code was the only way I could really figure out what was happening. Which is not the user experience we want.

Make sure to add selection of encryption type as well


error when install

go install github.com/golang/lint/golint
# github.com/golang/lint
/src/github.com/golang/lint/lint.go:1434: assignment count mismatch: 2 = 3
go version:1.4.2
system:mac x 10.10

Use go get -u github.com/golang/lint/golint. You have an out of date dependency.


Init replacing service.Info

https://github.com/abelsilva/swaggerwcf/blob/753ab97746a1c875ca0e5a7a9add3ec9f7532a23/src/SwaggerWcf/SwaggerWcfEndpoint.cs#L59
Perhaps I have overlooked something, but when using the parameterless ctor this ends up in null for service.Info, which was set in buildService a line before.

i'll release a new version soon


mod hotbarplus not working

I downloaded advansedcompany and I had only 3 hotbar slots instead of 4. Along with this mod, I downloaded hotbarplus and put 8 slots in the configuration, I go into the game and there are only 6. I think okay, thanks for 6, but scrolling only works for 3 slots. The rest of the slots are just for decoration

AC already lets you upgrade your hotbar to multiple slots, and is configurable to how many slots you start with.
Read up on mods you install and their functionality to avoid problems in the future.

AC already lets you upgrade your hotbar to multiple slots, and is configurable to how many slots you start with. Read up on mods you install and their functionality to avoid problems in the future.

I don't know English well, tell me in which section it is configured

After selecting a save you see the lobby config / preset screen. Go to the tab Perks.
Look for Inventory slots

Base field is your start amount of hotbar slots (300% = 3 hotbar slots).
Change field is hotbar slots added per level in the Carry bags perk
Costs is the cost per Level in the Carry bags perk. This must not have more than 10 - ( base / 100 ) levels defined.

To have 8 hotbar slots when you start and upgradable to the max if 10 via perk leveling do this:

Change Base to 800%
Leave Change at 100%
Change Costs to 1000, 2000

And if you dont want AC to change the hotbar at all you can deactivate all inventory changes in Server configuration.


[Archetype builder] Eliminate "outer scope" constraints.

We no longer need "outer scope" constraints; they're just
explicit/redundant. Moreover, we no longer need to consider the
generic environment of the enclosing generic context when adding an
existing generic signature to the archetype builder.

@swift-ci please smoke test and merge

We are relying on the Self type in protocols in a ton of places without mapping it properly. Oops.

Lots more refactoring and fixing to do before I can eliminate the generic environment, so there's no point in keeping this open.


Pastel Wonderland Checklist

Things for the upcoming version, due before Blanketcon
Blocks/Plants

[ ] Pillow Block that can be thrown
[ ] Taffy Block

Physics-wise, behaves like a bed?

[ ] Purple spore puffs that can cause you to take damage when in proximity [toxic]
[ ] Blue glow mushrooms
[ ] Blue /Green decorational plants

WIP in planning for this one because forest stuff ideas is not done!
Items

[ ] Taffy rope
[ ] Taffy leads

Also WIP in planning
Foods related

[ ] Taffy [soft candy basically]
[ ] Cotton Candy
[ ] Sugar Ore
[ ] Pastel Ore
[ ] Spicy Sugar Cane
[ ] Peanut Brittled Ice Spikes
[ ] Hard Candy
[ ] Peanut Butter

Foods related tech/armour?
In extension by default, all the armour should have corresponding tools too.

[ ] Candy Machine for making Candy related stuff
[ ] Soft Candy Armour

Made out of Cotton Candy Blocks

[ ] Hard Candy Armour

Jolly-rancher-like, idk, there wasn't much on the ideaboard regarding this one.

[ ] Pillow Armour

Made out of pillows, reduces fall damage, makes a soft fall sound when landed

Worldgen
Above ground

[ ] Plains-like, potentially meadow clone

Sparse trees with flowers and plants

[ ] Dark-oak like forest
[ ] Tiaga-like forest
[ ] Desert-like
[ ] Mesa-like with Pastel blocks

Structures

[ ] King of the hill pillow towers
[ ] Burnable cotton villages

Other

[ ] Revamp textures to not be vanilla-edits

people working on this: please refer to the ideaboard and adjust as necessary.

also for anyone in devOS who wants to contribute: go to the development channel and join the BlanketCon thread. Easy channel access through pinned messages, or by click on the Threads button and then BlanketCon.


docs(@angular/cli): update docs for ng new

Update the docs for ng new command: remove source-dir, add new collection, service-worker, view-encapsulation, add aliases, update descriptions on docs/new.md.

Superseded by https://github.com/angular/angular-cli/pull/10470 .  Command option documentation is now auto-generated.


Having issue when sending a file : image , video etc

When I follow the example and try to send an image , the client fails with message:
image upload failed: upload responsed with %!d(float64=404
using latest version v0.1.0

Use the master branch not v0.1.0 or any release.

Use the master branch not v0.1.0 or any release.

Thanks, it helped

I am currently facing this issue. I tried downloading from the master but it didn't work for me.

Error sending message: to<PHONE_NUMBER> --> image upload failed: upload responsed with %!d(float64=404)
I also get the same error.. I tried compressing and resizing images but nothing helped.

have the same:
image upload failed: upload responsed with %!d(float64=404)


Outstanding CVEs in 3.9.0

Helm includes a version of go that has several high severity CVEs:
CVE-2022-23772
CVE-2022-23806
CVE-2022-23773
CVE-2022-24921
CVE-2022-24675
CVE-2022-28327
Is there an ETA for getting this lib updated?  If they are being deferred for non-exploitability, would you mind sharing the analysis?

@SatanicMechanic see https://github.com/helm/helm/pulls

This is still an issue because the circleci image we use for building is outdated. circleci is no longer updating it and the version of Go is outdates. They have newer images (in new locations) to use instead. See https://circleci.com/developer/images/image/cimg/go

This should now be fixed with the merge of #11161 and is out with the release of Helm 3.9.2


No such package @zig_sdk//platform on 1.0.1

Getting this on compilation using the upgraded package:
 com.google.devtools.build.lib.packages.RepositoryFetchException: no such package '@zig_sdk//platform': compilation failed:
return_code=1
stderr=
stdout=launcher.zig:296:19: error: expected ')', found ','
    for (want_args, res.exec.args.items) |want_arg, got_arg|

Looks like a simple syntax error.
FWIW, 1.0.0 still works fine :)

This is a new feature in zig: https://kristoff.it/blog/zig-multi-sequence-for-loops/ which the launcher uses.
Looks like you upgraded bazel-zig-cc but did not upgrade Zig?
As of bazel-zig-cc v1.0.1 it is recommended to use zig_toolchains() without parameters, unless you want to mirror Zig SDK yourself instead of downloading it from mirror.bazel.build. But then you will need to track such issues yourself and upgrade Zig from time to time.

Aye, that worked. Thanks mate.


OAK-11198: commons: add helper class for argument checking

For now, exactly for those variants used in Oak.
(Bikeshed: this is in a new package because it is internal and no other package seemed to fit; suggestions for better package/class names are welcome)

@reschke I would recommend using the name Preconditions or Validate over Checks.
@stefan-egli wdyt?

Let's use Validate for now.


Complete support for  Expr --> String

Is your feature request related to a problem or challenge?
This ticket tracks the remaining work to complete https://github.com/apache/arrow-datafusion/issues/9495
As @devinjdangelo  says in https://github.com/apache/arrow-datafusion/issues/9494#issuecomment-2007108672
We now have a solid foundation for converting Exprs --> SQL (see https://github.com/apache/arrow-datafusion/issues/9495  for why this is valuable).
It should now be straightforward to add support for the remaining Expr types and doing so is a great way to get more familiar with DataFusion's core data structures and optimization algorithms without already having expertise in database internals.
Describe the solution you'd like
The basic task is to:

Pick one (or a few) of the expressions below
Create a PR to Implement the Expr --> AST reverse code (in this match statement)
Add a test (for example, here)

Here is the remaining list of exprs (just note on the ticket which you plan to work on)

[ ] https://github.com/apache/datafusion/issues/10518
[ ] https://github.com/apache/datafusion/issues/10519
[ ] #10522
[ ] #10520
[ ] #10521
[ ] #10523
[x] https://github.com/apache/datafusion/issues/10197
[x] https://github.com/apache/datafusion/issues/10256

Describe alternatives you've considered
No response
Additional context
No response

take IsTrue, IsFalse, IsUnknown

I'd like to work on isNotNull, Not and Between

take ScalarFunction and InList

take Case

take Like

take Sort

take Negative

It looks like there are a few things not yet complete in https://github.com/apache/arrow-datafusion/blob/main/datafusion/sql/src/unparser/expr.rs#L52

[ ] Window functions
[ ] user defined functions
[ ] (maybe more)

take SimilarTo, IsNotTrue, IsNotUnknown,Negative

Here is a list that has not been done yet, for interested contributor to better see:

[ ]  GetIndexedField
[ ]  TryCast
[ ]  Sort
[ ]  Exists
[ ]  Wildcard
[ ]  GroupingSet
[ ]  Placeholder
[ ]  OuterReferenceColumn
[ ]  Unnest
[ ]  WindowFunction

Also Aggregate Function only supports BuiltInAggregateFunction, and it didn't use filter field.
Maybe we should use AggregateExpressionWithFilter when there's a filter? See: https://github.com/sqlparser-rs/sqlparser-rs/blob/e747c9c2af08f4ea12e8d1692adf95998209e2a1/src/ast/mod.rs#L643-L649

I noticed that in the sqlparser-rs crate, OrderByExpr is not part of the sqlparser::ast::Expr enum (which is the return type for expr_to_sql), which is a problem when I was implementing this for Expr::Sort.
So I wanted to get some thoughts on how best to proceed, should we make a change in sqlparser-rs, or should we just create a new enum to encapsulate the two, or anything else.

I noticed that in the sqlparser-rs crate, OrderByExpr is not part of the sqlparser::ast::Expr enum (which is the return type for expr_to_sql), which is a problem when I was implementing this for Expr::Sort.
So I wanted to get some thoughts on how best to proceed, should we make a change in sqlparser-rs, or should we just create a new enum to encapsulate the two, or anything else.

I think when converting Expr::Sort  to an sql::ast::Expr it should just return its inner Expr (and igore the ORDER BY information)
The ORDER BY information is required when turning the expression back into an entire query I think...
But at first just converting to sql::ast::Expr by ignoring the ORDER BY is probably correct in most cases

Thank you everyone who has contributed so far :pray:. I just filed a PR to update datafusion-federation to point upstream for plan->SQL going forward https://github.com/datafusion-contrib/datafusion-federation/pull/30. The additional expr implementations upstream here will enable datafusion-federation to handle many more queries than before.

take ScalarVariable

Here is a list that has not been done yet, for interested contributor to better view:

[ ]   GetIndexedField
[ ]  ScalarVariable
[ ]   TryCast
[x]   Sort
[x]   Exists
[ ]   Wildcard
[ ]   GroupingSet
[ ]   Placeholder
[ ]   OuterReferenceColumn
[ ]   Unnest
[ ]   WindowFunction

Take ScalarVariable if this is still needed.

Thank you very much @devanbenz  -- that would be awesome. Yes in general completing this list (or determining it is not possible) is very much needed

Sounds good. I have never contributed to this project before but am interested in helping out where I can and offering my support for this project -- @alamb if I needed to ask any questions should that be done in the issue thread or is there a slack irc or discord where datafusion discussion takes place?

Here are some thoughts to help people figure out what SQL matches to what expression:

GetIndexedField --> `SELECT col['field_name']
ScalarVariable --> Not sure (maybe @@foo@@?
TryCast * --> CAST (COL AS <datatype>>
Sort --> col ASC or col DESC NULLS FIRST
Exists --> col exists (SELECT 1 from foo)
Wildcard --> COUNT(*)
GroupingSet --> Not sure
Placeholder --> SELECT $1
OuterReferenceColumn --> not sure (I think this is the same as a column)
Unnest --> UNNEST(SELECT col FROM bar)
WindowFunction --> first_value(x ORDER BY y PARTITION BY z)

Here are some thoughts to help people figure out what SQL matches to what expression:

GetIndexedField --> `SELECT col['field_name']
ScalarVariable --> Not sure (maybe @@foo@@?
TryCast * --> CAST (COL AS <datatype>>
Sort --> col ASC or col DESC NULLS FIRST
Exists --> col exists (SELECT 1 from foo)
Wildcard --> COUNT(*)
GroupingSet --> Not sure
Placeholder --> SELECT $1
OuterReferenceColumn --> not sure (I think this is the same as a column)
Unnest --> UNNEST(SELECT col FROM bar)
WindowFunction --> first_value(x ORDER BY y PARTITION BY z)

@alamb I beleive ScalarVariable would just be something like @variable.

Thanks @devanbenz !

@alamb if you could please take a look at https://github.com/sqlparser-rs/sqlparser-rs/pull/1260 -- it's liked to this issue :)

@alamb if you could please take a look at sqlparser-rs/sqlparser-rs#1260 -- it's linked to this issue :)

Thanks @devanbenz  -- can you explain how this is related to this issue?

@alamb if you could please take a look at sqlparser-rs/sqlparser-rs#1260 -- it's linked to this issue :)

Thanks @devanbenz -- can you explain how this is related to this issue? (Sorry I am sure it is obvious but I can't keep everything in my head anymore )

All good - this is for the Unparser expr_to_sql  match statement https://github.com/apache/datafusion/blob/89443bff20015ef6c7646ca32754ece8c4093910/datafusion/sql/src/unparser/expr.rs#L391  I need to have a supported ast::Expr type for ScalarVariable from sql-parser. 👍

It seems to me that Expr::ScalarVariable currently is parsed from ast::Identifier (that starts with @)-- perhaps instead of adding a new ast node, you could just create an equivalent ast::Identifier 🤔
https://github.com/apache/datafusion/blob/accce9732e26723cab2ffc521edbf5a3fe7460b3/datafusion/sql/src/expr/identifier.rs#L28-L43

It seems to me that Expr::ScalarVariable currently is parsed from ast::Identifier (that starts with @)-- perhaps instead of adding a new ast node, you could just create an equivalent ast::Identifier 🤔
https://github.com/apache/datafusion/blob/accce9732e26723cab2ffc521edbf5a3fe7460b3/datafusion/sql/src/expr/identifier.rs#L28-L43

Sounds good - I'll have some time to come back to this in the next couple days! Will modify per your suggestion :)

I filed tickets for the remaining issues
These ones are likely straightforward.

[ ] https://github.com/apache/datafusion/issues/10518
[ ] https://github.com/apache/datafusion/issues/10519
[ ] #10522

The others are likely more involved as I don't know how to make a reproducer

With the completion of https://github.com/apache/datafusion/pull/10555 from @xinlifoobar I think this epic is now done!


Tests for flash loader and metadata

          This whole branch is not being tested for. I think it would be good to write a separate test for it, but could be done later.

Originally posted by @rettigl in https://github.com/OpenCOMPES/sed/pull/133#discussion_r1340976311
          There are no tests for metadata yet. I think mpes metadata are tested partly through the processor tests, maybe you can do something similar, or add a generic metadata test in test_loader?

Originally posted by @rettigl in https://github.com/OpenCOMPES/sed/pull/133#discussion_r1341004667

done by PR #239


[Event Request] codeunit 12 "Gen. Jnl.-Post Line" - OnPostCustOnBeforeInsertDtldCVLedgEntry()

Describe the request
    IsHandled := false;
    OnPostCustOnBeforeInsertDtldCVLedgEntry(GenJnlLine, TempDtldCVLedgEntryBuf, CVLedgEntryBuf, CustLedgEntry, IsHandled); //CustPub
    if not IsHandled then
        TempDtldCVLedgEntryBuf.InsertDtldCVLedgEntry(TempDtldCVLedgEntryBuf, CVLedgEntryBuf, true);

[IntegrationEvent(false, false)]
local procedure OnPostCustOnBeforeInsertDtldCVLedgEntry(var GenJnlLine: Record "Gen. Journal Line"; var TempDtldCVLedgEntryBuf: Record "Detailed CV Ledg. Entry Buffer"; var CVLedgEntryBuf: Record "CV Ledger Entry Buffer"; var CustLedgEntry: Record "Cust. Ledger Entry"; var IsHandled: Boolean) //CustPub
begin
end;

Additional context
Publisher request marked with //CustPub tag.Internal work item: AB#494234

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14819.


Relax panda and alembic requirements

Pandas 1.3+ has a regression: https://github.com/pandas-dev/pandas/issues/42549
This PR proposes relaxing alembic and panda requirements

I've mirrored this change in #119 so I can verify it builds. This change will be part of v2.5.2 releasing shortly.


WIP: Website builds

Hugo version:
Hugo Static Site Generator v0.69.2/extended darwin/amd64 BuildDate: unknown

Closing this PR in favor of another approach that doesn't require upgrade.


Add missing margin type from ITextOpts

This type is listed in the documentation and does work, but was not in the types file. I think these are the only two places it's needed to be added?

Thanks @cronin4392 !


Add unit and integration tests for Event Grid generated code

[x] Add unit tests for the code generated by AutoRest
[x] Add integration tests for the code generated by AutoRest

@bsiegel I assume generally autorest-generated code doesn't need unit test or integration test code. The convenience layer does.
Is there any special reason for this issue?

I believe we have tests for the autorest layer so that if the service behavior diverges from their swagger (or if our swagger doesn't match the service's actual API layer) our nightly live tests will catch it.

Close since it's done.