Protential memory leak.

queue.c/destroyQueue: release needjobs_responders and clients if needed
eg: GCQueue() will leak clients if (q->clients && listLength(q->clients) == 0)

Thank you! Good catch, the list of responders was definitely leaked. For the clients, the matter is a bit more convoluted... Usually we should free queues only when there are no longer clients blocked into those queues, so this should be more an assert, basically! However there is another bug in flushServerData that calls destroyQueue without checking if there are clients attached.
So the right thing to do is likely:

Add the assert instead of the code to free the clients dictionary. However free it without asserting if it's just an empty dictionary.
Create a function that disconnects all the clients sending them an error Queue deleted by the system administrator while the client was blocking on it or alike, and use it in flushServerData.

Makes sense? Thanks again.

Hi @antirez .
Yes, make sense! I overlook the flushServerData. And sorry for the delay, feel illness for a week (:3J∠).

Add the assert instead of the code to free the clients dictionary. However free it without asserting if it's just an empty dictionary.

--- a/src/queue.c
+++ b/src/queue.c
@@ -104,8 +104,10 @@ int destroyQueue(robj *name) {
     skiplistFree(q->sl);
     if (q->needjobs_responders)
         dictRelease(q->needjobs_responders);
-    if (q->clients)
+    if (q->clients) {
+        serverAssert(listLength(q->clients) == 0);
         listRelease(q->clients);
+    }
     zfree(q);
     return DISQUE_OK;
 }

Create a function that disconnects all the clients sending them an error Queue deleted by the system administrator while the client was blocking on it or alike, and use it in flushServerData.

write a new function unblockClientsBlockedOnQueue(queue *q) in queue.c to disconnect and send error.
--- a/src/disque.c
+++ b/src/disque.c
@@ -527,6 +527,7 @@ void flushServerData(void) {
     dictSafeForeach(server.queues,de)
         queue *q = dictGetVal(de);
         serverAssert(queueLength(q) == 0);
+        unblockClientsBlockedOnQueue(q);
         destroyQueue(q->name);
     dictEndForeach
 }
--- a/src/queue.c
+++ b/src/queue.c
@@ -210,6 +212,25 @@ unsigned long queueNameLength(robj *qname) {
     return queueLength(lookupQueue(qname));
 }

+/* Unblock clients blocked on a queue */
+void unblockClientsBlockedOnQueue(queue *q) {
+    if (q->clients) {
+        listNode *ln;
+        listIter li;
+        listRewind(q->clients,&li);
+        while((ln = listNext(&li))) {
+            client *c = listNodeValue(ln);
+            unblockClient(c);
+            addReplyErrorFormat(c, "Queue %s deleted by the system"
+                    "administrator while the client was blocking on it",
+                    (char *)q->name->ptr);
+        }
+        listRelease(q->clients);
+        q->clients = NULL;
+    }
+}
+
+
 /* Remove a queue that was not accessed for enough time, has no clients
  * blocked, has no jobs inside. If the queue is removed DISQUE_OK is
  * returned, otherwise DISQUE_ERR is returned. */
--- a/src/queue.h
+++ b/src/queue.h
@@ -83,6 +83,7 @@ job *queueNameFetchJob(robj *qname, unsigned long *qlen);
 unsigned long queueLength(queue *q);
 unsigned long queueNameLength(robj *qname);
 void unblockClientBlockedForJobs(client *c);
+void unblockClientsBlockedOnQueue(queue *q);
 void handleClientsBlockedOnQueues(void);

 #define NEEDJOBS_CLIENTS_WAITING 0 /* Called because clients are waiting. */

Does this patch make sense? :)

Sorry, suddenly I realize that the unblockClientsBlockedOnQueue is not correct, later I will give a new version. :-)

Hi @antirez , this is the new unblockClientsBlockedOnQueue()
It iter all the clients blocked on the queue, delete the queue from the client's bpop.queues.
If the size of bpop.queues is zero after remove the clients, unblocked it.
Does this make sense? :-)
void unblockClientsBlockedOnQueue(queue *q) {
    if (q->clients) {
        listNode *ln;
        listIter li;
        listRewind(q->clients,&li);
        while((ln = listNext(&li))) {
            client *c = listNodeValue(ln);
            dictDelete(c->bpop.queues, q->name);
            addReplyErrorFormat(c, "Queue %s deleted by the system"
                    "administrator while the client was blocking on it",
                    (char *)q->name->ptr);
            if (dictSize(c->bpop.queues) == 0) {
                dictEmpty(c->bpop.queues,NULL);
                c->flags &= ~DISQUE_BLOCKED;
                c->btype = DISQUE_BLOCKED_NONE;
                server.bpop_blocked_clients--;
                /* The client may already be into the unblocked list because of a previous
                * blocking operation, don't add back it into the list multiple times. */
                if (!(c->flags & DISQUE_UNBLOCKED)) {
                    c->flags |= DISQUE_UNBLOCKED;
                    listAddNodeTail(server.unblocked_clients,c);
                }
            }
        }
        listRelease(q->clients);
        q->clients = NULL;
    }
}

Thanks @sunheehnus, I'll merge ASAP, currently trying to figure out what's wrong with #73, reported after #72 (both crashes fixed for this issue hopefully). If you happen to have a few spare minutes I would love to get your point of view about how such a stack trace as #73 happens, since the dictionary should have different hashing methods.

Hi @antirez ,
I find out that my former commit was not all correct and it messes with exported APIs.
I make the changes more clean and into one commit. :-)

Thanks :-) I'll review once ready.

Hi @antirez , It is ready now. :-)

Merged! Thanks.

@sunheehnus changed just a couple of things in 9a57b51


ISSUE#42 Added API to accept schema version as file instead of text on REST service and client

@shahsank3t You can use the newly added API for uploading schema versions.


【Neo3-GUI】Discussion Summary, Design & Implementation

Neo core developers and communities have been discussing about Neo3-GUI for a while. There are a few things that got most developers' attention about Neo-GUI. After meaningful discussion on GitHub as well as during Neo 2019 Community assembly event, now we have reached to a point that most of us have agreed on what Neo-GUI should be and how should we improve it. We think next step will be summarizing the whole discussion and move forward with Neo-GUI design and implementation.
This issue describes the overall discussions and corresponding summary, and will also include design and possible implementations as well as any up coming topics should be discussed by core developers and community for Neo-GUI.
Goals for Neo3-GUI
From all the discussions for Neo-GUI on GitHub as well as Community Assembly event, some goals/targets have been confirmed for Neo3-GUI. In order to make Neo3-GUI more user-friendly and widely adapted by more developers, we should improve Neo-GUI for the following:

Neo-GUI should be cross-platform
Neo-GUI should be the UI version of a full node and keep updated with Neo-CLI release
Enhance UI/UX  to provide better user's experience for developers as well as users (blockchain purist)
Back-end need to be updated to Neo3 and full function support

Topics have been discussed

