Question: React native component update cycle and Sagas

I'm having trouble understanding how to update a React Native UI from within the saga framework. As a simple example, I have two actions: spinnerShow() and spinnerHide() which show and hide a UI element, and a fetchData() action that, via dispatch(), triggers off fetchData_saga() using takeLatest().  If fetchData_saga() determines that it is going to take more than a few seconds I want it to call spinnerShow().   (The spinner is a Modal component that is shown/hid with a boolean state variable. fetchData_saga() has a 5-second delay() to simulate the workload.)
The Modal component UI doesn't update until the saga exits. After reading the Saga docs, what I assumed should work is using a parent saga that calls spinnerShow() then spawns fetchData_saga(), which before exiting calls spinnerHide(). However, it doesn't work - the UI doesn't update until fetchData_saga() completes. I've verified all the functions are being called and the state is being set in the middle of fetchData_saga(), and if I don't spinnerHide() the spinner appears when the Saga exits.
Google doesn't return me anything specific enough about Saga and React Native to help me understand what to do - maybe I'm googling the wrong terms, but I'm also new to React/RN and Redux. I've been at this all day and am out of ideas.
Any suggestions for resources or examples would be appreciated.

Could you post your saga?

Hey, going to throw this hail mary out there.
Are you running remote debugging in React Native?  If so, turn that off for a sec.  Was chatting with a fella on Friday with the exact same problem.

Thanks for the replies. I found my problem while preparing a minimal saga to post, and it was so simple I'm embarrassed to post it.
My problem was I was doing a call() to the action which just returns an event object. Switching to using put() solved my issue. It was showing the modal after the saga exited because at one point I had tried stacking two dispatch() calls, one to show the spinner, and one to my saga.


🐛 [BUG]: Race condition during adding/removing nodes when hooking into change handlers

Is there an existing issue for this?

[X] I have searched the existing issues and this is a new bug.

Current Behavior
This API is really amazing for what it does and for the most part has made my life extremely easy for what I set out to do. But the behaviors of removing nodes/edges has legitimately driven me insane for the entirety of my time with VueFlow. I may somehow just be using it wrong, but it's been wildly inconsistent.
All of the code for my post can be found here and you can launch the deployed version from its git page. I was able to mostly finish my program:
https://github.com/SkyAphid/corkboard
As far as I could tell by searching, no one else has brought up the problem I've been having. You can find the code I'm going to display below in its App.vue class, since I was able to find a decent solution around it for now.
My problem comes from deleting nodes. At first I thought it was as simple as calling removeNodes(id). The edges seemed to delete with it. However when I went to finally use my JSON files, I found that the edges weren't being removed, but hidden in the JSON file for some reason. I came back to edit the code, to find that removeEdges(id) wasn't working either when I tried to add it below removeNodes(). The node would disappear, but then pop right back up in the export, depending on how you ordered things (though I've forgotten what combination caused this, since I was so frustrated at the time and trying various things to just get it to work and move on).
I did experiments with using the edge.value filter functions, and applyEdgeChange, and still no luck. So I went back to trying to brute-force the removeEdges function to work. I was able to somewhat nail down the problematic behavior that way.
If you delete an edge, the node doesn't delete. If you delete a node, the edge doesn't delete. You can't do it at the same time for some reason. If I delete a node, its edges will still in the JSON output. If I try and do them at the same time, one or the other will stay hidden in the JSON output. Which resulted in this code, which "works:"
onNodeDoubleClick((nodeEvent) => {

  let pointerEvent = nodeEvent.event;
  let node = nodeEvent.node;

  pointerEvent.preventDefault();

  let items = [
    {
      label: "Delete Node", onClick: () => {

        removeNodes(node.id);

        let edges = getEdges;

        for (const edge of edges.value){
          if (edge.source == node.id || edge.target == node.id){
            removeEdges(edge.id);
          }
        }

      },
    },
  ];

etc, etc...

Calling this function will only delete one of the two at a time despite being in the same call. So I'll right click and tell it to delete the node, but only the edge will disappear. Then the next time, the node. But it gets even worse: If I have multiple edges connected to a node, it will only delete one edge at a time, then the next edge, then the next edge, and then finally it deletes the node.
Thankfully, I was able to get past the issue with the edges by doing this:
      label: "Delete Node", onClick: () => {

        removeNodes(node.id);

        let edges = getEdges;
        let edgesToRemove;

        for (const edge of edges.value){
          if (edge.source == node.id || edge.target == node.id){
            edgesToRemove.push(edge);
          }
        }

        removeEdges(edgesToRemove);

      },

So now at the very least, I have to click two times and it'll delete the edges, THEN the node on the second click. BUT it still asks me if I want to delete each node one by one. I honestly have no idea how to turn the confirmations off or define what warrants one. The least I could do was just make sure it's at least styled like the rest of the program.
I'm by far no expert in Vue/Javascript, I learned it along with this project as I went and built this in a week's time. But I think I've worked with it enough at this point to say that this is likely a bug of some kind.
Expected Behavior
Ideally, I don't want to have to call removeEdges at all. The removeNode function should frankly just delete any attached edges itself or be a toggleable setting. But I doubt that is something that will be compromised on, so at the very least I want to be able to call removeNode(id) and removeEdges(edges) in one call please, and they both disappear and I just go on with my day instead of having to repeat my inputs to get the desired result. Being able to control when the warning is displayed could help a lot too, since you don't really want users being able to control whether or not a disconnected/useless edge should be deleted or not.
Also, in my opinion, you guys should commit to edges working like everything else and not disappearing when a node is deleted because that strongly implies it's been deleted to new users. The resulting JSON output should be a representation of exactly what you see on the screen, with no surprises.
And as always, I know that maintaining APIs like this takes time and effort that I'm not entitled to. So I give you guys my upmost thanks for taking the time to read this long post and hopefully help me out. You're appreciated and I thank you very much!
Steps To Reproduce
I'll be deploying a version to my github page with the "fix" I was able to come up with. Simply delete a node with an edge attached to it to see the behavior.
Relevant log output
No response
Anything else?
Anyway, thank you so much for the amazing software! It really got my project off the ground and I was able to completely replace a commercial software that had gotten a little too expensive for us in a short amount of time. Hopefully this gets padded out so I can call it a day on this thing finally. I'm ready to get onto other things.
To whomever reads this, have a wonderful week/day/evening!

Thanks for the long post and detailed report.
Please provide a minimal reproduction of the issue since it makes it a lot easier to actually debug the problem since a long message doesn't really give me much of an idea what's up and I don't have the time to research a full github repo ^^
You can find a template right here.

@SkyAphid Any repro yet? :)

I'm writing up something now, will post in a little bit

Alright. Sorry it took so long. Codesandbox.io was a nightmare to try and actually code in. Everytime I'd CTRL-S it'd keep reverting my code back and the preview would constantly shut down, etc. It got to the point where it just completely stopped working, so I just started working locally and I managed to get it done in a few minutes.
https://codesandbox.io/p/devbox/priceless-brook-lkrh89
Bug explanation
I copied the result back to sandbox, since that was your preferred place. I managed to discover more about the bug, as it seems (for me) to have originated from using the Confirm Delete Node example on the VueFlow website.  If you don't use the confirm dialogs at all, it actually has the desired result. So that's what I may go for in my project, is just removing the dialogs to save the trouble.
The exact line of code that starts triggering the behavior is apply-default="false" in . You'll see in the example above that when you try to delete the node/edges at the same time, it'll do the edges and then the node itself. Depending on your perspective, this may not be a bug at all.
How I'd change it
I'd consider  tweaking the way the dialogs/apply-default works to where if I've called both removeNodes/removeEdges, they're all added to a queue in one go. So if the dialog appears and I agree to delete them, it does it all at once instead of just one at a time.
Otherwise if you don't want to change anything, I'd suggest at the very least adding a note to the tutorial page that this behavior can occur. I'm pretty much the perfect example here, I learned VueJs/Javascript/CSS with this project, and didn't realize the implications of the dialog appearing like this.
Again, thanks for the software! I'll just have to make some adjustments to my program to take what I learned into account.

it seems (for me) to have originated from using the Confirm Delete Node example on the VueFlow website

Sounds like the example causes you a race condition, which isn't what I want to happen but it can happen.
To solve this use-case more elegantly I'll be adding a onBeforeDelete handler in the next minor version which can be used for this case instead of doing the whole changes workaround which isn't as elegant and intuitive anyway (the whole concept of changes is most likely going to be trashed in VueFlow 2.x anyway).
I'll keep this issue open to track the implementation of onBeforeDelete though, thanks again for the report.

No worries! I'll keep an out and good luck. If onEdgesChange/onNodeChanges is removed, please consider adding a built in undo/redo system in too then, since that's how I'm tracking them in my program right now.

Hey, we ran into this recently while building an undo/redo feature, it used to work in older releases.
Having the deleted node in theonNodeChanges entry could help, or alternatively the onBeforeDelete can help us find the node before it gets removed.

That is already what is happening, where did you get the impression that it would be otherwise?

Its an observation, upgrading > 1.30 or so changed the timing for us. Before we used to be able to find the deleted node in the onNodeChanges handler. Then we couldn't, I'm not 100% on each release changes so take my words with grain of salt.

If you handle changes "manually" (by setting apply-default="false")

I have tried it and this works well for us, thank you and I appreciate taking the time to reply and maintain this project.

Its an observation, upgrading > 1.30 or so changed the timing for us. Before we used to be able to find the deleted node in the onNodeChanges handler. Then we couldn't, I'm not 100% on each release changes so take my words with grain of salt.

If you can pinpoint which version introduces this issue for you and/or provide a minimal repro of the issue, I'd appreciate that a lot. Then I could figure out what changes I made that might have changed the behavior (if it really did 😅) and fix it ^^

sure, I will try to pinpoint the release over the weekend.

@logaretm Hey! You said you were building something of an undo/redo feature, I was just looking to also do that and was surprised to find that it wasn't exactly already a plugin as a part of the core library. Perhaps the vue-flow library should have the node/edge changes stored as history and be able to undo/redo them? Seems like changes is going to be removed in the next minor release though from the commnet

perhaps the vue-flow library should have the node/edge changes stored as history and be able to undo/redo them?

This will most likely not happen - VueFlow is concerned with providing you tools to build this but not with solving specific use-cases for you.
Might be I'll add a basic example for it to the docs though, we'll see.

Seems like changes is going to be removed in the next minor release though from the commnet

If they are removed it's gonna be with the next major update, not minor, since it'd be a breaking change.

Makes sense, I was thinking too close to editors when seeing examples like https://tiptap.dev/docs/editor/extensions/functionality/undo-redo, but simply having some basic examples would also be great, thanks for the explanation

@logaretm any updates on the repro? :)

@bcakmakoglu Sorry for the delay, I was traveling and couldn't find time to do it.
I believe I have found the versions where this change occurred for this. Finding the node during the onNodeChange callback with findNode function was possible in <= 1.21.x releases but after >= 1.22 it isn't possible anymore.
Here is a sandbox with minimal repro.
I realize this was a long time ago so probably it isn't worth reverting, the applyChanges API works great for us now with the applyDefault set to false.

I’ll check it out once I’m back from Japan 🫶


Tweet on SIG Cloud Provider

Provide the actual tweet content between the commented colons

Looking for Cloud Engineers for SIG Cloud Provider to help run the cloud provider extraction working group specifically with k8s/k8s “cluster” directory and resolving how we properly  test in the absence of a cloud provider.Visit here for more information:https://bit.ly/2YBf2nL

/validate

/validate


url path bug

Describe the bug
When the url has a path include . will get a 404 and no response to the page at all, as in the following two cases:

✅  http://localhost:3333/client/v1/v100

❌ http://localhost:3333/client/v1/v1.0.0

vite.config.ts

Is it possible that this is a bug in the underlying devServer?
Reproduction
.
System Info
mac, chrome

Used Package Manager
pnpm
Logs

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of #2415


Misc doc changes from kianmeng

See #57 with thanks to @kianmeng - I saw them too late with too many conflicts and too many changes I wanted to do that were cumbersome with the conflicts and in a big commit. So redid them with small commits over here.

failure is only a connection error

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


Custom methods available to every factory (extending a module globally)

Problem this feature will solve
I want some code to be executed in multiple factories. The way I have done it is by:

Making a module
Extending the module in the factory
Calling a method from the module inside the factory

e.g.:
module FactoryBotEnhancements
  def add_active_trait
    trait :active do
      active { true }
      updated_at { Time.now }
      last_used_by { "System" }
    end
  end
end

FactoryBot.define do
  factory :my_factory do
    extend FactoryBotEnhancements
    add_active_trait

    some_attribute { 42 }
    other_attribute { "hello" }
  end
end

FactoryBot.create(:my_factory, :active)

Desired solution
I would like to extend every factory, so I can call the module's methods in any factory without needing to extend each factory manually.
I.e. instead of:
factory :my_factory do
  extend FactoryBotEnhancements
  my_method
end

factory :my_other_factory do
  extend FactoryBotEnhancements
  my_method
end

I want simply:
factory :my_factory do
  my_method
end

factory :my_other_factory do
  my_method
end

Additional context
Note that this is focused on methods that are called in the factory definition, not in the blocks where values are created. I.e.:
factory :my_factory do
  my_method_here

  some_attribute { not_something_here }
end

I found a way (whether it's recommended by the devs, not so sure):
# config/initializers/factory_bot.rb

module FactoryBotEnhancements
  def my_custom_method
    after(:build) do |record|
      puts "whoop"
    end
  end
end

class FactoryBot::DefinitionProxy
  include FactoryBotEnhancements
end

Then when defining factories:
FactoryBot.define do
  factory :my_factory do
    my_custom_method

    name { "yolo" }
    age { 42 }
  end
end

Yeah, that's pretty much what other folks have done. Related to https://github.com/thoughtbot/factory_bot/issues/1252. I don't think it was ever officially supported, but enough folks are doing it now that we're not likely to change the behavior, so you should be good.


MRG, MAINT: Clean up resource use

Clean up ResourceWarning in our codebase.

Also

removes --showlocals from pytest default config -- it makes the tracebacks super long even when --tb=short is used, and there is no way to disable it from the command line, so make it opt in.
adds -vv to the runs so that the name of the test currently executing is printed (should help identify segfaults) -- previously the output was unusable due to --showlocals on error when combined with -vv, but we no longer have that problem
fixes a matplotlib master deprecation involving plt.colorbar


Add support for ppc64le in dockerfiles

Enable dockerfile editing environment for ppc64le
Add new .Dockerfile and .partial.Dockerfile files for ppc64le architecture.

Related to https://github.com/tensorflow/tensorflow/issues/14934

Nagging Assignee @ymodak: It has been 25 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Oop improvements

New methods for exporting layouts as text, csv, excel files; and as figures. Method to randomly permute study (case-control) groups.

Pull request approved. But need to add tests for ramdomize_group_order, possible also for file and/or figure export (but how to test?)


Non-unreal server worker types [stage 2]

Description
External schema components are defined as SpatialOS components not generated by the Unreal GDK.
Currently, users must handwrite serialization and deserialization code from SpatialOS C API datatypes to interact with external schema. That implementation was defined in this document.
The new implementation will provide a code generation tool which receives a schema directory (or possibly a list of schema files) as input and will generate more user-friendly typed C++ classes for external schema components and a new interface for sending and receiving network operations using these types.
This is intended as a more robust implementation of the existing interface for interacting with external schema components from within the Unreal GDK. It is not intended as the ideal future workflow, which could also extend to A) facilitate callback registration using blueprints B) facilitate dynamic spawning of entities with external schema components.
Example generated interface for a single component:
class ExternalSchemaInterface
{
public:
	ExternalSchemaInterface() = delete;
	ExternalSchemaInterface(USpatialWorkerConnection* Connection, USpatialDispatcher* SpatialDispatcher)
		: SpatialWorkerConnection(SpatialWorkerConnection), SpatialDispatcher(SpatialDispatcher) { }
	~ExternalSchemaInterface() = default;

