weed mount with TLS

Describe the bug
seaweed mount doesn't work when gRPC is secured via TLS. (i.e. works if I comment out the grpc lines in security.toml, leaving disableHttp options and jwt key, but breaks if those lines are left in). This may be a misunderstanding of what does/does not work right now on my part, but I interpreted the security documentation as at least implying this would work (and everything except for mount does).
System Setup
weed -logdir=/var/log/seaweed/master master -mdir=/var/lib/seaweedfs -ip=<IP_ADDRESS> -disableHttp
weed -logdir=/var/log/seaweed/volume volume -port=9380 -dir=/mnt/seaweed000/data -max=50 -ip=<IP_ADDRESS> -mserver=<IP_ADDRESS>:9333 -dataCenter=Room -rack=Rack1
weed -logdir=/var/log/seaweed/filer filer -ip=<IP_ADDRESS> -port=8888 -master=<IP_ADDRESS>:9333 -disableHttp
weed -logdir=/var/log/seaweed/mount mount -filer=<IP_ADDRESS>:8888 -dir=/mnt/seaweed

OS = Ubuntu 20.04 LTS (very fresh install - only setup done prior to this was Cassandra, for the db)
Weed = version 30GB 2.18 d2ead72 linux amd64
(initially tried 2.16 - same issue)
filer.toml:
[filer.options]
recursive_delete = false

[cassandra]
enabled = true
keyspace="seaweed"
hosts=["localhost:9042",]
username="seaweed"
password="***"

security.toml
[jwt.signing]
key = "***"

[grpc]
ca = "/etc/seaweedfs/out/hex.crt"

[grpc.volume]
cert = "/etc/seaweedfs/out/volume.crt"
key  = "/etc/seaweedfs/out/volume.key"

[grpc.master]
cert = "/etc/seaweedfs/out/master.crt"
key  = "/etc/seaweedfs/out/master.key"

[grpc.filer]
cert = "/etc/seaweedfs/out/filer.crt"
key  = "/etc/seaweedfs/out/filer.key"

[grpc.client]
cert = "/etc/seaweedfs/out/client.crt"
key  = "/etc/seaweedfs/out/client.key"

with above certificates created by following instructions in wiki.
master log:
Log file created at: 2020/12/29 11:56:50
Running on machine: jeannie
Binary: Built with gc go1.15.6 for linux/amd64
Log line format: [IWEF]mmdd hh:mm:ss threadid file:line] msg
I1229 11:56:50 03374 file_util.go:23] Folder /var/lib/seaweedfs Permission: -rwxr-xr-x
I1229 11:56:50 03374 master.go:168] current: <IP_ADDRESS>:9333 peers:
I1229 11:56:50 03374 master_server.go:107] Volume Size Limit is 30000 MB
I1229 11:56:50 03374 master_server.go:192] adminScripts:
I1229 11:56:50 03374 master.go:122] Start Seaweed Master 30GB 2.18 d2ead72 at <IP_ADDRESS>:9333
I1229 11:56:50 03374 raft_server.go:70] Starting RaftServer with <IP_ADDRESS>:9333
I1229 11:56:50 03374 raft_server.go:129] current cluster leader:
I1229 11:56:50 03374 master.go:146] Start Seaweed Master 30GB 2.18 d2ead72 grpc server at <IP_ADDRESS>:19333
I1229 11:56:52 03374 masterclient.go:78] No existing leader found!
I1229 11:56:52 03374 raft_server.go:154] Initializing new cluster
I1229 11:56:52 03374 master_server.go:141] leader change event:  => <IP_ADDRESS>:9333
I1229 11:56:52 03374 master_server.go:143] [ <IP_ADDRESS>:9333 ] <IP_ADDRESS>:9333 becomes leader.
I1229 11:56:55 03374 masterclient.go:126] redirected to leader <IP_ADDRESS>:9333
I1229 11:56:55 03374 master_grpc_server.go:250] + client master@<IP_ADDRESS>:52234
I1229 11:59:08 03374 node.go:278] topo adds child Room
I1229 11:59:08 03374 node.go:278] topo:Room adds child Rack1
I1229 11:59:08 03374 node.go:278] topo:Room:Rack1 adds child <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:73] added volume server <IP_ADDRESS>:9380
I1229 11:59:08 03374 volume_layout.go:352] Volume 3 becomes writable
I1229 11:59:08 03374 volume_layout.go:352] Volume 4 becomes writable
I1229 11:59:08 03374 volume_layout.go:352] Volume 5 becomes writable
I1229 11:59:08 03374 volume_layout.go:352] Volume 2 becomes writable
I1229 11:59:08 03374 volume_layout.go:352] Volume 1 becomes writable
I1229 11:59:08 03374 volume_layout.go:352] Volume 6 becomes writable
I1229 11:59:08 03374 volume_layout.go:352] Volume 7 becomes writable
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 3 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 4 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 5 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 2 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 1 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 6 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:110] master see new volume 7 from <IP_ADDRESS>:9380
I1229 11:59:08 03374 master_grpc_server.go:155] master send to master@<IP_ADDRESS>:52234: url:"<IP_ADDRESS>:9380" public_url:"<IP_ADDRESS>:9380" new_vids:3 new_vids:4 new_vids:5 new_vids:2 new_vids:1 new_vids:6 new_vids:7 data_center:"Room"
I1229 12:01:02 03374 master_grpc_server.go:250] + client filer@<IP_ADDRESS>:8888

volume log:
Log file created at: 2020/12/29 11:59:08
Running on machine: jeannie
Binary: Built with gc go1.15.6 for linux/amd64
Log line format: [IWEF]mmdd hh:mm:ss threadid file:line] msg
I1229 11:59:08 03489 file_util.go:23] Folder /mnt/seaweed000/data Permission: -rwxr-x---
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/2.idx to memory
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/1.idx to memory
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/2.dat, replicaPlacement=000 v=3 size=8 ttl=
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/6.idx to memory
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/7.idx to memory
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/3.idx to memory
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/1.dat, replicaPlacement=000 v=3 size=232 ttl=
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/6.dat, replicaPlacement=000 v=3 size=384 ttl=
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/4.idx to memory
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/7.dat, replicaPlacement=000 v=3 size=784 ttl=
I1229 11:59:08 03489 volume_loading.go:123] loading index /mnt/seaweed000/data/5.idx to memory
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/3.dat, replicaPlacement=000 v=3 size=152 ttl=
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/4.dat, replicaPlacement=000 v=3 size=8 ttl=
I1229 11:59:08 03489 disk_location.go:131] data file /mnt/seaweed000/data/5.dat, replicaPlacement=000 v=3 size=576 ttl=
I1229 11:59:08 03489 disk_location.go:173] Store started on dir: /mnt/seaweed000/data with 7 volumes max 50
I1229 11:59:08 03489 disk_location.go:176] Store started on dir: /mnt/seaweed000/data with 0 ec shards
I1229 11:59:08 03489 volume_grpc_client_to_master.go:52] Volume server start with seed master nodes: [<IP_ADDRESS>:9333]
I1229 11:59:08 03489 volume.go:334] Start Seaweed volume server 30GB 2.18 d2ead72 at <IP_ADDRESS>:9380
I1229 11:59:08 03489 volume_grpc_client_to_master.go:114] Heartbeat to: <IP_ADDRESS>:9333

