Fixing select component size variations

Task: task-531897
Link: preview-355
Fixing the select component sizes.
Testing

Check the code
Visit Select page

I just noticed that there's a subtle difference with the new change. The corners of the small variant is less round. Also the carats are now the same size for all size variants. Are these changes expected?
new

old

I just noticed that there's a subtle difference with the new change. The corners of the small variant is less round. Also the carats are now the same size for all size variants. Are these changes expected?
new
old

Border change is expected and what I'm going for on current PR actually to match the input :D
Good catch on the carats, fixed now!


zsh-syntax-highlighting: unhandled ZLE widget 'vi-backward-delete-char-incr'

I tried to source the plugin and it was fine until I reload my terminal.
It complains about this error every time I launch my terminal
zsh-syntax-highlighting: unhandled ZLE widget 'vi-backward-delete-char-incr'

zsh-syntax-highlighting: unhandled ZLE widget 'vi-backward-delete-char-incr'

That's not a widget from zsh so it must be defined by some other plugin.
This error generally means that other plugin did bindkey x vi-backward-delete-char-incr foo without defining the widget foo.

Thanks for the input. I have found the cause of the problem. It was a program I installed, Bit. Bit injected a zsh dot file and its causing the problem

Thanks for the input. I have found the cause of the problem. It was a program I installed, Bit. Bit injected a zsh dot file and its causing the problem

Thanks for sharing the caused.  Glad it's resolved.


Fix a bug where RequestLog is not complete under certain circumstances

Motivation:
The RequestLog should be complete because the active request count of an EventLoop
is decreased when the ReqeustLog is complete.
However, if an exception is raised even before sending an HttpRequest by specifying
invalid URI, failing to resolve URI, etc, the RequestLog is not complete.
Modifications:

Fix to complete RequestLog
Fix to abort the HttpRuest

Result:

Less bugs

Thanks, @minwoox!


any way to add icons ?

is there any way to add images or icons in the items ?

You can use the renderOption function prop and render whatever you like.


fix: use an action to get the go version instead of hardcoding it

Signed-off-by: Harikrishnan Balagopal<EMAIL_ADDRESS>

Can we parameterize the version in dockerfile too?

done


Dead link to Snyk guidelines

The link to the Snyk guidelines
https://github.com/equinor/appsec/blob/1a2c052a351200fb675f0e7d2dc13be979bfafeb/docs/snyk/3-snyk_support.md?plain=1#L9
does not work for me. I get an 404 error when trying to access the page. It seems that mkdocs does not generate a page for the "Guidelines" group.

fixes coming in #81


how to edit kernel weight at certain global_step using estimator

i want to set some weights to zero at certation global step.

This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!


Reset tool rename

The reset tool needs to be renamed to Reset extent or something similar for the mouseover label. Example is one of the arrow circle things in the second cell of Annotators.ipynb

With modifiable bokeh icons, can we also modify labels?

Modifiable icons and labels were merged at the same time, unfortunately I only implemented that for the drawing tools, so currently there is no way to rename the reset tool.

Got it. I'll leave this open as a reminder for future enhancement if we decide on that.


Update Dockerfile

Thanks for the PR @LukeLIN-web! However, I don't think these changes are necessary. Thus, close it for now. Feel free to leave comments explaining why you want these changes.

Thanks for the PR @LukeLIN-web! However, I don't think these changes are necessary. Thus, close it for now. Feel free to leave comments explaining why you want these changes.


chore: switch to tinyglobby to drop 21 dependencies

https://npmgraph.js.org/?q=fast-glob - 17 dependencies
https://npmgraph.js.org/?q=tinyglobby - 2 dependencies

@anncwb @jinmao88 any chance you might be able to take a look at this? thanks!


how to install only on specified macOS version or arch of OS

like >= sonoma, and is arm64
the example Brewfile in README does not show such function, does homebrew-bundle support it?

Just use Ruby and Homebrew functions for this e.g. if Hardware::CPU.arm? or MacOS.version >= :sonoma


Support multiple encoders for future proofing

I was simply wondering since encoding is currently handled by SVT-AV1 directly and the likely arrival of AV1 hardware encoding on the next generation of video cards if it would not be more future-proof to be able to choose the encoder to use, like using ffmpeg for encoding?
This way it simply need to have an updated version of ffmpeg to encode in future formats as well (and also for those already present, h264, h265 etc.).
I think that a flag addition to choose to pass commands to ffmpeg for encoding would be enough.

It would be nice to run comparisons of different encoders all "auto-encoding" + across presets all from ab-av1.
On the other hand the main use case is to use the "best" av1 encoder to re-encode stuff. Just having one makes the code simpler, easier to maintain, the cli easier to design & understand etc.
What kind of use cases were you thinking about?

The use case I was referring to is using ffmpeg for encoding while retaining the ab-av1 feature of the VMAF sample to evaluate quality and file size.
Unlike the current version which is based on the svt-av1 encoder then based on the cli flags accepted by the included version of the encoder which could be incompatible with new versions I would entrust the encoding handling to ffmpeg with the advantage of being able to use other encodings as well.
Worst case scenario: ffmpeg changes the syntax and is no longer compatible with the implemented version. So the same fate that could happen in the current state using the SVT-AV1 encoder.
In other cases: You can get the same results as the current ab-av1 version but also more flexibility by being able to choose a different supported encoder that is already included in ffmpeg. This means greatly reducing the need for upgrades for future encoders such as GPU intensive AV1 ones. Also confirming the use case of using the "best" av1 encoder to re-encode stuff.


issue-424: remove benchmark from pythainlp

Related to #428

Coverage decreased (-0.03%) to 97.273% when pulling 265e6c7456d92fe51a26c31d5911e5f94b98dace on remove-benchmark into d460708c8a480465492d51fe48c367c9439cc9df on dev.

Hello @heytitle! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file pythainlp/cli/benchmark.py:

Line 13:1: E302 expected 2 blank lines, found 1
Line 23:80: E501 line too long (102 > 79 characters)
Line 44:1: E302 expected 2 blank lines, found 1
Line 64:80: E501 line too long (94 > 79 characters)
Line 105:80: E501 line too long (97 > 79 characters)
Line 108:80: E501 line too long (94 > 79 characters)
Line 118:80: E501 line too long (124 > 79 characters)
Line 143:80: E501 line too long (91 > 79 characters)
Line 147:58: W292 no newline at end of file


Revert "Dehardcode cbor hexes + doc"

Reverts Plutonomicon/cardano-transaction-lib#607

It looks like I have a fix in my stash, don't merge it now

Review this instead: https://github.com/Plutonomicon/cardano-transaction-lib/pull/753


restrictToParentElement modifier not working with current setup

Current Problem

Have a DndContext with the restrictToParentElement modifier in the modifiers array
Child element of the DndContext is a droppable div (with the ref set to setNodeRef coming from useDroppable) and css styling of position: "absolute", overflow: "hidden", width: "100%", height: "100%". This is inside CanvasElementsArea component shown below
The child of the droppable div is a mapping of elements, each rendering a draggable element. The Element component below
Each element can be dragged and dropped correctly, however the problem is that the restrictToParentElement modifier is not obeyed

Question
Is there something I am missing with my approach of is there any styling that I need to adhere to to make this work? Code snippet posted below
DndContext (highest-level)
<DndContext
  onDragStart={onDragStart}
  modifiers={[restrictToParentElement, snapToGrid]}
  sensors={sensors}
>
  <CanvasElementsArea isEditing={isEditing} isMobile={layout === "mobile"} />
</DndContext>

CanvasElementsArea component
const CanvasElementWrapper = styled("div")<IsMobile>`
  position: ${({ isMobile }) => isMobile ? "relative" : "absolute"};
  pointer-events: none;
`;

const CanvasElementsArea = ({
  isEditing,
  isMobile = false,
}) => {
  const elements = ["element 1", "element 2"];
  const { setNodeRef } = useDroppable({
    id: "droppable-area"
  });

  return (
    <div
      ref={setNodeRef}
      style={{
        position: "absolute",
        overflow: "hidden",
        width: "100%",
        height: "100%"
      }}
    >
      {elements.map((text, index) => (
        <CanvasElementWrapper key={text} isMobile={isMobile}>
          <Element text={text} />
        </CanvasElementWrapper>
      ))}
    </div>
  )
};

export default memo(CanvasElementsArea);

Element component
const Element = ({
  text,
}) => {
  const [{x, y}, setCoordinates] = useState<Coordinates>({x: 0, y: 0});
  const {
    attributes,
    isDragging,
    listeners,
    setNodeRef,
    transform,
  } = useDraggable({
    id: text, // this is always unique in my case
  });

  useDndMonitor({
    onDragEnd(event: DragEndEvent) {
      const newCoordinates = {x: x + event.delta.x, y: y + event.delta.y};
      setCoordinates(newCoordinates);
    },
  });

  const style = {
    display: "inline-flex",
    position: "relative",
    background: "#fff",
    boxShadow: isDragging ? defaultBoxShadow : "none",
    transform: CSS.Transform.toString(transform),
    transition: "200ms box-shadow ease-in-out",
    pointerEvents: "all",
    cursor: "grab"
  } as CSSProperties;

  return (
    <div
      ref={setNodeRef}
      style={
        {
          ...style,
          top: y,
          left: x,
          width: 200,
          height: 200
        } as CSSProperties
      }
      {...listeners}
      {...attributes}
    >
        {text}
    </div>
  );
};

export default memo(Element);

Would appreciate any help here, thanks!

anybody else seen this? still can't find the solution

Got the same issue but solved it with adding an extra div with ref={setNodeRef} on the outer, and then the {...listeners}       {...attributes} on the inner.

Do you mean like the following:
return (
    <div
      ref={setNodeRef}
      style={
        {
          ...style,
          top: y,
          left: x,
          width: 200,
          height: 200
        } as CSSProperties
      }
    >
      <div {...listeners} {...attributes}>
        {text}
      </div>
    </div>
);

yep!

sadly this didn't work for me personally! thank you though
still struggling with this so i may have to start from scratch to see where it goes wrong


merge: true and optional fields

Let's say a a field is defined as:
Category = Class.create({
	...
        fields: {
		type: {
			type: String,
			optional: true
		},
       }
});

category: {
  type: Category,
  optional: true
}

When category is null or undefined, doc.set({category: {type: ""}}, {merge: true, cast: true}) fails to update category.type. It succeeds only if category is an Object.
EDIT: Actually it happens in case of a double nested scenario: eg. page.category.type

It's expected behavior as astronomy doesn't know how to merge undefined value. Yoo can only merge objects.

But then we lose the advantage of merge. If we have to remember the field types and set them as objects, it again becomes an extra task.
Can't Astronomy have an extra check in case of merge and automate it?