	void RemoveCallback(USpatialDispatcher::FCallbackId Id);

	// Component improbable.restricted.Worker = 60
	void SendComponentUpdate(Worker_EntityId EntityId, const improbable::restricted::Worker::Update& Update);
	USpatialDispatcher::FCallbackId OnAddComponent(const TFunction<void(const improbable::restricted::Worker::AddComponentOp&)>& Callback);
	USpatialDispatcher::FCallbackId OnRemoveComponent(const TFunction<void(const improbable::restricted::Worker::RemoveComponentOp&)>& Callback);
	USpatialDispatcher::FCallbackId OnComponentUpdate(const TFunction<void(const improbable::restricted::Worker::ComponentUpdateOp&)>& Callback);
	USpatialDispatcher::FCallbackId OnAuthorityChange(const TFunction<void(const improbable::restricted::Worker::AuthorityChangeOp&)>& Callback);

	// command Disconnect = 60
	Worker_RequestId SendCommandRequest(Worker_EntityId EntityId, const improbable::restricted::Worker::Commands::Disconnect::Request& Request);
	void SendCommandResponse(Worker_RequestId RequestId, const improbable::restricted::Worker::Commands::Disconnect::Response& Response);
	USpatialDispatcher::FCallbackId OnCommandRequest(const TFunction<void(const improbable::restricted::Worker::Commands::Disconnect::RequestOp&)>& Callback);
	USpatialDispatcher::FCallbackId OnCommandResponse(const TFunction<void(const improbable::restricted::Worker::Commands::Disconnect::ResponseOp&)>& Callback);

private:
	void SerializeAndSendComponentUpdate(Worker_EntityId EntityId, Worker_ComponentId ComponentId, const ::improbable::SpatialComponentUpdate& Update);
	Worker_RequestId SerializeAndSendCommandRequest(Worker_EntityId EntityId, Worker_ComponentId ComponentId, Schema_FieldId CommandIndex, const ::improbable::SpatialType& Request);
	void SerializeAndSendCommandResponse(Worker_RequestId RequestId, Worker_ComponentId ComponentId, Schema_FieldId CommandIndex, const ::improbable::SpatialType& Response);

	USpatialWorkerConnection* SpatialWorkerConnection;
	USpatialDispatcher* SpatialDispatcher;
};

where the component definition is:
namespace improbable {
namespace restricted {

class Worker;

// Generated from C:\Users\alastair\Projects\Unreal\UnrealGDKThirdPersonShooter\Game\Plugins\UnrealGDK\SpatialGDK\ExternalSchemaCodegen\Scripts\..\..\..\..\..\..\spatial\build\dependencies\schema\standard_library/improbable/restricted/system_components.schema(53,1)
class Worker : public SpatialComponent
{
public:
	static const Worker_ComponentId ComponentId = 60;

	// Creates a new instance with specified arguments for each field.
	Worker(FString WorkerId, FString WorkerType, improbable::restricted::Connection Connection);
	// Creates a new instance with default values for each field.
	Worker();
	// Creates a new instance with default values for each field. This is
	// equivalent to a default-constructed instance.
	static Worker Create() { return {}; }
	// Copyable and movable.
	Worker(Worker&&) = default;
	Worker(const Worker&) = default;
	Worker& operator=(Worker&&) = default;
	Worker& operator=(const Worker&) = default;
	~Worker() = default;

	bool operator==(const Worker&) const;
	bool operator!=(const Worker&) const;

	// Serialize this object data into the C API argument
	void Serialize(Schema_ComponentData* ComponentData) const override;

	// Deserialize the C API object argument into an instance of this class and return it
	static Worker Deserialize(Schema_ComponentData* ComponentData);

	// Field WorkerId = 1
	FString GetWorkerId() const;
	FString& GetWorkerId();
	Worker& SetWorkerId(FString);

	// Field WorkerType = 2
	FString GetWorkerType() const;
	FString& GetWorkerType();
	Worker& SetWorkerType(FString);

	// Field Connection = 3
	const improbable::restricted::Connection& GetConnection() const;
	improbable::restricted::Connection& GetConnection();
	Worker& SetConnection(const improbable::restricted::Connection&);

private:
	FString _WorkerId;
	FString _WorkerType;
	improbable::restricted::Connection _Connection;

public:
	class Update : public SpatialComponentUpdate
	{
	public:
		// Creates a new instance with default values for each field.
		Update() = default;
		// Creates a new instance with default values for each field. This is
		// equivalent to a default-constructed instance.
		static Update Create() { return {}; }
		// Copyable and movable.
		Update(Update&&) = default;
		Update(const Update&) = default;
		Update& operator=(Update&&) = default;
		Update& operator=(const Update&) = default;
		~Update() = default;
		bool operator==(const Update&) const;
		bool operator!=(const Update&) const;

		// Creates an Update from a ::improbable::restricted::Worker object.
		static Update FromInitialData(const ::improbable::restricted::Worker& Data);