filer log:
Log file created at: 2020/12/29 12:01:02
Running on machine: jeannie
Binary: Built with gc go1.15.6 for linux/amd64
Log line format: [IWEF]mmdd hh:mm:ss threadid file:line] msg
W1229 12:01:02 03561 filer_server.go:114] skipping default store dir in ./filerldb2
I1229 12:01:02 03561 filer.go:101] existing filer.store.id = 921739869
I1229 12:01:02 03561 configuration.go:28] configured filer store to cassandra
I1229 12:01:02 03561 filer.go:173] Start Seaweed Filer 30GB 2.18 d2ead72 at <IP_ADDRESS>:8888
I1229 12:01:03 03561 filer_grpc_server_sub_meta.go:182] + listener filer:<IP_ADDRESS>:8888@<IP_ADDRESS>:49218
I1229 12:01:03 03561 filer_grpc_server_sub_meta.go:70]  filer:<IP_ADDRESS>:8888@<IP_ADDRESS>:49218 local subscribe / from 2020-12-29 12:00:02.42229329 +0000 UTC
I1229 12:01:03 03561 filer_grpc_server_sub_meta.go:85] after local log reads, filer:<IP_ADDRESS>:8888@<IP_ADDRESS>:49218 local subscribe / from 2020-12-29 12:00:02.42229329 +0000 UTC

Mount doesn't output anything at all (doesn't even create a log file) or result in a mount point appearing. Afraid I have no experience of go or debugging it, but strace shows all but two of the many threads are just sitting there, waiting for a mutex (worker threads?), while one thread seems to be waiting for a mutex, timing out, sleeping and then waiting again (main thread?) and finally another thread is polling the 4 sockets it has opened, timing out, then polling again and so on (networking thread?). Which is to say at the system level it looks like about what you would expect!
Expected behaviour
To mount! They key detail I note is that commenting out all of the [grpc] blocks in security.toml results in it working. Also, with certificates weed shell works fine, and if I enable the web interface of filer then I can upload/download files as expected - enabling certificates seems to only affect mount.
Even if this is an expected failure mode, it would still be very helpful for something to be outputted to the log, such that I can see what has gone wrong.
Additional context
I am doing this "properly", as in putting weed at /usr/local/bin/weed, setting everything up as services with systemctl and running all the processes as a user I've created called seaweed, except for the mount process which is root because of the shared access/fuse issue. Mentioning in case it's getting stuck trying to write a log file somewhere it has no permission to do so! It does have permission to write in it's current working directory when started (I've used /var/lib/seaweedfs for all of them; no idea if that is wise but did try a different directory for mount just incase). Have triple checked permissions on everything and all looks correct.

Thanks for the detailed issue content!
I can reproduce this and found it was caused by a regression. Added the fix.

Thank you for the speedy response - that's brilliant!


Support annotations

This project could really benefit from incorporating - or at least absorbing as a module - Super CSV Annotations.
https://github.com/mygreen/super-csv-annotation
This is supremely useful for reducing code complexity.  However, the biggest key missing feature I've found so far is that there is no current annotation support for custom cell processors.

I've used https://github.com/FasterXML/jackson-dataformat-csv for this before and I agree it's quite nice to have a CSV-specific DTO with annotations.
Some thoughts:

You wouldn't be able to use deep/indexed mapping so this would be limited to CsvBeanReader and CsvBeanWriter
The header could be supplied or as an annotation
The cell processor (which could be a chain of cell processors) for each column would be annotated on the relevant field (I'm picturing something like @CellProcessor(new Optional(new FmtDate()) to keep things simple, instead of specific annotations like the super-csv-annotation project).

Despite the limitations/problems @jamesbassett  mentions, I think it'd be cool if someone makes a PR for this.

Yeah I agree - and that's not a list of problems, more of an idea of how it could be implemented :)

I was thinking how to create a generic annotation that would allow to use any CellProcessor including custom or chained ones. The problem is that CellProcessor object cannot be an annotation member.
This is what I came up with.

For really simple cases:

@CellProcessor(NotNull.class)

For cell processors with arguments (including chained processors).

@CellProcessor(value=Optional.class, args={@CellProcessor(value=FmtDate.class, args ={“DD-MM-YYYY”})})
Limitation: will not be possible to use something other than allowed annotation member types or other cell processors as cell processor constructor arguments.

For use with any processors, but not typesafe:
@CellProcessor(cellProcString="new Optional(new FmtDate(\”DD-MM-YYYY\”))")

CellProcessor will be created by building a String containing its source code (a wrapper around a cell processor which is defined in cellProcString) and compiling it into actual class using the JavaCompiler class.
In case of custom/third-party cell processors one will need to supply package name:
@CellProcessor(cellProcString=”new MyCustomProcessor(new AnotherProcessor())”, customProcessorPackages={“org.mycompanyname.myprocessors”, “org.othercompany.otherprocessors”})
Does that sound like a viable idea?
Suggestions?

Hi @singularityfx
Thanks for your thoughts. There is something inside of me that feels wrong when I read your suggestion. I have always been of the impression that annotations were short and declarative rather than imperative.
while your approach is more general. it does entail dumping java into strings with no compiler to support the user.
Personally, I think we get the most millage from doing bespoke annotations to the current set of processors - despite the limitations it imposes.
Any oppinions or objections? @jamesbassett  ?  @ZioberMichal  ?

Hi @singularityfx , @kbilsted,
I agree with @kbilsted that we should not go in direction where at the end we have to parse String to "Java code".
Names
I think that CellProcessor is a good name but it is already used and we can not reuse it because it will be confusing to have interface and annotation with the same name.
Concept and implementation
Option 1) and 2) look good and they should cover most of cases. At the beginning, I think, we should put our effort to implement annotations for CellProcessors. Things related with configuration we can skip and I think they are not so important.
I propose to create two simple annotations:
public @interface Processor {

    Class<? extends CellProcessor> value();

    String[] args() default {}; //??? It will be tricky to instantiate a class.
}

and
public @interface Processors {

    Processor[] value();
}

according to Repeating Annotations article.
I prefer repeating annotation because they are "eye-friendly" and for long chain of processors it will be easier to register all of them. Chain order: 0-processor invokes 1-processor which invokes ... i-processor which ..... (0, 1, ..., i, ... - indexes of processors in array).
The biggest problem with implementation which I see it is creating instances of given CellProcessor. Regarding to args attribute we should be able to invoke proper constructor and try to parse arguments for it.
What if, for example, given CellProcessor has more than one constructor with the same number of parameters? Should we try first and if it will fail we can try invoke another?
We can add additional attribute:
    Class<?>[] argsClasses() default {};

and use it when we have more than one constructor with the same number of parameters but this solution obscures annotation declaration.
What do you think guys?

Repeating annotations are Java8 only. Is it a good idea to drop backward compatibility?

Oh, sorry. I meant to use only one part from this page "Step 2: Declare the Containing Annotation Type" to show my point of view. Sorry for misunderstanding.
I am thinking about something like this:

public class Bean {

    @Processors({@Processor(Trim.class), @Processor(ParseDate.class)})
    private Date birthDay;

    ....
}

It must be possible in the current version which we are using.


Trips

Updated. Add/delete trips from myTrips with their id's. When you get trips, it will return the entire trip object now.

:+1:
Quick question: what is the expected format/structure/schema is the body (updateMyTrips) supposed to have?

This should be more straightforward. To add trip to myTrips, call post, accounts/:id/mytrips with body format: { "myTrips": id }. Very similar for delete

@addisonnick Are we good to go? If so, let's :shipit:!


fix: plugins/permissions: channels/forums/createForumPosts

Check #2219 first!

Please resolve the merge conflicts

@itohatweb I have resolved the merge conflicts!


Release v0.10.0

Merge develop to master, creating of the new release after this merge.

Codecov Report

Merging #288 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #288   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           1      1
=====================================
  Hits            1      1

Continue to review full report at Codecov.

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