You can use default value for that field and it will merge with it. I can't merge with undefined values. Because it can be any value. What if someone set field's type to String and want to merge some value with it? I can't merge anything with string. Merging only applies to objects, so there should be an object already in a field. And as I said you can use default values for that.

I generally avoid default as it causes problem in updating docs where I have not fetched all fields. It sets field which is not fetched to its default value, overwriting its original value.
Is there a way to use default safely?

Yes there is a way of using default safely. Assuming that in the User class there is the address field whose default values is {}, then you can fetch it in the following way:
User.findOne({}, {fields: {address: 0}, defaults: false});

thanks!


Swift 5 infinite loop - fix optionals

As explained in this issue, this PR removes the optional methods
https://github.com/lucianomarisi/JSONUtilities/issues/31
Either this PR or #32 should be merged, not both

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #33   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines         144    186   +42
=====================================
+ Hits          144    186   +42

@Jon889 I'd prefer this approach as opposed to #32, are you happy for me to merge it?
By the way really cool that you provided code for both approaches!

Yep, happy with either way, just we might need to keep an eye on it init the future as it’s not recommended to overload based throwing or not.
Np, just trying to make it as easy as possible :)

Thanks for merging it :) can you deploy it as well?
(I’m happy to do the boring stuff if you want to add me as a collaborator, don’t worry I wouldn’t make changes without your input/approval, completely understand if you don’t want to though! :))

I've added you as collaborator to the project, so you should be able to merge new stuff in the future.

Awesome thanks! Also just in general, thanks for making this framework, it's been really useful and the API using generics and throwing errors is much better than say SwiftyJSON.


M24H-20 generate case number in jsbox

@KaitCrawford ready for review, this one's likely a bit more tricky. I've left some explanatory comments in it, hope it helps.

:+1: Thanks for the explanations :smile:


Added two-sum solution in python

You should use leethub extension to send PR's. Also read Readme.md


.wq! Deletes All data except the line where the cursor is located.

Steps to reproduce

Create any file with the following content (for example):string1
string2
string3
string4
string5

Open the file with VIM:vim test.txt

Position the cursor on any line and type the following command::.wq!

Expected behaviour
The file should be saved and the VIM editor should exit normally.
Actual Behavior:
All lines in the file are deleted except the string where the cursor is located. The content of the file after the command is:
[root@fedora ~]# cat test.txt
string3
[root@fedora ~]#

Version of Vim
9.0.1575
Environment
- OS: Fedora Linux 37 (Workstation Edition)
- Terminal: PuTTY Release 0.81
- value of $TERM: xterm
- shell: bash 5.2.15(1)
Logs and stack traces
### Full Command Sequence

# Create the test file
echo -e "string1\nstring2\nstring3\nstring4\nstring5" > test.txt

# Open the file in VIM with verbose logging
vim -V9vimlog.txt test.txt

# Inside VIM, execute the command to reproduce the issue
# :.wq!

# Exit VIM and check the log file
cat vimlog.txt

chdir(/etc)
fchdir() to previous dir
sourcing "/etc/vimrc"
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 90: sourcing "/usr/share/vim/vim90/syntax/syntax.vim"
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 19: sourcing "/usr/share/vim/vim90/syntax/synload.vim"
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 21: sourcing "/usr/share/vim/vim90/syntax/syncolor.vim"
chdir(/usr/share/vim/vim90/colors/lists)
fchdir() to previous dir
line 53: sourcing "/usr/share/vim/vim90/colors/lists/default.vim"
finished sourcing /usr/share/vim/vim90/colors/lists/default.vim
continuing in /usr/share/vim/vim90/syntax/syncolor.vim
finished sourcing /usr/share/vim/vim90/syntax/syncolor.vim
continuing in /usr/share/vim/vim90/syntax/synload.vim
finished sourcing /usr/share/vim/vim90/syntax/synload.vim
continuing in /usr/share/vim/vim90/syntax/syntax.vim
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 25: sourcing "/usr/share/vim/vim90/filetype.vim"
not found in 'runtimepath': "ftdetect/*.vim"
finished sourcing /usr/share/vim/vim90/filetype.vim
continuing in /usr/share/vim/vim90/syntax/syntax.vim
Executing FileType Autocommands for "*"
autocommand 0verbose exe "set syntax=" . expand("<amatch>")

Executing BufRead Autocommands for "*"
autocommand if !did_filetype() && expand("<amatch>") !~ g:ft_ignore_pat | runtim                                                                                e! scripts.vim | endif

chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 0: sourcing "/usr/share/vim/vim90/scripts.vim"
chdir(/usr/share/vim/vim90/autoload/dist)
fchdir() to previous dir
line 30: sourcing "/usr/share/vim/vim90/autoload/dist/script.vim"
finished sourcing /usr/share/vim/vim90/autoload/dist/script.vim
continuing in /usr/share/vim/vim90/scripts.vim
finished sourcing /usr/share/vim/vim90/scripts.vim
continuing in BufRead Autocommands for "*"
Executing BufRead Autocommands for "*.txt"
autocommand if getline('$') !~ 'vim:.*ft=help'|   setf text| endif

Executing FileType Autocommands for "*"
autocommand 0verbose exe "set syntax=" . expand("<amatch>")

Executing BufRead Autocommands for "*"
autocommand if !did_filetype() && expand("<amatch>") !~ g:ft_ignore_pat    && (g                                                                                etline(1) =~ '^#' || getline(2) =~ '^#' || getline(3) =~ '^#'^I|| getline(4) =~                                                                                '^#' || getline(5) =~ '^#') |   setf FALLBACK conf | endif

finished sourcing /usr/share/vim/vim90/syntax/syntax.vim
continuing in /etc/vimrc
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 98: sourcing "/usr/share/vim/vim90/filetype.vim"
finished sourcing /usr/share/vim/vim90/filetype.vim
continuing in /etc/vimrc
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 98: sourcing "/usr/share/vim/vim90/ftplugin.vim"
finished sourcing /usr/share/vim/vim90/ftplugin.vim
continuing in /etc/vimrc
finished sourcing /etc/vimrc
chdir(/root)
fchdir() to previous dir
could not source "$HOME/.vimrc"
chdir(/root/.vim)
fchdir() to previous dir
could not source "~/.vim/vimrc"
chdir(/root)
fchdir() to previous dir
could not source "$HOME/.exrc"
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
sourcing "$VIMRUNTIME/defaults.vim"
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 85: sourcing "/usr/share/vim/vim90/filetype.vim"
finished sourcing /usr/share/vim/vim90/filetype.vim
continuing in /usr/share/vim/vim90/defaults.vim
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 85: sourcing "/usr/share/vim/vim90/ftplugin.vim"
finished sourcing /usr/share/vim/vim90/ftplugin.vim
continuing in /usr/share/vim/vim90/defaults.vim
chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 85: sourcing "/usr/share/vim/vim90/indent.vim"
finished sourcing /usr/share/vim/vim90/indent.vim
continuing in /usr/share/vim/vim90/defaults.vim
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 121: sourcing "/usr/share/vim/vim90/syntax/syntax.vim"
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 15: sourcing "/usr/share/vim/vim90/syntax/nosyntax.vim"
Executing BufEnter Autocommands for "*"
autocommand syn clear

autocommand if exists("b:current_syntax") | unlet b:current_syntax | endif

finished sourcing /usr/share/vim/vim90/syntax/nosyntax.vim
continuing in /usr/share/vim/vim90/syntax/syntax.vim
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 19: sourcing "/usr/share/vim/vim90/syntax/synload.vim"
chdir(/usr/share/vim/vim90/syntax)
fchdir() to previous dir
line 21: sourcing "/usr/share/vim/vim90/syntax/syncolor.vim"
finished sourcing /usr/share/vim/vim90/syntax/syncolor.vim
continuing in /usr/share/vim/vim90/syntax/synload.vim
finished sourcing /usr/share/vim/vim90/syntax/synload.vim
continuing in /usr/share/vim/vim90/syntax/syntax.vim
Executing FileType Autocommands for "*"
autocommand 0verbose exe "set syntax=" . expand("<amatch>")

finished sourcing /usr/share/vim/vim90/syntax/syntax.vim
continuing in /usr/share/vim/vim90/defaults.vim
finished sourcing $VIMRUNTIME/defaults.vim
not found in 'packpath': "pack/*/start/*"
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/getscriptPlugin.vim"
finished sourcing /usr/share/vim/vim90/plugin/getscriptPlugin.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/gzip.vim"
finished sourcing /usr/share/vim/vim90/plugin/gzip.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/logiPat.vim"
finished sourcing /usr/share/vim/vim90/plugin/logiPat.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/manpager.vim"
finished sourcing /usr/share/vim/vim90/plugin/manpager.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/matchparen.vim"
finished sourcing /usr/share/vim/vim90/plugin/matchparen.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/netrwPlugin.vim"
finished sourcing /usr/share/vim/vim90/plugin/netrwPlugin.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/rrhelper.vim"
finished sourcing /usr/share/vim/vim90/plugin/rrhelper.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/spellfile.vim"
finished sourcing /usr/share/vim/vim90/plugin/spellfile.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/tarPlugin.vim"
finished sourcing /usr/share/vim/vim90/plugin/tarPlugin.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/tohtml.vim"
finished sourcing /usr/share/vim/vim90/plugin/tohtml.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/vimballPlugin.vim"
finished sourcing /usr/share/vim/vim90/plugin/vimballPlugin.vim
chdir(/usr/share/vim/vim90/plugin)
fchdir() to previous dir
sourcing "/usr/share/vim/vim90/plugin/zipPlugin.vim"
finished sourcing /usr/share/vim/vim90/plugin/zipPlugin.vim
not found in 'packpath': "pack/*/start/*"
not found in 'runtimepath': "plugin/**/*.vim"
Reading viminfo file "/root/.viminfo" info oldfiles
"test.txt"
"test.txt" 5L, 40B
Reading viminfo file "/root/.viminfo" marks
Executing BufRead Autocommands for "*"
autocommand if line("'\"") > 0 && line ("'\"") <= line("$") |   exe "normal! g'\                                                                                "" | endif

Executing BufRead Autocommands for "*"
autocommand if !did_filetype() && expand("<amatch>") !~ g:ft_ignore_pat | runtim                                                                                e! scripts.vim | endif

chdir(/usr/share/vim/vim90)
fchdir() to previous dir
line 0: sourcing "/usr/share/vim/vim90/scripts.vim"
finished sourcing /usr/share/vim/vim90/scripts.vim
continuing in BufRead Autocommands for "*"
Executing BufRead Autocommands for "*.txt"
autocommand if getline('$') !~ 'vim:.*ft=help'|   setf text| endif

