GON for large images (64x64)?

Thanks for the great work.
I try your GON with additional layers to generate large images from CeleA at 64x64 resolution.
However, the results are blurred and distorted.
I have tried adding a smooth Conv layer or Residual Conv layers after the ConvTrans, but the problem persists.
Could you kindly share your GON for large images as used for Figure 9 in your paper? Thank you.

Simply increasing the latent dim does the trick. 😄
Smooth Conv and Residual Conv contribute slightly to the performance.


Split frontier logic into Frontiers & Frontier

We also agreed to move frontier slot logic to a separate entity in a similar way.
Please, review.

@chekunkov Ready for next review, I also added integration tests for frontier logic and added new cassettes to #40.

After we merge this PR we will probably need to split client.py into smaller modules - 1.3k lines is a bit too much

@chekunkov I think that's it, take a look pls when you have time.


Error when turning tv off

Getting this error with turning off tv:
Home Assistant Core 0.110.7
My configs:
- platform: samsungtv_tizen
  name: Main TV
  host: <IP_ADDRESS>
  port: 8002
  update_method: "smartthings"
  mac: "d0:d0:03:04:eb:84"
  api_key: "<key>"
  device_id: "<id>"
  source_list: '{"TV": "KEY_TV", "Apple Tv": "ST_HDMI1", "Xbox": "ST_HDMI2"}'
  app_list: '{"YouTube": "111299001912"}'

- platform: samsungtv_tizen
  name: Bedroom TV
  host: <IP_ADDRESS>
  port: 8002
  mac: "00:7c:2d:d0:3e:e7"
  api_key: "<key>"
  device_id: "<id>"
  source_list: '{"TV": "KEY_TV", "HDMI 1": "ST_HDMI1", "HDMI 2": "ST_HDMI2"}'
  app_list: '{"Youtube Kids": "5SLwh7m3En.TizenYouTubeKids","RedBull TV": "4OAwqypfRT.RedBullTV","Showmax": "Nm4zJjYmVG.Shomwax","DSTV Now": "CXNLCPYe82.Dstvnow","Emby": "vYmY3ACVaa.emby","Netflix": "11101200001", "YouTube": "111299001912"}'

Logger: homeassistant.components.websocket_api.http.connection.2852310352
Source: custom_components/samsungtv_tizen/media_player.py:599
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 8:29:26 AM (3 occurrences)
Last logged: 8:29:36 AM
'SamsungTVDevice' object has no attribute '_auto_gen_installed_app_list'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 130, in handle_call_service
connection.context(msg),
File "/usr/src/homeassistant/homeassistant/core.py", line 1253, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1288, in _execute_service
await handler.func(service_call)
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 213, in handle_service
self._platforms.values(), func, call, required_features
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 454, in entity_service_call
future.result()  # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 597, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 485, in _handle_entity_call
await result
File "/usr/src/homeassistant/homeassistant/components/media_player/init.py", line 530, in async_turn_off
await self.hass.async_add_job(self.turn_off)
File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/samsungtv_tizen/media_player.py", line 599, in turn_off
if self._auto_gen_installed_app_list == True:
AttributeError: 'SamsungTVDevice' object has no attribute '_auto_gen_installed_app_list'

set app_list manually as described in the readme

@jaruba As in my example configs above?

oh, i see the issue now, the code will need to be fixed regarding this, thanks for reporting

I've made a new release, it's best to uninstall the old version and install the new one to avoid issues.


[BUG] GPT-J + init_inference + replace_with_kernel_inject returns copy error with multiple GPUs

Describe the bug
Using the replace_with_kernel_inject option in init_inference returns an error when using multiple GPUs (with a GPT-J model).
To Reproduce
Steps to reproduce the behavior:

Create an inference script using HF Transformers and GPT-J
Run the deepspeed command with multiple GPUs

import os
local_rank = int(os.getenv('LOCAL_RANK', '0'))
world_size = int(os.getenv('WORLD_SIZE', '1'))

import torch
from transformers import AutoTokenizer, AutoModelForCausalLM
import deepspeed
from transformers import pipeline as t_pipeline

model = AutoModelForCausalLM.from_pretrained("EleutherAI/gpt-j-6B", torch_dtype=torch.float16)
tokenizer = AutoTokenizer.from_pretrained("EleutherAI/gpt-j-6B")
generator = t_pipeline('text-generation', model=model, tokenizer=tokenizer, eos_token_id=50256,  device=local_rank)

generator.model = deepspeed.init_inference(generator.model,
                                            mp_size=world_size,
                                            dtype=torch.float16,
                                            replace_method= 'auto',
                                            replace_with_kernel_inject= True
                                        )

input_list = ["This is the input "]

res_ds = generator(input_list, do_sample=True, max_length = 1000, eos_token_id=50256, temperature=0.25, pad_token_id=50257)

Expected behavior
No error.
ds_report output
Unavailable, not currently in the compute node.
Screenshots

System info (please complete the following information):

OS: Linux - Ubuntu
One machine with 8x A100 40gb PCIE
Python 3.8
Using the following docker image: pytorch/pytorch:1.9.1-cuda11.1-cudnn8-devel

Launcher context
Deepspeed command line
Docker context
Base image is: pytorch/pytorch:1.9.1-cuda11.1-cudnn8-devel
Additional context

The problem does not exist when replace_with_kernel_inject is set to False
Things work fine with replace_with_kernel_inject = True and running the script directly with a single GPU.
The error appears to come from here:
https://github.com/microsoft/DeepSpeed/blob/master/deepspeed/module_inject/replace_module.py#L74

In addition, I was able to replicate the issue on a different box with Fedora and 8x a6000 GPUs.

Hi @TiesdeKok,
I will take a look at this.
Thanks,
Reza

Hi @TiesdeKok,
I will take a look at this.
Thanks,
Reza

Thanks a lot!

Hi @TiesdeKok
Can you please try this PR and see if this is fixed?
Thanks.

Appreciate the quick turnaround here @RezaYazdaniAminabadi!
The copy error is gone and the inference starts now, so that appears resolved. 🥳
However, I am running into another problem where everything works great with one GPU, however, with multiple GPUs, the inference will hang indefinitely. I can make a separate issue if you prefer, but let me describe what I am observing:

"EleutherAI/gpt-j-6B" with float16 with one GPU without kernel inject --> works
"EleutherAI/gpt-j-6B" with float16 with one GPU with kernel inject --> works
"EleutherAI/gpt-j-6B" with float16 with 2+ GPU without kernel inject --> hangs indefinitely
"EleutherAI/gpt-j-6B" with float16 with 2+ GPU with kernel inject --> hangs indefinitely

No errors are shown, it just pins the GPUs at 100% and nothing happens. I have tried this on two different machines and the behavior is the same. I noticed the same issue already yesterday without the kernel inject and letting it run for hours (on one prompt) clearly indicates that things are stuck.
To dig into this further, I have also tried using the distilgpt2 model, the same issue pops up:

"distilgpt2" with float 16 with one GPU --> works
"distilgpt2" with float 16 with 2+ GPU --> hangs indefinitely
"distilgpt2" with float 32 with one GPU --> works
"distilgpt2" with float 32 with 2+ GPU --> hangs indefinitely

I am a little lost here, the code I am running is essentially the same as:
https://github.com/microsoft/DeepSpeedExamples/blob/fix-inferen-test/inference/huggingface/gpt-neo.py
Which I run with deepspeed --num_gpus X gpt-neo.py (pseudo-code).
I tried looking for a verbose option to see if I could get better logging once things are on the GPUs, however, I could not find it. Any ideas on what might be happening here? 😕

@TiesdeKok,
This is a known issue that we have with the integration of DeepSpeed Inference and HF. This is happening since one GPU is finished with the generation while the other one is waiting to continue for the next token-generation. Would you mind setting the min_length and max_length to the same number and see if this issue is resolved?
Thanks

After reading your description it immediately hit me that the hanging issue is caused by a random.shuffle() line in my code, that created a different input for every machine and caused it all to hang. 🤦🏻♂
With that out of the way, I am now seeing weird behavior with the kernel inject:

