Highlighting the word under the cursor

Hi,
I'm confused by this part of the config:
https://github.com/nvim-lua/kickstart.nvim/blob/8d1ef972bc32faa86fee21a57f9033b41f612ebb/init.lua#L555
This line and the following lines define functions for highlighting words under the cursor. But I know that this functionality already exists in the plugin https://github.com/RRethy/vim-illuminate.
Maybe it's worth defining this functionality in the additional plugin section lua/kickstart/plugins with the corresponding config?
I'm using a configuration that exactly matches the current functionality of the init.lua configuration; this configuration is taken entirely from the plugin's page:
  -- highlight (underline) the word under the cursor
  -- https://github.com/RRethy/vim-illuminate
  {
    'RRethy/vim-illuminate',
    opts = {},
    config = function()
      require('illuminate').configure {
        -- providers: provider used to get references in the buffer, ordered by priority
        providers = {
          'lsp',
          'treesitter',
          'regex',
        },
        -- delay: delay in milliseconds
        delay = 120,
        -- filetype_overrides: filetype specific overrides.
        -- The keys are strings to represent the filetype while the values are tables that
        -- supports the same keys passed to .configure except for filetypes_denylist and filetypes_allowlist
        filetype_overrides = {},
        -- filetypes_denylist: filetypes to not illuminate, this overrides filetypes_allowlist
        filetypes_denylist = {
          'dirvish',
          'fugitive',
          'alpha',
          'NvimTree',
          'lazy',
          'neogitstatus',
          'Trouble',
          'lir',
          'Outline',
          'spectre_panel',
          'toggleterm',
          'DressingSelect',
          'TelescopePrompt',
        },
        -- filetypes_allowlist: filetypes to illuminate, this is overridden by filetypes_denylist
        filetypes_allowlist = {},
        -- modes_denylist: modes to not illuminate, this overrides modes_allowlist
        modes_denylist = {},
        -- modes_allowlist: modes to illuminate, this is overridden by modes_denylist
        modes_allowlist = {},
        -- providers_regex_syntax_denylist: syntax to not illuminate, this overrides providers_regex_syntax_allowlist
        -- Only applies to the 'regex' provider
        -- Use :echom synIDattr(synIDtrans(synID(line('.'), col('.'), 1)), 'name')
        providers_regex_syntax_denylist = {},
        -- providers_regex_syntax_allowlist: syntax to illuminate, this is overridden by providers_regex_syntax_denylist
        -- Only applies to the 'regex' provider
        -- Use :echom synIDattr(synIDtrans(synID(line('.'), col('.'), 1)), 'name')
        providers_regex_syntax_allowlist = {},
        -- under_cursor: whether or not to illuminate under the cursor
        under_cursor = true,
      }
    end,
  },

Hi. I don't fully understand the issue you're having.
You're saying that kickstart is doing something with code of its own that you'd prefer be done using a plugin?
I have no opinion on that. Maybe create a pull request so people can evaluate a fully fleshed out proposal in code?


Incorrect render priority with occluder material

I was using occluder material to render a portal with a red plane in the bottom on an Android device. The model I use is showing below.

The outside of the portal is a mask that is applied occluder material. I set the render priority as camera = 2, mask = 3, the red plane = 1, other elements = 4. I was expecting that the mask occludes only the things inside it. However, as shown below, it also occludes the red plane which has the highest render priority.

Is your plane transparent by any chance? Transparents are always rendered after opaque objects, regardless of priority.

@romainguy Thank you for your help. By setting the plane to opaque, the render result is as expected now. Is there any way I can get the expected render order if I need the plane to be transparent?

We need to render transparent objects after opaque objects. You could render the occluders twice maybe? Once as opaque and once as transparent?

It seems doesn't help to render the occluders twice. I am not sure I understand this correctly. How should I render the occluders twice with java API?

@romainguy I found that occluder doesn't render correctly on some device. Following is the render result I get on Oneplus 6T with exactly the same code as the previous one.

@romainguy I found that occluder doesn't render correctly on some device. Following is the render result I get on Oneplus 6T with exactly the same code as the previous one.

Hi, @romainguy do you have any idea of this problem？I found that this also happens on Sony Xperia G8142. The occluder material is as follows.
material {
    "name" : "Face Mesh Occluder",
    "parameters" : [
        {
            "type" : "float",
            "name" : "unused"
        }
    ],
    "shadingModel" : "unlit",
    "colorWrite" : false,
    "depthWrite" : true,
}

It's a known issue unfortunately and seems to be related to a driver issue. We haven't found a satisfactory fix.

Closing this bug to avoid multiple duplicates. We'll keep tracking in the other issues mentioned above.


Improve UX Performance on Moving Bars

Resizing by Viewport Size
If Mobile, Hide Moving Bars

Remove useless bottom space * ;<


feat: add next-seo setup

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


Null message in response are filled with empty values

Hi !
When I receive a protobuf message with a field that is a message and is null, the decode will read the message and set default value for it. However it is null.
Could you please fix this bug ? :)

Hey @alexisvisco, a few questions:

Is this during a JSON or Protobuf decoding?
Is this an optional field?

It is with the protobuf decoding
It is not an optional field but it is a message (not a native type)

I don't think message are serialized with default value because in Go message field are generated with pointers and when you provide nothing for a field that is a message it put a null pointer.
Is this more clear to you ?

message FindByIdResponse {
  Dashboard dashboard = 1;
}

If I sent {} through the rpc method I will get a nil dashboard and this is the correct behavior.

What's the reason to avoid marking dashboard as optional? https://developers.google.com/protocol-buffers/docs/proto#optional
message FindByIdResponse {
  optional Dashboard dashboard = 1;
}

This is more flexible because it enables consumers to decide the nullability semantics of a given field.

https://stackoverflow.com/a/42634681/8810797
In protobuf 3 all fields are optionals :)

That is not a correct interpretation of optional in protobuf -- we're overloading the term here.

In proto3, all fields are "optional" (in that it is not an error if the sender fails to set them).

The author of the post is correct that it's not an error if the sender fails to send them -- in fact we generally don't send fields that are the default value under protobuf regardless of what the sender sets. That is an optimization we apply at the serialization layer to decrease the size of the message over the wire.
optional is used to determine the semantics on the deserialization side. Specifically, do we leave the value as a a bottom type (undefined / null / nil, etc) or do we "hydrate" with the default value.

This is maybe a misunderstanding because optional was recently added in proto 3.
However setting optional prefix to a field which is a message does not solve the problem. In my server implementation I return the field with a null value and the client get a message with default value, how am I supposed to know if the field is set or not ?

Optional message deserialization had a bug where the default message value was always being supplied. This has been fixed. Now if the server supplies an optional field with a null value the client will receive undefined for the field.


bug: spell plugin shows default spell-check list on keyboard layouts that have numbers under shift modifier

Did you check docs and existing issues?

[X] I have read all the which-key.nvim docs
[X] I have searched the existing issues of which-key.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0-dev-186+g191e8b406 Build type: RelWithDebInfo LuaJIT 2.1.0-beta3 Compilation: /usr/bin/gcc-10 -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -fno-common -Wno-unused-result -Wimplicit-fallthrough -fdiagnostics-color=always -fstack-protector-strong -DUNIT_TESTING -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/__w/neovim/neovim/.deps/usr/include/luajit-2.1 -I/usr/include -I/__w/neovim/neovim/.deps/usr/include -I/__w/neovim/neovim/build/src/nvim/auto -I/__w/neovim/neovim/build/include -I/__w/neovim/neovim/build/cmake.config -I/__w/neovim/neovim/src -I/usr/include -I/__w/neovim/neovim/.deps/usr/include -I/__w/neovim/neovim/.deps/usr/include -I/__w/neovim/neovim/.deps/usr/include -I/__w/neovim/neovim/.deps/usr/include -I/__w/neovim/neovim/.deps/usr/include -I/__w/neovim/neovim/.deps/usr/include     system vimrc file: "$VIM/sysinit.vim"   fall-back for $VIM: "/__w/neovim/neovim/build/nvim.AppDir/usr/share/nvim"  Run :checkhealth for more info
Operating system/version
Linux TPad 6.3.8-arch1-1 #1 SMP PREEMPT_DYNAMIC Wed, 14 Jun 2023 20:10:31 +0000 x86_64 GNU/Linux
Describe the bug
When in the spell plugin list and want to select an option under a number label, the default spell-check list shows. This occurs only when the due to the keyboard layout, you have to write a number using a shift modifier.
Example

typed 6 (pressed shift + ž on my cz layout keyboard)

The choice had to be confirmed using <CR> key
Steps To Reproduce

change layout to one that has numbers under modifier (cz for example)
select number option in the spell plugin

Expected Behavior
The default spell-check list should not appear after selecting the number
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/which-key.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Neovim can't differentiate between that shift. Not possible to fix. You can probably fix it in your terminal settings instead. No idea how, but I'm sure you'll figure it out

OK. Thank you for the info (and for all the amazing work you are doing in the neovim ecosystem!). I will figure something out.

For anyone who is experiencing a similar problem... A reading of documentation of how mappings work in wezterm, this is a solution that works for my layout. It does this by intercepting SHIFT++ in the terminal and sends it as a 1 instead thus neovim is totally unaware of the SHIFT being ever pressed.
Anyone using wezterm can modify this to his setup specifically without any trouble.
for _, v in pairs { '1', '2', '3', '4', '5', '6', '7', '8', '9', '0' } do
  table.insert(M.keys, {
    key = 'phys:' .. v,
    mods = 'SHIFT',
    action = wezterm.action.SendKey { key = v },
  })
end

where M is the config table being returned


Changed name of NPM package

Codecov Report

Merging #501 into rc-2.0.0 will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           rc-2.0.0   #501   +/-   ##
=======================================
  Coverage       100%   100%
=======================================
  Files            54     54
  Lines          1709   1709
  Branches        249    249
=======================================
  Hits           1709   1709

Flag
Coverage Δ

#builder
100% <100%> (ø)
:arrow_up:

#runtime
100% <ø> (ø)
:arrow_up:

#srcapp
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

config/sky-pages/sky-pages.config.js
100% <ø> (ø)
:arrow_up:

cli/build-public-library.js
100% <ø> (ø)
:arrow_up:

cli/utils/run-build.js
100% <ø> (ø)
:arrow_up:

