[SMALLFIX] Include -bin in the name for release tarballs

@Xenorith

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20593/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.server.ft.ZookeeperFailureIntegrationTest.zkFailure
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20597/
Test PASSed.


Pick an existing image

There is an example on this page that uses https://aka.ms/campus.jpg
However, that link is defunct, and since people are coming to this page for help, it would be great if when you paste in the code and run it it doesn't fail because of this.
I recommend something from xkcd.com 😀

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

ID: cb4d16da-8c0f-5b7e-54ee-1fe36c549837
Version Independent ID: cb4d16da-8c0f-5b7e-54ee-1fe36c549837
Content: Image - .NET MAUI
Content Source: docs/user-interface/controls/image.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Associated WorkItem - 56011

LOL
Thanks for reporting this!

Hi @rogersm-uwosh
Fixed!


Write a guide to use Blender and MaterialX materials.

Request description
Write a document.

Here's the blend scene.
gltf_pbr_blend_scene.zip

Here are the files you need to put into BlenderUSDHydraAddon_1.1.1_Blender_3.1_Windows.zip before installing the addon.
gltf_pbr_materials.zip
copy gltf_pbr.mtlx -> BlenderUSDHydraAddon_1.1.1_Blender_3.1_Windows.zip\hdusd\libs-3.10\libraries\bxdf
copy gen_gltf_pbr.py -> BlenderUSDHydraAddon_1.1.1_Blender_3.1_Windows.zip\hdusd\mx_nodes\nodes\

Delayed due to low desirability from the team.


Issues with timestamps in History

no space after numeric value
the translatable string is "minutes ago". 5 minutes ago. But in French for instance, we must say: "Il y a 5 minutes" >> include a placeholder for the number in the string

@teolemon i am working on this

@iml-v are you working on this?

Fixed in #1600


load_model requires h5py

I used the code and everything from the here.
https://github.com/autorope/donkey2
I was at the point were I went to run the code and see if the machine learning has worked and mz car can  go through a path detecting the lines
However I'm getting this
(env) pi@raspberrypi:~/donkey2 $ python drive.py --model model_2019-07-24__18-43
using donkey version: 2.5.8 ...
/home/pi/.local/lib/python3.5/site-packages/env/lib/python3.5/importlib/_bootstrap.py:222: RuntimeWarning: compiletime version 3.4 of module 'tensorflow.python.framework.fast_tensor_util' does not match runtime version 3.5
return f(*args, **kwds)
/home/pi/.local/lib/python3.5/site-packages/env/lib/python3.5/importlib/_bootstrap.py:222: RuntimeWarning: builtins.type size changed, may indicate binary incompatibility. Expected 432, got 412
return f(*args, **kwds)
loading config file: /home/pi/donkey2/config.py
config loaded
pygame 1.9.6
Hello from the pygame community. https://www.pygame.org/contribute.html
Webcam loaded.. .warming camera
Starting Donkey Server...
Traceback (most recent call last):
File "drive.py", line 142, in
drive(cfg, model_path=args['--model'])
File "drive.py", line 52, in drive
kl.load(model_path)
File "/home/pi/.local/lib/python3.5/site-packages/env/lib/python3.5/site-packages/donkeypart_keras_behavior_cloning/part.py", line 17, in load
self.model = load_model(model_path)
File "/home/pi/.local/lib/python3.5/site-packages/env/lib/python3.5/site-packages/tensorflow/python/keras/engine/saving.py", line 187, in load_model
raise ImportError('load_model requires h5py.')
ImportError: load_model requires h5py.

@python1995
Just install necessary packages
sudo apt-get install libhdf5-dev
pip install h5py

This should work.
I am closing the issue. But if you are still facing the issue, you can reopen it again. Thanks!

@gowtham-kp
One quick question. As u saw I was working in an environment.
Do I still use  sudo when installing  the packages _

@python1995 Please refer to this link which has been beautifully answered.


Add fuzzy search for mentions

Fixes #470

Used Fuse.js to implement the fuzzy search logic for mentions.

@AbhayVAshokan Please Review.

@AbhayVAshokan Please review.


I/O blocking when editing in mounted network directories

When I mount an ssh directory on my file system on Ubuntu 15.04 and open a file the editor freezes until the file is loaded. This also happens when saving files. This can be problematic if my network is slow or experiences interruptions since it can completely lock up the editor and cause it to crash.

Thanks for the report, @hiyou102. There's already a few issues related to slowness/problems when working with network drives, e.g. https://github.com/atom/atom/issues/2456 and https://github.com/atom/atom/issues/1766. I think those two cover your report as well, so I'm going to close this in favor of those issues, but please let me know if I misunderstood anything.


bug: plugins not loaded due to cond are not contained in the lazy-lock.json file

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)
v0.9.1
Operating system/version
Linux
Describe the bug
If a plugin is disabled via cond, in my case it is:
return {
    'goolord/alpha-nvim',
    cond = function()
        -- don't start when opening a file
        if vim.fn.argc() > 0 then
            return false
        end

        -- skip stdin
        if vim.fn.line2byte '$' ~= -1 then
            return false
        end

        -- Handle nvim -M
        if not vim.o.modifiable then
            return false
        end

        for _, arg in pairs(vim.v.argv) do
            -- whitelisted arguments
            -- always open
            if arg == '--startuptime' then
                return true
            end

            -- blacklisted arguments
            -- always skip
            if
                arg == '-b'
                -- commands, typically used for scripting
                or arg == '-c'
                or vim.startswith(arg, '+')
                or arg == '-S'
            then
                return false
            end
        end

        return true
    end,
    dependencies = { 'nvim-tree/nvim-web-devicons' },
    config = function()
        require('alpha').setup(require('alpha.themes.startify').config)
    end,
}

then the plugin will not be shown in the lazy-lock.json file.
Steps To Reproduce

Add plugin with cond
Open nvim when cond is false
Run Lazy sync
Check the lazy-lock.json

Expected Behavior
Described plugin in the lazy-lock.json
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

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

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

-- install plugins
local plugins = {
	"folke/tokyonight.nvim",
	{
		"goolord/alpha-nvim",
		cond = function()
			-- don't start when opening a file
			if vim.fn.argc() > 0 then
				return false
			end

			-- skip stdin
			if vim.fn.line2byte("$") ~= -1 then
				return false
			end

			-- Handle nvim -M
			if not vim.o.modifiable then
				return false
			end

			for _, arg in pairs(vim.v.argv) do
				-- whitelisted arguments
				-- always open
				if arg == "--startuptime" then
					return true
				end

				-- blacklisted arguments
				-- always skip
				if
					arg == "-b"
					-- commands, typically used for scripting
					or arg == "-c"
					or vim.startswith(arg, "+")
					or arg == "-S"
				then
					return false
				end
			end

			return true
		end,
		dependencies = { "nvim-tree/nvim-web-devicons" },
		config = function()
			require("alpha").setup(require("alpha.themes.startify").config)
		end,
	},
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})

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

I think the issue is caused by enabled being set together with cond in plugin.lua.
Commenting that line out locally made it so that the plugin is included in the lock file again, but still prevented loading like expected.

Hello @Mephistophiles,
I thought about this issue, and would like to share a non-authoritative opinion.
The cond keyword indeed either disables or enables the plugin in the current vim session.
It differs from the enabled keyword in two ways:

lazy.nvim does not offer to clean the plugin, as it may be needed in another session.
lazy.nvim applies another color in the disabled section of its UI.

Quoting the description of cond from the specs:

When false, or if the function returns false, then this plugin will not be loaded.
Useful to disable some plugins in vscode, or firenvim for example.

An example for vscode can be found here.
Note that the checker and change_detection are disabled.
An unofficial example for firenvim can be found in this discussion.
Also, important info can be found in this pull request.
Now, both vscode and firenvim are external environments.
In my opinion, the maintenance of plugins(affecting the contents of lazy-lock.json),
is not expected to be done inside those environments.
Thus, I think that the cond keyword is not the best approach to accomplish the behaviour you are looking for.
As an alternative, the plugin could be loaded only on an event.
The value of the event property can be a function.
Your use case can be implemented by conditionally returning an empty table.
Example:
  {
    "goolord/alpha-nvim",
    event = function()
      local should_load = function()
        -- don't start when opening a file
        if vim.fn.argc() > 0 then
          return false
        end

        -- more logic here

        return true
      end

      if should_load() then
        return { "VimEnter" }
      else
        return {}
      end
    end,
    dependencies = { 'nvim-tree/nvim-web-devicons' },
    config = function()
        require('alpha').setup(require('alpha.themes.startify').config)
    end,
  }

Best regards!

Hi @abeldekat !
Thanks for the answer!
I'll try to do as you suggested. Another small question: how to properly filter by filetype AND event? eventName *.filetype? or is there a more convenient variant?

Regarding your question, as an example, this could work:
return {
  {
    "linux-cultist/venv-selector.nvim",
    event = "BufReadPost *.py",
    opts = {},
  },
}

I don't think that will work for the events lazy.nvim provides.
See also this discussion.


How do we migrate all the existing bags to Azure?

We have ~260,000 odd bags already in the storage service, which only exist in S3 and Glacier. How do we get them to Azure?

Here's my proposal:

Deploy a replicator/verifier pair that can store Azure replicas. Update the replica aggregator to expect three replicas.
All newly-ingested bags are now being stored in Azure. Get this up and running, watch it ingest new bags, leave it for a few days.

Modify the bag register (temporarily) so that it can replace an existing manifest, if it can't already.
I haven't checked, but I think it would throw an error if you ask it to register a bag that it already knows about. This is a temporary fix that means it would be able to create new manifests if it got a new message from the replica aggregator.

Send messages to the Azure replicator of the following form:
{source => (warm S3 replica), destination => azure}

One for every bag in the replica aggregator database that only has a Glacier replica.
This will trigger a replication + verification of the bag in Azure. Then the replica aggregator counts the third replica, sends that to the bag register, and it creates a new storage manifest that includes both replicas.

Repeat step 3 until every bag known to the register/aggregator has a replica in Azure.

This sounds sensible. My reading is that we are not creating "new versions" of the bags to do this and the mechanism you describe will overwrite existing storage manifests at their existing version.

My reading is that we are not creating "new versions" of the bags to do this and the mechanism you describe will overwrite existing storage manifests at their existing version.

Yes. Normally each version of a storage manifest is immutable after it's written, but we want to temporarily change that.
(Or add a mechanism by which the register can add a replica to an existing manifest without regenerating the whole thing, but that seems like more work unless we routinely plan to add new replicas.)

BTW, the overarching bent here is to do as little jiggery-pokery as possible. We push as much work as possible into the pipeline apps, to minimise duplicated effort and avoid mistakes.

I realised last night that this might not be what we want.
Having the bag register recreate the manifests from scratch will reset the created date, which probably isn't what we want – it would make all bags appear to be created as they were replicated to Azure. We want to preserve the original creation date, right? It's the creation date of the bag, not the JSON manifest?
As a data point: whenever I've tweaked the manifests in the past, I've preserved the original creation date.

@weco-bot Yes, we might need some custom code that reads the existing storage manifest and just appends to the replica rather than recreating it from scratch.

Alice and I discussed this at some length after planning today; a revised plan:

We get to a point where we can reliably ingest new bags that are replicated, verified and tagged in Azure.

We create a new instance of the bag register (the “Azure backfill register”), which writes bags to an empty VHS.

It receives updates from the replica aggregator
Newly-ingested bags will be written to the live bag register (and thus available in the external API) and copied to the empty VHS
This Azure backfill register does not send updates to the ingest tracker – it may register a bag before it’s available in the external API, and we don’t want to break the success->callback->get bag flow for external users.

We give the replicator/verifier temporary permissions to read from the primary storage bucket.

Once we're happy the Azure backfill register is storing bags successfully, we send messages to the replicator telling it "replicate bag from (warm storage) to (azure)". This bag passes through the replicator, verifier and replica aggregator as normal.

When a backfilled bag reaches the live bag register, it will fail, because it doesn't know how to replace the existing manifest
When a backfilled bag reaches the Azure backfill register, it will create a new manifest in the empty VHS

Eventually the two VHS instances will be in sync – each bag will have a manifest in both VHS instances.

We run a script that copies the creation date from (old VHS) to (new VHS).

We modify the live bag register to point at the (new VHS), which now has a record of every bag with a correct creation date, and we turn off the Azure backfill infra. We may reindex the bags into Elasticsearch at this point.

New bags will be replicated to Azure automatically, and we gradually backfill the existing archive without interrupting new ingests or bags.

One note:
The bag register can only register bags in order – e.g. you can only register v1 before you register v2.
We’ll need to order the messages sent to the replicator accordingly.

Closing in favour of #4589 – we've got an approach that we're all happy with, I think.


Fixed aws_vpn_gateway_route_propagation for default route table

Fix #85

@antonbabenko Was there a reason this hadn't been merged yet? I agree with @tyrken , it would be nice to have this feature in the VPC module. Let me know if you need help with testing or something else. Thank you!

Yes, we should merge this and it is primarily correct, but should be rebased because aws_main_route_table_association and aws_default_route_table were removed since this PR was created.
Please finish this and test it properly.

@antonbabenko Without write access I cannot change the branch directly. Do you want me to spin up a new PR based off a fork?

Yes, please check out this PR and make a new fork. I can't grant write access to this repo.

This PR can be closed. Re-opened here: https://github.com/terraform-aws-modules/terraform-aws-vpc/pull/133

@antonbabenko can we close this?


As a user, I want to be notified when a new version of my node is available, so I don't fork.

An idea for this, maybe we can have a notification area above VersionList that if the selected release is older than the latest remote version, it can say something like
<!> You are using an older version of geth (1.2.2)
New releases contain performance and security enhancements.
Click here to download|use the newest version (1.2.3)

Another nice feature could be an advanced settings checkbox that says "Always download and use the latest version of geth". On VersionList component load it could auto download and select the newest version.


pipe symbol attached to tokens and other issues with punctuation

When tagging the following non-Norwegian fragment (usually inside an otherwise Norwegian text, of course, but it also replicates when isolated), we get pipe symbols (not present in the source text) attached to tokens and punctuation symbols (present in the text) attached to them as well:
«… Fräulein von Kulp

may turn, her hand upon the door;

The output is:
"<«>"
	"$«" <anf>
"<…>"
	"…"

Det gjelder også norsk tekst:
"Når det gjelder de som er begravd i Ćelarevo, så ønsker ungarerne at de er ungarere, jødene at de er jøder og muslimene at de er mongoler, men ingen ønsker at de skal være kazarer.
Selv om dette er det mest sannsynlige…
Gravplassen er full av knuste leirkar innrisset med menoraer.

