🚧 Dev ➾ add full quickstart user scenario to E2E smoke test

🔗 Epic|Feature Link
No response
🚥 Definition of Done

[ ] 🎯 Planning and Design
[ ] 🏎 Dev Implementation Complete
[ ] ♾ PR Merged
[ ] 🏆 Product Acceptance

🧰 Work Description
The quickstart scenario in the docs has a good representative use case touching on several essential resources and processes. It is an excellent example to use in a smoke test to judge the acceptability of a pre-release or build quickly.
✅ To Do
Add the flextesa and Taquito steps of the scenario to what is already in the smoke test
⚖ Acceptance Criteria
✨ Functional:

[ ] working tests that replicate the quickstart scenario from start to finish
[ ] the test runs quickly enough to be in smoke

🔭 Scope
🙅♂ Out of Scope:

...

specific tests  of any plugins
error messaging
Code of Conduct

[X] I agree to follow this project's Code of Conduct

merged


feat: add vsock support to cyclone-server

Allows cyclone-server to connect via vsock.
The changes for deadpool and cyclone-client (if any) are not ready to be merged. The changes introduced by this should be non-functional/dead until the other half of the change is merged into main.

bors r+

bors merge


MySQL loadColumnSchema: determine boolean columns

The method loadColumnSchema determines a column as a boolean only if db type is 'BIT(1)'.
https://github.com/yiisoft/yii2/blob/master/framework/db/mysql/Schema.php#L160
At the same time Schema::TYPE_BOOLEAN maps on 'tinyint(1)' for MySQL.
https://github.com/yiisoft/yii2/blob/master/framework/db/mysql/QueryBuilder.php#L41
The method loadColumnSchema for MS SQL is implemented in the right way. It considers both TINYINT(1) and BIT(1) types.
https://github.com/yiisoft/yii2/blob/master/framework/db/mssql/Schema.php#L203
I propose to change the line 160 in mysql/Schema.php to make it corresponding to the same method for MS SQL. After that the data types mapping will be proper.
if ($column->size === 1 && ($type === 'tinyint' || $type === 'bit')) {
This issue isn't theoretical. I created a migration with boolean column. Then I used gii to generate CRUD. But it generates a textField for boolean colums (not checkbox!). And it's not a gii fault, it's all about implementation of loadColumnSchema method for MySQL.

related to #9324 #9339 #10202

I found this commit: https://github.com/yiisoft/yii2/commit/8ecceb395f35bfa06b357749622ad9c96b2468a7
But if it was a reasonable decision, why the behaviour of MySQL and MS SQL drivers are different?
I also have looked #9324
Either of 2 solutions (my or #9324) will solve the problem.
But now there is inconsistency in boolean type mapping.

Imo decision was wrong because in MySQL boolean is synonym for tinyint(1). See https://github.com/yiisoft/yii2/issues/10202#issuecomment-158435363

Any progress on this issue? What should be done? Make the migration add a BIT() or adapt gii to understand tinyint again.


bug: which-key hydra mode not working?

Did you check docs and existing issues?

[X] I have read all the which-key.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of which-key.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-1315+g668d2569b4
Operating system/version
linux
Describe the bug
the docs tell me to do this:
require("which-key").show({
    keys = "<c-w>",
    loop = true, -- this will keep the popup open until you hit <esc>
})

but when i actually put in my config, i get this error:
Error detected while processing /home/ingenarel/.config/nvim/init.lua:
E5113: Error while calling lua chunk: .../share/nvim/plugged/which-key.nvim/lua/which-key/buf.lua:69: attempt to index field 'modes' (a nil valu
e)
stack traceback:
        .../share/nvim/plugged/which-key.nvim/lua/which-key/buf.lua:69: in function 'attach'
        .../share/nvim/plugged/which-key.nvim/lua/which-key/buf.lua:138: in function 'update'
        .../share/nvim/plugged/which-key.nvim/lua/which-key/buf.lua:52: in function 'new'
        .../share/nvim/plugged/which-key.nvim/lua/which-key/buf.lua:188: in function 'get'
        ...hare/nvim/plugged/which-key.nvim/lua/which-key/state.lua:288: in function 'start'
        ...share/nvim/plugged/which-key.nvim/lua/which-key/init.lua:18: in function 'show'
        .../ingenarel/.config/nvim/lua/_plugins_/_load-plugins_.lua:91: in main chunk
        [C]: in function 'require'
        /home/ingenarel/.config/nvim/init.lua:103: in main chunk

Steps To Reproduce
copy paste this in the config:
require("which-key").show({
    keys = "<c-w>",
    loop = true, -- this will keep the popup open until you hit <esc>
})

Expected Behavior
the error shouldn't happen?
Health
No response
Log
No response
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "folke/which-key.nvim", opts = {} },
    -- add any other plugins here
  },
})
require("which-key").show({
    keys = "<c-w>",
    loop = true, -- this will keep the popup open until you hit <esc>
})

I had the same issue. I figured out that the correct usage of this is for it to be the lhs of a mapping:
      vim.keymap.set("n", "<c-w><space>", function()
         require("which-key").show({
            keys = "<c-w>",
            loop = true,
         })
      end)

i'm just currently using this function in my config:
---@nodoc CK_Submap_n() {{{1
---@nodoc docs {{{2
---### creates submaps
---uses |nvim_set_keymap| for setting the keymaps
---@param keymap string keymap that you want to set it to.
---@param action string stuff you want to do for the keymap
---@param submaplevel string if your keymap is is `abcd` and you want to just hit `d` after doing `abcd` to execute the keymap again, your submaplevel would be `abc`
---@param opts? {wk: boolean?, desc:string?, wait:integer?, mode:string?} --
---`wk:` nil and true are treated the same -> it executes whichkey before executing the keymap again. if false, it doesn't
---`desc:` the description from the keymap
---`wait:` time to wait before executing the keymap again. uses |vim.wait()|
---`mode`: the mode. normal mode, visual mode, insert mode etc.
---@nodoc docs }}}2
function CK_Submap(keymap, action, submaplevel, opts)
    if opts == nil then
        opts = {}
    end

    local iswhichkey = opts.wk
    local description = opts.desc
    local waiting_time = opts.wait
    local mode = opts.mode

    ---@type string
    local whichkey = ""

    ---@type string
    local wait = ""

    if iswhichkey == true or iswhichkey == nil then
        whichkey = ":WhichKey<lt>CR>"
    end

    if waiting_time ~= nil and waiting_time >= 1 then
        wait = "vim.wait(" .. waiting_time .. ") "
    end

    if mode == nil then
        mode = "n"
    end

    vim.api.nvim_set_keymap(
        mode,
        keymap,
        action
            .. ":lua "
            .. wait
            .. "vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes('"
            .. whichkey
            .. submaplevel
            .. "', true, false, true), 'nt', false)<CR>",
        {
            noremap = true,
            silent = true,
            desc = description,
        }
    )
end -- }}}1

I had the same issue. I figured out that the correct usage of this is for it to be the lhs of a mapping:
      vim.keymap.set("n", "<c-w><space>", function()
         require("which-key").show({
            keys = "<c-w>",
            loop = true,
         })
      end)

How do you use it when you are using which-key.add for your mappings already though? I've been trying to make a nested window so that I can make stepping lines in a debugger much less tedious but I have not been able to figure out how to use .show or it's loop parameter correctly in tandom with the standard .add hotkeys and the default trigger. This plugin has been wonderful but the documentation has been lacking. Been trying to look at the source code for answers but no luck yet.

have the same issue


How can I fine-tune a model for a downstream task?

I now need to validate the performance on the MSRVTT dataset. How can this be implemented? Could you provide a corresponding tutorial?

README.md


Adding cypress test for expiration field

Related to #1501

@tisto I have rebased my pr :)

This is old and it's not completely clear to me what it's trying to test; I'm going to close it.


Support for HDFS Quota Metrics

This patch provides the ability for Azkaban to monitor and report (via the internal Metrics framework) HDFS quota stats for a configured directory and also a polling interval. By default, the polling interval is 1 min.

Pausing this for now.


Default require path to features unless specified explicitly.

In general there seem to be some weird dependencies (or lack there of) between
configuration options and information in Cucumber::Cli::Configuration#feature_files
that could probably be clarified with a little refactoring.
One example that this feature exposed directly is that
Cucumber::Cli::Configuration#feature_files does not
depend on Cucumber::Cli::Configuration#feature_dirs though their names
certainly imply dependence.
I had deleted Cucumber::Cli::Configuration#feature_dirs to eliminate
the confusion, but added it back in when I realized it was used in
features/docs/post_configuration_hook.feature.
I took a stab at re-architecting the configuration code awhile back, but got mired and gave up. Just realized I never submitted this pull request.
Looks like there's plans to give the configuration some attention for 2.1, but maybe this is still a useful step.

#611

Thanks for the contribution!


work item number in commit messages for TFS Client Library version 12.0 but not 14.0

This is not necessarily an issue that has to be fixed but should at least be well-known.
Our deployed code is still from VS 2013 but our new development is on VS 2015, and consequently I have one development machine for each of the two versions. For a TFS changeset that is associated with a work item, git-tfs fetch will decorate the commit message with the line "work-items: #ID" when the TFS Client Library is at version 12 (for the system running VS 2013) but won't when the TFS Client Library is at version 14. Consequently the commit hashes will be different and the two repositories will diverge.
git-tfs is at version <IP_ADDRESS> on both machines.

VS2013 suport is no longer available in git tfs, so issue can't be solved.
Please close


Stricter needed parameters check and save meta data to samples

Drop all unneeded parameters when initializing the context
Save meta data to MCMC samples(h5 file)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

