Some problems about  running code

Thank you for sharing the code!I am the Python beginner，so it is hard for me to run the code successfully,and I need your suggestions!
First is run RAFT to get flow,and then run TRAIN.PY.IF I misunderstand the process.please tell me.
When I run run_inference.py in RAFT,I can only get the output folders,but there are nothing in the folders,the folders are empty.I set the data_path 、rgbpath and  mode,I will be appreciated for your advice!
Thanks a lot!

I have already run the run_inference.py successfully,but I get output of 3045 pictures in Flows_gap1 ,which DIVAS dateset has 3455 original images，and I think it's why I failed to run train.py that I cannot solve it.I would be happy if you could give me some advice.

I have same problems and how you solve it?Thanks

I have same problems and how you solve it?Thanks
I don't know if you have the same problem as me, but I can tell you some solutions.
In config.py,find the code:pair_list = [p for p in itertools.combinations(pairs , 2)]and change it to pair_list = [(1,-1),(2,-2)], the file train.py can run normally

I have same problems and how you solve it?Thanks

I don't know if you have the same problem as me, but I can tell you some solutions.
In config.py,find the code:pair_list = [p for p in itertools.combinations(pairs , 2)]and change it to pair_list = [(1,-1),(2,-2)], the file train.py can run normally

ok thanks. But I got the blank dataset after run raft, the same problem as your first step. How can I solve this?
发自我的iPhone
------------------ Original ------------------
From: AzulYang @.>
Date: Wed,Aug 17,2022 8:27 PM
To: charigyang/motiongrouping @.>
Cc: lotus0903 @.>, Comment @.>
Subject: Re: [charigyang/motiongrouping] Some problems about  running code(Issue #17)
I have same problems and how you solve it?Thanks
I don't know if you have the same problem as me, but I can tell you some solutions.
In config.py,find the code:pair_list = [p for p in itertools.combinations(pairs , 2)]and change it to pair_list = [(1,-1),(2,-2)], the file train.py can run normally
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

ok thanks. But I got the blank dataset after run raft, the same problem as your first step. How can I solve this? 发自我的iPhone
…
------------------ Original ------------------ From: AzulYang @.> Date: Wed,Aug 17,2022 8:27 PM To: charigyang/motiongrouping @.> Cc: lotus0903 @.>, Comment @.> Subject: Re: [charigyang/motiongrouping] Some problems about running code(Issue #17) I have same problems and how you solve it?Thanks I don't know if you have the same problem as me, but I can tell you some solutions. In config.py,find the code:pair_list = [p for p in itertools.combinations(pairs , 2)]and change it to pair_list = [(1,-1),(2,-2)], the file train.py can run normally — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

At that time, I set the wrong rgb path. Just need to set data_path in run_inference.py,you can print path to make sure the Correctness of settings.

I set rgbpath to 'Davis/JPEGImages' and 'Davis/JPEGImages/480p/boat' and all failed. I only got empty folders with no images.

I set rgbpath to 'Davis/JPEGImages' and 'Davis/JPEGImages/480p/boat' and all failed. I only got empty folders with no images.

Here is my setting:
data_path = "/home/m/桌面/motiongroup"
rgbpath = data_path + '/DAVIS/JPEGImages'  # path to the dataset
If you want to use one folder in 480p like boat will lead to failure.It seems like only input the full folders will work.

Thank you for sharing the code!I am the Python beginner，so it is hard for me to run the code successfully,and I need your suggestions! First is run RAFT to get flow,and then run TRAIN.PY.IF I misunderstand the process.please tell me. When I run run_inference.py in RAFT,I can only get the output folders,but there are nothing in the folders,the folders are empty.I set the data_path 、rgbpath and mode,I will be appreciated for your advice! Thanks a lot!

Hi i have the same problem? Have you solved it?

I got 48.5 with iteration 399,000 on DAVIS2016,here is my running_command,how can I get the result of 68.2?(Cause my GPU is RTX3070,I have to set batch_size to 8)
{
"batch_size": 8,
"lr": 0.0005,
"num_train_steps":<PHONE_NUMBER>.0,
"warmup_steps": 200,
"decay_steps": 80000.0,
"decay_rate": 0.5,
"dataset": "DAVIS",
"with_rgb": false,
"flow_to_rgb": true,
"num_slots": 2,
"num_iterations": 5,
"verbose": null,
"resume_path": true,
"inference": false,
"resolution": [
128,
224
]
}
@charigyang

Thank you for sharing the code!I am the Python beginner，so it is hard for me to run the code successfully,and I need your suggestions! First is run RAFT to get flow,and then run TRAIN.PY.IF I misunderstand the process.please tell me. When I run run_inference.py in RAFT,I can only get the output folders,but there are nothing in the folders,the folders are empty.I set the data_path 、rgbpath and mode,I will be appreciated for your advice! Thanks a lot!

Hi i have the same problem, Have you solved it?  These are my settings

Maybe need a direct path,but I'm not sure it can solve your problem.

Thank you for sharing the code!I am the Python beginner，so it is hard for me to run the code successfully,and I need your suggestions! First is run RAFT to get flow,and then run TRAIN.PY.IF I misunderstand the process.please tell me. When I run run_inference.py in RAFT,I can only get the output folders,but there are nothing in the folders,the folders are empty.I set the data_path 、rgbpath and mode,I will be appreciated for your advice! Thanks a lot!

Hi i have the same problem, Have you solved it?  These are my settings

Maybe need a direct path,but I'm not sure it can solve your problem.

Can't create any flow images either... So sadly

Data_path="Path to your folder motiongroup”
rgbpath= data_path+/DAVIS/JPEGImage
It seems that you cannot use a single one in 480p folder as input.
发自我的iPhone
------------------ Original ------------------
From: lotus0903 @.>
Date: Thu,Aug 18,2022 3:50 PM
To: charigyang/motiongrouping @.>
Cc: AzulYang @.>, Author @.>
Subject: Re: [charigyang/motiongrouping] Some problems about  running code (Issue #17)
I set rgbpath to 'Davis/JPEGImages' and 'Davis/JPEGImages/480p/boat' and all failed. I only got empty folders with no images.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thank you for sharing the code!I am the Python beginner，so it is hard for me to run the code successfully,and I need your suggestions! First is run RAFT to get flow,and then run TRAIN.PY.IF I misunderstand the process.please tell me. When I run run_inference.py in RAFT,I can only get the output folders,but there are nothing in the folders,the folders are empty.I set the data_path 、rgbpath and mode,I will be appreciated for your advice! Thanks a lot!

Hi i have the same problem, Have you solved it?  These are my settings

Maybe need a direct path,but I'm not sure it can solve your problem.

Can't create any flow images either... So sadly

I think the reason is probably you are using a newer python version. Assume you input the full DAVIS folder, for python >= 3.5, within "predict.py", you should use this:
images = glob.glob(os.path.join(args.path, '', '*.png'), recursive=True) +
glob.glob(os.path.join(args.path, '', '*.jpg'), recursive=True)

I have the same problem.
The DAVIS dataset has class directories under JPEGImages and it has png files but, run_inference.py needs png files under JPEGImages. Perhaps, you should rewrite the image path on predict.py.
https://github.com/charigyang/motiongrouping/blob/e7d955e1bffe9fc973b38f076758fbb8d596e851/raft/predict.py#L57

I'm sorry. My answer was wrong.
It seems that run_inference.py needs to be modified.
https://github.com/charigyang/motiongrouping/blob/e7d955e1bffe9fc973b38f076758fbb8d596e851/raft/run_inference.py#L15-L20

Hi, I have solved the problem.  No files are generated because of the path errors. Just try to modify the code run_inference.py and don't use 'folder = gb.glob(os.path.join(rgbpath, '*'))'

Hi, I have solved the problem. No files are generated because of the path errors. Just try to modify the code run_inference.py and don't use 'folder = gb.glob(os.path.join(rgbpath, '*'))'

Hi, if I don't use the line of code you mentioned, then what should I change the folder to? Can you show me the modified code? Thank you very much! ~

I have same problems and how you solve it?Thanks

I don't know if you have the same problem as me, but I can tell you some solutions. In config.py: find the code:pair_list = [p for p in itertools.combinations(pairs , 2)].change it to pair_list = [(1,-1),(2,-2)],and the file train.py can run normally

Hi, I have successfully generated the optical flow, but I am having some problems regarding the running of the train.py file. When I run the train.py file, the system doesn't report an error and it doesn't end the program, but it just doesn't continue on to the next level. When I debugged it, I found that the code is stuck as shown here.
Have you encountered similar problems when trying to run the train.py file successfully? Or what other efforts have you made? For example, you said to change the contents of pair in config.
Thank you very much for your answer!

I have same problems and how you solve it?Thanks

I don't know if you have the same problem as me, but I can tell you some solutions. In config.py: find the code:pair_list = [p for p in itertools.combinations(pairs , 2)].change it to pair_list = [(1,-1),(2,-2)],and the file train.py can run normally

Hi, I have successfully generated the optical flow, but I am having some problems regarding the running of the train.py file. When I run the train.py file, the system doesn't report an error and it doesn't end the program, but it just doesn't continue on to the next level. When I debugged it, I found that the code is stuck as shown here.
Have you encountered similar problems when trying to run the train.py file successfully? Or what other efforts have you made? For example, you said to change the contents of pair in config.
Thank you very much for your answer!

I have same problems and how you solve it?Thanks

I don't know if you have the same problem as me, but I can tell you some solutions. In config.py: find the code:pair_list = [p for p in itertools.combinations(pairs , 2)].change it to pair_list = [(1,-1),(2,-2)],and the file train.py can run normally

Hi, I have successfully generated the optical flow, but I am having some problems regarding the running of the train.py file. When I run the train.py file, the system doesn't report an error and it doesn't end the program, but it just doesn't continue on to the next level. When I debugged it, I found that the code is stuck as shown here. Have you encountered similar problems when trying to run the train.py file successfully? Or what other efforts have you made? For example, you said to change the contents of pair in config. Thank you very much for your answer! ![image](https://user-images.githubusercontent.com/14450282Based on my experience, it seems that the data was not loaded correctly. Please debug step by step to confirm that the dataset is loaded correctly. This may be due to differences in image format, png, and jpg3/280291802-6c88a97a-ee48-425b-bf4b-0f1533b79b7a.png)

Based on my experience, it seems that the data was not loaded correctly. Please debug step by step to confirm that the dataset is loaded correctly. This may be due to differences in image format, png, and jpg

Based on my experience, it seems that the data was not loaded correctly. Please debug step by step to confirm that the dataset is loaded correctly. This may be due to differences in image format, png, and jpg

@ZhangWeihao3014

Based on my experience, it seems that the data was not loaded correctly. Please debug step by step to confirm that the dataset is loaded correctly. This may be due to differences in image format, png, and jpg

Thanks for your answer, I will try again!

fix the problem:


Support discord_arch_electron aur package

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

discord arch electron is a aur(arch users repository) build of discord that patches discord to use the system version of electron which fixes some issues and improves the performance of discord. when I attempt to install betterdiscord via betterdiscordctl and pointing it to the location of discord with the patches(/usr/lib/discord/) a error about injector.js not being found is shown. I would like betterdiscord to support this patched version as it feels like it would be the best version of discord too me

Perhaps @ObserverOfTime would have some insight towards this as betterdiscordctl is their installer and I am unfamiliar with linux discord.

Perhaps @ObserverOfTime would have some insight towards this as betterdiscordctl is their installer and I am unfamiliar with linux discord.

See bb010g/betterdiscordctl#84. I can't fix it but if someone finds a way, do submit a pull request.

See bb010g/betterdiscordctl#84. I can't fix it but if someone finds a way, do submit a pull request.

@advaithm if you could provide more information on the structure of how the discord_arch_electron package works, such as where the discord modules are stored (specifically discord_desktop_core), that would help move this along. I am going to move this over to the installer repository as it more likely requires a change there than here.

I guess GitHub won't let me transfer it for whatever reason... So @advaithm please open a new issue at https://github.com/BetterDiscord/Installer with the information above

I've moved to powercord but it should be pretty easy to figure it out based on the PKGBUILD, its a bash scripts. functions are called in the order of prepare()->packager()
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=discord_arch_electron

Submitted in BetterDiscord/Installer#41.


Get memory alignment from device

Currently we are using defines L1_ALIGNMENT, PCIE_ALIGNMENT, and DRAM_ALIGNMENT:
  std::vector<::flatbuffers::Offset<tt::target::ChipDesc>> chipDescs = {
      ::tt::target::CreateChipDesc(
          fbb, toFlatbuffer(device.arch()), &deviceGrid, (1 << 20), 12,
          (1 << 20), L1_ALIGNMENT, PCIE_ALIGNMENT, DRAM_ALIGNMENT),
  };

But these actually need to come from the metal device i.e. device->get_l1_alignment(), so that we don't have hard coded architecture specific numbers in our runtime.  If these APIs don't exist in metal, let's file an issue and get signoff from the metal team, should be relatively uncontroversial.  Please tag any filed issues against metal on this issue.

Added this request to metal ticket linked, nothing to do here for now, will replace with API call once ready.


add new obd commands

from #108:
add new obd commands from obd-java-api 1.0-RC2 to https://github.com/pires/android-obd-reader/blob/master/src/main/java/com/github/pires/obd/reader/config/ObdConfig.java so that they become available

tested but none of the new commands seems to be supported by my car


Placement UI bugs

Erase button no longer highlighted red when erase mode is on.
Client crashes on changing placement mode.

Actually I'll just fix these myself because I need em for content.


Code adjustments to fix the API for readthedocs

Readthedocs currently isn't compiling our API correctly, these changes resolve the issues for my local html compiling.
The only caveat is that I can't get the make_psfmasks.py file to be documented correctly. When running the Makefile it struggles to get through the file due to the following lines:
pxsc = nircam._pixelscale_long  # arcsec
xoff, yoff = crpix1 - xcen - offset_lwb[filt] / pxsc, crpix2 - ycen

It's unable to recognize the pxsc as a float, so can't step through the calculation. If I run the file on its own everything is fine, so must be some issue with the Makefile behaviour.

Is the addition of the 3 binary mask files in this PR intentional? That doesn't seem like it has much to do with docs building, or does it?
e.g. spaceKLIP/resources/transmissions/JWST_MIRI_F1065C_transmission_webbpsf-ext_v2.fits

@mperrin It doesn't, let me undo that.

@mperrin Okay added some more tweaks to get the API sharpened up / connected to the source code. I won't make any changes on top of this.


dhcp-proxy: fix typo for --activity-timeout

The flag was spelled incorrectly, lets fix this before the release.

@baude @mheon PTAL

LGTM

/lgtm


Non-existing export is imported

Hi guys,
When trying to use this module with the follow code, I get the following error:
Code:
import TaskRunner from "concurrent-tasks"; const runner = new TaskRunner();
Error:
node_modules\concurrent-tasks\es\index.js (11:74) Error: Non-existent export 'setAppropriateConcurrency' is imported from node_modules\concurrent-tasks\es\PrivateFunctions.js 9: function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } } 10:  11: import { startCheckAndRun, runPending, startCheck, addCheck, removeCheck, setAppropriateConcurrency } from './PrivateFunctions';                                                                               ^ 12: import log from './log';
Can anyone help with this?

Hello @WardBrink,
Thanks for the report and sorry for the extreme delay in responding.
Can you try out the v2 version? It has a better API and more extensibility as well as being the version that's going to replace this package in the future.
Let me know if you still have this issue with it.


Discovered another issue with AdditionalValidationPackagesFromPackageSetFn

What happens when we don't have metadata for a package that we want to include? Welp we're screwed! I don't want to call Get-AllPkgProperties in my custom function when we CAN PASS IT IN.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Can't connect to anything

I updated to 1.31, and it loads for 10 minutes and comes back with "can't connect to a node".
I can't even get the wallet open to do anything, and the localhost rejects any connection attempts via browser since the wallet isn't actually loading.

I have the same issue.... did you ever get this resolved ?

No. I have tried on a few machines, Linux and Windows.
Can't get anything to connect, and have not been able to get a response from the Stratis project. See if pinging them here helps...
@sondreb @YakupIpek @Daegalus

I'm a linux user, updated to v1.3.1. Compiled and started the node with npm run package:linux then entering the StratisCore.UI/app-builds/linux-unpacked directory and starting core with ./stratis-core
This resulted in one connection so far with:
      OUT Peer:[::ffff:<IP_ADDRESS>]:16178,            (r/s/c):412193/412193/0               R/S MB: 4.78/0.11               agent:StratisNode:3.0.5 (70012)

I still get the same thing.
Cloned repo, built, ran from directory as instructed .
Shows the loading screen for 4-5 minutes, then I get "Failed to load, can not connect to a stratis node"

What are you seeing in the terminal ?

@AGowans have you tried shutting down Core, removing peers.json in ~/.stratisnode/Stratis/StratisMain and starting Core again?


Memory leak in TasksFragment

Detected leak path by LeakCanary:
┬───
│ GC Root: System class
│
├─ android.view.inputmethod.InputMethodManager class
│    Leaking: NO (InputMethodManager↓ is not leaking and a class is never leaking)
│    ↓ static InputMethodManager.sInstance
├─ android.view.inputmethod.InputMethodManager instance
│    Leaking: NO (DecorView↓ is not leaking and InputMethodManager is a singleton)
│    ↓ InputMethodManager.mNextServedView
├─ com.android.internal.policy.DecorView instance
│    Leaking: NO (RecyclerView↓ is not leaking and View attached)
│    mContext instance of com.android.internal.policy.DecorContext, wrapping activity com.example.android.architecture.blueprints.todoapp.tasks.TasksActivity with mDestroyed = false
│    Parent android.view.ViewRootImpl not a android.view.View
│    View#mParent is set
│    View#mAttachInfo is not null (view attached)
│    View.mWindowAttachCount = 1
│    ↓ DecorView.mAttachInfo
├─ android.view.View$AttachInfo instance
│    Leaking: NO (RecyclerView↓ is not leaking)
│    ↓ View$AttachInfo.mScrollContainers
├─ java.util.ArrayList instance
│    Leaking: NO (RecyclerView↓ is not leaking)
│    ↓ ArrayList.elementData
├─ java.lang.Object[] array
│    Leaking: NO (RecyclerView↓ is not leaking)
│    ↓ Object[].[1]
├─ androidx.recyclerview.widget.RecyclerView instance
│    Leaking: NO (View attached)
│    mContext instance of com.example.android.architecture.blueprints.todoapp.tasks.TasksActivity with mDestroyed = false
│    View.parent android.widget.LinearLayout attached as well
│    View#mParent is set
│    View#mAttachInfo is not null (view attached)
│    View.mID = R.id.tasks_list
│    View.mWindowAttachCount = 1
│    ↓ RecyclerView.mAdapter
│                   ~~~~~~~~
├─ com.example.android.architecture.blueprints.todoapp.tasks.TasksAdapter instance
│    Leaking: UNKNOWN
│    ↓ TasksAdapter.viewModel
│                   ~~~~~~~~~
├─ com.example.android.architecture.blueprints.todoapp.tasks.TasksViewModel instance
│    Leaking: UNKNOWN
│    ↓ TasksViewModel._snackbarText
│                     ~~~~~~~~~~~~~
├─ androidx.lifecycle.MutableLiveData instance
│    Leaking: UNKNOWN
│    ↓ MutableLiveData.mObservers
│                      ~~~~~~~~~~
├─ androidx.arch.core.internal.SafeIterableMap instance
│    Leaking: UNKNOWN
│    ↓ SafeIterableMap.mStart
│                      ~~~~~~
├─ androidx.arch.core.internal.SafeIterableMap$Entry instance
│    Leaking: UNKNOWN
│    ↓ SafeIterableMap$Entry.mNext
│                            ~~~~~
├─ androidx.arch.core.internal.SafeIterableMap$Entry instance
│    Leaking: UNKNOWN
│    ↓ SafeIterableMap$Entry.mKey
│                            ~~~~
├─ com.example.android.architecture.blueprints.todoapp.util.ViewExtKt$setupSnackbar$1 instance
│    Leaking: UNKNOWN
│    Anonymous class implementing androidx.lifecycle.Observer
│    ↓ ViewExtKt$setupSnackbar$1.$this_setupSnackbar
│                                ~~~~~~~~~~~~~~~~~~~
╰→ androidx.coordinatorlayout.widget.CoordinatorLayout instance
     Leaking: YES (ObjectWatcher was watching this because com.example.android.architecture.blueprints.todoapp.tasks.TasksFragment received Fragment#onDestroyView() callback (references to its views should be cleared to prevent leaks))
     key = 080d1590-6d2b-47da-8b64-4be379e522aa
     watchDurationMillis = 8201
     retainedDurationMillis = 3178
     key = 4e988abe-ac80-43b1-8004-e3c84d5d8bcd
     watchDurationMillis = 8205
     retainedDurationMillis = 3179
     mContext instance of com.example.android.architecture.blueprints.todoapp.tasks.TasksActivity with mDestroyed = false
     View#mParent is null
     View#mAttachInfo is null (view detached)
     View.mID = R.id.coordinator_layout
     View.mWindowAttachCount = 1

The reason is that the sample keep a lateinit listAdapter in TasksFragment. When we navigate to AddEditTaskFragment, the RecyclerView destroyed but the adapter is still keep the reference of it which causes the leak.
Do you have any suggestion to fix this?
Thanks!

If this is the case, then in the onDestroyView() method of TasksFragment we can set the viewDataBinding.tasksList.adapter = null. Gotta still test to make sure nothing breaks 💯

is there any more elegant way to fix this memory leak?


Gracefully return an error instead of panicking in non-X11 contexts

If this crate is used in a non-X11 Linux context, Cliboard::get_text() double panics.  Instead, I think an error should be returned from Clipboard::new() if it's not possible to connect to a X11 server.
I'd be happy to implement this change, but since it will move the initialization to an earlier time, I thought it would be wise to first ask if you have any concerns or objections.

First panic:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ClosedParseErr', src/x11_clipboard.rs:98:57
stack backtrace:
   0: rust_begin_unwind
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:475
   1: core::panicking::panic_fmt
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/panicking.rs:85
   2: core::option::expect_none_failed
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/option.rs:1221
   3: core::result::Result<T,E>::unwrap
             at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/result.rs:973
   4: arboard::x11_clipboard::LockedObjects::new
             at ./src/x11_clipboard.rs:98
   5: arboard::x11_clipboard::ensure_lo_initialized
             at ./src/x11_clipboard.rs:1173
   6: arboard::x11_clipboard::X11ClipboardContext::get_text
             at ./src/x11_clipboard.rs:1218
   7: arboard::Clipboard::get_text
             at ./src/lib.rs:65
   8: hello_world::main
             at ./examples/hello_world.rs:5
   9: core::ops::function::FnOnce::call_once
             at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/function.rs:227
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Second panic:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "PoisonError { inner: .. }"', src/x11_clipboard.rs:501:51
stack backtrace:
   0:     0x560ad3feeb00 - std::backtrace_rs::backtrace::libunwind::trace::h577ea05e9ca4629a
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/../../backtrace/src/backtrace/libunwind.rs:96
   1:     0x560ad3feeb00 - std::backtrace_rs::backtrace::trace_unsynchronized::h50b9b72b84c7dd56
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/../../backtrace/src/backtrace/mod.rs:66
   2:     0x560ad3feeb00 - std::sys_common::backtrace::_print_fmt::h6541cf9823837fac
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/sys_common/backtrace.rs:79
   3:     0x560ad3feeb00 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::hf64fbff071026df5
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/sys_common/backtrace.rs:58
   4:     0x560ad4008f7c - core::fmt::write::h9ddafa4860d8adff
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/fmt/mod.rs:1082
   5:     0x560ad3fed017 - std::io::Write::write_fmt::h1d2ee292d2b65481
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/io/mod.rs:1514
   6:     0x560ad3ff0de0 - std::sys_common::backtrace::_print::ha25f9ff5080d886d
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/sys_common/backtrace.rs:61
   7:     0x560ad3ff0de0 - std::sys_common::backtrace::print::h213e8aa8dc5405c0
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/sys_common/backtrace.rs:48
   8:     0x560ad3ff0de0 - std::panicking::default_hook::{{closure}}::h6482fae49ef9d963
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:200
   9:     0x560ad3ff0b2c - std::panicking::default_hook::he30ad7589e0970f9
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:219
  10:     0x560ad3ff1443 - std::panicking::rust_panic_with_hook::haa1ed36ada4ffb03
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:569
  11:     0x560ad3ff1019 - std::panicking::begin_panic_handler::{{closure}}::h7001af1bb21aeaeb
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:476
  12:     0x560ad3feef8c - std::sys_common::backtrace::__rust_end_short_backtrace::h39910f557f5f2367
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/sys_common/backtrace.rs:153
  13:     0x560ad3ff0fd9 - rust_begin_unwind
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:475
  14:     0x560ad40082a1 - core::panicking::panic_fmt::h4e2659771ebc78eb
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/panicking.rs:85
  15:     0x560ad40080c3 - core::option::expect_none_failed::h448b58a024c2c33a
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/option.rs:1221
  16:     0x560ad3f6e040 - core::result::Result<T,E>::unwrap::h687c87c3f313b615
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/result.rs:973
  17:     0x560ad3f5d7fe - arboard::x11_clipboard::Manager::destruct::hd0e565fd5595762c
                               at /home/jonas/Code/experiments/rust/clipboard/arboard/src/x11_clipboard.rs:501
  18:     0x560ad3f521ef - <arboard::x11_clipboard::X11ClipboardContext as core::ops::drop::Drop>::drop::h307ccff1f5befb5d
                               at /home/jonas/Code/experiments/rust/clipboard/arboard/src/x11_clipboard.rs:1205
  19:     0x560ad3f4e526 - core::ptr::drop_in_place::hf42d2093737883dc
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ptr/mod.rs:184
  20:     0x560ad3f4e4fe - core::ptr::drop_in_place::hab210301fc93f3d4
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ptr/mod.rs:184
  21:     0x560ad3f4e1fb - hello_world::main::h844a495b302b237f
                               at /home/jonas/Code/experiments/rust/clipboard/arboard/examples/hello_world.rs:10
  22:     0x560ad3f4e49b - core::ops::function::FnOnce::call_once::h13d94cf6f0dabc3e
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/function.rs:227
  23:     0x560ad3f4e43e - std::sys_common::backtrace::__rust_begin_short_backtrace::hce407c5b19206a5a
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/sys_common/backtrace.rs:137
  24:     0x560ad3f4e981 - std::rt::lang_start::{{closure}}::h2d710c588bfdbe82
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/rt.rs:66
  25:     0x560ad3ff1841 - core::ops::function::impls::<impl core::ops::function::FnOnce<A> for &F>::call_once::h6a3209f124be2235
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/core/src/ops/function.rs:259
  26:     0x560ad3ff1841 - std::panicking::try::do_call::h88ce358792b64df0
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:373
  27:     0x560ad3ff1841 - std::panicking::try::h6311c259678e50fc
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:337
  28:     0x560ad3ff1841 - std::panic::catch_unwind::h56c5716807d659a1
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panic.rs:379
  29:     0x560ad3ff1841 - std::rt::lang_start_internal::h73711f37ecfcb277
                               at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/rt.rs:51
  30:     0x560ad3f4e957 - std::rt::lang_start::hee4923ac0b96e659
                               at /home/jonas/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/rt.rs:65
  31:     0x560ad3f4e42a - main
  32:     0x7f5d6a551152 - __libc_start_main
  33:     0x560ad3f4e09e - _start
  34:                0x0 - <unknown>
thread panicked while panicking. aborting.

I've never encountered this but I image that this could be relevant in a test environment. Either way this would be a very useful enhancement, I have no objections at all.

Should this be waited on, for the next release, or should I release a new version with your GNOME fix as soon as I can?

While there's no need to rush, I think releasing the GNOME1 fix shouldn't be blocked by this one.  The problems are orthogonal, and users are more likely to run into that one.
1 Based on the clip issue, #4 may also affect KDE.

Is this still applicable? https://github.com/ArturKovacs/arboard/pull/13 seems to have removed all the unwraps in the X11 implementation so I would assume this now works as expected.

Thank you, @mathieulj for pointing that out.
I tested this by switching to a no-gui terminal (Ctrl+Alt+F4 on Ubuntu) and running cargo test on arboard. I got the following error returned form Clipboard::new():
Unknown { .. } - "Unknown error while interacting with the clipboard: Display parsing error"

So with that, I'm closing this. Please let me know if this should be re-opened for any reason.


Add Issue and Contribution Template

Now project does not have any guidelines for opening Issue and for making a #PR.
So, Add Issue and Contribution Template

Hi @therajanmaurya. I can take of this.
You an assign this to me and we an discuss the requirements.

@Abrasha welcome.

Add #37


[Keras] Conv2DTranspose constfolding issue with variable batch size.

TF Version
TF 2.10
Gist of the issue
The Conv2DTranspose does not behave the same when batch_size is fixed (e.g. batch_size=32) and when it's variable (e.g. batch_size=None). The problem seems to reside in the way constant_folding is working or in the way input_sizes is managed (which in this context is actually meaning the output shape).
Google Colab for Repro
Full repro case in detail: https://colab.research.google.com/drive/1sv3c8KqcK1lcUKmvJAAQheSU9VPiJH6N?authuser=1#scrollTo=VimLmENQOHQX
A picture is worth a thousands word
1. Let's create the model
Picture the most basic model using Conv2DTranspose
def create_model(batch_size=None):
  inputs = keras.layers.Input((128, 128, 3), dtype=tf.float32, batch_size=batch_size)

  net = keras.layers.Conv2DTranspose(filters=32, kernel_size=(3, 3))(inputs)

  model = keras.models.Model(inputs=inputs, outputs=net)
  return model

Fixed BS:

Variable BS:

So far so good, everything looks the same ...
2. Freeze + ConstFold + Plot
Now let's freeze the graph and plot the resulting graphdef:
Fixed BS:

Variable BS:

Conclusion
I hope the difference is fairly self explanatory, and that's a problem because that's a very different graph for inference and a lot more complicated graph to optimize by compilers (e.g. TF-TRT, XLA, TF Lite, TVM, etc).

@reedwm @bixia1 CC

@DEKHTIARJonathan
This issue seems to be Keras issue.  Please post this issue on keras-team/keras repo. as Keras development is fully moving to github.com/keras-team/keras. All issues and PRs related to keras will be addressed in that repo.
To know more see this TF forum discussion ;
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999
Thank you!

Will do thanks. I keep this one opened for reference

Could you please refer the comment here https://github.com/keras-team/keras/issues/17094#issuecomment-1284609390 and let us know if you still need assistance. Thanks!


Derive ord when possible

Fixes #884
r? @fitzgen

@bors-servo r+

:pushpin: Commit 27dad0b has been approved by fitzgen

:hourglass: Testing commit 27dad0b64d0a86107e61550afe2d08b3bd0e5155 with merge de0180e326de222e6f750c62323947390405b66c...

:broken_heart: Test failed - status-travis

(Note that Travis CI got stuck so I canceled and restarted a job, that isn't actually a test failure)

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing de0180e326de222e6f750c62323947390405b66c to master...


Can we manually set an outline color for all labels?

from discussion with @aeisenbarth

Use case

I want to plot contours only, so that I can overlay multiple labels without them occluding each other.
I want to plot all contours of a labels image with a uniform color, so that I can distinguish labels images (colorize by element, not by annotation).

Example
from spatialdata import SpatialData
from spatialdata.models import Labels2DModel
import numpy as np
import spatialdata_plot

# Two labels
array1 = np.zeros((10, 10), dtype=np.uint16)
array1[1:4, 1:4] = 1
array1[4:9, 4:9] = 2
# One label intersecting a label in array1
array2 = np.zeros((10, 10), dtype=np.uint16)
array2[2:7, 6:9] = 1
sdata = SpatialData(
    labels={
        "labels1": Labels2DModel.parse(array1, dims=("y", "x")),
        "labels2": Labels2DModel.parse(array2, dims=("y", "x")),
    }
)
sdata.pl.render_labels(
    elements="labels1",
    contour_px=3, # Surprising! contour_px=1 does not render at all
    outline=True,
    outline_alpha=1.0,
    # outline_color="red", # Not yet existing
    fill_alpha=0.0,
).pl.render_labels(
    elements="labels2",
    contour_px=3,
    outline=True,
    outline_alpha=1.0,
    # outline_color="green", # Not yet existing
    fill_alpha=0.0,
).pl.show(figsize=(2,2))

Actual:

Desired:

Both cmap and palette affect the fill, not the contour.

I don't know whether there are use cases for having the outline color by annotation value.

Analogously, one could want pass an actual color name `color="red" (instead of an obs column/ var name) to apply a uniform fill color to all labels.

Issues

Labels rendering currently requires the region to be annotated in the table, or no table at all. This is because _render_labels selects labels to render based on the table. However, a SpatialData with labels only is perfectly valid and there can be interest in visualizing them.

Adding fill_alpha=1.0 to above examples should render some default, but raises an error. This is because _map_color_seg is passed a SpatialImage instead of a Numpy array (seg=labels.values).

Hey @aeisenbarth, does this comment do what you wanted do describe here:
https://github.com/scverse/spatialdata-plot/issues/252#issuecomment-2316559464
?

My request was to specify an outline color per labels element, that is uniform to all label values of that element. It was already possible enable outline without fill, but the outline is always black.
From the comment, I couldn't find anything that solves this issue (two render_labels commands, `na_color="green"``).
I think we would need a separate argument like outline_color that can be set to values other than black.


Add more classNames to customize suggestions related elements

Hey there, awesome library!
It'd be nice to have an option to pass custom classes not only to suggestions wrapper, but also to ul and li elements. We wouldn't have to create an extra .css file just to add those:
.ReactTags__suggestions {
  @apply z-50 absolute mt-2 w-full overflow-hidden rounded-lg bg-white shadow-lg;
}
.ReactTags__suggestions ul {
  @apply max-h-64 overflow-y-auto;
}
.ReactTags__suggestions li {
  @apply cursor-pointer p-2 text-sm text-gray-700 hover:bg-gray-100;
}

I think, it shouldn't be hard to accept more props there, and I could work on a PR, if you like the idea :)
Cheers!

@sitek94 not sure what you mean by extra css file but you will not need an extra .css file to add these, you can add it in the same file where your other CSS is added


habib/add spacing fix login button

Changes:
Issue: Space between "Log in" button and language icon is more than usual in responsive mode.
Step 1: Navigate to https://deriv-com-3j1raadnt.binary.sx/es/partners/
Step 2: From the side menu tap on Affiliates and iB's page
Type of change

[x] Bug fix
[ ] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[x] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

LGTM

Hi @yashim-deriv Made another commit for a similar fix for Tablet. Please have a look.

Closing this as it is no longer relevant.


Persist the activation of virtual environments in CI

Also, don't create a virtual environment just to install Ansible.

Fixes #455.

This should also close #454 and #436 when it gets merged.
Can you explain why we need a virtualenv for installing girder-dandi-archive but not for Ansible?

We want to install Girder in isolation, without any of the clutter / complication from the system Python packages coming with the circleci/python:3.6 image.
Ansible is a straightforward tool to install, and we don't care about what exact library versions are used, so there's no need to have one more line of configuration code to make a virtualenv tool for it.


Consider running multiple instances without overhead

in order to make the daemons HA we could deploy multiple copies of it. though, only one of them should be active at any point in time.
each instance could try to grab a lock in etcd (or use config maps?) in order to elect a leader.
this is also how kubernetes' controller manager and scheduler work.

leader election itself can be easily implemented with: https://github.com/coreos/coreos-kubernetes/blob/fb30f43c4ad01a987ae8b5dcc97ba031d4ff33e9/multi-node/generic/controller-install.sh#L428-L434
doesn't require etcd, uses kube apiserver directly

I suggest this delayed until this PR is merged https://github.com/kubernetes/kubernetes/pull/39173
Currently k8s.io/kubernetes/... packages have internal conflicts with office Go CLI. Upon the merge I expect much easier integration -> easier implementation of leaderelection mechanism.


Add a step to run setup.sh script

It is required to run the setup.sh script to initialize RHDH environment when creating the RHDH instance via the operator.
Fixes FLPATH-1954
Signed-off-by: Moti Asayag<EMAIL_ADDRESS>rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

/lgtm


Reverse SyntaxKind query shows misleading result

TypeScript Version:  2.4.2
Code
console.log(ts.SyntaxKind[8]);

Expected behavior: NumericLiteral
Actual behavior: FirstLiteralToken
Description:
Maybe this is a duplicate of a more broad issue (which I did not find), and I am not sure if there is a workaround for this specific case, but currently I would presume that I can use SyntaxKind enum for reverse lookups of values I got from TS ast node's kind property.
As this enum describes not just unique values but also ranges (with FirstSomething and LastSomething  elements), reversing the enum values is not unambiguous, and currently it gives back the less expected result (FirstLiteralToken) and not the expected one (NumericLiteral).
I think changing the order of the enum items or putting these range items to outside of the enum could solve this issue, but this would break compatibility of course.
Footnote: other tools (for example https://astexplorer.net/) also ran into this mistake and it makes harder to beginners to understand without reading TS's source code what's actually happening here. (I thought for a while that FirstLiteralToken is some kind of special token, but then I empirically found out it only shows this token type for numeric values and just then I looked into TS's source code...)
((I'll report this issue to AstExplorer too.))

See #3103.

We probably (?) should have had "first in wins" semantics for the reverse map, but it'd be a runtime breaking change (extreme risk) to change it now.

Maybe moving FirstLiteralToken-like things to the top of the enum would help?

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Android support for V2 tokens

What are the plans for Android support?
It looks like it should be fairly easy to make the V2 tokens compatible for Android using lazysodium-android. It's also stated in the README that GSON will be supported in the future which would make compatibility easier.

Hey,
I had thought about adding Android support initially but couldn't think of a use case. It should be simple to add another v2 crypto provider which uses lazysodium-android. I'll take a look, as I suspect it should be nearly identical to the already existing lazysodium provider. I suspect v1 will already work on Android.
As for GSON support, it should be pretty easy to add. I have always intended to add support for GSON, just haven't had a reason to look into it yet.

I've opened a pull request with our changes (https://github.com/atholbro/paseto/pull/4).
These changes work fine when importing the code directly into an Android application, but I haven't achieved to make an Android library out of it.

I also started working on this by making a new v2 crypto provider for Android. There are other issues though, as I had initially wrote this library to support only JDK8+, which means that on Android it will only work with API level 26 or higher. I don't want to just replace the standard LazySodium with the Android variant as that will likely break support outside of Android.
If support for lower API levels are needed (and it probably should be), then at least these changes will be required:

Add a way to override the Base64 implementation used internally, as right now the library is using the Base64 support including with Java 8. I already had this, but removed it as the rest of the library requires Java 8 as well. This is pretty easy to solve.
The higher level API currently uses the new Java 8 Date/Time classes. These classes also require API level 26 on Android. I could switch to ThreeTenBP, but that will make usage awkward in any project which actually uses Java 8, which is the primary focus the project. I guess I could split the higher level api off into it's own jar, and provide an alternative which uses ThreeTenBP for Java 7 support.
The Jackson encoder is also dependent upon the Java 8 time APIs so that it can (de)serialize dates & times. This will also be a requirement for the future GSON encoder.

As it stands right now I have the low level Paseto API working on API level 17 with the Base64 change and the Android v2 crypto provider. On API level 26 everything should work, although I've only done minimal testing.
I'm going to work on adding GSON and cleaning it up a little then I'll push out a new release.

Yes, this could be implemented as PasetoAndroid and PasetoJava extending the Paseto interface, like Lazy Sodium is doing.
(Note that PasetoJava could also be used from Clojure, which I have been able to do without problems.)

Seconded. @atholbro are you going to implement this? Android Support would be great!

Yes! I do intend on getting back to this.

Hey all,
Lazysodium maintainer here. I really like the work you're doing here. Let me know if you need help with this 👍
PS: If you want, you can use jCenter() as a remote repository rather than our maven { url  "https://dl.bintray.com/terl/lazysodium-maven" }. Most developers already include jCenter() into their projects so could be easier for developers to integrate into their projects. However, jCenter() does sync slower than our own Bintray repository which can be a big disadvantage for some projects.

What was the issue/confusion you ran into with the KeyPair/Key classes? Maybe I can tweak the README to clear it up.

I don’t think it needs any explanation, it’s just that your PASETO library could use the KeyPair class from LazySodium instead of having another very similar implementation.

Still planning to do this, just been busy.
I'm leaning towards using 310BP internally and changing the Token class to return times as a unix timestamp (long) rather then an OffsetDateTime. Otherwise I'd either need to expose 310BP, which would make interfacing with any java 8 code annoying as you'd have to convert from JSR310 to 310BP (which would be similar to dealing with the timestamps). Or I'd have to split the library even more to have JSR310 and 310BP versions, which was my initial idea but it would also require splitting the json encoders as well, which just ends up becoming crazy when we add GSON into the mix.

Changing the Token class to use unix timestamps would be a great solution.
Would you then have the deserialiser take a formatted string and convert it to unix timestamps?

I'm playing around with this idea in the threeten-abstract branch. It goes a bit further by creating some small abstraction classes around the few Java 8 Time API features the library uses. There are two implementations of these abstractions, one using JSR310 and another using ThreeTenBP. By changing which dependency is imported via gradle, the library can now be switched to work with either JSR310 or ThreeTenBP without further code changes.
The public interfaces now expose times as unix timestamps stored in Longs. The JSON serializer/deserializer now uses these internal abstraction classes, and seamlessly switches between JSR310 / ThreeTenBP without code changes.
Essentially you can now swap between JSR310 / ThreeTenBP by changing the gradle dependency. The meta project pulls in the JSR310 variant, but I can provide a 'paseto-java7' library which pulls in the ThreeTenBP variant. The selecting between the two is just a matter of which you library you use.

I have this mostly working.
V2 public tokens aren't working on Android yet, due to what seems like a bug in LazySodium-Android. I've opened an issue, and when its fixed Android support should be ready for release.
https://github.com/terl/lazysodium-android/issues/19

I finally finished this, the new Android support is contained in a new repository:
https://github.com/atholbro/paseto-android/tree/master
The API is exactly the same as Java, the only 2 changes from the user's point of view are:

Use the Android dependencies.
Setup ThreeTen-ABP if you're not already using it (one line of code in your Application class).


Some documentation fixes

Type

[ ] Bug Fix
[ ] New Feature
[ ] Sample
[x] Documentation

Related Issues?
n/a
What is in this Pull Request ?

Added Connection parameter where missing
Added parameter descriptions where missing
Removed WhatIf parameter description where not present
Adjusted some spelling to follow the convention of https://support.microsoft.com/

Thanks @PowershellScripts , much appreciated this update !


Dev

Coverage remained the same at 100.0% when pulling 8e71cb74487fdfd196e4e7579839ec5b446aef7c on dev into 4fd60f62f6e4d771e8b5565770fa558aa39f478a on master.


Webpack fails when testing extension

Hi, I'm running this on Mac OS X 10.13.6 with Node v10.10.0.
Using the base installation of the template, I ran npm run dev and got a warning:

{ parser: "babylon" } is deprecated; we now treat it as { parser: "babel" }.

After that, making hot changes worked.
However, when:

I loaded the extension as unpacked at: "project/build";
clicked the button in the web browser;
and then clicked the "New tab" button, I got:

internal/streams/legacy.js:57
throw er; // Unhandled stream error in pipe.
^
Error: read ECONNRESET
at TCP.onStreamRead (internal/stream_base_commons.js:111:27)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: webpack --config ./core/webpack.dev.js --hide-modules
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/macbookpro/.npm/_logs/2019-04-19T10_05_16_155Z-debug.log

Where the debug is:

0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/Cellar/node/10.10.0/bin/node',
1 verbose cli   '/usr/local/bin/npm',
1 verbose cli   'run',
1 verbose cli   'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'predev', 'dev', 'postdev' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/path_to_server/project/node_modules/.bin:/Users/macbookpro/bin:/usr/local/bin:/usr/local/mysql/bin:/Users/macbookpro/.composer/vendor/bin:/usr/local/ec2/ec2-api-tools-<IP_ADDRESS>/bin:/usr/local/opt/php@7.1/sbin:/usr/local/opt/php@7.1/bin:/opt/local/bin:/opt/local/sbin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/local/git/bin
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /path_to_server/project
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c',
10 silly lifecycle   'webpack --config ./core/webpack.dev.js --hide-modules' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec dev script
13 verbose stack Error<EMAIL_ADDRESS>dev: webpack --config ./core/webpack.dev.js --hide-modules
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter. (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:301:16)
13 verbose stack     at EventEmitter.emit (events.js:182:13)
13 verbose stack     at ChildProcess. (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:182:13)
13 verbose stack     at maybeClose (internal/child_process.js:962:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:251:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /path_to_server/project
16 verbose Darwin 17.7.0
17 verbose argv "/usr/local/Cellar/node/10.10.0/bin/node" "/usr/local/bin/npm" "run" "dev"
18 verbose node v10.10.0
19 verbose npm  v6.4.1
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>dev: webpack --config ./core/webpack.dev.js --hide-modules
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>dev script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

I'm able to start Webpack up again, but it dies during testing, which is a shame.

I opened the DevTools in Chrome and found a lot of errors.

I got this working dropping by back to node v8.16.0

Interesting, but a downgrade wouldn't be viable for me.


Laravel 5.5 auto-discovery

Laravel 5.5 allows the provider to be auto-discovered using an entry in composer.json. That would be a very nice update for this package.
I'll send a PR after I've played with this package a bit first, if it's something you would like.

Sure send along a PR for this and I'll merge it.

I have a fork with updates for all three issues I've raised. I'm not sure if you would want all three? I have a feeling a config option may be useful to switch the level of detail that is logged. But regular cleanups of the table may be all that is needed.
https://github.com/academe/laravel-queue-monitor

@judgej If you submit separate PRs for each of the issues I'll review them individually.

Okay, then that will have to come later in the week, as unfortunately I didn't do them in separate branches.


Mohit Bharti : Mein Kampf

Book Title?
Mein Kampf
Author?
Adolf Hitler
Genre of Book?
Auto Biography

Hey @themohitbharti Kindly assign this issue to me.

Hey @themohitbharti Kindly assign this issue to me.

Hey @dakshsinghrathore , a PR already exists for this issue. You can raise a PR for any other book. More in Contribution Guidelines

#36


Feature: Initial assortment of refactor changes for review

TBD
(Obviously, do not merge these)

Cannot give huge useful feedback on the refactoring as a whole aside that it looks very promising.
The work on regrouping the different commands in IItemCommandService.cs is a very good idea.
Been only playing with commands around the recycle bin, one can feel there is a lot of convoluted code around creating new commands. Something here begs to be streamlined.
Even if the behaviors are slightly different depending from where we're coming (Sidebar / Former IShellPage for exemple), maybe it'd be possible to create a common base for each command? This refactor gives a small starting ground for it at least.

Looks like my code selection for the review has been a bit messy... Hope you'll be able to make something out of it @lukeblevins.
If some modifications I suggested look good to you but you don't have time for them, hit me up and I'll see what I can do!
First code review for me as well on Git, if you've got any feedback, I'll take it!

I think the #10185 is partially adressed in this refactoring.


Cacuhy , laplace plot the scale value

If you have the validation/test dataset output .

can you upload here
what is the max and the min value of the scale value for cauchy and laplace

biwi_cauchy.csv
biwi_laplace.csv
facedata_cauchy.csv
facedata_laplce.csv
These are the csv files of the test data output for both datasets .


Better Update Handling

Instead of running RobloxPlayerLauncher twice when Roblox updates, it should only run once. roblox_steam_launcher_master should watch for the new RobloxPlayerLauncher.exe and apply the roblox_steam_launcher_substitute changes to it without running the new one.

Updates don't always run properly. Sometimes they crash the roblox exe, and sometimes roblox_steam_launcher does not apply the substitute to the new version.

I have written an experimental change to update handling. Roblox will run once with arguments in order to update and join the game. roblox_steam_launcher will automatically detect when RobloxPlayerLauncher.exe is created and it will apply roblox_steam_launcher_substitute to it.
Possible issues:

Steam might not hook into the game if Roblox is started this way
The new RobloxPlayerLauncher.exe might be used to finish updating its version after it's created. It might not be possible to apply changes in time, or it might not update itself properly if not named RobloxPlayerLauncher.exe
It might not be possible to apply changes if RobloxPlayerLauncher.exe is still being written to by the updater
Roblox may not run properly if it runs through the substitute again, which is what should happen


Improve confidence scoring algorithm

To be updated.

Improve on the current confidence scoring algorithm here. Kaldi's AM and LM scores are scaled and unbounded negative log-likelihoods, we want to compute an aggregate % confidence score similar to that of Google ASR.
Relevant links:

kaldi-help thread about confidence scores


Add HostsDock app

Add HostsDock app to Electron app list.
Thanks!

Thank for adding you app! Could you move it to the bottom of the list please? If you could also get a higher quality icon, without the compression loss, that would be great too ✨

I have a new commit named 'update HostsDock app position and icon' within this pull request. Please check it.
Thanks!

Thanks!


ILoggingBuilder is not part of Logging.Abstractions

Any reason why ILoggingBuilder.cs is not part of the Logging.Abstractions namespace but rather a part of the implementation assembly?

See #679

Dup of #679


Add exercise solutions for "Replacing Matches"

The "Replacing Matches" section is missing from your chapter, and the exercises are likewise missing.
I've found your repository very helpful as I work through the book, and I'm hoping only to help make better what is already the best resource available. Thank you for what you've already shared!

Thanks. Glad you found it useful.


Z~!NEW-VIDEOs]* Jaden Newman Leaks Viral Leaked Video 2024 LINK Viral on Social Media X Today (Trending)

10 sec ago, Jaden Newman, ZFULL Jaden Newman Leaks Video On Social Media X Twitter[2B75K]

Jaden Newman latest video has taken social media by storm, captivating viewers and sparking widespread discussions. This young digital creator has garnered significant attention for her engaging content, particularly with her recent Jaden Newman-themed video that has gone viral. Fans are eagerly searching for where to watch the full video, turning to platforms like Instagram, X (formerly Twitter), and Discord for access.
As the buzz around Jaden Newman grows, fans are actively sharing their reactions online. The viral success of her video highlights the powerful role of social media in propelling creators into the spotlight. With her unique style and creativity, Jaden Newman has quickly become a notable figure in the digital landscape, leaving many eager to learn about her future projects and content.
...
....
....
...
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
...
....
....
...
...
....
...
....
...
[

vbnvbn


Reactivate frontend tests

🔧 (typescript_test.yml): Update job name to "Run Playwright Tests" for clarity and consistency
🔧 (typescript_test.yml): Remove unnecessary 'ref' input from 'actions/checkout' step

LGTM


Methods rename

closes #247

Thanks for the contribution!
When it comes to changing the names, I think that we should deprecate the old names first.
But I don't know much about scala2 macros, so I'm not sure how hard it would be. @adamw

@KacperFKorban ideally - yes - we would want to support both names with a deprecation notice. However, given that our resources are limited, and I'd like to get magnolia 1.0 (for both Scalas) in finite time, I think it would be better to do the renaming so that both versions use the same vocabulary, rather than keeping the status quo. There's still two major changes that should come in for the Scala 2 version (dropping mercator & possibly https://github.com/softwaremill/magnolia/issues/297 - cc @joroKr21 @deusaquilus) before releasing 1.0 anyway.
But of course that's only my opinion, if others would have a different one, that's fine :)

Yeah I think a deprecation cycle might not be feasible, given that it would be just another milestone.


Error message for failed sidechain deposits

If a sidechain deposit fails we need to alert the user and explain why it failed and what they need to do.

Let me add that (unless I've misunderstood) this is specific to this issue here: https://github.com/drivechain-project/docs/blob/master/bip1-hashrate-escrow.md#inconvenient-race-condition

Yes that is correct ^

This looks like this has been fixed in sidechaindepositdialog.cpp L87
https://github.com/drivechain-project/bitcoin/blob/mainchainBMM/src/qt/sidechaindepositdialog.cpp#L87

@ghabs That's true there are actually error messages now. They are very programmer friendly and not so user friendly though. I'll update the name of this issue, as the error messages just need to be made more clear now.


Add Windows deployment (?)

How complicated is the current configuration of the project? If it's not extra-complicated, I suggest having a deploy phase set up (an automatic Windows VM that runs the script to see it at least exists successfully). For deployment, AppVeyor should be a good match.
I'm not sure if you have any prior experience deploying a CI. Some inspiration can be found in CRoCS CI demo: https://github.com/crocs-muni/git-ci-demo

I don't have experience of deploying my own CI. But I have been working with automatic setup of VM's in OpenStack. I prepared a yaml file where I specified what has to be done when the machine starts up for the first time. So I think it should be almost the same, shouldn't it?

Yes, it sounds similar enough.
What I'm not sure about OpenStack: Can you spin up the machine for free
(reasonably)?
The idea of ApoVeyor is to run an automatic (!) test built automatically at
every (or just selected) push. It's also configured via an YML file, I
guess it's pretty similar to OpenStack.
On Wed, 14 Aug 2019, 13:16 filipgontko<EMAIL_ADDRESS>wrote:

I don't have experience of deploying my own CI. But I have been working
with automatic setup of VM's in OpenStack. I prepared a yaml file where I
specified what has to be done when the machine starts up for the first
time. So I think it should be almost the same, shouldn't it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/crocs-muni/tls-warning-collector/issues/17?email_source=notifications&email_token=AAOI6PB56KDK6OL6VDRBZTLQEPSPJA5CNFSM4ILJNMGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4IPHAI#issuecomment-521204609,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOI6PF2PPCOY4KQ2FU3ESTQEPSPJANCNFSM4ILJNMGA
.

OpenStack is paid but I don't think it is the best choice for CI. The AppVeyor seems very reasonable but it is paid as well.

AppVeyor is free for open source projects as far as I know.
I've enabled the build (
https://ci.appveyor.com/project/mathius/tls-warning-collector) and it seems
to work.
Have a look at the yml config, I really think the necessary adjustments
will be minimal (if there are not small, let me know and we can discuss
whether to go for it or not).
Martin.
On Wed, 14 Aug 2019 at 13:29, filipgontko<EMAIL_ADDRESS>wrote:

OpenStack is paid but I don't think it is the best choice for CI. The
AppVeyor seems very reasonable but it is paid as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/crocs-muni/tls-warning-collector/issues/17?email_source=notifications&email_token=AAOI6PDIAG5NDQNX7URD7M3QEPUBTA5CNFSM4ILJNMGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4IQDSA#issuecomment-521208264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOI6PDPWQFOZMEEAOVESQTQEPUBTANCNFSM4ILJNMGA
.

Try just a simple version which will try to install it and run one case and one browser version.

I guess this will not be done? If not, let's close the issue.

This won't be done now. Closing.


error when trying to start Web UI Server in linux

I tried playing with serverport in  Servers-config, but keep getting above error

can you try to run apps with root ? and let me know ?

running as root gives me
Exception in thread "main" java.awt.AWTError: Can't connect to X11 window server using ':10.0' as the value of the DISPLAY variable.
at java.desktop/sun.awt.X11GraphicsEnvironment.initDisplay(Native Method)
at java.desktop/sun.awt.X11GraphicsEnvironment$1.run(X11GraphicsEnvironment.java:99)
at java.base/java.security.AccessController.doPrivileged(AccessController.java:312)
at java.desktop/sun.awt.X11GraphicsEnvironment.(X11GraphicsEnvironment.java:58)
at java.desktop/sun.awt.PlatformGraphicsInfo.createGE(PlatformGraphicsInfo.java:36)
at java.desktop/java.awt.GraphicsEnvironment$LocalGE.createGE(GraphicsEnvironment.java:93)
at java.desktop/java.awt.GraphicsEnvironment$LocalGE.(GraphicsEnvironment.java:84)
at java.desktop/java.awt.GraphicsEnvironment.getLocalGraphicsEnvironment(GraphicsEnvironment.java:106)
at java.desktop/java.awt.Window.initGC(Window.java:477)
at java.desktop/java.awt.Window.init(Window.java:497)
at java.desktop/java.awt.Window.(Window.java:539)
at java.desktop/java.awt.Frame.(Frame.java:423)
at java.desktop/java.awt.Frame.(Frame.java:388)
at java.desktop/javax.swing.SwingUtilities$SharedOwnerFrame.(SwingUtilities.java:1919)
at java.desktop/javax.swing.SwingUtilities.getSharedOwnerFrame(SwingUtilities.java:1995)
at java.desktop/javax.swing.JWindow.(JWindow.java:183)
at java.desktop/javax.swing.JWindow.(JWindow.java:135)
at org.magic.gui.components.dialog.MTGSplashScreen.(MTGSplashScreen.java:30)
at org.magic.api.main.MtgDesktopCompanion.(MtgDesktopCompanion.java:36)
at org.magic.api.main.MtgDesktopCompanion.main(MtgDesktopCompanion.java:30)

Standard linux user can't open ports below 1024. Try to put 1025 then go to http://localhost:1025.

maybe you should restart app

that worked. Thanks!

Where is Servers-config?


Image not centered

The image on the index page does not appear to be centered when changing screen resolution or resizing the browser.

bug fixed in commit ce131f0a7109f0fcc1d3f5f7f5761489dbd3725a


How to conduct inductive experiments?

Would you mind giving an example of inductive node classification on PPI?

@Jhy1993 Hi,  have you soled this question? I also want to ask this.

嗨，你解决了这个问题吗？我也想问这个问题。

Hello， have you soled this question?


Proposal: Enable the JSON extension by default

This is not an issue, but a proposal to have the json extension loaded by default into the driver.
I'm currently developing a framework/library that requires the go-sqlite3 with the json extension loaded, and while you could build it with -tags json1, this also requires the consumers of my library to pass the same build tags when building their project which results in a poor dev experience.
This topic was brought several times in the past (ex. #710, #832 and https://github.com/golang/go/issues/43288) and since the json extension is frequently used, I want to suggest enabling it by default. As Russ Cox mentioned in the related go issue, I don't think this will increase the size significantly and will result in better out-of-the-box dev experience when using the driver.

Actually, after reading the release notes of SQLite 3.38.0, it seems that the JSON functions are already built-ins and the DSQLITE_ENABLE_JSON1 flag is a no-op.


What is the license?

Can you add license to this repo? I am interested in contributing to it and using it on my own project. To have license clearly stated would be necessary.
Thanks.

Sure, added a license.


Add a macros entry to the markdown-math configuration section

It would be nice to have a macros entry in the configuration section of markdown-math extension. A place where users could put their custom math macros.

This bug has been fixed in the latest release of VS Code Insiders!
@AlbertHilb, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version b4952d14a465572e33f0635ef58de089227b8876 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

/verified


[BUG] Stand Alone detail page has no Nav Bar

Description of the Bug
when the StandAlone detail is enabled, the page of the detail page does not have a NavBar
How To Reproduce
Go to settings, go on display, pick a table, turn on stand alone detail, on the list view you can now click on a row of that table and the detail page will table your whole screen. Now you can see there is no Navbar
Expected Behavior
The stand alone detail page needs the NavBar
Actual Behavior
The stand alone detail page has no NavBar
Priority
Low

video for Stakeholder is in #219's video

Signing off - fixed in 219


Sampling function raises error instead of return samples

In nomal.py line 68, the function raises something instead of returning it. Change raise for return.
Thanks

Thank you, I fixed it and published a new version


how does standalone connect to DAW?

Hi,
I've just installed this, thought i'd be able to spit out a MIDI file from the standalone app, now trying and failing to see how to connect to DAW... BTW my file is drums only... could this be why nothing is showing up?
Are there any instructions available?
Tx
Mark

Hi,
You should be able to export a MIDI file from the standalone app to any DAW, or any other app that accepts external file drag (for example File explorer or Finder). To do this, simply click and drag from the Drag the midi file from here banner to where you want to move the file (a midi track on your DAW for example).
NeuralNote does not support drums/percussion transcriptions, so yes that might be the reason why you don't get any notes in the transcription.
Let me know if it still does not work for tonal instrument or a voice samples.
Hope this helps!

Hi, thanks for letting me know... I did try an mp3 with audio just to have
a lookie see and it does indeed work... I'd used Google Gemini this morning
to ask what addons it could suggest to convert a drum pattern wav to midi
and it suggested neuralnote... be well, and thanks again for explaining.
I'm pasting an image here, not sure if it will publish:
[image: image.png]
Mark.
On Tue, 27 Feb 2024 at 16:00, Damien Ronssin @.***>
wrote:

Hi,
You should be able to export a MIDI file from the standalone app to any
DAW, or any other app that accepts external file drag (for example File
explorer or Finder). To do this, simply click and drag from the Drag the
midi file from here banner to where you want to move the file (a midi
track on your DAW for example).
NeuralNote does not support drums/percussion transcriptions, so yes that
might be the reason why you don't get any notes in the transcription.
Let me know if it still does not work for tonal instrument or a voice
samples.
Hope this helps!
—
Reply to this email directly, view it on GitHub
https://github.com/DamRsn/NeuralNote/issues/93#issuecomment-1966891181,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKO4LB664HPCB4XF6GCQTLYVX7KXAVCNFSM6AAAAABD4LQ4I6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRWHA4TCMJYGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

here is what Gemini said:


Could not open edgetpu.tflite

I have generated my own object detection model. For that I have used pre quantized 'ssd_mobilenet_v1_quantized_300x300_coco14_sync_2018_07_18' model.  I have used TensorFlow 1.15 version, I haven't converted the model using Bazel and TOCO. I have quantized the model before converting it to the edgetpu model because it is already a quantized, model.
I have converted the frozen graph model using:
tflite_convert \
--graph_def_file=tflite/tflite_graph.pb \
 --output_file=tflite/detect.tflite \
 --output_format=TFLITE \
 --input_shapes=1,300,300,3 \
 --input_arrays=normalized_input_image_tensor \
 --     output_arrays='TFLite_Detection_PostProcess','TFLite_Detection_PostProcess:1','TFLite_Detection_PostProcess:2','TFLite_Detection_PostProcess:3' \
 --inference_type=QUANTIZED_UINT8 \
 --mean_values=128 \
 --std_dev_values=127 \
 --change_concat_input_ranges=false \
 --allow_custom_ops

Then I have converted that model into edge tpu using:
! edgetpu_compiler "/content/drive/My Drive/detect (1).tflite"

The  compilation message:
 Edge TPU Compiler version 2.1.302470888

 Model compiled successfully in 568 ms.

 Input model: /content/drive/My Drive/detect (1).tflite
 Input size: 5.34MiB
 Output model: detect (1)_edgetpu.tflite
 Output size: 5.75MiB
 On-chip memory used for caching model parameters: 5.66MiB
 On-chip memory remaining for caching model parameters: 1.96MiB
 Off-chip memory used for streaming uncached model parameters: 0.00B
 Number of Edge TPU subgraphs: 1
 Total number of operations: 64
 Operation log: detect (1)_edgetpu.log

 Model successfully compiled but not all operations are supported by the Edge TPU. A percentage of the model will instead run on the CPU, which is slower. If possible, consider updating your model to use only operations supported by the Edge TPU. For details, visit g.co/coral/model-reqs.
 Number of operations that will run on Edge TPU: 63
 Number of operations that will run on CPU: 1
 See the operation log file for individual operation details.

When I am running the example script I got from 'https://github.com/EdjeElectronics/TensorFlow-Lite-Object-Detection-on-Android-and-Raspberry-Pi/blob/master/TFLite_detection_video.py'
It gives me this kind of error.
python3 detection_picam.py --model /home/pi/ detection_model/detection_edgetpu.tflite --labels /home/pi /detection_model/labelmap.txt --edgetpu
Traceback (most recent call last):
   File "detection_picam.py", line 135, in <module>
     experimental_delegates=[load_delegate('libedgetpu.so.1.0')])
  File "/home/pi/.local/lib/python3.7/site-packages/tflite_runtime/interpreter.py", line 204, in __init__
    model_path, self._custom_op_registerers))
ValueError: Could not open '/home/pi/detection_model/detection_edgetpu.tflite/edgetpu.tflite'.

Is there any problem with the detection_edgetpu.tflite model? What can be the solution here?

@MevadaRavikumar,
In order to expedite the trouble-shooting process, could you please provide the detection_picam.py file, .pb, .tflite and all other supporting files required to reproduce the issue reported here. Thanks!


Controller Inheritance support

Hi there! It would be nice to support controller inheritance, which can allow create (for example) a CrudController with CRUD methods for a REST API, calling a service to fetch data, and then inherit from that controller for each specific entity type

Prefer composition over inheritance - put your CRUD methods logic into generic services and inject it to the controller.

Yes @19majkel94, is what I'm doing, but I have to repeat the route handlers on every controller. Not a big deal, but a lot of repeated code.

Good point @jselesan

CrudController with CRUD methods for a REST API, calling a service to fetch data, and then inherit from that controller for each specific entity type
I have to repeat the route handlers

So you want something like a generic controller? Or can you show me your current use case and how this feature should looks&works 😉

I think something like this (may not compile ;))
export abstract class CrudController {
  private _service: CrudService;
  protected setService(service: CrudService) {
    this._service = service;
  }

 @Get("/")
 all() {
   return this.service.getAll();
 }

 /* the other methods are similar */
}

@JsonController("/employees")
export class EmployeesController extends CrudController {
  constructor(private service: EmployeesService) { //EmployeesService should extend CrudService
    super()
    this.setService(service);
  }
}

I'm not sure if it is possible, is just an idea :)

@jselesan code you provided does not look good even if it was implementable. In my experience inheritance in controllers never bring somethings really valuable, its only makes things complex and unpredictable. Controllers should be simple as hell and if you have duplication in logic - simply extract this logic into services - thats correct places for such logic. If you have duplication in decorators and routes - thats not an issue, having duplicated route decorators just makes things more explicit. Really really no need to make things complex (and make life harder for everyone except you) with such patterns.

If you have duplication in decorators and routes - thats not an issue, having duplicated route decorators just makes things more explicit.

+1000 😉
I also have something like @Get("/")  all() {    return this.service.getAll();  }
in almost every controller but it's not a big deal if you have repeated logic extracted into services.
@jselesan
Imagine that you use the extending pattern to reduce boilerplate. So in some time you can get the inheritance chain that consist of 10 extending controllers and there are difference for every controller. So instead of look into controller mapped to REST API resource path and see what's happening there (like reading a doc) you have to traverse all the way up to see all actions, etc. Good luck with debugging 😉
For easier development I recomend you to create your own template or even snippet for IDE so you can just type the service name and type for import and inject, controller name or path and the base thing will be configured in 30s 😃

Feel free to continue discussion

+1

Actually this makes sense in a bit more complex implementation like:
import Bluebird from 'bluebird'
import { Request } from 'express'
import { Get, Post, Put, Delete, Body, Param, Req, OnNull, NotFoundError } from 'routing-controllers'
import ResourceRepositoryService from '../services/ResourceRepositoryService'
import { ResourceNotFoundError } from '../errors'
import Dto, { DtoClassType } from './Dto'

export type ResourceRepositoryServiceClass<
  TService extends ResourceRepositoryService<TEntity, TId, TContent>,
  TEntity = {},
  TId = {},
  TContent = {}
> = Function & { prototype: TService }

export default function ResourceController<
  TDto extends Dto<TDto, TObject>,
  TContentDto extends Dto<TContentDto, TContentObject>,
  TService extends ResourceRepositoryService<TEntity, TId, TContent>,
  TEntity = {},
  TId = {},
  TContent = {},
  TObject = {},
  TContentObject = {}
> (
  resourceServiceIdentifier: ResourceRepositoryServiceClass<TService>,
  dtoClass: DtoClassType<TDto>,
  contentDtoClass: DtoClassType<TContentDto>
) {
  abstract class ResourceController<
    TController
  > {
    protected readonly packDto = Dto.pack.bind(dtoClass)
    protected readonly unpackDto = Dto.unpack.bind(dtoClass)
    protected readonly packContentDto = Dto.pack.bind(contentDtoClass)
    protected readonly unpackContentDto = Dto.unpack.bind(contentDtoClass)

    constructor (
      protected readonly resourceService: TService
    ) {}

    @Post()
    public async create (
      @Body({ required: true }) data: TContentDto
    ): Promise<TDto> {
      const resource = await this.resourceService.create(this.unpackContentDto(data))
      return this.packDto(resource)
    }

    @Get('/:id')
    @OnNull(404)
    public async get (
      @Param('id') id: number
    ): Promise<TDto | null> {
      const resource = await this.resourceService.get(id as any)
      if (!resource) {
        return null
      }

      return this.packDto(resource)
    }

    @Put('/:id')
    public async update (
      @Param('id') id: number,
      @Body({ required: true }) data: TContentDto
    ): Promise<TDto> {
      try {
        const resource = await this.resourceService.update(id as any, this.unpackContentDto(data))
        return this.packDto(resource)
      } catch (err) {
        if (err instanceof ResourceNotFoundError) {
          throw new NotFoundError(err.message)
        }
        throw err
      }
    }

    @Delete('/:id')
    public async delete (
      @Param('id') id: number
    ): Promise<void> {
      try {
        await this.resourceService.delete(id as any)
      } catch (err) {
        if (err instanceof ResourceNotFoundError) {
          throw new NotFoundError(err.message)
        }
        throw err
      }
    }

    @Get()
    public async query (
      @Req() request: Request
    ): Promise<TDto[]> {
      const resources = this.resourceService.query()

      return Bluebird.map(resources, resource => this.packDto(resource))
    }

    @Get('/count')
    public async count (
      @Req() request: Request
    ): Promise<number> {
      return this.resourceService.count()
    }
  }

  return ResourceController
}

My suggestion would be to add an extends: Function[] options to the Controller and JsonController, and check that in MetadataBuilder.createControllers to assemble actions from extended controllers as well

@19majkel94 any hints on how this may be implemented in routing-controllers? :)

@alexey-pelykh Not really, I don't know this codebase very well so it's hard for me to dig in and implement this. Also as pleerock is against, it might not be a good idea 😜

@19majkel94 I wonder what changed your mind!

Dear fellows, @alexey-pelykh , @pleerock ,
Is this abstract controller functional?
Sorry to repoen the issue, I was wondering how to base controllers, services and repository for a plenty similar CRUD.
@alexey-pelykh example seems very interesting as it aggregates a lot of work that is repeatable for cruds.
Where can I find more information or samples, would you provide @alexey-pelykh .
Appreciate your work

@lixaotec it wasn't last time I visited the feature, I ended up having MyEntityController + MyEntityCrudController

so, is there a way ？ i also meet this problem.
BaseControll
import { Model } from 'sequelize-typescript';
import { AnyObject } from '../config/global';
import Repository from "./Repository";

export default class Controller {
    constructor(public repository: Repository) {
      this.repository = repository;
    }

    async all(body: object) {
      const res = await this.repository.findAll(body);
      return res;
    }

    async findById(id: number) {
      const res = await this.repository.findOne(id);
      return res;
    }

    async one(body: AnyObject) {
      const { id = -1 } = body;
      if (typeof +id !== 'number' || id < 0) {
        return {
          status: 'error',
          code: 1001,
          message: 'id无效'
        };
      }
      const res = await this.repository.findOne(+id);
      return res;
    }

    async save(body: Model) {
      await this.repository.save(body);
      return { message: 'success' };
    }

    async update(body: AnyObject) {
      await this.repository.update(body);
      return { message: 'success' };
    }
}

ServiceController
import { JsonController, Post, Body } from "routing-controllers";
import { Service } from "typedi";
import Controller from '../Controller';
import { AnyObject } from '../../config/global';
import Model from "./model";
import ServiceRepository from "./repository";

@Service()
@JsonController('/service')
export default class RuleController extends Controller {
  constructor(repository: ServiceRepository) {
    super(repository);
  }
  @Post("/query")
  async all(@Body() body: object) {
    return super.all(body);
  }

  @Post("/detail")
  async one(@Body() body: AnyObject) {
    return super.one(body);
  }

  @Post("/save")
  async save(@Body() body: Model) {
    return super.save(body);
  }

  @Post("/update")
  async update(@Body() body: AnyObject) {
    return super.update(body);
  }
}

I must repeat the code in every service


Remove completely unresolvable interfaces

Professor,
This PR removes interfaces that cannot be resolved for successful compilation  These are interfaces from the java packages. This issue only happens if we isolate the input file from its codebase. Please take a look. Thank you.

Professor,
Following our discussion during the meeting, I have added a comment so that future maintainers are aware of the current limitation of this approach. Please take a look. Thank you.


available for ARM?

Just judging by the readme, it reads "suitable for X86..." or "suitable for X86_64...". What about ARM processors, which is recommended?
The xxhash github for the C implementation says it's highly portable, so thought I could run this on ARM.

@evbo It only talks about hardware accelerations.
If there is no hardware acceleration (I didn't implement NEON instructions yet) then it fallbacks to default scalar implementation.
So it should work without issue on any 32bit processor


Changing FTP recalulates intensity of historical efforts

Bug description:
When changing FTP in the extension, it reports you need to resync. When this is done the relative intensity is changed of historical efforts. This is poor, because those efforts were at that quoted FTP and not necessarily of the new one.
Actual Behavior:
Changing FTP causes sync of history, and recalculates all previous workout intensity
Expected Behavior:
Changing FTP does not recalculate prior workouts/rides
Steps to Reproduce:

Enter an FTP
Upload a ride, note intensity
Change FTP, perform sync
view old activity and see new intensity factor

Chrome version
60.0.3112.113
Plugin version:
5.10.1
Activities links?:
Console errors? (press F12 to see developer console, and copy paste here):
put console errors here if exist

Link screenshots or youtube video link if necessary:

See https://github.com/thomaschampagne/stravistix/issues/450#issuecomment-338066266

@philipwhiuk +1
Duplicate of https://github.com/thomaschampagne/stravistix/issues/456


Unclear language content

This samples lack clarity which language is being used for the samples provided.
I suggest to ad a magic in the code like %%scala for example to clarify this.

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

ID: 65331fde-51d0-365e-e182-2c3b01de8f99
Version Independent ID: 79b4f02b-27a5-019a-1b36-4e80a04f539f
Content: Quickstart: Get started analyzing with Spark - Azure Synapse Analytics
Content Source: articles/synapse-analytics/get-started-analyze-spark.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @saveenr
Microsoft Alias: saveenr

Thanks for the feedback! I have assigned the issue to the doc author to update appropriately.

Thank's @SamaraSoucy-MSFT

Hi @ralphke, Thank you for contacting Microsoft. The issue has been resolved in the article. I will go ahead with closing this issue. The @notification mechanism continues to work even after an item is closed, in case further communication is needed. We hope to hear from you again.

#assign:julieMSFT

#please-close


[Open Terminal] is not working for a local Ubuntu machine with Fish shell

Describe the bug
I can not Open Termianl for local Ubuntu machine
To Reproduce

Set up your default shell to Fish
Create a Ubuntu machine
Click [Open Terminal]

Expected behavior
It should open Fish or Bash and then connect to Ubuntu machine.
But it open Fish and did not connect to Ubuntu machine.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.5.1
Commit: 4cfac15e1080617c70eb163966e1cb2009dac1c2 (v1.5.1)
System info:
macOS: 14.1 (23B2073)
CPU: arm64, 11 cores
CPU model: Apple M3 Pro
Model: Mac15,6
Memory: 18 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-04-03T01-30-32.778556Z.zip

Screenshots and additional context (optional)
No response

Can't reproduce. Please share a screen recording.

https://github.com/orbstack/orbstack/assets/9758711/4220c41c-429b-4408-b589-652b8818d0a1
@kdrag0n  If you need more information, please let me known


Add isinstance behavior

This should probably work and do the obvious thing:
if isinstance(x, A & B)

since it works for unions.

As special cases, it might be worth adding what happens with empty intersections / unions:

empty Union must alias to Never, hence testing empty Union must always return False

[x] isinstance(x, ()) currently always returns False
[ ] isinstance(x, Union) currently unsupported
[ ] Union[()] currently unsupported

empty Intersection must alias to Any, hence testing empty Intersection must always return True

if simplification is implemented (https://github.com/CarliJoy/intersection_examples/issues/5) then

I suggest rethinking to allow Union and Intersection to accept no arguments, and return the appropriate Never or Any. (or Union[Never] and Intersection[Any]): https://github.com/python/typing/issues/1362

Union[] is already discussed in a different thread.  Since Union is deprecated, it's simply going to go away on its own.  There's no need to bring it up in this PEP.
Similarly, empty intersection doesn't need to work or be mentioned in my opinion.  When Intersection disappears in a few Python versions, you won't be able to write an empty intersection.

@NeilGirdhar The same reasoning as in https://github.com/CarliJoy/intersection_examples/issues/12#issuecomment-1657069873 applies as to why the actual types Union and Intersection can't disappear.

I guess if you want this, you should convince people in this thread: https://github.com/python/typing/issues/1362?  Anyway, I think this should have been a separate issue.

@randolf-scholz

empty Intersection must alias to Any

I believe that this should alias to object and don't really want to rehash all the stuff from #1
Can we agree on this simply, or should we wait to see a resolution on #1 ?
Note: The reason An empty Union is Never is the same reason why Never is removable from a Union with other types

Yes, it should have been object.

Note: The reason An empty Union is Never is the same reason why Never is removable from a Union with other types

This is right.  It's also a consequence of a property of subtypes of union types.  A type is a subtype of a union type if it is a subtype of at least one component of the union.  Trivially, there is no type that is a subtype of at least one component of an empty union type.  Therefore, an empty union type is a type that has no subtypes.
Dually for intersections (though it might seem a bit weird): a type is a subtype of an intersection type if it is a subtype of all the components of the intersection.  Trivially, every type is a subtype of all the (none) components of an empty intersection type.  Therefore, an empty intersection type is a type that is a supertype of every type.
But note that this is not necessarily object.  There can be more than one top type.

My opinion is that raw Intersection is nonsense.  If Intersection were a generic type, then this would be implicitly instantiated somehow (Intersection[Any]?).
But Intersection is not a generic type, it's just some special form and there's no reason that raw Intersection has to mean anything at all (except a static error).
We would thus avoid the confusion between programmers who expect that Intersection is a top type wrt. subtyping and programmers who think it's like a raw generic and it's Intersection[Any].  We avoid saying whether it is a distinct type from object or not.  We don't need it, because we already have object.

@kmillikin Here's an example for empty intersection:
from typing import TypeVarTuple, Intersection

protocols = TypeVarTuple("protocols")
# note: bounds currently not supported, but what is meant is
# TypeVarTuple("protocols", bound=type[Protocol])

def create_intersection_protocol(protos: tuple[*protocols]) -> Intersection[*protocols]]: ...
    """Synthesizes the intersection-class of pairwise disjoint protocol classes."""
    # if intersection is empty, returns a protocol that matches with any class.

EMPTY_TUPLE = ()
create_intersection_protocol(EMPTY_TUPLE)  # what is the type?

I think the revealed type should be either object, or the unique empty Protocol "Top" from my last comment.

@randolf-scholz I don't understand your example at all, but I think it's irrelevant to the question of what raw Intersection is, since it doesn't use raw Intersection.

It's an interesting observation that protocols allow arbitrarily many distinct user-defined top types.  That's probably a bit of a mess for Python, but I think an intersection PEP can be silent on these.

@kmillikin The function create_intersection_protocol should synthesize the intersection of disjoint protocol classes. For example
merge_protocol(Hashble, Sized) should return a Protocol like
class HashableAndSized(Protocol):
    def __len__(self): ...
    def __hash_(self): ...

Which is essentially equivalent to Hashable & Sized.

Still seems irrelevant.

I think the question is simply whether bare Union and Intersection should be considered equivalent to Union[()] and Intersection[()]. (as a shorthand, since these look quite awkward)
I know that Union[()] currently raises an error, but imo it shouldn't. In the example I gave, the question is what is reveal_type(merge_protocols(()))? It should be Intersection[()] without simplification, and object or the suggested typing.Top empty-Protocol class with simplification. A similar example for Union can be made that shows how Union[()] can pop up naturally.

Can we have this discussion in https://github.com/CarliJoy/intersection_examples/issues/20 where other people can find it?


Timers

Is there a way to add a timer functionality to AlexaPi? Amazon has a page detailing how it should work but I don't have enough expience to implement it. https://developer.amazon.com/public/solutions/alexa/alexa-voice-service/reference/timers-and-alarms-conceptual-overview

To do this, we'd need to switch over to the new API version, which we'll do in the new upstream: https://github.com/alexa-pi/AlexaPi ... some people are already looking at it.
Not relevant for this repo -> closing.


make use of the new references API

In the all modalities codelab, we make use of reference BSgenome.Hsapiens.UCSC.hg19.  It takes several minutes to install via biocLite("(BSgenome.Hsapiens.UCSC.hg19") since its rather large.  (But of course this only happens once.  And all regular users of BioConductor will likely have already installed this.)
It would be interesting to make use of the new references API instead at this point in the codelab.

This repository has been simplified to focus on the BigQuery queries for variant quality control.


[WFLY-14625]: WARN message jaegertracing: FlushCommand execution failed!

replacing Jaeger ThriftSenderFactory with our own to swallow
exceptions thrown by the UdpSender which were swallowed before.
some module.xml magic for the ServiceLoader to load our
implementation.

Jira: https://issues.redhat.com/browse/WFLY-14625

@ehsavoie never mind about my question about whether this is an old experiment -- I see now this PR is only 1 day old!

@bstansberry there were 4 approaches:

use the logging subsystem to disable this trace :-1:
ask jaeger client for a change (PR submitted) but they refused saying this was a logging configuration issue :-1:
change our default config not to provide opentracing: this is not backward compatible :-1:
this fix

@ehsavoie Thanks. Please note the comment about the WARN log in the WildFlySender constructor.

One thing I'm still not completely clear on is why, I'm guessing it's the client itself not WildFly, it just attempts to start sending data even if no server is there to receive the data. That was my only point for not adding the subsystem by default. If there is no server to receive the data what's the point in attempting to send it?
That said I don't really know much about the spec or implementation so I could misunderstand some things :)

@jamezp I think it's legit that it tries to send the data and logs if that fails. It's configured to send and that doesn't work.  Repeatedly logging isn't great, though.
Changing our default configs to not have the sender is the more correct solution, really, but it's an incompatible change.

@jasondlee WDYT?

@bstansberry Apologies. I thought this was closed. I think the change sounds reasonable. They're not willing to change on their end, and the "right" fix isn't backwards compatible, so we're left with this.
+1