Customizing release container: Unable to find image 'apolloauto/apollo:local' locally

I'm trying to follow the quick start guide and have followed steps 1-7 in the "Customize Your Release Container" section. However when I get to the "Launch the Local release env Docker Image" step, running bash docker/scripts/release_start.sh local results in these errors:
Unable to find image 'apolloauto/apollo:local' locally
docker: Error response from daemon: manifest for apolloauto/apollo:local not found.
See 'docker run --help'.
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Error response from daemon: No such container: apollo_release
Am I missing a step? Any guidance would be appreciated
link to guide: https://github.com/ApolloAuto/apollo/blob/master/docs/quickstart/apollo_1_0_quick_start.md](url)

hello, I have also the same problem. Have you found the solution towards this?

Oops, I just found that the TAG name in docker/scripts/release_commit.sh has become "local_release".
To confirm, please run
docker images | grep apolloauto
# There should be an image named "apolloauto/apollo:local_release"

Then the script to start it should be
bash docker/scripts/release_start.sh local_release

I will make the fix. Thanks very much for reporting!

Yes, I can confirm that bash docker/scripts/release_start.sh local_release launches the HMI as expected. (I actually figured it out on my own but did remember to update this issue!)


Wrap work execution in temporary build operation

So we can track execution times for all work in Gradle Profiler.
This is a precursor of the stable operation we'll add in https://github.com/gradle/gradle/issues/9117.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

I've added @ServiceScope. I believe I addressed all your requests, so I'm going to merge this once master is green.


Update the Gem?

redactor-rails gem 0.5.0 was released on February 24, 2015
Current codebase has commits from 2nd August 2015.
The reason I ask is because I just got caught out with the original image upload being returned instead of the content image.  The github codebase had the changes but the gem did not.  :/

Why not pull from master branch in the Gemfile:
#Gemfile
gem 'redactor-rails', github: "sammylin/redactor-rails"

@richpeck Because at any given moment your production code could stop working with a bad commit in the project.

And it does not work with turbolinks 5 :(

@NetScrn to make it works with Turbolinks 5 you can just do the following in config.js (generated by rails generate redactor:config): replace the ready page:load with turbolinks:load, so it will look like this:
before:
$(document).on( 'ready page:load', window.init_redactor );

after:
$(document).on( 'turbolinks:load', window.init_redactor );`


Set available device orientations for specific page

Hello guys.
I want to add specific page support for a few orientations.
Depending on the orientation, the view should be different.
In common StatefullWidget we can make it with:
  @override
  void initState() {
    super.initState();

    SystemChrome.setPreferredOrientations([
      DeviceOrientation.portraitUp,
      DeviceOrientation.portraitDown,
      DeviceOrientation.landscapeRight,
      DeviceOrientation.landscapeLeft,
    ]);
  }

  @override
  void dispose() {
    SystemChrome.setPreferredOrientations([
      DeviceOrientation.portraitUp,
      DeviceOrientation.portraitDown,
    ]);
    super.dispose();
  }

But how to specify available device orientations for specific pages using flutter_hooks?

I found solution:
    useMount((){
      SystemChrome.setPreferredOrientations([
        DeviceOrientation.portraitUp,
        DeviceOrientation.portraitDown,
        DeviceOrientation.landscapeRight,
        DeviceOrientation.landscapeLeft,
      ]);
    });
    useUnmount(() {
      SystemChrome.setPreferredOrientations([
        DeviceOrientation.portraitUp,
        DeviceOrientation.portraitDown,
      ]);
    });
    ```


Feature request: multiple entries

Is your feature request related to a problem? Please describe.
I would like to create separate bundles by providing two entry points for my Vue app - one will be loaded in index.html normally, the other one will be loaded dynamically using fetch() and eval() - I cannot use neither dynamic import() nor Vue Router in my use case.
Describe the solution you'd like
I would like to provide an array of entries in vue.config.js, as it is possible with Webpack,
Describe alternatives you've considered
There is some unofficial way in vue3-multiple-entry-points, but it is using dynamic import().
Additional context
This is the same as the closed issue #561. Possibly related to #257.

https://vitejs.dev/guide/build.html#multi-page-app

https://vitejs.dev/guide/build.html#multi-page-app

Is there a way to have multiple entries in library mode?


fix: More thoroughly clean up expired URLs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (f24f9cf) to head (356fea6).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #38   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          166       166
=========================================
  Hits           166       166

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


[tf.keras.layers.LSTM] reset_state performance issue

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): modified tensorflow/python/keras/layers/recurrent.py to get timings of LSTM reset (see code change below)
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: -
TensorFlow installed from (source or binary): 1.12.0, binary from PyPi
TensorFlow version (use command below): v1.12.0-0-ga6d8ffae09 1.12.0
Python version: 3.6.7
Bazel version (if compiling from source): -
GCC/Compiler version (if compiling from source): -
CUDA/cuDNN version: -
GPU model and memory: -

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
Time required to execute tensorflow.keras.Model reset_states() increases non-linearly with the number of stateful tensorflow.keras.layers.LSTM units, but only the first time reset_states() is called. The second time reset_states() it basically doesn't take any time at all.
A few numbers to highlight this - all numbers refer to the reset of a single LSTM (one model layer), not the whole model:

2 LSTMs: reset of the first LSTM takes ~20ms and of the last takes ~30ms (could be just a random variation)
20 LSTMs: reset of the first LSTM takes ~300ms and of the last ~700ms
200 LSTMs: reset of the first LSTM takes ~3500ms and of the last ~7000ms
I traced the delay down to tensorflow.keras.backend.set_value, so it obviously is really setting of the state that takes extremely long.

Describe the expected behavior
I would expect that the reset of a single LSTM (one model layer) stays constant independent of the number of LSTMs the model consists of. More importantly, I would expect that the time it takes to reset a single LSTM doesn't increase, but is constant for all of the LSTMs of a model..
Code to reproduce the issue
Import module datetime in tensorflow/python/keras/layers/recurrent.py and replace method of reset_states of class RNN by
  def reset_states(self, states=None):
    import datetime
    if not self.stateful:
      raise AttributeError('Layer must be stateful.')
    batch_size = self.input_spec[0].shape[0]
    if not batch_size:
      raise ValueError('If a RNN is stateful, it needs to know '
                       'its batch size. Specify the batch size '
                       'of your input tensors: \n'
                       '- If using a Sequential model, '
                       'specify the batch size by passing '
                       'a `batch_input_shape` '
                       'argument to your first layer.\n'
                       '- If using the functional API, specify '
                       'the batch size by passing a '
                       '`batch_shape` argument to your Input layer.')
    # initialize state if None
    if self.states[0] is None:
      if _is_multiple_state(self.cell.state_size):
        self.states = [
            K.zeros([batch_size] + tensor_shape.as_shape(dim).as_list())
            for dim in self.cell.state_size
        ]
      else:
        self.states = [
            K.zeros([batch_size] +
                    tensor_shape.as_shape(self.cell.state_size).as_list())
        ]
    elif states is None:
      if _is_multiple_state(self.cell.state_size):
        now = datetime.datetime.now()
        for state, dim in zip(self.states, self.cell.state_size):
          K.set_value(state,
                      np.zeros([batch_size] +
                               tensor_shape.as_shape(dim).as_list()))
        print(f"LSTM reset time: {datetime.datetime.now() - now}")  # TODO
      else:
        K.set_value(self.states[0], np.zeros(
            [batch_size] +
            tensor_shape.as_shape(self.cell.state_size).as_list()))
    else:
      if not isinstance(states, (list, tuple)):
        states = [states]
      if len(states) != len(self.states):
        raise ValueError('Layer ' + self.name + ' expects ' +
                         str(len(self.states)) + ' states, '
                         'but it received ' + str(len(states)) +
                         ' state values. Input received: ' + str(states))
      for index, (value, state) in enumerate(zip(states, self.states)):
        if _is_multiple_state(self.cell.state_size):
          dim = self.cell.state_size[index]
        else:
          dim = self.cell.state_size
        if value.shape != tuple([batch_size] +
                                tensor_shape.as_shape(dim).as_list()):
          raise ValueError(
              'State ' + str(index) + ' is incompatible with layer ' +
              self.name + ': expected shape=' + str(
                  (batch_size, dim)) + ', found shape=' + str(value.shape))
        # TODO(fchollet): consider batch calls to `set_value`.
        K.set_value(state, value)

Then set COUNT_LSTMS to the number of LSTMs that should be used in the following script and execute it:
import tensorflow as tf
import numpy as np

COUNT_LSTMS = 200

BATCH_SIZE = 100
UNITS_INPUT_OUTPUT = 5
UNITS_LSTMS = 20
BATCHES_TO_GENERATE = 2
SEQUENCE_LENGTH = 20

# build model
my_input = tf.keras.layers.Input(batch_shape=(BATCH_SIZE,
                                              None,
                                              UNITS_INPUT_OUTPUT))
my_lstm_layers = [tf.keras.layers.LSTM(units=UNITS_LSTMS,
                                       stateful=True,
                                       return_sequences=True)(my_input)
                  for _ in range(COUNT_LSTMS)]
my_output_layer = tf.keras.layers.Dense(UNITS_INPUT_OUTPUT)
my_output = tf.keras.layers.TimeDistributed(my_output_layer)(
    tf.keras.layers.concatenate(my_lstm_layers, axis=-1))
my_model = tf.keras.Model(my_input, my_output)

# generation
pred_input = np.random.randn(BATCH_SIZE, 1, UNITS_INPUT_OUTPUT)

for batch in range(BATCHES_TO_GENERATE):
    print('resetting states')
    my_model.reset_states()
    print(f"start generation of batch {batch}")
    for _ in range(SEQUENCE_LENGTH):
        pred_input = my_model.predict(pred_input, batch_size=BATCH_SIZE)

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.