26 of 33 (78.79%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 77.436%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

appletree/context.py
20
27
74.07%

Files with Coverage Reduction
New Missed Lines
%

appletree/context.py
1
81.13%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
1462

Relevant Lines:
1888

💛  - Coveralls

Thanks! Can we also save the instruction of context in meta as well? Then in principle it should be able to recover the whole fitting.


The console displays too many logs

Because pipStatusStream is referenced in PiPSwitcher, pipStatusStream starts a very frequent timer, resulting in frequent access to activity.packageManager.hasSystemFeature(PackageManager.FEATURE_PICTURE_IN_PICTURE), which will keep outputting logs on the console.
The solution is to save the init state when accessing isPipAvailable, and return the last queried state the next time you access it.

Can it be fixed soon?

The goal of the frequent access to the API is to provide up to date PiP status, shall it be changed e.g. by a system event, so I don't plan to remove/change it at the moment as it's necessary for the correct behaviour of the plugin.
I tested the package on many devices, both real and emulated, and haven't experienced excessive logs -- might be that your test device is set up with a highly verbose logs output causing the wall of logs. You can also filter out the logs easily, as it has predictable structure to prevent it from distracting you.

I see the issue, feel free to post a PR with the change that fixes your issue, add this will greatly speed things up.

PR had commited

Fixed in 5.0.1, thanks!


Wrong Status Code Description

Status code description is duplicated on all codes as shown in screenshot below:

Router is defined as follows:
var DeployProject = router.New(
	&DeployProjectInput{},
	router.Summary("Deploy Project"),
	router.Description("Deploy project for given configuration"),
	router.Responses(router.Response{
		"204": router.ResponseItem{
			Model:       model.Created{},
			Description: "Project created",
		},
		"400": router.ResponseItem{
			Model:       model.BadRequest{},
			Description: "Bad request",
		},
		"404": router.ResponseItem{
			Model:       model.NotFound{},
			Description: "Not found",
		},
		"500": router.ResponseItem{
			Model:       model.InternalServerError{},
			Description: "Error response",
		},
	}),
)

This can be fixed in swagger/swagger.go with following code:
func (swagger *Swagger) getResponses(response router.Response) openapi3.Responses {
	ret := openapi3.NewResponses()
	for k, v := range response {
		schema := swagger.getResponseSchemaByModel(v.Model)
		content := openapi3.NewContentWithJSONSchema(schema)
		description := v.Description
		ret[k] = &openapi3.ResponseRef{
			Value: &openapi3.Response{
				Description: &description,
				Content:     content,
				Headers:     v.Headers,
			},
		}
	}
	return ret
}

Why this?

Here is an explanation of this issue: https://stackoverflow.com/questions/29923956/golang-reusing-memory-address-copying-from-slice

Get it, thanks!

Fixed with PR #7


Inlined citations should either be part of the sentence or parenthesized

In your introduction, you have 2 references that are inlined but a bit awkward to read where. I feel like this isn't exactly your issue, but Drasil's. However, for now, I'll create the ticket here since it looks like you want inlined citations to be treated as full words in a sentence (i.e., treated as nouns) since you have other cases of that.

In IEEE (the style I tend to use for engineering documents), citations can be used as "full words" or as "inlined". I'm not sure how they should be used in Drasil; I agree that the current method is unclear (and I personally don't like it), but think this is an issue larger than ChemCode.

To be clear, I wasn't saying you shouldn't use citations as “full words” or anything. I just meant that when using them as full words, they should blend/flow in the sentence. When they're meant to be “attached” without blending into the sentence, they're usually just in parentheses or brackets.

In that case, I think this issue should be moved to the Drasil repo

It certainly can be. There are at least two paths forward:

blending the citation into the sentence (in which case, it's part of the sentence and no issue),
parenthesize the citation manually and create a ticket about citation style preferences (specifically, which is the default).

My thought is that the citations are currently being displayed incorrectly/citation style preference can't be selected, so this should be an issue for Drasil (and I may or may not get to doing it in my version). Unless there is a specific citation style that Drasil uses that interacts with in-line citations differently that I'm not aware of, in which case the issue becomes improving documentation for that (and me fixing the citations in my version).

Thinking of it like a feature request definitely sounds fair to me, especially given that our code for inline referencing is a smart constructor! :smile:

Interesting to note that there is precedent for wrapping these citations in parentheses, for example this unital in GlassBR.

Migrated


Support for electron App

Hello,
It looks like it won't be possible to use Amazon Connect Streams in an electron app as ideally, the electron apps don't open the website URL, they instead open a local index.html file in the browser. So the document.URL will be a file path instead of some HTTPS URL. And since Amazon connect streams require the URL to be explicitly whitelisted to allow it to embed the CCP, there won't be any way to use it in electron APP unless I open the URL instead of the local HTML file in the electron App.
I would still like to know if there is some way around it to make it work in an Electron App.
Thanks

Hey @imankurpatel000 please see #166, closing this as it's a duplicate.


Investigate adding DslMarker annotations to public APIs for better suggestions

The DslMarker annotation was mentioned by JetBrains as a way we can influence type suggestions in Intellij.  It was mentioned that this annotation is not stable so we cannot count on it.  We could provide feedback back to JB to prioritize stabilization but first we should know:

how much does stability matter w/ something like this?
does application of the annotation meaningfully improve the customer IDE experience

@DslMarker appears to be stable at this time.


Parallel link is dead

Link to parallel @ 8a570f8 in src/ is dead, which prevents speedseq from building.
Trying to use git clone git://git.savannah.gnu.org/parallel.git as suggested in #124 hangs.
Trying to access git://git.savannah.gnu.org/parallel.git or even the domain https://savannah.gnu.org/ times out completely.
I was lucky enough to have a clone already in another directory I could use for this, but most people won't have that.

Hi, I tried and had the same issue. but I found a new link that seemed to have worked
git clone https://git.savannah.gnu.org/git/parallel.git

Link to parallel @ 8a570f8 in src/ is dead, which prevents speedseq from building.
Trying to use git clone git://git.savannah.gnu.org/parallel.git as suggested in #124 hangs.
Trying to access git://git.savannah.gnu.org/parallel.git or even the domain https://savannah.gnu.org/ times out completely.
I was lucky enough to have a clone already in another directory I could use for this, but most people won't have that.

Can you please share the clone?


pam: Allow backing the MOTD with a file

Description of changes
Right now, the motd, e.g., on SSH login can only be set to a fixed string via users.motd.
This change adds the option to configure users.motdFile instead, which is read by PAM on every login, and can be changed without a NixOS rebuild. This does not replace the existing users.motd option, and does not allow to set both options at the same time.
There had been a PR (https://github.com/NixOS/nixpkgs/pull/37602) suggesting similar functionality a few years back, that tried to implement a more user-friendly interface for configuring dynamic motd updates, but it never got finished.
This PR does not try to offer such a high-level interface. Any dynamic updates to the MOTD still need to be built through other measures, e.g., systemd timers.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Could we have a release note mention please too? As this is a quite cool change I think.

Please squash the nixos/pam: fix formatting of users.motdFile commit, since it's a fixup to something added in this PR.

Oh, I just noticed nixos/pam: swap motd branches to simplify condition should be squashed too.

Oh, I just noticed nixos/pam: swap motd branches to simplify condition should be squashed too.

Done :slightly_smiling_face:


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20221017153540816 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20221017153540816 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 42.349%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/connection/browser/connectionDialogWidget.ts
1
73.05%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
29424

Relevant Lines:
64627

💛  - Coveralls


Search gives unexpected result for data token symbol

Try the following:

Take any known data set's symbol, e.g. RUMNAR-48, and type it into the search of market.oceanprotocol.com.
The result will surprise you; mainly because in this specific case RUMNAR-48 is not presented as a confident first result

See screenshot:

It'd in the search results - but what'd be less confusing or more pleasing to the users is showing it as result number 1 and potentially also removing the other false-positive results.

we have a greatly enhanced search query in #755 but can confirm that your exact example behaves the same there. Fun thing is that for any other picked symbol I really get only 1 result over there but not for RUMNAR-48
This specific issue might be also solved in combination with #699 where then I would expect based on relevancy to at least get RUMNAR-48 as first result

This happens because elastic search removes the hyphen when it analyses the text. I think getting RUMNAR-48 as the first result should be a satisfactory result, I don't think we necessarily need to remove all the other results.
If this doesn't get solved in #699, I'm happy to work on another search PR to address this

@TimDaub - can you check again ?

The screenshot shows that RUMNAR-48 comes first. Anyways IMO when a symbol is searched, I think RUMNAR-48 should be specificallly highlighted to signal the user that indeed the token symbol that hey're looking for exists.
FYI: There are now 8 false positives instead of 3. So maybe this fix was a regression?

searching for any datatoken symbol always presents this as the first result so not sure what exactly the issue here is anymore. So closing


Allow updating permissions of resource items

Permissions of resource items currently cannot be updated by calling update.
Paperless-ngx requires us to send a set_permissions field to write permissions, but securable models only have a permissions field. We need to add a conversion of field names to the UpdatableMixin, when permissions were modificated by a user.

Remove stale. Ya I will do that soon. 😌


.scala-build/ide-options-v2.json not generated by compile

Version(s)
0.1.18
Describe the bug
.scala-build/ide-options-v2.json file is not generated by compile command without additional flags, but it's referred to inside .bsp/scala-cli.json, causing metals not to work properly
To Reproduce
scala-cli compile file.scala

Expected behaviour
.scala-build/ide-options-v2.json should be generated, even if its content would be just an empty json object

Interestingly this seems to work for me on a new project. Are you able to reproduce it on a clean project?

I am able to reproduce it when I deleted the .scala-build directory, but the .bsp still exists. In this case, the .scala-build/ide-options-v2.json file is not generated.


Support single line comments.

// this should be removed
//! this should be kept

Need someone to write a custom parser, once PostCSS supports it, that supports Sass-style single line comments.

:+1:

This is supported if you use postcss-scss to process the input.


implemented receive data of request

Implements getting information about the query for logging the data

@yapro Thanks for your PR :)


Ability to check if the Red Hat Plugin is installed before extension activation.

Need to add extensionDependencies in your extension-manifest


Gofmt

Description
Fix errors reported by gofmt and add checks to prevent future
regressions.

make check-gofmt will check for files that aren't gofmt
compliant. This will be run by TravisCI.
make gofmt will fix any gofmt errors in Aminal's source
All existing gofmt have been fixed.

Vendored files are ignored.
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Tested that Aminal still builds and runs.
Test Configuration:

OS: Ubuntu
OS version: 18.04
Go version: 1.11.5

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own 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
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

That, probably, should be rebased and merged with priority since it will introduce conflicts to others PRs or others merged PRs will block that PR again and again

@liamg @MaxRis rebased

@mjs oh, it seems that Travis won't pass for your PR. That might be because you're doing it from your fork, but should do from the remote branch of the same repo. I have the same issue in another PR. will update mine and let you know if it helps.

@mjs you need just to push exactly the same branch from your fork into liamg/aminal repo directly. After you will do push the travis build will magically start for that your PR. No additional actions.


Swift Packager Manager Support

Adds Package.swift which enable usage with SPM.
Note, that the most correct usage with SPM will be possible only when there is new tag with full semantic version(e.g. "1.2.0", not "1.2") that includes Package.swift. Until then, dependency can be added by branch or commit hash which is not allowed in published packages.

@TwoDollarsEsq Thanks, great addition! 🎉


Members/rulan wei branch

What is the purpose of the change

Add personal info of RulanWei.
Post Hugo-on-Mac.

Brief change log

Modify /content/authors/Rulan-Wei
Modify /content/post/hugo-install-Mac

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark:<PHONE_NUMBER>:x: RulanWei:x: Wence-MayYou have signed the CLA already but the status is still pending? Let us recheck it.


✨ add iptables and conntrack to ubuntu images

What this PR does / why we need it: This PR adds iptables and conntrack tools to ubuntu images. This is needed as a requirement of kubeadm to deploy k8s clusters using provider.

I'd rather prefer packages to be added to the provider repositories - however those sounds core packages that should be in the base image. Thanks!


Access Denied on Android

1. Response on android (WORKS ON IOS):
{
url: 'https://www.zomato.com',
title: 'Access Denied',
description: undefined,
mediaType: 'website',
images: [],
videos: []
}
2. Even most of the time the 'images' array is empty for valid urls.
 The 'title' field returns correct names but 'images' array is empty.

Have you added permissions to your android manifest.xml file?

May I know which permissions?

https://stackoverflow.com/questions/2169294/how-to-add-manifest-permission-to-android-application

Internet permission was already added in Android manifest.

Even sometimes 'https://www.google.com' receives empty 'images' array in response but the 'title' field has correct response i.e 'Google'.
This happens many times both on android & ios

Sorry I could not reproduce this issue, could it be it has to do with some external factor either in your project or internet connection? firewall or something?
Feel free to submit a pull request if you manage to find the root of the problem and fix it, but it seems I cannot help you

Please try this on both Android and iOS:
Type 'https://www.myntra.com'

Sometimes it shows the preview
Most of the times 'No Preview Available'
Sometimes preview is shown but the preview goes away automatically and shows 'No Preview Available'

ok

I had to double check the code of the library to make sure I'm not returning a 'no preview available', so the library will never return this, SO: this sounds like the UI representation you have built on top of the library, so possible source of errors might be there, double check your code

I have built a simple script to test this,

COUNTER=0
         while [  $COUNTER -lt 100 ]; do
             echo The counter is $COUNTER
             yarn test
             if [ $? -eq 1 ]
             then
              echo 'FAILED TO FETCH'
              exit
             fi
             let COUNTER=COUNTER+1
         done```

and the test:

```it('should handle image urls', async () => {
    try {
      const info = await LinkPreview.getPreview('https://www.myntra.com');
      // const info = await LinkPreview.getPreview('Not a link');

    } catch (e) {

      expect(e.error).to.be('FORCE FAIL')
    }
  })```

the final output:

The counter is 99
yarn run v1.3.2
$ jest --forceExit
PASS  tests/index.spec.js
link preview
✓ should handle image urls (298ms)
Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        1.162s
Ran all test suites.```
it has ran the 100 times and not failed once, so again, this seems to be you doing something wrong on your end, check for firewalls, proxies, your async logic in your app, but I repeat, this cannot be a problem with the library, it is plain and simple javascript code, no android/iOS configuration attached to it.

just FYI, updated the test to check for JSON content and increased tests to 1000 and still not a single failure.
also, I'm in spain/germany so no government/ISP rate limiting/blocking, don't know where you are, but could also be a possibility.


Client can't download from filestore after hostname change.

I've started goiardi without a hostname parameter, uploaded a cookbook. The client could download it. I've restarted the server with a hostname parameter, and the client tried the filestore urls with the old (default) hostname. After I've uploaded the cookbook again it worked.

Strange. I remember this coming up at some point before, but I thought I fixed it. I'll take a look.

Aha, I found it. (Been busy at work, I'm afraid.) Running tests on it now, but while the hostname for file URLs was being fixed almost everywhere, it was broken exactly one place. Unfortunately when I tested it before it managed to fetch it from the old place. A fix for this, assuming I didn't break something subtly with this, should be up in the morning.

There's a fix in this for master. I'll be cutting a new release pretty soon too. Thanks for finding that this was still a problem.

This has been released for about a month. Unless I hear otherwise I'm going to assume this fixed it.


Add Zipkin integration document

@trustin All fixed. PTAL 🙇

@adriancole Anything to add or update?

@adriancole Added! PTAL 🙇

excellent!
On Fri, Jul 5, 2019 at 2:14 PM minux<EMAIL_ADDRESS>wrote:

@adriancole https://github.com/adriancole Added! PTAL 🙇
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/line/armeria/pull/1877?email_source=notifications&email_token=AAAPVV453RTIICL6ZNKWUQDP53REZA5CNFSM4H5BPL3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZIUL6A#issuecomment-508642808,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAPVVZ5OEU5WR7SUVXOILDP53REZANCNFSM4H5BPL3A
.

@trustin, all fixed PTAL 🙇

Thanks for reviewing. 😉


Docs: Getting Started Page Not Found

docs maybe need to update, the url of Getting Started is not found:

@marcusbooyah Getting Started link through github README file


Security: Updated Rack::Attack alerting for specific routes only

Alert separately only if this matches a valid route, which is the primary concern.  Requests for /garbage are lower priority.

selfie


Sync from website

I am unsure how to best resolve the merge-conflict in topics/concepts/modern-lb-and-proxying/index.md
between

https://github.com/datawire/getambassador.io/pull/802
https://github.com/datawire/ambassador-docs/pull/374 (more helpful diff: https://github.com/datawire/ambassador-docs/compare/f655f24af52b9f650c397c0d68a7fcd47e82bd55..1bfa64125a155eedf5e1046ef82ed326ffe7bd2a)

Nevermind, figured it out.


question： why there is no reducer to handle INDEX_NAV, did i miss anything?

greped reducers directory and got nothing there. :(

Thanks for reporting this, if you look at the action we are using the history api to manage state.
https://github.com/grommet/grommet-ferret/blob/2628df0c57d314e127928647a8e43006e59f0036/src/js/actions.js#L193
That's why we don't have a reducer. Let us know if you have a different approach for that.


Voucher and Memebership card vs Get It

When a voucher/membership card is on the home screen the GET IT is removed
If the voucher/membership card is deleted the GET IT is displayed

same thing as for Over18

when tyeh card is deleted from home it does not appear on discover

duplicate of Voucher and membership card is no more available in the Android build 75 #845


Builtin: CairoKeccak builtin

Implement cairo keccak builtin hints and fix integration test with keccak.
Rust implementation: https://github.com/lambdaclass/cairo-vm/blob/main/vm/src/hint_processor/builtin_hint_processor/cairo_keccak/keccak_hints.rs

assign me


[WIP] Allow operator functions to have extra arguments with default values

This changes how operator functions are detected. Before they were required to only have an explicit number of parameters that matches the operator arity. After this change, they can have extra arguments, as long as all of those arguments have default values.
The upshot of this is that you're able to create operator functions that capture things like the #file and #line of where they're used. This is incredibly useful while debugging.

@davedelong waves.

I would appreciate some guidance on the SILGen and Executable tests. I've been trying to fudge together some SILGen tests by poking and prodding at existing ones, but I'm not having much success.
Also, what sorts of things should I be checking for in these tests?

In the executable test I would check that calling a simple operator with a default argument works. Perhaps the operator can just return a struct with its input values and defaulted argument, and you can check that the result is correct. For the SILGen test, ensure a call to the default argument is emitted. Look at the existing default argument tests for ideas.

@davedelong Are you still interested in pursuing this change?

This is a change I would like to see made, but I am not skilled enough to make all of the necessary changes, nor do I have time to invest in figuring out what I’d need to know.

Alright.  I've filed SR-11885 to track this if you or somebody else would like to return to it.  In the mean time, I'm going to close out this pull request.
Thanks for your interest here.


odo project list -o json does not display new project info on 4.0/4.1 cluster

[kind/bug]

What versions of software are you using?

Operating System: macOS
Output of odo version: master

oc version:
Client Version: version.Info{Major:"4", Minor:"0+", GitVersion:"v4.0.0-alpha.0+b93a8e6-2141-dirty", GitCommit:"b93a8e6", GitTreeState:"", BuildDate:"2019-04-23T21:16:33Z", GoVersion:"", Compiler:"", Platform:""}
Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.4+b93a8e6", GitCommit:"b93a8e6", GitTreeState:"clean", BuildDate:"2019-04-23T21:16:31Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

How did you run odo exactly?
$ git clone https://github.com/openshift/odo
$ make configure-installer-tests-cluster
$ odo project create xftcyybzou -w && odo project list -o json
Check details here - https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_odo/1612/pull-ci-openshift-odo-master-e2e-core/938
Actual behavior
{
          "kind": "List",
          "apiVersion": "odo.openshift.io/v1alpha1",
          "metadata": {},
          "items": [
            {
              "kind": "Project",
              "apiVersion": "odo.openshift.io/v1alpha1",
              "metadata": {
                "name": "myproject",
                "creationTimestamp": null
              },
              "spec": {
                "apps": null
              },
              "status": {
                "active": false
              }
            }
          ]
        }

Expected behavior
{
          "kind": "List",
          "apiVersion": "odo.openshift.io/v1alpha1",
          "metadata": {},
          "items": [
            {
              "kind": "Project",
              "apiVersion": "odo.openshift.io/v1alpha1",
              "metadata": {
                "name": "myproject",
                "creationTimestamp": null
              },
              "spec": {
                "apps": null
              },
              "status": {
                "active": false
              }
            },
            {
              "kind": "Project",
              "apiVersion": "odo.openshift.io/v1alpha1",
              "metadata": {
                "name": "xftcyybzou",
                "creationTimestamp": null
              },
              "spec": {
                "apps": null
              },
              "status": {
                "active": true
              }
            }
          ]
        }

Any logs, error output, etc?
Creating a new project: xftcyybzou
Running odo with args: [project create xftcyybzou -w]
[odo]  •  Waiting for project to come up  ...
[odo]  ✓  Waiting for project to come up
[odo]  ✓  Project 'xftcyybzou' is ready for use
[odo]  ✓  New project created and now using project : xftcyybzou
Running odo with args: [project list -o json]
[odo] {"kind":"List","apiVersion":"odo.openshift.io/v1alpha1","metadata":{},"items":[{"kind":"Project","apiVersion":"odo.openshift.io/v1alpha1","metadata":{"name":"myproject","creationTimestamp":null},"spec":{"apps":null},"status":{"active":false}}]}
Deleting project: xftcyybzou
Running odo with args: [project delete xftcyybzou -f]
[odo]  •  Deleting project xftcyybzou  ...
[odo]  ✓  Deleting project xftcyybzou
[odo] Deleted project : xftcyybzou

• Failure [7.106 seconds]
odojsonoutput
/go/src/github.com/openshift/odo/tests/integration/json_test.go:14
  odo machine readable output on empty project
  /go/src/github.com/openshift/odo/tests/integration/json_test.go:30
    should be able to return project list [It]
    /go/src/github.com/openshift/odo/tests/integration/json_test.go:32

    Expected
        <string>: {
          "kind": "List",
          "apiVersion": "odo.openshift.io/v1alpha1",
          "metadata": {},
          "items": [
            {
              "kind": "Project",
              "apiVersion": "odo.openshift.io/v1alpha1",
              "metadata": {
                "name": "myproject",
                "creationTimestamp": null
              },
              "spec": {
                "apps": null
              },
              "status": {
                "active": false
              }
            },
            {
              "kind": "Project",
              "apiVersion": "odo.openshift.io/v1alpha1",
              "metadata": {
                "name": "xftcyybzou",
                "creationTimestamp": null
              },
              "spec": {
                "apps": null
              },
              "status": {
                "active": true
              }
            }
          ]
        }
    to match JSON of
        <string>: {
          "kind": "List",
          "apiVersion": "odo.openshift.io/v1alpha1",
          "metadata": {},
          "items": [
            {
              "kind": "Project",
              "apiVersion": "odo.openshift.io/v1alpha1",
              "metadata": {
                "name": "myproject",
                "creationTimestamp": null
              },
              "spec": {
                "apps": null
              },
              "status": {
                "active": false
              }
            }
          ]
        }

    first mismatched key: "items"

    /go/src/github.com/openshift/odo/tests/integration/json_test.go:35

/kind bug

Same failure is observed in #1612

/assign @surajnarwade

However it works fine on 4.1 cluster locally
odojsonoutput odo machine readable output on empty project
  should be able to return project list
  /Users/amit/go/src/github.com/openshift/odo/tests/integration/json_test.go:32
Creating a new project: ichvwhnvgo
Running odo with args: [project create ichvwhnvgo -w]
[odo]  •  Waiting for project to come up  ...
[odo]  ✓  Waiting for project to come up
[odo]  ✓  Project 'ichvwhnvgo' is ready for use
[odo]  ✓  New project created and now using project : ichvwhnvgo
Running odo with args: [project list -o json]
[odo] {"kind":"List","apiVersion":"odo.openshift.io/v1alpha1","metadata":{},"items":[{"kind":"Project","apiVersion":"odo.openshift.io/v1alpha1","metadata":{"name":"ichvwhnvgo","creationTimestamp":null},"spec":{"apps":null},"status":{"active":true}},{"kind":"Project","apiVersion":"odo.openshift.io/v1alpha1","metadata":{"name":"myproject","creationTimestamp":null},"spec":{"apps":null},"status":{"active":false}}]}
Deleting project: ichvwhnvgo
Running odo with args: [project delete ichvwhnvgo -f]
[odo]  •  Deleting project ichvwhnvgo  ...
[odo]  ✓  Deleting project ichvwhnvgo
[odo] Deleted project : ichvwhnvgo
•
------------------------------
odojsonoutput odo machine readable output on empty project
  should be able to return empty list
  /Users/amit/go/src/github.com/openshift/odo/tests/integration/json_test.go:39
Creating a new project: leeitkmjzr
Running odo with args: [project create leeitkmjzr -w]
[odo]  •  Waiting for project to come up  ...
[odo]  ✓  Waiting for project to come up
[odo]  ✓  Project 'leeitkmjzr' is ready for use
[odo]  ✓  New project created and now using project : leeitkmjzr
Running odo with args: [app list -o json --project leeitkmjzr]
[odo] {"kind":"List","apiVersion":"odo.openshift.io/v1alpha1","metadata":{},"items":[]}
Deleting project: leeitkmjzr
Running odo with args: [project delete leeitkmjzr -f]
[odo]  •  Deleting project leeitkmjzr  ...
[odo]  ✓  Deleting project leeitkmjzr
[odo] Deleted project : leeitkmjzr
•
------------------------------

This looks like an issue in odo project create. It is probably not waiting for a new project properly.

Please uncomment the test once this is fixed
Test file - json_test.go
Test Spec -
*It("should be able to return project list", func() {
			actualProjectListJSON := helper.CmdShouldPass("odo", "project", "list", "-o", "json")
			desiredProjectListJSON := fmt.Sprintf(`{"kind":"List","apiVersion":"odo.openshift.io/v1alpha1","metadata":{},"items":[{"kind":"Project","apiVersion":"odo.openshift.io/v1alpha1","metadata":{"name":"myproject","creationTimestamp":null},"spec":{"apps":null},"status":{"active":false}},{"kind":"Project","apiVersion":"odo.openshift.io/v1alpha1","metadata":{"name":"%s","creationTimestamp":null},"spec":{"apps":null},"status":{"active":true}}]}`, project)
			Expect(desiredProjectListJSON).Should(MatchUnorderedJSON(actualProjectListJSON, WithUnorderedListKeys("items")))
		})

/priority hgh

@amitkrout , works for me though
$ odo project list -o json |  jq
{
  "kind": "List",
  "apiVersion": "odo.openshift.io/v1alpha1",
  "metadata": {},
  "items": [
    {
      "kind": "Project",
      "apiVersion": "odo.openshift.io/v1alpha1",
      "metadata": {
        "name": "proj1",
        "creationTimestamp": null
      },
      "spec": {
        "apps": null
      },
      "status": {
        "active": false
      }
    },
    {
      "kind": "Project",
      "apiVersion": "odo.openshift.io/v1alpha1",
      "metadata": {
        "name": "proj2",
        "creationTimestamp": null
      },
      "spec": {
        "apps": null
      },
      "status": {
        "active": true
      }
    }
  ]
}

versions are:
$ ./oc version
Client Version: version.Info{Major:"4", Minor:"1+", GitVersion:"v4.1.0-201904211700+86afce9-dirty", GitCommit:"86afce9", GitTreeState:"", BuildDate:"2019-04-21T23:20:35Z", GoVersion:"", Compiler:"", Platform:""}
Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.4+81fc896", GitCommit:"81fc896", GitTreeState:"clean", BuildDate:"2019-04-21T23:18:54Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

I tested this on CRC

@amitkrout , works for me though
$ odo project list -o json |  jq
{
  "kind": "List",
  "apiVersion": "odo.openshift.io/v1alpha1",
  "metadata": {},
  "items": [
    {
      "kind": "Project",
      "apiVersion": "odo.openshift.io/v1alpha1",
      "metadata": {
        "name": "proj1",
        "creationTimestamp": null
      },
      "spec": {
        "apps": null
      },
      "status": {
        "active": false
      }
    },
    {
      "kind": "Project",
      "apiVersion": "odo.openshift.io/v1alpha1",
      "metadata": {
        "name": "proj2",
        "creationTimestamp": null
      },
      "spec": {
        "apps": null
      },
      "status": {
        "active": true
      }
    }
  ]
}

versions are:
$ ./oc version
Client Version: version.Info{Major:"4", Minor:"1+", GitVersion:"v4.1.0-201904211700+86afce9-dirty", GitCommit:"86afce9", GitTreeState:"", BuildDate:"2019-04-21T23:20:35Z", GoVersion:"", Compiler:"", Platform:""}
Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.4+81fc896", GitCommit:"81fc896", GitTreeState:"clean", BuildDate:"2019-04-21T23:18:54Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

I tested this on CRC

Can you check with OpenShift 4.0. I think there should be some flexibility of using OpenShift version through CRC

@surajnarwade This is a consistent CI failure on prow (OpenShift 4.0)

@surajnarwade Did you get any chance to test it on 4.0 cluster ?
BTW our prow config is using 4.1 cluster. So i will uncomment those tests to see the behavior on prow+4.1 cluster.

@surajnarwade Also in CI (4.1 OpenShift) it fails. Seems it only happens in CI irrespective of cluster version due to odo project list -o json fails to fetch the latest data.
I do not think it's a timing issue because project creation waits till the project is up and ready to use.
Failure log - https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_odo/1749/pull-ci-openshift-odo-master-e2e-core/1117

We are planning to club all the machine readable output related issues in later sprint and hence reducing the priority

@amitkrout , can you please confirm this issue again because, I still can't reproduce the error

@surajnarwade Sure, Will send a pr just to check if that reproduce in CI setup. Locally it works as i mentioned above.

Context("odo machine readable output on empty project", func() {
		JustBeforeEach(func() {
			project = helper.CreateRandProject()
			context = helper.CreateNewContext()
			os.Setenv("GLOBALODOCONFIG", filepath.Join(context, "config.yaml"))
		})
		JustAfterEach(func() {
			helper.DeleteProject(project)
			helper.DeleteDir(context)
			os.Unsetenv("GLOBALODOCONFIG")
		})
		It("should be able to return project list", func() {
			// time.Sleep(5 * time.Second)
			actualProjectListJSON := helper.CmdShouldPass("odo", "project", "list", "-o", "json")
			partOfProjectListJSON := fmt.Sprintf(`{"kind":"Project","apiVersion":"odo.openshift.io/v1alpha1","metadata":{"name":"%s","creationTimestamp":null},`, project)
			Expect(actualProjectListJSON).To(ContainSubstring(partOfProjectListJSON))
		})
	})

@surajnarwade You can reproduce the issue on any cluster by using the above test code. If you uncomment // time.Sleep(5 * time.Second) and run can be run successfully. So it seems the api call is done before the actual list updated.

/unassign @amitkrout
/assign @surajnarwade

/assign @amitkrout
/unassign @surajnarwade

@surajnarwade Sure, Will send a pr just to check if that reproduce in CI setup. Locally it works as i mentioned above.

@amitkrout is there a PR for this? It works for me locally. If it's working fine, can we close this?
$ odo version
odo v1.2.0 (b26a1f522)

Server: https://api.crc.testing:6443
Kubernetes: v1.16.2

$ crc version
crc version: 1.9.0+a68b5e0
OpenShift version: 4.3.10 (embedded in binary)

$ odo project list -o json | jq
{
  "kind": "List",
  "apiVersion": "odo.openshift.io/v1alpha1",
  "metadata": {},
  "items": [
    {
      "kind": "Project",
      "apiVersion": "odo.openshift.io/v1alpha1",
      "metadata": {
        "name": "ci-operator-hub-project",
        "namespace": "ci-operator-hub-project",
        "creationTimestamp": null
      },
      "spec": {},
      "status": {
        "active": false
      }
    },
    {
      "kind": "Project",
      "apiVersion": "odo.openshift.io/v1alpha1",
      "metadata": {
        "name": "newproject",
        "namespace": "newproject",
        "creationTimestamp": null
      },
      "spec": {},
      "status": {
        "active": true
      }
    }
  ]
}

This looks like an issue in odo project create. It is probably not waiting for a new project properly.

Right.
After the -w flag implementation in project create it works fine, i mean the test script can list the current project.


EncodingProviders must use abs(machine id)

Negative machine IDs are possible, and SnowflakeEncodingProvider left-shifts these without checking, resulting in a small negative number that is OR'ed with the timestamp, erasing the timestamp.  Example:
    static class FixedIdProvider implements MachineIdProvider {
        final long id;
        FixedIdProvider(long id) {
            this.id = id;
        }
        public long getMachineId() {
            return id;
        }
    }

    public static void main(String[] args) throws Exception
    {
        IdGenerator g1 = newSnowflakeIdGenerator(new FixedIdProvider(1));
        System.out.println(g1.generateId(0).asString());
        IdGenerator g2 = newSnowflakeIdGenerator(new FixedIdProvider(-1));
        System.out.println(g2.generateId(0).asString());
    }

Output:
080f6b18b1001000
fffffffffffff000

Creating a wrapper on e.g. MacPidMachineIdProvider is an easy temporary workaround.

@rholder Is there an ETA on this pull?
@jonross There's another pull from @azderski with a different implementation but similar issue. Is it a different problem?

It looks like the other implementation fixes the MAC address value, however another machine ID implementation could still return < 0.  Also I included a test.

Good. Then I'll keep using your branch while this gets merged.


fix: heap profile

Which issue does this PR close?
Closes #471
Rationale for this change

What changes are included in this PR?
Use https://github.com/mkmik/heappy/ to heap profile

Are there any user-facing changes?
No

How does this change test

Manually
For now, following errors will be thrown
error: failed to select a version for `libc`.
    ... required by package `heappy v0.1.0 (https://github.com/mkmik/heappy?rev=99a7ae15a49de3fdb4cc82abc0ead01df7a0db7d#99a7ae15)`
    ... which satisfies git dependency `heappy` of package `profile v1.0.0-alpha01 (/code/misc/CeresDB/components/profile)`
    ... which satisfies path dependency `profile` (locked to 1.0.0-alpha01) of package `server v1.0.0-alpha01 (/code/misc/CeresDB/server)`
    ... which satisfies path dependency `server` (locked to 1.0.0-alpha01) of package `catalog_impls v1.0.0-alpha01 (/code/misc/CeresDB/catalog_impls)`
    ... which satisfies path dependency `catalog_impls` (locked to 1.0.0-alpha01) of package `ceresdb v1.0.0-alpha01 (/code/misc/CeresDB)`
versions that meet the requirements `^0.2.138` are: 0.2.138

all possible versions conflict with previously selected packages.

  previously selected package `libc v0.2.127`
    ... which satisfies dependency `libc = "^0.2.94"` (locked to 0.2.127) of package `backtrace v0.3.66`
    ... which satisfies dependency `backtrace = "^0.3.9"` (locked to 0.3.66) of package `common_util v1.0.0-alpha01 (/code/misc/CeresDB/common_util)`
    ... which satisfies path dependency `common_util` (locked to 1.0.0-alpha01) of package `analytic_engine v1.0.0-alpha01 (/code/misc/CeresDB/analytic_engine)`
    ... which satisfies path dependency `analytic_engine` (locked to 1.0.0-alpha01) of package `benchmarks v1.0.0-alpha01 (/code/misc/CeresDB/benchmarks)`

failed to select a version for `libc` which could resolve this conflict

Closed this because #484 has addressed #471


Identifiers validation failed for dataset Etnomicología en la Sierra de Santa Marta, Veracruz

Identifier validation failed for the dataset Etnomicología en la Sierra de Santa Marta, Veracruz:

Crawler attempt: 323
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 78
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 78 total records; 78 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3740095/step

New IDs sample:

Old IDs sample:

14b847d1a78fae0483ad296c64248c28
168e2b7b23497a83edc92c095514ee3d
1c3952825572e28b3ec45dd182ab47e3
2441b7bb7c9d0beb559a77ea7ed9bd5e
35582a586bb44127963e97350a69a3ea
4b2fea02f169d9d949ed1f5f68d16d4c
53794b7d5bc0beaf3c413d16e1996af1
5ab706a69134fd87864f9deabb003a01
6b707fd896520393a72636ff847d0404
76bc37e91a0d11cdde163c1559a739f2

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Etnomicología en la Sierra de Santa Marta, Veracruz : https://doi.org/10.15468/j0reuf.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Etnomicología en la Sierra de Santa Marta, Veracruz:

Crawler attempt: 324
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 78
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 78 total records; 78 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3741057/step

New IDs sample:

Old IDs sample:

14b847d1a78fae0483ad296c64248c28
168e2b7b23497a83edc92c095514ee3d
1c3952825572e28b3ec45dd182ab47e3
2441b7bb7c9d0beb559a77ea7ed9bd5e
35582a586bb44127963e97350a69a3ea
4b2fea02f169d9d949ed1f5f68d16d4c
53794b7d5bc0beaf3c413d16e1996af1
5ab706a69134fd87864f9deabb003a01
6b707fd896520393a72636ff847d0404
76bc37e91a0d11cdde163c1559a739f2

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Etnomicología en la Sierra de Santa Marta, Veracruz : https://doi.org/10.15468/j0reuf.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Etnomicología en la Sierra de Santa Marta, Veracruz:

Crawler attempt: 326
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 78
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 78 total records; 78 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3741670/step

New IDs sample:

Old IDs sample:

14b847d1a78fae0483ad296c64248c28
168e2b7b23497a83edc92c095514ee3d
1c3952825572e28b3ec45dd182ab47e3
2441b7bb7c9d0beb559a77ea7ed9bd5e
35582a586bb44127963e97350a69a3ea
4b2fea02f169d9d949ed1f5f68d16d4c
53794b7d5bc0beaf3c413d16e1996af1
5ab706a69134fd87864f9deabb003a01
6b707fd896520393a72636ff847d0404
76bc37e91a0d11cdde163c1559a739f2

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Etnomicología en la Sierra de Santa Marta, Veracruz : https://doi.org/10.15468/j0reuf.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


We do not require square brackets for httplib.

Resolves #707.
I need to come up with a test for this that doesn't suck, but it should be considered illuminating that no tests break with this change.

@t-8ch You added this in 208ba74c796116bafc95d46dfcf4325449643ab6, any idea why? Is it safe to remove now?

The original issue was https://github.com/shazow/urllib3/pull/199

BTW, I legitimately have no good idea how to test this. I guess I could manually replace the socket to force httplib to do its validation but not actually use the socket, but...that doesn't feel good.

According to https://github.com/shazow/urllib3/pull/199#issuecomment-19837893, it sounds like it happens on a specific Py3 version when port is not given? :/

What's our supported set of Py3 versions? The oldest I have installed right now is Python 3.3.6, which does not reproduce the bug in #199.

Pip supports 3.2 still
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Ugh, ok, I'll try to reproduce on 3.2.

Python 3.2.6 also does not reproduce the bug from #199. Looks like this must have been fixed in a version of 3.2 prior to 3.2.6.
@dstufft What's the scope of your supported 3.2 versions? Put another way, how much effort should I be going to to track down a release that actually exhibits this bug?

What versions of X does pip support is always a sort of hard question. In this case I think that the only reason someone would be using Python 3.2 is either if they are using PyPy3 (which I'm not sure if pip even works on....) or if they are getting Python 3.2 from their OS. Typically our support for anything other than the latest version in a particular X.Y series is best effort based on how big of a deal the bug seems to be, and how hard the fix is.
Looking at the bug, it appears this only affects cases where someone is passing a raw IPv6 into urllib3? So for pip that would probably map to someone is using a raw ipv6 address as an --index-url, --find-links, or are using it in a link from a private index. I suspect that very few people are going to be switching to IPv6 and still be using Ubuntu Precise. There's a good chance the next major version of pip drops support for 3.2 anyways.
tl;dr I wouldn't spend much effort into it unless someone comes out of the wood work to explicitly complain.

Thanks @dstufft. From my perspective that makes me want to merge this, but then again I proposed it so I'm naturally in favour. ;)

I don't have an opinion on the change itself, but I wouldn't let a nebulous concern about a <3.2.6 Python 3.2 prevent you.

If you're interested - https://groups.google.com/forum/#!topic/pypa-dev/Ef0PF2ZGAv0

I've no feelings about this, @Lukasa totally your call, feel free to hit whichever buttons speaks to you.

Yeah, 3.2 is broken in many ways with httplib. I'd say we just stop trying to support 3.2 in this case. :+1:

Screw it, let's do it.


Installation issue

Error when installing

Try installing using pip instead, I cannot reproduce the issue.

use this command sudo python3 setup.py install

Thanks It's Worked


flycheck-plantuml failed to check

(flycheck-define-checker plantuml
  "A checker using plantuml.

See `http://plantuml.com"
  :command ("java" "-Djava.awt.headless=true" "-jar" (eval plantuml-jar-path) "-syntax")
  :standard-input t
  :error-patterns ((error line-start "ERROR" "\n" line "\n" (message) line-end))
  :modes plantuml-mode)

The plantuml checker defined by flycheck-plantuml uses the jar path.
Sets plantuml-jar-path explicitly like that will fix the issue:
(use-package flycheck-plantuml
  :ensure t
  :after (flycheck plantuml-mode)
  :config (flycheck-plantuml-setup)
  ;; flycheck-plantuml only support the jar style
  :custom (plantuml-jar-path "/usr/share/java/plantuml/plantuml.jar"))

Thanks, I didn't know there was a person who was actually trying to use my config.
I install most external dependencies using Nix via home-manager. I also save machine-specific configuration in custom-file. You can change the value of the variable with M-x customize-variable RET plantuml-jar-path.


Limit search results.

Is there anyway I can limit number of output results? Like I need only 10 Google search results...
I used below code and it gave too many results...
`from search_engines import Google
engine = Google()
results = engine.search("my query")
links = results.links()`
Thanks in advance.

Nevermind, god it.

@chsaiujwal can you post your answer for others?


Edit gradle application plugin to set application scripts in distribution root directory

For ease of use, have gradle application plugin place the scripts in the distribution root directory.

Additional information is required to see if this is possible. Configuration allows for startup scripts to be placed in the root directory of the distribution, but it renders the scripts useless because the classpath is not configured correctly causing Java to fail when finding the lib directory.

Closing due to #31


Permission inheritance is wrong on DrvFs with -o metadata

Your Windows build number:  (Type ver at a Windows Command Prompt)
17063

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

correct behavior on local file system
me@devbox:~$ mkdir /tmp/foo
me@devbox:~$ chmod 777 /tmp/foo
me@devbox:~$ ls -al /tmp/foo
total 0
drwxrwxrwx 1 me   me   4096 Dec 20 19:42 .
drwxrwxrwt 1 root root 4096 Dec 20 19:42 ..
me@devbox:~$ sudo su - you
you@devbox:~$ touch /tmp/foo/bar
you@devbox:~$ chmod 600 /tmp/foo/bar
you@devbox:~$ ls -al /tmp/foo/bar
-rw------- 1 you you 0 Dec 20 19:42 /tmp/foo/bar

incorrect behavior on drvfs mount:
me@devbox:~$ mkdir /mnt/d/foo
me@devbox:~$ chmod 777 /mnt/d/foo
me@devbox:~$ ls -al /mnt/d/foo
total 0
drwxrwxrwx 1 me me 4096 Dec 20 19:46 .
drwxrwxrwx 1 me me 4096 Dec 20 19:46 ..
me@devbox:~$ sudo su - you
you@devbox:~$ touch /mnt/d/foo/bar
you@devbox:~$ chmod 600 /mnt/d/foo/bar
chmod: changing permissions of '/mnt/d/foo/bar': Operation not permitted
you@devbox:~$ ls -al /mnt/d/foo/bar
-rwxrwxrwx 1 me me 0 Dec 20 19:47 /mnt/d/foo/bar

What's wrong / what should be happening instead:

files created on drvfs with -o metadata are receiving wrong user & groups, causing chmod and other metadata tools to fail later on.

my mistake. drive wasn't mounted with -o metadata


AttributeError: 'int' object has no attribute 'get'

$ flex --source http://openfisca.sgmap.fr/api/1/swagger
Traceback (most recent call last):
…
  File "/usr/local/lib/python2.7/site-packages/flex/decorators.py", line 112, in inner
    kwargs[key] = obj.get(key, EMPTY)
AttributeError: 'int' object has no attribute 'get'

Thanks for the big report.  I'll look into this today.
On Thu, Apr 2, 2015, 7:23 AM Matti Schneider<EMAIL_ADDRESS>wrote:

$ flex --source http://openfisca.sgmap.fr/api/1/swagger
Traceback (most recent call last):
…
File "/usr/local/lib/python2.7/site-packages/flex/decorators.py", line 112, in inner
kwargs[key] = obj.get(key, EMPTY)
AttributeError: 'int' object has no attribute 'get'
—
Reply to this email directly or view it on GitHub
https://github.com/pipermerriam/flex/issues/68.

Also.  What version are you using.
On Thu, Apr 2, 2015, 9:04 AM Piper Merriam<EMAIL_ADDRESS>wrote:

Thanks for the big report.  I'll look into this today.
On Thu, Apr 2, 2015, 7:23 AM Matti Schneider<EMAIL_ADDRESS>wrote:

$ flex --source http://openfisca.sgmap.fr/api/1/swagger
Traceback (most recent call last):
…
File "/usr/local/lib/python2.7/site-packages/flex/decorators.py", line 112, in inner
kwargs[key] = obj.get(key, EMPTY)
AttributeError: 'int' object has no attribute 'get'
—
Reply to this email directly or view it on GitHub
https://github.com/pipermerriam/flex/issues/68.

3.5.0, just installed it  ;)
Let me know if I can give any additional information!  :)

You can get the fix for this in 3.5.1 which is up on pypi
https://pypi.python.org/pypi/flex/3.5.1

Wow, pretty impressive reaction time! Congratulations and thanks  :)

Happy to have you using it.  Please let me know if you find anything else.
On Thu, Apr 2, 2015, 11:35 PM Matti Schneider<EMAIL_ADDRESS>wrote:

Wow, pretty impressive reaction time! Congratulations and thanks :)
—
Reply to this email directly or view it on GitHub
https://github.com/pipermerriam/flex/issues/68#issuecomment-89169741.


Documentation should not mention ingress controller specific annotations [File: /content/en/docs/concepts/services-networking/ingress.md]

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
The ingress resource documentation describes examples that contain an annotation that will not work with other ingress controllers besides nginx: nginx.ingress.kubernetes.io/rewrite-target
Proposed Solution:
While this may help some users, there is also a generic rewrite-target annotation called ingress.kubernetes.io/rewrite-target. This annotation will also work with traefik and other ingress controllers.
The documentation should be kept as implementation-agnostic as possible, so the reference to nginx should be dropped.
Page to Update:
https://kubernetes.io/docs/concepts/services-networking/ingress/

Working on it

Cool, thank you.
I did a little bit of research, and it seems that behaviour differs a bit between different ingress controllers, but at least ingress-nginx and Træfik have supported the generic annotation for a while.
I don't know about others.

Apparently, what I said is not quite true, as can be seen in this comment: https://github.com/kubernetes/ingress-nginx/issues/3109#issuecomment-422802472
But there's a somewhat more pragmatic discussion for Træfik: https://github.com/containous/traefik/pull/1723
However, I think it would make a lot of sense to have a common syntax for different ingress controllers. The ingress.kubernetes.io/rewrite-target annotation seems to work well in many cases, but a controller-specific prefix may be needed. The documentation should reflect this, or a design decision may be needed (well out of scope for this bug report).
Here is the documentation for Træfik: https://docs.traefik.io/configuration/backends/kubernetes/#general-annotations
And for nginx: https://github.com/kubernetes/ingress-nginx/blob/master/docs/examples/rewrite/README.md


whitenoise config in demotemplate/wsgi.py

from whitenoise.django import DjangoWhiteNoise

application = DjangoWhiteNoise(application)

also see settings.py line 149-152:
# Simplified static file serving.
# https://warehouse.python.org/project/whitenoise/
# See also https://devcenter.heroku.com/articles/django-assets
STATICFILES_STORAGE = 'whitenoise.django.GzipManifestStaticFilesStorage'

needed for all Django/Heroku apps


Extend WebLayer API to support content filtering.

Enables consumption of easylist style rules encoded using components/url_pattern_index as a protobuf.
ContentFilteringService is an interface provided by the browser to renderer proceses, which use it to fetch the protobuf via a file descriptor that can be memory mapped.
This enables the protobuf to be stored as part of the application APK and efficiently shared to renderer processes without any copies. (Important for the easylist protobuf files to not be compressed as part of the APK however.)
ContentFilterManager is the main controller interface used by the embedding app to configure the content filter. The embedding app can also observe when filtering happens by attaching a ContentFilterCallback instance to each Tab.
There's a command line tool called make_filter_rules that takes an easylist format file and outputs the corresponding protobuf.
See also Network Filtering w/ WebLayer for more details on creating this PR.

@yozuysal - This is ready for review now. Let me know if you want to setup some time to walk through it.


https://github.com/github/explore/issues/588#issuecomment-466796720

https://github.com/github/explore/issues/588#issuecomment-466796720

Thank you

Good

https://github.com/github/explore/issues/602#issuecomment-473997427


Add ClearML Server Deployment Option in Qhub

ClearML is a way to track experiment results for ML Models.  It'd be nice to add to Qhub.
Looks like some instructions for deployment exist at https://allegro.ai/clearml/docs/rst/deploying_clearml/index.html.  Initially
Likely, I'll plan to use the Helm deployment option.
More details:
ClearML is broken into roughly 3 parts. A local python package, a ClearML server, and a ClearML Agent for ML-Ops orchestration.  The priority is the ClearML server, if the ClearML agent is not too heavy a lift that would be great too (unclear how it’s orchestration functionality integrates with QHub).
https://github.com/allegroai/clearml/#clearml-architecture
Ideally, finish this in the next 2 weeks.

More details:
ClearML is broken into roughly 3 parts. A local python package, a ClearML server, and a ClearML Agent for ML-Ops orchestration.  The priority is the ClearML server, if the ClearML agent is not too heavy a lift that would be great too (unclear how it’s orchestration functionality integrates with QHub).
https://github.com/allegroai/clearml/#clearml-architecture

Some useful resources on the ClearML side:

Installation Instructions (https://github.com/allegroai/clearml-server-k8s/tree/master/clearml-server-cloud-ready)
ClearML server architecture image (https://github.com/allegroai/clearml-server#launching-the-clearml-server)

Also, they just released v1.0 of ClearML, but the helm chart isn't ready yet.

Looks like we'd need to create some PV's for the various components of ClearML:

Persistent Volumes for pvc-apiserver.yaml, pvc-fileserver.yaml, and pvc-agentservices.yaml.
Persistent volumes for elasticsearch, mongodb and redis (redis is optional). See relevant information for each chart:
elasticsearch
mongodb
redis Make sure to define the following values for each PV:
elasticsearch - in the values.yaml set elasticsearch.persistence.enabled=true and set elasticsearch.volumeClaimTemplate.storageClassName to the storageClassName used in your elasticsearch PV.
mongodb - in order to define a persistent volume for mongodb, in the values.yaml set mongodb.persistence.enabled=true and set mongodb.persistence.storageClass to the storageClassName used in your mongodb PV. Read here for more details.
redis - in order to define a persistent volume for redis, in the values.yaml set redis.master.persistence.enabled=true and set redis.master.persistence.storageClass to the storageClassName used in your redis PV. Read here for more details.

I tried to deploy ClearML after deploying Qhub.  It didn't work.  It seemed like I needed a standard storage class which didn't exist (at least not on Azure).  I tried adding a standard storage class that was similar to default, but it still didn't seem to work.  I kind of want to try on GCP to see if it that storage class exists there.  I may just need to ping them on their slack channel with my questions, bc I'm not sure why it's not working.

@Adam-D-Lewis ping them on the slack they are very active there and can help.

As I suspected, the standard storage class exists on GCP, and the deployment progressed further.  However, the elasticsearch container is still erroring out.  I've pinged clearML team on slack.  I'll post my question below

Hi, I'm trying to deploy clearML on GKE on google cloud via helm using app version 1.0.2 and chart version 2.0.2+1.  I'm seeing the following Cannot allocate memory warnings and errors in my clearml-elastic-master-0 pod.

| elasticsearch OpenJDK 64-Bit Server VM warning: Option UseConcMarkSweepGC was deprecated in version 9.0 and will likely be removed in a future release.
│ elasticsearch {"type": "server", "timestamp": "2021-05-28T13:59:54,162Z", "level": "WARN", "component": "o.e.b.JNANatives", "cluster.name": "clearml-elastic", "node.name": "clearml-elas
│ tic-master-0", "message": "Unable to lock JVM Memory: error=12, reason=Cannot allocate memory" }
│ elasticsearch {"type": "server", "timestamp": "2021-05-28T13:59:54,165Z", "level": "WARN", "component": "o.e.b.JNANatives", "cluster.name": "clearml-elastic", "node.name": "clearml-elas
│ tic-master-0", "message": "This can result in part of the JVM being swapped out." }
│ elasticsearch {"type": "server", "timestamp": "2021-05-28T13:59:54,166Z", "level": "WARN", "component": "o.e.b.JNANatives", "cluster.name": "clearml-elastic", "node.name": "clearml-elas
│ tic-master-0", "message": "Increase RLIMIT_MEMLOCK, soft limit: 16777216, hard limit: 16777216" }
│ elasticsearch {"type": "server", "timestamp": "2021-05-28T13:59:54,166Z", "level": "WARN", "component": "o.e.b.JNANatives", "cluster.name": "clearml-elastic", "node.name": "clearml-elas
│ tic-master-0", "message": "These can be adjusted by modifying /etc/security/limits.conf, for example: \n\t# allow user 'elasticsearch' mlockall\n\telasticsearch soft memlock unlimited\n
│ \telasticsearch hard memlock unlimited" }
│ elasticsearch {"type": "server", "timestamp": "2021-05-28T13:59:54,167Z", "level": "WARN", "component": "o.e.b.JNANatives", "cluster.name": "clearml-elastic", "node.name": "clearml-elas
│ tic-master-0", "message": "If you are logged in interactively, you will have to re-login for the new limits to take effect." }

... later in logs

| elasticsearch ERROR: [1] bootstrap checks failed                                                                                │ elasticsearch [1]: memory locking requested for elasticsearch process but memory is not locked

I imagine I could make the changes specified in the warning to /etc/security/limits.conf in my own slightly modified docker image, but I was wondering if anyone has seen this before and if there's something obvious I'm missing.  I'm using all defaults in the values.yaml file currently.

Vini and I also upgraded some nodes to use n1-standard-8 becase we got some error indicating the n1-standard-2 instance was too small for ClearML on GCP.

I'm assuming the error is due to ulimit missing:
try adding 16777216 to both soft/hard ulimit
https://github.com/allegroai/clearml-server/blob/09ab2af34cbf9a38f317e15d17454a2eb4c7efd0/docker/docker-compose.yml#L58

From the ClearML developer:

I'm assuming the error is due to ulimit missing:
try adding 16777216 to both soft/hard ulimit
https://github.com/allegroai/clearml-server/blob/09ab2af34cbf9a38f317e15d17454a2eb4c7efd0/docker/docker-compose.yml#L58

Owh, thanks for the feedback on that. I will try twerking that and see if it solves the issue.

@vfdev-5 found this reference to the changes I believe we need to make (but this applies to the single node deployment of clearML only rather than the cloud-ready version as I understand it) https://allegro.ai/clearml/docs/docs/deploying_clearml/clearml_server_kubernetes_helm.html

I think I may have found a workaround for the moment.  I created my own docker image based on the original docker image, and created a custom entrypoint to run ulimit -l unlimited before running the normal entrypoint as specified in the link below.  https://github.com/kubernetes/kubernetes/issues/3595#issuecomment-288451522

I think I may have found a workaround for the moment. I created my own docker image (balast/elasticsearch:5:56 on DockerHub) based on the original docker image, and created a custom entrypoint to run ulimit -l unlimited before running the normal entrypoint as specified in the link below. kubernetes/kubernetes#3595 (comment). I also ran the elasticsearch container as root and added the capabilities IPC_LOCK and SYS_RESOURCE as mentioned in
kubernetes/kubernetes#3595 (comment).
Now the issue is that the apiserver isn't discovering the elasticsearch server. I don't see an clearml ingress pod up, maybe we have to create one?

It seems that the docker image was succefully initalized, but it restarts once elasticsearch /custom-entrypoint.sh: line 5: ulimit: max locked memory: cannot modify limit: Operation not permitted

@Adam-D-Lewis @viniciusdc reading your messages, does it mean that this installation step : https://allegro.ai/clearml/docs/docs/deploying_clearml/clearml_server_kubernetes_helm.html#step-1-modify-elasticsearch-default-values-in-the-docker-configuration-file does not work ? Or we are in a bit different situation ?

@aktech and @viniciusdc, can you comment on this?  These are the remaining steps as I see them:

Try a daemon set to modify the ulimits on the node
Set up a new node pool for clearML in qhub
the rest of integration into qhub (add feature flag, etc.)
Support for other cloud providers (will only work on GCP right now AFAIK)

Add standard storage class for other cloud providers (Push off for now)

@vfdev-5 Those instructions don't apply to the cloud-ready clearML helm deployment directly (https://github.com/allegroai/clearml-server-k8s/tree/master/clearml-server-cloud-ready).  Those apply to https://github.com/allegroai/clearml-server-k8s/tree/master/clearml-server-chart.  The instructions for the clearML cloud-ready helm deployment is https://github.com/allegroai/clearml-server-k8s/tree/master/clearml-server-cloud-ready, but it didn't work for us off the bat.  First thing to know was you can't use Azure (currently), but GCP should work.  Second thing was how to get around the ulimits manually (which we did via a custom elasticsearch image, but are going to try Daemon Sets to rectify next).

@aktech and @viniciusdc, can you comment on this? These are the remaining steps as I see them:

Looks good to me, other than that the obvious one is organizing the full yaml file into logical pieces.
By the way a trivial example of daemonset to start off with:
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: clearml-limits
  namespace: kube-system
  labels:
    k8s-app: clearml-limits
spec:
  selector:
    matchLabels:
      name: clearml-limits
  template:
    metadata:
      labels:
        name: clearml-limits
    spec:
      nodeSelector:
        app: clearml

      containers:
        - name: clearml-limits
          image: busybox
          volumeMounts:
            - mountPath: host-mount
              name: root-mount
          command: ["sh", "-c"]
          args:
            - cat host-mount/etc/docker/daemon.json;
              echo "vm.max_map_count=262144" > host-mount/tmp/99-clearml.conf;
              mv host-mount/tmp/99-clearml.conf host-mount/etc/sysctl.d/99-clearml.conf;
              cat host-mount/etc/sysctl.d/99-clearml.conf;
          securityContext:
            privileged: true
      volumes:
        - name: root-mount
          hostPath:
            path: /

@aktech and @viniciusdc, can you comment on this? These are the remaining steps as I see them:

Set up a new node pool for clearML in qhub

Try a daemon set to modify the ulimits on the ClearML nodepool nodes

the rest of integration into qhub (add feature flag, etc.)

Support for other cloud providers (will only work on GCP right now AFAIK)

Add standard storage class for other cloud providers (Push off for now)

Sure, I think you addressed all the general steps needed. Just a quick question, those are related only to get the ClearML server running with QHUb right? Are we considering connecting it to the ClearML client too?

Status with this: The clearml branch now has the kubernetes manifest in the terraform and it deploys clearml components correctly with QHub. The next step is exposing the webserver via Traefik.

Update so far: It seems there isn't a way to access clearml server on a path say qhub-url/clearml as the server doesn't redirects the static files to the correct path. I am going to try the subdomain approach as suggested in the doc:
https://allegro.ai/clearml/docs/docs/deploying_clearml/clearml_server_kubernetes.html#accessing-clearml-server

@vfdev-5 and I pair programmed for most of today in order to get the feature flag and node labels added.

I fixed some minor terraform formatting on the PR.  The checks on the PR are still failing with the error below and I don't know why. (https://github.com/Quansight/qhub/pull/640/checks?check_run_id=2840961922

Run nslookup github-actions.qhub.dev
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53
** server can't find github-actions.qhub.dev: NXDOMAIN


[BUG] FastFlags Are not saving

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
When i change value of the fastflag and save it it doesnt save it says it saved but its not
Bloxstrap Log
N/A

what fflag are you trying to put

what fflag are you trying to put

FFlagDisablePostFx
the bug happening on every fastflag

did you check if bloxstrap is running and not the normal roblox

did you check if bloxstrap is running and not the normal roblox

yes i did

hmm idk really

Upload your configuration: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

When i change value of the fastflag and save it it doesnt save it says it saved but its not

So first of all you did click save right. (am guessing yes). When you reopen the bloxstrap menu can you still see the fast flag in bloxstrap ff editor? Make sure that you did open bloxstrap using the tiny icon at the bottom on the taskbar.

Issue has stale for over a month, and will be reopened if further information is given.


Distribute the load of data on two kafka consumer using kafka.net client

How I can divide the load of data on my two kafka consumer using kafka .net client.

we need to write separate coordinator service to re-balance the consumers in the same group . unfortunately Kafka-net does not provide this support!


Improve Syntax Coloring Scope

Copy and paste the code below. The brackets are with the wrong color
<?php if ($test) { ?>

Lorem ipsum dolor, sit amet consectetur adipisicing elit. Molestiae ipsam eius quos voluptate hic recusandae excepturi cum. Dolores vitae sunt optio assumenda soluta aliquam numquam voluptatum modi minima, labore illo.

<?php } else { ?>

Lorem ipsum dolor, sit amet consectetur adipisicing elit. Molestiae ipsam eius quos voluptate hic recusandae excepturi cum. Dolores vitae sunt optio assumenda soluta aliquam numquam voluptatum modi minima, labore illo.

<?php } ?>

As you can see it provides us just a generic source.php meta.embedded.line.phpthat we can't target. This is sort of bug because the second curly bracket (the opening one) have a different scope.

Closing this issue because it seems to be fixed on the latest version of Insiders.
@andremacola feel free to reply if this issue still occurs for you after updating.

@rzhao271 nope, the scope still wrong in 1.51.0-insiders. Some themes you can't notice the problem but if you show the TM Scope you see the problem.

Filed upstream https://github.com/atom/language-php/issues/402


[BUG] IntramodalMerge fails when input dimensions are not consistent

This happens, for instance, with ds114/sub-02. The T1w images of the two sessions are merged, but it fails for a potential misorientation problem:
$ nib-ls /scratch/03843/crn_plab/fmriprep-ohbm-2/ds000114_R2.0.1/sub-02/ses-retest/anat/sub-02_ses-retest_T1w.nii.gz /scratch/03843/crn_plab/fmriprep-ohbm-2/ds000114_R2.0.1/sub-02/ses-test/anat/sub-02_ses-test_T1w.nii.gz
/scratch/03843/crn_plab/fmriprep-ohbm-2/ds000114_R2.0.1/sub-02/ses-retest/anat/sub-02_ses-retest_T1w.nii.gz float32 [256, 156, 256] 1.00x1.30x1.00   sform
/scratch/03843/crn_plab/fmriprep-ohbm-2/ds000114_R2.0.1/sub-02/ses-test/anat/sub-02_ses-test_T1w.nii.gz     float32 [156, 256, 256] 1.30x1.00x1.00   sform

Solving this issue involves:

[ ] Checking that input dimensions are consistent
[ ] Checking that input orientations are correct and consistent.

WDYT @chrisfilo ?
/ref #285 - for the multiple T1w problem.

ds114 is a longitudianl dataset with a separate T1w for each session. In such cases to avoid bias I would treat each session like a separate subject (don't combine T1ws across sessions, coregister EPIs to their respective T1ws).

Ok, that behavior is definitely not implemented.

Superseded by #310.


Add jekyll-compose

Turns out this isn't as useful as I hoped:

it hardcodes not adding a date to anything that isn't a post, but I need dates on events
it ignores default front matter for from the config file and defaults to post, requiring adding a redundant layout to plugin defaults


Guidelines/config for running a no_std envrionment

Hello,
Getting programs to work with no_std is pretty tricky on the playground. For example, the simple program:
#![no_std]
fn main()->(){}

This won't compile (as expected) because of the missing eh_personality. The best resolution for the playground isn't clear.
no_main also typically goes along with no_std, but how to specify the entrypoint also kind of varies and usually relies on target crates.
My suggestion would be maybe to add a template, or add a dropdown option that provides the appropriate configuration. Or just a little blurb on the help page about what to do to use no_std, since I'm sure it's possible but just haven't found the right options. Using #[cfg(no_std)] works but I don't believe this guarantees the entire program will be no-std, unless you remember to use it everywhere
A way to still print to the console (as println! is part of std) would be helpful as well.

I have found a workable solution, which may help anyone who comes across this. You can write the main code in no_std, but enable std for the tests. This allows for the helpful printing/debugging where needed, while enforcing that the main code not use std.
#![cfg_attr(not(test), no_std, no_main)]
#![allow(dead_code)]

fn testable() -> u16 {
    4 * 10
}

#[cfg(test)]
mod tests {
    use super::*;
    extern crate std;

    #[test]
    fn test_testable() {
        assert_eq!(testable(), 40);
    }
}

https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=febe456279ff7c917c08a6ecfc30d4c4

Additional note: I have read the policy but would still like to suggest considering adding the heapless crate. It is not mentioned in the cookbook, but it is mentioned multiple times in the embedded book, and is really a core crate for editable strings/vectors on the stack

A better way to go about this all might be to run the binary in qemu. cross could be used if there's enough desire


Add bot.getMessageUser function to primary platforms

Adds a standardized mechanism for loading a standardized user profile for Slack, Teams, Cisco Spark and Facebook.
As a promise:
bot.getMessageUser(message).then(function(profile) { ... });
As a callback:
bot.getMessageUser(message, function(err, profile {...});
profile object will contain:
{ id: username: first_name: last_name: full_name: gender: timezone: timezone_offset: email: }

To clarify, you want this to be a Botkit specific function, uniform experience between all platforms? I'm not sure of a case for such uniformity at this time, but as the controller gets unified maybe it'll be important. But this does leave out some things that are important platform specific uses, and will likely trip up people who have trouble finding things in the mountain of docs as it stands now.
This feature would lead to having two different profile api calls, so maybe it can be made clear it is an internal Botkit function?
In Facebook.js we don't currently have an API method to just get at the FB request for the messenger user profile info, I think there will be confusion for people as to why this doesn't include some of the more important FB profile information.
Such as profile_pic, last_ad_referral, is_payment_enabled, locale
Would be a good PR for someone to add at the user profile API call, ala this middleware someone wrote.
Profile pic is used as a hack to identify a user outside your Bot's scope (we get a page scoped ID, and it cant be used to link to someone using your other FB apps unless you compare the profile pic url)
Ad referral is getting more important as FB Messenger ads become more popular, helps with segmenting.
Locale is important too for internationalization


OpenComputers support

In the server console see:
its when player place OC Robot to world or take as item from ground
[15:34:33] [Thread-12/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[15:34:33] [Thread-12/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput
....

When place robot (creatix)

[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.rednetInput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput
[10:47:10] [Thread-14/ERROR] [prism]: Unsupported json list data type: [I for key oc:rs.bundledOutput

Any progress? This issue appear also when OC robot move and generate big log and console spam...


Trigger production bundle release for v0.15.3

Trigger production bundle release for v0.15.3
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/LGTM

/lgtm
/approve

/unhold


Favicons moved to client side.

I was up late and got inspired.  This pull request fixes most of #77.  Express server no longer creates favicons, and the client now creates a favicon if it can't get one from the server, and then posts it back to the server at /favicon.png.  Express server then saves it to disk if it didn't already have one.
This doesn't pass the create new favicon integration test, i'll have to track down the exact problem, but as best I can tell manually it is working as intended.
client/plugin/favicon.coffee could be a lot more elegant if someone who knows the canvas api takes a crack at it with the built in gradient stuff, this is just a client side port of the server side brute force port of Ward's ruby favicon generator.  I just went with what I knew created acceptable favicons for now.
Shouldn't effect the ruby server as long as it is still generating favicons.

This commit solves an unpleasant dependency issue for node. Although I see it as a step sideways from the point of view of new site creation, I'm happy to see activity in this area continue.


Update to 1.20

This will resolve #44.
Note: this still needs to be tested since paper server is not out yet.

Just a note for documentation purposes:
This PR makes JDA compile into the jar, which is not ideal, but paper doesn't seem to want to load the library and panics at the start.  Hopefully this can be fixed in a future version.


MAINT: Removed redifinition of trace1 in spatial/qhull

Reference issue

Refer https://github.com/rgommers/scipy/pull/28#issuecomment-871204576
What does this implement/fix?

Additional information

cc @rgommers

Thanks, lgtm