Executing FileType Autocommands for "*"
autocommand call LoadFTPlugin()

chdir(/usr/share/vim/vim90/ftplugin)
fchdir() to previous dir
line 18: sourcing "/usr/share/vim/vim90/ftplugin/text.vim"
finished sourcing /usr/share/vim/vim90/ftplugin/text.vim
continuing in <SNR>9_LoadFTPlugin
Executing FileType Autocommands for "*"
autocommand call s:LoadIndent()

not found in 'runtimepath': "indent/text.vim"
Executing FileType Autocommands for "*"
autocommand 0verbose exe "set syntax=" . expand("<amatch>")

Executing BufRead Autocommands for "*"
autocommand if !did_filetype() && expand("<amatch>") !~ g:ft_ignore_pat    && (g                                                                                etline(1) =~ '^#' || getline(2) =~ '^#' || getline(3) =~ '^#'^I|| getline(4) =~                                                                                '^#' || getline(5) =~ '^#') |   setf FALLBACK conf | endif

Executing BufRead Autocommands for "*"
autocommand if line("'\"") >= 1 && line("'\"") <= line("$") && &ft !~# 'commit'                                                                                |   exe "normal! g`\"" | endif

Executing BufWinEnter Autocommands for "*"
autocommand call s:Highlight_Matching_Pair()

Executing BufEnter Autocommands for "*"
autocommand sil call s:LocalBrowse(expand("<amatch>"))

Executing VimEnter Autocommands for "*"
autocommand sil call s:VimEnter(expand("<amatch>"))

Executing CursorMoved Autocommands for "*"
autocommand call s:Highlight_Matching_Pair()

Executing CursorMoved Autocommands for "*"
autocommand call s:Highlight_Matching_Pair()

Executing CursorMoved Autocommands for "*"
autocommand call s:Highlight_Matching_Pair()

"test.txt"
"test.txt" 1L, 8B written

Also works with many other rpm-based distributions and older versions of VIM.

Isn't this behavior correct?
:[range]wq[!] [++opt] [file]
			Same as above, but only write the lines in [range].

Actually you are right

Op za 1 jun. 2024 11:58:

Position the cursor on any line and type the following command:

:.wq!

I guess that the . indicates the range of lines to write, the current one...
Christ van Willegen


Skip integrations not binded to the current client and fetch their options from it

Q
A

Branch?
2.1

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
yes

License
MIT

As noticed by @HazAT in #847, some integrations were executed even though the client binded to the current hub was not using them. I also changed how we pass the options to the integrations by stopping to use the constructor and fetching them from the client binded to the current hub so that we can avoid having a circular reference between the two

The reason is just for backwards compatibility. I could simply stop using the options passed to the constructor and always use the client to fetch them, however people that implemented the integrations wrong (like we did!) and that (unlikely case, but possible so we must take it into account) change the hub during execution and have a different client bound to it will suddenly find out that their app breaks because the behavior changed and the options are now different from the ones they expected (even though they are now the ones that should be)


How to impose external forces on boundary vertices in ex2?

Hello everyone,
In ex2, the external forces are constant over the boundary, the code is
   VectorArrayCoefficient f(dim);
   for (int i = 0; i < dim-1; i++)
   {
      f.Set(i, new ConstantCoefficient(0.0));
   }
   {
      Vector pull_force(mesh->bdr_attributes.Max());
      pull_force = 0.0;
      pull_force(1) = -1.0e-2;
      f.Set(dim-1, new PWConstCoefficient(pull_force));
   }

   LinearForm *b = new LinearForm(fespace);
   b->AddBoundaryIntegrator(new VectorBoundaryLFIntegrator(f));
   cout << "r.h.s. ... " << flush;
   b->Assemble();

However, in my case, the external forces vary in each boundary vertex.
Before posting this issue, I tried to solve this by myself.
By using different bdr_attributes in each boundary segement and assumng that the force on each boundary segement is half of the forces on each boundary segement.
The results are not ideal. I think this is because the forces on each boundary segement are not continuous, I can't take a combination of the half of forces in the two vertices as the average forces in the boudary segement.
Hopefully, someone would give me some advice on this issue! Thanks in advance!
Sincerely,
Jianhua

Is your desired boundary value a continuous function f(x,y)?

Hi @jandrej,
The boundary forces are obtained from a fluid solver. Generally, they are discontinuous.

Do you get the solution of the fluid solver on the same mesh? If yes, does the solution vector contain values at the nodes of the mesh? If you could get geometry information from the solver, for example a map which tells you fluid_solver_solution(x, y) -> R^n, then you could use a FunctionCoefficient or VectorFunctionCoefficient and implement a callback function that handles this mapping.

@jandrej
Thank you for your advice!
The fluid solver and MFEM share the same boundary vertices/nodes. Therefore, it is possible to use force(x, y) -> R. By doing so, we can construct a coefficient that contains only the vertex number and solution verctor of the force.
For example, (0, 1.0e-2, 1.0e-2), (1, 2.0e-2, 1.0e-2), (2, 1.0e-3, 1.0e-3)...
What I want to know is how to construct LinearForm *b.
In MFEM, I can only find b->AddBoundaryIntegrator and b->AddDomainIntegrator.
However, neither of them can be used for nodal forces.

If I had to do it, I would make a GridFunction which would take on the values you get from the fluid solver and then send it to b->AddBoundaryIntegrator(new VectorBoundaryLFIntegrator).

@vikramsg
It seems to be a solution. But I don't know how to make the GirdFunction. The fluid solver is based on Lattice Boltzmann Method (LBM) and Immersed Boundary Method (IBM). Therefore, the forces are only on nodal points.

Hello, @JianhuaQin ,
I also like @vikramsg 's solution but unfortunately there is an outstanding issue (#565 ) in MFEM which will cause this to fail.
I see two ways to work around this issue but both of them require that you create a GridFunction on the MFEM mesh which approximates your force vector field from the LBM solver.  Once you have this GridFunction you can compute the needed LinearForm by performing a matrix-vector multiplication.  The matrix you'll need is a mass matrix computed on the boundary of the mesh:
BilinearForm m(fespace);
m.AddBoundaryIntegrator(new MassIntegrator);
//or
m.AddBoundaryIntegrator(new MassIntegrator, bdr_marker);
m.Assemble();
If you choose to use different bases for the LBM solver and the elasticity solver you can use a rectangular operator in the above by using a MixedBilinearForm with the same MassIntegrator instead.
Presumably you have a mapping from your LBM vertices to your MFEM boundary vertices.  If not then hopefully you can build it by looping through the MFEM mesh and using GetVertex and comparing these coordinates with your LBM boundary coordinates.  Clearly this could be expensive so you should do it once and store the result if possible.
Once you have this mapping you can rely on the fact that the order of the GridFunction degrees of freedom for a first order H1 basis in serial is only determined by the vertex number and the dimension of the space.  Specifically, the dof index is "dim*node+d" where 'd' is 0, 1, or 2 corresponding to the 'x', 'y', or 'z' component of the field.
The other option is to write a Coefficient object for each component of the force and incorporate the above mapping into these coefficients so that you can look up force values given nodal coordinates.  You can then call GridFunction::ProjectBdrCoefficient(Coefficient*coeff[], Array &attr).  This method will look cleaner and it has the potential to work in parallel but it would probably be less efficient.
I'd be happy to give more details if any of this is unclear.
Best wishes,
Mark

Hi @mlstowell ,
Thank you so much for the discussion! Your suggestions help me a lot!
By using a first order H1 basis, I constructed a GridFunction force by
   ifstream ifilef("../../Turek/force.data");
   GridFunction force(mesh, ifilef);
   ifilef.close();

This gridfunciton contains values of the whole domain, not only boundary values.
Then I can easily construct the linear form by
   LinearForm *bb = new LinearForm(&fespace);
   VectorGridFunctionCoefficient forcec(&force);
   bb->AddDomainIntegrator(new VectorDomainLFIntegrator(forcec));
   bb->Assemble();

Is anything wrong by doing like this?
Thank you again for your help!
Best regards,
Jianhua

Hello, @JianhuaQin ,
The first approach you showed will impose a force throughout the entire volume of the mesh.  If the grid function is equal to zero in most of the mesh then this force will also be zero so that may be ok.  You should be warned though that the LinearForm object in this case is performing a volume integral and not a surface integral so, even if the GridFunction is only nonzero on the boundary, the effective force is acting on the outermost layer of elements and not on the surface of those elements.  The difference will depend on the thickness of the outer layer of elements and this could easily lead to bad results.
Regarding the time to write the file force.data.  I presume you are mostly writing zeros to this file because, if I understand correctly, the force is only nonzero on the boundary.  You could speed this up by only writing the non-zero values and their coordinates or vertex indices.  You could then read this much smaller file and use a suitable mapping from LBM indices to MFEM indices to place the values in the GridFunction one by one.
The matrix-vector multiplication scheme has some problems as you've written it.  The vectors used in finite element schemes come in two flavors; primal and dual.  This is a subtle distinction and MFEM's class documentation does not help to clarify this distinction so it's easily missed.  MFEM's GridFunction class is used to store primal vectors.  These store the degrees of freedom that we solve for, they are the expansion coefficients in a finite element representation of a function e.g. they are the a_i in \sum_{i=0}^n a_i \phi_i(x).  MFEM's LinearForm class is used to store dual vectors.  These are linear functionals which store the values of integrals of the basis functions e.g. they are the b_i in b_i = \int_\Omega f(x)\phi_i(x)dx for some function f(x).  If you wanted to compute the integral of the product of two functions f(x) and g(x),  MFEM provides two ways to accomplish this:

Project f(x) and g(x) onto two Gridfunctionss called fPrimal and gPrimal, compute a mass matrix, M, using a BilinearForm  and compute fPrimal^T * M * gPrimal.
Project g(x) onto a GridFunction and build a LinearForm called fDual with f(x) and compute fDual^T * gPrimal.
These two methods should produce exactly the same answer as long as the BilinearForm and LinearForm use the same integration rule.  In other words fDual = M * fPrimal.  With this in mind your scheme becomes the following where *bb is the dual vector associated with the primal vector stored in force.

   ifstream ifilef("../../Turek/force.data");
   GridFunction force(mesh, ifilef);
   ifilef.close();
   LinearForm *bb = new LinearForm(&fespace);
   BilinearForm m(&fespace);
   const double ref_density = 1.0*dr; // density in the reference configuration
   ConstantCoefficient rho0(ref_density);
   m.AddBoundaryIntegrator(new MassIntegrator(rho0));
   m.Assemble();
   m.Mult(force, *bb);  //matrix-vector multiplication

Best wishes,
Mark