UI framework discussion - Done  (#497)
Community have proposed few great cross-platform UI frameworks for Neo-GUI, including Xamarin, Avalonia, Electron, etc. We have done some research and testing, and then discussion the conclusion on GitHub. Eventually, most community members agree on Electron.

Wireframes for Gui  - Done (#498)
Under this issue, we have discussed the functions for Neo3-GUI. And thanks for the great work from @lock9 's team, they have finished the design for Neo3-GUI wireframe (Prototype).

Add gui for neocli.  (#462)
After discussion with @lightszero , we think that it may not be the best solution. The reasons are as follows:
Increase the cost of version control and release.
Redundant client file will make size increase a lot.
Gui has different target users, better to implement it independently.

At this point, we have discussed and agreed on the goals, UI framework and wireframes for Neo-GUI.
With the release of Neo3 preview1, we believe that we should start design and  develop Neo3-GUI asap and deliver a more mature and user-friendly version along with Neo3 development progress.
Tech-Stack for Neo-GUI
Based on the above detail, we have discussed with Neo community and decided to develop Neo3-GUI with  the following framework:

Back end: .Net core 3.0
Front end: Electron + React
Communication: WebSocket + RPC

We will use MVC model to decouple the front-end with back-end so that it will be easy and more organized to develop Neo3-GUI as well as maintain and enhancement later on.
Overall structure of Neo3-GUI
We have designed an overall structure for Neo3-GUI based on the above discussion results, which is open to discussion for further improvement. Thanks again for the great discussion.

Next Step - Development & UI design
If there is no more  suggestions and discussions about overall structure and tech-stack, we will start UI design based on prototype that Rick provided as well as front-end development and back-end development for Neo3-GUI.
@neo-project/core, @erikzhang @shargon @lock9 @lightszero @vncoelho @igormcoelho  @devhawk ,  Your suggestions and comments will be well appreciated and valuable for the enhancement of Neo-GUI.
If you agree with above plan, please react with 👍 . if not, react with  👎 and some comments .
Thanks for all suggestion!

I am not used to neo-gui, but I will try to read as soon as possible.

We are going to start designing UI/UX for Neo-GUI and also build the foundation based on discussed cross-platform framework.

I don't really understand the diagram. Is this the required structure for Electron?

I don't really understand the diagram. Is this the required structure for Electron?

Right, this diagram represents the basic structure which front end is electron, back end is process depends on Neo.dll

I have updated the description of this issue.
Added the Neo-GUI page list to help prototype & UI/UX design. Also Neo-GUI prototype design is already in progress.
Next step will be UI/UX design for GUI page by page, also development.


bug fix for left hand panel view on mobile (presence indicators and u…

…nread message counts were not populating properly)

Testing with deploy preview and everything looks good.  Merging


Create 0000-01-02-IbukunOD.md and edit it

created a file and edited the comments section

Second comment


sudo command not found? in /amplify.sh?

** Please describe which feature you have a question about? **
Build fails on deploying Frontend
** Provide additional details**
2020-02-16T03:14:38.602Z [INFO]: DONE  Build complete. The dist directory is ready to be deployed.
2020-02-16T03:14:38.602Z [INFO]: INFO  Check out deployment instructions at https://cli.vuejs.org/guide/deployment.html
2020-02-16T03:14:38.647Z [INFO]: # Executing command: sudo apt-get install awscli
2020-02-16T03:14:38.648Z [WARNING]: /codebuild/output/src661412030/src/****/amplify.sh: line 18: sudo: command not found
2020-02-16T03:14:38.648Z [ERROR]: !!! Build failed
2020-02-16T03:14:38.648Z [ERROR]: !!! Non-Zero Exit Code detected
2020-02-16T03:14:38.649Z [INFO]: # Starting environment caching...
2020-02-16T03:14:38.649Z [INFO]: # Environment caching completed
Terminating logging...

i removed the line from Build settings & it worked.


Add feature to use deployments in replicate

This is refering to issue https://github.com/eth-sri/lmql/issues/252.

This pull request also contains a small fix in the docs changing the word transport to endpoint.

Thanks for this. Maybe @charles-dyfis-net can also have a look, if they are around.
Interestingly, according to this line, we already support lmql.model("replicate:<user>/<model>") and with this PR, lmql.model("replicate:deployment/<user>/<model>"), which should be even more convenient then specifying a model name * and * endpoint.

Just tested this successfully. Thanks again, merging now.


[Bug]: Function mocking failure when package contains test source with only constants

Description
If a package has a test source with only const variables,
when using function mocking to call a mock function, the mocking implementation fails to load the particular class and results in a FunctionCallError.
This is thrown when trying to load the classes list from the mockFunctionClasses. When only constants are added in a test source file, a separate class is not generated for this file.
During mock implementation, it tries to load each class from a pre-decided list(At desugar level https://github.com/ballerina-platform/ballerina-lang/blob/v2201.3.0/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/desugar/MockDesugar.java#L379) and looks for the mocked function. When only constants are added in a test source file, a separate class is not generated for this file.
So, when trying to load that class at https://github.com/ballerina-platform/ballerina-lang/blob/v2201.3.0/misc/testerina/modules/testerina-core/src/main/java/org/ballerinalang/testerina/natives/mock/FunctionMock.java#L178, it results in a ClassNotFound exception.
We need a way to skip adding the test sources with only constants to the class list.
Steps to Reproduce
Create a ballerina project and create the following structure.
.
├── Ballerina.toml
├── main.bal
└── tests
    ├── main_test.bal
    └── myconst_test.bal

Add the following to main.bal file.
import ballerina/io;

public function main() {
    io:println("Hello, World!");
}

function isRoleExist(string roleName) returns (error|boolean)? {
    return true;
}

main_test.bal
import ballerina/test;

@test:Mock { functionName: "isRoleExist" }
test:MockFunction isRoleExistsMockFn = new();

@test:Config {
}
public function testGetValidAppRole() {
    test:when(isRoleExistsMockFn).call("mockRoleExist");
    test:assertEquals(isRoleExist("roleName"), false);
}

function mockRoleExist(string roleName) returns (error|boolean)? {
    return false;
}

myconst_test.bal
const testVal = 90;

bal test will result in a test failure with the below output.
error message : <FunctionCallError> 'error FunctionCallError ("dilhashanazeer.mocktest1$test.0.tests.myconst_test")'
		    Assertion Failed!

		    	expected: <boolean> 'false'
		    	actual	: <FunctionCallError> 'error FunctionCallError ("dilhashanazeer.mocktest1$test.0.tests.myconst_test")'

		    	at ballerina.test.0:createBallerinaError(assert.bal:43)
		    	ballerina.test.0:assertEquals(assert.bal:112)
		    	dilhashanazeer.mocktest1$test.0.tests.main_test:testGetValidAppRole(tests/main_test.bal:10)

		0 passing
		1 failing
		0 skipped

Affected Version(s)
2201.2.3, 2201.3.0
OS, DB, other environment details and versions
No response
Related area
-> Test Framework
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

The same behavior might be reproducible for bal files with:

Variables only
Comments only

We need to investigate why we can't add only the classes with mocks to the mockFunction classes list instead of adding every bal file.
https://github.com/ballerina-platform/ballerina-lang/blob/v2201.3.0/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/desugar/MockDesugar.java#L379


Face detection using IP camera

Is your feature request related to a problem? Please describe.
We have surveillance IP camera to detect faces from 10-20m distance. we want to detect faces at low lighting conditions using javascript framework.
Describe the solution you'd like
We use dlib for face recognition (using python framework). Using detected faces from jeeliz framework and process it using python framework.
Describe alternatives you've considered
we  tried using haarcascade.xml for face recognition it is detecting  non-faces as faces and the image quality is very low.
Pls suggest how can we use your js framework along with python framework.

Hello,
Thank you for your interest for our library
Our framework requires an access to the GPU, it cannot run on standard server side.
We make it run on server side to provide an image fallback for https://jeeliz.com/sunglasses with these tricks:

the server runs with ubuntu 16.04 with a blackbox window environment (quite minimalist)
we use slimerjs with node.js, which is a headless browser. Unlike phantom.js, it supports WebGL and it is based on firefox. It has the same interface than phantom.js.
But this is only to process images, and there is a problem because slimerjs seems not maintained and it does not work with the last firefox versions (above FF57 since they changed the rendering engine), so we have to use FF52.

I think if you want to do things simply, you have to get the video stream in the web browser, inside a  element (not a flash video or other fancy plugin and to process it fully client side. I see 2 solutions:

maybe you can stream the video image of the webcam into a  element using mediaStream API and then process it with our API. We can then add a tweak to disable the user direct webcam request, it will be quick to implement :) ,
or you can use a standard USB webcam and use it as a wireless webcam with usb2air : https://www.amazon.com/HuddleCamHD-HC-USB2-AIR-Wireless-Extender-Bandwidth/dp/B073RSW19H

Hi,
I have added the possibility to detect faces on custom  elements. There is a demo here (included in the repo) where the faces is detected from a MP4 video file:
https://jeeliz.com/demos/faceFilter/demos/canvas2D/fromVideoFile/
I think if you manage to stream the video from your IP camera to a  element (you can take a look here : https://www.wowza.com/community/questions/2442/how-to-re-stream-video-from-an-ip-camera-to-html-5.html ) then you will be able to use our lib.
I am interested by your feedbacks !
Cheers,
Xavier

Hi @vijaysagi1
Have you managed to use this lib for you problem ? Have you tried the solutions I have suggested above ?

No news -> I close the issue. If you have the same problem feel free to re-open the issue.


Register in packagist

Hi @zpao ,
I am really sorry to bother you with that.
I have a final request to ask you. Is this possible to publish it on packagist ?

It's already published - https://packagist.org/packages/reactjs/react-php-v8js. cc @talyssonoc who did that (also, can you add me as an owner)

Sure, @zpao, what's your Packagist username ?

zpao

Just did it !

Thanks @talyssonoc!
@qpautrat Looks like it's updated now for 1.0.0 and I setup the webhook so it'll get autoupdated in the future.

@zpao Great, thanks !


I want to replace sharing to VK with sharing to whatapp. Can you help me? Thank you!

I want to replace sharing to VK with sharing to whatapp. Can you help me? Thank you!
I have replaced VK as the whatsapp in my topic, but I whatsapp not be able to call thumbnails when sharing, because I am at the entry level, maybe my code is not standardized enough, can you help me correct it? My website is: www.djkcc.cn。

{if $theme_config->social_vk}
<a href="whatsapp://send?text={$current.TITLE},view on Link:{$http_scheme}://{$smarty.server.HTTP_HOST}{$smarty.server.REQUEST_URI}{if isset($current.derivatives.large)}&media={$http_scheme}://{$smarty.server.HTTP_HOST}{$smarty.server.REQUEST_URI}/../{$current.derivatives.large->get_url()}{/if}"
onclick="window.open(this.href, 'vk-share', 'width=490,height=530');return false;" title="Share on Whatsapp"{if $theme_config->social_buttons} class="btn btn-sm btn-social btn-raised btn-vk"{/if}>
{if $theme_config->social_buttons} Whatsapp{/if}

{/if}

The correct thumbnail cannot be displayed here.


fix(Glaze): Fix playback bar

@CharlieS1103 can you review this?

@afonsojramos it all looks good, however I already patched these issues on my own glaze theme repo, i'm going to merge this in and continue all glaze theme development on the official spicetify-themes repo.


Create _config.yml

User description

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

[!NOTE]
I'm currently writing a description for your pull request. I should be done shortly (<1 minute). Please don't edit the description field until I'm finished, or we may overwrite each other. If I find nothing to write about, I'll delete this message.

Description

Introduced a new _config.yml file for Jekyll site configuration.
Configured the theme, title, and description for better site presentation.

Changes walkthrough 📝
Relevant filesEnhancement

_config.ymlAdd Jekyll configuration for site customization

_config.yml
Added configuration for Jekyll theme.  Set title and description for the site.  Enabled downloads for the site.

+4/-0

💡 Penify usage:
Comment /help on the PR to get a list of all available Penify tools and their descriptions

Summary by CodeRabbit

New Features

Introduced a new theme for the Jekyll site.
Updated site title to reflect the purpose of the GitHub Action.
Added a description for setting skills and generating badges.
Enabled the display of download statistics.

@gstraccini npm dist

@gstraccini prettier


[Feature Request] Longer code segments

Could you please change the title to

[Feature Request] Longer code segments

I agree that 600 might be too short.  The picture size is limited enough already.

Could you please change the title to

[Feature Request] Longer code segments

I agree that 600 might be too short. The picture size is limited enough already.

can we make it  1200 word it's size of 2 sinippet  and merge them toghter with some package like
https://github.com/lukechilds/merge-images
with check of words like
take the first 600  words and give theem snippet   same with the rest of code (600 words )
and grab those 2 images merge them together

This depends on the resolution of #136.


Like functionality is not working for searched output for People tab

Staging env - Tested on Android 9 Mobile and Windows 10 laptop on Chrome Browser
https://www.hippovideo.io/video/view/6758057

Please the video if the above link is not working
ezyzip.zip

Steps 1. Put any keyword and search
2. Goto people tab
3. See any person  posts
4. Like the post

LikenotworkingforPeoplesearchedresult2.zip

Hi, Its not about like button count issue. Its the User not able to like the posts itself. #1754 needs recheck as its not fixed. Please confirm if I am correct!

Hi, Its not about like button count issue. Its the User not able to like the posts itself. #1754 needs recheck as its not fixed. Please confirm if I am correct!
Hi , Please refer enclosed GIF ,the earlier like button is not working correctly  before. After the fix like button fuctionality is working fine also user able to like the post itseslf. Once you review the GIF , if you still have clarification please let me know

this is NOT an issue :rotating_light:
What you are seeing in the "People" (and "Organizations") tab is NOT a post, it's a user card, a mini version of a user profile showing their name and their "about" section.

think of it like this:

this is NOT an issue 🚨
What you are seeing in the "People" (and "Organizations") tab is NOT a post, it's a user card, a mini version of a user profile showing their name and their "about" section.

think of it like this:

Actually @Zaydme
the issue is when you navigate to user post from 'People' then on that post user is not able to click 'Like' but can comment.
see this GIF
The user is able to like his own post but not other users.

Ayyy :joy:
Thanks for the clarification @rabbi08


FIX: Add examples to CMake install correctly.

Coverage remained the same at 96.127% when pulling e36df096a7de9a62324f947427e9d6a3031de644 on zanebeckwith:fix-install-examples into d2e8beeabe96da6dfe0ff3415f761f9991f60b68 on xaptum:master.


DensityEnergyFromPressureTemperature can produce undesirable results

When calling DensityEnergyFromPressureTemperature for a SpinerEOS using inputs near STP (e.g. P = 1.0e6, T = 297), the solution returned is one of the "wrong" solutions.
Specifically with a sesame2spienr input of simply matid = 3337 to generate the materials.sp5 file, calling
DensityEnergyFromPressureTemperature(1.0e6, 297.0, nullptr, rho, e); returns rho=0.00257975 -- the vapor phase.  While this is technically a root of the table, it's not the one I want when nothing else is specified.
For a host code/end user wanting to specify initial conditions in PTE for a material like a metal, the likely situation is that they will want the solid phase rather than the vapor phase.  Perhaps there could be some way to override this behavior, but I don't think simply treating rho, e as initial guesses is the best choice because they could reasonable be uninitialized variables or zeros or something.

Is it actually true that you will want the solid phase when initializing in PTE? Because if it's the solid phase, it's probably in tension, and will not find an equilibrium with, say, air.

If I'm specifying a positive pressure, I would expect that the return value would not be in tension.  I think you are confusing with a PTE solver given a certian state Initializing in PTE means that I specify P and T and want the other state variables.  I want to specify an initial pressure and temperature (e.g. STP) and have it return the state that I want (solid phase in the case of metals).  This is already done in the PTE solver by bracketing the solution above the minimum pressure along the isotherm.
The example I gave returns a valid solution for that isotherm --- the 297K isotherm crosses 1 bar three times, I just want the one at 8.9 g/cc and not at 2e-3 g/cc.
We could argue about the use cases for DensityEnergyFromPressureTemperature but I don't see it being used much outside of setting up an initial state for a simulation.

Aaah I see what you mean. I think this probably requires some thought. A few things:

Should DE from PT be split into two or more scalar output functions?
I don't like the idea of automatically always picking the lower root, but we could perhaps try to offer a switch or something?

One comment that may be helpful: of the three roots available, the one Chad wants is the lowest energy state. In most cases I can think of, the lowest energy state corresponds to how we understand the material in our daily experience.
I agree with Chad that 99% of the time somebody uses this function, they are asking what the conditions are for a material at atmospheric conditions to initialize a hydro run of some sort. Sure it's possible that somebody may want to initialize copper vapor at 298K, but perhaps it's a better idea to provide a flag or switch for that case instead of the most-used case.

Could the rhopmin function be useful in this context ?


Redirect a manual and sections

Adds a bin script wrapped around a utility class
Passing a base_path and destination will find the manual with the slug matching the base_path and redirect it and the associated sections by publishing redirects to the publishing api.

The task for Specialist Publisher Rebuild is here.
eg. bin/redirect_manual_and_sections /manual-url /redirected-url
with @steventux

Does this deal with the <manual-slug>/updates route ?  I don't think it does, but TBH I'm not sure, maybe the updates route is in the :links in the content store?  It's fairly confusing that this works over many different levels of the data - uses the content store to get links, directly uses ManualRecord and SpecialistDocumentEdition rather than the repository objects in the wiring.  But!  It's arguably simpler for doing it this way.  As we're re-writing SP anyway, maybe it's not a problem to just Get It Working™ for now and make sure the re-write includes similar rake tasks (or UI) for this functionality.

Most command-line utilities in this app seem to be triggered from bin, rather than rake, and having these two places has caused confusion this week. I'm planning to create a PR which removes lib/tasks/publishing_api.rake to simplify things(the other tasks in this file are obsolete, I believe). Is it possible to turn this into a bin script?

I've updated this code to turn the rake task into a bin script. This ought to be ready to merge – could somebody cast an eye on this? /cc @tuzz

Actually, just saw @h-lame's comment about the <manual-slug>/updates route... Closing until that's added.

This hasn't been touched since 10 May 2016 so I'm deleting the branch.


Split liquibase-core and liquibase-commercial jar files

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[X] Breaking change (fix or feature that would cause existing functionality to change)

Description
Rather than having liquibase-core contain both the functionality from this repository as well as the commercial code, keep "liquibase-core" as the base, open source library and create a new "liquibase-commercial" as the additional proprietary/commercial functionality.
The new liquibase-commercial.jar file will be available:

In maven central
As an additional asset on release pages
As an additional jar in the liquibase-artifacts.zip and liquibase-artifacts-<BRANCH_NAME>.zip build artifacts

For consistency, the liquibase-core.jar file will now be stored as liquibase-core.jar on the build and release assets pages rather than liquibase.jar like it previously was.
Impacts
CLI
The liquibase CLI distributions will continue to include both liquibase-core.jar and liquibase-commerical.jar in the internal/lib directory**, so there is no impact for CLI users,** regardless of what distribution is used (tar.gz file, zip file, or docker image). The existing liquibase launch scripts have been updated to use the correct jar files.
While we will not ship a CLI distribution without liquibase-commercial.jar, removing any non-open source code is simply a matter of deleting internal/lib/liquibase-commercial.jar
NOTE: The liquibase-core.jar file was previously called liquibase.jar and stored in the root of LIQUIBASE_HOME. So anyone working directly with the liquibase.jar file will need to update their usage to use internal/lib/liquibase-core.jar.
NOTE: If you upgrade over an existing LIQUIBASE_HOME installation, the old liquibase.jar file will continue to exist because there is no logic to clean up old files. It can be deleted at your leisure, and it will not be used by Liquibase if left.
Maven Plugin
The Liquibase-maven-plugin will now depend on both liquibase-core and liquibase-commercial instead of just liquibase-core. This means that anyone using the maven plugin as:
            <plugin>
                <groupId>org.liquibase</groupId>
                <artifactId>liquibase-maven-plugin</artifactId>
                ...

will continue to be able to use both free and paid functionality with no changes to their configuration.
Servlet / Spring / Ant Integrations
See "User Managed Dependencies" below
User Managed Dependencies
Any projects that were directly or indirectly referencing or using "liquibase-core" will no longer include the commercial functionality. To continue to use Liquibase Pro features, you need to have both liquibase-core.jar and liquibase-commercial.jar as part of your project.
If you are managing your project using a tools that follows Maven managed dependencies (Maven, Gradle, Ivy, etc.), the easiest approach is to change your dependency on liquibase-core to liquibase-commercial. Because liquibase-commercial defines liquibase-core as a dependency, you will automatically get both included in your project without having to manage multiple dependencies yourself.
If you are not directly referencing liquibase-core yourself, but it is comes along as a transitive dependency (like if you use spring-boot), add org.liquibase:liquibase-commercial as an additional dependency in your project.

Things to test:

In the build's liquibase-artifacts-*.zip artifacts:

Both liquibase-core.jar and liquibase-commercial.jar exist

The liquibase-core.jar:

Does not include a liquibase/pro directory

The liquibase-maven-plugin.jar's:

META-INF/maven/.../pom.xml file has both liquibase-core and liquibase-commercial as dependnecies

The liquibase-commercial-jar:

Has liquibase-core as a dependency
Has correct license and other metadata
Does not include any other dependencies

I think this is all working now, including the "re-version" logic. I tested that with a test run of the action.

TEST:

MacOS Installer:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key: PASS

Windows Installer:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key: PASS

Ant Integration:

Pro Features access denied without Pro Key and with liquibase-commercial.jar included: PASS
Pro Features access with Pro Key and with liquibase-commercial.jar included: PASS
Pro Features unavailble with Pro Key but without liquibase-commercial.jar included: PASS
Upgrade from core to Pro with liquibase-commercial.jar included and Pro Key: PASS

Servlet Integration:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key (not available at the moment but the code is accesable): PASS
Pro Features unavailble with Pro Key but without liquibase-commercial.jar included: PASS
Upgrade from core to Pro with liquibase-commercial.jar included and Pro Key(not available at the moment but the code is accesable): PASS

.zip file:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key: PASS

.tar file:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key: PASS

Spring-boot/Spring integration:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key (not available at the moment but the code is accesable): PASS
Pro Features unavailble with Pro Key but without liquibase-commercial.jar included: PASS
Upgrade from core to Pro with liquibase-commercial.jar included and Pro Key(not available at the moment but the code is accesable): PASS

Maven plug-in:

Pro Features access denied without Pro Key: PASS
Pro Features access with Pro Key: PASS
(there is no way to exclude liquibase-commercial.jar at the momment)

Post Release distributions pending:

Gradle Integration
liquibase.org/download
Hombrew
Choclatery
Docker

this PR makes me so happy
that is all


Restrict HTTPS survey to specific good-SSL sites

This restricts what HTTPS sites the HTTPS survey is shown for, to a set of known good-SSL sites. Also adds tests. Fixes #233

I also discovered that a recent Blink change broke the other UrlHandler method (GetMinimizedUrl). Since this new code depends on it, I fixed it in this CL also.

I can't find how to reply on outdated diffs so adding them here :p

Older versions of Chrome used to just output : when it fails to parse into a proper scheme://host. Now it outputs file:.

Acknowledged.

Good question. It works with chrome: because the bounds are (0, 6) and not (0, 5). To prove it to myself I added a test case for chrome://interstitials.

Ah, interesting. Sgtm.

I did that intentionally bc I want to catch all google.co.uk, google.co.in, etc. without adding a super long (and possibly incorrect) list. I figure google.example.com will be rare. What do you think of that decision? I added a test & comment to document.

Sounds reasonable.

FYI, here's how to comment on an older diff. In the timeline, you'll see this...

Click on the right and it'll open up the comment, where you can reply like usual.

OK, please take another look.

LGTM, and thanks for the tip about the comments.


[FLINK-1594] [streaming] [wip] DataStream supports self-connection

Added StreamEdge abstraction, so more than one edge can be defined between stream operators.
Added OutputSelectorWrapper to avoid directed emission spreading over all the code.
Added factories for reader and writer creation

Travis fails on StreamVertexTest.coTest() (it gets stuck). Locally it passes.

Looks good to me, and travis passes, I will merge tomorrow.

What about the stuck test? Did you look into it or did just a re run of Travis pass?

It was missing about 10 commits from the master, and after rebasing the tests ran properly. But I didnt investigate the issue

Ok seems like there is an issue with chained tasks, so lets not merge it until that is fixed. Gabor is working on it


support for navigating back to past stages

TODO:

actually disabled editing in individual components when viewing results from past (completed) stages

This is really nicely done!


CORE-19444 UTXO ledger batch inserts

Batch inserts in the UTXO ledger persistence code.
Postgres specific insert statements following the structure of INSERT INTO table VALUES (?, ?, ?), (?, ?, ?) where (?, ?, ?) is repeated multiple times so that multiple rows can be inserted within a single insert statement. Batching in this way significantly improves performance, especially compared to using the generic form of batching that PreparedStatement provides.
All UTXO ledger inserts that were previously inserted by looping over collections and inserting a row one at a time have been switched to insert in this way.
BatchPersistenceService has been added to encapsulate the batching logic to a single class. This takes care of two levels of batching:

Batching the number of rows inserted per insert statement.
Batching the number of inserts included in a batch of statements before contacting the database.

To support this behaviour change, queries need to be dynamic so that they can be told the number of rows included in a single statement and then produce a query with the correct number of (???) / value blocks.

build e2e


[Enhancement] heroku

Description
As a developer, I want  ot quickly test and deploy my code to ensure it is running as expected and can be reviewed. Heroku provides a nice solution for app deployments, based on different languages, repositories and code. It can also be integrated via ci.
Criteria

check if heroku can be used and how
document the possible options
provide an example to deploy something automatically
provide documentation how it can be used via cirrus-ci

Additional information

https://heroku.com/
https://www.bencollier.info/devops/architecture/2019/03/26/deploying-docker-containers-gitlab-heroku.html

I would like to participate in this enhancement.

Awesome, I will keep this in mind for the first application.


Don't generate dictionary representation

Until we can verify it compiles and works for all schema permutations.
The current implementation leads to a number of compilation errors and warnings that I'd rather ensure work before adding this to the specification.
Most notably the warnings are related to strict-selector-matching due to messaging an id variable.
The compilation error I hit was related was due to a property of type Map<String,Array<String>>. There is likely a simple fix but I'm disabling since models on master should always compile.

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger


Documentation, examples and demos for VIE are spread over too many locations

Take the http://github.com/IKS/VIE and create a http://github.com/IKS/VIE-widgets and move all VIE related stuff there. Maybe use viejs.org as project entry point and reference all: docs, examples, demos, descriptions. REMOVE all other related VIE pages, locations.

There may be a good reason for different placement of some widgets, for example CreateJS is its own project. I think more important would be to maintain the widgets list on the VIE website, as mentioned in #81


Cannot set button text background.

Normal

Hover

ui:stylePush({
        button = {
          normal = '#99999944',
          hover = '#99999900',
          ['text background'] = '#ffffff00'
        }
      })
			if ui:button('Button') then
				print('button pressed!')
			end
      ui:stylePop()

It's seems that the text background is the same as normal or hover.

I'll look into this, but one thing you can do to speed up the process is create a version of your test in C. Ideally, the C version would work as expected, meaning the problem is with LOVE-Nuklear and not Nuklear itself. You can find C tests in nuklaer/demo.

In C version, it has no text background and cannot change the text background.

OK, so if I understand correctly, the C version lists text_background as a button style property, but the property doesn't actually do anything. This would explain why it doesn't work in LOVE-Nuklear. In this case, you might want to start an issue with the upstream repository. If I misunderstood you, please let me know. Thanks for the issue.

I just copy the C version description and create a new issue https://github.com/Immediate-Mode-UI/Nuklear/issues/158

So, I think we should fix the problem of love version. Button should remove the text background.


How many frames per packet?

Overview
How many frames can there be in any video packet? I'm trying to index video frames without decoding but I'm unsure as to how many frames can be in each packet. From my limited testing, I've only seen packets with 0 or 1 frames.

This is really out of scope for PyAV, I think you'd be better served checking the FFmpeg documentation, PyAV just binds to its libraries.


WMS ID: 11049 - Updating "Implement custom authentication in APEX"

Numbers from lab folders now removed.


fix errors in DocStats comments

Comments for two fields of DocStats are reversed

Hi @wormday - sorry for the late reply. Do you have the idea of the reviewer? Committers are usually too busy to watch all pull requests and often overlook good PRs.

Hi @wormday - sorry for the late reply. Do you have the idea of the reviewer? Committers are usually too busy to watch all pull requests and often overlook good PRs.

What work do reviewers need to do? I have nothing to do recently, I am reading the source code of lucene, and I hope to do something.

Anyone can be a reviewer for this project. Browse the opened/active pull requests (there are always tons of PRs), choose one you are interested in and able to review, and make comments on it to improve the code. Only committers can merge it, though.

Anyone can be a reviewer for this project. Browse the opened/active pull requests (there are always tons of PRs), choose one you are interested in and able to review, and make comments on it to improve the code. Only committers can merge it, though.

I see. I'll try
thanks


(PUP-5983) Enable interpolation of nested data in hiera data provider

This commit improves the Hiera data provider so that it performs
interpolation of data found in arrays and hashes. The interpolation of
such data is recursive. Prior to this commit, only interpolations found
in strings at the topmost level were interpolated.

Sure this is exactly what hiera does both behavior- and datatype- wise? (I would not be surprised if hiera did something odd).

Yes I'm sure. Hiera produces the exact same result given the same values. I have a test for that in the Hiera code-base. Do you want me to submit that as a (maint) commit?

If you have an additional test for hiera that would be great (there are way too few tests there). Just push it as a (maint) as you suggested, no need to review.
However, stable branches are in lockdown until release, so both this merge and the push of the (maint) to hiera has to wait a bit.

Good to merge when block is lifted.

Ping @thallgren. This needs a rebase and you can then merge it to stable.


Add Contribution guidelines for keycloak-benchmark project

@ahus1
Thought to add the contribution guidelines based on the Keycloak template. I tried to trim it a little bit, hence kept this in a draft state. Its also not super urgent., but something we wanted to complete some time back.
Please add your suggestions or changes as needed.
Closes #244

Hi Kamesh, thanks for preparing this PR. I've reviewed the docs, and pushed one more commit.
Part of the change was due to Keycloak maintaining their docs in the in a separate repository. In this project, the docs should be part of the same project.
You write that this is not super urgent, which I agree with. At the same time as this is now started, we should complete it :-)
Even if we change this document later, it is better to have this document than not having it. We can still revise it later.

Hi Kamesh, thanks for preparing this PR. I've reviewed the docs, and pushed one more commit.
Part of the change was due to Keycloak maintaining their docs in the in a separate repository. In this project, the docs should be part of the same project.
You write that this is not super urgent, which I agree with. At the same time as this is now started, we should complete it :-) Even if we change this document later, it is better to have this document than not having it. We can still revise it later.

Thanks for reviewing it very quickly, just dropped a question on Building and working with the code base. Let me know your thoughts on this. @ahus1


[web] Ignoring assets MIME type

Use case
I have a js file that gets loaded from a service worker from the assets, but in debug I get the error that the MIME type is wrong. When running in production the web server handles the MIME type.
Proposal
Flutter seems to have no way of setting the MIME type of an asset.
Maybe a key like assets-mime-type could be added to the pubspec.yaml to set the type for every asset file if needed.
But there is the problem that not every character is valid yaml character.
Something like this:
flutter:
  assets:
    - image.png
    - test.js
  assets-mime-type:
    image.png: image/png
    test.js: application/javascript

You don't need to specify the mime type in the asset manifest (in fact, that will do nothing for release builds since a normal web server doesn't care about the manifest). Instead we just need to set the content type header correctly

I try
html.document.head.append(html.LinkElement()
  ..rel = 'stylesheet'
  ..href = 'assets/packages/foobar/assets/foobar.css'
  ..type = 'text/css');

and get Refused to apply style from 'http://localhost:8686/assets/packages/foobar/assets/foobar.css' because its MIME type ('text/plain') is not a supported stylesheet MIME type, and strict MIME checking is enabled.
Couldn't that be the same errror, Maybe a regression?


WIP: cmake: Initial standalone overlays builds. This is just the cmake

changes, there is a utils/build-overlays that will follow tomorrow.
rdar://problem/28707246

@swift-ci Please clean test

@swift-ci Please clean test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - e34bc36fa45424befc7592e744c1e9a6f0c6869a
Test requested by - @erg

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - e34bc36fa45424befc7592e744c1e9a6f0c6869a
Test requested by - @erg

@hughbe When I use PROJECT_SOURCE_DIR cmake sets that value to the overlay's directory, not the base swift/ directory. I had to revert the pathnames.
I added a utils/build-overlay script for people who might want to build one standalone.

@swift-ci Please clean test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - d3d0ee8c3b72ab99ff5a0a0519aeaf7c57b2716c
Test requested by - @erg

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - d3d0ee8c3b72ab99ff5a0a0519aeaf7c57b2716c
Test requested by - @erg

@swift-ci Please smoke test


[iPadOS] Stage Manager has incorrect inset when in top corner

Requirements to Reproduce

Must be using one of the following devices:

iPad Pro 12.9-inch (3rd generation and later)
iPad Pro 11-inch (1st generation and later)
iPad Air (5th generation)

If using a simulator, Xcode version must be at least 14.1
Must have Stage Manager enabled (Settings < Home Screen & Multitasking < Stage Manager = On)
Must be in Portrait orientation

Steps to Reproduce

Execute flutter run on the code sample.
Grab the handle in the bottom right corner and drag inward to make it as small as possible.
Click the three dots at the top of the app and select Add Another Window and then select a different app
Grab the handle in the bottom right corner and drag inward to make other app as small as possible too.
Grab three dots at top of the flutter app and drag it up so it's closer to the top of the screen.
Click the input field in the flutter app to open the keyboard.

Expected results:
Expect the flutter app would have no inset below the input field (so it should look the same as it did when the keyboard was closed).
Actual results:
Very small inset below the input field.

Notes
For some reason, when it's in this size and position in Stage Manager, the info provided by UIKeyboardFrameEndUserInfoKey does not seem accurate.
For example, with an iPad Pro 11 inch:
The screen's dimensions are 834 x 1194.
If the app is full screen, the keyboard's dimensions are 834 x 340. And its origin position is (0, 854). This is as expected. The origin.y is 340 (the keyboard's height) from the bottom of the screen (1194 - 854 = 340).
If the app is top right/left (like screenshot above), the keyboard's dimensions are 834 x 340. However, its origin position is (0, 661). This is not expected. For some reason, the origin.y is wrong for this use case.
If the app is middle or bottom right or left, the keyboard's frame is the same as if it were full screen.

Code sample

https://github.com/flutter/flutter/issues/104689#issuecomment-1139253289

flutter doctor -v

[!] Flutter (Channel master, 3.7.0-10.0.pre.50, on macOS 12.6.1 21G217 darwin-arm64, locale en)
    • Flutter version 3.7.0-10.0.pre.50 on channel master at xxx
    ! Upstream repository git@github.com:vashworth/flutter.git is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to git@github.com:vashworth/flutter.git to dismiss this error.
    • Framework revision 32da25053d (67 minutes ago), 2022-12-19 12:16:28 -0500
    • Engine revision a12dadfda6
    • Dart version 3.0.0 (build 3.0.0-35.0.dev)
    • DevTools version 2.20.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to
      perform update checks and upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at xxx
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.74.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.54.0

[✓] Connected device (4 available)
    xxx

[✓] HTTP Host Availability
    • All required HTTP hosts are available

The PR https://github.com/flutter/engine/pull/37719 made a best effort to fix Stage Manager issues, but this issue remains.
https://github.com/vashworth/engine/blob/aa6276e10ad6ded0beb653b624b47693b3b83b32/shell/platform/darwin/ios/framework/Source/FlutterViewController.mm#L1460-L1464
This part calculates how much of the keyboard intersects with the view to calculate the inset.
https://github.com/vashworth/engine/blob/aa6276e10ad6ded0beb653b624b47693b3b83b32/shell/platform/darwin/ios/framework/Source/FlutterViewController.mm#L1480-L1485

CC @MitchellGoodwin @cyanglaz
This doesn't seem related to text input but correct me if I'm wrong.


Misalignment in admonitions when using the ones provided by Admonition Panels plugin

Hi I'm using Logseq Flow Nord theme with admonitions provided by the Admonition Panels plugin (https://github.com/nmartin84/admonition-panels)
The visual rendering of such admonitions is somehow broken (see the misalignment between upper and lower part of the admonition)

No problems when using the regular admonitions provided by Logseq itself
#+BEGIN_TIP
Tutorial Videos: [Tutorial Videos | Cypress Documentation](https://docs.cypress.io/examples/examples/tutorials)
#+END_TIP

results in

My environment:

Logseq 0.9.8
Admonition Panels plugin 0.1.1
Logseq Flow Nord  0.1.2

Sorry, I can't access https://github.com/nmartin84/admonition-panels now :(

Me too. I wonder whether that plugin is still "alive" (i.e. "supported")
Nevertheless the plugin is available in the Logseq Plugin Marketplace. Thus anyone can install it and encounter the issue described above, right?
If that plugin is "dead", I guess it should be removed from the list of the ones that can be installed but I've no idea who to talk to in Logseq management team to handle this situation

Me too. I wonder whether that plugin is still "alive" (i.e. "supported") Nevertheless the plugin is available in the Logseq Plugin Marketplace. Thus anyone can install it and encounter the issue described above, right?
If that plugin is "dead", I guess it should be removed from the list of the ones that can be installed but I've no idea who to talk to in Logseq management team to handle this situation

Could you please tell the plugin name in the Logseq Plugin Marketplace.

Adminition Panels (as in the title of this issue)

Adminition Panels (as in the title of this issue)

Can you find in  https://github.com/logseq/marketplace/tree/master/packages

Cannot find the Admonition Panels (logseq-admonition-panels) in the link you provided
I had it installed in my Logseq since months ago but now, after an uninstallation, I cannot re-add it from the Marketplace
Another indication of a "dead" plugin, presumably
By tyhe way, I simply got rid of the custom admonitions, replacing them with the standard ones provided by Logseq itself.
Do you think we can simply close this issue?

Cannot find the Admonition Panels (logseq-admonition-panels) in the link you provided I had it installed in my Logseq since months ago but now, after an uninstallation, I cannot re-add it from the Marketplace Another indication of a "dead" plugin, presumably
By tyhe way, I simply got rid of the custom admonitions, replacing them with the standard ones provided by Logseq itself. Do you think we can simply close this issue?

Sure, I think we can close this issue.  Have a good day.


Archives contain unexpected directory name

I would expect that extracting the archive influxdb-0.8.6.amd64.tar.gz would result in a new directory called influxdb-0.8.6.
However, the current downloads contain a directory called build instead, which is a bit unexpected.
Perhaps this could be an improvement to packaging?
Example:
[joe@ryuko ~]$ wget --quiet http://s3.amazonaws.com/influxdb/influxdb-0.8.6.amd64.tar.gz
[joe@ryuko ~]$ tar -zxvf influxdb-0.8.6.amd64.tar.gz
build/benchmark_config.toml
build/config.toml
build/influxdb
build/influxdb-benchmark
build/scripts/
build/scripts/influxdb-daemon.sh
build/scripts/init.sh
build/scripts/post_install.sh
[joe@ryuko ~]$

Fixed by https://github.com/influxdb/influxdb/commit/387b91cce35588a7f3f6e16337d9ad69105bc58c


[QA] Missing music in DollHouse scene (-102, -97)

Explorer Alpha build versions:

v0.14.0

Issue Description:
The DollHouse scene at coordinates (-102, -97) is missing background music.
STR:

Launch the explorer
/goto -102, -97 DollHouse scene.
Listen for background music.

Expected behavior:
Background music should play consistently in the DollHouse scene.
https://github.com/user-attachments/assets/9c91fe22-a37b-4268-b3de-ee2efaac8347
Current behavior:
https://github.com/user-attachments/assets/cca6f22a-366c-47ca-9c84-76b7bc568abb
Additional Notes:
Windows 11. 12th Gen Intel(R) Core(TM) i7-12650H 2.30 GHz. nvidia geforce RTX

on prod .zone it is also not playing. Cors issue.


Feat refactor task widget

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:
Automating creation of task widget and AINDBehaviorTaskLogicModel
What issues or discussions does this update address?

NA

Describe the expected change in behavior from the perspective of the experimenter

NA

Describe any manual update steps for task computers

NA

Was this update tested in 446/447?

[ ] 446/7

A little side project I've been working on this week. I basically made a robust schema for all the task types, fed those schemas into a previous widget tool that I've made that automatically creates, formats, and validates widgets, and then created a lightweight custom widget on top that will handle and update task changes and updating the AindBehaviorTaskLogicModel. This approach makes it so if the task schema needs to be updated, all downstream widgets and AindBehaviorTaskLogicModel will be updated and handled automatically, hopefully saving time and effort in the future. Let me know what you folks think. Right now it's not integrated, but that's what I'll be working on next unless people don't think this is worth the effort.
https://github.com/user-attachments/assets/87b0b9e7-f6f0-4e4f-bda0-53ae22426cf1

Also there is an example in the task_widget.py file if anyone wants to play around

This looks cool. Let's review this PR after the break.


Crash recovery

Ongoing discussion, might as well have a mockup attached

One thing I've seen (at least with FOSS apps and Android) is that they offer an option to view the contens of the crash report before sending. For example in case people want to make sure it does not contain any personal information.
Usually this is done in form of a link at the end of the checkbox ("view log" or something like that).
Assuming our crashlogs will be submitted in a text format, I think that might be something we'd want to have.

I think this is ready now. The mockup is here, and the crashed plane asset is here.


Prevent partial_apply cloning.

partial_apply cannot be cloned, even in OSSA. OSSA lowering does not know how to allocate for multiple partial applies.
Fixes rdar://119768691 (OwnershipModelEliminator triggers assertion when lowering certain [on_stack] partial_applys in certain circumstances)
This fixes a regression caused by the move-only representation change in 5.9.

@swift-ci test

@swift-ci benchmark

LGTM

@swift-ci smoke test

LGTM. Would it be worth adding verification for?

Ideally yes. PartialApplyInst::visitOnStackLifetimeEnds is the logic we need for verification. It runs when lowering OSSA. It isn't worth it to me to integrate into the verifier right now--there's no real danger of ad-hoc cloning of closures. I changed the assert to a release-mode informative diagnostic:
partial_apply [on_stack] use:
   %7 = argument of bb2 : $FakeOptional<Klass>    // users: %13, %8
     %8 = select_enum %7 : $FakeOptional<Klass>, case #FakeOptional.some!enumelt: %2, case #FakeOptional.none!enumelt: %3 : $Builtin.Int1 // user: %13
     %11 = mark_dependence %10 : $@noescape @callee_guaranteed () -> () on %1 : $*Klass // users: %13, %12
     %12 = begin_borrow [lexical] %11 : $@noescape @callee_guaranteed () -> () // user: %13
->   br bb5(%7 : $FakeOptional<Klass>, %8 : $Builtin.Int1, %11 : $@noescape @callee_guaranteed () -> (), %12 : $@noescape @callee_guaranteed () -> ()) // id: %13
LLVM ERROR: partial_apply [on_stack] cannot be cloned


Ask user to swith account when reading code while signed in

Task/Issue URL: https://app.asana.com/0/72649045549333/1208755822305612/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#5271  👈
develop

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

Join @cmonfortep and the rest of your teammates on  Graphite


Adding onPress to the list of properties for Row and Col in index.d.ts

onPress is a property accepted in Row (and similarly in Col). In both cases, the Views are wrapped with a TouchableOpacity that receive onPress as a property.
This property, though, is not defined in index.d.ts, so typescript projects that use onPress on rows/cols will get an error. This PR adds this definition of onPress.
The signature for onPress is the same one used for TouchableOpacity's onPress.

@fgiobergia Please update your PR considering #65


Fix: update dead documentation links

/setup -> /getting-started

Thank you very much for the PR!


Pasting text using Shift-Insert doesn't work when tmux is running in alacritty

Which operating system does the issue occur on?
 OS: Arch Linux
 Kernel: x86_64 Linux 4.16.13-1-ARCH
 Shell: zsh 5.5.1
 WM: i3

If on linux, are you using X11 or Wayland?
Display: x11 server: X.Org 1.20.0 driver: i915 ...

Package versions:
alacritty-git <IP_ADDRESS>4.g2fcdd40-1
tmux 2.7-1
gpaste 3.28.2-1

Issue description

unable to paste text from the system clipboard into a tmux session running inside alacritty :crying_cat_face:
pasting text from the system clipboard works fine when using another terminal emulator and tmux ( tested with kitty , tilix, gnome-terminal, terminator )
pasting text from the system clipboard works fine when using alacritty without tmux
when pasting text I'm using Shift+Insert in all test cases.
I'm using gpaste as the clipboard manager.

Your clipboard manager shouldn't matter, on X11 alacritty currently uses XClip for copy/pasting. The key bindings used to do so should be defined in your alacritty.yml configuration file.
Do you have xclip installed? If not, that should probably be the first step. After that you need to define a shortcut for it, by default one is only defined for Ctrl+Shift+C/V.

Thank you for looking into this and for your clarifications.
I already have xclip installed:
xclip 0.13-1

This is the keybinding part of my alacritty.yml config file, these
haven't changed:
key_bindings:
  - { key: V,        mods: Control|Shift,    action: Paste               }
  - { key: C,        mods: Control|Shift,    action: Copy                }
  - { key: Q,        mods: Command, action: Quit                         }
  - { key: W,        mods: Command, action: Quit                         }
  - { key: Insert,   mods: Shift,   action: PasteSelection               }
...

Looks like I've always been using what the alacritty config calls
PasteSelection to paste into tmux running inside alacritty and it has
always worked.
If I use the keybindings for Paste then pasting inside a tmux pane
running in alacritty works.
I wonder if this is not some sort of regression with the alacritty code
around the PasteSelection functionality...
On 06/05/2018 04:05 PM, Christian Duerr wrote:

Your clipboard manager shouldn't matter, on X11 alacritty currently uses
|XClip| for copy/pasting. The key bindings used to do so should be
defined in your |alacritty.yml| configuration file.
Do you have |xclip| installed? If not, that should probably be the first
step. After that you need to define a shortcut for it, by default one is
only defined for Ctrl+Shift+C/V.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jwilm/alacritty/issues/1352#issuecomment-394744652,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARKmQxf1bCATJcA-WKstT2a1YevWWatks5t5p5RgaJpZM4UaJun.

@chrisduerr thanks for explaining the subtle differences between Paste
and PasteSelection !
Note:
I've made no changes to my alacritty.yml, but since
alacritty-git <IP_ADDRESS>4.g2fcdd40-1

PasteSelection has stopped working.
PasteSelection was working before this build version.
Switching my keybindings to:
  - { key: Insert,   mods: Shift,   action: Paste               }

allows me to successfully use Shift-Insert to paste inside a tmux
window running in alacritty.
Thanks for your help!
On 06/06/2018 07:24 PM, Christian Duerr wrote:

@stefancocora https://github.com/stefancocora IIRC |PasteSelection| is
used as an alternative to middle-mouse pasting, similar to the way it
works on linux. So this will not work in tmux because tmux has their own
selection and alacritty doesn't know anything about a selection you make
with tmux.
If you want normal "paste", you should just switch the key binding to
use |Paste| instead and everything should be good to go. If that works
for you and you have no more problems, feel free to close this issue. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jwilm/alacritty/issues/1352#issuecomment-395166999,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARKmd9LiI65GixGAGD_ITmpFeWLCvUQks5t6B5NgaJpZM4UaJun.

I think there is still a problem?
If I select something using shift+Mouse and want to paste it using shift+insert it do not work in tmux.
Selection works fine, and xclip has the data. Problem only with shift+insert.
Why PasteSelection action does not work inside tmux?
I am switching from gnome-terminal, everything fine there.

@sazarkin See https://github.com/jwilm/alacritty/pull/1300 and https://github.com/jwilm/alacritty/issues/1298.


Add comments to tasks

Is your feature request related to a problem? Please describe.
There's no way for a mentee and mentor to communicate. They can only assign tasks and complete them.
Describe the solution you'd like
Use the comments endpoint from the backend to add comments to tasks when a user is in a relationship to communicate.
UI to do this would be open a task on a new screen and load the comments below it right side messages for the current user and left for the person he/she is in a relationship with. (like a chat screen) . Feel free to share your views on the UI or if you have something else in mind :)

@anitab-org/coding-team does anyone want to take this?

@anitab-org/mentorship-managers Can someone assign me this?

On hold until the sent_by_me field is added on backed and fix the internal server error :)

