Fixed manualExit option on Mac machines

Changes

Making manualExit option work on mac machines.

Successful Workflow Run Link
Tested on private self-hosted runner, cannot share here.
Checklist

[x] Read the contribution guide and accept the
code of conduct
[x] Docs (If new inputs or outputs have been added or changes to behavior that should be documented. Please make a PR
in the documentation repo)
[x] Readme (updated or not needed)
[x] Tests (added, updated or not needed)

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

Comparison is base (ff23166) 37.40% compared to head (4400366) 37.40%.

:exclamation: Current head 4400366 differs from pull request most recent head a917f3a. Consider uploading reports for the commit a917f3a to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@           Coverage Diff            @@
##             main     #619    +/-   ##
========================================
  Coverage   37.40%   37.40%
========================================
  Files          75       75
  Lines        3048     3048
  Branches      632      605    -27
========================================
  Hits         1140     1140
- Misses       1759     1908   +149
+ Partials      149        0   -149

see 36 files with indirect coverage changes


Animating video dimensions on Android?

I'm trying to shrink my video to the bottom right corner of the screen in response to a pan responder for another view:

  localVideoView() {
    const videoStyles = {
      width: this.state.overlayPan.x.interpolate({
        inputRange: [-SCREEN_WIDTH, 0],
        outputRange: [MIN_LOCAL_VIDEO_WIDTH, MAX_LOCAL_VIDEO_WIDTH],
        extrapolate: 'clamp',
      }),
      height: this.state.overlayPan.x.interpolate({
        inputRange: [-SCREEN_WIDTH, 0],
        outputRange: [MIN_LOCAL_VIDEO_HEIGHT, MAX_LOCAL_VIDEO_HEIGHT],
        extrapolate: 'clamp',
      }),
      bottom: this.state.overlayPan.x.interpolate({
        inputRange: [-SCREEN_WIDTH, 0],
        outputRange: [OUTER_PADDING, 0],
        extrapolate: 'clamp',
      }),
      right: this.state.overlayPan.x.interpolate({
        inputRange: [-SCREEN_WIDTH, 0],
        outputRange: [OUTER_PADDING, 0],
        extrapolate: 'clamp',
      }),
    };

    return (
      <Animated.View style={[styles.localVideoContainer, videoStyles]}>
        <RTCView zOrder={1} objectFit="contain" style={styles.localVideo} streamURL={this.state.localVideoURL}/>
      </Animated.View>
    );
  }

However, it turns black whenever it's animating and flickers back on when I stop and go.
If I try to use transform: {scale: .. } instead, the video doesn't scale proportionally to the Animated.View container (it also shrinks to the middle which is not what I want, but at least it doesn't turn black / flicker)
    let scale = this.state.overlayPan.x.interpolate({
      inputRange: [-SCREEN_WIDTH, 0],
      outputRange: [0.2, 1],
      extrapolate: 'clamp',
    })

    let scaleStyles = {transform: [{scale}]};

@EdmundMai did you resolve this finally?

@ramneekhanda No, I had to change my design to work around this.

I used both scale and translate to animate my view (from fullscreen to top right corner). It actually worked quite nicely, with no flickering. I used Animated.timing instead of interpolation though.


[Fix #10877] Fix crash with Layout/BlockEndNewline heredoc detection.

Autocorrection for Layout/BlockEndNewline tries to handle heredocs but now crashes in some cases where heredoc? is not defined.
Fixes #10877.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


[TEMPLATE] - Vue + Django - FullStack

What domain will the project template be based on?
FullStack
If 'Others', please specify the preferred domain.
No response
Particular directory path for the template.
templates/FullStack/Vue+Backend/Vue+Django/
What tech stacks do you want to include?
Vue.js, Django, Django REST Framework
Brief Description
This template combines Vue.js with Django to build a powerful web application. The frontend leverages Vue.js for dynamic rendering, while the Django backend provides a robust API using Django REST Framework. This combination is perfect for developers looking to create scalable applications with a rich user experience.
Relevant Links:
Vue.js Documentation
Django Documentation
Django REST Framework
👀 Have you spent some time checking if this issue has been raised before?

[X] Yes
[ ] No

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Would you like to work on this issue?
No

Hi @Abhishek-Mallick, I can take care of this issue. Can you assign me?
PS. I can add also another fullstack template, Golang on the server + Vue on the client. If you are OK with it,i will add the [TEMPLATE REQUEST] issue.

Assigning this to you
Sure go ahead and create an issue for Vue + Golang I'll assign it to you

Thanks! I opened the issue.

Hi @Abhishek-Mallick, I opened PR. Can you take a look in the meanwhile?


Tracking upstream: Wayland support not implemented/buggy

UI thread spawned
fontkey from: Roboto, 14
thread 'main' panicked at 'internal error: entered unreachable code', /home/sawyer/.cargo/registry/src/github.com-1ecc6299db9ec823/skulpin-0.5.0/src/renderer/window_support.rs:45:14
stack backtrace:
   0:     0x55c26be318f4 - backtrace::backtrace::libunwind::trace::h65597d255cb1398b
                               at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.40/src/backtrace/libunwind.rs:88
   1:     0x55c26be318f4 - backtrace::backtrace::trace_unsynchronized::hd4f479d7150ec4a0
                               at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.40/src/backtrace/mod.rs:66
   2:     0x55c26be318f4 - std::sys_common::backtrace::_print_fmt::h015072984a2b172c
                               at src/libstd/sys_common/backtrace.rs:77
   3:     0x55c26be318f4 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h6df05d3335f32194
                               at src/libstd/sys_common/backtrace.rs:61
   4:     0x55c26be57d6c - core::fmt::write::h1f444f4312eb6c27
                               at src/libcore/fmt/mod.rs:1028
   5:     0x55c26be2ea37 - std::io::Write::write_fmt::h8d147888220078ef
                               at src/libstd/io/mod.rs:1412
   6:     0x55c26be33e4e - std::sys_common::backtrace::_print::h8a6df0fa81d6af62
                               at src/libstd/sys_common/backtrace.rs:65
   7:     0x55c26be33e4e - std::sys_common::backtrace::print::h6f05b4733407e509
                               at src/libstd/sys_common/backtrace.rs:50
   8:     0x55c26be33e4e - std::panicking::default_hook::{{closure}}::h0d0a23bd02315dd8
                               at src/libstd/panicking.rs:188
   9:     0x55c26be33b41 - std::panicking::default_hook::h8d15a9aecb4efac6
                               at src/libstd/panicking.rs:205
  10:     0x55c26be3454b - std::panicking::rust_panic_with_hook::hbe174577402a475d
                               at src/libstd/panicking.rs:464
  11:     0x55c26bdb1783 - std::panicking::begin_panic::hfc51a6f59961fcca
                               at /rustc/73528e339aae0f17a15ffa49a8ac608f50c6cf14/src/libstd/panicking.rs:400
  12:     0x55c26b5d8efa - skulpin::renderer::window_support::create_surface::hdafadb24e11b4148
                               at /home/sawyer/.cargo/registry/src/github.com-1ecc6299db9ec823/skulpin-0.5.0/src/renderer/window_support.rs:45
  13:     0x55c26b5b3055 - skulpin::renderer::device::VkDevice::new::hb87b290a8f6fd6cc
                               at /home/sawyer/.cargo/registry/src/github.com-1ecc6299db9ec823/skulpin-0.5.0/src/renderer/device.rs:50
  14:     0x55c26b5aa1c1 - skulpin::renderer::renderer::Renderer::new::hc5d4e7bf2204550e
                               at /home/sawyer/.cargo/registry/src/github.com-1ecc6299db9ec823/skulpin-0.5.0/src/renderer/renderer.rs:249
  15:     0x55c26b5a9d7a - skulpin::renderer::renderer::RendererBuilder::build::h4991be65e14a1fbf
                               at /home/sawyer/.cargo/registry/src/github.com-1ecc6299db9ec823/skulpin-0.5.0/src/renderer/renderer.rs:163
  16:     0x55c26b3dea5b - neovide::window::ui_loop::hdcad53034d51e52c
                               at src/window.rs:42
  17:     0x55c26b4429f9 - neovide::main::hcc0d95a0303618df
                               at src/main.rs:91
  18:     0x55c26b43f520 - std::rt::lang_start::{{closure}}::hfbadcd0574f266cc
                               at /rustc/73528e339aae0f17a15ffa49a8ac608f50c6cf14/src/libstd/rt.rs:61
  19:     0x55c26be33f73 - std::rt::lang_start_internal::{{closure}}::h6ea535ec5c50fc3e
                               at src/libstd/rt.rs:48
  20:     0x55c26be33f73 - std::panicking::try::do_call::h631c6408dfccc6f5
                               at src/libstd/panicking.rs:287
  21:     0x55c26be3c2ca - __rust_maybe_catch_panic
                               at src/libpanic_unwind/lib.rs:78
  22:     0x55c26be34a2d - std::panicking::try::hab539b2d1255d635
                               at src/libstd/panicking.rs:265
  23:     0x55c26be34a2d - std::panic::catch_unwind::hd5e0a26424bd7f34
                               at src/libstd/panic.rs:396
  24:     0x55c26be34a2d - std::rt::lang_start_internal::h3bdc4c7d98181bf9
                               at src/libstd/rt.rs:47
  25:     0x55c26b43f4f9 - std::rt::lang_start::h1cd89b6af8d283da
                               at /rustc/73528e339aae0f17a15ffa49a8ac608f50c6cf14/src/libstd/rt.rs:61
  26:     0x55c26b442a7a - main
  27:     0x7f3a6ddfd153 - __libc_start_main
  28:     0x55c26b385e9e - _start
  29:                0x0 - <unknown>

Might be upstream, but should look at the issue here to be sure

Not reachable on a later version, may be hidden by an earlier issue though (currently trying to resolve on my end)

Yep, still present for me. Backtrace updated with any new offsets

Looks like skulpin may not support wayland? Shouldn't be an issue though since it should be able to just run through xwayland
https://github.com/aclysma/skulpin/blob/master/src/renderer/window_support.rs

Yeah unfortunately the decision to use Vulcan is making life hard on non windows platforms seems like. I've been considering some other renderer targets such as pathfinder and the like, but they run into problems when it comes to remote desktop on windows.
In the end I will likely build a generic renderer trait which I build multiple implementations for. But for now and I think for the initial release I will need to add better readme support for getting vulcan running on the various platforms.

How well do other rust vulkan libs work cross-plat?

Upstream is working on adding wayland support

Not sure how to debug this at the moment, after applying the patch this runs and does a couple loops in the event loop but a window never appears

Unfortunately I don't have a linux box easily accessible, so testing wayland is somewhat difficult at the moment. Further I've got some other fish to fry (emoji support, documentation, random other crashes :P). For the moment, I'm going to rely on upstream skulpin support to land for this. If however it doesn't work after I have finished the immediate next tasks, I will build a linux test environment and dig in.
Thanks for testing and trying what you have so far! It helps a lot.

So is Wayland supported now? Neovide still launches as a xwayland client by default on my system. I don't see Wayland mentioned in the README or the Wiki, so how do I instruct it to be a Wayland client? :man_shrugging:


Compatible binary files

For compatible with binary files, add “ ||xhr.responseType==='arraybuffer'  " when status = STATUS_OK.

Is this a comment? I’m going to close, reopen if you would like to clarify.


Corrected spelling and spacing and some wording in messages

Codecov Report

Merging #354 into trunk will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              trunk     #354   +/-   ##
=========================================
  Coverage     58.19%   58.19%
  Complexity    10210    10210
=========================================
  Files          1161     1161
  Lines         74189    74189
  Branches       7322     7322
=========================================
  Hits          43174    43174
  Misses        28517    28517
  Partials       2498     2498

Impacted Files
Coverage Δ
Complexity Δ

...mpler/hc/JMeterPoolingClientConnectionManager.java
27.77% <0%> (-3.18%)
10% <0%> (-1%)

...apache/jmeter/extractor/TestBoundaryExtractor.java
97.88% <0%> (ø)
17% <0%> (ø)
:arrow_down:

...ocol/jms/org/apache/jmeter/protocol/jms/Utils.java
56.98% <0%> (+2.15%)
15% <0%> (ø)
:arrow_down:

...s/org/apache/jmeter/timers/PoissonRandomTimer.java
78.37% <0%> (+5.4%)
10% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Chef Infra Client 17.3 release

Chef Infra Client monthly release
Estimated date: Between July 12th - 15th

Closing to create a real release in ZH


[Bug][Connector-V2] Fix the bug of incorrect path in windows environment

Purpose of this pull request
fix #2979

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Can UT be added?

config file:

#
# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements.  See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to You under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License.  You may obtain a copy of the License at
#
#    http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#

env {
  # You can set spark configuration here
  spark.app.name = "SeaTunnel"
  spark.executor.instances = 1
  spark.executor.cores = 1
  spark.executor.memory = "1g"
  spark.master = local
  job.mode = "BATCH"
}

source {
  FakeSource {
    schema = {
      fields {
        c_map = "map<string, string>"
        c_array = "array<int>"
        c_string = string
        c_boolean = boolean
        c_tinyint = tinyint
        c_smallint = smallint
        c_int = int
        c_bigint = bigint
        c_float = float
        c_double = double
        c_bytes = bytes
        c_date = date
        c_decimal = "decimal(30, 8)"
        c_timestamp = timestamp
        c_row = {
          c_map = "map<string, string>"
          c_array = "array<int>"
          c_string = string
          c_boolean = boolean
          c_tinyint = tinyint
          c_smallint = smallint
          c_int = int
          c_bigint = bigint
          c_float = float
          c_double = double
          c_bytes = bytes
          c_date = date
          c_decimal = "decimal(30, 8)"
          c_timestamp = timestamp
        }
      }
    }
  }

  # If you would like to get more information about how to configure seatunnel and see full list of source plugins,
  # please go to https://seatunnel.apache.org/docs/flink/configuration/source-plugins/Fake
}

transform {

  # If you would like to get more information about how to configure seatunnel and see full list of transform plugins,
  # please go to https://seatunnel.apache.org/docs/flink/configuration/transform-plugins/Sql
}

sink {
  LocalFile {
    path="/tmp/seatunnel/dist/text/"
    row_delimiter="\n"
    partition_dir_expression="${k0}=${v0}"
    is_partition_field_write_in_file=true
    file_name_expression="${transactionId}_${now}"
    file_format="text"
    filename_time_format="yyyy.MM.dd"
    is_enable_transaction=true
  }

  # If you would like to get more information about how to configure seatunnel and see full list of sink plugins,
  # please go to https://seatunnel.apache.org/docs/connector-v2/sink/File
}


Questions about Global Memory Bandwidth

Hello,
I have one question about global memory bandwidth.
I find that global memory bandwidth may decrease for float8 and float16 in most of devices.
I hope to know the reason why global memory bandwidth decreases.
The following is my log from my MacPro.
Platform: Apple
  Device: Intel(R) Iris(TM) Graphics 6100
    Driver version  : 1.2(Apr 11 2017 16:38:15) (Macintosh)
    Compute units   : 48
    Clock frequency : 1050 MHz

    Global memory bandwidth (GBPS)
      float   : 13.71
      float2  : 14.28
      float4  : 14.75
      float8  : 7.58
      float16 : 3.97

    Single-precision compute (GFLOPS)
      float   : 631.50
      float2  : 641.30
      float4  : 640.04
      float8  : 638.97
      float16 : 634.22

    No half precision support! Skipped

    No double precision support! Skipped

    Transfer bandwidth (GBPS)
      enqueueWriteBuffer         : 4.81
      enqueueReadBuffer          : 5.41
      enqueueMapBuffer(for read) : 447.69
        memcpy from mapped ptr   : 4.64
      enqueueUnmap(after write)  : 6376.14
        memcpy to mapped ptr     : 5.27

    Kernel launch latency : 71.60 us

Thanks a lot.

While I don't know the exact reason, I can only speculate.
This might be to do with memory bus width. Spec says, it has 128 bit wide bus (aka 4 floats). I guess when it loads float8/float16, it is not fitting the cache line & trashing heavily

Thanks. I will close the issue.


build: only run prince if command found

This should automatically decide whether to generate the pdf or not, depending on the availability of prince on $PATH.
Please test both in docker-mei and on a local system. Report back on the local system OS, whether Saxon, xerces or prince were available on your $PATH.

@bwbohl This PR is missing the full section of the old PR where the prince-available command was introduced, so at the moment it is never initiated at all. Also, what is the env variable used for? Is it never called?
https://github.com/music-encoding/music-encoding/pull/1221/files#diff-766797f233c18114f9499750cf1ffbf3829aeea50283850619c01bd173132021R137-R148

@musicEnfanthen reintroduced missing section, please have a second look ;-)