Hello, @mlstowell ,
Thank you so much for the thorough explanation. I will keep learning how to use MFEM!
Of course the thickness of the element has a significant influence on the results.
Yours sincerely,
Jianhua


ESC7 - Error when attempting to add an officer

When attemting to exploit ESC7 and the account I use for authentication does not have the right Manage Certificates, I must add that account as a new officer in order to grant the account that right. This fails for me using Certipy 2.0.6.

Once this works, can I delete/remove the officer and possibly other remaining changes after the attack?

Yes, you can remove the CA officer afterwards with -remove-officer. The request fails because you're trying to connect to the DC. You should connect to the CA

I actually tried connecting to the ADCS server when I tested this but that failed as well :)

The second image makes more sense to me. It's a mistake in my implementation. I accidentally assumed the CA and the DC was on the same server, which results in Certipy trying to connect to LDAP on the CA, or connecting to the CA DCOM on the DC. Thanks for reporting this. I'll look into a fix soon

Should be fixed in 2.0.8. Can you please verify?

Using 2.0.8 I can now add the officer. Good!

The next step in my case is to enable the certificate template SubCA but that fails for me.

Should be fixed in 2.0.9. The new LDAP DNS resolution was not applied for the -enable-template. Can you verify that it is working now? I really appreciate that you report these issues. Thanks!

Great! This works now. I actually managed to execute the complete ESC7 attack this time. Don't worry about asking me to verify fixes, I will have so much fun owning my client's infrastructure using your tool :D No more hassle having to own a domain-joined machine, bypass AV, AMSI, Applocker, execution policies, language constrained mode and what not then having to upload and import powershell scripts! All I need now is essentially Responder, Hashcat, Bloodhound and Certipy :D

Great.


Possibly missing check in SCION path DecodeFromBytes

Hi all,
There might be some checks missing in this function:
This decoder first decodes the MetaHdr, then it counts the number of Info fields and hops. The MetaHdr decoder first decodes the current info field and hop number. The *Base decoder never checks if CurrINF and CurrHF are valid indices.
Is this behavior intended?
https://github.com/scionproto/scion/blob/362c4622084037cd1d3ac593d2d78128a86638fd/go/lib/slayers/path/scion/base.go#L51-L70
https://github.com/scionproto/scion/blob/362c4622084037cd1d3ac593d2d78128a86638fd/go/lib/slayers/path/scion/base.go#L125-L137

Hi @LinoTelschow
Thanks for reporting this. Indeed the decoder could be enhanced by also verifying that the CurrINF and CurrHF indices indeed point to an existing info- and hop-field. We will take this up and add it.


fix git repo url

copy/paste url pointed to the java app, not the python app
Additionally, the referenced image workshop/content/images/nationalparks-import-from-git-url-builder.png illustrates the Java builder image rather than the python builder image. It would ideally be the latter.

Thanks @dlbewley , we'll add the right screnshot for Python, in the while fixing this issue with Python git repo.


[Fixed] Web page issue on https://code.visualstudio.com/download#

Does this issue occur when all extensions are disabled?: N/A

VS Code Version: v169.2
OS Version: All

Steps to Reproduce:
Try to download latest VSCode setup in whatever version from the official webpage https://code.visualstudio.com/download#
... it doesn't work
Change to https://code.visualstudio.com/insiders/#win and there it works. Click a button of whatever version and you get redirected to a new page like https://code.visualstudio.com/docs/?dv=winzip&build=insiders (if you click on zip) and a popup to download the very file.
[Workaround]
As I didn't want insider build, I just removed '&build=insiders' from the URL and got the one I wanted :
https://az764295.vo.msecnd.net/stable/3b889b090b5ad5793f524b5d1d39fda662b96a2a/VSCode-win32-x64-1.69.2.zip
instead of :
https://az764295.vo.msecnd.net/insider/12b08be500f8a307f30e92cbc3ee39ba115eab69/VSCode-win32-x64-1.70.0-insider.zip

fyi @gregvanl

I'm very sorry to bother,
It was a cookie story. I feel concerned with privacy but am fed up with cookies notifications and use "I don't care about cookies" plugin on Waterfox to automatically dismiss  the notifications. It leads sometimes to a weird behavior. It was the case on this page.
Added an exception for VScode and refused cookies. The magic works again now. ;-).

I'm very sorry to bother,
It was a cookie story. I feel concerned with privacy but am fed up with cookies notifications and use "I don't care about cookies" plugin on Waterfox to automatically dismiss  the notifications. It leads sometimes to a weird behavior. It was the case on this page.
Added an exception for VScode and refused cookies. The magic works now. ;-).


How should we distinguish DesiredHealthcareResult and DesiredHealthCondition::FunctionalOrMentalStatus?

Following Pieter's answer in issue number #111  it's a great idea to modulate FunctionalOrMentalStatus directly in the profile instead of a reference. Below is a quote from the issue:
However, the Goal resource offers the necessary parts to represent a FunctionalOrMentalStatus in the future. Therefore it seems like a better approach to model FunctionalOrMentalStatus directly as part of this profile.
Each element gets its own slice and a code to distinguish. Unfortunately there is no definition code available for the element DesiredHealthCondition::FunctionalOrMentalStatus. How can we clarify for this concept anyway?
Potential solutions:

We ask the zib center for a definition code
Accompanying text in the comment section

In the Goal resource, the element description is a mandatory element. This makes it impossible to add a constraint, because desccription must always be filled.

Potential solution number 1 is a dead end: https://bits.nictiz.nl/browse/ZIB-1549

Would it be possible to deduce that Goal.description is a DesiredHealthCondition when Goal.target and/or Goal.addresses  is present?

I think Goal.target only is enough to deduce Goal.description, but there must be a piece of explanation written down that Goal.target is linked with Goal.description

In consultation we have decided that it is not important that the recipient can't make a distinction. The current situation meets the requirements.

To add to Ashwin comments: that means that if a DesiredHealthcareResult is given in the Goal resource, the Goal.description get the value of a concatenated value derived from the DesiredHealthcareResult which may be received as a DesiredHealthCondition for the recipient. We don't think that will be a big problem.


QnAMakerDialog Support for IBotTelelmetryClient

Is your feature request related to a problem? Please describe.
QnaMaker Cognitive Service does not return Telemetry info. QnAMaker supports Telelmetry if an Implementation of the IBotTelelmetryClient is provided at time of Construction.
Describe the solution you'd like
Add a telelmetryClient  parameter of type IBotTelelmetryClient on Line 117 of The QnAMakerDialog.cs constructor signature defaulting it to null.
In the constructor store it to a local private field.
on Line 232 of QnAMakerDialog.cs add this private field as the 4th parameter of the initialization call changing
return new QnAMaker(endpoint, options, HttpClient);
to
return new QnAMaker(endpoint, options, HttpClient,telelmetryClient);
Describe alternatives you've considered
Inherit and override GetQnAMakerClientAsync.
Additional context
none
[enhancement]

We are also experiencing the same issue as above. We recently migrated to Bot.Builder Version 4.7.2 to be able to add skills to our VA, but our Power BI Dashboard for QnA Telemetry is no longer being recorded.
In the EventTelemetry table in Power BI Dashboard -  QnA_knowledgeBasedId and QnA_Answer are no longer being recorded.

Hi, we are investigating this issue. Will update shortly.

@pchoudhari @vipeketi I will take this one.  It is related to the work I am doing today for adaptive telemetry.

Duplicate of #3472. Closing.


A new parameter for SQL Server in 3.13.0.

Reverts prisma/docs#3049 which reverted accidental merge of https://github.com/prisma/docs/pull/3047
Original message from @pimeys:

Parameter trustServerCertificateCA should point to a certificate authority, which we use to validate the server certificate.

Tsk tsk tsk

OK, so more context might need to be added to the descriptions here... should #3047 not have been merged? I think it was approved by @janpio so I took a quick look and then merged it.
Or, is it that we DO want 3047 merged, and you are reverting the accidental reversion?

#3047 should not have been merged, I just approved the content change in there so it is known that this can be merged when the feature was released. So I reverted the change to not have misleading info on the website, and then reverted the revert so we have a PR for later to be able to merge.

Ah, thanks @Jan! I didn't notice the Milestone on it, of 3.13, sorry. :-(


Update readme: no System: Gateways: Single option in OPNSense 24.1.1

They moved it to the bottom right now, see the plus button

Fixed in https://github.com/FingerlessGlov3s/OPNsensePIAWireguard/commit/0922b2e9d054c2cf9ddd6053e6f303870bf02736


SwipeToSlide

Swipe to Slide - Allow user to swipe with indeterminate step
Don't swipe farther then last slide

What about PR?

Thanks so much for this React port of the slick carousel!
Would love to see this feature implemented!

this pull request is working great, can we merge it so i can use directly from this repo? thanks

Merged #463 which implements this feature

thanks!


Remove remaining references to Registry