Outdated ndarray

The ndarray crate has been bumped to 0.16, while this crate still depends on 0.15. User may suffer from version conflict when using this crate and another ndarray related crate simultaneously.

Is it possible to update ndarray dependancy to 0.16. My crates are blocked to 0.15

Is someone working on this?


Token overcounting?

I'm a bit puzzled by how Helicone counts tokens for streaming responses.
I wanted to summarize a Python file, so I ran it through tiktoken to get a sense of how many tokens it would have used -- got a result of about 4000 tokens. I then sent the request to OpenAI with a very short question at the end of the prompt (something like "Explain this script to a total newcomer to the project"), and got a relatively short answer.
When I checked the request details on the Helicone dashboard, it listed almost 8000 tokens for the prompt, which seems like a pretty extreme difference, even considering all the hidden message tokens and the system prompt. The response was estimated at 304 tokens, and I get 285 when running its plain text through tiktoken, which seems a bit more in line.
Any idea why the difference would be so large?
EDIT: I also just realized that the base gpt-4 has an 8k context, so if the whole prompt had really been 8000+ tokens I should have gotten a cut-off response, right? I think that further points to some overcounting on Helicone's part.

Hi @iamjackg sorry for the incredibly late response here. If you are using ChatGPT there are hidden tokens for each system and assistant message. Helicone accounts for these hidden tokens.
Hopefully this answers your questions..
The 8k context window is the input window, there is no technical max on the response tokens

If you read my original submission you'll see that I mentioned the extra tokens for message delimiting, but they can't possibly account for the doubling of the total token count I'm seeing.

Thanks @iamjackg for your response - reopening the issue and will take a look


Button improvements

@magicae How can I add you to the pull request?

merge to your own repo.
I'll update it later.


When the menu is hidden by default, toggling should be temporary, and dismissed by non-menu actions

VSCode Version: 1.10.2
OS Version: Windows 10

Steps to Reproduce:

Set the menu to be hidden by default
Press ALT to display the menu
Change your mind about needing the menu and continue typing in the editor instead

Issues:

The menu doesn't go away unless ALT is pressed a second time
Hotkey mnemonics continue to be displayed

If the menu is hidden by default, displaying it should be treated as a transient interaction. If the next interaction does not involve the menu, the menu should be re-hidden automatically.

We will change to custom menus on Windows. For now you can set "window.titleBarStyle": "custom".


updated voting

Description
Fixes # (issue)
fixed voting updates dynamically, also implemented voting on the guestview page
Type of change
Please delete options that are not relevant.

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

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

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

need to fix an issue


simplyfing npm run tasks

👋
I saw the command line was getting too convoluted and repetitive due to babel path (and option) so I would like to suggest simplifying the command line a bit.
I think this would be helpful for further understanding of what files are being transformed in every command.
If adding sourceMaps or other available  flags to babel were needed, it would be needed only in the build line.
I didn't do it in every package.json but I would be happy to do it  If you like this change.
thanks!

Nice trick. I included it in ch12 in npm scripts sidenote. Thanks!


[14.0.x] ISPN-14008 indexing annotations, query updates

backports #10785
https://issues.redhat.com/browse/ISPN-14008

LGTM. Merged. Thank you!


Mailer tests in the request tests were removed on Rails 6 upgrade

Remove mailer tests in the request tests. This is because sidekiq is overridden in request tests by the test adapter (in Rails 6) and thus sidekiq mailer tests in the requests were faulting "rails/rails #37270" and the solution is to continue using the test adapter as the mailers are tested seperately.
Rails 6 upgrade PR

This is less of an issue but something to watch for when it's fixed so that we can then reverse that change. It'll make the request tests singing emails testing a little bit more robust as the mailers are already tested in mailer tests.

Email functionality was refactored in AP-2011 and test re-written then, so I'll close this issue as it's no longer relevant.


You should enalbe Discusion here on Github

Instead of people asking random questions on issues they could use discussion section here with GitHub.

Thank you so much mylastore for information about the great github feature. Discussion is now enabled.


How to convert the live preview image into Numpy array in real-time

I am trying to convert the live preview image into Numpy array. However, the function get_raw_pixels  was quite slow, it would cost more than 4 seconds to exact 2160 x 4096 x 4  pixels. There should be some way to do it much faster since the preview shows the FPS is 17. Any alternative way to achieve this? I do not have too many experiences in using Micro-Manager. Any help would be grateful.
My code was as follow:
import numpy as np
from pycromanager import Bridge
import time

bridge = Bridge()
studio = bridge.get_studio()
snap_list = live_manager.snap(False)
img = snap_list.get(0)

start_time = time.time()
raw_pixels = img.get_raw_pixels()
print("--- time_profile ", "get raw pixels", " %.4f seconds ---" % (time.time() - start_time))

--- time_profile get raw pixels  4.4569 seconds ---

Same issue #90, I think. I looked into it a bit and it seems that there is a major slowdown caused by the library that pycromanager is using to serialize data to transmit it between Java and Python. Should be fixable, just need to swap in an alternative library. Until that happens, any alternative way of trying this will likely run into the same problem

Should be fixed with a 10-100x speedup in tomorrows nightly build

I'm guessing this is fixed for you now? Reopen if you run into something else


Running with Odyssey?

Hey, great work!
Nice feature for grinding mats ;)
Is it working with Odyssey as well?
Thank you :)

Closing because of no feedback. Feel free to reopen it if you are still having issues.


Do not accepting ScrollView in second component

<ScrollableTabView tabBarUnderlineColor="#ED9A11" tabBarActiveTextColor="#ED9A11" tabBarInActiveTextColor="#000"> 		<Projects tabLabel="Projects" navigator={this.props.navigator} token={this.props.token} /> 		<KPIMetrics tabLabel="KPI Metrics" navigator={this.props.navigator} token={this.props.token} /> 	  </ScrollableTabView>
I am displaying listview in both components
So I need to put ScrollView(with RefreshControl) in first component and it worked..
but when putting the same in second one gives me error..

Probably problem in your component. Anyway I can't help you without Projects and KPIMetrics components.

reopen if you still think that problem in react-native-scrollable-tab-view


Apple  Music connection to FCP is atrocious

Apple Feedback Assistant ID: MISSING
DESCRIBE THE BUG:
Apple Music implementation is a joke; you can't drag and drop anymore straight from music. Search in FCP is not working most of the time; I can't sort by the same columns as in Music. Once upon a time, under Steve Jobs Apple placed a large focus on usability and ensure it that functionality is system and application white if appropriate for instance, drag and drop between Apple applications was a no-brainer. Now I need to spend 20 minutes to simply access my Music in FCP and the only way is to export from music and re-import in FCP because often music which is clearly in my library cannot be found in Final Cut Pro
TO REPRODUCE:
xxx