@eega,
Sorry for the delayed response. Can you please confirm if the problem still persists? Thanks!

Sorry, no, I'm not in the position anymore to verify that issue.


reject: private keys are checked in: new:usr: adding code for P008

Code for Project 008

Private key is encrypted using ansible-vault. It cannot be viewed.  This was placed to enable cross SSH and use one floating IP.

We are not acceoting any checkins of any private key. This is policy.
Please fix

On Apr 24, 2017, at 8:26 PM, naveenkumar2703<EMAIL_ADDRESS>wrote:
Private key is encrypted using ansible-vault. It cannot be viewed. This was placed to enable cross SSH and use one floating IP.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/cloudmesh/sp17-i524/pull/444#issuecomment-296857532, or mute the thread https://github.com/notifications/unsubscribe-auth/AAZ8VSliwzVtktY9HA8rGNNz577nuAnkks5rzT2wgaJpZM4NFqQT.


add tests for orch. and a cross. .travis.yml file

We are not going to run full integration test on travis.


URGENT: Fixes compile time bug based on non-existent member

Fixes:
/home/bcarlson/testing/OneWifi//source/core/wifi_ctrl_rbus_handlers.c:415:47: error: ‘wifi_preassoc_control_t’ has no member named ‘tcm_client_deny_assoc_info’; did you mean ‘client_deny_assoc_info’?
  415 |         strncpy(vap_info->u.bss_info.preassoc.tcm_client_deny_assoc_info, str,
      |                                               ^~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                               client_deny_assoc_info
/home/bcarlson/testing/OneWifi//source/core/wifi_ctrl_rbus_handlers.c:416:50: error: ‘wifi_preassoc_control_t’ has no member named ‘tcm_client_deny_assoc_info’; did you mean ‘client_deny_assoc_info’?
  416 |             sizeof(vap_info->u.bss_info.preassoc.tcm_client_deny_assoc_info));

compile-time errors introduced in commit ec9235ccd55fce1c1f76dd804187a2582cab04c6
To Replicate
mkdir testing && cd testing
git clone https://github.com/rdkcentral/OneWifi.git
git clone https://github.com/rdkcentral/unified-wifi-mesh.git
cd OneWifi && git checkout ec9235ccd55fce1c1f76dd804187a2582cab04c6
make -f build/linux/makefile setup
make -f build/linux/makefile all

@HarshavardhanP1

Problematic commit ec9235ccd55fce1c1f76dd804187a2582cab04c6 has been reverted. Closing this PR.

Problematic commit ec9235ccd55fce1c1f76dd804187a2582cab04c6 has been reverted. Closing this PR.


Enable config manipulation after parsing

Provide a way to safely (if possible without direct downcasting) manipulate configurations after parsing. Possible use-case: Transform relative into absolute Target-paths.

qzed@c4bc3d54fd6ed27f597a56ca76366b970a455333 adds visitor-functions which simplify editing Config trait-objects. The whole Target-path-changing-situation could be implemented directly on top of that, however, I don't think this is a good approach (e.g. it's quite easy to miss a visitor for some type). A solution to this should provide direct support for the built-in defaults and a structure that does not make it too easy to miss a visitor-implementation for a custom-defined type.


Transition from NextJs to ReactJs

Transition from NextJs to ReactJs due to import of PDF to GovTool

@ValMBO @teske00 @vpaunovic Should this be considered done?

@bosko-m I think yes, we are done with this and we have react package

Marked it as done


Using Ionic2 Without CLI