Projects that rely on BOSL exceptions should say that they do

This is important licensing information since it affects how those projects can be forked. zcash_script depends on orchard via zcash_primitives and zcash_note_encryption: see
https://github.com/ZcashFoundation/zcash_script/blob/c9fbc441efd78593ba6a9828be45baf2d6469757/Cargo.toml#L57-L58
and https://github.com/zcash/librustzcash/issues/576.
Suggested wording is:

Downstream code forks should note that the code in this repository depends on
the 'orchard' crate, which is licensed under the Bootstrap Open Source License.
A license exception is provided allowing some derived works that are linked or
combined with the 'orchard' crate to be copied or distributed under the original
licenses (in this case MIT / Apache 2.0), provided that the included portions of
the 'orchard' code remain subject to BOSL.
See https://github.com/zcash/orchard/blob/main/COPYING for details of which
derived works can make use of this exception.

See also:

https://forum.zcashcommunity.com/t/bosl-or-mit-orchard/41439/118
https://github.com/zcash/zcash/issues/5931 (similar issue for zcashd)
https://github.com/zcash/librustzcash/pull/577 (PR for librustzcash)

I'm proposing the following actions and would like your input on whether we should implement them now. My plan is to submit a PR with the following changes:

Incorporate dual license files (APACHE/MIT) into this project, aligning with what Zebra and Zcash do.

Update the README by adding a license section, using a wording similar to Zebra's README as a reference: Zebra License Section.

Integrate the recommended wording from @daira regarding the BOSL into the newly created license section on the README.

Your thoughts on this would be greatly appreciated!


"or" operator doesn't seem to work within a refinement list (disjunctive facet search).

Hi 👋
Description
"or" operator don't work within a refinement list.
Current behavior
When I click on a refinement list, only the options that have a document in common with the option selected are shown - even if the refinement list is set with the "or" operator.
So - when I have a refinement list with exclusive options (ie. none is shared by a document - in a belong_to/has_one association), I can only display a single option within the refinement list.
But - with the "or" operator, I'd like to see other options as well that I can select, so I can select two different options from the refinement list and see the documents that have either one or the other.
Screenshots or Logs
      searchBrands.addWidget(
        panelForIndustries({
          container: "#categories",
          attribute: "industry.name",
          showMore: true,
          autoHideContainer: true,
          keepZeroFacets: true,
          operator: "or"
        })
      );

Behaviour - I have industries to search on brands. Each brand can only have one industry.
Before selecting anything:

Here's what I'd like as a result (using Algolia, which set the default operator as "or" - so I can select and view results from two different industries - as I would expect using the "or" attribute)

Here's the actual result by Meilisearch (the default operator being "and" - and the setting in the RefinementList doesn't seem to work)

Environment (please complete the following information):

Meilisearch version: MeiliSearch 0.27.1
instant-meilisearch version<EMAIL_ADDRESS>instantsearch.js version: InstantSearch.js 4.43.0

Hey @alcanari
Unfortunately, the way it works today is that if you are doing and you have the following results:

and if you're using or you have the following:

I agree with you that this is counterintuitive! It is planned to update this behavior to what we call disjunctive facet search.
When the next release of meilisearch (the engine) is out, I will be working on instant-meilisearch and on this issue as well.
It should come in the following weeks.
I will keep you updated once the issue is created on how disjunctive facet search works and if it seems to be what you are looking for.
meanwhile, can I suggest using keepZeroFacet option which at least will still showcase your other facets!

Hey,
Thank you. I'm not sure I follow. In your examples, you have 3 games that are both Multiplayer and Single Player? If you'd only have games that would be Multiplayer or Single Player (none can be both) - would other facets show up on the using or?
Just to make sure I undestand, because I'm already using the keepZeroFacet option :)

When you say "this is counterintuitive" -> you mean, the fact that you go from 3 to 7 for Single Player games in the using or option?

When you say "this is counterintuitive" -> you mean, the fact that you go from 3 to 7 for Single Player games in the using or option?

Yes ! My bad  I wasn't very clear

Thank you!
Yes agreed - I understand it's related to the disjunctive facet search - got it. Just to be 100% clear - the or option on the refinement list right now changes the results of the search, but not the available options to select for now? :)
Also, did you saw my previous message? We're already using keepZeroFacets but it doesn't seem to work, which makes it hard to use in a setup where there are exclusive refinement lists options.

Hey,
Thank you @bidoubiwa ! :)
I'm not sure I follow. In your examples, you have 3 games that are both Multiplayer and Single Player? If you'd only have games that would be Multiplayer or Single Player (none can be both) - would other facets show up when you're using or?
Just to make sure I understand, because I'm already using the keepZeroFacet option :)

Is there any idea of when this issue will be resolved? Thanks!

Hello @johnwbaxter
I cannot guarantee any time for the moment unfortunately :/ I will try to create the issue explaining in details how it should be implemented asap so that contributors may implement it if they want it to be released sooner

I add a flag on this (we add keepZeroFacets on configuration for the moment but i'm really interesting on that behaviour

Would be great to keep moving forward on this where possible, this is a critical feature for us

This is a pain point for us too.
We have "or" based filter
This is the refinement list without any filters

When we select 1 this is what we would see (notice other options becoming 0)

But in fact, the document count available is not 0 but 6 since this is an "or" based filter. When we select document as well, our total search hits become 10 (6+4):

This is what we expect to see: (filter count available, not filter count in current hits)

Does anyone know when we will get an update on this one?

Hey @jeremylynch. It is the next task we are working on :) I'll keep you updated

Hello there,
I'd like to ask about this feature, is there any update with that?

Working on it for the moment! We have been slowed down with hacktoberfest unfortunately

Working on it for the moment! We have been slowed down with hacktoberfest unfortunately

Hi @bidoubiwa, any update on this?
Happy to test on our 20k+ product database which has lots of facets if it helps.

Hey @anthonycook, I'm currently working on it :) Both multi-index search and disjunctive facet search are being added simultaneously.
You can follow along #888 Which introduces the multi-index search, and should be a gate opener to adding disjunctive very efficiently

Hey @anthonycook, I'm currently working on it :) Both multi-index search and disjunctive facet search are being added simultaneously.
You can follow along #888 Which introduces the multi-index search, and should be a gate opener to adding disjunctive very efficiently

Amazing! Thanks for the update, I will follow along 😊

Thank you for the update @bidoubiwa !

Sorry for my question, and maybe it's a stupid question, but when will this feature be available, would be happy to use it in existing project.
Thanks!

Hey @kamopeter
The beta should be released this week on npm

Thanks! :)

The disjunctive search beta is released! Can you try it out? It is published on npm here

It is released on the latest dist tag in npm and should be acting as expected with v0.11.0!


Table export a value of len 0

The CPU architecture family 68000 (68040, 68030, 68020 and coldfire) all include the table fcc (variant sf) export a value of len 0.
It's unclear how Sleigh should handle a values of len 0.

Merged into ghidra


[MM-44815] Duplicate task as next task instead of the last task

Summary

Duplicates the item next to itself. The behaviour before this was to add to the end of the list.
Technical Description
Instead of a simple append towards the end of the list, this PR creates a new list which is a copy of all the items before and after the duplicated item and returns it after adding the item at the supposedly new position.
Ticket Link

Fixes: https://github.com/mattermost/mattermost-server/issues/20824
Checklist

[ ] ~Telemetry updated~
[ ] ~Gated by experimental feature flag~
[ ] ~Unit tests updated~

Hello @varundey,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Looks mostly good, though I think the ordering logic has one issue where the new item is being put before the old item instead of after. Please let me know your thoughts on this.

Sure we can put the new item after the original one instead of before.

Also do we need to update this function?

I looked at this but I could not find an instance where playbookRunId is not present here. If you could help me figure out when and how this function gets called, I can try testing this as well.

I looked at this but I could not find an instance where playbookRunId is not present here. If you could help me figure out when and how this function gets called, I can try testing this as well.

I'm not too familiar with the project, so I'm not sure what conditions would cause this. You could force the condition and make it so the duplicate function is always called, for the purpose of testing your feature.

I'm thinking the client-side function still needs to be updated, so I'll defer approval until the discussion is resolved

Fixed this too

I'm thinking the client-side function still needs to be updated, so I'll defer approval until the discussion is resolved

Fixed this too

Whoops. Clicking on re-review removes other reviewers. 😬


Rewrite makefile

[to be filled in]
Currently just testing in CI, and some minor cleanup is expected to come soon.  But comments welcome!

Coverage decreased (-0.04%) to 64.502% when pulling 4ad25442290debf3f396c0b6f3dabcb01f7a0e0e on rewrite_makefile into 0c9a74d39babe1d3bdfd030b10055c06fd7e4bdb on master.


Fixed Donate page issue

I have changed the donate page where if user wants to enter their name while donating they can click on Name Donation and if a user wants to donate Anonymously then do that also.

@gunajidiksha  Please resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Space in path causes exception

Code point https://github.com/Netflix/feign/blob/master/httpclient/src/main/java/feign/httpclient/ApacheHttpClient.java#L105
will cause exception when URI has space(s) in path: http://localhost/path /to/resource

Fixed in https://github.com/Netflix/feign/pull/274


Move bundling of Worker under a flag