EXPECTED BEHAVIOUR:
Have a seamless integration between music and Final Cut Pro being able to search for tracks and sort the columns the same way as I could in music
SCREENSHOTS:
(https://drive.google.com/file/d/15MlpD1i8qd3beEaAWVt4EiBoTuXUyuse/view?usp=sharing)

SPECS:

2021 16-inch MacBook Pro (M1 Max, 64GB RAM, 8TB SSD)
macOS Sonoma 14.1.2
Final Cut Pro 10.7.1

ADDITIONAL COMMENTS:
xxx

Not to mention the connection between FCP and music is not live if any changes are made to Music i.e. new music imported you need to shut down and restart FCP. It's like in the middle ages lol


mvnd overwrites PATH for vanilla mvn

I use sdkman and both maven (3.9.0) and mvnd (0.8.2, 0.9.0) are installed.
If mvnd 0.9.0 is set as default version, the command "mvn --version" shows the version 3.8.7.
If mvnd 0.8.2 is set as default version, the command "mvn --version" shows the version 3.9.0 (as expected).
The PATH variable looks like:
/Users/henning/.sdkman/candidates/mvnd/current/bin:/Users/henning/.sdkman/candidates/maven/current/bin:...
Since mvnd 0.9.0 the bin-directory contains a 'mvn', which is the first one the shell found. So the 'mvn' from the maven bin-directory is not used.
Is this an issue for mvnd or sdkman? I feel like ordering should not matter for PATH variables.

@gnodet We are receiving complaints from our SDKMAN users that the mvn command is being clobbered by mvnd. Is there any chance that this change can be reverted?

Similar issue happening on homebrew side as well.
Not a PATH conflict but symlink problem.
Homebrew installed 'maven' formula creates symlink for mvn and mvnd tries to overwrite it and fail.

That's a good question. I think mvnd should become just another maven installation, rather than a separate project.  So that people can switch between stock mvn or mvnd, just like they do between graalvm / temurin or another JDK.
Would that make sense ?

As an sdk user, I for one prefer the previous (0.8.2) situation: maven package provides the mvn binary, and mvnd package provides the mvnd binary. I wish for both to be the latest version promoted by their respective publishers (which might be different versions of the Maven library, as they are as of this writing). If I want to use mvnd, I will run mvnd. If I want to use stock Maven, I will run mvn. They are interchangeable in many, but not all, situations, and I certainly do not want to be flipping some global configuration back and forth between shell commands.

As an sdk user, I for one prefer the previous (0.8.2) situation: maven package provides the mvn binary, and mvnd package provides the mvnd binary. I wish for both to be the latest version promoted by their respective publishers (which might be different versions of the Maven library, as they are as of this writing). If I want to use mvnd, I will run mvnd. If I want to use stock Maven, I will run mvn. They are interchangeable in many, but not all, situations, and I certainly do not want to be flipping some global configuration back and forth between shell commands.

I agree that mvnd and mvn do behave slightly differently in some cases, however, the mvn script that now comes with mvnd >= 0.9.0 does not use the daemon and it should simply boot maven, so there should be no differences really.  Have you hit any problem ?

@gnodet the problem isn't really with the behaviour of your script, but rather with the fact that your mvn script clobbers the real mvn provided by Maven. There can only be one mvn on the path.

@gnodet the problem isn't really with the behaviour of your script, but rather with the fact that your mvn script clobbers the real mvn provided by Maven. There can only be one mvn on the path.

Yes, I understood that.  That's why I think mvnd may be better advertised as a maven distribution instead of being advertised as a separate project.  Kinda like graalvm is considered a java flavour and not a separate project.

That's why I think mvnd may be better advertised as a maven distribution instead of being advertised as a separate project.

While this is an approach that may work with SDKMAN! (but @marc0der should know much better than I do), I am not sure other package managers (such as Homebrew) support a similar approach.

Not a homebrew expert, but mvnd formula probably need to declare that it conflicts with maven formula or declare it as keg_only so that it does not try to symlink (=don't try to show up in PATH).
https://docs.brew.sh/Formula-Cookbook#specifying-conflicts-with-other-formulae
What's the benefit of mvnd taking over mvn as a command?
Suggesting people to define an alias to those who want to use mvnd as drop in replacement of mvn not an option?

FWIW, I was hit by #795 because of this overlap/overwrite.
Removed all mvnd sdkman packages for now (which oddly leaves behind non-existing /home/fmo/.sdkman/candidates/mvnd/current/bin in PATH).

@famod you will need to open a new terminal to clean up the residual state. Let us know if you still see this entry in your path after doing that.

@gnodet any update on this yet? I don't think we can advertise mvnd as a Maven distribution because it isn't multiplatform like mvnd. Because this is still causing issues for our users, I'd prefer if we remove 0.9.0 from SDKMAN until this is resolved.

@famod you will need to open a new terminal to clean up the residual state. Let us know if you still see this entry in your path after doing that.

Ah, thanks. I was merely opening a new tab in Ubuntu console, which obviously didn't suffice.

Ah, thanks. I was merely opening a new tab in Ubuntu console, which obviously didn't suffice.

@famod It might be that you are using a login shell, which means that this problem might persist until you log out and back in again.

Ah, thanks. I was merely opening a new tab in Ubuntu console, which obviously didn't suffice.

@famod It might be that you are using a login shell, which means that this problem might persist until you log out and back in again.

@marc0der all is well with the PATH cleansing; just opening a new tab is not enough but opening an entirely new Terminal window works. I don't have to log out/in.

I personally solved this (hacked?) by removing exec bits from ~/.sdkman/candidates/mvnd/0.9.0/bin/mvn

@marc0der I'm fine with removing 0.9.0 for now or any other quick fix. Not sure how to do that though.
Though I don't understand how the fact that mvnd is multiplatform is relevant.  It also works on platforms where the native binary is not available, so this should not be a problem.

I'm fine with removing 0.9.0 for now or any other quick fix. Not sure how to do that though.

Great, you can remove any version using our API, it has a DELETE method. If you struggle I can help. Also, you would need to make 0.8.9 the new candidate default.

Though I don't understand how the fact that mvnd is multiplatform is relevant.

This is very relevant as SDKMAN classifies this at a candidate level. Besides the fact that it simply isn't possible, I also don't feel comfortable adding mvnd to the Maven candidate.

I'm fine with removing 0.9.0 for now or any other quick fix. Not sure how to do that though.

Great, you can remove any version using our API, it has a DELETE method. If you struggle I can help. Also, you would need to make 0.8.9 the new candidate default.

> for PLATFORM in LINUX_64 MAC_OSX MAC_ARM64 WINDOWS_64
  do curl -s -X DELETE \
    -H "Consumer-Key: ${SDKMAN_CONSUMER_KEY}" \
    -H "Consumer-Token: ${SDKMAN_CONSUMER_TOKEN}" \
    -H "Content-Type: application/json" \
    -H "Accept: application/json" \
    -d '{"candidate": "mvnd", "version": "0.9.0", "platform": "'${PLATFORM}'"}' \
   https://vendors.sdkman.io/release
done
{"message":"Conflict: mvnd 0.9.0 LINUX_64","status":409}{"message":"Conflict: mvnd 0.9.0 MAC_OSX","status":409}{"message":"Conflict: mvnd 0.9.0 MAC_ARM64","status":409}{"message":"Conflict: mvnd 0.9.0 WINDOWS_64","status":409}

Though I don't understand how the fact that mvnd is multiplatform is relevant.

This is very relevant as SDKMAN classifies this at a candidate level. Besides the fact that it simply isn't possible, I also don't feel comfortable adding mvnd to the Maven candidate.

I've been testing that locally (by using sdk install maven 0.9.0 /path/to/mvnd-0.9.0 and similar other calls, and this seems to work fine enough for me.   I'll experiment a bit more.  I'm actually working on modifying mvnd to support maven 3.9.x and 4.0.x with two different distributions generated from the same source tree (i.e. a single maven build / release would produce 2 distributions for a given platform, one embedding maven 3.9.x and another one embedding 4.0.x).
For the platform, I think it would make sense to publish a UNIVERSAL distribution containing just the mvnd.sh script as indicated by https://sdkman.io/vendors#endpoints in addition to the distributions containing the native mvnd binary.

@gnodet I'd prefer it if we do not follow this approach. Having multiple packages for mvnd will be confusing for SDKMAN users.
I'd like to keep things as they are (one native package per version per platform). I also do not want the mvnd package to clobber the Maven binary.

@gnodet I'd prefer it if we do not follow this approach on SDKMAN. Having multiple packages for mvnd will be confusing for our users.
I'd like us to keep things as they are (one native package per version per platform). I also do not want the mvnd package to clobber the Maven binary.

@marc0der So the next mvnd release will support both maven 3.9.x and 4.0.x, but it will provide 2 different distributions for each platform (not a single distribution supporting two different maven versions).  The reason is that most of the code is common, so it's easier to maintain two distributions from a single source tree, rather than having to port any change to 2 different branches. See https://github.com/apache/maven-mvnd/actions/runs/4374385157.
I suppose those will have to be published as mvnd version = 1.0.0-m5-m39 and 1.0.0-m5-m40
Was your objection was related the above point ? Else is there another way that mvnd publish a distribution for non natively supported platforms ?  Reading the doc, I would think publishing a UNIVERSAL flavour would work, but you'll know better...
On the mvnd / mvn binaries mix, I'll revert the change that mixes up mvnd and mvn binaries, though I may come back to it in a few months when things have matured a bit more.


Initial policy code.

Description
Adds the first policy code to validate the shareProcessNamespace pod configuration.
Fix https://github.com/kubewarden/kubewarden-controller/issues/598
Test
make test e2e-tests

@kubewarden/kubewarden-developers just in case you've started the review. I've just updated this PR updating the owner name and email address, as well as removing some dependencies from the Cargo file.


unnit test for adding a new line at the end of source files

Signed-off-by: Pratik Dey<EMAIL_ADDRESS>A fix for the issue #999

@pdxjohnny I used the recursive approach to enumerate all files and the directories .

Hi @pdxjohnny thanks for the review ,I will be making the suggested changes very soon
Thanks 😊

Hi @pdxjohnny, Sorry for the delay I made a separate PR for this, I am closing this PR as there is a small typo in the branch name
New PR is #1015 .


$currencyManager being accessed before initialization.

So I obviously added BedrockEconomy to my server.  The plugin is great and works exactly how it should, until the server is reset.  It doesn't always happen, but sometimes when the server is shutdown I get an error.  It usually is fixed if I delete the plugin data but keep the .phar, and then try to run the server again.  However, as my server begins to get members it is becoming a problem because I can't delete the data anymore without resetting the balance of players.
Here is my error:
[Server thread/CRITICAL]: Error: "Typed property cooldogedev\BedrockEconomy\BedrockEconomy::$currencyManager must not be accessed before initialization" (EXCEPTION) in "plugins/BedrockEconomy (1).phar/src/cooldogedev/BedrockEconomy/BedrockEconomy" at line 131

I need the full exception including the trace


[Event Change] Codeunit 99000832 "Sales Line-Reserve", OnBeforeVerifyQuantity

Hi Team,
could you please add parameter OldSalesLine to publisher OnBeforeVerifyQuantity in codeunit Sales Line-Reserve
Current definition:
[IntegrationEvent(false, false)]
local procedure OnBeforeVerifyQuantity(var NewSalesLine: Record "Sales Line"; var IsHandled: Boolean)
begin
end;

Requested definition:
[IntegrationEvent(false, false)]
local procedure OnBeforeVerifyQuantity(var NewSalesLine: Record "Sales Line"; var OldSalesLine: Record "Sales Line"; var IsHandled: Boolean)
begin
end;

Publisher is triggered in procedure VerifyQuantity
procedure VerifyQuantity(var NewSalesLine: Record "Sales Line"; var OldSalesLine: Record "Sales Line")
    var
        SalesLine: Record "Sales Line";
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeVerifyQuantity(NewSalesLine, IsHandled);
        // OnBeforeVerifyQuantity(NewSalesLine, OldSalesLine, IsHandled); // Requested definition
        if IsHandled then
            exit;
...

Thanks in advance,
Christian

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


Added verified publisher metadata

Added verified publisher metadata for ArtifactHub

@cpanato you should have an invite to join the org

yep, received, thanks!


Yikes! The Kusto function URL returns a 404

The URL for the Kusto function "https://aka.ms/sentinel-ciscoise-parser" returns a 404 error. Please advise. Thanks!

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e5c4d13c-831f-023c-980f-c271edfab93f
Version Independent ID: 6424cca0-c370-08d1-4aea-21024fbc885f
Content: Cisco Identity Services Engine connector for Microsoft Sentinel
Content Source: articles/sentinel/data-connectors/cisco-identity-services-engine.md
Service: microsoft-sentinel
GitHub Login: @cwatson-cat
Microsoft Alias: **cwats

@jpicart-atsg
Thanks for your feedback! We will investigate and update as appropriate.

@v-dvedak @prtanej

@cwatson-cat
Thanks for your inputs.
@jpicart-atsg
Thanks for pointing this out. Required changes have been made and visible in the document itself. I am going to close this thread now. Please feel free to reach out if you have any querys.


[WIP]optimize: optimized compatibility issues

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report
Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (c08345e) 52.35% compared to head (40dcdd8) 52.41%.
Report is 1 commits behind head on 2.x.

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6366      +/-   ##
============================================
+ Coverage     52.35%   52.41%   +0.06%
- Complexity     5214     5247      +33
============================================
  Files           920      924       +4
  Lines         31942    32071     +129
  Branches       3815     3835      +20
============================================
+ Hits          16723    16810      +87
- Misses        13570    13602      +32
- Partials       1649     1659      +10

Files
Coverage Δ

...ceptor/handler/AbstractProxyInvocationHandler.java
50.00% <100.00%> (+12.50%)
:arrow_up:

...r/parser/GlobalTransactionalInterceptorParser.java
30.55% <100.00%> (+1.98%)
:arrow_up:

...handler/GlobalTransactionalInterceptorHandler.java
29.68% <68.18%> (-0.84%)
:arrow_down:

... and 9 files with indirect coverage changes


Change from NavigtionView to BottomNavigationView

change id nav_view to bottom_nav_view
change NavigationView to BottomNavigationView

app:menu="@menu/nav_menu" should be app:menu="@menu/bottom_nav_menu" as well but I didn't change it


jquery: add second signature for triggerHandler.

This one takes an event instead of an event type. See documentation
(http://api.jquery.com/triggerHandler/) and related documentation
ticket (https://github.com/jquery/api.jquery.com/issues/393).

@johnnyreilly

Great PR! Thanks @hesselink


feat(cli): include ldflag version in --version command

#199  👈
main

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

Join @jakim929 and the rest of your teammates on  Graphite


The property of Struct Record should add [] for the read operation

I just read the article https://docs.microsoft.com/en-us/dotnet/csharp/write-safe-efficient-code,
and checked. I see that when I use C # to write the following code:
    struct Bar
    {
        public int Item1 { get; set;}
        public int Item2 { get; set;}
    }

Actually Item1 is compiled as:
public int Item1
{
	[IsReadOnly]
	[CompilerGenerated]
	get
	{
		return <Item1>k__BackingField;
	}
	[CompilerGenerated]
	set
	{
		<Item1>k__BackingField = value;
	}
}

See, It  adds a [IsReadOnly] attribute in front. So, I think it can be recognized and optimized by the C # compiler when it is passed as a parameter.
But if I write F # code like this:
[<Struct>]
type Bar = {
    Value: int
    mutable Text: string
}

compiled as:
[CompilationMapping(SourceConstructFlags.Field, 1)]
public string Text
{
	get
	{
		return Text@;
	}
	set
	{
		Text@ = value;
	}
}
[CompilationMapping(SourceConstructFlags.Field, 0)]
public int Value
{
	get
	{
		return Value@;
	}
	set
	{
		Value@ = value;
	}
}

Neither Value nor Text adds the [IsReadOnly] attribute above get.
Now if I pass the record to a C # function, even if I only do a read operation, the ldobj instruction will be generated.
But C # struct will not.

Please raise this over at http://github.com/dotnet/fsharp/issues instead of this repository. As the Issue template says, this repository is for packaging only, and all suggestions/bugs/issues must be opened upstream.

Please raise this over at http://github.com/dotnet/fsharp/issues instead of this repository. As the Issue template says, this repository is for packaging only, and all suggestions/bugs/issues must be opened upstream.

OK


Inconsistency in height of cards

There is an inconsistency in cards height , which looks bad and unprofessional, I want to work on this issue, if you can assign it to me.

It is like this right now:

My improvisation:

Assign it to me, I can resolve this issue

@Hemant1101 do it


Remove http_request_get_headers() from Request

https://github.com/yiisoft/yii2/pull/13780#discussion_r125824228
We should remove support for PECL http_request_get_headers()  from Request.
https://github.com/m6w6/ext-http/blob/45fb8dff8bc2981151fbca023408c61acbba0f08/src/php_http_env.c#L36
It gets the data from $_SERVER which we also support so there's no real advantage to use this.

Done since PSR-7 request is now used.


Abstract PCS trait added

Was split into https://github.com/w3f/fflonk/pull/8 and https://github.com/w3f/fflonk/pull/10


Catch up fork with origin

This PR catches up fake-s3 to its origin branch. We needed to do this because the 2.3 AWS SDK and this fork were not meshing on multi-part uploads. A local JSON import that was larger than 5 MB was throwing odd errors. I was pretty sure this was the culprit after I found this commit.
This library now offers most everything that was added to the fork. The content-disposition metadata and our specific set of CORS headers were the big differences between our fork and origin.
We will need to update some of our private code by changing our handling of some AWS exceptions to NoSuchKey from NotFound. According to treverrowe in this issue, NoSuchKey is just a more distinct NotFound error that the new aws-sdk dependency now raises. We would push this to production by pushing this and the exception handling simultaneously.
We got a clean test run on circle. The id is 34717; I don't want to link it directly.
You're prime @fgarces! I assume you will want to take a look as well, @jturkel

@jfo84 can you tell me which classes you modified?

lib/fakes3/file_store.rb in the create_metadata method. Before the lib wasn't passing any metadata so we added it, but origin now has all metadata except the content-disposition stuff
lib/fakes3/s3_object.rb for the getter for the above
lib/fakes3/server.rb the difference from origin is the apply_cors_headers method, which they are now doing but not in exactly the same way. Line 68 is their headers but everything else is ours
That should be it

assert_equal(response.headers[:access_control_allow_origin],"http://localhost:3535") from the post_test.rb file is also ours

looks good to me :shipit:


Version 1.0 NER4OPT Library

This is version 1.0 of our library. Please review and let me know any changes you would appreciate. Will address them accordingly.

Great update over first version. Only minor comments
The only thing is LICENSE, that I am not sure..

Thanks. I updated as per your suggestions, removed data to avoid any license conflicts.


Charges: Replace Retired and RetiredDateTime with RetiredByChargeOperation

Description:
With this story we want to implement the data model improvement, suggested by @BjarkeMeier.
Acceptance Criteria:

RetiredByChargeOperation has replaced Retired and RetiredDateTime in tables; ChargePrices and ChargePeriodDetails in in the database
Code that handles persistence has been refactored to align with AC#1
Charges data model diagram has been updated and uploaded to the Charges repo.

What is RetiredByChargeOperation ? I am having trouble figuring it out from the description.

What is RetiredByChargeOperation ? I am having trouble figuring it out from the description.

I've tried to elaborate on this. Hope it helps.


fix: Google analytics tracking of gclid

1. Why is this change necessary?
Currently if the customer navigates to the next page and came through Google with a gclid set, this id is not saved in any way and thus the click cannot be assigned to the ad.
2. What does this change do, exactly?
Save the gclid and resubmit it for further page views.
3. Describe each step to reproduce the issue or behaviour.
-
4. Please link to the relevant issues (if any).
NEXT-18984
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-20946
Please use this issue to track the state of your pull request.

Thanks for your contribution @aragon999


refactor(ast_tools): handle paths as strings

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6928
#6927
#6926
#6925  👈
#6924
main

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

Join @overlookmotel and the rest of your teammates on  Graphite


lime rebuild extension-webm windows error

i type lime rebuild extension-webm windows and say this

help me pls

this is technically the wrong place to put this issue and extension-webm is discontinued

Closing because extension-webm is not a part of Lime.

that too actually


Add workflows

see https://github.com/PhilipMay/mltb/tree/master/.github/workflows

PYPI_TOKEN secret added

all wf added


Chat window not opening on iOS client side, but shows in admin dashboard

Hi there
here is my AppDelegate Code:
LiveChat.licenseId = "MY LICENSE ID"
LiveChat.delegate = self
But whenever I call the method .presentChat, nothing appears.
No error message, nothing. Any help ?
PS: The only log I get is this:
WebViewBridge unknown messageType: {     messageType = uiReady; }

Just follow the step mentioned in the README of this repo
class SceneDelegate: UIResponder, UIWindowSceneDelegate {
var window: UIWindow?
func scene(_ scene: UIScene, willConnectTo session: UISceneSession, options connectionOptions: UIScene.ConnectionOptions) {
    LiveChat.windowScene = (scene as? UIWindowScene)
}

}

Closing as already resolved.


perf: use vector for dispatcher deferred deletion

Using a linked list leads to repeated allocations for no reason.
A vector once large enough is stable and will not allocate anything.

@lyft/network-team


METRON-1736 Enhance Batch Profiler Integration Test [Feature Branch]

The integration test for the Batch Profiler should use the Profiler Client API and PROFILE_GET to validate the values that are produced.  This is more effective end-to-end validation that the Batch Profiler is working.
This is a pull request against the METRON-1699-create-batch-profiler feature branch.
This is dependent on the following PRs. By filtering on the last commit, this PR can be reviewed before the others are reviewed and merged.

[x] #1161

Pull Request Checklist

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?
[x] Have you included steps or a guide to how the change may be verified and tested manually?
[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
[x] Have you written or updated unit tests and or integration tests to verify your changes?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

I feel like this makes it easier to understand what this is testing.  Nice improvement!  +1

Thanks for the review.  This has been merged.


Specify jobs responses and queue behavior for MIS specification

Task from @Eugene-Yesakov

[x] Specify ALL error_types (validation_failed and so on)
[ ] Synchronize job responses with each other and with the specification
Bug report
Dev task https://zube.io/edenlabllc/e-health/c/6178
New manifest
[x] Describe how does hash impact job

@warehousman , @Eugene-Yesakov  I did all I could. Decide, please, either we are going to do task https://zube.io/edenlabllc/e-health/c/6178 or we close this task without further fixes.


lock function mentioned in README is not available

There is this consul.lock() which is not available actually. Does anybody know where it's gone?

It's not in a released version yet. I'll take a look at releasing it this weekend and close this ticket when I do.

I released version 0.15.0 to npm, although I marked consul.lock as experiment in the README for now.


Transfer from keras to caffe 【error forSeparableConv 】

Platform (like ubuntu 16.04/win10):ubuntu 16.04
Python version:python2.7.12
Source framework with version (like Tensorflow 1.4.1 with GPU):keras 2.2.4
Destination framework with version (like CNTK 2.3 with GPU):caffe
Pre-trained model path (webpath or webdisk path):
Running scripts:
mmconvert -sf keras -iw age.h5 --dstNodeName age -df caffe  -om mycaffemodel
Using TensorFlow backend.
2019-07-12 17:28:20.255156: I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 AVX512F FMA
2019-07-12 17:28:20.361369: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1212] Found device 0 with properties:
name: Quadro P2000 major: 6 minor: 1 memoryClockRate(GHz): 1.4805
pciBusID: 0000:65:00.0
totalMemory: 4.94GiB freeMemory: 4.28GiB
2019-07-12 17:28:20.361401: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1312] Adding visible gpu devices: 0
2019-07-12 17:28:20.573748: I tensorflow/core/common_runtime/gpu/gpu_device.cc:993] Creating TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 4032 MB memory) -> physical GPU (device: 0, name: Quadro P2000, pci bus id: 0000:65:00.0, compute capability: 6.1)
IR network structure is saved as [cf3c22ff6bc040c4b67fefb242b96e02.json].
IR network structure is saved as [cf3c22ff6bc040c4b67fefb242b96e02.pb].
IR weights are saved as [cf3c22ff6bc040c4b67fefb242b96e02.npy].
Parse file [cf3c22ff6bc040c4b67fefb242b96e02.pb] with binary format successfully.

Target weights are saved as [cf3c22ff6bc040c4b67fefb242b96e02.npy].
Traceback (most recent call last):
  File "/home/bill/.local/bin/mmconvert", line 10, in <module>
    sys.exit(_main())
  File "/home/bill/.local/lib/python2.7/site-packages/mmdnn/conversion/_script/convert.py", line 112, in _main
    dump_code(args.dstFramework, network_filename + '.py', temp_filename + '.npy', args.outputModel, args.dump_tag)
  File "/home/bill/.local/lib/python2.7/site-packages/mmdnn/conversion/_script/dump_code.py", line 32, in dump_code
    save_model(MainModel, network_filepath, weight_filepath, dump_filepath)
  File "/home/bill/.local/lib/python2.7/site-packages/mmdnn/conversion/caffe/saver.py", line 9, in save_model
    MainModel.make_net(dump_net)
  File "cf3c22ff6bc040c4b67fefb242b96e02.py", line 100, in make_net
    n = KitModel()
  File "cf3c22ff6bc040c4b67fefb242b96e02.py", line 38, in KitModel
    n.batch_normalization_4 = L.BatchNorm(n.separable_conv2d_1, eps=0.0010000000475, use_global_stats=True, ntop=1)
  File "/home/bill/E/dl_base/caffe_program/caffe_mobilenet/python/caffe/net_spec.py", line 180, in __getattr__
    return self.tops[name]
KeyError: 'separable_conv2d_1'

Before I can successfully convert mxnet's depthwise conv to caffe's groupconv, how to deal with the keras's SeparableConv and transfer to caffe's groupconv. Thanks!


[Feature] Cloudflare Workers AI Support

Cloudflare has a cheap and good api with multiple models.
Any plans to implement this ?
I think I have the time to implement this.

Hey @saturnphp,
It's a great idea. No one has started working on this so feel free to contribute


[webview_flutter] Added 'allowsInlineMediaPlayback' property

Description
The PR includes following changes:

adds allowsInlineMediaPlayback property to WebView to support inline media playback in iOS.
switched to using collection literals for Set.

Related Issues
#25630
#42099
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@blasten This is ready to be reviewed.

Although I made all the changes, the ipad is working in full screen but the iphone does not work
https://user-images.githubusercontent.com/57063848/104122686-8abf8700-5357-11eb-96b4-beb18cc5c4e1.mov
https://user-images.githubusercontent.com/57063848/104122691-901cd180-5357-11eb-9dcc-aa6b842263de.mov

Although I made all the changes, the ipad is working in full screen but the iphone does not work
https://user-images.githubusercontent.com/57063848/104122686-8abf8700-5357-11eb-96b4-beb18cc5c4e1.mov
https://user-images.githubusercontent.com/57063848/104122691-901cd180-5357-11eb-9dcc-aa6b842263de.mov


Email Notifications

Describe the solution you'd like
Email notifications via SMTP
Describe alternatives you've considered
Slack, Webex
Additional context
It would be great to also have the option to use email notifications.

Hey @gianlucam76, I'll take this one.


feat(ssh-addition): added option and new sg

Added option and new sg
Updated examples

This PR is included in version 0.3.0 :tada:


Update README info and prune package dependencies

Still resolving packages, but I wanted to push up the updated README and LICENSE info.

Nvm, all resolved now! Ready for changes to be reviewed @ayuspark.


Update Bulma to latest version

The version used by this project is now almost 2 years old and a lot of features have been released since.

Hi @JonasUJ, thanks for opening this issue!
This project hasn't been very actively maintained, however, this organisation is a hotbed of activity and we haven't forgotten about this repo. We'd be happy to accept a pull request that updates the version, if you're willing to open one.

Closed by #48


what size

Hi，What size images did you use when evaluating, still 480x224 or origin size of 1600x900

And you rotated and  translate the camera image before training, but you only  adjust the camera intrinsics and didn't adjust the camera extrinsics. I don't know if this is right.

the camera image is only resized and cropped during training which only affects the camera intrinsics
for all stages, both train and eval, we use 480x224


Node.js Nightly version 15.0.0-20201006

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

Microsoft Reviewers: Open in CodeFlow

/azp run


Print loss on the terminal through the slim.learning.train () function

How to print the loss value on the terminal, such as ：INFO:tensorflow:global step 2511: loss = 2510.0000 (1.71 sec/step).

the same question


Add aggregation tables reconfiguration after import metadata

Related #13882

Tests are failing :red_circle:
Also, please add some specific tests for this, and log errors with CartoDb::Logger instead of puts

@javitonino Yep, I just want to validate the idea because I don't have an idea of the migrator :)

Yeah, the main approach seems like the way to go :)