		/**
		  * Converts to a ::improbable::restricted::Worker
		  * object. It is an error to call this function unless *all* of the optional fields in this
		  * update are filled in.
		  */
		::improbable::restricted::Worker ToInitialData() const;

		/**
		  * Replaces fields in the given ::improbable::restricted::Worker
		  * object with the corresponding fields in this update, where present.
		  */
		void ApplyTo(::improbable::restricted::Worker&) const;

		// Serialize this update object data into the C API component update argument
		void Serialize(Schema_ComponentUpdate* ComponentUpdate) const override;

		// Deserialize the C API component update argument into an instance of this class and return it
		static Update Deserialize(Schema_ComponentUpdate* ComponentUpdate);

		// Field WorkerId = 1
		const SpatialGDK::TSchemaOption<FString>& GetWorkerId() const;
		SpatialGDK::TSchemaOption<FString>& GetWorkerId();
		Worker::Update& SetWorkerId(FString);

		// Field WorkerType = 2
		const SpatialGDK::TSchemaOption<FString>& GetWorkerType() const;
		SpatialGDK::TSchemaOption<FString>& GetWorkerType();
		Worker::Update& SetWorkerType(FString);

		// Field Connection = 3
		const SpatialGDK::TSchemaOption<improbable::restricted::Connection>& GetConnection() const;
		SpatialGDK::TSchemaOption<improbable::restricted::Connection>& GetConnection();
		Worker::Update& SetConnection(improbable::restricted::Connection);

	private:
		SpatialGDK::TSchemaOption<FString> _WorkerId;
		SpatialGDK::TSchemaOption<FString> _WorkerType;
		SpatialGDK::TSchemaOption<improbable::restricted::Connection> _Connection;
	};

	class Commands
	{
	public:
		class Disconnect
		{
		public:
			static const Schema_FieldId CommandIndex = 1;
			struct Request
			{
				using Type = improbable::restricted::DisconnectRequest;
				Request(Type Data) : Data{ Data } {}
				Type Data;
			};
			struct Response
			{
				using Type = improbable::restricted::DisconnectResponse;
				Response(Type Data) : Data{ Data } {}
				Type Data;
			};
			using RequestOp = ::improbable::CommandRequestOp<Request>;
			using ResponseOp = ::improbable::CommandResponseOp<Response>;
		};
	};

	using AddComponentOp = ::improbable::AddComponentOp<Worker>;
	using RemoveComponentOp = ::improbable::RemoveComponentOp<Worker>;
	using ComponentUpdateOp = ::improbable::ComponentUpdateOp<Update>;
	using AuthorityChangeOp = ::improbable::AuthorityChangeOp<Worker>;
};

inline uint32 GetTypeHash(const ::improbable::restricted::Worker& Value);
} // namespace restricted
} // namespace improbable

Release note
Feature: A tool for external schema component C++ code generation, creating wrapper types and interface for sending and registering callbacks for these types.
Tests
Manual testing of generated interface is in progress.
Exhaustive worker SDK was used to generate C++ code and check this compiled.
Need to check edited Setup.bat flow on other machines.
Documentation
Documentation (being written) will be part of the existing non-unreal server worker type documentation.
/uncc /nocc

Given the length of this PR could we rewrite the history to make it clear what goes together? E.g. adding the code generator, specify the type of codegen we want, changes to the core gdk.
If we can read this in either a series of PRs or a series of commits this will be a lot easier.

Have we tested this with a comprehensive sample schema (like the Worker team would have)? Even if we can't add unit testing for the time being (should be JIRA'd), we should test once and commit the Test schema imo.

Worker have added a comprehensive test schema package that we now use in newer versions of the code generator tests: spatial package get schema test_schema_library <version> <zip>.zip

I used the Worker team exhaustive schema throughout dev but only tested as far as eyeballing the code and ensuring it compiled. Checking the compilation would be an ideal automated test for new releases when we have some kind of framework for running them? Not sure I understand what you're request testing 2) ExternalSchemaInterface codepaths for callbacks and sending updates/commands/events. @girayimprobable ? I've tested the basic functionality of sending and receiving commands and component updates but by no means covered a significant proportion of the types available. Not sure there's much of an efficient way to achieve this. Worker have some C API functionality arriving soon which can be used to validate serialized outgoing proto message structure but that still requires us writing code to send such updates and even then it won't have any correctness assertions on the content of data.

@alastairdglennie Testing I was asking for is basically what you mention that you did. 1) Compile a relatively complex schema in UE4 2) Do basic functionality verification sending/receiving updates/commands/authority in UE4. Automated testing is out of scope for this, and I'm also ok if we have a well defined set of unsupported complex schema situations for the time being.
Maybe the code has changed after you've done some of these tests? Because I think there's at least one compile error that would have been caught (SourceGenerator.cs:220), and it doesn't involve a complex schema at all. And I think sending a component update from Unreal simply wouldn't have worked due to Unreal/InterfaceGenerator.cs:50. Similarly with commands HeaderGenerator.cs:229.
I suspect that these were changed during the PR process so maybe they didn't get the full attention.

ah okay, I haven't had a chance to look through all your comments yet, will aim to later this week after MW stuff!

Fixed up all comments I can see, waiting on Giray for 2nd review.


Resolved issue #45, added support for UTF8 encoding to prevent the is…

Added UTF-8 encoding for the filename string. Problem persists only on Python 2.7.

Coverage remained the same at 9.62% when pulling fe70dac5c2a8b81c95775a80ebd50eee48509988 on regmibijay:master into f94d3bb509eb0741164149acbef0788769a869e4 on DLTK:master.


halloy: 2023.5 -> 2024.1

Diff: https://github.com/squidowl/halloy/compare/refs/tags/2023.5...2024.1
Changelog: https://github.com/squidowl/halloy/blob/2024.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 288556 run on x86_64-linux 1

1 package built:

halloy


add id after growl has been created

Hi
How can I access the growl object after it has been created? I would like to add an id to the growl container. Unfortunaley this does not work:
var growl = $.growl({...},{...});
growl.attr('id','test');
Thanks,
Frank

I would recommend either using the template as you did or use the onOpen function to assign a id and then when recalling your function that would generate the notification check to see if the id exists first.


feat: include 2 new methods toCapitalize and trimSpaces

toCapitalize: Convert the first word of each thread into a capital letter.
trimSpaces: Eliminate double / triple / ... spaces leaving them to only one
Thanks!

Coverage decreased (-0.3%) to 99.677% when pulling a628ee0cb37491f179770125c1eef9a21a6988c1 on madebyivan:master into 9ebcae203fac7a78c391bfc7076497f591fe0be6 on epoberezkin:master.


Feature/add profile drawer with static content

Description
Creates a profile content drawer which can open and close in accordance to the UX design document. Material UI's Drawer component was used to achieve this end. Paired with conditional styling on Drawer close and open as well as local state set up through the use of useState. The current styling and components are completely static and will require additional functionality and styling to achieve then end goal for the drawer.
Type of change

[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] Test A
Ascertained functionality through the hot-loaded webpack server (toggled in and out of the drawer)
[ ] Test B

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

There were unwanted commits that got carried over to this branch accidentally the reverts you see in the commit history rectified this problem

 RC2: Work on Header components (account button, search, notifications, title)


Cypher search: Check if generated Cypher is provided in backticks

Check if generated Cypher code is wrapped in backticks
Some LLMs like the VertexAI like to explain how they generated the Cypher statement and wrap the actual code in three backticks:

I have observed a similar pattern with OpenAI chat models in a conversational settings, where multiple user and assistant message are provided to the LLM to generate Cypher statements, where then the LLM wants to maybe apologize for previous steps or explain its thoughts. Interestingly, both OpenAI and VertexAI wrap the code in three backticks if they are doing any explaining or apologizing. Checking if the generated cypher is wrapped in backticks seems like a low-hanging fruit to expand the cypher search to other LLMs and conversational settings.

lgtm! maybe worth adding a unit test if you have time

Any hints where to put the tests for the extract_code_block function?

@hwchase17 @dev2049 I've added the unit test


not working

hi there!
the status interface isn't working. i clicked submit a bunch of times, but it didn't do anything.
is there anyway you can help?

Hello,
Did you already try restarting your computer/discord?: (Y/N)
What operating system are you using:
What version of Discord are you running:
What version of my application are you running:

N
Windows
N/A
Latest version

Well go ahead and try rebooting. I am unable to troubleshoot without knowing the version number of Discord and my application that you are running.


Update Globals “theme”

Automatically generated by Netlify CMS

👷 Deploy Preview for joseluizcoe-site processing.

Name
Link

🔨 Latest commit
bd9a8045052cb932dac23f6250031407582a7ec5

🔍 Latest deploy log
https://app.netlify.com/sites/joseluizcoe-site/deploys/63455eecbb10d200084a2c1b


[send] doesn't seem to work with Wwise source plugins

Issue #13 seems to imply that [send] has been implemented and works within Wwise souce plugins, but I don't see anything in Wwise with this PD patch:

Is it expected to work? Did I do something wrong?
Patch: https://enzienaudio.com/h/martinb/send/

ohh sorry @martinbussyparis, yeah this isn't expected to work yet. Changes coming soon that will address expectations like this! But that previous post was more outlining the direction of the API to access hooks in other frameworks like Wwise.
Apologies for any confusion.


Discover the status of discuss.openai.com in 2019

TL;DR: discover if discuss.openai.com is temporary or permanent disabled; if the second case, look for some official explanation

The https://discuss.openai.com/ is (or was) one of the places open on the internet to discuss philosophy and A/IS ethics.
I remember that people around the world were discussing topics that were not just about how to develop AI, but about morality, risks, impact, etc, in one of the sub forums. And some of these threads where big.
At this moment, the adress "discuss.openai.com" does not resolve to a IP address. I tested from different IPs and proxies, and still not able to connect.
Looking from cached pages of the site, I'm able to find that at least until 2018-12-23 the discuss.openai.com was online.

For reference, please look at https://ai.stackexchange.com/questions/10282/status-of-discussion-forum-discuss-openai-com-in-2019. But I guess is bad news.
Maybe I will open a different topic on the AI stack exchange on how to handle shutdown of AI forums. Draft here https://github.com/EticaAI/forum/issues/65#issuecomment-461253123.
But for this issue, I will close and remove the OpenAI forum.


BF-Streams: Review of OpenML