Add support for turning off bundling of Worker, for situations where we only need to bundle SharedWorker.
Basically, it allows projects that want to use SharedWorker (and don't need Worker) to use this plugin without facing issues such as https://github.com/GoogleChromeLabs/worker-plugin/issues/12

@gbmatt Are you able to sign the CLA? Cheers!

@googlebot I signed it!

Published in 4.0.1!


Add comment Italic

Thank you for your work. I like your theme very much. I hope you can add this function in your spare time.

@Mofiqul  Thank you for your work.


Privacy link doesn't redirect to the log in page

When the app is running and in the Log out state, privacy link doesn't redirect to the log in page even when the [Authorize] attribute to the Privacy page is in place.

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

ID: 94bfd895-d2af-390b-0801-428b36a27ff4
Version Independent ID: 0e0d870e-e49a-e16c-c259-68d58e701b8f
Content: Introduction to Identity on ASP.NET Core
Content Source: aspnetcore/security/authentication/identity.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I can't reproduce the problem.

I followed your tutorial step by step and created the app, Scaffolded the Register, Login, Logout pages. Then as per the Test Identity section wrote as follows:
Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Mvc.RazorPages;
namespace WebApp1.Pages
{
[Authorize]
public class PrivacyModel : PageModel
{
public void OnGet()
{
}
}
}
When the app was run and I navigated to the Privacy link it didn't require login .
If you wish I could share the code as a GitHub link

I am using Visual Studio 2017 where putting [Authorize] as an attribute on the Privacy page is showing a compile-time green squiggly "AuthorizeAttribute cannot be applied to Razor Pages handler methods....."
Probably I shall need to upgrade to VS2019 to get it going.

Apply it to the class.

Per the article
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Mvc.RazorPages;

namespace WebApp1.Pages
{
    [Authorize]
    public class PrivacyModel : PageModel
    {
        public void OnGet()
        {
        }
    }
}


[1/n] torchtune <> llama-stack integration skeleton

Context
This is the 1st of series PRs that integrate torchtune with llama-stack as meta reference post-training implementation. For MVP, we will focus on single device LoRA SFT.
Though this PR is still WIP, we want to get early feedback on the high level design of this skeleton while still working on several details
Scope
To limit the scope of this PR, we focus on the skeleton of the implementation.
What are included?

refine the post-training SFT apis
skeleton of supervised_fine_tune implementation. We verified that we can call the supervised_fine_tune API successfully from llama stack client SDK (client side PR: https://github.com/meta-llama/llama-stack-client-python/pull/51)
a very basic single device LoRA training recipe based on torchtune core components

What are not includes?

detailed unit tests, numerical parity check etc (will be added in this PR)
implementation of other job management, get training artifacts apis (separate PR)
several necessary functionality in the training recipe such as logging, validation etc (separate PR)

Testing
Although we haven't added detailed testing and numerical parity check with torchtune yet, we did a simple E2E test from client to server

setup server with llama stack build --template meta-reference-gpu --image-type conda and llama stack run meta-reference-gpu
On client, run  llama-stack-client --endpoint http://devgpu018.nha2.facebook.com:5000  post_training supervised_fine_tune
Training finishes successfully. On server side, get the finetune checkpoints under output dir. On client side, get the job uuid

server

client

added some initial comments


Location of hyper's C API

In providing a C API for hyper(#2265, #2278), a decision is needed on where exactly the code should live: either inside the hyper crate, or as a separate one. As usual, there's some pros and cons for both options, which I'll list out.
A separate crate (hyper-capi)

The API can be versioned separately. Breaking changes in the Rust API don't necessarily mean that the C API must also break.

Optional part of the hyper crate (hyper::ffi)

If a module in hyper, we can take advantage of crate-internal details. For example, hyper::Error internally is a wrapper for a Box<ErrorImpl>. In order to expose a hyper_error *, we can just return the already existing box, instead of boxing a second time if the code were in a separate crate.
We don't need to make "intermediate" public APIs if we wish to add an option to the C API that isn't normally exposed in the Rust API.
We can more easily tie the version of hyper to the version the C API sees, such as when asking for hyper_version().
We can keep the same library name for both Rust and C: libhyper. If using a separate crate, we can't name it libhyper (so no -lhyper), but something like libhyper_c.
This module can be an optional ffi (or capi or similar) feature, so it isn't included by default.

This was done in https://github.com/hyperium/hyper/pull/2278/commits/618f03449d3c4f51d9ce6e540ec8c6010309ed89


Of Interest:  SwiftPM support for code generation

SwiftPM has proposed a way to support code generators such as SwiftProtobuf.  If you're interested in such support, please take a look and let the SwiftPM team know what you think!
https://forums.swift.org/t/se-0303-package-manager-extensible-build-tools/45106
(Posting this as an issue so that people watching this project will be aware.  We can close this issue soon.)

Closing as I believe we now have this.


Offline help is incomplete

Using the Help Viewer in VS2017 with all relevant books added, lots of content, especially Win32 API documentation, is missing. Some categories aren't included at all, others have only some functions available. In the latter case a warning on the top says "You are viewing online content. The topic is not available on your computer, but it is available in one or more books. Select a book to download." but the book it suggests (e.g. "Windows Desktop App Development") is installed. Re-installing the book(s) does not help.
Examples of missing docs I can remember:

GDI reference (missing completely)
DLL functions (missing e.g. DisableThreadLibraryCalls, FreeLibrary, GetProcAddress, …)
Memory management (missing VirtualQuery, GetProcessHeap, HeapAlloc, …)
Direct3D 11 (missing e.g. ID3D11Device, ID3D11Device1, ID3D11Device4, ID3D11Device5 interfaces while their member functions are mostly available, …)
Media Foundation (missing e.g. IMF2DBuffer interface while its member functions are mostly available, seems to be a recurring pattern with COM interfaces)

@dastaffo Is this your area?

@ShevaDas owns this area now


Improve OPF-058/059 usage messages

The OPF-058 and OPF-059 usage messages state that "Spine item has no TOC/NCX entry reference." without mentioning which item the message actually refers to.
However, that information is available when the message is reported in EpubNavCheck.java:196 and EpubNCXCheck.java:176 as the location’s path property. Looking at the report message formatter (for terminal use in my case, DefaultReportImpl.java:90) I see that the location’s path property (i.e. location.getPath()) isn’t actually being added to the usage message. Adding it breaks a bunch of test cases so I did not pursue this matter further for now.
I think it would be quite useful if the usage messages contained the above path information.

Hi Jens,
any chance you can send an initial PulLRequest with changes? This would raise the chance that this change will make it into the 4.1 release end of the month.
I can handle the failing tests then...

Hi Tobias 🤓
How about I add the path alongside the remaining location information in the format string of formatMessage()? Something like this:
    return String.format("%1$s(%2$s): %3$s%4$s(%5$s:%6$s,%7$s): %8$s",
        message.getSeverity(),
        message.getID(),
        epubFileName,
        fileName,
        location.getPath(),
        location.getLine(),
        location.getColumn(),
        fixMessage(args != null && args.length > 0 ? message.getMessage(args) : message.getMessage()));

If a path can be null (can it?) then that should be accounted for, of course. I’d change this in every formatMessage() method of all classes that derive from the MasterReport class. Anywhere else?

Hi Jens,
I'm not sure whether the formatMessage() is the correct spot to make this fix as it would affect ALL messages.
Normally, fileName parameter should contain the filename of the file where the error was thrown. Unfortunately you only posted a truncated message string in our original post so I can't reproduce at the moment.
My suggestion is, to alter the OPF58 and OPF59 messages and add the offending path as a part of the message.
Can you post a complete error message string with error code and filepaths, etc.?
And then another version which contains the changes you would like to see in the message?
Thanks!

Not to butt in on this thread, but the messages could use some improvement language-wise:

OPF_058=Spine item is not referenced from the TOC.
OPF_059=Spine item is not referenced from the NCX.

I had to read the descriptions to understand that it wasn't an error about a missing toc attribute on the spine, and that these weren't duplicates.

Hi,
I need a little help
I have this doubt. take this space in blank between two tags.
how do I do this automatically?

(space in blank)

this is to fix in epub xhtml
thanks
On Fri, Nov 10, 2017 at 11:33 AM, Matt Garrish<EMAIL_ADDRESS>wrote:

Not to butt in on this thread, but the messages could use some improvement
language-wise:
OPF_058=Spine item is not referenced from the TOC.
OPF_059=Spine item is not referenced from the NCX.
I had to read the descriptions to understand that it wasn't an error about
a missing toc attribute on the spine, and that these weren't duplicates.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IDPF/epubcheck/issues/804#issuecomment-343474257, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFyQ-znyuLt2YickTZyiDyihdPFJTjawks5s1FChgaJpZM4QVoIj
.

--
AC
Antonio Carlos Espilotro
A2 estudio de criação
Rua Francisco Leitão 339 - 1º andar - Pinheiros - São Paulo
11 991121257
www.a2.com.br
www.acespilotro.com

@Espilotro this is the wrong place for such a question. Sorry.

Adding this to the 4.1 milestone. If @jenstroeger feels like taking it, take it! Otherwise if I don't hear from you Jens, I'll try to fix it this weekend or beginning of next week.

Can you post a complete error message string with error code and filepaths, etc.?
And then another version which contains the changes you would like to see in the message?

This would still be great! @jenstroeger or @mattgarrish, feel free to post rephrased messages which should get implemented...
Based on Matt's suggestion I think we should rephrase it like

OPF_058=Spine item 'spine-id' is not referenced from the TOC.
OPF_059=Spine item 'spine-id' is not referenced from the NCX.

And: shouldn't TOC be rephrased to NAV doc? WDYT?

Spine item 'spine-id'

If the actual id can be given there, that would be perfect.

And: shouldn't TOC be rephrased to NAV doc

Yes, I was wondering about that. The navigation document is technically the holder of the table of contents, just as the NCX does the same for EPUB 2 (plus there's the toc attribute, just for fun). They've sort of become synonymous with the table of contents, but I wonder if we should be specific here what we mean:
OPF_058=Spine item 'spine-id' is not referenced from the TOC in the Nav Doc.
OPF_059=Spine item 'spine-id' is not referenced from the TOC in the NCX.
No one's ever suffered from too much clarity.

@tofi86, sorry my planning tumbled and I won’t get to this for the rest of the week 🤔

alright, no worries! I'll try to take care of it on the weekend.
thanks matt, that sounds good!

Started to work on this in PR #811. Actual fix: 5min. First test cleanup: 1h, still 40 other test to go... 😢


Feature request: Drag and drop tags

It would be nice to be able to drag and drop tags which would in turn be reflected in the filesystem as well. This would make organizing really easy.

I don't understand the idea.
In my understanding tags are not related to the file system and a document can be labeled with several tags. How this could be reflected in the file system without duplicating documents?
Or do you mean nested tags can be change using drag and drop and the changed tags will be written back into the documents by changing the content of the yaml front matter?


In a code:
for fleet in empire.fleet():
    if fleet.mission == mission.expedition:
        print("Expedition arrival at:", fleet.arrival)

The 'fleet.arrival' sometimes picks up a wrong arrival time. (that is an arrival time of a different fleet with different ID and Mission type).
Haven't yet found out why, however still filling a ticket for tracking puropes, in case anone else would encounter this.
Using Python 3.7.7

it could be cause by a fleet that has no arrival time causing to shift the arrival time to other fleets

I will look into this. Maby it will be bedder to redo the hole thing. its the second attemt of this function i have done

Fixed with https://github.com/alaingilbert/pyogame/commit/fb82fcaf202ec9d5d45649948d24c24bc56b7bc0


feat(#73): assets url environment variable for images

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
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 change)

Checklist:

[x] My code follows the code style of this project.
[] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@diegolezana could you also take a look at https://github.com/vuestorefront/spree/blob/main/docs/guide/configuration.md? I think we should mention the new config param there


Added domain opolelo3.onmicrosoft.com.

Highschool site: ~~http://www.lo3.opole.pl/index2.php~~
Edit: Now it is just https://www.lo3.opole.pl.

@ankiedos Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Street adress: Poland, Opole, ul. Stanisława Dubois 28 45-070
Proofs:

Anything else like a document or a letter or an instruction I can not provide because we only have got pieces of paper with our email adresses and temp passwords.

@ankiedos Thank you for the clarification. Pull request merged.


General Application Improvements

Overview

Epic that tracks functionality and improvements applicable to the DP Creator 2 application as a whole

Issues

[ ] #88
[ ] #89
[x] #105
[ ] #155
[x] #154
[x] #153
[x] #150
[ ] #149
[x] #141

Dependencies

None

Related

https://github.com/IQSS/dataverse-pm/issues/337
https://github.com/IQSS/dataverse-pm/issues/331

Resources

DP Creator II POC plan

As a group, this it too vague to be useful. Individual issues can remain open.


Feature Request: A "pseudo-user" class with methods that, e.g., trigger events ordinarily triggered by a real user

The idea here is that this would be useful for unit testing GUI components.  Data attributes would govern things like where a "location-possessing" event (such as a touch) is intended to have originated from; the programmer could then assign such values using either bounded random functions, literals, expressions, or whatever is appropriate for the case at hand.  Thanks!

I'm missing a bit of context because I don't fully understand what the idea is and where it'd be implemented. Can you expand, with maybe an example?

Example methods would be HitButton and MissButton, which, given as input
the size and position of a button, would generate ten, say, random
positions inside and outside the button, respectively, then serially feed
the button these positions as if ithey were the result of a user
interaction: in this way a unit test (the basic use case for such a
pseudo-user) could test a programmer's implementation of a button, for
example.
On Saturday, February 28, 2015, matham<EMAIL_ADDRESS>wrote:

I'm missing a bit of context because I don't fully understand what the
idea is and where it'd be implemented. Can you expand, with maybe an
example?
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3019#issuecomment-76563698.

--
David Lawrence Goldsmith (DLG)
Software Engineer
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center – NOAA
2725 Montlake Blvd E.
Seattle, WA 98112
(206) 302-2420 <%28206%29%20860-3341> (w)
(360) 481-3800 (c)

I don't think this belongs in kivy. Kivy already provides the tools to simulate inputs (http://kivy.org/docs/api-kivy.input.recorder.html) and what you're suggesting is useful in your specific case but is not generic enough, or at least you have not proposed a generic enough system for this to belong in kivy.

Ok, thanks, I didn't know about inputs, that does sound like what I had in
mind, more-or-less.  @matham: do you mind if I email you off-list with
Q's about garden.graph?
On Monday, March 2, 2015, matham<EMAIL_ADDRESS>wrote:

I don't think this belongs in kivy. Kivy already provides the tools to
simulate inputs (http://kivy.org/docs/api-kivy.input.recorder.html) and
what you're suggesting is useful in your specific case but is not generic
enough, or at least you have not proposed a generic enough system for this
to belong in kivy.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3019#issuecomment-76757582.

--
David Lawrence Goldsmith (DLG)
Software Engineer
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center – NOAA
2725 Montlake Blvd E.
Seattle, WA 98112
(206) 302-2420 <%28206%29%20860-3341> (w)
(360) 481-3800 (c)

Well, try to catch me on IRC. It's been a while since I looked at Graph, and my hope is that it will be replaced with matplotlib using a kivy backend so I had been looking at that direction rather than focus energy on Graph. If you ask on IRC and it's not suited we can do it offline, but on IRC others may help as well.

Interesting, as the features I want I know are already in matplotlib: do
you have a release target?  Is any of it alpha-ready?  (I particularly need
the sub-plot functionality.)? I'm afraid I'm an IRC novice: can you suggest
a windows client?  Or do you use a browser?  And do you post messages: if
no one is on when I post, what happens?
On Monday, March 2, 2015, matham<EMAIL_ADDRESS>wrote:

Well, try to catch me on IRC. It's been a while since I looked at Graph,
and my hope is that it will be replaced with matplotlib using a kivy
backend so I had been looking at that direction rather than focus energy on
Graph. If you ask on IRC and it's not suited we can do it offline, but on
IRC others may help as well.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3019#issuecomment-76769397.

--
David Lawrence Goldsmith (DLG)
Software Engineer
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center – NOAA
2725 Montlake Blvd E.
Seattle, WA 98112
(206) 302-2420 <%28206%29%20860-3341> (w)
(360) 481-3800 (c)

Unfortunately I'm only past the exploratory stage of figuring out that it is possible and the probable best ways to approach it. I have not had time to write actual code and then we decided to put it up as a GSOC project so I put it off hoping someone will take it up there (http://kivy.org/docs/gsoc.html#project-ideas).
Regarding IRC, here's a brief intro: http://www.pyladies.com/blog/irc-resources/ and one slightly longer: http://www.instructables.com/id/How-to-use-IRC-Internet-Relay-Chat/. Personally, I use Quassel as a client and I like it. However, before that, I just used the website (http://webchat.freenode.net/) as described here: http://kivy.org/docs/contact.html#irc.

OK, thanks!  One last follow-up and then I'll reserve further Q's for IRC:
in your explorations, were you able to gauge the likelihood that you'll be
able to provide a backward-compatible API, so that I won't need to
re-factor code I'm writing now, using graph, once the change-over is made?
I'll need this info in discussions w/ my boss re: how best to proceed.
(The only reason we're using Kivy is for support of hand-helds in the
future (not the present), and the only reason we're using garden.graph is
because Kivy doesn't support matplotlib.  But if the plan is for Kivy to
support matplotlib in the future, and our hand-helds support is in the
future, maybe I should just be writing the graphics portion of our package
in matplotlib from the outset; it would certainly make my life a lot
simpler!)  Thanks!
On Mon, Mar 2, 2015 at 10:37 AM, matham<EMAIL_ADDRESS>wrote:

Unfortunately I'm only past the exploratory stage of figuring out that it
is possible and the probable best ways to approach it. I have not had time
to write actual code and then we decided to put it up as a GSOC project so
I put it off hoping someone will take it up there (
http://kivy.org/docs/gsoc.html#project-ideas).
Regarding IRC, here's a brief intro:
http://www.pyladies.com/blog/irc-resources/ and one slightly longer:
http://www.instructables.com/id/How-to-use-IRC-Internet-Relay-Chat/.
Personally, I use Quassel as a client and I like it. However, before that,
I just used the website (http://webchat.freenode.net/) as described here:
http://kivy.org/docs/contact.html#irc.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3019#issuecomment-76773899.

--
David Lawrence Goldsmith (DLG)
Software Engineer
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center – NOAA
2725 Montlake Blvd E.
Seattle, WA 98112
(206) 302-2420 <%28206%29%20860-3341> (w)
(360) 481-3800 (c)

The API with matplotlib would be for you to call mpl methods like you normally do with mpl, except the graph would show up in a kivy widget. So no, it'd not be a drop in replacement because currently you modify the Plot.points list, but with mpl you'd call e.g. plt.plot. However, I'm not sur ethat this is a significant difference?
But I wouldn't expect to have a working prototype, even in alpha phase in the next few months, unless it's taken up as a GSOC project, or someone else here adapts it.

OK, thanks.  TBC on #kivy (or do either kivy.garden or kivy.garden.graph
have their own channels?)
On Mon, Mar 2, 2015 at 11:21 AM, matham<EMAIL_ADDRESS>wrote:

The API with matplotlib would be for you to call mpl methods like you
normally do with mpl, except the graph would show up in a kivy widget. So
no, it'd not be a drop in replacement because currently you modify the
Plot.points list, but with mpl you'd call e.g. plt.plot. However, I'm not
sur ethat this is a significant difference?
But I wouldn't expect to have a working prototype, even in alpha phase in
the next few months, unless it's taken up as a GSOC project, or someone
else here adapts it.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3019#issuecomment-76789371.

--
David Lawrence Goldsmith (DLG)
Software Engineer
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center – NOAA
2725 Montlake Blvd E.
Seattle, WA 98112
(206) 302-2420 <%28206%29%20860-3341> (w)
(360) 481-3800 (c)

Nop, it's on #kivy.

OK, I just posted my question on #kivy.  Thanks!
On Mon, Mar 2, 2015 at 11:39 AM, matham<EMAIL_ADDRESS>wrote:

Nop, it's on #kivy.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3019#issuecomment-76794637.

--
David Lawrence Goldsmith (DLG)
Software Engineer
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center – NOAA
2725 Montlake Blvd E.
Seattle, WA 98112
(206) 302-2420 <%28206%29%20860-3341> (w)
(360) 481-3800 (c)


Slight audio latency/delay (about 130ms) even when using Tone.immediate() / Tone.context.currentTime and zero lookAhead.

Describe the bug
I set up a basic webpage which plays a sound when either a keyboard key is pressed, or when a MIDI input command is detected where velocity is not zero. The webpage also changes the background color of a rectangle on the page when such events occur.
This is the code invoked:
// Start the context when the button is pressed.
document.getElementById('button-start').addEventListener('click', async () => {
        await Tone.start();
});

Tone.context.lookAhead = 0; // I tried doing this, but doesn't seem to change anything.
player = new Tone.Player('/sound/0027.ogg').toDestination();

// ...
const onKeyEvent = () => {
        // Play the sound.
        player.start(Tone.context.currentTime);
        // I have also tried: player.start(Tone.immediate());

        // Change the background color.
        // ... code removed for brevity ...
};

I have noticed that there is a delay of about 130ms between the input and the time at which the sound begins to play. The background color changes instantly, but the sound begins playing a bit later. This is consistent even when the sound is played multiple times, i.e. cached.
I am running the webpage on Chrome under Windows 10, and while it's possible Windows has inherent audio lag, or perhaps even my audio card (integrated Realtek), I want to make sure it's really not because of a mistake in my code.
Expected behavior
I would be happy if there was a way to trigger the sound instantly, i.e. at the exact same time that the input is detected, the time at which the background changes.
Additional context
Please see attached video. If you analyze it frame by frame, you will notice that the background color changes a few frames before the sound begins. The video is 30fps and there are 4 frames where the background changes but still no sound, and therefore I presume the delay is about 130ms.
https://user-images.githubusercontent.com/7698618/133801589-dae04b00-b21c-4980-bb12-0fc12464c3c7.mp4

@marcelblum Thank you so much for your help! Your suggestion of tweaking the settings on Windows, i.e. uncheck "Enable audio enhancements" did the trick for me, my new screen recording shows that the latency has pretty much been eliminated! I have also changed my call to be player.start() as you suggested, keeping Tone.context.lookAhead = 0 in my code.
In fact this also helps me with an entirely other issue, where I had input/audio lag in a game, which has also become eliminated by changing the above Windows setting. Thanks!


background seed can't import specific seed files.

/api/background?seed=true
Ran into an error!
ENOENT: no such file or directory, scandir '/var/task/node_modules/@socialbaking/karma/src/karma/data/seed/medical-cannabis-new-zealand'

https://karma.patient.nz/products


Editor vs Journal Manager Roles

Editor focussed on content.
Only Journal Manager can do:

[x] Settings
[x] Workflow steps
[x] Contacts
[x] Design
[x] Theme
[x] Board

closes with https://github.com/okulbilisim/ojs/commit/915eff7ad7e63d9bb02cb2a569fd380c62b8de6f


add watch_stderr() to RunningJob and incrementally get stderr

This PR will add a new interface to the RunningJob object to watch the stderr log and print to stderr or stdout. This also will incrementally get stderr and update cached stderr.

Coverage remained the same at 89.056% when pulling 886033351e31f58082a5b5f3c358f0a5ba8b2509 on irontable:master into 08a51be5f86195a342b150df714257575d9f3e82 on Netflix:master.

Coverage remained the same at 89.056% when pulling 886033351e31f58082a5b5f3c358f0a5ba8b2509 on irontable:master into 08a51be5f86195a342b150df714257575d9f3e82 on Netflix:master.

Coverage remained the same at 89.057% when pulling 3a3a6e106b8d202d35b5257b051cfb7a4daef077 on irontable:master into 03651629c5ca463e980800bd4ebdda3cb6cb889d on Netflix:master.


net/firewall: Aliases in use can be deleted

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
Plugin version in use: 1.2
Aliases that are referenced in os-firewall automation-rules can be deleted.
I would think that this behaviour could break the ruleset (?) or at least bring problems with it.
As this problem is directly linked to the os-firewall rules I guessed the issue belongs to the plugins-repository.
To Reproduce
Steps to reproduce the behavior:

Install the 'os-firewall' plugin
Create a host-alias in 'Firewall - Aliases'
Create a rule that uses the alias in 'Firewall - Automation - Filter'
Save the rules
Go back to 'Firewall - Aliases' and delete the alias
Now the rule exists and references a non-existent alias

Expected behavior
The deletion of the referenced alias should not be possible.
The user 'Fright' in the OPNSense forum mentioned the following:

may be additional check (in Filter model) should be added at
https://github.com/opnsense/core/blob/7601e9cab3badfd78673bee3f486f8cb9f092266/src/opnsense/mvc/app/models/OPNsense/Firewall/Alias.php#L129

Reference: https://forum.opnsense.org/index.php?topic=30077.msg145270#msg145270
Environment
OPNsense 22.7.4-amd64
FreeBSD 13.1-RELEASE-p2
OpenSSL 1.1.1q 5 Jul 2022
Common KVM processor (4 cores, 4 threads)
Hypervisor: Promox PVE

We don't have a mechanism in place for handling constraints originating from plugins automatically in this case, but since this is only a single plugin managed by us, I don't mind adding the optional config path the list so it does check for our os-firewall plugin


Assembly code is broken for modulus (64 bit)

Turns out the 32 bit modulus left the x87 stack in a bad state since a fadd should've been a faddp.


[feature request] Support the setting of configuration parameters via CLI

It's handy to be able to set and/or override configuration parameters from the command line, particularly for development. I'm not yet that familiar with the cfg4j internals but I imagine this would involve creating a source that accepted CLI param configuration settings for some getopt-like library, and the returned values could be used, merged with other settings, etc. in the usual way.

Hey @crimeminister,
Thanks for the suggestion - I'm already working on adding this feature in the release 4.3.0. It's being tracked as part of the #128 issue. I'm closing this one as a duplicate. Feel free to add more comments to the other issue.


[HUDI-5956] fix spark DAG ui when write

Change Logs
DataSet.ofRows will create new QueryExecution, which is not registered, so the dag metric will not update when running job
Impact
None
Risk level (write none, low medium or high below)
None
Documentation Update
None
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@codope done


Improve config validation for group

Description:
Before an invalid entity id for groups generated this error message:
16-06-02 15:09:37 ERROR (MainThread) [homeassistant.bootstrap] Invalid config for [group]: Group petersroom is invalid: not a valid value for dictionary value @ data['entities'] for dictionary value @ data['group']

With this change it will be:
16-06-02 15:46:24 ERROR (MainThread) [homeassistant.bootstrap] Invalid config for [group]: Group petersroom is invalid: Entity ID group.tracke--rs is an invalid entity id for dictionary value @ data['entities'] for dictionary value @ data['group']

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Totally missed this, looks good 🐬


Server replication

 Issue by baj0
Sunday Oct 02, 2016 at 11:28 GMT
Originally opened as https://github.dev.global.tesco.org/baj0/mewbase/issues/41

Implement server replication where some servers can be leaders, and others followers. The followers replicate logs on the server.
We should allow all document and event logs to be replicated.
Replication can be implemented by the follower subscribing to events on the leader and then updating their own logs locally.
When the changes have been written to the logs an ack should be sent back to the leader. This should all be pipelined and async so we don't have RTT latency for every operation.
It should be configurable to send ack back to leader when either:

entry has actually been persisted to disk
entry has been accepted on replica and will be persisted to disk asynchronously
will provide better durability guarantees (no loss if the entire data centre goes down)
will provide better performance.

Now that mewbase is a library this has gone beyond scope and has moved to the underlying Event store.


Discriminator with new GridFS

Q
A

Version
2.1.2

Support Question
Is there a way to add discriminator with the new GridFS system ?
In my case I wanted to make a generic File class with basic fields (about the same as in the example) and sime others like Image, Archived, etc. that will have more fields at the root or inside the metadata.
The problem is, without discriminator when i find() a File that was an Image, it load a File object, not an Image object, so some infos are missing.
With the old GridFS, as it was document like anything else it was possible, but it seems it's not anymore. is it intended ?

The file itself can't have a discriminator mapping, as there can't be additional fields in the root document of a GridFS bucket. You can however use a discriminator when mapping the metadata document: this is what will vary between different file "types", the rest is always the same.

Yeah didn't think about that.
I even forgotten discriminator on embeddedDocument was possible.
Thanks, I'll manage with that.


net: frequent TestTCPListener failures on linux-amd64-wsl

#!watchflakes
post <- pkg == "net" && test == "TestTCPListener" && builder ~ `.*-wsl`

(attn @golang/wsl; CC @qmuntal @ianlancetaylor @neild)

I wonder if this is something to do with running afoul of the the Windows firewall, like in #59497.
I notice that several of those tests also omit the localhost address:
https://cs.opensource.google/go/go/+/master:src/net/listen_test.go;l=35-59;drc=50f2b7b76481f045649d12d2a26dd36b7cc248a3


hello thanks, get error ..

json returned "Request contains an invalid argument.". Details: "[{'message': 'Request contains an invalid argument.', 'domain': 'global', 'reason': 'badRequest'}]">

Hello @GreenAA ,
Please send a screenshot that contains the following error so I can
check and resolve it

i try
method 1 / 2
dorker -q "inurl:api site:bentley.com" --output google_dorks.txt
dorker -d bentley.com -q inurl:api -o google_dorks.txt

Hello @GreenAA ,
I have tried the same request and dorks with my api keys and configs and I got results for that also, I think possibly you api usage is exhausted for today thats why it causing this error because from my side I can't reproduce this error with command given by you and instead I got some results and you can see those results from the attachements

please try again the same dork tomorrow and if you are getting the same error I will reinvestigate and will make a update
on google dorker tool :)
kind regards,
@sanjai-AK47

thanks bro

bro now is working , thanks for all .. working.


npm run lint throws TS version warning

** PLEASE READ THIS BEFORE FILING AN ISSUE **
I'm submitting a:

[x] bug report
[ ] feature request

What OS are you using?

MacOS
What browser(s) is this bug affecting:

Node
Current behavior:

After merging canary branch to master, lint throws a warning:
WARNING: You are currently running a version of TypeScript which is not officially supported by typescript-estree.

You may find that it works just fine, or you may not.

SUPPORTED TYPESCRIPT VERSIONS: >=3.2.1 <3.6.0

YOUR TYPESCRIPT VERSION: 3.6.3

Please only submit bug reports when using the officially supported version.

Expected behavior:
No warning.
Steps to reproduce:

run npm run lint
Related code:
Other information:

This issue is caused because typescript-estree is updated in version 2 of @typescript-eslint/eslint-plugin and @typescript-eslint/parser
After trying to update those I found a couple of new lint errors
/Users/fernandopasik/Sites/material-components-web-components/packages/dialog/src/mwc-dialog-base.ts
  54:3  error  Type boolean trivially inferred from a boolean literal, remove type annotation  @typescript-eslint/no-inferrable-types
  56:3  error  Type boolean trivially inferred from a boolean literal, remove type annotation  @typescript-eslint/no-inferrable-types
  62:3  error  Type string trivially inferred from a string literal, remove type annotation    @typescript-eslint/no-inferrable-types

/Users/fernandopasik/Sites/material-components-web-components/packages/notched-outline/src/mwc-notched-outline-base.ts
  28:3  error  Type number trivially inferred from a number literal, remove type annotation    @typescript-eslint/no-inferrable-types
  30:3  error  Type boolean trivially inferred from a boolean literal, remove type annotation  @typescript-eslint/no-inferrable-types

/Users/fernandopasik/Sites/material-components-web-components/packages/tab-bar/src/mwc-tab-bar-base.ts
  166:28  error  Unexpected empty method 'firstUpdated'  @typescript-eslint/no-empty-function

/Users/fernandopasik/Sites/material-components-web-components/packages/textarea/src/test/mwc-textarea.test.ts
  71:27  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
  74:24  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any

/Users/fernandopasik/Sites/material-components-web-components/packages/textfield/src/test/mwc-textfield.test.ts
  371:23  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any
  374:20  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any

✖ 10 problems (6 errors, 4 warnings)
  5 errors and 0 warnings potentially fixable with the `--fix` option.

If it's ok I can make a branch fixing those and updating the dev dependency

Please tag me in the PR for review!


VORTEX EDITOR in the charter menu.

What is your question?
i want to know all about this setting on charter editor

and this

if theres no problem

yeah me too ima keep this open

Duplicate of https://github.com/ShadowMario/FNF-PsychEngine/issues/1924


feat(@ngtools/webpack): always use ngc to discover lazy routes

This changes the plugin to always use the @angular/compiler-cli TypeScript program but disables all generation of factory code and metadata, as well as template checking, when in JIT mode. Lazy route discovery is then always performed using the more comprehensive metadata analysis method that was previously only used in AOT mode. Lazy routes are also now only updated when TS files are changed. Further optimizations could allow for further limits on updates based on the content of the changed TS files (i.e., presence of Angular metadata, etc.).

@clydin do we still need this? Closing since it's on the devkit repo


Any chance TF version will be realeased?

Great work!
Looking forward to try tensorflow implementation is there will be any

Great work!
Looking forward to try tensorflow implementation is there will be any

May I ask if you have implemented Dadaptation in TensorFlow?
Or have you found someone else's implementation?

I'm not aware of an implementation yet.


Document fan modes

Need to document all the different modes. Byte number 2 from sensor data characteristic.

[x] Humidity
[x] VOC
[x] Light
[ ] Airing
[x] Constant
[x] Boost
[x] Pause

My fan hasn't been inactive for >= 26h, so haven't tested the airing mode. Everything is logged, so will check if we're away and this is being triggered.

Airing function
If the fan has been inactive for 26 hours, it will run an airing program for 60 minutes.
The airing function is not active when the fan in run- ning continuously. The air flow can be adjusted from the control panel or via the app.

Not sure if timer and light share the same code. My fan isn't connected to the light switch so this will never be triggered here.

Timer
Triggered via light sensor or igniter wire. Possibility of a 2 minute delay, and selectable run-on time 15 or 30 minutes. The air flow can be adjusted from the control panel or via the app.


Deployment of application is showing error

Describe the bug
Tried to deploy the project many times, used a lot of tokens:
Error message:
An error occured while streaming this message.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-kkbjzb
Steps to reproduce

go to "deploy"
after processing...An error occured while streaming this message.

Expected behavior
deployment of application
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 77140606,
  "usedJSHeapSize": 71435538,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = AntonW
Chat ID = c6129a79dcfe

Additional context
Error:
An error occured while streaming this message.

Hi @AntonWir,
We are aware of this issue that occurs during deployment and are tracking it here: #1022.


dev/bots/analyze-sample-code.dart failing with NNBD flag flip

See the failure here https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8865360821001700848/+/steps/flutter_analyze/0/stdout, this is the flutter analyze step of the flutter-engine-linux-try bot.
It looks like this is failing because the samples are being opted in.
Marking as a P1 since this will be blocking the 2.12 release.

This is still failing it looks like, but on different samples https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8865278440977658192/+/steps/flutter_analyze/0/stdout

I think https://github.com/flutter/flutter/pull/69122 should have fixed the rest, please LMK if that is not the case

it looks good now thanks :)


Yarn required to build from source on MacOS?

👋 hi, I'm a new roacher starting this week. Trying to build Cockroach from source on MacOS, I saw the following error. Should it be documented that users need to install yarn in order to build Cockroach on MacOS?
paulkernfeld@Pauls-MBP cockroach % make build
Running make with -j16
GOPATH set to /Users/paulkernfeld/go
Warning: 'ccache' not found, consider installing it for faster builds
Warning: 'bazel' not found (`brew install bazel` for macs)
build/node-run.sh -C pkg/ui yarn install --offline
cd /Users/paulkernfeld/go/src/github.com/cockroachdb/cockroach/c-deps/jemalloc && autoconf
cd /Users/paulkernfeld/go/src/github.com/cockroachdb/cockroach/c-deps/libedit && autoconf
rm -rf /Users/paulkernfeld/go/native/x86_64-apple-darwin19.6.0/proj
go install -v uptodate
bin/prereqs ./pkg/cmd/uptodate > bin/uptodate.d.tmp
make: *** [pkg/ui/yarn.installed] Error 127
make: *** Waiting for unfinished jobs....
build/node-run.sh: line 51: yarn: command not found
...then lots of more lines of code, but it seems like the build failed here

Oh, I was looking at Install CockroachDB on Mac from the Cockroach user docs but I think I should actually have been looking at Building from source on MacOS on the CockroachDB wiki.


Add Swift version in the podspec file

@Dimillian thanks for your contribution! Just so I can track this, and help future people who may have questions, what problem does this solve?


Mish inplace question

Hi, thanks for the great repo! What's the reason that the x.mul_(inner) line is commented out? Does the commented line return different results? Thanks!
https://github.com/rwightman/gen-efficientnet-pytorch/blob/d2eadab6eb4427014c3b80eeaff1ff82d2d1f3f4/geffnet/activations/activations.py#L20-L25

It's dead code now, I should get rid of it.. when I first implemented mish I though I'd be smart and make it a little more efficient with the inplace variation... however it triggered an inplace gradient error (during training)

Oh, that's strange. I trained with this same implementation without any errors, but got slightly worse results than Swish, wasn't sure if the inplace operation had anything to do with it.
https://github.com/ultralytics/yolov3/blob/429d44282cec6f07640057ff7782f5b751b02e4a/models.py#L124
class Mish(nn.Module):  # https://github.com/digantamisra98/Mish
    def __init__(self):
        super().__init__()

    def forward(self, x):
        return x.mul_(F.softplus(x).tanh())

@glenn-jocher Maybe enabling anomaly_detection would let you know if there was a silent failure wrt to the gradient calc?

@rwightman that's an interesting idea. How is it enabled exactly?

I believe you put your training loop, or call to it under a detect_anomaly context manager... been a while since I've used it, has a performance impact but this sort of error should be detected quickly.
with autograd.detect_anomaly():
    training_loop()


Text in editionView on each of reference_many doesn´t show

I'm having trouble loading an editionView on a field of type reference_many. The inner label for each reference doesn´t show. Adding new works fine and they show up with text. My configuration worked fine in 0.7 and have have doublechecked the configuration according to NG Admin Demo and see no difference. Did I miss something?

All the best
Henrik

Hi, I came across the same problem, any update here?

Sorry, my bad. I was changing the same field type from reference to reference_many
My is okay now.

Simple solution to this. The API needs to have and endpoint returning single entity by entity ID. This wasn´t needed in earlier version.

There was a regression with reference.singleApiCall(), which was fixed a couple days ago. Could you check that the master branch has your bug fixed?

+1 I'm also having trouble with this. The server response is:

And I'm using the simple field:
nga.field('features', 'reference_many')
    .targetEntity(admin.getEntity('cases'))
    .targetField(nga.field('title'))

The correct requests are triggered in my case /cases/:id, and I know the filed title is present in every one of those response (I've tried various other fields too), but I'm getting the javascript error:

Resulting in:

@ajarbol are you using the master branch, or the latest tagged version?

@fzaninotto I've built from source on the master branch and I'm still seeing this error, thank you for your time.

+1 having the same issue

I just had exactly the same problem, and it turned out it was caused by a type mismatch.
The list of reference-able items had ids as strings, while the list of references had ints.
I suppose that happens because ids are compared with ===, and it's fine, but a warning in debug mode would have been useful.

Thank you all for your feedback.
Feel free to reopen the issue you face it again with the master version, I close it for now.


Scripts to test if the build is broken

maybe using docker (not travis to make it portable)
this is useful when using with a pre-commit hook or to verify project's build status.

Added Circle CI script


User/Person's Position is not set properly in Fedora from Registration.

When a new user registers using the "other" field for Position, it's not being set properly in Fedora. So, it's not being displayed properly.
Example:
Position: {:value=>"other", :class=>"custom"}

I just pushed a modification. I needed to get rid of the "user-defined" and "custom" classes in _position.html.erb. I originally put those in thinking I would have time to write some javascript to populate those if selected, but I didn't and forgot to change it back. I did the same for _referral_method.html.erb.
Then,  I updated the user_to_person_sync_service.rb to set the position properly in Fedora.

Added another change, though it's not an optimal fix. (We need to get solr indexing fixed.)
Also, note -- with this change, any existing and affected user registrations/profiles will need to be updated and sync'ed with Fedora through admin panel in order for the changes to be seen properly.


How to make something like growsThenShrinksHel ?

Thank you for very interesting library. I was trying to make a small page with it and found a problem:
Example:
slide:
  nbText: "Init"
  fragment @[grows], @[shrinks]:
    nbText: "11"
  fragment @[grows], @[shrinks]:
    nbText: "22"

The thing I do not like here - too many arrow-downs: one to grow "11", next to shrink "11", next to grow "22"
Is it possible to make it with just one arrow-down? like we have for fadeInThenOut but growInShrink ?
Also, looks like it needs a lot of other pairs, that is why - is it possible to make the then condition without the built-in animations with "Then" ?

Hi, nice to hear :)
Do I understand you correctly that this is the behavior you want?
Press 1: grow "11"
Press 2: shrink "11" and grow "22"
Reveal.js supports adding custom fragments by creating CSS classes so it should be possible to achieve. What kind of API would you like? I propose something like this:
slide:
  fragmentThen(grows, shrinks):
    nbText: "11"
  fragmentThen(grows, shrinks):
    nbText: "22"

where fragmentThen creates the necessary CSS class for the combination for us. So you wouldn't get an explicit animation called growsThenShrink but use this instead. Implementation-wise this should be easier than having to deal with macros to modify the enum. How does this sound?

@HugoGranstrom I think that the framentThen looks very good.
fragmentThen(@[grows, fadeIn],@[shrinks, fadeOut]) for multiple animations?
fragment grows=>shrinks:   # probably too much syntax sugar

Probably I have one more idea, not sure how easy to implement it.
It is clear that about every animation has it reversed equivalent. like grows=>shrinks, fadeIn=>fadeOut and etc.
Probably it would be possible to find the reverse automatically and to simplify fragmentThen(grows, shrinks)  into something like fragmentReverse(grows) which will do the same. Not sure that the fragmentReverse is the good name

fragmentThen(@[grows, fadeIn],@[shrinks, fadeOut])

This is a great idea :+1: I have come up with an easier way of solving this now so this should be possible. Instead of creating new classes, we play with the order of the fragments instead.

fragment grows=>shrinks:   # probably too much syntax sugar, fragmentThen is better

Yeah, I'd like to keep things as simple as possible. The meaning of => is very unclear for those who wouldn't know about this feature.

Probably it would be possible to find the reverse automatically and to simplify fragmentThen(grows, shrinks) into something like fragmentReverse(grows) which will do the same. Not sure that the fragmentReverse is the good name

This is something which is too niché IMO to include. It is easy enough to create a template yourself for things like this. For example:
template growThenShrink(body: untyped) =
  fragmentThen(grow, shrink):
    body

Now you can reuse it easily and define templates for the combinations you personally use the most often.

I have a somewhat working code locally now, there is just one problem. It's a general problem really, when you do fragment(@[fadeIn, grow]), the fadeIn animation is ignored and it is visible from the beginning. Hence you need to separate them: fragment(@[fadeIn], @[grow]) which first fadeIn and then grow (two clicks needed). So this becomes problematic for your example:
fragmentThen(@[grows, fadeIn],@[shrinks, fadeOut])

as the fadeIn is overridden by the grows. So we have two options, either allow multiple pre-animations like this (requires multiple clicks for fadeIn and grows):
fragmentThen(@[@[fadeIn], @[grows]], @[shrinks, fadeOut])

Or we would basically need something which can add a fadeIn at the same time as an animation. Something like this (only one click required):
fragmentFadeInAtSameTime:
  fragementThen(@[grow], @[shrink, fadeOut])

Where the fadeIn inserted by fragmentFadeInAtSameTime takes place at the same time as whatever the next animation is. This second idea could be generalized to support any animation like:
fragmentNext(fadeIn):
  fragment(grows):
    nbText: "11"

Here the fadeIn happens at the same time as the next animation, grows in this case. And a "shortcut" template fragmentFadeInNext could be created to simplify this process.
Do you have any opinions on this? Or a better proposal even? :)

It is just an idea that it could be solved with fragmentReverse, but I am not sure.
I like an idea with nested fragments more, but need more time to play with it. The reasons I asked - to find possible simple solution, like highlighted lines of code

fragmentThen is implemented as of nimiSlides 0.2.
fragmentReverse isn't general enough. What would be the reverse animation of highlightGreen? The reverse of strike?
You now have the needed building blocks to implement your own fragmentReverse template.


Add multiscale framework to physics with an example

Not in a state to be merged yet, but gives visibility on the work. If you have any feedback, do not hesitate.
Checks to be done before submitting your PR

[ ] python3 -m pytest tests/ runs successfully.
[ ] black . runs successfully.
[ ] make html runs successfully (in the docs/ directory).
[ ] Updated docstrings related to the changes (as applicable).
[ ] Added an entry to the CHANGELOG.rst.

Should the downsample etc. functions be methods of Physics? It feels quite specialised, maybe it's better to put them in a separate class e.g. MultiScaleBase and when you want to use it you could do
class MultiscaleInpainting(Inpainting, MultiScaleBase):
    pass

Marking as draft

Thank you for this interesting feedback @Andrewwango. I agree that having a separated multiscale framework could allow for more general algorithms, and that multiscale is not necessarily related to Physics class (e.g. an upscaling algorithm for handling non-uniform deformation with antialiasing as we discussed).
Maybe, as a first step, multiscale could be associated with Physics (with the currently proposed structure) and adapted later if a more relevant structure becomes easier to spot. I think about this strategy pro/cons as follows:

'+' provides tools and interface for handling coarser scale in the context of inverse problems concisely (as shown in the example)
'-' will change the interface of physics : if we choose the proposed structure, it means that if we change Physics interface later, it might break users code

I think that if the interface on Physics makes sense, then we can accept the proposed structure (because if we implement in a separate class later, we can always use it in Physics afterward). How do you feel about it ? @Andrewwango, @tachella

I think that it makes sense to add it to Physics directly with a NoImplementedError by default, since it doesn't add overhead to the physics class, and doesn't require defining more intermediate classes.

@Andrewwango I like this idea. MultiscaleBase would then have multiscale related methods (so Physics would be unchanged) and the downsampling operator would be a member of MultiscaleBase. I have to think about how MultiscaleBase would use fine scale physics methods such as A_dagger (probably by sending "self" to init of MultiscaleBase, which would be member, I will see if this is possible)

@Nils-Laurent @tachella agreed, it was more a conceptual point whether to_coarse should be that general (esp when lots of physics can't implement it) - just thinking ahead if we have many more functionalities like this it might crowd Physics ("cognitive overhead").
How about every time we want to let a new physics have multiscale functionality we let it derive from MultiScaleBase too? Eg
class Physics:
... unchanged

class MultiscaleBase:
... as above

class MRI(DecomposablePhysics):
... unchanged

class Inpainting(DecompPhysics, MSBase):
...
... def to_coarse(...):
...

This way the user doesn't need to implement their own class (unlike what I initially suggested) but it still separates the multiscale stuff into their own logical class

I think this solution is better, but I still don't agree with the idea that many operators will not be able to implement the multiscale approach. We could potentially provide a default implementation in physics of to_coarse that returns $A' = AS$ where $S$ is an upsampling operator, and this should work for any operator.

@tachella This is a good point. Does that mean that we would target LinearPhysics ?

@tachella This is a good point. ~Does that mean that we would target LinearPhysics ?~ Nevermind, I agree it works in the general case

Cool - I didn't realise that there could be a default implementation - in which case that looks good to me. Looking forward to seeing what that would look like.

I have changed Physics  class to have a general to_coarse function.
I am unsure on one point, should there be a initialize_downsampling_operator ? (It has one usage in downsample_signal but it is mentioned in to_coarse, which cannot be used if there is no downsampling operator).


Feature: Add one line install script and instructions to README

Install cli could be more convinient for for users to not guess b/w arm, x86_64 and aarch etc..
Plus it will provide better installation for dev-ops team as one interface.
Verify
╰─$ cat install.sh| sh -
######################################################################## 100.0%
Installed successfully to: /Users/pulkit.kathuria/git/action-coveritup/app/frankenphp

╰─$ ls -lh frankenphp
-rwxr-xr-x  1 user.name  679754705   105M Feb 25 20:15 frankenphp

Looks like some issues with shellcheck in GHA (mostly variables need to be quoted). Can you address that?

Thanks for the review @withinboredom
I have fixed the lint errors on spellchecker, I think rest of the lint errors are un-related to changes in this pull request. May want to re-run the CI.

Ideally it'd be also nice to have this script hosted on https://frankenphp.dev/install.sh instead of raw.github... But that is topic for another day - and may depend on how is the docs pipeline structured.

There are still remaining issues (double quoting to prevent globbing). The paths should be quoted because if my current pwd is /home/withinboredom/bin*2024 it will fail in fun ways trying to download the file.

Squashed commits and lints fixed.

Is this safe?
Composer uses a checksum for instance: https://getcomposer.org/download/

Is this safe?
Composer uses a checksum for instance: https://getcomposer.org/download/

No comments on this.
I mean I guess you could add this extra layer of security and have checksum. At the same time I also don't see the point as it is not a package manager. Homebrew installation doesn't have similar either and not many other curl shell installations.

I also don't see the point

Incomplete downloads, bad actors/broken caches on corporate middle-boxes, and a host of other things can cause checksums to be invalid. It's worth detecting it if we can, if for no other reason than to save people some time when things aren't working as expected.

Ok, that works for me. That would be nice to have an official brew and maybe APT repositories too (easier to update).

Hello,
They are remaining tasks to do before merging this PR ? Would be great to have this in the README.md file

I believe this is ready. All changes from my side have been pushed.

Thank you @kevincobain2000! I refactored a bit the script.

Thanks!


Have PackageVariant set readiness gate on PackageRevisions

Implements #615

PackageVariant controller now uses a readiness gate to allow a PackageRevision to complete its mutation pipeline before it is allowed to be proposed/approved
refactors conversion of Kptfiles to YAML since readiness condition information is stored in the package Kptfile

unifies all cases to the same kyaml/yaml-based method (KptFile.ToYamlString() and ToYamlString(*fn.KubeObject))
this ensures consistency in the Kptfile YAML (indentation, field order etc.)
and reduces the chances of Git conflicts when setting and updating readiness conditions

adds more info to error message in case of Git conflict when applying a patch

/test presubmit-nephio-go-test

/test presubmit-nephio-go-test

/test presubmit-nephio-go-test

/test presubmit-nephio-go-test

/test presubmit-nephio-go-test

/test presubmit-nephio-go-test

/test presubmit-nephio-go-test


Fixed Watch and Code description.

Should be "Watch and Code", not "Watch N Code".

@gordonmzhu Thanks for the correction 😊


[devops:bot] update deps in nix/sources.json (from: rhino-core@39330eb

update nix refernces for rhino-core and|or nixpkgs to match https://github.com/rhinofi/rhino-core/blob/39330eb5d7c6025a4a309e5f8b2be5c001cb267f/nix/sources.json (branch: dev)

superseded by #180


copy CPython implemention

This change copies the asyncio implementation from CPython, trims it down, and adapts it for CircuitPython. The benefit of this approach is that we get a fully-featured, well-tested implementation that we don't need to maintain ourselves. The disadvantage is the large code size it has for microcontrollers. I see this change just as a starting point from where we can make code size optimizations as needed going forward.
The following submodules of asyncio are at least partially imported: events (aka event loops), futures, tasks, locks, queues, runners, taskgroups, timeouts.
The following submodules/features are not currently imported: sockets, protocols, transports, threads, subprocess and anything OS-specific, contextvars.
BaseExceptionGroup and ExceptionGroup are not currently implemented in the version of MicroPython we are using, so I created placeholder implementations of them in order to support task groups.
Anything socket related has been removed from the imported code. I plan to support an async socket API in the future, but in a way that doesn't couple the socket implementation with the event loop implementation.
The streams submodule is not imported as-is because it depends on the machinery of transports and protocols, which I didn't import. Instead I have started a new stream submodule that contains the same StreamReader and StreamWriter interfaces as CPython, but has a lighter weight implementation for CircuitPython based on the MicroPython stream protocol.
The current compiled code size of the library is 44 kB of mpy files. (It is larger than my previous values because I've included more submodules now.) Here is a complete list of submodule sizes.
44681   total
9387    tasks.mpy
7440    base_events.mpy
4679    locks.mpy
3571    futures.mpy
2597    heapq.mpy
2441    taskgroups.mpy
2313    events.mpy
2207    queues.mpy
1764    circuitpython_streams.mpy
1723    runners.mpy
1375    timeouts.mpy
1081    base_tasks.mpy
899     coroutines.mpy
843     base_futures.mpy
748     exceptions.mpy
569     format_helpers.mpy
376     __init__.mpy
242     mixins.mpy
220     circuitpython_events.mpy
206     log.mpy

In addition to importing the code, I also imported the corresponding tests from CPython. The tests run on a desktop not a microcontroller. Run tests from the Adafruit_CircuitPython_asyncio directory with:
python3 -m test.test_asyncio

I've tried to import the code in such a way to make diffing with the CPython original easy in the future so that we can take updates. I did this by trying to only delete blocks, or make very localized changes. The code was copied from the 3.11 branch of CPython here. If you want to review this code, I recommend cloning both CPython and my repos, and using your favorite visual diff tool to see the differences.

The CI job failures are from the imported code not conforming with the CP lint rules. We would have to disable lint in order to preserve taking diffs from upstream.

I am getting back to looking at this now that 8.1.0 is out (see also https://github.com/adafruit/circuitpython/pull/7711#issuecomment-1570368908).
My only real worry about this is the size of the library, which is about 45kB. Some of large sensor libraries are in the 20kB region, and, say, portal_base + pyportal, begin to approach this size. By comparison, the current asyncio library .mpy's are about 9.4kB.
I tried some simple trimming: __all__ and __slots aren't implemented, so I just removed those things. I also removed some fancier repr code, and some but not all of the debugging. I only reduced things about 10%, though. Another idea is whether the traceback printing actually works properly, given our limited traceback functionality.
Do you think that there could be further trimming of functionality?

If one considers only "modern" use of asyncio in CPython, is there functionality here that is not needed and could be removed? I.e., is there historical baggage that could be removed?
The MicroPython asyncio (and our current library) does not provide Awaitable or Future, only Task. Is that viable for this, or do you need Future for the new _asyncio in the core?
Could some things (e.g. locks.py) be split up so lazy imports could be used, so that unused functionality is not in RAM?
Another idea is to freeze this library when possible, to avoid using up RAM.

If copying code from CPython, please be aware of the different licensing requirements (compared to MIT).


Add cert-rotation e2e test

This PR adds an e2e test for the Ironic certificate rotation feature.
The Ironic certificate rotation feature introduces the use of cert-manger to generate new TLS certificates and renew these certificates when they are expired. The current integration test can ensure if ironic can adopt the new certificates generated by cert-manager, so this PR aims to test the if certificates can be renewed, and ironic can continue to function well with the new certificates.
Test scenario:

Manually trigger the certificate renewal by deleting the secrets that contain Ironic certificates.
Check if ironic contains which use the certificates can restart. After restarting, they will adopt the new certificates.
Check if the containers are in the running state.

/test-integration
/test-e2e-v1a5
/test-e2e-centos-v1a5

/test-v1a5-centos-e2e

/cc @fmuyassarov
/assign @kashifest

/test-v1a5-e2e

/test-v1a5-e2e

/test-v1a5-e2e
/test-v1a5-centos-e2e

/test-v1a5-e2e

/test-integration

/test-integration

/test-v1a5-e2e

/test-v1a5-e2e

/test-v1a5-e2e

/test-v1a5-e2e

/test-v1a5-e2e
/test-v1a5-centos-e2e
/test-integration

/test-v1a5-e2e
/test-v1a5-centos-e2e
/test-integration


Integrate temp tagging with document store

The downloader now ensures that stuff being downloaded is not going to be deleted, by creating a temp tag at the beginning of the download and keeping it alive during the download. So the result of a successful download is now a TempTag.
What is missing to make things totally safe is to ensure that hashes are in the document store before the temp tag is deleted.
Maybe this is already the case because the document gets synced before the linked content. Just need to make sure this is safe.
CC @Frando

Yea, the downloads are only triggered when the actor receives insert events from the replica. So, only after entries are inserted. So this should be done already I think.