Upgrade guide from vbuild 5.2.0 to 6.x

Need upgrade guide to upgrade existing project from vbuild 5.2.0 to 6.x. Tried just updating vbuild in package.json to ^6.1.1 but error regarding missing index.js. Looked at the vue.config.js of 6.x and noticed that it is quite different to that of 5.2.0.
This is my 5.2.0 vue.config.js:
const webpack = require('webpack')

module.exports = {
    title: 'test-proj',
    template: './src/index.html',
    resolve: true,
    mergeConfig: {
        plugins: [
            new webpack.ProvidePlugin({
                jQuery: "jquery",
                $: "jquery",
                "window.jQuery": "jquery",
                "window.$": "jquery"
            })
        ],
        performance: { hints: false }
    },
    vendor: ['jquery', 'vue', 'vue-router', 'vuex', 'vuex-router-sync', 'vee-validate']
}

title and template are merged into html option, and this option accept all options in html-webpack-plugin
resolve is gone, you can manually modify webpack config
mergeConfig changed to webpack
vendor keeps the same

const path = require('path')

module.exports = (options, req) => {
  const webpack = req('webpack')

  return {
    html: {
      title: 'test-proj',
      template: './src/index.html',
    },
    webpack: {
        // this is the `resolve: true` in vbuild 6
        resolve: [path.resolve('src')],
        plugins: [
          new webpack.ProvidePlugin({
            jQuery: "jquery",
            $: "jquery",
            "window.jQuery": "jquery",
            "window.$": "jquery"
          })
        ]
    },
    vendor: ['jquery', 'vue', 'vue-router', 'vuex', 'vuex-router-sync', 'vee-validate']
  }
}

And how to set different config for production and development environment ?

You can manually set this by using if (options.dev)

something like:
const common = {}
const prodConfig = {}
const devConfig = {}

return Object.assign({}, common, options.dev ? devConfig : prodConfig)

@throrin19 this feature is back in v6.6.0 https://github.com/egoist/vbuild#shorthand


Hardcoded Account ID

The account ID should not be hardcoded.
https://github.com/newrelic/nr1-slo-r/blob/bdefd359f371cba8f6a9e826b98f344b4b1ca636/nerdlets/shared/services/alert-policies.js#L14

fix implemented - new version generation required ASAP - https://github.com/newrelic/nr1-slo-r/pull/149

fix implemented - new version generation required ASAP - https://github.com/newrelic/nr1-slo-r/pull/149

Fixed in 1.7.6

Fixed in 1.7.6


Enhancement Request: Support for self.get_blockname

For user error messages (i.e. which block failed) and other uses

How would you use this? Could you give me an example usage?

Sure.  One of the use cases is that the compute blocks go one by one and if there is a non-standard error that is not captured in logging for the app itself, streamlit, or another library the user never sees see what block was executing.  Once the execute button is run, everything just kind of runs in the background...
Here's one example.
At the top of the page, initiate a streamlit placeholder with st.empty().  This is overly simplified but should get the point across... The user gets a streamlit info message at the top of the page for every block when it is executing.
``
execution_block_placeholder = st.empty()
begin compute engine functions
def my_compute_engine(self):
# compute logic for input_block_1
block_name = self.get_blockname(self) # <--- need to think through this, we don't know the name of the block
block_execution_message = f('Now executing block {block_name}')
execution_execution_block_placeholder.info(block_execution_message)
we could also have warning, error and other messages here and either re-use execution_block_message OR add a new placeholder e.g. warning_block_placeholder = st.empty
input_block_1 = Block(name='Input Block 1')
input_block_1 .add_input(name='Save Orgs and Networks to Local Cache')
input_block_1 .add_compute(my_compute_engine)
``

@krish-adi - I don't know if this should be filed under a new enhancement req or this one but it would be nice to also get the last block or have an array where we could say block X of Y blocks.  Case is to notify the user of which block is currently running and to let them know when all blocks have run successfully (or not).  Some blocks may take some time to run with large API datasets. If something goes wrong, need to know where and notify the user in addition to internally with logger.

In v 0.7.0, now each block can store a private state. This state can store anything you want to access after execution. All the errors during the compute function execution of the block are stored in the state. When a block fails, only the blocks dependent on it are skipped, and the rest of the schema is executed.
This rather is the better way of keeping track of the errors and the status when compared to st.empty() or session_state. I didn't want this to rely too much on streamlit's workflow. For the st.empty() the update did not accur as the placeholder object is bounced around from one function to the other.


Snapshot temporary tables are created in other schemas

Current Behavior
When snapshot is running, the temp table is created in the default schema, although a custom schema is specified in the configuration.
Also, if the script worked with an error, then intermediate tables remain in the layer.
Expected Behavior
When you specify a target schema, all objects are created in the specified schema.
If a snapshot execution error occurs, delete all temp tables.
Steps To Reproduce
{% snapshot snap_stg_fct_pair_channel_sell_buy_90d %}
{{
    config(
      target_schema='snapshots',
      tags = ["overflow","overflow_2_1", "overflow_full"],
      unique_key="token_a_id::String||'-'||token_b_id::String||'-'||dttm::String",
      strategy='check',
      check_cols=['row_hash'],
      invalidate_hard_deletes=True,
      engine = "MergeTree()",
      order_by ='dbt_scd_id',
      pre_hook = [
        "set max_bytes_before_external_group_by=1000000000,
        max_bytes_before_external_sort=500000000,
        join_algorithm = 'grace_hash',
        join_on_disk_max_files_to_merge = 2,
        partial_merge_join_optimizations = 1,
        partial_merge_join_rows_in_right_blocks = 1000,
        max_bytes_in_join=1000000000"
      ],
    )
}}

select
  *
from
  {{ ref('stg_fct_pair_channel_sell_buy_90d') }}
where
  toYYYYMMDD(dttm) in (
    {{ get_incremental_period(
      env_var('DBT_ENV_CUSTOM_ENV_INCREMENTAL_PERIOD')| int,
      env_var('DBT_ENV_CUSTOM_ENV_INCREMENTAL_SHIFT')| int,
      "day"
    ) }}
  )

{% endsnapshot %}
log:
I see in log that insert in not target_shema from config
insert into snap_dim_token__dbt_tmp (....)
with snapshot_query as (
Relevant log output
No response
Environment

OS:Ubuntu 20.04
Python: Python 3.8.10
dbt:

Core:

installed: 1.4.5
latest:    1.5.0 - Update available!

Your version of dbt-core is out of date!
You can find instructions for upgrading here:
https://docs.getdbt.com/docs/installation
Plugins:

clickhouse: 1.4.1 - Up to date!

i create issue in dbt-core https://github.com/dbt-labs/dbt-core/issues/7493
But they say that may by problem in clickhouse materialization.

PR merged and released in v1.4.2


librespot built with lewton support crashes after first song

 Issue by awiouy
Tuesday Sep 05, 2017 at 19:35 GMT
Originally opened as https://github.com/plietar/librespot/issues/248

librespot built with lewton support crashes after first song with the following messages:
thread '<unnamed>' panicked at ' error: OggError(NoCapturePatternFound)', /checkout/src/libcore/result.rs:860:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "SendError(..)"', /checkout/src/libcore/result.rs:860:4

Running with RUST_BACKTRACE=1 does not provide further information:
thread '<unnamed>' panicked at 'Vorbis error: OggError(NoCapturePatternFound)', /checkout/src/libcore/result.rs:860:4
stack backtrace:
   0: <unknown>

The above messages were issued while using ALSA backend.
librespot was built with Rust 1.20.0:
$CARGO_BUILD --no-default-features --features "alsa-backend pulseaudio-backend with-lewton"

Here's a full backtrace (run with export RUST_BACKTRACE=full):
thread '<unnamed>' panicked at 'Vorbis error: OggError(NoCapturePatternFound)', src/libcore/result.rs:906:4
stack backtrace:
   0:        0x106d62b73 - std::sys::imp::backtrace::tracing::imp::unwind_backtrace::h3421d74c3554d1d6
                               at src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1:        0x106d5f4cc - std::sys_common::backtrace::_print::h81d2920454574615
                               at src/libstd/sys_common/backtrace.rs:68
   2:        0x106d66b71 - std::panicking::default_hook::{{closure}}::h046b82ccd70886b8
                               at src/libstd/sys_common/backtrace.rs:57
                               at src/libstd/panicking.rs:381
   3:        0x106d66876 - std::panicking::default_hook::h00ff9adc172e30f9
                               at src/libstd/panicking.rs:397
   4:        0x106d670b5 - std::panicking::begin_panic::hf76e4af07ba1dfa4
                               at src/libstd/panicking.rs:577
   5:        0x106d66f4e - std::panicking::begin_panic::hf76e4af07ba1dfa4
                               at src/libstd/panicking.rs:538
   6:        0x106d66e22 - std::panicking::try::do_call::h3cbb81c41447d36b
                               at src/libstd/panicking.rs:522
   7:        0x106d66d82 - std::panicking::try::do_call::h3cbb81c41447d36b
                               at src/libstd/panicking.rs:498
   8:        0x106d9f2f3 - <core::ops::range::Range<Idx> as core::fmt::Debug>::fmt::h7e1bfae6b8dbdbf2
                               at src/libcore/panicking.rs:71
   9:        0x1062b4bdd - core::result::unwrap_failed::hbe65d52cf173ad35
                               at /Users/travis/build/rust-lang/rust/src/libcore/macros.rs:23
  10:        0x1062af83d - <core::result::Result<T, E>>::expect::h70e37a3941dae0b2
                               at /Users/travis/build/rust-lang/rust/src/libcore/result.rs:799
  11:        0x1062a6ded - librespot::player::PlayerState::playing_to_paused::h71fcca76ec5f692b
                               at src/player.rs:230
  12:        0x1061674b2 - librespot::player::Player::new::{{closure}}::h65a1946639d1fb1e
                               at /Users/sashahilton/Documents/librespot/src/player.rs:66
  13:        0x10619bf81 - std::sys_common::backtrace::__rust_begin_short_backtrace::h0be8dc656e65d3c4
                               at /Users/travis/build/rust-lang/rust/src/libstd/sys_common/backtrace.rs:133
  14:        0x10615c868 - std::thread::Builder::spawn::{{closure}}::{{closure}}::hdc03551cfbaa331b
                               at /Users/travis/build/rust-lang/rust/src/libstd/thread/mod.rs:406
  15:        0x10619bc71 - <std::panic::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once::h4b6768b0a02dfca6
                               at /Users/travis/build/rust-lang/rust/src/libstd/panic.rs:300
  16:        0x106164bc1 - std::panicking::try::do_call::hcfa55e02944d1224
                               at /Users/travis/build/rust-lang/rust/src/libstd/panicking.rs:480
  17:        0x106d7252e - panic_unwind::dwarf::eh::read_encoded_pointer::h3e27ca5c61a4336a
                               at src/libpanic_unwind/lib.rs:101
  18:        0x106164a9c - std::panicking::try::h601e7203e01262ee
                               at /Users/travis/build/rust-lang/rust/src/libstd/panicking.rs:459
  19:        0x10619c039 - std::panic::catch_unwind::h50b081a0b01fed10
                               at /Users/travis/build/rust-lang/rust/src/libstd/panic.rs:365
  20:        0x10615c6a2 - std::thread::Builder::spawn::{{closure}}::hda706cb4bc3ce64c
                               at /Users/travis/build/rust-lang/rust/src/libstd/thread/mod.rs:405
  21:        0x106163615 - <F as alloc::boxed::FnBox<A>>::call_box::hc5af7a547988440e
                               at /Users/travis/build/rust-lang/rust/src/liballoc/boxed.rs:762
  22:        0x106d65fbb - std::sys::imp::thread::Thread::new::thread_start::h8c15ceac1351d4e1
                               at /Users/travis/build/rust-lang/rust/src/liballoc/boxed.rs:772
                               at src/libstd/sys_common/thread.rs:24
                               at src/libstd/sys/unix/thread.rs:90
  23:     0x7fff9329c93a - _pthread_body
  24:     0x7fff9329c886 - _pthread_start
DEBUG:librespot::player: drop Player[0]

@awiouy The problem looks to be coming from https://github.com/librespot-org/librespot/blob/932441db45775a55c2e97d60cb9b6a018c4c821d/src/player.rs#L230

Could it be related to https://github.com/RustAudio/lewton/issues/18

Notes from gitter for easy reference.

Sasha Hilton @sashahilton00
Lewton is pure rust. Unfortunately it panics on the spotify files, most likely due to malformed headers, as we know that there is a section in the files which is bad/unknown. The solution in other players has been to skip it.
Here's how despotism handled it for example: https://sourceforge.net/p/despotify/code/HEAD/tree/java/trunk/src/main/java/se/despotify/client/player/SpotifyOggHeader.java#l106

XpLoDWilD @xplodwild Feb 01 14:46
just skip the 167 first bytes

Sasha Hilton @sashahilton00 Feb 01 14:49
Where'd you get 167 from?

XpLoDWilD @xplodwild Feb 01 14:53
that's the length of the header read by despotify
https://sourceforge.net/p/despotify/code/HEAD/tree/java/trunk/src/main/java/se/despotify/client/player/ChannelPlayer.java#l195
so the 167 first bytes are spotify's own header metadata, then the ogg/vorbis data begins


Moving Platforms (Elevators, etc)

Another core tool that might prove useful is allowing the player (playspace) to be moved along by an object, such as an elevator (The Gallery: Call of the Starseed), in-game vehicle where the player can embark/disembark at will (Hover Junkers) such as a boat or spaceship and so-on. Ideally there's an option toggle on/off inheriting the motion of the vehicle (boat sway) as well.
Thanks!

You can do it moving the [CameraRig] object with your own code script.

Boy aren't you a party pooper! Three for three Hyppo :smile:
Yes I'm aware all of what I've been suggesting to date can be done with my own code, including this whole toolkit. This here is merely another suggestion along with the multitudes that have already been suggested and implemented. Clearly people are going to want to have moving platforms of a sort in their games, so this seemed like a general/global enough of a suggestion to add to the kit.
Cheers man

I will actually have an elevator in my current project so if I come across something generic I will do a PR for it :-)