The incorporation of code related to OpenML in RTD and the unit tests caused huge delays during workflow processing. However, the wrapper and related howtos are running properly. Problems occured with current version 0.12.2 and hopefully it will be solved in higher versions...

The problems of inpredictable delay seem to be solved by OpenML API version 0.14.2 and the corresponding adjustments of our wrapper code...


feature/139-refactoring

As a first step of the refactoring, I broke down the read_standards file into 12 smaller files/modules based on logical grouping.
I tested it on my local machine, but it would be good if Erin could also test out the scenarios for submission checklist generation, as I might have missed some corner cases.
Once this is merged, I can improve readability and documentation and better annotate the regular expressions.

@eschltz @drpaulralph first PR for refactoring!

I tested some different checklist combinations and everything still seems to be working fine.

Hello Erin, Hello Prof. Ralph,
Sorry, it took a while to get to it.  But the changes are now done and can be reviewed again!
@eschltz, can you try some combinations and let me know if everything is working fine?


Saia, Inc : 2013 Annual Report

Saia, Inc : 2013 Annual Report

Let's Get Quantifiable

Source URL:
https://assets.ctfassets.net/kjv8gs2ccggb/7q8zvEXBsEXFTMfYo6l8xP/27c0d48b4b60ecf7ff77fa78695f556b/Saia_2013_Annual_Report.pdf
BrowserChrome 85.0.4183.59OSWindows 10 64-bitScreen Size2560x1080Viewport Size3840x1405Pixel<EMAIL_ADDRESS>Level100%

img optimization by sqoosh.app

Saia OR : Time of Tenure (2008-2013; again - TEAM EFFORT )


Set config from env

Add --env / -e to set a config's value from the environment.
export ASDF=123
defang config set -e ASDF #ok
defang config set -e ASDF file #error
defang config set -e ASDF=1234 #error

See also #598

@lionello lgtm for what it is. I think the CSV of vars would be helpful, though looking at it, I'm not sure what the best DX would be?
Maybe another command like defang config create from-env ENV1 ENV2 ENV3
Just passing csvs like defang config create -e ENV1,ENV2,ENV3 feels off to me.
The most docker-like would be defang config create -e ENV1 -e ENV2 -e ENV3 which I think is pretty doable, though it would take a bit of massaging to have a good experience with the GH action...

@raphaeltm of those options, I prefer defang config from-env ENV1 ENV2 … (without create) but lets leave it as-is for now. The action can invoke defang config N times.


withFallback does not replace other substituted config values

---- base.conf ----
value = 20
substitute {
  value = ${value}
}

---- replace.conf ----
value = 10

The result of substitute.value is still 20 when do replace.withFallback(base).  Should it be 10?

This is documented semantics, please read: https://github.com/typesafehub/config#note-about-resolving-substitutions-in-referenceconf-and-applicationconf


Update logic class diagram

Pull Request Test Coverage Report for Build 1024

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.124%

Totals

Change from base Build 1001:
0.0%

Covered Lines:
3110

Relevant Lines:
4435

💛  - Coveralls

The rest is good.


ConnectivityPlugin.ConnectivityChanged not working on iOS for an XForms App

I'm using the ConnectivityPlugin for detecting changes in the connections for a XF App.
Connectivity.Plugin.CrossConnectivity.Current.ConnectivityChanged += (sender, e) => { ....; };
The event is triggered on Android, that works excellent but on iOS it is never triggered.
I tried with iPhone using iOS 9 and also on Tablets with iOS 7/8 and the event is never triggered on iOS.
According to the readme for iOS there's nothing special to configure on the App to make it work.
Is this a known issue? Do I need to assign any background Modes to the iOS App?
Plugin Version: 1.0.4
Xamarin.Forms Version: <IP_ADDRESS>47
Cheers
Ariel

Are you putting them into Airplane mode? This has been working fine for me since I originally launched the plugin.

No, no airplane mode.
I have the Connectivity.Plugin.CrossConnectivity.Current.ConnectivityChanged += (sender, e) => { ....; }; on a ViewModel that is created when the MainPage is created. Again on Android works excellent.
I placed a Debug.Writeln("ConnectivityChanged"); in the event to make sure it is not a problem with the breakpoint or on my business code so I have only one line with "Debug.Writeln("ConnectivityChanged");" as the event implementation.
I do as follows:
with the App running, I tap the home button of the iPhone, move to settings. Here I changed from one Wi-Fi network to another. That didn't triggered the event. I also tried to turn off the Wi-Fi settings so no connection will be in place and that does not work either.
I just tried this on a iPhone 5 with iOS 9 running the App in Debug mode.

Hi,
One of our client has a problem with connectivity detection when he switch into into Airplane mode. He has an Apple iPad Air. Our app using Xamarin.Forms and the Connectivity.Plugin for it.
Is there a solution for this problem?
Best Regards,
Aleksandrs

I can not reproduce this at all on iOS 9 on a physical device. I have tried over and over again and no issues at all as it triggers the change each and every time.  You may receive multiple events when coming to and from the application though. If you swipe up from the bottom of the screen you can toggle to airplane mode and see it working. I have tried 2 apps with the nuget and with the dlls directly.
I am using Reachability from here: https://developer.xamarin.com/recipes/ios/network/reachability/detect_if_network_is_available/

Ok, no worries. I'll reinstall the plugin again to see if it helps.
Thanks for the support


⚠ Samsung has degraded performance

In 69c4397, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1888 ms

Resolved: Samsung performance has improved in ab7660a.


Sensor logs exist in both warmup test split and training split

Hello,
I noticed that the following logs in warmup_test_e2e.yaml are presented as train_logs in default_train_val_test_log_split.yaml. I'm currently testing on the mini dataset. Does this meaning that these logs are used in both training and testing?
Thanks in advance,
2<IP_ADDRESS>.00.38_veh-35_01008_01518
2<IP_ADDRESS>.28.35_veh-35_00620_01164
2<IP_ADDRESS>.36.44_veh-35_00152_00504
2<IP_ADDRESS>.36.44_veh-35_01133_01535
2<IP_ADDRESS>.36.44_veh-35_02035_02387
2<IP_ADDRESS>.54.15_veh-12_04366_04810
2<IP_ADDRESS>.39.51_veh-26_01943_02303
2<IP_ADDRESS>.39.51_veh-26_05620_06003
2<IP_ADDRESS>.03.17_veh-12_02584_02970
2<IP_ADDRESS>.58.55_veh-35_01095_01484
2<IP_ADDRESS>.58.55_veh-35_01894_02311
2<IP_ADDRESS>.23.18_veh-38_00773_01140
2<IP_ADDRESS>.23.18_veh-38_02526_03027
2<IP_ADDRESS>.37.09_veh-12_00404_00864
2<IP_ADDRESS>.32.09_veh-35_05038_05402
2<IP_ADDRESS>.48.02_veh-12_04057_04438
2<IP_ADDRESS>.48.02_veh-12_04978_05337
2<IP_ADDRESS>.26.26_veh-38_04544_04920
2<IP_ADDRESS>.33.41_veh-35_03901_04264
2<IP_ADDRESS>.42.45_veh-12_03445_03902
2<IP_ADDRESS>.22.11_veh-38_01480_01860
2<IP_ADDRESS>.56.12_veh-16_00839_01285
2<IP_ADDRESS>.54.19_veh-35_00808_01256
2<IP_ADDRESS>.31.36_veh-16_00016_00377
2<IP_ADDRESS>.06.37_veh-35_00258_00748
2<IP_ADDRESS>.59.12_veh-38_01208_01692
2<IP_ADDRESS>.51.05_veh-17_01352_01901
2<IP_ADDRESS>.06.21_veh-38_03231_03712
2<IP_ADDRESS>.06.21_veh-38_04471_04922
2<IP_ADDRESS>.06.21_veh-38_04933_05307
2<IP_ADDRESS>.19.22_veh-35_00440_00858
2<IP_ADDRESS>.45.29_veh-35_00600_01084
2<IP_ADDRESS>.45.29_veh-35_01095_01486
2<IP_ADDRESS>.57.11_veh-08_01200_01636
2<IP_ADDRESS>.17.01_veh-45_02314_02798
2<IP_ADDRESS>.54.02_veh-45_00665_01065
2<IP_ADDRESS>.16.42_veh-28_02011_02410
2<IP_ADDRESS>.16.42_veh-28_03307_03808
2<IP_ADDRESS>.43.07_veh-28_00508_00877
2<IP_ADDRESS>.57.41_veh-50_00352_00535
2<IP_ADDRESS>.57.41_veh-50_01522_02088
2<IP_ADDRESS>.12.18_veh-50_00211_00304
2<IP_ADDRESS>.31.07_veh-50_01750_01948

Hi @dentou,
the warmup-test split is indeed based on the mini split - not only the test logs within mini. Thus, there can be overlaps between the mini train logs and the warmup test ones.
Warmup-test‘s mere purpose was to give participants an opportunity to get familiar with the submission procedure. As such, it was not meant as a competitive benchmark to evaluate performance.
After the competition we launched the public leaderboard based on navtest. I encourage you to use the navtest split as a held out test split instead of warmup_test. Since the open leaderboard has less strict limits regarding the number of submissions, you can also directly submit there and there is no need start by running experiments on mini.
(I am closing this for now. Feel free to re-open)

Hi @mh0797,
Thanks for the quick response. I think we can update this answer. In particular, my misunderstanding came from this section

test-logs in mini vs. warmup_test_e2e
The warmup_test_e2e scene-filter filters the test-logs from the mini dataset for challenging scenarios by employing the same strategy that is used to generate navtrain from trainval. Thus it subsamples these logs.

Since the test set is ~50% more than the mini set, I think a test split on mini could be helpful as a initial sanity check. Anyway, thanks for the suggestion about the navtest split.

Hi,
thanks for pointing this out. I updated the answer accordingly.
You can always reduce the number of scenarios by setting train_test_split.scene_filter.max_scenes=<some_value>


repeat not working as expected

I can't give you a demo right now since I'm waiting on new releases since the repeat binding is currently broken but fixed in #46. This has been working before so some kind of regression must slipped in. I have a sortable plugin that adds a placeholder when you start sorting in a repeatbinding. The placeholder adds an element.style.visibility = "hidden"since it should simulate a real item in the repeat. However when a sort is finished the placeholderis removed and the itemsare sorted. The problem is now that the item that you are moving to gets the placeholderelement (but correct executionContext) since it still has the element.style.visibility = "hidden". So it feels like the wrong view is used in this particular scenario.
Let me know if you need more info and also I will get up a demo asap.

