Kudos!

Not an issue!
I just wanted to say that I absolutely fell in love with this project. My jaw keep dropping as I explored the app and discovered new features. It's looking great, congrats! 👏
Thanks for building this!
ps: I would love to build something similar for fun and learning purposes and use this project as an inspiration (and reference it of course).
ps2: I had no idea that Microsoft's svelte design system was so complete (and seems like it's good with accessibility as well) 👀

Its not Microsoft's design system we are using, Fluent Svelte is a component library based on msft's fluent design created by an individual.
https://github.com/Tropix126/fluent-svelte


Add option --version (short -V) to print goldplate version

Add option --version (short -V) to print goldplate version.
This is very easy to do with optparse-applicative and kind of standard, to have a --version option.

ping @jaspervdj-luminal

Let me know if you want a new Hackage release for this, btw.

Let me know if you want a new Hackage release for this, btw.

It's not urgent, thanks.  This can wait, maybe there is a new release necessary for GHC 9.2.


How to set the input method to switch automatically

Feature description
How to set the input method to switch automatically, for example, input Chinese in INSERT mode, this time back to normal mode, automatically switch to English mode;
Additional information
No response

we already have fcitx5.nvim configed, just kept it commented out. check it's doc.

https://github.com/ayamir/nvimdots/blob/bf7f33ec6d57e5371c1b0abbee2929625782cb67/lua/modules/plugins/tool.lua#L7-L13

fcitx5.nvim caused serious performance problem for me

fcitx5.nvim caused serious performance problem for me

Because I rarely use this feature, you can investigate it on your own and make a pr to replace fcitx5.nvim if possible.

Because I rarely use this feature, you can investigate it on your own and make a pr to replace fcitx5.nvim if possible.

Just a reminder, I also rarely use this.

Close as no further response.


Separate Code Requirement for each arm mode

Checklist

[x] I checked for similar existing requests (both open and closed) before posting.
[x] My request is generic, other users may benefit from it too.

Proposal
It would be great if the setting "Require code for arming/disarming" can be defined for each arm mode.
I'd like to secure all arm modes except the 'armed_home' mode with a code.
Additional info
The reason for this is, that I want to enable my kids to activate the 'armed_home', even if they can not handle the code. In my setup, this can only be done from any device inside of our home.
All external device are unable to access home assistant in a direct way.

This is still requested

This is still requested

A great workaround would be a button card that triggers an automation with a code to alarm hode. I am doing this with Aqara Zigbee buttons to set the alarm into home mode with an auto code that can only be used to set the alarm.


Sleeping mat doesnt dismount properly

Minecraft version & Mod Version:
1.12; ;latest; sevtech 3.0.6
Single player or Server:
Single player
Describe problem (what you were doing / what happened):
Independently of me, someone else encounterd this issue here https://github.com/Darkosto/SevTech-Ages/issues/1641
When you use the sleeping mat on a horse (while mounted); after you wake up the game thinks you're still on a horse. E.g. being unable to re-mount the horse (since the 'game' thinks you're already mounted probably) and me specifically, got attacked by a zombie, ran away and the zombie was able to kill me as if i was on the horse)
Personally i've tried to replicate this behaviour with other 'sleeping' mods like the Tipi and the vanilla bed, they all seem to be dismounting the player correctly before/after the sleep happens. Since this works with vanilla horses + cyclic sleeping mat ( i did not test this in a clean environment with only vanilla+cyclic, i must admit.) im assuming its a problem where the sleeping mat doesnt do a dismountEverything(); or something like that.
Note that a relog fixes the issue.
Crash log (if any):

Should work in latest https://minecraft.curseforge.com/projects/cyclic/files/2550423

Thank you @0xb00b for submitting my bug to the right mod maker. I didn't think to take that next step and find out which mod was causing it.


Small scalability improvements

More scalability improvements
read per commit basis

test-me-please

Coverage increased (+0.03%) to 36.903% when pulling 816b893a71997351c77667e2cff54fd33dbf98c8 on pr/small-k8s-fixes-2 into 7fb10afc0604bf72435bf88fca23527449b884f4 on master.

retest-netnext

retest-net-next

CI failures are known flakes

CI failures are known flakes

Are they? I couldn't find an opened issue for Suite-k8s-1.11.K8sPolicyTest Basic Test Denies traffic with cnp default-deny ingress policy.

CI failures are known flakes

Are they? I couldn't find an opened issue for Suite-k8s-1.11.K8sPolicyTest Basic Test Denies traffic with cnp default-deny ingress policy.

I just opened it since I hit this flake on a different PR: https://github.com/cilium/cilium/issues/11741

test-me-please

K8s 1.18 kernel 4.9 hot bug that is going to be fixed by https://github.com/cilium/cilium/pull/11714


RAM-based interrupt vector tables

To improve performance for interrupts and allow for dynamically hooking interrupt vectors at runtime, provide some facilities for creating and using interrupt vector tables.
This should also increase what we can do with multicore.
This code is currently completely untested.

Didn't test the code either, but it looks sensible.
Maybe find a better place, though? I don't think it's multicore-related.

I still don't have a good mental model of whether the padding bytes (256 bytes - 192) will be used by other static variables, so avoiding writing to them seems the safest option.

This seems to be an open question:

https://github.com/rust-lang/rfcs/issues/1397
https://github.com/rust-lang/unsafe-code-guidelines/issues/176

Thanks for those threads. I think the answer (right now) is that the padding bytes are unused.
And, it doesn't seem like any existing target will support stride != size, so that should hold in the future.
Would be nice to have a way to overalign the start of an array to avoid this though - maybe we'll get magic syntax for that some day but I don't want it so badly that I'll push for it.


Can you provide a list of movie names included in the MAD dataset?

I couldn’t find movie names in the MAD dataset.
Could you please provide a list of movie names included in the MAD dataset?

Dear @ghbGC, you can find the mapping between the MAD IDs and the IMDb IDs in the attached file. You can recover the name of the movies by scraping the IMDb pages.
Feel free to reach out for any additional questions.
MAD_id2imdb.json

Thank you for providing the mapping file.


"Please install custome custome lovelace map" issue

Hi, map works fine If I add it myself manually but it won't work properly on the main page where it's added automatically. Any ideas?

That screenshot is not from the Lovelace Valetudo Map Card. You can't use another card and expect it to work. You need to use the Lovelace Valetudo Map Card.


Any update plan of the tutorial for the Docker for Mac ?

Hi, firstly thanks a lot for writing the tutorial ! and I found the tutorial in jenkins2 is for local Docker-Toolbox version of a fully ephemeral jenkins environment(tested up to Docker 1.11). while now docker is 1.13 and docker for mac is ready. recently I tried a lot on docker(1.13) for mac and met some questions, such as: YADP plugin can not connect to docker damon in jenkins, no .docker under $home folder ..etc  so just wonder if there is plan for updating the tutorial which will be great help. thanks in advance

There are! Sorry for the late response, I have a local version working on Docker for Mac, I just need to massage it and I'll get it up here t hen I plan to update the blog series or perhaps a slightly new blog referencing the evolutionary changes. Trying to get ahead of all the new Jenkins 2 stuff.

(Give me until next week to iron out the details and make sure everything is easy to understand, conceptually the problem with Docker for Mac is that it doesn't natively listen on a port, it only uses the docker.sock file, so I found an awesome proxy container I use to proxy the sock file to a port locally to work around it.  "ehazzlet/docker-proxy" is really easy to use and I'll  probably add that as part of the base configuration.).

Hi @maxfields2000, thank you for this awesome repo. Any updates on the tutorial for 'Docker for Mac'?

@rahulakash Updated! Take a look at the new "jenkins2" folder, it should be fully compatible with Docker for Mac (tested by many folks for me). Worked around Docker for Mac not having a native Docker TLS setup by using Evan Hazzlet's awesome docker-proxy container to proxy the docker socket file.  Works for Docker-toolbox as well!


Marker blinking issue

Summary
On Android side, I am currently facing issue when i use marker. Markers are blinking automatically in map
Reproducible sample code
<Marker

            onPress={() => {
              console.log("Clicked");
              setselected(spot);
            }}
            key={spot._id}
            // image={selected == spot ? images.MapPathThree : images.GreenMarker}
            coordinate={{
              latitude: parseFloat(spot.latitude),
              longitude: parseFloat(spot.longitude),
            }}
          >
            <View>
              <Image
                source={
                  selected === spot ? images.MapPathThree : images.GreenMarker
                }
                style={{
                  width: 55,
                  height: 55,
                  tintColor: selected === spot ? colors.redFF4F5A : "#004D16",
                }} // Adjust the size of the image as needed
              />
              <Text
                style={{
                  color: selected === spot ? colors.white : colors.white,
                  fontSize: fontSize.size14,
                  fontFamily: fontFamily.bold,
                  position: "absolute",
                  alignSelf: "center",
                  top: "25%",
                }}
              >
                {Currency}
                {spot.charges}
              </Text>
            </View>
          </Marker>

Steps to reproduce
In android using markers
Expected result
I have to stop that from happening.
Actual result
Currently, markers are blinking on android side
React Native Maps Version
1.7.1
What platforms are you seeing the problem on?
Android
React Native Version
0.71.15
What version of Expo are you using?
Not using Expo
Device(s)
Android
Additional information
No response

Same issue But not in andorid , in ios I am facing that markers blinking issue

Just realised I created a duplicate, sorry ^^
Also added a video screenshot of the issue.
Referencing it here: https://github.com/react-native-maps/react-native-maps/issues/5009

@JoshiGautamB, you can use the tracksViewChanges prop and set it to false. It should work correctly.

tracksViewChanges = {false}
works for me

Hi, thanks for reporting this. We're aware of the issue and are trying to fix it urgently. As other mentioned, this is a duplicate of https://github.com/react-native-maps/react-native-maps/issues/4997 so I'm closing this issue

tracksViewChanges = {false}
works for me

thanks, works for me too

where should I put it tracksViewChanges = {false} ?

@fatima-elboubakri you must use it in Marker, as shown below.
<Marker
   ....
   tracksViewChanges={false}
   ....
/>

If I use that, my markers disappear after a while.

I'm having this same issue. I believe it is causing a memory leak which is leading to an out of memory error... I'm not sure if the blinking markers are related though.

If I use that, my markers disappear after a while.

i am facing the same issue as soon as i add tracksViewChanges={false} marker disappears


[TF:TRT] Create execution context with device memory if shape output is present in TRT 7

Workaround for TRT 7 to handle shape outputs.
Previously, we use createExecutionContextWithoutDeviceMemory to create IExecutionContext, so that TF-TRT can manage the needed device memory to support the execution using the IExecutionContext. This triggers a bug in TRT 7. To workaround the bug, we switch to use createExecutionContext for creating IExecutionContext when the TRTEngine has int32 output.
Tracker #45481
Tagging @bixia1 for review.

Looks good to me. I modified the PR description with more details. Please check.

Based on the code then it's not the fact that there is a shape tensor, but that it is int32 data type. That's not the same thing, right? Because shape tensor has additional qualifications of being 0-d or 1-d.
Does this same behavior occur when index output of "topK" operation is set as output?
Is there no op that we can insert at the output to force TRT into correct behavior?

Thanks @christopherbate for your comment! You are right, we could and should be more specific when checking the output tensor. I have added checks to ensure that it is 0-d or 1-d.

Does this same behavior occur when index output of "topK" operation is set as output?

The error happens at an assertion of a shape tensor size: "Assertion failed: size == shapeTensorVolume(out.extent)"
I expect that the problem would only appear with int32 outputs that are shape tensors.

Is there no op that we can insert at the output to force TRT into correct behavior?

I am not aware of any.

I am not sure whether we can/should add check for 0-d or 1-d to filter the cases. It depends on the situation of the bug. For example, if we have this in the network:
shapeOp(4-d) -> reshape(2x2)
Do we need to allocate context with GPU memory to make this work?

@bixia1, you are right. I have checked it, and the error happens if I insert the reshape. I will revert the change, which filters for the 0 and 1d cases.
I will also add two unit tests that would trigger the bug. To add the tests, I will need to rebase this PR to #52181.


Update nginx-logs.yaml to accept host:port combined NCSA patter

Expand grok-match for NCSA vcombined format

Hey 👋🏻
Just to ask because I never seen this before, you are using the server_name directive in nginx to have domain  + port so when you have the log format set to:
    log_format main '$server_name $remote_addr - $remote_user [$time_local] '
                '"$request" $status $body_bytes_sent '
                '"$http_referer" "$http_user_agent"';
    # Logging
    access_log             /var/log/nginx/access.log main;

The $server_name is what you changed here?

Okay I just tested it with ip's but would be the same for domains:
<IP_ADDRESS>:8080 <IP_ADDRESS> - - [14/Nov/2024:09:02:03 +0000] "GET / HTTP/1.1" 200 11 "-" "curl/8.11.0"
<IP_ADDRESS> <IP_ADDRESS> - - [14/Nov/2024:09:02:05 +0000] "GET / HTTP/1.1" 200 615 "-" "curl/8.11.0"

 cat sites-available/* | grep server_name
        server_name <IP_ADDRESS>;
        server_name <IP_ADDRESS>:8080;

We are investigating an issue on our side, you changes are published, however, there currently issue downloading them remotely.
I will update this PR once we have it resolved.

Update: this should now be live and can be downloaded via updates
cscli hub update && cscli hub upgrade


Ambiguous Coordinates Dialog Box Showing for Coordinates that are not ambiguous

The ambiguous dialog box gets shown when it is not required. To reproduce open up Coordinate Conversion and for the input enter 0 100.

There is no requirement to show the confirm dialog box here as the input must be lat - long as latitude cannot be above 90

This was seen in both ArcMap and Pro

I have updated the first comment of this issue with more informaiton. @adgiles if I have missed something please add to this issue.

@CTLocalGovTeam I have updated the first comment in this issue to hopefully provide more clarity.

confirmed this is no longer happening in

ArcMap
ArcGIS Pro
added issue addressed to
CC June 2018 release (cms)
MT June 2018 release (cms)

Worked with @dfoll to verify doc changes:

[ ] Resources tab on both single addin and MT
[ ] PDF for Coordinate Conversion
[ ] PDF for Military Tools


Phone Number Validation Logic to Account for Extension number

Currently, our phone number validation logic throws a warning message and blanks the phone number when an extension number is included as part of the phone number. This is true for Ordering Facility Phone Number but it might be the same case for any phone number field.
Warning message:

Acceptance criteria

Modify phone validation logic to accept phone numbers with extension number: (555) 555 - 5555 ext. 5555

This might need further investigation as I can't find anything related to having extension number for ORC-23.

May be related to all of the NJ issues

Hey team! Please add your planning poker estimate with ZenHub @brick-green @greene858 @jbiskie @jorg3lopez @MauriceReeves-usds @oslynn @TomNUSDS


fix(bot,types)!: update DiscordAuditLogChange to be less manual

Our current DiscordAuditLogChange is pretty much outdated, on both the types and bot package. This is because while discord has been adding properties, we haven't updated the audit log change to reflect those, as it is very easy to forget about it.
This PR removes completely the AuditLogChange type and refactors the DiscordAuditLogChange to use the actual objects as source for all the values + some manually done for handling the exceptions discord has for some objects. The reason why I chose this is the following note in the Discord Developer Docs:

You should assume that your app may run into any field for the changed object, though none are guaranteed to be present. In most cases only a subset of the object's fields will be in the changes array.

So, while discord may never send a field, we should still have it typed in case it does send it.
For the removal of AuditLogChange: It is simply a mess to keep up to date, keeping it implies keeping the transformer up to date, and handing 230+ properties in the transformer is very hard to handle even with type errors, especially because the current solution is .map with a switch that will make the entire .map arrow function error, not just the single case that has an issue.
This pr also makes the values in OptionalAuditEntryInfo actually optional even if the type tecnically doesn't define them as such, however they will never exist all at the same time, so unless we create a discriminated union for every single case it is probably better to have the user null-check what they are going to use. Also, this adds the code for the transformation of application_id which for some reason was omitted, not sure exacly why, maybe the fact that it is duplicated? however, I would expect the reverse transformer to not just omit the application_id because it can...

[!WARNING]
This is breaking as removing the transformed type is going to break some code

I'm not sure about this... I do think it's a pain to maintain the transformer, but it feels wrong that this specific (can you even call this) "transformer" (anymore?) is not transforming the very thing it should be transforming.
@Skillz4Killz Thoughts on this?

is not transforming the very thing it should be transforming.

be clear, the transformer transforms the AuditLogEntry, i have removed the one for the AuditLogChange, however i do understand what you mean, the issue is that with 230+ props keeping track of everything is hard


Scale without UI/HUD acting strange?

Hello,
I've been using this plugin for... about a day now? My main issue is that, when I try to use the "Scale/Stretch" option, while EXCLUDING the use of UI/HUD, it starts to zoom in for a frame or two, and then just... forces the camera to focus on the lower right corner? When I use the Full Screen version that INCLUDES UI/HUD, it works as planned. That said, my game relies heavily on Pictures, so being able to zoom in while affecting Pictures and NOT the UI is really important to me.
Please feel free to tell me if I'm doing anything wrong in setting up the Zoom-in plugin. Attached is how I have it set up WITHOUT UI/HUD that makes it break. Video showcases what happens when I execute it. This is on a blank project with literally only this plugin.

https://github.com/user-attachments/assets/4a1b531e-a5b8-4275-8a07-38e86fd4e183

I guess that has something to do with that layer having a different resolution or a pivot point in another place. I'll look into it tomorrow.
In the meantime, you could try to disable "set pivot" as a workaround. I'm not sure if this'll work. I'll look for a real fix later.

Sorry for taking so long with fixing this. Fixed now with #b63b3ee but take a look at the plugin parameters.


Adding CUB200 dataset

This kind of dataset is interesting because of the attributes it has in addition of the labels. We may want to think how to integrate that. In the dataset, or in the scenario?

First in dataset and after maybe we can use attributs to create specific scenarios

Attributes on CUB200 are not so simple to use, we can create specific scenario with attributes on another dataset.


Use GitHub's facebox

Pardon the completely useless commit message... one thing led to another and I couldn't stop. I'll comment inline to explain most things. Views look 99% the same, I think.
Screenshots:
Options

Dropdown

Edit view

Diff is actually smaller than it seems! see https://github.com/notwaldorf/github-canned-responses/pull/13/files?w=1. :star:

Wahhhh I never merged this! Sorry @muan, I'm a bad maintainer :sob:


ci/create-docker-image.sh: Remove unused BASE build-arg

Both Docker.compile and Docker.crosscompile don't use the BASE
build-arg at all.  The output log of the CI (example) shows this:
[Warning] One or more build-args [BASE] were not consumed

This patch removes this unused build-arg from the
skydive-compile-build image building (it is still used in the skydive
or snapshot images to allow setting BASE per architecture).

run skydive-scale-tests


Loosing connections? Surely I am the problem.

Hi :)
I have this very small snippet:
extern crate mio;

use mio::tcp::TcpListener;
use mio::Token;
use mio::Poll;
use mio::Ready;
use mio::PollOpt;
use mio::Events;

use std::io::{Read, Write};
use std::str;
use std::io::ErrorKind;

const SERVER: Token = Token(0);

fn main() {
    println!("Hello, world!");

    let addr = "<IP_ADDRESS>:1883".parse().unwrap();
    let server = TcpListener::bind(&addr).unwrap();

    let poll = Poll::new().unwrap();

    poll.register(&server, SERVER, Ready::readable(), PollOpt::level()).unwrap();

    let mut events = Events::with_capacity(1024);

    let mut buffer: [u8; 1024] = [0; 1024];

    loop {

        poll.poll(&mut events, None).unwrap();

        for event in events.iter() {
            if event.readiness().is_readable() {
                match server.accept() {
                    Ok((mut tcp_stream, socketAddr)) => {
                        match tcp_stream.read(&mut buffer) {
                            Ok(n) => {
                                println!("Read {} bytes", n);
                                println!("Value: {}", str::from_utf8(&buffer).unwrap());
                                tcp_stream.write(&buffer);
                                buffer = [0; 1024];
                            }
                            Err(e) => {
                                if e.kind() == std::io::ErrorKind::WouldBlock {
                                    println!("Would Block");
                                }
                                println!("Error: {:?}", e);
                            }
                        }
                    }
                    Err(e) => {
                        println!("Error: {:?}", e);
                    }
                }
            }
        }
    }
}

The idea is a basic echo server.
However sometimes it doesn't work.
I send a TCP stream with nc and I would expect the same bytes back. Sometimes it work some other times it doesn't.
$ echo -e "foobar" | sudo nc localhost 1883 -D #OK
foobar
$ echo -e "foobar" | sudo nc localhost 1883 -D #OK
foobar
$ echo -e "foobar" | sudo nc localhost 1883 -D #OK
foobar
$ echo -e "foobar" | sudo nc localhost 1883 -D # <- ERROR
$ echo -e "foobar" | sudo nc localhost 1883 -D # <- ERROR
$ echo -e "foobar" | sudo nc localhost 1883 -D #OK
foobar

And what I see in the log is something like:
$ cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
     Running `target/debug/mett`
Hello, world!
Read 7 bytes
Value: foobar

Read 7 bytes
Value: foobar

Read 7 bytes
Value: foobar

Would Block
Error: Error { repr: Os { code: 11, message: "Resource temporarily unavailable" } }
Would Block
Error: Error { repr: Os { code: 11, message: "Resource temporarily unavailable" } }
Read 7 bytes
Value: foobar

The issues are not those two EAGAIN that I receive, the issues is that I never see the answer to those two request.
What is happening here?
It is me not understanding some basic concept ?

Thanks for the report! This issue has to do with the fact that mio uses non-blocking I/O. The error you're seeing, "temporarily unavailable", means that the bytes haven't arrived yet and you'd have to block waiting for a "readable" notification from the OS before attempting to read the bytes again. Whether or not it's immediately ready for being read is nondeterministic and not guaranteed, which is why it works sometimes but not others.

Hi Alex :)
still I am not sure I understand why I send 6 messages but I receive only 4.
I am completely ok with poll that sometimes wakeup without messaging ready to be read. However, next time poll wake up I would be expecting to see all the previous messages.
Is this expectation correct?

The two messages "dropped" are in transit and it's just nondeterministic that you received 4/6, it could have been possible that you received 0/6. To receive all messages you'd need to keep the socket alive until the mesage has been read.

Wait a second.
Each time I re-run nc, I am expecting that each run of nc open a new connection (3-way handshake and everything), and also I am expecting that every time that TCPStream goes out of scope the connection just created is closed.
Maybe I am starting to get it...
What I am doing wrong is that I do not poll on the TCPStream but only on the TCPListener.
The poll on the TCPListener tells me when a new connection is ready.
From there I create the TCPStream.
Then I poll the TCPStream to know when a new message is actually in the stream; at this point I am assured that there is an actual message to read.
Am I correct?

Yeah you'll want to register the accepted TCP stream with the Poll and then use that to wait for the stream to be readable, and that should do it!


fix(icon): export icon registry as part of icon

Currently, if you want to use md-icon, you have to import MdIcon and MdIconRegistry from separate files.  This PR consolidates the import, so you can do:
import {MdIcon, MdIconRegistry} from '@angular2-material/icon'
instead of:
import {MdIcon} from '@angular2-material/icon'
import {MdIconRegistry} from '@angular2-material/icon/icon-registry'

Now needs rebasing on my mega change #384

LGTM


feat(showcase): showcase paragraph widget

Relates to https://github.com/ratatui-org/website/issues/249
Followed the instructions in Widget Showcase Readme to add a simple example of the Paragraph widget.

Hey thanks for this, what about showing a little bit more. I think centering this and having something bold/underlined/colored could be nice.

@Valentin271 no problem and thanks for the feedback 🎉
By centering do you mean centering the Paragraph in the Frame ?

Done.
Are you also happy for me to attempt to provide PR's for the other widgets that are missing a showcase?

Of course no problem!


Neo4j docker images broken for version 4

I'm using the images on: https://hub.docker.com/_/neo4j?tab=tags
It seems the images for version 4 are all broken. I get this error when I build and run the container. Trying to create a Neo4j v4 database via the Neo4j desktop browser also fails.
Starting Neo4j.
2020-03-05 22:47:18.567+0000 INFO  ======== Neo4j 4.0.1 ========
2020-03-05 22:47:18.602+0000 INFO  Starting...
2020-03-05 22:47:34.712+0000 ERROR Failed to start Neo4j: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@3757e8e2' was successfully initialized, but failed to start. Please see the attached cause exception "org.neo4j.graphdb.event.TransactionEventHandler". Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@3757e8e2' was successfully initialized, but failed to start. Please see the attached cause exception "org.neo4j.graphdb.event.TransactionEventHandler".
org.neo4j.server.ServerStartupException: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@3757e8e2' was successfully initialized, but failed to start. Please see the attached cause exception "org.neo4j.graphdb.event.TransactionEventHandler".
	at org.neo4j.server.exception.ServerStartupErrors.translateToServerStartupError(ServerStartupErrors.java:45)
	at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:164)
	at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:114)
	at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:89)
	at org.neo4j.server.CommunityEntryPoint.main(CommunityEntryPoint.java:35)
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.server.database.LifecycleManagingDatabaseService@3757e8e2' was successfully initialized, but failed to start. Please see the attached cause exception "org.neo4j.graphdb.event.TransactionEventHandler".
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:465)
	at org.neo4j.kernel.lifecycle.LifeSupport.start(LifeSupport.java:111)
	at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:157)
	... 3 more
Caused by: java.lang.RuntimeException: Error starting database server at /data/databases
	at org.neo4j.graphdb.facade.DatabaseManagementServiceFactory.startDatabaseServer(DatabaseManagementServiceFactory.java:167)
	at org.neo4j.graphdb.facade.DatabaseManagementServiceFactory.build(DatabaseManagementServiceFactory.java:145)
	at org.neo4j.server.database.CommunityGraphFactory.newDatabaseManagementService(CommunityGraphFactory.java:36)
	at org.neo4j.server.database.LifecycleManagingDatabaseService.start(LifecycleManagingDatabaseService.java:88)
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:444)
	... 5 more
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.procedure.impl.GlobalProceduresRegistry@27a97e08' was successfully initialized, but failed to start. Please see the attached cause exception "org.neo4j.graphdb.event.TransactionEventHandler".
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:465)
	at org.neo4j.kernel.lifecycle.LifeSupport.start(LifeSupport.java:111)
	at org.neo4j.graphdb.facade.DatabaseManagementServiceFactory.startDatabaseServer(DatabaseManagementServiceFactory.java:158)
	... 9 more
Caused by: org.neo4j.internal.kernel.api.exceptions.ProcedureException: Failed to compile procedure defined in `Trigger`: org/neo4j/graphdb/event/TransactionEventHandler
	at org.neo4j.procedure.impl.ProcedureCompilation.compileProcedure(ProcedureCompilation.java:384)
	at org.neo4j.procedure.impl.ProcedureCompiler.compileProcedure(ProcedureCompiler.java:283)
	at org.neo4j.procedure.impl.ProcedureCompiler.compileProcedure(ProcedureCompiler.java:225)
	at org.neo4j.procedure.impl.ProcedureJarLoader.loadProcedures(ProcedureJarLoader.java:114)
	at org.neo4j.procedure.impl.ProcedureJarLoader.loadProceduresFromDir(ProcedureJarLoader.java:85)
	at org.neo4j.procedure.impl.GlobalProceduresRegistry.start(GlobalProceduresRegistry.java:342)
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:444)
	... 11 more
Caused by: java.lang.NoClassDefFoundError: org/neo4j/graphdb/event/TransactionEventHandler
	at java.base/java.lang.ClassLoader.defineClass1(Native Method)
	at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1016)
	at java.base/java.security.SecureClassLoader.defineClass(SecureClassLoader.java:174)
	at java.base/jdk.internal.loader.BuiltinClassLoader.defineClass(BuiltinClassLoader.java:800)
	at java.base/jdk.internal.loader.BuiltinClassLoader.findClassOnClassPathOrNull(BuiltinClassLoader.java:698)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClassOrNull(BuiltinClassLoader.java:621)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:579)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
	at java.base/java.lang.Class.getDeclaringClass0(Native Method)
	at java.base/java.lang.Class.isTopLevelClass(Class.java:1693)
	at java.base/java.lang.Class.getSimpleBinaryName(Class.java:1678)
	at java.base/java.lang.Class.getSimpleName0(Class.java:1562)
	at java.base/java.lang.Class.getSimpleName(Class.java:1553)
	at org.neo4j.procedure.impl.ProcedureCompilation.iteratorName(ProcedureCompilation.java:725)
	at org.neo4j.procedure.impl.ProcedureCompilation.generateIterator(ProcedureCompilation.java:615)
	at org.neo4j.procedure.impl.ProcedureCompilation.compileProcedure(ProcedureCompilation.java:347)
	... 17 more
Caused by: java.lang.ClassNotFoundException: org.neo4j.graphdb.event.TransactionEventHandler
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
	... 34 more
2020-03-05 22:47:34.714+0000 INFO  Neo4j Server shutdown initiated by request

What did you do to produce the error?
If I run docker run --rm neo4j:4.0.1 the neo4j 4.0.1 starts up as expected.

What did you do to produce the error?
If I run docker run --rm neo4j:4.0.1 the neo4j 4.0.1 starts up as expected.

Think I'm getting something similar. It works if I run that command, but if I use a docker-compose, then it breaks. The docker-compose works for images 3.5.14 however
version: '3.5'

services:
    database:
        container_name: n4j-database
        image: neo4j:3.5.14
        ports:
            - "7474:7474"
            - "7687:7687"
        networks:
            - backend
networks:
    backend:

@silverinkbottl3 thanks for the more detailed information! I tried running docker-compose up on your config, with neo4j:latest, 4.0.0, 3.5.15 and 3.5.14 and they all worked as expected.
The error message suggests that there's neo4j libraries missing from the classpath, which is odd in a docker environment. Especially if the same image works on your machine without docker-compose.
Could you try:

clearing your cache of docker images, especially the neo4j ones.
making sure your computer has enough free hard drive space to store the images
running one neo4j service on its own, so there are no concurrent services.

Hi :)
We were getting this exact error when I tried updating from neo4j 3.5 to 4.0.1. It turned out to be that we were still using the 3.5 version of apoc and deleting that (or updating it) allowed neo4j to start.
Hope that helps, and let me know if you need any more information from me.

Had a similar issue, and it was my first time running Neo4J on that particular machine.  Removed old images with  docker system prune -a  and ran the docker run --rm neo4j:4.0.1 again, it works now.

Hi :)
We were getting this exact error when I tried updating from neo4j 3.5 to 4.0.1. It turned out to be that we were still using the 3.5 version of apoc and deleting that (or updating it) allowed neo4j to start.
Hope that helps, and let me know if you need any more information from me.

thanks @catchouli , lifesaver!

I have the same issue with neo4j 4.0.1 and 4.1.3.
I've tried ocker system prune -a but it did not help.
Do you have any ideas?

Basically breaks on a regular basis. Quite annoying


new(ci): added github actions for non release jobs.

Signed-off-by: Federico Di Pierro<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind tests

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area commands

/area pkg

/area events

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 507ccb59ffd1fa413e097f180525c3eba3fa934e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


iOS: Invariant Violation: requireNativeComponent: "AirMap" was not found in the UIManager

Is this a bug report?
Yes
Have you read the Installation Instructions?
Yes
Environment
react-native: "0.57.5"
react: 16.6.1
react-native-maps<EMAIL_ADDRESS>Steps to Reproduce
The bug appears on iOS:

Create a new RN project: react-native init rnmapsbug
Install react-native-maps directly from git: npm install --save<EMAIL_ADDRESS>Link it: react-native link react-native-maps
Import it and use it:

// in App.js
import MapView from "react-native-maps";

// in your component class
<MapView
  style={{ ...StyleSheet.absoluteFillObject }}
  initialRegion={{
    latitude: 37.78825,
    latitudeDelta: 0.0922,
    longitude: -122.4324,
    longitudeDelta: 0.0421
  }}
/>

Run your app. The error will pop up: Invariant Violation: requireNativeComponent: "AirMap" was not found in the UIManager

Follow my guide above. I followed the installation instructions for manual installations on iOS (because I mustn't use CocoaPods in my project) and combined them with the fixes in this issue, because I also had the same bug, when I just ran npm install --save react-native-maps.
Expected Behavior
Maps should show on iOS.
Actual Behavior
The app crashes (see below).

Reproducible Demo
See steps to reproduce. You will end up with a demo in 3 minutes.

Yes i got the same error.

Yes! I have the same problem

Any solution yet?

Hello guys,
I solved this problem following this steps:

@mmatheusmmiguel you saved my day. Excellent answer, thank you. After juggling for a whole day with pod files, running into ton of issues with use_frameworks! etc, at the end, your solution worked the best. :)

@mmatheusmmiguel thank you for the clear and simple steps! Finally!

I have the same problem, but above solution does'n work for me. Maybe someone have other solution this problem.
This my configuration:
"react-native-maps": "0.25.0"
System:
    OS: macOS 10.14.5
    CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
    Memory: 342.55 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.15.3 - /usr/local/bin/node
    Yarn: 1.16.0 - /usr/local/bin/yarn
    npm: 6.10.0 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 12.4, macOS 10.14, tvOS 12.4, watchOS 5.3
  IDEs:
    Android Studio: 3.4 AI-183.6<IP_ADDRESS>22156
    Xcode: 10.3/10G8 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.8.6 => 16.8.6
    react-native: 0.60.4 => 0.60.4
  npmGlobalPackages:
    react-native-cli: 2.0.1```

@janhesters All is in the
linking natives librairies in the react native docs

for iOS, what worked for me was I did cd ios and then pod install, it showed new dependency react-native-maps is installed and my map showed up.

@coranmelia Thanks for your comment :D! it fixed by issues I can see the map now , but it outputs this error when starting the app on ios
$ react-native run-ios
error React Native CLI uses autolinking for native dependencies, but the following modules are linked manually:
  - react-native-maps (to unlink run: "react-native unlink react-native-maps")
This is likely happening when upgrading React Native from below 0.60 to 0.60 or above. Going forward, you can unlink this dependency via "react-native unlink <dependency>" and it will be included in your app automatically. If a library isn't compatible with autolinking, disregard this message and notify the library maintainers.


Fix #5: Adjust exception type

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
✔

Fixed issues
#5

TODO:  Also, I think we should move NotFoundException to a container and factory.


allow for limiting datasets in queries

currently all queries run on the whole KG. This should be limited to the source datasets only and parameterized, so that datasets can be changed in the query.

fixed in be113cc


[Perf] Read the TSConfig once then cache it.

Currently, it reads the call the getTSConfig on each process call:
https://github.com/kulshekhar/ts-jest/blob/ea5b0a5bc61c475c07f775981115f14cb5f74aab/src/preprocessor.ts#L22-L26
The process function will be called on each TS file. It means, for each file, there is an IO call to read the TSConfig file. However, during the process, the TSConfig will not be changed. It can cache the first TSConfig result and don't need to make IO call for the upcoming times.

I think this is a good idea. We need to keep in mind that the tsconfig can possibly change between builds though, if jest builds subprojects.

Hmm...? How does it happen? If that is a possible case, a cache Map may help.


not working as expected

I was expecting some plugin to add watermark to blog images. For the first time loading, it took ages to load images from a post, and images were loaded with the watermark successfully. I have not checked the source code yet, but I guess it was creating new images that with watermarks. However, when I refreshed the page (actually any page), the watermark was gone forever. pretty wierd.

Hi Frank
How many images per post do you have? How big are they?
What happens if you disable cache?
I think I was able to reproduce the bug, let me check ...

Ok I think I fixed the problem.
It is pushed to master and waiting to be deployed to OctoberCMS repository.
If you wish you can modify code by hand as shown in commit https://github.com/webeks/octobercms-imageking/commit/2c58371c709eb5f8f84d07304a64356f4365bb53
Let me know if it works :)

since no reply I will assume this works now ... closing.


Cannot reproduce the robustness results in Tensorflow

I use the PointCutMix-k augmentation method provided in this repo to try a DGCNN in Tensorflow. And attack the model with Saliency Map Attack(20iters, 5 points dropped in each iter). However, the accuracy under attack is only 80.97%, which is far lower than results in your paper.

hey, thanks for your attention, can you opensource your code?

The code I use is from https://github.com/WangYueFt/dgcnn/tree/master/tensorflow. I use the data augmentation part from your PointCutMix-k. I transform the numpy data into torch.tensor to obtain the mixed point cloud, and then I transform them into numpy to feed them into the tensorflow model. You can reproduce the result from this repo and attack the model with the Saliency Map Attack, which is from https://github.com/tianzheng4/PointCloud-Saliency-Maps.

What's the results did your point out? In table 4 or 5?

Table 5, of course. DGCNN can achieve 91.05% under Drop 100 attack. However, I only obtain 80.97%, when using Tensorflow. By the way, the DGCNN I use is from https://github.com/tianzheng4/PointCloud-Saliency-Maps.

Sorry, I find the attack in your paper is Black-Box. I test the model under White-Box. Can you provide more accuracy under White-Box attack? Because the Black-Box is not convincing enough to show the robustness, and there are no gradient masking in your model, the White-Box attacks can be easy to do.

Actually, if without using the PointCutMix-k to do data augmentation, the model can achieve 80%~81% accuracy under White-Box Saliency Map Attack as well.

Sorry, I find the attack in your paper is Black-Box. I test the model under White-Box. Can you provide more accuracy under White-Box attack? Because the Black-Box is not convincing enough to show the robustness, and there are no gradient masking in your model, the White-Box attacks can be easy to do.

I think the results in Table 4 are what you want.

Yes, accuracy in Table 4 is 85.86%. However, I can only achieve 80.97%, which is closer to the accuracy of the model without any data argumentation. The baseline, which is without using PointCutMix-k, in Tensorflow of DGCNN is 80.64%. And the baseline in Tensorflow of PointNet is 57.39%. I think there are some mistakes in your baselines.

Yes, accuracy in Table 4 is 85.86%. However, I can only achieve 80.97% when I only drop 100 points, which is closer to the accuracy of the model without any data argumentation. The baseline, which is without using PointCutMix-k and dropping 100 points, in Tensorflow of DGCNN is 80.64%. And the baseline in Tensorflow of PointNet is 57.39%. I think there may be some mistakes in your baselines or robust models. Or could you try to test on Tensorflow?

Please note that our results in Table 4 are drop 200 attack, and please make sure all the training scheme, standard data augmentation, learning rate, the checkpoint is the best test result's model are the same.
The baseline in Table 4 was adopted from IF-Defense, and actually, our baseline is lower than that, we will update our baseline in the future version.
I am not familiar with TF, but I think the difference shouldn't be that big, please check your code carefully.

Actually, if without using the PointCutMix-k to do data augmentation, the model can achieve 80%~81% accuracy under White-Box Saliency Map Attack as well.

I think the key difference is the drop attack of the IF-Defense baseline and the original drop attack.

Could you release the attack code as well for better evaluation on the robustness? Because I find officially released attacks are based on TF.

hey, we used the IF-Defense baseline to produce our results, I strongly recommend it.

I have spent a day to implement all parts of your methods. Here are some updates. I can achieve 81% accuracy under drop-200, which I think is acceptable, for there are still some differences between Tensorflow and Pytorch. However, when I try to do some ablation studies on every part of the code, I find that the label smoothing and data argumentation form PointcloudScaleAndTranslate improve the robustness of models more than PointCutMix-k. You could explore more about this. I think the statement that PointCutMix can enhance the robustness is kind of exaggerating. Hope this is useful for your work.

Huge thanks for your great work.
But I was wondering how did you validate the PointcloudScaleAndTranslate improve the robustness of models more than PointCutMix-k? Because we train models with PointCutMix-k and PointcloudScaleAndTranslate augmentation at the same time.
Anyway, thanks for the contribution, and I hope those findings may useful in your future research.

If you are willing to open source the reproduced TF code, I will be very grateful and recommend it in my repo.

Huge thanks for your great work.
But I was wondering how did you validate the PointcloudScaleAndTranslate improve the robustness of models more than PointCutMix-k? Because we train models with PointCutMix-k and PointcloudScaleAndTranslate augmentation at the same time.
Anyway, thanks for the contribution, and I hope those findings may useful in your future research.

Oh, I just replace the PointcloudScaleAndTranslate with a simple jitter, shifting and rotating translation. I need to do more explorations on my reproduced code before releasing it.

I find a bug in my Tensorflow code. When I use data augmentation like rotating and shifting, two point clouds may be not aligned. It can decrease the robustness when under attacks. So I see you only use the scale transformation before mix up them. After more experiments, the robustness brought by the PointCutMix is significant. There are no more questions from me. Thanks.
And for the Tensorflow code, I think I would release them later.

I find a bug in my Tensorflow code. When I use data augmentation like rotating and shifting, two point clouds may be not aligned. It can decrease the robustness when under attacks. So I see you only use the scale transformation before mix up them. After more experiments, the robustness brought by the PointCutMix is significant. There are no more questions from me. Thanks.
And for the Tensorflow code, I think I would release them later.

If you are interested in adversarial robustness, I recommend you to look at our newly released paper IT-Defense.


bump cudnn frontend to v0.9

v0.9 of the cudnn frontend has been released (https://github.com/NVIDIA/cudnn-frontend/releases/tag/v0.9).
Followed a format similar to PR #59675

@pgpetrak, can you update cudnn frontend internally and review this PR?
CC @kaixih

Will do.

Hi @pgpetrak Any update on this PR? Please. Thank you!


New Crowdin updates

@all-contributors please add @NM-EEA-Y for translation

@all-contributors please add @NM-EEA-Y for translation


Feature Request: Upsert

Is it possible to have atomic upsert functionality similar to postresql's INSERT ON CONFLICT or mysql's INSERT ON DUPLICATE KEY UPDATE statement? Use case is to save roundtrips on remote databases and to minimize lines of code.

This issue somehow depends on #127. Although, it's possible to define the conflict on any constraint.
I don't see any problem with adding this after #127 will be addressed.

@a8m do you have already a syntax for upsert in mind? Since upsert is different in Postgres/Sqlite and Mysql, Ent should probably abstract that common case.
Just a draft of my current thinking:
pedro, err := client.Pet.
    UpsertOneID(id).
    Create(
	pet.Create().SetName("pedro")
    ).
    Update(
        pet.Update().SetOwnerID(owner)
    ).
    Save(ctx)

This would allow us to reuse the Settters for Create and Update and ensure that Create and Update can include different properties.

This looks rather promising.

Any ETA on this? :)

I want one.

I need it.

yes please

Sorry to bug but is this being worked on? 🤦🏾♂

Has anyone made a start on this? I'd like to contribute!

Has anyone made a start on this? I'd like to contribute!

A few thoughts on this — at least from a Postgres standpoint — using ON CONFLICT requires some sort of annotation to indicate that a field is unique (already supported in the schema) or supply it with a named constraint (also generated by using index.Field(...).Unique(). Beyond that we need to update the graph inserter with the necessary pieces to write the conflict statement. Assuming something like @chris-rock suggested above, I've started to put this together:
// Pet holds the schema definition for the Pet entity.
type Pet struct {
	ent.Schema
}

// Fields of the Pet.
func (Pet) Fields() []ent.Field {
	return []ent.Field{
		field.String("name"),
	}
}

func (Pet) Indexes() []ent.Index {
	return []ent.Index{
		index.Fields("name").Unique(),
	}
}

// Usage:
pedro, err := client.Pet.
    Upsert().
    Create().SetName("pedro").
    Update().SetName("pedro-2").
    Save(ctx)

WIP https://github.com/facebook/ent/pull/1121

A few thoughts on this — at least from a Postgres standpoint — using ON CONFLICT requires some sort of annotation to indicate that a field is unique (already supported in the schema) or supply it with a named constraint (also generated by using index.Field(...).Unique(). Beyond that we need to update the graph inserter with the necessary pieces to write the conflict statement. Assuming something like @chris-rock suggested above, I've started to put this together:
// Pet holds the schema definition for the Pet entity.
type Pet struct {
	ent.Schema
}

// Fields of the Pet.
func (Pet) Fields() []ent.Field {
	return []ent.Field{
		field.String("name"),
	}
}

func (Pet) Indexes() []ent.Index {
	return []ent.Index{
		index.Fields("name").Unique(),
	}
}

// Usage:
pedro, err := client.Pet.
    Upsert().
    Create().SetName("pedro").
    Update().SetName("pedro-2").
    Save(ctx)

WIP https://github.com/facebook/ent/pull/1121

Has anyone made a start on this? I'd like to contribute!

I started an experiment for handing this, the only open question about it is whether we want to add a new hook/policy type for UpsertOne and Upsert (bulk).
Regarding the API, I used something as follows:
client.Pet.
    Create().
    // Setters...
    OnConflict(...).
    Update()
    // Setters...
    Save(ctx)

Has anyone made a start on this? I'd like to contribute!

I started an experiment for handing this, the only open question about it is whether we want to add a new hook/policy type for UpsertOne and Upsert (bulk).
Regarding the API, I used something as follows:
client.Pet.
    Create().
    // Setters...
    OnConflict(...).
    Update()
    // Setters...
    Save(ctx)

The explicit OnConflict API is nice — would it take a list of columns or options of some sort?
IIRC Postgres requires the conflict columns to be set to columns with a unique constraint and throws an error otherwise, would this be handled by Indexes/Field annotations?

The explicit OnConflict API is nice — would it take a list of columns or options of some sort?
IIRC Postgres requires the conflict columns to be set to columns with a unique constraint and throws an error otherwise, would this be handled by Indexes/Field annotations?

Also interested in bulk upsert. By now inserting or updating multiple objects can be done only one by one and that's very slow on large sets.

@DanielTitkov I've been slowly chipping away at this. It's quite complex when introducing Bulk changes, but I'm pretty close to having a PoC with Postgres to shake out how the API feels.

Hey all, and thanks @ivanvanderbyl for working on https://github.com/ent/ent/pull/1325.
We really want to push this feature forward, but I have some concerns/questions regarding the API (as mentioned in our Slack channel). I'll appreciate getting feedback/help/suggestions. Thanks 🙏

We prefer to start with UpsertOne and then add support for BulkUpsert (when we feel comfortable with the new API).
What about Hooks and Privacy, do we need to provide a new operation for it? If so, do we need to provide an API that indicates if an entity was created or updated (post hook/privacy-rule).
MySQL doesn't support the RETURNING clause, and  SQLite added support for it 2-3 months ago. What should be the return value of the Upsert function? Do we need to provide a few alternatives like Exec/Save and fail on runtime if the driver doesn't support getting the rows back? Or, maybe we can handle this using SELECT FOR UPDATE?
@sneakywombat proposed a new API for it here - https://github.com/ent/ent/pull/1325#issuecomment-856901461. I actually like it, but using Where(...) is not possible with @ivanvanderbyl's implementation (right?). What about supporting both APIs?// #1
client.User.Create().
    SetEmail("boring@entgo.io").
    OnConflict(user.FieldEmail).
    Update().
    SetEmail("boring2@entgo.io").
    Save(ctx)

// #2 (Will execute 2 statements to the databases. 1 for query records, and another one for creating or updating records).
client.User.Upsert().
    Where(user.Email("boring@entgo.io")).
    Create(... /* ent.UserMutation */)
    Update(... /* ent.UserMutation */)
    Save(ctx).