Sweet! Thank you 10Finger!

Now this is getting interesting because it's a natural part of my game. It's important to inheritate the speed from the platform.
Also try throw objects on a moving platform. This is also why a tracked object behaves more realistic than the other options in VRTK.

Were you guys simple deriving the translation/rotation of the moving platform and applying it to the [CameraRig] or were you successful in using physics?
I wondered if VRTK PlayerPresence would help in this? If not, would the player fall if stepping off the platform (as happens in the Climbing example when you step off any ledge)?

I think the simple way is like you said inherit  the translation/rotation and apply it to the base VR object.

I think the new Player Body Physics (Player Presence) PR should resolve this.
If you're naturally being pushed upwards by a floor and naturally falling to the nearest floor then this would work.
In fact the existing Player Presence script would probably resolve this.
However, this is only for elevator style platforms. Thinigs that move in forward/left+- directions wouldn't apply physics to you, they would slide around under you. One fix for this would be to child the camera rig to the platform (or just apply a follow script similar to the ferris wheel seats in Funfair)
The final thing is what do do when throwing something. The objects on throw need to have the play area velocity added to them as well, I believe someone did a PR for this, it's just waiting on the new Player Body Physics stuff to be implemented first.
https://github.com/thestonefox/VRTK/issues/603

Hi @thestonefox, is this something that is currently being worked on?
I am trying to get a drive-able car working. I also want the player to be able to interact with items that are inside the car. I ran into some issues though.
There is more info about this issue including a video on the Unity Forums thread that I posted:
VRTK-Interacting with Object while Moving
Maybe if you or someone could point me in the right direction or even just let me know if it's being worked on (or not possible to do yet), that would be great. Thank you!

@amandalinn what's happening to the collider when you're moving? If you look in the scene view is it becoming detached from the object? Or at least displaced.
Also check what's happening on the colliders on the controller. Are they displacing?

@thestonefox  Thank you for such a quick response!
I am not sure if I am able to see what is happening in the scene view since the car and game camera are moving (unless I am able to move the scene view with the car somehow?)
May I send the project folder over to you via email/message and you can quickly take a glance at it?

@amandaelinn if you want to zip up your project and put it on Dropbox or something then share the link then I'll try take a look (hopefully tomorrow)

@thestonefox  Yeah, I can definitely do that.
What email should I send the link to? Sorry I couldn't find your email or where to message you (I'm new to github haha)
Also, no rush. I really appreciate you taking the time to look at it and also working on VRTK :)

You can use thestonefox at gmail dot com

Parenting the camera rig to an object or using body physics with height adjust teleporter resolves this.


openness Controller deploy failed

Hi,
I am trying to deploy openness controller but failed at below step.
Can you please help on this? How to resolve?
TASK [kubernetes/cni/kubeovn/master : wait for running ovs-ovn & ovn-central pods] *******************************************************************
task path: /home/sysadmin/Murali/openness/openness-experience-kits/roles/kubernetes/cni/kubeovn/master/tasks/main.yml:149
./deploy_ne.sh controller last prints:

Pods status is:

Thanks & Regards,
Devika

Retry for pod creation failed.

@Jaladi-Devika  as far as i know, docker images may be downloading and timeout ansible playbook at end.
you can pull all required images using docker pull command and then try it.

New issue found can you please help me on this?

@ishanVshah, playbook at end issue resolved after we cleanup and try to re-deploy again. But now new issue came.. have posted the screenshots above. can you check if you can help on this?

Hi @Jaladi-Devika
I am not sure but the issue is the version of golang. minimal version is required to install intel-device-plugin in kubernetes cluster is 1.13
you can skip this step if you are no need to use "intel-device-plugin".
set below parameter to false inside group_vars directory.
k8s_device_plugins_enable: false

This ticket is open for a while now. Given that we have multiple releases since can you please check if this on the latest and open a ticket if it's still an issue.


Add tests for Workflow objects

Following #136, the main way to perform a reduction is to use the Workflow objects (e.g. Sans2dTutorialWorkflow).
We have kept the old unit tests that work on the Pipeline and providers, but we should also test the workflow wrappers.
This is probably not much more than ensuring all the default parameters are set up correctly?

Recent refactoring kind of did this implicitly --- most test now create pipelines using the pre-defined workflows.


Investigate disk space

Codecov Report
Patch coverage: 27.55% and project coverage change: -0.17 :warning:

Comparison is base (6540041) 62.95% compared to head (a27e92a) 62.79%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1943      +/-   ##
==========================================
- Coverage   62.95%   62.79%   -0.17%
==========================================
  Files         137      137
  Lines       26524    26610      +86
==========================================
+ Hits        16698    16709      +11
- Misses       9826     9901      +75

Impacted Files
Coverage Δ

client/cli/src/main.rs
0.23% <0.00%> (-0.03%)
:arrow_down:

client/client-lib/src/utils.rs
45.16% <0.00%> (-2.30%)
:arrow_down:

fedimint-core/src/api.rs
79.29% <0.00%> (-2.02%)
:arrow_down:

fedimint-dbtool/src/dump.rs
0.00% <0.00%> (ø)

fedimint-server/src/consensus/debug.rs
96.55% <0.00%> (ø)

fedimintd/src/fedimintd.rs
0.00% <0.00%> (ø)

fedimint-core/src/module/mod.rs
74.81% <7.69%> (-3.54%)
:arrow_down:

fedimint-server/src/consensus/mod.rs
77.53% <46.66%> (-0.56%)
:arrow_down:

fedimint-server/src/lib.rs
73.51% <71.42%> (ø)