Will resume work after offline feature is complete. Anyone else wanna pick this up untill then?


Play from SMB path

I can't play videos from SMB share.
<IP_ADDRESS> = Server IP (Windows 10)
<IP_ADDRESS> = Local Mizuu IP
I try open the video with opera browser to see the error:

This is the movie information from Mizuu:

Notes:

Mizuu get the movie information ok
I can play the movie with ES file explorer (from the same network share)
I have other tablet with Mizuu working ok (same server, same share, same user, etc.)

Can you help me?

The local SMB server is only accessible when you've started playback. Anyway, I'm not supporting Mizuu anymore. Sorry.

What you mean with "started playback"? Thanks.
PS:  It is a shame that you no longer supports Mizzu, is an excellent app in my opinion. What App substitute recommend? Thanks again

I love Mizuu. This is the end or someone else take the project? Thanks

Please read this and disregard any newer posts on the blog:
http://mizuu.tv/this-is-it/

Good work with Jolly. Unfortunally my devices (tv-box and projector) have android 4 :(
Sorry, but I can't loose the hope to get mizuu working. Please, can you help me a little more?
I think the problem is that the routing to smb server is not working (port 50002). What part of the mizuu code, links the local ip with port 50002 to server IP? Thanks

I don't think you quite understood the message of the blog post I linked to. Please don't continue this.


fix seaborn issue in plotting of shapefiles

Hello @bbuzz31! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tools/ARIAtools/util/shp.py:

Line 179:80: E501 line too long (80 > 79 characters)

The changing of the function name seems to be a mistake: (def shp_area(... was changed to def shp_are(...)

Per the below link, seaborn-v0_8 was introduced in matplotlib 3.6.0, what version is raising this issue? https://matplotlib.org/stable/api/prev_api_changes/api_changes_3.6.0.html
Another possible resolution is to bump the required version of matplotlib

the problem is with with plt.style.context(('seaborn')): which was deprecated at some point. The change I put in should be backward compatible.


[INLONG-7060][Sort] Support sink redis

Prepare a Pull Request
(Change the title refer to the following example)

[INLONG-7060][Sort] Support sink redis

Fixes #7060

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[x] This change is a trivial rework/code cleanup without any test coverage.

[x] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

finished


feat: add null driver

resolves #217
I might have misunderstood what was meant by "null driver".
But a driver that simply does not store data would be great for temporarily deactivating caching within Nitro.

Thanks!


fix template.sol

see https://github.com/fluidex/solidity_plonk_verifier/commit/d5dd215cdd152b2c7b6c56b01912bdcceed7df51

Codecov Report

Merging #97 (ced9cdd) into master (d5f62dc) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##           master      #97   +/-   ##
=======================================
  Coverage   53.06%   53.06%
=======================================
  Files           9        9
  Lines         522      522
=======================================
  Hits          277      277
  Misses        245      245

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 d5f62dc...d802a2b. Read the comment docs.


[Broken Link] A 403 Error in Hugging Face Demo

Hello!
Thank you for providing such an excellent model!
I noticed a minor issue in the Hugging Face Demo. Specifically, the link on this line (truetype_url) appears to be broken. It seems that the correct URL is: https://huggingface.co/internlm/internlm-xcomposer2d5-7b/resolve/main/SimHei.ttf.
Thank you so much!

Thanks very much.
The issue has been fixed.

Thanks!


game.settings.set is not a function errors

FtC 1.4.5 w/ Foundry 10.286, didn't happen in previous version of FtC.
Starting the process:
find-the-culprit.js:350 Uncaught (in promise) TypeError: game.settings.set(...) is not a function
[Detected 1 package: find-the-culprit]
    at deactivationStep (find-the-culprit.js:350:3)
    at Object.callback (find-the-culprit.js:78:11)

Reactivating all modules:
Uncaught (in promise) TypeError: game.settings.set(...) is not a function
    at reactivateModules (find-the-culprit.js:361:3)
    at Object.callback (find-the-culprit.js:239:25)
    at Dialog.submit (foundry.js:55745:35)
    at Dialog._onClickButton (foundry.js:55674:10)
    at HTMLButtonElement.dispatch (jquery.min.js:2:43064)
    at y.handle (jquery.min.js:2:41048)

Manually refreshing after makes things work as expected though? Expect when attempting to reactivate all.

The problem is I think because the game.settings.set is not terminated with semicolon, so the line after gets run in because of parenthesis usage.

Just noticed there's new version with fix for this, nevermind.


Make user can control when to stop a job.

I think it's good for user can stop the job (just return False in job_func).

Hi
How using these function with parallel thread you are showing here:
https://github.com/dbader/schedule/blob/master/FAQ.rst
Thank you


Drop ice-multi-config.sh

If this still works, it will be CentOS6 only. Since
we have moved everythong to CentOS7, this can be removed.

Note: https://ci.openmicroscopy.org/view/Mgmt/job/ICE-centos6-archive/ can likely be deleted once this is merged (though it may currently point to https://github.com/jburel/ome-sysadmin-scripts/tree/ice-multi-zip/ice-multi-archive) cc: @jburel

No objections to merging today.

See also https://github.com/openmicroscopy/zeroc-ice-py-centos7


[S] Secondary (pills) navigation bar

AC:

All stages, and content within stages are separate pages
The highlighted pill should indicate the current page
Next/Previous CTAs at the bottom of the page
Desktop + Tablet + Mobile

If this ticket is specifically just for the pills and slider then looks good. However noticed a few issues on the content below it, but not sure if that's being sorted with ticket #42

Padding on edges and top of content looks too much on mobile
-Seem to be missing the old breadcrumb design
-I haven't been able to test the school years, further education etc navigation. I assume this will be populated when we populate the content for these pages

@tkeburia @rjkerrison @cameronramsay @carinapope13


Small additions to different parts of the app

Tutorial Page:

Add a complete button at the end of each tutorial. This will help with two things. The first is that it allows the user to feel a sense of completion when finishing a tutorial, which will encourage them to continue to keep learning. And the second reason is that it makes the ux a bit easier. It allows the user to exit the tutorial without much hassle, and return to the list of tutorials.
Have back arrows at the top left corner of the individual tutorial pages so that if the user doesn't want to complete that tutorial they have an easy and clear way to return to the tutorial list.  Having an arrow on any page will help the user easily exit what page they're on with minimal confusion.  An example of this was in one of the interviews I conducted. The user opened up the "send bitcoin" page but she didn't want to send a transaction just yet. She wanted to go back but didn't know how. It took her a few seconds to figure out how to go back. If there was a simple back arrow she wouldn't have run into that problem.
Make the tutorials more concise, add diagrams or gifs, and add exercises that the user can complete. During the interviews, users said that the tutorials were good and had helpful information, but there was a little too much information. By adding more visuals and interactions, the users will be more engaged and have more fun with the app.

Love these!
I'm getting a copywriter to help me edit and simplify the tutorials, so I do agree that this is key. Even if in the end we have a few more tutorials, it's good to keep them simple and short, focused on teaching you one thing at a time.
The completion buttons and the arrow back buttons are for sure required! I think the guys are working them in the Figma files at the moment but I should double check.

I think we did most of those! The last thing is to add images/gifs/diagrams to the tutorials, and I'll open individual issues for those. Closing this now!


What is the typical setup of nebula? Install nebula on every host if possible?

I apologize if this is not the right place to ask this question.
I read through various docs I could find but it's not clear what is the typical/recommended setup of nebula: one single huge flat net with nebula installed on every node, except where it's not possible (unsupported devices like printer)?

That does seem to be how most people use Nebula: To string them into a distributed network when they're not close to each other.  It's more of an "expose services to those hosts" thing than an "access those hosts" thing.

Yes, that's correct. Nebula should be installed on every node that you want accessible on the network, or that you want to have access to the network. For devices like printers you can take a look at unsafe_routes.


Remove pydantic dependency

Description
kedro-mlflow is using pydantic for KedroMlflowConfig class. This is used both for its syntactic sugar to create class, to benefit from auto-completion from a dev perspective and for input validation logic.
Context
However, kedro has recently decided to use attrs instead of pydantic. The pros and cons are heavily discussed in https://github.com/kedro-org/kedro/pull/1465. I want to be as consistent as possible and limit the number of extra dependencies used to avoid potential conflicts and decrease maintenance burden.
Possible Implementation

Remove pydantic from requirements.txt
Define KedroMlflowConfig as a standard python class and validate inputs with "field"
Move part of the validation logic outside of the class. This will be done in #314 which is a pre-requisite for this ticket.

Possible Alternatives
Use a pure dataclass if all the validation logic is moved ouside of the class.

After multiple trials and errors, it turns out that pydantic makes manipulations much simpler to :

validate at instantiation each data
convert from dict ->pydantic class (attrs only enables conversion the other way around, i.e. froma  class to a dict).
These defaults may be overcome with attrs but need much more boilerplate code (write a validator for each entry) + another third party library (cattrs for serialization).

➡ I choose to keep pydantic for the following reasons:

Even if attrs more explicit, this is less readable given the amount of extra code needed
adding another library cattrs is against the initial motivation of this PR
No conflicts has been identified between pydantic, kedro and mlflow, so the initial motivation is not very prevalent


chore: Bump submodule

Description of your changes

Bump golangci-lint to v1.54.0
Superseedes #1859
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

 Received 504542006 of 504542006 (100.0%), 159.8 MBs/sec
  Error: /usr/bin/tar<EMAIL_ADDRESS>Cannot open: File exists
  Error: /usr/bin/tar<EMAIL_ADDRESS>Cannot open: File exists
  Error: /usr/bin/tar<EMAIL_ADDRESS>Cannot open: File exists
  Error: /usr/bin/tar<EMAIL_ADDRESS>Cannot open: File exists
  Error: /usr/bin/tar<EMAIL_ADDRESS>Cannot open: File exists
  Error: /usr/bin/tar<EMAIL_ADDRESS>Cannot open: File exists

https://github.com/golangci/golangci-lint-action/issues/23#issuecomment-647177770
I think we need to rewrite the lint action
The reference code from README.md is just https://github.com/golangci/golangci-lint-action#golangci-lint-action
  golangci:
    name: lint
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - uses: actions/setup-go@v4
        with:
          go-version: '1.21'
          cache: false
      - name: golangci-lint
        uses: golangci/golangci-lint-action@v3
        with:
          version: v1.54

I extended this PR with the following:

Use Go v1.21.2

Add ignore opts to some tests because they failed due to the unexported context.backgroundCtx struct

Bump golangci-lint to v1.54.2

Bump build submodule

Copy (most of) .golancilint.yml from github.com/crossplane/crossplane and fix the linter issues accordingly.
Go import order is now

Go standard packages
3rd party packages
local packages

@chlunde can you check again?


[ZEPPELIN-5296]. NPE when calling completion for %spark.sql

What is this PR for?
Trivial PR to fix NPE in Spark interpreter when calling code completion.
What type of PR is it?
[Bug Fix
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5296

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Will merge if no more comment


updated manual

Codecov Report

Merging #24 (c9981d8) into master (8ed6289) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   49.74%   49.60%   -0.15%
==========================================
  Files          14       14
  Lines        1371     1375       +4
==========================================
  Hits          682      682
- Misses        646      650       +4
  Partials       43       43

Impacted Files
Coverage Δ

pkg/version/version.go
0.00% <0.00%> (ø)

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 8ed6289...c9981d8. Read the comment docs.


Change day filter from radio button to checkbox

Idea Description
We can't uncheck filter so we would like to make it checkboxes

Reference images and links

I would like to start on this issue.

Thanks!


The new created resource group disappears after clicking "Focus this Group" for it

OS: Win10/M1
Build Version: 20220525.12
Regression: Not a regression
Repro Steps:

Make sure the resource groups by Resource Group.
Create a resource group and click "Focus this Group" for it.
Check whether the new created resource group is focused or not.

Expect:
The new created resource group is focused after clicking "Focus this Group" for it.
Actual:
The new created resource group disappears after clicking "Focus this Group" for it.

More Info:

Refresh the resources view, the new created resource group is focused.
This issue also reproduces on the latest released resource group extension.

Caused by #285


Move 'other location' data in to a separate fetched file

Currently the data for other locations is baked in to the JS, it'd be better to fetch it separately like we do with other data.

Let's create another repo just for crowdsourced locations. And fetch it via githubusercontent just like we do for vaxxnzlocations. Feel free to come up with a cool new name for it.

Honestly it works perfectly fine as it is, I think it's mint.


validate_token changes/encrypts cookie

Context
When we call the validate_token endpoint using Cookies, it overrides the cookie which contains important information like the access token, etc. I can't tell if it is either encrypting the token (when it was not encrypted upon signing up/logging in, or not.
I have created a minimum-replication Rails app, which can be found here. I have also showcased this issue in Postman, with example endpoints to run, as well as examples of what their responses are.
The main issue is when calling this endpoint with a cookie gained from signing or logging in, the cookie is then set to a new cookie, and encrypted/scrambled.
Template Information

Version: which version of this gem (and ng-token-auth, jToker or Angular2-Token if applicable) are you using?

gem 'devise_token_auth', '>= 1.2.0', git: "https://github.com/lynndylanhurley/devise_token_auth" - this was needed to get it working with Rails 7

Request and response headers: these can be found in the "Network" tab of your browser's web inspector.

I have examples saved on this public Postman workspace.

Environmental Info: How is your application different from the reference implementation? This may include (but is not limited to) the following details:

I am using Cookies, so I have done the following. Secure will be set to true in a PROD environment, but it is needed as false in order to showcase this on Postman.
  config.change_headers_on_each_request = true
  config.token_cost = Rails.env.test? ? 4 : 10
  config.cookie_enabled = true
  config.cookie_name = "testing-dta"
  config.cookie_attributes = {
    http_only: true,
    secure: false,
    same_site: "None"
  }

Routes: are you using some crazy namespace, scope, or constraint?

N/A

Gems: are you using MongoDB, Grape, RailsApi, ActiveAdmin, etc.?

Active Admin/Postgres

Custom Overrides: what have you done in terms of custom controller overrides?

N/A

Custom Frontend: are you using ng-token-auth, jToker, Angular2-Token, or something else?

I am using this in API only mode, to be used with a React SPA later on. However, this is a backend issue.

Hey @mdodell ! Check out this doc about SameSite=None. It looks like your current config will result in the cookie being blocked.

Hey @theblang - I tested this with a Heroku app as well, with the following config:
  config.cookie_enabled = true
  config.cookie_name = "testing-dta"
  config.cookie_attributes = {
    http_only: true,
    secure: true,
    same_site: "None"
  }

You can test it locally for yourself here: https://github.com/mdodell/devise-token-auth.
Here is it running in Postman: https://www.postman.com/winter-star-976257/workspace/devise-token-auth-cookie-issue/overview.
I'll leave that Heroku app running for now so you can also play with it yourself.
As you can see, the response header for the validate-token route is as follows:
[{"key":"Set-Cookie","value":"testing-dta=myrAWLEsUDXd24MTWQpGe4OznN62sWsqiWu0cF9qcghaBPhxVVgC4Q4Luvrbg0TbE996lY6cxoSlkqm%2F5n6gprjqjekEAdl5a1wz%2FkjZyy1wFaNpjuVXErf1YICHa7d4Y%2BYZftdz2rHnoJEvMfeZZYbEjcAn%2BAYNa9MuCfUuYPiIipl%2B1If6c3YxHn9vUVFv0YLuaTob9m0PMb49diKY7saDS03IG2RT3VgelzYMKxEx7zIZB5yXbPsPuiWJP4EydpOnlP%2BFksgt4e36XmEAAwWkHg%3D%3D--OqIZoXHw3CkWjdEb--bUxopeO9VFnUF8Plu0S2Og%3D%3D; path=/; HttpOnly; SameSite=Lax"}]

SameSite is set to "Lax" and there is no Secure - even though they are configured in they are configured to be SameSite "None" and "Secure" as true in the config here.


working with avro decimal type

Since this crate use apache avro [to_value] (https://github.com/gklijs/schema_registry_converter/blob/main/src/avro_common.rs#L144) function to convert serializable item to Value type, it might not work with avro schema with decimal field as the avro serializer might not convert it properly.
use apache_avro::Decimal;

#[derive(Serialize, Clone, Debug)]
pub struct item {
    // indexes
    pub total_successful_transaction_blocks: Decimal,

The total_successful_transaction_blocks will be serialized to
 "total_successful_transaction_blocks", Record([("value", Array([Int(1), Array([Int(7)])])), ("len", Long(1))]))

The issue is not very clear to me...
Is there something that should be fixed/improved in the apache_avro crate or it is an issue only in schema_registry_converter ?

Clsoing this, as it was fixed by https://github.com/gklijs/schema_registry_converter/pull/114


Usage of ruby updated

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

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: Your added content was already referenced in the Frameworks (gems) section of the article.
Thank you and happy coding.


Improve Docker Setup

Close #17

Improvements:

Add database tables via migrations with npm package db-migrate
Run migrations on docker-compose up
Add plain sql of tables for documentation
Provide MySQL credentials via environment variables and in Docker setup
Add mailhog for email debugging
Configure mailer with environemnt variables & provide mailer environment vars in Docker setup
Document all changes & add instructions for Docker setup

#17

@matthib I updated the description of the PR. Once we merge this PR, issue #17 will be auto-closed. Check out this special feature of Github.

@matthib OK, good, we can merge this once the few issues are resolved.
There are no apikeys in my database so I will open another ticket for the seeds.

Regarding the Api keys: I didn't add them because of issue https://github.com/Human-Connection/Clock-of-Change-API/issues/13
You can add them manually, as the MySQL service is exposed to the host and you can connect to it via localhost (add least in Docker).

@matthib if you want me to create the api keys manually, please update the README. I'm playing the ~ignorant~ unaware and will not do anything unless specified in the README.
(Well, of course I checked out the code and hoped to see some seeds but couldn't. So I tested the code up to this point)

@matthib if you want me to create the api keys manually, please update the README. I'm playing the ignorant unaware and will not do anything unless specified in the README.
(Well, of course I checked out the code and hoped to see some seeds but couldn't. So I tested the code up to this point)

Okay, then I'll just mention the documentation task in the relevant issue.


Questions regarding changes to JaxRDDLCompiler - params to jax_expr

There has been some changes in JaxRDDLCompiler which has altered the lambda functions returned for CPF evaluation. In the file the logic is jax_cpfs[cpf] = self._jax(expr, info, dtype=dtype). Earlier expr was a lambda function that had two params - 1. dictionary of state, action and interim variables with their current values and 2. a PRNG key.
Now I think it is expecting three values. I couldn't find documentation of what the three params should be. Can someone please let me know where should I be looking or just explain what the three params that expr expects now?
Thank you.

Hi,
If I am understanding correctly, you are referring to the new 'params' argument in the wrapped jax expressions for RDDL calculations.
In short, this refers to a dictionary of per-node weight parameters that provide fine control of the model relaxations for discrete calculations approximated by parameterized expressions, aka sigmoid.
You can currently define and tune these per node weights if you like (e.g. using Bayesian optimizataion). There is an unused function 'print_parameterized_exprs' in the JaxExample that you could call to retrieve the keys it expects as well as their current values.
===
The story behind this is that, in the previous version, there was a single global tuning parameter 'w' to control the accuracy of the relaxations, e.g. x >= y -> sigmoid(w * (x - y)) in FuzzyLogic. However, in principle, it is possible to use per-node weight parameters where each 'w' can be locally tuned, e.g. using some local errors. We did not want to limit users who wish to adapt these parameters and have better control over the model approximation. It is not really used anywhere, nor is it currently clear how to without having better control of intermediate calculation in jax, something we like to work on in the future. (FuzzyLogic currently defines these parameters for some relaxations, so you can look there to see the technical details how they are propagated.)

In the future, 'params' could also be used for propagating other information/parameters through the computation graph that one does not want to bake in, so it is really meant as a "catch-all" for propagating information through Jax.

Thank you for the pointers @mike-gimelfarb. I will have a look at them.


first pass at packaging

Fixes #10

[x] things still work
[x] can consume this locally in Desktop

[x] type declarations aren't being found ?!?!?!
[x] webpack can't resolve the native module

[ ] what to name it????

I've been able to integrate this into Desktop and port our registry usage over. I'll save the rename for another day.


added missing !default

#346
fix: added !default for breakpoints list in harmonium-settings.scss

:tada: This PR is included in version 4.3.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


es2017  can  not build

> 279 |                         <button onClick={ ::this.handleOrder} >
      |                                            ^
   280 |                         </button>

It does require a specific babel transform that is not included in the next's babel preset, you can add it yourself following intructions here
however using the bind operator inside render is considered an anti-pattern because of performance reasons, more info here.

Thanks @kandros for the help.


Element missing end tag in  component after recent update.

Environment

Operating System: Linux
Node Version:     v16.20.0
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
Somewhat surprising it hasn't been noticed yet but this bug is occurring in the StackBlitz environment linked on the Installation page in Nuxt's documentation  https://stackblitz.com/github/nuxt/starter/tree/v3-stackblitz

and locally when I attempt to do the following
npx nuxi init test
cd test
npm install
npm run dev -- -o

Describe the bug
500 error caused by lack of end tag in<EMAIL_ADDRESS>
Seems to be related to https://github.com/nuxt/nuxt/commit/9741f267be1c04a69f3a25204c175f98d834565b and changes to @nuxt/ui-templates mixing with vite-plugin-vue-inspector

Safe to say I am not the only one experiencing this given it is occurring in one of nuxt's featured starter environments.
Additional context
Noticed this started happening when I build new nuxt applications in the past week or so and believe this coincides with these recent changes https://github.com/nuxt/nuxt/commit/9741f267be1c04a69f3a25204c175f98d834565b
Here is a screenshot of logs from browser that were too long to post:

Logs
Logs from `npm run dev`output:
ERROR  Internal server error: Element is missing end tag.                                        3:50:08 PM
  Plugin: vite-plugin-vue-inspector
  File<EMAIL_ADDRESS>      at createCompilerError (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:18:17)
      at emitError (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:1487:5)
      at parseElement (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:1031:5)
      at parseChildren (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:844:18)
      at parseElement (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:1008:20)
      at parseChildren (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:844:18)
      at parseElement (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:1008:20)
      at parseChildren (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:844:18)
      at parseElement (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:1008:20)
      at parseChildren (C:\Users\ayoung.PRESCIENT\firefly\node_modules\@vue\compiler-core\dist\compiler-core.cjs.js:844:18) (x2)

I was also bootstrapping a new app this evening and run to the same issue. I can confirm that we have the same errors. I can reproduce and provide more context if needed.

I am having this issue on cloudflare's pages deployment as well

Thanks. This was triggered by a release of @nuxt/ui-templates. On it.

For anyone looking for a workaround. Open this file.
<EMAIL_ADDRESS>Search the the following tag:
<nuxtwelcome>

and replace it with:
<nuxtwelcome />

Save and re-run the dev server script.

The issue is already resolved upstream with release of v1.3.2.

Same error here using bun
bun --version
1.0.25
Log
$ nuxt dev

Nuxt 3.10.0 with Nitro 2.8.1                                                                                9:41:58 PM
                                                                                9:41:58 PM
  ➜ Local:    http://localhost:3000/
  ➜ Network:  use --host to expose

  ➜ DevTools: press Shift + Option + D in the browser (v1.0.6)                                                                                9:41:59 PM

 ERROR  Pre-transform error: Element is missing end tag.                                                                                9:42:00 PM

 ERROR  Pre-transform error: Illegal tag name. Use '<' to print '<'.                                                                                9:42:00 PM

ℹ Vite client warmed up in 413ms                                                                                9:42:00 PM
ℹ Vite server warmed up in 466ms                                                                                9:42:00 PM
✔ Nitro built in 338 ms                                                                                nitro 9:42:00 PM

Borwser :