@stoffeastrom I've got this reproduced in the demo you linked to in the Gitter channel. Will look into it and get back to you.

@martingust great!

@stoffeastrom Instead of pulling views out of the DOM and adding them back again, we are trying to reuse the views as much as possible, this improves the performance a lot. The reason your sortable plugin worked before was due to the lack of view reuse in the repeat that got introduced in the v0.10.0 release of templating resource.
In certain scenarios like yours and 3:rd party jQuery plugins that mutate the DOM we would like to have way to opt out of view reuse, I have created an issue for this here: https://github.com/aurelia/templating-resources/issues/49. Also we are planing on implementing a cache mechanism for views, that would most likely solve this kind of problems without the need of disabling view reuse in the repeat.
However when looking at your sortable plugin I'm wonder if there is a way for you to achieve what you are doing without mutating the DOM? Would it be possible to data bind to the image.media.placeholder property and based on that set a css class that hides the element?
Like:
<div class="col-sm-6 col-md-3 ${image.media.placeholder ? 'placeholder' : '' }" repeat.for="image of images" sortable-item="item.bind: image;">

@martingust That is ofc possible. However, doing that the plugin relies on that the user sets the correct css. It would be nice if that just works ootb. Maybe it will be possible to have plugin scoped css in the long run which would be awesome.

@stoffeastrom I see, hmmm. Would it be possible to implement the detached or unbind callback and reset the styling?

Definitely, if your plugin alters html, then I would make sure to appropriately destroy the plugin during the detached or unbind callback.

@martingust  @EisenbergEffect. I will try that. As long as the unbind is called for the placeholder element before setting the new executioncontext it wouldn't be a problem.

detached is not called since the view isn't destroyed which makes sense. The unbind is called but not for the element that I'm removing..

Sry, I forgot to use latest build.. it actually calls detached.

So I implemented the detachedcallback and it works as expected!

Awesome!

Great, looking forward to see the demos of your plugin.

Sorry guys but after some more testing I found out that it doesn't work 100%. When moving a element left to right it works however moving from right to left it doesn't. Neither detached nor unbind is called for the placeholder element

Ok, sorry won't be able to check it out tonight. Let me know if you find out anything or i'll take a look.

@martingust @EisenbergEffect
I would argue that this is a bug after some more investigation. I get different view lifecycles and it doesn't seem consistent. It's kind of hard to get a minimal example but in pseudo
After a repeat is rendered add an item
I expect the following view functions to be called
bind, attached
When moving the newly added item I expect nothing
When removing the newly added item I expect
unbind, detached
to be called.
Trying out different scenarios for above. Meaning moving the item forward or backwards in the array yields in different view lifecycle events and they are not always triggered for the correct view.
Best thing would be if we could get some scenarios up and running in unit tests but I'm not sure how that should be done.

@stoffeastrom You are absolutely right. Since the view gets reused detached and attached never get fired. This would get solved when the view caching is in place, which should be within the next couple of weeks. Hope this doesn't cause too much trouble for you or you can do temporary workaround maybe with CSS.

Actually I'll reopen this, so we can track when caching is in place.

@martingust @EisenbergEffect
Even if a view is reused shouldn't we make sure to follow the view lifecycle?
In my opinion we should still follow the view lifecycle but with the correct execution context.
What I'm saying is that I still want view caching for performance but ensure the view lifecycle.
In my case, if bind and unbind would be called with correct execution context it would just work.
Thoughts?

Thinking more of it we could even call attached and detached meaning that the view is attached, detached. If there are use cases you would want to know if it's attached/detached to the DOM we could add those events

Yes, absolutely. We have a "quickie" version in the repeater. But, we are going to remove that and implement caching at the view factory and view slot level. This will ensure enable the full lifecycle to be preserved even while re-using views. The repeater then will have it's custom logic removed. That's the plan at least.

Great!


ancp-bids 0.0.10 crashes

Trying to load BIDSLayout with ancp-bids 0.0.10 produces the following error:
Traceback (most recent call last):
File "/data2/lmichalke/github/ANCP/voxelwiseencoding/voxelwiseencoding/test_bids_loader.py", line 173, in
test_bids_loader()
File "/data2/lmichalke/github/ANCP/voxelwiseencoding/voxelwiseencoding/test_bids_loader.py", line 8, in test_bids_loader
layout=bids.BIDSLayout('/data2/azubaidi/ForrestGumpHearingLoss/BIDS_ForrGump/derivatives/fmriprep')
File "/home/lmichalke/anaconda3/envs/testbids2/lib/python3.9/site-packages/ancpbids/pybids_compat.py", line 10, in init
self.query = XPathQuery(self.dataset, self.sc)
File "/home/lmichalke/anaconda3/envs/testbids2/lib/python3.9/site-packages/ancpbids/query.py", line 21, in init
super(XPathQuery, self).init(dataset, scm)
File "/home/lmichalke/anaconda3/envs/testbids2/lib/python3.9/site-packages/ancpbids/query.py", line 13, in init
self.root = dataset.to_etree(mapping_=self.id2x, reverse_mapping_=self.x2id, nsmap_=schema.NS_MAP)
File "/home/lmichalke/anaconda3/envs/testbids2/lib/python3.9/site-packages/ancpbids/model.py", line 4754, in to_etree
dataset_description_.to_etree(element, name_='dataset_description', mapping_=mapping_, reverse_mapping_=reverse_mapping_, nsmap_=nsmap_)
File "/home/lmichalke/anaconda3/envs/testbids2/lib/python3.9/site-packages/ancpbids/model.py", line 4221, in to_etree
GeneratedBy_.to_etree(element, name_='GeneratedBy', mapping_=mapping_, reverse_mapping_=reverse_mapping_, nsmap_=nsmap_)
AttributeError: 'dict' object has no attribute 'to_etree'

Thanks for reporting,  @lmichalke !
Fixed in 0.0.11
FYI: the implementation had not yet processed complex JSON objects, i.e., in your case, derivatives have a complex structure GeneratedBy which was not read from its JSON representation

Sorry, had to fix one more issue. Please upgrade to 0.0.12.


⚠ Bofrost.de has degraded performance

In ac349af, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1773 ms

Resolved: Bofrost.de performance has improved in dfe0438 after 20 minutes.


cilium-cli: 0.12.1 -> 0.12.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cilium-cli/versions.
meta.description for cilium-cli is: "CLI to install, manage & troubleshoot Kubernetes clusters running Cilium"
meta.homepage for cilium-cli is: "https://www.cilium.io/"
meta.changelog for cilium-cli is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.12.2 with grep in /nix/store/sk3mxzcr5vznc33qrm816s95hqwrnfj5-cilium-cli-0.12.2
found 0.12.2 in filename of file in /nix/store/sk3mxzcr5vznc33qrm816s95hqwrnfj5-cilium-cli-0.12.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cilium-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sk3mxzcr5vznc33qrm816s95hqwrnfj5-cilium-cli-0.12.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cilium-cli https://github.com/r-ryantm/nixpkgs/archive/21c18d68ba25125260cc4215e9b7f2f8102edc3a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sk3mxzcr5vznc33qrm816s95hqwrnfj5-cilium-cli-0.12.2
ls -la /nix/store/sk3mxzcr5vznc33qrm816s95hqwrnfj5-cilium-cli-0.12.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cilium-cli

Maintainer pings
cc @humancalico @bryanasdev000 for testing.

Result of nixpkgs-review pr 188578 run on x86_64-linux 1

1 package built:

cilium-cli


[Feature Request] -RequiredResource(File) for Save-Module

There are many circumstances where you want to install modules to somewhere other than the powershell modules folder, for instance if creating the equivalent of a python "virtual environment" for CI builds, or other circumstances where you want a module but don't want to deploy it to the main path.
Save-Module usually fulfills this use path, and as such it should support the same items for "restore"-like functionality
Alternatively, add -Path or -Destination parameter to Install-Module to specify an alternate location. It should be in a different parameter set then -Scope

This is actually covered in the RFC as the -DestinationPath parameter. I suggest it be altered to -Destination because this would be more familiar to most Powershell users since Move-Item, Copy-Item use -Destination and not -DestinationPath

@JustinGrote thanks for opening this issue, we certainly intend to provide this functionality...one open question Amber and I have been discussing is if it makes more sense to provide this experience through a -Destination parameter on Install or through the Save cmdlet...do you have any insight as to which experience you prefer?

I would suggest it on Save since most users are familiar with that workflow, and the only difference is that it doesn't drop the XML metadata for the repository into the destination.
Once that's done it'd be trivial to make a "Restore-PSResource" command that would basically automate looking for a requirements.psd1 or something file in the current directory, and output to a predefined path, to give that "dotnet restore" or "nupkg restore" or "pip install -r requirements.txt" behavior users coming from other languages may be expecting.

Just ran into this bug today.
I need to add a handful of PowerShell modules to a directory inside a GitHub Actions worker so I can publish an AWS Lambda layer containing my PowerShell function's dependencies. I can call Save-PSResource with individual modules just fine, but calling it per-module doesn't spark joy. I had hoped to use the new RequiredResource functionality here, but it doesn't appear to have been added past just Install-.
It seems like this bug has gotten kicked around quite a bit over the last 3 years - @alerickson @SydneyhSmith would you prefer a separate feature request that's more up-to-date with the current state of the module?

Thanks @sean-r-williams feel free to add any additional thoughts to this issue, I will update status here

@sean-r-williams as a workaround (and shameless plug), ModuleFast  supports this.
iwr bit.ly/modulefast | iex
Install-ModuleFast -Path /path/to/specfile -Destination /path/to/target -DestinationOnly


Failed prop type: prop type is invalid (Text.propTypes.style.isRequired)

Description, Reproduction Steps and Sample Code
I wrote a React Native component:
export const OneLineText = ({ textStyle, title }) => (
  <Text style={textStyle} numberOfLines={1}>
    {title}
  </Text>
);

OneLineText.propTypes = {
  textStyle: Text.propTypes.style.isRequired,
  title: PropTypes.string.isRequired,
};

Run jest --watch, and it shows warning:
Warning: Failed prop type: OneLineText: prop type `textStyle` is invalid; it must be a function, usually from React.PropTypes.
    in OneLineText (created by LeftTextContainer)
    in ...
    ...

After deleting the isRequired in the line textStyle: Text.propTypes.style.isRequired, the warning is gone, but I need this prop to be required.
Additional Information