fedimint-server/src/net/api.rs
84.86% <90.90%> (+0.32%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[type: refactor] refactor shenyu filter logic and method name

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3322 (627f1dc) into master (2eef279) will increase coverage by 0.06%.
The diff coverage is 60.00%.

@@             Coverage Diff              @@
##             master    #3322      +/-   ##
============================================
+ Coverage     64.27%   64.34%   +0.06%
- Complexity     5169     5173       +4
============================================
  Files           743      743
  Lines         20972    20971       -1
  Branches       1920     1918       -2
============================================
+ Hits          13479    13493      +14
+ Misses         6210     6199      -11
+ Partials       1283     1279       -4

Impacted Files
Coverage Δ

...ava/org/apache/shenyu/web/filter/HealthFilter.java
0.00% <0.00%> (ø)

...pache/shenyu/web/filter/LocalDispatcherFilter.java
0.00% <0.00%> (ø)

...rg/apache/shenyu/web/filter/AbstractWebFilter.java
100.00% <100.00%> (ø)

...va/org/apache/shenyu/web/filter/ExcludeFilter.java
100.00% <100.00%> (ø)

...a/org/apache/shenyu/web/filter/FallbackFilter.java
100.00% <100.00%> (ø)

...henyu/admin/service/impl/UpstreamCheckService.java
70.14% <0.00%> (-1.50%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
58.51% <0.00%> (-1.07%)
:arrow_down:

...ng/rocketmq/rocketmq/RocketMQLogCollectClient.java
76.00% <0.00%> (+4.00%)
:arrow_up:

.../logging/rocketmq/utils/LogCollectConfigUtils.java
71.11% <0.00%> (+4.44%)
:arrow_up:

...che/shenyu/sync/data/http/HttpSyncDataService.java
83.73% <0.00%> (+6.50%)
:arrow_up:

... and 2 more

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 2eef279...627f1dc. Read the comment docs.


Add PR Function Tests

Signed-off-by: Zachary Kayyali<EMAIL_ADDRESS>open-cluster-management/backlog#12973

/hold


Determine if Router is active

Using preact-router I was able to detect if the Router is active by checking:
          <Router
            onChange={(event) => {
              // open/close sidebar
              setRightSidebarCollapsed(event.active.length === 0);
            }}
          >

Is it possible to mimic this behavior or somehow to detect if the Router is active?

What do you mean by "is active"?

The Router here has a onRouteChange prop that may suffice?

How I did it using preact-router is shown above.

You don't really show anything, just that you use the onChange callback. Without more code, I can't give much actionable advice.
However, I probably wouldn't do this with onChange or onRouteChange, instead using path params in the route component itself.
function MyRoute() {
    const { path } = useLocation();
    let showSidebar = false;

    if (path == someCondition) showSidebar = true;

    return (
        <>
            <Sidebar />
            <RouteContent />
        </>
    );
}

I'm sorry. I thought this would be sufficient as it shows that the onChange was passed an object compared to the onRouteChange which is passed the url. Using this I would have to match all the pathes in the Router against the url to check if any of the components is to be mounted. So I'd do the same thing the Router is doing again.
The full code looks like:

const AdminRouter = ({ url }) => {
  const [rightSidebarCollapsed, setRightSidebarCollapsed] = useState(true);

  return (
    <>

        <Navbar.Brand
          className={clsx(
            style.toggleRight,
            rightSidebarCollapsed ? style.grey : style.red,
          )}
          onClick={() => route(determineSidebarCloseRoute(url));}
        >
          »
        </Navbar.Brand>

        <Router>
          <Admin path="/admin" key={url} />
          <Users path="/admin/users/:p*" key={url} />
          <Changelog path="/admin/changelog/:p*" key={url} />
          <Organizations path="/admin/organizations/:p*" key={url} />
        </Router>

          <Router
            onChange={(event) => {
              // open/close right sidebar
              setRightSidebarCollapsed(event.active.length === 0);
            }}
          >
            <UserCreateAndEdit path="/admin/users/create" key={url} />
            <UserCreateAndEdit
              isEdit
              path="/admin/users/edit/:userId"
              key={url}
            />
            <CreateChangelog
              path="/admin/changelog/create/:system?"
              key={url}
            />
            <EditChangelog
              path="/admin/changelog/edit/:changelogId"
              key={url}
            />
            <CreateOrganization path="/admin/organizations/create" key={url} />
            <EditOrganization
              path="/admin/organizations/edit/:organizationId"
              key={url}
            />
          </Router>

    </>
  );
};

.rightSidebarCollapsed {
  width: 0;
  height: 0;
  overflow-y: auto;
  padding: 0;
}

Here it's just a few components but I use this in other context with >10 components.

Thanks, that helps.
We don't really provide a method for doing this, no. I'd still recommend passing setRightSidebarCollapsed as a prop/putting it in a context and having those router children set it to true, rather than trying to do it at the router level.

Ok. Thank you.


Improved/Corrected the Arabic translations of the validation messages in form_builder_validators.

The Arabic version of the validation error messages wasn't so great. It even had some grammatical/spelling mistakes. I fixed it and made improvements where possible.

Thanks @ProgrammerXDesigner for contributing and @WilliamCunhaCardoso for reviewing


engine: clearer wording about update fallback [ch4492]

currently we just say "falling back to next update method", which is pretty opaque esp. to someone who doesn't know how our updating works under the hood. In particular, it's not at all obvious that this message means "we can't run a live update, doing a full build instead".
this is the first step towards making this messaging better:

name the update that we failed to do (so the user knows this is about not being able to LIVE UPDATE e.g.)
HOPEFULLY being a little clearer about "fall back" means?

But my understanding (from talking to @hyu ) is that the words aren't really the problem, it's that they get lost in a sea of other words, and that we were looking for ways to visually distinguish them

Can one of you tell me the evidence behind this? My instinct is that the difficulty understanding LU fallbacks came from both the difficulty find the log lines AND the uninformative message once you found them. Before this change, it was a crap-shoot whether the fallback message even mentioned the words "live update", and i can't imagine many people were able to decipher it.

hmmmm...i wasn't really looking for a debate, was just curious what the current thinking was, happy to chat more about it in person

Thanks, @maiamcc — here's my own bits of minor feedback:

Can we consistently use "Live Update"? I see some variations but we just need to pick one.
I also think we should pretty up the file list. The absolute path is a lot of characters.
I do think the error message can be more concise. This is a high ratio of "ink to data":

Couldn't perform update via method: Live Update because:
       found file(s) not matching a LiveUpdate sync, so performing a full build
(Files: /blah/blahb/blah/blahb/blahb/blahb/blahb/blahb)

Falling back to next update method

I would prefer something like:
Live Update didn't work, so trying a Full Build. Reason:
    Found file(s) not matching a LiveUpdate sync:
    - /blah/blah

Also, @nicks — the words are actually still a problem since we don't specifically name "Live Update" fallbacks. My prior PR does some work to visually distinguish all fallbacks, but that was not specific to Live Update. @maiamcc's work here also sets us up to further distinguish Live Update with, say, an icon.

Yes, I for sure want to find a way to trim down the error text!

Live Update didn't work, so trying a Full Build...

We can't do this at the moment because of this -- though I'm inclined to fix this in a later PR.
Give me a sec to go trim copy on the error message itself, though.

@nicks @hyu ready for re-review.
FYI re: "[Image Build and] Docker Compose Deploy" -- going to leave that for now and fix it in a PR I'm making right now, where I calculate the human-readable name of the update method from the specs we're passing.

Yes, I for sure want to find a way to trim down the error text!

Live Update didn't work, so trying a Full Build...

We can't specify what build is coming next at the moment because of this -- though I'm inclined to fix this in a later PR.
Give me a sec to go trim copy on the error message itself, though.

I'm a doofus, the only thing we fall back to from a Live Update is a full build -- I was trying to make this code too generic, but the only fallback that really needs messaging around is Live Update ->> full build, I'll just special-case that.

...okay NOW it's ready for review. This should be a lot simpler.


Ability to run backend separately

For now its up to 2 days while I'm struggling with postgres docker container and it would've been great if there was any guidelines/ ready compose file to launch them locally and debug that locally.

@tiangolo I appreciate what you did for us all but please, mention somewhere that default values you've put int /backend/app/alembic/env.py
def get_url():
    user = os.getenv("POSTGRES_USER", "postgres")
    password = os.getenv("POSTGRES_PASSWORD", "")
    server = os.getenv("POSTGRES_SERVER", "db")
    db = os.getenv("POSTGRES_DB", "app")
    return f"postgresql://{user}:{password}@{server}/{db}"

Do you mean run the DB locally and FastAPI backend locally in a terminal?  for env.py include your .env file and then use python-dotenv:
`from future import with_statement
import os
from alembic import context
from dotenv import load_dotenv
from sqlalchemy import engine_from_config, pool
from logging.config import fileConfig
load_dotenv()
`
Same concept for the backend API which his referenced here.

No, I meant to startup backend project locally via docker compose

that's my yml file
# Use postgres/example user/password credentials
version: '3.1'
services:

  backend:
    image: backend
    ports:
    - 10088:10088

  db:
    image: postgres
    restart: always
    environment:
      POSTGRES_SERVER: server
      POSTGRES_USER: user
      POSTGRES_PASSWORD: pwd
      POSTGRES_DB: db
    ports:
      - 5432:5432

  adminer:
    image: adminer
    restart: always
    ports:
      - 8080:8080

and that what I'm passing to settings
kwargs = {
    'SERVER_NAME': '',
    'SERVER_HOST': 'http://<IP_ADDRESS>:10088',

    'PROJECT_NAME': 'CookieBack',

    'SENTRY_DSN': '',
...
    'POSTGRES_SERVER': '<IP_ADDRESS>:5432',
    'POSTGRES_USER': 'user',
    'POSTGRES_PASSWORD': 'pwd',
    'POSTGRES_DB': 'db',
    'SQLALCHEMY_DATABASE_URI': 'postgresql://user:pwd@<IP_ADDRESS>:5432/db',

    'FIRST_SUPERUSER'<EMAIL_ADDRESS>    'FIRST_SUPERUSER_PASSWORD': 'example'}

settings = Settings(**kwargs)

by

docker-compose -f stack.yml up --force-recreate --build

Still I'm getting this:
backend_1  | WARNING:__main__:Finished call to '__main__.init' after 17.066(s), this was the 18th time calling it.
backend_1  | INFO:__main__:Starting call to '__main__.init', this is the 19th time calling it.
backend_1  | ERROR:__main__:(psycopg2.OperationalError) could not connect to server: Connection refused
backend_1  |    Is the server running on host "<IP_ADDRESS>" and accepting
backend_1  |    TCP/IP connections on port 5432?

How I've could fix that?

It seems something wrong with my yml setting, because when I'm trying to access postgre container this what I get:
(pythonProject) ➜  backend docker exec -it 8b3340b4f924 bash
root@8b3340b4f924:/app# curl -v -sL -I localhost:5432/ping
* Expire in 0 ms for 6 (transfer 0x56521c6abf50)
* Expire in 1 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 1 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 1 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 1 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 1 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
* Expire in 0 ms for 1 (transfer 0x56521c6abf50)
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Expire in 149999 ms for 3 (transfer 0x56521c6abf50)
* Expire in 200 ms for 4 (transfer 0x56521c6abf50)
* connect to <IP_ADDRESS> port 5432 failed: Connection refused
*   Trying ::1...
* TCP_NODELAY set
* Expire in 149999 ms for 3 (transfer 0x56521c6abf50)
* Immediate connect fail for ::1: Cannot assign requested address
*   Trying ::1...
* TCP_NODELAY set
* Expire in 149999 ms for 3 (transfer 0x56521c6abf50)
* Immediate connect fail for ::1: Cannot assign requested address
* Failed to connect to localhost port 5432: Connection refused
* Closing connection 0

If you are running the DB in docker along with the API as you yaml file indicates, you need to refer to the DB by the docker name, not <IP_ADDRESS>.  You're passing it the localhost IP and the message shows nothing it talking on that IP & port.  Try changing
'POSTGRES_SERVER': '<IP_ADDRESS>:5432', to 'POSTGRES_SERVER=db:5432,

If you are running the DB in docker along with the API as you yaml file indicates, you need to refer to the DB by the docker name, not <IP_ADDRESS>. You're passing it the localhost IP and the message shows nothing it talking on that IP & port. Try changing
'POSTGRES_SERVER': '<IP_ADDRESS>:5432', to 'POSTGRES_SERVER=db:5432,

No, thats not what helped, but what really helped is to get postgre container IP by running:

docker inspect -f '{{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}' b154e0fe2bda
Taking that IP and then putting it to SQLALCHEMY_DATABASE_URI and to

def get_url():
    user = os.getenv("POSTGRES_USER", "example")
    password = os.getenv("POSTGRES_PASSWORD", "example")
    server = os.getenv("POSTGRES_SERVER", '<IP_ADDRESS>:5432')
    db = os.getenv("POSTGRES_DB", "example")
    return f"postgresql://{user}:{password}@{server}/{db}"

and when your IP changes?  This is why you refer to the service name.

Yes, I've got this, just needed to launch that at least once

I'll fork and put that mention to readme and then close this issue.

#258

#258


tflite_runtime standalone always results in "ModuleNotFoundError: No module named '_interpreter_wrapper'"

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

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): openSUSE Leap 15.1
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version: tflite_runtime 1.14.0
Python version: 3.6.5
Installed using virtualenv? pip? conda?: pip3 (followed the instructions from https://www.tensorflow.org/lite/guide/python)
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0
GPU model and memory: NVIDIA 2080ti 11GB

Describe the problem
When trying to install just the TensorFlow Lite interpreter, I'm able to install everything with no errors and I can even import tflite_runtime, but when I put in "from tflite_runtime.interpreter import Interpreter", I always run into a ModuleNotFoundError
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
I also installed Miniconda, installed python 3.7, pip3 installed the correct tflite_runtime for python 3.7, and ran into the exact same ModuleNotFoundError.

I got it to work when I had full tensorflow also installed, but then I did a full reinstall of openSUSE and ran into the same error. I was hoping to only install the Lite Interpreter instead of a full TF.

@wmol4 ,
Can you please try pip install tf-nightly.
You may also need to change tf.contrib.lite.Interpreter to tf.lite.Interpreter
Kindly refer this link for more information.

@oanush
The hope is to only need to install "tflite_runtime" as per this page:
https://www.tensorflow.org/lite/guide/python#install_just_the_tensorflow_lite_interpreter
The issue is that I can only get the tflite_runtime library to work if I also have full tensorflow installed, which defeats the whole purpose.
It works when I am running both TF 2.0 and tflite_runtime 1.14. I run into the ModuleNotFoundError when I only install tflite_runtime 1.14. Any ideas how I can get tflite_runtime working without needing to install the full tensorflow as well?

Is anyone else getting this issue or is it just me??

I'm trying to deploy AWS Lambda package containing tflite_runtime and getting the same error. Locally it is working correctly.

Interesting. Full disclosure, I was trying it on Ubuntu and OpenSUSE 15.1 via Windows Subsystem for Linux. But after reading @Helmars message, I decided to try it on a Fedora laptop I have laying around, and it was able to install just fine.

It seems there is a workaround to copy _interpreter_wrapper.so file from tflite_runtime one directory out (to site-packages or current directory). This didn't help for AWS Lambda though, because Amazon Linux uses older version of glibc. I ended up creating minimal C program instead of using TF Lite Python package.

Any updates on this? Running into this issue with a RPi3.

Same problem in AWS Lambda. Any news about this?

Hi, the interpreter_wrapper is migrated to pybind11 from SWIG, and the cause of the error might be different. Can you give more details &
steps to reproduce?

Facing the same problem using the wheel on Android. It tries to lazy-load the full TensorFlow module, which surely fails. I need to bundle only the tflite_runtime... The import is on line 30 in interpreter.py

Hi hematogender, as I commented above the backend for _interpreter_wrapper has changed, and it might be a different bug. Can you file a separate bug with steps to reproduce?
Seems you're trying to use python over Android rather than using it directly. If importing .aar works it would be great, but seems not so far :(
Can you try with arm64 and see if the same error happens? Also, can you share why cross-compile is not an option? You can crossbuild whl from your machine, following https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/tools/pip_package

Same Error for AWS lambda
Response:
{
  "errorMessage": "Unable to import module 'lambda_function': No module named '_interpreter_wrapper'",
  "errorType": "Runtime.ImportModuleError"
}

What i did:
## create docker container from python image ##
docker pull python:3.7-slim-buster
docker run -it python:3.7-slim-buster bash
cd home

## zip layer ##
layer=tensorflowLite2
mkdir -p $layer/python/lib/python3.7/site-packages/
cd $layer/python/lib/python3.7/site-packages/
pip install -t . https://dl.google.com/coral/python/tflite_runtime-2.1.0.post1-cp37-cp37m-linux_x86_64.whl
rm -r *.dist-info
rm -r __pycache__/
cd ../../../../
apt-get update
apt-get install zip -y
zip -r9 ../$layer.zip .

Any progress?

This stackoverflow thread solved my problem with AWS Lambda:
https://stackoverflow.com/questions/59570081/how-to-use-tensorflow-lite-on-aws-lambda

I was running into the same issue ModuleNotFoundError: No module named '_interpreter_wrapper' on Ubuntu 16.04 x86-64. The problem was, as the complete error message writes ImportError: /lib/x86_64-linux-gnu/libm.so.6: version 'GLIBC_2.27' not found, that Ubuntu 16.04 is delivered with GLIBC 2.23. You can get your version by writing: ldd --version.
For me the solution was to use Ubuntu 18.04, which is delivered with GLIBC 2.27.
It might worth considering to write a note for Linux users at page https://www.tensorflow.org/lite/guide/python that Ubuntu 16.04 is not supported.
Hope this helps

It seems there is a workaround to copy _interpreter_wrapper.so file from tflite_runtime one directory out (to site-packages or current directory). This didn't help for AWS Lambda though, because Amazon Linux uses older version of glibc. I ended up creating minimal C program instead of using TF Lite Python package.

This worked for me. It required me to remove "tensorflow_wrap_interpreter_wrapper.pyo, tensorflow_wrap_interpreter_wrapper.py, _tensorflow_wrap_interpreter_wrapper.pyd out of tflite_runtime" from the tflite_runtime folder and put them one directory up.

It seems there is a workaround to copy _interpreter_wrapper.so file from tflite_runtime one directory out (to site-packages or current directory). This didn't help for AWS Lambda though, because Amazon Linux uses older version of glibc. I ended up creating minimal C program instead of using TF Lite Python package.

This worked for me. It required me to remove "tensorflow_wrap_interpreter_wrapper.pyo, tensorflow_wrap_interpreter_wrapper.py, _tensorflow_wrap_interpreter_wrapper.pyd out of tflite_runtime" from the tflite_runtime folder and put them one directory up.

How did you do this.. I am getting same error..
I created a virtual env and installed necessary packages (tflite and other).. then zipped the site-packages to a folder.. now in the zipped folder i placed the lambda_function.py.. when I am running this on lambda console.. getting the same error for tflite... other dependencies are imported smoothly..
Thanks in advance..

virtualenv->Lib->site packages, find TFlite and remove the above-mentioned
files up one directory. to be the same directory as the TFlite folder.
On Thu, Aug 13, 2020 at 5:30 AM Asheesh<EMAIL_ADDRESS>wrote:

It seems there is a workaround to copy _interpreter_wrapper.so file from
tflite_runtime one directory out (to site-packages or current directory).
This didn't help for AWS Lambda though, because Amazon Linux uses older
version of glibc. I ended up creating minimal C program instead of using TF
Lite Python package.
This worked for me. It required me to remove
"tensorflow_wrap_interpreter_wrapper.pyo,
tensorflow_wrap_interpreter_wrapper.py,
_tensorflow_wrap_interpreter_wrapper.pyd out of tflite_runtime" from the
tflite_runtime folder and put them one directory up.
How did you do this.. I am getting same error..
I created a virtual env and installed necessary packages (tflite and
other).. then zipped the site-packages to a folder.. now in the zipped
folder i placed the lambda_function.py.. when I am running this on lambda
console.. getting the same error for tflite... other dependencies are
imported smoothly..
Thanks in advance..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/33153#issuecomment-673371191,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALD7SX7VPTGCLBTOKSK3M2LSAOXC3ANCNFSM4I6WHYTA
.

Hi, i had this similar issue when running my application on docker. I was using python:3.6.8 at that time. And by changing the python image to 3.6.11 solves my problem. Something that i notice is that there is a difference in gcc version that is being used by them (6.3 and 8.3 as i recall).

It seems there is a workaround to copy _interpreter_wrapper.so file from tflite_runtime one directory out (to site-packages or current directory). This didn't help for AWS Lambda though, because Amazon Linux uses older version of glibc. I ended up creating minimal C program instead of using TF Lite Python package.

This worked for me. It required me to remove "tensorflow_wrap_interpreter_wrapper.pyo, tensorflow_wrap_interpreter_wrapper.py, _tensorflow_wrap_interpreter_wrapper.pyd out of tflite_runtime" from the tflite_runtime folder and put them one directory up.

How did you do this.. I am getting same error..
I created a virtual env and installed necessary packages (tflite and other).. then zipped the site-packages to a folder.. now in the zipped folder i placed the lambda_function.py.. when I am running this on lambda console.. getting the same error for tflite... other dependencies are imported smoothly..
Thanks in advance..

After copy files in tflite_runtime to the ../, there turn out to be:
{
"errorMessage": "Unable to import module 'handler': /lib64/libm.so.6: version `GLIBC_2.27' not found (required by /var/task/_interpreter_wrapper.so)",
"errorType": "Runtime.ImportModuleError"
}

Having same issue... Any one better solution for this ?

Have same error "Unable to import module 'handler': /lib64/libm.so.6: version `GLIBC_2.27' not found .." on AWS lambda. Building it locally on Ubunt18.04 works fine though.

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

This issue still exits with tf2.x
tflite runtime needs GLIBC_2.27 which ubuntu16.04 does not come shipped with (it comes with GLIBC_2.23)
Is there a hard requirement of tflite runtime interpreter for GLIB_2.27 ? Is there any workaround for ubuntu16.04, please let know.

@Lucky94 did you find any solution for this. I am in same situation


sends birthdate to customer.io as timestamp instead of formatted date

What's this PR do?

Customer.io wants the user's birthdate sent as a timestamp, not formatted date. This PR updates the logic to do that!

How should this be reviewed?
👀
Relevant Tickets
Fixes this Pivotal Card.
Checklist

[ ] Documentation added for changed endpoints.
[ ] Tests added for new features/bug fixes.
[ ] Post a message in #api if this includes something that causes a rebuild!

@katiecrane in the Pivotal Card, the third task says that we need to update the BackfillCustomerIo job but I don't think we need to since it sends through toCustomerIoPayload updated in this PR anyway?

@chloealee I think we want to update it to remove $query = User::where('cio_full_backfill', '!=', true);  because we want to make sure to re-send everyone


Full path generation

Let's add generation full path generator, which will generate path from message name to field name.
Problem: for example in one proto file can be several duplicated field names:
syntax = "proto3";

package casesa;

option go_package = "cases/a;a";

message Foo {
  string bar = 1;
  int32 baz = 2;
}

message Bar {
  string bar = 1;
}

After generation we will receive smth like this:
f := Foo{}
f.FieldMaskPaths().Bar() // will return "bar"
b := Bar{}
b.FieldMaskPaths().Bar() // will also return "bar"

It is a problem due to one proto file can be like namespace. So it can be useful to have absolute path of each field in namespace.
f := Foo{}
f.FullFieldMaskPaths().Bar() // will return "foo.bar"
b := Bar{}
b.FullFieldMaskPaths().Bar() // will return "bar.bar"

Hi @andrey308 ,
Full path is supported starting from the first known field name.
Notice that even in your example, foo and bar are not fields name but just the lower case name so the messages. for foo or bar to be part of the fieldmask path, they need to be the name of a field in message that contains them.
For example:
message Baz {
  Foo foo = 1;
}

baz := Baz{}
baz.FieldMaskPaths().Foo().Bar() // will return "foo.bar"

I'm closing this issue for now, but feel free to respond if you still disagree.


Shouldn't the last four commits be added to v2.0.0?

It looks like there are really important commits in master which should be in v2.0.0 See https://github.com/driftyco/ionicons/compare/v2.0.0...master for more

created v2.0.1: https://github.com/driftyco/ionicons/releases/tag/v2.0.1


Web - Emoji and send icon do not center align vertically just like the plus icon inside of the composer

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a chat
Click inside the composer
Press shift + enter to add a new line
Observe and compare the plus icon with the emoji and send icons

Expected Result:
All the icons, i.e the plus icon, the emoji icon and the send icon should be either center aligned vertically or should be aligned to the bottom
Actual Result:
Only the plus icon becomes center aligned vertically but the emoji and the send icons become bottom aligned
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.35-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/cc8789e5-63e3-436f-8b54-49080be5a425
https://github.com/Expensify/App/assets/93399543/6998058e-f0f0-4cc3-a236-9f595cc8f924
Expensify/Expensify Issue URL:
Issue reported by: Nathan Mulugeta
React JS development and Quality Assurance (GH @Nathan-Mulugeta)
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688221459620419
View all open jobs on GitHub

I think we can close this out, this is expected based on the composer height change we made.


[Fix] allow line breaks before method calls and before function arguments for two Data Visualization challenges

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #35368

I've tested locally. Works for me. The changes are similar to changes made in https://github.com/freeCodeCamp/freeCodeCamp/pull/35093

Hi @Ensive
@Ensive We are looking for a more robust solution to keep us from having to add \s* throughout the challenge tests.  The changes you made for the Access the JSON Data from an API challenge would cover a few cases, but would still fail valid JavaScript like the following:
console.log(
  json[2] [ 'codeNames'] [1]
);

OR
console.log(
  json[2]
  .codeNames [1]
);

I suggest we discuss possible approaches back on the original issue itself.  Once we come to an agreement on the best approach, then you can create the PR the fulfills that approach.  I am going to close this PR.  You can create a new one once we have ironed out the details.  Thank you for your understanding.


fix: fix bug in retrieving the caller in the set_config endpoint.

Problem:
When calling the set_config API, the following error is returned:
"ic0_msg_caller_size" cannot be executed in reply callback mode

The reason here is that the caller is fetched after making an async request, at which point it becomes unavailable.
Solution:
Store the caller in a variable before making the async request.

LGTM.


Bharath Kumar S added

BHARATH KUMAR

Hey can you do the commit again, there are some issues with the commit.


Python: added num_records parameter to text memory skill

Motivation and Context

The text_memory_skill in the core skills was setup to only return 1 record, while the underlying memories all have support for multiple, so I added a parameter to the recall function to use that.
Description

Added parameter, passed that on to the memory call.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

I already solved that!


add functionality to open files in other apps from Filebrowser

@zebulasampedro an initial look? I have not used the BroadcastService to open the file in the editor here

@zebulasampedro another look? I have added the provider in the editor.js file, where I register the Filebrowser actions


Doc for DebuggerTypeProxyAttribute and DebuggerDisplayAttribute could mention VisualStudio options that impact their behavior in the debugger

I would've found it helpful if the Doc for DebuggerTypeProxyAttribute and DebuggerDisplayAttribute would have linked or mentioned the Visual Studio Debugging option to "Show raw structure of objects in variables windows" (and any other settings that may impact how the debugger uses these attributes).
The  "Show raw structure of objects in variables windows"  option appears to partially disable the debugger from using these attributes.
I say partially because the attributes seem to still be applied when hovering the mouse over some of the variables in the source text document window regardless of this setting.
It would also be helpful to know if there was a setting to fully disable the use of these attributes in the debugger when working with code that may have poorly written customizations using these attributes.
In any case I think it's important for people thinking of using the Debugger customizing attributes in their code to know that the "Show raw structure of objects in variables windows"  option exists.

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

ID: 4ba74624-2169-35d8-f2be-ae51f9d367d8
Version Independent ID: ad2b7ddf-b5f6-b3f3-76e2-7974a3d94cbf
Content: Display custom type using DebuggerTypeProxy - Visual Studio
Content Source: docs/debugger/using-debuggertypeproxy-attribute.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Adding a note in this doc.


Upgrade libXfixes to version 6.0.1 for Mariner 3.0

Upgrade libXfixes to version 6.0.1 and corresponding changes to cgmanifest and signature.json
Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

This PR upgrades libXfixes to version 6.0.1.  README file has been renamed as README.md in this version.
Change Log

Upgrade libXfixes to version 6.0.1
No other packages impacted
No previous CVEs

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=501550&view=results

Thanks @sindhu-karri and @mfrw for review!


chore: bump default graalvm version

It seems that there are situations where the default still pops up,
which seems to cause some issues. For example today I saw someone
mention seeing this issue:
Exception in thread "main" coursier.jvm.JvmCache$JvmNotFoundInIndex: JVM graalvm:19.3 not found in index: No graalvm version matching '19.3' found
    at coursier.jvm.JvmCache.$anonfun$getIfInstalled$1(JvmCache.scala:35)
    at coursier.jvm.JvmCache.$anonfun$getIfInstalled$1$adapted(JvmCache.scala:34)
    at coursier.util.Task$.$anonfun$flatMap$extension$1(Task.scala:14)
    at coursier.util.Task$.$anonfun$flatMap$extension$1$adapted(Task.scala:14)
    at coursier.util.Task$.wrap(Task.scala:82)
    at coursier.util.Task$.$anonfun$flatMap$2(Task.scala:14)
    at scala.concurrent.Future.$anonfun$flatMap$1(Future.scala:307)
    at scala.concurrent.impl.Promise.$anonfun$transformWith$1(Promise.scala:41)
    at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:64)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
    at java.lang.Thread.run(Thread.java:833)
    at com.oracle.svm.core.thread.PlatformThreads.threadStartRoutine(PlatformThreads.java:704)
    at

I think it might be a good idea to default to a newer version that
aligns with the version being used in coursier.

Note that someone on Discord just hit on this as well with
coursier install bloop                                                                                10:27:28
https://repo1.maven.org/maven2/io/get-coursier/apps/maven-metadata.xml
  No new update since 2022-12-21 18:35:06
https://repo1.maven.org/maven2/org/scala-lang/scala-library/maven-metadata.xml
  No new update since 2022-10-08 23:59:56
https://repo1.maven.org/maven2/org/scala-lang/scala3-library_3/maven-metadata.xml
  100.0% [##########] 29.1 KiB (81.3 KiB / s)
https://repo1.maven.org/maven2/ch/epfl/scala/bloopgun_2.13/maven-metadata.xml
  No new update since 2022-12-22 22:00:47
https://repo1.maven.org/maven2/ch/epfl/scala/bloopgun_2.13/maven-metadata.xml
  No new update since 2022-12-22 22:00:47
https://repo1.maven.org/maven2/org/scala-lang/scala3-library_3/maven-metadata.xml
  No new update since 2023-01-04 09:54:46
https://repo1.maven.org/maven2/ch/epfl/scala/bloopgun_2.13/maven-metadata.xml
  No new update since 2022-12-22 22:00:47
Exception in thread "main" coursier.jvm.JvmCache$JvmNotFoundInIndex: JVM graalvm:19.3 not found in index: JVM graalvm not found
    at coursier.jvm.JvmCache.$anonfun$getIfInstalled$1(JvmCache.scala:35)
    at coursier.jvm.JvmCache.$anonfun$getIfInstalled$1$adapted(JvmCache.scala:34)
    at coursier.util.Task$.$anonfun$flatMap$extension$1(Task.scala:14)

So this morning I also saw #2650. I still think this is a good idea to bump, but I also don't think it will fix the linked issue. Even when trying to pass in --graalvm-default-version=22.1 you then get the error about 22.1 not being found (or so they had, I can't reproduce locally). However, looking at the app descriptor of Bloop I don't really get why coursier is even trying to download a graalvm version here? If it's a native image shouldn't it simply download the binary and call it a day?

I think that if it can't find it for an architecture it tries to build a native image form a jar. OR maybe it does always try to do that?

Related https://github.com/coursier/coursier/issues/1628

So this morning I also saw #2650. I still think this is a good idea to bump, but I also don't think it will fix the linked issue. Even when trying to pass in --graalvm-default-version=22.1 you then get the error about 22.1 not being found (or so they had, I can't reproduce locally). However, looking at the app descriptor of Bloop I don't really get why coursier is even trying to download a graalvm version here? If it's a native image shouldn't it simply download the binary and call it a day?

I think that if it can't find it for an architecture it tries to build a native image form a jar. OR maybe it does always try to do that?

But in this case with Bloop
https://github.com/coursier/apps/blob/f2a1d410a525e72f52b8a99f067cc7b807b70818/apps/resources/bloop.json#L6
latest should be 1.5.6, and the assets are there https://github.com/scalacenter/bloop/releases/tag/v1.5.6 for apple-darwin, which is what the user should have gotten.

latest should be 1.5.6, and the assets are there https://github.com/scalacenter/bloop/releases/tag/v1.5.6 for apple-darwin, which is what the user should have gotten.

Maybe the name is wrong or wrongly inferred by coursier?

latest should be 1.5.6, and the assets are there https://github.com/scalacenter/bloop/releases/tag/v1.5.6 for apple-darwin, which is what the user should have gotten.

Maybe the name is wrong or wrongly inferred by coursier?

Alternatively, the issue could be that courier can't find the bloop launcher for the aarch64 architecture and tries to build it locally, which requires downloading graalvm for aarch64. However, it cannot find this version of graalvm:19.3 and fails.

By the way, building native images from cs install should removed.
cs may not be able to properly generate a native launcher for each repository, because the native launcher may have custom options and cs does not have a mechanism to obtain this native options.

By the way, building native images from cs install should removed.

cs may not be able to properly generate a native launcher for each repository, because the native launcher may have custom options and cs does not have a mechanism to obtain this native options.

Totally agree. I think in these type of situations it'd be nice to just fallback to the jvm launcher.

Note that someone also had this same issue when trying to install Bleep. Look at the app descriptor in https://raw.githubusercontent.com/oyvindberg/bleep/master/coursier-channel.json arm64 is there and they were on a mac. Shouldn't that have pulled from there install of trying to build on its own?

Note that someone also had this same issue when trying to install Bleep. Look at the app descriptor in https://raw.githubusercontent.com/oyvindberg/bleep/master/coursier-channel.json arm64 is there and they were on a mac. Shouldn't that have pulled from there install of trying to build on its own? Or is that an issue in the descriptor that should really be aarch64?

If you look at the configuration of scala-cli, you will see that aarch64-apple-darwin is used and it works. Therefore, it might be a bug.
However, there is still a missing aarch64 launcher of bleep for Linux, which causes this error in Linux:
Exception in thread "main" coursier.jvm.JvmCache$JvmNotFoundInIndex: JVM graalvm:19.3 not found in index: JVM graalvm not found
    at coursier.jvm.JvmCache.$anonfun$getIfInstalled$1(JvmCache.scala:35)
    at coursier.jvm.JvmCache.$anonfun$getIfInstalled$1$adapted(JvmCache.scala:34)
    at coursier.util.Task$.$anonfun$flatMap$extension$1(Task.scala:14)

but for M1 it should work now, so I think, it's a bug with descriptor name.
cc: @Baccata

but for M1 it should work now, so I think, it's a bug with descriptor name.

In all frankness, I'm not sure it categorises as a bug in the descriptor : for starters coursier doesn't seem to document the expectation very well.  Moreover, if aarch64 and arm64 are the same, shouldn't coursier attempt to recover when one isn't found by trying the other one ? That'd save library maintainers from having to upload the same file twice under different names.

By the way, building native images from cs install should removed.

cs may not be able to properly generate a native launcher for each repository, because the native launcher may have custom options and cs does not have a mechanism to obtain this native options.

Totally agree. I think in these type of situations it'd be nice to just fallback to the jvm launcher.

In general, we can still merge this PR because it is okay, even though it doen't solve the problem it was intended to solve.
This https://github.com/coursier/coursier/pull/2652 should fix the problem with:
Exception in thread "main" coursier.jvm.JvmCache$JvmNotFoundInIndex: JVM graalvm:19.3 not found in index: JVM graalvm not found

but I'm not convinced that it is the proper way. cc: @tgodzik


Resolving assets based on requested platform

Potential fix for #499 .

FYI. This is not RFR

Updated: RFC


Limit of articles? (Memory issue)

Hi.
Looking at PHP logs we found this memory problem:

[20-Apr-2016 14:57:04 UTC] PHP Fatal error:  Allowed memory size of 94371840 bytes exhausted (tried to allocate 72 bytes) in /wp-includes/meta.php on line 838

Did you guys know about this issue? As I told you we have a lot of articles (790).
Could you help us?
Thanks.

Hi.
Currently we have about 790 articles and the plugin stopped to work for no reason.
We are receiving this error message: Failed. Could not perform action (entity_list).
Is there any limit of articles for the plugin?
Thanks.

@solerc, this will be fixed in the next plugin update. We've e-mailed with some steps you can take in the meantime to limit the amount of articles. Please let us know if that doesn't work.
Thank you,
Studio Mercury

@solerc, thank you for your patience. Last night we pushed an update to the plugin that brings lazy loading to the article list view. This solves the error you were experiencing. Please let us know how the new update works for you.
Thank you,
Studio Mercury


Tribute to Contributors in diffrent way

Is your feature request related to a problem? Please describe.
As of now, we are Thanking Contributors like this 👇🏻

I think we can do it in a more better and personalized way !
Describe the solution you'd like
We can shift this section at bottom of reading me the file and we can add everyone's card so If anyone wants to reach out to them they can visit their GitHub profile card will contain their GitHub avatar and name where the link will be added.
Describe alternatives you've considered
Or we can just add links to their profiles to the existing places this will also help but the card will look a more attractive and a small tribute to great people!!
Will like to add this feature

I think it would be nice to add not only @nickname, but also real name and surname (for those, who'd like to share, of course) of contributors :)

That section is thanks to authors; contributors are thanked elsewhere


added 5.0 error messages

added some missing error messages from BT5

@bitcloud thanks! Do you have a reference section of the core spec for the new error codes? Have you experienced these error messages?

You'll find them in the spec (e.g.): http://www.mouser.li/pdfdocs/bluetooth-Core-v50.pdf
I think I experienced one of the last two. But that was quite a while ago while I was debugging some Bluetooth stuff.

May I suggest to submit your patches to:
https://github.com/abandonware/noble


[nrf528xx] align flash driver to the new otPlatFlash* API

This PR fixes #4738.

Please wait a bit with merging, waiting for internal tests to be finished.

It's ok from my side.


Switch for --uninstallPrevious for upgrade

Description of the new feature / enhancement
The upgrade behavior can be set to uninstall the previous version of the application by default in the manifest. Usually, upgrade behavior is set to install unless uninstallPrevious is needed for a specific reason.
As a power user there are times when it would be nice to override the behavior and uninstall the previous package anyways. One example is when a user has a local build of wingetcreate, the application will show for upgrade but cannot be upgraded unless it is uninstalled and re-installed.
This would also allow for the bypassing/overriding of the Install Technology Differs error by forcing a clean install
Proposed technical implementation details
No response

@denelon - Experimental


documentation: scipy.spatial.KDTree vs. scipy.spatial.cKDTree

It seems to be not documented that scipy.spatial.KDTree accepts complex arrays, but scipy.spatial.cKDTree doesn't.
The docs seem to imply that they are equivalent.

Thanks @keithbriggs, I wasn't aware of that. There seem to be zero tests for complex dtypes, so I'm not sure if it works on purpose or by accident. And if it e.g. works with all values for p of the minkowski distance. If you could supply a self-contained example that can be used as a test, that would be useful.

Looking at the KDTree code, the minkowski_distance does work for complex. But I don't think any of the tree building a querying code will work correctly. There's nothing that suggests to me that it was written with complex in mind.
Fundamentally, a KDTree relies on an ordering of points along each dimension, but no such ordering exists for complex numbers. The only reason it accepts complex numbers at all is a quirk in NumPy which uses lexicographic comparison of the real and imaginary components.
Also, the helper class Rectangle discards the imaginary component:
https://github.com/scipy/scipy/blob/a49e96058264eab8b220da8ee7b64ab1c4a62a07/scipy/spatial/kdtree.py#L90
That means at least: query_ball_point, query_ball_tree, query_pairs, count_neighbours and sparse_distance_matrix are all completely broken.
For completeness, here's a counterexample for KDTree:
In [1]: import numpy as np
In [2]: from scipy.spatial import KDTree, minkowski_distance
In [3]: coords = np.random.rand(1000, 2).view(complex)
In [4]: t = KDTree(coords)
In [5]: point = np.array([0 + 1j])
In [6]: np.count_nonzero(minkowski_distance(point, c) < 1) # Brute force query
Out[6]: 802

In [7]: t.query_ball_point(point, r=1)
/home/pbell/git/scipy/build/testenv/lib/python3.7/site-packages/scipy/spatial/kdtree.py:90: ComplexWarning: Casting complex values to real discards the imaginary part
  self.maxes = np.maximum(maxes,mins).astype(float)
/home/pbell/git/scipy/build/testenv/lib/python3.7/site-packages/scipy/spatial/kdtree.py:91: ComplexWarning: Casting complex values to real discards the imaginary part
  self.mins = np.minimum(maxes,mins).astype(float)
Out[7]: []

That said, I can imagine a complex KDTree would be possible. It would essentially be a 2K dimensional tree which is split on imaginary and real components separately. Only the metric functions would need to be specifically aware of the complex numbers. Unless there's a very good reason to add this feature though, I'm inclined to just add an error to KDTree if any of the inputs are complex.

Ok, thanks for checking. I agree it should report an error if the results aren't going to be meaningful.

Yes, it should split on the real an complex parts separately, which means that all we need to do for this to work correctly is to view n x k complex array as n x 2k real data.


Revert H/2 codec back to nghttp2

Risk Level: Low (unlikely that someone depends on oghttp2 specific behavior, since the change was very recent).
Testing: Unit Tests
Docs Changes: N/A
Release Notes: Yes
Platform Specific Features: N/A
Fixes #32611 #32371

https://github.com/envoyproxy/envoy/pull/32751 merged, closing this


Scaling CFF while retaining hints?

I'm just wondering, that seems a pretty difficult task. Has anyone done that before with fonttools?

Humm.  Why?  Sounds straightforward to me.

Relative coordinates, hint info interspersed with coordinates. Dealing subroutines one way or another. It's definitely more work than simply applying a transform pen, which is what I would call straightforward :)

Either way, it's non-trivial enough that in case anyone ever does this, a snippet contribution would be welcome.

Any number of that subroutine numbers will be scaled uniformly.

Err. *Any number out than...

Not necessarily. The same shape at a different position may end up with different relative coordinates after scaling (with rounding to ints).

See. Now you are talking about translation, rounding, etc.  If you just scale and don't do anything else (no translation, no rounding), it should all work.

If you accurately define what you need to do, I can assess how feasible it is.  I've written a lot of custom T2CharString decoders to do things of this scale (pun intended!)

Well actually, it seems I probably won't need this right now.

I wrote scaleUpem module now, but it desubroutinizes.

It probably can be done without desubroutinizing...


Automate import/export

In all of my projects I have set connections that would be useful for entire team to use. I now export all of them and put on share, then they import them.
It would be great to simplify importing/exporting projects via command line. Here are some ideas of how this could work:
#export all projects with drivers
dbeaver --export --projects '*' --with-drivers -o ~/dbeaver.dbp

#export only selected projects
dbeaver --export --projects 'General, ProjectXYZ' -o ~/dbeaver.dbp

#export only projects starting with Project_ preffix and General.
dbeaver --export --projects 'Project_*, General' -o ~/dbeaver.dbp

#import all projects from dbp file, overwrite any existing projects
dbeaver --import --projects '*' --with-drivers -f ~/dbever.dbp  --overwrite

#import all projects from dbp file, use specific prefix for project names
dbeaver --import --projects '*' --with-drivers -f ~/dbever.dbp  --prefix 'share_' -overwrite

#import all projects from dbp file, map input names to given names
dbeaver --import --projects 'XYZ:MY_XYZ' --with-drivers -f ~/dbever.dbp

This would allow database manager to put this in a cron job and export to common share or git repository for others/new memebers. Others could use cron job or script to sync, and with given prefix to import into specific 'namespace' so that their own stuff doesn't get changed (so that they can use copy/paste or some other method to update their local projects).
Saved passwords would become a problem then if they are not to be shared. Currently connection fails if password is not given but dbeaver could prompt for it. This applies to scenario in which domain login (OS Auth) is not used.

This is similar to #1271 (additional command line processing).

Its not exactly similar, although it would be a nice addition.
I somehow missed Open Project from File System option. Now, I saved project to shared location and each developer can open it from there. Each developer now can add bookmarks, sql files and new connections. Do you consider there to be any problem with that approach if they work on different things for example ? So far I observed the following:

Working on same SQL files wont be a problem since dbeaver monitors for changes. Anything else that comes to mind perhaps in this workflow ?
To see bookmarks of others F5 is required in running projects.
Changing connection properties requires F5.

All looks like something I can work with. Perhaps automatic refreshing on entire project would be nice to have.

I'd propose to put your shared project under some version control (git, svn, etc). Otherwise eventually there will be many conflicts in SQL scripts and other artifacts.
You can install version control plugins thru Eclipse Marketplace in EE version or use external tools (like Tortoise) in CE version.

This feature (opening external project) is already there.
Switch to Projects view->Create new project (in toolbar)->Set location of external project.


Constrain variable to integer values

Is it possible to constrain a variable to integer values? My problem requires that a variable be discrete integers and not continuous. Thanks!

Yes, please have a look at https://pymoo.org/customization/discrete_problem.html

Thanks!


Spacing fixes made by pre-commit.

Wouldn’t a lot of these have been caught by EditorConfig? @ErinvanderVeen shouldn’t another EditorConfig file be dropped into tests folder?

Wouldn’t a lot of these have been caught by EditorConfig? @ErinvanderVeen shouldn’t another EditorConfig file be dropped into tests folder?

Yes! We could definitely commit two editorconfigs, if that would solve the issue.


Sensu init fails to restart

In the past month running sensu `0.12.6-2``
I've seen how sensu client, server and api after running for a long time they hang upon restart.
Also today after running for 1 week without incidences sensu-client just died. Don't see errors on logs or anywhere else.
Runnin on top of AWS ubuntu precise 12.04.

Has this still not been fixed?
All of this relates to systemd and Debian 7.

For some reason I don't understand is the $service variable in line 151 of the /etc/init.d/sensu-service file empty - but only in that line. That means the exec variable contains only the path where the binary resides in instead of the binary itself.
Setting a different variable to the binary path at beginning of the script solved the problem for me - I'm using Debian 8.
--- a	2015-03-12 22:01:22.000000000 +0100
+++ b	2015-03-12 22:00:42.000000000 +0100
@@ -23,6 +23,7 @@
 LOG_DIR=/var/log/sensu
 LOG_LEVEL=info
 PID_DIR=/var/run/sensu
+SERVICE_BIN=/opt/sensu/bin/$service
 USER=sensu
 SERVICE_MAX_WAIT=10

@@ -148,7 +149,7 @@

 cd /opt/sensu

-exec=/opt/sensu/bin/$service
+exec=$SERVICE_BIN

 pidfile=$PID_DIR/$service.pid
 logfile=$LOG_DIR/$service.log

Same systemd issue applies to Ubuntu 15.04+

The patch from @ydkn above solves the issue for me on Ubuntu 15.04, would be great if someone would apply it.

ping @portertech

Closing due to inactivity (mostly on our part), but also because I'm pretty sure this has been fixed. Please feel free to create a new issue if this matter is not resolved. #monitoringlove

I don't work at that conpany anymore so I can't really test. FWIW in my personal setup runing latest sensu it seems to work properly.

@rhoml great, thanks for the follow-up!

I've had the same bug on CentOS 7 in my case I didn't needed to change the daemon, but my hostname was missconfigured (not present on /etc/hosts) after I've done that, everything worked fine.

I am trying sensu in Ubuntu know, because I could not solve this problem in Debian. However, this error came back and it went away when I modified a json file in config.d
So, json files with mistakes trow this bug or whatever?

Hi @holyyoshi777,
Given the age of this issue and the lack of detail around the problem you are encountering, it's hard for me to understand the underlying issue.
If you are having trouble managing your Sensu processes after changing config files in /etc/sensu/conf.d, you might want to make use of the --validate_config flag which was added to Sensu recently. https://github.com/sensu/sensu/issues/1244 includes some relevant discussion around Sensu's approach to config validation.
In light of the above, if you are still experiencing behavior that you consider to be a bug, will you please open a new issue to document your concerns?


Use another semaphore to improve tests

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

Comparison is base (05e9834) 98.27% compared to head (ca64dd3) 98.27%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           5        5
  Lines         290      290
=======================================
  Hits          285      285
  Misses          5        5

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


NeRF studio

Hi,
I'm using nerfstudio as the original NeRF but I couldn't save it using JIT (torchScript).
Do you have any pipeline for nerfstudio?
Alternatively, is it possible to skip the torchScript and load the model using .ckpt file?
I found the nerf.py file attached to an earlier issue, but I didn't understand the context. Can you provide any pipeline for end to end evaluation? From images to distilled NeRF.
Thanks,
Leo

Hi Leo,
Unfortunately the code was not built on top nerfstudio, with nerfstudio not available at the time. The NeRF base code to train the real scene was a version of the multiNeRF code, the weights were then saved and an appropriate loader was written in PyTorch (nerf.py file attached to the mentioned issue) which then together with the weights was saved using torchScript.


Google page title length in Content Guide

Current guidance on Google page titles says 55 characters. The wording also refers to 'Google searches'. Does this mean the title tag? I was a little unsure. If so, the character length for Google is now 70 for Google.

Review new guidance from Steve/Ilan to make sure we are crossing our i's etc re SEO application to the Content Guide URL

Now being tracked in Trello


Adding a second call to ForNpgsqlHasEnum causes the first to be regenerated which leads to a type already exists error.

In migration one, which works as expected:

modelBuilder.ForNpgsqlHasEnum("typeone", new [] { "one", "two", "three" })
dotnet ef migrations add One
dotnet ef database update

Then get ready for migration two, which will not work:

The existing line for "typeone" still exists.
Add: modelBuilder.ForNpgsqlHasEnum("typetwo", new [] { "apple", "cider" })
dotnet ef migrations add Two

This causes a migration like this to be generated:
migrationBuilder.AlterDatabase()
.Annotation("Npgsql:Enum:typetwo", "apple,cider")
.Annotation("Npgsql:Enum:typeone", "one,two,three")
....
.OldAnnotation("Npgsql:Enum:typeone", "one,two,three")
Also added to the Annotation and OldAnnotation are the postgis extensions (which these are not present during non-Enum migration operations).  But those are protected with:
CREATE EXTENSION IF NOT EXISTS
Try to run dotnet ef database update results in:
type "typeone" already exists
Manually removing the reference to typeone in migration Two allows the migration to be applied to the database.  Migration Three works as long as there are no new calls to ForNpgsqlHasEnum.  Any later migrations that call ForNpgsqlHasEnum will add Annotations for typeone and typetwo, etc.
And of course the Down migration in Two is incorrect as well as it tries to add then remove typeone.
Versions:

Npgsql 4.0.1
Npgsql.EntityFrameworkCore.PostgresSQL 2.1.1
ASP.Net Core 2.1.1

Sorry for the delay.  I did test this and the type already exists error is gone.  Thanks!
It would of course be very useful to have the ability to update an existing db enum type (by adding, removing values) during the migration.

It would of course be very useful to have the ability to update an existing db enum type (by adding, removing values) during the migration.

I agree - but unfortunately PostgreSQL support for altering enum types is quite limited - we can add new values but not remove them. Note that we haven't implemented support for adding values in the provider, please open an issue on that if you're interested in seeing it happen.


MWPW-138488 - Arabic font family

Updated Arabic pages to use adobe-arabic font family.

Corrected AR to point to Arabic languages.
Updated typekit for adobe-arabic
Updated Argentina country code to "ARG" and lang code to "ar_es" for Spanish speaking Argentina
Bumped up font size for Arabic to be on par with legacy pages.

Resolves: MWPW-138488
Test URLs:

Before: https://main--milo--adobecom.hlx.page/ar/font-test?martech=off
After: https://arabic--milo--adobecom.hlx.page/ar/font-test?martech=off
Before: https://main--milo--adobecom.hlx.page/ae_ar/font-test?martech=off
After: https://arabic--milo--adobecom.hlx.page/ae_ar/font-test?martech=off

Fonts certainly appear readable to me. When I copied content from a sample page on helpx, for example, content viewed from your arabic branch seems much better to my eye than the original.
https://helpx.adobe.com/mena_ar/photoshop/using/workspace-basics.html
https://arabic--milo--adobecom.hlx.page/drafts/echampio/arabic-test

Thanks for taking a look @echampio-at-adobe!

@rgclayton  - TGTM verified


Cache local candidates till local description is sent

@davidzhao Going to merge this as I want to get this on staging and check if it helps with tests. If you have comments, please drop them here and I will address them in a subsequent PR.


is this pytorch-version can running with  no cuda?

for example,my old computer has no cuda ,but I really want to test this fantasitic network for my slam project(to handle the.pcd files),..what should I do? or how to change the run.py file ?
sincerely!  please~

You can drop every .cuda() in main function :)

You can set use_gpu = False in run.py


References to go-1.13 seem to break go build in Vagrant

https://github.com/solo-io/packer-builder-arm-image/blob/aafbfd7244266ffc49cb9e2f98dc97a1266b415b/provision-build-env.sh#L50
References to go-1.13 in this file seem to break the Vagrant up build, as it currently installs go-1.14, at least on my machine. Not sure whether you want to lift the version up to 1.14 in this script or create a non-dependent way of doing it.

Or tie the apt-get install down to 1.13.

@jzr1991 , @yuval-k i have slightly updated the way we're installing Go so that we have control on its version. I included a couple of updates like some cleanup steps and packer version upgrade since the deployment was complaining for an incompatible API version. Please let me know if any comments.


Performance Issue: Larger then 256 Sizes

When attempting to edit Terrain larger then 256, editor slows down when attempting to modify terrain.

Thanks for posting this!
For my own project, having a big map was not a requirement that's why there is no LOD at the moment. I'll check if I can add the support of something like clipmaps (probably after the holidays).

@eumario
That should be better with the new clipmap feature merged into main.
Let me know if it helps, I'll close the issue after. 🙂

@spimort This looks good.  Able to create a terrain 4500m Squared, with no issues.

Awesome, I'll close the issue then!
Thank you @eumario for taking the time to test it 🙂


Newrelic logging low data mode

Is this a new chart
No
What this PR does / why we need it:
This PR fixes deploying the Windows daemonset, without this change all attempts to install via helm fail at the include statements for newrelic.lowDataMode
Which issue this PR fixes
None at the time of PR
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [mychartname])

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.John Harris seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Parse error: Missing ':' when using #comments in .pa.yaml source files

The following code gives the error Error   PA3003: Parse error: Missing ':'. If this is a multiline property, use | at Screen1.pa.yaml:2,18-2,18
Screen1 As screen:
    # Some comment
    sldSlider As slider:
        # Another comment
        Max: =sldSlider.Width
       ...

but this code packs ok:
Screen1 As screen:
# Some comment
    sldSlider As slider:
        # Another comment
        Max: =sldSlider.Width

When I unpack after a pack with comments - the pa.yaml is missing all the detal after the comment and is just:
Screen1 As screen:

Are yaml comments supported in pa.yaml files?

We do not support Yaml comments in the .pa.yaml file.
We do support PowerApps comments in the formulas (use // )
We should be giving parser errors in the cases above to explicitly warn which parts of Yaml are not in the supported subset.
We're capturing the docs for this here:
https://github.com/microsoft/PowerApps-Language-Tooling/blob/master/docs/YAMLFileFormat.md#yaml-comments

We do not support Yaml comments in the .pa.yaml file.
We do support PowerApps comments in the formulas (use // )
We should be giving parser errors in the cases above to explicitly warn which parts of Yaml are not in the supported subset.
We're capturing the docs for this here:
https://github.com/microsoft/PowerApps-Language-Tooling/blob/master/docs/YAMLFileFormat.md#yaml-comments

Rather than block the pack - could comments be allowed but a warning emitted to indicate that they won't be retained if you round trip - I can think of situations where comments would be useful in code-gen type scenarios.

Rather than block the pack - could comments be allowed but a warning emitted to indicate that they won't be retained if you round trip - I can think of situations where comments would be useful in code-gen type scenarios.

We'll allow comments on their own line.
But we still reject yaml comments on a line with other formulas - because it's too easy that the intention was actually a formula and we're trying to protect against confusion (and against the formulas being accidentally interpreted as yaml comments and stripped )
Screen1 As screen:
    # A comment    <-- this yaml comment is ok
    FillColor: = # FF00CC   <-- this is an error.  Yaml comment not allowed,

We'll give a warning now like:
Pack: D:\dev\pa2\Samples\FullDemo\Weather\MyWeather_src --> D:\dev\pa2\Samples\FullDemo\Weather\temp.msapp
Warning PA2010: Yaml comments don't roundtrip. at Screen1.pa.yaml:14,1-14,1
0 errors, 1 warnings.

We'll allow comments on their own line.
But we still reject yaml comments on a line with other formulas - because it's too easy that the intention was actually a formula and we're trying to protect against confusion (and against the formulas being accidentally interpreted as yaml comments and stripped )
Screen1 As screen:
    # A comment    <-- this yaml comment is ok
    FillColor: = # FF00CC   <-- this is an error.  Yaml comment not allowed,

We'll give a warning now like:
Pack: D:\dev\pa2\Samples\FullDemo\Weather\MyWeather_src --> D:\dev\pa2\Samples\FullDemo\Weather\temp.msapp
Warning PA2010: Yaml comments don't roundtrip. at Screen1.pa.yaml:14,1-14,1
0 errors, 1 warnings.


fixed windows path issues

Fixed a few path issues on windows. In this state I am able to run tina-cloud-starter in the mono repo on my windows machine using powershell.

This has not been tested back in unix/linux/macos


feat: add clear method to clear events

Please check if the PR fulfills these requirements

[x] It's submitted to right branch according to our branching model
[x] It's right issue type on title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has description for the breaking change

Description

add clear method and its test

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@lja1018 Jest 테스트 실패하는 부분 확인해주세요.


ControlledEditor doesn't update when value changes in readOnly mode

Hi, thanks for making this package! The lack of webpack configuration is very nice. 😄
I'm trying to use MonacoEditor in readOnly: true mode as an output visualizer for some text transforms. When the value provided to the editor is updated, the editor's contents don't change. It's quite unfortunate.
The issue can be kind of worked around by providing key={value}, but that causes the editor to refresh all its DOM elements every update. Rather unfortunate.
Edit: here's a reproducible sample: https://github.com/general-language-syntax/GLS-Web-Client/commit/cd7f0c71ff9953ba6c53652b26a7e2589cba9e16#diff-caa2478e3f773087d1ce9d097daf1d16R35.
Edit 2: interestingly, this bug goes away when readOnly: true is not provided.

Hello, thank you for your feedback, and for issue.
I could reproduce the issue. It’s quite strange. I will examine it more deeply. Will keep you up to date here.

@JoshuaKGoldberg the issue is solved in version 1.2.1. Could you please check it. Let me know if it covers your case. Thank you! 🙂

Confirmed, this works great. Thanks so much for the rapid response & turnaround time! 🙌

You're welcome 🙂 🎉 🙌


Improve TfLiteParser to support more models

It supports now:

Mobilenet_V2_1.0_224
ResNet_V2_101
Inception_V3_quant
Inception_V4_quant
MnasNet_1.3_224

Tested with models in this website:

Thanks very much for this, @brunomorishita
I'm sorry to be a pain, but please could you resubmit your work to Gerrit at https://review.mlplatform.org/#/admin/projects/ml/armnn as that is where we host the master development.
If you need any help using Gerrit please let me know.
Matthew


[FLINK-26214] Publish Docker images to GitHub Registry

I managed to make this work with simply using flink-operator for the docker image name, after reconsideration I prefer this for brevity.
We only publish images for the main branch after this PR, I will proceed with working on the release branch/tag setup.
We should improve this ideally where we only tag the image with the latest tag if the CI finishes successfully.

I tested via mbalassi/packages, and could already manually push to apache/packages.

Thanks, @tisonkun I addressed your comments. I went with an approach based on the Semantic versioning example of the docker metadata action.


[Package Issue]: Ryujinx.Ryujinx.Ava - Ryujinx organization no longer exists

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
As with Switch emulator Yuzu, the Switch emulator Ryujinx and its org has also been taken offline.
The relevant GitHub org/project pages already 404, so packages will no longer install.
Steps to reproduce
Attempt to install the package
Actual behavior
Fails, because the project no longer exists.
Expected behavior
N/A
Environment
N/A

Screenshots and Logs
No response

Since all mentioned PRs are closed, I'll close this Issue. If this is in error, please @ me and I'll reopen.


Submit doesn't handle header name-value pairs array

Given an action that would generate a request body, the following:
submit(action, {
  requestInit: {
    headers: [['Foo', 'bar']]
  }
});

generates the following request:
POST /foo HTTP/1.1
Content-Type: application/x-www-form-urlencoded
0: Foo,bar

...

@miller79


Cannot put a space using the MSG action

Trying to do <msg /mine reset>
Outputs - "MessageCommand received multiple arguments. If you want to send a message containing spaces escape them using "."

I should read the errors to the end, apologies :D I feel stupid 🤦

Okay I don't understand how to do it, I tried doing msg /mine "reset but it doesn't work either. Could you give me an example?

I think it should be `msg "/mine reset"

Ahhh gotcha, thanks for the help :)


Line break at sentences' end in a single toast of px-alert-message

Hi,
in px-alert-message is there a way  to insert line break at the end of the sentence, when 2 sentences with a line break are passed a string to messageTitle property?
Currently it doesn't seem to read the new line in the string.
If i pass:
'ABcsdf
sdfk'
in message Title, it shows 'ABcsdf sdfk' itself is shown.
Is there a support in px-alert-message to break the string at full stop or read the newline break in message inserted in messageTitle property?
If not, is there going to be any provision in the future?
Need urgent help on this issue. Thanks in advance.

You can slot in custom html into the content slot to achieve what you want. See this demo for an example: https://github.com/predixdesignsystem/px-alert-message/blob/master/demo/demo.html


chore: add debug option to roi-detected-objects-fusion

PR Type

Improvement

add arg to debug roi-detected-objects-fusion
Related Links

Description

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

:cry: