Jump-over/moveright is broken for triple-quoted strings in Python

Description
When filetype=python and in insert mode with the cursor within a """-delimited string before the closing quotes (at the | in """some text|"""), pressing " causes the cursor to jump over only one of the closing " characters. Pressing " additional times causes one " character to be inserted on the first press, then for every subsequent press four "s are inserted (one to the left of the new cursor position and three to the right).
It looks like the """ rule doesn't actually have a move_cond, and adding a basic one seems to work, though it probably misses some edge cases:
require('nvim-autopairs').get_rule('"""'):with_move(function() return true end)

Edit: One problem with enabling moveright is that the space bar will also trigger it for some reason. I vaguely remember encountering this issue before in a different context.
Edit 2: This only occurs if one of your pairs causes nvim-autopairs to map <space> in insert mode, though the pair can be otherwise completely unrelated.
Incidentally, there is no ''' pair included by default, though both """ and ''' are valid Python syntax.
Mapping  bug
No response
Steps to reproduce

In an empty buffer, set ft=python and enter insert mode.
Type """ and observe that the closing """ is auto-inserted.
Type " and observe only one closing " is jumped over.
Optionally, continue typing " and receive as many """ as you want without escaping the """string""".

Minimal config
vim.cmd [[set runtimepath=$VIMRUNTIME]]
vim.cmd [[set packpath=/tmp/nvim/site]]
local package_root = '/tmp/nvim/site/pack'
local install_path = package_root .. '/packer/start/packer.nvim'
local function load_plugins()
  require('packer').startup {
    {
      'wbthomason/packer.nvim',
      {
        'windwp/nvim-autopairs',
      },
      -- ADD PLUGINS THAT ARE _NECESSARY_ FOR REPRODUCING THE ISSUE
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. '/plugin/packer_compiled.lua',
      display = { non_interactive = true },
    },
  }
end
_G.load_config = function()
  require('nvim-autopairs').setup()
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing nvim-autopairs and dependencies.")
  vim.fn.system { 'git', 'clone', '--depth=1', 'https://github.com/wbthomason/packer.nvim', install_path }
end
load_plugins()
require('packer').sync()
vim.cmd [[autocmd User PackerComplete ++once echo "Ready!" | lua load_config()]]

I think I understand the issue with jump-over being triggered by unrelated bindings now. It is definitely an issue I encountered before, but I didn't look deeply enough into its cause at the time. A rule's with_move / move_cond callbacks actually get called whenever any mapped key is pressed when the cursor is before the closing pair for the rule, regardless of whether the actual key pressed matches the rule:
https://github.com/windwp/nvim-autopairs/blob/06535b1f1aefc98df464d180efa693bb696736c4/lua/nvim-autopairs.lua#L448-L451
This means that if you set a move condition for a rule, you currently need to check that opts.char matches the key you want to trigger the move. This seems unintuitive and doesn't seem to be documented, but it might not be an oversight as cond.move_right already does this check:
https://github.com/windwp/nvim-autopairs/blob/06535b1f1aefc98df464d180efa693bb696736c4/lua/nvim-autopairs/conds.lua#L227-L230
This doesn't work for multi-character closing sequences, by the way, but this can be fixed by changing opts.next_char to opts.next_char:sub(1, 1), at least so long as the first character of next_char is ASCII, otherwise Lua will not make your life easy. Actually, opts.next_char:sub(1, #opts.char) would probably work, but if you need Unicode support for auto pairs you're probably using it for the wrong thing. Or maybe you want to have auto pairs for «», I don't know.
Anyway, it seems to work fine for now with this in my config:
require('nvim-autopairs').get_rule('"""'):with_move(function(opts)
    return opts.char == opts.next_char:sub(1, 1)
end)

Ideally it should be smarter than this. """-delimited strings can have unescaped " within them, except at the beginning or end ("""a string with "quotes" in it""" parses fine, but """quote at the end"""" doesn't), though I'm not sure this causes problems on its own, as singular " characters don't currently expand when typing between """. On the other hand, the fact that you can have unescaped "s within """-strings means it could be undesirable to have a single " cause jump-over at all, for the same reasons as #228.
The next best thing might be to prevent a single " from causing jump-over within a """-string and have """ automatically clear itself and jump over the closing """ if typed before the closing """, but I'm not sure if this can be done cleanly with the current rules interface, and even if it could it might be too complex to have as default behaviour. """ can also be escaped (\"""), and as in #228 it should be possible to type this without triggering jump-over.

maybe the variable name next_char is make you think it is only one character and the key press but it is not.
It combine the key press and the string after the cursor so if you press " then next_char is """.
https://github.com/windwp/nvim-autopairs/blob/06535b1f1aefc98df464d180efa693bb696736c4/lua/nvim-autopairs.lua#L448-L451
it need to compare the end pair with next_char to check the rule because the default result of function can_move() is true (if it doesn't have any condition return false)
nvim-autopairs support autopairs with unicode character i don't test for multiple of unicode character but it have a test for 1 unicode character pairs.
https://github.com/windwp/nvim-autopairs/blob/master/tests/nvim-autopairs_spec.lua#L491-L505
btw: you can get the length of unicode string by vim.api.nvim_strwidth()

maybe the variable name next_char is make you think it is only one character and the key press but it is not. It combine the key press and the string after the cursor so if you press " then next_char is """.

Yes, the name is not ideal when the closing pair is multiple characters, but I did realize this. My point is that char is not checked at all by autopairs_map, so if a move_cond exists and returns true without checking that char matches what the rule expects, any key mapped by nvim-autopairs will cause jump-over so long as the characters after the cursor match rule.end_pair. i.e. if the "{", "}" rule was defined with only one move condition via :with_move(cond.done()), pressing ], ), " (and even openers such as {, (, [) would cause jump-over/moveright when the character after the cursor is }.

btw: you can get the length of unicode string by vim.api.nvim_strwidth()