multiple GPUs without kernel inject works great.
multiple GPus with kernel inject and do_sample=False completes without errors but it generates garbage output. The output looks like this (\n####\n is where my prompt ends):

\n####\n!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"

multiple GPus with kernel inject and do_sample=True throws an error:

I added a quick print statement right before that torch.multinomial() step and it shows:
print(probs)
#tensor([[nan, nan, nan,  ..., nan, nan, nan]], device='cuda:1')
#tensor([[nan, nan, nan,  ..., nan, nan, nan]], device='cuda:2')
#tensor([[nan, nan, nan,  ..., nan, nan, nan]], device='cuda:0')
#tensor([[nan, nan, nan,  ..., nan, nan, nan]], device='cuda:3')

The above issue and error also occur when settings max_lenght and min_length  to the same value.
Any thoughts on what might be the issue here? Thanks again for your help!
Ps. my torch version is 1.10.0 and my transformers version is 4.16.0.dev0

I did test this on the same versions as you mentioned. Just that I am using PyTorch1.9. The code snippet I am using is as follows:
import os
import torch
import transformers

from deepspeed import module_inject
from transformers import pipeline

local_rank = int(os.getenv('LOCAL_RANK', '0'))
world_size = int(os.getenv('WORLD_SIZE', '1'))

generator = pipeline('text-generation',
                     model='EleutherAI/gpt-j-6B',
                     device=local_rank,
                     )
generator.model = deepspeed.init_inference(generator.model,
                                           mp_size=world_size,
                                           dtype=torch.half,
                                           replace_method='auto',
                                           replace_with_kernel_inject=True)
string = generator("DeepSpeed is ", do_sample=True, min_length=50)
print(string)

Here is some part of the result I am seeing for this example:

That little code snippet was very helpful to debug what is happening here, my observations:
I was using the float16 revision so I had to download the float32 version and I figured that might be it, but that didn't change anything. I got the same error as before when running the exact code you provided (I only fixed the deepspeed import):

When turning off sampling I also saw the same weird behavior with the exclamation marks:

However, given that it worked for you there had to be something about my setup that was causing it, so I started changing dials:

Changing to transformers==4.15 --> no change
Changing to 2 GPUs --> no change

But then I tried deepspeed==0.5.10 and it all works again! Both your code snippet started working as well as my code. This suggests to me that something else got introduced that causes things to break.

Hi @TiesdeKok,
I think taking a look on this issue I opened might be relevant to your use case:
https://github.com/microsoft/DeepSpeed/issues/1797
I think it at least explains why you got the exclamation marks outputs and also probably raise your attention regarding the outputs you're getting in case you pad some of your inputs.

Hi @TiesdeKok I am also facing the garbage output issue. Not sure if it is related to the issue you were having previously: https://github.com/microsoft/DeepSpeed/issues/2113


Is it possible to import simple svg that has just one path?

Thanks for the package, it looks great!  I have a couple of existing svgs that are super simple, just one path.  How do I convert that to the json and import it, is that possible?

Hi. This package is purposed for creating border shapes (must be a closed path). If your path is simple enough, you can try to recreate it here: https://fluttershape.com/ and export it as JSON. Sorry I didn't implement an svg import feature.

No probs, it works though!  The problem I have is the morphable doesn't smooth the lines so it looks very blocky compared to the svg.  I may be missing something though as I'm not really a graphics person.
I wrote this function that converts a raw path (copy and paste from the svg!)  Maybe it helps someone
Map<String, dynamic> morphableConvertCoOrdinates(String coOrdinates) {
  var blim = coOrdinates.replaceAll("\r", "");
  var blah = blim.replaceAll(RegExp('\\s+'), ' ');
  var splut = blah.split(" ");
  var result = splut.map(
    (e) {
      var x = e.split(",");
      return {
        "pos": {"dx": double.tryParse(x[0]), "dy": double.tryParse(x[1])}
      };
    },
  ).toList();

  var json = {
    "type": "Path",
    "border": {"color": "ff000000", "width": 0, "style": "none", "strokeCap": "butt", "strokeJoin": "miter"},
    "path": {
      "size": {"width": 600, "height": 600},
      "nodes": result
    },
  };

  return json;
}

actually this works in converting the list of svg co-ordinates (I previously misunderstood how the svg worked).  I could add it somewhere as a pull request if you think it might be cool.
Offset _getXY(String coOrdinates) {
  var split = coOrdinates.split(",");
  return Offset(double.parse(split[0]), double.parse(split[1]));
}

Map<String, dynamic> morphableConvertCoOrdinates(String coOrdinates) {
  var nodes = coOrdinates.split("\n").where((element) => element.length > 10).toList();

  var result = nodes.map(
    (e) {
      var nodeLine = e.trim();
      var positions = nodeLine.split(" ");

      return {
        "prev": {"dx": _getXY(positions[0]).dx, "dy": _getXY(positions[0]).dy},
        "pos": {"dx": _getXY(positions[1]).dx, "dy": _getXY(positions[1]).dy},
        "next": {"dx": _getXY(positions[2]).dx, "dy": _getXY(positions[2]).dy},
      };
    },
  ).toList();

  var json = {
    "type": "Path",
    "border": {"color": "ff000000", "width": 0, "style": "none", "strokeCap": "butt", "strokeJoin": "miter"},
    "path": {
      "size": {"width": 600, "height": 600},
      "nodes": result
    },
  };

  return json;
}

btw, I really like it!  I think if you update your examples you should do something like this.
https://user-images.githubusercontent.com/4139336/219431014-cc0ae078-9b0d-4ece-b9fb-4cb3ade68fe9.mov

@atreeon could you provide a full example ? How about the performance ?
Thanks


Markdown sites don't work anymore, DatArchive is not defined error

Operation System: elementary OS 5 Juno (Ubuntu 18.04)
Beaker version: 0.8.0

I've been making use of Beaker's Markdown website functionality (as explained here: https://beakerbrowser.com/docs/guides/create-a-markdown-site), where you just put a Markdown file, some CSS and a nav.md file into a folder and Beaker then automatically turns it into a website with a sidebar.
It worked fine in Beaker 0.8.0-prerelease.9, but it doesn't work anymore in Beaker 0.8.0 final, CSS and the sidebar aren't displayed anymore and there is an error in the console.
This is what it looks like and the error shown in the console:

This is what it should look like (old screenshot):

Here is the link to my site shown in the screenshot if it helps: dat://b6201615277a04b958afd25b3531b90083f9dc4e2c38e3cbedb0a257def456fe/
And the error message in text form:
Uncaught (in promise) ReferenceError: DatArchive is not defined
    at renderNav (<anonymous>:102:16)
    at Object.1.remarkable (<anonymous>:99:3)
    at o (<anonymous>:1:265)
    at r (<anonymous>:1:431)
    at <anonymous>:1:460

Hmm, I cant get your dat site to load, what's your dat.json look like? Are you using the content_security_policy directive by chance?

Oh can you be more specific about what's broken? Is it just the nav?

Scratch that, I see. It's the nav and the CSS. I was able to reproduce.
Okay this isn't cool. I'm on it. Thanks for reporting.

@pfrazee Ah good that you can connect now, I'm seeding the site with hypercored (as explained here: https://docs.datproject.org/server) so it should always be available even when I have Beaker closed, but it's on my home internet connection so there might be some firewall/router issues going on.
The dat.json is the default created by Beaker, I don't think I've ever manually edited it (just used the Beaker UI to add a donation link) so it should be fine, there is no content_security_policy directive in there but you have probably seen that now that it works for you.

Okay should be fixed in master


Added missing @throws tags

Added missing @throws annotations.

Closing for same reasons as zendframework/zend-expressive-authorization#3.


Enables lookups for custom file extensions

Hiera backends like eyaml allow for custom file-extensions. This allows for example to use the .yaml extension for eyaml, instead for the default .eyaml extension.
This commit enables hiera_explain to lookup and use the custom file-extension, if configured for a backend.
On eyaml extensions: https://github.com/voxpupuli/hiera-eyaml/blob/master/README.md
Example output of eyaml backend using the .yaml extension:
# /opt/puppetlabs/puppet/bin/ruby -I$PWD/lib bin/hiera_explain -f foo
Backend data directories:
  * eyaml: /etc/puppetlabs/hieradata

Expanded hierarchy:
  * common

File lookup order:
  [X] /etc/puppetlabs/hieradata/common.yaml

Priority lookup results:
   * hiera('foo') => bar

Array lookup results:
   * hiera_array('foo') => ["bar"]

Hash lookup results:
   * hiera_hash('foo') => Not a hash datatype in ["common.yaml"]

Example output of eyaml backend using the default (or no configured) extension:
# /opt/puppetlabs/puppet/bin/ruby -I$PWD/lib bin/hiera_explain -f foo
Backend data directories:
  * eyaml: /etc/puppetlabs/hieradata

Expanded hierarchy:
  * common

File lookup order:
  [ ] /etc/puppetlabs/hieradata/common.eyaml

Priority lookup results:

Array lookup results:

Hash lookup results:

@binford2k can you glance this one, when you're at it of course 😊

Thanks @mpepping! (this will still output cipher text for eyaml, of course. Adding that support is on my todo list.)


Zstandard decompression support

Support decompression of files using Facebook's Zstandard(zstd) algorithm.

is license ( https://github.com/luben/zstd-jni/blob/master/LICENSE ) ok for apache ?

That looks like 2-clause BSD, which is ok (http://apache.org/legal/resolved.html#category-a). And zstd itself is 3-clause BSD, which is also ok.

LGTM then


Implement icon upload in API Client Connector edit page

This is a...

[ x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Documentation issue or request

Description
File upload of icon files needs to be implemented. New icons are saved when the edit page is saved.

This is done now, right?

Yes @riccardo-forina fixed by #306


AAR build from latest meidapipe can't be used fo android studio project

Thanks for your nice work.
I hava cloned this project and run it in my phone successfully:
https://github.com/jiuqiant/mediapipe_multi_hands_tracking_aar_example
However, the app crashed if I build the aar from source code and replace the original one.
I don't know how to fix this. I guess maybe because the updating of mediapipe.
BTW, I build the aar following this document:
https://github.com/jiuqiant/mediapipe_multi_hands_tracking_aar_example
Hoping for help

MediaPipe hand graph along with the calculator source code and model files are updated in the mediapipe repo. If you build the aar from source code, you need to regenerate the binary graph and copy the new model files as well.

Thanks for your reply.
Now, I am going to rebuild the binary graph. But I meet another issue.
https://github.com/google/mediapipe/blob/master/mediapipe/docs/android_archive_library.md
In this insturctions , build command for binary graph is:
bazel build -c opt mediapipe/examples/android/src/java/com/google/mediapipe/apps/facedetectiongpu:binary_graph
I find there is no pre-defined binary_graph in BUILD file located in  android\src\java\com\google\mediapipe\apps\multihandtrackinggpu.
But there is mediapipe_binary_graph in BUILD file located in  mediapipe\graphs\hand_tracking
Is this the binary graph for multihandtracking app?

Seems like the instruction needs to be fixed...
You can get hand_tracking_mobile_gpu.binarypb by running
$ bazel build mediapipe/graphs/hand_tracking:hand_tracking_mobile_gpu_binary_graph
INFO: Analyzed target //mediapipe/graphs/hand_tracking:hand_tracking_mobile_gpu_binary_graph (4 packages loaded, 184 targets configured).
INFO: Found 1 target...
Target //mediapipe/graphs/hand_tracking:hand_tracking_mobile_gpu_binary_graph up-to-date:
  bazel-bin/mediapipe/graphs/hand_tracking/hand_tracking_mobile_gpu.binarypb
INFO: Elapsed time: 5.855s, Critical Path: 0.10s
INFO: 1 process: 1 darwin-sandbox.
INFO: Build completed successfully, 2 total actions

BTW, the String variable BINARY_GRAPH_NAME in the MainActivity.java should match the binarypb name. For example,
private static final String BINARY_GRAPH_NAME = "hand_tracking_mobile_gpu.binarypb";

understoon. I think it will be helpful if the instruction is fixed.

Both the instructions and the example repo are updated.

AAR和图形之间的不匹配将使应用程序崩溃。详细说

Can you help me see how to solve this problem?Thanks. https://github.com/google/mediapipe/issues/1450

AAR和图形之间的不匹配将使应用程序崩溃。详细说

Can you help me see how to solve this problem?Thanks. https://github.com/google/mediapipe/issues/1450

Can you help me see how to solve this problem?Thanks. https://github.com/google/mediapipe/issues/1450

Can you help me see how to solve this problem?Thanks. https://github.com/google/mediapipe/issues/1450


Postcode comparison template

Addresses issue #215
Comparison template for postcode column. The default arguments will give a comparison with levels:
- Exact match on full postcode
- Exact match on sector
- Exact match on district
- Exact match on area
- All other comparisons
with an optional 'distance in km' comparison level

@zslade this looks good at first look (but currently on my phone so will look at properly tomorrow).
One thing I think should be added is the ability to add multiple km distance levels.
You can see an example of how to deal with multiple levels at https://github.com/moj-analytical-services/splink/blob/6e3a2e177f53a372f29b5fca57cfc3877b2d4c4d/splink/comparison_template_library.py#L195
So you will need to use ensure_is_iterable then distance_threshold_comparison_levels to get this working for the levels themselves.
Plus add to the comparison description similar to https://github.com/moj-analytical-services/splink/blob/6e3a2e177f53a372f29b5fca57cfc3877b2d4c4d/splink/comparison_template_library.py#L275
With distance_threshold_description

Also, it would be good to update the feature engineering topic guide with this function for the postcode section

And there are postcode columns in some of the splink demos which should be updated with this function instead


fixed LogsBloom

What was wrong?
Related to Issue #
How was it fixed?
This is a regression created by #275. This was breaking #280
Cute Animal Picture

Picture credit: Hanny Naibaho

Codecov Report

Merging #305 (3f307e7) into master (c2d0aac) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #305   +/-   ##
=======================================
  Coverage   93.31%   93.32%
=======================================
  Files          29       29
  Lines        1736     1737    +1
=======================================
+ Hits         1620     1621    +1
  Misses        116      116

Flag
Coverage Δ

unittests
93.32% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/ethereum/frontier/spec.py
87.31% <100.00%> (+0.09%)
:arrow_up:

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 c2d0aac...3f307e7. Read the comment docs.


Build should put html and js files into "public" subfolder of dist and server should expose only that folder

Right now, all of dist is exposed. Only html and js should be (not the data folder within).

Is this bug obsolete with the move to gh-pages?

Yes.
--Jonathan

On Sep 22, 2016, at 9:37 PM, Jill Huchital<EMAIL_ADDRESS>wrote:
Is this bug obsolete with the move to gh-pages?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

OK, closing.


html for User Home page

Please do work for this task in a branch named issue-08.
Includes users favorites
Includes a side bar listing recipes that the user has added to the site
give the option to view / edit?

Layout for home page is finished on issue 8 and merged into main.


Resolve issue with broken CI/CD Release Pipeline

Description
This PR adds the working-directory directive to the release workflow, ensuring the correct path is used for operations like comparing the Git tag with the Cargo metadata. As a result, the CI/CD release pipeline now runs successfully and triggers as expected when a tag is pushed.
Related Issue(s)

closes #64

Documentation

No additional documentation changes required

oh duh :) that happened right when I moved the repo structure around


Error trying to start EC2 cluster for paraview workflow

@TristanWright Looks like we need the same fix, you did for PyFR? ( is it not the same code ?)
HPCCloud.js:53512 Uncaught Error: missing machine in ec2 cluster payload

hmm that should be acting the same, a debug componentWillUpdate says otherwise, looking into it.

I was on the wrong branch, but there was something off with the payload.

closeable?


web development roadmap

plz add this my first issue

/assign

Can you please elaborate more on this issue?

/assign

Can I get more details regarding this about how to add a roadmap

Can I get more details regarding this about how to add a roadmap

@payalpm

Hey @payalpm can I get more details regarding this about how to add a roadmap


the document icon on headers tab

the icon in tab headers is not loaded

Hey @ManiRamezanpour are you talking about the https://typespec.io website icon?

Yes
On Thu, Jan 25, 2024 at 7:11 PM Timothee Guerin @.***>
wrote:

Hey @ManiRamezanpour https://github.com/ManiRamezanpour are you talking
about the https://typespec.io website favicon?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/typespec/issues/2828#issuecomment-1910465160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATWAQXWBQNRFNYMLMR6NTQ3YQJ4L5AVCNFSM6AAAAABCKK6U2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQGQ3DKMJWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Valentine (testing out how to fill these out)

audio: not sure what to do here
author: JoseDiazRohena
changes: ''Increased maximum attack time to 55ms
Adjusted crush/saturation/clip button spacing"
date: '2024-04-27T07:00:00.000Z'
description: An open source compressor meant to pump and breathe
files:

architectures:

x64
contains:
so
format: tar.gz
sha256: 858c16bf594ace122cee1854024fb8719c232f593d3ecf3dccee72f22c0b2a08
systems:
type: linux
size: 491519
type: archive
url: https://github.com/tote-bag-labs/valentine/releases/download/v1.0.1/valentine-1.0.1-Linux.zip

architectures:

x64
contains:
dll
format: zip
sha256: 7728372c985c4cb0019f5000858d9486a0444f70fed3c44cac724c05db19c94d
systems:
type: win
size: 853466
type: archive
url: https://github.com/tote-bag-labs/valentine/releases/download/v1.0.1/valentine-1.0.1-windows.exe
image: see below
license: gpl-3.0
name: Valentine
tags:

Effect
Compressor
Distortion
type: effect
url: https://github.com/Tote-Bag-Labs/Valentine

not done yet, need to figure out how to fill in the other info!

ignore this for now (unless you want to finish it which is fine, in which case i'll find another plugin to work on just so i can figure this out)

Closing this issue as there is a PR to track it


Points to analysis overlapping access

Overlapping accesses functionality for DSA.
Addresses overlapping accesses to Stack and Global Memory Locations through Addresses, Input/Output Parameters and Indirect Accesses

[x] Revised Memory Locations for DSA
[x] Function Memory Locations represented by cells expanded to their size
[ ] Overlapping Accesses Soundness #276

This seems to have fixed the issues (I'm surprised it was that simple) I was having yesterday but I'm going to test it with some more complex cases to make sure there aren't still any underlying problems.

This seems to have fixed the issues (I'm surprised it was that simple) I was having yesterday but I'm going to test it with some more complex cases to make sure there aren't still any underlying problems.

Did your issue's program include function pointers?
The subsequential change is that instead of initialising nodes for function memory locations with a cell of size 0 at offset 0, now DSA initialises a node with a cell of the size of the memory location at offset 0. When the function locations are loaded, they would be merged into the differences between offsets of their pointers in the SSA variable node. For example, in jumptable/clang since function cell sizes were 0 the two function memory cells wouldn't be merged unifying their nodes once the. Now that their sizes are 20 they would be merged at offsets 0 and 8 unifying their nodes.
The other global/stack/heap have their cell sizes initialised to 0 since DSA learns their sizes through accesses but the function locations are never read/written to.

Yes, it seemed to be that incorrect sizes were causing some weird problems.

I've added a test which demonstrates the remaining issue - if two pointers are loaded together into a single 128-bit register, then it is not correct to merge their pointees as if they are contiguous - the cells should always be collapsed.

You've misunderstood the problem, the "overlapping access" test was fine before.

Although I guess it doesn't make a difference right now since it would never be possible for the cells to point to different things

I agree it doesn't make a  lot of difference with the current approach, it's simpler and definitely sound to grow the size of the pointer cell to equal the size of the access, it will then merge with it's adjacent cells that are also getting accessed which then in turn also unifies their pointees.
Keeping the multiple pointer cells getting dereferenced as separate is trickier. Currently the pointer cells are kept separate and their pointees are merged and also collapsed.
I'm fine with either specially if we want to be able to get more precision out of overlapping accesses, we will need to consider the pointer cells separate, so it's good to keep them that way and pick up on any issues if they still exist

For now it's we just get clearer results if we do merge & collapse the cells like that I think, but it's good that you've figured out how to make the more precise version work because we'll likely want to build on that in the future. Sorry for the confusion.

I tried to merge this in but I can't merge this in without breaking things so I will leave it for another time.

will have a look after the workshop

I'm not sure about why some examples (secret_write and basic_function_call_caller) fail verification now


[Doc update][API-based minor updates]

updated

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/api-based-overview.md
92
6
link
:white_check_mark:

msteams-platform/messaging-extensions/build-api-based-message-extension.md
94
14
link
:white_check_mark:

msteams-platform/sbs-api-based-message-extensions.yml
82
85
link
:white_check_mark:

msteams-platform/sbs-api-me-ttk.yml
85
53
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/api-based-overview.md
92
6
link
:white_check_mark:

msteams-platform/messaging-extensions/build-api-based-message-extension.md
94
14
link
:white_check_mark:

msteams-platform/sbs-api-based-message-extensions.yml
82
85
link
:white_check_mark:

msteams-platform/sbs-api-me-ttk.yml
85
53
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


#311 - StorageHasMetadata matcher

Created StorageHasMetadata to avoid code duplications and used it in RpmITCase and RpmTest  for #311.

@0crat out

@0crat out (here)

@olenagerasimova Job gh:artipie/rpm-adapter#312 is not assigned, can't get performer

@0crat out (here)

@olenagerasimova Pull request #312 was not merged, no payment for ARC, see §28

@olegmoz thanks, corrected, please, take a look

Job gh:artipie/rpm-adapter#312 is not assigned, can't get performer


📄🚀 Add factor labels

Describe the feature you want and how it meets your needs or solves a problem

As a TIDES user, I want the labels of enumerated values to be documented in the spec so that their descriptions can be self-documenting.
Describe the solution you'd like

New support for this in Frictionless: https://github.com/frictionlessdata/specs/pull/844

I'm pretty sure enums are enumerated by value in the spec, e.g., https://github.com/TIDES-transit/TIDES/blob/b4d722cff27b0c35a12b27703ebd3a6732df66ef/spec/vehicle_locations.schema.json#L170 and https://tides-transit.org/main/tables/#vehicle-locations.
Are you asking for option 1 (adding "enumOrdered": true), or 2, using integers for the enum values and adding enumLabels? It doesn't look like either of those would make enums any more self-documenting.


feat: switch to official elastic provider

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improve templates examples

From a user:
Make the “Example files” download link more obvious and also only including the fields in those files that are necessary for submission (it appears some of them are just hypothetical fields for a hypothetical study but people may confuse this with the actual required fields, not all of which are mentioned in the Templates Tutorial page)
Agree, and we can have more than one example.

This was fixed some time ago, new version


Varprefix

better attempt at variable prefix for #16 - would like another set of eyes thru docs

Ah darn, so we just added Fedora support as well, which is now failing here.
Looks like https://github.com/sensu/sensu-ansible/blob/master/tasks/Fedora/redis.yml#L9-L10 for sure need to be updated to their prefixed version.

Submitted a PR to @pdaugavietis' branch and verified that fixes the Travis build. Will give him some time to review and merge to ensure he gets credit for the work he's done.

Hi! I'm going to close this PR out for the time being. I would love to get this change in to support https://github.com/sensu/sensu-ansible/issues/16 so, if you'd like to revisit this PR, feel free to reopen.


bug when used with classnames package

I have a typescript file that contains a react component and one of HTML elements uses the classnames utility to manage the element's classNames property. When trying to save the component + headwind.nvim resorts the classnames, it outputs invalid JSX. (see gif)

BTW thanks for this plugin! All the existing headwind integrations are outdated and don't work with Tailwind 2 or JIT.

I'll try out this scenario... are you using treesitter integration? This scenario is a little tricky using the pattern approach...

I would highly recommend using the treesitter setting instead of using the pattern approach. This scenario is hard to target using lua patterns. I will add steps on how to use treesitter. If you already are using treesitter and have the parser installed, you just need to enable the treesiter flag during setup.

It does look like using treesitter fixes the issue; however, I haven't been using treesitter because it causes some problems that are unrelated to your plugin.
Would any of the regexes mentioned in this PR help?

I used the wrong github account for those replies.... here they are for context.

I'll try out this scenario... are you using treesitter integration? This scenario is a little tricky using the pattern approach...
I would highly recommend using the treesitter setting instead of using the pattern approach. This scenario is hard to target using lua patterns. I will add steps on how to use treesitter. If you already are using treesitter and have the parser installed, you just need to enable the treesiter flag during setup.

@kyle-mccarthy Treesitter is just a parser. It won't cause you any problems to have the parser installed. You don't have to use the any highlighting or any other plugins that use treesitter. You just need the parser for your language installed.
I think the problem here is how it calculates the range to replace... the new line is messing it up.

Is there a new line? I think that it just wraps currently without inserting a line break.
Also to preface this, I know this isn't the right place for me get treesitter assistance but I want to explain why I haven't enabled it. My main problems with treesitter are errors when editing a file with a language (like vim) that doesn't have a parser, also, for some unknown reason, saving a file takes longer with treesitter enabled, and saving a file intermittently triggers an error and writes Error detected while processing BufWritePre Autocommands for '*'.
I haven't had the time to debug the errors further though, but since treesitter didn't exactly work out of the box I haven't been using it.

@kyle-mccarthy You don't have to enable any highlighting module for treesitter. You can just install the parser and this plugin will use it. Treesitter does nothing on it's own. It's just an api for plugins to use. I recommend just installing the nvim-treesitter plugin and installing the parser for javascript (or typescript). You don't even need to setup the nvim-treesitter plugin. It's purely just for compiling the parser.

Also, I think it's treating the function call as a class and that's why it's messing it up.


Package testing migrated to nosetests.

Also some quality of life improvements:

Docstrings.
Better directory structure
Removed unneeded files
Better travis test script

FIXES Ericsson/codechecker#278

FIXES Ericsson/codechecker#259


FAQ: Typos, grammar and understandability of vac_cert_invalid

Where to find the issue
https://www.coronawarn.app/en/faq/#vac_cert_invalid and
https://www.coronawarn.app/de/faq/#vac_cert_invalid
Describe the issue
The articles (English and German) have been revised several times and they include typos, grammatical errors and are not very understandable.
Mentioning June 14 twice in the first sentence doesn't read well.
EN
"... there are certificates with QR codes are issued ..." (grammar is incorrect)
"digitial" (typo)
DE
"eine Bescheinigungen" (doesn't agree singular/plural)
Suggested change

Fix typos and grammatical errors.
Make the text more oriented to the user who has just tried to scan in a vaccination certificate: why didn't it work? what should they do now? what can they expect to happen?

I can submit a PR for this, but I don't want to clash with @thomasaugsten who has lately submitted fast-track PRs without peer review, so I need to know if a PR from me would be useful or not.

PR are very welcome for anything

PR #1365 added to correct typos and grammar issues in
https://www.coronawarn.app/en/faq/#vac_cert_invalid and
https://www.coronawarn.app/de/faq/#vac_cert_invalid.

PR https://github.com/corona-warn-app/cwa-website/pull/1447 addresses some readability issues in the first sentence of the FAQ article.
https://www.coronawarn.app/en/faq/#vac_cert_invalid now starts with:
"The app can only successfully scan 'EU Digital COVIID Vaccination Certificates' and these are being issued by vaccination centers since 14 June 2021."
https://www.coronawarn.app/de/faq/#vac_cert_invalid now starts with:
"Die App kann nur erfolgreich ein 'EU digitales COVID-Impfzertifikat' scannen. Diese Bescheinigungen werden seit dem 14.06.2021 in den Impfzentren ausgegeben."
This is now improved enough to close this issue.

@MikeMcC399 Thank you very much!


smaller/greater than sign switched

Should this just be !=? This also seems to be part of PR #564. Since this is a separate bug fix, I'd like to keep it separate from #564

@wietsefranssen : I'll merge it (or @jhamman can), but you need to resolve the conflicts first

Ah I forgot to update! Done now!

@jhamman : Can you merge this after the latest travis problems are resolved?

merged with failing build,  will pass after merge due to #568


Updated 6.2.1 changelog so that it reflects what's in the release notes

Updated the changelog file for 6.2.1 so that it reflects what is in the 6.2.1 Release Notes and conforms to the documentation standard.
The changelog for subsequent versions should conform to this standard.

Added internally in 31821cb58596ec20c7ce2f6b5b31fee1fb0bd079


prettify converts decimals to int

before :
{"driving":8.0,"stopover":8.0,"trip":8.0}
after
{
    "driving": 8,
    "stopover": 8,
    "trip": 8
}

It looks like this is just the nature of how JSON.stringify / JSON.parse work.  Using non-zero floating point values such as {"driving":8.1,"stopover":8.2,"trip":8.3} result in the expected output of:
{
  "driving": 8.1,
  "stopover": 8.2,
  "trip": 8.3
}

The only other thing I can think of is storing the data as a string, like {"driving":"8.0","stopover":"8.0","trip":"8.0"}.


i ssue flv

I tiried to push the stream to flv  by using ffmpeg but giving erorr  any help please
livestreamer --http-header "Cookie=" "hlsvariant://http://hlslinear-l3c.med1.foxsportsgo.com/ls02/foxsports/FOX_SPORTS_2/live/master_wired_web_ads.m3u8" best --player | ffmpeg -i - -acodec aac -strict -2 -ab 48000 -ar 48000 -vcodec copy -f flv "rtmp://"

its work in vlc but i need to push via rtmp  can someone help please
Thnx


check isIgnored when compiling attrs to update

Hey @jinzhu (and community!) - thanks for the excellent work.  I found an issue where scope.Updates() weren't ignoring fields that had been tagged sql:"-" and tracked it down to this.  Note that while I was digging, I also found this introduction from a while ago: https://github.com/jinzhu/gorm/commit/c3d2746912d9e16af54b75e2baefca87ae7990cc#diff-ee3fac9069b0a4978ec2ce9e09640bf7R103 that appears to be no-op?  Was there another way that ignored fields where supposed to be ignored that was never completely implemented / is that a better solution?
thanks!

Looks good, thank you!


[C++][Parquet] Add API for writing boolean values as a packed bitmap

Presently, the TypedColumnWriter for booleans expects a bool* (i.e. 1 byte per value). Some writers may also use a LSB-ordered bitmap, and we could write that directly rather than having to copy the bits into a temporary bool*
Reporter: Wes McKinney / @wesm
Note: This issue was originally created as PARQUET-644. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Sounds good; I assigned the issue to you

Shuai Lin:
@wesm I'd like to work on it.

Gang Wu / @wgtmac:
This is resolved in arrow/cpp/src/parquet/column_writer.cc at 37bd413c8322e0d9527c14420468b99f8abd1715 · apache/arrow (github.com)

Wes McKinney / @wesm:
It's probably easiest now (after ARROW-3246) to implement TypedEncoder::Put(const arrow::Array&) for BooleanArray inputs


containers with --restart=always do not get restarted after switching consoles

RancherOS Version: (ros os version)
v0.6.1 and v0.7.0-rc5
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
AWS
I have containers with a restart policy of always and they don't get restarted after switching consoles.
Steps to Reproduce:

Create a container sudo ros -d --restart=always nginx
Switch consoles sudo ros console switch ubuntu

Result:
Console switches, but nginx container never starts.
Docker is running because I can find the container when doing docker ps -a
Note: They do restart after upgrade and doing sudo system-docker restart docker.

This appears to be magically fixed in v0.8.0-rc1 :tada:

Tested with v0.8.0-rc1 and added integration tests in #1436


Implement CloudSync according to the new design

Can we accept this raised PR as initial complete working model for CloudSync and a separate PR with the design discussed and finalised can be raised as next PR since the design changes are for all the data structure, I would suggest completing the functionality in this PR and rasing new PR for design
Originally posted by @nitu-s-gupta in https://github.com/lf-edge/edge-home-orchestration-go/pull/532#issuecomment-1082689932

In the Design Consideration We also need to store the client object.
Lets consider a scenario where there are two brokers say Broker1.com, Broker2.com
Now Client1 request for Broker1.com(publish) Then Client 2 for Broker2.com(subscribe), Client 3 for Broker1.com(subscribe)
The following info would be stored in the follwoing datastructures

URLData<string,[]string>(url,clientid array): Since its a first time entry the url entry shall be made in URLData with clientid(client1).
PublishData<{topic,url},data>: The entry for the published data shall be made in the PublishData
For Next Step, Since broker2.com is not present in URLData so the entry for it is made, and connection object for the broker is created.
SubscriptionInfo<{topic,url},[]clientid>Entry to the subscriptionInfo is also made with client id of client2
Now Client 3 request for again broker1.com. Now since we havent stored the client object for the Broker1.com, We cannot reuse the current homeedge object since the object is currently for connection to Broker2.com
Hence I would suggest to store clientobjects in separate hashmap mapped to urls. i.e. for every new url entry in the URLData, a entry to MQTTClient is made with url,clientobject
so In all there will be 4 Hashmaps
MQTTClient: <string,*Client>: Storing client object for every new url connection that homeedge makes.
URLData<string,[]string>: For every request publish/subscribe, If the user is not present for the url, entry to the Hashmap is made.
SubscriptionInfo<{topic,url},[]clientids>:For subscritpion request the client id is append for the key combination of topic,url if not present.
4.PublishData<{topic,url},data>: For every subscribed topic, entry to publish data is made with the latest data published by any client on the same url for the topic subscribed.
Plz let me know if any changes required

In the Design Consideration We also need to store the client object. Lets consider a scenario where there are two brokers say Broker1.com, Broker2.com Now Client1 request for Broker1.com(publish) Then Client 2 for Broker2.com(subscribe), Client 3 for Broker1.com(subscribe) The following info would be stored in the follwoing datastructures

URLData<string,[]string>(url,clientid array): Since its a first time entry the url entry shall be made in URLData with clientid(client1).
PublishData<{topic,url},data>: The entry for the published data shall be made in the PublishData
For Next Step, Since broker2.com is not present in URLData so the entry for it is made, and connection object for the broker is created.
SubscriptionInfo<{topic,url},[]clientid>Entry to the subscriptionInfo is also made with client id of client2
Now Client 3 request for again broker1.com. Now since we havent stored the client object for the Broker1.com, We cannot reuse the current homeedge object since the object is currently for connection to Broker2.com
Hence I would suggest to store clientobjects in separate hashmap mapped to urls. i.e. for every new url entry in the URLData, a entry to MQTTClient is made with url,clientobject
so In all there will be 4 Hashmaps
MQTTClient: <string,*Client>: Storing client object for every new url connection that homeedge makes.
URLData<string,[]string>: For every request publish/subscribe, If the user is not present for the url, entry to the Hashmap is made.
SubscriptionInfo<{topic,url},[]clientids>:For subscritpion request the client id is append for the key combination of topic,url if not present.
4.PublishData<{topic,url},data>: For every subscribed topic, entry to publish data is made with the latest data published by any client on the same url for the topic subscribed.
Plz let me know if any changes required

Thanks for your idea! I agree that client objects are necessary.
Please update the design diagram considering this concern share with us.

Resolved by #544, #549


Multiarch support for the operator

It would be really useful if it didn't get stuck with exec format errors. :)

/remove-lifecycle stale

/remove-lifecycle rotten

/remove-lifecycle rotten

This should be pinned..

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

Hi @samip5 Just to let you know that this repo has not been maintained in many years and is being archived due to inactivity.
For reference, here is the mailing list thread from earlier in the year: https://groups.google.com/g/kubevirt-dev/c/QpIaRzmNh4c/m/3lA87gdAAAAJ


bugfix:修改常用服务不需要项目权限和tab标签字体

Codecov Report

Merging #513 (5451a56) into v2.6.x_develop (0f1ba16) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           v2.6.x_develop     #513   +/-   ##
===============================================
  Coverage           62.69%   62.69%
===============================================
  Files                 441      441
  Lines               27270    27270
===============================================
  Hits                17096    17096
  Misses              10174    10174

Impacted Files
Coverage Δ

config/default.py
86.02% <0.00%> (ø)

Continue to review full report at Codecov.

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


Specify host <IP_ADDRESS> for Jekyll

Allows for remote connections on same network (used to be the default behavior, but Jekyll now seems to default to <IP_ADDRESS> which means it does not react to incoming external requests)

The need for this seems related to Jekyll changing its default configuration a while back. Wondering if it kept working until recently here because we hadn't updated dependencies... https://stackoverflow.com/questions/24108302/jekyll-serve-suddenly-switched-from-localhost-to-0-0-0-0-any-fix


[Bug]: [Android] Get SORA Card - 500 error appears when tapping on the “see the list” link

Preconditions
Fresh install
Steps to reproduce
1.Go to the wallet tab
2.Get SORA Card
3. Tap on the “see the list” link
Actual behavior
Get SORA Card - 500 error appears when tapping on the “see the list” link.

Expected behavior
Get SORA Card - There should be redirect to the  Blacklist page after tapping on the “see the list” link.
Environment
{"label"=>"Development"}
OS version
Android 13
Additional context
Soralution version

fixed

Verified on <IP_ADDRESS> version.


[Material 3] Using NestedScrollView with SliverOverlap* and Material 3 SliverAppBar does not show collapsed SliverAppBar when the child happens not to start scrolling

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Build a NestedScrollView
Set SliverOverlapAbsorber that wraps SliverAppBar.large as header sliver
Set CustomScrollView containing SliverOverlapInjector and some low-height components (they are placed no higher than the viewport height) as body
Scroll up until no more scrollable

Expected results
The collapsed SliverAppBar will display
Actual results
The collapsed SliverAppBar won't display
Code sample

Code sample
https://dartpad.dev/?id=2e84b36de33d6a00e59b4f6536531e07
NestedScrollView(
  headerSliverBuilder: (context, innerBoxIsScrolled) {
    return [
      SliverOverlapAbsorber(
        handle: NestedScrollView.sliverOverlapAbsorberHandleFor(context),
        sliver: SliverAppBar.large(
          title: const Text('Title'),
        ),
      ),
    ];
   },
  body: CustomScrollView(
    slivers: [
      Builder(
        builder: (context) {
          return SliverOverlapInjector(
            handle:
                NestedScrollView.sliverOverlapAbsorberHandleFor(context),
          );
        },
      ),
      const SliverToBoxAdapter(
        child: Text('Content'),
      ),
    ],
  ),
)

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.0, on Microsoft Windows [Version 10.0.22621.1702], locale zh-CN)
    • Flutter version 3.10.0 on channel stable at D:\SDKs\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 84a1e904f4 (10 days ago), 2023-05-09 07:41:44 -0700
    • Engine revision d44b5a94c9
    • Dart version 3.0.0
    • DevTools version 2.23.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\XYX\scoop\apps\android-sdk-base\current
    • Platform android-33-ext5, build-tools 33.0.2
    • ANDROID_HOME = C:\Users\XYX\scoop\apps\android-sdk-base\current
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.5+0-b2043.56-9484017)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.5.3)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.5.33516.290
    • Windows 10 SDK version 10.0.22621.0

[✓] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.5+0-b2043.56-9484017)

[!] Android Studio (version 2023.1)
    • Android Studio at C:\Users\XYX\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\231.7864.76.2311.10101468
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at C:\Users\XYX\AppData\Local\JetBrains\Toolbox\apps\IDEA-U\ch-0\231.8770.65
    • Flutter plugin version 73.1.1
    • Dart plugin version 231.9065

[✓] VS Code (version 1.79.0-insider)
    • VS Code at C:\Users\XYX\AppData\Local\Programs\Microsoft VS Code Insiders
    • Flutter extension version 3.62.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.1702]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 113.0.5672.126
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 113.0.1774.35

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

I found out that the problem can be solved like this. In lib/src/material/app_bar.dart:
- final bool isScrolledUnder = overlapsContent || forceElevated || (pinned && shrinkOffset > maxExtent - minExtent);
+ final bool isScrolledUnder = overlapsContent || forceElevated || (pinned && shrinkOffset >= maxExtent - minExtent);

Hi @xfqwdsj
I check your sample code on both latest Flutter channels and 3.7.12 but it seems to work as expected (SliverAppBar title is collapsed)

Demo video
https://github.com/flutter/flutter/assets/104349824/06aa385d-35d6-4841-b6e8-61bb41829913

Can you please provide the recorded video that illustrates issue more visible?

Sorry, I didn't describe the problem well.  I mean, in this case, the collapsed AppBar, that is, the elevated one, does not display normally, but instead displays a blank area.  If the body part of the NestedScrollView is scrolled a little, it will be displayed normally.
This demo video is done using CustomScrollView, but when the height of the NestedScrollView's body is sufficient, their effects are the same.

Video
please ignore the text spacing bug
https://github.com/flutter/flutter/assets/37076189/1352360e-86e3-48ce-a068-da491718669f

In the video, I mainly show two states of the AppBar (sorry I can't edit this video for now to explain it): the position of the AppBar is blank, and a "collapsed" or "elevated" AppBar is shown.  Showing the collapsed AppBar is what I want to achieve in the code sample of this issue.  If not showing it is the expected behavior by which Flutter was designed, I also hope that the Flutter team will consider modifying it.

Can you check this updated sample code to see if it's your expected behavior?

Sample code
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(
        useMaterial3: true,
      ),
      home: Scaffold(
        body: NestedScrollView(
          headerSliverBuilder: (context, innerBoxIsScrolled) {
            return [
              SliverOverlapAbsorber(
                handle: NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                sliver: SliverAppBar.large(
                  title: const Text('Title'),
                  pinned: true,
                  forceElevated: innerBoxIsScrolled,
                ),
              ),
            ];
          },
          body: CustomScrollView(
            slivers: [
              Builder(
                builder: (context) {
                  return SliverOverlapInjector(
                    handle:
                    NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                  );
                },
              ),
              SliverFixedExtentList(
                itemExtent: 50.0,
                delegate: SliverChildBuilderDelegate(
                      (BuildContext context, int index) {
                    return Container(
                      alignment: Alignment.center,
                      color: Colors.lightBlue[100 * (index % 9)],
                      child: Text('List Item $index'),
                    );
                  },
                ),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Demo
https://github.com/flutter/flutter/assets/104349824/88328571-52e2-4deb-bd77-21071e6ab5d4

For this code, the behavior is what I expected.  But the problem I want to feedback is that when the height of the content is not high enough for the CustomScrollView to scroll, no matter how I scroll, the collapsed AppBar will not be displayed.
I changed the content to a Text again, and the same behavior as before appeared.

Code
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(
        useMaterial3: true,
      ),
      home: Scaffold(
        body: NestedScrollView(
          headerSliverBuilder: (context, innerBoxIsScrolled) {
            return [
              SliverOverlapAbsorber(
                handle: NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                sliver: SliverAppBar.large(
                  title: const Text('Title'),
                  pinned: true,
                  forceElevated: innerBoxIsScrolled,
                ),
              ),
            ];
          },
          body: CustomScrollView(
            slivers: [
              Builder(
                builder: (context) {
                  return SliverOverlapInjector(
                    handle:
                    NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                  );
                },
              ),
              const SliverToBoxAdapter(
                child: Text("Content"),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Thanks for your response. I also see the issue when the CustomScrollView's content is not scrollable.
Labeling the issue for further insights. This is reproducible on the latest stable and master channels.

Sample code: at comment
Demo (has issue): at comment
Demo (no issue when the CustomScrollView's content is scrollable): at comment

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.10.1, on macOS 13.0.1 22A400 darwin-x64, locale en-VN)
    • Flutter version 3.10.1 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d3d8effc68 (26 hours ago), 2023-05-16 17:59:05 -0700
    • Engine revision b4fb11214d
    • Dart version 3.0.1
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

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

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

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

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

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 113.0.5672.126

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.11.0-11.0.pre.22, on macOS 13.0.1 22A400 darwin-x64, locale en-VN)
    • Flutter version 3.11.0-11.0.pre.22 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 31a665c3eb (26 minutes ago), 2023-05-22 19:48:40 -0700
    • Engine revision a342a9186e
    • Dart version 3.1.0 (build 3.1.0-129.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

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

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

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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.0.1 22A400 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 113.0.5672.126

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

@Piinks - maybe related to https://github.com/flutter/flutter/pull/126718?

Could be, https://github.com/flutter/flutter/issues/127191#issuecomment-1554549735 is the same place I traced that to.


added rectifier diode

Issue: #122
Screenshot:

Reference: https://guidepartantepast.z21.web.core.windows.net/rectifier-diode-circuit-symbol.html

It's more like this:

it should be "rectifier diodes" (plural) and have 4 diodes. Rectifier diode AFAIK isn't a thing and doesn't turn up on google :/
A rectifier diode circuit typically has 4 diodes

thanks for the clarification, i was confused regarding that too, and this is the only reference i had found
https://guidepartantepast.z21.web.core.windows.net/rectifier-diode-circuit-symbol.html
i'll start working on the "rectifier diodes"

@seveibar
made the changes mentioned

It's more like this:

Updated Screenshot:

Issue: #122

Quick questions about this PR:

Should we add 'rectifier_diodes' as a separate symbol like I did here, or scrap it since it’s just a combination of diodes?
If we’re keeping it, I accidentally messed up when resolving conflicts. Should I fix it with a new commit, or would it be better to close this PR and start fresh?

@mrudulpatil18 sorry for the delay, the pr looks good, merging!


[IMPROVE] Add a loader screen while data loading #106

* Add a fullscreen loader style and HTML in style.css and index.html
* Add a function to stop loading in index.js
* Invoke stop loading function if the data is not empty

Preview  ( Wait till 17 Sec)

LGTM!

Hey , @shiqimei Can you merge it? Or request the concerned person to merge it!?🙂

Hey , @shiqimei Can you merge it? Or request the concerned person to merge it!?🙂

@Sing-Li Sing, can you help merge this pr?


[REVIEW] Add AWS MNMG example

Adding Dask-cloudprovider example to run MNMG on AWS EC2

Just minor comments. I will run this tomorrow and check cloudprovider functionality.

I was able to run this with minor changes. Should be fine once addressed :D

Closes #122


Reproducing results on BAIR

Hi,
I do not want to mix up the issues, I open another issue for this. Again, thank you for providing the code.
The metrics for BAIR dataset is reported (Table 5) as 19.59 ± 0.27 (PSNR), 0.8196 ± 0.0084 (SSIM). However, when I run the test code with the instructions given in the README file (I add device 0 and batch size 32) and the pre-trained weights you shared, the results from the script are:
psnr 18.830994 +/- 0.274060565829277
ssim 0.80773425 +/- 0.008860558923333883
lpips 0.040237267 +/- 0.0024190666200593113
Do I miss something?
Thanks.

Hi!
We ran our test code with the command given in the README and the pretrained models that we provide, and obtained the same results as in our article.
Without additional information, it is difficult for us to determine the problem that you encounter. What is the command that you executed to obtain the results that you report ? You might also consider checking that you use the same computing environment as ours (given in requirements.txt`).

Hi,
thank you for the answer.
The environment I am using has the same modules with the requirements file and it python 3.7.6 with cuda 10.1. The comment I use:
python test.py   --data_dir /datasets/BAIR   --xp_dir models   --nt_gen 30   --lpips_dir lpips_weight/  --device 0  --batch_size 32
The xp_dir consists the pre-trained weight and the config file you shared.

We executed our test script with this command by cloning this repository, preprocessing the dataset and downloading the model from scratch, but we could not reproduce your issue as we still obtained the results reported in the paper. Your LPIPS score is particularly surprising as it is significantly better than the ones of all compared models in our work!
A possible explanation could still lie in your computing environment. Is it standard? We are operating on a Linux system and Nvidia GPUs such as Titan Xp, Titan V and RTX 2080 Ti.
In order to help us investigate your issue, could you please provide examples of ground truth and model predictions so that we can examine them? These are saved as NumPy arrays by the test script in the model folder.

Hi,
I am using Linux environment with Tesla V100 (32 gb) gpu. I will upload conda environment. I also upload the results and prediction to drive, the link is below. I only uploaded ssim, psnr and lpips best predictions.
env_info.txt
results.zip
Thanks.

We noticed a mismatch between your predicted sequences and the ground truth on our side. This might explain the issue. Did you follow the steps described here to preprocess the dataset?
If you want us to further investigate this matter, you can send us your BAIR testing data.

Thank you for the feedback. I think our dataset was processed by the code from svg repository and I thought it was going to same with your dataset since in paper, it is reported that you are using the same train and test set. I will probably process the dataset again and report the resullts on that.
Thank a lot.

No worries!
We follow the exact same procedure as SVG to preprocess the dataset. Therefore, its test set should in principle be the same than ours; it is indeed the case on our side. Furthermore, the results that we report for all models being consistent with those reported, for instance, in the SAVP article indicates that we also use the same dataset as them.
Please let us know the results that you obtain; we are interested in resolving any technical issue in the reproduction of results of all models in our experimental evaluation.

Hi again,
I preprocessed the data files that were downloaded before and got the results below. The PSNR and SSIM results are similar but again the LPIPS score differs too much. Here are the results:
Results:
psnr 19.56701 +/- 0.26613611459732056
ssim 0.82046306 +/- 0.008313765414059163
lpips 0.035872065 +/- 0.0020749701419845224
I will download the dataset and preprocess it again. Then, I will evaluate the model.

I downloaded the dataset and preprocess it with using your repository. The results are exactly same.
Results:
psnr 19.56701 +/- 0.2661360853910446
ssim 0.82046306 +/- 0.008313765414059163
lpips 0.035872065 +/- 0.0020749701419845224
My environment again matches with the requirements file. The run command is below:
python test.py   --data_dir /datasets/BAIR/srvp_version   --xp_dir models   --nt_gen 30   --lpips_dir lpips_weight/  --device 0  --batch_size 32
I also upload the predictions, ssim, psnr, lpips best predictions.
results
If you have any other suggestions,  I will try them.
Thanks a lot.

Thank you for the update!
Regarding the issue with respect to LPIPS, we think that you use the outdated LPIPS weigthts. Indeed, we used the most recent ones (v0.1). When we tested our model with the older ones (v0.0), we actually obtained the same results that you report.
We are sorry about the confusion: we specified the LPIPS version in our code only. We just modified the README.md file to clarify this point.
Can you confirm that using the most recent LPIPS weights makes your results match ours?

You are correct.
I guessed the version considering the publication date and I guessed it wrong :)
The results I got with v0.1:
psnr 19.56701 +/- 0.26613611459732056
ssim 0.8204631 +/- 0.008313765414059163
lpips 0.057495013 +/- 0.0031978254998102783
Thank you for your time and effort.

 We're now closing the issue as the problem was solved. Feel free to contact us if you have any other question.


Add donate to potlock functionality

Resolves #400

Hey @Megha-Dev-19, for record keeping sake, what is left to complete on this branch?

hey @elliotBraem, donate is complete, I was waiting to complete register, so that all projects can enjoy it, but if you want to get this in irrespective, I can work on the conflicts and merge it.


Sortable Transition Conflicts with other transitions

Say I have a list, and each item in the list has a background-color. The background color can change, so we put a transition: background 0.5s linear.
This disables the sortable transition, so we extend this to transition: background 0.5s linear, transform 0.3s ease.
However, this transform transition sticks around after the list is reordered. If you take item 2 and drag it into position 2, the following happens:

Item 2 is given a transform (e.g. translateY(-20px))
The items in the DOM are re-ordered, and item 2's transform is reset

Because the transform went from translateY(-20px) to (effectively) translate(Y), when the items in the DOM are re-ordered, item 2 is initially shifted, and transitions into its normal place.
See below for an example (animations slowed down)

Is there a nice way to handle this?
Workaround
If I reorder my transition to read transform 0s ease, background 0.5s linear, when you set transition-duration: 300, it'll update the first transition (but also clear the backgound transition).

bump
material-ui listitem same issue

@jacobp100 hi! just to cause you some nice minutes, thanks for posting this! i was struggling to figure out the problem, and it was a great hint that the transitions might conflicts with other transitions! :)
so, thanks again! 🤝


listenToOnce space-separated events only attaching callback once

http://jsbin.com/wuriku/1/edit?js,console
events.once('foo bar', callback);
events.trigger('foo');
events.trigger('foo');
events.trigger('bar');
events.trigger('bar');

console.log(count === 2); // >> true

events.listenToOnce(target, 'foo bar', callback);
target.trigger('foo');
target.trigger('foo');
target.trigger('bar');
target.trigger('bar');

console.log(count === 2); // >> false

Removing the (seemingly overkill) _.once would fix this.

Hi @thejameskyle! Thanks for pointing this out!
You're right that the function should be evaluated once for each space-separated event, but I don't think removing the _.once is the right approach. I'll pull a fix shortly.

but I don't think removing the _.once is the right approach

Why's that?

Why's that?

It's needed for https://github.com/akre54/backbone/pull/23#discussion_r19637176


Add unflattened jet/lep/V variables to array outputs; Add first GNN code

Crossref: https://gitlab.cern.ch/cms-analysis/hig/vhcc-run3/tasks/-/issues/9
The changes on this PR are dependent on the following change in the PoCo repo: https://github.com/PocketCoffea/PocketCoffea/pull/276
I would wait for the PoCo PR to be merged first before merging this one.

With save_arrays set to False, I get
ValueError: in ListOffsetArray64, cannot convert to RegularArray because subarray lengths are not regular.
But works fine with save_arrays True. Need to fix this.

The issue with saved_arrays will be solved in my next PR

Let me know when it's ready for merge

I converted this to draft because I'm now working on the odd/even splitting and implementing a weighted loss.
Besides, we need to wait for the PocketCoffea PR to be merged. I'll let you know.

@andreypz I added reverse compatibility with PoCo's old way of reading jets, i.e., without https://github.com/PocketCoffea/PocketCoffea/pull/276 being merged. I have not explicitly tested it, so perhaps you can confirm if it works.
So I think this can be merged now (provided you don't find issues with the jet selector). Will have to remove some temporary code when the PoCo PR gets merged.


ChangeTypeConverter does not handle URIs

Looks like URIs are a limitation of the converter in the present form.  solution for this is not clean with existing code (as nice as the converter is 👍 ).  Since there is no direct conversion from string to URI (since this is how this came about), it would require either some hackish code or a different converter which would you prefer?

The converter is limited to types from/to which can actually be cast. I think it should have been called the CastTypeConverter. Because of this, I hope you can see why string to URI can never work. And, I hope you can see why this is not a bug in the behavior. That being said, writing a small handler in the converter to handle this explicit conversion would be an okay solution, I think.

Ok :)


date display is garbled

packlist displays the lists with their date range. Unfortunately, the displayed dates are garbled, the order of year, month, day is not what I configured in the Android settings.

Dates are formatted using system conventions : see Trip.java line 151
        res = DateFormat.getDateInstance(DateFormat.SHORT).format(mStartDate.getTime());

This results in format change according to system language-locale in settings.
For example if I put my device in... 26th of march 2016 is displayed
french (france) : 26/03/16
french (canadian ) : 16-03-26
english (united states) : 3/26/16

In Settings->Language & input, I've set "Language" to "English". There's only one English, no regional choices. In Date & time, for "Choose date format" I chose 2016/12/31 instead of the first option "Regional (12/31/2016)".
F-droid for example displays the packlist 0.6 release as "Added on 2016/03/11", conforming to my chosen format.
packlist displays a trip starting today as 3/20/16, seemingly conforming to the regional format I have not chosen.
Interestingly, some pre-installed apps like Phone (e.g. 11/25/2015) also seem to get it wrong.
Osmand~ gets it right, e.g. 2016/01/17.
Unfortunately, I hardly know anything about Android programming, but to me this feels like there are two alternate ways of doing it, and one does it better than the other. ;-)
Or there's yet another place in the system settings where to configure the date format?
It's an Android 4.2.2, in case this explains this mess.

Thanks for your study,
Will try fixing it using suggestion in :
http://stackoverflow.com/questions/6981505/how-to-get-user-selected-date-format-in-android

Fixed in develop

Fixed in release 0.7.1

Hello code-surfer, could you please install 0.7.1, available on f-droid and playstore and tell me if fix is working fine, then close or update this bug.
Best regards, Nicolas

Testing with the lastest on F-Droid, 0.7.1, I can confirm that the date now gets formatted according to the chosen date format in the Android settings. (I also set changed them back and forth.) Well done!


Improve lb_df_fstype() results

Use other tools to a better detection of filesystems.
Currently we cannot make differences between FAT16/32 (vfat) or NTFS/exFAT (fuseblk).

Will be done in #95


FLUID-6465: Updated to latest versions of dependencies and latest Nexus

Tests pass fine under Node version 12, but not under version 13. The error seems to have something to do with Promise handling. Comparison of the two versions at this gist:
https://gist.github.com/ptcher/0e2fc613913a11d4878f9f33a5c5cb76
Is this a known issue with Infusion, or is it more likely an issue with this repository?
Do we care whether it works with the latest version of Node, or should we just aim for supporting the LTS version?

Thanks, this is worth knowing!
I may be wrong, but doesn't node follow the Linux-like convention that odd-numbered releases are considered "unstable"? So this problem may be due to some transitional bug that may end up being fixed. That said if it persists for some time we should look into it, but the acid tests presumably will be what happens with node 14.x releases.
By eye, it seems that a callback that should only be called once is being called twice, causing a promise to double-resolve.

As far as I can tell, it's not that odd-numbered major versions are unstable, just that only the even-numbered ones become LTS versions. Anyway, I am happy to say we only striclty aim to support the current LTS version of Node.
I think this is okay to merge, then?


NeuralynxRawIO - factor parsing of contiguous blocks of Ncs records

This pull request factors the parsing of contiguous blocks of records into a separate class. Then uses the static methods of that class to parse the ncs files and provide the time segments. Also incorporates new parsing of file types from the file headers in order to handle older Cheetah created files with rounded sampling rate conventions.
This also includes the samples in partially filled records at the end of a block of contiguous records. It may thus obviate the need for PR #881 .
This is again a smaller chunk of the work which is aimed to permit other ncs file styles and multiple data collection cards with slightly different start times.

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

In the file neo/rawio/neuralynxrawio.py:

Line 482:100: E501 line too long (103 > 99 characters)
Line 524:100: E501 line too long (106 > 99 characters)
Line 538:66: W504 line break after binary operator
Line 539:72: W504 line break after binary operator
Line 540:68: W504 line break after binary operator
Line 544:82: E226 missing whitespace around arithmetic operator
Line 558:100: E501 line too long (100 > 99 characters)
Line 560:86: W291 trailing whitespace
Line 595:32: W504 line break after binary operator
Line 596:82: E226 missing whitespace around arithmetic operator
Line 636:100: E501 line too long (119 > 99 characters)
Line 664:100: E501 line too long (100 > 99 characters)
Line 669:48: E226 missing whitespace around arithmetic operator
Line 671:100: E501 line too long (104 > 99 characters)
Line 716:100: E501 line too long (107 > 99 characters)
Line 717:100: E501 line too long (115 > 99 characters)
Line 718:100: E501 line too long (112 > 99 characters)
Line 726:100: E501 line too long (103 > 99 characters)
Line 728:100: E501 line too long (111 > 99 characters)
Line 740:100: E501 line too long (113 > 99 characters)
Line 742:100: E501 line too long (105 > 99 characters)
Line 745:100: E501 line too long (105 > 99 characters)
Line 752:100: E501 line too long (101 > 99 characters)
Line 767:100: E501 line too long (100 > 99 characters)
Line 768:100: E501 line too long (103 > 99 characters)
Line 771:48: E226 missing whitespace around arithmetic operator
Line 775:100: E501 line too long (106 > 99 characters)
Line 786:100: E501 line too long (110 > 99 characters)
Line 793:100: E501 line too long (100 > 99 characters)
Line 819:100: E501 line too long (100 > 99 characters)
Line 827:100: E501 line too long (104 > 99 characters)
Line 832:100: E501 line too long (104 > 99 characters)
Line 847:100: E501 line too long (113 > 99 characters)
Line 854:100: E501 line too long (102 > 99 characters)
Line 856:100: E501 line too long (103 > 99 characters)
Line 860:100: E501 line too long (102 > 99 characters)
Line 868:100: E501 line too long (111 > 99 characters)

In the file neo/test/rawiotest/test_neuralynxrawio.py:

Line 149:35: E226 missing whitespace around arithmetic operator
Line 151:100: E501 line too long (101 > 99 characters)
Line 167:44: E226 missing whitespace around arithmetic operator

Rebased this branch onto NeuralEnsemble:master at e97ac970 with the hopes of simplifying the merge with that master.
Not sure this succeeded in that. Nonetheless, it has no conflicts with the master and passes tests and incorporates the primary changes of using the NcsBlocksFactory methods to parse the blocks of records in Ncs files. This lays the groundwork for subsequent treatment of other file types and more complex starts and stops of different ADC cards used in some of the neuralynx systems.
Would appreciate advice for this project on how to best deal with rebasing/merging branches for pull requests when the branch in the pull request started some time back on the master.

Closed and incorporated into new PR.


Texts in logpoint are being printed twice in the Debug Console when debugging cells

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

In a code cell, set up a breakpoint and right click on it, click on "edit breakpoint" and select "log message" in the drop down menu
Add a message and hit enter
Click on "debug cell" and open the Debug Console. Make sure you step over the line with the breakpoint and watch the output being printed twice

Actual behavior

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running: Local
Extension version: 2020.5.76036-dev
VS Code version: 1.44.2
Python and/or Anaconda version: Python 3.8.0 32 bit
OS: Windows
Virtual environment: venv

Developer Tools Console Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

This would be a debugpy bug.


Eslint Pre-commit hook

i tried, didn't work.
will look into this tomorrow

i tried, didn't work.
will look into this tomorrow

closing this as seems ubuntu 20.04 is giving me some wierd errors!

closing this as seems ubuntu 20.04 is giving me some wierd errors!


Colloquial names are not so meaningful

Calvin and Hobbes certainly were able to invent a -ier word, yet this is not a comic strip, and I found your colloquial verbs to be more cutely distracting than informative. Specifically Datefier, DateValizer, Matrifier, Outliernicer, Statifier, and Monotonicer seem to describe objects, rather than functions.
I saw these as part of the JuliaCon proceedings review, yet there's no need to change anything for that review. This comment is just a minor comment that you're welcome to ignore.

Thanks for the comments.  There's no need to change anything.


Feat at all

#2394

Could you please also add an unit test so that we can see the behavior of the new method?
Thank you very much!

Could you please also add a unit test to see the behavior of the new method?
Thank you very much!

This PR will require changes in wechaty-puppet first. I'm asking to confirm that this design is fine with you, becuase I find it not very elegant.
this.wechaty.puppet.roomAnnounce(
        this.id,
        finalText,
        false,
        metionList.map(c => c.id)
}


Add Support for Apple Silicon (M1)

Hardware info
Machine: MacBook Air
OS: macOS BigSur 11.2.3
Chip: Apple M1 (8cpu, 8gpu)
RAM: 16GB
Software info
Python:3.8
PIP: 21.0.1
Miniforge: 4.10.1
Tensorflow-macos: 2.4.0-rc0 (0.1a3)
Bazel: 4.0.0-homebrew
Homebrew: 3.1.3
Description
I couldn't use pip install tensorflow-recommenders-addons to install the addon on my system, I tried Rosseta x86 terminal, but the 2.4.1 tensorflow can't run in Rosseta. So I tried to compile the source code by Bazel.
To compile the Apple Silicon version, I changed:
./.bazelversion
from 3.1.0
to     4.0.0
./build_deps/build_pip_pkg.sh
from
BUILD_CMD="${BUILD_CMD} --plat-name macosx_10_13_x86_64"
to
BUILD_CMD="${BUILD_CMD} --plat-name macosx_11_0_arm64
./requirement.txt
delete tensorflow=2.4.1
./tensorflow_recommenders_addons/version.py
from
MIN_TF_VERSION = "2.4.1"
MAX_TF_VERSION = "2.4.1"
to
MIN_TF_VERSION = "2.4.0-rc0"
MAX_TF_VERSION = "2.4.0-rc0"
Compiling Process
git clone https://github.com/tensorflow/recommenders-addons.git
cd recommenders-addons

# This script links project with TensorFlow dependency
python3 ./configure.py

bazel build --enable_runfiles build_pip_pkg
bazel-bin/build_pip_pkg artifacts

pip install artifacts/tensorflow_recommenders_addons-*.whl

Here is the wheel file:
tensorflow_recommenders_addons-0.0.1.dev0-cp38-cp38-macosx_11_0_arm64.whl
The Forked Repo:
tf-recommender-m1
Release Page
Current issue
I still get some warning messages while running the tutorial ipynb file.
The dynamic_embedding_tutorial.ipynb works properly except for some warning messages, but the embedding_variable_tutorial.ipynb raises an error as follows:
InvalidArgumentError: type of indices is not match with EmbeddingVariable key type.
I also tried to use Colab to run the embedding_variable_tutorial.ipynb, and I got the same error. I think it might not be my problem.

@candyzone

Hi @poinwater , thank for your feeadback. We have not officially supported m1 yet, but we will incorporate it into the development plan based on demand.  And I suggest you pull another issue for the EmbeddingVariable exception.

I will close this issue, anyone want to run TFRA on MacOS with M1 plz reference to the .


feat: add support for update utils command

Hi,
I tried to implement the feature discussed in this issue #298.
The original issue is that there is no way to update the utils.ts file. I added a tiny bit of logic to enhance the update command by recognizing the special keyword utils as @huntabyte suggested in his comment.

This might be an issue if a component is named utils, but I think it is highly unlikely to happen.

If the CLI is called with only utils in the argument components then it stops after updating the utils.ts file. Otherwise, it continues as normal (updating the other provided components / asking the users what to update).
I'm not too much familiar with this code base, so please, correct me if:

The utils file is actually in the registry, but from my exploration it is not.
Tell me if it is normal for the config.resolvedPaths.utils value to not contain the file extension, like if it was a folder?

From a previous contribution, I think I need to run a release command for semversionning. Please tell me what to do. I could not find any information on that matter in the CONTRIBUTING.md.

Before submitting the PR, please make sure you do the following

[x] If your PR isn't addressing a small fix (like a typo), it references an issue where it is discussed ahead of time and assigned to you. In many cases, features are absent for a reason.
[x] Prefix your PR title with feat:, fix:, chore:, or docs:.
[x] This message body should clearly illustrate what problems it solves.
[x] Follows the contribution guidelines.
[ ] Format & lint the code with pnpm format and pnpm lint

pnpm lint does not work in the folder /package/cli, I believe it should be replace by pnpm format:check` which has been run and passed successfully.

🎉 Awesome, thanks so much!

thank you!


Update bashedu.txt

Pedagogical now is branch of Bashkir State University

@fortunto2 Thank you for the update! Pull request merged.


Mongodocker  does not log the logs to /var/log/mongodb

I am using MongoDB 3.0 official docker image , in the configuration option it says the logs are being persisted in /var/log/mongodb/mongod.log but when i navigate to /var/log/mongodb/ there is no such file.
I am using the default mongo configuration.
This is what i got in my init.d file
# where to write logging data. systemLog:   destination: file   logAppend: true   path: /var/log/mongodb/mongod.log
But no logs are being logged , am i looking in wrong place or do i need to enable any other setting for this?

We don't use any configuration file by default, and logs should be going to stdout, and thus available via docker logs your-mongodb-container -- if that's not the desired behavior, you should be able to provide flags to the container's startup to modify the behavior (docker run ... mongo --logpath /somewhere/specific.log).

@tianon Cool makes sense , is it right that mongo use journal for loggings which can then be accessed via docker logs
Are these journal logs are stored for ever in the container?

@sri85 You might be confused with mongodb's "journal" file and "log" file. mongodb by default writes to a journal file for durability. When run mongod with a config file, log files can be optionally kept within the container, or use docker's VOLUME option to store log files on host or another container. If run within the container, you can config mongod to reopen or rename to a new log file.

@dbsrv Thanks for the clarification :)

Closing old issue. Feel free to comment on further troubleshooting if you feel this was closed in error or on the solution to help users running into the same problem.

@tianon Is it possible to provide that flag in a docker-compose file, I've tried over command, but it's acting strangely, considering my .log file as a directory. When I provide the flag in docker run it does work though (with the same image and port etc.)

@ivorbaric yeah, you just have to be careful about YAML syntax so you don't get sh -c ... added -- I'd recommend using the array syntax, ala:
services:
  mongo:
    image: mongo:4.x
    command: ['--logpath', '/somewhere/specific.log']

or:
services:
  mongo:
    image: mongo:4.x
    command:
      - '--logpath'
      - '/somewhere/specific.log'


Announcing new objects doesn't work

at least not for broadcasts, probably not at all

Either I fixed it accidentally, or something weird happened - anyway, I can't reproduce the problem.


KafkaBinderHealthIndicator with KafkaMessageChannelBinder when only KStream Binder

Hi,
When using KStream Binder with spring-cloud-stream, somehow KafkaMessageChannelBinder and KafkaBinderHealthIndicator also initialized. Because of that KafkaBinderHealthIndicator returns DOWN, it expects at least 1 binding to return UP status see:
https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/blob/b4a2950acd88df0f96178ca7ea2ba31e9cff43da/spring-cloud-stream-binder-kafka/src/main/java/org/springframework/cloud/stream/binder/kafka/KafkaBinderHealthIndicator.java#L112
Also new Health Indicator for Kstream binder recently added. https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/issues/544
I think KafkaMessageChannelBinder  and KafkaBinderHealthIndicator  should not be initiated at all if there is only KStream binding inside context. Is this a bug or config issue.

And this is the workaround
management.health.binders.enabled: false

And just a note: Since there is no KafkaStreamsBinderHelthIndicator in this master branch yet, this also a problem when KStream dies It is not possible to detect


the "other" routes in devise

when the user click in  "forgot your password"  goes to:
"http://localhost:3000/password/new"
but if I want other route like:
http://localhost:3000/recovery_password
can.t be done, I tried :
devise_for :user, :path => '', :path_names => { :passwords => "recuperar_password" }
as :user do
get    "recovery_password" => "devise/passwords#new"
end
and a lot of  "routes configurations" but nothing works...it  would be good a only way (ok, maybe 2) to change all the redirects,  routes...or...maybe Im so "beginner" that I dont figure out

Please use the mailing list or stack overflow for questions.


Fix import issue due to case-sensitivity

Under Linux I encountered an issue when developing for this extensions due to a case mismatch.

Thank you !


new filter

PR Description

Adds the defaults 10s filter length
Adds filter_kwargs parameter

closes #35 #18
I dont know if this is what you have in mind...
I'm also a bit busy again ( new exams, deadlines approaching etc) but I managed advance this a bit.

Codecov Report

Merging #40 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   97.01%   97.02%   +0.01%
==========================================
  Files           5        5
  Lines         502      505       +3
==========================================
+ Hits          487      490       +3
  Misses         15       15

Impacted Files
Coverage Δ

pyprep/prep_pipeline.py
100.00% <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 4bcde5e...269f075. Read the comment docs.

Originally clean line uses like 2-4 second windows with 50% overlap.

okay, that's what I was interested in, thanks. So now we are deviating from that ... but it's better than before, so I am fine with it.

Thanks @yjmantilla