React Native version: 0.44
Dev tools: while jest testing

Same here. The solution would be to manually mock PropTypes, but I'm sure there should be something way cleaner at disposal to fix this. @xareelee, you might want to submit your issue here, as PropTypes has been detached from the main React package, and PropTypes aren't strictly bound to React Native.

@jkomyno Thanks for your reply and info.
I currently switch to use Flowtype system instead of PropTypes.
Close this issue anyway.


Fails to run with older versions of ESBuild

Error
UnknownEsbuildError: Error: Build failed with 1 error:
[redacted]/node_modules/esbuild/lib/main.js:239:12: error: Invalid option in build() call: "jsx"

Possible cause
@contentlayer/core lists an old version in peerDependencies:
https://github.com/contentlayerdev/contentlayer/blob/20ce9059ca7da34a2a7eb9b7650be2b6bcde6cdd/packages/%40contentlayer/core/package.json#L17
It uses the jsx option which was added in ESBuild v0.12.1:
https://github.com/contentlayerdev/contentlayer/blob/20ce9059ca7da34a2a7eb9b7650be2b6bcde6cdd/packages/%40contentlayer/core/src/getConfig/index.ts#L69
Proposed fix
https://github.com/contentlayerdev/contentlayer/pull/204/files

Thanks for opening this issue. Can you please provide a reproduction repo with instructions how to reproduce the problem you're experiencing? Thank you!

Hey @schickling, here's the reproduction repo forked from the https://github.com/contentlayerdev/next-contentlayer-example.
There are two way this can be reproduced:

By adding another package that requires an older version of esbuild: https://github.com/nayaabkhan/next-contentlayer-example/commit/0471b02de5c37fd111caa04c4ff4e48c09659835
By using overrides to manually set a version: https://github.com/nayaabkhan/next-contentlayer-example/commit/fe9b5c4d5c729b99fa56f09057b5f8df45aa50ab

This should be addressed with the 0.2.5 release. 🎉


Rename stackdriver to error-reporting

Renames the stackdriver sample to error-reporting for consistency with other repos

@tswast

LGTM.
@shun-fan Looks like there are some merge conflicts. Be careful resolving.

rebased to master fyi


Fix web rendering

The rendered website is not displaying the nginx config file correctly because of a missing newline

Thanks for your cooperation @raybuhr !


[ISSUE] More test coverage needed for handler type

Need more test coverage on handler type
Where?
Defined in https://github.com/mocktools/go-smtp-mock/blob/master/handler.go
and all uses of the handler type in handler_*.go files

New issue checklist

[x] I have updated smtpmock to the latest version
[x] I have read the Contribution Guidelines
[x] I have read the documentation
[x] I have searched for existing GitHub issues

Issue description

There is insufficient test coverage on handler and where it is embedded (e.g. in handlerRset). I was able to break it with just one change in https://github.com/mocktools/go-smtp-mock/pull/125 - which was fixed in https://github.com/mocktools/go-smtp-mock/pull/127.
If I could break something like this so easily without a test failing then it's clear we don't have the essential unit test coverage.

@bestwebua Can you assign this to me? I will have time on this weekend to do it.

Good catch, @benjamin-rood 👍 Seems like good point to add one more file, handler_test.go with one test for handler#clearError() method: https://github.com/mocktools/go-smtp-mock/blob/34c709d397802ee214a9946e4104c1be84e5adb9/handler.go#L13-L15
Just one straightforward test with sessionMock about session that exactly recieved the call of clearError(). Also we have indirect calls of this one therefore we have 100% for this case.

Seems we have test for this case:
https://github.com/mocktools/go-smtp-mock/blob/1b2ab0e18fb8d75ceb08caf9663cbe175f6a9e57/handler_test.go#L10-L18
We don't need tests for this method, it's my bad, sorry.


Wildcard host bind with proxy cause hostname been cached.

Caddy version: 0.9 beta 2
Running on Windows Server 2012 R2
Caddy config:
https://*.example.com {
	gzip
	header / -Server
	tls cert.crt cert.key
	header / Strict-Transport-Security "max-age=31536000; includeSubDomains; preload"
	proxy / https://internal.example.com {
		transparent
		header_downstream -Server ""
	}
}

Expect behavior:
When I access a.example.com, the proxy pass a.example.com as host to backend server
When I access b.example.com, the proxy pass b.example.com as host to backend server.
Real behavior:
The first host I access will be cached in proxy.
For example, after caddy are started, I first access a.example.com, everything works fine.
Then I access b.example.com, the proxy still pass a.example.com as host to backend server.

Thanks for trying the beta! My guess is this is either due to some sort of persistent connection (reusing the http.Client maybe?) or something wrong with how we're setting the headers going upstream. We'll look into it! (I'm away from my computer for a bit.)

I am able to replicate this behavior. I used this Caddyfile though to simply slightly:
:2015
gzip
header / -Server
tls cert.pem key.pem
proxy / https://localhost:2016 {
	transparent
	header_downstream -Server ""
}

And I added a fmt.Println to reverseproxy.go so it printed outreq.Host when the request was proxied to the backend. First I loaded https://localhost:2015/abc and saw localhost:2015 being passed upstream, as expected. Then I loaded https://<IP_ADDRESS>:2015/abc and still saw localhost:2015 being passed upstream, which was not expected. /cc @evermax

Leaving this issue open since @pedronasser and I agree there is a better way to fix this.

I'd like to bump this, hoping there's an easy fix.  It's the only remaining issue before we switch to Caddy completely.

@mholt can you point us in the right direction?  Perhaps we can write a patch ourselves.

I've been working on this today... seems the replacer is not replacing a {host} value on the second request. (The function that returns r.Host from within the replacer's map is not being called.)

Found the problem. It's right here. It modifies the rules in-place, it should be changing the copied values instead.

@vibbow @martindale @pedronasser I believe this now fixed. Could you try it out to confirm?

@mholt just installed from master, seems to work now.  Thank you so much for fixing this!


Improving the README.md file

Hello! I'm a beginner participating in GSSoC'24, and I'd like to contribute to the project by adding useful videos to the README.md file. These videos will help beginners learn how to build Generative AI models.
@ashitaprasad, could you please assign this issue to me?
Thank you!

Please read the contributing guide properly to understand what needs to be done.


added new file and added the hello world program

What does the program do ?
a simple hello world in malayalam language

In what programming language it is written?
python

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Moving resources to resource bundle

Hello @justindhill!
You wrote that you still come here from time to time to maintain the repository.
This is a wonderful and simple library with many ready-to-go integrations that we are to this day using in our app.
We want to optimise our pod installation process: at the times of Xcode 10 we had to add :disable_input_output_paths => true to Podfile to mitigate CocoaPods/CocoaPods#8122. These days we found that every other pod moved to using resources_bundle except this one. I wrote a simple PR ( #81 ) to fix this. It would be cool if you could merge it and add version 1.7.2 tag to the master branch.
Thank you in advance!

Hi @OneSman7, I'll try to have a look at this tonight and get it merged. Thanks for the PR!

Hi @OneSman7, I'll try to have a look at this tonight and get it merged. Thanks for the PR!


Rename project_Get to project_Teams

And the rest call from /project/get/#{project} to /project/teams/#{project}

It makes more sense as a name

Hello, how can change team name?


Push ci changes to maintenance v4

Why is this PR needed?
We need to merge the CI changes done in experimental v5 branch to the maintenance v4 branch or the CI won't work for maintenance v4 branch.
Fixes #1662
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
merge CI changes from experimental v5 branch to maintenance v4 branch

@pablochacin Tests are failing. Do you think this is relevant for this PR?

 Please run "terraform init".
2020-06-11 09:00:38 ERROR] Plugin reinitialization required. Please run "terraform init".
`

Could we be missing a commit?

The actual error is this:

CONFLICT (content): Merge conflict in ci/jenkins/jobs.yaml

```
+ make -f skuba/ci/Makefile pr_checks
/home/jenkins/workspace/asp-jobs_pr_test-libvirt_PR-1139/skuba/ci/tasks/rebase.py
First, rewinding head to replay your work on top of it...
Applying: Reorganize jobs per caasp version
Using index info to reconstruct a base tree...
M	ci/jenkins/jobs.yaml
M	ci/jenkins/templates/conformance-template.yaml
Falling back to patching base and 3-way merge...
Auto-merging ci/jenkins/templates/conformance-template.yaml
CONFLICT (content): Merge conflict in ci/jenkins/templates/conformance-template.yaml
Auto-merging ci/jenkins/jobs.yaml
CONFLICT (content): Merge conflict in ci/jenkins/jobs.yaml
error: Failed to merge in the changes.
Patch failed at 0001 Reorganize jobs per caasp version
The copy of the patch that failed is found in: .git/rebase-apply/patch
Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort".
Command 'git -c "user.name=CaaSP Jenkins" -c<EMAIL_ADDRESS>                  rebase origin/master' returned non-zero exit status 128.
Traceback (most recent call last):
File "/home/jenkins/workspace/asp-jobs_pr_test-libvirt_PR-1139/skuba/ci/tasks/rebase.py", line 20, in
subprocess.check_call(cmd, cwd=cwd, shell=True)
File "/usr/lib64/python3.6/subprocess.py", line 311, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'git -c "user.name=CaaSP Jenkins" -c<EMAIL_ADDRESS>                  rebase origin/master' returned non-zero exit status 128.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/jenkins/workspace/asp-jobs_pr_test-libvirt_PR-1139/skuba/ci/tasks/rebase.py", line 23, in
print(Format.alert("Rebase failed, manual rebase is required."))
NameError: name 'Format' is not defined
make: *** [skuba/ci/Makefile:60: git_rebase_check] Error 1
script returned exit code 2
</details>

Notice the problem was not reported due to this exception:
Traceback (most recent call last):
  File "/home/jenkins/workspace/asp-jobs_pr_test-libvirt_PR-1139/skuba/ci/tasks/rebase.py", line 23, in <module>
    print(Format.alert("Rebase failed, manual rebase is required."))
NameError: name 'Format' is not defined

Against which branch is it rebasing?

Against which branch is it rebasing?

should I change these to be maintenance-caasp-v4 instead of master?
branch_name   = os.getenv("BRANCH_NAME","master")
https://github.com/SUSE/skuba/blob/maintenance-caasp-v4/ci/tasks/rebase.py#L9
rebase origin/master'.format(change_author, author_email)
https://github.com/SUSE/skuba/blob/maintenance-caasp-v4/ci/tasks/rebase.py#L18

