v3.0.8 might break nextjs 14 (14.1.4) middleware

I think v3.0.8 might be breaking my middleware for nextjs 14.1.4 as per this issue I raised in sst: https://github.com/sst/ion/issues/870 .  It might be breaking when a node api is detected by nextjs 14.
In that issue I was getting  a cloudwatch error:
WARN	Routing failed. TypeError: globalThis.__import_unsupported is not a function
    at 8915 (file:///var/task/middleware.mjs:17:3341)
    at s (file:///var/task/middleware.mjs:17:1473)
    at 2818 (file:///var/task/middleware.mjs:27:603)
    at s (file:///var/task/middleware.mjs:17:1473)
    at 6872 (file:///var/task/middleware.mjs:27:1308)
    at s (file:///var/task/middleware.mjs:17:1473)
    at 9693 (file:///var/task/middleware.mjs:27:279)
    at s (file:///var/task/middleware.mjs:17:1473)
    at 9108 (file:///var/task/middleware.mjs:25:77770)
    at s (file:///var/task/middleware.mjs:17:1473)

If I look at the changes in the release: https://github.com/sst/open-next/compare/v3.0.7...v3.0.8 , the one that seems most germane to the error is the for packages/open-next/src/build/edge/createEdgeBundle.ts  which is this commit: https://github.com/sst/open-next/commit/75857cf11906b010ae34c0575e89f9108041556b
I could be wrong.  But I think it's related because it appears to be working around something to do with setting __import_unsupported .
I've tested by deploying with<EMAIL_ADDRESS>and middleware is nominally working for me in that version. (I tested by setting openNextVersion: "3.0.7" in my sst.config.ts .
cc @conico974 - don't know if you can verify this somehow?

That's very likely where the error is yeah.
The __import_unsupported function are supposed to be there to replace module that use node at runtime (to throw an exception).
If you could create a repro that would help a lot, or at least just find which of your import has a dependency on node so that i can test both with next 15 and 14.
Recreating the issue is probably harder than the fix itself here.

@conico974 sorry, I'll try to replicate with a vanilla setup.  Incidentally, when I install sst using npm , the sst cli is not present in node_modules/.bin/sst.  Although in my actual project there is one.  Do you know how to get a local cli sst command installed from an npm package or similar?

seems like npm i<EMAIL_ADDRESS>won't set up a cli in node_modules/.bin but npm i sst will.

Apologies, I've been struggling to get the time to do the repro.  I can see a possible fix in the above PR 504 .  Happy to test that change if you want (not sure how to get it into my setup)?

@danielbush You could do a pnpm patch with the fix in the PR (without the ! as mentionned in the review).
You'll have to install OpenNext and update the buildCommand args in the ion construct to use the locally installed OpenNext.

@danielbush It should be fixed by #504 in 3.1.2
If you still have the issue, just post a comment here, i'll reopen the issue


fix location of deprecation warnings

fixes https://github.com/chef-cookbooks/compat_resource/issues/34.
The logic to find the location of cookbook code already works well.
But the location found is assigned to new_location, which is not used by super class.

:+1:
/cc @jkeiser @pburkholder


ServerSide wrapper

Is there a way to implement a (let's call it) ServerSide object (similar to Option) to tell diesel that a particular field should be ignored from any insert/update?
The example from the guide would then become:
#[derive(Queryable, Insertable)]
pub struct Post {
    pub id: ServerSide<i32>,
    pub title: String,
    pub body: String,
    pub published: bool,
}

and then use it in insert as:
    let new_post = NewPost {
        id: None // Or another special value
        title: title,
        body: body,
        published: false
    };

    diesel::insert(&new_post).into(posts::table)
        .get_result(conn)
        .expect("Error saving new post")

The same could be applied to other fields like created_at and updated_at.
See also: #860 and this reddit discussion

What is your reasoning to make this a wrapper struct and not an attribute? Am I missing something or is this a subset of #860 (and thus a dupe of it)?

Am 03.11.2017 um 01:48 schrieb Hernan Grecco<EMAIL_ADDRESS>Is there a way to implement a (let's call it) ServerSide object (similar to Option) to tell diesel that a particular field should be ignored from any insert/update?
The example from the guide would then become:
#[derive(Queryable, Insertable)]
pub struct Post
{
pub id: ServerSide
,
pub title: String
,
pub body: String
,
pub published: bool
,
}
and then use it in insert as:
let new_post = NewPost {
    id: None // Or another special value
    title: title,
    body: body,
    published: false
};

diesel::insert(&new_post).into(posts::table)
    .get_result(conn)
    .expect("Error saving new post")

The same could be applied to other fields like created_at and updated_at.
See also: #860 and this reddit discussion
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


measurementFill property doesn't change Centile Chart point colour

Hi Team,
Have noticed that specifying the measurementFill property on measurementStyle doesn't show the colour change for the chronological age or corrected age measurement points on the Centile chart.
Looking at the source code (src/CentileChart/CentileChart.tsx), it seems that the colour is hardcoded to black for corrected age measurement (Line 683) and that the measurementFill doesn't filter through to either the chronological age measurement point or the corrected age measurement point.
Using version 6.1.4.
Thanks in advance,
Daniel

Thanks @dmc-cambric for finding this - fix applied and new version pushed. Note latest charts now are 6.1.10


[trivial] Fix counting tests in parallel loop

It was using non-atomic increment which can suffer from data races.

Why does docker cloud not even show up? Hmm.

Why does docker cloud not even show up? Hmm.

Maybe they banned you

For life


Cannot able to change back button in ApptentiveSurveyViewController

Hi, I can able to access ApptentiveSurveyViewController in my application in older version of apptentive and change the back button image. But with latest version I cannnot able to access ApptentiveSurveyViewController in my app. Can someone please tell me how can I change the back button image in latest version

We don't expose the ApptentiveSurveyViewController header file publicly, but you might be able to add it to your header search paths. This would depend on your method of integration (CocoaPods/Carthage/Subproject).
I should also mention that the navigation controller subclass used for Apptentive's UI (ApptentiveNavigationController) is exposed in the public headers. For UIAppearance targeting, this can be a good option.


[Bug]: ESM Support "Use ESM Presets" documentation has confusing redundancy

Version
29.2
Steps to reproduce
This is a documentation bug.  The Use ESM presets section appears to have a redundant transform config in the example.  Perhaps I am misunderstanding, but it looks like it was copied from the Manual configuration section.
From looking at the ts-jest/presets/default-esm preset code, it seems to essentially resolve to:
{
    extensionsToTreatAsEsm: ['.jsx', '.ts', '.tsx', '.mts'],
    transform: {
      ['^.+\\.m?tsx?$']: [
        'ts-jest',
        {
          useESM: true,
        },
      ],
    },
  }

This makes me question why the example that uses this default also includes:
  transform: {
    '^.+\\.tsx?$': [
      'ts-jest',
      {
        useESM: true,
      },
    ],
  },

If I am not mistaken I would be happy to make the updates to:
https://github.com/kulshekhar/ts-jest/blob/main/website/docs/guides/esm-support.md
Expected behavior
I would expect the example configs to look like this:
const jestConfig: JestConfigWithTsJest = {
  // [...]
  preset: 'ts-jest/presets/default-esm', // or other ESM presets
  moduleNameMapper: {
    '^(\\.{1,2}/.*)\\.js$': '$1',
  },
}

Actual behavior
The example configs look like:
const jestConfig: JestConfigWithTsJest = {
  // [...]
  preset: 'ts-jest/presets/default-esm', // or other ESM presets
  moduleNameMapper: {
    '^(\\.{1,2}/.*)\\.js$': '$1',
  },
  transform: {
    // '^.+\\.[tj]sx?$' to process ts,js,tsx,jsx with `ts-jest`
    // '^.+\\.m?[tj]sx?$' to process ts,js,tsx,jsx,mts,mjs,mtsx,mjsx with `ts-jest`
    '^.+\\.tsx?$': [
      'ts-jest',
      {
        useESM: true,
      },
    ],
  },
}

Debug log
N/A
Additional context
No response
Environment
N/A

PR is welcome
You should use utility functions like createDefaultEsmPreset for the example codes in the documentation.
I'm working on #4433 to adjust the preset page to inform about switching to use utility functions as best practices


Fix broken init time attempt

Instead of crashing when some NWP variables are missing, we reindex the dimension, effectively adding NaN for the missing variables.

Opened by mistake!


undefined local variable or method flash' in 0.10.3

I got this after generating a new project from scratch. I finally tracked the error down to the automatically generated Gemfile:
gem 'sinatra-flash'

should be
gem 'sinatra-flash', :require => 'sinatra/flash'

After making that change, everything worked great.

Absolutely right and fixed here: https://github.com/padrino/padrino-framework/commit/f4014fca5a6e706e88d110e9321459c208c06582


bug: hererocks build failed because "python: command not found"

Did you check docs and existing issues?

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

Neovim version (nvim -v)
NVIM v0.10.0
Operating system/version
22.04.1-Ubuntu
Describe the bug
After fresh installation, I can see the following error:
    ● hererocks 0.07ms ✔ build
        /bin/bash: line 1: python: command not found

It looks like the offending line happens here: https://github.com/folke/lazy.nvim/blob/9005e8ede7ff9e6434818c32d99860d7154d0432/lua/lazy/pkg/init.lua#L124
I think we'll need to check the availability of python before we start, and many people will have python3 in their path instead of python.
Steps To Reproduce
Install neovim with latest version of lazy. Open an empty file, lazy screen will pop up showing the aforementioned error.
Expected Behavior
Nothing should appear.
Repro
No response

Also, after I've symlinked python3 to python locally, the build failed again. This time because missing readline/readline.h:
luaconf.h:275:10: fatal error: readline/readline.h: No such file or directory

I had to install it on my machine to finally get the plugin built:
sudo apt-get install libreadline-dev

oh true lua headers can be another issue
🤔 at this point it might be worth writing something just to install luarocks on every system either with different tools and package managers or just binaries with sth similar to mason

Having the same issue when hitting Update for all plugins

Sigh, luarocks really is pretty fragile...
On Windows it's even worse.
Will make some changes...

What if I'd just disable automatic luarocks packages completely?
And then if a certain plugin needs luarocks, you need to manually add {"foo/bar.nvim", build = "rockspec"}

that just makes it a pain for plugin authors and users ig
here people can work together to make a nice solution which works universally
otherwise this will make plugin authors think twice if they really want to use luarocks (or just be a pain for the ones who need things from there) and also requires users of such plugins to install things just themselves

What if I'd just disable automatic luarocks packages completely? And then if a certain plugin needs luarocks, you need to manually add {"foo/bar.nvim", build = "rockspec"}

I think that's a sensible approach. I understand the intention, but I personally feel as a neovim plugin manager, we should not try to install a lua package manager especially when no binary distribution is provided for the various platforms we strive to support.

that just makes it a pain for plugin authors and users ig
A plugin can just add a lazy.lua file with the build = "rockspec".

The biggest issue right now is plugins that also have a rockspec where they define dependencies to other Neovim plugins.
There's currently no way I can resolve these deps to the correct github paths. I have no idea how to do that (without using luarocks). Parsing a rockspec is easy though.
Like this one https://github.com/mfussenegger/nvim-dap-python/blob/master/nvim-dap-python-scm-1.rockspec

Is there a workaround to disable building hererocks for now? I don't need this functionality, but I really need to get Lazy working again.

Is there a workaround to disable building hererocks for now? I don't need this functionality, but I really need to get Lazy working again.

Hi! Also faced with this issue. These instructions worked for me:

So try to set lazy.opts.rocks.enabled to false

In case this helps:
In rocks.nvim, we have a bootstrap script that works quite well on unix and doesn't depend on hererocks.
We also have luarocks itself as a dependency (you can install it with luarocks), so we can pin the version.
Our bootstrap script doesn't work with non cygwin Windows yet. It'll probably use the install.bat when it does eventually.
It's probably a good idea to silently disable luarocks packages if no luarocks executable is found and it can't be bootstrapped.

Is there a workaround to disable building hererocks for now? I don't need this functionality, but I really need to get Lazy working again.

Hi! Also faced with this issue. These instructions worked for me:

So try to set lazy.opts.rocks.enabled to false and issue should be fixed:

Thanks, this works for me

With hererocks disabled:

hererocks enabled:

Similar, but when lazy tries installing a plugin:

These errors should probably be more than enough for users?

I think this is a big improvement
the best solution imo would be to try bootstrapping it and disabling it if not found like mrcjkb suggested here https://github.com/folke/lazy.nvim/issues/1548#issuecomment-2188564402
Because ig that will work for a lot of users (even more if we'd extend the script to use hererocks as a fallback) which would just lead to less failed installs

Can't even get luarocks to work with my mingw installation in Widnows.
Had to install a bunch of additional things with scoop, but still fails with a non-descriptive error:
Warning: Failed searching manifest: Failed downloading https://nvim-neorocks.github.io/rocks-binaries/dev/manifest-5.1 - failed downloading https://nvim-neorocks.github.io/rocks-binaries/dev/manifest-5.1
Cloning into 'nvim-dap'...
sed: Bad locale: LC_CTYPE=C.UTF-8
sed: Bad locale: LC_CTYPE=C.UTF-8
sed: Bad locale: LC_CTYPE=C.UTF-8
C:/Program Files/Git/mingw64/libexec/git-core\git-submodule: line 613: cmd_: command not found
    Error: Failed installing dependency: https://luarocks.org/dev/nvim-dap-scm-1.rockspec - Failed to fetch submodules.

Does anyone know how I can find the rockspec for a rock name?

When I restart Neovim, luarocks also sometimes won't launch at all.
Then restarting a couple of times and it works again.
With "Won't launch" I mean vim.uv.spawn returns nil
So weird.

The whole luarocks ecosystem is a mess.
tbh, I've never used a package manager that works as terrible as this.
The big issue is that since I've added support to lazy, I will now need to support that mess.
I'll leave it as is for now, but I'm strongly considering to make luarocks opt-in.

Plugins should really just use a lazy.lua file to specify their dependencies and their configuration.
Nothing else is needed.
With luarocks it's not even possible for a plugin to declare that it need setup or that it can be lazy-loaded.
luarocks as a dependency in a lazy.lua file could still make some sense and will work, but not for plugins

I think the most important thing to be gained from luarocks is installing things which aren't neovim plugins like toml or e.g. fzy from luarocks
would that still work from a lazy.lua file?

Does anyone know how I can find the rockspec for a rock name?

Example for neorg 8.7.1-1
https://luarocks.org/manifests/neorg/neorg-8.7.1-1.rockspec
You can find versions in the manifest:
https://luarocks.org/manifest-5.1

@mrcjkb that requires the user name of the package to be known as well, but that's something that I don't know.
That nvim-dap-python package for example depends on nvim-dap.
I would like to be able to ideally resolve nvim-dap to its GitHub repo.
If a Neovim plugin only has a simple build step and depends on only neovim plugins, then I can skip luarocks altogether in that case.

I think the most important thing to be gained from luarocks is installing things which aren't neovim plugins like toml or e.g. fzy from luarocks
would that still work from a lazy.lua file?

That use case makes sense and this should of course still work.
I just want to prevent needing to use luarocks for things like telescope, plenary, nvim-dap and others that don't really need luarocks to work.
Ideally luarocks should only be used when needed.

Sweet! Thanks!

I found an error
when I rebuild neorg plugin with command lazy build neorg

checkhealth:

But I have installed neorg in lazy-rocks with previous version lazy.nvim

And the neorg plugin works, How to fix this wierd thing ?

@noahlias the issue is that Homebrew by default installs luarocks with lua 5.4.
I presume the previous version of lazy.nvim didn't check for the correct lua version yet.
You might be able to get it working with the homebrew formula for luajit (the lua@51 formulae has been disabled).
Unfortunately, the homebrew luajit formulae only provides a luajit binary and no lua symlink to it (@folke this may be relevant info for lazy.nvim).

So should I just make this a warning then? Instead of an error?

So should I just make this a warning then? Instead of an error?

It ought to be safe to check for either lua with version 5.1 or luajit (the version shouldn't matter for luajit).
The reason luarocks needs Lua is so that it can compile binary rocks (e.g. written in C but compiled to Lua) against the appropriate Lua headers.
And to run those with Neovim's luajit interpreter, they need to be compiled against either lua 5.1 or luajit.

@noahlias the issue is that Homebrew by default installs luarocks with lua 5.4.

fun side note
it's impossible to install lua 5.1 with homebrew because apparently it's outdated
there was a lot of discussion when this happened in the repo because there are many projects using 5.1
to get it you have to do your own patch or tap or however they call that

Allright, made some progress.
I just deployed a new Github action that fetches the luarocks root manifest and then for each rockspec that contains nvim in the name, it fetches the rockspec.
It then creates the mappings between rockspec names and urls or github short urls in lua/lazy/community/rockspec.lua
See for example here https://github.com/folke/lazy.nvim/commit/bc7bfb5d361cbb95937876eb5171bb78c1f6b4ee
My plan is to then use those mappings to resolve dependencies and only use luarocks if one of the dependencies is not a nvim rock or if it has a complex build step.

How about disable rockspec by default and let the plugin developer explicitly add build=rockspec in their repo’s lazy.lua?
I feel like current approach (fetch from luarocks manifest and use heuristic way to decide) is a little bit over-engineering and would still leave false positives?

Take a look at https://github.com/rockerBOO/awesome-neovim and find me three plugins without nvim in the name :)
It also doesn't really matter, since lazy is capable of installing any luarock url.
So I could effectively parse the full manifest and take over all dependency resolving on the lazy side, and just resort to luarocks for complex builds, but that's not really needed and would be too much.
The issue I'm solving is where some Neovim plugins are listed as a dependency in a luarock, but might also be in your lazy spec somewhere. Currently that would be two different things on your package path. This can potentially lead to issues.
Pretty much all of the Neovim plugin rockspecs I looked at, had either no dependencies or only Neovim plugin dependencies.
So for those cases, the rockspec should only be used to determine dependencies and let lazy handle the rest.
For more advanced plugins like Neorg, luarocks would still be used for some of its dependencies, like the tree-sitter-norg stuff.

I didn't realize how messy this was. It is obvious when looking at a plugin that it uses a Lua rock?

yes it is because of the rockspec

Take a look at https://github.com/rockerBOO/awesome-neovim and find me three plugins without nvim in the name ツ

So cmp basically. I'll add those as well

well this will also be the case for the completion plugin I'm currently working on (and perhaps there are also other things as well)

Just one plugin https://github.com/folke/lazy.nvim/commit/aff7ee8e8983947abab94a37aa3eec57f6fc3f6b

But again, that doesn't really matter that much. It will work nonetheless

The new code is live.
For neorg, you'll now see all it's depenencies and their depedencies as separate plugins in :Lazy.
Only the tree-sitter-norg one will be built by luarocks. The others are all managed by lazy.

My issue with nvim-dap-python and some others is now also resolved...

just one thing (would take too much time to go figure that out in the code)
what happens if two plugins have the same dependency with different versions?

also just for clarification because of the recent changes and discussions about opt-in or opt-out
lazy does by default now parse pack-spec, rockspec and lazy.lua
rockspec dependencies are installed like normal plugins if they don't have special build steps. Only if they have special build steps they need the luarocks binary correctly installed
If there are such plugins luarocks will be tried to be used by default ootb without any additional user config (but could be disabled)
is that correct?

just one thing (would take too much time to go figure that out in the code)
what happens if two plugins have the same dependency with different versions?

Good point, versions are currently silently ignore. ha.
Still need to add that part, but will add an option to disable rockspec versions.

rockspec dependencies are installed like normal plugins if they don't have special build steps. Only if they have special build steps they need the luarocks binary correctly installed

If there are such plugins luarocks will be tried to be used by default ootb without any additional user config (but could be disabled)

Yes, correct.
With the latest changes, if a user didn't need luarocks before v11, they won't need it now. That was the main thing I wanted to fix.

The whole luarocks ecosystem is a mess.
tbh, I've never used a package manager that works as terrible as this.

I didn't realize how messy this was. It is obvious when looking at a plugin that it uses a Lua rock?

To be fair, the luarocks ecosystem itself isn't a mess. It just isn't intended to be used in a way that doesn't treat lua rocks as first class citizens.
To elaborate:
luarocks installs rocks into a tree that separates library (lua, lib) and runtime (share/<name>/<version>/{plugin, ftplugin, ..., whatever}).
This allows to make Lua libraries available without impacting startup time in any meaningful way, and to add runtime data as needed.
Some Nvim libraries double as plugins and libraries, but don't need to be plugins for the average user.
For example, plenary.nvim is mostly a library, but also a plugin that creates commands for testing Neovim plugins (useless if you're not a plugin developer).
The idea of using luarocks to manage plugins as first class citizens is:

Users should not have libraries their plugins depend on pollute Nvim with unexpected behaviour or unnecessarily impact startup time.
The behaviour should only be added if users explicitly declare that they want that to happen (by installing it as a plugin).
Extensions of plugins should be able to declare version constraints on the plugin they're extending.

In our [nvim-neorocks org] experience, this works quite well.
luarocks even has the ability to load different versions of the same dependency.
The main PITAs we still have are;

macOS and Windows package managers being sh*te
(which is why we're working on better ways to bootstrap luarocks and ensure a compatible version is installed - and have also been working on a rewrite[^1]).
Slow bulk installations because luarocks doesn't yet have good mechanisms to allow parallel builds without potentially corrupting shared dependencies.

[^1]: It's too soon to say if we will use it to replace luarocks.
... which is why I was a bit surprised that this was merged and released so quickly.

With luarocks it's not even possible for a plugin to declare that it need setup or that it can be lazy-loaded.

There exists no standard for declaring such things, so it wouldn't be very useful for diversity.
There is, however, a very simple solution to this problem that works independently of the package manager.

I just deployed a new Github action that fetches the luarocks root manifest and then for each rock that contains nvim in the name (currently 177 rocks), it fetches the rockspec.

@folke many neovim plugins have 'neovim' or 'nvim' in the description.labels list.
Perhaps checking that could make the GitHub action more reliable.

The biggest issue is tooling as far as I can see.
Things like npm, rust etc all just work on Linux, Macos, Windows.
For me on Arch, luarocks also works great, but in any other system I tried, it was a big pita.
I still couldn't get it to work on Windows for example...

Slow bulk installations because luarocks doesn't yet have good mechanisms to allow parallel builds without potentially corrupting shared dependencies.
Lazy does, and also now does it for luarocks, since the those will be in separate trees.

... which is why I was a bit surprised that this was merged and released so quickly.

As I said before, with lazy v11.0, all of the sudden, some of user's plugins needed luarocks.
If a uses didnt need luarocks before v11, then they shouldn't need it after.
This is now fixed with my recent changes.

many neovim plugins have 'neovim' or 'nvim' in the description.labels list.
Perhaps checking that could make the GitHub action more reliable.
This would require to download all rockspecs, unless there's a repo somewhere that contains them all?

Ah, I had assumed you were downloading the rockspecs already.
Yep, that might hit rate limits pretty quickly. Hadn't thought of that.
I guess you could have a cron job that crawls a subset of https://luarocks.org/manifest-5.1, but that might be overkill for the few plugins that don't have nvim in the name. It would probably be easier to just add the popular ones manually.

I'm currently using https://luarocks.org/manifest.
Should I use the 5.1 version instead?

Closing, since op's original issue was also fixed.

I'm currently using https://luarocks.org/manifest. Should I use the 5.1 version instead?

Probably not a bad idea. 5.2..4 rocks won't work with neovim anyway.

cmp

Also the neotest ecosystem:
https://github.com/nvim-neotest/neotest - is published to luarocks
https://github.com/mrcjkb/neotest-haskell/blob/master/neotest-haskell-scm-1.rockspec - is not correctly converted to lazy dependencies

@mehalter added!
https://github.com/folke/lazy.nvim/commit/6d60dc3c05440f9f79c738058af62c333543329b


[BUG] Trying to debug a test in VS Code with linetrace=True raises "TypeError: 'NoneType' object is not callable"

Describe the bug
When trying to debug a pytest test in VS Code that imports a Cython module that has linetrace enabled, I'm getting a "TypeError: 'NoneType' object is not callable" at import time.
=================================== ERRORS ====================================
________________________ ERROR collecting test_spam.py ________________________
test_spam.py:1: in <module>
    import spam
spam.pyx:4: in init spam
    def f():
spam.pyx:4: in __Pyx_PyMODINIT_FUNC PyInit_spam(void)
    def f():
E   TypeError: 'NoneType' object is not callable

To Reproduce
Code to reproduce the behaviour:
spam.pyx:
# cython: language_level=3str, linetrace=True
# distutils: define_macros=CYTHON_TRACE_NOGIL=1

def f():
    return 1234

test_spam.py:
import spam

def test_f():
    assert spam.f() == 1234

Build spam.pyx with cythonize -i spam.pyx
Then try and debug test_f with VS Code using the Python extension's Pytest support.
Expected behavior
A clear and concise description of what you expected to happen.
Environment (please complete the following information):

OS: Windows
Python version 3.10.0
Cython version 0.29.24

Would be nice to support, but I'm not surprised that Python debugging has issues. I don't think anyone is seriously doing that. I suggest either a) using a C level debugger instead or b) debugging the uncompiled Python (.py) code and not writing  .pyx files (not sure if that's an option for you when wrapping libpcap & friends).
gdb has support for debugging Python, Cython and C code. Don't know if there is anything similar for Windows/MSVC.

(Not sure what a good label is – line tracing is (most?) commonly used for coverage reporting, so I'll put it there.)

I am having the same issue with cython 3.0.0a9 and PyCharm.
It would be nice to step through the code to debug issues if that is even possible. Not sure if that would require work from the PyCharm team as well or just on your end.
Coverage reporting would also be good as well.
The code runs just fine when run normally.
  File "C:/Users/james_000/AppData/Roaming/JetBrains/PyCharm2021.2/scratches/debug.py", line 1, in <module>
    import amulet_nbt
  File "D:\Data\GitHub\Amulet-NBT\amulet_nbt\__init__.py", line 1, in <module>
    from .value import BaseTag, BaseImmutableTag, BaseMutableTag, BaseValueType
  File "amulet_nbt\\value.pyx", line 1, in init amulet_nbt.value
  File "amulet_nbt\\util.pyx", line 1, in init amulet_nbt.util
  File "stringsource", line 1, in init amulet_nbt.errors
  File "amulet_nbt\\errors.pyx", line 1, in __Pyx_PyMODINIT_FUNC PyInit_errors(void)
TypeError: 'NoneType' object is not callable

Yeah, it's probably something that they should investigate on their side first. If only to find out what we should do better on ours. It's not unlikely that they would at least have to do … something … in the end. Or maybe could even do something on their side already.
In any case, it would appear easier for them to debug the problem than for us.

It looks like Cython is handling f_trace wrong. This is an internal member used by trace_trampoline to support a Python trace function that returns a different trace function than itself, basically the thread local trace function. Cython is setting it to None, but it should (Surprisingly and unlike other PyObject*) be NULL. Actually, Cython should just avoid touching the it at all, as it is only used by trace_trampoline internally for it's own operation and isn't supposed to be touched by code externally calling the c_tracefunc (Which again is a Python implementation detail, FUN!).

According to a comment in the code, it does that in order to enable line number reporting:
https://github.com/cython/cython/blob/f848bdf69717e68b919ef7a43b651f77bf4fb980/Cython/Utility/Profile.c#L292-L296

Yeah. But I'm not sure why... I don't see any reference to that effect in CPython...

https://bugs.python.org/issue42823

Hi and sorry for reviving this thread a year and half later.
I'm having trouble understanding the underlying issue and related PR. Is there any hope that this could be fixed in the near future?
Thanks in advance !


Adding of nodes more user friendly

Adding data nodes is not extremely user friendly. I think that people are use to click on a node and then drag them to the place where one needs them. Clicking on the datanode and then clicking on the canvas seems counterintuitive to me.

Agreed. Both should work.

this is based on how power point works but I agree it would be nice if both would work. I'll add the lowhangingfruit tag to this issue.


[BUG] Copy/paste does not work?

Trying to copy/paste any entity/brush, but doesn't work at all.

The issue seems to be happen with interface buttons. Shortcus are working.

Are you talking about 3D view?

Yes, probably

Fix with 5d314d0 in #1, please review and report if bug is still remain. If so - please provide more info.

"Paste" toolbar button still doesn't work. Nor in 3D, nor in 2D.

Aww, the toolbar button
fix with 491b788

Loks like working now, thanks.


Sensor_plus package add to macOs.

Use case
Our team was developing an fitness app where , we want to track the user's steps so we used sensor_plus package but it was not allowing to use on macOs.
Proposal
Our team was developing an fitness app where , we want to track the user's steps so we used sensor_plus package but it was not allowing to use on macOs.

sensors_plus is not maintained by the flutter team but the flutter community instead. This issue should be reported on their repo https://github.com/fluttercommunity/plus_plugins/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc.
Closing from here.
Thank you


Investigate shading the kotlin-runtime into Javalin

The Kotlin runtime could be shaded into Javalin. I'm not sure if it's required or beneficial. Example code:
<plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-shade-plugin</artifactId>
    <configuration>
        <minimizeJar>true</minimizeJar>
        <artifactSet>
            <includes>
                <include>org.jetbrains.kotlin:kotlin-stdlib</include>
                <include>org.jetbrains.kotlin:kotlin-stdlib-common</include>
                <include>org.jetbrains.kotlin:kotlin-stdlib-jdk7</include>
                <include>org.jetbrains.kotlin:kotlin-stdlib-jdk8</include>
            </includes>
        </artifactSet>
        <relocations>
            <relocation>
                <pattern>kotlin</pattern>
                <shadedPattern>javalin_shaded_kotlin_runtime</shadedPattern>
            </relocation>
        </relocations>
    </configuration>
    <executions>
        <execution>
            <phase>package</phase>
            <goals>
                <goal>shade</goal>
            </goals>
        </execution>
    </executions>
</plugin>

What would be the effect of this change?
As far as I can understand it would reduce the classpath polution for Java projects that rely on Javalin.
Does it have any effect on Kotlin-based Javalin applications?
I can imagine some conflicts if somebody using a different Kotlin version tries to create a fat jar of their app.

My understanding was that this will bundle the Kotlin runtime with Javalin, but under a different package structure. It would be like Javalin is a Java project without any dependency on Kotlin. Kotlin projects that use Javalin then have to include the Kotlin runtime like any other dependency.
I'm not sure if my understanding is correct though.

I tested the shade plugin and the generation seems to work. I tested the changes in my local project and they work great. Javalin's Kotlin runtime is shaded properly.

Thanks, that's good to know. We just need to figure out if it's necessary / good theb

No one seems to be bothered by how this works currently, so I'll just close it.


Support textures in EXT_feature_metadata

Expands EXT_feature_metadata support by adding convenient "views" for feature textures and feature id textures. This helps facilitate CPU-based picking implementations when combined with a front-end implementation.
In particular, this PR adds:

Utility to query the feature ID at a given texture coordinate in a feature ID texture.
Utility to query the property value at a given texture coordinate of a feature texture property.
Several miscellaneous improvements to the metadata API to better support rendering engine integrations (this has been integrated successfully here: https://github.com/CesiumGS/cesium-unreal/pull/698).

@nithinp7 is this one ready for another review? It looks good for the most part but I think there are is a small comment or two not addressed (even by saying we're not going to address it right now).

Sorry for the delay on this one @nithinp7, it all looks good! Please just update CHANGES.md in main.


Anti-Always On Top Feature

Description of the new feature / enhancement
To disable AOT on programs or mini applications that don't give you the option to disable AOT.
Scenario when this would be used?
It's annoying when programs or mini applications force you to keep them AOT when you don't want them to.
Supporting information
No response

can you give an example of a program? /needinfo

Like those little small programs on open source applications you find on Source Forge for an example like this auto-clicker one:

https://sourceforge.net/projects/hf-auto-clicker/
There other applications.  This one was just an example.


Typo: "Creat" > "Create".

Fixing the typo:
-##   Creat a plan guide using T-SQL
+##   Create a plan guide using T-SQL

@adyavanapalli : Thanks for your contribution! The author, @MikeRayMSFT, has been notified to review your proposed change.

Merging typo fix
@adyavanapalli Thanks.


[BUG]: Disk Space Warnings in Azure DevOps Pipeline Agent on CoreOS 41+ with ComposeFS

What happened?
Description
When running the Azure DevOps pipeline agent on CoreOS 41 or later, disk space checks result in warnings due to the use of composefs as the root filesystem. Since composefs is designed to always appear as fully utilized (100%), the agent's check incorrectly reports insufficient disk space on /.
Steps to Reproduce

Deploy a CoreOS 41+ host with composefs as the root filesystem.
Install the Azure DevOps pipeline agent.
Execute any pipeline job that triggers the disk space check.

Actual Behavior
A warning message is displayed in the timeline:
##[warning]Free disk space on / is lower than 5%; Currently used: 100.00%

This warning appears for every job, cluttering the pipeline timeline and making it harder to identify genuine issues.
Expected Behavior
The disk space check should correctly evaluate the volume where the working directory resides, rather than relying on the root filesystem (/).
Additional Context

composefs was introduced as the default root filesystem in CoreOS 41. Its design always shows 100% utilization for the root filesystem (link), which causes the disk space check to fail incorrectly. This spams the timeline with warnings, making it harder to identify genuine issues.
In general, checking free disk space on the root filesystem (/) is unreliable, especially when the working directory resides on a separate mounted volume. The current approach does not account for the actual volume used by the working directory, leading to misleading results in many possible configurations.

Possible Solution
Update the disk space check logic to evaluate the actual working directory's mount point or volume instead of the root filesystem (/).
Versions
v3.248.0 / Fedora CoreOS 41.20241109.3.0
Environment type (Please select at least one enviroment where you face this issue)

[x] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
No response
Version controll system
No response
Relevant log output

Hi @sebb0, thanks for the detailed report on this issue, we will take a look. By the way, I think you can disable the warnings by setting the agent knob AZP_ENABLE_RESOURCE_UTILIZATION_WARNINGS to false.


[feat] a CONTRIBUTING.md

Description 📝
To contribute effectively, it would be useful to know:

What is the general file structure/architecture and how various pieces fit together (e.g. engine_fragment, three.js, BIM Tiles, etc.)
What is the develoment cycle for a feature/bugfix which includes:

how to clone & setup a local environment e.g. github's guide
how to build & test the code against the examples (e.g. via a local python3 -m http.server 8000)
formatting & code style
commit message format e.g. conventional commits

The entrypoint for all this pieces of info is expected to be a CONTRIBUTING.md file
Suggested solution 💡
A detailed CONTRIBUTING.md file in the project root containing the above information
There is some information about getting started e.g. https://docs.thatopen.com/components/getting-started, but it's not geared towards working on this codebase
Alternative ⛕
No response
Additional context ☝
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.

Thanks a lot @davidgtl! We will keep this in mind to do it asap.

We have created a contributing page in our docs. I think it includes all the points you mentioned, but let us know otherwise. Cheers!


about liacence

hi, is this template is free?

Yes it is free to use for commercial purposes or otherwise!


support both gherkin 2 and 5 formats

update data and tests to cater for both gherkin formats

Noticed that you haven't bumped up the version in build.sbt.


Autocomplete duplication

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I've just loaded the last beta portable version for the first Atom try. The autocompletion test causes unexpected behavior: the object name is doubled before the property (the same in the safe mode / with factory defaults). Sorry, if I get something wrong.
Steps to Reproduce

Expected behavior: [What you expect to happen]
console.log();

Actual behavior: [What actually happens]
console.console.log();

Reproduces how often: [What percentage of the time does it reproduce?]
Persistently.
Versions
Atom    : 1.14.0-beta3 (atom-x64-windows.zip)
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
OS: Windows 7 x64

This is expected: you're using a feature called "snippets" (denoted by the green tab marker on the left).  Those expand when you autocomplete them, as opposed to normal autocompletion.

Well, it seems to be too confusing for a default behavior(

Yeah, I agree it's pretty confusing to new users.  If you have any suggestions for how to improve the UX though, please create new issues on the atom/autocomplete-plus repo.  We'd be glad to hear about them :).

Unfortunately, I know very little about Atom UX. I've also tried VSC the same day, this autocompletion works fine there. Maybe atom/autocomplete-snippets should not be toggled on by default?

It seems atom/autocomplete-snippets even disables common autocompletion, what is even more confusing to new users. Compare
atom/autocomplete-snippets is off:

atom/autocomplete-snippets is on:


Color Decision List (CDL) support

Some people would like OTIO to handle Color Decision List metadata in a structured way.
For example, an EDL can contain ASC_SOP and ASC_SAT like this:
*ASC_SOP (1.0000 1.0000 1.0000)(0.0000 0.0122 0.0305)(1.0000 1.0000 1.0000)
*ASC_SAT 1.0000
For reference:
https://en.wikipedia.org/wiki/ASC_CDL

@adaro can you take a look at this to see if it parses your CDL data correctly?

CDL metadata is now supported in the EDL adapter (both read and write).
CDL metadata on a Clip in OTIO looks like this:
                        "metadata": {
                            "cdl": {
                                "asc_sat": 0.9,
                                "asc_sop": {
                                    "offset": [
                                        1.0,
                                        0.0122,
                                        0.0305
                                    ],
                                    "power": [
                                        1.0,
                                        0.0,
                                        1.0
                                    ],
                                    "slope": [
                                        0.1,
                                        0.2,
                                        0.3
                                    ]
                                }
                            }
                         }

We still need to add support for CDL to the FCP XML adapter.

Here is a reference for how CDL data is stored in AAF (as a Video Effect).
https://www.amwa.tv/downloads/specifications/AS-05_AAF_Effects_protocol_v1.pdf

Here is a reference for how CDL data is stored in FCP X XML (as an XML comment inside a color correction video filter).
https://developer.apple.com/library/content/documentation/FinalCutProX/Reference/FinalCutProXXMLFormat/Adjustments/Adjustments.html#//apple_ref/doc/uid/TP40011227-CH14-SW8

I think CDL support in OTIO would be a great candidate for expanding our support for effects. It really is a color correction effect with well defined parameters, so we can model it as a schema.Effect object on a Clip. Each adapter can then transcribe that to/from the various other formats as appropriate.

Addressed by #143


Unable to install Keybase

The file cannot be installed because the file cannot be found in cabinet file 'cab2.cab'. This could indicate a network error, an error reading from the CD-ROM, or a problem with this package.
Using Windows 11.

Works fine now, not sure what I did differently.


Issue with Maxthon V<IP_ADDRESS>0

Upon loading a video, the video shows for a couple of seconds, then it seems like the site refreshes for a second, and afterwards the video is no longer visible, though audio continues to play. Disabling the addon returns everything to full functionality, so it seems like it's literally only the addon causing the problem.

Have you changed the website to use the old style?
It won't work on the new style...

That fixed it, thank you.
Honestly I didn't even know there WAS an option to return to the old style.

I hope it will work on the new style. It's probably the most useful script I have.


Support multiple database driver

Objective
Make database driver configurable, other than default H2Driver, to support other databases such as MySQL and PostgreSQL in the future.
Design
Note: I have implemented it with reference to slick-examples.
Add Profile trait to combine it with Table.
trait Profile {
  val profile: ExtendedProfile
}

Import Profile.profile.simple._ instead of scala.slick.driver.H2Driver.simple._ directly in the table definition:
trait AccountComponent { self: Profile =>
  import profile.simple._
  object Accounts extends Table[Account]("ACCOUNT") {
    def userName = column[String]("USER_NAME", O PrimaryKey)
    ...

Services include corresponding Component to import the Table object.
trait AccountService extends AccountComponent { self: Profile =>
  def getAccountByUserName(userName: String, includeRemoved: Boolean = false): Option[Account] =
    Query(Accounts) filter(t => (t.userName is userName.bind) && (t.removed is false.bind, !includeRemoved)) firstOption
  ...
}

Now almost every controller requires an ExtendedProfile instance to inject into the services, combined with Profile.
class AccountController(override val profile: ExtendedProfile) extends AccountControllerBase
  with AccountService with RepositoryService with ActivityService with OneselfAuthenticator with Profile

class ScalatraBootstrap extends LifeCycle {
  override def init(context: ServletContext) {
    val driver = Database.driver(context)
    context.mount(new IndexController(driver), "/")
    context.mount(new SearchController(driver), "/")
    ...

Merged to #791


Update Defrag command with the "/#" parameter

The "/#" is missing from this documentation.  Reference here https://learn.microsoft.com/en-us/previous-versions/windows/it-pro/windows-server-2012-R2-and-2012/dn789160(v=ws.11)#to-optimize-storage-tiers-using-defragexe
defrag : /g /h /#
This command optimizes storage tiers (/g) on the volume, returning a Post Defragmentation Report and a Storage Tier Optimization Report. Include /h to process the job at normal priority instead of low priority (the default).

#assign: @robinharwood, @Xelu86
@robinharwood  @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Configure automatic release notes

Dependabot will go into its own category
PRs with breaking_change label will go in its own category

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.05%) to 89.941%

Files with Coverage Reduction
New Missed Lines
%

gnosis/eth/ethereum_client.py
1
86.74%

gnosis/safe/safe.py
1
88.43%

gnosis/util/util.py
3
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
3523

Relevant Lines:
3917

💛  - Coveralls

Just to clarify: the labels need to be on the Pull Requests right?

I understand it that way, issue tags should not be relevant


Every discord command can be use by any person on the Discord.

The check for if a Discord User is admin does not work, so every Discord command can be used by anyone.

See #7505.

It seems the line @commands.check(commands.is_owner()) in restartbot.py and rotate.py does not function correctly.

Duplicate of #7505


Branch add viewstate.

Added stateful view functionality for edit commands that require person to be in full view.
Format for edit homework, attendance, grade progress and sessions still using the format of:
edit 1 h/1 homework
instead of the intended
edit h/1 homework

Codecov Report

Merging #72 (99f52fe) into master (19150a9) will increase coverage by 2.13%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master      #72      +/-   ##
============================================
+ Coverage     69.85%   71.99%   +2.13%
- Complexity      504      561      +57
============================================
  Files            89       94       +5
  Lines          1692     1878     +186
  Branches        175      199      +24
============================================
+ Hits           1182     1352     +170
- Misses          440      441       +1
- Partials         70       85      +15

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
69.56% <0.00%> (-3.17%)
:arrow_down:

.../address/logic/parser/AttendanceCommandParser.java
0.00% <0.00%> (ø)

...edu/address/logic/parser/SessionCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...ava/seedu/address/model/person/AttendanceList.java
55.88% <40.00%> (-3.50%)
:arrow_down:

...in/java/seedu/address/logic/parser/ParserUtil.java
79.03% <55.55%> (+24.31%)
:arrow_up:

.../java/seedu/address/model/util/SampleDataUtil.java
55.55% <60.00%> (+5.55%)
:arrow_up:

...n/java/seedu/address/model/person/SessionList.java
61.29% <61.29%> (ø)

...c/main/java/seedu/address/model/person/Person.java
83.09% <70.00%> (+0.28%)
:arrow_up:

... and 15 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


parseToAliases not work

3.0.0 is ok, but the newer version 3.1.0 and 3.1.1 dose not work.

Hi,
Can you please give us some sample code? I have a project that relies heavily on this library and I'm having no issues other than incomplete emoji coverage, and Darn Kids somehow managing to use Regional Indicator Symbols independently of the flags they're supposed to make. But that's neither here nor there.
Point is, I see parseToAliases working just fine.

@Lolologist Uses the v3.1.0 or v3.1.1 could produce this issue.

@88250 , please provide us a concret example that is not working with:

The string in input
The string in output
The string that you expected to have in output
The code you are using

It is really hard to help you without this context :)

@vdurmont
The sample code:
public static void main(String[] args) {
    String str1 = "Here is a boy: \uD83D\uDC66\uD83C\uDFFF!";
    String str2 = EmojiParser.parseToAliases(str1, EmojiParser.FitzpatrickAction.REMOVE);

    System.out.println(str2);
}

The output is OK with v3.0.0:
Here is a boy: :boy:!

But v3.1.1 outputs:
Here is a boy: ??!

In debugging view:

I uses Maven central repository to setup emoji-java.
Env info:

Java: 1.8.0_60; Java HotSpot(TM) 64-Bit Server VM 25.60-b23
OS: Windows 10 version 10.0 running on amd64


After uninstalling the Auctionator addon, Journalator doesn't work anymore.

It now cause the followig LUA error whenever I do something inventory related, in my case I opened a few JC gem bundles:
10x Journalator/Source/Utilities.lua:178: attempt to index global 'Auctionator' (a nil value)
[string "@Journalator/Source/Utilities.lua"]:178: in function GetItemLocationFromGUID' [string "@Journalator/Source/Monitor/LootContainers.lua"]:346: in function AddToLogs'
[string "@Journalator/Source/Monitor/LootContainers.lua"]:54: in function <Journalator/Source/Monitor/LootContainers.lua:29>
And when I tried to open it using /jnr it didn't work and printed:
Journalator: Unable to open display. The Display addon is disabled
I looked in the addon panel and saw that three addons pertaining to Journalator had a dependency on Auctionator.
I'm not sure if this is normal as the addon is not listed as requiring Auctionator installed.
On a side note:
I actually removed Auctionator after the new buy interface made me do too many wrong purchases.
after a while you get used to always click the buy button that kind of constantly pop anyway when you buy something.
To seldom end up realizing that you just paid some material ten time the price that you expected based on the first given price.
Buying often probably rendered me not careful enough. But still, the standard buy interface doesn't cause me this problem.
Edit:
After reinstalling the addon using Curseforge, it seems to work fine without Auctionator now.

Journalator requires Auctionator for the UI components. The error you got was still a bug, so that's fixed in the latest alpha, but Journalator (Display) will need Auctionator enabled to work.
Side note:
The buy interface has the same number of confirms as the Blizzard interface, its just integrated into the Shopping tab instead. Or is the Auctionator buy interface missing a dialog somewhere?

Side note: The buy interface has the same number of confirms as the Blizzard interface, its just integrated into the Shopping tab instead. Or is the Auctionator buy interface missing a dialog somewhere?

It's just the price given at first by the interface that can be misleading.

That price issue should be fixed now (I've just moved the alpha with the fix to a release). There was an issue with the total price being really low in some scenarios.

Alright that is great news, I was missing my shopping lists :)

As always, report if something goes wrong, there's usually a fix I can add 😄

I'm going to close this as Auctionator is an intended dependency of Journalator


Failed to launch CARLA v0.9.0 on Linux

System information (version)

CARLA => v0.9.0(Compiled version)
Operating System / Platform => Ubuntu 16.04 64bit
Graphics Driver => nvidia-387
Graphics Card => GeForce GTX 1060

Detailed description
I tried to launch CARLA v0.9.0 using CarlaUE4.sh.
But, vehicle is flying in the sky. And, I can not control vehicle.

On the other hand, I can control vehicle in previous version(v0.8.4).

Steps to reproduce
$ ./CarlaUE4.sh

Any update?

This is the intended behaviour of 0.9.0 API, please check out #660.


Use syscfg setting for sysinit stage numbers

This allows the target to rearrange package initialization order via syscfg override

I'm not a big fan of allowing user to configure init priority for core packages. What is the reason you need to change it? If priorities are not correct because of some dependency it should be fixed instead. If you need to have some other init done before/after other init then we do support this via $after and $before syntax (https://github.com/apache/mynewt-newt/pull/457).

as I said, I'm not a big fan of such changes but I won't block them from being merged - feel free to merge at your discretion


Backspace removes one too many parenthesis

Issue Type: Bug
When you have an if-statement with a function call in it and press backspace to remove the functions parenthesis it removes the closing parenthesis of the if-statement too.
declare function someFunctionOrVariable(): boolean;
if (someFunctionOrVariable()) { // put the cursor between the two closing parenthesis and press backspace twice
}

Playground Link
Context: I have gotten into the habit of testing poorly named boolean variables/methods by adding a pair of parenthesis behind it to see if it is a function. When it isn't and I remove the parenthesis the if-statement breaks too.
Rant (no need to read): For me this would not be an issue if autocomplete inserted parenthesis too if it is a function. While this is not always wanted it probably is most of the time? Or if the popup was triggered after autocomplete so I can see what it is. Or I simply just learn to read the popup help before I press enter to autocomplete :) . Or learn / rebind that cmd + k + i shortcut.
Still feels like it shouldn't remove the final parenthesis though :)
VS Code version: Code 1.50.1 (d2e414d9e4239a252d1ab117bd7067f125afd80a, 2020-10-13T14:53:05.704Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1030NG7 CPU @ 1.10GHz (8 x 1100)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (1.87GB free)

Process Argv
-psn_0_323663 --crash-reporter-id 44640369-2a58-4ef9-8279-cc7b9068d6d4

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

markdown-pdf
yza
1.4.4

(2 theme extensions excluded)

This seems to be fixed now, I cannot reproduce it in VS Code 1.55. Please reopen if it is not fixed!

Great, it seems to be fixed for me too.


how to get/set storage items values

Hello,
I want to retrieve the attributes inside the list of dashboard Items/storage,
for example save to a json file every Item (color, position, value inside like text photo),
I think that Im not setting correctly this custom attributes inside the ColoredDashboardItem
BR,
Luca

@Mehmetyaz
I saw that you need to modify DashboardItem in order to be able to store more parameters on the storage

@Mehmetyaz
if I want to update the color of an Item like the thing that is doing right now with the position and size
Is correct to edit _onMoveStart and implement it to modify the color?


authy_services.py

i think line 32  if authy_user.ok:  should be  if authy_user.ok():

without the () i found it sometimes did not work for some reason.


Some of the quickstarts ReadMe refer to the process for deploying on Fabric8 v1

For example:
https://github.com/fabric8io/quickstarts/blob/master/quickstarts/java/camel-cdi/README.md

Ya a bunch of them haven't been migrated. Frankly its maybe even better to remove the readme's as the v1 likely confuses people more than it helps.


Recommend using Taiwanese Mandarin instead of Traditional Chinese

Hello MiuLab,
Could you consider using Taiwanese Mandarin in place of Traditional Chinese for this project? I feel that this choice of language name aligns more closely with the project's essence. I'd love to hear your thoughts on this matter :)
Alex

good ideas!

cmn-Hant-TW encodes the Mandarin language written in the Han (Traditional variant) script as used in Taiwan.
The suggested Taiwanese Mandarin means cmn-TW which implies -Hant without spelling it out.
Given that the model name Taiwan-LLaMa already encodes -TW, another possibility to consider is Traditional Mandarin which spells out the cmn-Hant part.
In either case — if MiuLab changes the description, I'm happy to sync my downstream GGML repo to match.

I've reviewed the suggestions and the recent change to Traditional Mandarin. It aligns well with the language encoding and clarifies the specific variant used. The commit at 93576b065f3780bed219d21234227e919c9c2014 seems appropriate, and I support this decision. If further modifications are necessary, please don't hesitate to propose them.


An error occurred while scanning the remote server

Error after error. Have noe clue on whats wrong.
LOG: https://pastebin.com/qY8qzJNj
Settings: https://pastebin.com/zxrtRnr1

I had this happen too, and ended up setting the server script path closer to my server directory, e.g. /deluge/scanfs/ and /deluge/completed which fixed it.

My path is as short as :
/files/completed
/files/scanfs
But I still got the same error
ERROR - RemoteScanner.RemoteScanner - Caught an SshError Traceback (most recent call last): File "/app/python/controller/scan/remote_scanner.py", line 66, in scan self.__remote_path_to_scan) File "/app/python/ssh/sshcp.py", line 143, in shell args=" ".join(args) File "/app/python/ssh/sshcp.py", line 109, in __run_command raise SshcpError(sp.before.decode()) ssh.sshcp.SshcpError:  exec request failed on channel 0


Proposed Solution

Isolates the reformatting into a single commit based on an already fine-tuned treefmt.toml (from #825)
Adds that commit to the ingore rev for blame file

Adopted into #825 after consultation with @tshaynik .


pouchdb v5.4.0 breaks pouchdb-dump-cli v2.0.3

Using<EMAIL_ADDRESS>works fine, here is the error I get with v5.4.0:
unexpected error
{ [TypeError: this._get is not a function]
  result:
   { ok: false,
     start_time: Tue Jun 07 2016 13:53:31 GMT+0200 (CEST),
     docs_read: 0,
     docs_written: 0,
     doc_write_failures: 0,
     errors: [ [Circular] ],
     status: 'aborting',
     end_time: Tue Jun 07 2016 13:53:31 GMT+0200 (CEST),
     last_seq: 0 } }
TypeError: this._get is not a function
    at PouchDB.<anonymous> (/node_modules/pouchdb/lib/index.js:2282:15)
    at PouchDB.<anonymous> (/node_modules/pouchdb/lib/index.js:207:21)
    at PouchDB.<anonymous> (/node_modules/argsarray/index.js:14:18)
    at /node_modules/pouchdb/lib/index.js:144:21
    at PouchDB.<anonymous> (/node_modules/pouchdb/lib/index.js:131:19)
    at PouchDB.get (/node_modules/argsarray/index.js:14:18)
    at getDocAttachmentsFromTargetOrSource (/node_modules/pouchdb/lib/index.js:7333:17)
    at /node_modules/pouchdb/lib/index.js:7408:18
    at Array.map (native)
    at /node_modules/pouchdb/lib/index.js:7395:50

Hm yeah, looks like I'm using a private API that's not in there anymore. Will investigate.

Gonna pin to 5.4.2 for the time being.

pinned in 2.0.4


Bug | When user clicks on publish button, no indications that the article is being published

Current:
When a user is writing a new article, on publish button click, the app takes time to publish the article and redirect to the published article. There is no indication (in the form of loader or something else) that the informs the user app is publishing.
Expectation:
When user clicks the publish button, disable the button & show a loader within the button until the event handler finishes execution. (Consider happy & sad flows)

Can you assign me the task?

Sure. Please fork the repo, create a branch, and raise a PR.
Also, link this issue to your PR in the description with appropriate details. (refer existing PRs that are closed)


loading vector tiles

There are 3 kinds of files:
.dbf
.shp
.shx
Is It possible to load this data files by using your project?
I have seen these files:
.pbf,
.mbtiles,
But I did not seen
.dbf
.shp
.shx
files in your project.
Thank You

Hi.
pbf and mbtiles are processed vector tiles, while shp, kml, geojson, etc are raw vector data. To show shapefile into your project, use a map control that can show those shapefiles (such as SharpMap).
If your raw vector data files are too big and dense to show directly, you can use tippecanoe to convert that into processed vector tiles, and then show them on your client. But it's a bit of a difficult thing to do, so i'd recommend the first method

There are 3 projects in this repository:

GMap>demo.WinForms
Mapsui.Demo.WPF
Static.Demo.WPF

When I run these projects, all of them shows same UI result even though they are different UI.

I have my own vector tiles should be load in this project. I don't know how to load my map data instead of yours. Can you guide me please?

I assume, that you misunderstand what VectorTileRenderer does. It isn't a mapping library. It converts vector map data into raster tiles. This tiles than could be displayed with GMap or Mapsui. If you want to display your own data on a map, you should look into GMap or Mapsui.

I tried to use gMap but it shows .gmdb file.
I need library which can load shapefile (.shp, .shx, .dbf)

It seems, that Mapsui could handle SHP files. See the shapefile sample.


make error syscall

I am using  Window's subsystem Ubuntu 18.04, when I call make, it gives an error
randombytes.c:77:10: warning: implicit declaration of function ‘syscall’; did you mean ‘sysconf’? [-Wimplicit-function-declaration]
    ret = syscall(SYS_getrandom, (char *)buf + offset, chunk, 0);

should I comment  #include <sys/syscall.h> from random.h file? Or what is the solution? Please help.
I am trying to use this library in EOS smart contract.

Hmmm. So it looks like sys/syscall.h is correctly included, but it does not actually contain the syscall function.
Are you running on WSL or WSL2?

I am using WSL.


Mimick curl POST post 1:1

Working with a specific server, this post request using curl succeeds just fine.
curl --location --request POST 'https://foo.bar' --header 'Authorization: Token {{TOKEN}}' --form<EMAIL_ADDRESS>
However trying the equivalent using reqwest fails:
let client = reqwest::blocking::Client::new();
let form = reqwest::blocking::multipart::Form::new().file("image", "/home/Downloads/image.jpg")?;
let resp3 = client
    .post("https://foo.bar")
    .header(
        "Authorization",
        "Token ".to_owned() + &token.to_owned(),
    )
    .multipart(form)
    .send()?;

with "Status: 405". Hence authentication is not an issue. Reads more like that the form data is not in the expected format.
Questions:

Is there an easy way to send exactly the same POST request as send by curl through reqwest?
Are there any other suggestions to debug this issue?

A 405 is Method Not Allowed, so like the endpoint wasn't expecting a POST... I see you're setting --location, are you expecting to post the file through a redirect? reqwest can only do that with body types is knows how to rewind, which a form it might not...

A 405 is Method Not Allowed, so like the endpoint wasn't expecting a POST... I see you're setting --location, are you expecting to post the file through a redirect? reqwest can only do that with body types is knows how to rewind, which a form it might not...

Thanks, @seanmonstar for the quick answer. I'm an absolute curl/REST n00b. However, I tried curl without the location flag, and it succeeds as well...
curl --request POST 'https://foo.bar' --header 'Authorization: Token {{TOKEN}}' --form<EMAIL_ADDRESS>

Thanks, @seanmonstar for the quick answer. I'm an absolute curl/REST n00b. However, I tried curl without the location flag, and it succeeds as well...
curl --request POST 'https://foo.bar' --header 'Authorization: Token {{TOKEN}}' --form<EMAIL_ADDRESS>

You can see what curl is doing with -v (not that might include your auth token).

You can see what curl is doing with -v (not that might include your auth token).

Can I check equivalently what reqwest is doing?

curl has special output being a CLI tool, reqwest doesn't have as nice of output. It does has logs, they're just a bit noisier.
If you add the env_logger crate to your example, and then set the environment variable RUST_LOG=debug, you should see the more-useful log messages.

curl has special output being a CLI tool, reqwest doesn't have as nice of output. It does has logs, they're just a bit noisier.
If you add the env_logger crate to your example, and then set the environment variable RUST_LOG=debug, you should see the more-useful log messages.

Thanks, I'll try that!

Thanks, I'll try that!

Nevermind, I had a typo, all working now as expected!

Nevermind, I had a typo, all working now as expected!


Mean and variance of data sets

hello，
I have a new dataset, and I organize it into coco. But I don't know if I need to modify mean and STD in configs。

As shown in the figure above, I calculated the mean and STD of the dataset myself, and the result is quite different from the original mean and STD. I don't know if this is correct
Looking forward to your reply！

I tried to modify and train, and all the losses became Nan. I don't know where the problem is. Does this data need to be modified?

I don't know what data you are，please confirm that the mean and variance are correct，the pix range is 0-255，not 0-1，so mean should not be so low

@hhaAndroid
Excuse me, do you have Python code to calculate the mean and variance of data sets?
If so, would you like to share it with me？


Feat: update sidemenu, fix #66

Fixes #66

Before update

After update

Closed as this didn't update the living whitepaper


Getting ResourceInUseException issue when creating multiple accounts

Hello Team,
Customer tried to create 310 linked accounts by referring the guide [1]
The DynamoDB after updating the table continuously invoke the lambda function resulting in this error “ResourceInUseException  Account Factory cannot create this enrolled account right now, because another execution is in progress. Try again later.”
Since we have no way of identifying why the second event occurred while the first one is in progress. . Also, this didn’t occur on the recreates.
Can you please help us to resolve this issue?

References:
https://aws.amazon.com/ko/blogs/mt/how-to-automate-the-creation-of-multiple-accounts-in-aws-control-tower/

Ack. As discussed, since we do not have a way to root cause the issue. I will look in to adding logic to handle this condition gracefully.


add noms show --raw

@rafael-atticlabs fyi

Diff Coverage
File Path

•••••••••• 100%
cmd/noms/noms_show.go

Powered by Codecov. Last update 22d8e17...d12dd96

It only shows the raw top level chunk which is exactly what I want. It's
useful for debugging the serialization.
On Sat, Dec 24, 2016 at 10:50 AM Erik Arvidsson<EMAIL_ADDRESS>wrote:

@arv commented on this pull request.
What happens with Ref? Collections that have ptrees?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/attic-labs/noms/pull/2991#pullrequestreview-14356329,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE6BFEw1aE2xx-LT0EYe5KELO5ylqWBks5rLWllgaJpZM4LVLsw
.

That is also what I expected/want. Can we have a test for that?
On Sat, Dec 24, 2016, 11:19 Aaron Boodman<EMAIL_ADDRESS>wrote:

It only shows the raw top level chunk which is exactly what I want. It's
useful for debugging the serialization.
On Sat, Dec 24, 2016 at 10:50 AM Erik Arvidsson<EMAIL_ADDRESS>wrote:

@arv commented on this pull request.
What happens with Ref? Collections that have ptrees?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/attic-labs/noms/pull/2991#pullrequestreview-14356329
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAE6BFEw1aE2xx-LT0EYe5KELO5ylqWBks5rLWllgaJpZM4LVLsw
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/attic-labs/noms/pull/2991#issuecomment-269097027, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACzFekc1VUXhAOz5kos4dCAGTyy7XYJks5rLXBOgaJpZM4LVLsw
.

Hm, It sorta already is testing that because it decodes the raw chunk. Assuming the decoder is working correctly (and it is tested elsewhere) it should decode chunks with random extra data in them.
But I did add some tests for more complex values.


Use progress bar for all longer lasting operations

-> Fetching contract details from IB

-> Fetching contract details from IB

Fixed with PR #97

Fixed with PR #97


'show log' button in container list

I have several running containers and view logs often,
every time I click container link, scroll to logs link and click it.
I am lazy.
Shouln't be a 'view logs' button in container list?
How should it look like?

I think it would work after the container status, care to open a PR for it?


selector-no-redundant-nesting-selector false positive when followed by a character other than ':'

'selector-no-redundant-nesting-selector' rule falsely assumes the selector is redundant in this common BEM use case:
.block {
  &__element {
    ...
  }
  &--modifier {
   ...
  }
}

More generally nesting selector is not redundant if immediately followed by a non white space character as it results in a new not nested rule.

Thanks for reporting! It looks like we need to add more tests for that rule and refactor the code a bit.


[feat] FlowPath - add conditional logic to be able to move to next step

In a flow, we have a call out to an external system.  While that is running there is a spinner over the fields of the component, but the buttons (Next most significantly) are available to users.  If they click next, the flow stops in error.  While we can error trapping logic, it would be a better user experience to hide the standard footer and only allow users to move forward once we have a successful reply from the external system.
This could also be based on other conditions of course (only activate a Next button once all required info is entered, for example).
This is currently on Flow roadmap, but > 1 year out, or else this solution looks like it could handle the requirement, https://unofficialsf.com/flow-button-bar-new-and-improved/

@jstukas actually the issue is not on this component but on the other one.
this componet should implement the validate() API method (see https://developer.salesforce.com/docs/component-library/documentation/en/lwc/lwc.use_flow_validate_user_input_for_custom_components) to prevent the "next" operation to actually execute.


Add missing option in example XML

Docs Build status updates of commit fc0b1fd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/web-service-reference/changes-items.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


How to use LSTM or GRU in C++ using c_predict_api.h?

I find nothing about how to use LSTM or GRU in C++ using c_predict_api.h?
Have we support these recurrent layers? If No,is there any plan?

@xiaogehh currently the Gluon LSTM and GRU RNN layers cannot be hybridized but the work towards should be soon completed. You can track the effort here: https://github.com/apache/incubator-mxnet/issues/10873
Alternatively you have an example of LSTM implemented in C++ here: https://github.com/apache/incubator-mxnet/blob/master/cpp-package/example/charRNN.cpp
@sandeep-krishnamurthy could you please consider closing the issue.
@xiaogehh if you would like to follow up, please create an issue on https://discuss.mxnet.io


Incomplete testing framework.

Completed most functionality tests as of today. Marking this issue as resolve, though, as with most active projects, pursuing better testing frameworks is an on-going goal.


[#4389]  fix(iceberg-rest-server): add conf directory to classpath if starting standalone Iceberg REST catalog server in Gravitino package

What changes were proposed in this pull request?
add conf directory to classpath if starting standalone Iceberg REST catalog server in Gravitino package
Why are the changes needed?
Fix: #4389
Does this PR introduce any user-facing change?
no
How was this patch tested?
start Iceberg REST server and check classpath

the conf directory is added to the classpath in aux service mode and separate Iceberg REST package.


Try detecting timer and activity resurrection

What changed?
When processing user timer or activity timeout task, if a timer is overdue for a long time (by default, 24 hours), scan workflow history to see if that timer has already been fired or if that activity has ready finished.

Why?
For detecting timer/activity resurrection and prevent duplicated timer fired or activity completion.

How did you test it?

Unit test.

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 4fdf41fc-3013-408b-8ce1-d7ff5a08a114

162 of 190 (85.26%)  changed or added relevant lines in 4 files are covered.
22 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.05%) to 56.421%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/history/task/timer_active_task_executor.go
150
178
84.27%

Files with Coverage Reduction
New Missed Lines
%

common/cache/lru.go
3
90.73%

service/history/queue/timer_gate.go
3
95.83%

common/persistence/sql/sqlExecutionStore.go
16
60.0%

Totals

Change from base Build 629223c8-e7e5-4d3b-8c1a-894fd126873a:
0.05%

Covered Lines:
78707

Relevant Lines:
139499

💛  - Coveralls


Move remaining helper functions to lib/

Clear the regular update cache between unit tests

Thanks for reviewing!
None of this is new code, and I prefer to avoid making changes as code is moved, since the changes are impossible to discern in the commit history.
I'll update the names, and open a new PR for the rest!

Nexus service tests are not passing locally, though the Nexus failure exists in e2ee910, when the code entered, so it's not related to this change. Also failing are github and amo, also unrelated.


Can jsdom follow links ?

Hi,
What is the config to tell jsdom to follow a link, execute the scripts of the new page and update the document content accordingly ?
Is there another way than doing a new jsdom.env by hand with the url of the link ?
Thx

And, will you please point out the test that document how jsdom follow links if it exists ?

jsdom is more for single pages than for entire web browsing sessions. For the latter you may want to look into zombie.js, which is based on jsdom.

Thank you for your quick answer


Android app

I appreciate the suggestion, but this is a significant undertaking and much larger than a single GitHub issue. There's not currently any plan to do this, so I'm closing this issue for now.


Revise specification documentation

This PR is based on #285 , solution to #286

@tomdele , @mgeplf I've put comments for discussion, please don't review the whole PR. I've pushed it only to start a discussion.

Not sure I'm helping or making things more confusing.

It's ready for the review. I've reflected the current behavior in the specs. This behavior might conflict with what you expected but it describes what MorphIO does. If we want to change something, like throwing an exception for two-point soma in H5, then we should do in the different PR. Here we need to finally describe how MorphIO interpret morphologies.

Few minors for me but looks good ! Thanks for this @asanin-epfl !

Thanks for making suggestions. No need to open IDE for me.

Thanks for making suggestions. No need to open IDE for me.

that's why I suggested :D


Authentication for secure source is not working Norconex HttpCollector

Hi,
I need to crawl a secure site. I am trying with the following configuration but it is not able to authenticate. I tried with 2.3 version but still not getting any clue as why it is not even looking for authorization. It is not even logging that whether it is going for authentication or not. Initially, I had issue with certificate, then i changed the config to add <trustAllSSLCertificates>true</trustAllSSLCertificates>
After adding it the error related to certificate has gone.
<crawlerDefaults>
 <httpClientFactory>
      <authMethod>form</authMethod>
      <authUsernameField>username</authUsernameField>
    <authPasswordField>password</authPasswordField>
	<authUsername></authUsername>
	<authPassword></authPassword>
     <authURL></authURL>
</httpClientFactory>
  </crawlerDefaults>
 <crawlers>
    <crawler id="Norconex eforge">
	<startURLs stayOnDomain="true" stayOnPort="true" stayOnProtocol="true">
	   <!-- <url></url>-->
	   <urlsFile>E:\search\spiderURL.txt</urlsFile>
	</startURLs>
.......

The above is the extract from my configuration.
Please have a look and let me know as what could be issue.
Thanks!
Shubham

Please format any XML you paste on github using Markdown as otherwise your HTML tags do not show. I edited your question this time to make sure we can see the XML.
I would need the actual URL to confirm if what you have is OK.  Your full config would help as well to reproduce.
With what you have pasted, I can see you are missing the values for authUsername, authPassword, and authURL.  Did you just remove them before pasting here and your real config has them?  You need them.
Without pasting credentials, can you share the URL?  You can always email the URL to me if too sensitive to paste here (you can get my email on my github profile).

Thanks for your response.
Yes. I have removed the values for the fields but those values are there in my actual configuration.
URL which I need to crawl is an intranet site and is not accessible outside. Please find attached configuration which I am using at present.I have removed some information which are sensitive.
Thanks,
Shubham
config.txt

Do you get any error? Authentication mechanisms can vary greatly.  The form-based authentication supported is fairly basic.  If yours has custom elements in it or uses a more complex mechanism (e.g. SAML), chances are it is currently not supported.  Can provide detailed information about the authentication technology being used?
Without that info or being able to reproduce, I am afraid I can't help much on this support channel.
One option is to write your own solution for it. A good place to start is by extending GenericHttpClientFactory#authenticateUsingForm
Another option is to contact Norconex for a private analysis of the authentication issue on your environment (via Remote Desktop or else).

I am closing due to lack of feedback and not being able to reproduce.  If you have more information to share that could help troubleshoot, feel free to re-open or create a new ticket.

Is SAML based authentication supported in any release?

No it is not, but there is now a feature request for it: https://github.com/Norconex/collector-http/issues/421


Fix typo issue in main.css

In order to participate in this project, I am going to insert a patch that just fix typo in the code.
It is a simple modification, but I hope you will remember that it is the beginning.
ISSUE=None

Thanks for fixing a typo, and welcome to the project. :-)
If you plan to contribute non-trivial stuff in the future, you may want to sign the ICLA beforehand: https://feedback.line.me/enquete/public/2183-v2w9HqrE

I already register ICLA. Thank you :)


rename: cobalt => cobalt2

otherwise, vim will complain: "can not find colorscheme cobalt"

@zhuangya thx for the pr.
I've never run into any issues with the name, but it definitely can't hurt to improve the colorscheme name

yeah, i didn't have any problems until i switch to neobundle.
not sure if there is any "strict mode" or something :)


Sourodip20kar.md

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Can you please merge my request or can you tell me if I need to do any type of modification?

Please try with better topics

Please try with better topics

Okay thank you 💓

Please try with better topics

I have uploaded one new file, i.e. CSS animation project, kindly accept that PR. Thank you so much.


Python: Add module-level description to messages.snip template

This ensures that when sphinx generates documentation for the API types, the module description is not blank.
We might use the template "Protocol buffer classes used by the Google {API Name} API".

See blank description for API Types in http://pythonhosted.org/gax-google-pubsub-v1/apis.html

@geigerj Is this still relevant?

Not sure, re-assigning to @lukesneeringer to confirm the current status of Python GAPIC docs.

Closing because this code appears to be removed.


In_exec would read from continuous output of command/script

Is your feature request related to a problem? Please describe.
I am having this script to monitor cpu utilization, which issues mpstat. The thing with mpstat is that if I set it to get result every 5 seconds, it is important to keep it running, stoping it and a starting again after every 5 second would not work because it could miss some time of utilization, only having it run gets continuous reliable data.  Actually arriving to its place, but its not continuously updating the destionation (the out_tail is here just for this example, in reality I want to forward it).
<system>
  process_name fluentd1
</system>

<source>
  @type exec
  tag test_tag
  command ./script.sh
  connect_mode read_with_stderr
  <parse>
    @type json
  </parse>
</source>

<match test_tag>
  @type file
  append true
  path /home/{{ user }}/test.log
  add_path_suffix false
  <buffer []>
    @type file
    path /home/{{ user }}/fluentd/test_buffer.buff
    flush_mode immediate
  </buffer>
  <format>
    @type json
  </format>
</match>

Describe the solution you'd like
I would like to issue command with in_exec without parameter run_interval, that would continuously create records from output of a script, line by line.
Describe alternatives you've considered
I can save it as a file in the script and watch the file with fluentd. I would have to create system service for running the script then, which is not always convenient, for example if some company is managing the server for you. Having exec handling this would be much leaner.
Additional context
Or simpler, I could do mpstat 5 directly in in_exec command and having it parsed with regex parser - also not possible currently.

Found out that mpstat 5 by itself works, somehow the solution using script is a problem.


Login is not working, because the JS endpoint is hardcoded to a specific IP address

See PR https://github.com/Seedstars/django-react-redux-jwt-base/pull/1
Thank you.

So your PR fixes this issue, right?

Yes
On Wed, 17 Feb 2016 at 12:35, Luís Rodrigues<EMAIL_ADDRESS>wrote:

So your PR fixes this issue, right?
—
Reply to this email directly or view it on GitHub
https://github.com/Seedstars/django-react-redux-jwt-base/issues/2#issuecomment-185163328
.

Ok thanks for your help


Use data property

Smaller, and little faster.
Unfortunately, it dirties the type checking a bit. It'd be nice if typescript could detect nodeType === 3 as a type guard for Text... Right now the best we could do is user defined type guard but that's too big.

Is .data really faster? Interesting...

@jridgewell where's .data documented? I can't find anything on MDN.

Is .data really faster? Interesting...

http://jsbench.github.io/#b266690cc58d684e9f9eb7de3158437e. It's because CharacterData holds the .data, while Node holds the .nodeValue, so we'll be one step closer in the prototype chain.

where's .data documented? I can't find anything on MDN.

MDN has poor documentation for DOM things. It's spec'd at https://dom.spec.whatwg.org/#characterdata, and both Text and Comment inherit from CharacterData

@jridgewell seems like .data is much faster than .textContent too. Nice :)
I merged and formatted your branch, so it's ready to go, but I need permission to push.

I went and merged this manually. Thanks!


feat(TCOMP-481): Add endpoints to access icon resources for definitions.

Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[X] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
The REST endpoints for icons and images pass exclusively via the wizard definitions.
What is the new behavior?
Icons and images can now be fetched from any Definition subclass.
Does this PR introduce a breaking change?

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: success :star:


Remove AliasSeq when following __traits is directly aliasable

There's just one remaining AliasSeq!(__traits but it's not aliasable (getAliasThis).
edit: it will be after fix for blocker merged.

Blocker is https://issues.dlang.org/show_bug.cgi?id=19420. I'm not against some help with that. It must be possible to get the symbol for an expression requiring this since it works in the AliasSeq but for now i don't see how with the public API.

Is this still blocked?

No, it will go into the green zone now.

Well, I forgot about that but  still need to fix D-Scanner

Buildkite is still failing but i'm not involved in this stuff. Hope it will be fixed ASAP so that we could get nice green checkers everywhere.

you can remove the blocked label now.

done.

The Buildkite failures are unrelated.

Buildkite is still failing but i'm not involved in this stuff. Hope it will be fixed ASAP so that we could get nice green checkers everywhere

Yup, that would be nice, but sadly seems to be a bit challenging. See e.g.
https://github.com/dlang/ci/pull/340
https://github.com/dlang/ci/pull/344
https://github.com/dlang/ci/pull/346
https://github.com/dlang/ci/pull/350
https://github.com/dlang/ci/pull/351

Thanks for the merge and my sorries for the remaining work ion dlang ci :wink:


Add legacy macro option

Fixes #3029

Adds legacy macro option to enable _FillValue, any other macros which may reveal themselves. As discussed in #2858, the issue with _FillValue is that it can, in some circumstances, conflicts with a symbol libc++18.  While _FillValue is historically important and widely used, (and technically tricky to move away from), we can't avoid the fact that _ is a reserved identifier and will potentially cause problems for users as libc++18 and later become more common in the wild.
We replaced _FillValue with NC_FillValue in #2858, and we encourage users to adjust their software to account for this.  But not all of our users are maintaining software they rely upon, and we do not (and cannot) put people in a position where software they rely upon but are not equipped to modify will not compile.
Invoking --enable-legacy-macros (autotools) or -DNETCDF_ENABLE_LEGACY_MACROS=TRUE (cmake) will define the _FillValue macro, for use when building netCDF as part of a larger, un-modified technical ecosystem.

I believe this should have the opposite default setting, as the number of users who will encounter problems with _FillValue is tiny compared to the number that will have problems with its absence.
At least give users a few releases with the new macros available but not yet mandatory.

@edwardhartnett It would be great if we could get more release candidate exposure so that we had a better feel for this.  Flipping it (as you suggest) doesn't do any harm and could (as pointed out) be better in the short run.  I have documentation to add before I was going to merge this, so I'll flip the polarity for the next rc.  No harm in changing a considered opinion, after all ;).

For some customer input, I am fine with either option.  We have already added code to test whether NC_FillValue is defined and if it isn't, then use the old _FillValue

Echoing @gsjaardema , I just amended NCO to use NC_FillValue, which NCO #defines if/when netCDF does not.


Feature request : ./manage.py runcrons --loop

This would run "runcrons" every minutes in foreground. The usecase is for docker containers.

E.g: python manage.py runcrons --loop 2

You can create custom command and run it whenever you want:
> python manage.py cronloop &

app/management/commands/cronloop.py:
from time import sleep

from django.core.management import BaseCommand, call_command

class Command(BaseCommand):
    help = 'Run cronjobs in loop.'

    def add_arguments(self, parser):
        parser.add_argument(
            '-s',
            '--sleep',
            dest='sleep',
            type=int,
            help="Sleep interval in seconds.",
            default=5 * 60,
        )

    def handle(self, *args, **options):
        s = options['sleep']
        while True:
            try:
                call_command('runcrons')
                sleep(s)
            except KeyboardInterrupt:
                break


process.env.PROJECT_CWD undefined

Hi guys,
when running npm run zcli:start fails because the destinationPath that is created during build resolves to undefined/dist for me. Below is the full log, plus that destinationPath variable logged out.

I am running Apple Silicon M2 - I suspect this may be related to that. For now, I am hard coding process.env.PROJECT_CWD to ../.. 🤷
Great job on this repo btw - I've used this already over half a year ago for an app that I already published and it's cool to see that it's still maintained. Loving the new docs too!

Hey @Than-DE , thanks for reporting this and for the workaround!
I am not sure where and how this env variable is available on my machine (PROJECT_CWD) but I guess only me then have this variable :D I thought this was a default available to everyone.
I will put together something more sophesticated to retrieve the root of the project if there are no other simpler solutions


Support for multiple Firebase databases

Right now the firebase_database plugin can only handle one database. It should support multiple databases.
/cc @xster

Any updates on this?
Tried using FirebaseDatabase(databaseURL:) with other database link, but didn't seem to work.


New release

Hi,
Is it possible to make a new release so we can use the Regex comparison?

Soon :) Hopefully.


Add datastore fill pattern configuration

Add a configuration option to define filling pattern & sort order for datastores that the vsphere-cpi picks.

Currently it always picks approx. the least used datastore with the most free space, but we'd like to be able to configure BOSH to choose the most used datastore, in order to fill them up efficiently.

This would be really helpful

@JamesClonk can you please add some unit test for storage_picker and datastore_picker using this new configuration
Once you do we can run test and merge it

@nehagjain15 I've updated the PR accordingly. Added some unit-tests for the fill pattern configuration.
The default behaviour stays the same as before, but it now allows to configure the CPI to try and fill up the datastores more efficiently based on choosing either the most full or least full datastore to place disks.

Thank you @JamesClonk.

@JamesClonk : Thanks for submitting the PR.
I have some concerns regarding the core logic of datastore picker. I have added review comments in the code. Let's discuss them first before moving on to syntactic review (if needed).

@EleanorRigby where can I see these comments?

Closing this. As the entire architecture for cluster and datastore picker has changed.


ASA-Notifier Lambda function throwing access denied error for s3 iam-auto-key-rotation-enforcement.html

Error while getting file contents for iam-auto-key-rotation-enforcement.html - An error occurred (AccessDenied) when calling the GetObject operation: Access Denied
Got everything working but the above. Can't figure out why I am getting access denied error...

Ok I have found two issues....  both in Primary Account
First issue:
ASA-iam-key-auto-rotation-NotifierFunctionExecutio-WDN8XIWP171N
that iam role needs more permissions to get at the file...
Second Issue:
ASA-Notifier lambda function has typo...
in the code notifier.py line 59 has /Template/  in the codecommit its lower case... change to /template/
Hope that helps someone else...

The error is because the folder template (where the template for the email is located) is created in lower letter, however the policy providing access to the S3 Object, called the folder with a cap letter "Template", you can just modify the name of the folder from template to Template.

In our case our <our-IAM-Key-rotation-stack>-NotifierFunctionExecution-5pNi5KC1w18c IAM role had the following Statement,
            "Condition": {
                "StringEquals": {
                    "aws:PrincipalOrgID": "<our_org_id_here>"
                }
            },

and manually deleting that section got to the next error related to the (MessageRejected) when calling the SendEmail operation error.


Add a regression test [RIPD-1040]

As reported in RIPD-1040, part way into the development of ed25519 key support a secp256r1 public key could cause rippled to seg fault.  This test reproduces the seg fault if the fix is not present and passes when the fix is in place.
reviewers: @ximinez, @nbougalis

Removed unnecessary comment and corrected commit message.

Nice test. Verified that rippled to crashes in this test if the mentioned commit is reverted.
:+1:

Current coverage is 54.21%

Merging #1457 into develop will increase coverage by +0.02% as of 935b4e5

@@            develop   #1457   diff @@
=======================================
  Files           965     965
  Stmts         67374   67401    +27
  Branches          0       0
  Methods           0       0
=======================================
+ Hit           36514   36539    +25
  Partial           0       0
- Missed        30860   30862     +2

Review entire Coverage Diff as of 935b4e5

Powered by Codecov. Updated on successful CI builds.


Delete duplicate typedef

Should these need a second typedef? I'll let you answer that, because I can't, because I don't know.

Even thought the date and duration types are represented by the same int64, they are semantically different. While not needed, adding types improves readability and maintainability.