Total duration and speed in progressbar.

Hi, it would be great to be able to fully replace tqdm with click. I am particularly missing the information of current duration and speed (depending on units) as the bar progresses, and the total duration at the end of the run.
I am sure I can do this by adding info parts manually, but I feel many of us would get very happy to see this two fields added by default to the progressbar. Neither the total duration nor the speed need to know the length of the iterator, this should be relatively easy to implement.
I just forked the code, and I am up to add this and create a PR. But I would like to know if you have some strong comments against this, or previous ideas on how to implement it.

I don't think we are interested in making the progress bar implementation more complicated. In fact, I think David recommends tqdm when people are looking for progress bar features that Click doesn't have. That being said, if your PR doesn't add too much complexity it might be considered.

Yes, please use tqdm, I don't plan to add more to our implementation.


Layout made by splits

Hi, thanks for maintaining this plugin
I'm trying to open three splits using a layout to get something like this

But the following code
local Split = require("nui.split")
local Layout = require("nui.layout")

local split_one = Split({
  enter = true,
  relative = "win",
})
local split_two = Split({
  relative = "win",
})
local split_three = Split({
  relative = "win",
})

local layout = Layout(
  {
    relative = "editor",
    position = "bottom",
    size = 30,
  },
  Layout.Box({
    Layout.Box(split_one, { size = "50%" }),
    Layout.Box({
      split_two, { size = "50%" },
      split_three, { size = "50%" },
    }, { dir = "col" }),
  }, { dir = "row" })
)

layout:mount()

gives the following error
E5113: Error while calling lua chunk: .../site/pack/packer/start/nui.nvim/lua/nui/layout/init.lua:338: missing child.size
stack traceback:
        [C]: in function 'error'
        .../site/pack/packer/start/nui.nvim/lua/nui/layout/init.lua:338: in function 'Box'
        splits.lua:23: in main chunk

Currently the Layout component only works with Popup.
But supporting Split is surely something I'd like to do.

FYI I've just implemented split mode in my plugin competitest.nvim
I had a hard time looking for a solution to open splits correctly with the correct size.
Then I found out I had to disable equalalways, to enable/disable winfixwidth or winfixheight at the right time and to keep the cursor out of the current window to prevent it to be resized.
You can look at my implementation here
layouts are defined as tables, examples here

Thanks for sharing that! @xeluxee 😃
https://github.com/MunifTanjim/nui.nvim/pull/165 does almost all the same things as your implementation. But I was struggling a bit with the sizing issue. The missing piece was setting equalalways to false before mounting and resetting it afterwards.
It'll try it out and try to merge that PR soon.

Turns out modifying equalalways is not a good idea. The moment equalalways is enabled, all the splits are automatically resized... so that defeats the purpose.
Anyway, #165 just got merged. NuiSplit can now be used with NuiLayout.

Great, thanks! I'll try it out as soon as possible

Turns out modifying equalalways is not a good idea. The moment equalalways is enabled, all the splits are automatically resized... so that defeats the purpose.

I'm not experiencing this behaviour in my plugin. I'll look into it


How can you get this link

  const collectionResponse = await fetch(`http://<IP_ADDRESS>:${proxyPort}/lol-champions/v1/inventories/${summoner.id}/champions`);
  const masteryResponse = await fetch(`http://<IP_ADDRESS>:${proxyPort}/lol-collections/v1/inventories/${summoner.id}/champion-mastery`);

I try to captrue but it doesn't work. How can you know this url

It's the internal LCU API. You can use this app to see all the methods https://github.com/Pupix/rift-explorer


Code coverage generation not working

$ npm run test-cov
...
No coverage information was collected, exit without writing coverage information

It's has not worked since the beginning, so it's not a bug we introduced.
Honestly, it's kind of weird:
the test-cov script is npm run build && babel-node node_modules/isparta/bin/isparta cover --report html --report text node_modules/.bin/_mocha, which does not work.
However, if i switch node_modules/.bin/_mocha for my globally installed _mocha, it works perfectly. The version is the same, so I reckon it has something to do with relative paths.

Done


In SPI/CGPA converter.

Rules is displayed like a button with hover effect but it's a heading in SPI/CGPA converter. User may try clicking it many times but wouldn't get any other output.

Please assign this to me.

Go ahead
Deadline 1
30 March,2021 (11:59 PM IST)


Mochongli patch 1

fast

Sorry for not being more responsive, but if this is important, perhaps it should be configurable instead.

Will include a configurable version of this in 5.0.0.


[ko] Update outdated files in dev-1.23-ko.1 (M25-M33)

Related issue: #30845 (dev-1.23-ko.1)
Related PR: #30850 (dev-1.22-ko.4)

[ ]  M25.  content/en/docs/concepts/services-networking/service-traffic-policy.md  | 1(+XS) 1(-)
[ ]  M26.  content/en/docs/concepts/services-networking/service.md  | 2(+XS) 2(-)
[ ]  M27.  content/en/docs/concepts/storage/persistent-volumes.md  | 42(+M) 13(-)
[ ]  M28.  content/en/docs/concepts/storage/storage-capacity.md  | 1(+XS) 2(-)
[ ]  M29.  content/en/docs/concepts/storage/storage-classes.md  | 2(+XS) 2(-)
[ ]  M30.  content/en/docs/concepts/storage/volume-pvc-datasource.md  | 1(+XS) 1(-)
[ ]  M31.  content/en/docs/concepts/storage/volume-snapshot-classes.md  | 1(+XS) 1(-)
[ ]  M32.  content/en/docs/concepts/storage/volume-snapshots.md  | 1(+XS) 1(-)
[ ]  M33.  content/en/docs/concepts/storage/volumes.md  | 82(+M) 159(-)

/language ko

/assign

@seokho-son 리뷰 감사합니다.
모두 반영했고, 2번째 변경 사항만 별도로 확인하실 수도 있도록 squash는 아직 하지 않았습니다. 😊

@seokho-son 감사합니다. squash 도 완료했습니다. 😊

/approve


gtk2-perl: init at 1.24993

Motivation for this change
I build this accidentally as I thought I need it for gcstar. So since I ended with a package which might be useful in the future... here it is this PR
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

It is already in nixpkgs as perlPackages.Gtk2

okay then... there is no need to have it again... sorry was not sure they are one and the same thing


Thaumometer causes lag Spike of death

Everytime that i hold the thaumometer i get affect by massive lag

Probably a duplicate of #478 which is being addressed, should be better in Beta 17.


AC: Syndicate with Mesa 19.1.1

There was a problem with the latest mesa point release.  Mesa 19.1.1  gave me a white screen
when playing Assassin's Creed Syndicate. It painted the HUD, though .
I decided to downgrade back to  19.1.0.
from inxi:
Graphics:  Display: server: X.Org 1.20.5 driver: amdgpu unloaded: fbdev,modesetting resolution: 1600x900~60Hz             OpenGL: renderer: Radeon RX 580 Series (POLARIS10 DRM 3.30.0 5.1.14 LLVM 8.0.0) v: 4.5 Mesa 19.1.0 (git-1517811f4f)  ... Kernel: 5.1.14 x86_64 bits: 64 Desktop: LXDE 0.10.0

If you it works on Mesa 19.1.0 and doesn't work on 19.1.1, wouldn't the Mesa bugtracker be a better place to report this?

Yes, you're are right of course. It is more likely that my llvm 8.0 is old and mesa developers are using
svn.

If I'm off tomorrow I'll do a git bisect on mesa.

Recompiling 19.1.1 didn't give me a white screen. So,  there is nothing to bisect. I did upgrade dxvk. Maybe that fixed it?


Renamed 7zip.7zipAlpha 21.02 alpha to 21.02-alpha

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Name          Id                  Version     Available       Source
--------------------------------------------------------------------
7-Zip - Alpha 7zip.7zipAlpha      21.02 alpha 21.02-alpha     winget

Why was version changed? Now it shows up all a time that there is new upgrade because version does not match.

Because previous manifest versions was that way which made the real version not appear as the latest version, feel like it should indeed be changed back..

Because previous manifest versions was that way which made the real version not appear as the latest version, feel like it should indeed be changed back..

Real version has different name and different id so I don't think that alpha should conflict with it? Issue is that version string in manifest is "21.02-alpha" when reported version string is "21.02-alpha"


ENH: Add macros to itk::kRegistrationParameterScalesEstimator ivars

Add getter and boolean macros to
itk::kRegistrationParameterScalesEstimator ivars.
Check the expected values in the test using the appropriate itk testing macros.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)

To be rebased on master after PR #4075 is hopefully merged.


Update of light status (switching between RGB and TW mode) needs 30-60 seconds and changing brightness shortly after RGB/TW mode change is not correctly done

It seems as if the integration only polls the state of the lights only every 30 or 60 seconds if a light has been switched from RGB color mode to a TW mode.
If only selection a new RGB color, when the light is already on RGB mode, the color is presented in the Home Assisant GUI immediately. However if one clicks on a TW white-color, the lamp immediately reacts, but the HA interface still shows the selected RGB color from before.
Even more important:
Additionally if one tries to change the brightness of RGB or TW directly after switching the mode (and not waiting 30-60 seconds for HA to update the status in the GUI) the light switches back to the prior mode and applies the brightness change on this prior mode instead of the color/tw selected seconds before.
A workaround (especially in automations) is to first set the target brightness and afterwards change to the desired RGB color or TW. Doing it in the opposite order, results in the bug described above. Due to the bug it's also not possible to directly set brightness and target color/mode in the same step within an automation GUI-based script (no source-code).
So for example it currently is not possible to switch from RGB red color with 80% brightness to a TW white color with 30% brightness in one step and even if divided in to steps a specific order has to be used, which is not necessary when using the original Awox app or the physical BLE remote.
A sure sign that this "fallback to the old color"-bug wont happen is to wait 30-60 seconds until the GUI shows the newly selected mode(RGB or TW) in the dashboard.
HAOS 11.1 with HA core 2023.11.2
home-assistant-awox release 0.1.5

See https://github.com/fsaris/home-assistant-awox/discussions/111#discussioncomment-7611114 for the delay issues.
The update cycle had been improved in https://github.com/fsaris/EspHome-AwoX-BLE-mesh-hub and could be ported back here once #110 is resolved.