cli/version.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Use factory pattern for creating \ activating \ model binding a controller in ControllerActionInvoker

We established a pattern of using delegates that close over state in the factory, activator and model binder for pages. This moves the onus of caching and managing state from individual components to a single piece in the ActionInvokerProvider. We should do something similar with controllers.

FYI @Eilon \ @rynowak

Fixed in https://github.com/aspnet/Mvc/commit/c5f771d96d77be8b7bb70e41feeb6d0bac13f7e5


[Suggestion] A few things for the Report

First of all, the Report function is freaking AWESOME, I don't use caps often, so you see how much I like it. I have a thing for stats, so this is really starting to tick all the boxes.
But a few suggestions for it (You probably already have plans for these):

Page numbers.
Totals (Transactions/ total cash/ total items).
Symbols instead of words for amounts ($ instead of Dollars, would clear up some space), if applicable of course.
Player list (A list of players which bought/ sold items to you).

Also should you be able to click on the details magnifying glasses? Because I can not.
I know you're busy but wanted to leave this here for when you have the time :)
And thanks for remaking SignShops and putting it on steroids, it's like my dream come true xD

Alice
DragonTechMC

I plan to add a lot of things when I got time. With per day/week/month stats.
I'll see what I can do for the symbol. I already adapt to the server currency.
Click the glass will pop more details about the transaction itself, like what is written on the sign.

Are you also able to add server shops to the report? Right now if i get a players report it doesn't show what they have bought from (ibuy) or sold to (isell) the server. I would also like to be able to get the report of the server signs, both all the server signs and then, just to limit the size of the log im looking at, report of a specific sign. Not sure how much of this is possible but I would like this feature.

I like the idea. I'll add that.

generate a page for the server
log the ibuy and isell signs

I already plan to filter the shop transactions by sign

I'll work on reports next week end.
Any more ideas while I'm at it?
 Might need a separate report, maybe something in-game instead, but as I said, I love stats
Top money spent (All time/ monthly/ weekly/ daily?)
Shops created
Shops used
^ stuff like that

Main page will be general stats (nb of transaction, nb of items, money... With customisable time window)
There will be subpages for list of transactions, which can be filtered by player, sign and items.
I'm still working on it with a ux designer

It would be nice to generate the report for only a date range, player, maybe by item as well.  This would give better insight in to hot items and active shoppers.
Lastly, it would be nice if you could get totals for both purchases and sales in the range or even the current report.
Similar to op thoughts but filters and filtered totals would be very nice.

Hello, the report is reporting only in player order now making it impossible to see what I had done since my last report.  Part of the order by date that works great is offline time can give you what you need to restock or empty.


chore: update protobuf generation to latest buf changes

Description
Thanks for helping out with this @julienrbrt ❤

Updates to latest buf changes to align with sdk protobuf codegen

This PR updates the proto codegen and buf tooling to align with sdk/v0.46.x. A subsequent PR will be required to update to the changes introduced for sdk/v0.47.x
ref: #2671

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Seeing some weird test failures re. duplicate type url registration in the codec...

I'm happy to keep https://github.com/cosmos/ibc-go/issues/2671 open until 0.47 and just take this as a stepping stone towards getting it closed. I think this covers the bulk of the work tho.
For 0.47 we will need the changes from https://github.com/cosmos/ibc-go/pull/2705#discussion_r1016792760 and to regenerate the codegen.

lgtm, you may want to have a github action that pushes the changes automatically: https://github.com/cosmos/cosmos-sdk/blob/main/.github/workflows/proto-registry.yml cc @tac0turtle for creating the buf repo and adding the right buf token.

created a repo here https://buf.build/cosmos/ibc. Could I get some emails to invite to the cosmos org in the buf registry

Could I get some emails to invite to the cosmos org in the buf registry

Not sure how many you need, but here are the codeowners of this repo.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

As a side note, is this expected?
Formatting Protobuf files
find: clang-format: No such file or directory
find: clang-format: No such file or directory
find: clang-format: No such file or directory
find: clang-format: No such file or directory
find: clang-format: No such file or directory
...

It is expected but not desired. So for 0.46 compatibility the docker image defined is still tendermintdev/sdk-proto-gen, we used to use v0.2 which didn't have this issue, but updating to v0.7 seems to not have the clang-format tool.
I've been trying to track down where this image's dockerfile is maintained but I can't seem to find it anywhere easily.
Since this is temporary until we migrate to ghcr.io/cosmos/proto-builder for 0.47 we could potentially just defined our own under contrib/devtools like below, I tested this locally and it was fine. Otherwise, we continue with no formatting.
What do you think? cc. @colin-axner @chatton
FROM tendermintdev/sdk-proto-gen:v0.7

RUN apk add --no-cache clang-extra-tools


chore: fix colors

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

v0.1.0
GitHub release

Your semantic-release bot :package::rocket:


Error in callback: message.State is null or empty

I have used one of your samples (IdentityServer4InMem) offering 2 login possibilities: local login with Identity server or external login with Microsoft Identity Web.
Apart from that, I have an SPA using the oidc-client library to request an access token using auth code with PKCE.
If I click on external login (in your sample you were using Google but I changed to Microsoft Identity Web), after logging in into the Microsoft login form the next error is thrown: OpenIdConnectAuthenticationHandler: message.State is null or empty.
I have checked a lot of things but not able to see what happens.
It seems that there are some conflicts between initial SPA calls and what Microsoft Identity Web sends, could you say to me if I miss something pls?
I attach log trace and repro project with testing instructions (very simple).
Many thanks.
IdentityServers.zip
trace.txt

Solved, my error was to mix the internal callback (/signin-oidc) with my custom one.


[Initialization] Remove swift initialization from SystemInitializerCommon

Swift initialization should occur in SystemInitializaterFull and not in the SystemInitializerCommon.
This PR has 3 changes:

I found that SwiftASTContext was being initialized in both Full and Common, which seems problematic. Removed the one in Common.
Moved SwiftREPL initialization from Common to Full.
Stopped linking lldbInitialization against the swift ExpressionParser plugin. This is good because it means lldb-server stops linking against it.

cc @compnerd @dcci @JDevlieghere

@swift-ci test


Question: With async support enabled, sync wasm functions work as async

It's difficult to understand the whole codebase so I'd better ask experts here to enlighten me. If I understand all these async logic provided by wasmtime, exported WASM functions actually work as async functions when async support is enabled.
So let's see an example (host and module pseudo):
// host
#[tokio::main(flavor = "current_thread")]
pub async fn main() {
    let mut config = Config::new();
    config.async_support(true);
    let engine = Engine::new(&config).unwrap();
    let module = Module::from_file(&engine, "./target/wasm32-wasi/xxx.wasm").unwrap();
   ...
    let mut linker = Linker::new(&engine);
    linker.func_wrap1_async("host", "process", |_caller, x: i32| Box::new(async move {
        sleep(Duration::from_millis(5000)).await;
        Ok(x * 100)
    })).unwrap();
    wasmtime_wasi::add_to_linker(&mut linker, |s| s).unwrap();
    ...
    for _ in 0..10 {
      tokio::spawn(async move {
         ...
         let mut store = Store::new(&engine, wasi);
         let instance = linker.instantiate_async(&mut store, &module).await.unwrap();

         let run = instance_echo.get_typed_func::<(), (), _>(&mut store, "run").unwrap();
         alloc.call_async(&mut store, ()).await.unwrap();

        localasync().await;

         println!("Done");
       });
    }
}
async fn localasync() { ... }

// module
#[link(wasm_import_module = "host")]
extern "C" {
    #[link_name = "process"]
    fn process(a: i32) -> i32;
}
#[no_mangle]
extern "C" fn run() {
  process(1); // can this be threated as safe/true async?
  process(2);  // can this be threated as safe/true async?
  println!("Done");
}

Will run and process be spawned in a dedicated thread where they will block or will the callable sequence of host functions inside run be somehow magically converted to "safe" non-blocking async sequence? I'd like to understand this in-depth so I'd appreciate someone explaining the behind-the-scene execution flow.

As far as I know it runs the wasm on the current thread, but on a different stack. When the wasm calls into an async function for which polling results in Poll::Pending, it switched back to the original thread and then returns Poll::Pending from the poll method of the original call into the wasm code,

@bjorn3 that's how I understand it as well. I'm always afraid of what I don't see/understand thus I"d just like to make sure this approach is not blocking. So it shouldn't block, no meter what you do there, right? I wonder how this separate stack execution works.

Wasmtime doesn't spawn any tasks or threads internally, all async happens in the original async task which wasmtime was invoked on. As a practical detail-specific implementation the WebAssembly must be able to be suspended because the host function may not be ready when invoked (as is the case in your sleep example). The Wasm itself uses stack switching to handle that but you're insulated from that in the sense that it's handled by Wasmtime and there's nothing you need to do about it.
Another way to think about this is that the poll function for the future returned from call_async will execute WebAssembly internally up to the point that a host future needs to block and returns NotReady, then the poll function propagates that signal.

@alexcrichton thank you, that's what I was looking for. So basically wasm implementation is smart enough, won't block threads, and will rather switch between stacks. I also realized that the so-called "stack switching" relates to underlying Crainlift and not OS (e.g Linux).


Create a python template app for exporters

Create or design an exporter could be a challenge that it could be easy to do if pelorus provides a template or scaffold for that.
Pelorus's operators or developers could use it to develop new exporters easily with the same structure, configuration patterns and developer guidelines.
That template could be part of the Developer Guide in this repo with a folder including the files and minimum structure to develop, test and run locally.
Also it could help the review process to add to pelorus stack.

@rmarting that's interesting. We've been working towards doing a bit of a refactor of our code to standardize the data structure of the "metrics" that we're exporting, which would make something like this a lot more feasible.
#225


Split creation functions from creation functions with Schedulers

Currently we have creation functions like so:

from(obj) and from(obj, scheduler)
range(a, b) and range(a, b, scheduler)
(... and many more)

Problem
Schedulers and scheduling adds a LOT of bloat to bundles and we can't currently avoid it.
If you look at the implementations of these you'll see an obvious pattern, especially in the ones I more recently created as "just functions":
function fromSomething<T>(something: Something<T>, scheduler?: IScheduler) {
  if (!scheduler) {
     return new Observable(/* a little bit of code */);
  } else {
    return new Observable<T>(subscriber => {
      /* OMG WHAT?!! A HUGE MOUNTAIN OF CODE AND A BUNCH OF TYPES
          WE NEED TO IMPORT FOR THIS SO WE CAN DO THINGS
          WITH SCHEDULERS, WHICH DON'T GET USED 95% OF THE TIME */
    });
  }
}