retest this please

@javitonino :green_apple: second check?

@javitonino last one?


Fix host failures

Running failure tests result in errors.
This has to do with the state of a host when it fails.


Widget with label: size issue after text update

Hello,
I have an embedded widget in a custom node, containing a layout and a frame with text.
When initializing the widgets, the frame size is OK (fully inside node):

But when updating the text of the label (after clicking on it, through code in click event), the size is KO (label is out of node perimeter):

Thank you for your help..

Hi yvanblanchard, i figure it out and find quite easy.
If you're using deafult DataFlowGraphModel and DataFlowGraphicsScene like me, everything is almost ok.
This work is based on main brach commit 5465ddc91f1b30823c5fc3def93138cbec4c9394
What you need to do is:

add a Q_SIGNALS in DataFlowGraphModel.hpp named void embeddedWidgetSizeUpdated(NodeId const);
connect this in DataFlowGraphModel.cpp in DataFlowGraphModel::addNode(QString const nodeType) function with connect(model.get(),&NodeDelegateModel::embeddedWidgetSizeUpdated,this,[newId, this](){Q_EMIT embeddedWidgetSizeUpdated(newId);});
PS: You will see lots of connect method there, you can't miss it. ;)
last step is connect this in DataFlowGraphicsScene.cpp in the constructor with connect(&_graphModel,&DataFlowGraphModel::embeddedWidgetSizeUpdated,[this](NodeId const nodeId){ onNodeUpdated(nodeId); });
now you can use this->embeddedWidgetSizeUpdated(); in your DataModel