Da får man "<sannsynlige…|>". Når jet bare tar de siste to linjer, får jeg "<sannsynlige…>"... også med ...knyttet til ordet, men uten|`.

More problems:

hyphen/dash disappears, but an extra full stop appears at the end of "sentence":

$ echo "Gymnastikk … 6.15-6.30 -"|mtag/mtag.py -wxml
<word>Gymnastikk</word>
"<gymnastikk>"
	"gymnastikk" subst appell mask ub ent
<word>…</word>
"<…>"
	"…" symb
<word>6.15</word>
"<6.15>"
	"6.15" subst <klokke>
<word>6.30</word>
"<6.30>"
	"6.30" det kvant
	"6.30" subst <klokke>
<word>-</word>
"<->"
	"$-" <strek>
<word>.</word>
"<.>"
	"$."  clb <<< <punkt>

$ echo "19.30-sending"|mtag/mtag.py -wxml
<word>19.30</word>
"<19.30>"
	"19.30" subst <klokke>
<word>sending</word>
"<sending>"
	"sending" subst appell fem ub ent
	"sending" subst appell mask ub ent
<word>.</word>
"<.>"
	"$." clb <<< <punkt>

no results at all (in this way several sentences may completely disappear at the end of a text file):

$ mtag/mtag.py -wxml
1 gammelt persisk teppe ca kr 700,-
1 nytt teppe ca kr 500,-
$

This seems to be a problem of particular combination of punctuation marks and whitespace at the end of the input.

dash/hyphen changed to number ("700,-" changes into "700,1"):

(The last example with the problematic final punctuation removed.)
$ mtag/mtag.py -wxml
1 gammelt persisk teppe ca kr 700,-
1 nytt teppe ca kr 500
<word>1</word>
"<1>"
	"1" det ent kvant
<word>gammelt</word>
"<gammelt>"
	"gammel" adj nøyt ub ent pos
<word>persisk</word>
"<persisk>"
	"persisk" adj nøyt ub ent pos
	"persisk" adj ub m/f ent pos
	"persisk" subst appell mask ub ent
	"persisk" subst appell nøyt ubøy
<word>teppe</word>
"<teppe>"
	"teppe" subst appell nøyt ub ent
	"teppe" verb inf
<word>ca</word>
"<ca>"
	"ca" adv
	"ca." adv fork @adv>
<word>kr</word>
"<kr>"
	"kr" subst fork
<word>700,1</word>
"<700,1>"
	"700,1" det fl kvant
<word>nytt</word>
"<nytt>"
	"ny" adj nøyt ub ent pos
	"nyte" adj nøyt ub ent <perf-part>
	"nyte" adj ub m/f ent <perf-part>
	"nyte" verb perf-part
	"nytt" subst appell nøyt ubøy
	"nytte" verb imp
<word>teppe</word>
"<teppe>"
	"teppe" subst appell nøyt ub ent
	"teppe" verb inf
<word>ca</word>
"<ca>"
	"ca" adv
	"ca." adv fork @adv>
<word>kr</word>
"<kr>"
	"kr" subst fork
<word>500</word>
"<500>"
	"500" det fl kvant
<word>.</word>
"<.>"
	"$."  adv fork prep+adj @adv
<word>.</word>
"<.>"
	"$." clb <<< <punkt>

When tagging the following non-Norwegian fragment (usually inside an otherwise Norwegian text, of course, but it also replicates when isolated), we get pipe symbols (not present in the source text) attached to tokens and punctuation symbols (present in the text) attached to them as well:
«… Fräulein von Kulp

may turn, her hand upon the door;

The pipe symbols mark the end of detected headlines, defined as "sentences" with no more than 8 words and no sentence final punctuation, but followed by a blank line. I have now added a check that the text contains at least one following non-blank line, and that it starts with a capitalized word, and then you get the desired result for your example (with another bugfix giving the correct analysis for the ellipsis):
"<«>"
	"$«" <anf>
"<…>"
	"$..." clb <ellipse>
"<fräulein>"
	"Fräulein" subst prop <*>
"<von>"
	"von" prep
	"von" subst appell fem ub ent
	"von" subst appell mask ub ent
	"vone" verb imp
"<kulp>"
	"Kulp" subst prop <*>
"<may>"
	"may" ukjent
"<turn>"
	"turn" subst appell mask ubøy
	"turne" verb imp
"<,>"
	"$," <komma>
	"$," clb <komma>
"<her>"
	"her" adv
	"her" prep
"<hand>"
	"hand" subst appell fem ub ent
	"hand" subst appell mask ub ent
"<upon>"
	"upon" ukjent
"<the>"
	"the" ukjent
"<door>"
	"door" subst appell fem ub ent samset-analyse <+or>
	"door" subst appell mask ub ent samset-analyse <-or> <+or>
"<;>"
	"$;" clb <semi> <<<

Additionally, if want to skip the detection of headlines altogether, you can now provide the -no-headlines flag.

More problems:

hyphen/dash disappears, but an extra full stop appears at the end of "sentence":

$ echo "Gymnastikk … 6.15-6.30 -"|mtag/mtag.py -wxml

$ echo "en colaflaske (plast, 1/2-liter)"|mtag/mtag.py -wxml

$ echo "19.30-sending"|mtag/mtag.py -wxml

The problem with the disappearing hyphen is fixed now. The full stop is appended because the rules of the Constraint Grammar component of The Oslo Bergen Tagger (which is the main target of the mtag output) expects sentences to be terminated. However, you can now apply the -no-add-stop if you want to prevent this from happening.

no results at all (in this way several sentences may completely disappear at the end of a text file):

Your example works now:
$ python3 mtag.py -wxml -no-add-stop
1 gammelt persisk teppe ca kr 700,-
1 nytt teppe ca kr 500,-
<word>1</word>
"<1>"
	"1" det ent kvant
<word>gammelt</word>
"<gammelt>"
	"gammel" adj nøyt ub ent pos
<word>persisk</word>
"<persisk>"
	"persisk" adj nøyt ub ent pos
	"persisk" adj ub m/f ent pos
	"persisk" subst appell mask ub ent
	"persisk" subst appell nøyt ubøy
<word>teppe</word>
"<teppe>"
	"teppe" subst appell nøyt ub ent
	"teppe" verb inf
<word>ca</word>
"<ca>"
	"ca" adv
	"ca." adv fork @adv>
<word>kr</word>
"<kr>"
	"kr" subst fork
<word>700,-</word>
"<700,->"
	"700,-" det <beløp>
<word>1</word>
"<1>"
	"1" det ent kvant
<word>nytt</word>
"<nytt>"
	"ny" adj nøyt ub ent pos
	"nyte" adj nøyt ub ent <perf-part>
	"nyte" adj ub m/f ent <perf-part>
	"nyte" verb perf-part
	"nytt" subst appell nøyt ubøy
	"nytte" verb imp
<word>teppe</word>
"<teppe>"
	"teppe" subst appell nøyt ub ent
	"teppe" verb inf
<word>ca</word>
"<ca>"
	"ca" adv
	"ca." adv fork @adv>
<word>kr</word>
"<kr>"
	"kr" subst fork
<word>500,-</word>
"<500,->"
	"500,-" det <beløp>

dash/hyphen changed to number ("700,-" changes into "700,1"):

This was not actually the hyphen being changed to a number, it was the tagger interpreting the hyphen as a mark that a word was split between two lines. Hence, it took the "700," from the first line and the "1" from the following line and concatenated them into what it believed was a word.
I have now added the constraint that the character preceding the hyphen must be a letter or a digit (e.g. "spise-bord" or "10-talls"), so now it works correctly, as the previous output shows.

an extra full stop appears after abbreviations (which may be an intention, but undesired for us, and I was told it is not consequent either!)

You can now provide the -no-add-stop flag to prevent this from happening.  adv fork prep+adj @adv
<word>.</word>
"<.>"
	"$." clb <<< <punkt>

Thanks, that is great! As for the question of terminating sentences: we have been lately discussing this again as well. I am not sure about the final decision, but my opinion is that sentences do not necessarily need to be terminated by a full stop. Many sentences end with another punctuation (question mark, exclamation mark, ellipsis, dash, etc.) or nothing at all (e.g. the headlines). So, in the case of abbreviations, I believe the last dot can still be considered part of the token and the sentence may remain without any additional termination symbol as well. If the parser needs a termination token, I think there is a simple solution - an "empty" token:
<word></word>
"<.>"
	"$." clb <<< <punkt>

But that is just my personal theoretical preference. I guess it does not really matter which solution is applied as long as the users are well informed about the solution chosen.

I totally agree that other solutions might be valid as well - now it just so happens that when no additional full stop is added (i.e., when the -no-add-stop flag is applied), the tagger interprets the final dot as a sentence terminator and tags it that way, leaving the rest of the abbreviation as a separate token.
Since the final dot is kind of filling two roles simultaneously (as part of the abbreviation and as end-of-sentence marker), I agree that both solutions would be acceptable, but since this solution follows naturally while the one you suggest would require additional programming, I think I am going to leave it like this at the moment.
And of course, as you mention, there are a range of other characters that may mark the end of a sentence - in addition to full stop, question marks, colons, exclamation marks and ellipses are considered end-of-sentence markers by the tagger (and marked with '<<<') if they are followed by a capitalized word.

@wanthalf is it OK to close this issue now?

Whatever is best for you. We have not tested it yet. We may come with more feedback later and either add it here or open a new issue. What is better?

Ok, I'll close the issue, and if there are problems directly related to the examples you have given here, you could just reopen it. If you discover unrelated problems I think a new issue would be better.


Support JOIN

Hi,
Thanks for this project. I was looking for a SQL-like interpreter to learn how it works and I found it really useful.
Do you have plans / ideas how to support JOIN across records?

I don’t think it’s feasible given that charlatan works on stream-like data and a JOIN operator requires indexed (or at least sorted) fields.

So, given those two JSON, a JOIN is impossible even if we pass the two JSON files as records?
people.json
{"name": "Nico", "age": 54, "stats": {"walking": 42, "running": 104, "biking": 13}, "country_id": 2}
{"name": "Vincent", "age": 32, "stats": {"walking": 100, "running": 2, "biking": 3}, "country_id": 1}
{"name": "Cedric", "age": 52, "stats": {"walking": 2, "running": 2, "biking": 1000}, "country_id": 3}
{"name": "Michel", "age": 83, "stats": {"walking": 0, "running": 0, "biking": 0}, "country_id": 2}
{"name": "Gerard", "age": 55, "stats": {"walking": 54, "running": 12, "biking": 33}, "country_id": 4}

countries.json
{"id": 1, "code": "USA"}
{"id": 2, "code": "UK"}
{"id": 3, "code": "Spain"}
{"id": 4, "code": "France"}


stage0: use rootfs image permissions with overlay

When we use overlay, the permissions that end up in "rootfs/" are the
ones in the upper directory. To fix it, we need to set them as they are
in the image.
Fixes #1581

@krnowak can you explain the mentioned future changes in "We hardcode the permissions for now because this part of the code will be removed soon."?

In tests/rkt_userns_test.go, there are a couple of "TODO #1581". Can you fix them as well?

@krnowak can you explain the mentioned future changes in "We hardcode the permissions for now because this part of the code will be removed soon."?

It was actually me who wrote that :D. @krnowak told me that code will be gone with the fix for https://github.com/coreos/rkt/issues/1420. I'll add it to the commit message.

In tests/rkt_userns_test.go, there are a couple of "TODO #1581". Can you fix them as well?

Yes, I'll do that.

One question above.
The build system part is "ew!", but other parts LFAD.

In tests/rkt_userns_test.go, there are a couple of "TODO #1581". Can you fix them as well?

Yes, I'll do that.

I read tests/rkt_userns_test.go again and it actually only tests with --no-overlay. We would like to test this:
| With userns | Without userns
------------ | ------------ | -------------
With overlay | :x: (not testable, see #1057) | :interrobang: (if possible; Semaphore does not support overlay)
Without overlay | :white_check_mark: (always) |  :white_check_mark: (always)

I tested with overlay enabled on my machine (removing the rkt group because of #1602) and the test passes.
It'd be great to run all the tests with and without overlay but Semaphore doesn't support overlay...

Added a comment about testing with overlay fs.

LGTM but can you add a follow-up issue/PR for the overlay fs test (even if it is skipped on Semaphore)?

https://github.com/coreos/rkt/issues/1613

Thanks!


Sorting problem

All data disappears after you go back to the non sort state,
asc ->  desc -> no sort (data gone)
while if you sort by another column before you go back to no sort on the same column, it is ok.

I can't reproduce this,
is this the JSON behavior?

yes the JSON behavior
and in the console i noticed
Uncaught TypeError: Cannot set property '__i' of undefined

that means one of the json objects you've provided was replaced with undefined somehow.
in order to get back to unsorted state I put __i index on the objects in order to have the native sort be stable

I'm going to close this for now as I can't seem to reproduce it.  Let me know if it still is happening for you and we can dig in a little deeper.

i ran it through debugger, found that inside the toggleSort function
this copying makes use of __si  , but this is undefined
         if (sortStateIndex === 0) {
                var newData = new Array(this.data.length);
                for (i = 0; i < this.data.length; i++) {
                    var each = this.data[i];
                    newData[each.__si] = each;
         }
                this.data = newData;
            }

are you inserting new objects into the collection after things are running?
if so things need to be initialized.
On Mon, May 4, 2015 at 11:35 PM, eduardotang<EMAIL_ADDRESS>wrote:

i ran it through debugger, found that inside the toggleSort function
this copying makes use of __si , but this is undefined
     if (sortStateIndex === 0) {
            var newData = new Array(this.data.length);
            for (i = 0; i < this.data.length; i++) {
                var each = this.data[i];
                newData[each.__si] = each;
     }
            this.data = newData;
        }

—
Reply to this email directly or view it on GitHub
https://github.com/openfin/fin-hypergrid/issues/28#issuecomment-98929609
.

so everytime after i insert new objects , i have to call initDataIndexes?

yes, unless you don't care about stable sorting and being able to go back
to the initial unsorted state
On Tue, May 5, 2015 at 12:44 PM, eduardotang<EMAIL_ADDRESS>wrote:

so everytime after i insert new objects , i have to call initDataIndexes?
—
Reply to this email directly or view it on GitHub
https://github.com/openfin/fin-hypergrid/issues/28#issuecomment-99136159
.

I've just done a small refactoring, so if you change the underlying data
call "dataChanged()" on the behavior object...
    setData: function(jsonData) {
        this.data = jsonData;
        this.dataChanged();
    },

    dataChanged: function() {
        this.initDataIndexes();
        this.initColumnIndexes();
        this.changed();
    },

let me know how things work out....
On Tue, May 5, 2015 at 12:44 PM, eduardotang<EMAIL_ADDRESS>wrote:

so everytime after i insert new objects , i have to call initDataIndexes?
—
Reply to this email directly or view it on GitHub
https://github.com/openfin/fin-hypergrid/issues/28#issuecomment-99136159
.

now the data wont be gone, and it stays at the descending state.
it's good enough.
thx

I'll create an issue and implement a more robust solution,
Hypergrid should do something reasonable after inserting new data.
On Wed, May 6, 2015 at 4:38 AM, eduardotang<EMAIL_ADDRESS>wrote:

now the data wont be gone, and it stays at the descending state.
it's good enough.
thx
—
Reply to this email directly or view it on GitHub
https://github.com/openfin/fin-hypergrid/issues/28#issuecomment-99383444
.

your last commit move the call to initDataIndexes to setData, cause problem adding data to original array again

reverted,
thanks :)
On Sun, May 17, 2015 at 6:55 AM, eduardotang<EMAIL_ADDRESS>wrote:

your last commit move the call to initDataIndexes to setData, cause
problem adding data to original array again
—
Reply to this email directly or view it on GitHub
https://github.com/openfin/fin-hypergrid/issues/28#issuecomment-102783172
.


Lab files target unclear

Lab/Demo: 00
Exercise: Exercise 5 – Copy Lab Resources
Task: 00
Step: 01 - Copy the lab resources folder to C:\Labs
Description of issue
Is the target for the C:\Labs directory the VM just created or the Lab or local workstation?
Repro steps:

Hi Chris, the lab files just have to be on the same place on the machine you run Power Automate Desktop and the VM you deploy in the lab for unattended execution. For simplicity, c:\Labs on both but you can use any path as long as you use the same in both places.

Thanks David, I wasn’t sure. Any updates on getting the bank app to install properly? Thanks!
Chris Utter
Worldwide Learning
Microsoft

From: David Yack @.>
Sent: Tuesday, July 5, 2022 3:19:35 PM
To: MicrosoftLearning/PL-500T00-Microsoft-Power-Automate-RPA-Developer @.>
Cc: Chris Utter @.>; Author @.>
Subject: Re: [MicrosoftLearning/PL-500T00-Microsoft-Power-Automate-RPA-Developer] Lab files target unclear (Issue #2)
Hi Chris, the lab files just have to be on the same place on the machine you run Power Automate Desktop and the VM you deploy in the lab for unattended execution. For simplicity, c:\Labs on both but you can use any path as long as you use the same in both places.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftLearning%2FPL-500T00-Microsoft-Power-Automate-RPA-Developer%2Fissues%2F2%23issuecomment-1175461981&data=05|01|chris.utter%40microsoft.com|7c04db9667784859391b08da5ec3ae5f|72f988bf86f141af91ab2d7cd011db47|1|0|637926491792482136|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=8LuTH7Jd%2FI%2BWov3PC7I2OyRos%2BTvarL5e%2FmoMrjTEJs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAP7VVV4VMPSCU7DDXD4UUUTVSSKFPANCNFSM52J4NAQA&data=05|01|chris.utter%40microsoft.com|7c04db9667784859391b08da5ec3ae5f|72f988bf86f141af91ab2d7cd011db47|1|0|637926491792482136|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=aCW4ew0j4LFm2b4LnVWky6hnkIL5RV4%2FoOvLmcnGDxo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


fix(plugins/plugin-kubectl): getDirect try/catch does not handle erro…

…rs with no code
Fixes #7239

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

/lgtm


added callback to catch

Hi,
somehow it seems that the catch of the waitForUser promise is not that much into catching if it is not called with a catch handler
import angular from 'angular'
import 'angular-meteor'
import 'angular-meteor-auth'

angular.module('app',['angular-meteor','angular-meteor.auth'])
.run(function($auth,$log){
  $log.info('$auth testing');
  $auth.requireUser().then(u=>{
    $log.info('requireUser SUCCESS',u);
  }, err=>{
    $log.warn('requireUser ERROR',err);
  });
  $auth.waitForUser().then(u=>{
    $log.info('waitForUser SUCCESS',u);
  }, err=>{
    $log.warn('waitForUser ERROR',err);
  });
})

Meteor.startup(function(){
  angular.bootstrap(document.body, ['app']);
})

Has a the same result in my test app.
When I put in a handler like in the PR it works as expected.

@Urigo any thoughts on that?

@mxab The job of waitForUser is just to return a promise and resolve it once the user has logged in, therefore it should not return any error. Does it make any sense? Well it's not even something to discuss about since it only exists to implement an older API of angular-meteor-auth. Instead I recommend using awaitUser which was implemented in the new API and it the functionalities of waitForUser, requireUser and requireValidUser all at once. For more information see our docs page.

@DAB0mB I'm quite aware of what waitForUser does, you got me wrong. My problem is that they currently act the same
both result in error if there is no user
See this sample application
https://github.com/mxab/angular-meteor-auth-issue/blob/master/client/main.js
The log output is:
$auth testing
requireUser ERROR AUTH_REQUIRED
waitForUser ERROR AUTH_REQUIRED

About the awaitForUser, sure I could use this as well and simply create my own waitFor/requireUser wrappers around it but why should I if this package provides it already. I actually thought they weren't simply in the docs, since the one you pointed out is about 0.3.0 and the current one is 1.0.0

Hey, I referenced a pull request which should fix this issue. Can you please confirm it that the issue has been fixed? It would be great!

Sure, I'll give it a try.
Two things tough, at one point someone should figure out why the catch with no handler results in not catching it and the the point is just a very personal opinion that catch blocks should never silently swallow stuff as it might give debugging nightmares (I'm talking to you Blaze), that was the reason it put a debug log there

was addressed here:  https://github.com/Urigo/angular-meteor-auth/pull/23


Build crash at 1%

Build Process
Clone from AUR and run makepkg. I think I followed the instruction pretty close... download all the dependencies, include optional ones. I am not using Ananconda, just python. Also googled around build() error, and still can't figure out...
Build Log
==> Making package: polybar 3.4.0-4 (Mon Sep 30 19:02:04 2019)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Found polybar-3.4.0.tar
==> Validating source files with sha256sums...
    polybar-3.4.0.tar ... Passed
==> Extracting sources...
  -> Extracting polybar-3.4.0.tar with bsdtar
==> Starting prepare()...
==> Removing existing $pkgdir/ directory...
==> Starting build()...
-- Trying to enable ccache
-- Using compiler cache /usr/bin/ccache
-- Could not detect version with git, falling back to built-in version information.
-- No build type specified; using Release
-- Using supported compiler GNU-9.1.0
--  Build:
--    Version: 3.4.0
--    Type: Release
--    CXX: /usr/bin/c++ -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -O3 -DNDEBUG
--    LD: /usr/bin/ld -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now
--  Install Paths:
--    PREFIX:  /usr
--    BINDIR:  /usr/bin
--    DATADIR: /usr/share
--    DOCDIR:  /usr/share/doc/polybar
--    MANDIR:  /usr/share/man
--  Targets:
-- [X]   polybar-msg
-- [ ]   testsuite
-- [X]   documentation
--  Module support:
-- [X]   alsa (1.1.9)
-- [X]   curl (7.66.0)
-- [X]   i3
-- [X]   mpd (2.16)
-- [X]   network (libnl) (3.5.0)
-- [X]   pulseaudio (13.0-dirty)
-- [X]   xkeyboard (1.13.1)
--  X extensions:
-- [X]   xcb-randr (1.13.1)
-- [X]   xcb-randr (monitor support) (1.13.1)
-- [X]   xcb-composite (1.13.1)
-- [X]   xcb-xkb (1.13.1)
-- [X]   xcb-xrm (1.3)
-- [X]   xcb-cursor (0.1.3)
--  Log options:
-- [X]   Trace logging
-- XCB[XCB]: Found component XCB
-- XCB[ICCCM]: Found component ICCCM
-- XCB[EWMH]: Found component EWMH
-- XCB[UTIL]: Found component UTIL
-- XCB[IMAGE]: Found component IMAGE
-- Found XCB: /usr/lib/libxcb.so;/usr/lib/libxcb-icccm.so;/usr/lib/libxcb-ewmh.so;/usr/lib/libxcb-util.so;/usr/lib/libxcb-image.so
-- Searching for xcbgen with python2
-- Searching for xcbgen with python2.7
-- Searching for xcbgen with python3
-- Found xcbgen in /usr/lib/python3.7/site-packages
-- XCB[RANDR]: Found component RANDR
-- Found XCB: /usr/lib/libxcb.so;/usr/lib/libxcb-icccm.so;/usr/lib/libxcb-ewmh.so;/usr/lib/libxcb-util.so;/usr/lib/libxcb-image.so;/usr/lib/libxcb-randr.so
-- XCB[COMPOSITE]: Found component COMPOSITE
-- Found XCB: /usr/lib/libxcb.so;/usr/lib/libxcb-icccm.so;/usr/lib/libxcb-ewmh.so;/usr/lib/libxcb-util.so;/usr/lib/libxcb-image.so;/usr/lib/libxcb-randr.so;/usr/lib/libxcb-composite.so
-- XCB[XKB]: Found component XKB
-- Found XCB: /usr/lib/libxcb.so;/usr/lib/libxcb-icccm.so;/usr/lib/libxcb-ewmh.so;/usr/lib/libxcb-util.so;/usr/lib/libxcb-image.so;/usr/lib/libxcb-randr.so;/usr/lib/libxcb-composite.so;/usr/lib/libxcb-xkb.so
-- xpp: including xcb proto composite.xml
-- xpp: including xcb proto randr.xml
-- xpp: including xcb proto xkb.xml
-- xpp: including xcb proto xproto.xml
-- Font not found: fixed:pixelsize=10
-- Font not found: unifont:fontformat=truetype
-- Font not found: siji:pixelsize=10
-- Configuring done
-- Generating done
-- Build files have been written to: /home/will/Downloads/polybar/src/polybar/build
[  1%] sphinx-build html: see doc/builder-html.log
Traceback (most recent call last):
  File "/usr/bin/sphinx-build", line 11, in <module>
    load_entry_point('Sphinx==2.2.0', 'console_scripts', 'sphinx-build')()
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2852, in load_entry_point
    return ep.load()
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2443, in load
    return self.resolve()
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python3.7/site-packages/sphinx/cmd/build.py", line 23, in <module>
    from sphinx.application import Sphinx
  File "/usr/lib/python3.7/site-packages/sphinx/application.py", line 27, in <module>
    from sphinx.config import Config
  File "/usr/lib/python3.7/site-packages/sphinx/config.py", line 23, in <module>
    from sphinx.util.i18n import format_date
  File "/usr/lib/python3.7/site-packages/sphinx/util/i18n.py", line 19, in <module>
    import babel.dates
  File "/usr/lib/python3.7/site-packages/babel/dates.py", line 29, in <module>
    from babel.util import UTC, LOCALTZ
  File "/usr/lib/python3.7/site-packages/babel/util.py", line 20, in <module>
    from babel import localtime
  File "/usr/lib/python3.7/site-packages/babel/localtime/__init__.py", line 74, in <module>
    LOCALTZ = get_localzone()
  File "/usr/lib/python3.7/site-packages/babel/localtime/__init__.py", line 70, in get_localzone
    return _get_localzone()
  File "/usr/lib/python3.7/site-packages/babel/localtime/_unix.py", line 125, in _get_localzone
    with open(tzpath, 'rb') as tzfile:
IsADirectoryError: [Errno 21] Is a directory: '/etc/localtime'
make[2]: *** [doc/CMakeFiles/doc_html.dir/build.make:58: doc/CMakeFiles/doc_html] Error 1
make[1]: *** [CMakeFiles/Makefile2:334: doc/CMakeFiles/doc_html.dir/all] Error 2
make: *** [Makefile:130: all] Error 2
==> ERROR: A failure occurred in build().
    Aborting...

Environment:
Arch Linux
Additional context
I also tried installation instructions from markdown, and below is what output after I run
sudo make install
Scanning dependencies of target doc_html
[  1%] sphinx-build html: see doc/builder-html.log
Traceback (most recent call last):
  File "/usr/bin/sphinx-build", line 11, in <module>
    load_entry_point('Sphinx==2.2.0', 'console_scripts', 'sphinx-build')()
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2852, in load_entry_point
    return ep.load()
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2443, in load
    return self.resolve()
  File "/usr/lib/python3.7/site-packages/pkg_resources/__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python3.7/site-packages/sphinx/cmd/build.py", line 23, in <module>
    from sphinx.application import Sphinx
  File "/usr/lib/python3.7/site-packages/sphinx/application.py", line 27, in <module>
    from sphinx.config import Config
  File "/usr/lib/python3.7/site-packages/sphinx/config.py", line 23, in <module>
    from sphinx.util.i18n import format_date
  File "/usr/lib/python3.7/site-packages/sphinx/util/i18n.py", line 19, in <module>
    import babel.dates
  File "/usr/lib/python3.7/site-packages/babel/dates.py", line 29, in <module>
    from babel.util import UTC, LOCALTZ
  File "/usr/lib/python3.7/site-packages/babel/util.py", line 20, in <module>
    from babel import localtime
  File "/usr/lib/python3.7/site-packages/babel/localtime/__init__.py", line 74, in <module>
    LOCALTZ = get_localzone()
  File "/usr/lib/python3.7/site-packages/babel/localtime/__init__.py", line 70, in get_localzone
    return _get_localzone()
  File "/usr/lib/python3.7/site-packages/babel/localtime/_unix.py", line 125, in _get_localzone
    with open(tzpath, 'rb') as tzfile:
IsADirectoryError: [Errno 21] Is a directory: '/etc/localtime'
make[2]: *** [doc/CMakeFiles/doc_html.dir/build.make:58: doc/CMakeFiles/doc_html] Error 1
make[1]: *** [CMakeFiles/Makefile2:334: doc/CMakeFiles/doc_html.dir/all] Error 2
make: *** [Makefile:130: all] Error 2

IsADirectoryError: [Errno 21] Is a directory: '/etc/localtime'

This seems like an arch installation error on your side. /etc/localtime is generally a symlink not a directory. See the Arch Wiki


Duplicated actions are found

I notice that the same action is found multiple times, and creates duplicates
Is the search query maybe searching on all the branches? Meaning if there is let's say a develop and main branch, it will find the same action twice?
This is of course very confusing when people trying to lookup Actions in the marketplace. Maybe only stick to main or master? Or.. you need to detect duplicates some how using binary file comparison or checksums for instance. Or any other way to make the result unique (using org + repo + action name) as a unique ID.

No it does not. The search API we use here only searches the default branch by default, see the docs here.
Closing this issue.


Selecting unicode characters does not highlight correctly

reproduce:

start-> run "control intl.cpl" -> administrative

now reboot (the windows system UI language will not be changed , just the text display of programs that do not support Unicode

Fixed in 4443ee11f98a7848ef0e11f02feb2a010499cfd6

all should work now in v1.0.33 , please let me know if something is still not working
https://github.com/CobaltFusion/DebugViewPP/releases/tag/v<IP_ADDRESS>


Fix: Remove phpmd/phmd

Since there seems to be some consensus on removing phpmd/phpmd, this PR removes it.

@localheinz thanks!

Sure, you're welcome!


Knockout example doesn't work with 4.17.42

This sample code makes the date picker behave really strangely.  For any month, when you click day 1 to 12 it sets that as the month and 20 as the day
JS Fiddle demo: https://jsfiddle.net/w4dt40pw/
It works fine with 4.17.37

https://jsfiddle.net/aptajosh/2xc34wd9/


Initialize window after window is created

This prevents race conditions when mb.preloadWindow is set to false.
This would allow to disable window rendering if the user is not logged in yet.

@seppestas Is this one still good to merge?

Yes, it might a good idea to squash the commits though...

A couple of remarks:

It seems to break the initial login flow
It's not really resolving an issue that users have reported

Based on this I've decided to close this issue and not merge it.


Updated MMD related docs to new article author

Related to the info provided by jaimeo on PR #353

@jaimeo - PR is ready for final review.
Thank you.


Locale change banner design specs

Jacob will move this work to the Ideation board.  When done this issue will be closed.


Smith-waterman test broken

It looks like the TestSmithWatermanDecoder test is failing under test_sw_cuda.py.
https://github.com/flatironinstitute/deepblast/pull/145

the sw tests are just regression tests, so a bugfix in 81ac6e36 broke this. Will make a PR with updated test soon


Relative path to compile.log

oyster% gem install nokogiri -v '1.6.1'
Building native extensions.  This could take a while...
ERROR:  Error installing nokogiri:
        ERROR: Failed to build gem native extension.

        /home/me/.rbenv/versions/1.9.3-p484/bin/ruby extconf.rb
Extracting libxml2-2.8.0.tar.gz into tmp/x86_64-linux-gnu/ports/libxml2/2.8.0... OK
Running 'configure' for libxml2 2.8.0... OK
Running 'compile' for libxml2 2.8.0... ERROR, review 'tmp/x86_64-linux-gnu/ports/libxml2/2.8.0/compile.log' to see what happened.
*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of
necessary libraries and/or headers.  Check the mkmf.log file for more
details.  You may need configuration options.

Provided configuration options:
        --with-opt-dir
        --without-opt-dir
        --with-opt-include
        --without-opt-include=${opt-dir}/include
        --with-opt-lib
        --without-opt-lib=${opt-dir}/lib
        --with-make-prog
        --without-make-prog
        --srcdir=.
        --curdir
        --ruby=/home/me/.rbenv/versions/1.9.3-p484/bin/ruby
/home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/mini_portile-0.5.2/lib/mini_portile.rb:265:in `block in execute': Failed to complete compile task (RuntimeError)
        from /home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/mini_portile-0.5.2/lib/mini_portile.rb:257:in `chdir'
        from /home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/mini_portile-0.5.2/lib/mini_portile.rb:257:in `execute'
        from /home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/mini_portile-0.5.2/lib/mini_portile.rb:69:in `compile'
        from /home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/mini_portile-0.5.2/lib/mini_portile.rb:109:in `cook'
        from extconf.rb:101:in `block in '
        from extconf.rb:119:in `call'
        from extconf.rb:119:in `block in '
        from extconf.rb:109:in `tap'
        from extconf.rb:109:in `'

Gem files will remain installed in /home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/nokogiri-1.6.1 for inspection.
Results logged to /home/me/.rbenv/versions/1.9.3-p484/lib/ruby/gems/1.9.1/gems/nokogiri-1.6.1/ext/nokogiri/gem_make.out

The important line is this:

Running 'compile' for libxml2 2.8.0... ERROR, review 'tmp/x86_64-linux-gnu/ports/libxml2/2.8.0/compile.log' to see what happened.

That is not an absolute path, therefore I don't know where to find this file.
 find /tmp -name compile.log finds nothing.
A separate issue is nokogiri appears to build libxml2 when the latter is already installed on my system.

Hi,
Thanks for asking this question. I apologize for not responding earlier, but there was a period of time where github issues weren't watched closely.
The gem compilation stage is generally done in a directory determined by your Ruby (or your Ruby version manager, e.g. rvm / rbenv / what-have-you). That's generally where you'll need to look for that compile.log file.
You may want to ask questions like this on the nokogiri-talk mailing list. Details are at http://www.nokogiri.org/tutorials/getting_help.html


Item in enum is not of correct class type

Using pykwalify 1.6.1 seeing this error:
RuleError: <RuleError: error code 4: Item: 'linear' in enum is not of correct class type: '<type 'str'>': Path: '/mapping/execution/mapping/progression'>
The schema is here: https://github.com/openstack/shaker/blob/master/shaker/resources/schemas/scenario.yaml
And the YAMl is here: https://github.com/openstack/shaker/blob/master/shaker/scenarios/openstack/dense_l3_north_south.yaml
Is this an error on my end or did something change?

I tried the linked schema/data combo and i get OK validation. Could it be that you have loaded the data or schema wrong? Could you debug your output with -vvvvv and that the data is proper strings.


Error : destination size is smaller than source size

When trying to migrate a VM, i got this error :
nbdcopy: error: destination size<PHONE_NUMBER>6) is smaller than source size<PHONE_NUMBER>0)

I've solved this here:

Indeed, solved


the server could not find the requested resource error

What did you do
can I ask for advice you?
I tried installed rancher on my k8s cluster but dose not worked rancher
I have no idea, help me....

How was the cluster created?

k8s

What did you do afterwards?

k3d commands?
docker commands?
OS operations (e.g. shutdown/reboot)?

Screenshots or terminal output
2021/08/20 16:34:37 [INFO] Rancher version v2.5.9 (3c5418944) is starting
2021/08/20 16:34:37 [INFO] Rancher arguments {ACMEDomains:[] AddLocal:true Embedded:false BindHost: HTTPListenPort:80 HTTPSListenPort:443 K8sMode:auto Debug:false Trace:false NoCACerts:false AuditLogPath:/var/log/auditlog/rancher-api-audit.log AuditLogMaxage:10 AuditLogMaxsize:100 AuditLogMaxbackup:10 AuditLevel:0 Agent:false Features: ClusterRegistry:}
2021/08/20 16:34:37 [INFO] Listening on /tmp/log.sock
2021/08/20 16:34:37 [FATAL] the server could not find the requested resource

If applicable, add screenshots or terminal output (code block) to help explain your problem.
Which OS & Architecture

Linux amd64

Which version of docker
Client:
Context:    default
Debug Mode: false
Plugins:
app: Docker App (Docker Inc., v0.9.1-beta3)
buildx: Build with BuildKit (Docker Inc., v0.6.1-docker)
scan: Docker Scan (Docker Inc., v0.8.0)
Server:
Containers: 40
Running: 20
Paused: 0
Stopped: 20
Images: 27
Server Version: 20.10.8
Storage Driver: overlay2
Backing Filesystem: extfs
Supports d_type: true
Native Overlay Diff: true
userxattr: false
Logging Driver: json-file
Cgroup Driver: systemd
Cgroup Version: 1
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
Default Runtime: runc
Init Binary: docker-init
containerd version: e25210fe30a0a703442421b0f60afac609f950a3
runc version: v1.0.1-0-g4144b63
init version: de40ad0
Security Options:
apparmor
seccomp
Profile: default
Kernel Version: 4.15.0-154-generic
Operating System: Ubuntu 18.04.5 LTS
OSType: linux
Architecture: x86_64
CPUs: 6
Total Memory: 7.631GiB
Name: master-1
ID: NX6I:UWWM:W2W2:KBVR:N6A6:JSE6:CTX4:FTGI:ASO2:RBYH:USM3:SIVU
Docker Root Dir: /var/lib/docker
Debug Mode: false
Registry: https://index.docker.io/v1/
Labels:

hi @kerry-Cho , I am curious why you opened this, marked it as a bug and closed it?
What you describe does not seem to have any relation to k3d.
There is also not enough information to give enough clue to what you did.
I suggest to ask on the forum or stackoverflow if you have a question that is not specific enough to be handled as a bug.
People get emails and spend time on these.
If you still have an issue, please use the rancher project to file any bug and make sure you have at least :

the steps or process you followed for installation
the exact issue that you encounter, preferrably a message from the log or a stacktrace

Rancher installation can be found here :
https://rancher.com/docs/rancher/v2.x/en/installation/
as you can see, it matters a lot what kind of installation path you follow.
Please do not respond to this bug, because that too will send email.
thank you


PageView counter at top of the posts using Goatcounter

Checklist

[X] I have read and accepted the contributing guidelines.

Is your feature request related to a problem? Please describe
Add page view count on top of the blog post.
Describe the solution you'd like
Recently support for goatcounter has been provided. As per this documentation it is very much possible to print page view on our posts.
We can call the API and get the page count as follows -
// Fetch JSON data from the provided URL
fetch('https://[MYCode].goatcounter.com/counter/[relative URL].json')
  .then(response => response.json())
  .then(data => {
    console.log(data.count);
  })
  .catch(error => console.error('Error fetching JSON:', error));

I am not very good at web development, and hence I am not able to integrate it with the templates. Even adding to the index.html page gives errors (Error fetching Json).

If it helps, Chatgpt 3.5 states the following as a possible error "If you're able to fetch the JSON data successfully using the same JavaScript code in a .js file when running it with Node.js but encountering an error when trying to fetch it within an HTML file, the issue might be related to CORS (Cross-Origin Resource Sharing) restrictions."
From my understanding of the chirpy template, one has to add a working HTML code (shown above) in post.html. And add some CSS to make it look clean. Maybe display it to the right, alongside posted and updated (at the start of the post), as in below.

Describe alternatives you've considered
No response
Additional context
No response

@sukhjinder-kumar  I have implemented views counter recently using GoatCounter, here what I have done.

I have copied original _layouts/post.html to my project, so I can override it to add views counter where I want it to be

        {% include views.html tooltip=true lang=lang %}
        {{ site.data.locales[lang].post.pageview_measure }}

Created a new file _includes/views.html to have views counter logic

    ∞

The JS code is taken from GoatCounter site https://www.goatcounter.com/help/visitor-counter#json-851
That's about it. I hope this helps you get what you want while waiting for the changes in the theme.

Thank you for taking your time and helping me out. This solves my issue. Just to be sure, is there something we can do so that Adblocker doesn't block the request? I saw it on your github.io too, ublock is blocking the request (works in incognito).

New to open source, but you should release the above code as PR. I think many people will find it useful. Thanks once again.

I was happy to help @sukhjinder-kumar

is there something we can do so that Adblocker doesn't block the request?

I don't think so. If user decided to protect his privacy and so Adblock extension prevents to send requests to analytics, there is no way to bypass easily, I guess.
Please check additional info available in GoatCounter's FAQ regarding this problem as there are options described.


Added Multifunctioned Notes app

🛠 Fixes Issue #640

👨💻 Changes proposed
Added Multifunctioned Notes app

✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project (Zero Octave-Javascript-Projects).
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
GSSoC Contributor

📷 Screenshots
https://user-images.githubusercontent.com/82600388/167369948-540c4227-faf0-41c7-879a-6793eb2b2b7e.mp4

Directory not followed

@Astrodevil Please review now

update your forked repo once

Done @NOiR-07 Please review now

also add your project in readme file

@aaryahjolia make new PR properly


[Visual] Mobile View Columns Cutoffs

I was testing the app out on my mobile and noticed that the date column is cutoff in all but the fuel tab.  I am on an iphone 13 using the safari browser with the latest ios if that helps.

See #112

See #112

Sorry i thought one was firefox specific.


New version: JetBrains.Rider.EAP version 241.14494.107

Created by 🥟 Dumplings in Run #3758.
Microsoft Reviewers: Open in CodeFlow

@wingetbot run

This package installs and launches normally on a Windows 10 VM.

This package installs and launches normally on a Windows 10 VM.


no-extraneous-dependencies: better root package.json support

our setup looks like this:
├── package.json
├── .eslintrc.js
├── Avatar
│   └── package.json
├── Button
│   └── package.json
Avatar and Button reuse devDependencies from root package.json but they don't share much more
(Avatar and Button is used by 3rd component in same level)
if I try this config in .eslintrc.js I can get green in Button if devDependency is in Avatar only and missing in root.
            packageDir: [
              './', // mono-repo root
              './Avatar',
              './Button',
            ],

at the moment I need to replicate dev dependencies in all package.json to get around although setup is supported by node module resolution logic.

Scratch that.
we agreed in team that we're doing things wrong and should not rely on dev dependencies in root of the projects.

We agreed in team that we're doing things wrong and should not rely on dev dependencies in root of the projects.


AHC holds onto its body stream writer even after the request is complete

AHC seems to hold onto its body streaming delegate closure even after the request is complete. That means it's pretty much impossible to use it for bidi streaming without leaking it.
test case
    func testStuff() throws {
        let group = MultiThreadedEventLoopGroup(numberOfThreads: 1)
        defer {
            XCTAssertNoThrow(try group.syncShutdownGracefully())
        }

        let server = try ServerBootstrap(group: group)
            .childChannelInitializer { channel in
                channel.eventLoop.scheduleTask(in: .milliseconds(500)) {
                    channel.writeAndFlush(ByteBuffer(string: "HTTP/1.1 200 ok\r\ncontent-length: 2\r\n\r\nok")).map {
                        channel.close(promise: nil)
                    }.whenComplete { result in
                        print("server write complete", result)
                    }
                }
                return channel.eventLoop.makeSucceededFuture(())
            }
            .bind(host: "127.1", port: 0).wait()
        defer {
            XCTAssertNoThrow(try server.close().wait())
        }
        var request = try HTTPClient.Request(url: "http://\(server.localAddress!.ipAddress!):\(server.localAddress!.port!)/",
                                             method: .POST)
        let writerPromise = self.group.any().makePromise(of: HTTPClient.Body.StreamWriter.self)
        let donePromise = self.group.any().makePromise(of: Void.self)
        class LeakDetector {
            init() {
                print("INIT")
            }

            deinit {
                print("DEINIT")
            }
        }
        var leakDetector: LeakDetector? = LeakDetector()
        request.body = .stream { [leakDetector] writer in
            _ = leakDetector
            writerPromise.succeed(writer)
            return donePromise.futureResult
        }
        weak var weakLeak: LeakDetector? = leakDetector
        leakDetector = nil

        let resultFuture = self.httpClient.execute(request: request)
        request.body = nil // this is pointless but whatever
        writerPromise.futureResult.whenSuccess { writer in
            writer.write(.byteBuffer(ByteBuffer(string: "hello"))).map {
                print("written")
            }.cascade(to: donePromise)
        }
        print("HTTP sent")

        let result = try resultFuture.wait()

        XCTAssertEqual(.ok, result.status)
        XCTAssertEqual("ok", result.body.map { String(buffer: $0) } ?? "none")
        print("HTTP done", result)
        while weakLeak != nil {
            print("not nil")
            sleep(1)
        }
    }

Presumably the easiest fix is to stop holding onto Request.body in the RequestBag type once we've started sending the body.
The retain cycle is something like RequestBag -> Request -> (if it captures a promise to be fulfilled with BodyStreamWiter) BodyStreamWriter -> RequestBag


Failed to marshal state to json: unsupported attribute

What happens:
Executing tftui results in early exit during startup.
The TUI loads, and shows:
Executing Terraform init
Executing Terraform show

Then exits, with the message
unsupported attribute "http_protocol_ipv6"
and prints my Terraform state
This was on Terraform CLI v1.3.5, I upgraded it to v1.5.5, tftui still exits after Executing Terraform show, the error message is a little longer:
Failed to marshal state to json: unsupported attribute "http_protocol_ipv6"
but it doesn't print my Terraform state
I checked my Terraform state, and this appears to be an attribute of an aws_instance, under metadata_options
I tried switching to another Terraform repo, tftui exits early there too, this time with:
Failed to marshal state to json: unsupported attribute "secondary_allocation_ids"
I checked my Terraform state, and this appears to be an attribute of an aws_nat_gateway
Versions:
tftui v0.4.2
Installed via pip install tftui
macOS 13.5 (22G74)
M1 Max
Python 3.11.2
Terraform CLI v1.3.5 and v1.5.5
My Terraform state is stored remotely on Terraform Cloud, but tftui / terraform CLI is able to read the state
What I'd like to happen:

Graceful handling of unsupported attributes

Steps to Reproduce:

Run tftui

Those two error messages are originated in the terraform command itself.
Looks like a known bug, you first need to fix the state. Also, consider upgrading your terraform cli version.
https://github.com/hashicorp/terraform-provider-aws/issues/31637

Thank you for your help @idoavrah!
The cause of the issue for me is that the AWS Terraform Provider and Terraform CLI on my laptop were a few versions behind the provider and binary used on Terraform Cloud (latest), so I had to run:
brew upgrade terraform
terraform init -upgrade

I also ran a refresh-only Terraform apply via the TFC web UI, I'm not sure if that was required.


Feature Request: Warts.MapValues and Warts.FilterKeys

mapValues and filterKeys on MapLike don't create a new collection, but instead create a view on the values or keys of the map. This is possibly a huge performance impact since their names do not imply this is what they do.
It could be great to have a wart that warns when these methods are used.

Depending on usage patterns, these could also introduce a huge performance benefit (and I have utilized that myself). I'm not sure warning against their usage is warranted in light of that.

You're right - the use case does count. However, the desire to use this functionality should be explicitly stated by using a stream or a view, which is the standard way. This is a pitfall just like many others pointed to by WR.
This is even an open issue in Scala itself, namely SI-4776.

FYI I wrote a wart for this and it's now a pull request on wartremover-contrib: https://github.com/wartremover/wartremover-contrib/pull/30

@omervk thanks for the implementation! i don't think this is questionable enough to deserve being shunted off to wartremover-contrib, though. of course i might be biased after (too) spending hours on debugging an obscure OutOfMemoryError caused by mapValues once.

https://github.com/wartremover/wartremover-contrib/pull/32


OSLogin prevents Jenkins from SSH'ing into worker nodes.

When enable-oslogin=TRUE is set at the project metadata level, Jenkins is unable to SSH into any worker agents.  I have verified that the Jenkins server itself, using gcloud compute ssh, can ssh into worker agents as the service account the GCE instance is running under.  Setting enable-oslogin=FALSE allows Jenkins to SSH into any worker agents immediately.  It would be great if Jenkins could be enhanced to be able to use a service account and OSLogin instead of relying on ssh keys.

Thanks for letting us know. I gave some suggestions* in the #gcp-jenkins channel on the GCP community slack for ways to modify your IAM configuration to attempt to enable this. The way that we are connecting to GCE VMs at the moment means we can not use gcloud compute ssh.
*Here's a transcription:
"First, see https://cloud.google.com/compute/docs/instances/managing-instance-access#service_account
So the service account that you use with Jenkins should have the roles listed there
If simply adding the roles doesn't work, you can go a step further if your Jenkins master is also running as a GCE instance, and associate that instance with the service account
If the second isn't feasible for your current setup, that is understandable
It is also possible to grant the needed IAM roles to the Compute Engine default Service account: https://cloud.google.com/compute/docs/access/create-enable-service-accounts-for-instances#using_the_compute_engine_default_service_account"


Add option to excuse domain from --require-https

There's a site I link off to for which the html pages load fine over https, but for which the page assets don't. In this circumstance, I do want the page to be linked via http even though the request could go through via https.
I'd therefore love a command line flag something like --http-ok-for-domain example.com to exempt these links from --require-https
Thanks! 🙏

that's a server configuration issue i believe you should address it directly, not ignore it.

@lebensterben - Linking to someone else's domain. I'd love to fix their website for them, but unfortunately I'm not that good of a hacker yet! 😂
Also, --require-https appears to be recommending I use https for e.g. this link even though it gives an invalid cert error...

both http://www.palitext.com/JPTS_scans/JPTS_2007_XXIX.pdf and https://www.palitext.com/JPTS_scans/JPTS_2007_XXIX.pdf are 404 so i don't understand why would it tell you to use https.

we currently group results by their status, success, failure, timeout, ignored, etc.
I now think we should instead group them by levels such as

Info: Success, ignored
Warn: Not using HTTPS, not wrapping URL in angle bracket in Markdown file (this is not implemented)
Error: Failure, timeout

Oh, I'm sorry. The real link was the home page: http://www.palitext.com/ which is a redirect.

In my browser at least

something wrong with your browser

Oh? The https version works fine in your browser? No CERT_NAME error?

I object this proposal because it will make CLI options very complicated.
Instead I proposed https://github.com/lycheeverse/lychee/issues/960#issuecomment-1442770795 so that this becomes a warning instead of an error for CI.

https://www.palitext.com/ produces a cert error for me as well.
Don't want to add a CLI option for it, because it's quite a special case.
You could run lychee twice:
lychee --require-https --exclude palitext.com ...
lychee --exclude .* --include palitext.com ...

It's a little tedious, but it has the effect you want I guess.

produces a cert error for me as well

Ah good! As well it should. @lebensterben, you may want to check your security settings!

You could run lychee twice:

Ah, good idea. --include is ...erm... exclusive? Meaning it will only check what's included (if an include is... erm... included)?

@buddhist-uni
I don't think my security settings is wrong... I'm testing on ios

Should I open a new Issue for the Cert Error thing?

What is the output you're getting?
I tried this and it looks reasonable:
echo 'https://www.palitext.com/JPTS_scans/JPTS_2007_XXIX.pdf' | lychee --require-https -                                                                                ✘
Issues found in 1 input. Find details below.

[stdin]:
✗ [ERR] https://www.palitext.com/JPTS_scans/JPTS_2007_XXIX.pdf | Failed: Network error: The certificate was not trusted.

🔍 1 Total ✅ 0 OK 🚫 1 Error (HTTP:1)

Even though the https certificate is invalid (wrong domain name), Lychee thinks it's fine:
echo 'https://www.palitext.com' | lychee --require-https -
⠚ 1/1 ETA 0s ████████████████████ ✔ [200]
🔍 1 Total ✅ 1 OK 🚫 0 Errors

Shouldn't that be an error?

Strange, I'm getting the expected error:
echo 'https://www.palitext.com' | lychee --require-https -                                                                                ✘
Issues found in 1 input. Find details below.

[stdin]:
✗ [ERR] https://www.palitext.com/ | Failed: Network error: The certificate was not trusted.

🔍 1 Total ✅ 0 OK 🚫 1 Error (HTTP:1)

...even without --require-https:
echo 'https://www.palitext.com' | lychee -                                                                                ✘
Issues found in 1 input. Find details below.

[stdin]:
✗ [ERR] https://www.palitext.com/ | Failed: Network error: The certificate was not trusted.

🔍 1 Total ✅ 0 OK 🚫 1 Error (HTTP:1)

Openned #966 for the SSL issue  😊

So... should I close this issue and open a new issue for that?

Yup, let's move the discussion over to #966. Thanks!


Release Google.Api.Gax version 3.5.0

Changes since 3.4.0:

Set the gRPC max-receive-size to 2GB by default
Support for self-signed JWTs in gRPC client builders

Created release for Google.Api.Gax-3.5.0


Make SECRET_KEY_BASE defined in runtime

I've done this for my other elixir site deployed to kubernetes, so I know how now.

Fixed


[bug] Next.js search pipe fails to start due to TextEncoderStream reference error

describe the bug
When starting the search pipe, the Next.js development server fails to start due to a ReferenceError with TextEncoderStream. The error occurs during the initialization of Next.js edge runtime primitives.
system info

os: macOS 15.2
screenpipe version: 0.21.9

additional context
Error occurs when starting the search pipe with turbopack enabled. Full error log:
2024-12-29T04:17:24.346486Z ERROR screenpipe_core::pipes::pipes: [search] ReferenceError: Can't find variable: TextEncoderStream
at load (/Users/f/.screenpipe/pipes/search/node_modules/next/dist/compiled/@edge-runtime/primitives/load.js:18591:5)

The error seems to be related to Next.js edge runtime primitives initialization. This might be an incompatibility between Bun (v1.1.21) and Next.js's edge runtime.
Environment details:

Bun version: 1.1.21
Architecture: macOS arm64
Port: 46880

oops. I upgraded to bun 1.1.42 and it worked.


Cannot create new tables of complex data model

I am following Razor Pages with EF Core in ASP.NET Core and in step 6 of creating complex data model. I am trying to apply migration using 'Apply the migration to the existing database.' but  the steps dont clearly explain how to apply migration. I am not able to create those tables. Can someone please help me with this or point me to right place if I have missed anything.
Thanks,
Kruthika

Can you add the link and text to the page you're having problems with?


exit code null when --kill-others specified

This one's a little odd, but occasionally (not always), concurrently will receive an exit code of null from one of the processes when I specify --kill-others (like here).
This may be related to #37 and #80 which also indicate an exit code of null. Anyone know how that could happen? And better yet, how to fix it? Here's the code where the exit code comes from.
For now, the workaround is to not use --kill-others, but that's kinda annoying... :-/ I'm happy to submit a fix if someone can give me an idea of how this is even possible.

Was there a resolution @kentcdodds?
I get the same issue when running a node.js app (even when the node.js app is an empty file) and any other command, such as an echo:
concurrently -k 'node app.js' 'echo "hello world"'

If I replace the node app.js with anything else such as echo then the exit code is 0.
However with node I receive an exit code of null:

[1] echo "..." exited with code 0
--> Sending SIGTERM to other processes..
[0] npm run start exited with code null

Well, I opened a PR to fix it and it was merged/released. Maybe what you're seeing is unrelated to what I experienced. You may need to dig deeper.


Extremely slow

Requires some optimization 🙂
Based on my testing, this seems to be ~150x slower than the Python version

Turns out that I was using the wrong dataset on the JS test.
Instead of being ~150x slower, this is actually ~1100x slower, so there's definitely some room for improvement.


show line number for each result text

Hi, this plugin is so wonderful and I use it everyday. And I found an bug (if we can call it a bug) for the line number, the pull request is used to fix it.
During the search, if there are more than one result found in the same file, we can only see the the line/column numbers of the first matched result showing behand the filename.
Here is an examle:

I think it is useless, because we still can't know the line number of each matched result.
The following formats is more useful. we can see which file the results belong to, and we can also know the line number of each result.

This pull request is used to show the new formats, please help to check it.

After such a long time, nothing happens. So, close this.

Well, the highlight is moved to left.
https://github.com/nvim-pack/nvim-spectre/blob/817944747413ead7438ec28334201a340898b7e6/lua/spectre/ui.lua#L31-L33C54

If you busy, I can add myself. Sorry, for late reply. I was busy.

Thanks for reopen this pull request and very sorry about my rude words before. I have rebased the changes with the master branch and added a new changeset to fix the highlight issue.
It seems ok on my server, here is the test result:

I didn't notice the highlight before and forget to check this. Sorry about that and sorry about my rude words in previous comment again.

There is another 3 call in init.lua which calls render_line. Can you add them also item_line?
I got error when typed replace word because it calls another render_line.

Sorry about that and sorry about my rude words in previous comment again.
It helped me finally look for this project again. Thank you, lol.

There is another 3 call in init.lua which calls render_line. Can you add them also item_line? I got error when typed replace word because it calls another render_line.
new changeset is pushed, please help to check it.

Well, it fails because text doesn't match up because of number row. Can you add option to disable it and enable it by default. I will disable on tests later. So if it done, pretty much ready for MR.

new option lnum_for_results is added:
lnum_for_results   = true, -- show line number for search/replace results

LGTM


incorrect real_name parse

According to Yandex, real_name is <first_name> <last_name> and you have reverse (https://yandex.ru/dev/passport/doc/dg/reference/response-docpage/)
https://github.com/gurugray/passport-yandex/blob/master/lib/passport-yandex/strategy.js#L98

Yeah, same issue


Each prefix feature is on its own line

Blocked by #462
Looks like this:

There's probably a better way to present ALL of this, but this is better than how it used to be! 😄
Fixes: #373

Codecov Report

Merging #464 into revamp-word-class will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           revamp-word-class     #464   +/-   ##
==================================================
  Coverage              86.45%   86.45%
==================================================
  Files                     67       67
  Lines                   2650     2650
==================================================
  Hits                    2291     2291
  Misses                   359      359

Continue to review full report at Codecov.

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

Three questions/comments:

Is the ê- preverb actually missing from the list of preverbs, or is that just a result of the screenshot?

FOR WHATEVER REASON, the ê- preverb is missing in the test database, but it's present in the production database. ¯_(ツ)_/¯

At times, I've been considering whether one should show the prefixal features before/above the lemma/stem - but in any event this makes things much better in comparison to the current state.

Yeah! We should iterate on the presentation of this for sure, but just putting things on their own line is a huge improvement!

Though it may not be easy, we might want to consider shorter English relabelings for the preverbs, when presenting the linguistic analysis.

I think this is a great idea!


Wireshark

How can I get the chanel? And how can I get the package in Wireshark? It found only broadcast packages. Do you have any tutorial or something?

Ok I did it.

can you explain how you did it?


Make it safe to run the test suite under different Pythons in parallel

Some of the tests involve making and deleting a temporary folder.
This folder should be safely named while currently it only has one name which can lead to errors if running tests in parallel.

Mostly closed by #23 - though sometimes the temporary folders seem to be removed early still


[INFINITY-1171]Centralize cli downloading in our scripts.

@susanxhuynh & @nickbp I'd appreciate your comments.

Extract cli download to separate module.
Expose as script in case shell needs it (I don't need that now) -- also enables easy testing
Update cli atomically in case of goofy redownload while running
Automatically get the right version for the cluster
launch_ccm_cluster fetches dcos cli itself if required.

LGTM


Chrome reports GSP violation

Issue:
Clicking Generate Answer, opens a new window and an error is logged to the console:
Refused to execute inline event handler because it violates the following Content Security Policy directive: "script-src 'self' 'nonce-Fl6mtwZocgt1Y7dCXgcTbDUbE'". Either the 'unsafe-inline' keyword, a hash ('sha256-...'), or a nonce ('nonce-...') is required to enable inline execution. Note that hashes do not apply to event handlers, style attributes and javascript: navigations unless the 'unsafe-hashes' keyword is present.
Tested in latest Chrome Version 120.0.6099.5 (also incognito) and Edge.
Expected behaviour:
Click Generate Answer creates a new ChatGPT response based on saved prompt.
May attempt a fix and pull request when I have some time.

I can confirm, same issue happens on my end.

Disabling app.csp_enabled fixed this issue on my end

Yes, there is such a problem. It arose due to some updates in FreeScout, but the module has not yet been updated to work correctly with new versions. I will try to release a new version of the module with a fix for this problem within a few days, unfortunately, due to my studies I don’t have time to work on the module.

https://github.com/freescout-helpdesk/freescout/issues/3443


refactor: Improve performance, seo, accessibility and security

Performance improvement

style.scss & _fonts.scss: Self host font
teams.html: Set width and height for img
header.html: Set width and height for img
home.html: Set width and height for img

SEO improvement

about.md: Add description label
contact.md: Add description label
services.md: Add description label
team.md: Add description label
home.html: Fix #45

Accessibility improvement

footer.html: Fix HTML hierarchy problem (replace h3 by h2)
team.html: Increase contrast ratio (add $primary-dark variable in style.scss and set a color to $primary-dark in _type.scss)

Security improvement

teams.html: Fix LinkedIn link (add rel="noreferrer" attribute)

For all the pages (except subcategories of services and team, coming soon):

100/100 Google Lighthouse speed score
100/100 Google Lighthouse seo score
100/100 Google Lighthouse accessibility score
100/100 Google Lighthouse best practices score

@


Output location needs to be configurable

Right now it's based on the relative path to the code which makes it a touch tricky to migrate the code. Lets see if we can make it configurable.

@lukechurch Is this ready to be closed?


Upgraded to build with Mesos 0.21.1

Upgraded to build with latest versions of Mesos 0.21.1 and Jenkins LTS (at least the Long Term Support release) 1.580.2.
Required some updating to Vagrantfile as well, which additionally resulted in fix for issue #76, plus now has the option to use the local plugin source instead of explicit release version if desired within the VM. This is helpful for some testing in VM, but as per previous code still defaults to released version of plugin (now latest version as well - 0.5.0)

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

LGTM. Just one question about hpi-plugin version. What was the reason for the version bump?

+1

@vinodkone there can be java8, databoundsetters or other things. My suggestion will be remove hpi update and bump core to 1.565.3

I agree with @KostyaSha ^^

@vinodkone Yes as per @KostyaSha comments, I had originally bumped the jenkins version up to cater for being able to build in a Java 8 setup without errors (which I had) and I tried to align the hpi version with that as well as the same time. However there was a bit of backwards and forwards regarding whether this would limit existing setups or not. It was thus a convenience for those building in a Java 8 setup, however all still works with Java 7. As per suggestions happy to bump jenkins version and downgrade hpi.

Rebased, squash commits and adjusted as per comment, i.e.
Mesos - 0.21.1
Jenkins - 1.565.3
HPI - 1.95

Thanks for the cleanup. Merging it now.


Release 1.0!

[x] tidy up all the broken tests #327
[x] submit to pypi
[ ] submit to anaconda
[x] do all the release things
[x] update the changelog #323
[x] write up a blog post? #271

Conda support dropped for now. We need to remove a few 3.5 specific things and re-release on conda at a later point.


Access rethinkcli through dtr-rethinkdb

dtr-rethinkdb now houses the rethinkcli which means we no longer need
to pull the rethinkcli container from hub.

Signed-off-by: Kyle Squizzato<EMAIL_ADDRESS>Proposed changes
We no longer need to rely on the rethinkcli pulled from dockerhubenterprise as the tool now sits inside the dtr-rethinkdb container by default.  Now, users can just exec into this container to access the rethinkcli directly.
This patch modifies the instructions for users on how to access rethinkcli on v2.4

@joaofnfernandes I'm able to load rethinkcli from the dtr-rethinkdb:2.4.1 image:
> [ec2-user@net-stack-deploy-2 ~]$ docker ps --filter name=dtr-rethinkdb
CONTAINER ID        IMAGE                      COMMAND                 CREATED             STATUS              PORTS               NAMES
03e37d0729e1        docker/dtr-rethink:2.4.1   "/bin/rethinkwrapper"   2 days ago          Up 2 days                               dtr-rethinkdb-49b9dc85e2d8
[ec2-user@net-stack-deploy-2 ~]$ docker exec -it 03e37d0729e1 rethinkcli
> r.dbList()
[ 'dtr2',
  'jobrunner',
  'notaryserver',
  'notarysigner',
  'rethinkdb' ]

See: https://github.com/docker/dhe-deploy/issues/6135


Fix/backend/emailing bugs

Description
Updated integration testing for 80% of api-endpoints, changed structures. Need to update docs before merge.
Fixes # (issue)
#94
Type of change
Added more validation in api-endpoint functions.
Added more integration tests
Fixed database models.
Please delete options that are not relevant.

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

How Has This Been Tested?
Integration test ran for 80% functions. Did an invalid request, valid request and a 404 test.

[x] Invalid Request Test
[x] Valid request Test
[ ] 404 Not found test

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@Rethakgetse-Manaka , now that we have a working local mongodb to test with, could you consider making a new branch once you have merged this branch to make integration tests for api handlers

Okay cool. @waveyboym The reason why I requested your review is because of that linting fail I am experiencing. I wanted your perspective

@Rethakgetse-Manaka , It's saying that for the two test functions you wrote, TestCheckIn and TestCancelBooking, the programming logic is the same so either one of them have to be deleted.
I would suggested instead taking the test cases from either one of the functions and merging them into one test function because from what I can see, the only place where these two test functions differ is the test cases and the linter doesn't like that.


Improvements for "attachment" concept

[ ] Automatically save multiple versions of an attachment if it is "re-uploaded"
[x] Save the user who last modified / uploaded the file
[x] Save the date on which the file was uploaded

See https://github.com/inventree/InvenTree/pull/802


Lsp local dev

LSP-mode and docker support for localdev on C++ projects.

No idea what to do to fix the test failures in test-exordium-flip-string-quotes-point-flip-inner, something changed in upstream emacs. Broken in emacs 28.

Needs docs before merging.

LSP-mode and tramp have changed. Need to set up https://emacs-lsp.github.io/lsp-mode/page/remote/ configuration and https://github.com/emacs-lsp/lsp-docker/ for docker localdev.

@philippe-grenet @pkryger
I'd appreciate any review or suggestions. Missing docs, but the code is working.

I’ll give it a try next week. You can also ask in the IB chat


Add support for Python v3.13 and drop support for v3.9

coverage: 98.77% (+0.8%) from 97.93%
when pulling 0192c37b71c482aa90120f530b38d5da874de8e9 on python-3.13
into e6b9b5045820a170614751d6c4abd11afd35e919 on master.


[ISSUE-173][FOLLOWUP] The size of single buffer flush should reach rss.server.flush.cold.storage.threshold.size

What changes were proposed in this pull request?
follow https://github.com/apache/incubator-uniffle/issues/173, when single buffer flush is triggered, the buffer size should reach rss.server.flush.cold.storage.threshold.size, we keep the same logic as MultiStorageManager#selectStorageManager
Why are the changes needed?
Make sure cold storage is valid
Does this PR introduce any user-facing change?
No
How was this patch tested?
Already added

Codecov Report

Merging #180 (aec72e1) into master (6cbd9b7) will decrease coverage by 1.36%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##             master     #180      +/-   ##
============================================
- Coverage     58.38%   57.01%   -1.37%
+ Complexity     1266     1190      -76
============================================
  Files           158      149       -9
  Lines          8417     7928     -489
  Branches        780      751      -29
============================================
- Hits           4914     4520     -394
+ Misses         3251     3167      -84
+ Partials        252      241      -11

Impacted Files
Coverage Δ

...che/uniffle/server/storage/HdfsStorageManager.java
90.00% <83.33%> (-2.69%)
:arrow_down:

...he/uniffle/server/buffer/ShuffleBufferManager.java
82.77% <100.00%> (ø)

.../apache/uniffle/coordinator/CoordinatorServer.java
60.21% <0.00%> (-5.38%)
:arrow_down:

...storage/handler/impl/DataSkippableReadHandler.java
81.25% <0.00%> (-3.13%)
:arrow_down:

...e/spark/shuffle/reader/RssShuffleDataIterator.java

.../org/apache/spark/shuffle/writer/WriterBuffer.java

...che/spark/shuffle/writer/BufferManagerOptions.java

...org/apache/spark/shuffle/writer/AddBlockEvent.java

...pache/spark/shuffle/writer/WriteBufferManager.java

...ava/org/apache/spark/shuffle/RssShuffleHandle.java

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Replace proxy-list with outbound-socket-bindings

I don't know what's up with the commits in this pull request.  Just cherry pick the last one.

cherripicked, done, updated pull https://github.com/jbossas/jboss-as/pull/1429
thanks


ahv-vm-reporting-v_2.2.sh - error with comma in description field

if the VM desciption contain a comma (,) the output csv format line is disaligned

good point, never tested. Need to figure that out, easiest would be to let the script either remove the comma or change it to e.g. dot (.) but that is kind a ugly..


Request: Add environment variable reference/list

Hello there! I really like the image Readme's like https://github.com/actions/virtual-environments/blob/main/images/macos/macos-10.15-Readme.md. But when I need to add/write scripts for the CI runner, I am always confused as on what environment variables are available and how they are called.
Is it $ANDROID_HOME, $ANDROID_SDK_HOME, $ANDROID_SDK_ROOT or what is it? I pretty much have to find out doing trial and error.

Hi @carstenhag!
Thanks for your proposal! We'll investigate how many custom env variables (like $ANDROID_HOME, $ANDROID_SDK_HOME, etc) we have and think about the best way of adding them to the docs. Probably they can be added to the related tool section as we currently do for Java

Hi @carstenhag!
Thanks for your proposal! We'll investigate how many custom env variables (like $ANDROID_HOME, $ANDROID_SDK_HOME, etc) we have and think about the best way of adding them to the docs. Probably they can be added to the related tool section as we currently do for Java

Sounds good!

Sounds good!

Found the following environment variables that are mentioned in documentation:
All platforms:
ANDROID_HOME
ANDROID_SDK_ROOT
ANDROID_NDK_HOME
CHROMEWEBDRIVER
EDGEWEBDRIVER
GECKOWEBDRIVER
VCPKG_INSTALLATION_ROOT
CONDA
ANT_HOME
GRADLE_HOME
M2_HOME

Ubuntu:
GOROOT
GOROOT_1_13_X64
GOROOT_1_14_X64
GOROOT_1_15_X64

ANDROID_* and GOROOT definitely should be in documentation. Other variables are optional since those tools are already in PATH and variables are not needed to use. Moved this work item to backlog

Found the following environment variables that are mentioned in documentation:
All platforms:
ANDROID_HOME
ANDROID_SDK_ROOT
ANDROID_NDK_HOME
CHROMEWEBDRIVER
EDGEWEBDRIVER
GECKOWEBDRIVER
VCPKG_INSTALLATION_ROOT
CONDA
ANT_HOME
GRADLE_HOME
M2_HOME

Ubuntu:
GOROOT
GOROOT_1_13_X64
GOROOT_1_14_X64
GOROOT_1_15_X64

ANDROID_* and GOROOT definitely should be in documentation. Other variables are optional since those tools are already in PATH and variables are not needed to use. Moved this work item to backlog

Thanks for integrating this :) We can close it now imo, or is there still something to do?

@carstenhag yes, I'm going to close the issue. Thanks for your proposal!


Code Release Date

Thank you much for this interesting work! Can I ask when you will release the source code and the pretrain models?

Hi, I have reproduced RegionViT. You can follow up my code  for your research.

Hi, the codes have been released. closed this issue. please open another issue if you encounter any other problems. Thanks.


install warn

npm WARN deprecated<EMAIL_ADDRESS>gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
npm WARN deprecated<EMAIL_ADDRESS>Please use the native JSON object instead of JSON 3
/usr/local/bin/gka -> /usr/local/lib/node_modules/gka/bin/gka.js

Can a higher version of node be supported :)


Explicit Peering Agreement implementation

Context
Teku implements the concept of a “trusted”/”static” peer, which guarantees the ability to always remain connected to a peer ignoring defensive measures such as peer scoring, however it provides no guarantees of being part of their mesh or to send/receive full messages from them unconditionally.
Why explicit peering agreement
Gossipsub v1.1 introduces the concept of "Explicit Peering Agreement", which is an agreement that must be reached from a pair of nodes in order to “remain connected to and unconditionally forward messages to each other outside of the vagaries of the peer scoring system and other defensive measures”.
This feature if enabled in Teku could allow nodes to explicitly connect to some relays networks such as Fiber or Bloxroute to boost the propagation of their blocks.
Possible implementation
Most of the work for supporting explicit peers (called "direct peers" in Teku and in jvm-libp2p) has been done in https://github.com/Consensys/teku/pull/3576. However, direct peers seem to be never assigned (I'd be happy to know I'm wrong). As such, there are only few things that needs to be done:

set them from the command line
allow them to be added and removed dynamically
exclude them from peer scoring or other defensive measures

Lastly this list of nodes needs to be passed to the libp2p implementation.
If there is interest, we can open a PR that implements this functionality.

Thanks for raising. We are evaluating it.

Hello @tbenr, any questions we can answer to help with evaluation?

We have implemented explicit peering agreements in Lighthouse as trusted peers:

https://github.com/sigp/lighthouse/pull/5333

If you agree with this approach, we'll open a PR that implements this.

Hey, sorry for the delay. The only concern I have is this:
If direct peers are always part of node meshes, it is not necessarily true that we want to have static peers being direct peers too. Image you have a node with several static peers in several regions of the globe, you may want to have direct peers to be only co-located ones. So if my reasoning is true we may want to specify only a subset of static peers to be direct peers.
WDYT?

I think that makes sense. In the context of Lighthouse they didn't want to introduce another CLI flag, which is why we went with this approach. If you agree, we can start drafting a PR that adds an --explicit-peers flag and related functionality in the networking stack.

Hey @tbenr, if you want you can check out a draft of what an implementation of this could look like in #8098. Let me know what you think! Thanks a lot.