/* MORE STUFF DOWN HERE FOR SCHEDULING SUPPORT */
function dispatch() {
  /* you get the idea */
}

Proposal

Split creation methods into "normal" and "scheduled" flavors: from and fromScheduled, defer and deferScheduled, range and rangeScheduled etc.

If we do this, then the majority of RxJS usage (which doesn't involve schedulers) can avoid pulling in scheduler-related classes and scheduling-related logic.
That would make the above into:
95% use case
function fromSomething<T>(something: Something<T>) {
     return new Observable(/* a little bit of code */);
}

and else where
5% use case
function fromSomethingScheduled<T>(something: Something<T>, scheduler: IScheduler) {
    return new Observable<T>(subscriber => {
      /* OMG WHAT?!! A HUGE MOUNTAIN OF CODE AND A BUNCH OF TYPES
          WE NEED TO IMPORT FOR THIS SO WE CAN DO THINGS
          WITH SCHEDULERS, WHICH DON'T GET USED 95% OF THE TIME */
    });
}

/* MORE STUFF DOWN HERE FOR SCHEDULING SUPPORT */
function dispatch() {
  /* you get the idea */
}

Risks
It will be a breaking change for people using schedulers today. Easy to fix, but annoying.

Done.


Missing back button in Remote Repositories picker

Testing #177534

"Open Remote Repository"
"Open Repository from Azure Repos"
I might change my mind and want to go back here but I have to escape and start over

This is available again since we restored the Remote Repos picker in order to break the extension dependency between Remote Repos and the git extension


fix empty editorState sidebar jump

As soon as you click on an empty page, the calculation for the top margin for the sidemenu is done, but it comes out as -4, as there are no elements to count. This means the sidebar jumps, but not allowing the top margin to go below 0 fixes this.

Hi Nvveen,
Thanks for contributing to Megadraft!
I've tested your patch and the sidebar does not return to the start position. An alternative approach to fix this problem could be to set the initial top state to -4. (I don't know if this is the best fix).

Hey @Nvveen, I'm closing this PR ok? Feel free to reopen if you want to give it another try.


[SPARK-40406][CORE] Change default logging to stderr to consistent with the behavior of log4j1

What changes were proposed in this pull request?
As @Kimahriman mentioned, the default logging now goes to stdout instead of stderr, so this pr change it back to stderr.
Why are the changes needed?
Keep consistent with the behavior of log4j1.
Ref to
https://github.com/apache/spark/blob/78a5825fe266c0884d2dd18cbca9625fa258d7f7/core/src/main/resources/org/apache/spark/log4j-defaults.properties#L18-L23
and the log4j2.properties.template also points to SYSTEM_ERR.
https://github.com/apache/spark/blob/78d492c1b153240dddc636ec6002e7bfc6b94b3b/conf/log4j2.properties.template#L20-L32
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GitHub Actions

friendly ping @srowen @HyukjinKwon @dongjoon-hyun @viirya

Thanks for fixing it.

Thanks. Merging to master.

thanks @HyukjinKwon @viirya


Workload with Ingress Quick Start: 504 Gateway Time-out

Rancher versions:
rancher/server or rancher/rancher: rancher/rancher:v2.0.8
rancher/agent or rancher/rancher-agent: rancher/rancher-agent:v2.0.8
Docker version:
`Client:
Version: 17.03.3-ce
API version: 1.27
Go version: go1.7.5
Git commit: e19b718
Built: Thu Aug 30 01:06:10 2018
OS/Arch: linux/amd64
Server:
Version: 17.03.3-ce
API version: 1.27 (minimum version 1.12)
Go version: go1.7.5
Git commit: e19b718
Built: Thu Aug 30 01:06:10 2018
OS/Arch: linux/amd64
Experimental: false
`
Operating system and kernel:
CentOS7 3.10.0-862.el7.x86_64
Type/provider of hosts:
VMWare
Setup details:
Fully offline
One host: <IP_ADDRESS> (also hosting docker registry)
3 agents: <IP_ADDRESS> & <IP_ADDRESS>& <IP_ADDRESS>
Steps to Reproduce:
I am trying to follow this tutorial:
https://rancher.com/docs/rancher/v2.x/en/quick-start-guide/workload/quickstart-deploy-workload-ingress/
I deployed the workload:

But when I click on the target the app opens on a 504 Gateway Time-out nginx/1.13.12 page
Did anyone run into this problem?

Answering my own question: on a rancher full offline deployement the default ingress configuration uses an http://xip.io/ generated hostname. This cannot work as the xip.io server cannot be reached while offline. Instead the hostname for the proxy must be manually defined in rancher and accessible in the DNS (it should point to one of the hosts defined in the cluster)


SILGen: Fix invalid SIL emitted when protocol requirement witnessed by base class convenience init

If the derived class is final and the base class is not, the
protocol requirement can be witnessed by a non-required
initializer. Non-required initializers have initializing
but not allocating entry points in the vtable.
The fix here statically dispatches to the allocating
initializer instead. The test verifies that the allocating
entry point uses alloc_ref_dynamic, so you will get an
instance of the correct subclass.
This fix does not handle the resilient case where the
conformance is defined in a different module, and the original
module adds an override of the convenience init later.
I filed rdar://problem/40639124 to track the resilient case.
Fixes rdar://problem/40003840.

@swift-ci Please smoke test


Remove B2C from maximum requests limit

I would like to propose removing "B2C" from Maximum requests per IP per Azure AD B2C tenant       |6,000/5min

We have run into an issue with the B2C where multiple Azure AD B2C tenants were throttled at the same time, where it was only one environment/Azure B2C tenant causing the issue. They were being called from the same IP-Address.
Through a support request with Microsoft we received the following response:
"Throttling 429 errors happen at AAD gateway level. The gateway is the first hop into MS infrastructure and all requests go through it. After the requests get to the gateway they are forwarded to the proper service, in this case to B2C service.
The values you have in that document mean that if there are over 250 request per sec coming through the gate with final destination at the same b2c tenant, only 200 requests will pass and 50 will be throttled. You'd get 50 http 429 errors.
Now, the same logic is applied to the IP address. If there are over 6000 requests per 5 min coming from the same IP address only 6000 will be redirected to the tenant, all others will be throttled the same way."
So it seems like throttling happens at the AAD gateway level and not the B2C for a specific IP-Address.

@kengaderdus
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Casper-Olsen : Thanks for your contribution! I need to confirm this with our product team, before we merge this contribution.

Sure :)
Let me know if it needs to be written in some other way, perhaps referring to AAD gateway in some way.
it just doesn't seem like throttling is fully isolated to a single Azure B2C tenant.

@kengaderdus
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team

@Casper-Olsen
Please see @saraswatshantanu's comment.
#please-close


pull request for e2e test at 12/18/2016 14:45:19

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


torch.transpose seems to be mapped to different ops depending on requires_grad

Note: If you have a model or program that is not supported yet but should be, please use the program coverage template.
🐛 Bug

I admit I'm not sure if this is a bug of an expected behavior, but torch.transpose is mapped to torch.transpose if requires_grad=False, torch.permute otherwise.
To Reproduce
Code sample
import torch
import thunder

@thunder.jit
def f(x):
    return x.transpose(0, 1)

for requires_grad in (False, True):
    print("#" * 120)
    x = torch.rand((4, 2), requires_grad=requires_grad)
    f(x)

    print(f"### {requires_grad = }")
    print(f"### thunder.last_traces(f)[-1]\n{thunder.last_traces(f)[-1]}")

output
########################################################################################################################
### requires_grad = False
### thunder.last_traces(f)[-1]
# Constructed by Unwrap the actual return value
import torch
from thunder.executors.torchex import no_autocast

@torch.no_grad()
@no_autocast
def computation(x):
  # x: "cpu f32[4, 2]"
  t0 = torch.transpose(x, 0, 1)  # t0: "cpu f32[2, 4]"
    # t0 = ltorch.transpose(x, 0, 1)  # t0: "cpu f32[2, 4]"
      # t0 = prims.transpose(x, (1, 0))  # t0: "cpu f32[2, 4]"
  return t0
########################################################################################################################
### requires_grad = True
### thunder.last_traces(f)[-1]
# Constructed by Delete Last Used (took 0 milliseconds)
import torch
from thunder.executors.torchex import no_autocast

@torch.no_grad()
@no_autocast
def computation(x):
  # x: "cpu f32[4, 2]"
  t0 = torch.permute(x, (1, 0))  # t0: "cpu f32[2, 4]"
    # t0 = ltorch.permute(x, (1, 0))  # t0: "cpu f32[2, 4]"
      # t0 = prims.transpose(x, (1, 0))  # t0: "cpu f32[2, 4]"
  return {'output': t0, 'flat_args': [x], 'flat_output': (t0,)}, ((), ())

Expected behavior

Environment