DataFlowGraphModel.hpp
Q_SIGNALS:
     void inPortDataWasSet(NodeId const,PortType const,PortIndex const);
     +void outPortDataUpdated(NodeId const)

DataFlowGraphModel.cpp
void DataFlowGraphModel::onOutPortDataUpdated(NodeId const nodeId)
{
      +Q_EMIT outPortDataUpdated(nodeId);
      ...
}

DataFlowGraphicsScene.cpp
+connect(&_graphModel,
             +&DataFlowGraphModel::outPortDataUpdated,
             +[this](NodeId const nodeId) { onNodeUpdated(nodeId); });

Thank you !


[IA-3750] Convert Analysis Modal to TypeScript.

https://broadworkbench.atlassian.net/browse/IA-3891
Description
We want to convert AnalysisModal to typescript to get the benefits of type safety.
This PR far is the first step for the ticket - get the type script version of the file functioning.

One challenge I encountered was with the loading spinners and effectively updating the state.
In our original code, AnalysisModal would trigger a loading spinner on the file screen behind it. However, when using useAnalysisFiles, updating that loading seems more difficult.
How do we want to handle these loading spinners/busy state? In this PR I added a busy/setBusy for AnalysisModal - the UX seems a bit clearer than the current implementation, but I'm not sure my approach is what we want long term.

@nawatts
I feel compelled to once again pontificate about small, focused PRs. It would have been nice to have some of these changes (especially the rename that touched many files) in separate PRs.
I agree - that's my mistake to rename it in this PR. More than happy to revert that change and make a new ticket if you would prefer.

More than happy to revert that change and make a new ticket if you would prefer.

I won't request that. Just meant it as a reminder.


log/slog: slog.Time, doesn't handle dates after year 2262

Go version
1.21, 1.22
Output of go env in your module/workspace:
/

What did you do?
package main

import (
	"log/slog"
	"os"
	"time"
)

func main() {
	logger := slog.New(slog.NewJSONHandler(os.Stdout, nil))
	logger.Info("time test", slog.Time("expires_at", time.Date(2300, 1, 1, 0, 0, 0, 0, time.UTC)))
}

This is due to converting time.Time to nanosecond in slog.Time
What did you see happen?
{"time":"2009-11-10T23:00:00Z","level":"INFO","msg":"time test","expires_at":"1715-06-13T00:25:26.290448384Z"}
What did you expect to see?
I expected to see date in future e.g 2300-01-01 00:00:00.0Z in this case

Please check out Time.UnixNano.

Can I  have an explanation on why this was closed. This is a legit bug report. I know that this is because of the use of time.UnixNano, and I have also written this into the original ticket.  This is due to converting time.Time to nanosecond in slog.Time
IMHO this should be reopened and the slog must log any valid date time.Time supports. Unless you are saying that's OK, to have weird dates in the log, when you use dates after a certain one.