@jordimassaguerpla  I'm still diagnosing the issue. I'll update the card when I have more information.

PR https://github.com/SUSE/skuba/pull/1158 should fix the issue with the rebase.
@jordimassaguerpla  can you cherry-pick the changes in https://github.com/SUSE/skuba/pull/1158 and add to this PR?

PR #1158 should fix the issue with the rebase.
@jordimassaguerpla can you cherry-pick the changes in #1158 and add to this PR?

done

@pablochacin tests pass :) . Should I merge?

Yes, test pass but the logic for multiple PR target branches is somehow broken:
+ make -f skuba/ci/Makefile pr_checks
/home/jenkins/workspace/caasp-jobs_pr_test_PR-1158/skuba/ci/tasks/rebase.py
Not a PR. Rebase not required.

@jordimassaguerpla https://github.com/SUSE/skuba/pull/1158 should be fixed. If you can cherry-pick it, I think this time it should pass.


R3LIVE数据集的点云染色是如何完成的？

按照r3live代码着色方式实现即可，可以把代码移植到mapOptmization.cpp中.

Would be nice have a clue to do this, as the algorithm is amazing.  Please consider add it, I would do it, but no clue off programming


Missing pipeline image in README.md

Hi @ritwikraha @ariG23498, nice initiative. 👏
The pipeline image is missing from README.md. Can you please add it. Thanks!

Thanks for your interest in the work. We will make modifications to the entire repository in some time.
In that process the README would change considerably.
Will keep the issue open and update once the changes are made.

Hi @ritwikraha @ariG23498, nice initiative. 👏
The pipeline image is missing from README.md. Can you please add it. Thanks!

Sometimes, images don't load....

Wow, looking at the pipeline, it seems really interesting. I am willing to contribute to it too.

Wow, looking at the pipeline, it seems really interesting. I am willing to contribute to it too.

More than welcome, the more the merrier! 😇


Minor: Generate the supported Spark builtin expression list into MD file

Which issue does this PR close?

Follow up on #331
Closes #282
Related #240
Rationale for this change
Generate the supported Spark builtin expression list into MD file instead of txt file. Txt file still exists as it provides the reason why some expression is not supported

What changes are included in this PR?

How are these changes tested?

This is very cool @comphead but it looks like it is not detecting any of the aggregate functions that we support?

@andygrove @advancedxy I fixed the test, implementing extra parsing and manual small tests if the parsing is complicated. I hope now we have better picture.

Thanks @advancedxy I fixed the flaws you mentioned. However I'd like to make refactoring you recommended in followup PR, this PR getting too large for review

@andygrove I fixed all the comments, however you are right, sometimes we support partially the function.
means part of syntax or some value range not supported.
here comes an idea for follow up PR to introduce partially supported status(or similar) with the reason why it is supported partially

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 34.23%. Comparing base (9ca63a2) to head (e8f3b77).
Report is 27 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #455      +/-   ##
============================================
+ Coverage     34.18%   34.23%   +0.04%
+ Complexity      851      806      -45
============================================
  Files           116      105      -11
  Lines         38570    38488      -82
  Branches       8531     8562      +31
============================================
- Hits          13187    13175      -12
+ Misses        22612    22554      -58
+ Partials       2771     2759      -12

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


Open the popup of a marker in the programm

Hi,
My application needs to programmatically open the popup of a marker. After hours of trying I am stuck. The default Leaflet functions are not working in my code (openPopup, getPopup, ....). The data object also does not contain a reference to the popup.
Does anybody know how to open a popup of a certain marker by code? I could not find a method to access the array with prunecluster markers.
Best regards.
Mart

Hi,
This feature is not implemented. The markers and popups are created on the fly so that's why you cannot find them.
It's not trivial because if you want to open a popup attached to marker, the marker must be displayed. But it's perhaps inside a cluster, and then do you want to disable the clusters or zoom enough to not have a cluster anymore?
You could hack a bit by accessing some private properties on the leaflet map object or prunecluster if you know for sure that it is currently displayed.
In the past, if I wanted to have more interactions with a marker, I was removing/filtering it from prunecluster and adding it again with normal Leaflet methods.

Hi,
Thx, I will hack around a bit.
Best regards.


Microsoft.PowerShell version 7.2.2

[x] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Should I close this PR

Yes.

Should I close this PR

Yes.

Thanks for your guidance and I hope I've not caused much trouble to you.


Fix warning in small feature

from https://groups.google.com/d/msg/vim_dev/xjRF0mGFmg4/2V2nKmXzBAAJ

Codecov Report

Merging #3255 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3255      +/-   ##
==========================================
+ Coverage   76.48%   76.49%   +0.01%
==========================================
  Files          93       93
  Lines      136826   136825       -1
==========================================
+ Hits       104648   104663      +15
+ Misses      32178    32162      -16

Impacted Files
Coverage Δ

src/ex_docmd.c
77.69% <100%> (ø)
:arrow_up:

src/ex_cmds.c
79.81% <0%> (-0.23%)
:arrow_down:

src/os_unix.c
57.38% <0%> (-0.19%)
:arrow_down:

src/ex_cmds2.c
82.63% <0%> (-0.1%)
:arrow_down:

src/normal.c
74.35% <0%> (-0.08%)
:arrow_down:

src/getchar.c
75.16% <0%> (-0.05%)
:arrow_down:

src/if_py_both.h
76.57% <0%> (-0.01%)
:arrow_down:

src/gui_gtk_x11.c
48.03% <0%> (ø)
:arrow_up:

src/channel.c
83.11% <0%> (+0.03%)
:arrow_up:

src/screen.c
76.75% <0%> (+0.04%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Simple comparative benchmarks

We should have a simple comparative benchmark (with prose) comparing using AnnData and our on disk format to alternative structures.
Initial measurements could look at:

IO time
IO memory
On disk size
In memory size

Alternatives to compare against include:

Serialized R object (SeuratObject, maybe also SingleCellExperiment)
Loom

Variables to consider here include file format (hdf5 vs zarr), chunking, and compression.

Good resources to use here include:

Benchmarking chapter from Jake VanderPlas' Python Data Science HandBook
loompy docs
Seurat docs


10 rmi 00009 moderator deletes a comment - new

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[Check] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[Check] My code follows the style guidelines of this project
[Check] I have performed a self-review of my own code
[Check] I have commented my code, particularly in hard-to-understand areas
[ ] Three other reviewers have approved this PR

Finished use case for Moderator deletes comment (Missing UI and Database connection)
Added new instance variable to user (Access level, moderator has access level 1, all other users have access level 0)
Added new moderator class and comment class

I don't see any changes... Maybe check if you pushed the changes files

Can you see it now?


Readme: fix typo

Replace aims the combine the capabilities with aims to combine the capabilities in the readme.

Sorry this fell through the cracks. Thank you for the fix!


Add entry point

I'm a distro packager and the generated /usr/bin/pgcli doesn't work for me and I need more control. I need my own entry point so I can execute the module using python -m pgcli.main.

:+1: Thank you for the PR.
:rabbit2:


Emulator modification

Closes #29
Implemented mods for the emulator by adding objects, roads and trees to imitate a city environment.
:ballot_box_with_check: Definition of Done checklist

[x] Meaningful title and valuable description in pull request
[x] Code change tested manually if not covered by automated tests
[x] Pull request linked to one or more issues/stories
[x] Acceptance criteria of the linked stories satisfied
[ ] Code change comprehended by those who approved it
[ ] State of target branch maintained or improved after the code change

Needs more roads and obstacles

Dont add the .import folder to github please

This is tied to the obstacle avoidance milestone because the emulator mod is created for that exact purpose


java.lang.NoClassDefFoundError: codechicken/nei/config/KeyBindings

Hi covers,
A quick bug report for you;
Important Info

Minecraft 1.10.2
Minecraft Forge <IP_ADDRESS>5
CodeChicken Core 1.8+ <IP_ADDRESS>
NotEnoughItems 1.8+ <IP_ADDRESS>

Issue Description
Crash to desktop when looking up recipes for certain machines still supporting the old NEI handler causing this crash to occur (http://pastebin.com/jP7a9vw9)

That crash report is probably mouse tweaks somehow, the title of the issue isn't in that crash either..

oh woops wrong darn pastebin

@covers1624 there u go

Ohh, tell TFox to update NEI to the latest from CB's site.

infact, ill push it to curse.


REGRESSION: CMS_final: do not ignore CMS_dataFinal result

The CMS_dataFinal result is important as signature may fail, however, it
is ignored while returning success from CMS_final.
The SMIME_crlf_copy result is returned while it is ignored in the rest
of the openssl code.
Prefer the CMS_dataFinal result.
This is regression from openssl-1.
Blame finds c1669f41ei as root cause of regression.
Signed-off-by: Alon Bar-Lev<EMAIL_ADDRESS>
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@mattcaswell please review regression from https://github.com/openssl/openssl/pull/11884

We would need a signed CLA to be able to accept this if the check is added. See https://www.openssl.org/policies/cla.html

We would need a signed CLA to be able to accept this if the check is added. See https://www.openssl.org/policies/cla.html

Doesn't it falls in the trivial definition?

Without adding the check for the return value I suppose this could be acceptable with CLA: trivial. We can handle the check separately.

@t8m: I added a new commit to handle the SMIME_crlf_copy status in all usages.

The first commit would count as trivial IMO, both together wouldn't.

The first commit would count as trivial IMO, both together wouldn't.

In any case, I sent the cla :)

This pull request is ready to merge

Merged to master and 3.0. Thank you.

Hi,
Thank you for your prompt response.
Should I send pull request for openssl-3.0 branch or is it done automatically?
Thanks!

Hi,
We (Canonical/Ubuntu) would like to know the security implications of this issue or if this has not been analyzed. It looks like data that should fail authentication would be reported as succeeding but the lack of any mention of a security issue for this is surprising.

This function is used during the finalization step of creating CMS signed or encrypted messages, not when decrypting or verifying signatures.
It means some broken message that will later fail parsing/decryption/signature verification can be produced.
I do not see any security implications here.

I see, thanks a lot for your answer. I'll be discussing this with the
reporter of the issue on our side to better understand their need.


typo fix : change docker version to 17.03.2

change docker version from 17.03.02 to 17.03.2

LGTM

thanks!


Android: Fix Gyroscope startUpdates native API

Hi guys! Thank you for the package.
There's one problem with Gyroscope on Android introduced by recent commits: the observable callback is never called, so basically it doesn't work. I use the following workaround for now:
DeviceEventEmitter.addListener('Gyroscope', d => { console.log(d); });

The culprit is startUpdates method. Actually, promises are kind of pointless here (and in Accelerometer code as well), but it's just a quick fix. I'd be great if you release the next major release with appropriate fixes on npm soon.

Hey @baygeldin, thank you for taking the time to a PR and thank you for raising the bug. Rather then doing this quick fix at only one place I would like to see these methods changed back to non-promise ones, as they can't fail anymore due to absent sensors. We would need to change all occurences through all platforms.
It would be cool to see another PR from you, otherwise I will try to tackle this this weekend 👍

No problem, see #56.


Sluggish MSE feed for Wyze v2 rtsp

I have both Wyze v2 and v3 on Wyze's rtsp firmware.  Tried with both go2rtc v1.2 and v1.3.  Wyze v2 camera MSE is very sluggish (verified on two units).  See spin wheel almost every 2-3 seconds.  The feed is really unusable.  Wyze v3 cameras work fine.  Did not find any report on such issue.  Anybody know why?   Thanks.

MSE only works fine for good stream and good network between go2rtc and viewer.
There is no way to fix wheel on MSE if you have it because you received data not in real time.


Enhancement - Notes or Description for model.

Can i ask to add the ability to add a small (say 500 char) description or notes per model?
When managing my library of models, i also have note files that i have had to manually keep, regarding print results, and future modifications for a better print result.

i believe it could be an additional column (varchar(500) on van_dam.models, and then have an entry on the page that read from that field, and a button to update the text in that field, similar to the one for printed, pre-supported, y axis up?.

Good idea! Such a good one, it's already on the todo list, in #675 😄 👍🏻


NumberBox not showing keyboard on Xbox when IsTabStop is set

Describe the bug
When a NumberBox control has IsTabStop = "True", it does not show the number keyboard, so it is effectively disabled as user cannot change the value at all.
Steps to reproduce the bug

Place a NumberBox control on page
Set IsTabStop property to "True"
Set IsFocusEngagementEnabled  = "False" (or "True")
Set TabIndex = "1" (or any number)
Deploy to Xbox One

Expected behavior
Soft number keyboard appears when user selects the control and/or presses "A" button (the latter one depends on IsFocusEngagementEnabled )
Screenshots
No response
NuGet package version
Microsoft.UI.Xaml 2.7.0
Windows app type

[X] UWP
[ ] Win32

Device form factor
Xbox
Windows version
May 2021 Update (19043)
Additional context
No response

We try to make our controls accessible by default, so you shoulnd't have to mess with the IsTabStop settings for that reason (most controls are TabStops by default)


Added functional tests for soft deleting and restoring content

Added functional tests for soft deleting and restoring content, fixed the restore content job to also restore content children.

Coverage increased (+0.07%) to 69.631% when pulling 4c1c0e5867b7757ab147d9c60db289a0a73b5fb9 on tests_for_soft_deleting_content into 513252f2ddd8b01a277077c33ad8ac3a8f77b1bd on master.


Error: The method 'call' isn't defined for the class 'Progress'

mason make macosui_starter throws the following error:
Unable to execute hook: /Users/ben/.mason-cache/hosted/registry.brickhub.dev/macosui_starter_1.2.1+2/hooks/post_gen.dart.
Error: The method 'call' isn't defined for the class 'Progress'.
 - 'Progress' is from 'package:mason_logger/src/mason_logger.dart' ('../.pub-cache/hosted/pub.dartlang.org/mason_logger-0.1.3/lib/src/mason_logger.dart').
Try correcting the name to the name of an existing method, or defining a method named 'call'.
  flutterFormat();
               ^

This will be fixed in the next release, thanks for reporting!


Don't encode the colons separating the query fields in searchTracks()

As we can see in the api docs, we should not encode the colons which separate the query fields (artist, track, album etc)
https://developer.spotify.com/web-api/search-item/
The searchTracks() example from the main page call this url (with 'artist:Love')
https://api.spotify.com/v1/search?query=artist%3ALove&offset=0&limit=20&type=track
It should be https://api.spotify.com/v1/search?query=artist:Love&offset=0&limit=20&type=track
Otherwise the returned results aren't correct.

Sorry, looks like I was wrong...

That's OK @jitowix, let me know if there's anything else.


TypeError: Can not convert a list into a Tensor or Operation.

When I run "./scripts/train.sh 0,1 8" ：

Traceback (most recent call last):
File "train_pixel_link.py", line 293, in
tf.app.run()
File "/ai/installs/anaconda3/envs/py27_tf/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "train_pixel_link.py", line 288, in main
train_op = create_clones(batch_queue)
File "train_pixel_link.py", line 252, in create_clones
train_op = control_flow_ops.with_dependencies(train_ops, pixel_link_loss, name='train_op')
File "/ai/installs/anaconda3/envs/py27_tf/lib/python2.7/site-packages/tensorflow/python/ops/control_flow_ops.py", line 3368, in with_dependencies
with ops.control_dependencies(dependencies):
File "/ai/installs/anaconda3/envs/py27_tf/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 5004, in control_dependencies
return get_default_graph().control_dependencies(control_inputs)
File "/ai/installs/anaconda3/envs/py27_tf/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 4543, in control_dependencies
c = self.as_graph_element(c)
File "/ai/installs/anaconda3/envs/py27_tf/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 3490, in as_graph_element
return self._as_graph_element_locked(obj, allow_tensor, allow_operation)
File "/ai/installs/anaconda3/envs/py27_tf/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 3579, in _as_graph_element_locked
types_str))
TypeError: Can not convert a list into a Tensor or Operation.

opencv_version:<IP_ADDRESS>
python:2.7.15
tensorflow:1.12
cuda:9.0
After I change tensorflow1.12 to tensorflow1.1,the problem still remains.

if bn_update_op is not None:
if bn_update_op!=[]:
    train_ops.append(bn_update_op)


ip routing when host has a ipsec tunnel

Version: v1.0.0 (18bd921c)

Describe the bug
The server is in a datacenter and it has ipsec tunnels to the local offices. The server can access all network devices from the local office. But inside the pods they can't access devices from the local office.
To Reproduce

Server with ipsec tunnel
Start a pod
curl from inside the pod to a web service (webcam in my case) and you will only see trying.

Expected behavior
Access the device like the host can.
Actual behavior
Timeout on the connection
Additional context
Might be related to #951 but the set-up is now without ipsec from the flannel and moved to the host(master) and router of the office to be able to connect more hosts on the office network.

I have a similar issue to this @ibuildthecloud @erikwilson
Server is on DO
Agent is at home on RPi3
Scheduled two pods with nodeSelector and appropriate architecture - both run a webserver. When I schedule alpine on the server, I can hit the webserver on the server, but not the one on the RPi. The reverse is also true - running alpine on the RPi - I can hit the RPi webserver, but not the DO webserver pod.
Using ipsec made no difference.

@erikwilson @ibuildthecloud PTAL. This seems like a basic use-case that should work, or be documented clearly that it’s not supported.
Agent behind NAT, i.e. Intel NUC or RPi
Server on public cloud with public IP
Agent joins server with IPSec or Wireguard option, both nodes can be listed, but containers scheduled on either host cannot communicate.

@johansmitsnl did you get any further with this? Have you tried the wireguard option instead?

@alexellis The way we run ipsec it will not traverse a NAT without configuration and is extremely limited. In short traversing a NAT is not supported with our ipsec setup.

I think if users want this use case of traversing a NAT they should open a different issue for a feature request to add NAT traversal. We have to use a different approach, most likely not ipsec.

@alexellis wireguard seems to work with NAT.

@alexellis I have not. I switched to a infrastructure VPN tunnel and not use the ipsec from K3S.

I just opened a bug that is really similar to this one (#1613), however to me it seems like it's not related to NAT transversal though.
@johansmitsnl given that you have found a workaround, mind describing your setup?

The work around I did was to run the pod in a node that in the target network.
The set-up I have is like this
Node 01 -- IPSEC on host (not in K3) --- IPSEC tunnel --- router -- Node 03
   |
Node 02

Node 01

ens3 Public IP
ens7 Local lan

Installation of Node 01 is: $ curl -sfL https://get.k3s.io | INSTALL_K3S_EXEC="--flannel-iface ens7" sh -
When a pod in node 01 wants to access the network of the node 03 I get this issue. But when I am not wihtin a pod on node 01 it works.
When a pod in node 02 want to access the network of node 03 it works.
Note that node02 routes the network to node03 though node01.
It looks like some routing issue where it does not see the route available for the IPSEC tunnel.


[spike] DCOS-56720: correct which side gutterSize appears for column direction

When setting a gutterSize and setting a direction prop to "column" or "column-reverse", the padding on the FlexItem remained on the left instead of moving to the top.
Screenshots:
Before:

After:

The only changes in this PR are:
packages/shared/styles/styleUtils/layout/flexbox.ts
packages/styleUtils/layout/flexbox/components/Flex.tsx
packages/styleUtils/layout/flexbox/stories/flex.stories.tsx
The rest are just snapshot updates


Connectivity package fails - Execution failed for task ':app:mergeDebugNativeLibs'.

Hi, everyone, I have this trouble:
`FAILURE: Build completed with 2 failures.
1: Task failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDebugNativeLibs'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
File 'com.android.builder.files.ZipCentralDirectory@24e15651' was deleted, but previous version not found in cache

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

2: Task failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDebugJavaResource'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
File 'com.android.builder.files.ZipCentralDirectory@51a6811e' was deleted, but previous version not found in cache

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

connectivity: ^0.4.4+1
Can anyone help me? Thank you

Hi @cristianfb1989,
Can you try flutter pub cache repair or flutter clean and then flutter run to see if it resolves your issue ?
If problem persists, please provide flutter doctor -v and a minimal complete reproducible code sample that shows the issue you are facing ?
Also curious, any reason you are using ^0.4.4+1 and not the latest version of it ?

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.

Open the project in Android Studio android and clean the project
Build -> Clean Project


Package is not published as a NPM module

Please publish this latest package on NPM.
When I'm doing "npm view is-number-object" I can see only<EMAIL_ADDRESS>Latest tag 1.0.4 is not published. Getting following error
is-number-object: No matching version found for<EMAIL_ADDRESS>

It is published; just like the issue you filed on is-boolean-object, this is likely caused by a broken internal registry.
https://npmjs.com/is-number-object clearly shows v1.0.4 is published.