PyTorch Version (e.g., 1.0):
OS (e.g., Linux):
How you installed PyTorch (conda, pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Additional context

I think what's happening here is that there's no grad rule for torch.transpose defined, so the grad decomposes it to a prims.transpose call, which then has a grad rule defined here:
https://github.com/Lightning-AI/lightning-thunder/blob/ea0159ca20b4109a84f572020b73220fb3c85328/thunder/core/transforms.py#L947
And then prims.transpose maps to torch.permute in the Torch executor here:
https://github.com/Lightning-AI/lightning-thunder/blob/ea0159ca20b4109a84f572020b73220fb3c85328/thunder/executors/torchex.py#L581
Because prims.transpose is consistent with jax.lax.transpose and torch.permute, which is a little confusing that not everyone agreed on what transpose and permute should mean.
One way to address this oddity might be to add a comment with the provenance of the final operation. Another idea, which we've discussed for awhile, would be to add more custom grad rules for torch operations, so that, for example, torch.transpose would just have its own grad rule that would also call torch.transpose. This might even improve performance, since custom grad rules can be more efficient than autogenerated ones.
fyi @IvanYashchuk; maybe @beverlylytle would be interested in identifying speed or memory differences between the autograds that Thunder generates and PyTorch eager's (and torch.compiles?)? Maybe instead of identifying the differences it's more interesting to just add some more autograd formulas. An automated mechanism to identify speed and memory differences would probably help us identify the most important operations to cover, however, and make it easy to talk about the improvement after a custom formula was added. It would also be important to verify that the custom handwritten formula actually had better performance, too.

We have a simple way to measure speed and memory differences using pytest-benchmark (speed is printed in the terminal output, memory is not but it's saved in the benchmark results file). Here's an example of gelu:
https://github.com/Lightning-AI/lightning-thunder/blob/fef423b39661511b1bbb5a5e29b80c6260c228e6/thunder/benchmarks/targets.py#L215
The challenge is determining the appropriate input shape for comparison. This is easy for elementwise operations but more difficult for operations like scaled dot product attention. When we have concrete shapes to analyze (for example from logs of ThunderFX recorded in SubgraphInfo) then we can create benchmarks per each PyTorch operation from the input graph similar to per graph benchmarking in https://github.com/Lightning-AI/lightning-thunder/blob/fef423b39661511b1bbb5a5e29b80c6260c228e6/thunder/dynamo/compiler_graph_benchmark.py#L33-L76


Typo in az aks create -h

Describe the bug
az aks create -h, the output:

--nodepool-name                           : Node pool name, upto 12 alphanumeric characters.
                                            Default: nodepool1.

Expected behavior
It should be up to?
Environment summary
Windows

aks

This should be fixed! Closing for now.


Omni for Alacritty

Add a new theme to the organization?
If you have already created a theme, make sure you check the following requirements:

[x] The theme follows the template.
[x] The theme follows the Omni Color Palette.
[x] The branch name is main (we will keep this name pattern from now on)
[x] The theme has instructions on how to install it.

Application: Alacritty terminal emulator
GitHub URL: https://github.com/Mdsp9070/alacritty
Yeat another theme that I'd created as a standalone version haha, here's the old repo URL: https://github.com/Mdsp9070/alacritty-omni

Hey @Mdsp9070. Thanks for creating the theme!
I have reviewed it, and we can include the theme in the organization. 🎉
I'll be sending an invite for you to join the organization. After that, you can transfer your repository to the org.
Just make sure to open a PR and include the theme on the main repository 👊🏻
Also, after the transfer, make sure to change all URLs to match the new repository link.

Ok! Thank you! I already openned the PR and changed the url!

That's perfect. Remember to accept the invitation to join the org and transfer the repository.
Let me know if you have any questions!

So, I successfully merged the PR. Thank you for the attention!


enable abi split doesn't work

Tell us about the problem
app doesn't build if you enable abi split
Which platform(s) does your issue occur on?
Android
Please provide the following version numbers that your issue occurs with:

CLI: 4.1.0
Runtime(s): 4.1.1

Please tell us how to recreate the issue in as much detail as possible.
I enable abi split here:  app/App_Resources/Android/app.gradle
with following code:
splits {
    abi {
      enable true //enables the ABIs split mechanism
      reset() //reset the list of ABIs to be included to an empty string
      include 'arm64-v8a', 'armeabi-v7a', 'x86'
      universalApk true
    }
  }

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> Conflicting configuration : 'armeabi-v7a,x86' in ndk abiFilters cannot be present when splits abi filters are set : x86,armeabi-v7a,arm64-v8a

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 7s
Command ./gradlew failed with exit code 1

I can confirm  that it does not work anymore with latest cli. You can't have split with abiFilters.
For now i had to disable split. I hope this gets fixed soon.
Thanks

@farfromrefug
There isn't a problem with abi splits. Please enable it at app/App_Resources/Android/app.gradle with following code:
android {
....
  ndk {
    abiFilters.clear()
  }
  splits {
    abi {
      enable true //enables the ABIs split mechanism
      reset() //reset the list of ABIs to be included to an empty string
      include 'arm64-v8a', 'armeabi-v7a', 'x86'
      universalApk true
    }
  }
....

@miroslavaivanova it fails to build if use the "ndk" section. Seems like it does not know it.
Any idea?

@miroslavaivanova still having the issue. gradle does not recognise ndk

@farfromrefug, @miroslavaivanova
Contrary to what the {N} documentation shows, the ndk section should be nested inside the defaultConfig section and not inside android:
android {
  defaultConfig {
    generatedDensities = []
    applicationId = "org.nativescript.testapp"

    ndk {
      abiFilters.clear()
    }
  }

  splits {
    abi {
      enable true //enables the ABIs split mechanism
      reset() //reset the list of ABIs to be included to an empty string
      include 'arm64-v8a', 'armeabi-v7a', 'x86'
      universalApk true
    }
  }
}

@darind amazing I can confirm that it works!
Thanks

I'm getting the following error:
`A problem occurred evaluating project ':app'.

Cannot invoke method clear() on null object`

I'm getting the following error:
A problem occurred evaluating project :app.
 Cannot invoke method clear() on null object

I got same problem

@KunNiu try this
        ndk {
            abiFilters "armeabi-v7a", "x86", "arm64-v8a", "x86_64"
            abiFilters.clear()
        }


Async calls to get user groups for azureAD

Problem:
If a user has many groups the server calls azureAD for all groups in
sequence which can greatly increase login time
Solution:
Async the calls to azureAD when doing login
Issue https://github.com/rancher/rancher/issues/15858

This is also adding debug logging to help support diagnose a customer issue with slowness while using azureAD.

LGTM


Use actions.select_default instead of deprecated function

This fixes
E5108: Error executing lua ...cker/start/telescope.nvim/lua/telescope/actions/init.lua:26: `goto_file_selection_edit` is removed and no longer usable. Use `require('telescope.actions').select_` instead. Take a look at developers.md for more Information.

Oh this was reported in #2

Just saw that it was reported, closes #2

Closes #2

Fixes #2


Add Keyboard class for handling raylib keyboard related functions

I noticed there are classes defined for handling mouse (Mouse.hpp) and gamepad (Gamepad.hpp) inputs. However, there are also keyboard input functions in raylib that are not defined in raylib-cpp.

I think it would make sense to define these in a Keyboard class. If you agree, would you like me to make a PR for this?

Also, perhaps the SetExitKey() function should be added as a method of the Window class?

yaaaaaaaaaaaaaaaaaaaaaaaaaaaaaas

Added in #275


Enabled 2nd passphrase modal when sending transaction

If the wallet has a 2nd passphrase registered, it will now ask for this before sending the transaction instead of simply failing. Fixes #71
This was actually easier to solve than I had expected 😛

While using the second passphrase, it turns out that it sometimes is not properly saved (I assume). I now have an account where I'm unable to send a transaction, as it will result in an error. According to the runtime error, it has to do with "hasOwnProperty" in the Tx.generate function in the ark-ts module, which I assume is this line:
if (this.secondPrivKey && !tx.asset.hasOwnProperty('signature'))
Still looking into why it does work on my other wallet. The only difference I'm aware of between the two wallets is that I didn't immediately save my passphrase the first time. Steps to replicate the issue:

Have a wallet with a 2nd passphrase enabled
Send a transaction
On the 'transaction sent' screen, choose to save your 2nd passphrase
Go through the screens until it states that is has been successfully encrypted / saved
Try to send another transaction, and it will fail.

On my other wallet I did not save it after the first transaction. Instead I made another one and then followed steps 3-5, but with that wallet I'm able to send transactions without any issue.

Hey!
I had a similar problem in my other PR.
I think the problem is that in transaction-send.ts the secondPassphrase is set.
  onEnterPinCode(keys: WalletKeys) {
    const amount = new BigNumber(this.transaction.amount);
    const data: TransactionSend = {
      amount: amount.times(constants.WALLET_UNIT_TO_SATOSHI).toNumber(),
      vendorField: this.transaction.smartBridge,
      passphrase: keys.key,
      secondPassphrase: keys.secondKey, // remove this line!
      recipientId: this.transaction.recipientAddress,
    };

    this.arkApiProvider.api.transaction.createTransaction(data).subscribe((transaction) => {
      this.confirmTransaction.open(transaction, keys);
    }, () => {
      this.toastProvider.error('TRANSACTIONS_PAGE.CREATE_TRANSACTION_ERROR');
    });
  }

I think the "keys" only have to be passed in on the  this.confirmTransaction.open(transaction, keys); method and not already when creating the transactions.

@Nasicus It seems you are right. After removing that line the transactions goes through without any issue. Thanks again! :)

Well, you found a bug in ark-ts, so I fixed and updated the package! 👍🏻


Can't save

I can't change some of the fields. I have no ideas why that is.

can you write your ORM setup with the schema

Thanks @wojtek-krysiak I have fixed the issue. It turned out I can't use mongoose schema to create model for admin-bro. :D


Rating Blocks not counting ratings.

Concrete v9.3.4
PHP v8.3
Macareux Ratings v.1.0.1
When a rating block is put on the website, and you click it, the rating number does not go up, but it stays static at 0.
The table btC5jRatings is updated with all the Rating Buttons that are put on the website, same with the table btC5jRatingPageList, it is updated with all the Rating Page Lists you put on the website.
But the table C5jRatings does not register anything, it stays empty even though the rating buttons and rating page lists are clicked.

@SashaMcr
Could you please confirm this issue? I think you had resolved it previously.

@biplobice
it works as expected
@leal-k
Is it problem on some project or on clear project?

I just replied to Biplob on the other conversation, on fix #34 , explaining the whole bug and fixes, but basically I was getting this error when I fixed the array to string conversion. When I updated the SQL query both bugs disappeared, and the counts were being stored, as I commented on fix #36 .

@leal-k
Cc: @SashaMcr
If this issue isn't valid anymore, please feel free to close the issue and the PR.


Login Screens Design

Go over all login flow screens and define ux/ui/text improvements or if the screen is ok. Some parts we already had with the prototype test.
Acceptance criteria

[x] Select User
[x] Username
[x] Password
[x] Setup MFA Choose
[x] Setup MFA OTP
[x] Setup MFA U2F (Universial Second Factor)
[x] Setup MFA Done
[x] Passwordless Choose
[x] Passwordless Registration
[x] Passwordless Registration Done
[x] Init user
[x] Init user done
[x] Password Change
[ ] Link User (External Identity Provider)
[x] Register

I added all my findings very crudely here in this doc and - as mentioned by @fforootd - not made any issues out of them.
https://docs.google.com/document/d/195TRoG6_JF7DPDgwJbunNLUw5gmM8RK-kW7js83cXx8
Let me know what you think and if you need more informations on some parts.

@juergrinaldi is this issue complete, then? Do we have everything as per checklist above?

except Link User, yes.

except Link User, yes.

Are you still working on this? Since it is not assigned I am confused 😁

Isn't that a set design from the ID providers and not up for discussion?
i.e. if you choose Google/Gmail Login, that it will get handed over to Google for the identification process and with that coming with the Google UI?
Or did I missinterpreted the input above?

Well yes but as the data is returned from an external provider there are two potential ways of dealing with it:
A) we autoregister the user there is no action needed
B) The user is allowed to link his user to an existing user in zitadel, in which case there is the need of a guided process to make this happen