also #63844

I'm happy to leave this open, but as pointed out here, it is just another consequence of what #63844 is trying to fix. So I don't think there is any action to take now.

maybe we can just document the more limited range here?

maybe we can just document the more limited range here?

That seems to be the only thing we can do so far.

This issue is about slog happening to have picked a very tiny time format compared to time.Time in its zeal for avoiding allocations. This has nothing to do with #63844, which is about optimizing the time package. It's true that the issue suggests restricting the time range, but to something like 60,000 years, not the mere 500 that slog allows.
I think we should probably fix this by making the time constructor store an actual time.Time into v.any when the value is out of range for UnixNano. That would still be allocation-free for all the times anyone would ever see, but it also avoids slog introducing time range restrictions that do not exist in package time.
/cc @jba


Reset password is always redirecting

I have a custom template setup
<template name="resetPwd">

                    {{> atForm state='resetPwd'}}

</template>

And my route
Router.route('resetPwd', {
    path: '/reset-password'
});

http://localhost:3000/#/reset-password/d2RQzmh_mWzVi6fMvF2iscCoKNRROlXRzxym2ENWMNY
But when I navigate to the reset url it's always redirecting to the main site page and not loading the reset password template

hello @almogdesign, you should be configuring the resetPwd route like this:
AccountsTemplates.configureRoute('resetPwd', {
    template: 'resetPwd',
});

so that the needed configurations and logics are put in place for you by UserAccounts.
Let me know whether this solves your problems or not.
Cheers,
Luca

No that did nothing it still redirect back to http://localhost:3000/#
if I got to http://localhost:3000/reset-password it's fine but the when it has the token it does not work

@splendido any news regarding this kind of a major issue?

@almogdesign could you have a look at the boilerplates?
They're all up to date with latest versions of Meteor and packages and everything's working fine for them...

I have the following packages installed is this incorrect?
useraccounts:iron-routing
useraccounts:bootstrap
and my router which is pretty straight forward
https://gist.github.com/almogdesign/6c35399ad9f92a691655

The boilerplate examples are not using custom templates I'm creating my own templates and using
{{> atForm state='signUp'}}

these lines should be changed to ask UserAccounts to set up them properly for you.
See these lines in the boilerplate.
Could you please try to insert your custom template into the bootstrap-iron-routing boilerplate and see whether it works or not?
it should be done somehow like this:
AccountsTemplates.configureRoute('resetPwd', {
  template: 'resetPwd',
});

Btw, within the above linked lines I cannot see how you're using your custom template.
What it left out from the gist by purpose?

@splendido  Nope that did not help or do anything attached gif showing the issue

@splendido the example you sent is incorrect -
AccountsTemplates.configureRoute('resetPwd');
even this does not work
AccountsTemplates.configureRoute('resetPwd', {
template: 'resetPwd',
});
Also the boilerplate is not using custom template but using the templates you provide. I thought the whole point of {{> atForm state='signUp'}} is that I can create what every template I want

Ok this issue happens even if I don't use a custom template the default useracconts template

This only happens when you try to set custom templates - https://github.com/meteor-useraccounts/core/blob/master/Guide.md#templates
I just tested it with the default templates and it's fine as you can see from the screenshot

Got everything working I had to create the templates with the same name and then use
AccountsTemplates.configureRoute('verifyEmail', {
template: 'verifyEmail'
});

Glad you got it working!
Btw, your code snippet looks exactly like the one I posted above. The only difference is you're now configuring the verifyEmail route while you claimed the problem was with the resetPwd one...
Was the issue title wrongly written?
In any case, just to verify there are no bugs, could you please try to change the name of your custom template to, e.g., myTemplate and see whether it still works or not?
...from your last post it might seem that you had to use the verifyEmail name for the custom template because different names didn't work.
Is this what you really meant?

No the reset password was an issue I had copied the wrong code

@AlmogRnD can you share how did you fix the reset password url issue?
After I upgrade to Meteor 1.3,  I get Urls like
http://localhost:3000/#/reset-password/xxxxx
http://localhost:3000/#/verify-email/xxxxxx
I'm using flow-router
Any idea of how to fix it is grealy appreciated!

Below is to fix this issue
Accounts.urls.resetPassword = (token) => Meteor.absoluteUrl(reset-password/${token});
Accounts.urls.verifyEmail = (token) => Meteor.absoluteUrl(verify-email/${token});
Accounts.urls.enrollAccount = (token) => Meteor.absoluteUrl(enroll-account/${token});
Accoutns.urls url_server.js srouce code: https://github.com/meteor/meteor/blob/be986fd70926c9dd8eff6d8866205f236c8562c4/packages/accounts-base/url_server.js

You can customize the resetting page by change the link sent to the user.
  Accounts.emailTemplates.resetPassword = {
    from: ()=><EMAIL_ADDRESS>    subject: ()=> "Reset Your Account Password",
    text: (user, url)=> {
      const newUrl = url.replace("#/reset-password", "setpswd");
      return `Hi,Click the Link below to reset your password:\n${newUrl}`;
    }
  };

I have written a gist about the whole customization process.
Check it out here for more information.