Hey all, and thanks @ivanvanderbyl for working on https://github.com/ent/ent/pull/1325.
We really want to push this feature forward, but I have some concerns/questions regarding the API (as mentioned in our Slack channel). I'll appreciate getting feedback/help/suggestions. Thanks 🙏

We prefer to start with UpsertOne and then add support for BulkUpsert (when we feel comfortable with the new API).
What about Hooks and Privacy, do we need to provide a new operation for it? If so, do we need to provide an API that indicates if an entity was created or updated (post hook/privacy-rule).
MySQL doesn't support the RETURNING clause, and  SQLite added support for it 2-3 months ago. What should be the return value of the Upsert function? Do we need to provide a few alternatives like Exec/Save and fail on runtime if the driver doesn't support getting the rows back? Or, maybe we can handle this using SELECT FOR UPDATE?
@sneakywombat proposed a new API for it here - https://github.com/ent/ent/pull/1325#issuecomment-856901461. I actually like it, but using Where(...) is not possible with @ivanvanderbyl's implementation (right?). What about supporting both APIs?// #1
client.User.Create().
    SetEmail("boring@entgo.io").
    OnConflict(user.FieldEmail).
    Update().
    SetEmail("boring2@entgo.io").
    Save(ctx)

// #2 (Will execute 2 statements to the databases. 1 for query records, and another one for creating or updating records).
client.User.Upsert().
    Where(user.Email("boring@entgo.io")).
    Create(... /* ent.UserMutation */)
    Update(... /* ent.UserMutation */)
    Save(ctx).