oh. I see. I wasn't aware of that. (I was expecting A)
I'll take another look then.

I will close this issue. We have created others for each page and I think most of it is already implemented


MACS2 to CEAS

Hi Tao,
After MACS2, I want to run CEAS, and I found MACS2 give bdg and narrowpeak, so how can I change bdg to wig file and narrowpeak to bed file? I tried to run MACS and got bed and wig, but the peaks in MACS and MACS2 were different, I also tried use bdg and narrowpeak to run CEAS, it failed.
Thank you
Spencer

There is a special version of CEAS which supports bigwig file [https://bitbucket.org/cistrome/cistrome-applications-harvard/src/7bcbf7953a8b46392fa8580969d7671df6f84364/published-packages/CEAS/?at=default]. You can convert bdg to bigwig using this script: https://gist.github.com/taoliu/2469050. After install the new CEAS, use the 'ceasBW' command instead.

Close since it's not related to MACS.


Nimble stack resets with reason 0x13

Hello again,
I'm implementing the nimble Bluetooth stack (esp-idf v4.1.1) into my ESP32-WROOM-32D project which has been set as BLE peripheral. When I walk away with my Phone or disable bluetooth, and then come back in range or enable bleutooth and try to connect again, the nimble reset_cb is called with error 0x13 (BLE_HS_ETIMEOUT_HCI).
I've got two devices which are currently bonded, 1 still connected, on the other I execute the steps as described above. The problem is that I cannot always reproduce the issue, my guess is that this is due to Bluetooth addresses changing is not always at the same moment?
My bluetooth device has been set up as followed:
void CEspNimbleBluetoothDevice::Init()
{
  esp_err_t result;
  result = esp_nimble_hci_and_controller_init();
  Assert(result == ESP_OK);

  nimble_port_init();

  ble_hs_cfg.reset_cb = PeripheralOnResetCallback;
  ble_hs_cfg.sync_cb = PeripheralOnSyncCallback;
  ble_hs_cfg.gatts_register_cb = GattRegisterEventCallback;
  ble_hs_cfg.gatts_register_arg = reinterpret_cast<void*>(this);
  ble_hs_cfg.store_status_cb = ble_store_util_status_rr;

  ble_hs_cfg.sm_io_cap = BLE_SM_IO_CAP_NO_IO; // No input/output capabilities (just works)
  ble_hs_cfg.sm_bonding = 1; // Enable bonding
  ble_hs_cfg.sm_sc = 1; // Use secure connection
  ble_hs_cfg.sm_our_key_dist = BLE_SM_PAIR_KEY_DIST_ENC | BLE_SM_PAIR_KEY_DIST_ID; // Enable LTK + IRK
  ble_hs_cfg.sm_their_key_dist = BLE_SM_PAIR_KEY_DIST_ENC | BLE_SM_PAIR_KEY_DIST_ID; // Enable LTK + IRK

  ble_svc_gap_init();
  ble_svc_gatt_init();

  memset(m_serviceTable, 0, sizeof(m_serviceTable));
  memset(m_characteristicRegistrationTable, 0, sizeof(m_characteristicRegistrationTable));
}

See SDK config for more information: sdkconfig.txt
The image below contains my debug log:

The following things in my log stand out:

Unhandled event 0x10 - BLE_GAP_EVENT_ENC_CHANGE
Unhandled event 0x15 - BLE_GAP_EVENT_MTU

Looking at the bleprph example both events are not required to be handled but will be returned with 0 directly.

The reset_cb is called, with error: 0x13 - BLE_HS_ETIMEOUT_HCI
Advertising start error: 0x15 - BLE_HS_ENOADDR, probably because the nimble stack just reset, see error 0x13

Hopefully somebody can help me with this.

Hi all, I'm still having trouble with this issue. Can anyone help?


Issue when Canoe is already running

When Canoe is running and i want to send data multiple time then error occur.

When Canoe is Running i  just want to Transmit and Receive data. How to solve it?

Hello
First i run def startmeasurment() then Canoe started.
Then I comment out  #startmeasurment() and only run def TxRx()
When Canoe is running and i just try to send data then blow error occur.

How to Tx/Rx data whith out again Start/Stop measurment?

def open(self, canoe_cfg: str, visible=True, auto_save=False, prompt_user=False) -> None:
if os.path.isfile(canoe_cfg):
self.log.info(f'CANoe cfg "{canoe_cfg}" found.')
self.__dispatch_canoe()
self.__canoe_app_obj.Visible = visible
try:
self.__canoe_app_obj.Open(canoe_cfg, auto_save, prompt_user)
except:
print ("Already open")
self.log.info(f'loaded CANoe config "{canoe_cfg}"')
self.__fetch_canoe_cfg_general_data()
self.log.info('Fetched CANoe System Variables.')
else:
self.log.info(f'CANoe cfg "{canoe_cfg}" not found.')
Updated code

Win32com object will be created when CANoe is initialised.
Before exiting python code user should make sure CANoe application is stopped.
If user not stopping CANoe application then 2nd script cannot start CANoe application instance since CANoe is already running.
This is a known limitation right now.
So my recommendation is to execute both CANoe start stop in the same run.

I have updated code, Now if Canoe is running then also works
def open(self, canoe_cfg: str, visible=True, auto_save=False, prompt_user=False) -> None:
if os.path.isfile(canoe_cfg):
#self.log.info(f'CANoe cfg "{canoe_cfg}" found.')
self.__dispatch_canoe()
self.__canoe_app_obj.Visible = visible
#==========================================
try:
self.__canoe_app_obj.Open(canoe_cfg, auto_save, prompt_user)
except:
print ("Canoe already open")
#===========================================
#self.log.info(f'loaded CANoe config "{canoe_cfg}"')
self.__fetch_canoe_cfg_general_data()
#self.log.info('Fetched CANoe System Variables.')
else:
self.log.info(f'CANoe cfg "{canoe_cfg}" not found.')

I have updated code, Now if Canoe is running then also works def open(self, canoe_cfg: str, visible=True, auto_save=False, prompt_user=False) -> None: if os.path.isfile(canoe_cfg): #self.log.info(f'CANoe cfg "{canoe_cfg}" found.') self.__dispatch_canoe() self.__canoe_app_obj.Visible = visible #========================================== try: self.__canoe_app_obj.Open(canoe_cfg, auto_save, prompt_user) except: print ("Canoe is already Running") #=========================================== #self.log.info(f'loaded CANoe config "{canoe_cfg}"') self.__fetch_canoe_cfg_general_data() #self.log.info('Fetched CANoe System Variables.') else: self.log.info(f'CANoe cfg "{canoe_cfg}" not found.')

def open(self, canoe_cfg: str, visible=True, auto_save=False, prompt_user=False) -> None:
if os.path.isfile(canoe_cfg):
#self.log.info(f'CANoe cfg "{canoe_cfg}" found.')
self.__dispatch_canoe()
self.__canoe_app_obj.Visible = visible
#==========================================
try: self.__canoe_app_obj.Open(canoe_cfg, auto_save, prompt_user) except: print ("Canoe is already Running")
#===========================================
#self.log.info(f'loaded CANoe config "{canoe_cfg}"')
self.__fetch_canoe_cfg_general_data()
#self.log.info('Fetched CANoe System Variables.')
else:

could you take a look and suggest me how you have resolved this issue.
I am also facing similar issue.
https://github.com/chaitu-ycr/py_canoe/issues/8

I have updated code, Now if Canoe is running then also works def open(self, canoe_cfg: str, visible=True, auto_save=False, prompt_user=False) -> None: if os.path.isfile(canoe_cfg): #self.log.info(f'CANoe cfg "{canoe_cfg}" found.') self.__dispatch_canoe() self.__canoe_app_obj.Visible = visible #========================================== try: self.__canoe_app_obj.Open(canoe_cfg, auto_save, prompt_user) except: print ("Canoe is already Running") #=========================================== #self.log.info(f'loaded CANoe config "{canoe_cfg}"') self.__fetch_canoe_cfg_general_data() #self.log.info('Fetched CANoe System Variables.') else: self.log.info(f'CANoe cfg "{canoe_cfg}" not found.')

Where did u put this code ? I'm also having trouble to open 2 CANoe files at the same time


docs: update links to flowchart in 4.contribution.md

Links to flowchart lead to empty page if they contain a token.
Deleting the token fixes the problem.

My bad - just checked, that without token flowchart is displayed, but it's a wrong chart.
Then it's an issue - not a PR.
Please replace the token to correct one.

This link isn't correct - try viewing it in an incognito tab.
The hash contains the entire mermaid diagram and is required to display it.
I'd ideally like to include/render the mermaid diagram inline rather than linking out to another service - PR would be welcome, but might require needing to do some work in https://github.com/nuxt/nuxt.com to make that possible.


Not really an issue but can this be added to a CDN like https://cdnjs.com ?

See #32

i have looked at the above but i dont really understand how to go about getting this added to cdn js.. ? Am i missing something.. why would you not want to get this added? if i knew how i would definitely do it. thanks

https://github.com/cdnjs/cdnjs/blob/master/CONTRIBUTING.md


update RHEL8.2 repo to a new one (the old one is gone)

What about using http://download-ipv4.eng.brq.redhat.com/rhel-8/nightly/RHEL-8/latest-RHEL-8.2.0/?

What about using http://download-ipv4.eng.brq.redhat.com/rhel-8/nightly/RHEL-8/latest-RHEL-8.2.0/?

That was my first idea as well, but I think we don't like repositories that change content which would be the case for this one.

As it's only nightly repository and we are using it only for development, I don't care using changing repository. :)

As it's only nightly repository and we are using it only for development, I don't care using changing repository. :)

You are probably right. As opposed to Fedora, we don't have any test that would compare output images with known image-info, so it should be fine even though it changes.
@teg WDYT?

@teg made overriding repositories from the file system possible as part of #186: https://github.com/osbuild/osbuild-composer/pull/186/commits/32c6a40dd8219fe13e993c86b7f1cfdf46319256
He also changed it to the latest nightly (the symlink). I think we don't need this one.
Sorry!


Indicator plan export: Inconsistent treatment of text truncation/wrapping in columns

I'm noticing inconsistency in that some columns appear to be truncating text while others spread across neighboring columns. An example:

@KennyMonster what do you think about setting text to wrap instead of truncating or sprawling? Any risks, layout-wise? Could we get some wacky results if we do that?

Attached is an XLS with wrapped text (top aligned)
indicator_plan with wrapped text.xlsx

Deployment Verification

[x] Confirmed “Ready for Deploy” status
[x] Media Verifying:


Issues related to the next comfy frontend or the execution inversion PR

Main issue to gather any issues about the upcoming upstream merge in comfy:

the new UI
the execution inversion (should have 0 impact on mtb and on the contrary allow cleaner approach for some parts)

[ ] Litegraph object is no longer globally accessible (might be reintroduced)


weird issue when running the lint task

I'm getting the following when using implementation "com.google.android.gms:play-services-ads:15.0.1":

What went wrong:
Execution failed for task ':app:lint'.
Could not resolve all artifacts for configuration ':app:myAppDebugUnitTestCompileClasspath'.
Failed to transform file 'play-services-gass-15.0.1.aar' to match attributes {artifactType=android-exploded-aar} using transform ExtractAarTransform
GC overhead limit exceeded

Any ideas what can go wrong here?

I'm not sure what's going on here. I recommend reposting this in our support forum:
https://groups.google.com/forum/#!forum/google-admob-ads-sdk
We've got a team of support engineers that can dig in, try to replicate the issue, and see what's going on.


Render states

Keep track of the render states to reduce redundant driver state changes, it lazily updates them. For now it's only concerning the states we are modifying/would need to modify style-wise (blending/depth test/depth write), we could update with stencil if needed in future.

:+1:  Just wondering if using the () operator wouldn't be clearer - I recall that I first wondered what the assignment does (if it actually triggers an action)

Yes good suggestion! I'll make that change
On Mon, Jul 13, 2015 at 6:34 PM, Hannes Janetzek<EMAIL_ADDRESS>wrote:

:+1:  Just wondering if using the () operator wouldn't be clearer - I recall that I first wondered what the assignment does (if it actually triggers an action)
Reply to this email directly or view it on GitHub:
https://github.com/tangrams/tangram-es/pull/186#issuecomment-121080928

One thing to note in here, blending geometry may have blending with not all of the opaque layers, this is because non-ordering between opaque/non-opaque geometry.


Biggs and Magicite

The magicite screen shows that Biggs has mastered magicite when he hasn't learned the spells yet

This is an effective duplicate of issue #28 - I'm not sure if I can do anything about this... I'll keep this open though.


[fixed unknown query string theme] bug fix | unknown theme in query | Theme Not Found #34

Theme Not Found #34

Add validation for animation and layouts


Change api endpoint to frontend.binaryws.com

Coverage decreased (-0.5%) to 80.533% when pulling 745dac629790c1843187b4d7fe261b4e3a8da78a on aminmarashi:aminmarashi/change-api-endpoint-to-frontend into f5923c0d82b0c3a16a1e3797f489ca7775433470 on binary-com:master.

Coverage remained the same at 81.067% when pulling 7ca053fbbc8eb2ac87aef1e450216ef094eaa3b8 on aminmarashi:aminmarashi/change-api-endpoint-to-frontend into f5923c0d82b0c3a16a1e3797f489ca7775433470 on binary-com:master.


Cannot disable upload packing in the UI

Describe the bug
The UI displays an error when I try to save the config with the upload packing set to false.
Expected behaviour
Should be able to switch the upload packing on and off and save.
Additional context

General Information
Are you running a fork of renterd? N/A
Are you on the mainnet or on the testnet? N/A

I confirm, it works now!


⚠ Fortnite has degraded performance

In e966413, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1781 ms

Resolved: Fortnite performance has improved in a3708bc.


evaluate @State annotations from implemented interfaces (fixes #646)

In this PR @State annotations from default methods in implemented interfaces will be evaluated as well.
This makes the use case possible that is described in #646, thus allowing to modularize the provider tests for pacts with more than just a few interactions.

When implementing this, I noticed that multiple @State annotations with the same state don't create an exception. I would have expected an exception saying that the same @State is not allowed more than once. Probably either the first or the last method is then executed? Or both?
Shouldn't there be a validation that we have each state only once?

btw, if this is accepted, I'll gladly try to do the same for the junit5 module

Just checked the code with respect to multiple @State annotations, and I think that may have been by design. It will run all methods annotated with the correct state annotation.

I'll have a look at the junit5 module some time to implement the same functionality there, too.


[Features] improve scheduling performance on batch jobs

Is this a BUG REPORT or FEATURE REQUEST?:
more like feature, but bug when scheduling tasks with Critical Source.

Uncomment only one, leave it on its own line:

/kind feature
What happened:
1,  Looping checking the whole nodes of the cluster on each loop of scheduling one task
2, Allocated resource for part of tasks within one job, even the idle resource of the cluster can not satisfy all the tasks of the job.
What you expected to happen:
1, For action of  allocate as an example:
https://github.com/kubernetes-sigs/kube-batch/blob/c4896a41a061cd2e3d071fc01b1dd15df06b84ea/pkg/scheduler/actions/allocate/allocate.go#L112
before allocating  resources for the tasks  of one job, better filtering the nodes  and calculate  the idle of the nodes to check if they can satisfy the req resources of the whole job, if not just leave it in the queue and continue to handle the next one...
2, And for https://github.com/kubernetes-sigs/kube-batch/blob/c4896a41a061cd2e3d071fc01b1dd15df06b84ea/pkg/scheduler/framework/session.go#L144
Is it better to release the resources  of the tasks which are not bound to node  as the idle resources of the cluster can not satisfy the whole job?
Currently, it often  occurred that 2 jobs were pending even the idle resources of the cluster can satisfy the last submitted one.
How to reproduce it (as minimally and precisely as possible):
Suppose the idle resources of the cluster is: 100 cores, 100GB mem
1, submit  1st  job with 4 pods, each  req: 30 cores, 40GB mem
2, submit 2nd job with 4 pods, each req:  20 cores, 20GB mem
Anything else we need to know?:
Environment:
kube-batch:  master branch

/cc @k82cn

Is it better to release the resources of the tasks which are not bound to node as the idle resources of the cluster can not satisfy the whole job?

We need to care about starvation of "huge job" by "smaller job"; the other part is ok to me :)

/kind feature
/sig scheduling

@jiaxuanzhou

before allocating resources for the tasks of one job, better filtering the nodes and calculate the idle of the nodes to check if they can satisfy the req resources of the whole job, if not just leave it in the queue and continue to handle the next one

With this design, big jobs (i.e. a job with many tasks) are more likely to be starved because we keep allocating resources to smaller jobs. If we add a starvation prevention mechanism, big jobs will eventually be scheduled. But this behavior is not ideal because starvation prevention mechanism will not kick in unless the job has waited for long than specified starvation threshold.
PR 821 proposed a slightly different approach. We always allocate resources to big jobs. But if the big job is not ready to run, allocated resources will be released in backfill phase, and following smaller jobs will be scheduled as backfill jobs. In the following scheduling rounds, if the scheduler notices that the big job is conditionally ready, it will either 1) preempt backfill jobs and start big job right away (in preemption mode), or 2) disable backfilling, wait for backfill jobs to finish, and start the big job (in non-preemption mode). In either case, the big job is likely to start earlier.

l

looks good overall, looking forward the pr of backfill part, thanks 👍

There're two issue here: 1. fragment because of queue's algorithm 2. starvation. In this issue, we only need to handle first one :)

@jiaxuanzhou PR 805 implements backfill.


Parser::new_with_broken_link_callback() needs documentation / example

Can you provide an example of an implementation of Parser::new_with_broken_link_callback() in the docs?
Thank you

Right, it is missing and the docs and it would be useful. Meanwhile, you can view the example of main.rs:
https://github.com/raphlinus/pulldown-cmark/blob/1a5e54546b40d79eec8001d4e268b436571a78bb/pulldown-cmark/src/main.rs#L33
Thanks.

I ended up writing the following:
https://github.com/john-cd/mdbook-utils/blob/main/src/parser/mod.rs
(at the bottom)

An example was already in the examples directory, definitively a mdBook (#863) would provide more visibility for this kind of things: https://github.com/pulldown-cmark/pulldown-cmark/blob/master/pulldown-cmark/examples/broken-link-callbacks.rs


Moving config mesh utils to subpackage

This is part of the cleanup of pkg/config. Planning on moving everything here to subpackages.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: nmittler
To complete the pull request process, please assign ozevren
You can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

galley/OWNERS
istioctl/OWNERS [nmittler]
pilot/OWNERS [nmittler]
pkg/OWNERS [nmittler]
tests/OWNERS [nmittler]
tools/OWNERS [nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nmittler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
fcec0a6e12737018831b9d33faa0442e8c87dc8c
link
/test istio-racetest-master

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

/test istio-racetest-master

@nmittler: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-racetest-master
fcec0a6e12737018831b9d33faa0442e8c87dc8c
link
/test istio-racetest-master

integ-new-install-k8s-presubmit-tests-master
fcec0a6e12737018831b9d33faa0442e8c87dc8c
link
/test integ-new-install-k8s-presubmit-tests-master

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn, nmittler
To complete the pull request process, please assign ozevren
You can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

galley/OWNERS
istioctl/OWNERS [howardjohn,nmittler]
pilot/OWNERS [howardjohn,nmittler]
pkg/OWNERS [nmittler]
tests/OWNERS [howardjohn,nmittler]
tools/OWNERS [howardjohn,nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, howardjohn, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [geeknoid]
istioctl/OWNERS [geeknoid,howardjohn,nmittler]
pilot/OWNERS [geeknoid,howardjohn,nmittler]
pkg/OWNERS [geeknoid,nmittler]
tests/OWNERS [geeknoid,howardjohn,nmittler]
tools/OWNERS [geeknoid,howardjohn,nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


ISourceGenerator will be deprecated and the article should be updated

ISourceGenerator interface is about to be deprecated soon.

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

ID: 99a4410c-4c8d-0107-845b-9a951bb76682
Version Independent ID: f1ed9873-3e29-b4bd-2dbb-44dd9a05d790
Content: Source Generators
Content Source: docs/csharp/roslyn-sdk/source-generators-overview.md
Product: dotnet-csharp
Technology: csharp-roslyn
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Adding @KathleenDollard

ISourceGenerator related APIs were banned in https://github.com/dotnet/roslyn-analyzers/pull/6478


[Bug]: bad escape \m at position 11

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits

What happened?
A few minutes ago, EdgeGPT library stopped working on my Python project, it can't catch a response, the only error information I get is "bad escape \m at position 11", I'm aware of 0.4.4 update, I already upgraded, generated the cookies.json file again from bing.com on Edge and re-read docs, but it seems to be good from my end.
I know you're working really hard on this project, no pressure, just wanted to notice you and, by the way, congratulate you for your awesome work 😊
Steps to reproduce the problem

bot = await Chatbot.create(cookie_path='./settings/cookies.json')
response = await bot.ask(prompt=prompt, conversation_style=ConversationStyle.creative)

bad escape \m at position 11

What should have happened?
I should get an object with bot's response
Version where the problem happens
I've tested on 0.4.3 and 0.4.4
What Python version are you running this with?
I've tested on 3.10.10 and 3.11.3 python
What is your operating system ?
No response
Command Line Arguments
No

Console logs
bad escape \m at position 11

Additional information
No response

Windows? You guys have different paths iirc
Something like \\.settings\\cookies.json

I guess it was a temporary error from bing, I just wait 1 hour, tried again and it worked normally, so, that's it, thanks a lot

That's weird. I'll try to add some error handling

Yes, so weird, sorry for the issue report, its closed now, thank you for your work and your patience :)


New Website Logo: Artsy

Please fill the following out:

Name: Artsy
Homepage url: https://www.artsy.net
Brand Guidelines/Licensing: n/a
Logo: SVG https://cl.ly/0B3G071n0v2s

One of our OSS sample apps got featured in the “What’s new in TypeScript” talk at MS Build, so I guess we’re friends? :) http://twitter.com/alloy/status/862987779839586304

@alloy can you repost the logo? the link above does not seem to work any longer.

@mhegazy On it!

Here we go

thanks!


401 Unauthorized only returning

Good Afternoon,  I have been trying to implement this capability but i am having some trouble.
The routes are located in /routes/users.js
The passport.use(new ClientCertStrategy..... is located in auth/authroization.js
I am attempting to invoke the Client-cert strategy on the user.js line:
router.use('/user', auth.authenticate('client-cert'), function(req, res, next) {
console.log('authenticated');
next();
});
But all I get is a 401 error.
In the Authorization.js i have simplified it to return a user but still nothing:
passport.use(new ClientCertStrategy(function(clientCert, done)
{
var user = "test";
done(null, user);
}
));
module.exports = passport;
anysuggestions?

About 7 hours later i solved it.  In fact the service was working correctly, i had some other settings improperly set.


FStar.Comp and FStar.Relational are not ported to universes

FStar.Comp and FStar.Relational are not ported to universes

Hi ! Both seem to be in ulib:
ulib\FStar.Relational.Comp.fst
ulib\FStar.Relational.Relational.fst

Yes, they are there and they typecheck (since a couple of weeks).
Closing.


[WIP] xdg shell resize support for rootston

[x] There's an issue with get_input_bounds - not large enough for resize handles
[x] Properly implement resizing in cursor.c
[x] Offset issues when resizing
[ ] wlr_xdg_toplevel_v6_set_resizing

Resizing now works correctly. I ended up removing get_input_bounds (to let the client send resize requests) and add get_size (to send the correct size). Feedback is appreciated, not sure it's a good way to solve the issue.
We may want to change view_begin_move to use view_x et al.

Thanks! I think we can deal with input bounds separately.


Basic setup for Python interface using SWIG

Signed-off-by: Marcos Wagner<EMAIL_ADDRESS>🎉 New feature
Summary
Basic setup for a Python interface using SWIG to address #210 . (Related to https://github.com/ignitionrobotics/ign-math/issues/101) This creates a python module with the cpp classes with an associated .i file. To achieve dual interface output with Python and Ruby, the ruby.i and the python.i used at compilation had to be moved into separate subdirectories, to avoid collisions between the auxiliary files created by SWIG for Python and Ruby.
Test it
To test this interface

Build ign-math with colcon
Run export PYTHONPATH=/ws/install/lib/python
Open a Python interactive session running python3.
Inside of the interactive session run import ignition.math.
Library inside python ready  to use.

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[x] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Some of the changes I've pushed:

Install path was modified to install math.py in install folder which simplifies PYTHONPATH setting up. 3e31a3a
Module was renamed from pymath to math. 3e31a3a
Examples were added for Angle, Vector2, and Vector3 py classes.  f0ad4df
Fixes ruby_TEST after moving ruby.i to a subfolder fab308e
The configuration using Swig 3(bionic) and Swig 4(focal) is slightly different. Added support for both, however if we want to target focal it would be a bit simpler. 5fa9264

The interface has been implemented for Python using SWIG for the files that already had Ruby implementation. At the moment we have pushed tests for these files, to match the Ruby implementation.
Something to note is that when running add_test() for Python in the CMakeList.txt the --gtest_output=xml argument was not added to match the Ruby add_test(), because it was not being able to run it. Ruby is using the test-result folder inside the build directory as output for the xml produced, but no files are created when the colcon test cmd is run. Even after these two cases, running colcon test-result outputs the correct amount of passed, errors, failures and skipped tests (including the Python and Ruby ones)

I've just left some minor comments however It is in a good shape to be reviewed @chapulina.
One comment, there is no a defined style for python in the ignition contribution, so let us know if we must align to a different style.

Originally there was a file called ign_math.i listing the files that SWIG would use to create a Ruby interface. We now changed it into two different files, a python.i file and a ruby.i file as we won't have the same list of files for both languages.

This PR should fix ubuntu CI https://github.com/ignition-tooling/release-tools/pull/496

I think the following will fix the ABI checker: https://github.com/ignition-tooling/release-tools/pull/497

looks like the only remaining CI issue is that 9e2dafc3cbd087147a22e6152ddc9dd5eb8e375b needs to be signed @francocipollone

the test failure in the GitHub actions occur because make test is run without make install. We created the FAKE_INSTALL target in the test folder to support testing without make install. I've updated the python tests to use FAKE_INSTALL_PREFIX in 3e9109a. It required moving the definition of the FAKE_INSTALL_PREFIX variable to the root CMakeLists.txt in order to be accessed by both the src and test folders, which seems acceptable to me

I see, thanks @scpeters for both the patch and the explanation.

I just pulled the latest ign-math6 branch and got into a weird compile error:
set_target_properties Can not find target to add properties to: _pymath
Full log

[0.020s] Invoking command in '/home/chapulina/dev_bionic/ws_fortress/build/ignition-math6': /usr/bin/cmake /home/chapulina/dev_bionic/ws_fortress/src/ign-math -DBUILD_TESTING=true -DCMAKE_INSTALL_PREFIX=/home/chapulina/dev_bionic/ws_fortress/install
[0.062s] -- The C compiler identification is GNU 8.4.0
[0.102s] -- The CXX compiler identification is GNU 8.4.0
[0.111s] -- Detecting C compiler ABI info
[0.150s] -- Detecting C compiler ABI info - done
[0.159s] -- Check for working C compiler: /usr/bin/cc - skipped
[0.159s] -- Detecting C compile features
[0.159s] -- Detecting C compile features - done
[0.161s] -- Detecting CXX compiler ABI info
[0.200s] -- Detecting CXX compiler ABI info - done
[0.209s] -- Check for working CXX compiler: /usr/bin/c++ - skipped
[0.209s] -- Detecting CXX compile features
[0.210s] -- Detecting CXX compile features - done
[0.215s] -- ignition-math6 version 6.8.0
[0.215s] -- Operating system is Linux
[0.218s] -- Found CPack generators: DEB
[0.223s] -- Looking for eigen3 - found
[0.223s]
[0.232s] -- Searching for swig - found.
[0.696s] -- Searching for Ruby - found.
[0.704s] -- Searching for Python - found.
[1.374s] --
[1.375s] -- Searching for host SSE information
[1.413s] -- SSE2 found
[1.413s] -- SSE3 found
[1.413s] -- SSE4.1 found
[1.413s] -- SSE4.2 found
[1.422s] -- Configuring library: ignition-math6
[1.423s] -- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
[1.472s] -- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
[1.472s] -- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY
[1.520s] -- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY - Success
[1.520s] -- Performing Test COMPILER_HAS_DEPRECATED_ATTR
[1.570s] -- Performing Test COMPILER_HAS_DEPRECATED_ATTR - Success
[1.572s] -- Adding 43 UNIT tests
[1.593s] -- Adding 6 UNIT tests
[1.614s] CMake Error at src/CMakeLists.txt:120 (set_target_properties):
[1.614s]   set_target_properties Can not find target to add properties to: _pymath
[1.614s]
[1.614s]
[1.614s] CMake Error at src/CMakeLists.txt:126 (target_compile_options):
[1.614s]   Cannot specify compile options for target "_pymath" which is not built by
[1.614s]   this project.
[1.615s]
[1.615s]
[1.615s] CMake Error at src/CMakeLists.txt:131 (install):
[1.615s]   install TARGETS given target "_pymath" which does not exist.
[1.615s]
[1.615s]
[1.632s] -- Adding 1 INTEGRATION tests
[1.648s] -- No tests have been specified for PERFORMANCE
[1.663s] -- No tests have been specified for REGRESSION
[1.681s] -- Configuring library: ignition-math6-eigen3
[1.682s] -- Adding 1 UNIT tests
[1.720s] -- Adding codecheck target
[1.720s] -- Build configuration successful
[1.720s] -- Build type: RelWithDebInfo
[1.721s] -- Install prefix: /home/chapulina/dev_bionic/ws_fortress/install
[1.722s] -- Looking for ronn to generate manpages - found
[1.759s] -- Found Doxygen: /usr/bin/doxygen (found version "1.8.13") found components: doxygen missing components: dot
[1.801s] -- Configuring incomplete, errors occurred!
[1.801s] See also "/home/chapulina/dev_bionic/ws_fortress/build/ignition-math6/CMakeFiles/CMakeOutput.log".
[1.802s] See also "/home/chapulina/dev_bionic/ws_fortress/build/ignition-math6/CMakeFiles/CMakeError.log".
[1.807s] Invoked command in '/home/chapulina/dev_bionic/ws_fortress/build/ignition-math6' returned '1': /usr/bin/cmake /home/chapulina/dev_bionic/ws_fortress/src/ign-math -DBUILD_TESTING=true -DCMAKE_INSTALL_PREFIX=/home/chapulina/dev_bionic/ws_fortress/install

I'm not sure yet how to fix my workspace. Looking into it

I just pulled the latest ign-math6 branch and got into a weird compile error:
set_target_properties Can not find target to add properties to: _pymath
Full log
I'm not sure yet how to fix my workspace. Looking into it
Update
Solved for now with sudo apt purge .*swig.*

what OS and swig version do you have?

what OS and swig version do you have?

This happened on Bionic, swig/bionic 3.0.12-1 amd64. I see this is the same version installed in the successful CI build. I can try to reproduce in a clean container later.

So removing swig and installing it again did the trick?

Not exactly :sweat_smile:  Removing SWIG just allowed me to skip that entire block of code, so I didn't run into the problem and could compile the core library. Reinstalling SWIG brings back the issue to me.

My problem is the CMake version, trying a few things here: https://github.com/ignitionrobotics/ign-math/pull/223


docs: fix typo for menu

fix typo for lefthand menu

Summary
fix typo for lefthand menu
Details
fix typo for lefthand menu

Thnks for the fix @marconato7


Internal Server error... write: broken pipe

Summary
I'll be honest, this is a tough one to reproduce. After 24-48 hours of running, our argo-servers start throwing logs with this repeated over and over again:
time="2021-03-25T10:01:09.560Z" level=error msg="finished unary call with code Unknown" error="write tcp <IP_ADDRESS>:50542-><IP_ADDRESS>:5432: write: broken pipe" grpc.code=Unknown grpc.method=ListWorkflows grpc.service=workflow.WorkflowService grpc.start_time="2021-03-25T10:01:0

From a UI perspective, no users can see any workflows, they just see this error at the top of the screen:

The only fix we have found is to kill the argo-server pods and let them recreate themselves.
Diagnostics
What Kubernetes provider are you using? EKS-1.19
What version of Argo Workflows are you running? 3.0 rc9
We are using SSO to gihub (via argocd)

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

This just happened again. The server pods had been up for 25h

Are you able to indicate what the two IPs are?

Postgres?

hmm that would have been helpful wouldn't it?
Definitely not postgres, that's hosted externally to the cluster. I imagine it's just the other pods in the argo namespace (we run 2 servers and one controller). Next time it dies I'll trace the IPs.

Ah, you might be right to a degree...
So we use EKS and use an RDS Postgres db. To access the postgres DB, the pod gets allocated a secondary IP address - that's probably the one with :5432 on it. Summarised here: https://aws.amazon.com/blogs/containers/introducing-security-groups-for-pods/
So it's likely that both IP addresses are for the pod that's complaining. But I will confirm next time it happens.

So we're probably getting disconnected and not reconnected. No other users have complained about this, and this code is >= 1y6m old. I expect it maybe something specific with your set-up. Do you know if RDS drops connections?

It's not something I'm aware of but I will increase monitoring and get back to you.

Your assumption was correct. RDS was helpfully disconnecting. I believe I have addressed this now, even if I haven't, the database is at fault, not argo-workflows. Thanks.

Your assumption was correct. RDS was helpfully disconnecting. I believe I have addressed this now, even if I haven't, the database is at fault, not argo-workflows. Thanks.

Quick question what did you change on your DB side to fix this?

@alexec We are expiriencing a similar issue. Our connection to Postgres is routed via service mesh and looks like the connection is periodically killed.
I know that a lot of postgres drivers can re-establish a connection, any reason it is not happening in our case. The error is: broken pipe


refact: use async mode for computing task & fix some bugs

problems list:

kube conf only support absolute path (fixed)
Computer Task belongs to SysJob ( fixed)
Computer Task type is wrong,  isComputerTask return false (fixed)
cancel API return success but not work indeed  (TODO)

And due to update hugegraph-computer (k8s api to async method), we need update computer dependency first then the ci could run

compile error:
Error:  Failed to execute goal on project hugegraph-core: Could not resolve dependencies for project com.baidu.hugegraph:hugegraph-core:jar:0.12.0: com.baidu.hugegraph:computer-k8s:jar:0.1.1 was not found in https://repo.maven.apache.org/maven2 during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]

compile error:
Error:  Failed to execute goal on project hugegraph-core: Could not resolve dependencies for project com.baidu.hugegraph:hugegraph-core:jar:0.12.0: com.baidu.hugegraph:computer-k8s:jar:0.1.1 was not found in https://repo.maven.apache.org/maven2 during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]

wait for computer v0.1.1 version, now ci is OK


Fix issue 455

Task/Issue URL:  #455
Steps to test this PR:

Launch a search query.
Scroll up and down such that the navigation bar is hidden and then revealed; verify that the navigation bar does not clip into the action bar.
Test other UI elements to make sure nothing else is broken.

Given the intermittent nature of this problem, I cannot say unequivocally that these changes will solve the problem.  What I can say, however, is that on my Pixel XL 9.0 I was always able to reproduce this bug within fifteen seconds, and after the changes I have been thoroughly abusing the app for days and have not seen any trace of this bug.

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks for submitting this @JeffAment. I never managed to reproduce the original bleed issue you reported but would have accepted this change, however it introduces a new issue. When switching from landscape to portrait the omnibar can no longer be scrolled away.
To reproduce:

Launch the app in landscape
Enter a search
Rotate to portrait
Scroll to make the omnibar disappear, it is now glued in place

Thanks for being perceptive and finding this bug, @subsymbolic!  I have not been able to reproduce this on my device and two emulators, and was wondering if you could tell me what kind of device this happened on?  I would be happy to look into this a bit more to see if I can find a solution that will work for both, but at the same time I perfectly understand if this has become a bigger time investment for you than you had originally anticipated.  If this is the case, please feel free to close this PR as I'd hate to ask more of your time and take you away from solving more important issues.

It happens on a Pixel 3 XL running Android 9. In case it helps, I have attached a gif of the issue.

@JeffAment I'm closing this PR however you are welcome to open it again if the no-scroll issue on Pixel 3 is fixed.

@subsymbolic,
Sorry for the late reply!  I normally try and be more prompt than this, but had some unexpected personal issues arise.  I was not able to reproduce the issue that you had reported, as I do not have access to a Pixel 3 XL or the ability to emulate it.  I did try emulating devices with the same screen resolution and with notched screens, but was still not able to reproduce.  As of now, I do have some ideas that may work (and one that I believe will most likely work, but is not an ideal fix as it would add unnecessary overhead), but it is not fair to ask anyone to review a possibly endless stream of PR's that are based off complete guess work.  With that being said, I will keep this in mind and will open another PR should I ever find a way to reproduce this.  Conversely, I am also willing to test anyone elses future PR's (or even work in collaboration with anyone who can reproduce the Pixel 3 XL bug) if someone else should attempt to fix this, so please feel free to ping me should you ever need someone to test a fix for this, since I can very easily reproduce the original bug that I documented on my device.


Can't disallow space in shorthand function declaration

With the following config:
{
	"disallowSpacesInFunctionDeclaration": {
		"beforeOpeningRoundBrace": true
	},
	"requireSpacesInAnonymousFunctionExpression": {
		"beforeOpeningRoundBrace": true
	},
	"esnext": true
}

The following code outputs a warning:
{
	foo() {
	}
}

The warning: "Missing space before opening round brace".
It seems jscs considers the function as anonymous. Maybe a new option is needed for shorthand function declaration?
Might be linked to https://github.com/jscs-dev/node-jscs/issues/1890

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28127235-can-t-disallow-space-in-shorthand-function-declaration?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

I guess this is one of the most confusing addition to ES6, those methods, are in fact anonymous functions, see discussion in - https://github.com/jscs-dev/node-jscs/issues/1671#event-438122364
If you would like to send us pull to include exceptions to this rule, we would welcome it.


Embedded regex language provider support does not seem to escape regex constructs correctly

Hi, currently using c# extension version 1.23.2 and testing embedded language providers, mainly regex.
If I type that somewhere in code Regex.Match("test", ""); and try to trigger suggestions in the empty quotes of second argument, I see that the regex provider is triggered. Then, if I try to select some backslash suggestion like \s, what I get after pressing enter is just "\s". However, because the string is non verbatim, it should be escaped as "\s"`.
In addition, if I do the same but trigger completion after first typing two backslashes and then select something like \A, I get "\\\A" instead of just "\A".
I believe it does work in vs proper, so reporting it here.
CC: @333fred

We likely need to resolve the edit for this eagerly as well, like we do for doc comment tags.

@JoeRobich can you move this to omnisharp-roslyn?


⚠ TROJAN 🇺🇸 United States USF 1 has degraded performance

In fcba91d, TROJAN 🇺🇸 United States USF 1 (https://ust-1.opensvr.net/) experienced degraded performance:

HTTP code: 200
Response time: 14208 ms

Resolved: TROJAN 🇺🇸 United States USF 1 performance has improved in e1ecf3a.


minimumZoomDistance requires call to setView?

This has no effect:
viewer.scene.screenSpaceCameraController.minimumZoomDistance = 1;

But this does:
viewer.scene.screenSpaceCameraController.minimumZoomDistance = 1;
viewer.camera.setView({
    position : Cesium.Cartesian3.fromDegrees(-71.1106, 42.3736, 5000.0)
});

See https://groups.google.com/forum/#!topic/cesium-dev/7x8pHXBHxvk

This is no longer an issue.

Cool, can you respond to the forum post: https://groups.google.com/forum/#!topic/cesium-dev/7x8pHXBHxvk


Eliminate async creation of i18next instance

Closes #72

As for the client, i will improve the handing in another PR #58