failed to copy text with mouse after using nvim-osc52 inside the tmux in kitty

Reproduction steps:

launch kitty, run tmux locally.
inside tmux, select text A using mouse, and paste with ctrl-shift-v, which output text A as expect
inside tmux, launch a pane, open nvim with nvim-osc52 installed and edit a file
copy text B in neovim with yy
paste in another pane in tmux, which works well
select text A using mouse again, and paste with ctrl-shift-v, which output text B
change kitty to other terminals like alacritty, repeat steps 1-6, the problem disappears.
launch kitty, ssh to remote server and run tmux, repeat steps 1-6, the problem also disappears.

Minimal Configuration:
tmux
set -ga terminal-overrides ",xterm*:Tc"
set-option -g status-interval 5
set-option -g automatic-rename on
set-option -g automatic-rename-format '#{b:pane_current_path}'
set-environment -g COLORTERM "truecolor"
set -s set-clipboard on
set -g mouse on

kitty: default configuration
nvim-osc52 configuration:
require('osc52').setup {
    max_length       = 0,    -- Maximum length of selection (0 for no limit)
    silent           = true, -- Disable message on successful copy
    trim             = true, -- Trim surrounding whitespaces before copy
    tmux_passthrough = true, -- Copy from tmux into vim
}

I'm very confused about this issue, since if I running the following osc52 command in tmux and kitty, the problem won't happened:
esc="\033]52;c;$(base64 <<< 'hello world')\a"
esc="\033Ptmux;\033$esc\033\\" # tmux needs sequence escapement
printf "$esc"

Finally, thanks for your plugin which improves my remote development experience.

Close the issue as this is kitty's problem


Let the examples actually resolve to Atomic Data

People will follow these links and get 404s, and that limits how fast people will understand how this all works.
So, we have at least few options on making the URLs resolve:

Use existing RDF resources (with the downside that it's not actually atomic data)
Create IPFS links (that don't work for most people, who don't have ipfs resolvers in their browser)
Set up a server... And actually host atomic data. Seems like the way to go, but doing this right also requires tooling.

Any other ideas?

Many of the URLs currently actually resolve, except for some of the demonstrative example.com URLs. I think the easiest approach is to get rid of these - or host them on my surfy.ddns.net domain.


fix to issue searching for caption at startTime = 0

A start time of zero created an undefined search result which resulted in some weird behaviours.


First data date in daily field is wrong

The date of the first element of daily property of the latest summary is wrong since it's 2002-05-10: 18 years ahead. Also the month and day are wrong, it should be something like 2020-01-08, so I suppose there is a problem with the original data.
"daily": [
    {
      "confirmed": 1,
      "confirmedCumulative": 1,
      "deceased": 1,
      "deceasedCumulative": 1,
      "recovered": 0,
      "recoveredCumulative": 0,
      "critical": 0,
      "criticalCumulative": 0,
      "tested": 0,
      "testedCumulative": 0,
      "active": 0,
      "activeCumulative": 0,
      "cruiseConfirmedCumulative": 0,
      "cruiseDeceasedCumulative": 0,
      "cruiseRecoveredCumulative": 0,
      "cruiseTestedCumulative": 0,
      "cruiseCriticalCumulative": 0,
      "date": "2002-05-10",
      "confirmedAvg3d": 0,
      "confirmedCumulativeAvg3d": 0,
      "confirmedAvg7d": 0,
      "confirmedCumulativeAvg7d": 0
    },

Looks like this was caused by some typos in the data. Fixed all the occurances I can find, should be fixed in the next push in 15 mins.

Resolved, thanks!


add support for node 10.x

@niftylettuce any reason we don't support 10 yet?

will be releasing new version soon, will post a comment here once done.

See v2.0.0 via npm install frisbee@latest or yarn add frisbee@latest.
This release includes refactoring for Node v10+ support, new tests with ava (instead of mocha), and use of @babel/preset-env targeted to Node v6.4+ and the last 2 versions of browsers (see .babelrc and .browserslistrc).

Thank you so much @niftylettuce for doing and maintaining the project !


Art stuff

Updated the art. Think everything should work fine. Changed some file names, but you might want to change them back, I don't know. The perks seem to work fine as is, but couldn't find were to change capitalization stuff.
Added new art for ambrosia luck, and blueberry time, but didn't mess with actual code since it is much more complicated now and I don't want to put the effort to learn it enough to change it, right now. But here is how I imagined it if anyone wants to implement it.

Absolutely beautiful, once again. Thank you very much for the images!


Always getting error "No such file or directory: 'webdriver-py-profilecopy'"

tbselenium 0.5.3 (or 0.5.2)
selenium 3.141.0 (tried some other >3 versions)
tor browser 10.0.7 (linux64)
Code worked for some time, but now after launch getting error like No such file or directory: '/tmp/tmpgpl9c40g/webdriver-py-profilecopy'
Running code as root and directory actually exists..

@l4zygreed thank you for reporting.
Unfortunately, this is a generic Selenium error, so it's hard to say what's going on just based on this message.
Some questions that may help understand the problem:

Do you see a geckodriver.log in the profile directory? You may search for unexpected error messages in that log. Please avoid sharing it publicly.
Is this a windowless (e.g. over SSH) session? If so, do you use XVFB or similar virtual display servers?
Does the same code work on other computers?
Do the examples work?

@l4zygreed thank you for reporting.
Unfortunately, this is a generic Selenium error, so it's hard to say what's going on just based on this message.
Some questions that may help understand the problem:

Do you see a geckodriver.log in the profile directory? You may search for unexpected error messages in that log. Please avoid sharing it publicly.
Is this a windowless (e.g. over SSH) session? If so, do you use XVFB or similar virtual display servers?
Does the same code work on other computers?
Do the examples work?


No way to provide style for the TextInput itself

No way to set fontsize for it for example.

Did you try to set the style on the AutocompleteInput prop?
<Autocomplete
  style={{
    fontSize: 20,
  }}
 //...
/>

You can also render your own input by passing a render function into renderTextInput:
<Autocomplete
  renderTextInput={(props) => <TextInout {...props} style={/*...*/} />}
 //...
/>


InfluxDB 1.8: V2 compatibility write API doesn't support unsigned integers

The line protocol in InfluxDB 2 adds support for unsigned integers (suffix 'u').  V2 compatibility write API in InfluxDB 1.8 doesn't handle this.
Steps to reproduce:
curl -v -XPOST "localhost:8086/api/v2/write?bucket=test" --data-raw "test f=3u"
Expected behavior:
No error
Actual behavior:
< HTTP/1.1 400 Bad Request
< Content-Type: application/json
< Request-Id: a4c703d5-808b-11ea-8007-d66d6d2aa003
< X-Influxdb-Build: OSS
< X-Influxdb-Error: unable to parse 'test f=3u': invalid number
< X-Influxdb-Version: 1.8.0
< X-Request-Id: a4c703d5-808b-11ea-8007-d66d6d2aa003
< Date: Fri, 17 Apr 2020 09:13:04 GMT
< Content-Length: 56
<
{"error":"unable to parse 'test f=3u': invalid number"}
* Connection #0 to host localhost left intact

Environment info:

System info: Windows 10 Professional
InfluxDB version:  InfluxDB v1.8.0 (git: 1.8 781490de48220d7695a05c29e5a36f550a4568f5)
Other relevant environment details: Container runtime, disk info, etc

see also https://github.com/timberio/vector/pull/2868


validate_endpoints does not check leader field of GetMembersResponse

Bug Report

What version of TiKV are you using?

nightly
What did happened?
During the initialization of TiKV, it will check pd connection in validate_endpoints with following steps:

call get_members for all pd endpoints
Use the leader information in the first GetMembersResponse to connect leader

By default, the retry time is unlimited. Sometimes the response may not contain the leader field, then TiKV will be hung with the failure to connect leader.

there are three members, named pd0,pd1, pd2. and the pd0 is leader, pd1 is unhealth, which means pd1 may not know the leader. the TiKV start with endpoints="pd0,pd1,pd2", which may cause cannot connect to PD forever because cannot know the leader of pd members.

there are three members, named pd0,pd1, pd2. and the pd0 is leader, pd1 is unhealth, which means pd1 may not know the leader. the TiKV start with endpoints="pd0,pd1,pd2", which may cause cannot connect to PD forever because cannot know the leader of pd members.

/assign

/assign


pyarrow.lib.ArrowMemoryError: realloc of size<PHONE_NUMBER> failed

Hi,when I use the img2dataset to download the dataset ,it raises the error.
Traceback (most recent call last):
File "/hdd/u202320081001041/CapsFusion/download_file.py", line 22, in
number_sample_per_shard=1000,
File "/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/img2dataset/main.py", line 267, in download
max_shard_retry,
File "/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/img2dataset/distributor.py", line 36, in multiprocessing_distributor
failed_shards = run(reader)
File "/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/img2dataset/distributor.py", line 31, in run
for status, row in tqdm(process_pool.imap_unordered(downloader, gen)):
File "/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/tqdm/std.py", line 1181, in iter
for obj in iterable:
File "/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/multiprocessing/pool.py", line 748, in next
raise value
pyarrow.lib.ArrowMemoryError: realloc of size<PHONE_NUMBER> failed
Could you tell me where the problem is?

Hi, it's saying you don't have 2GB of free memory
On Thu, Dec 19, 2024, 06:28 FFFFFhh @.***> wrote:

Hi,when I use the img2dataset to download the dataset ,it raises the error.
Traceback (most recent call last):
File "/hdd/u202320081001041/CapsFusion/download_file.py", line 22, in
number_sample_per_shard=1000,
File
"/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/img2dataset/main.py",
line 267, in download
max_shard_retry,
File
"/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/img2dataset/distributor.py",
line 36, in multiprocessing_distributor
failed_shards = run(reader)
File
"/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/img2dataset/distributor.py",
line 31, in run
for status, row in tqdm(process_pool.imap_unordered(downloader, gen)):
File
"/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/site-packages/tqdm/std.py",
line 1181, in iter
for obj in iterable:
File
"/home/u202320081001041/miniconda3/envs/XVLM/lib/python3.7/multiprocessing/pool.py",
line 748, in next
raise value
pyarrow.lib.ArrowMemoryError: realloc of size<PHONE_NUMBER> failed
Could you tell me where the problem is?
—
Reply to this email directly, view it on GitHub
https://github.com/rom1504/img2dataset/issues/443, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437S2KBZDYJCJ2UH7MPT2GJKPBAVCNFSM6AAAAABT4FISFCVHI2DSMVQWIX3LMV43ASLTON2WKOZSG42DSMRZG4YTKOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Update cluster roles for addon-controller

lease role in particular causes issues with OCP 4.11 if
not present

Fixes https://github.com/stolostron/backlog/issues/24115
Signed-off-by: Tesshu Flower<EMAIL_ADDRESS>

/honk


add support for changing typeaheadMinLength

One might want to change minLength no of characters condition based.
With the old implementation minLength is not changed when typeaheadMinLength is changed from outside, now with a watch it does change.

This needs tests and a docs update.

Test was added.
I think the documentation is already ok.

typeahead-min-length $ (Default: 1) - Minimal no of characters that needs to be entered before typeahead kicks-in. Must be greater than or equal to 0.


"native" attribute fails TypeScript type check

When I follow the doc to use Foo to let the browser perform routing on its own, the code generates an error during TypeScript compilation: Property 'native' does not exist on type 'HTMLAttributes<HTMLAnchorElement>'.

@kmxz use data-native instead.

@kmxz use data-native instead.

From what I see in https://github.com/preactjs/preact-router/blob/61671b38fd5e371e886a6b33db2c66df7a6444db/src/index.js#L116, only the attribute native is being checked. data-native won't work.

#369

@kmxz
I'm currently working around the issue by extending the PreactDOMAttributes interface. I created a preact.d.ts file (doesn't matter where) with the following:
import "preact";

declare module "preact" {
  interface PreactDOMAttributes {
    native?: boolean;
  }
}

Not the best solution, but moves the problem out of the way until the built-in types are updated.

@martinrue What about submitting a PR for that here? We already have an index.d.ts file that specifies the types for preact-router in this repo.

@martinrue What about submitting a PR for that here? We already have an index.d.ts file that specifies the types for preact-router in this repo.

Just sharing the workaround I'm using currently, but not sure it's the correct solution beyond a quick fix. For example, there's currently a PR to change the prop to another name to avoid the non-standard issue all together.

@martinrue ,
I've tested your solution, but something should missing , only dts. file you provided is not enough.
I'm still getting same error.
is there anything else should be done to make this work ?
tsconfig , etc ...

My workaround before #369 is merged is to use the spread syntax:
foo


Weird join and group exception

I have two tables in sql server. When i join and group them like this
from d in db.testq
join sk in db.test2 on d.test_id equals sk.test_id
where sk.kid == token.getID()
group d by d.test_id into a
select new { id = a.First().d.test_id });

I got the following Exception
System.InvalidCastException
Unable to cast object of type 'Microsoft.Data.Entity.Query.QuerySourceScope`1[<>f__AnonymousType10`2[service.Model.test1,service.Model.test2]]' to type 'Microsoft.Data.Entity.Query.QuerySourceScope`1[service.Model.test1]'.

I can't figure it out, is this implemented in EF7?

@buraktamturk I'm having hard time reproducing this issue. Do you still have the sample project? If so, could you share some more details about it? I'm mostly interested in the shape of the model. Ideally, I would love to see the entire repro code, but if the project is too large or you don't want/can't share it for any reason, the model should be sufficient for me to go ahead with the investigation.

I would like to post it but i converted entire project to EF6. But the models just having two integers and one string. (I specially made for this case)
It might be fixed with never version of EF7. I was using beta1 while trying it. I'll be in work in two days, i'll happily share the result. You could try reproducing this case in beta1 version. If it's not working in beta1 but working at now. You could close this issue. Thanks!

Thanks, will try Beta1. Did you seed the database with any particular data?

I was able to reproduce this in Beta1 and it seems to be fixed in the latest bits, closing


Move to Github Apps for Submitting License Requests

Currently, we use Github OAuth Apps to create issues in license-list and license-namespace repository. But Oauth apps have limited no of fixed permission with no way to modify read/write access. https://developer.github.com/apps/building-oauth-apps/understanding-scopes-for-oauth-apps/#available-scopes
Github Apps provide finer-grained permissions - https://developer.github.com/apps/differences-between-apps/
We would like to move to Github Apps. For this, I suppose, changes will be needed only on the code side (utils.py) and requesting only the permissions which is needed - able to create issues in the https://github.com/spdx/license-list/ and https://github.com/spdx/license-namespace/ repo.
Linked Issue - #192

@rtgdk - is this still live?

Because the #192 was closed, I re-reported it as #498. The all repos write permissions are potential security problem.


BUILDING.md instructions are incorrect on Windows

https://github.com/webmixedreality/exokit/blob/master/BUILDING.md#windows
Exokit's BUILDING.md says that users can install windows-build-tools as follows:
npm install -g windows-build-tools

However, the official instructions are:

Start PowerShell as Administrator and run npm install -g --production windows-build-tools

Unless there's a solid reason to deviate from the official instructions, it's best to stick with the officially-supported installation method.
Also note that the formatting is broken:

Hm, what problem will this difference cause?

The official build instructions say to run the command inside Powershell. Unless we've verified that Powershell is optional, the difference might cause an installation error.


If a property value is a String keep it as one when converting to JSON (...

...even if the value is convertible to numeric)

We'll address this more fully in M2.


Updated changelog for 540 master

Codecov Report

Merging #545 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #545      +/-   ##
==========================================
+ Coverage   84.74%   84.74%   +<.01%
==========================================
  Files          41       41
  Lines        6366     6367       +1
==========================================
+ Hits         5395     5396       +1
  Misses        971      971


[Question/Suggestion] Entire Tile reveals when only a small portion visible

Hi I'm guessing what is seen in the pic below is a bug but thought I'd ask 1st if it is intended behavior?
If it is intended, I'd like to request/suggest that only what is in the token's line of sight be visible. In other words, basically like real life, if I see a tiny bit of my neighbors house through the window, I wouldn't know the entire shape of their home.
If this is intended for some purpose I just haven't figured out yet, maybe the option to toggle view styles would be nice.
Otherwise, great mod, thanks for all the work on this and others recently!

Foundry 0.8.8; BR <IP_ADDRESS>; Win10; Tested with Electron app and Chrome
Tile Setting:

Entire 2nd roof is visible when only a small sliver should be able to be seen. Tile correctly goes back to unseen what token moves out of window line of sight:

Not really a bug, it’s just how the system behaves, adding additional masks to have it work how you are asking would not only have performance impact but also has some technical issues ok with masks on masks that are not easy to solve, if I ever find a solution I’ll consider adding it

OK so the system is treating the tile differently than others because of the way the applied mask works. I see what you're saying. I could see that definitely leading to some issues if each one in view has to calculate. Especially if in view of several roofs. Makes sense and thanks for considering it for the future if you can get a fix.

The problem is that the los does not intersect with the roof so I cannot use it as a mask. If you enable the foundry sight rays debug you’ll see what I mean


Do not submit; test unpinning python validates container from beam1

Do not submit; test unpinning python validates container from beam1

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Seed Job

Run Python Dataflow ValidatesContainer

Run Seed Job

Run Python Dataflow ValidatesContainer


Volume not persistent

If I set the volume to 50%, quit the application and open it again, the volume is again at maximum. Using the latest stable version on macOS Sierra 10.12.1

I’m pretty clueless, but do know that Google Music stores the volume in the cookie sjpref. What does strings "$HOME/Library/Application Support/Radiant Player/Cookies" | grep sjpref return? Should be something like VsjprefVv:100|#A where 100 is the current volume.

When I reported this issue, this options wasn't selected:

I ran the command while the application was running: VsjprefUv:90|#A
I closed the application and ran the command again: VsjprefUv:90|
Lowered the volume a bit.
I ran the application again: VsjprefVv:100|

Then I enabled that option:

Lowered the volume a bit, ran the command again, while the application was still open: VsjprefUv:75|#A
Closed the application, ran again: VsjprefUv:75|#A
Ran the application. VsjprefUv:75|#A, but volume was still at 100%.
Set the volume to about 50%, closed the application: VsjprefUv:75|#A
Ran the application again. VsjprefUv:75|#A, but the volume stayed at 50%.

Seems to be working now, but the cookie is still VsjprefUv:75|#A.

Oddly enough I’m now encountering the issue exactly as you describe. I don’t recall changing anything about my setup that would trigger this.

I've run into this issue today as well. Haven't changed anything as far as I know.

I constantly run into this issue as well.  Would be great if it could be fixed.


Wayland support

Currently we only support X11, with the fast adoption of wayland we need to provide native support

Judging from the ashpd demo, you can use the Remote Desktop portal to request control access, and Screen Cast to get the cursor metadata. I hope this simplifies it for you a bit.

Coming back to this, I'd like to suggest a Wayland protocol be added upstream that allows apps to both view and emulate input, similar to Android, and it would go through xdg-desktop-portal to obtain permission. How does this sound?


debug(KONFLUX-3482): Show durations in buildah-remote

Just for record, this is how I created mine own bundle with the task:
podman login -u='rhcloudperfscale+jhutar_tekton_bundle_buildah_remote' -p='...' quay.io
tkn bundle push quay.io/rhcloudperfscale/jhutar-tekton-bundle-buildah-remote:1.4 -f /home/jhutar/Checkouts/build-definitions/task/buildah-remote/0.1/buildah-remote.yaml

and then in tekton yaml:
    - name: build-container-arm64
      params:
        - [...]
      runAfter:
        - prefetch-dependencies-arm64
      taskRef:
        params:
          - name: name
            value: buildah-remote
          - name: bundle
            value: quay.io/rhcloudperfscale/jhutar-tekton-bundle-buildah-remote:1.4
          - name: kind
            value: task
        resolver: bundles
      when:
        - input: $(tasks.init.results.build)
          operator: in
          values:
            - "true"
      workspaces:
        - name: source
          workspace: workspace-arm64