[#181129158]
Co-authored-by: Seth Boyles<EMAIL_ADDRESS>Co-authored-by: Daniel Felipe Ochoa<EMAIL_ADDRESS>

Thanks! @sethboyles


Allow sparkctl flags for create being specified inside the application yaml

Currently, flags as --upload-to, --project and --namespace are passed separately from application yaml file. Could it be an interesting addition, to allow in sparkctl create to define these parameters inside the yaml?
This way SparkApplication yaml could specify which gcp project (in case of s3 - profile) to use, and location to upload dependencies for this specific app.

The reason those are specified through flags is the jars/files are local to the client running sparkctl. The controller typically runs in a k8s cluster and therefore doesn't have access to the files.  Defining them in the application spec probably makes little sense from the controller perspective because the controller cannot do much with those dependencies. Thoughts?

Oh, well - this was not much of the intention. I thought more about preparsing yaml at sparkctl, and submitting the adjusted to controller.

Got it. That could work with an optional field in the spec for specifying how to handle local dependencies. Particularly, --namespace is not really needed as the yaml itself can specify that.

Is this still relevant?


Update cli.py

Purpose
I've added a named snippet app instance section within the code for out-of-repo snippet references.
Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

Hello @ckittel,
I've added named snippet where we are creating the confidential app instance. Please review and merge.
Cc: @mmacy, @Dickson-Mwendia


new release and pykerberos requirement

hi! a new release for the last commit would be nice. additionally, the required pykerberos version is likely 1.1.10 since that has the added verify parameter.

just posted a new release, sorry for the delay


Cannot read configuration value contain # sign in ballerina

Description:
In ballerina.conf file there is a key-value pair as below
CONFIG="Nima#1234"
When I read that value using ballerina/config it gives following error.
"line 2:0 missing '"' at ''"
Steps to reproduce:
Run the attached code herewith.
Affected Versions:
Ballerina 0.982.0
OS, DB, other environment details and versions:
macOS 10.13.6,linux
Related Issues (optional):

Suggested Labels (optional):

Type/Bug,Severity/Critical
Archive.zip
Suggested Assignees (optional):

This is a known issue, tracked by #9778


Problem: there is no tesnet sepolia wallet for testing

Originally posted by @lmontero18 in https://github.com/Hoosling/smart-contracts/issues/4#issuecomment-2502634587

@sfabricito @lmontero18 @MY-RV we need a testnet wallet to testing the dapp without spend real money. SEPOLIA ETH is the testing env for ETH, we can use tools like  https://sepolia-faucet.pk910.de/ to fill up the wallet with SEPOLIA ETH (eth for testing).

@lmontero18 pls next time make sure you sync issue-goal


MAC address under "Host" is wrong capture

I believe the MAC for the 'host' configuration at config.json capture wrongly.
After checking from the error log, it seem like first and third number had been swap.
Actual MAC: AA:BB:CC:DD:EE
At the config reading MAC is : CC:BB:AA:DD:EE

Thanks lily - my AA and CC are equal hence the mess up!
run npm i broadlinkjs@"git+https://github.com/lprhodes/broadlinkjs.git" -g and you should be good to go


Move to firefox rather than Chromium for desktop browser

Codecov Report

Merging #1313 (a0766b7) into image_v2.7 (1997c6d) will decrease coverage by 0.94%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           image_v2.7    #1313      +/-   ##
==============================================
- Coverage       46.47%   45.53%   -0.95%
==============================================
  Files              28       28
  Lines            4953     4862      -91
  Branches          827      827
==============================================
- Hits             2302     2214      -88
+ Misses           2594     2591       -3
  Partials           57       57

Impacted Files
Coverage Δ

pynq/pl_server/device.py
43.97% <0.00%> (-4.40%)
:arrow_down:

pynq/pl.py
72.13% <0.00%> (-3.93%)
:arrow_down:

pynq/lib/cmac.py
29.78% <0.00%> (-2.87%)
:arrow_down:

pynq/pl_server/server.py
44.61% <0.00%> (-2.06%)
:arrow_down:

pynq/pl_server/xrt_device.py
27.24% <0.00%> (-1.93%)
:arrow_down:

pynq/pmbus.py
27.03% <0.00%> (-1.54%)
:arrow_down:

pynq/_3rdparty/tinynumpy.py
29.28% <0.00%> (-1.23%)
:arrow_down:

pynq/utils.py
23.46% <0.00%> (-0.67%)
:arrow_down:

pynq/lib/axigpio.py
62.59% <0.00%> (-0.57%)
:arrow_down:

pynq/overlay.py
33.18% <0.00%> (-0.46%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


Use OAuth Resource Server starter #73

I left the Keycloak client in the tests because that is a complex setup and requires a separate investigation of how to better reimplement it.
I also left the Javascript as is but I think we should look into replacing it with another Java module.
Same as before, I had to disable Trust Manager because our test and local Keycloak installations do not have proper SSL sertificates.

OpenShift test failed because the app failed to get configuration from the Keycloak. Rerunning with SSH to see what's happening.

If I run the application inside CircleCI it fails to access https://secure-sso-fmp.<IP_ADDRESS>.nip.io/auth/realms/master and gets 503 Service not available. However, if I curl the same address I get a valid response.

If I deploy a Keycloak server on Hetzner and try to run identical Spring Boot applications locally and on CircleCI, the local one works but the CircleCI fails in the same way as before.

Added jwk-set-uri property, hopefully that will solve the problem.

For some reason the test passes if a Keycloak is running on Hetzner but not if it's running on CircleCI.

On CircleCI NimbusJwtDecoder token validation fails. It tries verifying the certificate with Keycloak at ${SSO_AUTH_SERVER_URL}/realms/master/protocol/openid-connect/certs and get's 503 error. Both locally ran application and curl from CircleCI can access that same path without issues.

So finally I managed to find a working setup. I've configured Spring Security to verify a JWT with a locally stored public key instead of delegating this task to a Keycloak server. This is a valid solution but does need an extra configuration (default is to contact the server). I don't think there is another option with the current CircleCI OpenShift setup.
If we would have an OpenShift server with a publicly available IP address and a Keycloak server with valid SSL certificates the example could be much cleaner and smaller.
@aureamunoz @metacosm could you review this again?


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
3154a9b -> e1289a9

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.25.7 -> v1.26.2

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.26.2
Compare Source
Bug Fixes

deps: update module github.com/go-openapi/strfmt to v0.22.0 (#731) (4aa1e01)

v1.26.1
Compare Source
Bug Fixes

deps: update gomod (#726) (a42f8cf)

v1.26.0
Compare Source
Features

expose checkConsistency with public function (#724) (f9ab5b8)

v1.25.8
Compare Source
Bug Fixes

deps: update gomod (#723) (7596e10)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 3.9.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


test: fix remaining DaemonSet induced flakes

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 82.262%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
5403

Relevant Lines:
6568

💛  - Coveralls


Prevent Form From Popping Up On Update

Every time the plugin is updated, the form appears again asking for demographic information. It still keeps the information from the original popup, so it is not causing us to lose data, but it might annoy users and can introduce error if users have to fill in their information many times, because they might not always be perfectly consistent.
To solve this, I need to figure out what is triggered on update, and I need to make the popup only trigger on installation.

Chrome triggers runtime.onInstalled() whenever the plugin is installed or updated, but it gives an OnInstalledReason object that tells us whether the trigger comes from an install or an update.
https://developer.chrome.com/docs/extensions/reference/api/runtime#type-OnInstalledReason

This was surprisingly easy to fix. It is fixed as of version v1.0.2.


implement second/minute helpers for temporal

Hello!
Which issue does this PR close?
I am interested in the implementation of:
EXTRACT (SECOND FROM '')
EXTRACT (MINUTE FROM '')

in the DF.
Thanks

Codecov Report

Merging #493 (c297f3f) into master (4c7d418) will decrease coverage by 0.03%.
The diff coverage is 88.00%.

@@            Coverage Diff             @@
##           master     #493      +/-   ##
==========================================
- Coverage   82.65%   82.61%   -0.04%
==========================================
  Files         165      165
  Lines       45524    45652     +128
==========================================
+ Hits        37628    37717      +89
- Misses       7896     7935      +39

Impacted Files
Coverage Δ

arrow/src/compute/kernels/temporal.rs
88.97% <88.00%> (-0.64%)
:arrow_down:

arrow/src/array/transform/boolean.rs
76.92% <0.00%> (-7.70%)
:arrow_down:

arrow/src/ffi.rs
78.71% <0.00%> (-4.15%)
:arrow_down:

arrow/src/array/equal/mod.rs
93.93% <0.00%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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

BTW I plan to cherry pick this into the 4.4 release of arrow-rs, which I hope will be released this weekend or early next week (and then automatically be available to datafusion too)


EnableDebugging is gone from ScriptEngine

Just a question, is it debugging feature disabled?
As I found I cann't find it from ScriptEngine
Thanks

It's disabled in .NET core, as, last I checked, this feature (emitting debugging symbols) wasn't supported in .NET core. It should still be present in the .NET 4.5 version of the library.

Based on this doc page https://docs.microsoft.com/en-us/dotnet/api/system.reflection.emit.ilgenerator.marksequencepoint this is still the case.


Feature/migrate mui

Changes proposed in this Pull Request:

Move the icons-mui repo into icons repo

If you add new scripts, you need to chmod 777 them so that they are executable by circleCI:


Markdown Previewer doesn't show info bar for relative path blocked content

Markdown Previewer doesn't show info bar on top of the preview in case of relative path blocked content.
Preview of Markdown file with external link content blocked with info bar on top:

Preview of Markdown file with relative path content blocked without info bar on top:

@somil55  Can you take a look into this? Doesn't seems like a big change. I updated the Web browser control to block external content in this PR which blocks relative path content as well similar to Html previewer.

this was in 0.16.1


example in test.env doesn't match if statement

In test.env, it mentions MODEL_TYPE = LlamaCpp or GPT4ALL
However, in local_app.py the if statement is as follows:
elif model_type == 'GPT4All':
This causes it to never match, as the L's are uppercase in test.env, but lowercase in the if statement.
I tried to create a branch and submit a PR for this but got 401, though that could be entirely my fault

fixed, thanks!


Fix is_valid_connection_candidate

What was wrong?
The #1054 broke the PeerPool.is_valid_connection_candidate function since the connected_peers keys are no longer NodeAPI objects.
How was it fixed?
Added the right path to get to the address.
To-Do

[x] Clean up commit history

Cute Animal Picture

This one deserves a release note, since bug went out with the last release and it's pretty visible and appears to prevent inbound connections from succeeding.

https://github.com/ethereum/trinity/pull/1179


UnsupportedClassVersionError When connecting drawio to self-hosted GitLab instance

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
We recently updated our GitLab to version 15.1.0-ee, and drawio to version 20.0.4 (running in a docker container). With these updates, drawio can no longer connect to GitLab. (We were using GitLab 14.8.2-ee and drawio version 14.0.3, and these were working fine.)
I spent lots of time digging, checking other issues (https://github.com/jgraph/drawio/issues/2192 , https://github.com/jgraph/drawio/issues/2474) get to the bottom of this bug and updated several things following what other people have tried to try to get it to work:

ensured that all our self-signed certificates are both on the gitlab server and in the drawio container

these work, I can curl gitlab from drawio and vice versa without any issues.

ensured that my gitlab application settings are correct

callback to https://diagrams.xxxxxxx.local/gitlab (no .html at the end, and using https)
application has the correct scopes (api, read_repository, write_repository)

set all the correct environment variables in my docker container:

DRAWIO_GITLAB_URL=https://gitlab.xxxxxxx.local
DRAWIO_GITLAB_ID=9b0axxxxxxxxxxxxxxxxxxxxxxxxxxda0a4aeb21c518fcad78517b26b
DRAWIO_GITLAB_SECRET=3867f5xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx96fc353da7ebb562696
DRAWIO_BASE_URL=https://diagrams.xxxxxxxx.local

And when I spin up the drawio page and try to connect, it raises an "Unknown error". If I inspect the page and look at the actual request, I see that drawio tried to access endpoint https://diagrams.xxxxxxx.local/gitlab?getState=1 and returns a 500 error:
Exception

jakarta.servlet.ServletException: Error instantiating servlet class [com.mxgraph.online.GitlabAuthServlet]
	org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:542)
	org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:92)
	org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:690)
	org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:356)
	org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:399)
	org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:65)
	org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:867)
	org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1760)
	org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:49)
	org.apache.tomcat.util.threads.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1191)
	org.apache.tomcat.util.threads.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:659)
	org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
	java.lang.Thread.run(Thread.java:750)