This is useful but not for this problem. nvim_strwidth(str) returns how many display cells the characters of str occupy, but what I needed to do was test if opts.char matches the start of opts.next_char, hence opts.char == opts.next_char:sub(1, #opts.char). The relevant function would be one that returns how many UTF-8 bytes a string occupies, but that's just the length of the string, which I didn't think of at first. A function that returned the first (or nth) unicode character of a string would also be useful, but since char should match the keybinding (I think even if the binding is multiple key presses) it's much easier to do it this way.


3.1.0 version not scanning the classpath properly?

I tried to update from 3.0-M1 to 3.1.0 but no classes are shown in the generated swagger.json file.
Not sure why, but trying the official example failed for me as well, see: https://github.com/typekpb/swagger-maven-example
once running mvn compile swagger.json is generated with following structure:
	{
	  "swagger" : "2.0",
	  "info" : {
		"description" : "This is a sample for swagger-maven-plugin",
		"version" : "v1",
		"title" : "Swagger Maven Plugin Sample",
		"termsOfService" : "http://www.github.com/kongchen/swagger-maven-plugin",
		"contact" : {
		  "name" : "Kong Chen",
		  "url" : "http://kongch.com",
		  "email"<EMAIL_ADDRESS>		},
		"license" : {
		  "name" : "Apache 2.0",
		  "url" : "http://www.apache.org/licenses/LICENSE-2.0.html"
		}
	  },
	  "host" : "petstore.swagger.wordnik.com",
	  "basePath" : "/api",
	  "schemes" : [ "http", "https" ]
	}

am I missing here some important setting?

@typekpb 3.1.0 uses the new version of swagger-core.  In the latest swagger-core version, they changed the package names from com.wordnik.* to io.swagger.*.
Because of this, you will have to:

Use 1.5 of io.swagger.swagger-core in your pom
Change all of your swagger annotations from their com.wordnik.* namespace to to io.swagger.* namespaces.

works for me now, thanks!


Add embedding fonts to Studio docs

Embedding Fonts
While we include directions for embedding your own font directly in Mapbox Studio. There has been some confusion with this process. It could be helpful to include to also it the Studio docs, like on the Style Quickstart page.
Any objections @ajashton @geografa ?

:+1: @amyleew want to add a final step after Labelling roads that walks the user through downloading a font, adding it to their project and using it?

@yhahn Sure! :+1: I know a well-made, free & open-source font collective to download from League of Moveable Type.


[DE Number] Prebuilt number entity incorrectly recognizes "ein" / "einen" as numbers in German

For the pre-built number entity we noted that often the German articles „ein“ and „einen are recognised as numbers.
E.g.  „stelle lautstärke 5 ein“ (set volume to to 5). LUIS identifies „5“ and „ein“ as number.
Another example would be: „setze einen timer auf 23 Sekunden“ (set a timer on 23 seconds)  LUIS identifies „einen“ as a number.

@cahann Cases like articles being recognized as number as expected in certain circumstances and should be easy to filter application-side.
Having said that, cases like "stelle lautstärke 5 ein" are not article cases and should not be tagged as much as possible. We'll prioritize these.


ENH simplify the tree narration removing entropy explanation

Removing the narration linked to the internal split mechanism.
I added a couple of insights regarding the prediction.

LGTM overall. The things I am wondering about:

do we want to mention that the criterion can be either Gini or entropy or even skip this (I guess gini is mention in the output of plot_tree so maybe you need to mention it somehow)?
not sure what to do with https://275-246063957-gh.circle-artifacts.com/0/jupyter-book/python_scripts/trees.html#the-regression-criterion. Maybe showing a tree with two splits, 4 splits, etc ... similarly to what is done for classification to build up the intuition that the tree is piece-wise constant and that the split decides where the "jumps" happen. Right now the first plot for regression does not convey that very well:

(i.e. the piecewise constantness does not jump at you).
there is a "information gain" remaining somewhere, probably this can be removed and we can stay vague with a formulation like "best split according to some criterion that can be set"

(I guess gini is mention in the output of plot_tree so maybe you need to mention it somehow)?

Or maybe use plot_tree(tree, impurity=False) ...

do we want to mention that the criterion can be either Gini or entropy or even skip this (I guess gini is mention in the output of plot_tree so maybe you need to mention it somehow)?

Yes we can simplify. I would just mention that we use the Gini just to point to the tree plot and that's it.

not sure what to do with https://275-246063957-gh.circle-artifacts.com/0/jupyter-book/python_scripts/trees.html#the-regression-criterion. Maybe showing a tree with two splits, 4 splits, etc ... similarly to what is done for classification to build up the intuition that the tree is piece-wise constant and that the split decides where the "jumps" happen. Right now the first plot for regression does not convey that very well:

Yes I forgot that it was structured like this. Basically, we want to explain how it works in regression. Likewise the classification, we need 2 plots to explain the principle of predicting the mean of each partition which is in line with your comment. I will make this message to the point and reduce a bit the narration.

OK this should be better now. I apply your suggestion @lesteve

Merging, thanks!


A.17.IR22.conformity.degree.node

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
Note: in TG Requirement Coverage table (req# 22), the link to this test case returns a 404 error.
This test case seems to only fail if the degree of conformity is not mentioned. Perhaps it should also fail if there is no conformity node (which is checked in A.18.IR23.conformity.node).
This test case only needs to be executed in the context of 'scenario 2'.
Proposed Change
Update link in TG Requirement Coverage table.
Consider merging this test case with A.18.IR23.conformity.node.

typo? A.17.IR22.conformity.deegree.node should be degree (deegree is software)

Not sure if IR 22 and IR 23 are the same.
IR 23 is a test for some conformity statement in an ExtendedCapabilities section, where metadata for the service is supplied using a scenario 2 ExtendedCapabilities response.
IR 22 is more general, for a service where metadata is supplied in some external document, (where there is a scenario 1 ExtendedCapabilities response)  the conformity needs to be tested in that document

Agreed that merging with A.18 makes no sense.
Implemented precondition that test case metadataURL.node must indicate scenario 2
Typo in link fixed by commit 607ec0e


Represent the schemas as google sheet specs for easier feedback from DPCs

Produce the spreadsheets with the metadata attributes of the main schemas we hold under MorPhic. (Study, Cell Line, Differentiated Cell line, etc)
We will use those to gather feedback from DPCs in our follow up individually.

This format should only be done for Study, no need to do it for the others at least for now.


Custom filesystems - stabilization of FileSearchProvider and TextSearchProvider

FileSearchProvider and TextSearchProvider are API proposals that are needed to make virtual filesystems more usable. Both APIs have been created several years ago.
Many extension writers have been regularly (re-)asking for the stabilization of both APIs. Would it be possible to have the two corresponding issues fixed this year ? VSCode is a great code editor, and its extension ecosystem is rich because it exposes many APIs to the extension writers. Thank you !

#59921 (6 years ago)

https://github.com/github/vscode-codeql/issues/500
https://github.com/intersystems-community/vscode-objectscript/issues/296
https://github.com/iamhyc/Overleaf-Workshop/issues/60
https://github.com/ast-grep/ast-grep-vscode/issues/59
https://github.com/SchoofsKelvin/vscode-sshfs/issues/2
#201964

#73524 (5 years ago)

https://github.com/SchoofsKelvin/vscode-sshfs/issues/2

My use case
I am developping an extension that connects to a project server (LSP + custom methods) for development. There are no local files. I implement a FilesystemProvider that communicates with a virtual filesystem present on the project server.
Having FileSearchProvider and TextSearchProvider available (without the users needing to use VSCode Insiders) would greatly improve the UX.

Pinging @andreamah 🙏

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


error

crawl: error: running 'scrapy crawl' with more than one spider is no longer supported

Duplicated

Duplicated


LaunchWpaPerfToolsLinuxAndroid.bat does not work with latest Windows Performance Analyzer (Preview)

Windows Performance Analyzer (Preview) has been automatically updated in the end of last month and Microsoft-Performance-Tools-Linux-Android-1.5.5 release seems not to be compatible with it:

Thanks @jppenttila for reporting! Using your steps I was able to reproduce the same issue.
I have reported the issue to the WPA team and will let you know what the status is once they triage the issue

This should be fixed by #123 but I will not close until fully tested and a release is available

Closed - should be fixed in 1.6.1

Verified that version 1.6.1 works, thank you!


Pink model markers are rendered without shading

Might look better as spheres rather than flat circles. Perhaps the wrong Material?
3.3

4.0

Tom, objects live in scene and don't choose what lights are used to illuminate them. We can only change material pproperties. Totally open to changing defaults.

I don't know the details. You can specify which objects cast/receive shadows, so I figured it might be possible to specify which objects are illuminated by each light. I've edited by comment (above).

related to #509

The pink spheres look nice to me in OpenSim 03d9220-2018-04-09

Pink Markers seem to have shading. From distance and from different angles, they can look flat.

The model markers do have shading now (they are no longer flat circles) but they blend into a white background. Already noted on #509.


Split out GTI and TELARRAY in separate pages

Currently GTI and TELARRAY are listed as sub-sections of the event list spec:
http://gamma-astro-data-formats.readthedocs.org/en/latest/events/index.html
They should be split out in separate pages, because they are just as relevant for IRFs as for EVENTS.
For GTIs I'm not sure if we need to have our own page or if we can just link to
https://heasarc.gsfc.nasa.gov/docs/heasarc/ofwg/docs/rates/ogip_93_003/ogip_93_003.html#tth_sEc6.3
where we mention GTIs, or put it in the glossary.
Anyone wants to work on this or should I?

Hi, I can do this. I am just not sure where to put it.

There's some comments I'd like to make about TELARRAY ... this is stuff that's connected to event types and we'll specify this later, so it will be some extra section mentioning TELARRAY for now, but not a really useful spec. I still have to think what exactly to say and where it fits ... assigning this issue to me.


failed to pull docker image: Error response from daemon: manifest for public.ecr.aws/supabase/gotrue:vunspecified not found

Every time I try to do a supabase db pull, I am getting this error:
failed to pull docker image: Error response from daemon: manifest for public.ecr.aws/supabase/gotrue:vunspecified not found: manifest unknown: Requested image not found
Is anyone else running into this?
I have tried changing my IP address and using a VPN and all sorts of things. I have tried the latest homebrew version of the supabase CLI, and also the latest node version, and I run into this same issue on each of them.
When I try browsing to public.ecr.aws/supabase/gotrue directly, I get this error:
{"errors":[{"code":"DENIED","message":"Not Authorized"}]}

Ok, I did some digging. It looks like there is a way to over-ride the default registry that is used by specifying an environment variable.
I was able to finally get it all working, like so:
SUPABASE_INTERNAL_IMAGE_REGISTRY=docker.io supabase db pull

Someone on the supabase team might want to investigate why the public.ecr.aws registry is failing to serve the docker images?

Ok, I finally figured out what was going wrong.
The supabase folder had a ".temp" folder in it that had a number of text files specifying the version numbers of different images. The text file that was specifying the version number for gotrue just had a string that said "vunspecified".
This was apparently breaking everything.
I solved the problem by completely deleting the supabase/.temp/ folder in my project.


docs(babel-plugin-jsx-dom-expressions): improve config

Summary

Improve config documentations;
Fixed some typos.

Thanks


50 posts are not enough

There are some busy feeds out there and not being able to load more than fifty posts is an unnecessary restriction.

this is limited by the feed itself, there's no way to load something that's not there


May I ask how to understand the format of the train_set.txt?

Thank you for your code
Example of one line: train/target/000024.jpg, train/searching/000024.jpg, 0.29269230769230764, 0.22233115468409587, 0.7991794871794871, 0.7608061002178649
what's the meaning of the 0.29269230769230764, 0.22233115468409587, 0.7991794871794871, 0.7608061002178649?. And how did you calculate this training label form the bounding boxes of the current and previous frames?

In your case, it means the target should be in the bounding box with top left corner ()

Bounding box is in the form of <x1, y1, x2, y2>. In your case, it means if the second frame has size
(h, w) #h is height and w is width
the target should be in the bounding box with top left corner (y1 * h, x1 * w) and bottom right corner (y2 * h, x2 * w)

However the network output is 10x of the original bbx. Thus it always outputs values ranging from 0 to 10. Sometimes exceeding it though.

Thank you very much! The CaffeNet resize the crop image into 227227, the input of the network is 227227, no matter the size(w, h) of the crop image. So the output of the network need to multiply w and h in order to regain the size of the crop image.
Because of your code, I understand the detail of GOTURN.
Extremely grateful!

You are welcome~

Hi,
May I ask you why sometimes they have negative values ?
Evan in your test sest examples, I found some negative values
test_example/target/000020.jpg,test_example/searching/000020.jpg,-0.13129449838187718,0.29418666666666665,0.5079611650485438,0.95328
thanks

Hi, may I ask you the tensorflow version in this code?
I download the pretrained model from: GOTURN_MODEL.
Uncompress the checkpoints folder, then I got two items: checkpoint and checkpoint.ckpt-1.data-00000-of-00001
I got the error: Unsuccessful TensorSliceReader constructor: Failed to find any matching files.
I used the tf (1.1.0)
It seems that lack of the model.ckpt.index in the checkpoints folder.
Did you used the earlier tf version?

I used 1.0.0, but 1.1.0 should also work fine. I have added the checkpoint index file. It should fix the bug.

How Bounding box is in the form of <x1, y1, x2, y2>, usually from 0 to 1, but exceeding this range is also fine. bbox encoding related to encoding proposed in paper?
Is encoding in section 4.2 Learning Motion Smoothness is only used in train time for augmentation?

so does this mean when i use the output of the network to draw a bb,  fc4/10 should be my x1,y1,x2,y2   and then use the above x1w , y1h , x2w ,  y2h as a pixel wise output

Thanks for this important work!
According to the original article the results are relative to the search image  which is
and is based on source bbox (with spacing) resized to (227,227).
B.T.W -  In the code here I do not see spacing,
so I expect the result bbox to be:
x1 = int(arr[0] / 10 * search_bbox.width) + search_bbox.left
y1 = int(arr[1] / 10 * search_bbox.height) + search_bbox.top
x2 = int(arr[2] / 10 * search_bbox.width) + search_bbox.left
y2 = int(arr[3] / 10 * search_bbox.hegiht) + search_bbox.top
And in the original code (cpp) I see indeed something like this (Unscale and Unceneter )
However , testing on the images seems to give wrong bboxes. Could you please clarify how to draw the result bboxes?
Thanks,
Omer


True false operators for Maybe

I added true & | operators for Maybe<T> which gives Maybe.NoValue a meaning that feels more like None in python or nil in ruby (more like nil than None). Now you can borrow ruby's initialization syntax:
private Maybe<int> theAnswer;
private void DoSomething()
{
    // Initialize theAnswer to 42, if it doesn't already have a value
    theAnswer |= 42;
}

likewise, you can also use || for null coersion now:
list.Select(x => x || 0);

I'm aware this is already available through other iSynaptic facilities, but this gives it a very fluent syntax and makes C# start to feel even more like those scripting languages you've used.

Thanks for the pull request!  I'll take a look at it.

I'm really liking the addition of the | (or) operator.  I need to update it to be lazily evaluated, but that is really going to clean up some code I'm working on!  Can't believe I didn't think of that - nice catch!
I'm not so sure about the addition of the true and false operators.  It looks like (and I need to confirm this) all it offers is not having to type ".HasValue".  Additionally, testing HasValue should not be the norm; there should be an extension method that provides the desired logic, implicitly testing HasValue.  Also, given that it could cause confusion when T is bool (e.g.   Thoughts?

the true operator is required in order to make || work. For instance, in a |= b the compiler tests first if a is true (Maybe.true(a)) and uses b depending on what it finds. So yeah, this is basically just an untra-sweet shortcut for HasValue.
I left the false operator not implemented because it's only required for making && operator work. However, the code won't compile if you remove the false operator.

I should clarify a little more. The difference between | and || is that || short circuits, which requires the true operator. The trouble with | is that it always evaluates both sides, so in a |= b the compiler has b evaluated even if it won't be used. What you really want is a ||= b, but the c# compiler rejects it.  Because of this behavior, this pull request isn't quite as groundbreaking as it could have been, but the |= still is pretty cool.

I knew the || operator uses the true operator for short-circuiting. However, we don't need the compiler short-circuiting with Maybe < T >.  It already has built in support for lazy evaluation and short-circuiting.  So, leaving out the true operator would only allow the | operator to be used, but it would still behave the way you want - even when you use the |= operator.
Take a look at the Or extension method. That shows how you can "or" together two maybes without evaluating them.  I would just move that code into the | operator code.


feat(members): implement "member groups"

should be able to create "member groups".
a group will consist of an array of members.
this "member group" will be searchable in the "add member" action in the SideBar Menu on every board.
when adding the member group to the boards members:
all users on group will get privileges on board.
when creating a group the user must define a list of users and privileges for each user to be set on future boards.

UI - creating member groups can happen via side menu.
either by "create a new group" icon.
or by "save current members as a member group" option.


Fix runtime arg fuzzy matching

Fixes #119
Fixes #92
This fixes a lot of bugs with the 'runtime arg' which is used throughout rtx. It's the argument used in rtx local RUNTIMEARG, rtx where RUNTIMEARG, rtx install RUNTIMEARG, and many more places.
Originally the intention was for this without an "@" to behave the same as "@latest", but it would also sometimes use the version in .tool-versions.
This wasn't consistent throughout rtx and broken in several places that's hard to document fully.
What this change does is make a runtime arg like nodejs (without an "@"), look up the current version from .tool-versions. nodejs@latest will always use the latest available release, despite what may be installed. There were attempts to use the latest installed version to avoid a situation where a new runtime is released and this would require installing it, but for now I'm moving away from that model.
So now things are consistent: nodejs means .tool-versions. nodejs@latest means the latest available, not latest installed.

smoke testing seems to be working for this. I would not be surprised if this causes a bug or two because it's a pretty large change that impacts most of the project.
#119 and #92 seem to be resolved for sure though

Codecov Report
Base: 80.83% // Head: 80.27% // Decreases project coverage by -0.57% :warning:

Coverage data is based on head (f3d76ac) compared to base (0ccb4d0).
Patch coverage: 80.48% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   80.83%   80.27%   -0.57%
==========================================
  Files          79       79
  Lines        2437     2459      +22
==========================================
+ Hits         1970     1974       +4
- Misses        467      485      +18

Impacted Files
Coverage Δ

src/cli/activate.rs
85.71% <ø> (ø)

src/cli/env.rs
100.00% <ø> (ø)

src/cli/mod.rs
88.46% <ø> (ø)

src/cli/render_help.rs
100.00% <ø> (ø)

src/cli/args/runtime.rs
75.00% <61.11%> (-15.00%)
:arrow_down:

src/cli/uninstall.rs
78.57% <70.00%> (-6.05%)
:arrow_down:

src/cli/latest.rs
91.66% <80.00%> (-8.34%)
:arrow_down:

src/config/config_file/mod.rs
83.33% <83.33%> (-2.39%)
:arrow_down:

src/config/mod.rs
83.52% <87.50%> (+2.16%)
:arrow_up:

src/cli/install.rs
87.23% <90.90%> (ø)

... and 5 more

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

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


[EPIC] Tasks for a new Top Level Apache Project

Is your feature request related to a problem or challenge?
Update: the new project was approved by the board and we are now in the process of setting it all up
We are on track to have a new top level apache project for DataFusion https://github.com/apache/arrow-datafusion/discussions/6475 (see Proposal Document for more details)
Once this is approved by the ASF board, we will have some logistical things to change. This ticket tracks the work items
Describe the solution you'd like
Rename github repositories

[x] https://issues.apache.org/jira/browse/INFRA-25726

Note it is critical in my opinion that all existing links continue to work (as in the url https://github.com/apache/arrow-datafusion/issues/9691 will be redirected to the new repo). I believe this is the default behavior in Github when a repository is renamed but we should double check.

https://github.com/apache/arrow-datafusion/ → apache/datafusion
https://github.com/apache/arrow-datafusion-python → apache/datafusion-python
https://github.com/apache/arrow-ballista → apache/datafusion-ballista
https://github.com/apache/arrow-ballista-python  → apache/datafusion-ballista-python
https://github.com/apache/arrow-datafusion-comet → apache/datafusion-comet

Update docs

[x] https://github.com/apache/arrow-datafusion/pull/10130
[ ] Change hosting url from https://arrow.apache.org/datafusion/ to https://datafusion.apache.org (with a redirect from https://arrow.apache.org/datafusion/)
[ ] Change link under subprojects from https://arrow.apache.org/ to point at new docsite (or maybe we should just remove the link all together)
[ ] https://github.com/apache/datafusion/issues/10131

Infrastructure

[ ] https://github.com/apache/datafusion/issues/10151
[ ] https://github.com/apache/arrow-datafusion/issues/10133
[x] Create mailing lists<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>[x] (email trouble) https://issues.apache.org/jira/browse/INFRA-25727
[ ] (DNS trouble): https://issues.apache.org/jira/browse/INFRA-25731
[ ] Update ASF config https://github.com/apache/arrow/blob/main/.asf.yaml
[ ] Update committers list: https://github.com/apache/datafusion/issues/10154

Update release process

[ ] https://github.com/apache/arrow-datafusion/issues/10134

Announcements

[ ] https://github.com/apache/arrow-datafusion/issues/10135

We will also need to make some updates to the release scripts and associated documentation (link to reporter URL, mailing list name, and so on)

Note it is critical in my opinion that all existing links continue to work (as in the url https://github.com/apache/arrow-datafusion/issues/9691 will be redirected to the new repo). I believe this is the default behavior in Github when a repository is renamed but we should double check.

Yes, I just verified this behavior is the default. I created a repo at https://github.com/phillipleblanc/datafusion-rename-test - created an issue: https://github.com/phillipleblanc/datafusion-rename-success/issues/1 and renamed the repo to datafusion-rename-success - and now the above links redirect to the new repo name.

Thank you for double chekcing @phillipleblanc

Its official -- DataFusion is now its own Top Level Project!
https://projects.apache.org/project.html?datafusion
I plan to file individual tickets for the tasks listed above later this morning

🎉🎉

🎉🎉

I created https://github.com/apache/arrow-datafusion/pull/10130 to make some documentation updates to change the name from Apache Arrow DataFusion to Apache DataFusion

Regarding mailing lists, I think this may not be possible or hard to do, but just curious about if there are similar cases before. Is it possible to move previous threads related to DataFusion to new mailing lists?

Regarding mailing lists, I think this may not be possible or hard to do, but just curious about if there are similar cases before. Is it possible to move previous threads related to DataFusion to new mailing lists?

I am not sure -- maybe we could look in the INFRA JIRA tickets or file a request

I created https://github.com/apache/arrow-datafusion/issues/10133 and https://github.com/apache/arrow-datafusion/issues/10134 for logistics
I also filed https://github.com/apache/arrow-datafusion/issues/10135 to make a blog post -- I would be interested in anyone has opinions on the venue (should we post to the arrow blog or make the first post on a new datafusion specific blog)?

I got an email aknowledging the creation of the email lists, but the domain name does not appear to be working. I filed https://issues.apache.org/jira/browse/INFRA-25727 to track

@comphead  pointed out to me that the commiters list of the new repo is not updated yet. I will fix that now https://github.com/apache/datafusion/issues/10154

I also filed https://github.com/apache/datafusion/issues/10155, https://github.com/apache/datafusion/issues/10156, and https://github.com/apache/datafusion/issues/10157 to track the board reports (and build some institutional knowledge)

We are sill having DNS trouble -- filed https://issues.apache.org/jira/browse/INFRA-25731

Ok, the DNS issue has been resolved.
We have the website up (needs some links fixed) https://datafusion.apache.org/
The mailing lists are working as well. For example<EMAIL_ADDRESS>

Update .asf.yaml to point to to the new mailing list: https://github.com/apache/datafusion/pull/10189
I'm also taking a look at https://github.com/apache/datafusion/issues/10151

https://datafusion.apache.org/ now has the website live 🚀  -- thanks @phillipleblanc

this link currently still works and doesn't redirect -- will it redirect to the new one at some point? https://arrow.apache.org/datafusion/
(I know this is very new and in flight, appreciate the work here!)

We should rename slack and discord channels?

We should rename slack and discord channels?

Update here is that @andygrove  did so

Thanks to @kou  we have completed https://github.com/apache/datafusion/issues/10194 and the old doc site redirects to datafusion.apache.org now
I tested a few links like
https://arrow.apache.org/datafusion/library-user-guide/working-with-exprs.html
https://arrow.apache.org/datafusion/user-guide/cli/index.html

Update: @tisonkun has made a DOAP file 😄 -- #10233
I have created a proposed page with governance information: https://github.com/apache/datafusion/pull/10238

I filed a few more doc tweaks https://github.com/apache/datafusion/pull/10284 and https://github.com/apache/datafusion/pull/10285
I think all that is left for this epic is to write a blog post (https://github.com/apache/datafusion/issues/10135) and we can close it down

Actually, we also owe the ASF board a report each month for the first 3 months. I'll begin coordinating the first one shortly (tracked via https://github.com/apache/datafusion/issues/10281)

DataFusion Top Level Project announcement is live: https://arrow.apache.org/blog/2024/05/07/datafusion-tlp/

Also, we got a suggestion to make an official ASF press releas https://github.com/apache/datafusion/issues/10403


403 Responses with CAPTCHA

I'm scraping through a Tor connection.  But every request if its either get_tokens() or get() returns a 403 page with CAPTCHA.  When I browse to the site through a Tor browser I get right to the site without any CAPTCHA or 403.
My question is: how can I debug cfscrape in order to better understand what is going wrong?
My minimal functional code looks like:
`import cfscrape
proxies = {
"http": "socks5h://localhost:9050",
"https": "socks5h://localhost:9050"
}
url = "omitted"
tokens, user_agent = cfscrape.get_tokens(url, proxies=proxies)
print(tokens)`

There are several issues currently open for this problem. In my case, the server also returns captcha but in reality, it is not. Waiting for maintainers to handle if possible and feasible.

This page has a very close version of what I see.  So far my analysis is...
The script within "challenge-form" executes, passing the "data-ray" and "data-sitekey" parameters.  When it complete it'll generate a DOMContentLoaded event whose attributes include both the resulting hash and the time it took to complete.  These values are added as hidden input tags to the "challenge-form" form.  Although it's not immediately clear what is triggering the form submission.  Is it all the IFrame code?

@CalmLogarithm, would you please create a pastebin with the response content so it can examined? Are you sure that there's a CAPTCHA with that 403 response and that particular response is from Cloudflare?

@pro-src, Here is the HTML  from the page in question.  Aside from the frequent "cf-*" names and use of "CloudFlare", it also references a CloudFlare domain in the JavaScript: ajax.cloudflare.com.  And there is a CAPTCHA, it is pulled from Google's domain for reCAPTCHA.

@CalmLogarithm, since cloudflare-scrape is using the requests library and no exceptions are being raised, see this stackoverflow page on how you might try to debug that. You'll want to identify the difference between the requests that are being made in those two environments. Use Tor Browser's network monitor. If you will put the requests and responses for each environment in their own pastebin, I'll review it for you.

@CalmLogarithm, it's very possible that the only difference is the user agents. I would focus on the headers.

@pro-src With this Python requests log  the problem is obviously the User-Agent string.  Looking at the cfscrape code the User-Agent should be randomly selected from the provided list.  Is there a quick work-around I should use in the mean time to specify my own User-Agent string?

Yes. Since create_scraper returns an object that inherits from
requests.Session, simply initialize a scraper, ie by calling
create_scraper, and then set the user-agent dictionary key of the object's
headers attribute
On Sun, Mar 10, 2019, 10:21 PM Courtney Falk<EMAIL_ADDRESS>wrote:

@pro-src https://github.com/pro-src With this Python requests log
https://pastebin.com/0LLAKCFM the problem is obviously the User-Agent
string. Looking at the cfscrape code the User-Agent should be randomly
selected from the provided list. Is there a quick work-around I should use
in the mean time to specify my own User-Agent string?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Anorov/cloudflare-scrape/issues/190#issuecomment-471399740,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZHfLc02zKcfPS07jWYGqYsX0OabAmwmks5vVdnVgaJpZM4bV2uT
.

@Mattwmaster58 This definitely appears to be a bug because get_tokens is supposed to create an instance for you and override the user agent of that instance if the user_agent keyword argument is truthy.
https://github.com/Anorov/cloudflare-scrape/blob/25e875509e46392bf0363b380951ce69319428cc/cfscrape/__init__.py#L173-L177
@CalmLogarithm What version of requests is being used? Your minimal code would be:
`import cfscrape

proxies = {
"http": "socks5h://localhost:9050",
"https": "socks5h://localhost:9050"
}

url = "omitted"
user_agent = "Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:60.0) Gecko/20100101 Firefox/60.0"
tokens, user_agent = cfscrape.get_tokens(url, user_agent=user_agent, proxies=proxies)
print(tokens)`

I'll try to get around to debugging this but I'm currently helping to develop https://github.com/codemanki/cloudscraper which is a port of cloudflare-scrape.
Let me know if passing the user agent to get_tokens doesn't solve this for you and I'll prioritize this issue.

@Mattwmaster58 I can confirm that manually setting the User-Agent string via the session object works.  I was already passing a session object in order to specify proxies if that makes any difference.

@pro-src The requests library version is 2.21.0.


Add license

Hi, could you please specify the license for this repository? This project looks very useful, but others cannot use it legally when the project has no license file.
I know you've abandoned the project as you're no longer using it, but these few clicks in GitHub web could let others use your code.

Added a license.


Add support for syncing Linked files

Is it possible to add support for Linked files as well?

Ooop, I figured it out.  In a regular text file I just need to add <FORM></FORM> and anything I put in-between will be part of the linked file and anything outside will be part of the main file.  Todo lists inside linked files are awesome!


bugfix: the id can be string object, which contains ^[a-zA-Z0-9-_]+$.

Use uniform id rules for global rules, routes, service, upstream, stream_routes.
And use jsonschema to check it.

@shibingli you can create new PR with your test cases in https://github.com/apache/incubator-apisix/pull/1659

@shibingli you can create new PR with your test cases in #1659

OK


Update _forms.sass

This is awkward to overwrite.

@pandalion do you want me to merge this? Just saw it here

Hmm, I think this should be left as is.

A nicer solution would be to add a variable with a !default value of $form-error, you'll then be able to override that.


Casaia PIR323 integration - Node Descriptor response trigger a Leave

Hello,
I'm currently developing a Domoticz plugin using the zigpy library and especially the zigpy-znp lib.
In such I'm using a zzh USB key from electrolama.
While all of the device/brands I have tested works, I have an issue with the PIR323 from OWON and CasaIA which systematically send a Leave message after receiving a Node Descriptor Response from the coordinator.
While I cannot for sure say that the 0x000 manufacturer code is an issue, what I can say is using the zigpy layer with a Zigate doesn't generate such problem. This is why my suspicion are related to that.
I'm here attaching a sniff where you can see at packet 181 there is the Response, and then from packet 189 onward the device doesn't response any more (we see the retry done by zigpy) and finally packet 344 is the Leave message
PIR323-znp-pairing.pcapng.zip

Try to duplicate the same on other key sending Manuf.Code 0x0000 and there is no issue.
Closing this sorry


How does this differ from Expo?

Sorry, Discussions aren't enabled for this repo, but I'm super interested in your stack.

this isn’t released yet, but it’ll work with expo. it’s a middleware between expo and next, and a set of suggestions to work with both.
to learn more, see this discussion: https://github.com/axeldelafosse/expo-next-monorepo-example/pull/1

Thanks for the quick reply. Love your work!


chore(deps) bump lua-resty-openssl to 0.4.1

Summary
Bumps lua-resty-openssl from 0.3.0 to 0.4.1.

Superseded by https://github.com/Kong/kong/pull/5387.


Closes #144 | Add OpenLID Dataloader

Closes #144
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Hi @yana-xuyan, mind removing non-dataloader code change in this PR and opening another PR for that (or perhaps pull the most recent config from master and see it if it has been implemented)?
Thanks!

Let me check this issue later this weekend :)
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Salsabil Maulana Akbar @.>
发送时间: Thursday, January 4, 2024 11:04:57 PM
收件人: SEACrowd/seacrowd-datahub @.>
抄送: Yan XU @.>; Mention @.>
主题: Re: [SEACrowd/seacrowd-datahub] Closes #144 | Add OpenLID Dataloader (PR #241)
Hi @yana-xuyanhttps://github.com/yana-xuyan, mind removing non-dataloader code change in this PR and opening another PR for that (or perhaps pull the most recent config from master and see it if it has been implemented)?
Thanks!
―
Reply to this email directly, view it on GitHubhttps://github.com/SEACrowd/seacrowd-datahub/pull/241#issuecomment-1877244508, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJGALO3HVDVUUFNQBPRY2C3YM3AJTAVCNFSM6AAAAABBD5FNB6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZXGI2DINJQHA.
You are receiving this because you were mentioned.Message ID: @.***>

Hi, I have re-committed the dataloader. Please have a check when you're free. Many thanks!


movies_full_splunk.csv has numeric node_ref

node_ref should start with a alphabetic char, to allow using it as is.
| inputlookup movies_full_splunk.csv
| eval node_ref = "n" . node_ref
| eval rel_src_node = "n" . rel_src_node
| eval rel_dst_node = "n" . rel_dst_node

Fixed with 9236c2d2dfbed26c5ef83344be72b162366721f2


Update bluemonday

https://github.com/microcosm-cc/bluemonday/releases/tag/v1.0.20

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@64c3d55). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #21281   +/-   ##
=======================================
  Coverage        ?   47.00%
=======================================
  Files           ?     1017
  Lines           ?   138169
  Branches        ?        0
=======================================
  Hits            ?    64949
  Misses          ?    65277
  Partials        ?     7943

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

->  #21287


Add storage module and common module

close #86

Codecov Report

Merging #87 (af98a64) into main (0ea3df7) will decrease coverage by 0.01%.
The diff coverage is 70.58%.

@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
- Coverage   88.92%   88.90%   -0.02%
==========================================
  Files          66       68       +2
  Lines       11987    12002      +15
==========================================
+ Hits        10659    10670      +11
- Misses       1328     1332       +4

Impacted Files
Coverage Δ

xngin-common/src/bitmap/ffi_bitmap.rs
89.54% <ø> (ø)

xngin-common/src/bitmap/mod.rs
80.93% <ø> (ø)

xngin-common/src/bitmap/vec_bitmap.rs
95.68% <ø> (ø)

xngin-common/src/slice_ext.rs
89.58% <ø> (ø)

xngin-compute/src/col/mod.rs
100.00% <ø> (ø)

xngin-compute/src/col/nonsized.rs
0.00% <0.00%> (ø)

xngin-compute/src/col/sized.rs
89.71% <ø> (ø)

xngin-storage/src/codec.rs
0.00% <0.00%> (ø)

xngin-common/src/array/mod.rs
100.00% <100.00%> (ø)

xngin-compute/src/col/typed.rs
94.02% <100.00%> (ø)

Continue to review full report at Codecov.

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


Is it possible to include some default review comments? There should be a template wherein which default review comments will be set.

We as reviewers need a template in which we can put some default review comments.
When a pull request is created,these questions present in template should be created as review comments.
This will help developers as well as reviewers to not miss anything

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

ID: 0197b69a-56ae-8cef-88e3-2165b07e23de
Version Independent ID: 336f8906-e816-48ed-18ff-5ac7a8aa7a76
Content: Code review and feedback field reference - Azure Boards
Content Source: docs/boards/work-items/guidance/guidance-code-review-feedback-field-reference.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

Lindsey and wayne
On Thu, Jul 2, 2020, 4:29 PM krishnaGottuparthi<EMAIL_ADDRESS>wrote:

Closed #8768
https://github.com/MicrosoftDocs/azure-devops-docs/issues/8768.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-devops-docs/issues/8768#event-3508647665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6KK2N47RHM7WWTJ7DBHVDRZTU3NANCNFSM4OPFZ4YA
.

@krishnaGottuparthi - You can add comments to the Description field but not the Comments/Discussion field. See Sample work item templates.

Lindsey and wayne
On Thu, Jul 2, 2020, 3:32 PM krishnaGottuparthi<EMAIL_ADDRESS>wrote:

[Enter feedback here]
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com
http://docs.microsoft.com ➟ GitHub issue linking.

ID: 0197b69a-56ae-8cef-88e3-2165b07e23de
Version Independent ID: 336f8906-e816-48ed-18ff-5ac7a8aa7a76
Content: Code review and feedback field reference - Azure Boards
https://docs.microsoft.com/en-us/azure/devops/boards/work-items/guidance/guidance-code-review-feedback-field-reference?view=azure-devops
Content Source:
docs/boards/work-items/guidance/guidance-code-review-feedback-field-reference.md
https://github.com/MicrosoftDocs/azure-devops-docs/blob/master/docs/boards/work-items/guidance/guidance-code-review-feedback-field-reference.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE https://github.com/KathrynEE
Microsoft Alias: kaelli

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-devops-docs/issues/8768, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6KK2O7GQH2UQFRFA7HVBLRZTOE7ANCNFSM4OPFZ4YA
.


SOLR-15966 Get rid of WEB-INF/lib/jsr305-3.0.2.jar

https://issues.apache.org/jira/browse/SOLR-15966
WIP attempt of excluding jsr305 and javax.annotations jars from the release.
I did a full gradlew check on this branch, and it succeeded, so I think it does no harm simply excluding these?

I think we got rid of it now from the distro
gw dev
find solr/packaging/build/dev | grep -E "jsr305|javax.annotation-api"
# (nothing)

Should these be "more global" in some way? Now each module will need to exclude.
But if no more improvements are suggested I'll merge this soon.


Provide option to downgrade unknown import errors to warnings

When instantiating a component, it's probably safe to supply it with additional imports that aren't declared in its world. For example, consider a component that declares an import foo in its world but it doesn't use foo in its implementation. Consequently, foo won't be present in the component's WIT that is generated by witgen, which will cause a confusing unknown import error by wac. I think downgrading the error to a warning would be enough to let the user know about a potential logical error while allowing wac to run successfully.

I'm also running into this issue. For languages with good dead code elimination like Rust the resulting component binary might not reflect the exact targeted world if the any imports are not used - those imports will simply be removed. It would be nice if supply imports that don't exist was simply a warning as supplying an import that doesn't exist is essentially a noop.
Of course, this will largely go away with a programatic API where the actual world of the component can be dynamically inspected and only imports it actually requires can be provided.


⚠ Bofrost.de has degraded performance

In 59da613, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 911 ms

Resolved: Bofrost.de performance has improved in fdc8e68 after 21 minutes.


basic_json::value throws exception instead of returning default value

Bug Report
My apologies if this is expected behaviour, it just seemed somewhat unexpected.
I am using json objects to represent configuration files that may or may not be present. Missing files are represented as default constructed json objects.
When trying to retrieve a value using a json_pointer from an empty json object it is throwing an exception instead of returning my specified default value. This is happening because the empty json object is not an object and so it is failing the first test ( JSON_LIKELY(is_object()) ) and thus throwing an exception. This seems counter intuitive given the description of the function.
I would have preferred it that if for any reason the json object could not return a valid value that it should return the default. As it currently stands I will have to wrap all value requests to the json object in a helper function to make sure that regardless of the state of the json object I always either get the stored value or the defaulted value. This essentially makes the "value" function pointless, for my needs at least.
This is on Visual Studio 2017
Thanks for a great library by the way.

The value function documentation does say this:
@throw type_error.306 if the JSON value is not an object; in that cases,
using `value()` with a key makes no sense.

@nlohmann Typo: cases should be case.
It might make sense to allow returning the default value for a type of null, indicating that this particular thing was not provided, but continue to throw for anything else.
@unholytony Can you provide a complete sample of how you're using this, including the input that you expect to work and return a default value?

@gregmarr Thanks for the hint. Fixed with #875.

Yes sorry, I had not read the comment fully. My apologies. I see now that value is not meant to be safe in the way I had expected.
I have now modified my code to add a wrapper function that I will use instead of value.
My code was essentially
	nlohmann::json getJsonData( std::string const& filename )
	{
		auto const raw_data = getRawData( filename );	// returns a std::vector of the file content which may be empty if the file does not exist
		if ( raw_data.size() )
		{
			try
			{
				return nlohmann::json::parse( raw_data.begin(), raw_data.end() );
			}
			catch ( std::exception const& e )
			{
				LOG( "Failed to parse json data : ", e.what() );
			}
		}
		return {};
	}

	void readConfig()
	{
		auto const config_json = getJsonData( "some_optional_json_file.json" );
		int const width = config_json.value( "/backbuffer/width", 3840 );			// exception
		int const height = config_json.value( "/backbuffer/height", 2160 );			// exception
	}

Incidentally, when writing the safe wrapper I struggled to do so without resorting to a try catch scenario.
Is there any way of testing for the validity of a json_pointer without just trying to request it and catch the generated exception.
I had a brief look but couldnt find anything. It seems that even internally you use exceptions to indicate failure to find a key value.
which leads to the situation where
#define JSON_NOEXCEPTION
#include "json.hpp"

int main()
{
	auto const data = R"({ "hello" : "world" })"_json;
	auto const v1 = data.value( "/hello"_json_pointer, "default_value" );				// v1 == "world"
	auto const v2 = data.value( "/goodbye"_json_pointer, "please don't explode" );		// explodes
	return 0;
}

I don't personally use JSON_NOEXCEPTION so I might be missing something here but this doesnt seem right.
Thanks again.

So you would like to have a function like
bool json_pointer::is_valid(const json& j) const

returning true iff the JSON Pointer is valid for JSON value j? And this should work without exceptions under the hood?

not sure if helps, but what i do to avoid value() to throw an exception is:
            for (json::iterator it = j.begin(); it != j.end();)
              if (it.value().is_null()) it = j.erase(it); else ++it;

then when the object is clean of null values values() returns the default value if key is not present

wondering what the state of this issue is currently?
It would be great to have a value() method that has a no throw guarantee, and always returns the default value on failure.  This would remove a lot of boilerplate for config files.
Maybe the method signature could be changed to value(string key, T default, bool exception=true) , where exception=false guarantees no-throw.


Saving bookmarks doesn't list any existing bookmarks or folders to save to

Hey,
I recently commented on this WhiteSur3 on deviantart and I've noticed that the saving bookmarks window doesn't list any existing bookmarks or folders to save to. You can see in the attached image that I have bookmarks for sure and that the "Computer" folder is missing

This was dumb of me... - I clicked the arrow in light mode and then the bookmarks shows up
I think the arrow's appearance is wrong however.. downwards pointing arrow should in my view list available items where the rightwards pointing arrow should be the one be seen as closing the tree view?
Am I wrong about this?

Hi!
Sorry for the late reply, our time zones do not match : )
Anyway, could we know which version of WhiteSur are you using?

Anyway, could we know which version of WhiteSur are you using?
I was using the most recent beta of White Sur.
WhiteSur-V3-beta-2
https://github.com/FirefoxCSSThemers/WhiteSur-Tweaked/releases/tag/v3.0-beta-2

The behaviour of the arrows you present is the direct opposite of how the Firefox browser presents the bookmark arrows.
Expanded vs non-expanded tree

generally the bookmarks are not supposed to be used with the theme, so we didn't pay much attention to bookmarks.

That's of course alright 👍🏻  I wish you and your team continued success with WhiteSur.

Thank you!


Conduit Bundle low TPS

Issue Description:
It occurs in modpack, but when the mod is isolated the issue still occurs. I'm pasting the timings from the other mods enabled as well. Currently disabling conduit bundles to see if that alleviates the issue.
What happens:
Incredibly high TPS usage with only ~800 conduit bundles in the world.
Timings: https://timings.aikar.co/?id=bbe37e8316f14ef384a0e0cfe149aab8
What you expected to happen:
Normal tick rate for a block like this.
Steps to reproduce:

Place a large/moderate amount of conduit bundles in the world, connected to machines.
Leave server running for ~1 hour
Low TPS and other performance issues.
...

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.10.2-3.1.183
EnderCore: EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: 2447, using spongeforge

Your most recent log file where the issue was present:
No logs would be generated from this issue, though, I can provide debug information from the world snapshot.
[pastebin/gist/etc link here]

Just an update, disabling MEConduits and OCConduits seems to help alleviate some of the TPS issues, but blockConduitBundleTileEntity is still topping the charts, with very few of them actually in the world.

Including the timing report of the server with OC Conduits and ME conduits disabled. That seemed to be the culprit.
https://timings.aikar.co/?id=c0c3cbbf36894078b9148a14747604b7

Don't use aikar timings, use the vanilla profiler.
https://www.reddit.com/r/feedthebeast/comments/5mxn51/vanilla_has_an_built_in_profiler_that_not_a_lot/

Didn't know vanilla even had that... I'm still getting very high tick time from bundles, but I'll take a vanilla reading of timings as well, will paste after I let it run for a bit.
Any reason not to use aikar? I've been using them for quite a while now.

`---- Minecraft Profiler Results ----
// Hello world

Time span: 483397 ms
Tick span: 9334 ticks
// This is approximately 19.31 ticks per second. It should be 20 ticks per second
--- BEGIN PROFILE DUMP ---
--- END PROFILE DUMP ---
`

Going to assume this is not the desired results. Will be pulling the world tonight to test it without spongeforge entirely, but, that is the debug result from /debug on, /debug off.

aikar attributes everything to one thing unless it uses an event to trigger other things. That doesn't work very well with Forge mods that use capabilities and interfaces to talk to each other. So no matter which code causes the lag, if it's connected to a conduit aikar will blame the conduit bundle for it.
You see the same with the Farming station---not a single plant or block update under it. Just as if the Farming Station was just sitting there and burning CPU cycles to mine Bitcoin. Nope. 90% of the farm time is talking to plants, asking them if they are fully grown, asking them for their drops or resetting them to their "freshly planted" state.
Conduits are hit especially hard by this, as most they do is talking to other blocks.

I am having quite a bit of lag as well on my server Enclosed is the minecraft profiler and its results
---- Minecraft Profiler Results ----
// Maybe if you treated it better then it'll have more motivation to work faster! Poor server.
Time span: 73584 ms
Tick span: 1468 ticks
// This is approximately 19.95 ticks per second.


No code hint?

Why is there no code prompt? It's important that there are so many apis, and it's not an editor problem, I tried idea and VsCode

I have no idea why thats happening. Never happened to me. You can search for the Sprite class and the IDE finds it?

Your problem is independent of graphx, you will have the same issue with others libraries, check the Intellij documentation.
GraphX doesn't have a custom configuration of that.


Windows Guide

For the Remaining Noobs like myself  can you please complete a Windows Guide

On windows you have to have docker.
First install docker, then copy and paste the "Dockerfile" document and modify it with the required informations.
Then run where the Dockerfile is:
docker build -t tribeca .
docker run --restart=always -p 27017:27017 --name tribeca-mongo -d mongo
docker run --restart=always -p 3000:3000 --link tribeca-mongo:mongo --name tribeca -d tribeca

Easy!

I don't have windows but it looks like to be what you need.
What you want to be able is to run commands starting with "docker".
You can try a "docker info" or "docker version" to see if it's running.
The Dockerfile is there:
https://github.com/michaelgrosner/tribeca/blob/master/Dockerfile
You also have a guide there:
https://github.com/michaelgrosner/tribeca
Good luck!

would any one be willing to create a step by step how to get this running on windows , Willing to pay $100 USD for the hour to explain it to me

I cant even get it to run on a mac lol


Added methods for optional clause forUpdate, forReference and forView

Generally it looks good. After fixing the minor issue it will be ready to merged.

Thanks, could you please share what the issues are?

@coderdecoder01 I made some comments in your code. Are you able to see it?
Basically isEmpty() is a static String method, so you can't use optionalClause.isEmpty(). Instead String.isEmpty(optionalClause) would compile.

Ah, yes. Thanks

Also I'd recommend initializing the optionalClause with empty string ('' instead of ' ').

I am making a new pull request with changes and closing this one.

Actually, you can make incremental commit to your branch, and run git push to update the PR.
So that you don't have to close it and create another one.

Very nice

It's merged. Thank you.

Would you be able to write some test cases covering these locking statements?

yes, sure

Thanks for that. Since this PR is merged, you will have to create another PR whenever you're ready.


⚠ Bofrost.at has degraded performance

In 32c8621, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1121 ms

Resolved: Bofrost.at performance has improved in e476c7d.


Theming shadows requires 25

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

When overriding the theme shadows,  it needs 25 items in the array.
Expected behavior 🤔
A hard 25 item requirement is too specific.
What if I want 26?
What if I want 1?
Steps to reproduce 🕹
Steps:

Override shadows inside the object passed into createTheme
Watch TS complain

Context 🔦
I'm trying to create a theme which has only a couple shadows
Your environment 🌎

npx @mui/envinfo
  System:
    OS: macOS 12.2.1
  Binaries:
    Node: 14.17.4 - ~/.nvm/versions/node/v14.17.4/bin/node
    Yarn: 1.22.17 - ~/.nvm/versions/node/v14.17.4/bin/yarn
    npm: 6.14.14 - ~/.nvm/versions/node/v14.17.4/bin/npm
  Browsers:
    Chrome: 102.0.5005.115
    Edge: Not Found
    Firefox: 90.0.2
    Safari: 15.3
  npmPackages:
    @emotion/react: ^11.9.0 => 11.9.0
    @emotion/styled: ^11.8.1 => 11.8.1
    @mui/base:  5.0.0-alpha.81
    @mui/icons-material: ^5.6.2 => 5.8.0
    @mui/lab: ^5.0.0-alpha.80 => 5.0.0-alpha.82
    @mui/material: ^5.6.2 => 5.8.0
    @mui/private-theming:  5.8.0
    @mui/styled-engine:  5.8.0
    @mui/system:  5.8.0
    @mui/types:  7.1.3
    @mui/utils:  5.8.0
    @mui/x-data-grid: ^5.9.0 => 5.11.0
    @mui/x-data-grid-pro: ^5.10.0 => 5.11.0
    @mui/x-date-pickers: ^5.0.0-alpha.3 => 5.0.0-alpha.3
    @mui/x-license-pro:  5.11.0
    @types/react: 18.0.6 => 18.0.6
    react: 18.0.0 => 18.0.0
    react-dom: 18.0.0 => 18.0.0
    typescript: 4.6.3 => 4.6.3

Found on Google Chrome

Duplicate of https://github.com/mui/material-ui/issues/28820


[chip-test] Implement chip_sw_alert_handler_sleep_mode_alerts

Implements the chip-level test described in the issue #15764.
Signed-off-by: Bilgiday Yuce<EMAIL_ADDRESS>

Thanks @bilgiday - let's merge this so that nightlies can pick it up.

Thanks @bilgiday - let's merge this so that nightlies can pick it up.

Thanks @msfschaffner


Add cache to FedoraToOcflIndex getMapping

JIRA Ticket: https://fedora-repository.atlassian.net/browse/FCREPO-3712
Relies on: https://github.com/fcrepo/fcrepo-storage-ocfl/pull/39
What does this Pull Request do?
Add a caffeine cache to the DbFedoraToOcflIndex for the getMapping calls.
How should this be tested?
There should  be no functional difference, though I hope it is faster.
Interested parties
Tag (@ mention) interested parties or, if unsure, @fcrepo/committers

This will fail until the associated PR on fcrepo-storage-ocfl is merged.

Made the cache configurable, but I didn't do any value checking. So let me know if we should be limiting what the cache sizes can be set to. Ran a small test (11,000 containers) and with this change the ocf_id_map select happens and average of 3 times per transaction (versus 12 before).


MAINT: line.startswith shouldn't be bytes

closes #11107

pinging for a review.

Added test

Test failures are unrelated.

LGTM now, thanks @andyfaff


Clarify that ENetConnection's create_host and service must be called on client and server

Context: despite 3 years working coding servers and clients, it took me around 10 hours to connect two ENet peers, and I read docs + ENet website + searched in issues and forums (oof!).
So, in this PR I try to clarify some of things I wish were there.
(In a separate PR I plan on adding an example.)

I may be wrong as I am not an english native speaker, but I think stablish should be changed to establish in the notes.

@Mickeon

Also remember to squash your commits, see the Pull Request Workflow.

Sure I can do it, but GitHub already enables maintainers to squash and merge, and in the process, even rewrite the commit messages if they want to.
I see some back and forth in this repo like "change commit message" or "rebase every commit", while I see the reasons behind it, is there any reason why maintainers don't prefer to click on "Squash and Merge" button? Wouldn't it save time for both the maintainers and contributors?
I also ask this because I believe that fixup commits can be useful for reviewers!

Anyway! I won't block the PR on this question :) , applying changes and squashing, I appreciate the review.

Another small argument: Git beginners can have a hard time with interactive rebasing, IMHO this adds an unnecessary barrier to contributing, considering there is a button that solves it (it adds two more clicks, max).

Done, applied all changes, ty @ze2j too!!!

GitHub already enables maintainers to squash and merge

It does, but sadly it doesn't allow squashing and creating a dedicated merge commit, which we require for any PR to make the git history cleaner and more manageable, it also helps reverting changes, and helps keeping the history clean in the rare cases where we don't enforce single commits, having merge commits also helps with cherry picking I believe
And rebasing and squashing is a core part of the workflow in any repo and any project and is a good and important skill to know, so at least I feel that it's less of a barrier and more a part of the general process of learning how to work with the tools and workflows necessary for contributing, just as much as learning how to make commits themselves or create branches! And maintainers are ready to help if new contributors are having a hard time and ask for help to figure it out, along with the detailed documentation

@mhilbrunner

GitHub already enables maintainers to squash and merge

It does (though it's not enabled on this repo), but sadly it doesn't allow squashing and creating a dedicated merge commit, which we require for any PR to make the git history cleaner and more manageable, it also helps reverting changes, and helps keeping the history clean in the rare cases where we don't enforce single commits, having merge commits also helps with cherry picking I believe
And rebasing and squashing is a core part of the workflow in any repo and any project and is a good and important skill to know, so at least I feel that it's less of a barrier and more a part of the general process of learning how to work with the tools and workflows necessary for contributing, just as much as learning how to make commits themselves or create branches! And maintainers are ready to help if new contributors are having a hard time and ask for help to figure it out, along with the detailed documentation

Oh, I didn't know merge commits helped with cherry-picking and reverting changes! That's great to know.
Thanks for clarifying :)

CI failed after I applied this suggestion, it seems unrelated tho.

Thanks!

You don't call create_host on clients -- that's what you call to start serving a session. If the client is the server, they can't be a client to someone else, so maybe this isn't worded right?
You do need to create an ENetConnection on all sides, client and server, but you don't call create_host on clients, you call create_client.
If for some reason you needed to create a host on clients to get connections to work, that's a bug imo.

You do need to create an ENetConnection on all sides, client and server, but you don't call create_host on clients, you call create_client.

@Jimmio92 You are confusing ENetConnection with ENetMultiplayerPeer.
The first one, does indeed need either create_host or create_host_bound, and doesn't have a create_client method since it's almost a one-to-one mapping to official ENet API.

Ahh, I now see I absolutely did confuse the two. Sorry about that!


[WIP] Add Bintray settings to build.gradle

There're some SaaS/PaaS cloud services that are using Elasticsearch.
I want to write plugins for these services based on this plugin(want to use this plugin as a library).

@sakama thank you for improving. can you consider to transfer this repo under embulk account??

@muga Sounds good.
I don't have permission to transfer this repository since I don't have an admin permission.
Could you give me a permission?

@sakama I transferred this repo under embulk account. please confirm it.

Thanks.
I updated repository to "embulk" at build.gradle and created new organization at bintray.
https://bintray.com/embulk-output-elasticsearch
./bradlew bintrayUpload command works successfully.
https://bintray.com/embulk-output-elasticsearch/maven/embulk-output-elasticsearch


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #59 (5763498) into main (7f9ad27) will decrease coverage by 3.90%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
- Coverage   71.99%   68.09%   -3.91%
==========================================
  Files           4        3       -1
  Lines         457      351     -106
  Branches       28        0      -28
==========================================
- Hits          329      239      -90
+ Misses         94       78      -16
  Partials       34       34

Flag
Coverage Δ

cdk
?

go-lambda
68.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/index.ts

Continue to review full report at Codecov.

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


fix: standardised Snyk catalog errors handling

Applies Snyk Code standards when handling Snyk catalog errors .

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


UI is not blocking controls

Describe the bug
I'm noticing that when the dialog Loading is open, I can still touch the controls behind, causing some bugs in my app. Is there any way to lock the UI while user dialog is being displayed?
To Reproduce
Steps to reproduce the behavior:

Launch UserDialogs.Instance.ShowLoading(message, MaskType.Black);
Try click on controls from page
Page controls are accessible, where they should be blocked by the user dialog UI

Did you run Sample project from this repo, is there the same?
yes, the result is the same.
Smartphone (please complete the following information):

Device: any
OS: Android

Did you found a workaround?
Not

Thanks. Yes, can confirm this. But, unfortunately, I don't have enough time to fix this, and will do when I have time. If this is urgent for you, you may create a PR with the fix, or wait until I do this.

@Alex-Dobrynin I found the problem and applied the solution. Can I create a pull request? Can you grant me permission?

@Alex-Dobrynin I got it the problem and fix. Can I create a pull request? Can you grant me permission to create a branch?

@Alex-Dobrynin I got it the problem and fix. Can I create a pull request? Can you grant me permission to create a branch?

just fork this repo, do the fix in your fork, and the create a pr into this repo master branch

is this launched ?

@kberawala what do you mean by launched?


I install react-devtools and use npx react-scripts command to start it but failed. show as below

What docs page needs to be fixed?

Section:
Page:

What is the problem?
What should be changed to fix the problem?

This has nothing to do with Redux.


Precision is lost, and the id is recognized as a date.

{
    "id":<PHONE_NUMBER>143693812
}

url: https://jsonhero.io/j/rFjT37bQM4sV/tree

Hi @fuzhengyin,
We always display the original data as parsed by JSON.parse() at the top of the panel (under id in this case). Further down are just alternative ways of thinking about the data based on our best guess.
What you're experiencing here is that JavaScript uses the double number type for all numbers, and it's not 100% accurate. It can represent integers up to 2^53<PHONE_NUMBER>740992) accurately, but anything above that isn't guaranteed.

If you need to be able to accurately represent integers this large then you should use strings in JSON, and then in your code use BigInt(). This page has good info on this topic.
This is a brilliant article that explains floating point numbers in lots of detail.


[sqlpp11] update to v0.59

This PR updates sqlpp11 library to v0.59 and adds features to sqlpp11-connector-mysql to support MySQL connector.

Thanks for this PR! But I've fixed cpp-taskflow and fastrtps regressions, so please revert them.

Waiting for PR #11824 merged, this PR will fix qscintilla build failed on CI.

/azp run


fix(table): resizable table handles window width changes

Closes #1974
Summary
Added fix to make sure table columns can be manipulated (resized, hidden, shown) after the window has been resized. The main scenario solution is to add an "expander column" that grows and shrinks when a resizable table is resized.  For scenarios with useAutoTableLayoutForResize (i.e. resizable columns + table-layout:auto) where the browser resizes all the columns automatically there will be no expander column and we use a window resize listener to keep the actual column sizes and react code sizes in sync.
This PR also solved #1019
Change List (commits, features, bugs, etc)

added expander column to grow and shrink for resizable tables (unless the table uses useAutoTableLayoutForResize which makes resizing of all columns automated)
added window resize listener for resizable tables with useAutoTableLayoutForResize. Resize listener measures columns when the window is resized.
tested all storybook scenarios
added test cases
added role and aria-label to resize handle for better UI and easier testing
added dev dependency @testing-library/react-hooks to help with hook testing

Acceptance Test (how to verify the PR)

Test story from the issue and make sure the bug does not appear. Verify that column resize works. https://deploy-preview-2306--carbon-addons-iot-react.netlify.app?path=/story/watson-iot-table--with-resize-and-initial-column-widths-on-simple-stateful-with-row-selection-sort
Test a story with resizable columns + initial column widths.  Verify that each column gets the width assigned and that resizing the browser grows and shrinks the expander column and not the real columns. Verify that column resize works.  https://deploy-preview-2306--carbon-addons-iot-react.netlify.app/?path=/story/watson-iot-table--with-resize-and-initial-column-widths-on-simple-stateful-with-row-selection-sort
Test a story with resizable columns that do not have initial widths. Verify that column resize works. Verify that the expander column doesn't occupy any space before the browser window is resized.   https://deploy-preview-2306--carbon-addons-iot-react.netlify.app/?path=/story/watson-iot-table--stateful-example-with-expansion-max-pages-and-column-resize
Testa a story with resizable columns but that also has useAutoTableLayoutForResize. Make sure resizing works. Verify that all columns are resized when the browser is resized. Verify that there is no expander column. https://deploy-preview-2306--carbon-addons-iot-react.netlify.app/?path=/story/watson-iot-table--with-resize-and-no-initial-column-width-and-auto-adjusted-column-widths
Test a story without column resize and make sure it behaves like normal, i.e. columns are resized when the browser window is resized. https://deploy-preview-2306--carbon-addons-iot-react.netlify.app?path=/story/watson-iot-table--stateful-example-with-every-third-row-unselectable

Regression Test (how to make sure this PR doesn't break old functionality)
Try all (sorry) different table stories to make sure they look and act fine on render and on window resize. Only the last column should be resized for stories with hasResize unless useAutoTableLayoutForResize is used (which is only one story).

This works great almost everywhere, but I am running into some issues resizing the "expander" after hiding/showing columns in the table. We changed the behavior of this due to a bug report opened while you were out (see #2061). It now auto-distributes the space available to all columns rather than trying to reinsert the column with the same width as when it was removed. The "expander" column seems to fail to resize correctly after columns have been removed and re-added.

Good catch. Looks like this bug already exists in 'next', i.e. resizing fails after all columns are first hidden and then (some are) shown again. This is bug has probably been there for a while and is unrelated to the current changes. I would suggest that we open a new issue to fix that problem.

Has this bug been opened?

This works great almost everywhere, but I am running into some issues resizing the "expander" after hiding/showing columns in the table. We changed the behavior of this due to a bug report opened while you were out (see #2061). It now auto-distributes the space available to all columns rather than trying to reinsert the column with the same width as when it was removed. The "expander" column seems to fail to resize correctly after columns have been removed and re-added.

Good catch. Looks like this bug already exists in 'next', i.e. resizing fails after all columns are first hidden and then (some are) shown again. This is bug has probably been there for a while and is unrelated to the current changes. I would suggest that we open a new issue to fix that problem.

Has this bug been opened?

Yes it was opened when it was discovered. Perhaps I should have linked it from this PR for clarity. Here it is
https://github.com/carbon-design-system/carbon-addons-iot-react/issues/2340


[Logging v2] - container miss security context after daemonset re-deploy from Rancher

What kind of request is this:
bug
Steps to reproduce:

SELinux enabled RKE2 downstream cluster (CentOS 8.3)
Install Logging chart with selinux.enabled = true and select a Project
Chart installs properly, rke2 aggregator is present and available
Go to Rancher and in from the UI select re-deploy on the rke2 aggregator daemonset. (Ember UI)

Working security context after chart install
  spec:
      containers:
      - image: rancher/mirrored-fluent-fluent-bit:1.6.10
        imagePullPolicy: IfNotPresent
        name: fluentd
        resources: {}
        securityContext:
          seLinuxOptions:
            type: rke_logreader_t

Result:
container SecurityContext is missing if I check the YAML, the pod fails to create
CrashLoopBackOff: back-off 5m0s restarting failed container=fluentd pod=rancher-logging-rke2-journald-aggregator-z69zc_cattle-logging-system(39b0fd90-6746-41d4-ac9d-4fb47b1db930)
Environment information

Rancher version: v2.5.8-rc9
Installation option: HA RKE Centos 8 SELinux.

Found while testing #31309

The ember ui seems to set global.selinux.enabled to false.

@paynejacob I am not sure how the UI would be setting this value true. I talked to @izaac and he is not editing the resource in the Ember UI. He is simply redeploying the daemonset. I don't understand how that could be attributed to a UI issue because redeploy is an action with no inputs. Why do you think the ember ui is resetting the value?

Rancher version v2.5.8-rc12
This is working from the dashboard.
Managing these resources from the Ember UI (Cluster Explorer) is not a valid use case.

I am having issues with selinux enabled RKE2 environment and fluentbit daemonset. I get "Error: failed to create containerd task: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: failed to set /proc/self/attr/keycreate on procfs: write /proc/self/attr/keycreate: invalid argument: unknown" even reading the right label in "rancher-logging-fluentbit" daemonset definition manifest:
- image: rancher/mirrored-fluent-fluent-bit:1.6.10
imagePullPolicy: IfNotPresent
name: fluent-bit
resources:
limits:
cpu: 200m
memory: 100M
requests:
cpu: 100m
memory: 50M
securityContext:
seLinuxOptions:
type: rke_logreader_t
RKE2 is running with cis-1.5 profile and SElinux enabled.

I run into the same issue as @frjaraur. Any updates on how to resolve this?
Installed packages:
# rpm -qa | grep rke
rke2-selinux-0.9-1.el8.noarch
rke2-agent-1.22.8~rke2r1-0.el8.x86_64
rke2-common-1.22.8~rke2r1-0.el8.x86_64

I'm using Rancher Logging rancher-logging:100.1.1+up3.17.3.
Thanks & regards,
Philip


fix windows gpu build - add cusolver to cmake

required by:
https://github.com/tensorflow/tensorflow/commit/72c023d3967a3218cd3d830ce6e57f7c4d87a18c

Can one of the admins verify this patch?

Jenkins, test this please.


Credential window popup on PowerShell 2 and 3

Hello - it seems on PowerShell 2 and 3 (haven't tested 4) that we get a popup asking for credentials even if we don't specify that parameter (just the Test-PendingReboot function without any parameters).  I've commented out the parameter to bypass this, just wondering if you knew about it or if any other fix was available.

I wasn't aware, I'll have a look, thanks for opening an issue.


SEG-23 Reset custom dimensions before set

What does this PR do?
Reset custom dimensions before send for pageview events.
Are there breaking changes in this PR?
No.
Any background context you want to provide?
For single page application calling pageview event more that once without reloading the page or changing the page result in tracking custom dimensions which tracked in previous calls.
Is there parity with the server-side/android/iOS integration components (if applicable)?
No.
Does this require a new integration setting? If so, please explain how the new setting works
No.

@CarlosMecha Resolved PR comments.
Also we need not to do delete customDimension[prop]
Issue is due to how google analytics works, whatever property we set it persist as long as there is page reload or page change, analytics code does not persist any data on successive calls to page, it was google analytics holding the custom dimensions set.
So as a fix we just resetting the custom dimension.
There are 2 ways for resetting, either set it as empty or send reset event.

Closing this PR as it's merged to upstream.


ci: adding some checks

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


error: The default player (VLC) does not seem to be installed.  You must specify your path to a player executable with --player

Hey,
I'm quite new to this and I'm already a bit confused :D
It says I don't have VLC installed, even though I have it.
I've installed it at C/Programs/VLC.
If someone could help me, I would be very thankful :D
Freddy

Like it says you can use the --player command to specify the location of the player.
You can do this either in the command itself. For example:
livestreamer twitch.tv/twitch best --player="C:\Program Files\VideoLAN\VLC\vlc.exe"

Or you can change the value inside the livestreamerrc file.

Thank you very much :)

Hello every one!
I try this and no result for me why is VLC not responding? I even try removing the # in front of all VLC at the player list. can some one directer me to another solution please.
Thanks
Dennis


add return type for HCQCompatAllocator _alloc

While working on qcom allocator, spent some time to dive into the copyin, copyout, transfer and offset of HCQCompatAllocator to find out that they require allocated memory object to have "va_addr" and other stuff
So this change make allocator abstraction more transparent
It adds 3 new lines of code btw, so if it is considered to be superfluous, I'm ok :)

Types are always worth the lines! @nimlgen can you review?

Thanks!


MSFT_Disk: Implement workaround for maximum size discrepancy (fixes #181)

Pull Request (PR) description
This PR introduces a workaround that fixes issue #181 where the reported maximum supported size of a partition would be approximately 1MB bigger than the actual size of the partition. As long as it is unclear why the New-Partition and Get-PartitionSupportedSize cmdlets report different data if neither Offset nor Alignment parameters are used, this workaround should remain in place.
If this is not the case, Test-TargetResource for a disk configured to use the remaining size will always return false and trigger an endless cycle of LCM runs.
This Pull Request (PR) fixes the following issues

Fixes #181

Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md in the resource folder.
[ ] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[x] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #197 into dev will increase coverage by <1%.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           dev   #197    +/-   ##
===================================
+ Coverage   94%    94%   +<1%
===================================
  Files        8      8
  Lines      886    888     +2
===================================
+ Hits       834    836     +2
  Misses      52     52

I have 0 ideas why the build suddenly fails. I did a rebase since the changelog was getting in the way of a merge. Will have a look

Thanks @nyanhp - I'll get onto this review this weekend (finally will have some free time).

Hi @nyanhp - are you still working on this one?

Closing because replaced by #213


v.type in makeParams returns nil

For some users, the v.type (subCommand) returns nil.
Uncaught Error: /home/santa/deps/discordia-slash/Application.lua:33: attempt to call a nil value

Hi, can you provide the code you are using so I can test it?

It happens to any code, even with your ban example command.
It is related to specific phones/applications (for example an user who has an Asus phone, it breaks on him every time).

Oh, hm, this sounds like discord issue

put this on line 49


Notifications Page Redesign

We could probably run a animation so that they start as highlighted, then one second after opening the page, the background blinks and disappears which marks them as read.

I created a branch for this issue but no PR yet


Updates to the unified namespace documentation

Merged build finished. Test PASSed.

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

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.4 please


feat(textinputs): allow us handle cursor position in text inputs

Context:
I have an edit field for phone number, which auto formats number as it is typed.
When the auto formats adds a space between digits block (like in french phone numbers) or a dash (like in US phone numbers), the actual value length is incrase by 2 (the added separator, and the actual digit typed)  but unfortunately, the cursor only only registers one more char, i would need to force it to the end, but for that, i need the material impl using TextFieldValue instead of basic String
I can provide a video via slack, i have it ready i just can't upload it here.
I would like to be able to use
var textState by remember { mutableStateOf("Some text") }
VitaminTextInputs.Outlined(
    ...
    value = TextFieldValue(textState, selection = TextRange(textState.length)),
    ...
)

Thanks a lot @dimitriz09 for spotting this. We'll analyse when we can add this case. fill free to submit a contribution if you want it handled quicker :)


Set state for session instead of deleting it when deleting an applica…

…tion
Simplify by just setting the state of the active session belonging to an
application that is deleted. This is more in line with what we do for other
transitions.
Inactive sessions will be deleted by maintainers in config server after
some time.

How long are inactive sessions kept? Do we need more aggressive GC to keep disk usage down?

Deleting an application is not done very often, whereas periodic redeployments are done often, so I don't think GC will need to change due to this

It's the opposite for CD systems, but I guess you're right anyway.

👍


not working on android

hi, player controls are not working on android.
import React from "react";
import Navigators from "./src/navigators";
import Store from "./src/Store";
import { Provider } from "react-redux";
import {
initialWindowMetrics,
SafeAreaProvider,
} from "react-native-safe-area-context";
import VideoPlayer from "react-native-reanimated-player";
import { useSharedValue } from "react-native-reanimated";
export default () => {
const videoHeight = useSharedValue(300);
const isFullScreen = useSharedValue(false);
return (

<VideoPlayer
showOnStart={true}
source={{
uri: "https://famee.bitwork.tech/public/uploads/BkHyttGZb9.mp4",
}}
headerTitle={"Title in full screen mode"}
onTapBack={() => {
// Alert.alert('onTapBack');
}}
onTapMore={() => {
// Alert.alert('onTapMore');
}}
onPausedChange={(state) => {
//Alert.alert(onPausedChange: ${state});
//setPaused(state);
}}
videoHeight={videoHeight}
//paused={paused}
isFullScreen={isFullScreen}
disableControl={false}
/>

);
};
at start controls shows but after playing video, they disapears

@Nadimkhan120 Hi,
I just tested on android, It's no problem.
What's your react-native-reanimated & react-native-gesture-handler versions?

react-native-reanimated is 2.3.1 and  react-native-gesture-handler is 2.3.0

It's ok, could you show the video or gif?
Let's me see the problem

Hi, I am facing same issue please take a look
https://github.com/alantoa/react-native-reanimated-player/issues/9

same questiopn


What operation system did you use to test?

I tried to use Ubuntu to test. However, when I installed the Nvidia driver, there was a loop in the Ubuntu login interface. I solved this problem by installing the  Nvidia driver without installing openGL. And I installed the Mesa to use openGL. However, the version of openGL is 3.0. I want to use the openGL 4.x. I do not know how to upgrade the version of openGL.
And one more question, how did you install the openGL? Did you install openGL while installing the Nvidia driver?

These repository contains scripts to simplify the configuration process. You just need to follow these steps.