Hi,
I would like to integrate Ionic2 lib to my current working project which already has a lot of Webpack specific configuration + Cordova (It's a website + native apps).
I'd like to keep my current configurations I'm already using without a lot of changes - therefore I don't want to use the Ionic CLI.
What's the minimum requirements I need to do in order to integrate Ionic2 SDK into my current project without using the Ionic CLI?
Thanks

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum (forum.ionicframework.com). Thanks for using Ionic!


for cmake3 make tbb an imported target

addresses an error when exporting a target that depends on tbb
I described this error with more detail in #5. I am by no means a cmake expert, but I needed this change for my work, and I wanted to contribute back since I have found findTBB.cmake helpful.

No, I did not run the test project. Let me do that and let you know how things go. I just ran into some weirdness that I need to investigate, so put this pull request on hold for now. (Is there a formal way to do that?) I will send you a note when I have tried the test and think it is ready.
Phil
From: Justus Calvin<EMAIL_ADDRESS>Sent: Wednesday, October 26, 2016 11:56 AM
To: justusc/FindTBB<EMAIL_ADDRESS>Cc: Philip Miller<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [justusc/FindTBB] for cmake3 make tbb an imported target (#6)
@justusc requested changes on this pull request.
Thanks for the pull request.
The initial changes look OK, but similar changes will also need to be applied to the other branches below.
Did you by any chance run the test project?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/justusc/FindTBB/pull/6#pullrequestreview-5888452, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABcK4PMnvEaZZQ2-0IpulMdz-bUsItFaks5q33gGgaJpZM4KhV2E.

Exporting external targets into a config file is a non-trivial process. The documentation on the procedure is very sparse. The trick is that you have to recreate the external targets in your exported config file. The recommended procedure is: (1) export the find module into the same directory as your config file, and (2) write your config file to use the find module to recreate the external target (i.e. the TBB target). Another approach is to use the TBB variables to set the appropriate compile options for your exported libraries. Unfortunately the second option precludes using the TBB target in your project as it will result in errors.
The test project is just the project I used to ensure that FindTBB is working as expected and can be used to build an executable. To run the test project run cmake . in the test directory.
Try modifying the test project to be a minimal example of what you are trying to do in your project.
I will look into this issue in more detail later this week, if you don't beat me to it.

Well, I gave it my best effort. The problem with my first attempt was I used INTERFACE_LINK_LIBRARIES for an imported target, which is not correct. I just pushed a commit to my pull request where I use IMPORTED_LOCATION_<CONFIG>, which I think is the correct way to configure an imported target. I also added to the test project install export rules that demonstrate the original error that made me go down this path in the first place.
I tried to put in changes for the other two branches (where (1) only TBB_LIBRARIES_RELEASE is defined, or (2) neither TBB_LIBRARIES_RELEASE nor TBB_LIBRARIES_DEBUG are defined), but that logic seems a little strange and testing it with msvc14, which always defines both, seems to take more time than I have right now.
I hope this PR is helpful.

I think you are going in the right direction, but exporting targets in CMake is a tricky business. In my own projects, I just used the TBB variables directly to create executables and libraries in config files. I will try to get it working as soon as I have some free time. Hopefully this weekend.

I think the focus of this issue and the pull request is to change the tbb target from being an INTERFACE target to being an IMPORTED target. (This is what FindBoost.cmake does in cmake 3.6).
Yes, I understand that exporting targets in CMake is tricky; I admit that I understand little about it, so I appreciate you taking time to look at this. Also, by no means do I think my pull request has a good example of exporting a target. It is just a demonstration of the error I encountered when exporting a target that used the tbb target.

The changes you made in your PR were mostly correct. I made a minor change to the FindTBB.cmake file. Otherwise, everything looked good.
I also expanded the test to verify that the FindTBB script can be used by projects that export executables and shared libraries that depend on TBB.

Thank you! I appreciate you making FindTBB available and your time incorporating this request.


update Immich to v1.88.2

Thank you! Published at: https://community.bigbeartechworld.com/t/update-immich-to-v1-88-2-on-bigbearcasaos/310


An 'empty' tab spot on Help that also prevents any further tabbing

Can no longer confirm so closing


Refactor of notes handling for file ingredients

WIP on updates for notes handling for file ingredients.

Commit cbb8032 closes numerous TODOs for adding a test handler for directrory.

@taigrr fixes added and ready for review!


fix(wasm): remove hardcoded isolation level

Summary
This was added during local development to work around a bug but is no longer desired and disallows setting the isolation level via kong.conf.
Checklist

[ ] The Pull Request has tests - not sure how to test :thinking:...
[ ] There's an entry in the CHANGELOG
[ ] ~There is a user-facing docs PR~

Let's wait until https://github.com/Kong/kong/pull/11387 is merged, at which point I'll add a changelog entry and update kong.conf.default to reflect the fact that proxy_wasm_isolation is supported.


goboardup.com/wp-admin is my legit website.

Could you take out this warning?  It is my website.   I cannot access it right now

The domain is blocked due to the following sub-domain -
extension-metamask.com.goboardup.com
Once a scam appears on a sub-domain, the root domain is also blocked.


Factors covariate reads out calculations

This way the covariate read and selection is only done once. This gave a slight, but noticable, speedup in my testing.

Oh, you are right. I like it!


🧹 Cleanup a few TODOs

How to categorize this PR?

/area dev-productivity
/kind cleanup
What this PR does / why we need it:

Drop scaling of cluster-autoscaler in generic Worker actuator, follow-up of https://github.com/gardener/gardener/pull/8010
Drop clean-up of old fluent-bit resources, follow-up of https://github.com/gardener/gardener/pull/8240
Cleanup legacy kuberbacproxy component, follow-up of https://github.com/gardener/gardener/pull/8216
Drop removal of from-world-to-ports annotation, follow-up of https://github.com/gardener/gardener/pull/8302
Drop cleanup of Reconciled condition in Gardens, follow-up of https://github.com/gardener/gardener/pull/8238
Drop nil check for .status.machineDeploymentsLastUpdateTime in Worker, follow-up of https://github.com/gardener/gardener/pull/7774

Release note:

NONE

/approve


Better list empty state

Moves add button into the empty state itself
Before

After

Tweaked the "Add" button, not amazing but I think it improves the visual balance

Mobile
Web


highlight (size) titles and text headers, differentiate

text content wrapper box

bring back horizontal separators with tasteful border, one line per control can no longer be relied on


DEA doesn't properly manage nats server failure

Hi all,
I am working on some resiliency test that involve NATS and I am facing a weird issue with DEA upon a specific sequence of NATS crash simulation.
Environment
I am using a test cf deployment  and not a DevBox (BoshLite).
I've a deployment with two NATS servers:  NATS/0 and NATS/1.
Scenario
- STEP 1: CF is properly working, I simulate a crash of NATS/0 by stopping the process via monit stop;
- STEP 2: I verify that NATS/1 is being contacted by issuing a tcpdump command
- STEP 3: I start again the NATS/0 instances and stop NATS/1
Upon STEP3 I noticed several failures in DEA and CC.  I run a script that verify DEA connections to nat (it issues  netstat -alep | grep  nat on each DEA) and I didn't find  connections in place.
I gave a look to the code and according to me it seems we have an integration issue among the DEA code:
-https://github.com/cloudfoundry/dea_ng/blob/4b1a50ae5598b0c70cb3e5895ed800e0cff37722/lib/dea/nats.rb#L101-L110
and nats client at (these lines are the same as in gem nats-0.5.0.beta.16)):

https://github.com/nats-io/ruby-nats/blob/master/lib/nats/client.rb#L790-L807

When NATS/0 was stopped on the STEP 1, nats client remove that server from
server_pool list ,then when NATS/0 is up again, no one re-init this list, dea code doesn't initialize again the client.
At STEP 3 NATS/1 is removed from server_pool list  that remains empty.
To support this conjecture I was not able to find any entry in the DEA logs related to "nats.connecting".

Hi @aalbanes,
We've addressed this here in a similar issue that you opened recently; read details there.
Thanks for opening this issue — apologies to leave you hanging —
@Quintaminant & @rmasand of CF CAPI

Hi @aalbanes,
Just a follow up to what @Quintaminant and @rmasand said: while we are making larger architectural changes (Diego, Routing API, internal service discovery) to phase out NATS, it will require further investigation to understand the current impact of the issue you raised, and the cost of solving it especially as it pertains to delaying the longer-term solution of phasing NATS out. @fraenkel and @sykesm will investigate further, and open up new Issues/PRs as appropriate.
This comment is cross-posted here
Thanks,
Amit, CF Release Integration team


Package doesn't include generated files into project (Not usable)

Actual behavior
The generated files are not included in the project and are therefore unusable to the application code.
Expected behavior
Like any other generator I expect the generated output to automatically be included as code in the project.
Additionally, I'd expect to be able to set the full namespace of the generated content. (It seems quite limited at the moment and always ends up including a nested 'PInvoke' namesapce I don't want.
Repro steps

NativeMethods.txt content:

SetWindowPos

NativeMethods.json content (if present):

[NOTE NativeMethods.json is, at present an undocumented thing... WHat is it? Better docs would help a lOT, one GIF doesn't tell a clear story]

Any of your own code that should be shared?

Context

CsWin32 version: [e.g. 0.1.560-beta]
Target Framework: [e.g. net5.0-windows10.0.19041.0]

NativeMethods.json is, at present an undocumented thing

It is documented. The readme introduces it and if you use it as it guides you, then a decent JSON editor (including VS) will use the $schema property to give you auto-completion and intellisense documentation on the options available.

The generated files are not included in the project and are therefore unusable to the application code.

CsWin32 is implemented as a roslyn Source Generator and has no say in how the generated files are included in the project. This is a feature of the C# compiler and its integration with VS and msbuild. If this isn't working for you, my guess is there's something not configured right in your setup somehow or you're hitting a bug.
If you can include a build log (with dotnet build -bl) of your project's failed build we might be able to help here. But getting support from dotnet/roslyn may be where we send you if it's not a CsWin32-specific thing.


SegmentMetadataQuery should have flags for specific kinds of metadata

The reason being that some kinds of metadata (like estimated size) can be expensive to compute. Callers may not want all of them. I think a reasonable way to do this is to have an enum of flags and to have the query take an EnumSet of them. The default should probably be what exists now, for backwards compatibility.

which kinds of metadata do you think should be optional? Do you have anything specific in mind?

The estimated size for sure, since it ends up reading a bunch of parts of the columns that otherwise wouldn't need to be read.

But also possibly everything??

This was done through analysisTypes.


Make generate_mk.py Python 3 compatible

Shelve 28082022, imported from internal gitlab.

This is a change from gitlab that I moved here to merge -- can one of you approve this? Looks like I can't approve PRs that I created.


WIP feat: add webkms client support in web-wallet

closes #541
Signed-off-by: Baha Shaaban<EMAIL_ADDRESS>

closing this PR as it's now stale


x/pkgsite: package removal request for github.com/hugo-cachon/vitals

What is the path of the package that you would like to have removed?
github.com/hugo-cachon/vitals
Are you the owner of this package?
Yes.
The previous url of the project was 'github.com/hugo-cachon/vitals' before my username change. It is now 'github.com/xororist/vitals'.

What is the reason that you could not retract this package instead?
This package has only been made for testing purposes and I don't want it to be published publicly.

I confirm that github.com/hugo-cachon/vitals redirects to github.com/xororist/vitals and have made the necessary change. Please allow a few days for configuration to propagate.


Move expected_loglik here & additional design discussion

Complement to https://github.com/JuliaGaussianProcesses/ApproximateGPs.jl/issues/53
The expectation of the log-likelihood under q(f), ∫ q(f) log p(y | f) df, isn't specific to ApproximateGPs themselves, and if someone added a new likelihood here, it'd be great if they had visibility into also providing e.g. analytic implementation of expected_loglik or at least thinking about what _default_quadrature should be.
expected_loglik currently has the following signature:
expected_loglik(quadrature::QuadratureMethod, y::AbstractVector, q_f::AbstractVector{<:Normal}, lik)

Some questions on the design:

To me it would seem more natural to either order it as
expected_loglik(quadrature, q_f, lik, y) or expected_loglik(quadrature, lik, q_f, y) -- we're unlikely to dispatch on y (it's implicitly defined by what the likelihood is already), and the first form matches the common maths notation much more closely.

expected_loglik takes a Vector y, a Vector q_f, and a lik function that maps from scalar f to UnivariateDistribution; expected_loglik then does the broadcasting internally. Is that what we want?

Would it be better (cleaner) to have expected_loglik handle scalars only (but this might result in some performance loss for e.g. recomputing the Gauss-Hermite points and weights for each data point)?
Should we expect lik to take the full Vector fs and return e.g. a Product() of UnivariateDistributions (though this might make the expectation code more complicated)?
How would we want to handle the heteroskedastic case, where we do want to include the correlations across the two outputs at each data point, but independence between different data points (not sure how we would handle that on the AbstractGPs side given that multi-output is all "output as input")?

If we want to support e.g. Expectation Propagation as well, we should also provide ∫ q(f) p(y | f) df: the quadrature fallbacks are going to be exactly the same, just evaluating pdf instead of loglikelihood (NB- should that be logpdf? the difference is in summing over ys...). Should we have expected_pdf separately (maybe re-using internal quadrature methods)?

To me it would seem more natural to either order

Could we make lik the first argument such that one can use Julia's do syntax (see https://docs.julialang.org/en/v1/manual/style-guide/#Write-functions-with-argument-ordering-similar-to-Julia-Base)?
And maybe one could use the name expected_loglikelihood or mean_loglikelihood to be more consistent with StatsBase.loglikelihood?

Could we make lik the first argument such that one can use Julia's do syntax (see https://docs.julialang.org/en/v1/manual/style-guide/#Write-functions-with-argument-ordering-similar-to-Julia-Base)?
And maybe one could use the name expected_loglikelihood or mean_loglikelihood to be more consistent with StatsBase.loglikelihood?

I really like both ideas and have a preference for expected_loglikelihood.

How would we want to handle the heteroskedastic case

I think it's fair to provide specialized methods for the 2D and maybe 3D cases (by D I mean the dimension of the GP input to the likelihood), because quadrature would not scale higher.

And maybe one could use the name expected_loglikelihood or mean_loglikelihood to be more consistent with StatsBase.loglikelihood?

If that's what we mean, I'd agree (not mean_loglikelihood though, that's making the dependence on q(f) too subtle IMO), but I'm wondering if that's the case or what we actually mean here is expected_logpdf?

I think it's fair to provide specialized methods for the 2D and maybe 3D cases (by D I mean the dimension of the GP input to the likelihood), because quadrature would not scale higher.

From an implementation point of view, yes, that's why we used Monte Carlo for the expectation with softmax likelihood for multi-class classification. But it would be good if neither SVGP in ApproximateGPs nor the end user would have to call different methods depending on their type of likelihood.

Could we make lik the first argument such that one can use Julia's do syntax (see https://docs.julialang.org/en/v1/manual/style-guide/#Write-functions-with-argument-ordering-similar-to-Julia-Base)?

I don't think expected_loglikelihood (or whatever we end up calling it) would be called a lot directly by the user/ from the REPL, so I don't think there's much value in it. The lik likelihood object is specified very indirectly through the LatentGP constructor.

what we actually mean here is expected_logpdf?

I don't think it's what we mean since sometimes likelihoods are not pdfs...

would have to call different methods depending on their type of likelihood.

We can easily dispatch one unique method and have underlying _expected_loglikelihood_2d etc...

The lik likelihood object is specified very indirectly through the LatentGP constructor.

I actually like the idea of the do syntax, especially if one uses custom weird likelihood. It's similar to the problem you mention in #43 where you don't want tobuild a link explicitely

I don't think it's what we mean since sometimes likelihoods are not pdfs...

Ah, that might be an issue of terminology overloading. "Likelihood" can mean the object p(y | f) (which we can represent as (f, y) -> pdf(dist_y_given_f(f), y)), or the value of p(y | f) for a certain value f (for fixed y) - which generalises to the 1d function f -> pdf(dist_y_given_f(f), y). But in terms of code, I think it makes most sense to think of the likelihood as dist_y_given_f as a function from a value f to a Distribution.
Distributions defines logpdf and loglikelihood roughly as follows:
logpdf(univariate_distribution, ys::Vector) = map(y -> logpdf(univariate_distribution, y), ys)
loglikelihood(univariate_distribution, ys) = sum(logpdf(univariate_distribution, ys))

We could maybe think of the generalisation to VI and EP as
expected_(logpdf, qf, ys, quadrature) and expected_(pdf, qf, ys, quadrature)...


create the temporary package directory when performing copy_to. Fixes…

… if a version of the package was not cached in belay folder

Codecov Report

Merging #121 (68c2e6a) into main (9ac6ddd) will decrease coverage by 0.34%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   76.61%   76.27%   -0.34%
==========================================
  Files          32       31       -1
  Lines        1317     1311       -6
  Branches      213      213
==========================================
- Hits         1009     1000       -9
- Misses        239      243       +4
+ Partials       69       68       -1

Flag
Coverage Δ

unittests
76.27% <0.00%> (-0.34%)
:arrow_down:

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

Impacted Files
Coverage Δ

belay/packagemanager/group.py
76.13% <0.00%> (-0.88%)
:arrow_down:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Clicking x does not reset search

Description
On Chrome browser when searching a small cross appears next to the word. When it is clicked the word disappears but the search does not get reset so it continues to show only the issues which match the search.
to do: When the x is clicked the search term should reset to "" and all items should be shown again.
Screenshots
No response
Additional information
No response

Description
On Chrome browser when searching a small cross appears next to the word. When it is clicked the word disappears but the search does not get reset so it continues to show only the issues which match the search.
to do: When the x is clicked the search term should reset to "" and all items should be shown again.
Screenshots
No response
Additional information
No response

Yup, it's clearing it up from the bar, but it's not reseting the items.

@emmalearnscode , @KendallDoesCoding
I guess better approach to handle this UI scenario will be:

to  remove the default cross added by chrome in the type=search input
create a custom cross feature to work across different browsers.


[image caption] saved models have higher loss

when I load the saved model and resume the training
the loss is always higher than it supposed to be
I also have saved the optimizer but the result is same
the resumed training always have higher loss than it supposed to be
anyone know what make this happen? and how to fix it?

The provided checkpoints were saved at intermediate step, not final step. Therefore, it can have higher loss than the final checpoint.


Moving tool installation to step prior to matrix generation

Moving tool installation to step prior to matrix generation.
If the tools haven't been cached in a new branch, each test will clobber the cache, causing the tests to install the tools every time.  This re-arranges the order so that we ensure tools are installed prior to creating the test matrix.
This also modified the watchdog timer from always_comb to generate as discussed in https://github.com/chipsalliance/caliptra-rtl/issues/71

Two requests:
(1) Let's file an Issue regarding upgrade to Verilator 5.010 (or later if available) (2) Can we add a result checker for the Verilator Workflow? See #71 for recommendations.

https://github.com/chipsalliance/caliptra-rtl/issues/79
done - check out the latest changes


Gateway fails to run state machines until it's restarted

We are running a federation with v0.1.3 release. It is in a region with pretty poor internet connectivity in general. Everything was working smoothly for a few days. Then all-of-a-sudden lightning payments started to fail. If you generated an invoice using fedimint-cli, and called fedimint-cli wait-invoice it would get to the WaitingForPayment state and get stuck there.
From the gateway logs (which are pretty sparse) it seemed like the gateway's handle_htlc_stream was seeing the HTLC, and this database commit was being hit.
But the gateway's fedimint transaction buying the preimage from the federation was never being confirmed by the federation. In fact, it seemed like the submit_transaction RPC method wasn't getting called because this log wasn't showing up. But if we restarted the gateway, both that log and the transaction itself would show up. The fedimint-cli wait-invoice... command which was previously stuck on WaitingForPayment also promptly completed.
We thought the root cause was jsonrpsee slots getting exhausted so we tried https://github.com/fedimint/fedimint/pull/3471 and https://github.com/fedimint/fedimint/pull/3470 but neither worked.
We decided to spin this gateway down and setup another to get things working again. In the process of withdrawing funds, the withdrawal command from gateway-cli also failed to complete until the gateway was restarted which suggests the problem is somewhere in the state machine executor.

Call: we need better tooling to inspect state machines when debugging things like this

One of the problems is due to payment taking too long. Currently we have this:
impl GatewayPayInvoice {
    fn transitions(
        &self,
        global_context: DynGlobalClientContext,
        context: GatewayClientContext,
        common: GatewayPayCommon,
    ) -> Vec<StateTransition<GatewayPayStateMachine>> {
        vec![StateTransition::new(
            Self::await_get_payment_parameters(global_context, self.contract_id, context.clone()),
            move |_dbtx, result, _old_state| {
                Box::pin(Self::transition_buy_preimage(
                    context.clone(),
                    result,
                    common.clone(),
                ))
            },
        )]
    }

On transition_buy_preimage we will send a
client
                .router()
                .send_payment_v2(SendPaymentRequest {
                    payment_request: invoice.clone(),
                    allow_self_payment: true,
                    no_inflight_updates: false,
                    timeout_seconds: LND_PAYMENT_TIMEOUT_SECONDS,
                    fee_limit_msat,
                    ..Default::default()
                })

But what if we are paying a HODL Invoice?
It won't timeout because the payment is InFlight, it won't also fail. So it will wait for it to settle for minutes, hours, days...
But note that we are waiting inside a STATE TRANSITION, which is global for all state machines. So everything else on the gateway will freeze/not progress.
So the solution is probably to move this InFlight payments to a specific state. @m1sterc001guy thoughts?

Ah yes, nice catch. This is a problem. I think we perhaps forgot to move this back into the trigger function once we made our LN pay implementations idempotent. Either way, the fix should be to move this function into the trigger function.
I actually don't think we want a new state. I think we want to combine this with await_payment_parameters (and give that function a new name) since we should validate the payment parameters before attempting to pay.
I think almost all of transition_buy_preimage can be moved into the trigger function (or split into multiple trigger functions). The transition function should essentially just validate the result and return the new state.


Sort_by function not work very well with hungarian letters

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
st 0.9.1
Did you try the latest code to see if this problem got fixed?
Tried, but the problem still
yazi --debug output

    Version: 0.2.5 (53d658d 2024-05-26)
    OS: linux-x86_64 (unix)
    Debug: false

Ya
    Version: Ok(Output { status: ExitStatus(unix_wait_status(0)), stdout: "Ya 0.2.5 (53d658d 2024-05-26)\n", stderr: "" })

Emulator
    Emulator.via_env: ("st-256color", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect: Unknown([])

Adaptor
    Adaptor.matches: X11

Desktop
    XDG_SESSION_TYPE: Some("tty")
    WAYLAND_DISPLAY: None
    DISPLAY: Some(":0")

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL: Some("/bin/bash")
    EDITOR: Some("micro")
    ZELLIJ_SESSION_NAME: None
    YAZI_FILE_ONE: None
    YAZI_CONFIG_HOME: None

file(1)
    Version: Ok(Output { status: ExitStatus(unix_wait_status(0)), stdout: "file-5.44\nmagic file from /etc/magic:/usr/share/misc/magic\n", stderr: "" })

Text Opener
    default: Some(Opener { run: "$EDITOR \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block: Some(Opener { run: "$EDITOR \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

tmux
    TMUX: false

Ueberzug++
    Version: Ok(Output { status: ExitStatus(unix_wait_status(0)), stdout: "ueberzugpp 2.9.5\n", stderr: "" })

Describe the bug
Sort of accented characters does not yet work properly.

Here you can see that it doesn't handle accented characters properly because it actually sorts by the second letter.
Expected Behavior
The correct sequence is always the following regardless of what the second character contains
A....
Á....
...
E
É
...
I
Í
...
O
Ó
Ö
Ő
...
U
Ú
Ü
Ű
...
To Reproduce

Configuration
No response
Anything else?
No response

You need to enable it in your yazi.toml like this:
[manager]
sort_by              = "natural"
sort_transliteration = true

Note: I just resubmitted the commit https://github.com/sxyazi/yazi/commit/1ab3df6850640c034a624566abcc8ef968c77a89 and renamed sort_transliteration to sort_translit because it was too long lol.
See https://github.com/sxyazi/yazi/pull/1053 for more details.

sort_by              = "natural"
sort_transliteration = true

I did it before make this issue, and does not yet work properly.

Weird, it works for me.

Could you record a gif to show me your steps? Include the Yazi version, configuration file, and sorting operation.

The example you show only works because the second letter of words that start with accented letters is further away.
Create the files with the following names, and then it won't be good for you too:
ábrahám
aladár
balassagyarmat
édrián
elemér
ubul
údrira

Is this the correct order?

Is this the correct order?

No. The correct sequence is always the following regardless of what the second character contains:
A....
Á....
...
E
É
...
I
Í
...
O
Ó
Ö
Ő
...
U
Ú
Ü
Ű
...
So:
aladár
ábrahám
balassagyarmat
elemér
édrián
ubul
údrira

This is how sort_translit works: when you enable it, it removes diacritical marks from characters in a string, like converting Á to A for sorting purposes. This means Á and A are treated as equivalent and sorted according to the general character order, grouping similar characters together.
I'm not sure if there's a better way to achieve this.

Accented characters (such as á, é, í, ó, ö, ő, ú, ű) are not equivalent to the unaccented versions. Each language has its own accented characters, and they follow a given rule.

If different languages have different processing rules, it would be more complex.
Currently, Yazi has a built-in character mapping table that simply replaces characters without considering the differences between languages.


Release 0.4.0

Release 0.4.0

changelog
version bump
package name
linting

Codecov Report

Merging #158 (5b0961e) into main (9682a24) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #158      +/-   ##
==========================================
- Coverage   82.92%   82.88%   -0.04%
==========================================
  Files          39       39
  Lines        2799     2799
==========================================
- Hits         2321     2320       -1
- Misses        478      479       +1

Impacted Files
Coverage Δ

skbase/base/_pretty_printing/_pprint.py
86.00% <ø> (ø)

skbase/__init__.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Reduced requirements

Pull Request
I have scanned through the repository and have commented out all the extra requirements. Please review and then I will be sure and remove all the extra requirements.
Checklist:

[X] My code follows OCF's coding style guidelines
[X] I have performed a self-review of my own code
[X] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have checked my code and corrected any misspellings

This looks good, how do you know you havent missed a requirement?

Happy to merge @peach280 ?


require doesn't work properly with v1.7.4

Hi,
I'm using your plugin v1.7.3, and everything works fine. I'm using require to import your lib in a js module, and then browserify to compile the main javascript file.
Passing to v1.7.4, it seems that bootstrap-select isn't exported correctly, and then not available to use as jQuery plugin.
I'm suspecting this commit ...

+1 same problem here

RequireJS is working for me in the latest version.


Question: kubebuilder kustomize not exist or is not available for darwin/arm64

➜  app git:(feature/app) ✗ make install
/Users/fanux/work/src/github.com/labring/sealos/controllers/app/bin/controller-gen rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases
test -s /Users/fanux/work/src/github.com/labring/sealos/controllers/app/bin/kustomize || { curl -s "https://raw.githubusercontent.com/kubernetes-sigs/kustomize/master/hack/install_kustomize.sh" | bash -s -- 3.8.7 /Users/fanux/work/src/github.com/labring/sealos/controllers/app/bin; }
Version v3.8.7 does not exist or is not available for darwin/arm64.

Just install kustomize:
brew install kustomize

➜  app git:(feature/app) ✗ which kustomize
/Users/fanux/homebrew/bin/kustomize
➜  app git:(feature/app) ✗ cp /Users/fanux/homebrew/bin/kustomize /Users/fanux/work/src/github.com/labring/sealos/controllers/app/bin/kustomize

Then cp the bin file to your controller/bin dir


Wishlist frontend + Backend[Feature Request]:

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
So, when user visit the website and if he/she likes the book and wanted to buy afterwards then she can put the product into her wishlist.
Use Case
when again user will visit to buy the product he/she can go into wishlist and buy the product instead of searching all over the website for the same product again
Benefits
User convenience
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Hello, @Trisha-tech please assign me this issue under an appropriate label

Hello, @Trisha-tech please assign me this issue under an appropriate label

Hello @Sneha0019 , Go for it
All the Best


Add support for specific date restore

duplicity -t 3D --file-to-restore FILENAME<EMAIL_ADDRESS>/home/me/restored_file
How can I do restoration with specific date?

You can always check duplicitie's examples section for the required parameters: http://duplicity.nongnu.org/duplicity.1.html#sect3
Then check the contents of the image's restore script. You can always add Duplicity parameters through the script. Just attach them to your script call. Script parameters are taken as duplicty parameters:
$ restore  -t 3D

It was my mistake... haha
There was option to pass Duplicity Parameters in README :)
Thanks anyway


Support for mac with ARM processors

This pull request introduces support for Mac computers with M1/M2 processors.

Looks good! Thank you!


Some Settings not sticking

For example, Crouch Toggle doesn't take.
I started program, set option.
After exiting config tool, played game, crouch toggle doesn't work.
Restarted config tool, option unchecked.
Checked it saved.
Restarted config tool again, immediately after exiting tool.
Crouch toggle unchecked (Ran second time as Admin just incase).

Thanks, I'll check it out.
On Sun, Oct 8, 2017 at 3:09 AM, Wohlever<EMAIL_ADDRESS>wrote:

For example, Crouch Toggle doesn't take.
I started program, set option.
After exiting config tool, played game, crouch toggle doesn't work.
Restarted config tool, option unchecked.
Checked it saved.
Restarted config tool again, immediately after exiting tool.
Crouch toggle unchecked (Ran second time as Admin just incase).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TheLonerD/BorderlandsAdvancedConfig/issues/1, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGQbN-BK1bo-L06qewRBGjzhAfgKfHJ_ks5sqBKjgaJpZM4Pxih9
.

Update: If you leave it unchecked, the crouch works.
For K&M.
If you check it, it works for controller but not K&M.
WTF?


Sessions

[x] create
[x] destroy

Made the skeleton as best I could. I have the login route sending a JSON object with a boolean saying whether or not the login attempt was successful. Also made a route that Angular can send an $http request to for getting current session information.
Not sure if a login route is totally necessary, as it only exists to render the login form and that is something that we would likely handle with Angular, not EJS.


feat: implement blinky for nrf52833 on micro:bit

This example enables one to blink the first LED on BBC micro:bit v2 (nrf52833-based board).

Thank you for the PR 🚀
Please also add the examples to ci.sh.
I'm not sure if this should live here vs in a board support crate for the microbit. On one hand, the microbit is a pretty widespread starter board, but there are others like the adafruit boards as well, and it's not clear to me where the line should be drawn and considered too much maintenance overhead (i.e. devkits from chip manufacturer is ok, but any non-chip manufacturer board is not?)
Would love to see more opinions on this before merging.

Totally agree about the fact that this example should live in a specific micro:bit folder but I couldn't find any in this repo (didn't look thoroughly though) so I went with nrf52833 instead of micro:bit to follow the trend.
Given that no microcontroller is self-sufficient per se and requires a board to be exploitable (e.g. which LED does the nrf52840's blinky example blink ?), I also wonder how examples are usually added to each µC's folder.

The examples are intended to showcase how to use the HAL. HAL usage is identical in all nrf52 chips, any nrf52840 example will work on the microbit if you change the chip in Cargo.toml and .cargo/config.toml.
I would love to have ready-to-go example for popular nrf52 boards, but I'm afraid the maintenance burden will be too high, similar to what we're already running into with the stm32 examples. (We have stm32 examples for each chip familiy because there is sometimes relevant hardware differences so the way you'd use the HAL changes. Still there's a lot of overlap, and when you change some HAL API you often have to go and update ~10 examples that are almost identical copy-pastes from each other)
Thanks anyway for the PR!


Math functions were implemented

I implemented inverse sine, factorial, exponentiation and square root functions.

@moussacodes these should fix the problems.

thank you guys, merged @moussacodes @angelpilled


Feature Request - Table Row's Border Customization

Is there any way that We can Increase or Decrease the table row's  border length?   Also Can We give some space between two rows?

Is there any way that We can Increase or Decrease the table row's border length

Do you mean  border width？

Also Can We give some space between two rows?

Do you mean the padding or the row height?

how do I create a gap between rows in a table? like this...
[===ROW===]
[===ROW===]
[===ROW===]
[===ROW===]

@maksudur-rahman-maruf

gap between rows

Can you provide a capture with annotation or something else to make it more visually?

I am trying to achieve this...but I have encountered few issues like

First I had to hide the footer. thats okay but I am not able to hide the footer border.
I am not able to hide the header border.
I am not able to create a gap between rows in a table

This is my current table looks like

@maksudur-rahman-maruf After the release of the new header-slot feature, I think this UI can be achieved, please wait for my notice.

@maksudur-rahman-maruf Update to the latest version 1.2.25, and here is a reference: https://codesandbox.io/s/friendly-gwen-hqc370?file=/src/App.vue.


fava: Add missing dependency

Motivation for this change
Closes #118170
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'm not interested in bikeshedding.

setuptools-scm also belongs into nativeBuildInputs. Fixing that myself.


Bbox#24

I added it to the library, I'm not very good with design, but to give a brand and a face to the package it was cool, if you don't find it very interesting I can remove it, I organized the readme making it more objective for anyone using the tool

This is a blatant copy of the box.com logo. I won't be accepting any forgery as it can lead to copyright issues.


relationReferenceAttribute as Unsafe attrubutes

Is there a way to implement the feature? I'd like to set some attrubutes depends on user group. For example, only Admin can change groupIds.
I tried
 public function rules()     {         return [                        [['!groupIds'], 'safe'],         ];     }
controller:
 $post = Yii::$app->request->post(); if (Yii::$app->user->hasRole('admin') && isset($post[$model->formName()]['groupIds'])) {             $model->groupIds = $post[$model->formName()]['groupIds'];         }
but it has no effect.
After
$model->load($post)
anyone can change the attribute

sorry, it works!


Implement From<f32> for Val

Objective
Implement From<f32> for Val, returning a Val:Px value, and replace Val functions parameters with impl Into<Val>, so users can ellide the Val::Px variant specifier.
Ergonomics aside, it's probably more robust too as it seems much easier to confuse Val:Percent(100.) with Val::Px(100.) than 100..
Solution
Implement From<f32> for Val.
Change Size and UiRect to functions to non-const and replace the Val parameter with impl Into<Val>
Changelog

implemented From<f32> for Val
replaced 'Val' function parameters with impl Into<Val> and made those functions non-const where necessary.

Hmm. I don't love it. At least for me, it seems hard to keep track of whether the From impl turns it into pixels or percents (or ems or mm or...)

Well, it mirrors the CSS behaviour where px is the default if you don't specify a unit type.

CSS is often full of these "clever" tricks and implicit behavior. I'm not convinced that's a model to emulate, although I do agree that specifying values is far too verbose for now.
My personal preference would be to include the Val enum variants in the bevy_ui prelude.

Another option could be to have short functions that would look like this:
pub fn px(v: f32) -> Val {
	Val::Px(v)
}

but don't make it part of the prelude because I think this would pollute the scope too much. This could probably be in a bevy_ui::helpers or something.
Essentially, do something similar to what glam does with vec2(x, y)/vec3(x, y, z).
I don't really like the idea of adding more stuff to the prelude, especially when it's really short/generic stuff because it becomes really easy to clash with other things.

The problem with helper functions I was thinking of about is that for some style properties, like border values, the Auto variant of Val' is meaningless. Taffy has different LengthPercentageandLengthPercentageAuto` types for this reason. And if there is a new type introduced then, as far I can tell, helper functions couldn't be made that would work with both types.

The dissatisfaction with this pr for me is that it doesn't help when directly accessing fields. So it's probably not worth it.

And if there is a new type introduced then, as far I can tell, helper functions couldn't be made that would work with both types.

This is possible. See Taffy's style_helpers module https://github.com/DioxusLabs/taffy/blob/main/src/style_helpers.rs The trick is to make the return type of the helper generic over a trait and implement that trait for each type you want the helper to work with. Bevy could potentially even use Taffy's traits and helper functions, and implement them for it's own types.


add 'search' param to vlan endpoint

Signed-off-by: Marcin Franczyk<EMAIL_ADDRESS>Description
add 'search' param to vlan endpoint

Release Note
Release note for CHANGELOG:

None

References

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

/ok-to-test sha=0157efc

/ok-to-test sha=0157efc

/ok-to-test sha=db27d50

/ok-to-test sha=db27d50

/ok-to-test sha=a1ea86c

/ok-to-test sha=a1ea86c


shellz: init at 1.6.0

Motivation for this change
shellz is a small utility to manage your ssh, telnet, kubernetes, winrm, web or any custom shell in a single place.
https://github.com/evilsocket/shellz
Related to #81418
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


Diameter of a tree using DFS

I want to work on this topic in  C++ language. Please assign this issue to me

try something new instead of DSA


Cannot delete images in registry

Hi all,
I've followed Docs by running curl -X DELETE http://localhost:5000/v2/<image_name>/manifests/latest to delete image in registry.
However it returns
{"errors":[{"code":"DIGEST_INVALID","message":"provided digest did not match uploaded content"}]}
I've no idea about the error because when I run curl -X GET http://localhost:5000/v2/<image_name>/manifests/latest, it will return the correct manifest

@RyanDeng You should not use tag as the reference in DELETE request. For deletes, reference must be a digest or the delete will fail.
Please refer to: https://github.com/docker/distribution/blob/e16c23bb434ac3f8c4d65cbf8fa83b9f5a2141d4/docs/spec/api.md.tmpl#L659-L674

Thanks @xinxiaogang. I didn't read the docs carefully


Add area monitor callback error checking

Fixes #59628
I have the error only print once because with the referenced issue it felt like a good idea to only print the error once. If we want the error to print every time then that's good with me too.

Thanks! And congrats for your first merged Godot contribution :tada:


Add an option to reset the local changes

something similar to git reset --hard so that the local changes will be discarded.

Duplicate of #107


performance hint does not work on windows 10

on windows it needs to be
plugins: [        new webpack.LoaderOptionsPlugin({          // test: /\.xxx$/, // may apply this only for some modules          options: {            performance: ...          }        })      ]
but on linux it needs to be
performance: {   hints: false }
webpack version: 2.1.0-beta.27

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.


allow negative bymonthday

Following the work of @bartekbrak in pull-request #59 I implemented the possibility to use negative values for the monthly rules, i.e. count days from the end of the month.
The widget uses the four remaining cells to allow entering of the last, second, third and fourth but last day. I suppose there are very few scenarios where more is needed.
Translations are provided for all previous languages except basque.
@dominicrodger: by the way this is a very great django-app :+1:

This LGTM - thanks for all the work here @mir06! :+1:
I've not got a great deal of experience with translations - do you know what the implications of the missing Basque changes are? What difference will that make?
Any thoughts here @emperorcezar or @tamask? If there are no objections in the next week or so I'll get this merged.
Also - all the credit for django-recurrence is for @tamask who wrote it originally, and @emperorcezar who got it up on PyPI :)

@dominicrodger it's just about 7 strings that have to be translated (last, second last, 2nd last, third last, 3rd last, fourth last and 4th last). probably @unaizalakain could do this.

Great - thanks @mir06. I've just got this merged, I'll add some release notes shortly, and file an issue for getting those 7 string added to the Basque translation. Thanks again for all your work on this - particularly for getting all those strings translated.

@dominicrodger thank you :+1:

@dominicrodger Let me know if/when you want to cut a release. Also, could you make an issue to cut a release and assign it to me? Make it easier to track.

Will do, thanks @emperorcezar - there are a few more things that I'd like to get in before we get a release out (mostly #87 and #92).


Can the PB message structure be the input parameter for RegisterWorkflow or RegisterActivity function

Can the PB message structure be the input parameter for RegisterWorkflow or RegisterActivity function

Thank you for your feature request!  Currently, the RegisterWorkflow and RegisterActivity functions in Temporal do not directly accept Protocol Buffer (PB) message structures as input parameters.
However, you can achieve this functionality by:

Defining a Temporal Data Converter:  Create a custom data converter that maps your PB message structure to Temporal's supported data types (e.g.,  string, int, bool, or list). You can then pass these converted data types as arguments to the RegisterWorkflow and RegisterActivity functions.

Using Custom Input Serialization:  You can create a custom input serialization method that takes your PB message structure as input and outputs a Temporal-compatible format (like JSON). This serialization method can be used in your workflow and activity implementations to handle the input data.

Please let us know if you have any further questions.

Thank you for your suggestion! Currently, RegisterWorkflow and RegisterActivity functions in Temporal do not accept a Protobuf message structure as an input parameter.
To achieve this functionality, you can consider using a custom data type or a custom serializer/deserializer within your workflows and activities. This would involve defining a custom data type in your workflow and activity code, and then using a serializer to convert the Protobuf message to this custom data type before passing it to your workflow or activity.
For more information on defining custom data types and working with serialization, please refer to the following documentation:

Workflow Types
Activity Types

We value your feedback and will keep your suggestion in mind for future development!

Thanks for your feature request! Currently, RegisterWorkflow and RegisterActivity functions don't directly accept protobuf messages as input parameters.
You can work around this by:

Converting protobuf messages to JSON: Convert your protobuf message to JSON using libraries like jsonpb in Go or protobuf-to-json in Python, then pass the JSON string to RegisterWorkflow or RegisterActivity.

Defining input parameters as custom structs: Define custom structs that represent your protobuf message fields and pass them to RegisterWorkflow or RegisterActivity.

Let us know if you have any other questions or suggestions!

Thank you for your feature request!
Currently, the RegisterWorkflow and RegisterActivity functions in Temporal don't directly accept Protobuf (PB) messages as input parameters.
However, you can use the following workaround:

Define a Go struct: Create a Go struct that maps to your desired PB message structure.
Use proto.Marshal() and proto.Unmarshal(): Marshal your PB message into a byte array using proto.Marshal(). Pass this byte array to the RegisterWorkflow or RegisterActivity function.  Within the workflow or activity code, use proto.Unmarshal() to decode the byte array back into your Go struct.

This allows you to utilize your existing PB structures while still interacting with Temporal.
Please let me know if you have any other questions.


Ignore dirs with invalid source files

@shurcooL This fixes the issue with scanForPackages failing completely when a particular directory contains invalid source files. Now, source files in such directories will be ignored (although the subdirectories will be scanned).

LGTM.


google_sign_in account picker

Looking for a configuration like prompt=select_account+consent . Depending on what device I'm using, the google window is not consistent. On some devices, I have to enter my credentials. On others, I can choose which account to select. Why is the account picker not shown by default and only shown sometimes?

@kaarens93
Please provide flutter doctor -v and a minimal complete reproducible code sample that shows the issue you are facing. It'll also be helpful if you could add screenshots of the devices (along with OS versions )on which you are seeing this discrepancy.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Update instructions to reflect using Azure DevOps Extensions SDK

The instructions in this page seem dated, and should be updated to align with the sample provided here. For example, the npm install vss-web-extension-sdk --save instruction should now be npm install azure-devops-extension-sdk --save, etc. Otherwise the current page takes the reader to the GitHub samples page, which redirects back to this page for more details.

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

ID: 14955ee9-29f2-46fe-3a8d-a42666f53ffc
Version Independent ID: 5d417cd0-a671-ee30-8fca-5f8b0addd328
Content: Develop a web extension for Azure DevOps - Azure DevOps
Content Source: docs/extend/get-started/node.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @apawast
Microsoft Alias: apawast

It's very frustrating to try to navigate these documents. Is there not a single Microsoft developer who can read the documentation from page 1 and try to follow it through to create a new extension? Microsoft is a two trillion dollar company now. Surely someone can spend an hour on this.

@GaTechThomas -- Thomas, thank you for your follow-up.
@apawast -- Apeksha, please look into this issue.


Authentication + Syncing Issue

Please review previous closed issues before filling out a new one! Duplicate issues will be closed without comment.
Describe the bug
I'm unable to get the questions that I've completed to sync to the existing repository that I've been using. It's just not syncing for me. I uninstalled the extension to re-authenticate and connect my existing repo to LeetHub, but when I click on the authenticate button in the extension, it just redirects me to my home page in GitHub.
Expected behavior
I expected a request for LeetHub to get access to my GitHub account, and then for the extension to prompt me to create a repository to connect to, or an existing one. I was going to do the latter, because I've previously used the extension.

re-installing LeetHub is a 2 step process: 1) reinstall extension and 2) revoke access to LeetHub. To revoke access to LeetHub, you'd need to go to this link and click on Authorized OAuth Apps and click on ellipses icon besides LeetHub --> revoke access. See figure below.

Once that is done, you can reinstall the extension and try again the entire process. Lmk if it still doesn't work.

Yep, it worked! Reset my stats though in the extension (rip). Works perfectly now - thanks for the quick help!


Handling error codes in download manager

What

Handling error codes in download manager
What should we do ? Hide it ? Show a retry button ? Automatically retry after a time to determinate ?

Screenshot

@teolemon This is a server error (time-out): the best we could do is replace the text with "Server error: time-out".
If you want to retry, there's a "refresh" button in the top-right corner.
But usually it's not needed, as the background tasks are automatically relaunched when you go on some important pages of the app (e.g. product page).

Slightly related, https://github.com/openfoodfacts/openfoodfacts-server/issues/8601


Make the root repo a valid go package

Thanks for the PR, but I'm not convinced this is worth adding.
Having a go module where the root isn't a go package itself seems like it should be fine.
I can see from a side-channel discussion that the problem's rooted in
// unused build-flagged-off import just so `go.mod` versions it
import _ "github.com/euank/ifacepropagate"

Changing that to:
import _ "github.com/euank/ifacepropagate/cmd/ifacepropagate"
// or
import _ "github.com/euank/ifacepropagate/pkg/ifacepropagate"

seems preferable to me as a way to make it a more-valid import (with the second being totally valid, and the first being a main package, which probably works if it's build-flagged off?)
If there's another reason that having the root be a package makes sense, I'd be happy to hear it of course!

Yeah, that makes total sense. Not sure why I went straight to this. Thanks!


Implement Get Colors… Action

Read from binary file - compatibility with traditional paint?

Add error trap if file is not in the same format:
Warning:
FILENAME
Paint cannot open this file.
This file is not in the correct format.


Keybinding customization - only shows the keybindings of activated extensions

The vscode-tslint extension is contributing key bindings. However, these keybindings are only available for customization in the editor, once the extension is activated and the user has opened a TypeScript file.
The user should be able to customize keybindings independent from whether an extension is activated or not.

@jrieken Any idea why the command registry is not having the commands until activation?

@sandy081 The keybinding comes in right on startup (set brk-point here: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/keybinding/electron-browser/keybindingService.ts#L482). That binds the identifier of an existing or future command to some key/keycombination.
The command itself, the id->function-thing, comes in when someone called registerCommand, so when becoming active. I don't think you should validate the existing of a function but should just rely on the declarative part.

@jrieken Sorry if I am not clear. It's not about keybindings.. its about commands with no keybindings.
When a user open keybindings editor, he does not see those commands without keybindings from an extension which is not active yet. This will confuse the user. Since command are registered descriptively also, is not it possible to get the information from there? I understand that Command registry is a registry of command id -> Function which will not be available until the extension is activated. Do we have some thing like command extension point registry from where we can get descriptive details?

Is the MenuRegistry which you mentioned previously will have these details?

Yes, MenuRegistery contains everything an extension defined in the commands section and in the menus section. It is similar to a keybinding as that it registers a user-gesture (selecting a button) to run a command.

But there is a catch that this will not give me command description that can tell me that command takes args or not.. Currently I am excluding all commands with args from Keybindings editor.
Also is CommandsRegistry.getCommands() is a sub set of MenuRegistry.getCommands() ?

CommandRegistry holds commands which is a id->Function mapping. MenuRegistry holds menu items/command actions which are ui-description->id mapping. The latter are called commands (ICommandAction) because they invoke commands and I see how that is slight confusing. They have nothing in common, but join the id-parts
command: id -> function // dynamic, involves code

keybinding: key-event-description -> id // declarative

command_action: ui-description -> id // declarative

menu: [command_action, command_action] // declared an arrangement of command_actions

Even though MenuRegistry has all registered commands through commands contribution point, it does not tell if these commands take arguments or not. This information is available only after registering to CommandRegistry which happens after extension activation. So, it would be nice if commands contribution point also takes args description.

So, it would be nice if commands contribution point also takes args description.

Then the same applies to the keybindings contribution point


CPE definition on pkg.Package is coupled to an external package as a type alias

Today we have the CPEs field on the pkg.Package struct:
https://github.com/anchore/syft/blob/c5d15d1d6c2e79f92545cf8e973317deb04b7638/syft/pkg/package.go#L28
However, the cpe.CPE definition is a type alias:
https://github.com/anchore/syft/blob/c5d15d1d6c2e79f92545cf8e973317deb04b7638/syft/cpe/cpe.go#L11
This isn't good for a couple of reasons:

type aliases are really intended for making refactors easier, not for permanently defining a type
the underlying type behavior or shape can change out from underneath us without warning

Ideally we should have an internal type that represents a CPE and has the basic functionality we need within syft. Downstream consumers (such as grype) that require additional behaviors (e.g. CPE matching or comparison) should use abstractions to protect against direct use of the upstream wfn package.

I think that making an identically laid out struct that Syft controls, and wrapping the type conversion in helper methods will prevent us from having break changes here. Example conversion: https://go.dev/play/p/f5VDwwTYBb8.


[Feature Request] Add option (e.g. ZeroAsNull) to treat zero values equal to missing property (similiar to EmptyAsNull)

Test case:
package main

import (
	"testing"

	"github.com/sters/yaml-diff/yamldiff"
	"gopkg.in/yaml.v3"
)

var yaml1 = `
intA: 2
intB: 0
intC: 0
strA: foo
strB: ""
strC: ""
`

type MyType struct {
	IntA int    `yaml:"intA"`
	IntB int    `yaml:"intB"`
	IntC int    `yaml:"intC,omitempty"`
	StrA string `yaml:"strA"`
	StrB string `yaml:"strB"`
	StrC string `yaml:"strC,omitempty"`
}

func Test_YamlDiff_Omitempty(t *testing.T) {
	t.Parallel()

	yaml1Source, err := yamldiff.Load(yaml1)
	if err != nil {
		t.Error(err)
	}

	mytype := &MyType{}

	err = yaml.Unmarshal([]byte(yaml1), mytype)
	if err != nil {
		t.Error(err)
	}

	bYaml2, err := yaml.Marshal(mytype)
	if err != nil {
		t.Error(err)
	}

	yaml2 := string(bYaml2)

	yaml2Source, err := yamldiff.Load(yaml2)
	if err != nil {
		t.Error(err)
	}

	diffs := yamldiff.Do(yaml1Source, yaml2Source, yamldiff.EmptyAsNull())
	if len(diffs) > 0 {
		for _, diff := range diffs {
			if diff.Status() != yamldiff.DiffStatusSame {
				t.Log("diff detected\n" + diff.Dump())
				t.Fail()
			}
		}
	}
}

Test Result:
--- FAIL: Test_YamlDiff_Omitempty (0.00s)
    .../main_test.go:59: diff detected
          intA: 2
          intB: 0
        - intC: 0
          strA: "foo"
          strB: ""
        - strC: ""

If one of the diff sources contains a zero value for a property and the other has this property missing a diff will be found. This makes sense by default.
Kubernetes generates optional fields with omitempty tag (just looked up a random string property). So once a yaml is produced by kubernetes (e.g. kubectl), such output will have those properties missing. It's sometimes intentional by (because $reason) , that the other diff source has this property always set, even if set to zero.

Offered you a PR https://github.com/sters/yaml-diff/pull/31 for this issue. Please have a look and review


fuse.EPERM not defined

I could not find an error code matching syscall.EPERM defined in this package.
It should be defined.

Please don't open one issue per missing constant -- that's a little bit excessive.


Comprehensive spherical harmonics toolbox

I re-arranged the spherical harmonics functions from the stress repository for a better separation between front-end (napari-visible functions) and backend (stress code).
This will make the code easier to maintain in the long run as functionality (e.g., spherical harmonics expansion, manifold evaluation, etc) happens in a single place.
Relates to #80
Note: This does not change any functionality - so if the concerned functions (fit spherical harmonics & measure curvature) still work, everything should be fine.

Codecov Report

Merging #92 (4a73264) into main (48451a7) will increase coverage by 0.52%.
The diff coverage is 94.59%.

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   19.09%   19.62%   +0.52%
==========================================
  Files          23       23
  Lines        7222     7237      +15
==========================================
+ Hits         1379     1420      +41
+ Misses       5843     5817      -26

Impacted Files
Coverage Δ

...napari_stress/_spherical_harmonics/manifold_SPB.py
55.37% <ø> (+0.12%)
:arrow_up:

...apari_stress/_tests/test_spherical_harmonic_fit.py
92.85% <84.61%> (-7.15%)
:arrow_down:

..._spherical_harmonics/spherical_harmonics_napari.py
92.00% <92.00%> (ø)

...stress/_spherical_harmonics/spherical_harmonics.py
97.61% <97.61%> (ø)

src/napari_stress/__init__.py
100.00% <100.00%> (ø)

src/napari_stress/_utils/frame_by_frame.py
92.25% <0.00%> (+16.19%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Hi Johannes @jo-mueller , I have just tested the affected functions from this PR in napari and they are running without errors.

Seems like removing Python 3.7 from the tests resolves issue #93

I adjusted the demo notebooks to work with the re-arranged code in https://github.com/BiAPoL/napari-stress/pull/92/commits/ec2d0fe55045187d90114131e16db20f98fa9c0d


Update home assistant integration to support ESPHome

I have ESPHome Bluetooth Proxies in ideal locations connected via ethernet, throughout my house. I'd like to use them for my BLE ESLs.
Describe the solution you'd like
Update the home assistant integration to not need an openepaperlink AP.
I think the existing code from BLE should be able to be ported to python/homeassistant.

https://github.com/OpenEPaperLink/OpenEPaperLink/blob/master/ESP32_AP-Flasher/src/ble_filter.cpp
https://github.com/OpenEPaperLink/OpenEPaperLink/blob/master/ESP32_AP-Flasher/src/ble_writer.cpp

Describe alternatives you've considered
The main alternative is just using another ESP32S3 running the AP code for handling the BLE messages.
Additional context

I haven't created any feature requests in ESPHome or HomeAssistant.
That being said, with an ESPHome device acting as a bluetooth proxy, I think the feature request belongs in HomeAssistant. Which is where the net new code could need to live.

I'm not that technical to say where the code needs to live, but in my opinion, ESL is just a display, so one could create the displayed image in ESPHome (like drawcustom), and ESPHome would be responsible for communicating with the display.
Not sure if Bluetooth proxy functionality would help.
If the support could be added to ESPHome, then OpenEpaperLink and the HA integration wouldn't be needed.

I'm already familiar with ESPHome and did some projects with displays, so my personal preference would be to configure everything in ESPHome, but I agree that with multiple Bluetooth proxies around the house, it would be best to have a central place to manage display content, so when you move the display from kitchen to garage it will still work (because of Bluetooth Proxy)
Ref: https://github.com/esphome/feature-requests/issues/2782

It's very unlikely that it would ever be possible to use Bluetooth Proxy to talk directly to the tags. At least, not with WiFi proxies.
Maybe, if there would exist an ESP32-C6 board with Ethernet.


Add vertical style and "Always show workspace drawer" setting

Description
As requested in #248, some users prefer a more "vertical" style with the service list on top as seen in Rambox.
This PR will add a new setting "Use vertical style" that adds additional styling to move the sidebar to the top of the page, see this demo:

How Has This Been Tested?
Using the setting in addition to other appearance settings.
Currently known bugs

The sidebar hasn't got the correct height when using "Very slim sidebar" or "Extremely slim sidebar"
Ferdi Todos will overlay on top of the sidebar icons

Types of changes

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

Checklist:

[x] My code follows the code style of this project (run $ yarn lint).

@vantezzen nice one! thanks!


Project still alive?

First of all, thank you for this awesome extension - it is unique!
Is it still continued? The automatic folding of the svg elements would be a nice to have feature!

I'm waitting for the realization of this extension Visual Studio Code Custom Folding Extension
It is a preview feature Feature - Default Folding (Preview) in their extension.

Default folds define regions which will be collapsed whenever the file is opened. In the current preview version, default folds are only collapsed when the regionfolder.collapseDefault command is issued from the command palette.

It will be something like this
<!-- #region drawnote --><svg .../>
<!-- #endregion -->

I have not readed their source code carefully. Maybe it is easy.
It is alive, but I am lazy. Maybe the feature in this issue will be done in months.

Okay, thanks for the update. In my case this is a critical feature (collapsing long SVG areas) which would make this extension pretty useful - provided this issue is solved.
I just realized that if a SVG drawing gets loo long/big (in one line) VS Code does not allow folding this SVG area at all. This happens at the same time as the syntax highlighting stops working for this line - I guess because there is a maximal character limit per line for these functions to work.

Good catch!
seems it should be changed to things like
<!-- #region drawnote -->
<svg .../>
<!-- #endregion -->

When I deal with it, it should resolve.

Okay, good to hear!

done


Updates interactive window tests for VS 2015

Updates interactive window tests for VS 2015
Adds some of the interactive window commands that were missing from VS 2013
Adds workaround for Shift+Enter shortcut not working in VS 2015 RC
Fixes null reference error reading from standard input.
Fixes references in all projects
Fixes newlines in shared REPL window code.
Please pay particular attention to the files in Python\Product - these fix crashes. The rest are test related and may still have issues that can't be discovered without running the full pass on our test machines.

:+1:

:+1:


Calling Field's input.onChange(undefined) does not set field value to undefined.

Hey there,
I'm trying to reset a form field to be undefined but it doesn't seem to work. Here is a codesandbox showing it not working:
https://codesandbox.io/s/5w72x5rpmx
It does work passing null, but I am confused why undefined doesn't work. I couldn't find any mention of why this would be the case in the docs but maybe I missed something.
Besides this behavior being confusing, I would especially love it to be set to undefined (and not null or false) in my case because I often set certain default values for my form values using the
const {input: {value = "hey"}} = this.props

Which only works if the value gets reset to undefined.
Thanks!

Is there a specific reason you want it to be undefined instead of an empty string? Technically null is a value, so I'm not sure that's actually what you're looking for.
This kinda gets weird when comparing fields which were defined in initialValues vs fields which were not defined (or when not including initial values all together.) [See here.] (https://github.com/erikras/redux-form/issues/3366) Sounds like this is going to be updated in v8.
What's worked best for me is to assign an empty string when I want to act like something doesn't have a value. Here's an example of it working.
One thing to note in my example is that I applied a unary to the inc/dec buttons so they cast the empty string to a 0 when doing addition.

@Olliebaba Destructuring default values only work when the value is undefined. For example sometimes in my app I have a form value getting defaulted to an empty array and then use logic like array.length. Currently I can't unset that value because onChange with undefined doesn't work and if I did an onChange with null then that value would not be defaulted to an empty array and my app crashes.

@tgreen7 - I'm not sure what you're referring to. Could you make a sandbox example of what you're describing?

@Olliebaba I recreated it using formValues and the change function (I was having trouble getting a field to store an array). https://codesandbox.io/s/0nzq9x5m0

here is the line that is causing the issue https://github.com/erikras/redux-form/blob/de8beb08e888f99c30753fee93768c4e7e0788c3/src/createReducer.js#L234

I'm with the same problem.

Same problem here.
In redux-form-input-masks, we use normalize to create input masks, and we've been returning an empty string to clear the Field's value to undefined, but this behavior is detailed on the docs.
Also, using empty string to clear the Field will not work if the input has an initial value. This is some odd behavior: why should the initial value determine how the change action behaves for empty strings? This is caused by the line of code below:
https://github.com/erikras/redux-form/blob/a97606fee0ed365903fff80cc93956fe2313a3ce/src/createReducer.js#L232
I'm okay with empty string setting undefined on the Field's value, but from the user's perspective it just makes no sense that initial value would play a role in this case.

I'm also having a problem with the inconsistent behavior. The initial === undefined makes it so that there's always a value for a field, even when you don't want it.
My use case is the following

I have a various fields foo.bar1, foo.bar2 in the form
Each field can be selected and deselected, which changes it's value from {} when selected to '' when deselected, using onChange(value ? '' : {});
When submitting, the form values only include the fields that are selected (whose value is {}), fields that had a '' value are not in the values object passed to onSubmit

The CHANGE actions trigger when clicking on each field in this way

When the field isn't selected (value is '')

{
  type: '@@redux-form/CHANGE',
  meta: {
    form: 'foodForm',
    field: 'features.alcohol',
    touch: false,
    persistentSubmitErrors: false
  },
  payload: {}
}

When the field is selected (value is {})

{
  type: '@@redux-form/CHANGE',
  meta: {
    form: 'foodForm',
    field: 'features.alcohol',
    touch: false,
    persistentSubmitErrors: false
  },
  payload: ''
}

Which results in the following diffs to the state respectively
{
  form: {
    values: {
      foo: {
        bar1: {}  // foo bar1 key is added with {} value
}}}}

{
  form: {
    values: {
      foo: {}  // foo bar1 key is removed
}}}

But if I pass initialValues to the form this behavior is broken, it triggers the same CHANGE actions, but the diff of the second one results with a state of:
{
  form: {
    values: {
      foo: {
        bar1: ''  // foo bar1 key is still present with '' as it's value
}}}}

When the intended behavior is for bar1 to be removed from the form values

I put in a PR for this here: https://github.com/erikras/redux-form/pull/4441

That previous PR had issues because all tests were failing on master so I remade the PR against the latest version of master.

Hi, Do you know when this will be released? This will solve a very annoying problem of mine being able to reset a select component

This week should be released I think.

This week should be released I think.

Great, thank you for your quick response.

Published in v8.2.6.

Looks like we still can't set undefined for fields with initial values. Sandbox here
This is caused by shouldDelete check into createDeleteInWithCleanUp:
https://github.com/erikras/redux-form/blob/87beada1e8a42c62e66c297f1651870f8fe6f254/src/deleteInWithCleanUp.js#L33-L35
https://github.com/erikras/redux-form/blob/de8beb08e888f99c30753fee93768c4e7e0788c3/src/createReducer.js#L44-L54

We can reverse this if
https://github.com/erikras/redux-form/blob/master/src/createReducer.js#L306
or even more check only for payload === undefined but I don't know exactly if this may be a breaking change, there are to much responsibilities in redux-form and I don't know exactly all cases for CHANGE
@renatoagds any thoughts on this one?

Maybe we should not care about initial value at all, and let user set his desired value.

I think this issue should be reopened, it's crucial.

@khanilov what version do you have?

@iamandrewluca I have 8.2.6.
I am addressing the issue pointed in this comment:

Looks like we still can't set undefined for fields with initial values. Sandbox here
This is caused by shouldDelete check into createDeleteInWithCleanUp:
https://github.com/erikras/redux-form/blob/87beada1e8a42c62e66c297f1651870f8fe6f254/src/deleteInWithCleanUp.js#L33-L35
https://github.com/erikras/redux-form/blob/de8beb08e888f99c30753fee93768c4e7e0788c3/src/createReducer.js#L44-L54

As @YauheniKhadanionak pointed, shouldDelete would return false if there is initial value for field.
https://github.com/redux-form/redux-form/blob/v8.2.6/src/createReducer.js#L52-L56
So deleteInWithCleanUp not work if initial value presented.
Btw for what reason there are checks for initialValue in change handler?

The issue is still there in<EMAIL_ADDRESS>Not resolved.


feat: add Icon component for MDX pages. Use a single icon file per icon type

Having 12 different sketch icons and 12 different github icon files is nuts.
I think gatsby-image is overkill for these little guys, we can colocate them into a single component where we'll add icons as needed.
TODO: document adding an icon and using an icon

@emyarod yes, we don't use those anymore. Prob coulda been a separate PR but I was on an unused file tear.