Given I can build task bundle with modified task myself, I do not need this PR.


[BUG] C# setter for property named value prevents assignment.

Describe the bug.
A C# setter has a "magic" variable called value which contains the value being assigned.
If a model has a property called Value, then the generated setter is broken, as it will attempt to assign the magic variable, rather than the backing class field.
public partial class Duration
{
    private readonly decimal value;

    public decimal Value
    {
        get { return value; }
        // This setter is broken - it is assigning the setter-level value, not the class-level value.
        set { value = value; }
    }
}

### Expected behavior

The setter could be generated as `this.value = value`;

```c#
public decimal Value
{
    get { return value; }
    set { this.value = value; }
}

Screenshots
N/A
How to Reproduce

I opened the online playground.
I changed one of the field names to value.
I noted the C# setter was generated incorrectly, as described above.

🥦 Browser
Google Chrome
👀 Have you checked for similar open issues?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

/gfi typescript

Makes sense 👍


Add whisper 1 support

Adds whisper-1 support as outlined in the API docs for transcription and translation https://platform.openai.com/docs/api-reference/audio
I have tried my best to use a similar style to the rest of the repo, but happy to tidy things up where necessary.
Demonstration
https://user-images.githubusercontent.com/4222666/222873861-61137977-dd24-45ba-b38d-950e8d240545.mp4

Awesome, thank you for the PR!


[update] CONTRIBUTING.md

The information in CONTRIBUTING.md so far was very old, so I updated them.
Would you like to see these changes?

Hm, not sure. I'd like to have a simple vimrc example, on how to just run vim with vim-airline included, so one can easily reproduce issues. Perhaps we need to have this included in the root dir?

I think it's easier to run vim with vim-plug.
e.g
scriptencoding utf-8

call plug#begin('~/.vim/plugged')
Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'
call plug#end()

What do you think? @chrisbra

okay, thanks.


FR: Tag images in multiple projects simultaneously

Hello. The quantity of projects are growing and dedicated tag step for each project - it's not convenient. 6 projects and 20 different builds = 120 tag steps (all imagestream in all projects, except central image project).

@livelace are you thinking

1 src tag and many dst tags
N src tag and P dest tags, where N == P
both 1) and 2)
?

@bparees FYI

@gabemontero

It will be great.
It's the very rare case, because we use metajobs, which launch one specific build for different software == 1 build and 1 imagestream as result.
It will be nice if we will be have both possibilities: src 1+n, dst 1+n

Hello. Any news ? :)

@gabemontero is this reflected on one of the "jenkins enhancement" trello cards?  if not, let's get it there (or create a new one) and close this.

hey @livelace and @bparees - yeah originally I thought I might be able to squeeze this one in between official assignments but that proved to not be the case, and will remain as such given what's currently on my plate.
I'll review the existing trello cards first, see if it makes sense to add this to those cards.  Otherwise, I'll create a new one.
I'll update this issue in a bit with the trello card in question and close this out.

OK, @bparees and I agree we can add this to https://trello.com/c/Yl4qomsw/878-5-additional-jenkins-plugin-enhancement-requests-from-github, which is already on our Next list, and so is pretty close to getting scheduled.

@livelace - an update.  With commit https://github.com/openshift/jenkins-plugin/commit/1da103810da471618db2122583d88abf2efd40db from @jupierce the plugin is now enabling the following forms:

1 destination stream and 1 destination tag (i.e. single tag applies to single stream)
1 destination stream and N destination tags (i.e. all tags apply to the same stream)
1 destination tag and N destination streams (i.e. all streams will get the same tag)
N destination streams and N destination tags (i.e. each index will be combined to form a unique stream:tag combination)

Cool things. Thanks guys! :)

@gabemontero @jupierce
Hello. I did update to the version 1.0.23 and don't see anything in the build step for tagging in multiple projects. This FR realized only in pipeline working mode ?

@livelace Clicking the help icon next to the destination stream & tag in Jenkins should now describe support for a comma delimited list in each field.
A description of how the lists are handled is given in the pipeline section of the README.MD (https://github.com/openshift/jenkins-plugin ; search for "Tag OpenShift Image") -- the pipeline DSL supports an identical comma delimited list for the fields.

@gabemontero just talked to me offline and pointed out that you specifically mentioned "projects" . You are correct that there is currently no support for multiple src/dest projects in the build step (or pipeline DSL for that matter). Only one source project and one dest project is supported in each step.
Based on the issue conversation, it seemed that enabling 1->N, N->1, N->N  (destStream->destTag)   would be sufficient. Please let us know if I've missed the mark.

Oh no. I meant projects and multiple tags between them. I thought we discussed all relations in context of projects. Because we have a lot of projects and one contain images, which must be available for others.

@livelace You might be able to accommodate your project topology using the new feature and a pipeline DSL step/looping constructs, but if not, can you open another issue to enable multiple destination projects in the same tagging step?
The necessary change will require us to go back to the drawing board for a bit in order to ensure backwards compatibility with prior versions of the plugin.

@jupierce  We can't move to using the pipeline right now. Maybe in near future. Ok, we will be use multiple steps.  Thanks.


Add mkWrapper example

Alternative to https://github.com/MatrixAI/Haskell-Demo/pull/23
Instead of baking nix store path into the binary, use the wrapProgram utility in postFix stage to put the nix store path onto application's PATH.
This requires moving back to using cabal.nix as the auto generated expression from cabal2nix and override the drv in default.nix to add the postFix.

Related to #7


Bug fixes and error handling in Redis - Vectorstore

Bug fixes in Redis - Vectorstore (Added the version of redis to the error message and removed the cls argument from a classmethod)

Fixes  #3893 #4896
Who can review?
@dev2049 @tylerhutcherson

Hey -- What is this fixing exactly? We need this redis module check here otherwise it will fail for folks running vanilla redis

The confusion might be that there's Redis modules... RediSearch and RedisJSON for example (part of Redis Stack). These are what we use the code above to check for existence...
And then there's Python client libraries. The standard redis python library is all we need for langchain.

I have reverted the previous code deletion and refined the error message @tylerhutcherson

@nikhilkandur Please do pip uninstall redisearch and  do pip install redis==4.5.5
that'll solve the issue.

Sorry about all the chaos. I'll revert the changes done to this error message. I'll just probably add the version of redis (>=4.1.0) that has to be installed by the user, in order to be compatible with langchain here. Would that be fine?

Perfect. I think that will help a ton.

I installed redis==4.5.5 but i'm getting below error but that version of redisearch doesn't exists.
ValueError: Redis failed to connect: You must add the RediSearch (>= 2.4) module from Redis Stack. Please refer to Redis Stack docs: https://redis.io/docs/stack/

@nikhilkandur Here's what:
The Redisearch standalone project is discontinued. We won't need theredisearch python package to be installed as it is already available bundled within the standard redis>=4.1.0 package.
So, the error you get right now despite what the message suggests has nothing to do with redisearch (you can refer the previous conversations as well). I doubt it might have something to do with the redis connection itself. Please check on that front.

Exactly. The error you have @nikhilkandur is because you need to run the Redis Stack docker container (or sign up for Redis Cloud) that bundles RediSearch functionality into the database. The Python client side is different. redis>=4.1.0 is sufficient.

@nikhilkandur Here's what:
The Redisearch standalone project is discontinued. We won't need theredisearch python package to be installed as it is already available bundled within the standard redis>=4.1.0 package.
So, the error you get right now despite what the message suggests has nothing to do with redisearch (you can refer the previous conversations as well). I doubt it might have something to do with the redis connection itself. Please check on that front.

Exactly. The error you have @nikhilkandur is because you need to run the Redis Stack docker container (or sign up for Redis Cloud) that bundles RediSearch functionality into the database. The Python client side is different. redis>=4.1.0 is sufficient.

Currently, I'm making use of redis addon for heroku and using redis>=4.1.0 at the client side to make the vector store work. Isn't it sufficient ?

@nikhilkandur Here's what:
The Redisearch standalone project is discontinued. We won't need theredisearch python package to be installed as it is already available bundled within the standard redis>=4.1.0 package.
So, the error you get right now despite what the message suggests has nothing to do with redisearch (you can refer the previous conversations as well). I doubt it might have something to do with the redis connection itself. Please check on that front.

Exactly. The error you have @nikhilkandur is because you need to run the Redis Stack docker container (or sign up for Redis Cloud) that bundles RediSearch functionality into the database. The Python client side is different. redis>=4.1.0 is sufficient.

Currently, I'm making use of redis addon for heroku and using redis>=4.1.0 at the client side to make the vector store work. Isn't it sufficient ?

Sounds like Heroku's redis implementation does not include any of the add-on modules like RediSearch, RedisJSON, RedisGraph, etc. But to use Redis as a vector database, you need the RediSearch module...

There seems like another bug, not sure if it is fixed recently in unreleased version. I'm seeing kwargs field inside kwargs. You can see it here while instantiation
  File ".../site-packages/redis/connection.py", line 956, in __init__
    super().__init__(**kwargs)
TypeError: AbstractConnection.__init__() got an unexpected keyword argument 'kwargs'

There seems like another bug, not sure if it is fixed recently in unreleased version. I'm seeing kwargs field inside kwargs. You can see it here while instantiation
  File ".../site-packages/redis/connection.py", line 956, in __init__
    super().__init__(**kwargs)
TypeError: AbstractConnection.__init__() got an unexpected keyword argument 'kwargs'

yup fixing this

@iamadhee Can you change the name of this PR to be more specific to this fix and then will get this prioritized as well. Thanks!

@iamadhee Can you change the name of this PR to be more specific to this fix and then will get this prioritized as well. Thanks!

Have added the new title and description

@tylerhutcherson updated the redisearch error message to avoid further confusions

Is there anything else that is needed from my end on this PR? @tylerhutcherson

Made the suggested changes cc @tylerhutcherson @dev2049

thanks @iamadhee!


Support for Metadata Projection

It would be great if we can select specific fields to be queried rather than ship the entire metadata column over the wire. This is helpful if there is a lot of metadata.

I can see the benefit for projects that make heavy use of metadata but this would be a big increase in scope as we'd have to introduce our own language for projections and translate it into Postgres compatible json operators.
Overall I don't think its enough of a problem to justify the additional complexity we'd have to introduce so I'm going to close this for now


Which config file should we use?

Distro: Arch Linux
The wiki has this config file which I copied and edited: https://github.com/dylanaraps/neofetch/wiki/Config-File
I decided to check what would happen if I ran without a config file, and it generates one. But that one is very different from the one in the wiki:
https://gist.githubusercontent.com/tonijarjour/6f72ab9a4679272d1ba57c76f979115f/raw/7de9f2ae9d3b193ea18ac4dd0e966ce0f1160ae2/config.conf
And this begs the question, which one are we supposed to use? Is it a problem with the distro package having an old file, or is the wiki the old one?
Also the default config that's supposed to be in /etc/neofetch/config isn't there, nor is it inside /usr/share or /usr/share/doc. There is no neofetch folder at all.

Neofetch will create the config file in ~/.config/neofetch/neofetch.conf on first run. It respects the $XDG_ variables so the location will differ based on your configuration.
The Wiki config file is outdated as I forgot to update it with the latest release. The /etc directory is up to you or your distribution to create.

You could also do neofetch --print_config > neoconf.conf then if you want to update your local config with the latest one you use a diff tool to see what changed.

Can you please clarify where the default config is coming from if /etc/neofetch doesn't exist?
I copied my config to /etc/neofetch/config (and also tried /etc/neofetch/config.conf) hoping that would cause it to apply to any users who didn't yet have a config, but it doesn't.

It comes from the script itself. But I don't know how to apply config for all users, which path it should be in.


How Kylin Support self join query or complex sub query

Hi All!
rt.
1  I have a self  join query like
select count(*) from
(select t2.brand from
(select brand,count(1)  from DEMO
group by brand having count(1) >=2)t1
join
(select brand from DEMO )t2
on t1.brand = t2.brand)t3;

The ERROR
null while executing SQL: "select * from (select t2.brand from (select brand,count(1) from DEMO_FOR_TCL group by brand having count(1) >=2) t1 join (select brand from DEMO_FOR_TCL ) t2 on t1.brand = t2.brand group by t2.brand) t3 LIMIT 50000"

2  sub query like
select * from DEMO a
where a.brand in (select t1.brand from (select brand,count(1) from DEMO group by brand having count(1) >=1) t1);

ERROR LOG
[http-bio-7070-exec-243]:[2014-12-22 16:11:46,095][ERROR][com.kylinolap.rest.controller.QueryController.doQuery(QueryController.java:224)] - Exception when execute sql
java.sql.SQLException: error while executing SQL "select * from
(select t2.brand from (select brand,count(1) from DEMO group by brand having count(1) >=2) t1
 join
 (select brand from DEMO ) t2
 on t1.brand = t2.brand group by t2.brand) t3  LIMIT 50000": null
    at net.hydromatic.avatica.Helper.createException(Helper.java:39)

the query like self join or complex sub query  need  special cube setting, or i missed sth?
Thanks.

did you solve this problem? @Yancey1989 @lukehan

hi this repo is deprecated, pls visit kylin.apache.org to try latest kylin


Upgrade to Functions SDK 1.0.1

Breaking changes
## 1.0.0-dev.4.0

- Breaking: Upgraded to Functions SDK 1.0.1 and Admin SDK 5.12.0
  Official migration guide is located here: https://firebase.google.com/docs/functions/beta-v1-diff
  Changes in this library are identical and slightly adapted to Dart
  semantics:
  * `admin.config().firebase` field has been removed
  * Background functions (that is everything except HTTPS) now
    expect two arguments `data` and `context` instead of single `event`
    argument. For details see migration guide or updated examples in
    `example/` folder.

See official migration guide for more details: https://firebase.google.com/docs/functions/beta-v1-diff
To upgrade make sure to adjust your package.json to following constraints:
{
  "dependencies": {
    "firebase-admin": "5.12.0",
    "firebase-functions": "1.0.1",
    "@google-cloud/firestore": "0.13.1"
  }
}

Fixes #21 .

Please!

Sorry for the delay here. Will try to get this on Pub tonight.

Released as https://pub.dartlang.org/packages/firebase_functions_interop/versions/1.0.0-dev.4.0

:+1:


Laraflat Commands show just 3 options

Laraflat Commands just show 3 options
1 - form builder
2- add comment to module
3 - add rate to module
but another commands like laraflat:admin_model not show

soory laraflat:admin_model is form builder

laraflat:admin_model is  form builder
you can access to all command from php artisan


Taxonomies->Terms List Issue,case by TermsComparer's Compare method

When I create a child [cA ]term under A and [cA] show under B node as B's child in the terms list view.
I find this is case by the 'Compare' method in the 'TermPart' class. attached:

Say we have term with fullpath: /24 and other one's fullPath is /241/247, so u know it, bang!
Then I add a new property 'FullPathEnd' in TermPart Class to fix this, also attached:

Well, there could be more effective solution,its do the trick.

@jersiovic what do you think?

It looks the compare method is using FullPath instead of FullWeight the new property @jardg added.
We didn't changed this method because it is not used anywhere so we marked it as obsolete.
Is it posible the terms view you mention is a custom one in your theme?
We can fix Compare to work properly and remove the obsolete annottation.
Independtly of that we are going to send a fix for Compare method to be based on FullWeight.

Well it is not so easy because current Compare method is used by the Orchard.Upgrade module when it calls TermPart.Sort() for the migration of terms from previous versions, and for that it is needed as it is.
I imagine at some point you are using TermPart.Sort() method in your view instead of simply get terms ordered per FullWeight using TaxonomyService.GetTerms(). My advice is replace one per the other.
What we can do in Orchard for avoiding confusions like that is to mark also TermPart.Sort() as obsolete, but maybe it is not enough. Maybe we should rename obsolete methods to TermPart.ObsoleteSort() and ObsoleteTermsComparer() to allow an early detection of obsolete calls in custom code at compilation time.

@wthinkit  Looks like you could be using Sort method on the retrieved terms when generating a custom Terms View?. Sort actually should be Obsolete as @jersiovic says. Also could you specify the repro steps?. Did you have an existing taxonomy in which you added the new terms or you tested it creating a new taxonomy?. In case you had an existing taxonomy, did you apply the migration for terms using the Upgrade module? so you dont have to use Sort because the terms can be retrieved from db and ordered by FullWeight.

Wow, the new way to order terms,after I pull the newest code and checkout dev branch, good job!

TermPart.Sort now yields a build warning as it's still using TermsComparer, which marked as Obsolete (but there's no recommendation what to use instead, btw). Can we change it like this?
public static IEnumerable<TermPart> Sort(IEnumerable<TermPart> terms) =>
    terms.OrderBy(term => term.FullWeight);

Btw we should add some documentation to TermPart, as int Weight vs. string FullWeight looks confusing to me.

I will work on it and submit a pr with this change and documentation about the Weight

Where should I provide the documentation? I didn't find any written documentation about terms or taxonomies other than a simple mention to the Orchard.Taxonomies module.

I think some in-line documentation is enough. What do you think about the Sort method?

Yes. Should be changed to the way you say.I didn't change it at first because I didn't want to force the users to upgrade but this way it's better than just order manually everywhere you want to sort the items.
I added a static SortObsolete method with the old code because TaxonomyController needs in Upgrade module to migrate terms created in an old version.


irrPlot HPC flag

irrPlot has an HPC flag that has not be propagated through analysis() or analysis1axis(). Check wha'ts going on.

Fixed in development


JavaScript: Fix characteristic predicate of XMLParent.

The database type @xmlparent is defined a bit too loosely in that it includes all of @file, not just XML files. This meant that non-XML files were instances of XMLParent but not of any of its subtypes, thereby inheriting the default none()-implementation of getName(), causing their toString() predicates to be empty.
Fixing the extent of @xmlparent would involve fiddling with the extractor/dbscheme, while fixing XMLParent can be done at the QL level, so that's what I've done instead.
Fixes #2517.

I'm going to do a very quick dist-compare to ensure this doesn't cause optimiser wobbles.

A smoke-test evaluation (internal link) showed no changes.
I tried out DPM for this run, which worked like a charm (:tada:) and shows no changes to either tuple counts or predicted timings. I take that as evidence that join orders didn't change at all, so unless there is significant appetite for a more thorough evaluation I think this is good to go.


Replace old player model?

Is there any documentation on how to replace animated playable characters?

You better post that question on the forum, this is not the correct place for discussions.


Revisit whether ConnTrack should be a configurable for an endpoint

Example:
If we want to allow identity B to have ingress traffic from identity A, then without connection tracking enabled, we have to allow ingress traffic to A from B as well so that A can receive replies from B - traffic must be allowed in both directions. As such, the structures within Cilium for policy have to know that identity B is allowed to receive traffic from identity A, and vice versa.
If ConnTrack is disabled for the endpoints, within the Consumables that apply to those endpoints, we have to find a way to specify which identities have been specified as being allowed to talk to each other. The current implementation in the Consumable structure is to have a map of security identities (ReverseRules) which specify which identities have specified that they want to receive traffic from that Consumable; the Consumable must know that it must allow traffic in the reverse direction so the connection can be maintained. However, this has a variety of issues associated with it:

We enable ConnTrack for an endpoint dependent upon the policy:

pkg/endpoint/policy.go:regeneratePolicy:
// Apply possible option changes before regenerating maps, as map regeneration
	// depends on the conntrack options
	if c.L4Policy != nil {
		if c.L4Policy.RequiresConntrack() {
			opts[OptionConntrack] = optionEnabled
		}
	}

We thus change something that the user may have configured based on the policy. In my opinion, we should not be changing options that are configurable unless the user specifies them.

This does not work in a multinode setup

The only way that the ReverseRules map is updated is based off of the ConsumableCache, which maps identities to Consumables on the local host. However, if the cache does not contain an entry for the identity for which the reverse rule is being added, as is the case when both identities are on separate nodes in the system, then no such reverse rule is added. This will always be the case when ConnTrack is disabled for two endpoints that are on separate nodes, as the ConsumableCache is only updated with identities local to the host on which the cilium-agent is running.

This hasn't been working anyway upon changes in policy after the reverse rules have been updated:

in pkg/policy/consumer.go:BanConsumerLocked, which is called when an identity is no longer allowed ingress access for a Consumable, there is the following logic:
if consumer.Reverse != nil {
	c.deleteReverseRule(id, c.ID)
}

However, the Reverse field for a Consumer is never populated anywhere in the code. Thus, in the case where endpoints are running on both nodes with ConnTrack disabled, the deletion of an identity from the ReverseRules of a Consumable wasn't happening when it should.
I think this warrants us making this option immutable for an endpoint, or at least specifying clearly in our documentation in which cases this will work / not work (single node setup vs multinode setup).

We thus change something that the user may have configured based on the policy. In my opinion, we should not be changing options that are configurable unless the user specifies them.

That sounds like a pretty intuitive argument which I'm inclined to agree with. My guess would be that when this L4Policy check (and therefore L7 policy check) was introduced, the "disable CT" endpoint option was not considered. I believe that proxy_port is tightly coupled to CT in the datapath at the moment, so if we 'fixed' this to respect the user's wish to disable conntrack, it may end up breaking L7 policy application.

After discussion with @tgraf - ok to rip the ReverseRules out of the Consumable. I checked at what points in the ConsumableCache where identities are added to the Cache:
The only time the cache is added to is in this function:
pkg/policy/cache.go:
func (c *ConsumableCache) GetOrCreate(id NumericIdentity, lbls *Identity) *Consumable {
	c.cacheMU.Lock()
	defer c.cacheMU.Unlock()
	if cons, ok := c.cache[id]; ok {
		return cons
	}

	c.cache[id] = NewConsumable(id, lbls, c)
	return c.cache[id]
}

And this function is only called twice in Cilium:

pkg/policy/cache.go:Init, in which all reserved identities are added to the Cache.
pkg/endpoint/policy.go:SetIdentity, which is only called upon an endpoint (that is, only for endpoints on the host)

Thus, the cache only contains identities of endpoints managed on the node itself, and thus this 'ReverseRules' model breaks down when running Cilium in a multinode setup. An alternative to fix this is to have all identities computed on every node and added to the ConsumableCache, but this is not scalable at all, as it requires synchronization with the key-value store, and because policy computation is quite costly.
Thus, we should remove ReverseRules and add documentation specifying that users must add explicit policy rules to allow traffic in both directions in the case that ConnTrack is disabled for an endpoint.

Unfortunately, after trying to remove ReverseRules, we cannot do it so easily. When ConnTrack is disabled in an endpoint's configuration, despite adding rules that allow communication in both directions (from the host --> endpoint and endpoint --> host), connectivity doesn't work because adding a rule that applies to reserved:host doesn't work the same as applying a rule to an endpoint with label id.server, in that internally, the host is not really an endpoint; there is a Consumable with identity 1 in the ConsumableCache, though. The only way the Consumables representing the reserved identities to be updated is when the Consumable associated with endpoint(s) is updated to allow these reserved identities. This is done by updating the ReverseRules of the reserved identity Consumables. Thus, removing this will take some work, as we need to allow reserved-identity --> endpoint communication in the case ConnTrack is disabled for an endpoint.


[BACKLOG-21741] Creating a data-lineage analyzer for the Mapping (sub-transformation) step

Minot changes needed to allow the Mapping Analyzer to be added within the pentaho-metaverse-core project

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/mapping/MappingMeta.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -3% :small_red_triangle_down:
Complexity Change: -3.2% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.7% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.9% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -0.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

@pentaho/r2d2

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/mapping/MappingMeta.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -3% :small_red_triangle_down:
Complexity Change: -3.2% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.7% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:


VS Code as a Progressive Web App

vscode.dev would benefit greatly from the ability to install as a progressive web app. Benefits include faster load times, smoother interface, and offline access.

While I can install this on my iPad as a full-screen PWA, it does have some issues, namely:

Lack of an icon (uses a screenshot of the webpage)
Signing in with GitHub redirects me to a full-screen page saying that I may now close this page. There is no “Done” or close button, so I have to close and reopen the app at that point.

The first problem can be solved with the
Unfortunately isn't apple open-source and PWA friendly because this is an alternative to there AppStore😟

@joaomoreno I just ran an update just in case. Still no go on Chrome 94.
Mozilla/5.0 (X11; CrOS x86_64 14150.64.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.104 Safari/537.36

That's because Edge don't need a service worker but Chrome.

+1. I'd really like to see this be available as a PWA and integrate with the available and coming browser apis. Being available as a PWA enables vscode to be discovered and used by many more people as it will be 'natively' available on tablets and (hyped!) chromebooks.
The service worker also lets you implement offline functionality, so the PWA version can be used offline. It can then also be installed from the playstore via a 'Trusted Web Activity' and the microsoft store. Eventually, with enough support by coming browser apis, it could feature-wise get close to the native vscode. (All from a single codebase!)
As for the technical side, fireship has made a great starter video:
(You can use workbox for most caching/offline functions and pwa-asset-generator for the images)

To get a basic PWA working you'd have to:

Add a service worker javascript file.
Add a script that registers a service worker in your index html file:

   <script>
      if ('serviceWorker' in navigator) {
        navigator.serviceWorker.register('/path/to/service-worker.js').then(function(registration) {
          console.debug('ServiceWorker registration successful with scope: ', registration.scope);
        }).catch(function(err) {
          console.warn('ServiceWorker registration failed: ', err);
        });
      } else {
        console.warn('Service workers are not supported.');
      }
   </script>

Now with only these changes you should get the small 'install' button. But there are of course many more nicities of PWAs.

Generate all icons and splash images. run:
npx pwa-asset-generator starting-icon.svg icons-dir/ -i index-file.html -m manifest-file.json
This fetches the latest apple design specifications for icons and generates all needed icons and splash images (even the
maskable ones!) from the start-image
Implement caching using workbox. For this, you can import workbox from a cdn and create simple caching rules using it's easy-to-use api. I'd recommend to cache the main 'vscode.dev' site and all relevant javascript, wasm, image, and font data. The user's files are stored in local IndexedDB anyways.
This is what I did to get basic caching for images working using the CacheFirst strategy (everything is loaded from cache if there's a cache hit, else it's loaded over network):

    importScripts(
      "https://storage.googleapis.com/workbox-cdn/releases/6.2.0/workbox-sw.js"
    )

    workbox.routing.registerRoute(
        ({ request }) => request.destination === 'image',
        new workbox.strategies.CacheFirst()
    )

Populate the manifest with more info. There are many more supported keys that add ways to specify screenshots for PWA-stores, a background color for the loading screen, protocol-handlers, shortcuts, navigation scopes, and more.

I think i'm actually going to add that myself and create a pr if that's possible. @joaomoreno how do i add my service-worker.js file to the build script?

I think i'm actually going to add that myself and create a pr if that's possible. @joaomoreno how do i add my service-worker.js file to the build script?

We're not accepting PRs for vscode.dev. We are currently internally working on PWA for vscode.dev.

But vscode.dev is built from this repo, right? I mean, i can build 'vscode.dev' by running yarn web. So changes to this codebase would reflect in vscode.dev

@joaomoreno you can check my post above, should be a solid starting point

@joaomoreno if any help on getting started with / configuring / customizing / extending Workbox needed - I'm here to help!

I mean the App ist probably built using node, so you can Just import workbox and webpacks treeshaking will take care of the minifying

This is now pushed and deployed.


Updated operatorhub.sh to use the new CSV location

Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>

The new script was used to submit the PRs for 1.19.0 already.


Add GitHub actions

See #59  for details.
This adds automated workflows to

build and run unit tests of both the core library and the demo on any push to any branch
on merge to main:

bump the version in the version file according to the version string in the commit messages (#major, #minor, #patch, #none). #minor by default.
create a new release and add debian packages as well as tar.gz and zip release
run the unit tests against the released packages
build and push new docker images for the core library, the demo and the tutorial

Alright @orzechow , this one is ready for review. It should be very analogous to the util_caching pipelines now. Feel free to push your release-test-pipeline onto this branch if we want to do that in one go.

Alright @orzechow , this one is ready for review. It should be very analogous to the util_caching pipelines now. Feel free to push your release-test-pipeline onto this branch if we want to do that in one go.

Thanks!
One quick remark: the dockerfile is missing the release_builder stage etc.
Yeah, I'll add the release test pipeline after finishing the cpack PR

You're right. I think it's easier if one person just does it. I have moved the PR back to draft to wait until you finished the cpack PR.
I will then rebase this PR, add the Docker stage and release test pipeline and ping you once the PR is actually done.

Rebased on main

Ok @orzechow this one should truly be ready now.
See the edited PR description for all the things that are now going on the workflows.
I tested the workflow on a test branch where I didn't need a PR to trigger the workflow, see 82b31a0d2e551b35fb531e5b0fb21f45be565fc6. Other than that, the branch is identical to this one and the last run was succesful.
A couple of noteworthy changes I made since we last talked about this:

Combine the release and the docker build workflows. This is due to the fact that we need the version bump to happen first. Otherwise the docker images will be tagged with the old version. Also I guess it's nice to wait for the release tests before publishing new images.
The unit-test workflow now build the library core image before building the demo unit test stage. This is due to the fact that we want to test the state that the repo is in. Otherwise, the demo unit tests would run against the version that's currently hosted on ghcr (which doesn't even necessarily exist).


Change the name of isExpired method

Just a tiny thing, but is really confusing. The method to check is the JWT is expired is called "isExpired", but it returns true when the JWT is valid and false when is not. That is somewhat confusing, isExpired should return true when it is expired, or the method should be called isNotExpired so the semanting is correct.

Yes this was very confusing for me as well.

The code comment says this
 /// Tells whether a token is expired.
  ///
  /// Returns true if the token is valid, false if it is expired.
  ///
  /// Throws [FormatException] if parameter is not a valid JWT token.
  static bool isExpired(String token) {
    final expirationDate = getExpirationDate(token);
    // If the current date is after the expiration date, the token is already expired
    return DateTime.now().isAfter(expirationDate);
  }

But I think this is not true. This function will return true if the token is invalid and false if it is valid.


[rosidl] Ensure *_defs data is added to *_py and *_cc targets

Towards Anzu bugfix

This change is

Bazel Drake-ROS Continuous Integration tests pass locally for me.

bazel_ros2_rules/ros2/rosidl.bzl line 402 at r1 (raw file):

        interfaces,
        includes = [],
        data = [],

nit Can I ask you to add a corresponding doc string to the data argument?
Ditto for all the interface changes.

Previously, EricCousineau-TRI (Eric Cousineau) wrote…
The AMENT_INDEX_PATH environment variable has ~800 duplicate entries for rosidl_generate_ament_index_entry
These should be deduplicated

:(

Previously, EricCousineau-TRI (Eric Cousineau) wrote…
Done.

:)

Build drake_ros in CI failed with the following error:
ERROR: /home/runner/.cache/bazel/\_bazel\_runner/d07b19e4bc14c7eead45599d9bb941ba/external/vtk/BUILD.bazel:31:11: Middleman \_middlemen/@vtk\_S\_S\_Cvtkkwiml-cc\_library-compile failed: missing input file '@vtk//:include/vtk-9.1/vtk\_kwiml.h'

[136](https://github.com/RobotLocomotion/drake-ros/actions/runs/6661541550/job/18104550754#step:13:137)ERROR: /home/runner/.cache/bazel/\_bazel\_runner/d07b19e4bc14c7eead45599d9bb941ba/external/vtk/BUILD.bazel:31:11: Middleman \_middlemen/@vtk\_S\_S\_Cvtkkwiml-cc\_library-compile failed: missing input file '@vtk//:include/vtk-9.1/vtkkwiml/abi.h'

[137](https://github.com/RobotLocomotion/drake-ros/actions/runs/6661541550/job/18104550754#step:13:138)ERROR: /home/runner/.cache/bazel/\_bazel\_runner/d07b19e4bc14c7eead45599d9bb941ba/external/vtk/BUILD.bazel:31:11: Middleman \_middlemen/@vtk\_S\_S\_Cvtkkwiml-cc\_library-compile failed: missing input file '@vtk//:include/vtk-9.1/vtkkwiml/int.h'

[138](https://github.com/RobotLocomotion/drake-ros/actions/runs/6661541550/job/18104550754#step:13:139)ERROR: /home/runner/.cache/bazel/\_bazel\_runner/d07b19e4bc14c7eead45599d9bb941ba/external/vtk/BUILD.bazel:31:11: Middleman \_middlemen/@vtk\_S\_S\_Cvtkkwiml-cc\_library-compile failed: 3 input file(s) do not exist

Ah, nuts. For now, I'll merge this PR (since testing passed locally) and fix the vtk issue in a separate PR.


Filter biobliography results by publication date

solves https://github.com/opentargets/issues/issues/2578
include date filter for bibliography
update query to handle start and end date

I was having a look and the netlify preview does not seem to get along the API
https://deploy-preview-19--ot-platform.netlify.app/
Perhaps is due to some recent changes in the API?


Tabs to use ng-if over ng-class to delegate active state

Right now a .tab-pane is considered active by providing the .active class to toggle between the visibility of the tab. This means that all the tabs that are out of context to the user still remain in context to the system.
If, instead of ng-class="{active: tab.active}" we had ng-if="tab.active" then we could have the tab opt-in & opt-out of the digest cycle when it's active or inactive.
A couple issues with the ng-class approach, other than leaving inactive tabs wired into context, is that when a directive becomes linked, it has no access to accessing the size of elements because it's currently hidden.
For example, if we have a directive inside the tab accessing element.width(), we'll get an inaccurate value when calling this from the linking phase because it's accessed before the tab's .active class has been applied.

The downside is the lack of backward compatibility, since an ng-if on the tab content will create an isolate scope. So I'd vote that we could opt-in to using an ng-if approach in some way. Something like
<tab use-active-class="false"></tab>

A simple way to reproduce would be to take a directive like so
function directive () {
  return {
    restrict: 'E',
    link: function (scope, element) {
       console.log(element.closest('.tab-pane').hasClass('active'));
    }
  }
}

And toggle between the tabs -- it won't be able to reach a true state

Closing as per comment in https://github.com/angular-ui/bootstrap/issues/2637#issuecomment-141088998


Update CSI Manifests

What this PR does / why we need it:
csi provisioner, attacher and driver-registrar v0.3.0 image last updated 3 months back, we should run csi CI against latest attacher, provisioner, driver-registrar.
In the CI scripts, we are building against latest but not testing with the latest build.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:
NONE.

/approve
/lgtm
/override openlab/cloud-provider-openstack-acceptance-test-flexvolume-cinder


Make AWS access keys optional for DBT metadata ingestion

Is your feature request related to a problem? Please describe.
Currently when setting any ingestion to include dbt metadata stored in S3 the AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY are mandatory, at least on the UI. This prevents relying on other authentication mechanisms like credentials stored in the default ~/.aws/credentials location or using the EC2 instance's IAM role
Describe the solution you'd like
The ACCESS_KEY fields are optional when defining a dbt source so the authentication can fallback to other mechanisms
Describe alternatives you've considered
N/A, in our company we don't have access to aws access key pairs so this makes S3 unusable for dbt for us.
Additional context
Pic of the form input validation requesting the fields to be fulfilled

Thanks @Beetelbrox for reporting this. Marking it as a dupe and closing to track the change in https://github.com/open-metadata/OpenMetadata/issues/6291
Thanks 🙏


Feat: Support SSH for retrieving terraform modules in private git repo

Support SSH for retrieving terraform modules in private git repo
Enable support for terraform modules in private git repo through SSH. A kubernetes secret which holds the ssh private-key and known_hosts should be created and referenced in the component definition.
Terraform controller PR: https://github.com/kubevela/terraform-controller/pull/349
Feature for https://github.com/kubevela/terraform-controller/issues/292
I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Tested in a local cluster.

Create a kubernetes secret of type kubernetes.io/ssh-auth for the git ssh credentials & known-hosts

apiVersion: v1
kind: Secret
metadata:
  name: git-ssh
  namespace: vela-system
type: kubernetes.io/ssh-auth
stringData:
  ssh-privatekey: |
   <SSH Private Key> # the ssh private key used for authenticating git
  known_hosts: |
   <SSH known_hosts>  # use `ssh-keyscan github.com`  to generate known_hosts

Create a component definition

apiVersion: core.oam.dev/v1beta1
kind: ComponentDefinition
metadata:
  annotations:
    definition.oam.dev/description: Terraform module to provision a KMS key with alias
  labels:
    type: terraform
  name: aws-kms-key
  namespace: vela-system
spec:
  schematic:
    terraform:
      configuration: git@github.com:motilayo/terraform-aws-kms-key.git # use terraform configuration in private git repo to test
      gitCredentialsSecretReference: # reference to the secret created earlier
        name: git-ssh # secret name
        namespace: vela-system  # secret namespace
      type: remote

Create an application and verify

Special notes for your reviewer

@wonderflow Please review when you get a chance. I've added some tests and rebased my branch with the master branch.


Clicking 'View All' for Users and Orgs as Super Admin blows up server

The queries for the Users and Organizations tables include a join to provide plan and template counts - these queries are overloading the server when a Super Admin clicks the link to 'View All' in these tables (I got a proxy error for both)
Disable View All for Super Admin users on these tables - updated per @sjDCC and my confirmation that this is a good solution

Yeah, @jollopre expressed concern about how big some of these queries would be. I’m not sure we need the ‘view all’ so let’s just disable it for superadmin.

View All link is no longer present on Users page at all. Desired behavior is to remove the link for Super Admins only and retain it for Org Admins. In the latter case it's unlikely that the number of users will cause issues and admins want to be able to expand the table to see an overview of all their users.
please reinstate the ability to View All users for Org Admins @jollopre

looks good now on roadmap stage. closing issue.


[CSSolver] Use correct locator when matching function result types related to closures

Currently we always use 'FunctionResult' as a path element when matching
function result types, but closure result type is allowed to be implicitly
converted to Void, which means we need to be careful when to use
'FunctionResult' and 'ClosureResult'.
Resolves: rdar://problem/37790062

/cc @slavapestov I've reverted changes to thunks and made it so we coerce directly in CSApply because there is no thunk support for C style functions. Is this what you had in mind?

@swift-ci please test

@swift-ci please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - 72140b227914474dd1efd0389132f9f58757f265

@swift-ci please test source compatibility

@swift-ci please test source compatibility

Looks like I've found a safer solution related to the type variable bindings, will make it a separate PR.


Add Round B - BikeTour - Python solution

Issue Number for which I'm assigned & worked: #6

🚀merged!!


showBuildSettings sh:cmp not found

I'm getting the following error whenever I run my tests:
2014-05-14 11:43:05.047 xctool[603:2607] Warning: error running xcodebuild -showBuildSettings: sh: cmp: command not found
My test command is:
xctool.sh -workspace ABC.xcworkspace -scheme 'ABCTests' -sdk iphonesimulator7.1 -reporter junit:junit.xml test
Any ideas what is causing this? The junit file is created, tests are run, but this is causing my jenkins job to fail.
Thanks

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

I think my PR https://github.com/facebook/xctool/pull/393 fixed this problem. Please reopen if you see hit it again.


Various improvements

Added linting, and applied fixes in response to warnings
Added features build including option parsing
Renamed xilinx script to xilinx-ise
Run as non-root user inside container, and mirror and mount the host user and home directory for transparent operation, and for better interactive usage.
Added support for X11 to enable GUI apps to run.
Various tidy-ups

I pulled this PR and applied it and it makes for a better experience. In particular you can start ISE and the GUI just pops up on your screen. It doesn't yet work when you've forwarded X11 across an SSH connection to the machine where the container is running but it solved the problem I was having with getting the GUI to appear (at least on a local machine running the container). Now to have it pick up the programming dongle so that one can use the ISE tutorials as written (caveat starting it up) and I'm good to go.


Unable to load 2 different images in 2 different imageViews line-by-line

On start of Activity I set:
ImageLoader.getInstance().init(ImageLoaderConfiguration.createDefault(getBaseContext()));
Then, in my invalidateFields method I write:
ImageLoader.getInstance().displayImage("https://upload.wikimedia.org/wikipedia/commons/5/5b/Ultraviolet_image_of_the_Cygnus_Loop_Nebula_crop.jpg", topPicture);
ImageLoader.getInstance().displayImage("http://www.citrus.ua/upload/new_iblock/eeb/cc096ef5b585f8e57c787f5f8f46693e.jpg", circleLogo);
topPicture is ImageView
circleLogo is de.hdodenhof.circleimageview.CircleImageView
After activity starts only one of images was downloaded instantly (if I put line with circleView before line with ImageView - it will download picture to circleView only; so, the issue is not in using circleView).
Btw, after some time (or, sometimes, after onResume()) second image is downloaded.
But I need it to be downloaded at the same time as first one, is it possible?
Let me add some STR:

Create new project in Android Studio.

Put in activity_main.xml:

<de.hdodenhof.circleimageview.CircleImageView
android:id="@+id/circleLogo"
android:layout_width="150dp"
android:layout_height="150dp"
android:layout_marginTop="50dp"
android:layout_marginBottom="30dp"/>

Put in MainActivity.java:
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
ImageLoader.getInstance().init(ImageLoaderConfiguration.createDefault(getBaseContext()));
 ImageView topPicture = (ImageView) findViewById(R.id.topPicture);
 ImageView circleLogo = (ImageView) findViewById(R.id.circleLogo);

 ImageLoader.getInstance().displayImage("https://upload.wikimedia.org/wikipedia/commons/5/5b/Ultraviolet_image_of_the_Cygnus_Loop_Nebula_crop.jpg", topPicture);
 ImageLoader.getInstance().displayImage("http://www.citrus.ua/upload/new_iblock/eeb/cc096ef5b585f8e57c787f5f8f46693e.jpg", circleLogo);

}

Run project, no matter on emulator or real device.

Result remains the same, if you change circleImageView to just ImageView.

Sorry, sorry and one more time sorry!
Issue was with that https picture (
Your lib is very great!


Sometimes function list does not work for Python

Issue Type: Bug
Hi, when I start VSC, last Typescript project folder is opened, then:
I go to File -> Close Folder
Then, I choose a Python project at the Recent section on Welcome tab.
After that, the Python project fully loads, but the Outline window doesn't show function list in any python file saying "No symbols found in document '<filename.py>'", it does not matter how much I wait, method list is not shown.

Thanks for replying, pylance

Thanks for replying, pylance


G80 CAN: GetWriteBufferSize should be not zero

Don't set GetWriteBufferSize, just read, it is zero instead of 1.
After set, it becomes correct.

Found on G120 too

EMX and other devices have similar issues because use same library.

Fixed by merged #212


Make site work without unsafe-inline CSP

Is your feature request related to a problem? Please describe.
I've been setting up a Content Security Policy (CSP) for a react-static site and I've noticed that the way static html is generated currently requires script-src 'unsafe-inline' and style-src 'unsafe-inline'; policies. Adding these policies opens up XSS and other attack vectors, so I'd prefer if the site worked without these policies.
Looking at a react-static-example-basic site, there are two issues. One is an inline script with the route info:

And two divs with the following inline style:

Describe the solution you'd like
It would be great if there was an option to not to include the inline script and load the route info from the routeInfo.json at the cost of performance. I'm not sure where the outline:none style comes from and why it's necessary, but it would be great if that could be excluded somehow as well.
Describe alternatives you've considered
For the inline script, one could compute its hash and insert a meta tag with the CSP option  script-src '<hash-algorithm>-<base64-value>';, but I'm not sure how this would play with additional CSP configurations eg. from headers.
Happy to provide a PR with some guidance :)

This most likely comes from @reach/router, which we can't "fix". You should inquire there about this one.
We'll happily and eagerly accept a PR which adds a hash to the inline script tag, so that there is NO performance hit. You won't need additional headers.

This most likely comes from @reach/router, which we can't "fix". You should inquire there about this one.

Thanks! I've left a comment there, but it doesn't seem likely that there is a solution.

We'll happily and eagerly accept a PR which adds a hash to the inline script tag, so that there is NO performance hit. You won't need additional headers.

The issue is that the script hash has to be added to the CSP directive, not the script tag. There are two ways to define a CSP: either in a Content-Security-Policy header or in an HTML meta tag (the former is the preferred way). In react-static static it would be only possible in an HTML meta tag, so the page would have the following tags:
<meta http-equiv="Content-Security-Policy" content="script-src 'self' 'sha256-abc123';">

...

But now the user can't add any other script sources to their CSP like a CDN, as that would either overwrite the script-src directive in the meta tag if it came from a header or it would be ignored if it was added in a subsequent meta tag (I looked this up now in the CSP spec).
My use case would be covered by the above solution with the meta tag, but I'm not sure if it's a good solution in general. Alternatively, if react-static output a file with inline script hashes to the dist folder, so that I can read that file and add the hashes to my CSP header dynamically, that would also solve my problem, but I don't think this is a good general solution either. So I think the simplest thing would be an option to exclude the inline script even that costs some performance (the route info is only needed to make the site interactive, but the site can render without it, right?).

But now the user can't add any other script sources to their CSP like a CDN, as that would either overwrite the script-src directive in the meta tag if it came from a header or it would be ignored if it was added in a subsequent meta tag

That is correct -- I was suggesting adding it as a meta, and outputting it via the build plugins, so that one could ammend it/capture it.

(the route info is only needed to make the site interactive, but the site can render without it, right?).
Yes and no. The site will suspend without it, which breaks pre-rendering

Right now this is what we do:

inject the route info
render the page, where we set the "context" for the route info to match window.__routeInfo
the rendered page is stored

Wen we do 3, we can actually remove the window.__routeInfo, although hydration will fail -- and another roundtrip is needed to make the site interactive (as you say).
A much easier solution would probably be to inject a meta tag, or add it as a data-* attribute. That solves all these issues.

That is correct -- I was suggesting adding it as a meta, and outputting it via the build plugins, so that one could ammend it/capture it.
...
A much easier solution would probably be to inject a meta tag, or add it as a data-* attribute. That solves all these issues.

Ok, happy to work on this, but would need some pointers. My best idea would be to add the inline script hashes to the state prop of Document and then the user can construct a CSP in a meta element as they wish.

Hi there, I am trying to figure out how to use this new feature. I'm using an express server with helmet to serve the /dist directory. I can add a hash for every page to the CSP, but that seems cumbersome. The docs seem to imply that I can change a setting in static.config.js in order to set a single hash for all pages, or am I reading that totally wrong?
Can you provide an example for how to use Document in the static.config.js to configure inlineScripts?

Document: {
    state: {
      inlineScripts: {
        script: "script",
        hash: "sha256-kjhlkjhkljhlkjhkljhfg...",
      },
    },
  },
...

Thank you!

Hi @owenschoppe this feature is just to give you the hashes of the inline scripts generated by react-static at build time. You can then add these hashes to your CSP at build time to restrict scripts. The Document.state object is not meant to be set in the configuration file, it's something that react-static populates dynamically at build time to give you access to some internals like the inline script hashes in this case. Hope this helps!

@abiro @SleeplessByte
I am trying to follow the documentation related to creating site-wide meta tags.  However, after attempting to write out custom CSP meta tags I am finding that my strings are being encoded which I don't believe will work.
For instance, with the following code....
  Document: ({     Html,     Head,     Body,     children,     state: { routeInfo, siteData, renderMeta, inlineScripts },   }) => {     return (                           <meta charSet="UTF-8" />           <meta name="viewport" content="width=device-width, initial-scale=1" />           {Object.keys(inlineScripts).map((key) => (             <meta               http-equiv="Content-Security-Policy"               content={script-src='${inlineScripts[key].hash}'}             ></meta>           ))}                  hi's            );   },
my resulting meta tag looks like this (single quotes are encoded)...
    <meta       http-equiv="Content-Security-Policy"       content="script-src='sha256-og5vccCPe6Stmme6KiUIBOaCn8mrsuEM1XnPIxr791o='"     />
I have tried to switch to dangerouslySetInnerHtml, but that also does not work because the HEAD component is automatically injecting Link elements to the rendered head element and react throws an error saying you cant do both dangerouslySetInnerHtml and set children.
So how do I get this to render correctly?

There is no = inside the CSP value. See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy#using_the_html_meta_element

There is no = inside the CSP value. See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy#using_the_html_meta_element

thanks, but I realized that error after my post and edited it to match the correct value.  Still my question stands

hi @abiro could you possibly share the technique you used to generate the meta tags correctly?  I am having trouble here with encoding that I don't see how to solve.

There is no = inside the CSP value. See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy#using_the_html_meta_element

Any thoughts on how to solve the encoding issue?

@buddhamangler-cbre I had a look and found this function, but I haven't found any usages and it's been two years so I can't guarantee that it works:
function getCSPString(inlineScripts) {
  let scriptSources = ''
  for (let k in inlineScripts) {
    scriptSources += `'${inlineScripts[k].hash}'`
  }
  // TODO (abiro) remove style unsafe inline
  // Solution: https://github.com/reach/router/issues/63#issuecomment-575748441
  const csp = `
  default-src 'none';
  prefetch-src 'self';
  script-src 'self' ${scriptSources};
  connect-src 'self' https://api.example.com https://cognito-idp.us-west-2.amazonaws.com;
  img-src 'self';
  style-src 'self' 'unsafe-inline';
  base-uri 'self';
  frame-ancestors 'none';
  form-action 'self';
  `.replace(/(?:\r\n|\r|\n)/g, ' ')

  return csp
}

inlineScripts should be the same object that you get from Document.state in static.config.js.

Hi ,
Could you please help  me to understand the standard way to add the hash into index.html
is there any lib or just take a hash from console and paste it into csp directive


Key Fob - How to?

Hello,
Integration is working well. How to use Key Fob? I have Key Fob in Peripherals, but see only one "button" in devices. This button is doing nothing (doesn't react on Key Fob pressing). My Key Fob has 4 buttons..... How to map it to HA?
Maybe some guid would be helpful.
Thanks in advance

There's no support for "4 buttons keyfob" now. You can join the Discord, try to debug the keyfob via Wireshark and we may add the support.