Root Cause

java.lang.UnsupportedClassVersionError: com/mxgraph/online/GitlabAuthServlet has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0 (unable to load class [com.mxgraph.online.GitlabAuthServlet])
	org.apache.catalina.loader.WebappClassLoaderBase.findClassInternal(WebappClassLoaderBase.java:2515)
	org.apache.catalina.loader.WebappClassLoaderBase.findClass(WebappClassLoaderBase.java:877)
	org.apache.catalina.loader.WebappClassLoaderBase.loadClass(WebappClassLoaderBase.java:1413)
	org.apache.catalina.loader.WebappClassLoaderBase.loadClass(WebappClassLoaderBase.java:1257)
	org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:542)
	org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:92)
	org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:690)
	org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:356)
	org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:399)
	org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:65)
	org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:867)
	org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1760)
	org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:49)
	org.apache.tomcat.util.threads.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1191)
	org.apache.tomcat.util.threads.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:659)
	org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
	java.lang.Thread.run(Thread.java:750)

Note: any subsequent attempts to connect raise a 404. I can only get the 500 error on the first connection attempt when I startup the container.
And at this point, I'm very far down the rabbit hole, I don't know what to do anymore... Any help is greatly appreciated.
To Reproduce
Steps to reproduce the behavior:

Install GitLab 15.1.0-ee (on an Ubuntu 20.04.4 LTS machine)
Create an application for drawio
Install drawio 20.0.4 in a docker container
add the application credentials in the container's environment variables when starting it
Install self-signed certificates on GitLab server and drawio container
Try to connect to gitlab in the drawio page

Expected behavior
Successful connection to gitlab to see, edit, and commit diagrams to repository
D
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 20.0.4 (running in docker container)

It's built with Java 11 and you're running Java 8. Have you got 11?

Oh, that's not right, we'll look.

OK, that'll be fixed in the next release via https://github.com/jgraph/docker-drawio/commit/c153de6eee25404bd644b6cf9307bf95337f7215. Thanks for the report.

Great! thanks a lot!

Hello,
I saw this morning that you guys released a new version with the changes from this bug report. Sadly I still have no luck in connecting to GitLab (although we are making progress!)

Any ideas?

Having the same issue

Will be fixed in the next release. A manual fix is here https://github.com/jgraph/drawio/issues/2932


update share test, production manila audit map, add debug for tox

For unknown reasons, the create_notifier() in test_api_request_no_messaging used "oslo_messaging.Notifier" instead of the "_LogNotifier" that should be originally used.
This patch fixes this issue by changing "use_oslo_messaging" config to false


kbfsdokan unkillable

win 7 ultimate
kbfsdokan.exe cannot be killed - even the pid stays the same
upd.exe can be killed but spawnes again with a new pid

If you touch/try to open K:\.kbfs_unmount kbfsdokan will exit. Of course the service can restart it after an exit.


[unit-test]LocalStorage: add updateLocalVolumeGroupAccessibility test

What this PR does / why we need it:
add updateLocalVolumeGroupAccessibility test
also for #1280
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d2bc890) 36.47% compared to head (255205a) 36.47%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1281   +/-   ##
=======================================
  Coverage   36.47%   36.47%
=======================================
  Files          28       28
  Lines        2218     2218
=======================================
  Hits          809      809
  Misses       1306     1306
  Partials      103      103

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ogd file paths are hardcoded

in 0002
not ideal; better to have a lookup file somewhere.
Same for level texts, age categories etc.

done


depend on android.library?

My gradle fu is not great but was wondering if it is possible to not have to apply android.library/android.app plugin before dropshots. Maybe there is an api that can check if it is on path rather than requiring ordering?

example of test execution:
gradlew :myApp:myModule-sandbox:installDebug :myApp:myModule-sandbox:installDebugAndroidTest && adb shell am instrument -w com.twitter.sandbox.notifications.push.test/androidx.test.runner.AndroidJUnitRunner

Fixed in a2aa635


Implement SingleShaderBufferedDrawNode

Implement the issue #167
And found the reason of #159
.
What's done in this PR:

Not expose setter in the IMultiShaderBufferedDrawable interface.
Instead of always drawing white color in the LyricSpriteTextDrawNode, it should be better not to affect the FrameBufferDrawColour if lyric drawable color changed.
Add single shader buffer draw node and relative interface.
Switch the draw node base class from MultiShaderBufferedDrawNode into SingleShaderBufferedDrawNode in the lyric sprite text and karaoke sprite text.
Fix the test case.

Also, listed the benefit why use SingleShaderBufferedDrawNode instead of MultiShaderBufferedDrawNode.

No frame-buffer recycle issue.
Might not use more than one shader in the karaoke project now.


Don't collect all groups from https://git.unistra.fr

The purpose of this issue is to start a discussion on what groups should be listed from https://git.unistra.fr
The next step is to identify the right person at unistra to help us answer this.

Ping me if you need help on this issue

Actually, this is a duplicate of https://github.com/DISIC/politique-de-contribution-open-source/issues/190 - closing it here.


Unable to retrieve template for my-alias: 400

serverless 1.26.1.
serverless-alias 1.7.0
Recently I've been running into the same error message during deploy:
$ serverless deploy --stage dev --alias my-alias --verbose
...
Error --------------------------------------------------

  Unable to retrieve template for some-other-alias: 400

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Forums:        forum.serverless.com
     Chat:          gitter.im/serverless/serverless

  Your Environment Information -----------------------------
     OS:                     darwin
     Node Version:           8.11.1
     Serverless Version:     1.26.1

There are around 25 templates the plugin tries to pull down each time this runs and it will fail randomly on one of them. I'm also a little curious as to why it needs to pull them all down. I'd appreciate any help with this. Thanks!

I'm facing the same issue. It happens randomly as well here.


flutter update-packages --force-upgrade + analyzer fix