cc @rotemtam, @alexsn

Hey @a8m ,
My few 0.02$ on the issue:

We prefer to start with UpsertOne
I agree, start with UpsertOne. From my (limited) perspective, this is the common use case.

What about Hooks and Privacy, do we need to provide a new operation for it?
In general, as discussed, there are two approaches to upsert like behavior:

INSERT .. ON DUPLICATE KEY UPDATE
SELECT .. FOR UPDATE (SfU henceforth)
I think it will be much simpler to support upserts in ent using SfU since it fits very nicely with the current hooks/privacy definitions. In order for a mutation to succeed, it must pass both privacy checks instead of adding a new, somewhat weird "upsert" notion.
It has the additional advantage that, as you mentioned,

MySQL doesn't support the RETURNING clause
Using SfU simplifies this because we can then return the final value for any dialect.

The downside of course is SfU requires 2 round-trips to the database and holding a lock on a resource which in some use cases my cause substantial performance issues. I think those can be solved using something which was discussed in another issue (can't find it now), which would provide a generic API for modifying SQL queries in place in those edge cases were specific tweaking is required.

Summarizing a discussion with @a8m, @ivanvanderbyl, and @arielitovsky from June 22nd

INSERT .. ON DUPLICATE do not map nicely to ent's current hooks/privacy infrastructure. to keep those in place, upsert APIs will be available out of the box for working with a single record and be implemented with SELECT .. FOR UPDATE.

INSERT .. ON DUPLICATE are useful for bulk upserts, mostly in data pipeline job style setting to improve performance by reducing the number of calls to the database.
These will either be implemented by adding an extension/feature flag to generate bulk upsert methods or by adding a "query mutation" modifier to make low-level modifications to queries before they are executed. Either way, the API and documentation should make it clear to the user that this path bypasses hooks and privacy checks and should be used with caution.

Hey, the upsert and upsert-bulk APIs were added using a new feature flag. Please, feel free to share your thoughts on the PR. https://github.com/ent/ent/pull/1793

Closing, as it's been used in production for more than 3 months and work without any issues. Thanks all for the feedback ❤

Can we unpin this issue? :D


Bot delete old Role

Hey, you're bot is very cool, but i have a question, how to not delete OLD role in discord server ?
Like people have another role, but the bot delete all OLD role and not just add the role
It's possible to fix this ?

Second that one, as I have an already existing community on my discord I want to add people from my forum and give them permissions only once through the forum, I don't want old accounts to get removed through.

The bot only sync roles which are configured in the settings of the plugin.
Discord roles which are not configured in the plugin (mybb) should not be modified by the bot.


New variance measures: margin of error and coefficient of variation

Adding margin of error (MOE) and coefficient of variation (CV)
So far all of our data points had have been calculated with only one measure of variance: standard error (SE). This pull request implements processes to replace SE with MOE and add CV.
How calculations are done
This is a review but it's helpful context. The calculations are done with the R survey package which we call via the rpy2 python package. We use this package to get variance measures from the replicate weights. There are three files in /statistical responsible for using the survey package to do calculations: calculate_counts.py, calculate_fractions.py, calculate_medians.py.
Each of these was modified to return a CV and replace SE with MOE.
Calculate MOE
The MOE.py file in statistical takes a dataframe and calculates the MOE using the a z-score corresponding to the 90% confidence interval, calculated with the python stats package.
It can either replace the SE or keep the SE in. In retrospect this functionality was not worth the time invested to build, should have written it to always replace the SE.
All calls to any calculate code now includes an add_MOE flag and keep_SE flag that are to be set by whomever calls the code. add_MOE defaults to true and keep_SE defaults to false.
Calculate CV
The vartype parameter to the svyby function from the survey package can be set to return a CV. In all calls to svyby this parameter is set to ["se", "ci", "var", "cv"] which are 4 variance measures that svyby can return. We only need cv but it doesn't work when only cv is passed. I'm not sure why.
We take CV as a decimal instead of a percentage, it should be easy to convert back to percentage later on if we need to
Debug fractions crosstabs
The existing code get crosstabs functions backward. When we crosstab by race we want the fraction of people in each category that are each racial group. So for example in labor force we want the fraction of all people in labor force that are white. What we were doing is fraction of all white people in the labor force.
While debugging the code was simplified. The calculate_fractions_crosstabs function in calculate.py was removed and instead the calculate_fractions function takes a single flag parent_category that constructs the column label to distinguish whether the fraction is a crosstab.
Categories with no respondents
In these cases our counts and fractions should be reported as 0 and the associated measures of variance reported as NA.
The survey package reports the count of a category with zero respondents as NA. This is converted  to 0 in counts_to_zero in calculate_counts.py. It reports the variance as NA as well which is correct and doesn't need to be cleaned.
The survey package reports the fraction of a category with zero respondents as 0 which is correct (although not intuitive). It reports the associated SE as 0 however which is not correct. This is converted to NA in SE_to_zero_no_respondents
There is also the cleaning of median calculation for these situations. Because the only median we do now is age, the buckets are just PUMA-race and obviously we have survey respondents in each bucket. So there is no corresponding situation to test. However when we do median wages by industry we know there are some industry-race buckets with zero respondents. My decision is to wait until we start that indicator to do this cleaning on medians. I think that adding a dummy category to the existing pipeline is to much effort.
Tests
The tests are pretty simple. They were helpful to check that the code runs but were a pain to work with due to long run times.
Currently each file does it's own aggregation of each data point which takes forever. This could be refactored to use a single aggregated dataframe across multiple tests.
A couple changes to existing tests were added, maybe they shouldn't be in this pull request.
Miscellaneous
The aggregation tests don't run on each push as they never pass and it's not helpful
All calls to calculate code should pass a copy of the dataframe so that operations on the data in the calculate code don't effect the data stored in the class attribute. Good practice generally I think
The return f'occupation-{occupation_mapper.get(person["OCCP"], "none")} line will be made irrelevant when we correctly set the denominator for occupation.

Added files for internal review. These files should have margin of error, coefficient of variation, and standard error. Our final output won't have standard error but it's helpful for review

Ok neat. In my opinion we should merge these changes in and then worry about column naming later. @AmandaDoyle is that the right approach?


Open a directory prompt popup shows weird rectangle in sub-folder

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Godot v4.1.1.stable - Ubuntu 22.04.3 LTS 22.04 - Vulkan (Compatibility) - Mesa Intel(R) HD Graphics 4600 (HSW GT2) () - Intel(R) Core(TM) i7-4710HQ CPU @ 2.50GHz (8 Threads)
Issue description
I was using GUT which relies on folder selection to find test scripts, and the popups started showing weird blue rectangles when I entered the test subfolder.
The rectangle changes position depending on whether a folder is selected or not.

I assume GUT is using a native function and therefore this is a Godot bug.
I'm not familiar with the API to prompt folder selection myself so I couldn't write a true MWE but feel free to upload your own, or point me to said API (honestly it's more a matter of time as I want to go on with my project right now).
Steps to reproduce

In any project, install GUT and activate add-on, see https://github.com/bitwes/Gut/wiki/Install
Open GUT panel, on the right side scroll down, and under Test Directories, select Directory 1 > ...
In the folder selection popup, enter any subfolder and click on a folder, then outside a folder area to test both rectangle positions

Minimal reproduction project
N/A

Duplicate of #74115
Thanks for reporting the bug nonetheless!


Adding AccountInvitation into core model

Will enable a user (existing/new) to be invited into an Organisation account
implements issue: https://github.com/shesha-io/shesha-framework/issues/1825

closing this, misattributed


Make FlowDirection's Path of CheckBox fixed LTR

What does the pull request do?
When turning the Checkbox on and off the glyph would flip in RTL mode, this PR makes it fixed as LTR. Simple Theme works fine.
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes
No.

You can test this PR using the following package version. 11.0.999-cibuild0028001-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0028011-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Do not spam with Updated user setting 'typescript.check.tscVersion' to false

Please do not show the message "Updated user setting 'typescript.check.tscVersion' to false". The user just clicked on that, no need to show the second message and spam the user

Closing as this setting was removed


Allow passing comma in string quoted DHCP client options

For some ISPs it is necessary to send DHCP client options, which contain commas in a quoted string. E.g:
dhcp-class-identifier "100008,0001,,pfSense dhclient 2.1"

Up until now, inserting such an option into Interfaces > Lease Requirements and Requests >  Send options and related options creates invalid dhclient config files. This patch allows setting commas in quoted strings.

Hmm, this seems to duplicate #2793. This might be an alternative implementation ;-).

Consider using foreach loop.  Cleaner than a for count loop.
Consider making the function generic.  ex: csv_str_to_array, $str, $matches, $array.
What is the use case for the second regex substring?  Given the first substring is it possible for the second substring to begin with a double quote?
Don't know which method is more desirable, explicit regex or the preg match call back.  I'm on the fence myself.  Like the delimiter and enclosure flexibility of the preg match call back way, but like the "simplicity" of the explicit regex way.

Thank you for your review. I simplified my version quite a bit. I also tried to make it more flexible, but the regex became a monster as soon as i tried to interpolate variables. Therefore if we need a very flexible solution, then yours it is.
The first part of the regex [^",]*(?:"[^"]*"[^",]*)+(?:"[^",]*)? only matches tokes with quotes. The second part [^,]+ only matches tokens without quotes. The thing is, that if you replace the + of the first part with * it matches empty strings as well and would return them.
Anyway, I don't care which solution is going to be merged as long as one of them is.

What about something like this?
Regex parts broken out for example clarity.  Personally that's probably how I'd write it too.  I like breaking up long or complex regex patterns to be more manageable and comprehendible.  But others may not like it like that.
function csv_str_to_array($str, $enclosure = '"', $delimiter = ',') {
/*
	$part1 = '[^'.$enclosure.$delimiter.']*';
	$part2 = '(?:"[^'.$enclosure.']*'.$enclosure.'[^'.$enclosure.$delimiter.']*)+';
	$part3 = '(?:'.$enclosure.'[^'.$enclosure.$delimiter.']*)?';
	$part4 = '[^'.$delimiter.']+';

	$csv_pattern = $part1.$part2.$part3.'|'.$part4;
/**/
	$csv_pattern = '[^'.$enclosure.$delimiter.']*(?:"[^'.$enclosure.']*'.$enclosure.'[^'.$enclosure.$delimiter.']*)+(?:'.$enclosure.'[^'.$enclosure.$delimiter.']*)?|[^'.$delimiter.']+';
	preg_match_all('/'.$csv_pattern.'/m', $str, $matches, PREG_PATTERN_ORDER);

	return $matches ? $matches[0] : [];
}

I was referring to the second sub-pattern.  The second set of parenthesis.  Not "or" part.
Anyway, nice work.  Yes I'd like to see one or the other get merged too.  Don't know why it hasn't been merged.  Would be nice if someone would at least make some kind of feedback comment on it.  Like "no way not going to happen because xyz", or "like it but probably hold off until abc".  etc.  The crickets MO (mode of operation) leaves people in the dark.  And in my opinion is disrespectful.

I added some comments to the regex, hopefully they might help:
[^",]*        # match text in front of protected string
(?:
  ".*?"       # match protected string (with separators included)
  [^",]*      # match text after protected string
)+            # match one! or more protected strings in one token
(?:"[^",]*)?  # match string until next separator in case of unbalanced string quotes
|[^,]+        # match tokens without protected strings

The following two images should display the difference in result:
with the second sub-pattern:

without the second sub-pattern:

So instead of totally failing in case of unbalanced quotes, this part is trying to save, what can be saved. Probably this is worth a discussion.

Should the code try to interpret, what it can and avoid issues?
Should it just stubbornly do what it was being told?
Should it raise an exception and therefore fail hard?

I was actually writing this code with interpolated variables as well. But what a mess, this is is quite far from readable code. Compared to your solution with regex callbacks, yours feels more like reading a poem 😄 .


[PR6] Add api to determine if node is in a nat environment

Signed-off-by: Karim TAAM<EMAIL_ADDRESS>PR description
Java APIs to determine if a node is in a NAT environment, and then identify the kind of NAT environment.
This API will allow to detect the docker, kubernetes or other environment when it is feasible
In order to detect a nat environment you need to implement the NatMethodAutoDetection interface
public class SampleNatMethodAutoDetection implements NatMethodAutoDetection {
    @Override
    public AutoDetectionResult shouldBeThisNatMethod() {
        //ADD CHECK
        return new AutoDetectionResult(NatMethod.SAMPLE, true);
    }
}

And pass this method to the NatService when trying to detect
final SampleNatMethodAutoDetection natDetectionMethod = new SampleNatMethodAutoDetection();
final Sample2NatMethodAutoDetection natDetectionMethod2 = new Sample2NatMethodAutoDetection();
final NatMethod natMethod = NatService.autoDetectNatMethod(natDetectionMethod, natDetectionMethod2 );

Hi @RatanRSur Here is a new PR ready for the review. Thanks


Fix linker error

This linker error didn't show up locally but showed up on travis after disabling the xcpretty filtering.

Green travis run for FIAM at https://travis-ci.org/firebase/firebase-ios-sdk/jobs/550866129


added license to readme

saves a click. Not sure if it is what you want but I like it when licenses are called out in the readme

No thanks. Sorry 😬


UnderlineNav2 Flickering problem when moving pages (with nextjs)

Description
I found a new issue flickering UI about UnderlineNav2 while trying to reproduce issue #2651.
https://user-images.githubusercontent.com/15520015/206831112-5536736c-0119-43e2-b6ee-2a1a39c26d77.mov
Steps to reproduce

Go to ~https://codesandbox.io/p/sandbox/primer-underline-nav2-bug-ojlgdh~ https://stackblitz.com/edit/primer-undelinenav2-issue-2651-2676
Adjust the width of the preview window until the more button appears.
While switching pages, check the flickering on the right side of the nav bar.

Version
v35.15.1
Browser
Chrome

Codesandbox was unstable, so I recreated it with stackblitz.
https://stackblitz.com/edit/primer-undelinenav2-issue-2651-2676

Hi there! 👋
I believe this happens when components like UnderlineNav are expected to share state between route transitions. Because the routes are separate components, there is a flash as the component is mounted and then has to sync with the current state of the world.
This can be solved by moving the shared layout up into _app.js or alternatively by using per page layouts (link in: https://nextjs.org/docs/basic-features/layouts#per-page-layouts) in order to preserve state and keep the component mounted between transitions. Here's a quick example with the per-page layout approach: https://stackblitz.com/edit/primer-undelinenav2-issue-2651-2676-qkweh9
Hope this makes sense, let me know if you have any questions or if I misunderstood anything!

@b6pzeusbc54tvhw5jgpyw8pwz2x6gs I'm going to close this issue as guidance has been provided, but please comment back here if needed and we can reopen the issue. Thanks!


Salsa - reference search across commonjs fails

In the following setup:
version<EMAIL_ADDRESS>jsconfig.json
{
    "compilerOptions": {
        "module": "commonjs",
        "target": "es5"
    },
    "exclude": [
        "node_modules"
    ]
}

circle.js
const PI = Math.PI;

exports.area = function (r) {
  return PI * r * r;
};

geometry.js
const circle = require('./circle.js');
console.log( `The area of a circle of radius 4 is ${circle.area(4)}`);

Go to definition from geometry.js area works and navigates to the file circle.js.
Doing a reference search from circle.js exports.area doesn't result in any references.
When changing the code of circle.js to:
const PI = Math.PI;

export function area (r) {
  return PI * r * r;
};

then the reference search returns the correct result.

This should be fixed by #6632


You are also adding a duplicate sheet.json to the charsheets root as well. Please fix these and create a new pull request.


Look into implementing Mermaid plugin on Discourse

Enhancement Proposal
Mermaid is a tool for creating diagrams and charts and there is a plugin available for Discourse. When enabled, the plugin can render mermaid diagrams inside a [mermaid] block, enhancing documentation. The tool is widely used and should add value to a Discourse deployment. Potentially this can be an optional plugin that is not enabled by default?

mermaid plugin is enabled on the discourse charm and working (tested a deployment on microk8s), hence closing as done


[Feat][Spark] Update snake-yaml dependency to 2+ version

Is your feature request related to a problem? Please describe.
Spark/PySpark may be installed with both Hadoop2 and Hadoop3. Hadoop2 requires snakeyaml of the version 2 that is incompatible with version 1.26 that is used now.
Describe the solution you'd like
Update snakeyaml to version 2.0 and update code (few lines only). The new code may looks like:
val yaml = new Yaml(new Constructor(classOf[VertexInfo], new LoaderOptions()))

(currently there is no LoaderOptions passed)
I made a check and with updating snakeyaml to 2.0 and updating the code by passing LoaderOptions project is passing all the tests. I can open a PR with such an update.
Describe alternatives you've considered
.
Additional context
It is related to #300 because when you run PySpark session it will use priority classes from PySpark instead of classes from shaded jar.
Another important fact is that version 2 of snakeyaml fixed a known vulnerability (CVE-2022-1471). I can guess, that this vulnerability and it's fix were the reason of changing Constructor signature.

@SemyonSinchenko Thanks for the report, the solution looks good for me and the version of snake-yaml should be update indeed according the facts you describe.

solved.


inner join syntax

I've got the following query that works fine, and ive figured out how to transpose it to the Dart API, however I can't work out correct inner join syntax for the C# API. can u let me know the correct syntax to use for inner joins ? also how can I return fields from the different tables (as per select p.name, mc.job, mc.department, m.name as moviename) ?
        select p.name, mc.job, mc.department, m.name as moviename
        from movie_crew mc
            inner join movie m
                on m.id = mc.movie_id
        inner join person p
        on p.id = mc.person_id
        where m.id = '90330ca1-8e38-4349-aabd-f5d95ece43b1'
        order by mc.job;

I tried the following but it's not working correctly. Pretty sure I've got the wrong syntax for the inner join piece:
var res2 = await supabase
                .From<PostgresModels.MovieCrew>()
                .Select("*, movie!inner(*)")
                .Select("*, person!inner(*)")
                .Filter("movie_id", Constants.Operator.Equals, "90330ca1-8e38-4349-aabd-f5d95ece43b1")
                .Get();

hey all, should I perhaps post this in the Github discussions instead?

Apologies @ajonno, had a lot of family stuff over the last 2 weeks. I’ll look into this tonight/tomorrow and get back to you!

Thanks mate no rush, family comes first!

@ajonno thank brother, appreciate that! Could you post an example of the schema that you're using when you get a chance?

Appreciate your understanding! Needed the time with them, but all's settled now.
Given the following schema:
CREATE TABLE public.movie (
  id serial NOT NULL,
  created_at timestamp without time zone NOT NULL DEFAULT now(),
  name character varying(255) NULL
);
ALTER TABLE public.movie ADD CONSTRAINT movie_pkey PRIMARY KEY (id);

CREATE TABLE public.person (
  id serial NOT NULL,
  created_at timestamp without time zone NOT NULL DEFAULT now(),
  first_name character varying(255) NULL,
  last_name character varying(255) NULL
);
ALTER TABLE public.person ADD CONSTRAINT person_pkey PRIMARY KEY (id);

CREATE TABLE public.movie_person (
  id serial NOT NULL,
  movie_id integer NOT NULL,
  person_id integer NOT NULL
);
ALTER TABLE public.movie_person ADD CONSTRAINT movie_person_pkey PRIMARY KEY (id);

insert into "public"."movie" ("created_at", "id", "name") values ('2022-08-20 00:29:45.400188', 1, 'Top Gun: Maverick');
insert into "public"."movie" ("created_at", "id", "name") values ('2022-08-20 00:29:45.400188', 2, 'Mad Max: Fury Road');

insert into "public"."person" ("created_at", "first_name", "id", "last_name") values ('2022-08-20 00:30:02.120528', 'Tom', 1, 'Cruise');
insert into "public"."person" ("created_at", "first_name", "id", "last_name") values ('2022-08-20 00:30:02.120528', 'Tom', 2, 'Holland');
insert into "public"."person" ("created_at", "first_name", "id", "last_name") values ('2022-08-20 00:30:33.72443', 'Bob', 3, 'Saggett');

insert into "public"."movie_person" ("id", "movie_id", "person_id") values (1, 1, 1);
insert into "public"."movie_person" ("id", "movie_id", "person_id") values (2, 2, 2);
insert into "public"."movie_person" ("id", "movie_id", "person_id") values (3, 1, 3);

The following query to http://localhost:3000/movie?select=*,person!inner(*)&person.first_name=eq.Bob should produce:
[
  {
    "id":1,
    "created_at":"2022-08-20T00:29:45.400188",
    "name":"Top Gun: Maverick",
    "person":[{"id":3,"created_at":"2022-08-20T00:30:33.72443","first_name":"Bob","last_name":"Saggett"}]
  }
]

The same result can be replicated by the following in C#:
var movies = await client.Table<MovieWithPerson>()
                .Select("*,person!inner(*)")
                .Filter("person.first_name", Constants.Operator.Equals, "Bob")
                .Get();

Referencing the following model:
using System;
using System.Collections.Generic;
using Postgrest;
using Postgrest.Attributes;
using Postgrest.Models;

namespace PostgrestExample.Models
{
    [Table("movie")]
    public class MovieWithPerson : BaseModel
    {
        [PrimaryKey("id", false)]
        public int Id { get; set; }

        [Column("name")]
        public string Name { get; set; }

        [Column("person")]
        public List<MoviePerson> Persons { get; set; }

        [Column("created_at")]
        public DateTime CreatedAt { get; set; }
    }

    public class MoviePerson
    {
        [PrimaryKey("id",false)]
        public int Id { get; set; }

        [Column("first_name")]
        public string FirstName { get; set; }

        [Column("last_name")]
        public string LastName { get; set; }

        [Column("created_at")]
        public DateTime CreatedAt { get; set; }
    }
}

Hopefully that helps!
Joseph

@ajonno did that help? Or is something still not working on your end?

hey @acupofjose sorry been crazy busy last week will have time to check all this next few days stay tuned!

No worries! Just wanted to be sure your issue gets resolved


fix: Use number facade with namespace

This will fix the error appearing post-install when using the livewire component!

Thanks!


Noir Contracts: Migrate 'airdrop' claim functionality to its own 'zk_token_airdrop' example contract

The createClaim and  claim functions are unconventional enough to warrant their own example file, so as to keep the main zk_token example simple.
Also, if possible, the claim example could do with more commentary to make the intentions of the example clear.

I find myself doing this so that I can write some docs content.


Release 3.2.3

Summary
The changes between v3.2.3 and v3.2.2 can be found at https://github.com/mattermost/mattermost-plugin-jira/compare/release-3.2.2...release-3.2.3.
Ticket Link
Part of https://github.com/mattermost/mattermost-plugin-jira/issues/905

@hanzei or @mickmister - The build is failing in CI here.
I also when I try to compile myself I see the following
➜  mattermost-plugin-jira git:(release-3.2.3-bump) ✗ make dist
./build/bin/manifest apply
mkdir -p server/dist;
cd server && env GOOS=linux GOARCH=amd64 /opt/homebrew/bin/go build -ldflags '-X "github.com/mattermost/mattermost-plugin-jira/server/utils/telemetry.rudderWriteKey=1d5bMvdrfWClLxgK1FvV3s4U1tg" -X "main.BuildDate=Wed 25 Jan 2023 17:19:13 UTC" -X "main.BuildHash=49ac4cbf92a4d5a26ef73dd866ec7d0bf77939dc" -X "main.BuildHashShort=49ac4cb"' -trimpath -o dist/plugin-linux-amd64;
# github.com/mattermost/mattermost-plugin-jira/server
./issue.go:554:26: client.GetCreateMeta undefined (type Client has no field or method GetCreateMeta)
make: *** [server] Error 2

Please advise

@DHaussermann I fixed a typo that was causing the error

@DHaussermann https://github.com/mattermost/mattermost-plugin-jira/pull/909 is merged now. Are we good to go?


Support hashing different types for sub-partitioning

Relevant to https://github.com/rapidsai/cudf/issues/13000
This PR changes to leverage the Plugin way to partition batches instead of the cudf hashPartition in sub-partitioning.
Because the left and right join keys with the name value but different integral types(e.g. 100.toInt and 100.toByte) will produce different hash values by cudf hashPartition, breaking the join by sub-partitioning algorithm.

build


Wrapping Typedefs ITKOptimizersBase fails on VS2017 LEGACY_REMOVE

On VS2017 (fully updated) I get the following error message:
cmd.exe /C "cd /D C:\src\ITK-Release\Wrapping\Typedefs\python && C:\src\ITK-Release\Wrapping\Generators\SwigInterface\swigwin-3.0.12\swig.exe -c++ -python -O -features autodoc=1 -py3 -Werror -w302 -w303 -w312 -w314 -w361 -w362 -w350 -w383 -w384 -w389 -w394 -w395 -w467 -w508 -w509 -o C:/src/ITK-Release/Wrapping/Modules/ITKOptimizers/ITKOptimizersBasePython.cpp -IC:/src/ITK-Release/Wrapping/Generators/SwigInterface/swig/share/swig/3.0.12/python -IC:/src/ITK-Release/Wrapping/Generators/SwigInterface/swig/share/swig/3.0.12 -IC:/src/ITK/Wrapping/Generators -IC:/src/ITK-Release/Wrapping/Typedefs/python -IC:/src/ITK-Release/Wrapping/Typedefs -outdir C:/src/ITK-Release/lib C:/src/ITK-Release/Wrapping/Typedefs/ITKOptimizersBase.i"
Assertion failed: Getattr(n, "sym:name"), file Modules/lang.cxx, line 1657
ninja: build stopped: subcommand failed.
The last checkout that works is:
https://github.com/InsightSoftwareConsortium/ITK/commit/f426a2a27b66d3fbb69ad3d31370d6e3205a7cb0
Suggesting that the error was introduced in
https://github.com/InsightSoftwareConsortium/ITK/commit/35865cadf56fb5350561e1dbdd6ddb2d5144830a
This bug persists in Master/Head today.
CMake options used
ITK_LEGACY_REMOVE = ON
I thought it was due to the removal of the conditional defined(ITK_WRAPPING_PARSER) in several places, but restoring them does not fix the issue.

I can reproduce this:
1>------ Build started: Project: castxml (ExternalProjectTargets\castxml\castxml), Configuration: Debug x64 ------
2>------ Build started: Project: swig (ExternalProjectTargets\swig\swig), Configuration: Debug x64 ------
3>------ Build started: Project: ITKCommonCastXML, Configuration: Debug x64 ------
4>------ Build started: Project: ITKStatisticsCastXML, Configuration: Debug x64 ------
5>------ Build started: Project: ITKOptimizersCastXML, Configuration: Debug x64 ------
6>------ Build started: Project: ITKCommonSwig, Configuration: Debug x64 ------
7>------ Build started: Project: ITKStatisticsSwig, Configuration: Debug x64 ------
8>------ Build started: Project: ITKOptimizersSwig, Configuration: Debug x64 ------
9>------ Build started: Project: ITKOptimizersPython, Configuration: Debug x64 ------
9>Generating ITKCostFunctionsPython.cpp, ../../../lib/ITKCostFunctionsPython.py
9>Generating ITKOptimizersBasePython.cpp, ../../../lib/ITKOptimizersBasePython.py
9>Assertion failed: Getattr(n, "sym:name"), file Modules/lang.cxx, line 1657
9>C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Microsoft\VC\v150\Microsoft.CppCommon.targets(209,5): error MSB6006: "cmd.exe" exited with code 3.
9>Done building project "ITKOptimizersPython.vcxproj" -- FAILED.
========== Build: 8 succeeded, 1 failed, 107 up-to-date, 0 skipped ==========

I guess that this is a bug in SWIG.

The file mentioned in the error message is C:\Dev\ITK-2017\Wrapping\Generators\SwigInterface\swigwin-3.0.12\Source\Modules\lang.cxx.

I have a patch for this coming :-) -- will push after I have finished testing variants.

Our Windows Azure Python is green again :green_heart: :tada:


Generic Input/Output

Is it possible to work with generic input/output instead of defining in/outputs for every function call?

I am working on the subject. In the near future, I will add an improvement on the subject so that we can return results as DataTable or Object.
Thank you for your notification.

Unfortunately it was not completed.


Make sure all components are documented

The purpose of this ticket is to make sure that we "Maintain up-to-date documentation for all implemented components, APIs, and dependencies to facilitate smooth collaboration and knowledge transfer within MOJ Digital"
We should make sure every component has a readme or other documentation, and remove anything that is out of date. Consider moving content into the runbooks where it makes sense.

[x] Find MOJ data repo
[x] Custom frontend components
[x] The custom ingestion sources / transformers
[ ] Data catalogue repo

mostly updated and reviewed.
Outstanding is the data catalogue repo readme secrets section - Currently has a circular flow for some postgres and opensearch secrets which may be unnecessary now. Will need further investigating and potentially streamlining and then the readme here should be updated accordingly


Configure Web

What is Schedule ，Repeat every and  job data (json)？
How to configure? What is the format?

See the Agenda documentation at https://github.com/rschmukler/agenda#creating-jobs


Async/sync tests should be distinguishable in TestExplorer

For every test I have in a track2 library, they show up twice (one for sync and one for async), but I cannot distinguish between the two. Maybe I'm missing something.

This does not seem to be an issue anymore. The tests are distinguishable based on the async parameter.


Fix build issue during past PR conflicts

Description
This PR is mainly to fix the build issue with the latest packages/sdk code with PR #16.
Additionally, ensured the seed phrase stored points to unique auto ID, which is not verified on-chain.

WARNING: When run in react native, observe if more time is required in generating seed phrase.

Just some fixes & a minor feature added.


pyodbc and "Always Encrypted" feature of MS SQL Server 2016 Enterprise

MS SQL Server 2016 Enterprise introduced "Always Encrypted" feature
https://msdn.microsoft.com/en-us/library/mt163865.aspx.
Is there a plan to support it in pyodbc?

Here's the ODBC specific background I looked at: https://msdn.microsoft.com/en-us/library/mt637351(v=sql.110).aspx
I was under the impression that it is possible to use it today as long as ColumnEncryption was enabled in the connection string and parameters are used for passing values, which is what pyodbc already does.  In other words, I was under the impression that the driver and server do most of the work.
I looked through the new constants and didn't see anything that would be useful to implement, but I haven't actually used AE yet.
Can you first try the existing pyodbc with the appropriate driver (>= 13.1) and see how it goes.  I believe it will work.

It does not work with driver 17. The values I get are encrypted and not sure how to decrypt or how to insert! Help is appreciated!!

You need to add ColumnEncryption=Enabled to the connection string. This will be sufficient for decrypting data upon retrieval, but for insertion you need to use parameters instead of literals and keep in mind the constraints. See the documentation for more information -> https://docs.microsoft.com/en-us/sql/connect/odbc/using-always-encrypted-with-the-odbc-driver

Okay, I got it. The permissions I mentioned above are enabled by default.  We need to pass the following additional parameters in the connection string:
KeyStoreAuthentication=KeyVaultClientSecret
KeyStorePrincipalId={client_id}
KeyStoreSecret={client_secret}

Yes, if you are using AKV for your keys then you need to supply the credentials for it too.
To insert, you need to use parameters. That linked document has a list of limitations you should keep in mind when using AE.


Websockets issue with lists

So, while using lists, I cannot do .send(message) on a name, and using sets are alright, but I cannot lookup for a specific client for a game if I want to announce that they are the winner. So I've been using both, one to properly track people and the other to be used to send messages to clients.
Is there a method I can use to e.send to a specific client without dealing with 'str' object has no attribute 'send'?
PS. Is there a limit to the amount of players that can join?

Maybe post a short example what isn't working for you. Your text is very confusing.

@lgrahl Updated, thanks!


Enable batch participants to sponsor the batch by renewing their account

Bring back the sponsor a batch feature.
Accounts participating in a batch will be able to sponsor a batch by renewing their account. That applies to all the accounts, included the ones with the autorenew batch version, as long as the batch confirmation does not trigger an expiry extension.
Added some tests to the RenewAccount RPC endpoint. I created a couple of interfaces (Account and Order Managers) to decouple a bit the logic and use mocks for the tests. Same applies to the account rpc marshaler.

@bhandras: review reminder
@guggero: review reminder

!lightninglabs-deploy mute 2022-Feb-01


🌱 runtime-sdk: improve unit testing for runtime client and controller

Introduces unit-tests for internal/runtime/client.go
prevents panic for internal/runtime/client.go
improve unit-tests for exp/runtime/internal/controllers to use httptest

Perfect, Thx!


Always Get Error When Using Newest JQuery

I don't know why, it's always got error when I use Jquery 2.x, or 3.x

https://github.com/iprodev/iView/pull/16


Testdriving pt2

Hey,
thanks for the first batch of fixes, it's shaping up nicely :)
Here are some smaller things?

If one specifies item, the handler function is called directly, even if the entry also has children. Not sure it's intended, but should probably be documented (otoh, maybe you want to unify that, e.g. have an entry "result" that either is a string for calling the handler function with, or a table that represents the children).

I was happy to see that leaving out the question key for ask results in a menu that has no header and no header separator line, winning me 2 lines of screen estate. Unfortunately, the submenues all have a header + separator. I'd appreciate a way to remove that, too, my menues are quite big, but don't need a header, so I'd be quite fond of 2 additional lines for the menu.

Regarding 2., the submenus use the table key as header, even when there's a description. I don't mind, since I want to get rid of the header + separator alltogether, but it would make sense to use the description if it's available.

Fully unrelated: Would you want PRs? I feel like I should contribute :) I don't have anything substantial, but being a user of this lib I could maybe put things in the docs that seem helpfull.

Thanks for testing and coming up with enhancements!
Let's go:

Even though it's unintentional, I'm divided. Should the handler function deal with parents (and return false so the prompt remains open) or only deal with the leafs?
that might be configurable I think, allowing both scenarios but defaulting to the latter.

That's because of the breadcrumbs. I think it makes a lot of sense to not display it if there's no question/message on the prompt.

That was a poor choice thinking now. Using the description sounds more reasonable.

I'd love to get some PRs. I'm focusing on making this as friendly as possible for developers to use and extend impromptu, so feel free to add more docs. My goal is to hook luadoc with readthedocs.io altogether, so public functions are always documented.

More on this: 2 and 3 are fixed in the next fixup branch.
1 is actually a complicated design issue. I'd rather leave it like that and restrict having item and children set at the same time. It makes hard navigating the tree and I don't want to make this overly complex.
So I think I should simplify the way options are provided by:

Never taking items actually. They should be the key to that option;
Only allowing map-style declarations ({item =  <option-table> }), since that'd make navigation easier.


docs: Make it clearer that --locale uses #split

This PR tries to clarify that the option for locales should be used like:
wti push -l 'en sv'

Yes, absolutely. Thanks for the PR!


Faster and more robust feed type determination

Thanks for FeedKit! I'm using it in a podcasting app and it's been working like a champ. I have made a few of small tweaks that I'll upload as PRs. There's isn't really a general theme except perhaps that several of them have to do with Postel's law - they are adjustments to make out-of-compliance but sensible feeds less likely to fail parsing.
This first patch changes the implementation of XML vs. JSON determination, for a couple of reasons.
First, XML needn't be in UTF-8 encoding, and I have seen cases where String(data:encoding:) actually fails because of this. String(decoding:as:) is a more lenient conversion and in this context it doesn't matter if odd characters are not correctly converted.
Second, there's no reason to convert the entire data block. Profiling was showing that this could be quite expensive with large feeds. If the feed doesn't show up within 200 characters, there's something very strange going on.
Finally, instead of just trimming whitespace, this patch just defines clear sets of accept/reject/don't-care characters and acts on the first accept or reject character in the buffer.

Hi @GarthSnyder, what a great contribution. Love this PR, can't wait to merge. I've seen all other PRs too, awesome work! I'll merge them in the next couple of days and release a new version ASAP.
Thank you 🙏

You're a true politician, Nuno. :) Thanks, I appreciate the enthusiasm - hope the patches are useful.


Add required kwarg

I had trouble following the tutorial, as I was getting errors until I added the to= in front of the related model.

Hey @brylie, thanks for reporting!
I'm pretty certian that this can be passed through as a positional argument as well. Please can you paste the error you were getting here?

ParentalKey should indeed support passing the model name as a positional argument, and that's the version we'd want to encourage in the documentation, as it's slightly more concise and matches the version normally used for Django's ForeignKey. If you've found a place where it's not working as it should, please share more details of the error and we'll be happy to investigate further.