Supersedes https://github.com/flutter/flutter/pull/108178.
Fixes https://github.com/flutter/flutter/issues/108184.
First commit is result of flutter update-packages --force-upgrade.
Second commit is the fix for the analyzer errors (https://github.com/flutter/flutter/issues/108184).

Looks like the analyzer found some new different warnings now. I'll fix those up in a sec.

The analyzer check is happy now. 🎊

Need to rebase onto #107984 which rolled archive.


Vortex integration with newer compilers

The vortex integration is now in a separate library. That library is compiled separately, making sure that GCC 4.8 is always used. If you don't have GCC 4.8 available, you can use a prebuilt version.
In theory, we could use a single precompiled libRcsVortex.so for all users. Before that is uploaded here, however, we need to verify that that is allowed license-wise. While the library does not link vortex code statically, it does include stuff from the headers.
This pull request also includes some large-scaler refactoring of the Rcs physics classes. Both PhysicsFactory and PhysicsConfig have been moved to RcsCore, and PhysicsConfig underwent major changes.

Thanks!


Upload min targ.gz snapshots to publicly accessible bucket

The buildManifest just builds the snapshot but does not upload it anywhere. Its the publish-snapshot that uploads the artifacts to maven.
https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/opensearch/distribution-build.jenkinsfile#L45-L47
Should we directly upload from here (jenkins local) to snapshots (prod) s3? Also what about arm64? While I executed the jenkinfile to see the generated artifacts, I did not see anything generated for arm64 here.
OR Can we separate this problem from the release-candidates one?
Also talked to @mch2 seems like we have 2 possible solutions:

Rename the above received artifact to snapshots as it guarantees that user is running tests against an artifact that will get released
Build x64 (already doing) and arm64 snapshots and then upload to s3 (either just to prod or both staging and prod).

Originally posted by @gaiksaya in https://github.com/opensearch-project/opensearch-build/issues/1378#issuecomment-1007823438
Acceptance Criteria:
Min snapshot artifacts should be accessible via https://artifacts.opensearch.org/
Example: https://artifacts.opensearch.org/snapshots/core/opensearch/1.2.2-SNAPSHOT/opensearch-min-1.2.2-SNAPSHOT-linux-x64-latest.tar.gz
https://artifacts.opensearch.org/snapshots/core/opensearch/1.2.2-SNAPSHOT/opensearch-min-1.2.2-SNAPSHOT-linux-arm64-latest.tar.gz

The -SNAPSHOT build is fundamentally a different build from non-SNAPSHOT. It will never be released. For daily builds that are used for testing I think we want to keep it this way.
I noticed that with the requirements above the distribution build is looking increasingly similar to what we want the snapshot build to be (both x64 and arm), except that the latter is built with snapshot: true and uploads its artifacts to a different location in S3 + publishes snapshot JARs to Maven. We might consider having 2 jobs.

The -SNAPSHOT build is fundamentally a different build from non-SNAPSHOT. It will never be released. For daily builds that are used for testing I think we want to keep it this way.
I noticed that with the requirements above the distribution build is looking increasingly similar to what we want the snapshot build to be (both x64 and arm), except that the latter is built with snapshot: true and uploads its artifacts to a different location in S3 + publishes snapshot JARs to Maven. We might consider having 2 jobs.

Agree - I'd suggest we repurpose the existing arm/x64 builds to include snapshot: true and create a new script to build a release artifact off of a locked down manifest.
The intent of making those original builds 'release' builds was so that we did not have a special release build and any daily build used for testing could be promoted.  I'm not sure this is providing us much value if we can perform a release build against a locked down manifest.

The -SNAPSHOT build is fundamentally a different build from non-SNAPSHOT. It will never be released. For daily builds that are used for testing I think we want to keep it this way.
I noticed that with the requirements above the distribution build is looking increasingly similar to what we want the snapshot build to be (both x64 and arm), except that the latter is built with snapshot: true and uploads its artifacts to a different location in S3 + publishes snapshot JARs to Maven. We might consider having 2 jobs.

Agree - I'd suggest we repurpose the existing arm/x64 builds to include snapshot: true and create a new script to build a release artifact off of a locked down manifest.
The intent of making those original builds 'release' builds was so any daily build used for testing could be promoted. I'm not sure this is providing us much value if we can perform a release build against a locked down manifest.

I think that's not quite what was I was saying, we still want those dailies for people to download and try that aren't snapshot, but maybe I'm overthinking it.

Will they be used in addition to the 2 snapshot artifacts?  If they have their separate use then that makes sense.  If not, I think having 4 parallel builds is a bit redundant.  The snapshots "should" be identical to the release tarball outside of maven tagging it as a snapshot.

After talking to @peternied looks like this issue is urgent. Working on implementing a fix. This is what it looks like currently:
https://github.com/gaiksaya/opensearch-build/blob/snapshots/jenkins/opensearch/distribution-build.jenkinsfile#L57-L91

PR #1472 resolved this issue. Thanks!


add doc data encrypt

add doc of "data encrypt"

Codecov Report

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

@@            Coverage Diff            @@
##             master     #280   +/-   ##
=========================================
  Coverage          ?   58.49%
=========================================
  Files             ?       23
  Lines             ?     2159
  Branches          ?        0
=========================================
  Hits              ?     1263
  Misses            ?      829
  Partials          ?       67

Continue to review full report at Codecov.

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


Image Stretch Problem

Hi,
I want an image to be scaled and if it is small, it should be stretched as well according to given dimension.
For larger images , it's working fine but for small images it's getting scaled but not stretched
and that looks kind of improper to me.
This is my current configuration -
public $actsAs = array(
'Upload.Upload' => array(
'category_image' => array(
'path' => '{ROOT}webroot{DS}img{DS}uploads{DS}category{DS}',
'mimetypes' => array('image/bmp',
'image/gif',
'image/jpeg',
'image/pjpeg',
'image/png',
'image/vnd.microsoft.icon',
'image/x-icon',
'image/x-png'
),
'extensions' => array('png', 'jpg', 'jpeg'),
'thumbnailSizes' => array(
'281' => '[281x155]',
'thumb' => '[60x45]',
'130' => '[130x130]',
)
)
)
);
Please help me out. Thanks !

Your configuration is best fit, if you want to scale images you'd want to scale them to a specific size using
100w
See the docs.
https://cakephp-upload.readthedocs.org/en/latest/thumbnails.html

But 100w will resize according to width only, what about the height. Can I use something like 100w×100h.


Ambiguous call to operator error with QiCore function overload

The following CQL results in an Ambiguous call to operator 'isCommunity(org.hl7.fhir.r4.model.MedicationRequest)' error at runtime:
define fluent function isCommunity(medicationRequest MedicationRequest):
  exists (medicationRequest.category C
    where C ~ Community
  )

define fluent function isCommunity(medicationRequest MedicationNotRequested):
  exists (medicationRequest.category C
    where C ~ Community
  )

This code worked in v3.15.0. The following update appears to have introduced this issue.
I’ve already tried setting the signature level to Overloads using:
evaluationSettings.getCqlOptions().getCqlCompilerOptions().setSignatureLevel(LibraryBuilder.SignatureLevel.Overloads);
but this didn’t resolve the issue.
The apparent challenge here is that these are legitimately different signatures in CQL, but not in ELM. The ELM shows the same resultType for the medicationRequest operand, being {http://hl7.org/fhir}MedicationRequest.

@c-schuler - Potentially resolved by #1476. Could you validate?


[18.09] backport always hornor client side to choose which builder to use with DOCKER_…

…BUILDKIT env var regardless the server setup
backport of https://github.com/moby/moby/pull/37826 for 18.09
git checkout -b 18.09_backport_buildkit-cli-control ce-engine/18.09
git cherry-pick -s -S -x 5d931705e33927ba9f0b7251b74b6b3c450edaf6

cherry-pick was clean; no conflicts

ping @tiborvass @AntaresS @tonistiigi PTAL

All green; let's merge 👍


Add \gt and \lt

This makes it easier to typeset math, because it avoids using < and > as special HTML characters.
(CLA was signed)

@crepererum thank you for the pull request.  Although these aren't standard TeX commands, MathJax supports this probably for the same reasons that you've outlined.  I'm okay with merging this, but I'd like to hear @xymostech's thoughts first just in case she has any concerns.

We already talked about these and decided against it here: #149 but now I'm less sure about that decision. Getting a macro expansion of \lt into < for things like \left\lt looks like it works in TeX, so we might try that option once macros come around. Dealing with < and > inside HTML is a nightmare though, and if someone's using our auto-render plugin then they don't have a chance to do a replacement before running it through KaTeX.
@crepererum How are you using KaTeX? Would doing something like adding an option to the auto-render extension to turn \lt into < and \gt into > before sending the input to KaTeX, would that work for you?

My current setup is to type some text using Markdown which sometimes contains math, rendered by the auto-renderer. Escaping backslashes in Markdown is not a problem, but dealing with </> is a nightmare, so the proposed PR would simplify this a lot.
Sure you could extend the Markdown parser to handle this step, but for my app (Ghost) there is currently no way to do this without running your own fork (ref https://github.com/TryGhost/Ghost/issues/1724).
So the auto-render extension might work, but I think that the HTML-parser of the browser will already be a problem. What I can do instead is using </> when typesetting the math, because the auto-renderer uses textContent to get the data (https://github.com/Khan/KaTeX/blob/master/contrib/auto-render/auto-render.js#L55) which will included the transformed </>.
If the </>-workaround becomes somehow official, we might want to add this to the documentation of the auto-renderer so that users don't have to search through the source code to find this implementation detail.

@crepererum could you verify that \lt and \gt work as delimiters, e.g. that \left\lt\frac{1}{x}\right\gt produces the same rendering as \left<\frac{1}{x}\right>?

@kevinbarabash done, added tests for that.

@crepererum looks good except one of the lines in the test is too long.  Can you split it so it's on two lines so that we stay within the 80 column limit?

@kevinbarabash sry, fixed now.

@crepererum awesome, thanks!


Setup Adapter Pretraining and Fusion

Close #17

I'm getting this problem when trying to fuse the pre-trained adapters: https://github.com/Jacob-Chmura/plausibility-vaccine/issues/24

I'm getting this problem when trying to fuse the pre-trained adapters: #24

Addressed


How To add files to the tangle safe list

My problem
I have :noweb evaluations in my org file, with auto tangle the messages buffer says:
Tangling /home/rampedindent/.dotfiles/system.org...
forever. And emacs opens a new buffer called *emacs* that says
Lisp expression: Evaluate this python code block (toggle-packages) on your system? (yes or no)
To me this looks like it was meant to open a confirmation dialogue instead of a new buffer.

The thing i need help with
After looking around on this repo i think i need to add the file to the save list but i am unclear how to do that since my elisp skills aren't the best.
Currently i have the following in my emacs configuration
  (setq org-auto-tangle-babel-safelist '(
                                         ("/home/rampedindent/.dotfiles/system.org")
                                         ))

The help screen says i've set the variable but it does not appear to do anything

Removing the brackets fixed that error
  (setq org-auto-tangle-babel-safelist '(
                                         "/home/rampedindent/.dotfiles/system.org"
                                         "/home/rampedindent/tmp/test.org"
                                         ))


Add eslint for development

Is your feature request related to a problem? Please describe.
This comment https://github.com/nvh95/jest-preview/pull/79/files#r865053054 can be avoided by having eslint installed.
Describe the solution you'd like
Integrate eslint for development env
Describe how should jest-preview implements this feature
Install eslint and related plugins (prettier, node, typescript...)

@Rohitbels Thanks for your support. Anything not conflict with prettier is fine. I am looking forward to see your PR 🤩

Thank you @Rohitbels. I am converting this project into pnpm workspace, the progress is tracked in #255
I am not available to finish it yet. But aiming for the next few weeks.
For the current folder which have its own package.json. We might not need eslint for them (?). Since it's kind of "external" ones. For example, websites has nothing to do with jest-preview.
I think you can try to just add eslint for current state of the project, or you can wait a little bit longer after #255 get merged, then add eslint. It's up to you.
Again, thanks for your interested in helping us improve the DX of jest-preview


Mass gwas-curator emails causing problems for EBI listserver

Mass gwas-curator emails generated by the curation app upon status change are causing issues for systems. This needs addressing to generate only a single email when the publication page is used to perform mass action on all studies of a publication.  This is high priority requested by TSC.
Note, Sunny advised "mailman list is not designed for these types of automated email", we should look for an alternative solution.

@sajo-ebi as discussed, please stop the emails being sent as a short term solution

@ljwh2 the curation app has been deployed in Prod with Email sending for status change code block commented

Thanks Sajo! Keeping ticket open for longer term solution.


missing "percent sign" in locale_pl-PL.ini

file:   options/locale/locale_pl-PL.ini
action --> commit_repo
wrong version:
commit_repo=wypycha do %[3]s w[4]s
correct version
commit_repo=wypycha do %[3]s w%[4]s

Hi!
Our translations are kept on crowdin and need to be changed there. However, as this is quite a clear fix I've fixed it for you and approved it already.
FYI this one would be https://crowdin.com/translate/gitea/all/enus-pl#q=commit_repo
Thanks.

Thank you very much


More '>' signs after each disable/enable

I disabled and enabled the package for whatever reason and now it completes like this:
<html>
>>>
</html>

The '>' are getting more and more.

This is a known bug that this dude hasn't fixed in forever (known bug since 2016 on issue #151). This only happens when you do that disable/enable thing that you explained, so sadly for now you just got  to not mess with the setting to get it to work properly.  Uninstalling, restarting and reinstalling it should fix the problem though.


Support EasyAuth in JavaScript SDK

Need to support EasyAuth in JS SDK, including Cordova Plugin.

Client will need overrides for login host, prefix, (like iOS, Managed, Android) and a final redirect during login.

Transition to azure-mobile-apps-js-client#1


fix: fix pipeline

Summary
fix failing notebook
Tests
tested pipeline
Dependency changes
None

/azp run

Codecov Report

Merging #1544 (e965085) into master (9fc6428) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1544      +/-   ##
==========================================
- Coverage   83.79%   83.74%   -0.06%
==========================================
  Files         301      301
  Lines       15337    15337
  Branches      707      707
==========================================
- Hits        12852    12844       -8
- Misses       2485     2493       +8

Impacted Files
Coverage Δ

...osoft/azure/synapse/ml/core/utils/AsyncUtils.scala
75.00% <0.00%> (-5.00%)
:arrow_down:

...crosoft/azure/synapse/ml/lightgbm/TrainUtils.scala
83.64% <0.00%> (-2.52%)
:arrow_down:

...ft/azure/synapse/ml/cognitive/ComputerVision.scala
73.47% <0.00%> (-0.87%)
:arrow_down:

.../execution/streaming/continuous/HTTPSourceV2.scala
92.08% <0.00%> (-0.72%)
:arrow_down:

...zure/synapse/ml/stages/PartitionConsolidator.scala
95.74% <0.00%> (+2.12%)
:arrow_up:

Continue to review full report at Codecov.

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


failed releases don't indicate why

When a release "Fails" (not rejected) there's no information to help understand what happened:
https://origin-release.svc.ci.openshift.org/
(in this case the an image w/ an invalid file in the manifest was being included in the release payload so the release image could not be built)

In case we need breadcrumbs later ... the release controller runs these pods in the ci-release namespce


Simplfy the implementation of Mode Select Cluster

Problem
What is being fixed?  Examples:

Simplify the over-complicated implementation of Mode Select Cluster
Fixes #11163
Fixes #11162

Change overview
Removed the usage of std::map
Removed the abstract ModeOptionIterator and its implementation
Removed the abstract ModeOptionIteratorFactory and its implementation
Move the implementation of StaticSupportedModeOptionsManager to /example folders, as it contains app-specific logic.
Added the concrete class ModeOptionsProvider
Testing
How was this tested? (at least one bullet point required)

If manually tested, what platforms controller and device platforms were manually tested, and how?
Manually tested on MacOs with chip-tool

/rebase

@saurabhst @msandstedt @andy31415 ?

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

Fast tracking as this has had > 3 days to review, and comments are addressed


Error when using pandas dataframe for properties on Tracks layer

🐛 Bug
Error when using pandas dataframe for properties on Tracks layer.
To Reproduce
The Tracks layer throws an error when passing a dataframe for the properties kwarg:
Traceback (most recent call last):
  File "/home/quantumjot/napari/napari/_qt/event_loop.py", line 79, in gui_qt
    yield app
  File "<ipython-input-15-fdca00748db9>", line 4, in <module>
    viewer.add_tracks(data, properties=properties)
  File "/home/quantumjot/napari/napari/components/add_layers_mixin.py", line 854, in add_tracks
    layer = layers.Tracks(
  File "/home/quantumjot/napari/napari/layers/tracks/tracks.py", line 144, in __init__
    self.properties = properties or {}
  File "/home/quantumjot/anaconda3/envs/trackpy/lib/python3.8/site-packages/pandas/core/generic.py", line 1326, in __nonzero__
    raise ValueError(
ValueError: The truth value of a DataFrame is ambiguous. Use a.empty, a.bool(), a.item(), a.any() or a.all().

Expected behavior
Tracks layer should accept the dataframe.
Environment
napari: 0.3.9.dev31+g647d918
Platform: Linux-4.15.0-1097-oem-x86_64-with-glibc2.10
Python: 3.8.5 (default, Sep 4 2020, 07:30:14) [GCC 7.3.0]
Qt: 5.14.2
PyQt5: 5.14.2
NumPy: 1.19.1
SciPy: 1.5.2
Dask: 2.29.0
VisPy: 0.6.5
GL version: 4.6.0 NVIDIA 450.80.02
MAX_TEXTURE_SIZE: 32768

Additional context

Thanks flagging this, @quantumjot ! I can take a look over the next couple of days.

Notes from @quantumjot in this comment:

This is the offending line:
https://github.com/napari/napari/blob/647d9182a6dc3542521cbc99b8b59beb4f2ba901/napari/layers/tracks/tracks.py#L144

edit: if you want to make a PR that's great too!

@kevinyamauchi - Done!


cannot build with xcodebuild

First let me say I really like this repo- it's super handy! Thanks for the work.
Somehow I'm unable to build a project via xcodebuild with this package.
Steps to repro:

New xcode proj
Add spm nativemarkkit
from command line: xcodebuild -scheme 'scheme' -sdk iphoneos

I get the error:
The following build commands failed:
CompileSwift normal armv7 /Users/me/Library/Developer/Xcode/DerivedData/marktest-epvvpnjxgaeusqfjffrlspcvjsnw/SourcePackages/checkouts/NativeMarkKit/Sources/NativeMarkKit/render/NativeMarkText+UIKit.swift
CompileSwiftSources normal armv7 com.apple.xcode.tools.swift.compiler

Do you have advice on resolving this?
Thanks very much 💯
Using Version 12.3 (12C33)

@andyfinnell hope your 2021 is off to as good a start as possible! I'm having this same issue -- any thoughts? Love this project, thanks for making it available!

@andyfinnell hope your 2021 is off to as good a start as possible! I'm having this same issue -- any thoughts? Love this project, thanks for making it available!

Thanks for writing this up. I can confirm I can reproduce this. I'll work on a fix tonight.

Thanks for writing this up. I can confirm I can reproduce this. I'll work on a fix tonight.

It appears this is happening because the containing app is building the armv7 architecture, and SwiftUI doesn't support that architecture. I can see in my build logs it builds fine for armv64 though.
According to these two posts it can happen when the deployment target is iOS 10 or less. However, I'm seeing it in a test app that targets iOS 13.5. So I'm a bit baffled as to why it's trying to build armv7 at all.

It appears this is happening because the containing app is building the armv7 architecture, and SwiftUI doesn't support that architecture. I can see in my build logs it builds fine for armv64 though.
According to these two posts it can happen when the deployment target is iOS 10 or less. However, I'm seeing it in a test app that targets iOS 13.5. So I'm a bit baffled as to why it's trying to build armv7 at all.

I think I've fixed it in v1.1.1. Let me know!

I think I've fixed it in v1.1.1. Let me know!

It's working for me now -- woohoo! Thanks!

It's working for me now -- woohoo! Thanks!


[consensus] byzantine attack on state sync

Attack scenario:

f byzantine nodes collect a QuorumCert committed B_n, none of 2f+1 honest nodes are aware of this QC.
1 honest node is lag behind and start to sync to this QC from byzantine node
f byzantine nodes go offline
the system will stuck because of honest node can not finish the state sync(none of the 2f honest commit B_N) and crash looping, other 2f nodes can not move forward without this node.

Solution:
We used to solve this by processing the sync_to QC from synchronizer so that at least f+1 honest nodes who voted for are able to commit B_n and sync back to the lag behind node.
After we separate state synchronizer out of consensus, we lose this mechanism.

Right now, we require syncing to the 3-chain that commits B_n. Does this open up additional byzantine scenarios of any sort (are there situations that might discard that data)

Just to add that the solution to this problem is not obvious: state synchronizer is currently not aware of consensus so we can't easily force the state synchronizer to "commit" a ledger info (something that we could do when we were running it in the same event loop)

@bmaurer the receiver should verify the 3-chain is valid to continue the sync process, otherwise it'll just ignore the whole message.

closed via https://github.com/libra/libra/issues/1590


index.css not being included

Hey there! I can't seem to get css to register. I've deleted dist and .parcel-cache, and rebuilt. Any other ideas?

Is it possible to share the setup that reproduce this?

@dudo Is it possible to share the setup that reproduce this?

// package.json
{
  "name": "browse_app",
  "version": "0.1.0",
  "private": true,
  "description": "",
  "scripts": {
    "start": "parcel serve ./src/index.html -p 1264 --hmr-port 1265",
    "build": "parcel build ./src/index.html"
  },
  "dependencies": {
    "@babel/runtime": "^7.12.1",
    "body-scroll-lock": "^3.1.5",
    "framer-motion": "^2.9.4",
    "json-api-serializer": "^2.5.0",
    "react": "^17.0.0",
    "react-dom": "^17.0.0",
    "react-router-dom": "^5.2.0"
  },
  "devDependencies": {
    "@babel/core": "^7.12.3",
    "@babel/plugin-transform-react-jsx": "^7.12.1",
    "@babel/plugin-transform-runtime": "^7.12.1",
    "@babel/preset-env": "^7.12.1",
    "babel-eslint": "^10.1.0",
    "eslint": "^7.12.1",
    "eslint-plugin-react": "^7.21.5",
    "eslint-plugin-react-hooks": "^4.2.0",
    "parcel": "^2.0.0-nightly.433",
    "parcel-reporter-bundle-manifest": "^0.3.1"
  },
  "browserslist": {
    "production": [
      ">0.2%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  }
}

// .babelrc
{
  "plugins": [
    [
      "@babel/plugin-transform-runtime",
      {
        "regenerator": true
      }
    ],
    [
      "@babel/plugin-transform-react-jsx",
      {
        "runtime": "automatic"
      }
    ]
  ]
}

// .parcelrc
{
  "extends": "@parcel/config-default",
  "reporters": ["...", "parcel-reporter-bundle-manifest"]
}

Setup was pretty straight forward... thanks for taking a look =)

@dudo And the CSS file is being imported from ./src/index.html with something like <link rel="stylesheet" href="./styles.css" /> right? Would it be possible to share a minimal setup that could reproduce this via CodeSandbox?

@lukyth that was it! I was importing the main css file from my js entrypoint. Removing that in favor of including the link meta tag properly includes index.css.
Thanks!!

I see. Thanks for the confirmation! I think it should include that CSS file in the generated manifest as well so I'm gonna reopen this issue until it's resolved. And thanks for reporting btw! 👍

@dudo Which version of Parcel 2/this plugin/Node.js that you are using? With those info, I might be able to reproduce the issue and help you with it.
I tried to build with a CSS file imported from JS file, and I could see the CSS file in the manifest as well:

The setup I tested with is at https://github.com/autifyhq/parcel-reporter-bundle-manifest/tree/main/tests/fixtures/parcel-beta, with the JS import CSS added in https://github.com/autifyhq/parcel-reporter-bundle-manifest/commit/34a06905638c7bda3242945251a60501399b6ebc.

"dependencies": {
    "@babel/runtime": "^7.12.1",
    "react": "^17.0.0",
    "react-dom": "^17.0.0",
    "react-router-dom": "^5.2.0"
  },
  "devDependencies": {
    "@babel/core": "^7.12.3",
    "@babel/plugin-transform-react-jsx": "^7.12.1",
    "@babel/plugin-transform-runtime": "^7.12.1",
    "@babel/preset-env": "^7.12.1",
    "babel-eslint": "^10.1.0",
    "eslint": "^7.12.1",
    "eslint-plugin-react": "^7.21.5",
    "eslint-plugin-react-hooks": "^4.2.0",
    "parcel": "^2.0.0-nightly.433",
    "parcel-reporter-bundle-manifest": "^0.3.1"
  },

@dudo This should be fixed in v0.3.2 now. Let me know if you still find any problem! 😃


Migrate to softprops/action-gh-release.

Eric noticed both the actions/create-release and
actions/upload-release-asset GitHub repositories had been archived. This
converts our two-step create and upload into one step using the
reccomendd replacement action.

Thanks for pointing this our @Eric-Arellano - assuming this works, things got simpler anyhow.

How do you test this when iterating?

I don't. I did have the same nerves though and implemented a manual release using:
https://github.com/pantsbuild/pex/blob/8d953ba407cc75f56a3697a72b8b1cb34195d777/.github/workflows/release.yml#L6-L10
I had to use that in my 1st release or two when sorting things out.

@Eric-Arellano that worked just fine: https://github.com/pantsbuild/pex/runs/2242481723?check_suite_focus=true
You should have no troubles with Pants PEX.


Updated field dimensions with 2017 rule changes

@JosephusPaye / @Phasedd we should probably update this then https://github.com/NUbots/NUsight2/blob/master/src/shared/field/dimensions.ts#L72

@BrendanAnnable Made a pull: https://github.com/NUbots/NUsight2/pull/112